docs: add Copilot CLI to all documentation#653
Conversation
PR SummaryLow Risk Overview Updates the Copilot CLI integration one-pager ( Written by Cursor Bugbot for commit f49e5e9. Configure here. |
There was a problem hiding this comment.
Pull request overview
Updates project documentation to consistently include Copilot CLI as a supported agent for Entire CLI, aligning docs across user guides, architecture references, and contributor materials.
Changes:
- Add Copilot CLI to agent lists across README, contributing docs, architecture docs, and security/privacy docs.
- Extend E2E documentation to include Copilot CLI in example commands, env vars, and CI workflow descriptions.
- Update Copilot CLI agent documentation (
AGENT.md) with revised token-usage guidance.
Reviewed changes
Copilot reviewed 10 out of 10 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| e2e/README.md | Adds Copilot CLI to E2E docs (commands, env vars, CI workflow notes). |
| docs/security-and-privacy.md | Includes Copilot CLI in the “Where data is stored” agent list. |
| docs/architecture/sessions-and-checkpoints.md | Adds Copilot CLI to the overview agent list. |
| docs/architecture/agent-integration-checklist.md | Updates integration checklist examples to mention Copilot CLI formats and patterns. |
| docs/architecture/agent-guide.md | Adds Copilot CLI examples for hook config + updates lifecycle hook mapping table and naming section. |
| cmd/entire/cli/agent/copilotcli/AGENT.md | Updates Copilot CLI agent-format documentation, especially around token usage. |
| README.md | Adds GitHub Copilot CLI to prerequisites and introduces a Copilot CLI section with enable instructions. |
| CONTRIBUTING.md | Updates contributor docs to mention Copilot CLI among supported integrations. |
| CLAUDE.md | Updates developer reference to include Copilot CLI in E2E command examples and agent lists. |
| .github/copilot-instructions.md | Updates Copilot-specific repo overview to include Copilot CLI among supported agents. |
![claude[bot]](https://avatars.githubusercontent.com/in/1236702?s=60&v=4){kind=small}
There was a problem hiding this comment.
LGTM — straightforward docs-only PR adding Copilot CLI to agent lists; the two nits flagged inline are worth fixing but non-blocking.
Extended reasoning...
Overview
This PR adds Copilot CLI references across 10 documentation and config files: README, CLAUDE.md, CONTRIBUTING.md, copilot-instructions.md, agent-guide.md, agent-integration-checklist.md, sessions-and-checkpoints.md, security-and-privacy.md, e2e/README.md, and the mise e2e task script. The changes are mechanical — inserting Copilot CLI into lists of supported agents, adding the copilot-cli E2E command examples, and adding the Copilot CLI section to the README. It also fixes some pre-existing inaccuracies (e.g., updating cursor to cursor-cli in E2E docs).
Security risks
None. This is a documentation-only PR with no runtime code changes. The only non-markdown file touched is mise-tasks/test/e2e/_default, where the change is limited to updating the help text for the --agent flag.
Level of scrutiny
Low scrutiny appropriate. Documentation-only changes with no behavioral impact. The two inline nits (camelCase hook names in the agent-guide table and GITHUB_TOKEN vs COPILOT_GITHUB_TOKEN in e2e/README) are real inaccuracies but are minor and the GITHUB_TOKEN issue was already flagged by the Copilot reviewer (marked resolved).
Other factors
All Copilot reviewer inline comments are marked as resolved, suggesting the author addressed them (some fixes like adding cursor-cli E2E examples are visible in the diff). The two remaining nits from the bug hunting system are legitimate but low-impact documentation inconsistencies that do not affect any runtime behavior.
No description provided.