Skip to content

Keep leaf command paths discoverable in hierarchical help - #10

Merged
gersham merged 1 commit into
mainfrom
cursor/root-help-leaf-inventory-6ccc
Sep 10, 2026
Merged

gersham merged 1 commit into
mainfrom
cursor/root-help-leaf-inventory-6ccc

Conversation

@gersham

@gersham gersham commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Problem

PR #7 switched the CLI to Commander hierarchical help. Root screenrig --help now lists only top-level groups (media, account, agent, …), so the plugin skill-vs-binary check (scripts/test-skill-commands.py) no longer finds taught two-word command strings such as media generate, account show, and agent enroll.

Change

Keep Commander’s hierarchical immediate-child listing. For root and group human help, append an All commands inventory of descendant leaf paths (canonical names plus aliases) walked from the live command tree. Do not hardcode the list.

JSON --json --help HelpDocument stays hierarchical: commands remains immediate children; usage is the same human text, including the inventory.

Leaf --help is unchanged (no inventory). Group help stays focused — screenrig media --help still lists media children, and its inventory is only media leaves.

Tests

  • Root Commander section stays compact; every live leaf path appears in root usage.
  • Media group help still focuses on media and includes media generate etc.
  • Human screenrig --help stdout includes every COMMAND_SPECS path (the plugin substring check).
Open in Web Open in Cursor 

Root and group human --help still use Commander for immediate children,
then list descendant leaves from the live command tree so agents and the
plugin skill check can find two-word forms like `media generate`.

Co-authored-by: Gersham Meharg <gersham@users.noreply.github.com>
@gersham
gersham marked this pull request as ready for review September 10, 2026 20:49
@gersham
gersham merged commit 6b85d88 into main Sep 10, 2026
6 checks passed
@gersham
gersham deleted the cursor/root-help-leaf-inventory-6ccc branch September 10, 2026 20:49
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