feat(parse): support custom help and version actions - #1123
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Central YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
792f352 to
c068f02
Compare
c068f02 to
3189067
Compare
3189067 to
ceb9956
Compare
a966d16 to
23c2cb5
Compare
d713555 to
2c7bf90
Compare
Instruction counts
No instruction-count regression above 1%. Only instruction counts gate. Wall clock is shown for context — on identical hardware it moves 4-20% run to run. Measured by tak — instruction-counted CLI benchmarks, stored in this repository's git notes. Shadow comparisonParsing
|
e449097 to
1022515
Compare
1022515 to
a742df6
Compare
a742df6 to
3c7e217
Compare
4b1a6f3 to
b9c5e5e
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 4b1a6f3. Configure here.
b9c5e5e to
7cf0ea4
Compare
0c04ae1 to
12abcc0
Compare

Stack created with GitHub Stacks CLI • Give Feedback 💬
Note
Medium Risk
Touches core parse, help rendering, and completion walking in both Rust and Go, so regressions would change how help/version is requested. Behavior is additive and gated by explicit flags/actions.
Overview
CLIs can now disable the synthesized
--help/-h,helpsubcommand, and--version/-Ventries, and relocate those behaviors onto declared flags viaHelp,HelpShort,HelpLong, andVersionactions.The same properties travel through typed Rust, KDL, usage-lib, generated Go, and the clap bridge. Help pages omit the synthetic rows when disabled; custom flags keep their own help text. Completion walking no longer treats a declared help/version action as a help-topic parse, so later words still complete.
Disabling these built-ins is no longer a clap-bridge fidelity loss. Derive fields with a non-binding action must be
boolflags.Reviewed by Cursor Bugbot for commit 238871f. Bugbot is set up for automated code reviews on this repo. Configure here.