Skip to content

docs: correct MCP transport (SSE to streamable HTTP) and Claude Desktop config path#74

Merged
jaymeklein merged 1 commit into
mainfrom
docs/claude-rules-and-standards-check
Jul 20, 2026
Merged

docs: correct MCP transport (SSE to streamable HTTP) and Claude Desktop config path#74
jaymeklein merged 1 commit into
mainfrom
docs/claude-rules-and-standards-check

Conversation

@jaymeklein

Copy link
Copy Markdown
Owner

What changed

Corrects the MCP transport description in the README, which still called it SSE in three places even though the server runs stateless streamable HTTP (api/services/mcp/server.py — it deliberately abandoned SSE because the stream died on uvicorn --reload). The README's own MCP section already said "streamable HTTP," so the three stale references were internally inconsistent with both the code and the rest of the doc.

README.md

  • Features bullet — "MCP server over SSE" → "over streamable HTTP"
  • Architecture diagram — REST + MCP/SSEREST + MCP/HTTP
  • Ports table — REST + MCP/SSEREST + MCP/HTTP
  • Claude Desktop config path — corrected from the Linux-style ~/.config/claude/… to the macOS location ~/Library/Application Support/Claude/claude_desktop_config.json, with an explicit macOS / Windows split

api/main.py and api/routers/mcp.py

  • Updated two stale "SSE" code comments to "streamable-HTTP" so the comments match the transport actually mounted.

Why

Documentation-accuracy follow-up. A reader wiring up an MCP client would otherwise get contradictory signals about the transport, and the macOS config path pointed at a directory Claude Desktop does not use.

Reviewer notes

  • Docs + comment-only change — no runtime behavior changes. Verified against the code (FastMCP(..., stateless_http=True, json_response=True) in api/services/mcp/server.py).
  • While reviewing, I also confirmed the README's other claims are accurate and left them untouched: default ports, the MCP tool list/signatures, the 60 rpm rate limit, the presigned-URL 1 h expiry, the MinIO/GPU compose commands, and http://localhost:8000/docs (returns 200 on the live stack).
  • ruff passes on the two edited Python files; mypy/pytest are not meaningful for comment-only edits.
  • This branch was previously merged via docs(claude): lean CLAUDE.md index + path-scoped rules, and a standards-check skill #73; origin/main..HEAD is exactly this one commit, so the PR diff is just these fixes.

…op config path

The MCP server runs stateless streamable HTTP (api/services/mcp/server.py), but the README still described it as SSE in three places, contradicting its own MCP section and the code. This fixes the Features bullet, the architecture diagram, and the ports table. It also corrects the Claude Desktop config path to the macOS location (Library/Application Support/Claude) with an explicit macOS/Windows split, and updates two stale SSE comments in api/main.py and api/routers/mcp.py so the code matches the transport it mounts.
@jaymeklein
jaymeklein merged commit cf68fa4 into main Jul 20, 2026
8 checks passed
@jaymeklein
jaymeklein deleted the docs/claude-rules-and-standards-check branch July 20, 2026 17:08
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