[Automated] Update snyk CLI Options - #5352
Conversation
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (9)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ 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. |
There was a problem hiding this comment.
Claude Review
This is an automated regeneration PR for the Snyk CLI options package. All 8 enum "Allowed values for X" doc-comment summaries gained a trailing period, and Snyk.Generation.json's generatorSourceSha256 was bumped to reflect the newer generator source. No CLI commands, option types, or command tree changed (toolVersion and commandTreeSha256 are unchanged).
Review evidence
src/ModularPipelines.Snyk/Enums/SnykFailOn.Generated.cs: Summary doc comment now reads 'Allowed values for --fail-on.' with a trailing period; verified this matches the current generator template literal and an existing generator unit test, confirming the regeneration is a faithful, non-divergent output of the current tool source rather than an accidental or manual edit.
src/ModularPipelines.Snyk/Generated/Snyk.Generation.json: Only generatorSourceSha256 changed (37c8e0e0...9a6998b -> 755df85a...9132441d6); toolVersion (1.1306.1) and commandTreeSha256 are unchanged, confirming this PR is a pure generator-formatting regeneration with no CLI surface or command-tree changes.
No actionable findings.
Optional follow-up notes
The bumped generatorSourceSha256 was verified against current generator source: tools/ModularPipelines.OptionsGenerator/src/ModularPipelines.OptionsGenerator/Generators/OptionEnumFactory.cs:60 already emits $"Allowed values for {switchName}." with a trailing period, and CobraCliScraperTests.cs:44 asserts this exact format, so the regenerated output correctly matches the current generator rather than diverging from it.
No prior review findings are outstanding: CodeRabbit explicitly skipped review of these files due to its default path filters for generated/.generated. paths, and the Codex review shown in context was still 'Running' with no comments posted yet, so there is nothing unaddressed to reconcile.
|
Snyk acceptance for #5009: this snapshot was generated by https://github.com/thomhurst/ModularPipelines/actions/runs/35725645483 from main ae2f20a after #5330 and #5350 merged. The complete diff contains eight generated enum summary punctuation changes from the shared factory and the provenance fingerprint update. CLI value spellings, enum members, options, services, and the 18-command coverage manifest remain unchanged. No generated files or API baselines were manually edited. The guarded Release/net10.0 Snyk executable test project passes all 12 tests locally. The workflow solution build passed; PR integration CI and reviews remain merge gates. #5009 stays open until Google acceptance also completes. |
Summary
This PR contains automatically generated updates to snyk CLI options classes.
The generator scraped the latest CLI help output from the installed tool.
Changes
Command coverage
Command coverage report:
Verification
🤖 Generated with ModularPipelines.OptionsGenerator