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

Comment thread crates/forge_main/Cargo.toml Outdated
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.

@github-actions
Copy link
Copy Markdown

Action required: PR inactive for 5 days.
Status update or closure in 10 days.

@github-actions github-actions bot added the state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. label Apr 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

state: inactive No current action needed/possible; issue fixed, out of scope, or superseded. 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