Skip to content

ci: fix publish verification#14

Merged
anand-testcompare merged 1 commit into
mainfrom
fix/publish-oidc-verification
Feb 8, 2026
Merged

ci: fix publish verification#14
anand-testcompare merged 1 commit into
mainfrom
fix/publish-oidc-verification

Conversation

@anand-testcompare

@anand-testcompare anand-testcompare commented Feb 8, 2026

Copy link
Copy Markdown
Owner

Fix Publish Package workflow flake:

  • Parse npm _npmUser whether it is an object or a string (GitHub Actions OIDC publishes return a string).
  • Make CI publish reruns idempotent by skipping npm publish if the computed version already exists.

This unblocks the Verify published version step that was failing with _npmUser.*=''.

Summary by CodeRabbit

  • Improvements
    • Strengthened npm publisher identity verification and validation during the package publishing workflow to ensure more secure and reliable deployments.
    • Implemented pre-publication checks that prevent publishing if a package version already exists on npm, reducing the risk of accidental republishing and improving overall publishing reliability.

Handle npm _npmUser string format and make CI publish reruns idempotent.
@coderabbitai

coderabbitai Bot commented Feb 8, 2026

Copy link
Copy Markdown

Walkthrough

These changes enhance the npm publishing workflow by: (1) robustly parsing npm registry JSON data to handle flexible _npmUser field formats for OIDC provenance verification, and (2) implementing a pre-publish existence check to prevent republishing already-published package versions.

Changes

Cohort / File(s) Summary
Publishing Workflow Enhancement
.github/workflows/publish.yml, .mise/tasks/publish
Added pre-publish existence check via npm view to prevent republishing. Enhanced JSON parsing for _npmUser field to handle both object and string formats, including "Name " patterns. Extended email and name extraction logic. Improved error messaging with raw npm user data output when OIDC verification fails.

Possibly related PRs

Poem

🐰 With whiskers twitching, checks run deep,
Before the package takes its leap!
OIDC verified, strings all parsed,
No doubles published—mission starred! ✨

🚥 Pre-merge checks | ✅ 2
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'ci: fix publish verification' is directly related to the PR's main objective of fixing the publish verification step in the CI workflow.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix/publish-oidc-verification

No actionable comments were generated in the recent review. 🎉


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

@anand-testcompare anand-testcompare merged commit ab4a4c3 into main Feb 8, 2026
6 checks passed
@anand-testcompare anand-testcompare deleted the fix/publish-oidc-verification branch February 8, 2026 05:15
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