Skip to content

Modify publish workflow to include id-token permission#86

Merged
TylerJDev merged 1 commit intomainfrom
gp/oidc-migration
May 6, 2026
Merged

Modify publish workflow to include id-token permission#86
TylerJDev merged 1 commit intomainfrom
gp/oidc-migration

Conversation

@gracepark
Copy link
Copy Markdown
Contributor

Part of: https://github.com/github/web-systems/issues/4309

Updated publish workflow to include permissions and modify npm publish commands as we've migrated to OIDC.

Added write permission for id-token and updated npm publish command.
Copilot AI review requested due to automatic review settings May 5, 2026 22:29
@gracepark gracepark requested a review from a team as a code owner May 5, 2026 22:29
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the npm publish GitHub Actions workflow to support npm’s OIDC-based “trusted publishing” by enabling OIDC token minting and switching the publish command to emit provenance.

Changes:

  • Grant the workflow id-token: write permission to allow OIDC token issuance during publishing.
  • Remove token-based npm auth usage (NODE_AUTH_TOKEN / npm whoami) in favor of publishing with npm publish --provenance.
Show a summary per file
File Description
.github/workflows/publish.yml Adds id-token permission and updates the npm publish step to use provenance for OIDC migration.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

- run: npm whoami; npm --ignore-scripts publish
env:
NODE_AUTH_TOKEN: ${{secrets.npm_token}}
- run: npm --ignore-scripts publish --provenance
@TylerJDev TylerJDev merged commit 6a32acc into main May 6, 2026
8 checks passed
@TylerJDev TylerJDev deleted the gp/oidc-migration branch May 6, 2026 18:55
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.

3 participants