Skip to content

Conversation

@adityaoberai
Copy link
Member

@adityaoberai adityaoberai commented Sep 11, 2025

What does this PR do?

  • Adds Claude Code to docs homepage
  • Adds a redirect for the /docs/tooling/mcp/claude path to the Claude Desktop docs

Test Plan

  • pnpm i && pnpm dev
  • Visit /docs and scroll down to Build faster with AI section
image

Related PRs and Issues

#2338
#2365

Have you read the Contributing Guidelines on issues?

Yes

Summary by CodeRabbit

  • New Features
    • Added “Claude Code” entry to MCP docs with proper branding.
  • Documentation
    • Updated Claude Desktop card to link to /docs/tooling/mcp/claude-desktop.
    • Improved accessibility by correcting alt text for Windsurf and VS Code icons.
  • Chores
    • Added redirect from /docs/tooling/mcp/claude to /docs/tooling/mcp/claude-desktop for seamless navigation.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 11, 2025

Walkthrough

  • Added a new redirect in src/redirects.json: {"link": "/docs/tooling/mcp/claude", "redirect": "/docs/tooling/mcp/claude-desktop"}.
  • In src/routes/docs/+page.svelte:
    • Updated the Claude Desktop card to link to /docs/tooling/mcp/claude-desktop and changed its image alt to "Claude".
    • Added a new MCP card for "Claude Code" linking to /docs/tooling/mcp/claude-code with alt "Claude".
    • Adjusted alt texts: Windsurf to "Windsurf" and VS Code to "VS Code".
    • Kept the Cursor item unchanged.

Possibly related PRs

Suggested reviewers

  • ebenezerdon
  • arielweinberger

Pre-merge checks (3 passed)

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title "Add Claude Code to docs homepage" accurately and concisely summarizes the primary change in this PR—adding a "Claude Code" card to the docs homepage—which aligns with the modifications in src/routes/docs/+page.svelte and the PR objectives. The added redirect for /docs/tooling/mcp/claude is a minor ancillary change that does not need to be included in a short, focused title.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.

Tip

👮 Agentic pre-merge checks are now available in preview!

Pro plan users can now enable pre-merge checks in their settings to enforce checklists before merging PRs.

  • Built-in checks – Quickly apply ready-made checks to enforce title conventions, require pull request descriptions that follow templates, validate linked issues for compliance, and more.
  • Custom agentic checks – Define your own rules using CodeRabbit’s advanced agentic capabilities to enforce organization-specific policies and workflows. For example, you can instruct CodeRabbit’s agent to verify that API documentation is updated whenever API schema files are modified in a PR. Note: Upto 5 custom checks are currently allowed during the preview period. Pricing for this feature will be announced in a few weeks.

Please see the documentation for more information.

Example:

reviews:
  pre_merge_checks:
    custom_checks:
      - name: "Undocumented Breaking Changes"
        mode: "warning"
        instructions: |
          Pass/fail criteria: All breaking changes to public APIs, CLI flags, environment variables, configuration keys, database schemas, or HTTP/GraphQL endpoints must be documented in the "Breaking Change" section of the PR description and in CHANGELOG.md. Exclude purely internal or private changes (e.g., code not exported from package entry points or explicitly marked as internal).

Please share your feedback with us on this Discord post.

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch add-claude-code-docs-homepage

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.

Copy link
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.

Actionable comments posted: 0

🧹 Nitpick comments (4)
src/routes/docs/+page.svelte (4)

340-351: Make the logo decorative and disambiguate analytics for Claude Desktop.

  • The adjacent text “Claude Desktop” already conveys meaning; set the logo’s alt to empty to avoid SR duplication.
  • Add an item-specific event name to distinguish clicks in analytics.
-                            onclick={() => trackEvent(`docs-mcp-click`)}
+                            onclick={() => trackEvent(`docs-mcp-click--claude-desktop`)}
@@
-                                alt="Claude"
+                                alt=""

388-390: Logo alt should be empty here as well.

Adjacent text “Windsurf Editor” already names the target; make the logo decorative to avoid redundant SR output.

-                                alt="Windsurf"
+                                alt=""

402-404: Same a11y nit: make the VS Code logo decorative.

-                                alt="VS Code"
+                                alt=""

352-365: Add Claude Code — route exists; make logo decorative & analytics event unique

Route exists at src/routes/docs/tooling/mcp/claude-code/+page.markdoc. Update the link in src/routes/docs/+page.svelte:

-                            onclick={() => trackEvent(`docs-mcp-click`)}
+                            onclick={() => trackEvent(`docs-mcp-click--claude-code`)}
@@
-                                alt="Claude"
+                                alt=""
📜 Review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 84df194 and db0ab2c.

📒 Files selected for processing (2)
  • src/redirects.json (1 hunks)
  • src/routes/docs/+page.svelte (3 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: tests
  • GitHub Check: build
🔇 Additional comments (2)
src/redirects.json (2)

728-730: Redirect LGTM.

Mapping /docs/tooling/mcp/claude → /docs/tooling/mcp/claude-desktop aligns with the updated card. JSON remains valid.


728-730: Sanity check passed — no duplicate "link" entries; JSON parses cleanly.
Script output: OK: no duplicate links.

@adityaoberai adityaoberai merged commit 5a78ed0 into main Sep 11, 2025
4 checks passed
@coderabbitai coderabbitai bot mentioned this pull request Sep 23, 2025
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.

4 participants