Skip to content

fix: update minimum Node.js version to 20 for serialize-javascript compatibility#1806

Merged
chagong merged 1 commit intodevelopfrom
fix/update-node-version-for-serialize-javascript
Apr 8, 2026
Merged

fix: update minimum Node.js version to 20 for serialize-javascript compatibility#1806
chagong merged 1 commit intodevelopfrom
fix/update-node-version-for-serialize-javascript

Conversation

@chagong
Copy link
Copy Markdown
Contributor

@chagong chagong commented Apr 8, 2026

Problem

The serialize-javascript override (^7.0.3) added in #1792 for an RCE security fix requires Node.js >=20. The project was still targeting Node 18 (.nvmrc and engines.node), which doesn't have globalThis.crypto available by default. This causes a ReferenceError: crypto is not defined during extension:buildProd webpack builds.

Fix

  • Update .nvmrc from 18.20.4 to 20.18.1
  • Update engines.node in extension/package.json from ^18.20.4 to >=20.0.0

Node 18 reached EOL in April 2025, so this also aligns the project with a supported LTS version.

Fixes #1801

…mpatibility

The serialize-javascript override (^7.0.3) added for the RCE security fix
requires Node.js >=20. The project was still targeting Node 18 which
doesn't have globalThis.crypto, causing 'ReferenceError: crypto is not
defined' during webpack production builds.

- Update .nvmrc from 18.20.4 to 20.18.1
- Update engines.node in extension/package.json to >=20.0.0

Fixes #1801
@chagong chagong merged commit f1ea0c6 into develop Apr 8, 2026
5 checks passed
@chagong chagong deleted the fix/update-node-version-for-serialize-javascript branch April 8, 2026 01:42
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.

[Bug] Cannot build extension due to Java compilation errors

2 participants