Skip to content

fix: use pnpm publish instead of npm to prevent ENEEDAUTH#25

Merged
yltw27 merged 1 commit into
mainfrom
fix/npm-publish-oidc
Apr 14, 2026
Merged

fix: use pnpm publish instead of npm to prevent ENEEDAUTH#25
yltw27 merged 1 commit into
mainfrom
fix/npm-publish-oidc

Conversation

@yltw27
Copy link
Copy Markdown
Collaborator

@yltw27 yltw27 commented Apr 14, 2026

Summary

The publish step was failing with ENEEDAUTH because npm and pnpm have conflicting auth configurations. pnpm handles all earlier workflow steps (install, build, test), but switching to npm publish causes npm to pick up stale or empty auth tokens written by pnpm, preventing OIDC from working.

Changes

  • Changed npm publish to pnpm publish in the CI workflow
  • Added --no-git-checks flag to allow publishing after the version commit
  • Ensures consistent auth configuration throughout the workflow

Why this fixes it

pnpm's native OIDC support works without interference from npm's auth resolution. By staying in pnpm, we avoid the auth state collision where npm tries to use an empty/stale token instead of falling through to OIDC trusted publishing.


🤖 Generated with Claude Code

pnpm was already handling all package operations (install, build, test).
Switching the publish step from npm to pnpm ensures consistent registry and
auth configuration throughout the workflow. npm's OIDC flow can fail when
pnpm has written registry config that npm then misinterprets. Using pnpm
publish avoids this auth state collision and leverages pnpm's native
--provenance and OIDC support.

Also add --no-git-checks to allow publishing after version commit.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@yltw27 yltw27 marked this pull request as ready for review April 14, 2026 10:28
@yltw27 yltw27 requested a review from a team as a code owner April 14, 2026 10:28
@yltw27 yltw27 requested review from djslimbo, horlaarsco and matt-koevort and removed request for a team April 14, 2026 10:28
@yltw27 yltw27 merged commit 1d41860 into main Apr 14, 2026
2 checks passed
@yltw27 yltw27 deleted the fix/npm-publish-oidc branch April 14, 2026 10:31
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