Skip to content

feat(jsonrpc): add stdio-based json-rpc server with forge api integration#2896

Draft
amitksingh1490 wants to merge 4 commits intomainfrom
json-rpc-stdio
Draft

feat(jsonrpc): add stdio-based json-rpc server with forge api integration#2896
amitksingh1490 wants to merge 4 commits intomainfrom
json-rpc-stdio

Conversation

@amitksingh1490
Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions bot added the type: feature Brand new functionality, features, pages, workflows, endpoints, etc. label Apr 9, 2026
@amitksingh1490 amitksingh1490 marked this pull request as draft April 9, 2026 01:48
.unwrap_or_else(|| PathBuf::from("."));

// Initialize the API
let api = Arc::new(ForgeAPI::init(cwd, config, services_url));
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
let api = Arc::new(ForgeAPI::init(cwd, config, services_url));
let api = Arc::new(ForgeAPI::init(cwd, config));

Config already have service url

path = "src/main.rs"

[[bin]]
name = "forge-jsonrpc"
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't do two bins. We need to ship 1 bin.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: feature Brand new functionality, features, pages, workflows, endpoints, etc.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants