docs(runtime,metadata-protocol): correct the published 17.3.0 CHANGELOG's scope-less claim - #16493
Merged
Merged
Conversation
…pe-less clause in the published 17.3.0 CHANGELOG The 17.3.0 CHANGELOG entry for #14375 (`writable` verdict) said a scope-less `type: module` carried by a multi-package artifact lands as read-only alongside the marketplace/offline-import case. That is false: `defineStack` parses every `packages[]` entry through `ManifestSchema`, whose `scope` is `.default('project')`, so no package of a compiled artifact is ever scope-less (#14803 already fixed the live source comment and prose; this corrects the two published CHANGELOG.md files that still carry the original false sentence). Per the published-release-text policy: the original sentence stays verbatim, and a dated, referenced correction is appended next to it in both packages/runtime/CHANGELOG.md and packages/metadata-protocol/CHANGELOG.md, under their `## 17.3.0` entries. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Vbw3RPgdtqesx4azk9SbW8
Contributor
📓 Docs Drift Check
What this run could not see
Coarse fallback — 28 page(s) merely mention a changed package (the pre-#9192 predicate, kept for the deliberately-wide backstop): |
baozhoutao
marked this pull request as ready for review
September 7, 2026 04:57
baozhoutao
enabled auto-merge
September 7, 2026 04:57
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.
Fixes #16122
Ruling (quoted verbatim, binding)
Director seat, decision batch #65, 2026-09-07 (comment 5564448990 on the card), maintainer reply verbatim 「同意」:
What changed
Appended one sub-bullet to the existing
#14375entry in both## 17.3.0sections (nested alongside the entry's existing Why / Where / Additive sub-bullets, matching the file's own markup):The original sentence (
isWritablePackagereadsengine.manifestsFIRST … a scope-lesstype: modulecarried by a multi-package artifact lands there too) is untouched — appended-next-to, not rewritten.Why the correction holds
defineStackparses everypackages[]entry throughManifestSchema(packages/spec/src/kernel/manifest.zod.ts:319), whosescopefield isz.enum([...]).default('project')— verified directly onorigin/mainbefore writing this text.#14803's actual merge (commit4b0508e1bd, PR docs(runtime,metadata-protocol): the scope-less booted row is a marketplace / offline import, not a multi-package artifact's module #16123) already fixed the live source comment (packages/metadata-protocol/src/protocol.ts:7519-7524) and shipped a changeset (.changeset/scope-less-booted-row-attribution.md) carrying the correct explanation into the next release. This PR only corrects the two already-publishedCHANGELOG.mdfiles that still carry the pre-Five platform-side comments still name the multi-package artifact as the source of the scope-less served row — it never was:defineStackmaterialisesscope: 'project'#14803 false sentence from the 17.3.0 release.Changeset
None added. Precedent: PR #15454 (
fix(spec): correct the offer-set claim in the icon-withdrawal CHANGELOG entry) is the same shape — a docs-only correction to an already-published packageCHANGELOG.md, adding no new release content — and was labeledskip-changesetperpr-automation.yml's own Check Changeset rule ("It releases nothing … apply the 'skip-changeset' label"). Leavingskip-changesetapplication to the PM per this task's standing terms.Verification
Docs-only, two files, six lines added. Proportionate verification per the dispatch contract:
node scripts/check-nul-bytes.mjs— exit 0.node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstackon the final commit (14eeffb8ae) derived 36 commands (29 path-matched + 7 whole-tree-declared) for this diff's twoCHANGELOG.mdpaths.pnpm check:doc-authoring,pnpm check:published-files,pnpm check:nul-bytes, and everydocs-audit/check-*-census/check-*-shapegate the derivation matched.pnpm check:dts-closure,pnpm check:dual-build-cjs-loads) exit 3PREREQUISITE NOT MET— both readdist/for the full 80-package workspace, which a docs-only two-line CHANGELOG diff does not build locally (same as PR fix(spec): correct the offer-set claim in the icon-withdrawal CHANGELOG entry #15454's identical, precedented note oncheck:dual-build-cjs-loads). NOT MEASURED, not a failure.node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --ran <list>→36 derived, 36 run, 0 NOT-MEASURED, 0 UNRUN.No
pnpm test/pnpm typecheckrun — no source file touched, no package published.🤖 Generated with Claude Code
https://claude.ai/code/session_01Vbw3RPgdtqesx4azk9SbW8
Generated by Claude Code