docs: clean up mcp-server.md (archived link, Claude auth, Codex commands) [murmur:logfire-sdk/mcp-server-doc-cleanup] by dmontagu · Pull Request #2205 · pydantic/logfire

dmontagu · 2026-08-05T23:04:16Z

Summary

Remove archived repo link -- the sentence "You can check the Logfire MCP server repository for more information" pointed to an archived repo with no useful content; the page itself now documents the tool set. The deprecated local-server section's OLD_README.md link is kept.
Cross-link the in-product MCP page -- adds a note near the top of the Remote MCP Server section linking logfire-us.pydantic.dev/-/mcp and logfire-eu.pydantic.dev/-/mcp, which know the correct region URL for each account.
Claude Code: one copyable auth block -- replaces the manual /mcp slash-command steps with claude mcp login logfire shown together with claude mcp add in a single shell block.
Codex: lead with the region caveat -- rewrites "no separate MCP JSON configuration is required" (which reads as "you're done") to "configures the US endpoint by default", so EU readers do not stop before the region caveat.
Codex: remove redundant commands -- drops codex mcp remove logfire (does nothing when installed via the plugin, redundant anyway because codex mcp add overwrites) and codex mcp login logfire (codex mcp add auto-triggers OAuth; the extra login opened a second identical browser tab). The EU switch block is now just the codex mcp add command plus the new-conversation note.

Test plan

Render the docs page and confirm all links resolve
Verify claude mcp add logfire --transport http <url> followed by claude mcp login logfire completes authentication without additional steps
Verify codex mcp add logfire --url <eu-url> alone handles EU switch correctly

- Remove archived logfire-mcp repo link (no longer useful) - Cross-link the in-product MCP page (US + EU) near the top of the remote-server section so readers find the region-aware UI instructions - Claude Code: show `claude mcp login logfire` together with `claude mcp add` as one copyable block, replacing the manual /mcp slash-command steps - Codex: lead with "configures the US endpoint by default" so EU readers do not stop before the region caveat; remove redundant `codex mcp remove logfire` and `codex mcp login logfire` commands (`codex mcp add` overwrites and auto-triggers OAuth) Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

coderabbitai · 2026-08-05T23:08:08Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 9f3451c0-8fef-46bd-8ccc-e14b96f898b2

📥 Commits

Reviewing files that changed from the base of the PR and between 3fe564c and e5a35c4.

📒 Files selected for processing (1)

docs/how-to-guides/mcp-server.md

🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

pydantic/logfire (manual)
pydantic/pydantic-ai (manual)
pydantic/platform (auto-detected)
pydantic/pydantic (auto-detected)

🚧 Files skipped from review as they are similar to previous changes (1)

docs/how-to-guides/mcp-server.md

📝 Walkthrough

Walkthrough

Updated the MCP setup guide to remove the archived repository link. Claude Code instructions now use explicit claude mcp add and claude mcp login commands. Codex instructions now document automatic US endpoint configuration and authentication, with a separate EU endpoint command.

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly summarizes the MCP documentation cleanup, including the archived link, Claude authentication, and Codex command changes.
Description check	✅ Passed	The description directly explains the documentation changes and their relationship to the MCP onboarding cleanup.
Linked Issues check	✅ Passed	The changes address all coding-related objectives in issue `#2199`, including link cleanup, authentication guidance, region handling, and redundant command removal.
Out of Scope Changes check	✅ Passed	The pull request only updates the MCP server guide and contains no unrelated or out-of-scope changes.

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch murmur/w/logfire-sdk/github_oauth/dmontagu/mcp-server-doc-cleanup

_{Comment @coderabbitai help to get the list of available commands.}

