docs(skills): objectstack-ui puts the create/edit form under form, not a named formViews entry

[objectstack-fleet]

Fixes #21537
Clause-②: no

What this changes

ViewSchema.form is a container's "Default form view" and formViews are "Additional named form views" (packages/spec/src/ui/view.zod.ts lines 4782 and 4784 at origin/main 15fe567). Since PR #21535 landed, expandViewContainerWithDiagnostics serves form as the one default form and promotes no named form in its place, so a container whose only form is a named formViews entry declares no default form. Three positions in the published objectstack-ui skill taught exactly that shape. Each now authors the create/edit form under form:

1. skills/objectstack-ui/SKILL.md — the subforms example (option 2 under entering a record together with its child line items): formViews with a default key becomes form, one nesting level fewer; the inline comments are kept.
2. skills/objectstack-ui/rules/list-views.md — the os:check marked defineView container: the formViews block with an edit key becomes form, placed after list in the schema's own declaration order (list, form, listViews, formViews); the callout now reads "Every view lives under list / form / listViews / formViews".
3. skills/objectstack-ui/evals/views-apps-actions-pages.json — eval 1's expected_output now expects defineView({ list, form, listViews }) and "form with type: 'simple'". Its must_contain / must_not_contain reference no changed spelling and are untouched.

Every formViews occurrence in skills/objectstack-ui/** at the base, and its disposition:

Location	Text	Disposition
SKILL.md:14	"formViews type: 'wizard'"	legitimately named (wizard) — unchanged
SKILL.md:77	subforms example, formViews with default	position 1 — moved to form
SKILL.md:207	"Define default list + form, then named listViews / formViews"	the control row — unchanged; that table row is held by the in-flight sibling #21567, which is not addressed here
rules/list-views.md:14	"aggregating the default list, named listViews, and formViews"	outside the claimed region (the container and the callout) — unchanged; see Acceptance notes
rules/list-views.md:29	container formViews with edit	position 2 — moved to form
rules/list-views.md:38	callout	position 2 — names form now
evals/views-apps-actions-pages.json:7	eval 1 expected output	position 3 — expects form
evals/views-apps-actions-pages.json:57 and :60	wizard eval and its must_contain	legitimately named (wizard) — unchanged

Parse proof (built spec at a864ccc)

expandViewContainerWithDiagnostics('support_case', container) imported from packages/spec/dist, on the list-views.md container before and after:

• before (formViews with edit, no form): forms served = support_case.edit, isDefault absent; collisions none
• after (form): forms served = support_case.form, isDefault: true; collisions none

Readings

Whole-file lines and ratchet tokens, before → after (base 15fe567 → a864ccc):

File	Lines	Tokens	Ceiling (headroom before → after)
skills/objectstack-ui/SKILL.md	310 → 308	3854 → 3843	3856 (2 → 13)
skills/objectstack-ui/rules/list-views.md	302 → 300	3068 → 3065	3141 (73 → 76)
skills/objectstack-ui/evals/views-apps-actions-pages.json	65 → 65	1505 → 1502	1505 (0 → 3)

Whole package (sum of every skills/**/SKILL.md): 4397 → 4395 lines. No ceiling moved. No changeset: the diff publishes nothing from a released package (skill text only), so skip-changeset applies.

Verification (all at a864ccc; every exit code captured before any pipe)

• pnpm --filter @objectstack/spec build — exit 0 (under scripts/pm/os-verify-lock.sh, "VERDICT command-exit 0").
• pnpm --filter @objectstack/spec run check:skill-examples — exit 0: "260 prose examples type-check across 3 surface(s) — every marked block parsed, so tsc ran the SEMANTIC pass on all of them"; the edited os:check container is among them. A first run exited 3 (PREREQUISITE NOT MET: packages/client-react/dist unbuilt, nothing measured); @objectstack/types, core, client and client-react were then built directly and the gate rerun.
• pnpm --filter @objectstack/spec run check:skill-docs — exit 0 (skills/README.md, content/docs/ai/skills-reference.mdx in sync); check:skill-refs — exit 0 ("9 generated files in sync").
• node scripts/check-skills-token-ratchet.mjs — exit 0: "54 authored bundle file(s) within their ceilings"; --self-test exit 0 (65 cases).
• node scripts/pm/dispatch-gates.mjs --commands with no paths derived 24 families from the diff at a864ccc; all 24 run, every exit 0; --ran reconciliation: "24 derived famil(ies) accounted for — 24 run, 0 NOT-MEASURED". The derived set equals the dispatch's list minus check:skill-examples, which was run anyway. pnpm --filter @objectstack/lint run check:doc-formula-expressions exited 3 (PREREQUISITE NOT MET) on the first run and 0 after @objectstack/formula and @objectstack/lint were built. pnpm check:nul-bytes exit 0.
• Repo-wide pnpm lint is CI's run, not run locally (declared narrowing: the diff is Markdown and JSON only, no ESLint-governed file).

