Skip to content

[Automated] Update nbgv CLI Options - #4923

Merged
thomhurst merged 1 commit into
issue-4655-repeatable-continuationfrom
automated/update-cli-options-nbgv-ref-dd6d9693f925b4978d242ca6b0b284e0e51f06d3fc71bf0da37d7201e0738e7a
Sep 14, 2026
Merged

thomhurst merged 1 commit into
issue-4655-repeatable-continuationfrom
automated/update-cli-options-nbgv-ref-dd6d9693f925b4978d242ca6b0b284e0e51f06d3fc71bf0da37d7201e0738e7a

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

This PR contains automatically generated updates to nbgv CLI options classes.

The generator scraped the latest CLI help output from the installed tool.

Changes

  • Updated options classes to reflect latest CLI documentation
  • Added new commands if any were detected
  • Updated option types and descriptions

Assembly-wide public API impact

No active public API changes were detected in this assembly.

Command coverage

Command coverage report:

  • nbgv (3.10.91+e05abbcae4): 7 commands, tree 7a9c8d889f1ee1248bbdd066caabfaaf53477dea0c81e97ea9f405b33bda6b63
    • Baseline comparison: 7 commands at 3.10.91+e05abbcae4 -> 7 commands at 3.10.91+e05abbcae4

Verification

  • Solution builds successfully

🤖 Generated with ModularPipelines.OptionsGenerator

@thomhurst thomhurst added automated dependencies Pull requests that update a dependency file labels Sep 14, 2026
@coderabbitai

coderabbitai Bot commented Sep 14, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (1)
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 93131c9b-e726-42fc-8b7a-b58867c211a5

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

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.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 14, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-09-14T03:54:40.542719Z b957bb6 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@greptile-apps

greptile-apps Bot commented Sep 14, 2026

Copy link
Copy Markdown

Greptile Summary

This PR refreshes the nbgv generation provenance after generator-source changes.

  • Updates only generatorSourceSha256.
  • Leaves the tool version and command-tree fingerprint unchanged.
  • Does not alter generated command options or public APIs.

Confidence Score: 5/5

The PR appears safe to merge because the metadata-only checksum update matches the repository’s generation-provenance design.

The source fingerprint is intentionally independent of generated output and the command-tree checksum, and no actionable failures or rule violations remain.

Important Files Changed

Filename Overview
src/ModularPipelines.NerdbankGitVersioning/Generated/Nbgv.Generation.json Updates the generator-source provenance checksum consistently with changes to fingerprinted generator sources.

Reviews (1): Last reviewed commit: "chore: Update nbgv CLI options" | Re-trigger Greptile

@github-actions

Copy link
Copy Markdown
Contributor

