Incorporate MCP Server into Golem CLI #2605
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…registry - Gate the HTTP server behind the `mcp-server` Cargo feature - Add graceful error when `--serve` is used without the feature - Keep server code isolated in `serve.rs` behind cfg(feature = "mcp-server") - Implement clean JSON-RPC response model (`RpcResponse`, `RpcError`) instead of raw literals - Wire `tools/list` directly to the central registry of tools - Restrict `tools/call` to only allowed/registered tools with typed input schema - Align manifest discovery with repo’s canonical algorithm (`golem.yaml` walk-up)
…d subcommands field intro list_tools, cleanup,...
…rom list_tools to default response, add golem_server status, make call_tools return immediat response for golem server run,...
…tern, add resources, incorporate basic client with e2e testing of golem commands,...
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Hello @jdegoes
I didn't found a way to DM you.
My PR not AI slop, everything reviewed carefully, also I've tried to cover all requirements outlined within the issue.
Golem x Claude Code: https://youtu.be/t5dnCSYQg_0
Golem x Claude Code (suite): https://youtu.be/95BXexeZjj4
Golem MCP server/client e2e testing: https://youtu.be/ONUJ6BOyHDI
Please note it tooks me a while to did that changes ~2 weeks, not just copy paste AI response with minimal effort, without even compilation/build.
Algora.io
/claim #1926