Skip to content

ci: npm@latest for trusted publishing + OIDC diagnostics + npm pkg fix - #21

Merged
queso merged 1 commit into
mainfrom
ci/oidc-debug
Aug 22, 2026
Merged

ci: npm@latest for trusted publishing + OIDC diagnostics + npm pkg fix#21
queso merged 1 commit into
mainfrom
ci/oidc-debug

Conversation

@queso

@queso queso commented Aug 22, 2026

Copy link
Copy Markdown
Owner

The 0.2.0 attempt failed ENEEDAUTH with zero OIDC traces — npm never attempted the trusted-publishing exchange despite id-token: write and Node 24. Two suspects remain: the runner's bundled npm predating trusted-publishing support, or the OIDC env vars not reaching the job. This PR settles it:

  • npm install -g npm@latest before publish (guarantees a trusted-publishing-capable CLI)
  • diagnostic line prints node/npm versions and whether ACTIONS_ID_TOKEN_REQUEST_URL is present — the next failure, if any, names its own cause
  • npm pkg fix applied: bin without ./, normalized repository.url — npm's publish-time "auto-correction" warned it removed the bin entry, which would have shipped a CLI package with no CLI

If this run still ENEEDAUTHs with OIDC vars present and current npm, the remaining cause is the npmjs.com trusted-publisher registration not matching (queso / FlowSpec / release.yml, environment blank).

Merging this triggers the release run: semantic-release will again compute v0.2.0 from everything since v0.1.2. No tag/release was created by the failed runs — state is clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01GtAjJabU9hUkrzoWF3Xtzm

The 0.2.0 publish attempt hit plain ENEEDAUTH with no OIDC exchange traces
— npm never attempted trusted publishing. Upgrade npm to latest before
publish (Node-bundled npm may predate support) and log node/npm versions
plus ACTIONS_ID_TOKEN var presence so the next failure names its own cause.
npm pkg fix applied so npm stops auto-correcting bin/repository at publish
time (its 'correction' threatened to strip the bin entry entirely).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GtAjJabU9hUkrzoWF3Xtzm
@coderabbitai

coderabbitai Bot commented Aug 22, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@queso, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 50 minutes

Limit details: You’ve used the included review currently available.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

Wait for the limit to reset, then comment @coderabbitai review or push new commits to the PR.

An organization admin can change what happens after included review limits in Billing.

How do review limits work?

CodeRabbit enforces per-developer PR review limits within each organization.

For paid Pro and Pro+ reviews, CodeRabbit uses a developer's included PR review attempts over the past 7 days to set the current hourly allowance. At typical activity levels, the full plan allowance applies. Higher sustained activity can lower the allowance until earlier attempts leave the 7-day window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 28ca6471-a251-4b85-8bb1-45d1ae3c8c7e

📥 Commits

Reviewing files that changed from the base of the PR and between c4bdae7 and 0b023ad.

📒 Files selected for processing (2)
  • .github/workflows/release.yml
  • package.json

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.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Nitpick review — approve

The change updates the GitHub Actions release workflow and build configuration to support npm trusted publishing. Because this modifies the CI release pipeline without accompanying automated tests, human review is recommended to ensure the release process remains stable.

0 inline comment(s).

Verdict was approve, posted as a comment — this repo does not allow GitHub Actions to approve PRs.

@queso
queso merged commit d6efbd6 into main Aug 22, 2026
4 checks passed
@queso
queso deleted the ci/oidc-debug branch August 22, 2026 01:25
@github-actions

Copy link
Copy Markdown

🎉 This PR is included in version 0.2.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant