Skip to content

fix: use OIDC provenance for npm publish#72

Open
nicknisi wants to merge 1 commit into
mainfrom
nicknisi/fix-oidc-publish
Open

fix: use OIDC provenance for npm publish#72
nicknisi wants to merge 1 commit into
mainfrom
nicknisi/fix-oidc-publish

Conversation

@nicknisi
Copy link
Copy Markdown
Member

@nicknisi nicknisi commented May 13, 2026

Summary

  • Remove registry-url from actions/setup-node — it was creating an .npmrc with a placeholder _authToken that npm used instead of OIDC, causing 404 on publish
  • Without registry-url, no .npmrc is written, so npm publish --provenance authenticates via OIDC directly (using the id-token: write permission)

Test plan

  • Merge and verify release-please publish succeeds with OIDC provenance

Open in Devin Review

Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

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

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant