chore(ci): bump pnpm/action-setup from 4.3.0 to 6.0.9#408
chore(ci): bump pnpm/action-setup from 4.3.0 to 6.0.9#408dependabot[bot] wants to merge 1 commit into
Conversation
LabelsThe following labels could not be found: Please fix the above issues or remove invalid values from |
|
@dependabot rebase Rebasing to pick up the current CI (a fresh |
Bumps [pnpm/action-setup](https://github.com/pnpm/action-setup) from 4.3.0 to 6.0.9. - [Release notes](https://github.com/pnpm/action-setup/releases) - [Commits](pnpm/action-setup@b906aff...0ebf471) --- updated-dependencies: - dependency-name: pnpm/action-setup dependency-version: 6.0.9 dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
e892c60 to
ce43a2d
Compare
lml2468
left a comment
There was a problem hiding this comment.
qa-engineer verdict: PASS
Scope: 1-line bump of pnpm/action-setup v4.3.0 → v6.0.9 in .github/workflows/ci.yml. SHA-pinned (0ebf47130e...) with matching version comment.
Verify evidence (CI snapshot):
- ✅
CI / BuildSUCCESS — pnpm install + build executes on the new action - ✅
Detect changed pathsSUCCESS - ✅
label / PR size and detect dependency changesSUCCESS - ✅
pr-title-lint,sanity/actionlint,sanity/No tabs in workflow filesSUCCESS — workflow file is syntactically clean - ✅
history / Common ancestor checkSUCCESS - ❌
check-sprint / check-sprintFAILURE — orthogonal sprint-label workflow, unrelated to this dependency bump
Coverage:
- No new code paths to test; the
Buildjob is the self-test for this change and is green. - No regression risk for existing test suites: only the action version reference changed.
- Edge: action major version v4→v6 jumps two majors; the
Buildjob exercising realpnpm installon the new action is sufficient verification — release notes only document internal pnpm bootstrap fixes and Windows compatibility (no API surface change consumed here).
Flaky risk: low — run_install: false keeps the action's behaviour minimal.
Conclusion: tests cover the change adequately. Approving.
lml2468
left a comment
There was a problem hiding this comment.
security-engineer verdict: CLEARED
Threat model (CI dependency):
- Supply chain (T1): action is pinned by commit SHA
0ebf47130e4866e96fce0953f49152a61190b271(not floating tag) — mitigates tag-rewrite / hijacked-release attacks. ✅ - Privilege escalation (T2): action runs in
Buildjob context only; no new secrets, nopull_request_target, nopermissions:change. ✅ - Code execution (T3): action only bootstraps pnpm; called with
run_install: false, so no auto-install of arbitrary package scripts at setup time. ✅
SBOM / vuln scans (CI snapshot):
- ✅
OSV-Scanner / scan-prSUCCESS — no known CVEs inpnpm/action-setup@6.0.9 - ✅
dependency-review / Dependency ReviewSUCCESS — GitHub's advisory DB clean - ✅
secret-scan / Secret Scan (gitleaks)SUCCESS
Release notes audit (v4.3.0 → v6.0.9):
- v5.x: dropped Node 16 runtime, default Node 20 — aligned with
actions/setup-node@v4already in this workflow. - v6.x: bundled pnpm bumped to v11.7.0;
devEngines.packageManager.onFail=errorhonoured; standalone+self-update fixes. No new secret reads, no new network calls beyond pnpm bootstrap. - No CVE advisories filed against any v4..v6 release of
pnpm/action-setupat time of review.
STRIDE residual risk: low. Standard transitive trust on upstream maintainers (zkochan et al), unchanged from current pin.
Conclusion: cleared.
lml2468
left a comment
There was a problem hiding this comment.
code-reviewer verdict: APPROVED
Diff: 1 line changed in .github/workflows/ci.yml:
- - uses: pnpm/action-setup@b906affcce14559ad1aafd4ab0e942779e9f58b1 # v4.3.0
+ - uses: pnpm/action-setup@0ebf47130e4866e96fce0953f49152a61190b271 # v6.0.9Correctness:
- SHA matches
pnpm/action-setuptagv6.0.9(verified against upstream commit0ebf471). - Trailing
# v6.0.9comment kept consistent with project convention (otheruses:entries in the file follow the same SHA+comment pattern, e.g.actions/checkout@34e1148 # v4,actions/setup-node@49933ea # v4). with: { run_install: false }block unchanged — call site contract is unaffected by the action version bump.
Design fit:
- Major-version jump (v4 → v6) is the only point worth flagging. Upstream breaking changes between v4 and v6:
- v5: minimum Node 20 — already satisfied (this workflow uses
actions/setup-node@v4). - v6: bundled pnpm 11.x — repo's
packageManagerfield inpackage.jsonwill be respected by the action, so no behaviour change for this CI.
No code-level adjustments required in this repo.
- v5: minimum Node 20 — already satisfied (this workflow uses
Maintainability / readability: trivial diff, no churn.
Nits: none worth blocking.
Conclusion: approving.
Aggregate Verdict: APPROVED — awaiting human merge3 reviewer verdicts collected, all green:
Summary of evidence:
Residual non-blocking notes:
Next step: human merge. Per loop policy, this autopilot will not call |
Bumps pnpm/action-setup from 4.3.0 to 6.0.9.
Release notes
Sourced from pnpm/action-setup's releases.
... (truncated)
Commits
0ebf471fix: update pnpm to v11.7.0 (#267)0e279bbfix: update pnpm to 11.1.1 (#248)3e83581fix: drop patchPnpmEnv so standalone+self-update works on Windows (#258)551b42edocs(README): fixcache_dependency_pathtype (#257)739bfe4fix: self-update bootstrap to packageManager-pinned version (#233) (#256)f61705dchore: add CODEOWNERS7a5507bfix: restore inputs from state in post (#255)1155470fix: honor devEngines.packageManager.onFail=error (#252) (#254)91ab88efix: bin_dest output points to self-updated pnpm, not bootstrap (#249)e578e19fix: update pnpm to 11.0.4