Skip to content

chore: gate npm installs by 3-day minimum release age - #58

Merged
karlkeppner1 merged 3 commits into
chains/realio-testnetfrom
chore/min-release-age-gate-testnet
Jun 3, 2026
Merged

chore: gate npm installs by 3-day minimum release age#58
karlkeppner1 merged 3 commits into
chains/realio-testnetfrom
chore/min-release-age-gate-testnet

Conversation

@karlkeppner1

Copy link
Copy Markdown

Mirrors #57 onto the chains/realio-testnet branch.

Changes

  • Add .npmrc with supply-chain hardening: min-release-age=3, audit-level=critical, engine-strict=true, registry=https://registry.npmjs.org/.
  • Dockerfile (node:23): copy .npmrc before npm ci and RUN npm i -g npm@latest so npm ≥ 11.10.0 honors min-release-age.
  • Dockerfile-bitbucket (node:14, legacy): copy .npmrc before npm ci (npm too old for min-release-age; other keys apply).

@karlkeppner1 karlkeppner1 reopened this Jun 3, 2026
@karlkeppner1
karlkeppner1 merged commit 5d1db2f into chains/realio-testnet Jun 3, 2026
0 of 2 checks passed
@karlkeppner1
karlkeppner1 deleted the chore/min-release-age-gate-testnet branch June 3, 2026 17:32
karlkeppner1 added a commit that referenced this pull request Aug 7, 2026
The chains/realio-testnet build is currently broken:

  #8 RUN npm i -g npm@latest
  npm error code EBADENGINE
  npm error Required: {"node":"^22.22.2 || ^24.15.0 || >=26.0.0"}
  npm error Actual:   {"npm":"10.9.2","node":"v23.11.1"}

npm@latest resolved to 12.0.2, which dropped node 23 support. The image is
node:23-alpine, and .npmrc sets engine-strict=true, so this is a hard
error rather than a warning. Nothing in the repo changed to cause it --
the registry moved under an unpinned install.

11.19.0 is the newest npm whose engines (^20.17.0 || >=22.9.0) still cover
node 23, and it is >= 11.10.0, so the min-release-age=3 cooldown from #58
continues to apply. That was the entire reason this upgrade step exists.

This is the same unpinned-global-install class of bug that #60 addressed
in the workflow; this Dockerfile line was missed.

Co-authored-by: Claude Opus 5 (1M context) <[email protected]>
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.

1 participant