Skip to content

console: require Node 22.22.2, 24.15.0 or 26 - #40650

Merged
leedqin merged 1 commit into
MaterializeInc:mainfrom
leedqin:console-engines-node
Nov 11, 2026
Merged

leedqin merged 1 commit into
MaterializeInc:mainfrom
leedqin:console-engines-node

Conversation

@leedqin

@leedqin leedqin commented Nov 10, 2026

Copy link
Copy Markdown
Contributor

Motivation

console/package.json declares "node": ">=20.12.2", but the installed dependency tree no longer runs there. Checking every installed package's engines.node against candidate versions:

Node Packages whose engines.node rejects it
20.12.2 60 (jsdom's CSS packages, …)
20.19.0 18 (@mswjs/interceptors 0.45, @rolldown/plugin-babel, @testing-library/jest-dom 7, eslint-plugin-unicorn 79, …)
22.12.0 25 (@eslint/* needs 22.13, …)
22.18.0 7 (node-gyp 13, which 7, abbrev 5, nopt 10, lint-staged 17, undici 8.12, …)
24.11.0 5 (node-gyp 13 and its helpers)
22.22.2, 24.15.0, 26.0.0 none

msw 3 (#39684) and kysely 0.29 (#40550) both deferred this change to #39683, which is still open.

Description

engines.node becomes ^22.22.2 || ^24.15.0 || >=26.0.0, the intersection of every installed package's range. Volta (26.10.0), console/Dockerfile and CI already run Node 26. It is the same range as #39683, so that PR merges cleanly on this line.

🤖 Generated with Claude Code

engines.node still allowed Node 20.12.2, which 60 installed packages
reject, among them msw 3's interceptors, @rolldown/plugin-babel,
lint-staged 17 and npm's node-gyp 13. The new range is the intersection
of every installed package's engines.node, and matches what Volta, the
Dockerfile and CI already run (Node 26).
@leedqin
leedqin requested a review from a team as a code owner November 10, 2026 20:27
@leedqin
leedqin requested review from def- and jdonelson November 10, 2026 20:27
@leedqin
leedqin enabled auto-merge (squash) November 11, 2026 21:05
@leedqin
leedqin merged commit 4c61185 into MaterializeInc:main Nov 11, 2026
85 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants