Skip to content

fix(ci): add npm registry auth to release publish job#319

Merged
tmchow merged 1 commit intoEveryInc:mainfrom
tonymfer:fix/npm-publish-auth
Mar 20, 2026
Merged

fix(ci): add npm registry auth to release publish job#319
tmchow merged 1 commit intoEveryInc:mainfrom
tonymfer:fix/npm-publish-auth

Conversation

@tonymfer
Copy link
Copy Markdown
Contributor

Summary

  • Add registry-url: https://registry.npmjs.org to actions/setup-node@v4 step
  • Add NODE_AUTH_TOKEN env var to the npm publish step

Why

The publish-cli job in the release workflow calls npm publish --provenance --access public but the Node.js setup step doesn't configure the npm registry URL or auth token. This causes ENEEDAUTH on every release attempt.

Closes #317

Test plan

  • Verified the setup-node step now configures registry-url
  • Verified NODE_AUTH_TOKEN is passed to the publish step

🤖 Generated with Claude Code

The publish-cli job calls `npm publish --provenance --access public` but
the setup-node step did not configure the registry URL or auth token,
causing ENEEDAUTH on every release attempt.

- Add `registry-url: https://registry.npmjs.org` to setup-node step
- Add `NODE_AUTH_TOKEN` env var to the publish step

Closes EveryInc#317

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@tmchow
Copy link
Copy Markdown
Collaborator

tmchow commented Mar 20, 2026

Thanks @tonymfer :)

@tmchow tmchow merged commit 3361a38 into EveryInc:main Mar 20, 2026
2 checks passed
@github-actions github-actions Bot mentioned this pull request Mar 20, 2026
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.

Bug: release publish job fails without npm auth

2 participants