chore(release): record the published-metadata corrections as a changeset (META-325) - #41
Merged
qmarcelle merged 1 commit intoAug 17, 2026
Conversation
…set (META-325) The fixed group's release composition was missing its metadata half. Six changesets covered every code change since 0.4.4, but the manifest corrections that landed with the repository reconstruction and the OSS-polish pass carried none — repository/bugs pointers, the bare canonical homepage, author, two removed keywords, and the rules description. Those fields ship. So do README.md and CHANGELOG.md, both listed in `files`, which were corrected in the same passes with no release note. META-325's acceptance requires every intended public package change to map to an explicit changeset or an explicit no-release-impact decision; this closes the gap on the metadata side. Patch on its own terms. The group is taking a minor this release for reasons recorded in the accompanying changesets, so the number does not move because of this file.
|
No reviewable files after applying ignore patterns. |
Reviewer's GuideAdds a retrospective changeset documenting previously shipped metadata corrections for the @workspacejson/rules and @workspacejson/spec packages, ensuring the release notes and changelog accurately reflect the published tarball contents without changing runtime behavior or versioning semantics. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
|
There was a problem hiding this comment.
Hey - I've left some high level feedback:
- The changeset description is quite long and mixes implementation details with historical context; consider tightening it to a shorter summary focused on what changed and why from a consumer’s perspective.
- The explanation section inside the changeset (
**Why a metadata-only change gets a changeset at all.**) reads more like design rationale; you might move this kind of reasoning to an ADR or internal doc and keep the changeset itself more succinct.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- The changeset description is quite long and mixes implementation details with historical context; consider tightening it to a shorter summary focused on what changed and why from a consumer’s perspective.
- The explanation section inside the changeset (`**Why a metadata-only change gets a changeset at all.**`) reads more like design rationale; you might move this kind of reasoning to an ADR or internal doc and keep the changeset itself more succinct.Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
qmarcelle
deleted the
feature/meta-325-reconcile-changesets-and-pin-the-fixed-group-rc
branch
August 17, 2026 23:13
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.



Closes the composition gap in META-325's acceptance: every intended public package change is mapped to an explicit Changeset or an explicit documented no-release-impact decision.
What was missing
Ten commits touched
packages/since the 0.4.4 publish. Six map cleanly to existing changesets. Two need none — the union-denominator tests and negative fixtures are not infiles, and the@types/nodebump is a devDependency that does not ship. One was uncovered:repository.url+directory,bugs.urlhomepageto the bare canonical hostauthortoworkspacejson contributorsagents-auditandaaifkeywords removed@workspacejson/rulesdescriptionrewrittenAll of it ships.
README.mdandCHANGELOG.mdare both listed infiles, so they are published bytes, and the manifest fields are part of the published artifact even though no runtime behavior moves. These corrections reached consumers with no release note explaining why the npm page changed.What this adds
One changeset,
patchon both packages. It does not move the version — the fixed group is taking aminorthis release for reasons recorded in the accompanying changesets — and it is not meant to. The point is that the changelog records what shipped.Written without tracker identifiers: changesets become shipped changelog entries, and
check:docsfails on internal identifiers in public prose.Verification
check:docspasses.changeset statusstill resolves the fixed group tominor.Not in this PR
No Linear release wiring. Live release mutations are disabled for the 0.5.0 cut per the META-354 pilot verdict; the release will be backfilled from the verified version, commit and registry receipt after publication.
Summary by Sourcery
Enhancements: