chore: migrate publish to npm Trusted Publishing (OIDC)#237
Conversation
|
Warning Rate limit exceeded
You’ve run out of usage credits. Purchase more in the billing tab. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
WalkthroughThis PR migrates npm publishing from long-lived token authentication to GitHub Actions OIDC (Trusted Publishing). The publish workflow assigns the job to the ChangesTrusted Publishing Migration
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
Adds pnpm overrides patching 13 high-severity transitive advisories (tar, vite, picomatch, lodash, path-to-regexp, flatted, hono) to fixed releases within the same major. pnpm audit --audit-level=high now 0 high. Also drops the stale Node 20 reference from the CI Matrix header comment (matrices already run 22/24 only).
Summary
Migrates helixir to npm Trusted Publishing (OIDC), removing the long-lived
NPM_TOKENdependency. Part of the org-wide migration in response to the npm Mini Shai-Hulud token rotation event. Pattern validated on discord-ops@0.23.3 and create-helix@0.9.5.Changes
environment: npm-publishto publish jobNODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}from changesets/action envNPM_CONFIG_PROVENANCE: 'true'id-token: writealready present at workflow levelWhy no npm-install step
helixir publishes via pnpm (pinned 9.15.9). pnpm 9.15.9 has native OIDC trusted-publishing auth — confirmed working on create-helix@0.9.5 (same pnpm version).
npm-side configuration (required before merge)
bookedsolidtech, repo:helixir, workflow:publish.yml, environment:npm-publishNote
pnpm auditreports 42 vulns (13 high). These do NOT block publish (helixir's prepublishOnly has no audit gate). PR #226 already tracks transitive-dep patching.Test plan
Summary by CodeRabbit