Skip to content

ci: use OIDC for nightly npm publish (drop NPMJS_TOKEN)#1048

Open
chakAs3 wants to merge 2 commits intomainfrom
ci/nightly-oidc-without-npm-token
Open

ci: use OIDC for nightly npm publish (drop NPMJS_TOKEN)#1048
chakAs3 wants to merge 2 commits intomainfrom
ci/nightly-oidc-without-npm-token

Conversation

@chakAs3
Copy link
Copy Markdown
Collaborator

@chakAs3 chakAs3 commented Apr 11, 2026

Nightly releases already run in the same workflow file as tag releases and have id-token: write. With trusted publishing configured for this repo + release.yml, npm/pnpm can authenticate via GitHub OIDC without NODE_AUTH_TOKEN.

Change

  • Remove NODE_AUTH_TOKEN: secrets.NPMJS_TOKEN from the Nightly release step so behavior matches the tag release job (OIDC-only publish).

Why

  • Aligns nightly with trusted publishing and avoids relying on a long-lived (or missing/expired) granular token in GitHub secrets.

After merge

  • Confirm the next push to main runs pnpm nightly-release successfully. If NPMJS_TOKEN is unused elsewhere, the secret can be removed from repo settings.

Made with Cursor

Remove NODE_AUTH_TOKEN from the nightly job so npm/pnpm use the same
trusted publishing path as tag releases. NPMJS_TOKEN is no longer
required for nightly when GitHub Actions OIDC is configured for this
workflow.

Made-with: Cursor
@netlify
Copy link
Copy Markdown

netlify Bot commented Apr 11, 2026

Deploy Preview for nuxt-storybook failed. Why did it fail? →

Name Link
🔨 Latest commit 315c9d9
🔍 Latest deploy log https://app.netlify.com/projects/nuxt-storybook/deploys/69db7c5cd02fc40008af7df1

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Apr 11, 2026

Open in StackBlitz

npm i https://pkg.pr.new/@nuxtjs/storybook@1048
npm i https://pkg.pr.new/@storybook-vue/nuxt@1048

commit: 315c9d9

@tobiasdiez
Copy link
Copy Markdown
Collaborator

Thanks! I tried that in #1047, but it's still not working. Any idea?

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