Skip to content

docs(runtime,metadata-protocol): correct the published 17.3.0 CHANGELOG's scope-less claim - #16493

Merged
baozhoutao merged 1 commit into
mainfrom
claude/issue-16122-changelog-correction-line
Sep 7, 2026
Merged

docs(runtime,metadata-protocol): correct the published 17.3.0 CHANGELOG's scope-less claim#16493
baozhoutao merged 1 commit into
mainfrom
claude/issue-16122-changelog-correction-line

Conversation

@baozhoutao

Copy link
Copy Markdown
Contributor

Fixes #16122

Ruling (quoted verbatim, binding)

Director seat, decision batch #65, 2026-09-07 (comment 5564448990 on the card), maintainer reply verbatim 「同意」:

General rule (same as recorded on #16056): published release text is never silently rewritten; a factual error gets a dated, referenced correction line appended in place, and the correction also rides the changeset into the next release (route A, already in flight via #14803's PR).

This card. A docs-only PR appends one line under the ## 17.3.0 entry in both packages/runtime/CHANGELOG.md and packages/metadata-protocol/CHANGELOG.md: Correction (date): no package of a compiled artifact is scope-less — ManifestSchema defaults scope to project; the "scope-less type: module" clause above is withdrawn, see #14803 / #16122. The original sentence stays. This corrects the repository and GitHub view; the 17.3.0 npm tarball keeps the old text, and the correction line itself is the record of that difference.

What changed

Appended one sub-bullet to the existing #14375 entry in both ## 17.3.0 sections (nested alongside the entry's existing Why / Where / Additive sub-bullets, matching the file's own markup):

  - **Correction (2026-09-07):** no package of a compiled artifact is
    scope-less — `ManifestSchema` defaults `scope` to `project`; the
    "scope-less `type: module`" clause above is withdrawn, see #14803 / #16122.

The original sentence (isWritablePackage reads engine.manifests FIRST … a scope-less type: module carried by a multi-package artifact lands there too) is untouched — appended-next-to, not rewritten.

Why the correction holds

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 package CHANGELOG.md, adding no new release content — and was labeled skip-changeset per pr-automation.yml's own Check Changeset rule ("It releases nothing … apply the 'skip-changeset' label"). Leaving skip-changeset application 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/objectstack on the final commit (14eeffb8ae) derived 36 commands (29 path-matched + 7 whole-tree-declared) for this diff's two CHANGELOG.md paths.
  • All 36 run: 34/36 exit 0, including pnpm check:doc-authoring, pnpm check:published-files, pnpm check:nul-bytes, and every docs-audit / check-*-census / check-*-shape gate the derivation matched.
  • 2/36 (pnpm check:dts-closure, pnpm check:dual-build-cjs-loads) exit 3 PREREQUISITE NOT MET — both read dist/ 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 on check:dual-build-cjs-loads). NOT MEASURED, not a failure.
  • Reconciliation: 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 typecheck run — no source file touched, no package published.


🤖 Generated with Claude Code

https://claude.ai/code/session_01Vbw3RPgdtqesx4azk9SbW8


Generated by Claude Code

…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
@github-actions github-actions Bot added size/xs documentation Improvements or additions to documentation labels Sep 7, 2026
@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

📓 Docs Drift Check

⚠️ 2 changed file(s) yielded no anchor (packages/metadata-protocol/CHANGELOG.md, packages/runtime/CHANGELOG.md), so the pages documenting them are NOT COVERED by this run — this is not a clean bill of health for those files. Nothing else in this diff resolved to a documentable surface (no symbol, route or SDK anchor derived from 2 changed package(s)).

What this run could not see
  • 2 changed file(s) yielded no anchor (packages/metadata-protocol/CHANGELOG.md, packages/runtime/CHANGELOG.md) — pages documenting those are invisible to this run
  • a page that states a rule by its inputs shares no identifier with the emitter that implements the rule, so an emitter-only diff cannot list it — not on this run and not on any run. Measured on fix(driver-sql): emit varchar(maxLength) for a text field a declared index keys on #11430: content/docs/protocol/objectql/types.mdx documents the text-family column mapping by the ObjectQL type names it maps FROM (text / textarea / html) while the diff changed createColumn; it went unlisted, and it was the page that diff falsified, in four places. No shared token exists to detect this on, so a rule your change carries has to be re-read by hand in the pages that restate it.

Coarse fallback — 28 page(s) merely mention a changed package (the pre-#9192 predicate, kept for the deliberately-wide backstop): node scripts/docs-audit/affected-docs.mjs --json 4500bc5d7917324330f062424efebdadc62c6fb7packageMentionDocs.

@claude claude Bot added the skip-changeset PR has no user-facing published change; bypasses the changeset gate label Sep 7, 2026
@baozhoutao
baozhoutao marked this pull request as ready for review September 7, 2026 04:57
@baozhoutao
baozhoutao enabled auto-merge September 7, 2026 04:57
@baozhoutao
baozhoutao added this pull request to the merge queue Sep 7, 2026
Merged via the queue into main with commit 8d0aede Sep 7, 2026
39 checks passed
@baozhoutao
baozhoutao deleted the claude/issue-16122-changelog-correction-line branch September 7, 2026 05:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/xs skip-changeset PR has no user-facing published change; bypasses the changeset gate

Projects

None yet

2 participants