Skip to content

[Automated] Update pnpm CLI Options - #5013

Closed
thomhurst wants to merge 1 commit into
issue-4655-repeatable-continuationfrom
automated/update-cli-options-pnpm-ref-dd6d9693f925b4978d242ca6b0b284e0e51f06d3fc71bf0da37d7201e0738e7a
Closed

thomhurst wants to merge 1 commit into
issue-4655-repeatable-continuationfrom
automated/update-cli-options-pnpm-ref-dd6d9693f925b4978d242ca6b0b284e0e51f06d3fc71bf0da37d7201e0738e7a

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

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

  • pnpm (12.4.1): 103 commands, tree ed0aa4ad19ed65abb387f2af96478f8f91f9e15f197104835663fac5ba17c2d1
    • Baseline comparison: 103 commands at 12.4.1 -> 103 commands at 12.4.1
    • Excluded: pnpm audit signatures (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.), pnpm stage approve (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.), pnpm stage download (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.), pnpm stage list (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.), pnpm stage publish (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.), pnpm stage reject (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.), pnpm stage view (pnpm 12.4.1 no longer lists this path in a Commands section or exposes a distinct child synopsis. The parent command documents [PARAMS]... instead; use its generated Params operand.)

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: e6aa5181-5ef5-4dd7-8821-ae3dcc53a109

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-14T11:35:52.840212Z 5f4cebf New commits
ℹ️ 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

The current PR updates the pnpm generation provenance checksum while retaining pnpm 12.4.1 and the existing command-tree fingerprint. Changes since the previous review also introduced configurable generator command deadlines and scraper reliability fixes.

  • Updates pnpm generator-source provenance metadata without changing command coverage.
  • Adds configurable per-command deadlines, including a longer AWS workflow deadline.
  • Serializes Terraform stacks help calls and improves help-section parsing.
  • The configurable deadline is not propagated to HTML type-enhancement commands.

Confidence Score: 4/5

The PR is not yet safe to merge because the new command-timeout contract is bypassed during HTML type enhancement, allowing requested longer deadlines to be ignored and generated types to silently lose accuracy.

The configured timeout reaches normal scraper commands, but the enabled-by-default HTML type-enhancement path constructs its own executor with the old 30-second deadline and converts enhancement failures into fallback output. This leaves a concrete generator-correctness issue despite the current pnpm metadata-only diff being internally consistent.

Files Needing Attention: tools/ModularPipelines.OptionsGenerator/src/ModularPipelines.OptionsGenerator/OptionsGeneratorCommand.cs, tools/ModularPipelines.OptionsGenerator/src/ModularPipelines.OptionsGenerator/TypeDetection/OptionTypeEnhancer.cs

Important Files Changed

Filename Overview
src/ModularPipelines.Node/Generated/Pnpm.Generation.json Updates only the generator-source checksum while retaining the pnpm version and command-tree identity.
tools/ModularPipelines.OptionsGenerator/src/ModularPipelines.OptionsGenerator/OptionsGeneratorCommand.cs Adds command-timeout configuration, but the configured value is not propagated to the optional type-enhancement executor.
tools/ModularPipelines.OptionsGenerator/src/ModularPipelines.OptionsGenerator/TypeDetection/OptionTypeEnhancer.cs Its default factory creates an independent command executor that retains the fixed 30-second timeout.

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

@github-actions

Copy link
Copy Markdown
Contributor

Review

This PR updates a single value in src/ModularPipelines.Node/Generated/Pnpm.Generation.json: the generatorSourceSha256 provenance fingerprint, changed from a64e31201c7dc54dd2d30167e4060f2b00e65d6e1d51caccccb85b03f1c8df08 to 8cfabb81bf2ce63e8a5e77311e3c41400b9e8a18a86bb06c5143193e6fd850f4.

Verification:

  • This is a generated provenance file. Per repo conventions, generated options/provenance files must reflect current generator output and should never be hand-edited — this PR does exactly that (regeneration, not a hand edit).
  • The hash is a fingerprint of the generator source tree, computed by Get-GeneratedOptionsSourceFingerprint and validated by scripts/Assert-GeneratedOptionsFreshness.ps1. It correctly changes here because the immediately preceding commit (45b3ba1e, "fix(generator): parse pnpm 12 clap-style help (fix(generator): parse pnpm 12 clap-style help #4713)") modified the generator source itself, so a new fingerprint is expected and required to keep the freshness check passing.
  • The new value is a well-formed 64-character lowercase hex string, matching the format the freshness script enforces.
  • No other file in the repo references the old hash, so there's no stale cross-reference left behind.
  • The PR description confirms no active public API changes and identical command coverage (103 commands) before/after — consistent with this being a pure provenance/fingerprint refresh with no behavioral change.

No actionable findings. This is a clean, expected, mechanical follow-up to the generator fix in #4713.

@thomhurst
thomhurst force-pushed the automated/update-cli-options-pnpm-ref-dd6d9693f925b4978d242ca6b0b284e0e51f06d3fc71bf0da37d7201e0738e7a branch from 12e54af to 5f4cebf Compare September 14, 2026 11:33
@thomhurst thomhurst closed this Sep 14, 2026
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