Skip to content

fix(release): grant id-token write for npm provenance - #3

Merged
adhikjoshi merged 1 commit into
mainfrom
fix/npm-provenance-permission
Aug 24, 2026
Merged

fix(release): grant id-token write for npm provenance#3
adhikjoshi merged 1 commit into
mainfrom
fix/npm-provenance-permission

Conversation

@adhikjoshi

@adhikjoshiadhikjoshi commented Aug 24, 2026

Copy link
Copy Markdown
Contributor

npm publish --provenance mints a signed attestation from a GitHub OIDC token. Without id-token: write it fails the publish outright — it does not degrade to an unsigned publish.

The workflow had only contents: write, so the first tag would have died at the npm step after goreleaser had already created the GitHub release and pushed to Homebrew and Scoop — leaving a half-published version and a tag that can't cleanly be re-run.

Caught by reading the workflow before tagging rather than by tagging.


View with [code]smithAutofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

`npm publish --provenance` mints a signed attestation from a GitHub OIDC token
and fails the publish outright without `id-token: write` — it does not fall back
to an unsigned publish. The workflow only had `contents: write`, so the first
tag would have died at the npm step after goreleaser had already cut the release
and pushed to Homebrew and Scoop, leaving a half-published version.
Caught before tagging rather than by tagging.
@adhikjoshi
adhikjoshi merged commit 6cb4915 into mainAug 24, 2026
8 checks passed
@adhikjoshi
adhikjoshi deleted the fix/npm-provenance-permission branch August 24, 2026 09:01
Sign up for freeto 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

@adhikjoshi