Skip to content

fix(ci): use Node 20 + npm@latest for OIDC publish - #16

Merged
alec-bell merged 1 commit into
mainfrom
fix/node20-for-npm-publish
Apr 29, 2026
Merged

alec-bell merged 1 commit into
mainfrom
fix/node20-for-npm-publish

Conversation

@alec-bell

Copy link
Copy Markdown
Contributor

Node 22 on GHA runners is broken for npm publish. Switch to Node 20 + npm@latest which is the exact combination that published v1.0.0 successfully.

Node 22.22.2 on GitHub Actions has a broken npm (missing promise-retry
module) that can't self-upgrade. Even pinning to Node 22.11.0, the
bundled npm v10.x fails OIDC provenance publishing with 404.

Switch to Node 20 + npm@latest (v11.x) which is the exact combination
that successfully published v1.0.0 on March 18.
@alec-bell
alec-bell force-pushed the fix/node20-for-npm-publish branch from abc2b00 to 45e74df Compare April 29, 2026 00:04
@alec-bell
alec-bell requested a review from kolchinski April 29, 2026 00:05

@kolchinski kolchinski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM 👍

@alec-bell
alec-bell merged commit 8a3dd48 into main Apr 29, 2026
1 check passed
@alec-bell
alec-bell deleted the fix/node20-for-npm-publish branch April 29, 2026 00:06

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: abc2b00c6f

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

registry-url: "https://registry.npmjs.org"

- name: Upgrade npm
run: npm install -g npm@latest

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Pin npm to a stable version in release job

Installing npm@latest makes the publish pipeline non-deterministic and can break the release job as soon as npm's latest tag requires a newer Node runtime than the pinned Node 20 runner. In this workflow, that failure would occur before npm ci, blocking all releases on main/beta; pinning npm to a tested major/minor avoids unexpected breakage from upstream npm tag changes.

Useful? React with 👍 / 👎.

@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.1.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants