Skip to content

Pin npm 11.18.0 for Node 20 Docker builds - #7839

Merged
ar2rsawseen merged 1 commit into
release.24.05from
codex/fix-node20-npm-compat
Jul 23, 2026
Merged

Pin npm 11.18.0 for Node 20 Docker builds#7839
ar2rsawseen merged 1 commit into
release.24.05from
codex/fix-node20-npm-compat

Conversation

@ar2rsawseen

@ar2rsawseen ar2rsawseen commented Jul 23, 2026

Copy link
Copy Markdown
Member

What changed

  • Pin npm to exactly 11.18.0 in the Debian API/frontend Dockerfiles.
  • Apply the same exact pin to the CentOS API/frontend Dockerfiles.

Why

The release Dockerfiles used npm@latest with Node 20. npm 12 is now the latest major, but it requires Node ^22.22.2, ^24.15.0, or >=26, so the Docker build fails with EBADENGINE.

Pinning npm 10 would avoid that immediate engine error, but the current npm 10 release (10.9.8) still reports known vulnerabilities in its bundled dependency tree, including the critical node-tar advisory. npm 11.18.0:

  • supports Node ^20.17.0 || >=22.9.0;
  • includes the patched tar@7.5.19;
  • returned zero known vulnerabilities in a clean audit comparison performed on 2026-07-23.

The exact pin also keeps future npm major releases from unexpectedly breaking this release branch.

Validation

  • git diff --check
  • docker buildx build --check for all four changed Dockerfiles
  • Installed and ran npm@11.18.0 on the same node:iron-bookworm-slim base:
    • Node v20.20.2
    • npm 11.18.0
  • A full local Dockerfile-api build completed the base-image Python build and tests, then stopped before npm because this arm64 host cannot install the Dockerfile's hardcoded amd64 tini package. That architecture mismatch is pre-existing and unrelated to this change.

Failure reproduced from:

@ar2rsawseen
ar2rsawseen marked this pull request as ready for review July 23, 2026 12:32
Copilot AI review requested due to automatic review settings July 23, 2026 12:32

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ar2rsawseen
ar2rsawseen merged commit f13a3dc into release.24.05 Jul 23, 2026
6 of 7 checks passed
@ar2rsawseen ar2rsawseen changed the title Fix Node 20 Docker builds after npm 12 release Pin npm 11.18.0 for Node 20 Docker builds Jul 23, 2026
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.

3 participants