Skip to content

fix(version-dispatch): skip private packages in preview and dispatch#87

Merged
exo-mv merged 1 commit into
masterfrom
mv/skip-private-packages
May 21, 2026
Merged

fix(version-dispatch): skip private packages in preview and dispatch#87
exo-mv merged 1 commit into
masterfrom
mv/skip-private-packages

Conversation

@exo-mv

@exo-mv exo-mv commented May 20, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Filter packages with "private": true in package.json out of the computed bumps map immediately after computeBumpsForPr.
  • Sticky version-preview comment no longer lists private packages, and the dispatched version workflow no longer receives them.
  • If every touched package is private, the existing empty-bumps path clears any stale preview comment and skips the dispatch.

Test plan

  • yarn jest — 196 passing, including new coverage for dispatch-skips-private and preview-omits-private.

Private workspace packages (`"private": true` in package.json) are not
published, so they should never appear in the preview comment or be
sent to the version workflow. Drop them from the bumps map immediately
after computation so both the sticky preview comment and the dispatched
inputs are clean. If every touched package is private the existing
empty-bumps path takes over: stale preview comments get cleared and
no workflow dispatch happens.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude <noreply@anthropic.com>
@exo-mv exo-mv requested a review from exo-egor May 20, 2026 19:29
@exo-mv exo-mv marked this pull request as ready for review May 20, 2026 19:29
@exo-mv exo-mv enabled auto-merge (squash) May 20, 2026 21:42
@exo-mv exo-mv requested a review from sparten11740 May 21, 2026 14:32
@exo-mv exo-mv merged commit 393f3c6 into master May 21, 2026
2 checks passed
@exo-mv exo-mv deleted the mv/skip-private-packages branch May 21, 2026 15:01
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