fix(desktop): wire MCP server for Claude Code agents - #6604
Open
BradGroux wants to merge 1 commit into
Open
Conversation
Claude Code's entry in KNOWN_ACP_RUNTIMES had mcp_command: None, so
Desktop set BUZZ_ACP_MCP_COMMAND="" and the harness never spawned a
MCP server for Claude agents. Goose avoided this because it loads MCP
servers from its own config.yaml (supports_acp_native_config: true),
but Claude has no native config support — the only way it receives
MCP servers is through the session/new mcpServers field, which stays
empty when BUZZ_ACP_MCP_COMMAND is blank.
Set Claude's mcp_command to Some("buzz-dev-mcp"), matching Codex and
Buzz Agent. Added a test asserting Claude has the MCP command wired.
Closes block#6593
Co-authored-by: Brad Groux <brad@digitalmeld.com>
Signed-off-by: Brad Groux <brad@digitalmeld.com>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Summary
Claude Code agents get no MCP server from Buzz because the Claude entry in
KNOWN_ACP_RUNTIMEShasmcp_command: None. Desktop setsBUZZ_ACP_MCP_COMMAND="", and the harness'sbuild_mcp_serversreturns an empty list — the session/newmcpServersfield stays empty.Goose avoids this because it loads MCP servers from its own
config.yaml(supports_acp_native_config: true). Claude has no native config support — the only way it receives MCP servers is through the session/newmcpServersfield, which stays empty whenBUZZ_ACP_MCP_COMMANDis blank.Set Claude's
mcp_commandtoSome("buzz-dev-mcp"), matching Codex and Buzz Agent.Related issue
Closes #6593
Testing
New unit test
claude_has_mcp_commandinruntime/tests.rsasserts thatknown_acp_runtime("claude-agent-acp")resolves withmcp_command: Some("buzz-dev-mcp"). The existingcodex_has_mcp_commandandgoose_has_no_mcp_hookstests are unchanged, confirming Goose still uses its native config path and Codex still has its MCP command.Cannot run the full
cargo testsuite from this host (no Rust toolchain). The build and test gate will be verified on CI.