[AI-815] Surface --cursor/--copilot in kcap --help Plugin section#163
Conversation
The top-level `kcap --help` (help-usage.txt) Plugin summary still read `[--codex]` only, so users couldn't discover `plugin install --copilot` (or --cursor) from the main help — even though both have been supported since AI-731/AI-815 and the dedicated `kcap plugin --help` documents them. The import line and hook section already listed every vendor; this brings the Plugin two-liner in sync with PluginCommand's actual dispatch (--codex|--cursor|--copilot|--skills). Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
realtonyyoung
left a comment
There was a problem hiding this comment.
One consistency issue to fix before merging: this updates the top-level kcap --help Plugin summary to include --copilot, but the fallback usage emitted by PluginCommand.PrintUsage() still says Usage: kcap plugin <install|remove> [--project] [--codex|--cursor|--skills] [--if-installed]. So users who run an invalid kcap plugin ... command still get the stale target list, which undercuts the discoverability fix. Can we update that usage string to include --copilot as well?
The fallback usage emitted by `kcap plugin <bad-args>` still listed [--codex|--cursor|--skills], omitting --copilot — same staleness as the top-level help this PR fixes. Bring it in sync so the error path is also discoverable. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Fixed in 3642781 — |
realtonyyoung
left a comment
There was a problem hiding this comment.
One remaining docs consistency nit while this PR is already updating vendor discoverability: README.md still has the uninstall guidance saying kcap plugin remove [--codex|--cursor|--skills] for per-agent selective cleanup. Since plugin remove --copilot is now supported and this PR updates both top-level and fallback plugin usage to include it, can we add --copilot there too (or intentionally leave the README for a separate docs pass)?
…view) Review (kcap-cli#163): the uninstall section still listed `kcap plugin remove [--codex|--cursor|--skills]` for per-agent cleanup, omitting the now-supported --copilot. Matches the top-level help and PrintUsage fallback this PR already updated. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
realtonyyoung
left a comment
There was a problem hiding this comment.
Re-reviewed the latest revision. The prior consistency gaps are addressed: top-level help, PluginCommand.PrintUsage(), and the README per-agent cleanup line all now include --copilot. I do not have any further findings from source/diff inspection. Per request, I did not run tests.
help-usage.txt's Plugin: summary listed --codex|--cursor|--copilot|--skills but not --gemini — #163 rewrote that line after the #164 branch point, so the 3-way merge left Gemini off. The import line, hooks section, and full help-plugin.txt already advertise it; this aligns the one-line summary. Co-authored-by: Claude Opus 4.8 <noreply@anthropic.com>
Brings in the Gemini CLI vendor (AI-887, #164), help-text refresh (#163), and the YAML redactor fix (#166). All conflicts were parallel-vendor additions (Kiro vs Gemini) resolved by keeping both, Gemini before Kiro. Build + 1567 unit tests green; NativeAOT publish clean. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Problem
kcap --help(top-level usage) listed only[--codex]under Plugin, so a user reading the main help couldn't discoverplugin install --copilotor--cursor— even though both have shipped (AI-731 Cursor, AI-815 Copilot) andkcap plugin --helpdocuments them in full. Reported from the field: "looked at kcap --help and didn't see anything for Copilot."The
importline and theHookssection already enumerate all four vendors; only this Plugin two-liner was stale.Fix
Bring the Plugin summary in sync with
PluginCommand's actual dispatch (--codex|--cursor|--copilot|--skills):Docs-only (embedded help resource). Verified by running
kcap --helpfrom a release build off current main.🤖 Generated with Claude Code