Skip to content

[AI-815] Surface --cursor/--copilot in kcap --help Plugin section#163

Merged
realtonyyoung merged 3 commits into
kurrent-io:mainfrom
realtonyyoung:tonyyoung/ai-815-plugin-help-vendors
Jun 17, 2026
Merged

[AI-815] Surface --cursor/--copilot in kcap --help Plugin section#163
realtonyyoung merged 3 commits into
kurrent-io:mainfrom
realtonyyoung:tonyyoung/ai-815-plugin-help-vendors

Conversation

@realtonyyoung

Copy link
Copy Markdown
Contributor

Problem

kcap --help (top-level usage) listed only [--codex] under Plugin, so a user reading the main help couldn't discover plugin install --copilot or --cursor — even though both have shipped (AI-731 Cursor, AI-815 Copilot) and kcap plugin --help documents them in full. Reported from the field: "looked at kcap --help and didn't see anything for Copilot."

The import line and the Hooks section 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):

Plugin:
  plugin install [--project] [--codex|--cursor|--copilot|--skills]   Register hooks/skills (Claude default; flag picks the vendor)
  plugin remove  [--project] [--codex|--cursor|--copilot|--skills]   Remove hooks/skills (Claude default; flag picks the vendor)

Docs-only (embedded help resource). Verified by running kcap --help from a release build off current main.

🤖 Generated with Claude Code

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-code-review

Copy link
Copy Markdown

Qodo reviews are paused for this user.

Troubleshooting steps vary by plan Learn more →

On a Teams plan?
Reviews resume once this user has a paid seat and their Git account is linked in Qodo.
Link Git account →

Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center?
These require an Enterprise plan - Contact us
Contact us →

@realtonyyoung realtonyyoung left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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>
@realtonyyoung

Copy link
Copy Markdown
Contributor Author

Fixed in 3642781PluginCommand.PrintUsage() now emits [--codex|--cursor|--copilot|--skills], matching the top-level help and the actual dispatch. Verified by running kcap plugin <bad-args> from a release build; AOT publish clean (0 IL warnings).

@realtonyyoung realtonyyoung left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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

Copy link
Copy Markdown
Contributor Author

Both review nits addressed:

  • PluginCommand.PrintUsage() fallback now includes --copilot (3642781).
  • README uninstall per-agent cleanup line now lists --copilot (1f115be).

So the vendor list is consistent across the top-level help, the plugin fallback usage, and the README.

@realtonyyoung realtonyyoung left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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.

@realtonyyoung realtonyyoung merged commit e15ca4a into kurrent-io:main Jun 17, 2026
5 checks passed
realtonyyoung added a commit that referenced this pull request Jun 18, 2026
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>
realtonyyoung added a commit that referenced this pull request Jun 18, 2026
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>
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.

1 participant