Skip to content

MCP Servers#348

Open
pepeladeira wants to merge 1 commit intomainfrom
mcp-servers
Open

MCP Servers#348
pepeladeira wants to merge 1 commit intomainfrom
mcp-servers

Conversation

@pepeladeira
Copy link
Copy Markdown
Collaborator

@pepeladeira pepeladeira commented Apr 30, 2026

Summary by CodeRabbit

  • Documentation
    • Added comprehensive "Dub Links MCP Server" documentation with setup instructions for multiple MCP clients (Cursor, Claude Desktop, VS Code Copilot, and others) and available tools for link and analytics management
    • Added comprehensive "Dub Partners MCP Server" documentation with client setup guides and partnership management tools
    • Added new "MCP SERVERS" navigation group under Developer Docs for improved discoverability

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 30, 2026

📝 Walkthrough

Walkthrough

Three documentation files are added: a navigation configuration entry for an "MCP SERVERS" group and two new MDX documentation pages for the Dub Links and Dub Partners MCP servers, each including setup instructions and tool descriptions.

Changes

Cohort / File(s) Summary
Navigation Configuration
docs.json
Adds "MCP SERVERS" navigation group under Developer Docs, linking to two new MCP server documentation pages.
MCP Server Documentation
docs/sdks/mcp/dub-links.mdx, docs/sdks/mcp/dub-partners.mdx
Two new MDX pages documenting Dub Links and Dub Partners MCP servers with authentication requirements, multi-client setup instructions (Cursor, Claude, VS Code Copilot, Gemini CLI), and enumerated tool capabilities organized by feature category.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Poem

🐰 Hop hop! Fresh docs are here today,
MCP servers on display,
Dub Links and Partners in sight,
Setup instructions burning bright,
Configuration guide oh so right! ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'MCP Servers' directly and clearly summarizes the main change: adding documentation for MCP server integrations (dub-links and dub-partners) with setup instructions and available tools.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch mcp-servers

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
Review rate limit: 0/1 reviews remaining, refill in 60 minutes.

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

@mintlify
Copy link
Copy Markdown
Contributor

mintlify Bot commented Apr 30, 2026

Preview deployment for your docs. Learn more about Mintlify Previews.

Project Status Preview Updated (UTC)
Dub 🟢 Ready View Preview Apr 30, 2026, 10:26 PM

💡 Tip: Enable Workflows to automatically generate PRs for you.

Copy link
Copy Markdown
Contributor

@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.

🧹 Nitpick comments (1)
docs/sdks/mcp/dub-links.mdx (1)

26-27: ⚡ Quick win

Add an explicit API key safety note near setup instructions.

Please add a short warning to avoid storing/committing real MCP_DUB_TOKEN values in shared config files, and prefer local environment variables/secret managers.

Suggested doc tweak
 ## Setup

 Choose your MCP client below to get started. Replace `dub_xxxxxx` with your actual Dub API key.
+Never commit real API keys to source control. Store `MCP_DUB_TOKEN` in your local environment or a secret manager.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@docs/sdks/mcp/dub-links.mdx` around lines 26 - 27, Add a short explicit
safety warning next to the setup instructions that use
`dub_xxxxxx`/MCP_DUB_TOKEN: state not to store or commit real `MCP_DUB_TOKEN`
values in shared config or version control and instruct readers to use local
environment variables (e.g., .env or export), or a secret manager/CI secret
storage instead; include a one-line example suggestion and a brief note to
rotate keys if accidentally committed.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@docs/sdks/mcp/dub-links.mdx`:
- Around line 26-27: Add a short explicit safety warning next to the setup
instructions that use `dub_xxxxxx`/MCP_DUB_TOKEN: state not to store or commit
real `MCP_DUB_TOKEN` values in shared config or version control and instruct
readers to use local environment variables (e.g., .env or export), or a secret
manager/CI secret storage instead; include a one-line example suggestion and a
brief note to rotate keys if accidentally committed.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 742b518d-913d-49bf-8e64-00ff71d2beb2

📥 Commits

Reviewing files that changed from the base of the PR and between 7834cc0 and a49866c.

📒 Files selected for processing (3)
  • docs.json
  • docs/sdks/mcp/dub-links.mdx
  • docs/sdks/mcp/dub-partners.mdx

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.

1 participant