Skip to content

fix(ci): restore npm upgrade for OIDC provenance publishing - #14

Merged
alec-bell merged 1 commit into
mainfrom
fix/restore-npm-upgrade
Apr 28, 2026
Merged

alec-bell merged 1 commit into
mainfrom
fix/restore-npm-upgrade

Conversation

@alec-bell

Copy link
Copy Markdown
Contributor

Problem

npm publish with --provenance has been failing with 404 since PR #11 removed the npm install -g npm@latest step. The bundled npm v10.x in Node 22 doesn't handle OIDC provenance publishing correctly with npm's trusted publishers feature.

Fix

Restore the npm@latest upgrade with corepack disable first to prevent the promise-retry module error that caused us to remove it in the first place.

Context

  • v1.0.0 published successfully (March 18) — workflow had npm upgrade
  • PR fix(ci): remove broken npm self-upgrade step #11 removed npm upgrade → every publish since has failed with 404
  • npm trusted publisher config is correct (verified on npmjs.com)
  • The issue is npm v10 vs v11 OIDC token handling

The bundled npm (v10.x) in Node 22 doesn't handle OIDC provenance
publishing correctly with npm's trusted publishers. Restoring the
npm@latest upgrade (now with corepack disable to avoid the
promise-retry module error) to get npm v11.x which handles OIDC
provenance properly.
@alec-bell
alec-bell force-pushed the fix/restore-npm-upgrade branch from 62abf5f to 429b632 Compare April 28, 2026 23:55
@alec-bell
alec-bell requested a review from kolchinski April 28, 2026 23:56

@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 10922d1 into main Apr 28, 2026
1 check passed
@alec-bell
alec-bell deleted the fix/restore-npm-upgrade branch April 28, 2026 23:57
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 1.1.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@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