Skip to content

fix(cli): group-level help — rome <group> / <group> --help - #33

Merged
skansal-rome merged 1 commit into
mainfrom
fix-group-level-help
Jul 24, 2026
Merged

fix(cli): group-level help — rome <group> / <group> --help#33
skansal-rome merged 1 commit into
mainfrom
fix-group-level-help

Conversation

@skansal-rome

Copy link
Copy Markdown
Contributor

rome cookbook --help (and rome facts, bare) printed Unknown command followed by the full command catalog — the group-level gap left by #24, which covered per-command --help only.

Now:

  • rome <group> / rome <group> --help|-h|help → that group's usage (its commands + summaries), exit 0.
  • rome <group> <unknown-sub>Unknown command with the help scoped to the group's commands instead of the full catalog, exit 1.
  • Anything else keeps today's behavior (full help on truly unknown commands).

Test-first: three new behavioral cases in test/cli.test.ts (group --help exits 0 without "Unknown command", bare group is a help request, unknown-subcommand error excludes unrelated commands) — 125/125 pass locally, build clean.

…known-subcommand errors

A bare group or 'rome cookbook --help' printed 'Unknown command' plus the
full catalog. Both are now help requests (exit 0) scoped to the group's
commands; an unknown subcommand errors with the group's usage instead of
the full help.
@skansal-rome
skansal-rome merged commit 4909f65 into main Jul 24, 2026
6 checks passed
@skansal-rome
skansal-rome deleted the fix-group-level-help branch July 24, 2026 05:29
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