Repository navigation
chore: upgrade Node to 26.12.1 (CVE-2026-111497) - #324
Conversation
Bump the mise-pinned Node toolchain from 26.3.0 to 26.12.1 and update the mise.lock URLs and checksums (from nodejs.org SHASUMS256.txt). _Generated with `xum` • Model: `anthropic:claude-opus-5-5` • Thinking: `high`_
|
@codex review |
|
@codex security review |
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. |
|
Codex Review: Didn't find any major issues. Keep it up! Reviewed commit: ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
If Codex has suggestions, it will comment; otherwise it will react with 👍. Codex can also answer questions or update the PR. Try commenting "@codex address that feedback". |
🛡️ Codex Security ReviewSecurity review completed. No security issues were found in this pull request. Reviewed commit: Only the user who started this review can view the report in Codex. ℹ️ About Codex security reviews in GitHubThis is an experimental Codex feature. Security reviews are triggered when:
Once complete, Codex will leave suggestions, or a comment if no findings are found. |
Summary
Upgrades the pinned Node toolchain from 26.3.0 to 26.12.1 to fix Node2Shell (CVE-2026-111497). Fixed Node lines: 22.23.4, 24.22.1, 26.12.1 or newer.
Node is a dev and CI tool in this repo (prettier, the release-please scripts, and reviewfixer's ACP adapter). No plugin code runs on Node, and nothing here decodes attacker uploads with Brotli.
Files changed
mise.toml:node = "26.3.0"→node = "26.12.1".mise.lock: the node entry now points to the v26.12.1 tarballs for linux-x64 and macos-arm64. The checksums matchhttps://nodejs.org/dist/v26.12.1/SHASUMS256.txt.Other Node references checked
I searched the repo for every other Node pin. None needs a change:
.nvmrc,.node-version,.tool-versions, Dockerfile or Containerfile.scripts/package.jsonhas noenginesorvoltafield. Theenginesentries inscripts/package-lock.jsonare dependency minimums, not pins.jdx/mise-action(nosetup-nodeornode-version), so CI picks up themise.tomlpin..devcontainer/devcontainer.jsoninstalls the toolchain through mise and does not pin Node.Validation
All runs used Node from mise.
node -vprintedv26.12.1.mise install: passed.mise run setup: passed.mise run check: 0 warnings / 0 errors in 113 files.mise run format-check: 254 files, 0 changed.mise run test: 706 successes / 0 failures / 0 errors.npm ci --prefix scriptsandnpm --prefix scripts run test:release-please-runner: passed.CI runs the integration tests. They do not use Node.
Generated with
xum• Model:anthropic:claude-opus-5-5• Thinking:high