Thanks to visit codestin.com
Credit goes to github.com

Skip to content

MCP server #151

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 2 commits into from
Oct 6, 2025
Merged

MCP server #151

merged 2 commits into from
Oct 6, 2025

Conversation

@danciaclara
Copy link
Collaborator

@danciaclara danciaclara commented Oct 3, 2025 •
edited by coderabbitai bot
Loading

Description

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • Feature (non-breaking change which adds functionality)
  • Improvement (change that would cause existing functionality to not work as expected)
  • Code refactoring
  • Performance improvements
  • Documentation update

Screenshots and Media (if applicable)

Test Scenarios

References

Summary by CodeRabbit

  • Documentation
    • Added a new guide for integrating the Plane MCP server with SSE, including setup and troubleshooting across common tools.
    • Normalized titles: adjusted casing of "Build a Plane app (Beta)" and removed redundant sidebar titles for consistency.
    • Reorganized navigation: renamed top tabs to "API Reference" and "Dev Tools," consolidated related pages under "Build and extend Plane."
    • Added redirects from legacy URLs to the new docs paths to preserve existing links.

@vercel
Copy link

vercel bot commented Oct 3, 2025 •
edited
Loading

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

Project Deployment Preview Comments Updated (UTC)
developer-docs Error Error Oct 6, 2025 8:22am

@coderabbitai
Copy link

coderabbitai bot commented Oct 3, 2025 •
edited
Loading

Caution

Review failed

The pull request is closed.

Walkthrough

Metadata edits to two MDX docs, addition of a new dev-tools MDX guide for Plane MCP server, and site navigation/top-level tab updates in mint.json including new redirects from legacy paths to the new dev-tools locations.

Changes

Cohort / File(s) Summary of changes
Docs front matter tweaks
dev-tools/build-plane-app.mdx, dev-tools/intro-webhooks.mdx		 Adjusted front matter: changed casing of title/sidebarTitle in build-plane-app.mdx; removed sidebarTitle from intro-webhooks.mdx. No content changes.
New documentation page
dev-tools/mcp-server.mdx		 Added a new MDX page documenting Plane MCP server integration and usage (SSE, configuration snippets, activation, troubleshooting).
Site navigation and redirects
mint.json		 Renamed tabs ("APIs and BYOA" → "API Reference", "Webhooks" → "Dev Tools"); removed "App Development" nav entry; replaced the former "Webhooks" group with "Build and extend Plane" containing dev-tools/* pages; added redirects mapping legacy paths to new dev-tools paths.

Sequence Diagram(s)

(The changes are documentation and navigation-only; no runtime control-flow changes to illustrate.)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested reviewers

  • pushya22

Poem

I thump my paws on minty trails,
New tabs hop in, old paths set sails.
A server guide, carrots aligned,
Dev tools burrow, neatly defined.
Redirects map where links once roam—
Hippity hop, docs find their home. 🥕🐇

Pre-merge checks and finishing touches

❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title Check ⚠️ Warning The title “MCP server” is a vague noun phrase that does not clearly summarize the primary change or changes in the PR, which include adding a new MCP server guide as well as reorganizing and updating Dev tools navigation and redirects. It lacks a verb and fails to convey the scope of the update. A good PR title should be a short, clear sentence that highlights the most important change. As written, the title will not help teammates quickly understand the purpose of this changeset. Please rename the pull request to a concise sentence that highlights the main updates, for example “Add MCP server documentation and update Dev tools navigation”, to clearly convey both the new guide and related navigation changes.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between cda9f75 and e8f9062.

📒 Files selected for processing (2)
  • dev-tools/build-plane-app.mdx (1 hunks)
  • mint.json (3 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

coderabbitai[bot]
coderabbitai bot reviewed Oct 3, 2025
View reviewed changes
Copy link

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🧹 Nitpick comments (1)
dev-tools/build-plane-app.mdx (1)

42-46: Update internal links to avoid redirect and hash loss

These steps still point to /api-reference/byoa/build-plane-app#.... With the doc now under /dev-tools/build-plane-app, those links rely on a redirect and risk dropping the hash, so users land at the top of the page instead of the intended section. Please point them directly at the local anchors.

-1. [Register your app on Plane developer portal](/api-reference/byoa/build-plane-app/#registering-your-app)
-2. [Implement OAuth flow](/api-reference/byoa/build-plane-app#implement-oauth-flow)
-3. [Obtain and store access tokens securely](/api-reference/byoa/build-plane-app#obtain-and-store-access-tokens-securely)
-4. [Make authenticated API requests to Plane](/api-reference/byoa/build-plane-app#make-authenticated-api-requests-to-plane)
-5. [Handle token refresh](/api-reference/byoa/build-plane-app#handle-token-refresh)
+1. [Register your app on Plane developer portal](#registering-your-app)
+2. [Implement OAuth flow](#implement-oauth-flow)
+3. [Obtain and store access tokens securely](#obtain-and-store-access-tokens-securely)
+4. [Make authenticated API requests to Plane](#make-authenticated-api-requests-to-plane)
+5. [Handle token refresh](#handle-token-refresh)
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 94f9a09 and cda9f75.

📒 Files selected for processing (4)
  • dev-tools/build-plane-app.mdx (1 hunks)
  • dev-tools/intro-webhooks.mdx (0 hunks)
  • dev-tools/mcp-server.mdx (1 hunks)
  • mint.json (3 hunks)
💤 Files with no reviewable changes (1)
  • dev-tools/intro-webhooks.mdx

@danciaclara danciaclara merged commit cf59ae3 into preview Oct 6, 2025
4 of 6 checks passed
@danciaclara danciaclara deleted the mcp-server branch October 6, 2025 08:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

@Prashant-Surya Prashant-Surya Prashant-Surya approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@danciaclara @Prashant-Surya