chore(copilot): deprecate mcp server#4797
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
PR SummaryMedium Risk Overview Runtime: Docs: The Copilot MCP section (API keys, Cursor / Claude / VS Code config) is stripped from copilot pages in en, de, es, fr, ja, zh. Hygiene: Vitest coverage asserts 410 behavior (including with Reviewed by Cursor Bugbot for commit 98f7278. Configure here. |
Greptile SummaryThis PR deprecates the Copilot MCP server by replacing ~900 lines of active implementation across the MCP route, OAuth discovery endpoints, and v1 headless chat API with minimal 410 Gone stubs. Docs are cleaned up across all six supported languages, and the contract-validation script is updated to exempt the now-dead routes.
Confidence Score: 4/5Safe to merge once the Vim swap file is removed — all active code paths are correctly replaced with 410 stubs and tests cover every deprecated endpoint. The deprecation itself is clean and well-tested. The only concern is the committed packages/db/.env.swp binary file: it is a Vim swap file for the packages/db/.env file and may contain secrets if that file held real credentials at edit time. Until confirmed inert, merging it extends that content into the permanent git history. packages/db/.env.swp — should be removed before merging; verify whether the underlying .env file contained real credentials at the time the swap file was created.
|
| Filename | Overview |
|---|---|
| packages/db/.env.swp | Vim swap file for packages/db/.env accidentally committed to the repository; binary file that should not be tracked in git. |
| apps/sim/app/api/mcp/copilot/route.ts | 740-line MCP server implementation replaced with three stub handlers (GET, POST, DELETE) that all return 410 Gone; runtime and maxDuration exports correctly removed. |
| apps/sim/lib/mcp/copilot-deprecated.ts | New helper module with two 410 Gone response factories — a plain JSON response and a JSON-RPC-flavored one. Clean, minimal, well-commented. |
| apps/sim/app/api/v1/copilot/chat/route.ts | 151-line headless copilot chat handler replaced with a single POST stub returning 410 with a success:false error body. |
| apps/sim/app/api/mcp/copilot/route.test.ts | New test file with 6 test cases covering all deprecated handlers (GET, POST, DELETE, and both OAuth discovery sub-routes); good coverage of the 410 contract. |
| scripts/check-api-validation-contracts.ts | Correctly moves the four deprecated routes from RAW_JSON_BASELINE_ROUTES to INDIRECT_ZOD_ROUTES with appropriate comments explaining the exemption. |
Reviews (2): Last reviewed commit: "deprecate copilot api v1 route" | Re-trigger Greptile
|
@greptile |
|
bugbot run |
There was a problem hiding this comment.
✅ Bugbot reviewed your changes and found no new issues!
Comment @cursor review or bugbot run to trigger another review on this PR
Reviewed by Cursor Bugbot for commit 98f7278. Configure here.
Summary
Almost no usage, based on very old version of copilot that does not work for most things
Cleans up docs
Type of Change
Testing
N/A
Checklist