Skip to content

Make Cloud MCP spawn find search_docs from /agent - #20

Closed
alex-16moro wants to merge 1 commit into
mainfrom
cursor/mcp-cloud-discovery-3eed
Closed

alex-16moro wants to merge 1 commit into
mainfrom
cursor/mcp-cloud-discovery-3eed

Conversation

@alex-16moro

Copy link
Copy Markdown
Owner

Summary

Cloud search_docs was not working: stdio is spawned from /agent, and /agent/.cursor is root-owned, so python3 -u .cursor/mcp-diffusers-docs.py exited with file-not-found during live tool discovery.

.cursor/mcp.json now boots with python3 -c and globs /agent/repos/*/.cursor/mcp-diffusers-docs.py (Desktop still finds the relative repo path). mcpServerAllowlist permits both python3 (what mcp.json spawns) and diffusers-docs-mcp (PATH shim). Allowlisting only the shim blocked the python3 spawn.

This session cannot reconnect an already-failed MCP namespace. New agents need this revision (and a new environment build after merge).

Convention gate

  • Overlay mcp.json remains stdio diffusers-docs only (no Hub HTTP)
  • Allowlist includes python3 and diffusers-docs-mcp
  • Handshake from cwd /agent lists search_docs

Tests run

PYTHONPATH= python3 -m unittest tests.test_tooling.TestCloudMcpAllowlist tests.test_tooling.TestOverlayMcpDefaults tests.test_tooling.TestMcpFraming tests.test_tooling.TestMcpLauncher tests.test_tooling.TestAttachEmptyMcp -v
make doctor

13 tests OK. doctor OK. /agent handshake lists search_docs and returns scheduler docs.

Human judgment (not the gate)

  • Does not enable Hub HTTP MCP
  • Does not overwrite library AGENTS.md / .ai/
Open in Web Open in Cursor 

Cloud stdio cwd is /agent and /agent/.cursor is not writable, so
python3 -u .cursor/mcp-diffusers-docs.py never started. mcp.json now
boots with python3 -c and globs /agent/repos/*/.cursor/mcp-diffusers-docs.py.
Allowlist python3 (mcp.json) and diffusers-docs-mcp (PATH shim).

Co-authored-by: ale93.moro <ale93.moro@gmail.com>
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.

2 participants