Skip to content

Replace Node 18 with Node 24 in the development bundle#484

Merged
prydonius merged 1 commit into
mainfrom
zjj/prydonius/include-node-24-in-the-wkmrtuzwyqvm
Jul 24, 2026
Merged

Replace Node 18 with Node 24 in the development bundle#484
prydonius merged 1 commit into
mainfrom
zjj/prydonius/include-node-24-in-the-wkmrtuzwyqvm

Conversation

@prydonius

@prydonius prydonius commented Jul 23, 2026

Copy link
Copy Markdown
Contributor

Why

Production repl-runner cells use the full Nixmodules b024b55 artifact, where Node 24 is available. Goval's reduced local development bundle tops out at Node 22, so agent toolchain resolution behaves differently locally.

What changed

Replaces EOL nodejs-18 with nodejs-24 in the reduced development bundle. This brings local toolchain selection in line with production without growing the bundle.

Test plan

  • nix build .#bundle-squashfs --print-out-paths --no-link
  • Confirm the generated registry contains nodejs-24 and no longer contains nodejs-18.
  • Confirm the artifact contains nodejs-24.13.0-wrapped/bin/node.
  • The artifact decreased from 1.4 GiB to 1.3 GiB.
  • nix fmt .

Rollout

This only changes reduced development artifacts. Production full bundles retain every module. Safe to revert.

  • This is fully backward and forward compatible

Stack

Review in PRZerg

~ written by Zerg 👾 (veteran-vulture-6afc)

@prydonius prydonius added the zergling-authored Authored by Zerg agent label Jul 23, 2026
@prydonius
prydonius force-pushed the zjj/prydonius/include-node-24-in-the-wkmrtuzwyqvm branch from 1efc3f0 to 3a9963b Compare July 24, 2026 00:31
@prydonius prydonius changed the title Include Node 24 in the development bundle Replace Node 18 with Node 24 in the development bundle Jul 24, 2026
@prydonius prydonius changed the title Replace Node 18 with Node 24 in the development bundle Include Node 24 in the development bundle Jul 24, 2026
@prydonius prydonius changed the title Include Node 24 in the development bundle Replace Node 18 with Node 24 in the development bundle Jul 24, 2026
@prydonius
prydonius marked this pull request as ready for review July 24, 2026 03:07
@prydonius
prydonius requested a review from a team as a code owner July 24, 2026 03:07
@prydonius
prydonius requested review from ryantm and removed request for a team July 24, 2026 03:07
@prydonius

Copy link
Copy Markdown
Contributor Author

@replit/micromanager approve dev only

@prydonius
prydonius requested a review from cbrewster July 24, 2026 03:59
@prydonius
prydonius merged commit aa3f348 into main Jul 24, 2026
6 checks passed
@prydonius
prydonius deleted the zjj/prydonius/include-node-24-in-the-wkmrtuzwyqvm branch July 24, 2026 05:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

zergling-authored Authored by Zerg agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants