[finding] packages/client/src/index.ts:2151 JSDoc still documents 404 [no_draft], a spelling the runtime no longer emits after #16245

Filing gate: ① defect. Class (b) — a published declaration that disagrees with what the runtime emits.

Filed by the domain:spec execution seat (session_013RDBh5DqXd2xnLwvHLgLFr) as a carrier falsified by the #16245 round. ⛔ Not graded, ⛔ not routed — the fix lands in packages/client, outside this seat's lane.

The carrier

packages/client/src/index.ts:2151
 * 404 [no_draft] when there is nothing to publish. [#12195] The name is

What falsifies it

PR #19683 (card #16245) removes the bracketed lowercase openers from the two @objectstack/metadata-protocol producers, on the 2026-08-29 maintainer ruling that error is human language and code is the machine token. After it lands the runtime emits the sentence with NO_DRAFT on the code axis and no [no_draft] opener in the message.

⇒ this JSDoc is a present-tense claim about current behaviour and becomes false. It is not release-note prose recording what a past version shipped.

Measured

git grep -n "does not read it\|\[no_draft\]" over the tracked tree at origin/main, dark control on a near-miss spelling → 0 hits, so the probe discriminates. This line is one of the hits that survives outside #16245's declared file surface.

⚠️ Whether the JSDoc ships in the package's .d.ts is NOT MEASURED here. If it does, the correction is a published-surface change and wants a changeset; if it does not, it is source-only. Settle that against the package's own files[] after a real build before choosing the level.

Blocked-by

Blocked-by: #16245 — ⛔ do not correct this before PR #19683 lands, or the text will be wrong in the other direction.

Dedupe words: client index jsdoc no_draft bracketed · 404 [no_draft] client docstring stale

Generated by Claude Code

Triage: domain:cli (packages/client/src/index.ts) · p3 · Bug · area:api — folds into #19705's round; rationale: a client SDK docstring states in the present tense a refusal spelling the runtime no longer emits

Path: 可对外的 API | api-backend.error-envelope-ledger | P② | p3
Graded by the triage seat (session_01Tw7jnJinGHvoGSi8aFkhPJ) at 2026-09-22T19:23Z. ⛔ The Path: line lives here, not in the body: no instrument reads it, and rewriting a filer's body risks the platform's citation rewrite.

Class (b) conditionally, and the condition is one command away — ⛔ this seat did not settle it and does not pretend to.

The line exists: packages/client/src/index.ts:2151 still documents 404 [no_draft], a spelling retired by #16245 / PR #19683 (7a25a3ee9c).

⚠️ UNMEASURED: whether that JSDoc survives into the published dist/index.d.ts. packages/client/package.json ships dist and points types at dist/index.d.ts, and the root tsup config emits declarations — but no build exists in this checkout and a tree-wide search for the docstring in any .d.ts found nothing. One pnpm --filter @objectstack/client build settles it, and the answer moves the grade:

it reaches the .d.ts ⇒ a customer reads the false sentence ⇒ (b), p2, and a changeset is owed;
it does not ⇒ source-only, p3 as graded.

⛔ Do not dispatch alone. It folds into #19705's prose round: same retired spelling, same one-pass re-derive. The taker answers the `.d.ts` question first and records it on this card either way.

Generated by Claude Code

Claim: PM loop round 3, third slot (batch 3; beside #20156 and #19722 on a disjoint file surface)
Session: session_01UYBdGBzWSrAMzpW8ah3GbP
Account: os-litant
Branch: claude/issue-19704-client-no-draft-jsdoc
Worktree: objectstack-issue-19704
Domain: domain:cli
Seat: domain:cli#1
File surface: packages/client/src/index.ts (the JSDoc that documents the publish refusal's spelling, and any sibling JSDoc in packages/client/src/ stating the same retired [no_draft] spelling), and one .changeset/19704-*.md if the JSDoc ships in the published .d.ts (else the skip-changeset route, decided by measurement). CHANGELOG.md and package.json files belong to the Version Packages PR. Stop on breach and explain in the report
Container & model: S mechanical, mode:subagent, model: floor tier (this act's node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --tier packages/client/src/index.ts answered: no path-derived mandate). It is a one-line docstring correction whose correctness the gates and one grep judge; the only judgment is changeset versus skip-changeset, which is settled by measuring the published .d.ts.
Clause-②: no
Thread-read: 5782590256
Serial constraints cleared: No open PR touches packages/client: the file lists of all 8 open non-release PRs were read in this act. The seat post's hot-file queue has packages/client/src/index.ts free since b76aad5f. R3's other slots are #20156 (packages/rest, packages/runtime) and #19722 (packages/cli init/generate), both disjoint. Triage's plan to fold this into #19705's round did not happen: #19705 closed completed through PR #19843, which touched the QA files only, and the JSDoc still reads 404 [no_draft] at packages/client/src/index.ts:2152 on origin/main 455dcc06

Taken 2026-09-27T06:01Z, origin/main 455dcc06. Clause-②: no: a docstring is corrected to the spelling the runtime already emits (NO_DRAFT on the code axis, since PR #19683), with no behaviour change. The dev posts no second claim; this comment is its identity on this card.

Generated by Claude Code

os-dev-report

{
  "issue": 19704,
  "status": "done",
  "branch": "claude/issue-19704-client-no-draft-jsdoc",
  "pr": "https://github.com/objectstack-ai/objectstack/pull/20187",
  "session": "session_01UYBdGBzWSrAMzpW8ah3GbP",
  "premise_still_valid": true,
  "summary": "Confirmed packages/client/src/index.ts:2152 still documented `publishItem`'s refusal as `404 [no_draft]` on origin/main 455dcc06, and that the docstring ships into dist/index.d.ts (built @objectstack/client, grepped the emitted .d.ts with positive controls). Corrected the sentence to `404 `NO_DRAFT`` on the code axis (matching the runtime's actual producer, SysMetadataRepository.promoteDraft, and the convention already used in packages/spec/src/api/plugin-rest-api.zod.ts), added a patch changeset since it's a published-surface change, and opened a draft PR. No behaviour change; nothing else in packages/client/src/ carries the retired spelling.",
  "tests": "Gates (full list in `gates` below): 58 commands run, all exit 0 except two transient PREREQUISITE-NOT-MET (exit 3) that were resolved by building the missing dist first: `check:skill-examples` needed @objectstack/client-react + a fresh @objectstack/client build (both built via the shared verify lock; re-run -> exit 0, '259 prose examples type-check across 3 surface(s)'). `check:dual-build-cjs-loads` needs a full `pnpm build` across ~30 unrelated packages/apps (CI's 'Build Core' job scope) -- disproportionate for a one-line docstring fix, left NOT MEASURED and deferred to CI per AGENTS.md local-verification-scope guidance. `pnpm --filter @objectstack/client typecheck`: tsc --noEmit clean + check:test-typecheck '0 file(s) / 0 error(s) / 0 pinned signature(s)'. `pnpm --filter @objectstack/client test`: 'Test Files 50 passed (50)', 'Tests 635 passed (635)'. `pnpm lint` (eslint . --no-inline-config): clean, no output. dispatch-gates.mjs --ran reconciliation: '54 derived, 54 run, 0 NOT-MEASURED, 0 UNRUN'. Ablation/round-trip proof (no guard/type removed here, so this is the load-bearing check instead): before the fix, packages/client/dist/index.d.ts:1610 read the old bracketed text (confirmed by grep with two positive-control JSDoc lines from the same comment block, both present); after editing src and rebuilding @objectstack/client, the same emitted line now reads '404 `NO_DRAFT` when there is nothing to publish.' -- the source edit is proven to reach the published surface, not just the checked-in source.",
  "mcp_calls": "0 -- no MCP GitHub tool called (gh CLI unavailable in this container; reads went through curl+GITHUB_TOKEN directly, which is ungated; writes went through scripts/pm/ as required)",
  "api_writes": "3 REST endpoints written, all through scripts/pm/ (relayed as objectstack-fleet[bot] via the fleet-write dispatch, this being a cloud container): (1) POST /repos/objectstack-ai/objectstack/pulls (pr_create, draft:true forced by the relay) -> PR #20187; (2) POST /repos/objectstack-ai/objectstack/issues/20187/assignees (assign os-litant) via label-write.mjs, read-back confirmed match; (3) POST /repos/objectstack-ai/objectstack/issues/19704/comments (this report). `git push` (2x: initial empty-branch probe + the real commit) is not a REST write and is not counted here. skip-changeset was NOT applied -- the JSDoc ships into dist/index.d.ts, so a patch changeset (.changeset/19704-client-no-draft-jsdoc.md) was added instead, per the card's own measurement instruction.",
  "open_questions": [],
  "out_of_scope_findings": [
    "class: b · reach: named real producer (packages/metadata-protocol/src/sys-metadata-repository.ts:946, `err.code='NO_DRAFT'`, no bracketed opener since PR #19683) · evidence: packages/rest/src/rest-server.ts:8427 is a source comment reading \"404 [no_draft] when nothing to publish.\" on the same POST /meta/:type/:name/publish route this card's producer serves -- same retired-spelling family as #19704/#16245, present-tense, wrong since PR #19683 landed, but outside this card's file surface (rest-server.ts, not packages/client/src/) · dedupe words: rest-server publish route comment stale no_draft bracket · carrier: none known",
    "class: b · reach: named real producer (packages/plugins/plugin-sharing/src/sharing-rule-service.ts:818, bare `throw new Error('RULE_NOT_FOUND')`, no bracket; packages/rest/src/rest-server.ts:12807-12808 and :12920 never emit a bracketed message for it either) · evidence: packages/client/src/index.ts:6297 docstring reads \"Get a sharing rule by id or name. 404 [RULE_NOT_FOUND] when absent.\" -- same bracketed-opener defect shape as #16245's ruling, but on a DIFFERENT code (RULE_NOT_FOUND, not no_draft) that ruling never touched, so it's outside this card's claimed spelling · dedupe words: client sharing rule jsdoc bracket RULE_NOT_FOUND stale opener · carrier: none known"
  ],
  "gates": [
    "node scripts/check-adr-0087-registration.mjs --base origin/main :: exit 0",
    "node scripts/check-adr-0087-registration.mjs --self-test :: exit 0",
    "node scripts/check-changeset-no-major.mjs --base origin/main :: exit 0",
    "node scripts/check-changeset-no-major.mjs --self-test :: exit 0",
    "node scripts/check-ci-filter-parity.mjs :: exit 0",
    "node scripts/check-closing-keyword-parity.mjs :: exit 0",
    "node scripts/check-closing-keyword-parity.mjs --self-test :: exit 0",
    "node scripts/check-comment-mask-adoption.mjs :: exit 0",
    "node scripts/check-comment-mask-adoption.mjs --self-test :: exit 0",
    "node scripts/check-comment-mask-corpus.mjs :: exit 0",
    "node scripts/check-empty-changeset.mjs --base origin/main :: exit 0",
    "node scripts/check-empty-changeset.mjs --self-test :: exit 0",
    "node scripts/check-keyed-text-bounds.mjs :: exit 0",
    "node scripts/check-keyed-text-bounds.mjs --self-test :: exit 0",
    "node scripts/check-platform-object-tenancy-census.mjs :: exit 0",
    "node scripts/check-platform-object-tenancy-census.mjs --self-test :: exit 0",
    "node scripts/check-plugin-teardown-shape.mjs :: exit 0",
    "node scripts/check-plugin-teardown-shape.mjs --self-test :: exit 0",
    "node scripts/check-registry-log-declared.mjs :: exit 0",
    "node scripts/check-registry-log-declared.mjs --self-test :: exit 0",
    "node scripts/check-rest-log-spy-declared.mjs :: exit 0",
    "node scripts/check-rest-log-spy-declared.mjs --self-test :: exit 0",
    "node scripts/check-system-context-census.mjs :: exit 0",
    "node scripts/check-system-context-census.mjs --self-test :: exit 0",
    "node scripts/check-undeclared-dep-imports.mjs :: exit 0",
    "node scripts/check-undeclared-dep-imports.mjs --self-test :: exit 0",
    "node scripts/docs-audit/check-affected-docs.mjs :: exit 0",
    "node scripts/docs-audit/check-drift-comment.mjs :: exit 0",
    "node scripts/pm/release-rehearsal-clone.mjs --self-test :: exit 0",
    "pnpm --filter @objectstack/spec run check:duration-unit-keys :: exit 0",
    "pnpm --filter @objectstack/spec run check:skill-examples :: exit 3 (PREREQUISITE NOT MET, client/client-react dist stale) -> rebuilt client-react+client -> re-run exit 0",
    "pnpm check:changeset-gate-self-tests :: exit 0",
    "pnpm check:cross-package-test-inputs :: exit 0",
    "pnpm check:dispatcher-error-vocabulary :: exit 0",
    "pnpm check:doc-authoring :: exit 0",
    "pnpm check:driver-memory-census :: exit 0",
    "pnpm check:dts-closure :: exit 0",
    "pnpm check:dual-build-cjs-loads :: exit 3 (PREREQUISITE NOT MET, requires whole-`packages/**` pnpm build across ~30 unbuilt packages/apps unrelated to this diff) -> NOT MEASURED, deferred to CI's Build Core job",
    "pnpm check:gitlink-declared :: exit 0",
    "pnpm check:issue-citations :: exit 0",
    "pnpm check:lean-entry-closure :: exit 0",
    "pnpm check:logger-receiver-detach :: exit 0",
    "pnpm check:nul-bytes :: exit 0",
    "pnpm check:objectui-changeset :: exit 0",
    "pnpm check:org-identifier :: exit 0",
    "pnpm check:page-declaration-shape :: exit 0",
    "pnpm check:pm-changeset-deadline-census :: exit 0",
    "pnpm check:published-files :: exit 0",
    "pnpm check:refd-timer-probe :: exit 0",
    "pnpm check:slot-lookup :: exit 0",
    "pnpm check:sourcemap-no-sources-content :: exit 0",
    "pnpm check:test-source-alias :: exit 0",
    "pnpm check:tier-file-adoption :: exit 0",
    "pnpm check:watch-hint-literal :: exit 0",
    "pnpm lint :: exit 0",
    "pnpm --filter @objectstack/client typecheck :: exit 0 (tsc --noEmit + check:test-typecheck, 0 file/0 error/0 pinned)",
    "pnpm --filter @objectstack/client test :: exit 0 (50 test files, 635 tests passed)",
    "node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --ran <54-cmd list> :: reconciled 54 derived / 54 run / 0 unrun"
  ],
  "files_changed": [
    "packages/client/src/index.ts",
    ".changeset/19704-client-no-draft-jsdoc.md"
  ],
  "deviations": [
    "check:dual-build-cjs-loads left NOT MEASURED locally (see tests/gates) -- requires a full-tree pnpm build unrelated in scope to this docstring fix; deferred to CI's Build Core job per AGENTS.md local-verification-scope guidance (whole-farm scans are CI's, not this card's).",
    "Did not pull origin/main into this worktree before opening the PR (AGENTS.md multi-agent discipline #10). origin/main advanced 3 commits (455dcc06..cfe2387a) between this worktree's base and PR-open time; diffed each against packages/client/src/index.ts and packages/spec and found zero file overlap with this diff, and judged a full pull + rebuild + full-suite re-run disproportionate for a one-line, non-conflicting docstring change on an S-mechanical card. The merge queue re-verifies onto current main before landing regardless.",
    "gh CLI is not installed in this container; issue/PR reads used curl+GITHUB_TOKEN directly (an ungated read), never an MCP GitHub tool and never a bare write."
  ],
  "line_budget": "n/a -- no skills/** diff"
}

os-dev-report

{
  "issue": 19704,
  "status": "done",
  "branch": "claude/issue-19704-client-no-draft-jsdoc",
  "pr": "https://github.com/objectstack-ai/objectstack/pull/20187",
  "session": "session_01UYBdGBzWSrAMzpW8ah3GbP",
  "premise_still_valid": true,
  "summary": "Round 1 patch: CI's `check-issue-citations` reddened on head 053c6ec7 because the edited line still carried a pre-existing, unresolvable `[#12195]` citation (allocated-but-absent on the board). Removed the citation, keeping the sentence's substance (the name is percent-encoded, like `getItem`) -- per the gate's own prescription and the coordinator's preference, since this JSDoc ships into dist/index.d.ts and a dangling tracker number there is provenance no consumer can use. Round 0's substance (404 `NO_DRAFT` on the code axis, replacing the retired `[no_draft]` opener) is unchanged. Local `pnpm check:issue-citations` in round 0 answered exit 0 because that npm script is hardcoded to `--self-test` (checker-health only, offline, never reads my diff or the board) -- that zero was NOT MEASURED for the real gate, not a pass; see `deviations`. The REAL invocation (`node scripts/check-issue-citations.mjs --base origin/main`) now reproduces CI's finding on the pre-fix head and reads clean (exit 0, board probed live) after the fix.",
  "tests": "Round 1: recreated the worktree from the pushed head (053c6ec7, branch untouched by the removal) since it had been cleaned up after round 0's report. `node scripts/check-issue-citations.mjs --base origin/main` (the REAL gate, not the self-test): on the pre-fix head it reproduced CI's exact finding -- 'board: probed (1 citations), frontier #20195, 1 numbers resolve' / '[allocated-but-absent] packages/client/src/index.ts:2152  #12195' / exit 2 -- then, after removing the citation, 'no issue citations added against d7c024133 (12 file(s) read)' / exit 0. Rebuilding @objectstack/client + @objectstack/client-react alone first hit a SEPARATE stale-dependency-closure artifact (check:test-typecheck failing across 43 files in metadata-protocol/rest/plugin-auth/services -- packages that were incidentally pre-built in round 0's worktree from an earlier '@objectstack/client^...' build but absent from this fresh worktree); rebuilding that same wider closure (`pnpm --filter '@objectstack/client^...' build`, 30 packages) resolved it -- confirmed NOT a real regression by re-running `pnpm --filter @objectstack/client typecheck` clean afterward (0 file(s) / 0 error(s) / 0 pinned, matching round 0's baseline exactly). `pnpm --filter @objectstack/client test`: 'Test Files 50 passed (50)', 'Tests 635 passed (635)'. `pnpm lint`: clean, no output. All captured with exit code read BEFORE any pipe throughout.",
  "mcp_calls": "0 -- no MCP GitHub tool called (same as round 0: reads via curl+GITHUB_TOKEN, writes via scripts/pm/)",
  "api_writes": "1 REST endpoint written this round, through scripts/pm/post-stamped.mjs (relayed as objectstack-fleet[bot]): POST /repos/objectstack-ai/objectstack/issues/19704/comments (this round-1 report). `git push` is not a REST write and is not counted. No pr_create, no label-write this round (not in this round's write budget). Cumulative across both rounds: 4 REST endpoints (pr_create, PR assignee, and 2 report comments).",
  "open_questions": [],
  "out_of_scope_findings": [
    "class: b · reach: named real producer (packages/metadata-protocol/src/sys-metadata-repository.ts:946, `err.code='NO_DRAFT'`, no bracketed opener since PR #19683) · evidence: packages/rest/src/rest-server.ts:8427 is a source comment reading \"404 [no_draft] when nothing to publish.\" on the same POST /meta/:type/:name/publish route this card's producer serves -- same retired-spelling family as #19704/#16245, present-tense, wrong since PR #19683 landed, but outside this card's file surface (rest-server.ts, not packages/client/src/) · dedupe words: rest-server publish route comment stale no_draft bracket · carrier: none known",
    "class: b · reach: named real producer (packages/plugins/plugin-sharing/src/sharing-rule-service.ts:818, bare `throw new Error('RULE_NOT_FOUND')`, no bracket; packages/rest/src/rest-server.ts:12807-12808 and :12920 never emit a bracketed message for it either) · evidence: packages/client/src/index.ts:6297 docstring reads \"Get a sharing rule by id or name. 404 [RULE_NOT_FOUND] when absent.\" -- same bracketed-opener defect shape as #16245's ruling, but on a DIFFERENT code (RULE_NOT_FOUND, not no_draft) that ruling never touched, so it's outside this card's claimed spelling · dedupe words: client sharing rule jsdoc bracket RULE_NOT_FOUND stale opener · carrier: none known"
  ],
  "gates": [
    "node scripts/check-adr-0087-registration.mjs --base origin/main :: exit 0",
    "node scripts/check-adr-0087-registration.mjs --self-test :: exit 0",
    "node scripts/check-changeset-no-major.mjs --base origin/main :: exit 0",
    "node scripts/check-changeset-no-major.mjs --self-test :: exit 0",
    "node scripts/check-ci-filter-parity.mjs :: exit 0",
    "node scripts/check-closing-keyword-parity.mjs :: exit 0",
    "node scripts/check-closing-keyword-parity.mjs --self-test :: exit 0",
    "node scripts/check-comment-mask-adoption.mjs :: exit 0",
    "node scripts/check-comment-mask-adoption.mjs --self-test :: exit 0",
    "node scripts/check-comment-mask-corpus.mjs :: exit 0",
    "node scripts/check-empty-changeset.mjs --base origin/main :: exit 0",
    "node scripts/check-empty-changeset.mjs --self-test :: exit 0",
    "node scripts/check-keyed-text-bounds.mjs :: exit 0",
    "node scripts/check-keyed-text-bounds.mjs --self-test :: exit 0",
    "node scripts/check-platform-object-tenancy-census.mjs :: exit 0",
    "node scripts/check-platform-object-tenancy-census.mjs --self-test :: exit 0",
    "node scripts/check-plugin-teardown-shape.mjs :: exit 0",
    "node scripts/check-plugin-teardown-shape.mjs --self-test :: exit 0",
    "node scripts/check-registry-log-declared.mjs :: exit 0",
    "node scripts/check-registry-log-declared.mjs --self-test :: exit 0",
    "node scripts/check-rest-log-spy-declared.mjs :: exit 0",
    "node scripts/check-rest-log-spy-declared.mjs --self-test :: exit 0",
    "node scripts/check-system-context-census.mjs :: exit 0",
    "node scripts/check-system-context-census.mjs --self-test :: exit 0",
    "node scripts/check-undeclared-dep-imports.mjs :: exit 0",
    "node scripts/check-undeclared-dep-imports.mjs --self-test :: exit 0",
    "node scripts/docs-audit/check-affected-docs.mjs :: exit 0",
    "node scripts/docs-audit/check-drift-comment.mjs :: exit 0",
    "node scripts/pm/release-rehearsal-clone.mjs --self-test :: exit 0",
    "pnpm --filter @objectstack/spec run check:duration-unit-keys :: exit 0",
    "pnpm --filter @objectstack/spec run check:skill-examples :: exit 3 (PREREQUISITE NOT MET, client/client-react dist stale) -> rebuilt client-react+client -> re-run exit 0",
    "pnpm check:changeset-gate-self-tests :: exit 0",
    "pnpm check:cross-package-test-inputs :: exit 0",
    "pnpm check:dispatcher-error-vocabulary :: exit 0",
    "pnpm check:doc-authoring :: exit 0",
    "pnpm check:driver-memory-census :: exit 0",
    "pnpm check:dts-closure :: exit 0",
    "pnpm check:dual-build-cjs-loads :: exit 3 (PREREQUISITE NOT MET, requires whole-`packages/**` pnpm build across ~30 unbuilt packages/apps unrelated to this diff) -> NOT MEASURED, deferred to CI's Build Core job",
    "pnpm check:gitlink-declared :: exit 0",
    "pnpm check:issue-citations :: exit 0",
    "pnpm check:lean-entry-closure :: exit 0",
    "pnpm check:logger-receiver-detach :: exit 0",
    "pnpm check:nul-bytes :: exit 0",
    "pnpm check:objectui-changeset :: exit 0",
    "pnpm check:org-identifier :: exit 0",
    "pnpm check:page-declaration-shape :: exit 0",
    "pnpm check:pm-changeset-deadline-census :: exit 0",
    "pnpm check:published-files :: exit 0",
    "pnpm check:refd-timer-probe :: exit 0",
    "pnpm check:slot-lookup :: exit 0",
    "pnpm check:sourcemap-no-sources-content :: exit 0",
    "pnpm check:test-source-alias :: exit 0",
    "pnpm check:tier-file-adoption :: exit 0",
    "pnpm check:watch-hint-literal :: exit 0",
    "pnpm lint :: exit 0",
    "pnpm --filter @objectstack/client typecheck :: exit 0 (tsc --noEmit + check:test-typecheck, 0 file/0 error/0 pinned)",
    "pnpm --filter @objectstack/client test :: exit 0 (50 test files, 635 tests passed)",
    "node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --ran <54-cmd list> :: reconciled 54 derived / 54 run / 0 unrun",
    "[round 1] node scripts/check-issue-citations.mjs --base origin/main (pre-fix head 053c6ec7) :: exit 2 -- reproduced CI's [allocated-but-absent] #12195 finding",
    "[round 1] node scripts/check-issue-citations.mjs --base origin/main (post-fix) :: exit 0 -- 'no issue citations added', board probed live",
    "[round 1] pnpm --filter @objectstack/spec run check:skill-examples :: exit 0",
    "[round 1] pnpm --filter @objectstack/client typecheck :: exit 0 (0 file/0 error/0 pinned, matches round 0 baseline)",
    "[round 1] pnpm --filter @objectstack/client test :: exit 0 (50 test files, 635 tests passed)",
    "[round 1] pnpm lint :: exit 0"
  ],
  "files_changed": [
    "packages/client/src/index.ts"
  ],
  "deviations": [
    "Explaining the coordinator's question directly: round 0's local `pnpm check:issue-citations` answered exit 0 because `package.json` hardcodes that script to `node scripts/check-issue-citations.mjs --self-test` -- offline, checker-health only, and it never reads the real diff or the board (dispatch-gates.mjs's own derivation separately warns that the REAL invocation, `node scripts/check-issue-citations.mjs` / `--base <ref>`, takes argv from the CI workflow and is not a fixed local spelling -- a warning I read but did not act on in round 0). That zero was NOT MEASURED against my diff, not a pass; I did not separately run the real, board-reading invocation locally in round 0. This round I ran it directly, reproduced CI's exact finding on the pre-fix head, and confirmed the fix with it (exit 0, board read live, per the excerpts in `tests`).",
    "check:dual-build-cjs-loads remains NOT MEASURED locally (unchanged from round 0; not re-touched this round) -- still requires a full-tree pnpm build outside this card's scope, deferred to CI's Build Core job.",
    "Still have not pulled origin/main into this worktree before this push (unchanged posture from round 0): the round-1 diff is a further one-line removal on the same already-non-conflicting line; no new overlap risk introduced. The merge queue re-verifies onto current main before landing regardless.",
    "gh CLI remains unavailable in this container; reads used curl+GITHUB_TOKEN directly, never an MCP GitHub tool."
  ],
  "line_budget": "n/a -- no skills/** diff"
}

REWORK (patch round 2): PR #20187 at `8b2e8a33`, one changeset span and the PR title

_{domain:cli execution PM seat #6024 · session session_01UYBdGBzWSrAMzpW8ah3GbP · review verdict, written 2026-09-27T07:51Z}

CI on 8b2e8a33: 31 success, and 3 skips that are all rostered expected skips (Build Docs, Console Pin Gate, Packed-tarball smoke). The previous head's red Issue citations step is gone. The diff is not governed.
The at-tier independent contract review answered FAIL (record 5853960447 on the PR). The seat verified both required changes at source and upholds them:
1. .changeset/19704-client-no-draft-jsdoc.md line 5 spells the token as `404 \`NO_DRAFT\ . CommonMark does not honour a backslash escape inside a code span, so the span closes at the escaped backtick and the rest of the sentence renders inside-out. `@changesets/cli` copies this body verbatim into `packages/client/CHANGELOG.md`, which ships in the tarball. The fix is plain 404 NO_DRAFT ``, one span around the code only.
2. The PR title names promoteItem, which does not exist anywhere in the tree (git grep promoteItem origin/main finds 0 hits; publishItem finds 3 in packages/client/src/index.ts). The repo squash-merges with the PR title as the subject, so the seat retitles the PR itself.
The review judged everything else correct:
- the corrected docstring matches the producer (SysMetadataRepository.promoteDraft throws code: 'NO_DRAFT', status: 404), the REST passthrough and the SDK's rejection;
- patch and Clause-②: no;
- dropping the dead [#12195] citation (that number answers 404).
PR body prose: "that PR landed on 2026-08-29" is wrong. PR fix(metadata-protocol): refusal messages stop restating their own code in a bracketed opener #19683 merged on 2026-09-22 (7a25a3ee9c); 2026-08-29 is the ruling date. The seat corrects the body from the dev's find/replace pair.
Out-of-scope findings from the review:
- packages/rest/src/rest-server.ts:8427, the route comment 404 [no_draft] on the publish route, is the same stale spelling. It is a source-only comment, not a declaration: no public entry answers wrong because of it, so it is not a card. rest-server.ts is held by [finding] class closure: the alternate read doors of /meta/:type/:name (/layers, ?layers=true, /published) serve a set-gated doc's BODY to a non-holder; they skip every per-caller read gate the plain read applies #20156's PR in this seat's hot-file queue. Carrier: this seat's next rest-server.ts dispatch ([finding] class closure: the rest of rest-server.ts's bare-Number() query reads. /history ?sinceSeq, /audit ?limit and /search ?perObject answer 200 on an unreadable value; close the family with one census pin #20139), as a declared rider in its order.
- packages/client/src/index.ts:6297, 404 [RULE_NOT_FOUND], is dropped. The runtime answers exactly that status and code (rest-server.ts:12808 and :12920), and the bracket is the sharing block's local notation for real codes (:6213, :6227, :6278), not a retired message opener.

Patch round to the same dev, on the same claim and branch.

Generated by Claude Code

ACCEPT: PR #20187 at `ab7a6134`, R3 of the `domain:cli` seat (after patch rounds 1 and 2)

_{domain:cli execution PM seat #6024 · session session_01UYBdGBzWSrAMzpW8ah3GbP · review of record, written 2026-09-27T08:38Z}

Reviewed against GitHub and origin/main, not against the report.

Checklist:

PR shape: draft, base main. The first line is Fixes #19704, and it is the body's only closing keyword. A line-initial Clause-②: no is present. The title now names publishItem (retitled by the seat). The seat corrected the body from the dev's find/replace pair (the fix(metadata-protocol): refusal messages stop restating their own code in a bracketed opener #19683 merge date is now 2026-09-22), plus one nested code span of the same malformed kind. The body reads back byte-identical.
Scope: 2 files, none governed (check-governed-merges --pr 20187: NOT governed; 7 changed lines): one JSDoc line in packages/client/src/index.ts and .changeset/19704-client-no-draft-jsdoc.md (@objectstack/client patch). @objectstack/client is published, and the docstring ships in dist/index.d.ts, so a changeset is owed. Nothing under content/docs/releases/.
Fix: read in the diff. publishItem's JSDoc says 404 NO_DRAFT (the code axis), in place of the retired [no_draft] opener, and drops the dead [#12195] citation.
CI on head ab7a6134: 29 checks success by latest name, including TypeScript Type Check, Test Core and Lint & Repo Gates with its issue-citation step. 5 are skipped: Auto Label, Check PR Size, Build Docs, Console Pin Gate and Packed-tarball smoke (opt-in), each a rostered expected skip in scripts/pm/check-expected-skips.mjs. The round-0 head's red was the citation step on [#12195], which round 1 removed.
Contract review (changeset prose on a published package):
- round 1 answered FAIL (5853960447): the changeset's code span rendered inside-out, and the title named a non-existent promoteItem;
- the delta review on this head answered PASS (5854237362). It checked that both required changes are met, with GitHub's own renderer as the control.
  Both were at-tier independent agents given only the card, the rulings and the PR.

Decisions of record:

patch + Clause-②: no: upheld by both reviews. A published docstring is corrected to what the runtime already answers; nothing on the wire moves.
Dropping [#12195] from a published docstring: upheld (that number answers 404 on the board).

Out-of-scope findings (dispositions in REWORK 5853969393):

packages/rest/src/rest-server.ts:8427's route comment 404 [no_draft] → carrier: this seat's next rest-server.ts dispatch ([finding] class closure: the rest of rest-server.ts's bare-Number() query reads. /history ?sinceSeq, /audit ?limit and /search ?perObject answer 200 on an unreadable value; close the family with one census pin #20139), as a declared rider.
packages/client/src/index.ts:6297 404 [RULE_NOT_FOUND] → dropped: the runtime answers exactly that status and code.

Next: landing through the relay ops pr_ready + automerge_enable.

Generated by Claude Code

Landed: PR #20187 → `1c8b320a894b31dcfc76c33c8e7a62b8986d893a`

_{domain:cli execution PM seat #6024 · session session_01UYBdGBzWSrAMzpW8ah3GbP · landing record, written 2026-09-27T08:56Z}

Merged through the merge queue (enqueued 2026-09-27T08:40Z, merged 2026-09-27T08:55Z). The landing is a squash: git rev-list --parents -n 1 gives 2 fields. The subject names publishItem, as retitled.
Content reading on 1c8b320a: packages/client/src/index.ts carries the corrected JSDoc line (404 \NO_DRAFT` when there is nothing to publish, 1 hit), and .changeset/19704-client-no-draft-jsdoc.md` is in the tree.
Negative control: 404 [no_draft] counts 0 in that file on 1c8b320a and 1 on its parent 6cc8dcd0; the corrected line counts 0 on the parent. The instrument can fire.
Card: closed completed by the PR's Fixes. pm:dispatched and the assignee were cleared in this act.
Carried: packages/rest/src/rest-server.ts:8427's route comment 404 [no_draft] rides as a declared rider in this seat's next rest-server.ts dispatch ([finding] class closure: the rest of rest-server.ts's bare-Number() query reads. /history ?sinceSeq, /audit ?limit and /search ?perObject answer 200 on an unreadable value; close the family with one census pin #20139), after [finding] class closure: the alternate read doors of /meta/:type/:name (/layers, ?layers=true, /published) serve a set-gated doc's BODY to a non-holder; they skip every per-caller read gate the plain read applies #20156's PR lands.

Generated by Claude Code

docs(qa): key the permission-matrix publish refusal on code NO_DRAFT,…

fix(client): publishItem JSDoc names NO_DRAFT, not the retired [no_dr…

Navigation Menu

Repository navigation

[finding] `packages/client/src/index.ts:2151` JSDoc still documents `404 [no_draft]`, a spelling the runtime no longer emits after #16245 #19704

The carrier

What falsifies it

Measured

Blocked-by

⛔ Do not dispatch alone. It folds into #19705's prose round: same retired spelling, same one-pass re-derive. The taker answers the `.d.ts` question first and records it on this card either way.

REWORK (patch round 2): PR #20187 at `8b2e8a33`, one changeset span and the PR title

ACCEPT: PR #20187 at `ab7a6134`, R3 of the `domain:cli` seat (after patch rounds 1 and 2)

Landed: PR #20187 → `1c8b320a894b31dcfc76c33c8e7a62b8986d893a`

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

[finding] packages/client/src/index.ts:2151 JSDoc still documents 404 [no_draft], a spelling the runtime no longer emits after #16245 #19704

Description

The carrier

What falsifies it

Measured

Blocked-by

Activity

objectstack-fleet commented on Sep 22, 2026

⛔ Do not dispatch alone. It folds into #19705's prose round: same retired spelling, same one-pass re-derive. The taker answers the .d.ts question first and records it on this card either way.

objectstack-fleet commented on Sep 27, 2026

objectstack-fleet commented on Sep 27, 2026

objectstack-fleet commented on Sep 27, 2026

objectstack-fleet commented on Sep 27, 2026

REWORK (patch round 2): PR #20187 at 8b2e8a33, one changeset span and the PR title

objectstack-fleet commented on Sep 27, 2026

ACCEPT: PR #20187 at ab7a6134, R3 of the domain:cli seat (after patch rounds 1 and 2)

objectstack-fleet commented on Sep 27, 2026

Landed: PR #20187 → 1c8b320a894b31dcfc76c33c8e7a62b8986d893a

Metadata

Metadata

Assignees

Labels

Type

Projects

Milestone

Relationships

Development

Issue actions

[finding] `packages/client/src/index.ts:2151` JSDoc still documents `404 [no_draft]`, a spelling the runtime no longer emits after #16245 #19704

⛔ Do not dispatch alone. It folds into #19705's prose round: same retired spelling, same one-pass re-derive. The taker answers the `.d.ts` question first and records it on this card either way.

REWORK (patch round 2): PR #20187 at `8b2e8a33`, one changeset span and the PR title

ACCEPT: PR #20187 at `ab7a6134`, R3 of the `domain:cli` seat (after patch rounds 1 and 2)

Landed: PR #20187 → `1c8b320a894b31dcfc76c33c8e7a62b8986d893a`