Skip to content

ci: switch to npm trusted publishing via OIDC#78

Merged
enaboapps merged 1 commit intodevfrom
fix/npm-trusted-publishing
Mar 10, 2026
Merged

ci: switch to npm trusted publishing via OIDC#78
enaboapps merged 1 commit intodevfrom
fix/npm-trusted-publishing

Conversation

@enaboapps
Copy link
Copy Markdown
Owner

@enaboapps enaboapps commented Mar 10, 2026

Removes NPM_TOKEN in favour of npm's trusted publishing — GitHub Actions authenticates via OIDC so no secret is needed.

Summary by CodeRabbit

  • Chores
    • Updated npm publishing workflow configuration.

@enaboapps enaboapps merged commit 000f4f4 into dev Mar 10, 2026
@enaboapps enaboapps deleted the fix/npm-trusted-publishing branch March 10, 2026 16:37
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Mar 10, 2026

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 0dc4a3b1-c33d-45cc-9263-31358cfada1a

📥 Commits

Reviewing files that changed from the base of the PR and between f171332 and 312a4b4.

📒 Files selected for processing (1)
  • .github/workflows/publish.yml

📝 Walkthrough

Walkthrough

Removed the NODE_AUTH_TOKEN environment variable from the npm publish workflow step in the GitHub Actions configuration file. This eliminates the environment variable declaration from the step's env block without altering the publish step's execution logic.

Changes

Cohort / File(s) Summary
GitHub Actions Workflow Configuration
.github/workflows/publish.yml
Removed NODE_AUTH_TOKEN environment variable from the "Publish to npm" step's env block (2 lines deleted).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Poem

🐰 A token takes flight, removed from the flow,
No secrets whispered when workflows do go,
The publish still runs, swift and clean,
Simplest of changes, most elegant scene! ✨

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/npm-trusted-publishing

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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