feat(mcp): add windsurf host to init-client config#2608
feat(mcp): add windsurf host to init-client config#2608glorydavid03023 wants to merge 2 commits into
Conversation
Add `windsurf` to `gittensory-mcp init-client --print`. Windsurf reads ~/.codeium/windsurf/mcp_config.json using the same shared `mcpServers` JSON shape as claude/cursor/mcp, so it reuses that snippet branch rather than needing its own (unlike vscode's `servers`/`type` shape). Updates the usage text, the empty-client and unsupported-client error messages, and the README example + host description. A test asserts the windsurf snippet equals the shared mcpServers snippet. No issue because issue creation is restricted on this repo; this is a small additive host, mirroring the vscode host addition (JSONbored#1770), no schema/API change.
|
Caution 🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥🟥 🛑 Gittensory review result - reject/close recommendedReview updated: 2026-07-02 23:23:48 UTC
🛑 Suggested Action - Reject/Close
Review summary Blockers
Nits — 3 non-blocking
Why this is blocked
Review context
Contributor next steps
Signal definitions
🟩 Safe / merged · 🟦 Advisory · 🟨 Held for review · 🟥 Blocked / closed 💰 Earn for open-source contributions like this. Gittensor lets GitHub contributors earn for the work they already do — register to start earning →. Checked by Gittensory, a quiet PR intelligence layer for OSS maintainers.
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2608 +/- ##
==========================================
+ Coverage 96.07% 96.09% +0.02%
==========================================
Files 235 237 +2
Lines 26357 26444 +87
Branches 9559 9587 +28
==========================================
+ Hits 25322 25412 +90
Misses 425 425
+ Partials 610 607 -3 🚀 New features to boost your workflow:
|
JSONbored
left a comment
There was a problem hiding this comment.
Failing gittensory review, closing.
Summary
Adds
windsurftogittensory-mcp init-client --print, so Windsurf users can generate their MCP config the same way as the other supported hosts.Windsurf reads
~/.codeium/windsurf/mcp_config.jsonusing the same sharedmcpServersJSON shape asclaude/cursor/mcp, so it reuses that snippet branch (no new snippet is needed, unlikevscode'sservers/"type": "stdio"shape).This mirrors the earlier
vscodehost addition (#1770): the usage text, the empty-client and unsupported-client error messages, and the README example + host description are all updated, and a test asserts thewindsurfsnippet equals the sharedmcpServerssnippet.No issue because issue creation is restricted on this repo for outside accounts; this is a small additive host with no schema or API change.
Scope
Validation
git diff --checknpm run typechecknpm run build:mcp(node --checkon the CLI is clean)npm run test:mcp-pack(see note)npm run test:coverage(scoped:test/unit/mcp-cli-basics.test.ts— 20 pass, including the addedwindsurfsnippet assertion; the CLI is exercised by spawninginit-client --print windsurf)npm run ui:openapi:check/ui:*— N/A (no UI change; mirrors feat(mcp): add VS Code host to init-client config #1770 which touched no UI)npm audit --audit-level=moderateIf any required check was skipped, explain why:
src/**/*.tsinclude, so no patch-coverage obligation).build:mcppasses andinit-client --print windsurfprints the sharedmcpServerssnippet;mcp-cli-basics.test.tspasses (20). I could not runtest:mcp-packlocally because this machine runs Node 24 while the repo pins Node 22 (.nvmrc);scripts/check-mcp-package.mjsthrowsERR_INVALID_ARG_TYPEon Node 24 identically on unmodifiedmain. CI runs the full matrix on the pinned Node.Safety
Notes
vscodehost addition (feat(mcp): add VS Code host to init-client config #1770): CLI usage/errors/snippet + README + test, no UI files.