Skip to content

fix: run snapshot publish outside pnpm exec#656

Merged
brendanjryan merged 3 commits into
mainfrom
brendanjryan/fix-snapshot-publish
Jul 15, 2026
Merged

fix: run snapshot publish outside pnpm exec#656
brendanjryan merged 3 commits into
mainfrom
brendanjryan/fix-snapshot-publish

Conversation

@brendanjryan

Copy link
Copy Markdown
Collaborator

Motivation

Snapshot publishing rewrote the manifest through Zile, then pnpm revalidated the rewritten workspace manifest and rejected its intentionally stale lockfile.

Summary

  • Added a dedicated main-snapshot publish script that runs Zile and Changesets directly after setup.
  • Removed the unsupported Changesets publish snapshot flag.

Key design considerations

  • Preserved frozen-lockfile enforcement for all normal pnpm operations.
  • Kept Zile's manifest restoration in the publish sequence.

@pkg-pr-new

pkg-pr-new Bot commented Jul 15, 2026

Copy link
Copy Markdown

Open in StackBlitz

npm i https://pkg.pr.new/mppx@656

commit: 587e910

@brendanjryan brendanjryan marked this pull request as ready for review July 15, 2026 20:51
@cursor

cursor Bot commented Jul 15, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@brendanjryan brendanjryan merged commit a556718 into main Jul 15, 2026
14 checks passed
@brendanjryan brendanjryan deleted the brendanjryan/fix-snapshot-publish branch July 15, 2026 21:26
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