Fix npm trusted publishing workflows#87
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Run ID: 📒 Files selected for processing (4)
Summary by CodeRabbit
WalkthroughThis PR migrates npm package publishing workflows from 🚥 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)
✨ Simplify code
Comment |
|
@coderabbitai full review |
✅ Action performedFull review finished. |
Restores the publish workflows to the npm CLI path required for npm trusted publishing while keeping pnpm for repo setup and package preparation.
Changes
Why
npm trusted publishing is implemented by the npm CLI OIDC exchange. Keeping pnpm for install, versioning, and pack validation is fine, but the final publish step needs npm publish to avoid falling back to unauthenticated registry publishing.