[objectstack-fleet]

domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW

Part of #25919 (PR 2: the ui/AGENTS.md restructure. PR 1 landed as #26023.)
Ruling-ref: 6435359095 · 6485892110
Clause-②: no
Metadata-protocol: none
skip-changeset: the diff touches only ui/AGENTS.md and ui/.claude/**. No package ships either.

维护者速读

• 改了什么：ui/AGENTS.md 收成规范核心。本地开发、端口和多栈流程原样搬进 console-development.md，flaky 测试的成因、读数和前情原样搬进新的 guides/flaky-tests.md，核心里各留一行指针。新增两条写法规则，不加门禁。修了 5 处过期说法，并按契约复核的 S1 修正了 console 指南里的 MSW 说法。
• 为什么改：裁决 6435359095 和 6485892110 定了这次重构。核心只写规则，叙事放到主题页，过期说法会把 agent 带错路。
• 风险与回滚：只动文档，不动代码。27 行 ⛔ / never 全数留在核心。其中 26 行逐字节不变，第 27 行（Convert to monorepo with scoped packages #8）只按派发令补了 packages/ 路径前缀。回滚就是 revert 这个 PR。
• 先看：① ui/AGENTS.md §9 的两行新指针，加上 §5 新增的 Add comprehensive test suite for Zod schema validation #12 和 [WIP] Add Chinese version of the documentation #13；② guides/console-development.md 末尾新增的「Local dev and per-agent stacks」节；③ guides/flaky-tests.md；④ ui/CLAUDE.md 的一句话；⑤ console-development.md「Debugging & Browser Simulation Strategy」的 Rule Add metamodel interfaces for ObjectQL/ObjectUI contract #1（S1）。
• 两分钟验证：在仓根跑 git diff origin/main...HEAD --color-moved=dimmed-zebra -- ui。预期搬迁的行显示为 moved。没标 moved 的增删行只有下文列出的修正行和新写的指针、规则行。
• 需要你做的：审批这个 Tier H PR（ui/AGENTS.md、ui/.claude/**）。

What changes

ui/AGENTS.md goes from 36,140 bytes over 261 lines to 32,591 bytes over 221 lines, measured at head dbb5e1cb. There is no cap (6485892110).

Stays in the core: §1–§8 in full, plus the rule-bearing parts of §9: Housekeeping, the test-running subsection, and the 测试纪律 rules. Each rule appears once. Where a rule already named its instrument (pnpm check:*, a test file, a lint rule), the pointer stays where it was. No pointer was added to an existing line, because that would reword it.

New text:

• §5 gets two writing rules from 6407900763 item 2, written as prose:

  • Add comprehensive test suite for Zod schema validation #12: a comment in a workflow or a script is one line of why plus a link.
  • [WIP] Add Chinese version of the documentation #13: code comments state invariants, not incident history.

  Each says that no gate holds it and that existing comments are not swept. Neither contains ⛔ or "never", so the census is unchanged.

• ui/CLAUDE.md: its one line now reads "AGENTS.md is the source of truth for what differs under ui/ — it and the topic pages it links to are one instruction set, maintained in one place, under the root AGENTS.md" (6407900763 item 1; contract-review S3 6502595750, ruling 6503029051; surface widened by the seat for this file only). No rule is restated there.

• One pointer line per topic page, in §9: one under 测试纪律 and one in a new ### Local dev and per-agent stacks stub.

• One row in the objectui-contributor SKILL.md topic table for the new guide.

Moved-block map (byte-preserving)

From ui/AGENTS.md	To
§9 ### Local dev — console UI ↔ backend (read before debugging UI): the heading and its 7 bullets	ui/.claude/skills/objectui-contributor/guides/console-development.md, under a new ## Local dev and per-agent stacks
§9 ### 每个 agent 独立测试栈(端口隔离,多 agent 并行的推荐做法): the whole subsection, 4 steps and the 纪律 paragraph	same new section, after the block above
§9 测试纪律, the opening paragraph (the 976 ms / 1000 ms measurement)	new ui/.claude/skills/objectui-contributor/guides/flaky-tests.md
§9 测试纪律, the 「失败现场会直接指认根因」 bullet and the 「顺手体检」 bullet	guides/flaky-tests.md
§9 测试纪律, the 「前情:objectui#3010」 line	guides/flaky-tests.md

Nothing went to docs/postmortems/. The only incident write-up in the file is the one-line 前情 note, and it went with its section's reasoning into flaky-tests.md.

Move proof. Run git diff origin/main...HEAD --color-moved=plain -- ui. Of the 134 changed lines (blank lines included), 103 are marked moved. A second script compared the 57 removed lines against the added lines. Every removed non-blank line reappears byte-identical, except 9, and those 9 are exactly the stale-claim lines below. The full dimmed-zebra output is reproducible with the command in 维护者速读.

Stale claims fixed (the only rewording; reading 6455483141)

1. The :3001/_console claim (moved to console-development.md). It now says the backend serves /_console on its own port (:3000 in the single-stack setup). The built Console is still named as real, by its package: 「仓根 @objectstack/console 包的 dist/」. The package spelling replaces the literal root path because ui/scripts/check-skills-paths.mjs resolves a packages/ token against ui/. Once the line moved under its scan root, that gate went red on the root-relative path, which is unbuilt in a fresh checkout anyway. The package spelling keeps the claim and resolves cleanly.
2. §7 MSW bullet. It now says there is no @objectstack/plugin-msw package and no msw dependency in the workspace. It points to the real-backend stacks in the console guide and keeps the instruction not to hand-roll fetch interceptors.
3. §2 styling path. skills/objectui/rules/styling.md now says "at the repository root". The file's preamble reads paths relative to ui/.
4. §5 Convert to monorepo with scoped packages #8. app-shell/src/urlParams.ts → packages/app-shell/src/urlParams.ts. This is a census line. The dispatch ordered this fix, so the line differs from before by exactly those 9 bytes.
5. §6 registry snippet. The path is now packages/core/src/registry/Registry.ts. The invented registerComponent / resolveComponent Map snippet is replaced by an abridged view of the real Registry class (register, registerLazy, get) and the ComponentRegistry singleton. The renderer-loop snippet's one call to the removed resolveComponent becomes ComponentRegistry.get(schema.type), which is what SchemaRenderer.tsx calls. Without that change the snippet would name a function that no longer exists anywhere in the file.
6. Contract-review S1 (6502595750, ruling 6503029051). console-development.md § Debugging & Browser Simulation Strategy no longer claims a mock mode or an MSW plugin. Rule Add metamodel interfaces for ObjectQL/ObjectUI contract #1 is retitled «No MSW plugin — debug against a real backend» and points to «Local dev and per-agent stacks». The MSWPlugin Configuration bullet is removed. Rule ✨ Set up Copilot instructions #2 Alignment now names the /api proxy target (DEV_PROXY_TARGET) instead of a mock server. The SKILL.md row says «debugging against a real backend».

⛔ / never census (case-insensitive ⛔|never, one count per line)

	lines	bytes of those lines	sha256 of the line set (first 16)
before, origin/main 72b08eec (identical to ad3a640e35 for this file)	27	16,789	5c6693bd42200de7
after, head 261449bc (byte-identical to ccc751c5 and dbb5e1cb for this file, 32,591 B), all in the core	27	16,798	4a9f74261366934b

27 = 27: 26 lines byte-identical and in the same order, plus 1 that differs only by the ordered path fix. That line is §5 #8 ("UI state lives where it can survive"): its path app-shell/src/urlParams.ts gained the 9-byte prefix packages/, which is why the line set grows 16,789 → 16,798 bytes. The fix was ordered by the dispatch (reading 6455483141) because the old path does not resolve from ui/, and the seat kept it (OQ2 → A). No ⛔ / never line moved out of the core.

Dependent reader

scripts/check-agent-test-spelling.mjs COUNTER_EXAMPLE_FILES names ui/AGENTS.md for trap two (陷阱二). That bullet stays in the core with the rest of the test-running subsection, so the script and its test need no edit. pnpm check:agent-test-spelling passes and reports 1 declared counter-example(s).

Tests

• pnpm --dir ui exec vitest run --maxWorkers=2 scripts/__tests__/check-skills-paths.test.ts scripts/__tests__/check-shell-escape-residue.test.ts scripts/__tests__/check-doc-links.test.ts: Test Files 3 passed (3), Tests 196 passed (196). The lock printed VERDICT command-exit 0.
• Under ui/, three scripts exit 0:
  • node scripts/check-skills-paths.mjs: 86/87 stated paths resolve, 1 baselined.
  • node scripts/check-shell-escape-residue.mjs
  • node scripts/check-doc-links.mjs: "Links are valid across 14 scan roots."

Acceptance notes

• The core is still large. It is about 32.6 KB, because the 27 whole ⛔ / never lines alone are 16.8 KB and §5 carries most of them. The test-running subsection and the i18n-regex bullet stay whole. The first holds the dependent reader's counter-example and a 「上面的形态」 back-reference. The second states a rule inside the incident that proved it, and splitting it would reword a line.
• Stale claims outside this card's list were left alone:
  • The published skills/objectui/guides/data-integration.md still names @objectstack/plugin-msw (the seat files it). The console-development.md half was fixed here as S1.
  • The heading «Which console :3001/_console serves» in root-side docs/agents/build-and-test.md still names :3001.
  • §7's surviving census line still says "matches the mock server". Census lines stay verbatim.
• Trailers. The commits carry the model-free pair the root AGENTS.md requires.

Verification

dispatch-gates --commands (13 derived) and --ran reconciliation, head ccc751c (re-run after the ui/CLAUDE.md commit and the origin/main merge; same 13 commands, all 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-corpus.mjs :: exit 0
pnpm check:agent-test-spelling :: exit 0
pnpm check:driver-memory-census :: exit 0
pnpm check:gitlink-declared :: exit 0
pnpm check:nul-bytes :: exit 0
pnpm check:pm-governed-merges :: exit 0
pnpm check:published-files :: exit 0
pnpm check:refd-timer-probe :: exit 0
pnpm check:tier-file-adoption :: exit 0
pnpm check:undeclared-dep-imports :: exit 0
pnpm check:watch-hint-literal :: exit 0
✓ dispatch-gates --ran: 13 derived famil(ies) accounted for — 13 run, 0 NOT-MEASURED (a DERIVED zero — all 13 recorded an exit code and none of them is 3).

The first pass ran before pnpm install and four families exited 3 (PREREQUISITE NOT MET). All 13 were re-run after the install and are shown above.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QSeKYoC3JrEqdXJwZ69A8m

---

Generated by Claude Code

[objectstack-fleet]

domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW

Contract-review-claim: session_01QSeKYmTYfaPbLQD4BJfomW · head ccc751c · 2026-10-29T13:56Z

The lane reviews its own PR at CONTRACT_REVIEW_TIER, through an isolated subagent. The record follows on this PR.

---

Generated by Claude Code

[objectstack-fleet]

domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW

Contract-review-handoff: triage — CONTRACT_REVIEW_TIER unavailable (HTTP 429 rate_limit on the reviewer model, 2026-10-29T13:57Z) · head ccc751c

The lane's claim 6498630564 on this head is withdrawn in this act, so no live claim stands in triage's way. The seat does not review at a lower tier (contract-review.md). needs:contract-review goes on in the same act. The brief is the record-recognisers --brief-template output for the five hits paths, with the rulings on #25919: 6407900763 items 1–2, 6435359095, 6485892110 and the reading 6455483141. The dev report is 6498180390 with addendum 6498382703. CI on this head is green: 55 check-runs, 43 success, 12 skipped. The seat's ACCEPT and the approver hand-off follow the PASS.

---

Generated by Claude Code

[objectstack-fleet]

triage#1 · os-sales · session_01L6bhgV4fMGJHSdSdx3u9mK

Contract-review-claim: session_01L6bhgV4fMGJHSdSdx3u9mK · head ccc751c · 2026-10-29T16:25Z

---

Generated by Claude Code

[objectstack-fleet]

triage#1 · os-sales · session_01L6bhgV4fMGJHSdSdx3u9mK

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: ccc751c5f6a7d5e7df6ebd83d837bde5ef8f3c99
Local-runs: none
Scope: ui/.claude/skills/objectui-contributor/SKILL.md, ui/.claude/skills/objectui-contributor/guides/console-development.md, ui/.claude/skills/objectui-contributor/guides/flaky-tests.md, ui/AGENTS.md, ui/CLAUDE.md + ui/packages/core/src/registry/Registry.ts, ui/packages/react/src/SchemaRenderer.tsx, ui/packages/app-shell/src/urlParams.ts, skills/objectui/rules/styling.md, packages/console/package.json, packages/cli/src/commands/serve.ts, packages/cli/src/commands/dev.ts, ui/apps/console/vite.config.ts, ui/.claude/launch.json, scripts/check-agent-test-spelling.mjs, ui/scripts/check-skills-paths.mjs, ui/scripts/tests/check-i18n-dead-keys.test.ts, scripts/pm/check-governed-merges.mjs, pnpm-lock.yaml, AGENTS.md (root: Prime Directive #14, Multi-agent discipline 8–9)

Inputs read: card #25919 body and all 20 comments (rulings 6407900763 items 1–3, 6435359095, 6485892110 on #27297 with its brief 6485463740, reading 6455483141, sweep correction 6411838306, dev report 6498180390 and addendum 6498382703); PR #27576 body, file list and the net diff from the files endpoint; the 64 check-runs on this head. Governing rule texts: the rulings above, and root AGENTS.md Prime Directive #14 for the tier.

① Derived judgments

• Tier — right. All five hits paths are GOVERNED_SURFACES rows (ui/AGENTS.md, ui/CLAUDE.md, ui/.claude/**), every one Tier H under root AGENTS.md feat: Comprehensive CRM example demonstrating all ObjectStack protocol features #14; one Tier H path makes the whole PR Tier H. The PR body says so and asks for the maintainer's approval. This record lifts nothing on its own; it is the contract review the ruling 6435359095 spends, and the approval stays the maintainer's. The diff adds a new governed file (guides/flaky-tests.md) under the same row.
• Public surface — unchanged, right. No package source, schema, workflow or published skill changes. ui/.claude/skills/objectui-contributor/ carries internal: true and is not in the published skills/ catalog; check:published-files is among the 13 derived families the dev ran at exit 0, and UI Published Artifacts was not derived. skip-changeset is the correct declaration (see ②).
• Byte-preserving moves — right, re-derived from the net diff. 58 removed lines; every removed non-blank line reappears byte-identical in the added lines except 10: the 9 stale-claim lines the PR body lists (styling path, §5 Convert to monorepo with scoped packages #8, the four registry.ts snippet lines, the resolveComponent call, the MSW bullet, the :3001/_console bullet) and the ui/CLAUDE.md sentence. The dev's "9" was counted at dbb5e1cb, before the ui/CLAUDE.md commit; the tenth is that commit and is declared. git diff --color-moved=plain over merge-base..head marks 48 removed and 55 added lines as moved (the dev reported 103 of 134; the two extra coloured lines are the ui/CLAUDE.md pair). The moved blocks' internal cross-references still resolve inside the destination: 「见下「每个 agent 独立测试栈」」 and 「上面的 :5180 dev 服务」 point within the new section, and the root AGENTS.md Multi-agent discipline items it names are still item 8 (kill only a PID you recorded) and item 9 (refresh build state after merging main).
• ⛔ / never census — right. grep -ciE '⛔|never' gives 27 on origin/main and 27 on the head for ui/AGENTS.md; a line diff of the two census sets shows exactly one differing line, §5 Convert to monorepo with scoped packages #8, by the packages/ prefix. Every census line is in the core; none moved to a page. Size 36,140 → 32,591 bytes, as the body states.
• Dependent readers — right. scripts/check-agent-test-spelling.mjs COUNTER_EXAMPLE_FILES names ui/AGENTS.md for 陷阱二; that bullet stays in the core's test-running subsection, so the script needed no edit and check:agent-test-spelling is green. ui/scripts/__tests__/check-i18n-dead-keys.test.ts cites ui/AGENTS.md 测试纪律 for the beforeAll / hookTimeout reasoning; that bullet (「不要用 beforeAll 预热」) stays in the core. ui/scripts/check-skills-paths.mjs (SCAN_ROOTS = skills, .claude/skills) now scans the moved block; the one root-relative token it would have refused (packages/console/dist) was re-spelled as the package name, and the gate reports 86/87 with its sole baseline entry unchanged. No script or test pins the old ui/CLAUDE.md wording; the four readers of that path (CODEOWNERS, governed-merges, governed-prose, queue-guard) match the path only.
• Gate verdicts — green. 64 check-runs on this head across two push runs: 44 success, 20 skipped, 0 failure, 0 in progress. Governed Surface Queue Guard, PR-body guards, Lint UI, Type Check UI, Test UI (16 shards), Build & E2E UI, Live E2E UI, Lint & Repo Gates, Lint · docs & code guards all success.

② Semver level

skip-changeset label present; Check Changeset success on the first run and skipped (label honoured) on the second. The diff publishes nothing: no packages/**, no ui/packages/**, no published skill. Clause-②: no on the PR body is right — no declared capability is narrowed at any consumer; the only consumer-visible text change is instruction prose for agents in this repository. Matches.

③ Boundary flags

• OQ1 (6498180390) — ui/CLAUDE.md edit — answered A, done. Addendum 6498382703 applied it at 1a085dc7, merged to ccc751c5. The sentence says what ruling 6407900763 item 1 asks: the core plus its linked pages are one instruction set. One wording point remains, ④ S3.
• OQ2 — §5 Convert to monorepo with scoped packages #8 path fix versus "27 byte-identical" — answered A, right. Ruling 6485892110 binds both "every ⛔ / never line stays whole and verbatim" and "the stale claims listed in 6455483141 fixed in the same PR"; 6455483141 item 3 names §5 Convert to monorepo with scoped packages #8 by its path. The census the rulings count is lines (27 = 27), and the body discloses the 9-byte difference and its sha256. A stale path inside a ⛔ line is the misdirection the ruling exists to remove; the fix wins.
• OQ3 — os-dev.md signature order versus post-stamped.mjs — outside this diff; escalated to the domain:devx seat. No file of this PR carries it.
• Core size — escalated to the approver, not a FAIL. Ruling 6485892110: "the dev measures the resulting core before editing and reports the number in the PR; a number far above the thread's ~20 KB estimate is re-presented, not acted on." The core is 32,591 bytes, about 1.6× the estimate. The number and its reasons are on the PR body's first screen (《What changes》, 《Acceptance notes》: the test-running subsection holds the dependent reader's counter-example and a back-reference; the i18n bullet states its rule inside the incident that proved it; §1–§8 are rule-bearing), but the seat dispatched without re-presenting. The tree itself holds letter A — every ⛔ / never line whole in the core, no cap — and the ruling's size clause addresses the maintainer, who is the Tier H approver of this very PR. This record names the gap so the approval is an informed one: approving this head accepts 32.6 KB as the core; asking for a smaller one means moving §1 Role & Product, §4, §6, §8 and the i18n bullet verbatim onto pages (none carries ⛔ / never), which is a further PR under the same letter.
• docs/postmortems/ not created — right. The ruling offers it as an alternative to the guides ("or"); the only incident write-up (the 前情 objectui#3010 line) went with its reasoning to flaky-tests.md.
• No instrument pointer added to an existing rule — accepted. Ruling item 1 asks for "each rule once, with its instrument pointer". Every rule that has an instrument already names it (vitest-invocation-guard, check:test-path-roots, no-dynamic-import-in-test-hook, the product-neutral ratchet test, check:new-line-citations); every rule without one says so per [WIP] Create a new release version #9 (chore: version packages #10, Migrate documentation site to Fumadocs with monorepo structure and shared content #11's card clause, Add comprehensive test suite for Zod schema validation #12, [WIP] Add Chinese version of the documentation #13). Adding a pointer to a census line would break "verbatim"; adding one elsewhere had nothing to point at.
• No code comment swept — confirmed. The file list is five prose files.

④ Design judgment

ui/AGENTS.md — judged against rulings 6407900763 items 1–2, 6435359095, 6485892110, reading 6455483141, and the code it cites on this head. Holds.

• The five stale claims of 6455483141 are each fixed and each fix is true in the tree: skills/objectui/rules/styling.md exists at the repository root and nowhere under ui/; ui/packages/app-shell/src/urlParams.ts exists; ui/packages/core/src/registry/Registry.ts exports class Registry with register(type, component, meta?), registerLazy(type, loader, meta?), get(type, namespace?) and the ComponentRegistry singleton exported as a new Registry(...), and the types the abridged snippet names (ComponentRenderer, RegistryComponentMeta, LazyComponentLoader) are exported from that file; SchemaRenderer.tsx resolves through ComponentRegistry.get(...); no @objectstack/plugin-msw package exists and no tracked package.json (128 scanned) declares msw — the lockfile carries msw@2.15.0 only inside vitest's optional-peer resolution suffixes, so "nothing in this workspace depends on msw" holds at the declaration level, which is the level a reader checks. The old :3001 claim is replaced by "the backend serves /_console on its own port (:3000 in the single stack)": serve defaults --port to 3000 and mounts the Console "served from @objectstack/console (default: true)"; dev forwards --ui only when set and never --no-ui, so pnpm dev / dev:crm inherit that default; @objectstack/console ships files: ["dist"] and pnpm console:build builds it. The :5180 / DEV_PROXY_TARGET || http://localhost:3000 claims match ui/apps/console/vite.config.ts.
• §5 Add comprehensive test suite for Zod schema validation #12 / [WIP] Add Chinese version of the documentation #13 carry ruling item 2 (i) and (ii) as written — one line of why plus a link, rationale in the ADR or issue; invariants not incident history, history on the PR or issue — with "no gate and no numeric budget" and "existing comments are not swept" stated. Neither is a census line.
• The two pointer lines resolve: .claude/skills/objectui-contributor/guides/flaky-tests.md exists; console-development.md § Local dev and per-agent stacks is a heading in that file.
• S1 — should-fix: the §7 pointer lands in a page that contradicts it. §7 now reads "No MSW integration … debug the Console against a real backend (the stacks in .claude/skills/objectui-contributor/guides/console-development.md)". The guide it names still carries «Debugging & Browser Simulation Strategy → Rule Add metamodel interfaces for ObjectQL/ObjectUI contract #1: Official MSW Integration», instructing @objectstack/plugin-msw and MSWPlugin, twenty lines above the new section; and the SKILL.md topic table still advertises that guide for "MSW debugging". ui/CLAUDE.md now declares the core and its linked pages one instruction set, and on this one claim the set disagrees with itself. The hierarchy resolves it (the core governs; the dev disclosed it in 《Acceptance notes》; the ruling's fix list names only the core's claims), so this is not a FAIL — but both files are already in this diff under the same Tier H review, and the fix is three lines: retitle Rule Add metamodel interfaces for ObjectQL/ObjectUI contract #1 to say there is no MSW plugin and point at the new section, and drop "MSW debugging" from the SKILL.md row. The surviving §7 census line "matches the mock server" stays verbatim under the ruling and is correctly left alone.
• Note, no finding: the core links two topic pages; rules/no-touch-zones.md is not linked from the core, so the "pages it links to" phrase in ui/CLAUDE.md does not reach it. The no-touch rule itself is stated in §5 Remove explicit pnpm version from workflows to fix version conflict #7, so nothing is lost.

ui/.claude/skills/objectui-contributor/guides/console-development.md — judged against the move it receives and the code the moved block cites. Holds. The 7-bullet 「Local dev」 block and the 4-step 「每个 agent 独立测试栈」 block are byte-identical to what left the core, with the one declared rewording (the /_console bullet) true as read above. ui/.claude/launch.json has the console and console-build-test entries the block names; ui/playwright.live.config.ts and ui/e2e/live/global-setup.ts exist; examples/app-showcase and examples/app-crm exist at the root with dev scripts that run objectstack dev. S1 applies to this file's own Rule #1. Pre-existing, out of scope, noted for the seat: the console-build-test launch entry the moved text tells a reader to imitate hard-codes a /Users/... path and ports 4000 / 5181 from the objectui checkout.

ui/.claude/skills/objectui-contributor/guides/flaky-tests.md — judged against the scenario it is accepted for (an agent facing a test that is green alone and red in the full run) and the rules left in the core. Holds. The measurement paragraph (976 ms against the 1000 ms budget), the two diagnostic bullets and the 前情 objectui#3010 line are byte-identical to the removed core lines; the two framing lines are new and say where the rules stayed. The core keeps every actionable rule of 测试纪律 (module-scope import, no beforeAll warm-up with its lint, no timeout raising or heavyDomTests, the i18n regex rule, stale build, no prettier --write), so a reader who never opens the page still has every instruction; the page adds only why. The SKILL.md row describes it accurately. No ⛔ / never on the page.

ui/.claude/skills/objectui-contributor/SKILL.md — one row, link resolves, description matches the page. Holds. S1 touches its pre-existing console row.

ui/CLAUDE.md — judged against ruling 6407900763 item 1 and the file it points to. Holds the ruling's letter: the core plus its linked pages are named one instruction set, and the objectui-era "this repo" is gone.

• S3 — should-fix, wording. "the single source of truth for working under ui/" overstates: ui/AGENTS.md's own first paragraph says it adds to the root AGENTS.md and the root wins where they disagree, and root AGENTS.md feat: Comprehensive CRM example demonstrating all ObjectStack protocol features #14 governs both. A reader of ui/CLAUDE.md alone is told of one source; the file it sends them to corrects that one click later, so the misdirection is short-lived. Suggested: "AGENTS.md is the source of truth for what differs under ui/ — it and the topic pages it links to are one instruction set, maintained in one place, under the root AGENTS.md; read it before your first edit." No rule is restated, as the ruling requires.

Must-fix: none. Should-fix: S1 (console-development.md Rule #1 and the SKILL.md "MSW debugging" row contradict the fixed §7), S3 (ui/CLAUDE.md "single source of truth" against the root-wins preamble). Escalated to the approver: the 32.6 KB core against the ruling's "~20 KB … re-presented" clause (③).

Implemented-by: claude/issue-25919-ui-agents-core
Reviewed-by: session_01L6bhgV4fMGJHSdSdx3u9mK

VERDICT: PASS

---

Generated by Claude Code

[objectstack-fleet]

domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW

Contract-review-handoff: triage — CONTRACT_REVIEW_TIER unavailable (HTTP 429 rate_limit on the reviewer model at this seat, 2026-10-29T17:11Z; a delta review is owed by the reviewer of record, session_01L6bhgV4fMGJHSdSdx3u9mK) · head 261449b

Delta review requested for record 6502595750 (PASS on ccc751c5f6), per contract-review.md: a commit that only lands findings is delta-reviewed by the same reviewer. The maintainer ruled A on analysis 6502668243 (recorded 6503029051): the 32,591 B core is accepted, and S1 and S3 are fixed in this PR.

The delta is ccc751c5..261449bc: commit 261449bc on top of merge 9e875b51 with origin/main. Dev addendum 6503287503 describes it:

• S1: guides/console-development.md § Debugging & Browser Simulation Strategy:
  • the intro's "mock mode" claim is removed;
  • Rule Add metamodel interfaces for ObjectQL/ObjectUI contract #1 is retitled «No MSW plugin — debug against a real backend» and points to «Local dev and per-agent stacks», with the no-hand-rolled-interceptors sentence kept verbatim;
  • the MSWPlugin Configuration bullet is removed;
  • the Rule ✨ Set up Copilot instructions #2 Alignment bullet now names the /api proxy target.
    The SKILL.md row now reads "debugging against a real backend".
• S3: ui/CLAUDE.md now carries the reviewer's suggested sentence.
• ui/AGENTS.md is byte-identical: 32,591 B, still 27 ⛔/never lines.

The PR body was updated to match in this act. needs:contract-review goes on in the same act. After a same-head PASS, the seat writes the seat record and requests os-zhuang and hotlong.

---

Generated by Claude Code

[objectstack-fleet]

triage#1 · os-sales · session_01L6bhgV4fMGJHSdSdx3u9mK

Contract-review-claim: session_01L6bhgV4fMGJHSdSdx3u9mK · head 261449b · 2026-10-29T17:38Z

---

Generated by Claude Code

[objectstack-fleet]

triage#1 · os-sales · session_01L6bhgV4fMGJHSdSdx3u9mK

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: 261449bcabd5674295df32395e32784b8cafd6ab
Local-runs: none
Scope: ui/.claude/skills/objectui-contributor/SKILL.md, ui/.claude/skills/objectui-contributor/guides/console-development.md, ui/.claude/skills/objectui-contributor/guides/flaky-tests.md, ui/AGENTS.md, ui/CLAUDE.md + ui/apps/console/vite.config.ts, ui/apps/console/.env.development, ui/packages/app-shell/src/providers/AdapterProvider.tsx, ui/scripts/check-skills-paths.mjs, .github/workflows/ci.yml (the "Skill Guide Path Check" step), AGENTS.md (root: the preamble's "AGENTS.md wins", Build & Test "Frontend (Studio UI)", Prime Directive #14), and, re-checked for drift on main only, the cited set of record 6502595750 (ui/packages/core/src/registry/Registry.ts, ui/packages/react/src/SchemaRenderer.tsx, ui/packages/app-shell/src/urlParams.ts, skills/objectui/rules/styling.md, packages/console/package.json, packages/cli/src/commands/serve.ts, packages/cli/src/commands/dev.ts, ui/.claude/launch.json, scripts/check-agent-test-spelling.mjs, ui/scripts/tests/check-i18n-dead-keys.test.ts, scripts/pm/check-governed-merges.mjs, ui/playwright.live.config.ts, ui/e2e/live/global-setup.ts)

Delta review of record 6502595750 (PASS on ccc751c5f6a7d5e7df6ebd83d837bde5ef8f3c99, this seat), requested by hand-off 6503384625 under ruling 6503029051 (letter A on analysis 6502668243: the 32,591 B core is accepted; S1 and S3 are fixed in this PR; the same reviewer delta-reviews). Inputs read: card #25919 body and all 23 comments (the rulings 6407900763 items 1–3, 6435359095, 6485892110 and 6503029051; reading 6455483141; dev reports 6498180390, 6498382703 and the delta addendum 6503287503); PR #27576 body (updated for this head), its file list and the net diff from the files endpoint; the 64 check-runs on this head; locally, git log and git diff between the two heads, with the merge of origin/main separated from the PR's own commit.

What moved, separated. ccc751c5..261449bc is two commits: merge 9e875b51 of origin/main 4278c3b5 (209 files against the PR's previous head, all of them main's; its diff against 4278c3b5 is exactly the PR's five files, and its diff against ccc751c5 on ui/AGENTS.md, ui/CLAUDE.md and ui/.claude/ is empty — main touched none of the hits), and one PR commit 261449bc, three files, +6/−7: guides/console-development.md § Debugging & Browser Simulation Strategy, one SKILL.md table row, and the one sentence of ui/CLAUDE.md. ui/AGENTS.md (32,591 B) and guides/flaky-tests.md (1,242 B) are byte-identical to ccc751c5. The net diff against the new merge-base is still five files, no new path.

① Derived judgments

• Tier — unchanged, right. The file list is the same five GOVERNED_SURFACES rows; Tier H under root AGENTS.md feat: Comprehensive CRM example demonstrating all ObjectStack protocol features #14. The PR body still asks for the maintainer's approval; this record lifts nothing by itself.
• Public surface — unchanged, right. The commit touches two internal-skill prose files and ui/CLAUDE.md; no package, schema, workflow or published skill. skip-changeset stays correct.
• Byte-preserving moves and the ⛔ / never census — stand as judged in 6502595750. ui/AGENTS.md did not change between the heads; grep -ciE '⛔|never' reads 27 on the new merge-base 4278c3b5 and 27 on the head. The S1 hunk sits above the moved 「Local dev and per-agent stacks」 section and leaves every moved line byte-identical (re-read in the net diff: the 7-bullet and 4-step blocks are unchanged).
• Main's merge broke nothing the previous record judged. The cited files that record read — Registry.ts, SchemaRenderer.tsx, urlParams.ts, styling.md, packages/console/package.json, serve.ts, dev.ts, vite.config.ts, launch.json, check-agent-test-spelling.mjs, check-skills-paths.mjs, check-i18n-dead-keys.test.ts, check-governed-merges.mjs, root AGENTS.md, playwright.live.config.ts, global-setup.ts — are byte-identical between the old merge-base 1ed33609 and the new one 4278c3b5, so every claim the core and the guide make about them holds exactly as before.
• Dependent readers — right. No script, test or workflow pins the old wording («Official MSW Integration», "MSW debugging", "mock mode", MSWPlugin): a tree-wide grep finds those tokens only in CHANGELOGs, release notes, plugin-auth comments and the published skills/objectui/guides/data-integration.md and its eval (out of this card's scope, carried by the seat per 6498382703). ui/scripts/check-skills-paths.mjs scans the edited guide: the commit removes two stated tokens (/api/v1, @objectstack/plugin-msw) and adds package.json, which resolves under ui/; the gate runs in CI as the "Skill Guide Path Check" step of ci.yml, and every UI lane on this head is success. The four readers of ui/CLAUDE.md (CODEOWNERS, governed-merges, governed-prose, queue-guard) match the path only.
• Gate verdicts — green. 64 check-runs on this head: 44 success, 20 skipped, 0 failure, 0 in progress. Required contexts: Lint & Repo Gates, TypeScript Type Check, Test Core, Governed Surface Queue Guard, Lint UI, Type Check UI, Test UI (16/16 shards), Build & E2E UI, Live E2E UI all success; Build Core is skipped by its paths filter on this docs-only diff, as it was on ccc751c5 (same 44/20 split). PR-body guards and Lint · docs & code guards success. Check Changeset skipped with the label honoured.

② Semver level

Stands as judged in record 6502595750: skip-changeset, Clause-②: no, Metadata-protocol: none; the diff publishes nothing and narrows no declared capability at any consumer. The delta adds no path that could change that.

③ Boundary flags

• Core size — closed by ruling. Record 6502595750 escalated the 32,591 B core against 6485892110's "re-presented, not acted on" clause. The seat re-presented it (6502668243) and the maintainer ruled A (6503029051, 「A 收 32.6KB+本PR修 (Recommended)」): the core is accepted at this size. Nothing is owed on it.
• S1 and S3 — ordered fixed in this PR, done. Judged in ④.
• OQ1, OQ2 (6498180390) — stand as judged (A, A). OQ3 stays escalated to the domain:devx seat; no file of this PR carries it.
• Addendum 6503287503 — no open questions. Its one out-of-scope finding (ui/apps/console/.env.* and vercel.json still set VITE_USE_MOCK_SERVER, which no source reads; docs/deployment.md lists a vendor-msw chunk) is outside this diff and correctly left for the seat. Confirmed in the tree: no .ts/.tsx under ui/ reads USE_MOCK; the only non-env readers are a test that pins the value false and a playwright config comment.
• No code comment swept, no root AGENTS.md edit, docs/postmortems/ not created, no instrument pointer added to a census line — all stand as judged.

④ Design judgment

ui/.claude/skills/objectui-contributor/guides/console-development.md — S1 judged afresh against the core's §7 it must agree with, ruling 6503029051's wording, and the code it cites. Holds; S1 is closed. The section now reads: intro "When debugging the Console in a browser"; «Rule #1: No MSW plugin — debug against a real backend», whose one bullet says there is no @objectstack/plugin-msw package and no tracked package.json declares msw, points at 「Local dev and per-agent stacks」 "below", and keeps the sentence "Do NOT write custom fetch interceptors or manual mock servers unless absolutely necessary." verbatim; the MSWPlugin Configuration bullet is gone; Rule #2 Alignment says the client's baseUrl should match "the backend the Console's /api proxy targets (DEV_PROXY_TARGET, http://localhost:3000 by default)"; Rule #3 is untouched. Each claim is true in the tree: no plugin-msw directory exists and none of the 128 tracked package.json files names msw; ui/apps/console/vite.config.ts proxies '/api' to process.env.DEV_PROXY_TARGET || 'http://localhost:3000' on port 5180; the heading ## Local dev and per-agent stacks is in this file, after the section; the Console has no mock mode (VITE_USE_MOCK_SERVER is set to false in both .env.* files and read by nothing). The three edits beyond the literal retitle — the intro, the dropped Configuration bullet and the Alignment bullet — are the same false premise S1 named (a mock server that does not exist) and could not stay under the new title; they add nothing outside the finding. The core's §7 and this page now say the same thing in the same direction. Note, no finding: the Console's client baseUrl is import.meta.env.VITE_SERVER_URL || '' (AdapterProvider.tsx), and .env.development leaves it empty on purpose, so by default the client is same-origin and the proxy decides the backend — the Alignment sentence is true as written; naming VITE_SERVER_URL as the knob would make the check one step more concrete, which a later edit of this page may do.

ui/.claude/skills/objectui-contributor/SKILL.md — the console row now reads "debugging against a real backend"; it describes the page it links to, and the link resolves. The flaky-tests row is unchanged and stands as judged. Holds; the S1 half on this file is closed.

ui/CLAUDE.md — S3 judged afresh against ruling 6407900763 item 1 and the files it sits between. Holds; S3 is closed. The sentence is the reviewer's suggested text verbatim: "AGENTS.md is the source of truth for what differs under ui/ — it and the topic pages it links to are one instruction set, maintained in one place, under the root AGENTS.md; read it before your first edit." It agrees with ui/AGENTS.md's own preamble ("This file adds to the root AGENTS.md; it does not replace it … Where the two disagree, the root file wins") and with the root file (its preamble "AGENTS.md wins" and its Frontend section "ui/AGENTS.md adds that tree's own rules to this file's"). It names the core plus its linked pages as one instruction set, as the ruling asks, and restates no rule. The earlier note stands: rules/no-touch-zones.md is reached from SKILL.md, not from the core, and the no-touch rule itself is in §5 #7, so nothing is lost.

ui/AGENTS.md — byte-identical to the head of record 6502595750; stands as judged there in full (the five stale-claim fixes true in the tree, #12/#13 carrying ruling item 2, both pointer lines resolving, 27 = 27). One consequence of S1 to record, no finding: §7's surviving census line "Verify the client baseUrl matches the mock server." is now the only sentence in the set that still names a mock server; it stays verbatim under 6485892110, the page it points at is the truer text, and the previous record already judged it correctly left alone.

ui/.claude/skills/objectui-contributor/guides/flaky-tests.md — byte-identical to the previous head; stands as judged in record 6502595750.

Must-fix: none. Should-fix: none — S1 and S3 of record 6502595750 are closed on this head, and the delta introduces nothing new. The size escalation of that record is resolved by ruling 6503029051.

Implemented-by: claude/issue-25919-ui-agents-core
Reviewed-by: session_01L6bhgV4fMGJHSdSdx3u9mK

VERDICT: PASS

---

Generated by Claude Code

[objectstack-fleet]

domain:devx#2 · os-justin · session_01QSeKYmTYfaPbLQD4BJfomW

席位核对 — Tier H · awaiting approval · domain:devx#2 · head 261449bcab · 2026-10-29T17:51Z

维护者速读

• 改了什么：ui/AGENTS.md 收成只写规则的核心，从 36,140 B 减到 32,591 B。本地开发、端口、多栈流程原样搬进 console-development.md，flaky 测试的成因和读数原样搬进新建的 guides/flaky-tests.md，核心里各留一行指针。§5 新增两条写法规则（Add comprehensive test suite for Zod schema validation #12、[WIP] Add Chinese version of the documentation #13），不加门禁。修了 5 处过期说法。另按契约复核修了两处：console 指南不再教并不存在的 MSW 插件（S1）；ui/CLAUDE.md 改为「在根 AGENTS.md 之下」的措辞（S3）。
• 为什么改：执行你在 docs: narrative cost — 42.5% of source lines are comments, workflows 58.6%, AGENTS.md is 103 KB with three stale claims, ROADMAP.md still says v3.3.2 and 80 Storybook stories #25919 的裁决（6435359095、6485892110）：核心只写规则，叙事放主题页，过期说法会把 agent 带错路。核心大小和两条修正，按你刚才选的 A 定案（6503029051）。
• 风险与回滚：只改文档，5 个文件，+83/−64，不动代码，不发布任何东西。27 行 ⛔ / never 全部留在核心：26 行逐字节不变，第 27 行（§5 Convert to monorepo with scoped packages #8）只按派发令补了 packages/ 路径前缀。回滚 = 还原本 PR。
• 先看：① ui/AGENTS.md §9 的两行新指针，以及 §5 的 Add comprehensive test suite for Zod schema validation #12、[WIP] Add Chinese version of the documentation #13；② console-development.md 末尾的「Local dev and per-agent stacks」和「Debugging & Browser Simulation Strategy」的 Rule Add metamodel interfaces for ObjectQL/ObjectUI contract #1；③ guides/flaky-tests.md；④ ui/CLAUDE.md 那一句。
• 两分钟验证：在仓根跑 git diff origin/main...pr-head --color-moved=dimmed-zebra -- ui，搬迁的行应显示为 moved；没标 moved 的增删行，只有 PR 正文列出的修正行和新写的指针、规则行。
• 需要你做的：批准本 PR（os-zhuang 或 hotlong 任一位即可）。达档契约复核在这个 head 上是 PASS（评论 6504218316，Head-sha 261449bcabd5674295df32395e32784b8cafd6ab；它是上一轮 6502595750 的 delta 复核）。批准后席位翻 ready、挂自动合并。

席位意见：批准合入。两轮达档复核都是 PASS，最后一轮没有任何 should-fix。核心 32.6 KB 是在「⛔ 行必须留在核心」这条约束下能做到的结果，已按你的裁决接受。席位派发时漏掉了「远超 ~20 KB 要拿回给你看」这一步，这次已补上并请你裁过（6502668243）。

Shape: draft → main, Part of #25919 (PR 2 of the card; PR 1 landed as #26023). 5 files, +83/−64: ui/AGENTS.md, ui/CLAUDE.md, ui/.claude/skills/objectui-contributor/SKILL.md, guides/console-development.md and the new guides/flaky-tests.md. Tier H on all five (ui/AGENTS.md, ui/CLAUDE.md, ui/.claude/**). Labels documentation, size/m, skip-changeset. No assignees. In this act, reviewers are requested from GOVERNED_APPROVERS (os-zhuang, hotlong), needs-user-decision goes on the PR, and needs:contract-review comes off. Dev reports 6498180390, 6498382703 and 6503287503; claim 6497608495.

Premise measured:

• The rulings are 6407900763 items 1–3, 6435359095 and 6485892110, with the reading 6455483141.
• The size clause of 6485892110 was re-presented as analysis 6502668243 and ruled A, recorded as 6503029051, quoted verbatim: 「A 收 32.6KB+本PR修 (Recommended)」.
• Measured at this head, §5 and §9 together are 23.9 KB, so the ~20 KB estimate is unreachable while every ⛔ line stays in the core.

Contract review: both rounds at CONTRACT_REVIEW_TIER, by triage's isolated reviewer (session_01L6bhgV4fMGJHSdSdx3u9mK).

• 6502595750 on ccc751c5: PASS, with should-fixes S1 and S3, and the size question escalated to you.
• Delta 6504218316 on 261449bc: PASS. S1 and S3 are closed, no new finding, and ui/AGENTS.md and flaky-tests.md are byte-identical across the delta.
• The lane's own attempt at that tier was refused with HTTP 429 (handoff 6498673845). The seat did not review at a lower tier.

Gates:

• On head 261449bc: 64 check-runs, 44 success, 20 skipped, 0 failure; mergeable_state: clean. Lint UI, Test UI, Build & E2E UI, Live E2E UI and Governed Surface Queue Guard are all success.
• check-expected-skips --pr 27576: OK, all 20 skips are in main's roster (exit 0).
• check-governed-merges --pr 27576: governed, 5 of 5 paths; 147 changed lines, under the 3,000-line floor.
• The dev's union on this head: 13 derived, 13 run, all exit 0, plus check:pm-governed-prose and ui check:skills-paths.

Open questions:

• OQ1 → A (ui/CLAUDE.md edit, now S3's wording).
• OQ2 → A (§5 Convert to monorepo with scoped packages #8 path fix over 27 byte-identical).
• OQ3 (the os-dev.md signature order against post-stamped.mjs) is outside this diff and stays with the seat.

Accepted deviations: the ui/CLAUDE.md surface was widened by the seat for that one file. S1 also removed the guide's "mock mode" intro and the MSWPlugin Configuration bullet, which the review judged the same false premise and not new scope.

Out of scope, with destinations:

• The published skills/objectui/guides/data-integration.md and its eval still teach @objectstack/plugin-msw; the console's dead VITE_USE_MOCK_SERVER / vendor-msw remnants are in the same family → skills/objectui: the published data-integration guide and its eval teach @objectstack/plugin-msw / MSWPlugin, a package removed in 12.0 whose last release is 11.0.0 #27678.
• ui/.claude/launch.json's console-build-test entry hard-codes a /Users/... objectui path → ui/.claude/launch.json: the console-build-test entry that ui/AGENTS.md tells agents to imitate hard-codes a /Users/... objectui-checkout path and ports 4000 / 5181 #27668.
• §7's surviving census line still says "mock server", and stays verbatim under 6485892110.
• Skills-docs: n/a (internal agent instructions only).

Landing: the seat never approves or merges a governed PR. On an approval from GOVERNED_APPROVERS, the seat flips it ready and arms auto-merge, unless the approver did. On merge, the seat verifies the merged diff against this head, posts the landing note on #25919 and archives cloud session session_01QSeKYoC3JrEqdXJwZ69A8m. The card then closes or stays open per its remaining items.

---

Generated by Claude Code