fix(platform): validate Copilot and Antigravity ports#658
Conversation
code-review-graph reviewOverall risk: 0.80 (HIGH) — 25 changed function(s)/class(es), 0 affected flow(s), 14 test gap(s) Risk-scored changes
Test gaps
Token savings: this graph-backed report used ~91,518 fewer tokens (~95%) than reading every changed file in full (estimated, chars/4 approximation). Powered by code-review-graph — local-first analysis; no code leaves the CI runner. |
Use the current GitHub Copilot CLI MCP schema, detect the actual VS Code Copilot extension, and install current workspace-scoped Antigravity skills without importing the stale hook format. Co-authored-by: npkriami18 <pranaynannuri@gmail.com> Co-authored-by: 0deep <227102371+0deep@users.noreply.github.com>
28241bb to
c5a7815
Compare
|
Real-client validation found a blocking Copilot detection gap; this checklist item is FAILED, not passed.
Keep this PR in draft. Detection must recognize the released bundled extension, then VS Code must be reopened against the generated workspace MCP configuration and retested. No merge until that passes. |
|
Follow-up: the bundled-Copilot detection blocker from the previous comment is fixed in commit 5496791.
The VS Code checklist item is complete. Copilot CLI invocation, reinstall/uninstall, and all Antigravity items are still unverified. This PR remains draft and must not be merged yet. |
|
Released-client evidence for head
All GitHub Copilot gates are now satisfied. This remains a draft and must not merge until the four released Google Antigravity checks pass. |
rajpratham1
left a comment
There was a problem hiding this comment.
This pull request provides a well-scoped improvement to platform integration by strengthening GitHub Copilot detection, aligning GitHub Copilot CLI with the documented mcpServers configuration schema, and adding support for Antigravity workspace-scoped skills. The implementation updates the installation and uninstallation workflows, improves configuration accuracy, documents the new behavior, and includes comprehensive test coverage for the added functionality. The changes are consistent across the implementation, documentation, and tests, and no blocking issues were identified during review. LGTM and approved.
Latest refresh — 18 July 2026
0a3bd6c; current head5496791Status
Draft — do not merge until the released Antigravity checks below pass.
This completes the focused maintainer ports promised when source PRs #129 and #531 were closed. Contributor attribution is preserved.
Change
mcpServerscontract withtype: localandtools: ["*"].agents/skills/<skill>/SKILL.md; do not carry the source PR's obsolete hooksAutomated validation
git diff --check: cleanReal-client result
PASSED — GitHub Copilot:
GitHub.copilot-chat0.55.0 was detected.vscode/mcp.jsonwithout a configuration parse error~/.copilot/mcp-config.jsonfrom the user sourcelist_graph_stats_tool; the real result returnedtotal_nodes: 4334code-review-graphfrom both configs, and preserved the unrelated servers/settingsRequired real-client validation
Do not mark this ready or merge it until all items below are recorded with released clients.
GitHub Copilot
~/.copilot/mcp-config.jsonand invokes a code-review-graph tool successfullyGoogle Antigravity
.agentsskillCarries PR #129 with
npkriami18credited and PR #531 with0deepcredited.