coderabbitai

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/how-to-guides/mcp-server.md`:
- Around line 75-76: Update the Claude MCP add command so the --transport http
option appears before the logfire server name, matching Claude Code’s documented
argument order; leave the login command unchanged.

🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

Push a commit to this branch (recommended)
Create a new PR with the fixes

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 77a22ce6-dd3c-4a1e-98ee-839a72726866

📥 Commits

Reviewing files that changed from the base of the PR and between 5f1fb5f and 3fe564c.

📒 Files selected for processing (1)

docs/how-to-guides/mcp-server.md

🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

pydantic/logfire (manual)
pydantic/pydantic-ai (manual)
pydantic/platform (auto-detected)
pydantic/pydantic (auto-detected)

cubic-dev-ai

No issues found across 1 file

Confidence score: 5/5

Automated review surfaced no issues in the provided summaries.
No files require special attention.

_{Re-trigger cubic}

--transport http must come before the server name per Claude Code's documented CLI syntax: claude mcp add --transport http <name> <url> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

macroscopeapp · 2026-08-05T23:09:43Z

Thanks @coderabbitai -- all pre-merge checks passing. The inline argument-order fix (Message #2) has been applied in a follow-up commit.

^{Posted by Murmur}

alexmojaki · 2026-08-06T12:30:09Z


+!!! note
+    The Logfire UI has per-client setup instructions at
+    [logfire-us.pydantic.dev/-/mcp](https://logfire-us.pydantic.dev/-/mcp) (US) or


I don't think this change should merge as long as those pages are missing codex

split this note into another pr

coderabbitai Bot requested changes Aug 5, 2026

View reviewed changes

Comment thread docs/how-to-guides/mcp-server.md Outdated

cubic-dev-ai Bot reviewed Aug 5, 2026

View reviewed changes

docs: fix claude mcp add argument order

f52fdb9

--transport http must come before the server name per Claude Code's documented CLI syntax: claude mcp add --transport http <name> <url> Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

coderabbitai Bot approved these changes Aug 5, 2026

View reviewed changes

alexmojaki reviewed Aug 6, 2026

View reviewed changes

remove note

e5a35c4

alexmojaki enabled auto-merge August 6, 2026 13:56

alexmojaki added this pull request to the merge queue Aug 6, 2026

Merged via the queue into main with commit b6f8443 Aug 6, 2026
18 checks passed

alexmojaki deleted the murmur/w/logfire-sdk/github_oauth/dmontagu/mcp-server-doc-cleanup branch August 6, 2026 14:12

This was referenced Aug 6, 2026

Rework the MCP server guide around plugin installs and in-app setup instructions #2208

Merged

docs: point EU/self-hosted plugin users at a plugin update, not a manual MCP entry #2211

Merged

Navigation Menu

docs: clean up mcp-server.md (archived link, Claude auth, Codex commands) [murmur:logfire-sdk/mcp-server-doc-cleanup] - #2205

docs: clean up mcp-server.md (archived link, Claude auth, Codex commands) [murmur:logfire-sdk/mcp-server-doc-cleanup]#2205
alexmojaki merged 3 commits into
mainfrom
murmur/w/logfire-sdk/github_oauth/dmontagu/mcp-server-doc-cleanup

dmontagu commented Aug 5, 2026 •

edited by cubic-dev-ai Bot

Loading

Uh oh!

coderabbitai Bot commented Aug 5, 2026 •

edited

Loading

Walkthrough

Uh oh!

coderabbitai Bot left a comment

Uh oh!

Uh oh!

cubic-dev-ai Bot left a comment

Uh oh!

macroscopeapp Bot commented Aug 5, 2026

Uh oh!

alexmojaki Aug 6, 2026

Uh oh!

alexmojaki Aug 6, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

Conversation

dmontagu commented Aug 5, 2026 • edited by cubic-dev-ai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Test plan

Uh oh!

coderabbitai Bot commented Aug 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Uh oh!

coderabbitai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

macroscopeapp Bot commented Aug 5, 2026

Uh oh!

alexmojaki Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

alexmojaki Aug 6, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

dmontagu commented Aug 5, 2026 •

edited by cubic-dev-ai Bot

Loading

coderabbitai Bot commented Aug 5, 2026 •

edited

Loading