Skip to content

Adopt npm trusted publishing in CI#100

Merged
ryantm merged 1 commit intomainfrom
zerg/oidc-trusted-publishing
Mar 24, 2026
Merged

Adopt npm trusted publishing in CI#100
ryantm merged 1 commit intomainfrom
zerg/oidc-trusted-publishing

Conversation

@ryantm
Copy link
Contributor

@ryantm ryantm commented Mar 24, 2026

Why

  • npm trusted publishing via GitHub OIDC removes the long-lived NPM_TOKEN requirement from the release workflow.
  • npm's trusted publisher flow also requires a newer Node/npm runtime in the publish job.

What changed

  • reduced the workflow's default permissions to read-only and scoped id-token: write to the publish job
  • switched the publish job to actions/setup-node with Node 24 and the npm registry URL configured
  • removed token-based npm auth from npm publish and updated the publishing docs to point at the CI.yml trusted publisher setup

Test plan

  • git diff --check
  • did not run repo tests; this only changes GitHub Actions workflow and publishing docs

Revertibility

  • Safe to revert in git.
  • If NPM_TOKEN is removed during rollout, reverting this workflow also requires restoring that secret or updating npm publishing settings to match.

~ written by Zerg 👾

@ryantm ryantm added the zergling-authored Authored by Zerg label Mar 24, 2026
@ryantm ryantm marked this pull request as ready for review March 24, 2026 14:23
@ryantm ryantm enabled auto-merge (squash) March 24, 2026 14:25
@ryantm ryantm merged commit d1c2c8b into main Mar 24, 2026
15 of 16 checks passed
@ryantm ryantm deleted the zerg/oidc-trusted-publishing branch March 24, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

zergling-authored Authored by Zerg

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants