[Automated] Update dotnet CLI Options - #4648
Conversation
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. |
|
Important Review skippedReview was skipped as selected files did not have any reviewable changes. 💤 Files selected but had no reviewable changes (2)
⛔ Files ignored due to path filters (1)
⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
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 |
Code ReviewThis PR is the automated dotnet CLI options regeneration bot run — it touches only I spot-checked a broad sample of the ~1350 added / ~1200 removed PublicAPI entries (e.g. Additional checks: no duplicate entries in the new No actionable issues found. |
202fe2a to
82cf014
Compare
ReviewThis is a fully automated, mechanical PR — it touches only Two things worth flagging:
Neither point is a functional bug in this specific diff, but both affect how trustworthy/reviewable these automated PRs are over time, so I'd like to see them addressed (at least the reordering, since it directly hides real changes in every future auto-PR of this kind). |
82cf014 to
2bfdb75
Compare
|
Review This is the automated dotnet CLI options regeneration bot run. It touches only I spot-checked a sample of the added/removed
Following up on the prior automated review's BLOCKING finding (reordering churn) The earlier review flagged the That means the reordering in this diff is a one-time correction of entries in The PR description's boilerplate text ("scraped the latest CLI help output... added new commands if any were detected") is still generic auto-generated wording that doesn't reflect that this particular run only reconciled the tracking files with already-shipped source — same observation the earlier review made. This is cosmetic (PR body text, not code) and not something worth blocking on. No actionable code issues found in the current head. |
6e8eaa0 to
f0bfbe4
Compare
ReviewI verified the generator-provenance freshness check for this automated snapshot update, since that's the only substantive risk in an auto-generated options PR like this. Blocking: generated snapshot is stale against its own base branch
I recomputed That means Suggested fix: re-run the "Generate CLI Options" workflow for dotnet from current No other issues found; the generated options diff itself (added/removed/changed members) is consistent with the described upstream |
f0bfbe4 to
b3c7bf4
Compare
Review: PR #4648 — chore: Update dotnet CLI optionsThis is a fully automated, bot-generated regeneration of
No hand-written or generated Verification performed
ConclusionPer this repo's CLAUDE.md, this kind of breaking, tool-driven regeneration is the explicitly intended workflow ("Current tool output is the sole source of truth for generated options, even when regeneration causes breaking API changes"). The breaking removals (e.g. No actionable findings. |
2667932 to
e681342
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: e681342e2d
ℹ️ 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".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
e681342 to
702bbce
Compare
702bbce to
6f31eca
Compare
ReviewThis is the automated Verification performed:
Per this repo's CLAUDE.md, breaking, tool-driven regeneration is the explicitly intended workflow here ("current tool output is the sole source of truth for generated options, even when regeneration causes breaking API changes"), and the removed/changed members (e.g. No actionable issues found in the current head. |
6f31eca to
ff96b27
Compare
ReviewThis is the automated Verification performed on the current head:
Per this repo's CLAUDE.md, breaking, tool-driven regeneration is the explicitly intended workflow ('current tool output is the sole source of truth for generated options, even when regeneration causes breaking API changes'), and this run's removed/changed members ( No actionable issues found in the current head. |
ff96b27 to
1fa25a8
Compare
Review: PR #4648 — automated dotnet CLI options regenerationThis is a fully automated, bot-generated regeneration of
No hand-authored or Previously flagged blocking issue is now resolvedAn earlier review on this PR (2026-09-06 01:32 UTC) flagged that I also confirmed Spot-checked content correctnessSampled a handful of the larger PublicAPI churn clusters and they all read as genuine generator/data-quality fixes reflecting the real
Breaking changes are expected and acceptable here per this repo's convention (current CLI output is the source of truth; no compatibility shims for generated options). VerdictNo actionable issues remain. The one substantive risk for this kind of PR (a stale provenance stamp silently blocking merge) has been fixed and I verified the fix independently rather than just trusting the stamp. |
1fa25a8 to
9d96420
Compare
ReviewThis PR is a fully automated regeneration of the DotNet tool's options, touching only:
No hand-written Verification performed:
No correctness, design, or architectural issues found. Nothing here warrants blocking. |
|
Latest-options audit confirmed the baseline lifecycle problem is tracked in #4638 / #4644. The independent freshness-gate gap is now explicitly tracked in #4740. Current provenance matches main (773b4f943140dab43ee649323893d06346286caa0635dcf177d0bfb536a63497). Under the requested merge-latest workflow, these generator/workflow fixes remain follow-ups; no handwritten baseline or generated-options patch. Resolving review threads as tracked follow-ups, not as fixes in this generated snapshot. |
Summary
This PR contains automatically generated updates to dotnet CLI options classes.
The generator scraped the latest CLI help output from the installed tool.
Changes
Assembly-wide public API impact
Affected API families:
Assembly/common,DotNet.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.DotNet.Options.DotNetBuildOptions.Debug.get -> bool?ModularPipelines.DotNet.Options.DotNetBuildOptions.Debug.set -> voidModularPipelines.DotNet.Options.DotNetBuildOptions.Nologo.get -> bool?ModularPipelines.DotNet.Options.DotNetBuildOptions.Nologo.set -> voidModularPipelines.DotNet.Options.DotNetCleanOptions.Nologo.get -> bool?Representative added members:
ModularPipelines.DotNet.Options.DotNetNewDetailsOptions.Deconstruct(out string! PackageIdentifier) -> voidModularPipelines.DotNet.Options.DotNetNewDetailsOptions.DotNetNewDetailsOptions(string! PackageIdentifier) -> voidModularPipelines.DotNet.Options.DotNetNewDetailsOptions.PackageIdentifier.get -> string!ModularPipelines.DotNet.Options.DotNetNewDetailsOptions.PackageIdentifier.init -> voidModularPipelines.DotNet.Options.DotNetNewInstallOptions.Deconstruct(out System.Collections.Generic.IEnumerable<string!>! Package) -> voidCommand coverage
Command coverage report:
Verification
🤖 Generated with ModularPipelines.OptionsGenerator