Skip to content

fix: restore npm OIDC publish on Node 24 - #29

Merged
cursor[bot] merged 1 commit into
masterfrom
cursor/fix-npm-oidc-publish-a4e2
Aug 24, 2026
Merged

fix: restore npm OIDC publish on Node 24#29
cursor[bot] merged 1 commit into
masterfrom
cursor/fix-npm-oidc-publish-a4e2

Conversation

@rharutyunyan

Copy link
Copy Markdown
Contributor

Change Summary

What does this PR change?
Fixes publish-to-npm so a tag publish can use npm trusted publishing (OIDC) again. v2.0.6 already published to GitHub Packages; npm failed with E404 because actions/setup-node writes a dummy NODE_AUTH_TOKEN and npm then skips OIDC.

Related Issue/Ticket:
Follow-up to #28 / PT9-1372 release.

Testing & Verification

How was this tested?

  • Unit tests
  • Integration tests
  • Manual testing (describe steps)
  • Verified on staging

Risk Assessment

Risk Level:

  • Low - Minor changes, no operational impact
  • Medium - Moderate changes, limited impact, standard rollback available
  • High - Significant changes, potential operational impact, complex rollback

Risks & Impact
CI-only. Does not change library source. After merge, the next tag (v2.0.7) should publish to npm.

Changes Made

  • Use Node 24 for the npm job (ships npm ≥ 11.5.1).
  • Unset NODE_AUTH_TOKEN and strip the _authToken line setup-node writes before npm publish --access public.
Open in Web Open in Cursor 

v2.0.6 reached GitHub Packages but npm rejected the publish (E404).
setup-node writes a dummy NODE_AUTH_TOKEN that makes npm skip OIDC.

Co-authored-by: Radik Harutyunyan <rharutyunyan@users.noreply.github.com>
@rharutyunyan
rharutyunyan requested a review from krboktv as a code owner August 24, 2026 10:49
@cursor
cursor Bot merged commit 422a546 into master Aug 24, 2026
3 checks passed
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