Skip to content

docs: Cursor mcp.json snippet#69

Open
kauercurtis wants to merge 2 commits into
hoainho:mainfrom
kauercurtis:docs/cursorMCP
Open

docs: Cursor mcp.json snippet#69
kauercurtis wants to merge 2 commits into
hoainho:mainfrom
kauercurtis:docs/cursorMCP

Conversation

@kauercurtis

Copy link
Copy Markdown

Linked issue

Closes #4

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • Feature (non-breaking change that adds functionality)
  • Performance improvement
  • [X ] Documentation only
  • Refactor (no behavior change)
  • Test additions or fixes
  • Chore (tooling, deps, build config)

What changed

Testing notes

npm run test:run

npm run build

$ npm run test:run
...

$ npm run build
...

Claim confirmation

If this PR closes a good first issue or help wanted issue, please confirm:

  • [ X] I starred the repo ⭐ — see CONTRIBUTING.md → How to claim. This is now enforced by CI (.github/workflows/star-check.yml) — the "Star Check" status will block merge until you star.
  • [ X] I commented I'll take this (or similar) on the issue before starting work, so two people don't accidentally race on the same issue. (Honor-system; not CI-enforced.)

If this PR is from a maintainer or a follow-up to a tracked plan, both can be skipped — just delete this section. Maintainer/bot/tracked-plan-labeled PRs are auto-exempted by the Star Check workflow.

Checklist

  • [ X] npm run test:run passes locally (or pre-existing failures are unrelated to this change)
  • [ X] npm run build succeeds locally
  • [ X] CHANGELOG.md entry added under ## [Unreleased] (skip for docs-only or test-only PRs)
  • [ X] PR title follows Conventional Commits (e.g., fix(panel): ...)
  • [ X] No new runtime dependencies added without prior discussion in a Discussion

Screenshots / GIFs

Additional notes

The previous issues with Claude and Open Code weren't available so I had no idea if there was an actual endpoint for Cursor to find for the MCP server.

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds a new "Cursor" configuration section to the MCP_USAGE.md documentation, detailing the configuration file paths and providing a JSON snippet for integration. The review feedback points out a typo ("you system's" instead of "your system's") and incorrect configuration file paths for Cursor on Windows and macOS/Linux.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.

Comment thread cli/docs/MCP_USAGE.md

## Cursor

To use this AI integrated MCP client, add the configuration snippet below to you system's `mcp.json` file.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

There is a typo in this line: "you system's" should be "your system's".

Suggested change
To use this AI integrated MCP client, add the configuration snippet below to you system's `mcp.json` file.
To use this AI integrated MCP client, add the configuration snippet below to your system's `mcp.json` file.

Comment thread cli/docs/MCP_USAGE.md Outdated
Comment on lines +43 to +44
- **Windows:** `%USERPROFILE%\.CURSOR\mcp.json`
- **macOS/Linux** `~/.cursor.json`

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

The configuration file paths for Cursor contain a few issues:

  1. On macOS/Linux, the path is ~/.cursor/mcp.json (a directory named .cursor containing mcp.json), not ~/.cursor.json.
  2. On Windows, the directory is typically lowercase .cursor rather than .CURSOR.
  3. There is a missing colon after macOS/Linux.
Suggested change
- **Windows:** `%USERPROFILE%\.CURSOR\mcp.json`
- **macOS/Linux** `~/.cursor.json`
- **Windows:** `%USERPROFILE%\.cursor\mcp.json`
- **macOS/Linux:** `~/.cursor/mcp.json`

@hoainho hoainho left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

Review: ⚠️ Needs fixes

Thanks for the contribution! A couple of issues to fix before merging:

  1. Typo: "you system's" → "your system's" in the Cursor intro paragraph
  2. Wrong macOS/Linux path: ~/.cursor.json should be ~/.cursor/mcp.json. Cursor's MCP config file is ~/.cursor/mcp.json on both macOS and Linux, not .cursor.json.

Could you please fix both and push an update?

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.

[Easy] Add docs/MCP_USAGE.md mcp.json snippet for Cursor

2 participants