fix(ci): unblock npm publish and revert unintended version bump - #477
Merged
Conversation
- pin npm to 11.x; npm 12 rejects the --no-git-checks flag changesets passes for pnpm workspaces - drop continue-on-error so a failed publish fails the release run - revert 'ci(changesets): version packages'; the Maestro changeset is patch, not minor
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Two problems, one PR — because merging them separately would publish the versions we are trying to take back.
1. Publish is broken under npm 12
Run 30457694907 reported success but published nothing:
@changesets/cli2.31.1 detects the pnpm workspace (rootpackageManager: pnpm@9.0.0) and appends--no-git-checksto the publish command (changesets-cli.esm.js:851). That flag reachesnpm publish, and npm 12 turned unknown CLI flags into a hard error — npm prints it as--git-checksbecause it strips theno-prefix. The last green publish, on 2026-06-29, ran npm 11.18.0; npm 12.0.0 shipped 2026-07-08.Pinned to
npm@11(resolves 11.18.0, still >= 11.5.1 for trusted publishing). Upgrading pnpm to 10.x is the longer-term fix and deserves its own PR.Also removed
continue-on-error: truefrom the changesets step. That flag is why this went unnoticed: the job stayed green while all four packages failed to publish.2. Revert the version bump — patch, not minor
The Maestro changeset was marked
minor, which cascaded further than intended.devnet,aiken-uplc, andscalus-uplcall declare@evolution-sdk/evolutionas apeerDependency(workspace:*), and changesets escalates peer-dependents to a major bump when the peer dep gets more than a patch. Replayingchangeset versionboth ways:@evolution-sdk/evolution@evolution-sdk/devnet@evolution-sdk/aiken-uplc@evolution-sdk/scalus-uplcOne provider bugfix should not force three major releases. This reverts
588ebb37and restores the changeset aspatch. Nothing was published at 0.6.0, so there is no yanked version to work around.After merge
The release run will open a fresh version PR at 0.5.12 / 3.0.13 / 2.0.12 / 2.0.12; merging that one publishes.
Verification
Versions confirmed back at 0.5.11 / 3.0.12 / 2.0.11 / 2.0.11 with the changeset restored as
patch, and the 0.5.12 numbers above came from an actualchangeset versionrun on a scratch worktree. The publish path itself cannot be exercised outside a release run — the next one is the real test, and it will now fail loudly instead of silently.🤖 Generated with Claude Code