Repository navigation
chore(deps): select current patched Node runtimes - #194
Conversation
|
🦞👀 Pull request received. I will update this pull request when review starts. ClawSweeper review completeClawSweeper finished reviewing this revision. The review result is being finalized. |
clawsweeper
Bot
added
P3
|
Codex review: needs maintainer review before merge. What this changesUpdates selected Node runtimes and Crabbox’s pnpm version while preserving the public Node 22 minimum. Example: Run Linux validation on the selected Node 24 runtime.
Review scores
ProductKind: Maintenance · Worth it: Yes · Fix scope: Complete Merge readiness✅ Ready for maintainer review Keep open for landing: this PR makes a focused runtime refresh that is absent from main, preserves existing validation coverage, and has no actionable findings. Priority: P3 Before mergeNone. FindingsNone. Tests
Agent review detailsHow this fits togetherRuntime pins select the tools used for Kova development, website and release builds, CI validation, and Crabbox hydration; they do not select installed OpenClaw fleet runtimes. flowchart TD
A[Runtime pins] --> B[CI matrix]
A --> C[Website and release builds]
A --> D[Crabbox hydration]
B --> E[Kova checks and installed CLI smoke]
D --> F[Validation workspace]
Technical reviewBest possible solution: Land the focused pin refresh with the existing engine minimum and validation gates preserved. Do we have a high-confidence way to reproduce the issue? This is maintenance rather than a reported runtime bug; source comparison verifies the older selections on main. Is this the best way to solve the issue? Directly updating the existing pins and their matching gate condition is the smallest complete change. AGENTS.md: found and applied where relevant. Codex review notes: model internal, reasoning medium; reviewed against e4aabcbe7034. Provenance checked
TestingProof path: shipped entry point. SecurityNone. EvidenceWhat I checked:
Likely related people:
LabelsLabel changes:
Label justifications:
Rating scale6/6 🦀 challenger crab · 5/6 🦞 diamond lobster · 4/6 🐚 platinum hermit · 3/6 🦐 gold shrimp · 2/6 🦪 silver shellfish · 1/6 🧂 unranked krab. Overall follows the weaker of proof and patch quality; ✨ marks media proof (a screenshot, video, or linked artifact) that directly shows the changed behavior. WorkflowClawSweeper edits this one comment on every review. Comment Reviewed October 26, 2026, 3:01 AM ET / 07:01 UTC. |
Problem and change
Kova still selected older Node patches in its local toolchain, website and CI. Select Node 24.22.1 locally and for the website, qualify CI on Node 22.23.4/24.22.1/26.12.1, and use pnpm 12.15.0 for Crabbox hydration. Keep the Linux native command-owner gate on the updated Node 24 row. Public engine minima are unchanged.
Validation
Native macOS ARM64 task-owned UTM guest, official Node 24.22.1/OpenSSL 3.5.9 and pinned OCM 0.2.48: frozen install; core checks and 282/282 self-checks; 21/21 render snapshots; web payload/release contracts; release-shaped package build; actual checksum-verified installer with required OCM; installed version/setup and 282/282 self-checks; all installed documentation links. Independent review found no actionable issues; diff check passed.
Two initial receipt snapshots failed only with a long task TMPDIR. A controlled exact-base comparison reproduces both failures, and base/candidate both pass all 21 with the same short isolated TMPDIR. No expectations were changed. Broader Linux/Node/website qualification remains with this PR's existing CI.
Rebuilt archive SHA256:
0f46cbe96e919eec90a19babd19a84144a26cccaaaeb4ae6d727e47bbb80095b.No release or deployment is included. This updates selected validation runtimes; it does not establish delivery to installed fleets.