Skip to content

Use Node 24 in publish job to fix npm self-upgrade crash#531

Merged
asimgunes merged 1 commit intoeclipse-cdt-cloud:mainfrom
asimgunes:fix/publish-node-24
May 7, 2026
Merged

Use Node 24 in publish job to fix npm self-upgrade crash#531
asimgunes merged 1 commit intoeclipse-cdt-cloud:mainfrom
asimgunes:fix/publish-node-24

Conversation

@asimgunes
Copy link
Copy Markdown
Contributor

The Publish Package to NPM job has been failing in the Update npm (OIDC/provenance support) step because npm i -g npm@latest cannot self-upgrade from the npm 10.9.x bundled with Node 22.22.2 (upstream bug npm/cli#9151). This blocked the v1.8.0 release from reaching npm.

This PR bumps only the publish job to Node 24, whose bundled npm already includes the upstream fix from npm/cli#9152. CI's Build & Test jobs stay on Node 22 — the integration test matrix (Ubuntu + Windows, GDB breakpoints, async/non-stop, native PTY tests, serialport native builds) is not touched.

The publish job runs only yarn (which triggers prepublish: yarn buildtsc + esbuild) and npm publish. No tests, no nativebuild at publish time. The published tarball is JS produced by tsc/esbuild plus .cc/.h sources — runtime-Node-version-independent — so consumers are unaffected by the build host's Node version.

A v1.8.1 release preparation PR will follow once this is merged.

Fixes #530

The Update npm step was failing with "Cannot find module 'promise-retry'"
on Node 22.22.2 due to npm/cli#9151 — npm@latest cannot self-upgrade from
the npm bundled with Node 22.22.2. Node 24's bundled npm already includes
the upstream fix from npm/cli#9152.

Only the publish job is bumped; CI's Build & Test jobs stay on Node 22.

Fixes eclipse-cdt-cloud#530
Copy link
Copy Markdown
Contributor

@jreineckearm jreineckearm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM as long as CI finishes successfully.

@asimgunes asimgunes merged commit 4a04fe8 into eclipse-cdt-cloud:main May 7, 2026
4 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.

Publish Package to NPM job fails on tag push — npm self-upgrade crash

2 participants