Skip to content

fix(mcp): standardize server naming and improve OAuth docs#17642

Merged
rahulchhabria merged 6 commits intomasterfrom
fix/mcp-docs-consistency
May 6, 2026
Merged

fix(mcp): standardize server naming and improve OAuth docs#17642
rahulchhabria merged 6 commits intomasterfrom
fix/mcp-docs-consistency

Conversation

@sentry-junior
Copy link
Copy Markdown
Contributor

@sentry-junior sentry-junior Bot commented May 6, 2026

Description

Addresses user feedback about inconsistencies and gaps in the MCP docs page (docs/ai/mcp.mdx).

Changes

Standardize MCP server key naming:

  • Changed "sentry""Sentry" in Amp (VS Code) and OpenCode JSON config examples to match every other client config on the page
  • Updated OpenCode CLI commands (opencode mcp auth, opencode mcp logout) to match the updated config key

Improve OAuth documentation clarity:

  • Added an <Alert> in the main Setup section explicitly stating that Cloud transport requires OAuth + Streamable HTTP support, with links to the mcp-remote bridge and Stdio Transport as fallbacks
  • Expanded the "OAuth Authentication Problems" troubleshooting section with:
    • Link to mcp-remote workaround for clients without native OAuth/Streamable HTTP
    • Link to Stdio Transport as an alternative
    • URL verification step

Why

Users who don't get an OAuth prompt after following the main setup have no clear path to a fix — the mcp-remote workaround was only mentioned in the collapsed "Other Clients" section, not in the setup flow or troubleshooting. The naming inconsistency (Sentry vs sentry) across configs is a paper cut that erodes trust in the docs.


Action taken on behalf of Rahul Chhabria.

- Standardize MCP server key to "Sentry" (capital S) in Amp and
  OpenCode config examples to match all other client configs
- Update OpenCode CLI commands to match the config key
- Add alert in main Setup section clarifying that Cloud transport
  requires OAuth + Streamable HTTP client support, with fallback
  options (mcp-remote bridge or stdio transport)
- Improve OAuth troubleshooting with actionable guidance: link to
  mcp-remote workaround and stdio alternative, add URL check
@vercel
Copy link
Copy Markdown

vercel Bot commented May 6, 2026

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

Project Deployment Actions Updated (UTC)
develop-docs Ready Ready Preview, Comment May 6, 2026 4:37pm
sentry-docs Ready Ready Preview, Comment May 6, 2026 4:37pm

Request Review

Comment thread docs/ai/mcp.mdx Outdated

When you first connect, you'll be prompted to log in with your Sentry organization and grant access. During authentication, you select which tool groups to expose — this keeps the context window focused on the tools you actually need.

<Alert>
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

idk if this deems an alert. this is a pretty common thing for MCP servers to make sure it supports oauth and streamable http 🤔

Comment thread docs/ai/mcp.mdx Outdated
```json
"amp.mcpServers": {
"sentry": {
"Sentry": {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i would actually prefer we lowercase everything, i haven't seen any other mcp servers capitalize their mcp server names in configs

Lowercase all "sentry" keys in MCP client config blocks to match
ecosystem convention — capitalized server names are uncommon across
other MCP server docs.

Demote the OAuth + Streamable HTTP requirement from an Alert to inline
prose. This is a baseline expectation for cloud MCP servers, not a
warning that warrants a callout.

Co-Authored-By: Claude <noreply@anthropic.com>
@rahulchhabria rahulchhabria requested a review from sergical May 6, 2026 14:50
@rahulchhabria rahulchhabria marked this pull request as ready for review May 6, 2026 14:51
rahulchhabria and others added 2 commits May 6, 2026 09:07
…teps

Follow-up to the JSON config key lowercasing. The opencode CLI commands
and the Claude Code, Factory Droid, and v0 UI selection prompts also
reference the configured server name — keeping those capitalized while
the config keys are lowercase would mismatch what users see.

Co-Authored-By: Claude <noreply@anthropic.com>
The v0.app MCP flow moved — the registry is now under
Account Settings → Integrations → MCP Connections, and users search the
catalog for the Sentry brand entry rather than picking a configured key.

Co-Authored-By: Claude <noreply@anthropic.com>
@rahulchhabria rahulchhabria enabled auto-merge (squash) May 6, 2026 16:35
@rahulchhabria rahulchhabria merged commit 16d4055 into master May 6, 2026
18 checks passed
@rahulchhabria rahulchhabria deleted the fix/mcp-docs-consistency branch May 6, 2026 16: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.

2 participants