docs: list copilot in remaining harness enumerations and document --add-dir#459
Merged
Conversation
…dd-dir Sweep the reference/topic docs that still claimed only Codex and Claude Code where GitHub Copilot CLI is a shipped built-in harness, and add the missing --add-dir row to the coven run reference. Follow-up flagged in the bodies of merged PRs #453/#454/#457. Closes #458 Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Signed-off-by: Val Alexander <68980965+BunsDev@users.noreply.github.com>
Contributor
There was a problem hiding this comment.
Pull request overview
Docs-only sweep to bring remaining reference/topic pages in line with the shipped built-in harness set (Codex, Claude Code, GitHub Copilot CLI) and to document coven run --add-dir.
Changes:
- Update remaining “supported harness” enumerations to include GitHub Copilot CLI where appropriate.
- Add
--add-dirto thecoven runreference (flags table + behavior description). - Refresh
coven doctorand roadmap wording to reflect Copilot install/auth hints and recovery guidance.
Reviewed changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| docs/start/showcase.md | Updates the showcase “today” card to include Copilot alongside Codex and Claude Code. |
| docs/ROADMAP.md | Updates shipped-list bullets to include Copilot in built-in harness adapters, run examples, and recovery hints. |
| docs/reference/harness-adapters.md | Updates the page summary to include Copilot in the harness list. |
| docs/reference/cli.md | Updates the top-level CLI table row to include copilot in the harness ids list. |
| docs/reference/cli-run.md | Documents --add-dir in the Common Flags table, including repeatability and fallback behavior. |
| docs/reference/cli-doctor.md | Updates the “missing harness” section to include Copilot install/auth hints. |
| docs/concepts/features.md | Updates the features card copy to include GitHub Copilot CLI in the “today” harness list. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| | `coven status` | Ecosystem overview: daemon, sessions, familiars, skills, research, hub. Alias: `coven overview`. | | ||
| | `coven daemon start/status/restart/stop` | Manage the local daemon. | | ||
| | `coven run <harness> <prompt>` | Launch a project-scoped harness session. Current harness ids: `codex`, `claude`. | | ||
| | `coven run <harness> <prompt>` | Launch a project-scoped harness session. Current harness ids: `codex`, `claude`, `copilot`. | |
| @@ -1,5 +1,5 @@ | |||
| --- | |||
| summary: "The adapter shape Codex and Claude Code use today." | |||
| summary: "The adapter shape Codex, Claude Code, and GitHub Copilot CLI use today." | |||
| - Versioned `coven.daemon.v1` API contract with named apiVersion, machine-readable capabilities, structured errors, and monotonic event cursors. See [`docs/API-CONTRACT.md`](/API-CONTRACT). | ||
| - Compatibility tests for the external OpenClaw bridge against versioned daemon responses. | ||
| - First-run recovery hints for missing Codex or Claude Code CLIs. | ||
| - First-run recovery hints for missing Codex, Claude Code, or GitHub Copilot CLIs. |
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
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.
Context
coven run --add-dirflag. Follow-up flagged in the bodies of merged PRs docs: fix broken links and stale harness claims in entry-point docs #453/fix: make CLI help text truthful and consistent #454/docs: slim README and reconcile duplicated doc pairs #457.Implementation
crates/coven-cli/src/harness.rsbuilt_in_harness_specs()(codex, claude, copilot), copilot install hint from the copilot spec,--add-dirbehavior frommain.rs:234-238.Verification
cargo fmt --check— n/a (docs-only)cargo clippy --workspace --all-targets -- -D warnings— n/a (docs-only)cargo test --workspace --locked— n/a (docs-only)python3 scripts/check-secrets.py— clean (current tree + history)git diff --checkclean;node --test scripts/onboarding-docs-test.mjs13/13 pass; every replacement cross-checked against harness.rs/main.rs.Risk and Rollback
Agent Handoff