Skip to content

[Automated] Update helm CLI Options - #4614

Open
thomhurst wants to merge 1 commit into
mainfrom
automated/update-cli-options-helm
Open

[Automated] Update helm CLI Options#4614
thomhurst wants to merge 1 commit into
mainfrom
automated/update-cli-options-helm

Conversation

@thomhurst

@thomhurst thomhurst commented Sep 3, 2026

Copy link
Copy Markdown
Owner

Summary

This PR contains automatically generated updates to helm 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

Affected API families: Helm.

  • Added APIs: 22
  • Removed or changed APIs: 22
  • Members with matching names but changed signatures: 0

Breaking changes are present. Consumers may need to update method arguments, option property types or nullability, enum members, and references to removed APIs.

Representative removed or changed members:

  • ModularPipelines.Helm.Enums.HelmGetMetadataOutput.Json = 1 -> ModularPipelines.Helm.Enums.HelmGetMetadataOutput
  • ModularPipelines.Helm.Enums.HelmGetMetadataOutput.Table = 0 -> ModularPipelines.Helm.Enums.HelmGetMetadataOutput
  • ModularPipelines.Helm.Enums.HelmGetValuesOutput.Json = 1 -> ModularPipelines.Helm.Enums.HelmGetValuesOutput
  • ModularPipelines.Helm.Enums.HelmGetValuesOutput.Table = 0 -> ModularPipelines.Helm.Enums.HelmGetValuesOutput
  • ModularPipelines.Helm.Enums.HelmHistoryOutput.Json = 1 -> ModularPipelines.Helm.Enums.HelmHistoryOutput

Representative added members:

  • ModularPipelines.Helm.Enums.HelmGetMetadataOutput.Json = 0 -> ModularPipelines.Helm.Enums.HelmGetMetadataOutput
  • ModularPipelines.Helm.Enums.HelmGetMetadataOutput.Table = 1 -> ModularPipelines.Helm.Enums.HelmGetMetadataOutput
  • ModularPipelines.Helm.Enums.HelmGetValuesOutput.Json = 0 -> ModularPipelines.Helm.Enums.HelmGetValuesOutput
  • ModularPipelines.Helm.Enums.HelmGetValuesOutput.Table = 1 -> ModularPipelines.Helm.Enums.HelmGetValuesOutput
  • ModularPipelines.Helm.Enums.HelmHistoryOutput.Json = 0 -> ModularPipelines.Helm.Enums.HelmHistoryOutput

Command coverage

Command coverage report:

  • helm (version.BuildInfo{Version:"v3.21.4", GitCommit:"813176c51bb5c181dbbd7901298ddcc104cd3417", GitTreeState:"clean", GoVersion:"go1.26.5"}): 50 commands, tree 4899ed49663bc26b485e79579fc4dd8dc6db81e8115ef7343e12699488811782
    • Baseline comparison: 50 commands at version.BuildInfo{Version:"v3.21.4", GitCommit:"813176c51bb5c181dbbd7901298ddcc104cd3417", GitTreeState:"clean", GoVersion:"go1.26.5"} -> 50 commands at version.BuildInfo{Version:"v3.21.4", GitCommit:"813176c51bb5c181dbbd7901298ddcc104cd3417", GitTreeState:"clean", GoVersion:"go1.26.5"}

Verification

  • Solution builds successfully

🤖 Generated with ModularPipelines.OptionsGenerator

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

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Team

Run ID: 9a46a6d2-6b42-4e83-9457-38ddd8a81fd5

📥 Commits

Reviewing files that changed from the base of the PR and between 8abd675 and d1fbf6b.

⛔ Files ignored due to path filters (12)
  • src/ModularPipelines.Helm/Enums/HelmGetMetadataOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmGetValuesOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmHistoryOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmInstallOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmListOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmRepoListOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmSearchHubOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmSearchRepoOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmStatusOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmUninstallCascade.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Enums/HelmUpgradeOutput.Generated.cs is excluded by !**/*.generated.*
  • src/ModularPipelines.Helm/Generated/Helm.Generation.json is excluded by !**/generated/**
📒 Files selected for processing (2)
  • src/ModularPipelines.Helm/PublicAPI.Shipped.txt
  • src/ModularPipelines.Helm/PublicAPI.Unshipped.txt
💤 Files with no reviewable changes (1)
  • src/ModularPipelines.Helm/PublicAPI.Shipped.txt

Included review availability: Your plan provides up to 4 included reviews per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

The Helm public API removes unsupported shipped enum members and updates unshipped numeric mappings for output and uninstall cascade enums.

Changes

Helm public API

Layer / File(s) Summary
Update Helm enum contracts
src/ModularPipelines.Helm/PublicAPI.Shipped.txt, src/ModularPipelines.Helm/PublicAPI.Unshipped.txt
Shipped output enums retain Yaml, and HelmUninstallCascade retains Background. Unshipped declarations define the updated numeric mappings for Helm output and uninstall cascade enums.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Merge Risk: ⚪ Minimal · up to d1fbf

This updates Helm enum API declarations and numeric mappings to match the generated CLI option contracts. No actionable current-head merge risk remains.

Poem

A rabbit checks the Helm enum line
YAML remains, precise and fine
JSON and tables hop away
Background guards the uninstall day
New numeric values point the way

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly describes the main change: updating Helm CLI options. The title is concise and relevant to the pull request objectives.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch automated/update-cli-options-helm

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.

@thomhurst
thomhurst force-pushed the automated/update-cli-options-helm branch 2 times, most recently from fc20e14 to d1fbf6b Compare September 6, 2026 07:36
@github-actions

github-actions Bot commented Sep 6, 2026

Copy link
Copy Markdown
Contributor

Code review

This PR is a fully auto-generated update from ModularPipelines.OptionsGenerator, reordering the [EnumValue] members of several Helm output enums (HelmGetMetadataOutput, HelmGetValuesOutput, HelmHistoryOutput, HelmInstallOutput, HelmListOutput, HelmRepoListOutput, HelmSearchHubOutput, HelmSearchRepoOutput, HelmStatusOutput, HelmUpgradeOutput, HelmUninstallCascade) to match the latest scraped helm CLI output, plus the corresponding PublicAPI.Shipped.txt/PublicAPI.Unshipped.txt and Helm.Generation.json updates.

Observations:

  • All changes are confined to *.Generated.cs files and generated metadata — no hand-edits to generated output, consistent with this repo's rule that generated-options files must not be hand-tweaked (CLAUDE.md).
  • The reordering does change enum ordinal values (e.g. Json/Table swap from 1/0 to 0/1, and HelmUninstallCascade.Foreground/Orphan swap from 2/1 to 1/2). This is a genuine breaking change for any consumer relying on the underlying integer value, but it's explicitly called out in the PR description's "Assembly-wide public API impact" section and is expected behavior per this repo's policy that "current tool output is the sole source of truth for generated options, even when regeneration causes breaking API changes."
  • The PublicAPI.Unshipped.txt/Shipped.txt diffs are consistent with the enum changes (each swapped member appears once as *REMOVED* and once as newly added), so the public API tracking is correctly in sync.

No actionable issues found — this is a clean, mechanical regeneration.

@thomhurst
thomhurst force-pushed the automated/update-cli-options-helm branch from d1fbf6b to b3df26f Compare September 6, 2026 08:40
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