Review: [Automated] Update nbgv CLI Options (#4923)

This is a bot-generated provenance refresh, not a hand-written change. The diff touches a single file and a single field:

```diff

  • "generatorSourceSha256": "baaf09c947b655eca7003d9a905d7255e52dcfe8be5dc849bfb13af636805a4e"
  • "generatorSourceSha256": "0e7b8148ac358e31f6466ef6ad8bc639988519d3411edfb27a394b2bda1bdc16"
    ```

Verification performed:

  • Confirmed generatorSourceSha256 is written by scripts/Write-GeneratedOptionsProvenance.ps1 via Get-GeneratedOptionsSourceFingerprint, and is a hash over the generator's own source, independent of toolVersion/commandTreeSha256. Since the base branch (issue-4655-repeatable-continuation) recently modified the generator's help-timeout handling (Report exhausted help timeouts instead of phantom removed commands #4688), a fingerprint bump with no commandTreeSha256 or command-option changes is exactly the expected output — not a hand-edit and not a sign of drift.
  • toolVersion and commandTreeSha256 are unchanged, and no generated .cs option classes changed, consistent with the PR description's "No active public API changes were detected."
  • Per CLAUDE.md, generated options/provenance files must come from the generator, never hand-edited — this file is fully machine-written and requires no manual reconciliation.

No functional, design, or architectural concerns — there's no hand-written logic in this diff to critique. Safe to merge as-is.

@thomhurst
thomhurst merged commit 21ee578 into issue-4655-repeatable-continuation Sep 14, 2026
11 checks passed
@thomhurst
thomhurst deleted the automated/update-cli-options-nbgv-ref-dd6d9693f925b4978d242ca6b0b284e0e51f06d3fc71bf0da37d7201e0738e7a branch September 14, 2026 04:27
thomhurst added a commit that referenced this pull request Sep 14, 2026
thomhurst added a commit that referenced this pull request Sep 14, 2026
thomhurst added a commit that referenced this pull request Sep 14, 2026
thomhurst added a commit that referenced this pull request Sep 14, 2026
thomhurst added a commit that referenced this pull request Sep 14, 2026
…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 <[email protected]>

* chore: Update kind CLI options (#4889)

Co-authored-by: thomhurst <[email protected]>

* chore: Update yq CLI options (#4899)

Co-authored-by: thomhurst <[email protected]>

* chore: Update grype CLI options (#4897)

Co-authored-by: thomhurst <[email protected]>

* chore: Update nbgv CLI options (#4894)

Co-authored-by: thomhurst <[email protected]>

* chore: Update flux CLI options (#4896)

Co-authored-by: thomhurst <[email protected]>

* chore: Update docker CLI options (#4891)

Co-authored-by: thomhurst <[email protected]>

* chore: Update buildah CLI options (#4892)

Co-authored-by: thomhurst <[email protected]>

* chore: Update skopeo CLI options (#4898)

Co-authored-by: thomhurst <[email protected]>

* chore: Update syft CLI options (#4900)

Co-authored-by: thomhurst <[email protected]>

* chore: Update cosign CLI options (#4901)

Co-authored-by: thomhurst <[email protected]>

* 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 <[email protected]>

* chore: Update docker CLI options (#4916)

Co-authored-by: thomhurst <[email protected]>

* chore: Update cosign CLI options (#4917)

Co-authored-by: thomhurst <[email protected]>

* chore: Update buildah CLI options (#4918)

Co-authored-by: thomhurst <[email protected]>

* chore: Update kind CLI options (#4919)

Co-authored-by: thomhurst <[email protected]>

* chore: Update minikube CLI options (#4904)

Co-authored-by: thomhurst <[email protected]>

* chore: Update kustomize CLI options (#4920)

Co-authored-by: thomhurst <[email protected]>

* chore: Update yq CLI options (#4925)

Co-authored-by: thomhurst <[email protected]>

* chore: Update syft CLI options (#4924)

Co-authored-by: thomhurst <[email protected]>

* chore: Update nbgv CLI options (#4923)

Co-authored-by: thomhurst <[email protected]>

* chore: Update flux CLI options (#4921)

Co-authored-by: thomhurst <[email protected]>

* chore: Update grype CLI options (#4922)

Co-authored-by: thomhurst <[email protected]>

* chore: Update trivy CLI options (#4906)

Co-authored-by: thomhurst <[email protected]>

* chore: Update argocd CLI options (#4907)

Co-authored-by: thomhurst <[email protected]>

* chore: Update pulumi CLI options (#4905)

Co-authored-by: thomhurst <[email protected]>

* chore: Update podman CLI options (#4903)

Co-authored-by: thomhurst <[email protected]>

* chore: Update helm CLI options (#4902)

Co-authored-by: thomhurst <[email protected]>

* chore: Update eksctl CLI options (#4893)

Co-authored-by: thomhurst <[email protected]>

* chore: Update kubectl CLI options (#4895)

Co-authored-by: thomhurst <[email protected]>

* chore: Update gh CLI options (#4927)

Co-authored-by: thomhurst <[email protected]>

* fix(generator): scope repeatability to option declarations (#4655)

* chore: Update yq CLI options (#4929)

Co-authored-by: thomhurst <[email protected]>

* chore: Update helm CLI options (#4931)

Co-authored-by: thomhurst <[email protected]>

* chore: Update docker CLI options (#4939)

Co-authored-by: thomhurst <[email protected]>

* chore: Update flux CLI options (#4941)

Co-authored-by: thomhurst <[email protected]>

* chore: Update pulumi CLI options (#4940)

Co-authored-by: thomhurst <[email protected]>

* chore: Update kustomize CLI options (#4936)

Co-authored-by: thomhurst <[email protected]>

* chore: Update nbgv CLI options (#4930)

Co-authored-by: thomhurst <[email protected]>

* chore: Update minikube CLI options (#4932)

Co-authored-by: thomhurst <[email protected]>

* chore: Update syft CLI options (#4933)

Co-authored-by: thomhurst <[email protected]>

* chore: Update podman CLI options (#4942)

Co-authored-by: thomhurst <[email protected]>

* chore: Update trivy CLI options (#4943)

Co-authored-by: thomhurst <[email protected]>

* chore: Update argocd CLI options (#4944)

Co-authored-by: thomhurst <[email protected]>

* chore: Update skopeo CLI options (#4945)

Co-authored-by: thomhurst <[email protected]>

* chore: Update kubectl CLI options (#4946)

Co-authored-by: thomhurst <[email protected]>

* chore: Update cosign CLI options (#4947)

Co-authored-by: thomhurst <[email protected]>

* chore: Update gh CLI options (#4948)

Co-authored-by: thomhurst <[email protected]>

* chore: Update grype CLI options (#4935)

Co-authored-by: thomhurst <[email protected]>

* chore: Update eksctl CLI options (#4934)

Co-authored-by: thomhurst <[email protected]>

* chore: Update kind CLI options (#4938)

Co-authored-by: thomhurst <[email protected]>

* chore: Update buildah CLI options (#4937)

Co-authored-by: thomhurst <[email protected]>

* 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 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automated dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant