chore(deps): update toolchain and security actions - #174
Conversation
|
Codex review: needs maintainer review before merge. Reviewed August 2, 2026, 2:05 AM ET / 06:05 UTC. ClawSweeper reviewWhat this changesThe PR updates pnpm and development-tool versions, advances immutable CodeQL and TruffleHog pins, and makes CI, release, and Crabbox hydration read the pnpm version from Merge readinessThis is a focused, technically sound dependency and workflow update with passing hosted CI and security checks. It should remain open for maintainer review because it changes pnpm selection in the release and Crabbox hydration paths, which have not been directly exercised in the supplied proof. Priority: P3 Review scores
Verification
How this fits togetherClawpatch workflows provision pnpm and Node before installing dependencies and running CI, release publishing, or Crabbox remote-environment hydration. This PR centralizes the pnpm version declaration in the checked-in package manifest and updates the security scanners those workflows invoke. flowchart LR
Manifest[Package manifest]
CI[CI workflow]
Release[Release workflow]
Crabbox[Crabbox hydration]
Setup[pnpm setup action]
Validation[Install and validation]
Security[CodeQL and secret scan]
Manifest --> CI
Manifest --> Release
Manifest --> Crabbox
CI --> Setup
Release --> Setup
Crabbox --> Setup
Setup --> Validation
Security --> Validation
Decision needed
Why: The code change is small and the exact CI path passes, but release publishing and self-hosted hydration have different execution environments and operational consequences that are not fully settled by unit checks. Before merge
Agent review detailsSecurityNone. Review metrics
Root-cause clusterRelationship: Members:
Proposal only: this assessment does not dispatch repair, suppress jobs, mutate sibling items, close, or merge anything. Merge-risk optionsMaintainer options:
Copy recommended automerge instructionTechnical reviewBest possible solution: Keep Do we have a high-confidence way to reproduce the issue? Not applicable: this PR maintains dependency and automation configuration rather than repairing a reported runtime defect. The supplied hosted CI and security checks provide a concrete after-change validation path for the central CI and scanning behavior. Is this the best way to solve the issue? Yes for the version-source consolidation: using the checked-in AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning high; reviewed against 4a5028c27f01. LabelsLabel justifications:
EvidenceWhat I checked:
Likely related people:
Rank-up movesOptional improvements that raise the rating; they are not merge blockers.
Rating scale
Overall follows the weaker of proof and patch quality. Workflow
HistoryReview history (1 earlier review cycle)
|
Summary
packageManagerfield the single pnpm version source for CI, Crabbox, and release workflowsThis supersedes #166: that branch stops at CodeQL 4.37.3 and updates the TruffleHog action wrapper without updating the explicit 3.95.9 scanner input.
I intentionally left
actions/staleon v10. Version 11 is the only remaining newer major, and its live behavior mutates repository issues and pull requests; this sweep does not have a safe non-mutating live proof for that major.Proof
pnpm outdated --format json:{}The GitHub checks on this PR provide the live hosted proof for the updated CodeQL and TruffleHog actions.