维护者速读(草稿)

改了什么。 发布的 objectstack-ui 技能有三处把「新建/编辑表单」写在 formViews 的具名条目下(default、edit)。本 PR 把这三处改为容器的 form 键:SKILL.md 的子表单示例、list-views.md 的 defineView 示例及其提示框、评测文件第一条的预期输出。向导(wizard)表单是真正的附加具名表单,保持不动。

为什么改。 spec 把 form 定义为容器的默认表单、formViews 定义为附加的具名表单。PR #21535 落地后,运行时只把 form 当默认表单,不再把第一个具名表单提升为默认。技能照旧教的话,AI 作者按示例写出的应用在规范层面没有默认表单,只靠前端的兜底才显得正常。按 NORTH-STAR 优先级规则 4,写给 AI 的技能说错一句等于产品缺陷;两者应随同一个版本发布。

风险与代价(含回滚)。 纯技能文本,不涉及代码与发布包,无 changeset。三个文件的 token 读数均净减(见上表),没有抬任何上限;check:skill-examples 证明改后的示例仍能编译,并用构建后的 spec 解析验证了改后容器的 support_case.form 被标为默认。回滚即 revert 本 PR 的单个 commit。残余:list-views.md 第 14 行的引言仍只列出 list / listViews / formViews(在认领的区域之外),见 Acceptance notes。

席位意见。

