chore(changeset): release intent for the edit-path comment fix - #58
Merged
Conversation
systemfsoftware-maker
force-pushed
the
chore/release-intent-edit-path
branch
from
August 24, 2026 22:55
b599f95 to
0e24f61
Compare
…rehensive README rewrite The edit-path fix merged with no intent, so plan-release read phase=none (v0.1.5 already tagged, zero pending) and shipped it to nobody. This intent moves the next master push to phase=version. Also rewrites .changeset/README.md to a proper human-first guide: - Adds Mermaid state-machine diagram of the release automation contract - Explains the pnpm 11 built-in command and manual markdown structure - Documents the scope rule (Rust crate changes MUST carry intents) - Links directly to the release scripts and workflow definitions - Documents the single-paragraph changelog bullet constraint - Adds callout warning against merging PRs without intents
systemfsoftware-maker
force-pushed
the
chore/release-intent-edit-path
branch
from
August 24, 2026 22:59
0e24f61 to
129682f
Compare
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.
#57 merged with no change intent, so it shipped to nobody. This adds the intent and fixes two stale claims in
.changeset/README.mdthat are why it was skipped.Why nothing released
release.ymlruns on every master push and asksplan-release.tsfor the phase from durable repository state:The launcher manifest is at
0.1.5, tagv0.1.5exists, and.changeset/held onlyREADME.md— sopending = 0,tagged = true,phase = none. The fix has been on master since #57 and the newest tag is stillv0.1.5, so anyone installing from the registry still gets the old binary.Measured on this branch, A/B:
Merging this makes the next master push take the
versionphase, which opens the release PR; merging that leaves an untagged manifest version, and the push after it takespublish.Dry-run of the real
release-version.tsagainst a scratch copy of this branch: bumps0.1.5→0.1.6, consumes the intent, and appends one bullet under## 0.1.6. No run-on, no internals.Why the intent was skipped
Two signals pointed the wrong way:
npm/packages/comment-checker) MUST ship with an intent here." A crate-only PR reads as exempt — but the crate is the binary that package runs, so a crate-only PR changes exactly what a consumer receives.The rule as written excluded the case that most needs it. Corrected to key on consumer observability, and to say so explicitly for the crate.
Also corrected in that README
release-version.ymldoes not exist..github/workflows/holdsci,js-gate,mutation,platform,release,rust-gate,tools. The consuming logic isrelease.yml, whose version job runsrelease-version.ts. Replaced with the actual two-phase description, including the consequence that a merged fix with no intent ships to nobody.release-version.tsdoes(parts[2] ?? '').trim().split('\n').join(' '), so a multi-paragraph body collapses into one run-on bullet. Now stated, and this intent obeys it.pnpm changeis unchanged — it is a pnpm 11 built-in (pnpm change [--bump <type>] [--summary <text>] [<pkg>...], verified against pnpm 11.21.0 in the repo's dev shell), not a package script. An earlier revision of this branch wrongly claimed it did not exist, on the grounds that it is absent from the root manifest'sscripts; a built-in would never appear there. That claim is reverted and the original wording restored. The only edit to that line now is dropping thepnpm version -rparenthetical, since the bump is applied byrelease-version.ts.Verification
check_changeset.tson the intent: OK (a first draft failedB8on review vocabulary and was reworded).plan-release.tsrun locally, both with and without the intent — output above.release-version.tsrun against a scratch copy:0.1.5→0.1.6, one clean bullet, intent consumed.pnpm change --helpunder pnpm 11.21.0 — confirms the documented flags.No crate or test changes on this branch.