Skip to content

fix(docs): drop the inert bare /mcp redirect - #53

Merged
a-essawy merged 1 commit into
mainfrom
fix/drop-inert-mcp-redirect
Jul 21, 2026
Merged

fix(docs): drop the inert bare /mcp redirect#53
a-essawy merged 1 commit into
mainfrom
fix/drop-inert-mcp-redirect

Conversation

@a-essawy

Copy link
Copy Markdown
Contributor

Follow-up to #52, verified against production after that deploy landed.

/docs/mcp is Mintlify's own hosted MCP server endpoint for this docs site, not a page. It is POST-only JSON-RPC, so a GET correctly returns 405, and the platform route takes precedence over the redirects table.

$ curl -X POST https://rendobar.com/docs/mcp -d '{"jsonrpc":"2.0","id":1,"method":"initialize",...}'
event: message
data: {"result":{"protocolVersion":"2024-11-05","capabilities":{"tools":...},
       "serverInfo":{"name":"Rendobar","version":"1.0.0"},
       "instructions":"This Model Context Protocol server provides search and
       retrieval tools for the Rendobar site..."}}

So the /mcp -> /mcp-server rule #52 added never fires. It was written to cover a 405 that turns out not to be a defect. Removing it, because leaving it tells the next person reading this file that /docs/mcp redirects when it does not, and validate-nav waves it through (the destination is a real page, so the check cannot see that the source is shadowed).

The real fix from #52 is untouched and still live:

/docs/mcp/overview       308 -> /docs/mcp-server  200
/docs/mcp/installation   308 -> /docs/mcp-server  200

Test plan

$ npm run validate:nav          # Checked 16 pages, 16 nav entries, 22 redirects. All consistent.
$ npm run validate:frontmatter  # Checked 16 MDX files. All frontmatter valid.

No behaviour change to verify after deploy: the rule being removed has no observable effect today.

/docs/mcp is Mintlify's own hosted MCP server endpoint for this site, not a
page. It is POST-only JSON-RPC, so a GET correctly returns 405, and the
platform route wins over anything in the redirects table. Confirmed against
production: a JSON-RPC initialize returns serverInfo {"name":"Rendobar"}.

The /mcp -> /mcp-server rule added in #52 therefore never fires. It was
covering for a 405 that is not a defect. Leaving it in place would tell the
next person reading this file that /docs/mcp redirects, and validate-nav
passes it because the destination is a real page.

The actual fix from #52 is unaffected: /mcp/:slug* still sends every
/docs/mcp/<page> URL to /mcp-server. Verified live.
@a-essawy
a-essawy merged commit 54db1cf into main Jul 21, 2026
2 checks passed
@a-essawy
a-essawy deleted the fix/drop-inert-mcp-redirect branch July 21, 2026 21:39
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