Skip to content

fix: do not treat Cursor servers-only MCP files as configured - #253

Merged
SebTardif merged 1 commit into
mainfrom
fix/improve-mpi-20260828-s0825
Aug 28, 2026
Merged

fix: do not treat Cursor servers-only MCP files as configured#253
SebTardif merged 1 commit into
mainfrom
fix/improve-mpi-20260828-s0825

Conversation

@SebTardif

Copy link
Copy Markdown
Contributor

Summary

Inspect treated a Cursor `.cursor/mcp.json` with only `servers.patchloom`
as configured. Cursor does not load that key. The status bar showed
ready and Setup Workspace skipped Configure MCP.

Inspect now requires `mcpServers.patchloom`, matching Cursor and
what Configure MCP writes after #252.

Verification

  • Unit test: servers-only Cursor file is exists=true, configured=false
  • `npm run check`

Checklist

  • All commits in this pull request are signed off with `git commit -s`
  • I ran `npm run check` and it passes
  • I updated docs if user-facing behavior changed
  • I am contributing this work under the repository license (MIT)

Cursor only loads mcpServers. A pre-fix .cursor/mcp.json with
servers.patchloom made the status bar look done while Cursor ignored
the file. Inspect now matches what Cursor reads.

Signed-off-by: Sebastien Tardif <sebtardif@ncf.ca>
@SebTardif
SebTardif marked this pull request as ready for review August 28, 2026 15:18
@SebTardif
SebTardif enabled auto-merge (squash) August 28, 2026 15:18
@SebTardif
SebTardif merged commit 9d896a0 into main Aug 28, 2026
22 checks passed
@SebTardif
SebTardif deleted the fix/improve-mpi-20260828-s0825 branch August 28, 2026 15:23
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