Skip to content

ci: verify npm publication before release cleanup - #115

Merged
ByBrawe merged 1 commit into
mainfrom
ci/verify-npm-publish
Aug 18, 2026
Merged

ci: verify npm publication before release cleanup#115
ByBrawe merged 1 commit into
mainfrom
ci/verify-npm-publish

Conversation

@ByBrawe

@ByBrawe ByBrawe commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • make one-shot release/npm-v* branches the only npm publish trigger
  • remove the redundant v* tag trigger that could re-run the same publish workflow after the release branch creates its tag
  • poll the public npm registry for the exact published package version after npm publish
  • create the GitHub tag/release and delete the one-shot release branch only after registry verification succeeds

Why

The release branch already owns the full publication transaction: verify, test, pack, npm publish, tag, GitHub release, and branch cleanup. Listening to the generated v* tag as a second publish trigger can create a redundant follow-up run whose existing-tag guard necessarily fails.

The workflow also previously trusted the npm publish process exit alone. Registry propagation is now checked explicitly with bounded npm view <package>@<version> version retries before the release is finalized.

Scope

Release CI only. Runtime/package source is unchanged.

Copilot AI lite review requested due to automatic review settings August 18, 2026 02:05

Copilot AI 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.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@ByBrawe
ByBrawe merged commit 5177e67 into main Aug 18, 2026
6 checks passed
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