Skip to content

Add support to copilot - #35

Open
luisiul359 wants to merge 3 commits into
mainfrom
RAOPST-3715-copilot
Open

Add support to copilot#35
luisiul359 wants to merge 3 commits into
mainfrom
RAOPST-3715-copilot

Conversation

@luisiul359

@luisiul359 luisiul359 commented Jul 21, 2026

Copy link
Copy Markdown

This PR is for adding Microsoft Copilot as a supported harness, alongside the existing Claude Code plugin and Kiro Power.

  • Install prompts added to README.md for the three interactive surfaces — VS Code, Copilot CLI, and Visual Studio — in the same paste-into-your-assistant style as the Claude Code and Kiro sections.
  • copilot/mcp.json — a single canonical config carrying both top-level keys: servers (VS Code + Visual Studio) and mcpServers (Copilot CLI). One source of truth the install prompts copy the matching key out of.
  • copilot/skills.md — the conventions doc, adapted from root SKILL.md and structured like the Kiro steering file (intro → Copilot First use / setup → full conventions body). Read automatically by VS Code, Visual Studio, the CLI (which also honors AGENTS.md), and the cloud agent.

@luisiul359
luisiul359 requested a review from a team as a code owner July 21, 2026 15:21
@luisiul359 luisiul359 added the do not merge This pull request should not be merged even if checks pass label Jul 21, 2026
@luisiul359
luisiul359 force-pushed the RAOPST-3715-copilot branch from 26ea90b to 1736ca8 Compare July 27, 2026 11:38
@luisiul359 luisiul359 removed the do not merge This pull request should not be merged even if checks pass label Jul 27, 2026

@david-santos-os david-santos-os left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed with focus on two things: (a) parity of copilot/skill.md with the existing Claude Code / Kiro / root skill docs, and (b) whether the README changes affect the docs for the existing harnesses.

Overall: solid addition. I diffed copilot/skill.md section-by-section against the other three skill docs: every shared section (intro, Tools at a glance, Caveats, Rules, Names, Answering, Mentor session round-trip, Context Service visibility, Workflows) is byte-identical, with only the harness-specific First use / setup and Authenticating sections diverging — exactly the intended shape. The README additions are pure insertions and the existing harness sections are untouched. The dual-key canonical copilot/mcp.json, with comments explaining which surface reads which key, is a nice single-source-of-truth touch.

Five pre-merge asks, each as an inline comment:

  1. Update CLAUDE.md + CONTRIBUTING.md for the fourth lockstep doc — the most important one (anchored on copilot/skill.md because those files aren't in this diff).
  2. Restore an updated "harnesses we test against" sentence in the README.
  3. Add the missing "retry the user's original request" step to the Copilot setup flow.
  4. MINOR version bump per the CONTRIBUTING.md policy (anchored on the README because .claude-plugin/* isn't in this diff).
  5. A sweep of small nits (fence typo, step-4 wording, VS auth path, casing, trailing newline, stale PR description).

One thing I couldn't validate locally: the Copilot CLI syntax (copilot mcp add --transport http, /mcp reload, /mcp show) and the Visual Studio "Manage Authentication" CodeLens. GitHub iterates these surfaces quickly, so a quick confirmation in the PR that each of the three flows was run end-to-end on current versions would be great.

Comment thread copilot/skill.md
Comment thread README.md
Comment thread copilot/skill.md
Comment thread README.md
Comment thread README.md
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.

2 participants