Skip to content

Remove NODE_AUTH_TOKEN#64

Merged
dcramer merged 1 commit intomainfrom
fix-release-again
Jan 31, 2026
Merged

Remove NODE_AUTH_TOKEN#64
dcramer merged 1 commit intomainfrom
fix-release-again

Conversation

@dcramer
Copy link
Member

@dcramer dcramer commented Jan 31, 2026

No description provided.

@vercel
Copy link

vercel bot commented Jan 31, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
warden Ready Ready Preview, Comment Jan 31, 2026 6:25pm

Request Review

@@ -31,8 +31,6 @@ jobs:

- name: Publish to npm
run: npm publish --provenance --access public
Copy link
Contributor

Choose a reason for hiding this comment

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

🚨 npm publish will fail without NODE_AUTH_TOKEN authentication (high confidence)

Removing NODE_AUTH_TOKEN env var breaks npm authentication. The actions/setup-node with registry-url creates an .npmrc expecting NODE_AUTH_TOKEN for auth. The --provenance flag adds provenance attestation but does NOT replace authentication - it's supplementary. This will cause npm publish to fail with 401 Unauthorized.

Suggested fix: Restore the NODE_AUTH_TOKEN environment variable for the npm publish step

Suggested change
run: npm publish --provenance --access public
env:
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

warden: notseer

Copy link
Member Author

Choose a reason for hiding this comment

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

idk the robot said to try it

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.


- name: Publish to npm
run: npm publish --provenance --access public
env:
Copy link

Choose a reason for hiding this comment

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

Missing npm authentication token breaks package publishing

High Severity

The NODE_AUTH_TOKEN environment variable was removed from the npm publish step. When actions/setup-node is configured with registry-url, it creates an .npmrc that expects authentication via NODE_AUTH_TOKEN. Without this environment variable, npm publish will fail with an authentication error because npm cannot authenticate with the registry.

Fix in Cursor Fix in Web

@dcramer dcramer merged commit 667f677 into main Jan 31, 2026
11 of 14 checks passed
@dcramer dcramer deleted the fix-release-again branch January 31, 2026 18:28
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.

1 participant