[Automated] Update gh CLI Options - #4948
Conversation
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. 🗂️ Base branches to auto review (1)
Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced 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 |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
Greptile SummaryThis PR updates only the gh generated-options provenance fingerprint. The tool version and command-tree fingerprint remain unchanged, and the new generator-source fingerprint correctly matches the PR base.
Confidence Score: 5/5The PR appears safe to merge because the sole metadata update is internally consistent with the generator freshness check. No actionable failures remain; the replacement fingerprint matches the deterministic hash of the configured generator-source paths at the PR base.
|
| Filename | Overview |
|---|---|
| src/ModularPipelines.GitHub/Generated/Gh.Generation.json | Updates the generator-source provenance hash to the value expected for the current base generator inputs. |
Reviews (1): Last reviewed commit: "chore: Update gh CLI options" | Re-trigger Greptile
Code reviewThis PR is fully automated: the only change is the I verified the update is expected rather than spurious: No issues found. Checked for bugs and CLAUDE.md compliance. |
1026a48
into
issue-4655-repeatable-continuation
Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com>
Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com>
Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com>
…ines (#4664) * fix(generator): keep repeatable lookahead on wrapped option-looking lines HelpDeclaresRepeatableOption stopped its block scan at the first line matching OptionLinePattern, so a wrapped description line that started with another option's switch hid a repeatable phrase split across it. Walk the block with the shared IsContinuationLine rule instead, keeping the blank-line and option-row bounds gcloud relies on. A null declarationIndentation now means "no indentation floor", and the inline-description column derivation is shared with AccumulateWrappedDescription via GetDescriptionColumn. Closes #4655 Claude-Session: https://claude.ai/code/session_01EHe9J8SumuZb2NBZxpGBzc * fix(generator): locate the prose column past value hints and tab separators The repeatable lookahead anchored its description column on the first run of two blanks, so a padded value hint or a second switch form became the column and a single tab was not a separator at all. Walk the row's segments instead: split on two or more blanks or one tab, skip switch segments and single-token value hints that are followed by more text, and take the first prose segment. Covers padded hints, shellcheck-style dual switch forms and tab-aligned help, with lookahead regressions for a nested option row at the hint column and for tab-separated blocks. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): parse a terminal value hint as a hint, not prose A row such as "--env stringArray" whose description only starts on the next line made the last segment count as prose, so the hint column became the continuation threshold and a nested option row at that column was absorbed. Classify every single-token segment by shape instead: one token that is not a sentence-case word is a value hint wherever it sits, while a one-word description such as "Verbose" stays prose. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): infer the lookahead column from the first wrapped line With #4658 an unknown description column accepts any deeper line, so the repeatable lookahead absorbed every following option row after a declaration without inline prose (argparse-style help) until the next blank line. Take the column from the first accepted wrapped line, as AccumulateWrappedDescription already does, and make the terminal-hint regression realistic: prose deep, nested option row shallow. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): treat every lone token in an option row as a value hint TitleCase metavariables such as String, Path or File slipped past the sentence-case rule and became the prose column. A single token can be a hint or a one-word description, but the difference only matters when something wraps beneath the row, and the lookahead now takes its column from that wrapped line, so classify every lone token as a hint and drop the casing heuristic. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): stop a descriptionless option's lookahead at a sibling row With no inline prose and no column yet, an option-looking line at the same indentation as the declaration was accepted as the first continuation, so the sibling's description was scanned as part of the original option. While the column is unknown, apply the declaration indentation as a floor to option-looking candidates only: siblings start the next option, wrapped switch text (indented deeper) still continues, and same-indent prose such as gcloud's repeatability notes is unaffected. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): keep the repeatable lookahead conservative while the column is unknown This lookahead does not know a tool's layout, so it cannot tell a one-word description from a value hint or a nested option row from wrapped switch text. Until the row's inline prose has fixed the description column, any option-looking line now ends the block (a sibling row, a nested row, or the neighbour of a one-word description alike) while plain prose beneath a descriptionless row still establishes the column. Wrapped prose that starts with a switch is still kept once the column is known, which is the case the issue is about. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): borrow the help layout's description column for descriptionless rows A row with no inline prose left the description column unknown, so any option-looking first continuation ended the repeatable lookahead. That kept sibling and nested rows out of the block but also dropped wrapped prose that happens to open with a switch mention, hiding a documented repeatable option. The lookahead now learns the column the help text lays descriptions out at (the most common inline prose column, or the indentation of next-line prose beneath descriptionless rows) and seeds a descriptionless row with it. Prose at that column stays in the block; rows at a shallower indentation still end it. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): treat a run of placeholder tokens as a value hint LooksLikeValueHint only recognised single-token hints, so a two-word hint such as "KEY VALUE" read as prose and GetInlineDescriptionColumn anchored the description column on the hint. A sibling row indented between the hint and the real prose column then passed as wrapped continuation and could donate its repeatability note to the wrong option. A segment whose every token is placeholder-shaped (uppercase, bracketed, or an ellipsis) is now a hint as well; prose has lowercase words. Refs #4655 Claude-Session: https://claude.ai/code/session_01PkLNTUfwGXjqXZrYrHaDGC * fix(generator): scope repeatable lookahead and reject nested value rows (#4655) * fix(generator): bound repeatability sections and nested flags (#4655) * fix(generator): bound nested option descriptions (#4655) * fix(generator): use prose context for wrapped switch references (#4655) * refactor(generator): separate nested declaration detection (#4655) * fix(generator): distinguish reference phrases from connectors (#4655) * fix(generator): distinguish short prose from hints and headings (#4655) * fix(generator): infer padded type hints from row structure (#4655) * chore: Update kustomize CLI options (#4890) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kind CLI options (#4889) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update yq CLI options (#4899) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update grype CLI options (#4897) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update nbgv CLI options (#4894) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update flux CLI options (#4896) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update docker CLI options (#4891) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update buildah CLI options (#4892) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update skopeo CLI options (#4898) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update syft CLI options (#4900) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update cosign CLI options (#4901) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * fix(generator): separate repeatable sections and bracketed defaults (#4655) * style(generator): satisfy scraper diagnostics before regeneration (#4655) * chore: Update skopeo CLI options (#4915) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update docker CLI options (#4916) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update cosign CLI options (#4917) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update buildah CLI options (#4918) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kind CLI options (#4919) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update minikube CLI options (#4904) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kustomize CLI options (#4920) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update yq CLI options (#4925) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update syft CLI options (#4924) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update nbgv CLI options (#4923) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update flux CLI options (#4921) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update grype CLI options (#4922) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update trivy CLI options (#4906) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update argocd CLI options (#4907) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update pulumi CLI options (#4905) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update podman CLI options (#4903) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update helm CLI options (#4902) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update eksctl CLI options (#4893) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kubectl CLI options (#4895) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update gh CLI options (#4927) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * fix(generator): scope repeatability to option declarations (#4655) * chore: Update yq CLI options (#4929) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update helm CLI options (#4931) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update docker CLI options (#4939) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update flux CLI options (#4941) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update pulumi CLI options (#4940) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kustomize CLI options (#4936) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update nbgv CLI options (#4930) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update minikube CLI options (#4932) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update syft CLI options (#4933) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update podman CLI options (#4942) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update trivy CLI options (#4943) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update argocd CLI options (#4944) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update skopeo CLI options (#4945) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kubectl CLI options (#4946) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update cosign CLI options (#4947) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update gh CLI options (#4948) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update grype CLI options (#4935) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update eksctl CLI options (#4934) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update kind CLI options (#4938) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * chore: Update buildah CLI options (#4937) Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com> * fix(generator): distinguish switch-led prose from declarations (#4655) * fix(generator): retain detached descriptions on nested options (#4655) * fix(generator): recognize acronym section headings (#4655) * fix(generator): share configured executor with type enhancement (#4655) * fix(generator): honor caller captures for detached descriptions (#4655) * test(generator): follow configured process registration (#4655) --------- Co-authored-by: thomhurst <9139608+thomhurst@users.noreply.github.com>
Summary
This PR contains automatically generated updates to gh CLI options classes.
The generator scraped the latest CLI help output from the installed tool.
Changes
Assembly-wide public API impact
No active public API changes were detected in this assembly.
Command coverage
Command coverage report:
Verification
🤖 Generated with ModularPipelines.OptionsGenerator