docs: point EU/self-hosted plugin users at a plugin update, not a manual MCP entry - #2211
Conversation
…CP endpoint by hand
|
Note Reviews pausedIt looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the Use the following commands to manage reviews:
Use the checkboxes below for quick actions:
📝 WalkthroughWalkthroughThe Claude Code setup instructions now require Logfire plugin version 0.1.4 or later. They include marketplace and plugin update commands. They require restarting Claude Code. They document fallback installation commands for Possibly related PRs
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
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 38-46: Before publishing this documentation, update and publish
the pydantic/skills plugin manifest to version 0.1.4 or later so it supports
LOGFIRE_MCP_URL; if publication cannot happen first, add a clearly identified
pending-release fallback to the documented installation instructions.
🪄 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: b89029d6-d3b8-4c82-8d3c-c46ba2c3e763
📒 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)
There was a problem hiding this comment.
All reported issues were addressed across 1 file (changes from recent commits).
Tip: Review your code locally with the cubic CLI to iterate faster.
Re-trigger cubic
Follow-up to #2208.
The
LOGFIRE_MCP_URLnote told anyone on an older plugin to bypass the plugin and register the MCP server by hand (claude mcp add ...), which leaves them with a duplicate, unmanaged server entry and no plugin skills. Tell them how to update the plugin instead.Verified
claude plugin marketplace update/claude plugin update <plugin>@<marketplace>are the real commands (claude plugin update --help), and thatclaude plugin listreports the installed version.Note the
LOGFIRE_MCP_URLsupport itself is still pending in pydantic/skills#48 (latest published plugin is 0.1.3), so this note becomes accurate for everyone once that merges and the official marketplace picks it up.