你要做的。 skills/** 是 Tier H 受管面:需要你的一次 APPROVED review;之后由席位落地。PR 保持 draft,不翻 ready、不挂 auto-merge。

Acceptance notes

• rules/list-views.md:14 ("aggregating the default list, named listViews, and formViews") omits form the same way the callout did. It sits outside the claimed region (the container and the callout), so it is untouched here; folding it in costs about four tokens against a headroom of 76 and is reported to the seat.
• Eval 1's must_contain does not assert the form key (it never asserted the form key before either); adding "form:" would fit the remaining headroom of 3 tokens but is beyond the card's stated fix and is left to the seat.
• Ordering: the dispatch's suggested route placed form after listViews; this PR follows the schema's declaration order (list, form, listViews, formViews) in the container, the callout and the eval, which is also the order of SKILL.md's "Define default list + form, then named listViews / formViews" row.
• Sibling skills(objectstack-api): the published public-form opt-in names two of the three sharing keys the anonymous form endpoints require — an AI following it authors a form both endpoints answer 404 #21567 edits the same SKILL.md at the assembly table row; the regions are disjoint, origin/main was still at the base when this PR opened (no merge needed), and a second merge follows once that sibling lands.

---

Generated by Claude Code

[objectstack-fleet]

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: d66ec0254f5c7a0cb5a1ef0ab2e7c2ed1f9a3c70
Local-runs: none

Read-only shape: the diff against the merge base 15fe567 (two commits, a864ccc and the patch-round commit d66ec02), card #21537 with every comment, and this head's check-runs; nothing built or run locally. Reviewed by the dispatch seat in seat (served tier equals the constant's value, read from get_session). Review face: skills/**, governed rule text (Tier H). Readings taken at 2026-10-04T00:29Z.

① Derived judgments

• Accept set: unchanged. Prose and example code in three published skill files (skills/objectstack-ui/SKILL.md +8/−10, rules/list-views.md +3/−5 plus the patch round's +2/−2, evals/views-apps-actions-pages.json +1/−1); no schema, export, error code, route or runtime behaviour moves. Clause-②: no holds.
• The three ruled positions: the subforms example now authors the create/edit form under form (one nesting level fewer, inline comments kept); the os:check container now declares form after list, in ViewSchema's declaration order, and the callout names list / form / listViews / formViews; eval 1 expects defineView({ list, form, listViews }) and form with type: 'simple', its must_contain unchanged (it never named formViews). The wizard eval (:57 / :60) and the SKILL.md:207 control row are untouched, as ruled.
• The patch round (seat ruling 5974903312): the file's intro sentence now reads "the default list and form, named listViews, and formViews" — the same family as the callout, one word, re-wrapped at the original width; the os:check block is untouched by it.
• Correctness against origin/main: ViewSchema.form is the default form view and formViews the additional named ones (packages/spec/src/ui/view.zod.ts); PR fix(spec): a view container's form is its default form — never collapsed into a named form, and no named form is promoted #21535 (merged) serves form as the one default and promotes no named form. The dev's parse proof on the built spec (before: support_case.edit, no isDefault; after: support_case.form, isDefault: true) matches that contract.

② Semver level

• No released package publishes from this diff; no changeset owed; skip-changeset is the correct declaration. No ADR-0087 disposition applies.

③ Boundary flags

• Dev flags: open_questions empty both rounds; the residue it surfaced was taken by the patch round; the must_contain note stays as the seat ruled (beyond the card's fix).
• Ratchet, read off the head blobs by this seat against the base 15fe567: SKILL.md 310 → 308 lines, 3854 → 3843 tokens (ceiling 3856); rules/list-views.md 302 → 300 lines, 3068 → 3068 tokens (ceiling 3141); evals/views-apps-actions-pages.json 65 → 65 lines, 1505 → 1502 tokens (ceiling 1505); catalog SKILL.md sum 4397 → 4395. No ceiling moved; the dispatch's net ≤ 0 bound holds in the two zero-headroom files.
• origin/main has moved to 1968d5e since the base; git diff --stat 15fe567c origin/main -- skills/ is empty, so nothing on the review face moved under this PR.
• Check-runs on this head at this write: Governed Surface Queue Guard success; Lint & Repo Gates, the Type Check lanes and Test Core 1/6, 2/6 in progress — the enqueue gate reads them at landing, not this record.
• Cross-lane: the sibling PR docs(skills): name the complete public-form opt-in the anonymous form endpoints read #21650 (skills(objectstack-api): the published public-form opt-in names two of the three sharing keys the anonymous form endpoints require — an AI following it authors a form both endpoints answer 404 #21567) edits the same skills/objectstack-ui/SKILL.md at the :208 table row, disjoint from this diff's region; serial landing with a base merge after the first lands.

Implemented-by: claude/issue-21537-ui-skill-form-default
Reviewed-by: session_01CB6W87z22K2yjUCDyVrJRk

VERDICT: PASS

[objectstack-fleet]

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: d66ec025b124dbf47f735506a83aee848532955c
Local-runs: none

Supersedes record 5975004740, whose Head-sha: line carried a mis-expanded 40-hex value (the first eight characters were right, the rest were not read from git); this record names the head as git rev-parse and the PR object both answer it, and is otherwise the same review. Read-only shape: the diff against the merge base 15fe567 (two commits, a864ccc and the patch-round commit d66ec02), card #21537 with every comment, and this head's check-runs; nothing built or run locally. Reviewed by the dispatch seat in seat (served tier equals the constant's value, read from get_session). Review face: skills/**, governed rule text (Tier H). Readings taken at 2026-10-04T00:43Z.

① Derived judgments

• Accept set: unchanged. Prose and example code in three published skill files (skills/objectstack-ui/SKILL.md +8/−10, rules/list-views.md +3/−5 plus the patch round's +2/−2, evals/views-apps-actions-pages.json +1/−1); no schema, export, error code, route or runtime behaviour moves. Clause-②: no holds.
• The three ruled positions: the subforms example now authors the create/edit form under form (one nesting level fewer, inline comments kept); the os:check container now declares form after list, in ViewSchema's declaration order, and the callout names list / form / listViews / formViews; eval 1 expects defineView({ list, form, listViews }) and form with type: 'simple', its must_contain unchanged (it never named formViews). The wizard eval (:57 / :60) and the SKILL.md:207 control row are untouched, as ruled.
• The patch round (seat ruling 5974903312): the file's intro sentence now reads "the default list and form, named listViews, and formViews" — the same family as the callout, one word, re-wrapped at the original width; the os:check block is untouched by it.
• Correctness against origin/main: ViewSchema.form is the default form view and formViews the additional named ones (packages/spec/src/ui/view.zod.ts); PR fix(spec): a view container's form is its default form — never collapsed into a named form, and no named form is promoted #21535 (merged) serves form as the one default and promotes no named form. The dev's parse proof on the built spec (before: support_case.edit, no isDefault; after: support_case.form, isDefault: true) matches that contract.

② Semver level

• No released package publishes from this diff; no changeset owed; skip-changeset is the correct declaration. No ADR-0087 disposition applies.

③ Boundary flags

• Dev flags: open_questions empty both rounds; the residue it surfaced was taken by the patch round; the must_contain note stays as the seat ruled (beyond the card's fix).
• Ratchet, read off the head blobs by this seat against the base 15fe567: SKILL.md 310 → 308 lines, 3854 → 3843 tokens (ceiling 3856); rules/list-views.md 302 → 300 lines, 3068 → 3068 tokens (ceiling 3141); evals/views-apps-actions-pages.json 65 → 65 lines, 1505 → 1502 tokens (ceiling 1505); catalog SKILL.md sum 4397 → 4395. No ceiling moved; the dispatch's net ≤ 0 bound holds in the two zero-headroom files.
• origin/main has moved to 1968d5e since the base; git diff --stat 15fe567c origin/main -- skills/ is empty, so nothing on the review face moved under this PR.
• Check-runs on this head at this write: Governed Surface Queue Guard success; Lint & Repo Gates, the Type Check lanes and Test Core 1/6, 2/6 in progress — the enqueue gate reads them at landing, not this record.
• Cross-lane: the sibling PR docs(skills): name the complete public-form opt-in the anonymous form endpoints read #21650 (skills(objectstack-api): the published public-form opt-in names two of the three sharing keys the anonymous form endpoints require — an AI following it authors a form both endpoints answer 404 #21567) edits the same skills/objectstack-ui/SKILL.md at the :208 table row, disjoint from this diff's region; serial landing with a base merge after the first lands.

Implemented-by: claude/issue-21537-ui-skill-form-default
Reviewed-by: session_01CB6W87z22K2yjUCDyVrJRk

VERDICT: PASS

[objectstack-fleet]

维护者速读(终稿)— PR #21651(#21537)· skills seat 1 · 2026-10-04T00:43Z

改了什么: 发布的 objectstack-ui 技能有三处把「新建/编辑表单」写在 formViews 的具名条目下(default、edit):SKILL.md 的子表单示例、rules/list-views.md 的 defineView 示例及其提示框、评测文件第一条的预期输出。三处都改为容器的 form 键(spec 里 form 就是默认表单,formViews 是附加的具名表单);补丁轮又把 list-views.md 开头那句「aggregating the default list, named listViews, and formViews」补上 form。向导(wizard)表单是真正的附加表单,保持不动。

为什么改: PR #21535 落地后,运行时只把 form 当默认表单,不再把第一个具名表单提升为默认;技能照旧教的话,AI 作者按示例写出的应用在规范层没有默认表单,只靠前端兜底才显得正常。Version Packages PR(17.7.0)已带上 #21500 的改动,技能须与它同版发运(p1 的由来)。

风险与代价(含回滚): 纯技能文本,无 changeset(skip-changeset)。三个文件 token 均净减(SKILL.md 3854 → 3843、list-views 3068 → 3068、evals 1505 → 1502),没有抬上限;check:skill-examples 证明改后的示例仍可编译,构建后的 spec 解析显示容器现在服务 support_case.form 且 isDefault: true。席内契约复核 PASS(5975004740),必查门禁已绿。回滚 = revert 两个 commit(a864ccc、d66ec025)。

席位意见: 建议批准。示例顺序按 ViewSchema 的声明序(list、form、listViews、formViews),与 SKILL.md 既有的那行规则「Define default list + form, then named listViews / formViews」一致;评测第一条的 must_contain 未加 form:(超出卡的既定修复,留作后续)。与 PR #21650(#21567)共用 skills/objectstack-ui/SKILL.md 但区域不相交,先批哪个都可以,席位串行落地。

你要做的(一个动作): 在 PR #21651 上给一次 APPROVED review;批准后由席位清标、ready、挂 auto-merge 入队。