Skip to content

chore: Unpin Node 22 test matrix from 22.12 - #22447

Merged
timfish merged 1 commit into
developfrom
awad/fix-ci-node-2212-esbuild
Jul 21, 2026
Merged

chore: Unpin Node 22 test matrix from 22.12#22447
timfish merged 1 commit into
developfrom
awad/fix-ci-node-2212-esbuild

Conversation

@logaretm

@logaretm logaretm commented Jul 21, 2026

Copy link
Copy Markdown
Member

Unpins the Node 22 unit-test matrix from 22.12 back to the latest 22.x LTS.

Something about esbuild and Node 22.12 specific version crashes it when used with jsdom which is exercised in solidstart.

Node 22.12.0 (the only 22.12.x release) is the first release with unflagged require(ESM). Under that path the solidstart vite tests trip esbuild's startup invariant ("new TextEncoder().encode("") instanceof Uint8Array" is false), failing the Node 22 unit-test job on every run and blocking all PRs.

Float the 22 line back to the latest 22.x LTS, which the job was green on before it was pinned, and which matches the floating 24/26 entries.
@logaretm logaretm changed the title ci: Unpin Node 22 test matrix from 22.12 chore: Unpin Node 22 test matrix from 22.12 Jul 21, 2026
@logaretm
logaretm marked this pull request as ready for review July 21, 2026 16:45
@logaretm
logaretm requested review from a team, JPeer264, isaacs and timfish and removed request for a team July 21, 2026 16:45
@timfish
timfish merged commit 4878364 into develop Jul 21, 2026
84 checks passed
@timfish
timfish deleted the awad/fix-ci-node-2212-esbuild branch July 21, 2026 17:27
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