Skip to content

Refactor npm publish step in workflow for OIDC migration#116

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

Refactor npm publish step in workflow for OIDC migration#116
TylerJDev merged 1 commit into
mainfrom
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.

Simplified npm publish command by removing whoami step.
@gracepark gracepark self-assigned this May 5, 2026
Copilot AI review requested due to automatic review settings May 5, 2026 22:22
@gracepark gracepark requested a review from a team as a code owner May 5, 2026 22:22
Copy link
Copy Markdown
Contributor

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 align with npm’s OIDC/trusted publishing approach (and npm provenance), removing the dependency on a long-lived NODE_AUTH_TOKEN secret.

Changes:

  • Remove NODE_AUTH_TOKEN-based authentication from the publish step.
  • Simplify the publish command to rely on npm publish --provenance (with scripts disabled).
Show a summary per file
File Description
.github/workflows/publish.yml Refactors the npm publish step to no longer use secrets.npm_token, relying on OIDC/provenance-based publishing.

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: 0

@TylerJDev TylerJDev merged commit 2cf0cee into main May 6, 2026
8 checks passed
@TylerJDev TylerJDev deleted the gp/oidc-migration branch May 6, 2026 18:48
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