Skip to content

fix: pin npm to ^11 to avoid broken npm@12 bundled dependencies - #1444

Merged
ankita10119 merged 1 commit into
v4from
DXCDT-2775-1
Nov 12, 2026
Merged

ankita10119 merged 1 commit into
v4from
DXCDT-2775-1

Conversation

@ankita10119

Copy link
Copy Markdown
Contributor

Changes

Pins the npm version used by the release workflow's publish step on v4:

  • .github/actions/npm-publish/action.yml: npm install -g npm@latest → npm install -g npm@^11

npm@latest now resolves to npm 12, which requires Node ^22.22.2 || ^24.15.0 || >=26.0.0. The publish job runs on Node 20, so the v4.37.2 release failed at this step with EBADENGINE, before anything was published or tagged. npm 11 supports Node ^20.17.0 || >=22.9.0, so it works with the current runner.

This is the same change as #1367 on v5. It only affects how the CI job installs npm; no SDK code, endpoints, classes or methods change.

After this is merged, the v4.37.2 release can be re-run manually with workflow_dispatch on v4. The release commit (#1443) is already on v4, and no v4.37.2 tag or npm version exists yet.

References

Testing

This can't be tested locally, since it only runs inside the release workflow. Checked locally:

  • the YAML still parses
  • npm@^11 currently resolves to npm 11.16.0, whose engines.node is ^20.17.0 || >=22.9.0, which includes the runner's Node 20.20.2
  • there are no other npm@latest installs in .github

Checklist

@ankita10119
ankita10119 requested a review from a team as a code owner November 12, 2026 18:08
@ankita10119
ankita10119 merged commit 2f0d88c into v4 Nov 12, 2026
3 checks passed
@ankita10119
ankita10119 deleted the DXCDT-2775-1 branch November 12, 2026 18:14
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