Repository navigation
skills(pm-dispatch): references/contract-review.md states that the exports map is a package's published surface — an entry-unreachable, exports-unaddressable .d.ts is shipped bytes, not an accept set (ruling on objectui#9733) #19068
Description
Blocked-by: #18903
Filed by the director seat (objectstack #12708, summon #24, session_01Wj1HUjzyeiBQ8atRf1ZhaL) executing decision batch #167 item 5 — maintainer 「其他同意」 to B on objectui#9733. The ruling comment on that card carries the reasoning; this card carries the one deliverable, which lives in this repo.
The line to add
references/contract-review.md gains one sentence under 载体纪律 (or its successor section after PR #18903's rewrite):
已发布面以包的
exports映射为准:从入口类型图不可达、且exports不可寻址的.d.ts是出货字节,⛔ 不是已发布接受集,改它不触发Clause-②: yes;legacy node10 深引用在契约之外。
Grounds (measured by the director, 2026-09-18): both repos' tsconfigs use only bundler / nodenext / node16 — zero node10; Node's exports refuses unlisted subpaths (ERR_PACKAGE_PATH_NOT_EXPORTED); the batch #157 addendum on #18536 (5727164106, carried by PR #18903) already keys the Clause-② content limb to the protocol's public surface, so this line is the general statement that limb implies.
Sequencing
Blocked-by: #18903 — that PR rewrites references/contract-review.md wholesale (「按车道」); this line rides a references/**-only PR after it lands (queue-guard exception: ## Contract review + Served-tier: on the PR), ⛔ never pushed onto #18903. ⛔ No script changes: check-clause2-carriers.mjs is not taught to read exports maps — the sentence is the whole deliverable.
Dedupe words: exports map published surface · entry-unreachable d.ts · shipped bytes accept set · node10 deep import
Generated by Claude Code
Activity
hotlong commentedon Sep 19, 2026
Record: the blocker is spent — PR #18903 merged 2026-09-19T00:09Z;
pm:blocked→pm:queueon the ruling's own restart clause (batch #167 item 5, objectui#9733 letter B, 5732103712) · 2026-09-19T00:23ZDirector seat, summon #24,
session_01Wj1HUjzyeiBQ8atRf1ZhaL. Re-read at source:.claude/skills/pm-dispatch/references/contract-review.mdonorigin/main(60 lines, after PR #18903 and PR #19144 both rewrote it) carries no sentence on theexportsmap or an entry-unreachable.d.ts— the deliverable is still owed, and the wording in this card's body stands. It now lands under Tier S (AGENTS.md Prime Directive #14, landed by PR #19144:.claude/**is landed by the owning seat on a contract-tier review of record; the maintainer's click is no longer required), as one line under the rewritten file's 载体纪律 successor section, in areferences/**-only PR.Generated by Claude Code
objectstack-fleet commentedon Sep 23, 2026
补定级
priority:p3—— 维护者已裁的一行交付,本卡留pm:queue分诊席(
session_01Tw7jnJinGHvoGSi8aFkhPJ),2026-09-23T04:23Z。本卡在pm:queue里缺priority:*(三析取第 ④ 项)。origin/main的.claude/skills/pm-dispatch/references/contract-review.md,exports/出货字节/node100 命中 ⇒ 那一句还没落。.claude/**多处)的文件清单不含contract-review.md(本席读了该 PR 的文件列表)。5770886272,2026-09-22T03:42Z)退役了「载体纪律」与「降档保险丝」两节,但保留Clause-②: yes | no声明行 ⇒ 这句话的内容(什么算已发布接受集、什么不触发Clause-②: yes)仍然成立,落在Clause-②仍存在的那一节即可。p3判据:一行 PM 协议文字,不在任何产品路径上(NORTH-STAR.md优先级第 2 条)。Generated by Claude Code
objectstack-fleet commentedon Sep 23, 2026
Claim: PM loop round R3 (skills seat 1) —
batch3 on the maintainer's 「后续并发降到3」; in flight 0 → 2 (this act takes the two rows the 08:14 UTC landings of PR #19803 and PR #19805 released: #19068 and #19146's remainder)Session:
session_01Wnstp2kTth7sGXfr8fXypcBranch:
claude/issue-19068-contract-review-exports-map-sentenceWorktree:
objectstack-issue-19068Domain:
domain:skillsSeat:
domain:skills#1File surface:
.claude/skills/pm-dispatch/references/contract-review.mdon6eaa0f4a8(26 / 60 lines, headroom 34 ⇒ the one sentence from this card's body lands as ONE line under 「复核归属与资格(按面)」 beside the 复核面 definition at :8–:9, the section that now states what counts as the published surface) — ⛔ no script, no gate, no other file; stop on breach and explain in the reportContainer & model:
S,mode:subagent,model: TIER_DEFAULT opus—node scripts/pm/dispatch-gates.mjs --tier --repo objectstack-ai/objectstack .claude/skills/pm-dispatch/references/contract-review.mdat6eaa0f4a8(2026-09-23T08:21Z) printed verbatim: "Model tier — no path-derived mandate: the surface hits none of the 3 declared glob(s), derived here, not recalled." ⇒ the default tier for the dev; the## Contract reviewrecord of this Tier S.claude/**surface is rendered in-seat atCONTRACT_REVIEW_TIERClause-②: no
Thread-read: 5789021420
Ruling-ref: objectui#9733 comment 5732103712 — batch #167 item 5, letter B, maintainer 「其他同意」 2026-09-18T15:18Z, retrieved this act (HTTP 200); the triage seat's re-grade 5789021420 (p3, kept in
pm:queueunder that ruling, 0 hits forexports/出货字节/node10in the file at its read — re-measured 0 / 0 / 0 on6eaa0f4a8this act)Serial constraints cleared: all 17 open objectstack PRs' file lists read in this act — none touches
contract-review.md(PR #19793 touchesplatform-readings.mdonly); the fleet'spm:dispatchedclaims read (22 objectstack + 5 objectui) — noFile surface:names this file; PR #19803, the file's last writer, MERGED6eaa0f4a8at 08:14 UTC of this date;toolingin flight fleet-wide is #13597 (the maintainer's own program) — this card isdocumentation, nottoolingDispatch basis: a maintainer-ruled one-sentence deliverable (batch #167 item 5) unblocked since PR #18903 landed on 2026-09-19 and re-graded p3 by the triage seat at 04:23 UTC of this date under that ruling; product P0 / P1 read fresh this act — objectstack non-epic p0 0 · p1 25, objectui p0 0 · p1 8 — 北极星 第 3 条 bars p2 / p3
toolingand contract-hygiene cards under this seat's label-based reading (declared on seat post #7623); this card carries neither label and a ruling in hand; released by PR #19803's landing, the sentence's home section re-read on6eaa0f4a8.Generated by Claude Code
objectstack-fleet commentedon Sep 23, 2026
os-dev-report
{
"issue": 19068,
"status": "done",
"branch": "claude/issue-19068-contract-review-exports-map-sentence",
"pr": "#19821",
"pr_number": 19821,
"head_sha": "01b1ce45c979d3355c209fe6d79a5fbb7e0cd5cf",
"session": "session_01Wnstp2kTth7sGXfr8fXypc — subagent, so this is the dispatching seat's id (fleet-write route read it from the container; commit trailer Claude-Session carries the same id)",
"served_model": "the default opus tier, per the harness's own statement to this subagent (the exact model id is in the terminal message only: AGENTS.md bars a model identifier in a GitHub comment). get_session was reachable but answers for the dispatching seat's session, not this subagent, so it is not a reading of this run.",
"premise_still_valid": true,
"summary": "The ruled sentence is on the branch, verbatim from the card body, in .claude/skills/pm-dispatch/references/contract-review.md. It sits inside 「## 复核归属与资格(按面)」 at :10-:11, directly after the surface definition at :8-:9, and nothing else changed (diff +2/-0, one file; no script, gate, SKILL.md or AGENTS.md). One measured deviation from the dispatch: the sentence is ONE bullet on TWO physical lines, not one. As a single line it is 235 bytes, and check:pm-skill-ratchet's 120-byte max-line rule turns it red (measured below). So it is wrapped at the gate's own canonical wrapLine break, between 是 and 出. That is whitespace only, and the text is byte-identical after joining. Draft PR #19821 is open and left in draft, with label skip-changeset. The card's assignee was already set (the shared account); this dev wrote no assignee and posted no claim.",
"tests": "Union at final head 01b1ce4 (after merging origin/main 0e06f3b): node scripts/pm/dispatch-gates.mjs --repo objectstack-ai/objectstack --commands derived 17 commands (identical list at pre-merge 2297d9a; change set 1 path vs merge base 0e06f3b). All 17 plus 1 extra exited 0 (per-gate verdicts in
gates). --ran reconciliation: '✓ dispatch-gates --ran: 17 derived famil(ies) accounted for — 17 run, 0 NOT-MEASURED (a DERIVED zero — all 17 recorded an exit code and none of them is 3).' ONE-LINE-FORM MEASUREMENT (a mutation leg on committed HEAD 2297d9a, via node scripts/ablation-replace.mjs; anchor hit 1 -> 0; blob 5a2999b156f2 -> 275812908a4c): node scripts/pm/check-skill-line-ratchet.mjs exit 1, verdict '✗ check-skill-line-ratchet: .claude/skills/pm-dispatch/references/contract-review.md has 1 line(s) over the 120-byte budget: L10 (235B).' Restore proof: blob after restore 5a2999b156f2 == blob at HEAD; git diff HEAD empty; independent git status --porcelain empty; git hash-object == git rev-parse HEAD:path. Verbatim proof: a script stripped the continuation indent and joined the two lines; the result equals the card body's sentence exactly (and equals the dispatch's copy). H1 on base 6eaa0f4: grep -E 'exports|出货字节|node10' gave 0 hits, 26 lines, sole H2. H3: 16 open PRs' file lists read just before the PR opened; none touches contract-review.md. H4: node scripts/pm/check-governed-merges.mjs --pr 19821 exit 3 (= EXIT_TEST_GOVERNED): 'GOVERNED — Tier S(席内达档复核落地) ... .claude/** ×1 ... .claude/skills/pm-dispatch/references/contract-review.md; size: 2 changed line(s) (+2 / -0) ≤ 5000'. No package touched, so no build ① and no package test/typecheck ②. Not measured locally: the four lint.yml Type Check lanes (dispatch-gates prints them as CI-shell only; the diff has no TypeScript). CI on the head when the report was written: 12 success, 12 skipped, 5 in_progress (Lint & Repo Gates, and Type Check · source gates / workspace / consumer gates / debt ledger); Check Changeset, Governed Surface Queue Guard, Test Core and Dogfood Regression Gate are success.","gates": [
{
"command": "node scripts/check-closing-keyword-parity.mjs",
"exit": 0,
"verdict": "check-closing-keyword-parity: OK (3 parsers agree on all 9 keywords and both measured separators; sweep found 5 file(s) carrying the grammar across 9285 tracked file(s), all registered)."
},
{
"command": "node scripts/check-closing-keyword-parity.mjs --self-test",
"exit": 0,
"verdict": "✓ check-closing-keyword-parity --self-test: 40 assertions, 5 mutations of the shipped parsers each driven to red."
},
{
"command": "node scripts/check-comment-mask-corpus.mjs",
"exit": 0,
"verdict": "✓ comment-mask corpus sweep [scripts/js-comment-mask.mjs]: 7020 files, 0 disagree, 0 unparseable, 66.0s (comparator self-test: 26 cases pass)."
},
{
"command": "node scripts/pm/check-harness-current.mjs --self-test",
"exit": 0,
"verdict": "check-harness-current --self-test: all 26 cases passed."
},
{
"command": "pnpm --filter @objectstack/lint run check:doc-formula-expressions",
"exit": 0,
"verdict": "✓ check:doc-formula-expressions (field-level *When): 14 predicate(s) on a statically determinable field layer judged clean; 6 skipped as undeterminable. (First run in the fresh worktree exited 3, PREREQUISITE NOT MET: formula/lint dist unbuilt; built under os-verify-lock, VERDICT command-exit 0; rerun exit 0; rebuilt again after the merge.)"
},
{
"command": "pnpm check:agent-test-spelling",
"exit": 0,
"verdict": "✓ check-agent-test-spelling: 0 violations — 564 file(s) · 9370 bare -- token(s) · 1811 launcher-rooted run(s) · 13 separator(s) JUDGED"
},
{
"command": "pnpm check:cross-package-test-inputs",
"exit": 0,
"verdict": "OK: 29 package(s) read outside themselves, all declared, and turbo.json hashes every declared glob"
},
{
"command": "pnpm check:doc-authoring",
"exit": 0,
"verdict": "✓ doc authoring guard: 53 published skill files clean / 16192 customer-facing string(s) clean / sibling-package prose ids hold the baseline — 819 pinned site(s), no growth"
},
{
"command": "pnpm check:driver-memory-census",
"exit": 0,
"verdict": "check-driver-memory-census: OK — every declaration is ledgered, every ledger entry is live"
},
{
"command": "pnpm check:gitlink-declared",
"exit": 0,
"verdict": "check-gitlink-declared: OK (9285 index entries -- 0 gitlink(s) at mode 160000; nothing to declare)."
},
{
"command": "pnpm check:nul-bytes",
"exit": 0,
"verdict": "check-nul-bytes: OK (scanned 9278 text file(s) -- 9278 tracked, 0 untracked-not-ignored; skipped 7 binary; no raw ASCII control bytes)."
},
{
"command": "pnpm check:pm-governed-merges",
"exit": 0,
"verdict": "✓ check-governed-merges --self-test: 441 assertions; live: the real generator declared 9 output(s) and certified this tree"
},
{
"command": "pnpm check:pm-skill-id-lint",
"exit": 0,
"verdict": "✓ check-skill-id-lint: 34 file(s) clean (pattern /#[0-9]{3,}/g)."
},
{
"command": "pnpm check:pm-skill-ratchet",
"exit": 0,
"verdict": "✓ check-skill-line-ratchet: .claude/skills/pm-dispatch/references/contract-review.md is 28 lines (ceiling 60; headroom 32). / widest table row is 0 bytes (pin 0; headroom 0). Zero ✗ lines (the 120-byte line rule prints only on failure); self-test 157 cases pass."
},
{
"command": "pnpm check:refd-timer-probe",
"exit": 0,
"verdict": "OK check-refd-timer-probe: 7015 source file(s) swept"
},
{
"command": "pnpm check:skill-frame-sync",
"exit": 0,
"verdict": "✓ check-skill-frame-sync: the one declared copy of the decision frame is internally coherent (.claude/skills/pm-dispatch/SKILL.md)"
},
{
"command": "pnpm check:watch-hint-literal",
"exit": 0,
"verdict": "✓ check-watch-hint-literal: 71 declaration(s) across 4 rostered name(s)"
}
],
"gates_outside_derivation": [
{
"command": "pnpm check:pm-settings-deny-roster",
"exit": 0,
"verdict": "✓ check-settings-deny-roster: 17 content-write tool(s) declared = enforced in .claude/settings.json — run beyond the derivation because dispatch-gates marks its roster as living under .claude, a directory this diff touches"
}
],
"pushes": "3 git pushes to claude/issue-19068-contract-review-exports-map-sentence: empty-branch probe at 6eaa0f4, commit 2297d9a, merge 01b1ce4. Pre-push check:commit-card-trailers was green each time. No force-push.",
"mcp_calls": "0 MCP GitHub calls. 1 non-GitHub MCP call: mcp__Claude_Code_Remote__get_session, read-only, for the model reading.",
"api_writes": "3 REST writes, all through the fleet-write relay (transport dispatch, as objectstack-fleet[bot]): (1) pr_create = POST /repos/objectstack-ai/objectstack/pulls with draft forced true, relay run 35839052624 success, stored body byte-identical to the sent body (8147 bytes); (2) label-write --add skip-changeset = POST /repos//issues/19821/labels, relay run 35839125345 success, read back documentation · size/xs · skip-changeset, MATCHES; (3) this report = POST /repos//issues/19068/comments via scripts/pm/post-stamped.mjs. Reads were plain REST GETs.",
"deviations": [
"Dispatch Zone 1 asked for ONE physical line and said 'the file has no wrapped bullets'. Measured on base: the file already has wrapped bullets (:13-:14, :20-:21), and the one-line form is 235 bytes, red under check:pm-skill-ratchet's 120-byte rule (exit 1, L10 (235B), see tests). The sentence lands as one bullet in the gate's canonical wrapLine form, 118 + 119 bytes, broken between two CJK characters (renders as nothing). It is verbatim after whitespace normalisation, and the file goes from 26 to 28 lines. The ruling (objectui#9733 comment 5732103712) names the file and forbids any script or gate change; it does not specify physical lines, and no gate changed.",
"The injected os-dev agent definition for this run matches the shared checkout's .claude/agents/os-dev.md (branch claude/pm-dispatch-skills-nbzcw8, which still carries 'GitHub 写一律走 REST 代理'), not origin/main's, which routes writes through scripts/pm write tools and the fleet-write relay. Following the dispatch, I worked from origin/main's copy; every write went through the relay.",
"Commit trailers are the model-free AGENTS.md pair (Claude-Session + Co-authored-by: Claude). The harness reminder asked for a model-bearing Co-Authored-By and a two-line PR footer. AGENTS.md and os-dev.md take precedence, so the PR body ends in the single session-URL footer block.",
"Zone 3's route ('one commit') was taken with a second commit: the dispatch-ordered merge of origin/main 0e06f3b, a clean merge touching only packages/spec and a changeset from main.",
"The --ran record for check:doc-formula-expressions carries only the post-build run (exit 0). A first record listing both runs (3, then 0) was read as 3 by the tool, which then asked for a record that states one thing."
],
"cleanup": "rm -rf /home/user/objectstack-issue-19068/node_modules, then git worktree remove /home/user/objectstack-issue-19068: exit 0, no --force, directory gone, git worktree list no longer shows it. The local branch ref remains; the remote branch holds 01b1ce4. No background process, monitor or dev server was started. This comment was posted with the byte-identical scripts/pm tree of /home/user/objectstack-main-ro at 6eaa0f4 (git diff 6eaa0f4 HEAD -- scripts/ is empty).",
"open_questions": [],
"out_of_scope_findings": []
}
Generated by Claude Code
objectstack-fleet commentedon Sep 23, 2026
ACCEPT —
domain:skillsseat 1 (session_01Wnstp2kTth7sGXfr8fXypc, seat post #7623), 2026-09-23T08:56Z — PR #19821 at head01b1ce45c979d3355c209fe6d79a5fbb7e0cd5cf(round 0;Fixes).Checklist, read against GitHub and the fetched head on a review worktree, not the report: draft, base
main, first lineFixes #19068,Clause-②: noline-initial in the body and on the claim (5791511150, the newestClaim:); two commits on merge-base0e06f3b5d(the sentence, then the ordered merge oforigin/main); one file vs the merge-base, +2 / −0 —references/contract-review.md:10–:11, the ruled sentence (objectui#9733 comment 5732103712, batch #167 item 5 letter B) as one bullet on two physical lines; the seat joined the lines and compared them to the card body: byte-equal. The wrap is the ratchet's 120-byte line rule (one-line form 235 bytes, measured red by the dev; the file already wraps :13–:14 and :20–:21) — accepted as a whitespace-only deviation from the dispatch. No script, no gate, no other file (the ruling forbids them). Gates re-run by the seat at the head: ratchet 0 (28 / 60), id-lint 0, frame-sync 0;check-governed-merges.mjs --pr 19821exit 3 GOVERNED Tier S (.claude/**×1, 2 lines). The dev's 17-family derivation reconciled--ran17 / 17 / 0. CI at the seat's read: 25 latest-per-name check runs — 12 success, 11 skipped, 2 in progress (Lint & Repo Gates, Type Check · workspace), 0 other; the landing waits for those two and forcheck-expected-skips.mjs --pr 198210. Labelsdocumentation·size/xs·skip-changeset. Record: on the PR (PASS, rendered in-seat at the served tier). Report 5791876660 parses;mcp_calls0 GitHub (one read-onlyget_session); three relay writes plus three pushes; no body PATCH by the dev.No open question. Deviations recorded, none blocking: the wrap (above); the injected agent definition differing from
origin/main's — the dev followedorigin/main's and every write went through the relay; the model-free trailer pair by AGENTS.md; two commits (the ordered merge).Landing: Tier S — on all checks green the seat readies the PR and arms auto-merge (SQUASH) through the CCR routes; at MERGED the card closes by
Fixes,pm:dispatchedand the assignee come off in the same act, the sentence confirmed onorigin/mainby content.Generated by Claude Code
fix(pm): a pure-regeneration head move keeps the contract-review reco…
docs(pm-dispatch): contract-review states the `exports` map is a pack…