Repository navigation
docs(platform-objects): name the slotted page that declares the org record tab strip - #17750
Merged
Merged
Conversation
…ecord tab strip Three documents assert that the organization record page opens on a Members tab with Invitations and Teams beside it, and none of them said where that comes from. A measurement over `packages/platform-objects/src/identity/` found no `Field.relatedList` prominence key and read the absence as "declared by no metadata" — a two-way disjunction between missing declarations and stale prose. It is declared. `SysOrganizationDetailPage` (`packages/platform-objects/src/pages/sys-organization.page.ts`) is a `kind: 'slotted'`, `isDefault: true` record page for `sys_organization`, handed to the runtime by plugin-auth's `pages: [...]`. Its `slots.tabs` override carries exactly three `record:related_list` tabs — Members, Invitations, Teams — and objectui's synthesizer pushes the authored node instead of calling `buildDefaultTabs`, so it replaces the synthesized Details + stacked `Related` strip outright and Members is at index 0. `relatedList: 'primary'` is a different mechanism and would not move this page while the `tabs` slot is present. Prose only. No metadata changes, no behaviour changes. The two source comments and the QA checklist item now name the declaring page; the checklist item takes the revision and history bump its ledger requires, with its steps, acceptance clauses, oracles and negatives unchanged. Co-Authored-By: Claude <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01MkQhmuuJAVDjmeWNixwDDH
Contributor
📓 Docs Drift CheckNothing in this diff resolved to a documentable surface (no symbol, route or SDK anchor derived from 1 changed package(s)), so this run has no opinion about the docs. What this run could not see
Coarse fallback — 3 page(s) merely mention a changed package (the pre-#9192 predicate, kept for the deliberately-wide backstop): |
os-bill
marked this pull request as ready for review
September 12, 2026 02:17
This was referenced Sep 12, 2026
akarma-synetal
pushed a commit
to akarma-synetal/framework
that referenced
this pull request
Sep 17, 2026
…correct the residue and D3's pointer (objectstack-ai#17808) Fixes objectstack-ai#16270 **Why `Fixes` and not `Part of`.** This is the card's last residue. Its provenance half landed in PR objectstack-ai#17750 (the three documents that asserted the tab strip now name the record that declares it); the seat's merge comment recorded that the card stayed open carrying exactly one correction — this ADR. With it landed, nothing of objectstack-ai#16270 remains. - **Clause-②: no** — this PR puts no new key on any published payload.⚠️ **This PR is GOVERNED and stays draft.** `docs/adr/**` is a governed surface (AGENTS.md Prime Directive 14). ⛔ No seat flips it ready, enqueues it, or arms auto-merge; a human merge is the review record. That is the expected end state, not a problem to route around. --- ## What was wrong ADR-0133's "What this record does not decide", item 1, asserted **in the present tense** that "Opens on tab-0 Members" is *"declared by none of its metadata"*, and that the tab ordering is *"either an emergent property of the renderer or a claim that has gone stale."* Both halves are false, and were already false on the day they were written. **The tab strip is declared metadata.** `packages/platform-objects/src/pages/sys-organization.page.ts` exports `SysOrganizationDetailPage` — `type: 'record'`, `kind: 'slotted'`, `isDefault: true`, for `sys_organization` — whose `slots.tabs` override carries exactly three `record:related_list` tabs: | order | label | objectName | |:--|:--|:--| | 0 | Members | `sys_member` | | 1 | Invitations | `sys_invitation` | | 2 | Teams | `sys_team` | plugin-auth hands it to the runtime — `auth-plugin.ts` imports it and declares `pages: [SysOrganizationDetailPage, SysUserDetailPage]`. It is registered, not a dead export. **It already existed at `77781151d`, the very commit the ADR cites.** The contents API for that path at that ref answers **HTTP 200**, blob `2f56173ff2c84b1ee9fb3324577df71fd78d9b3b`, 4606 bytes. **Lit control:** the same call for a fabricated sibling path at the same ref answers **Not Found**. So the 200 is a reading. ## Why a correct measurement produced a wrong conclusion ⭐ **This is the reusable part, and it is what the correction carries.** The ADR's two greps reproduce **exactly** on my own head, with their own lit control — they were never the error: - `git grep -rn relatedList -- packages/platform-objects/src/identity/` — the two hits on today's tree are the *provenance comments* PR objectstack-ai#17750 added; **zero declarations**, which is what the sentence claimed. Read the sites, not the count. - Lit control, same sweep: `packages/drivers/driver-sql/src/builtin-column-collision.ts` still hits, so the pattern matches something. - `relatedLayout` — 7 hits, all prose (3 in ADR-0085, 1 here, 1 in `field.zod.ts`'s doc comment, 2 in a SKILL.md). Zero object declarations. The search was **exhaustive over the wrong space**: scoped to `src/identity/` and to the `relatedList` key, while the declaration lives one directory over in `src/pages/`, written in a different vocabulary — an **assigned Page**, not a field prominence key. A control proves a probe reaches; it cannot prove the probe is aimed at the right place.⚠️ **Neither branch of the disjunction was even available**, measured at the objectui sha this repo pins (`.objectui-sha` = `53ded82bf7a494f54e344e19099dbf00854b8694`, read with `git show`, ⛔ nothing there edited): - `buildDefaultTabs` seeds `items[0]` with `{ label: 'Details', value: 'details', … }` **unconditionally**, so a promoted related list could never be tab-0. - `buildDefaultPageSchema` never calls it at all when an assigned page supplies a `tabs` slot: `if ('tabs' in slots && slots.tabs !== undefined) { components.push(...toNodeArray(slots.tabs)); }`. ⇒ adding `relatedList: 'primary'` would have been **inert on this page**, not corrective. "An emergent property of the renderer" is not merely unproven; it is measurably not what happens. ## Item disposition — it STAYS, as a corrected residue Asked to choose whether the item stays in "What this record does not decide" or moves. **It stays**, for three reasons: 1. **The disposition is still true.** This record does not decide the tab set or its order — that was never the false part. What was false was the *ground* stated for it. Moving the item would imply the placement was the error. 2. **The section's preamble stays honest.** It says "⛔ None of it is resolved here." Still true: this record decides nothing new. A correction of fact recorded in place is not a decision this ADR now makes. 3. **A correction belongs beside the reading it corrects.** ⛔ The item is not deleted — it records a real measurement, and the honest edit says what corrected it. The original reading is preserved verbatim under "The reading as originally recorded", including the "emergent property / gone stale" conclusion, quoted so a reader sees exactly what was overturned. The item now reads: recorded reading → what corrected it → what the measurement missed → why neither branch was available → the unchanged disposition. ##⚠️ Declared scope expansion — one sentence in D3 The dispatch fenced this round to item 1. **A whole-file sweep found the same falsehood at a second site**, and I corrected it too rather than leave a governed record contradicting itself one section away: >⚠️ **The tab set and its ordering are NOT declared by this repository, and this record does not decide them.** See [What this record does not decide](#what-this-record-does-not-decide). That sentence is D3's **pointer into the very item being corrected**. Leaving it would have made the ADR assert the falsehood in its *decision* section while refuting it in its residue section. It is corrected minimally: the "not declared" clause is replaced with the declaring record and its registration, the true half ("this record does not decide them") is kept word for word, and the pointer is kept. This is the bounded in-place fix, and all four conditions were measured before taking it: ① same defect class as the card; ② mechanical, in a form already pinned by item 1's correction; ③ no other claim holds this file — all 16 open PRs were enumerated and their file lists read; two touch `docs/adr/`, namely PR objectstack-ai#17776 (ADR-0087) and PR objectstack-ai#17756 (ADR-0025), and **zero** touch ADR-0133; ④ same gate family, no new verification surface. ⭐ The sweep that found it is the same discipline this card is about: `NOT declared` → 1, `declared by none` → 1, `emergent` → 1, `tab order` → 2, on a whitespace-flattened, indent-stripped stream. Exactly two sites; both corrected. ⛔ Items 2, 3 and 4 of that section are **byte-untouched** (they appear in the diff only as context lines). No code, no metadata, no other ADR, nothing under `content/docs/releases/`. ## Verification⚠️ **Prose probes must be flattened before matching.** On the raw file, three of the four quotes I needed returned **0** — they wrap across lines. On a flattened, indent-stripped stream all four return **1**, with `relatedList` → **2** on the same stream as the lit control proving the probe reaches. | probe | raw file | flattened | |:--|--:|--:| | `declared by none of its metadata` | 1 | 1 | | `emergent property of the renderer` | **0** | 1 | | `no object in ... declares the relatedList prominence key` | **0** | 1 | | the deep-link-contract sentence | **0** | 1 | | **lit control** `relatedList` | — | **2** | **On-disk proof of the edit** (⛔ not the editor's exit code): replaced text → 0 occurrences each; injected text → present each; the preserved original reading → still 1 each; `relatedLayout` → 1 as the control on the same after-stream. **Gates — all run with the exit code captured BEFORE any pipe** (`cmd > file 2>&1; EXIT=$?`), the set derived by `node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack` (18 commands; the stderr provenance line names this repo and commit `310760d225`): | gate | exit | |:--|--:| | `check-adr-symbol-anchors` (+ `--self-test`) | 0 | | `check-adr-links` (+ `--self-test`) | 0 | | `check:adr-anchors` | 0 | | `check:doc-authoring` | 0 | | `check:nul-bytes` | 0 | | `check:pm-governed-merges` | 0 | | `check-closing-keyword-parity` (+ `--self-test`) | 0 | | `check-ci-filter-parity` | 0 | | `check-comment-mask-corpus` | 0 | | `check:cross-package-test-inputs` | 0 | | `check:driver-memory-census` · `check:refd-timer-probe` · `check:watch-hint-literal` | 0 | | `report-test-timings --self-test` | 0 | ⭐ **The new anchors are genuinely checked — proven by ablation, not by the green.** `check-adr-symbol-anchors` passes 2079 anchors across 139 records. Renaming the symbol inside my new anchor on disk drove it to **exit 1** with `[unresolved-symbol]` at **both** new sites; restoring reproduced the file **byte-identical** (`git hash-object` `e085c37a…` before and after, with a `trap … EXIT INT TERM` and absolute paths), and the gate returned to exit 0. On-disk proof preceded reading any result: injected marker → 2, original spelling → 0.⚠️ **The two `objectui:` anchors are only judged when a checkout is available.** Default run: 27 cross-repo anchors reported-not-judged. With `OBJECTUI_CHECKOUT=/home/user/objectui`: skipped drops to 11 and mine are not among them ⇒ judged and resolved. Both symbols were independently confirmed at the pinned sha with `git show`. ⛔ **One gate was NOT MEASURED at first and is now measured.** `check:doc-formula-expressions` returned **exit 3 — PREREQUISITE NOT MET** (`@objectstack/formula` and `@objectstack/lint` not built). Exit 3 is not a failure and ⛔ never counts inside a green tally. My file **is** in its corpus (`ROOTS` includes `docs`; `docs/adr` is not in `SKIP_PATHS`), so the prerequisite was satisfied and the gate re-run rather than waived. **Repo-wide scans are CI's.** No local narrowing is claimed for them. ## Changeset — `skip-changeset`, and this is why ⛔ Measured, not assumed. - **Target:** `docs/adr/0133-org-management-open-basics.md` resolves to **no owning package at all** — it sits outside every package directory, so no `files[]` can reach it. - Across the workspace: **70 published packages** (12 private skipped, 0 published without a `files[]`). The **union** of every `files[]` entry is `CHANGELOG.md | README.md | api-surface | dist | json-schema | liveness | llms.txt | prompts | spec-changes.json | src/**/*.zod.ts`. Entries naming `docs/adr` or escaping the package dir with `..`: **0**. - All 139 ADR files live at repo-root `docs/adr/`; **none** is nested inside a package. - No build or copy step pulls them in: `git grep docs/adr` over every `package.json`, the tsup configs and `turbo.json` → **no hits, exit 1**. **Lit control** on the same probe space: `dist` in those same `package.json` files → hits, exit 0. So the zero is a reading. - **Positive control (ships):** `packages/platform-objects/README.md` → direct `files[]` hit **true**. - **Negative control (does not):** `packages/platform-objects/src/identity/invite-entry-toolbar.test.ts` → direct `files[]` hit **false**. ⇒ nothing published moves, so `skip-changeset` applies — the label's own documented case in `lint.yml` is a PR that releases nothing.⚠️ Note for whoever lands this: the size-labeler's whole-set `PUT` has erased a seat-applied `skip-changeset` before; the label was applied additively and read back. ⭐ Deliberately the **opposite** call from PR objectstack-ai#17750 on the same card, and both are right — because both were measured. There, a comment-only diff moved published bytes (the new text appeared 4 times inside `packages/platform-objects/dist`), so a changeset was owed. Here the file cannot reach a tarball at all. ## 维护者速读(草稿) **改了什么** — ADR-0133「本记录不决定什么」第 1 条,以及 D3 里指向该条的那一句。原文用现在时断言「组织记录页开在 tab-0 Members」这件事「没有任何元数据声明它」,并推论 tab 顺序要么是「渲染器的涌现属性」,要么是「已经过时的说法」。两半都是假的,而且写下它的那天就已经是假的。 **为什么改** — tab 条本来就是声明出来的:`SysOrganizationDetailPage` 是 `sys_organization` 的 slotted 记录页,`slots.tabs` 里正好三个 related_list —— Members、Invitations、Teams,顺序如此,由 plugin-auth 注册进运行时。它在 ADR 自己引用的那个 commit `77781151d` 上就已经存在(HTTP 200,带「伪造路径回 Not Found」的对照)。⭐ ADR 的**测量是对的,推论是错的**:搜索被限定在 `src/identity/` 和 `relatedList` 这个键上,而声明在隔壁 `src/pages/`,用的是另一套词汇 —— 一次「把错误的空间穷尽搜索」必然回一个理直气壮的零。这条教训是本次修正真正要留下的东西。原读数**没有删**,原样保留并标注是什么推翻了它。 **风险与代价(含回滚)** — 风险很低:改的是散文,不动任何代码、元数据或其他 ADR;第 2、3、4 条逐字节未动。新引用全部是受门禁校验的符号锚,并用消融证明了它们真的会变红(改名后 exit 1,还原后字节一致、exit 0)。⛔ 不发布任何东西,因此 `skip-changeset`(已实测,带正负对照)。回滚 = 直接 revert 这一个 commit,单文件,无下游依赖。⚠️ 需要您注意的只有一处:派发把范围钉在第 1 条,我按「有界就地修」把 D3 里同一句假话也改了 —— 否则同一份受管记录会在决策节说 A、在残留节说非 A。若您认为不该扩,删掉 D3 那一段即可,第 1 条的修正独立成立。 **席位意见** — (留空,待席位补) **你要做的** — 这是**受管面**,按 Prime Directive 14,只有您手工合并;⛔ 任何 AI 席位都不会把它翻 ready、不入队、不挂 auto-merge。请确认两件事:① D3 的那处扩范围要不要保留;② 第 1 条留在「本记录不决定什么」作为「已更正的残留」是否合您的意 —— 我给的理由是:该条的处置本身没错(本记录确实不决定 tab 顺序),错的是它给出的理由,所以位置不动、理由更正。 ## 验收备注 - **noted, not filed** — 观察类,不立卡:`docs/qa/platform-checklist/runs/` 里仍然只有 `README.md`,`identity-auth.org-membership-team-management`(P1)从未对真实 console 执行过。它的 tab 断言是从 cloud ADR-0081 写出来的,没有被观察过。这是清单计划自己的 backlog,不是本树的缺陷。**承接者:** checklist-test lane / 下一次 platform-checklist 运行。 - **noted, not filed** — 命名撞车,不是缺陷:objectui 另有一个手写的 console 区域 `/organizations/:slug`(`OrganizationLayout.tsx`),tabs 是 Members / Invitations / **Settings**,没有 Teams,且确实 index-redirect 到 members。任何靠点界面来复测本卡的人都可能落到那个界面,然后在任一方向上得出一个很自信的错误答案。**承接者:** 本节本身 —— 下一个读这些文档的人在这里遇到它;PR objectstack-ai#17750 的验收备注已记过一次,这里是它在受管记录这一侧的对应位置。 - **noted, not filed** — 边界记录,不扩类:ADR-0133 的状态行仍是 `Proposed (2026-09-06)`,等待维护者手工合并这一「受管面的接受动作」。本 PR 不动状态行 —— 那是维护者的动作,不是更正的一部分。**承接者:** 维护者,在合并本 PR 时一并判断。 --- 🤖 Generated with [Claude Code](https://claude.com/claude-code) https://claude.ai/code/session_01MkQhmuuJAVDjmeWNixwDDH --- _Generated by [Claude Code](https://claude.ai/code/session_01MkQhmuuJAVDjmeWNixwDDH)_ Co-authored-by: Claude <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #16270
The card's premise is false, and that is the deliverable
The card asked for a measurement first: three documents assert that the organization record page opens on a Members tab with Invitations and Teams beside it, while
packages/platform-objects/src/identity/declares noField.relatedListprominence key. It framed a two-way disjunction — reading 1, the metadata is short threerelatedList: 'primary'declarations; reading 2, the three documents are stale — and the triage admitted the card on the grounds that both branches are defects.Neither branch holds. The tab strip is declared metadata. It is simply not declared by the key the measurement looked for, and not in the directory it looked in.
packages/platform-objects/src/pages/sys-organization.page.tsexportsSysOrganizationDetailPage: atype: 'record',kind: 'slotted',isDefault: truepage forsys_organizationwhoseslots.tabsoverride carries exactly threerecord:related_listtabs — Members, Invitations, Teams, in that order.packages/plugins/plugin-auth/src/auth-plugin.tshands it to the runtime aspages: [SysOrganizationDetailPage, SysUserDetailPage], and its own comment there already said so: "sys_organization gets a Members / Invitations / Teams tab strip" — a fourth document making the claim, and the only one that named the mechanism.The renderer half, read from objectui at the sha this repo pins (
.objectui-sha=53ded82bf7):packages/plugin-detail/src/synth/buildDefaultPageSchema.ts— when the assigned page supplies atabsslot, the synthesizer pushes that authored node and never callsbuildDefaultTabs. The authored strip replaces the synthesized one outright, so Members is at index 0.buildDefaultTabsalways seedsitems[0]with Details. So even under reading 1 — threerelatedList: 'primary'declarations added — tab 0 would have been Details, never Members. Reading 1 could not have produced the documented surface.packages/app-shell/src/utils/deriveRelatedLists.tssetsisPrimaryfromfieldDef.relatedList === 'primary', exactly as the card quoted. That key is a different mechanism, and while thetabsslot is present it would not move this page at all.sys-organization.page.tswas already in the tree at77781151d, the commit the card measured against (verified through the contents API at that ref: HTTP 200, 4606 bytes). The zero was real; the inference from it was not.What this PR changes — prose only
No metadata. No behaviour. The three documents that assert the tab order now name the page that declares it, so the next reader does not repeat the measurement:
packages/platform-objects/src/identity/sys-member.object.tsinvite_usermirror's rationale gains a note naming../pages/sys-organization.page.tsas the declaring page, and stating thatrelatedListis a different mechanism absent from this directorypackages/platform-objects/src/identity/invite-entry-toolbar.test.tsdocs/qa/platform-checklist/areas/identity-auth.jsonidentity-auth.org-membership-team-managementgains onesourceentry, plus therevision2 to 3 andhistorybump its ledger requiresWhat a grader will now do differently (the checklist is what QA grades, so this is stated plainly): nothing about the grade changes. The item's
steps,acceptanceclauses, their oracles andverifytext, and thenegativelist are byte-unchanged — the same screenshot, the same three tabs, the same FAIL condition for the Organization nav landing on the raw list. What is added is asourceentry, which is a pointer a grader reads to locate the surface, and it tells them one new thing: if the page renders Details plus a stackedRelatedtab instead of the three, that meansSysOrganizationDetailPagefailed to load — it does not mean the clause was wrong. That reading was not available before, and it is the reading this card was filed for lack of.Verification
pnpm lint(repo-wide,eslint . --no-inline-config)pnpm --filter @objectstack/platform-objects testpnpm --filter @objectstack/platform-objects typecheckpnpm --filter "@objectstack/platform-objects..." buildpnpm check:platform-checklistOK — 15 areas, 264 items; symbol anchors: 633/633 resolvedpnpm check:nul-bytes·check:doc-authoring·check:page-declaration-shape·check:published-files·check:test-source-alias·check:cross-package-test-inputs·check:type-check-coverage·check:comment-mask-adoption·check:comment-mask-corpus·check:keyed-text-bounds·check:closing-keyword-paritycheck-empty-changeset·check-changeset-no-major·check-adr-0087-registration, each--base origin/maincheck-partof-closing-keywordwith this body and this branch's commit listEvery exit code above was captured before any pipe (
cmd > file 2>&1; EXIT=$?), and the two lock-held runs are quoted from the wrapper's ownVERDICT command-exitline.Ablation — the checklist gate's new anchor really is checked. With the new
sourceentry's symbol renamed on disk toSysOrganizationDetailPageXX(injected text present 1, original present 0, both counted withgrep -oF),check-platform-checklistwent to exit 1 withABSENT SYMBOL ... is not in packages/platform-objects/src/pages/sys-organization.page.ts outside its comments. Restored from a pristine copy and proved byte-identical bygit hash-object(168dca14before and after). The green above is therefore a reading, not a gate that judged nothing.Why a changeset and not the publishes-nothing exemption.
@objectstack/platform-objectsshipsfiles: ["dist", ...], and itssrccomments reachdist: after the build, the new comment text appears 4 times underpackages/platform-objects/dist, with an exported symbol (SysMember, 16 occurrences) as the positive control and the edited test file's header absent at 0. Published bytes move, so apatchchangeset is owed —.changeset/16270-org-record-tab-strip-provenance.md, in the tree's issue-scoped spelling.Declared narrowing.
scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstackderived 62 runnable commands for this change set; the table above is a subset chosen by what this diff can move, and the rest is CI's farm. The derivation also warned that this branch's base is 3 commits behindorigin/mainand that one file it derives from (scripts/check-regen-pending.mjs) changed across that range; re-deriving aftergit fetch origin mainproduced a byte-identical command list.Why
Part ofand not a closing keywordThis PR carries the measurement and repairs the discoverability gap, but it does not discharge the card.
docs/adr/0133-org-management-open-basics.md, under "What this record does not decide", now states something false — that the tab claim is "declared by none of its metadata" and that the ordering is "either an emergent property of the renderer or a claim that has gone stale".docs/adr/**is a governed surface, outside this dispatch's declared file face, and its correction is the maintainer's call. Recorded as a finding in the round report rather than ridden in here.验收备注
53ded82bf7), taken withgit show 53ded82bf7:pathin the sibling checkout. That checkout's own HEAD is3fbdd4a2don an unrelated branch; nothing there was edited./organizations/:slug(packages/app-shell/src/console/organizations/manage/OrganizationLayout.tsx), whose tabs are Members / Invitations / Settings with an index redirect tomembers. It is reached from a "Manage" button on the organizations list, not from Setup'snav_organization, and it has no Teams tab. Noted so the next reader does not measure that surface and grade this one. Not filed: no defect, just a collision of names.docs/qa/platform-checklist/runs/contains onlyREADME.md, soidentity-auth.org-membership-team-managementhas no run record and its tab assertion had never been executed against a live console. Noted, not filed.Generated by Claude Code