Skip to content

Disable help flag on passthrough commands#2

Merged
andersonleal merged 1 commit intomainfrom
fix/help-command
Feb 27, 2026
Merged

Disable help flag on passthrough commands#2
andersonleal merged 1 commit intomainfrom
fix/help-command

Conversation

@andersonleal
Copy link
Contributor

@andersonleal andersonleal commented Feb 27, 2026

Summary

  • Disable Clap's automatic -h/--help handling for passthrough subcommands (console, create, motia, start).
  • Prevent the wrapper CLI from intercepting help flags that should be forwarded to downstream binaries.
  • Keep existing passthrough behavior (trailing_var_arg + allow_hyphen_values) while improving command compatibility.

Test plan

  • Review git diff main...HEAD to confirm only src/cli.rs changed.
  • Run iii-cli console --help and verify help is provided by downstream target.
  • Run iii-cli create --help and verify passthrough behavior remains intact.

Summary by CodeRabbit

  • Chores
    • Adjusted help flag handling behavior for Console, Create, Motia, and Start subcommands.

… Create, Motia, and Start commands to streamline user experience.
@coderabbitai
Copy link

coderabbitai bot commented Feb 27, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b9a5426 and dbd3b09.

📒 Files selected for processing (1)
  • src/cli.rs

📝 Walkthrough

Walkthrough

Four Clap subcommand attributes in the CLI configuration are updated to include disable_help_flag = true, preventing automatic help flag handling for Console, Create, Motia, and Start subcommands. No logic or signature changes are introduced.

Changes

Cohort / File(s) Summary
CLI Subcommand Configuration
src/cli.rs
Added disable_help_flag = true attribute to four subcommands (Console, Create, Motia, Start) to disable automatic help flag handling for these variants.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Poem

🐰 A rabbit hops through CLI land,
Four subcommands now take a stand,
Help flags silenced with a simple tune,
Custom behavior—done real soon!
hop-hop-hop

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Disable help flag on passthrough commands' accurately and concisely describes the main change—adding disable_help_flag = true to Clap subcommand attributes for passthrough commands.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/help-command

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@andersonleal andersonleal merged commit 0a30a24 into main Feb 27, 2026
1 check passed
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