Skip to content

Use npm trusted publishing - #105

Merged
fraserxu merged 1 commit into
masterfrom
codex/use-npm-trusted-publishing
Jul 11, 2026
Merged

Use npm trusted publishing#105
fraserxu merged 1 commit into
masterfrom
codex/use-npm-trusted-publishing

Conversation

@fraserxu

@fraserxu fraserxu commented Jul 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • remove the legacy NPM_TOKEN environment variable from the npm release job
  • rely on the verified npm Trusted Publisher connection for fraserxu/react-chartist and release.yml

Why

The package now uses GitHub Actions OIDC publishing. Keeping a token reference is unnecessary and would cause the release job to attempt token-based authentication even though no repository secret is configured.

Validation

  • npm Trusted Publisher verified in the package settings with npm publish permission
  • release workflow YAML parsed successfully
  • npm run check
  • 6 tests passed
  • ESM, CommonJS, and declaration builds passed

Note

Low Risk
CI-only change to npm publish authentication; no application runtime or package API changes.

Overview
The release workflow no longer passes NODE_AUTH_TOKEN from secrets.NPM_TOKEN on the Publish to npm step.

Publishing is expected to authenticate via npm Trusted Publishing (GitHub Actions OIDC), which aligns with the workflow’s existing id-token: write permission and setup-node registry configuration—avoiding token-based auth when no secret is configured.

Reviewed by Cursor Bugbot for commit 6ced663. Bugbot is set up for automated code reviews on this repo. Configure here.

@fraserxu
fraserxu merged commit 510360a into master Jul 11, 2026
5 checks passed
@fraserxu
fraserxu deleted the codex/use-npm-trusted-publishing branch July 11, 2026 00:27
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