Sorry, something went wrong.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Sorry, something went wrong.
📝 Walkthrough
WalkthroughThis PR substantially rewrites docs/dev-tools/mcp-server.md, reorganizing MCP server documentation around transport modes (HTTP OAuth/PAT, stdio, SSE) with authentication models, client-specific setup instructions (Claude Desktop, Claude Code, Cursor, VS Code, Windsurf, Zed, Claude.ai), self-hosted deployment configuration (PLANE_BASE_URL environment variables), expanded tool reference and common workflows, and updated troubleshooting with connectivity verification steps. ChangesPlane MCP Server Documentation Overhaul
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Poem🐰 A document reborn in structured light, ❌ Failed checks (1 inconclusive)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches 🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. ❤️ ShareComment @coderabbitai help to get the list of available commands and usage tips. |
Sorry, something went wrong.
There was a problem hiding this comment.
Actionable comments posted: 4
🤖 Prompt for all review comments with AI agentsFix all unresolved CodeRabbit comments on this PR:
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 9f83720e-4708-4dcb-9695-8ec3ffe32650
📥 CommitsReviewing files that changed from the base of the PR and between 92279d1 and 39bfb3c.
📒 Files selected for processing (1)
Sorry, something went wrong.
| ## Transport modes | ||
|
|
||
| ## Transport methods | ||
| The server supports four transport modes. The right one depends on your deployment and usecase. |
There was a problem hiding this comment.
⚠️ Potential issue | 🟡 Minor | ⚡ Quick win
Fix grammar: "use case" should be two words.
"usecase" should be "use case" for correct English grammar.
📝 Proposed fix‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| The server supports four transport modes. The right one depends on your deployment and usecase. | |
| The server supports four transport modes. The right one depends on your deployment and use case. |
[grammar] ~15-~15: Ensure spelling is correct
Context: ...ight one depends on your deployment and usecase. | Transport | For ...
(QB_NEW_EN_ORTHOGRAPHY_ERROR_IDS_1)
🤖 Prompt for AI Agents
Sorry, something went wrong.
| ``` | ||
| https://app.plane.so/acme-corp/ | ||
| ``` |
There was a problem hiding this comment.
⚠️ Potential issue | 🟡 Minor | ⚡ Quick win
Add language tag to fenced code block.
Fenced code blocks should specify a language for proper syntax highlighting and to comply with markdown linting rules. For this URL example, use text or plaintext.
🔧 Proposed fix[warning] 99-99: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Sorry, something went wrong.
|
|
||
| #### HTTP with OAuth | ||
|
|
||
| 1. Go to **Customize → Connectors ** in Claude.ai. |
There was a problem hiding this comment.
⚠️ Potential issue | 🟡 Minor | ⚡ Quick win
Remove extra space in emphasis markers.
There's an extra space before the closing emphasis marker: ** in Claude.ai. should be ** without the trailing space.
🔧 Proposed fixAs per coding guidelines, Prettier formatting checks must pass in CI.
📝 Committable suggestion‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.
| 1. Go to **Customize → Connectors ** in Claude.ai. | |
| 1. Go to **Customize → Connectors** in Claude.ai. |
[warning] 308-308: Spaces inside emphasis markers
(MD037, no-space-in-emphasis)
🤖 Prompt for AI Agents
Sorry, something went wrong.
| ``` | ||
| What is work item ENG-42 about? | ||
| ``` | ||
|
|
||
| Model calls `retrieve_work_item_by_identifier` with `project_identifier="ENG"` and `work_item_identifier="42"`. | ||
|
|
||
| ### Create a work item | ||
|
|
||
| ``` | ||
| Create a high-priority bug in the ENG project called "Login times out on Safari". | ||
| Description: The OAuth callback redirects to a blank page on Safari 17+. | ||
| Assign it to me. | ||
| ``` | ||
|
|
||
| Model calls `list_projects` → `retrieve_work_item_by_identifier` (or `get_me` to resolve "me") → `create_work_item`. | ||
|
|
||
| ### Update work item state | ||
|
|
||
| ``` | ||
| Mark ENG-88 as done and add a comment: "Fixed in commit abc1234, needs QA." | ||
| ``` | ||
|
|
||
| Model resolves the UUID, calls `list_states` to find the Done state UUID, calls `update_work_item` and `create_work_item_comment`. | ||
|
|
||
| ### Sprint planning | ||
|
|
||
| ``` | ||
| Create a cycle called "Sprint 15" in ENG starting 2025-06-02, ending 2025-06-15. | ||
| Then move all incomplete issues from Sprint 14 into it. | ||
| ``` | ||
|
|
||
| Model calls `create_cycle` then `list_cycles` to find Sprint 14's UUID, then `transfer_cycle_work_items`. | ||
|
|
||
| ### Log time | ||
|
|
||
| ``` | ||
| Log 90 minutes on ENG-42: "Implemented retry logic for the upload endpoint." | ||
| ``` | ||
|
|
||
| ### Search across the workspace | ||
|
|
||
| ``` | ||
| Show me all high-priority bugs assigned to me that are still in progress. | ||
| ``` | ||
|
|
||
| Model calls `list_work_items` with filters `priorities=["high"]`, `state_groups=["started"]`, and the current user's UUID as `assignee_ids`. | ||
|
|
||
| ### Manage a module | ||
|
|
||
| ``` | ||
| Add ENG-55, ENG-56, and ENG-57 to the "Checkout Redesign" module. | ||
| ``` |
There was a problem hiding this comment.
⚠️ Potential issue | 🟡 Minor | ⚡ Quick win
Add language tags to workflow example code blocks.
All fenced code blocks in the Common workflows section lack language tags. For these natural language user prompt examples, use text or plaintext to comply with markdown linting rules.
🔧 Proposed fix for all workflow examplesApply this pattern to all 7 code blocks in lines 1384, 1392, 1402, 1410, 1419, 1425, and 1433:
[warning] 1384-1384: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 1392-1392: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 1402-1402: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 1410-1410: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 1419-1419: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 1425-1425: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
[warning] 1433-1433: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Sorry, something went wrong.
Description
Type of Change
Screenshots and Media (if applicable)
Test Scenarios
References
Summary by CodeRabbit