feat: add GitHub Copilot platform support#382
Open
Vignesh-Thangamariappan wants to merge 3 commits intotirth8205:mainfrom
Open
feat: add GitHub Copilot platform support#382Vignesh-Thangamariappan wants to merge 3 commits intotirth8205:mainfrom
Vignesh-Thangamariappan wants to merge 3 commits intotirth8205:mainfrom
Conversation
- Add 'copilot' entry to PLATFORMS dict in skills.py - Config: .vscode/mcp.json with 'servers' key (VS Code MCP format) - Detection: checks ~/.vscode exists - needs_type: True (VS Code requires type: stdio) - Add .github/copilot-instructions.md to _PLATFORM_INSTRUCTION_FILES - Add 'copilot' to _PLATFORM_CHOICES in cli.py - Add TestCopilotPlatform test class with 8 tests covering platform entry, config install, idempotency, dry-run, and instruction file injection - Update inject_platform_instructions tests to include new file - Update README.md platform lists and install example Usage: code-review-graph install --platform copilot Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
- Add 'copilot-cli' entry to PLATFORMS dict in skills.py - Config: ~/.copilot/mcp-config.json (user-level, global) - Key: 'servers' (VS Code MCP format, same as copilot platform) - Detection: checks ~/.copilot exists - needs_type: True (type: stdio required) - Add 'copilot-cli' as co-owner of .github/copilot-instructions.md in _PLATFORM_INSTRUCTION_FILES so install --platform copilot-cli also injects graph-aware instructions into the project - Add 'copilot-cli' to _PLATFORM_CHOICES in cli.py - Add TestCopilotCLIPlatform test class (8 tests) covering: platform entry metadata, config install, idempotency, existing server preservation, instruction injection, dry-run, all-detection, and user-level config path assertion - Update README platform lists and install examples Unlike the 'copilot' platform (.vscode/mcp.json, project-level), copilot-cli writes to ~/.copilot/mcp-config.json making the MCP server available globally across all projects in the terminal. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
wax911
reviewed
Apr 25, 2026
| ## MCP Tools: code-review-graph | ||
|
|
||
| **IMPORTANT: This project has a knowledge graph. ALWAYS use the | ||
| code-review-graph MCP tools BEFORE using Grep/Glob/Read to explore |
There was a problem hiding this comment.
If we are truly targeting copilot instructions then we might as well reference the adequate tools in addition to terminal commands in the case the model does not text match on the explicitness of the tools when invoking runInTerminal
Suggested change
| code-review-graph MCP tools BEFORE using Grep/Glob/Read to explore | |
| code-review-graph MCP tools BEFORE using #tool:read/readFile #tool:search/fileSearch #tool:search/textSearch to explore |
…ool refs Per PR tirth8205#382 review feedback: - Rename .github/copilot-instructions.md → .github/code-review-graph.instruction.md to avoid conflicts with users' existing copilot-instructions.md files - Add YAML front matter (applyTo, description) so Copilot Chat applies the instructions workspace-wide as a skill - Replace generic 'Grep/Glob/Read' with Copilot-specific tool references (#tool:read/readFile, #tool:search/fileSearch, #tool:search/textSearch) so the model text-matches on explicit tool names - Add _COPILOT_SECTION constant and _PLATFORM_INSTRUCTION_CUSTOM_SECTIONS dict in skills.py to use the Copilot-tailored section for the instruction file - Update all test references to the new filename Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Author
|
Resolved both comments. Kindly re-review @wax911 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
GitHub Copilot (VS Code) platform support
copilotentry toPLATFORMSdict inskills.py.vscode/mcp.jsonwithserverskey (VS Code MCP format)~/.vscodeexistsneeds_type: True(VS Code requirestype: stdio)GitHub Copilot CLI platform support
copilot-clientry toPLATFORMSdict inskills.py~/.copilot/mcp-config.jsonwithserverskey (user-level)~/.copilotexistsneeds_type: TrueCopilot instruction file
.github/code-review-graph.instruction.mdto_PLATFORM_INSTRUCTION_FILEScode-review-graph.instruction.md(notcopilot-instructions.md) to avoid conflicts with users' existing filesapplyTo: '**',description) so Copilot Chat applies it workspace-wide as a skill#tool:read/readFile,#tool:search/fileSearch,#tool:search/textSearch) for explicit tool matchingskills.py internals
_COPILOT_SECTIONconstant with Copilot-tailored instruction content_PLATFORM_INSTRUCTION_CUSTOM_SECTIONSdict to route per-file content ininject_platform_instructionsCLI & tests
copilotandcopilot-clito_PLATFORM_CHOICESincli.pyTestCopilotPlatformandTestCopilotCLIPlatformtest classes (16 tests) covering platform entry, config install, idempotency, dry-run, instruction file injection, and detectioninject_platform_instructionstests to include the new instruction fileREADME
--platform copilotand--platform copilot-cliexamplesUsage