Skip to content

Serve MCP protocol on GET /mcp - #933

Merged
RhysSullivan merged 1 commit into
mainfrom
fix/mcp-get-protocol
Aug 28, 2026
Merged

Serve MCP protocol on GET /mcp#933
RhysSullivan merged 1 commit into
mainfrom
fix/mcp-get-protocol

Conversation

@RhysSullivan

Copy link
Copy Markdown
Collaborator

Summary

  • GET /mcp was serving the HTML install page to protocol clients that expect streamable HTTP or SSE
  • Rewrite/redirect to /mcp/setup only when Accept prefers HTML; SSE, JSON, and MCP Accepts reach the existing handler
  • Browser visits still land on the install UI

Test plan

  • Unit tests: browser GET /mcp rewrites to setup; SSE/JSON/MCP Accepts do not; POST /mcp unchanged
  • bun run lint and main-site typecheck
  • Live: browser GET /mcp still shows install UI; agent GET /mcp is MCP, not HTML

- GET /mcp was serving the HTML install page to protocol clients that expect streamable HTTP or SSE
- Rewrite/redirect to /mcp/setup only when Accept prefers HTML; SSE, JSON, and MCP Accepts reach the existing handler
- Browser visits still land on the install UI
@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
answer-overflow-main-site Ready Ready Preview Aug 28, 2026 7:23am

Request Review

@cloudflare-workers-and-pages

cloudflare-workers-and-pages Bot commented Aug 28, 2026

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
❌ Deployment failed
View logs
answeroverflow-main-site a9443bb Aug 28 2026, 07:22 AM

@RhysSullivan
RhysSullivan merged commit 3ea531e into main Aug 28, 2026
4 of 6 checks passed
@RhysSullivan
RhysSullivan deleted the fix/mcp-get-protocol branch August 28, 2026 07:26
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