Skip to content

fix(crew): auto-trust project MCP so the seeded first message sends#7

Open
dropdevrahul wants to merge 2 commits into
mainfrom
fix-mcp-seed
Open

fix(crew): auto-trust project MCP so the seeded first message sends#7
dropdevrahul wants to merge 2 commits into
mainfrom
fix-mcp-seed

Conversation

@dropdevrahul

Copy link
Copy Markdown
Owner

Bug

Starting a new chat from crew didn't send the first message. The repo's .mcp.json (nexum-delegate) makes every fresh Claude session in a new worktree show the one-time "trust MCP servers?" prompt. crew's seed-typing (pty.rs) then types the task into that dialog instead of the prompt box → the message is lost.

Fix

Launch and resume claude with --settings '{"enableAllProjectMcpServers":true}', which auto-enables project .mcp.json servers so the trust prompt never appears and the seed reaches the REPL. Verified the key name against the Claude Code CLI.

Standalone/opencode/cursor paths unchanged. 45 Rust + 415 Python tests green.

Adding .mcp.json means every fresh Claude session in a new crew worktree hits
the one-time 'trust MCP servers?' prompt. The blind seed-typing in pty.rs then
types the task into that dialog instead of the input box, so the first message
is never sent.

Launch/resume claude with --settings '{"enableAllProjectMcpServers":true}' to
auto-enable the project's .mcp.json servers, so no prompt appears and the seed
reaches the REPL.
…ll the trust prompt

The --settings enableAllProjectMcpServers approach did not reliably suppress the
'found new MCP in project nexum-delegate' trust prompt, which still swallowed
the seeded first message. Instead pass the repo's .mcp.json to claude via
--mcp-config <abs> --strict-mcp-config: a CLI-provided server is trusted (no
discovery prompt) and strict mode ignores auto-discovered configs. When the repo
has no .mcp.json, no MCP flags are added (nothing is discovered, so no prompt).
Verified the flags load headlessly (exit 0).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant