[objectstack-fleet]

Requested by Steedos · project thread

Part of #25921 — the zero-consumer barrel batch for @object-ui/plugin-gantt and @object-ui/plugin-designer (claim 6576816381), the item landings 6542309053 and 6543370760 left open. The card stays open: every other item it lists is untouched.

Clause-②: yes (narrowing)
Metadata-protocol: none

维护者速读

• 改了什么：@object-ui/plugin-gantt 入口不再导出 19 个名字（快速筛选栏、资源负载网格、负载计算、班次带工具及其类型、normalizeDependencies）；@object-ui/plugin-designer 入口不再导出 6 个值（CollaborationProvider、ConnectionStatusIndicator、useClipboard、useMultiSelect、useCanvasPanZoom、PropertyEditor）。模块文件全部保留，两个 README 不再教这些名字，每包一个 ! changeset。
• 为什么改：这 25 个名字在全仓（含 examples、文档、技能、测试）和 hotcrm 中，除自己包外零引用；甘特图和设计器内部照常使用它们，只是不再作为没人用的公开面发布。
• 风险与回滚：仓内无消费者，Console 与其它 @object-ui/* 包不受影响；外部 npm 用户若导入会在升级时编译报错，changeset 写明每个名字改用的写法（多数是在 gantt 块里声明 quickFilters / resourceView / timeSegments）。回滚即 revert 本 PR。
• 先看：ui/packages/plugin-gantt/src/index.tsx、ui/packages/plugin-designer/src/index.tsx、.changeset/25921-plugin-designer-internal-helpers-off-entry.md（协作入口退出后，设计器里的协作分支在包外不可达）。
• 两分钟验证：git grep -n -w -e QuickFilterBar -e computeWorkload -e CollaborationProvider -e useCanvasPanZoom -- ui/packages/plugin-gantt/src/index.tsx ui/packages/plugin-designer/src/index.tsx ui/packages/plugin-gantt/README.md ui/packages/plugin-designer/README.md，预期无输出（退出码 1）：两个入口和两个 README 都不再出现这些名字。
• 需要你做的：无，席位落地（入队前欠一次契约复审档的复审，由席位转总监）。

What changes

• @object-ui/plugin-gantt (src/index.tsx): the entry stops exporting QuickFilterBar, QuickFilterBarProps, QuickFilterField, QuickFilterOption, QuickFilterLabels, QuickFilterDef, ResourceWorkload, ResourceWorkloadProps, computeWorkload, WorkloadColumn, WorkloadOptions, ResourceCell, ResourceLoad, normalizeDependencies, normalizeShiftSegments, shiftDayStart, bandAt, ShiftSegmentsConfig and ShiftBandConfig. Every module stays: ObjectGantt still imports the bar, the grid and the shift helpers, and ResourceWorkload still imports the workload math.
• @object-ui/plugin-designer (src/index.tsx): the entry stops exporting CollaborationProvider, ConnectionStatusIndicator, useClipboard, useMultiSelect, useCanvasPanZoom and PropertyEditor. Every module stays: the four designers import the hooks, PropertyEditor and useCollaboration from them.
• Both README.md files stop teaching the retired names (check-readme-exports binds every README self-import to a real export). The gantt README now says the bar, the workload grid and the shift bands are configured through the gantt block; the designer README drops the collaboration feature bullet, its section and the Quick Start wrapper.
• Two changesets, patch with the **BREAKING** banner, both declaration lines, the ADR-0087 not-required (runtime-export-retired …) marker naming every operand, and a FROM → TO "no replacement" line with the one-line fix per group. New file names, distinct from the 25921-* changesets chore: version packages #26517 still holds.

Why

How the set was chosen, the #28385 method re-run at origin/main 738971d2 (hotcrm d4ffe99): every name each barrel exports was listed from the barrel's AST (35 / 38), then git grep -l -w -F ran for each over the whole tree minus the package's own directory, **/CHANGELOG.md, the lockfiles, .changeset/** and ui/.changeset/**; the brief's cut (ui/, packages/, apps/, examples/, content/, tests excluded) and hotcrm were counted beside it. Positive controls, same command: ObjectGantt 108 files (the #28385 reading), CreateAppPage 48, SchemaRenderer 1157 (hotcrm 3). A name was retired only when all three hold:

1. zero hits outside its package on every cut (any hit, prose included, keeps the name);
2. no in-package module loses its last importer (no module file is deleted);
3. no export that stays needs it, measured with the TypeScript checker over the declared signatures of every kept export.

package	barrel names	zero hits outside	of those, needed by a kept export	retired here
@object-ui/plugin-gantt	35	24	5	19
@object-ui/plugin-designer	38	15	9	6

Kept by rule 3: GanttViewProps (props of GanttView), GanttDependency and GanttDependencyObject (GanttTask.dependencies), NormShiftSegments and NormShiftBand (GanttViewProps.shiftSegments); the nine designer …Props types of components that stay.

Tests

• Code head 641e44d3; bb4ab640 merges 21 origin/main commits, none under either package (branch delta re-read: the same 6 files, +71 / -107).
• turbo run type-check --filter=@object-ui/plugin-gantt --filter=@object-ui/plugin-designer --filter=@object-ui/app-shell --filter=@object-ui/console → 74/74 tasks successful (os-verify-lock VERDICT command-exit 0). app-shell (the lazy designer pages) and the Console are the importers of either package.
• pnpm --dir ui exec vitest run --maxWorkers=2 packages/plugin-gantt/ packages/plugin-designer/ → Test Files 156 passed, 1 skipped (157); Tests 1248 passed, 114 skipped (VERDICT command-exit 0).
• pnpm --dir ui exec vitest run scripts/ (includes check-readme-exports) → Test Files 169 passed, 2 skipped; Tests 4903 passed (VERDICT command-exit 0).
• Full dogfood battery (the title carries !) on bb4ab640: closure turbo run build --filter=@objectstack/dogfood^... 68/68, then pnpm --filter @objectstack/dogfood test → Test Files 393 passed, 3 skipped (396); Tests 3243 passed, 28 skipped (3271); VERDICT command-exit 0, held the lock 2236 s.
• Head d0ee5c76 merges 29 more origin/main commits, none under either package; the census re-run on it still reads zero for all 25 retired names (control ObjectGantt 108). The suites above were not re-run on it.
• No ablation: export lines and README text removed; no tested behaviour changed.

Acceptance notes

• With CollaborationProvider off the entry, nothing outside the package can mount it, so the collaboration branches in PageDesigner, DataModelDesigner, ProcessDesigner and ReportDesigner (useCollaboration() is now always null for a host) and the provider and indicator bodies in CollaborationProvider.tsx are dead code. Deleting them edits modules this claim does not cover; same card family, left for a next pass.
• useConfirmDialog and VersionHistory have zero hits once tests are excluded; their only outside hits are comments in ui/scripts/__tests__/one-authority-per-exported-name-6273.test.ts naming their module files. Kept under rule 1 here.
• Gantt names kept with outside hits: ObjectGantt, ObjectGanttProps, ObjectGanttRenderer, GanttView, GanttTask, GanttTaskType, GanttViewMode, GanttLinkType, GanttMarker, normalizeTaskType, parseHHMM (several are prose or same-name hits, for example parseHHMM in service-messaging; rule 1 keeps them all).
• A CONTRACT_REVIEW_TIER review is owed before the queue (barrel narrowing); the seat routes it to the director.

Verification

Derived gates — 28/28 run, 0 NOT-MEASURED

node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack at 641e44d3: 28 commands; --ran: "28 derived famil(ies) accounted for — 28 run, 0 NOT-MEASURED". Every command exit 0, among them check-adr-0087-registration --base origin/main, check-empty-changeset --base origin/main, check-changeset-no-major --base origin/main, check-sdui-manifest and its --self-test, check:published-files, check:undeclared-dep-imports, check:nul-bytes and vitest run scripts/. check-comment-mask-corpus first timed out at 300 s beside the closure build, then exit 0 alone (20050 files, 0 disagree, 284 s).

Outside the derived set: check-platform-checklist --changed-from origin/main exit 0 (0 anchored files) at bb4ab640. dispatch-gates: 178 changed lines, under the 3000 threshold; no governed path.

Human Minimum

• 1. Schemas start as Zod; packages/spec holds no runtime logic.
• 2. Every name I added uses the casing for its kind.
• 3. No formatter ran; eslint on the two changed barrels exits 0 (7 warnings, 0 errors: react-refresh/only-export-components on the kept helper exports and one no-explicit-any, all present on main before this diff); the repo-wide pnpm lint is CI's run.
• 4. One issue, one branch off an up-to-date main, one pull request.
• 5. A change that publishes carries a changeset, and both declaration lines are filled in here and in the changeset.
• 6. packages/spec touched: its generated artifacts are regenerated and committed. (not touched)
• 7. No hand edits to content/docs/references/, packages/*/CHANGELOG.md or content/docs/releases/.
• 8. Title, body and comments are in English.

---

Generated by Claude Code

[zhuangjianguo]

director#1 · zhuangjianguo · session_01QsR2jV5sBSzCYNEDgpYLJw

Contract-review-claim: session_01QsR2jV5sBSzCYNEDgpYLJw · head d0ee5c7 · 2026-11-02T14:02Z

---

Generated by Claude Code

[zhuangjianguo]

director#1 · zhuangjianguo · session_01QsR2jV5sBSzCYNEDgpYLJw

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: d0ee5c769a6f6bb1ef6c0d69021050f415f2d79c
Local-runs: none
Scope: ui/packages/plugin-gantt/src/index.tsx, ui/packages/plugin-designer/src/index.tsx (the two published entries), .changeset/25921-plugin-gantt-internal-helpers-off-entry.md, .changeset/25921-plugin-designer-internal-helpers-off-entry.md, ui/packages/plugin-gantt/README.md, ui/packages/plugin-designer/README.md + ui/packages/plugin-gantt/src/ObjectGantt.tsx (ObjectGanttProps, ObjectGanttHandoff L324, GanttConfigRestated L192, QuickFilterDef L109), ui/packages/plugin-gantt/src/GanttView.tsx (GanttViewProps.shiftSegments L628), ui/packages/plugin-gantt/src/shifts.ts, ui/packages/types/src/objectql.ts (ObjectGanttSchema / GanttConfig), a git grep -w -F census on main of every retired name outside its package, record 6542128649 on PR #28385 (the precedent and its three keep rules), card #25921, claim 6576816381, os-dev report 6579032346

Read-only review of the committed tree. Check-runs on the head at claim time: 9 success, 11 skipped, 32 in progress; the seat lands on a green head, and this record is pinned to this sha.

① Derived judgments

• The narrowing is exactly the 25 names, and every one has zero readers outside its package. My own git grep -l -w -F on main for each of the 19 gantt names and 6 designer names, over the whole tree minus the package's own directory, CHANGELOGs, lockfiles and changesets, reads 0 for all 25; the positive control ObjectGantt reads 136 files. The dev's hotcrm reading (zero at d4ffe99) is not re-measured here (no checkout; read-only) and is accepted as the dev's. Right.
• Keep rule 3 holds: no kept export's declared signature names a retired type. The one place that looked like a counter-example is GanttConfigRestated (ObjectGantt.tsx L192–248), which types quickFilters as QuickFilterDef[] and timeSegments as ShiftSegmentsConfig; but ObjectGanttProps.schema is ObjectGanttHandoff = ObjectGanttSchema & ObjectGanttSearchFields (L324) with ObjectGanttSchema imported from @object-ui/types (L27), whose gantt?: GanttConfig carries the spec's own quickFilters / timeSegments twins (ObjectGantt.configPin.test.ts pins them mutually assignable). GanttConfigRestated was never on the barrel. The five types the dev kept (GanttViewProps, GanttDependency, GanttDependencyObject, NormShiftSegments, NormShiftBand) are exactly those a kept signature names (GanttTask.dependencies, GanttViewProps.shiftSegments L628). Right.
• No module leaves; no in-package importer breaks. The entries only drop export lines and one import line; ObjectGantt still imports the bar, the grid and the shift helpers, the four designers still import the hooks, PropertyEditor and useCollaboration. Right.
• Rule 1 applied conservatively: useConfirmDialog and VersionHistory (only test-comment hits) and parseHHMM (a same-name hit in service-messaging) stay. Any hit keeps the name; that is the refactor(ui)!: retire zero-consumer exports from @object-ui/collaboration and @object-ui/mobile #28385 rule and it errs toward keeping. Right.
• Public surface: two ui entries narrow; nothing under packages/spec, no registered node type, no stored shape. No security-relevant finding; nothing withheld from this record.

② Semver level

• Two changesets, one per package, patch with the **BREAKING** banner and refactor(…)!: subjects; Clause-②: yes (narrowing) and Metadata-protocol: none at line start in both and in the PR body. Under the repo's level rule (a non-protocol narrowing ships patch with its banner) and the refactor(ui)!: retire zero-consumer exports from @object-ui/collaboration and @object-ui/mobile #28385 precedent, patch is the required grade. Right.
• Migration present: a FROM → TO line per group, each "no replacement" with the one-line fix (declare gantt.quickFilters / resourceView / timeSegments on the node; pass shiftSegments in the NormShiftSegments shape; render the designers without the wrapper). Right.
• ADR-0087 marker not-required (runtime-export-retired …) names every operand in both files, with the reason (no spec schema, node type, conversion surface or stored shape names them). Same class and shape as refactor(ui)!: retire zero-consumer exports from @object-ui/collaboration and @object-ui/mobile #28385's. Right.
• New changeset file names, distinct from the 25921-* files PR chore: version packages #26517 holds, so no collision in the queue. Right.

③ Boundary flags

• Tier S (no governed path, 178 changed lines); the seat lands after this record and a green head. The PR is Part of #25921; the card stays open with its other items untouched, as the body says.
• Dead collaboration branches after CollaborationProvider leaves the entry (useCollaboration() is now always null for any host; the provider and indicator bodies are unreachable from outside): declared by the dev, left for a next pass because deleting them edits modules outside this claim. Right to leave; it is the same card family's next item, not this narrowing's.
• check:readme-exports could not be measured locally for cli (prerequisite dist); the README self-import binding is exercised by vitest run scripts/ (169 files green locally) and CI's Test UI/Lint & Repo Gates. The two README diffs remove every teaching of a retired name and keep only names the entries still export (useUndoRedo, useConfirmDialog, ConfirmDialog, Minimap, VersionHistory, PageDesigner). Right.
• The dev did not re-run suites on the final head d0ee5c76 (a 29-commit main merge touching neither package; the census re-run reads 0 on it). Acceptable for a merge that moves no file of either package; CI on the head is the verdict.
• No open questions.

④ Design judgment

ui/packages/plugin-gantt/src/index.tsx — judged against ObjectGantt.tsx, GanttView.tsx, shifts.ts and the census.

• What stays is the composable surface a host actually uses: the two components, the renderer, the task/dependency/link/marker vocabulary, normalizeTaskType, parseHHMM, and the two normalised shift types GanttViewProps needs. The new header comment states that rule, so a later author does not re-add an internal out of habit. Break attempt: a host that drove GanttView directly with raw ShiftSegmentsConfig would now have to normalise it — the changeset says so (pass shiftSegments already in the NormShiftSegments shape), and nothing in the tree or hotcrm does it. Holds.
• QuickFilterDef leaving while gantt.quickFilters stays authorable is right: the authored shape is the spec's GanttConfig.quickFilters, reachable from @object-ui/types, and the plugin's local twin was only ever the renderer's restatement. Holds.

ui/packages/plugin-designer/src/index.tsx — judged against the four designers and CollaborationProvider.tsx.

• The six names are designer internals (three canvas hooks, the property panel, the collaboration wrapper and its indicator); the comment above the hooks block records why they stay module-local. DESIGNER_DEFAULT_TRANSLATIONS keeps its export and its objectui#4401 reason. Holds.
• Break attempt: a host that wrapped PageDesigner in CollaborationProvider per the old README loses real-time co-editing with no replacement; the changeset says exactly that, and the census shows no such host in the workspace or hotcrm. A product decision to offer collaboration again would re-export the provider, additively. Holds.

READMEs — the gantt README now routes the bar, the workload grid and the shift bands through the gantt block, which is the surviving authoring path; the designer README drops the collaboration bullet, section and Quick Start wrapper. Both teach only what the entries export. Holds.

No must-fix found. No should-fix.

Implemented-by: claude/project-thread-g88f85
Reviewed-by: session_01QsR2jV5sBSzCYNEDgpYLJw

VERDICT: PASS

---

Generated by Claude Code

[objectstack-fleet]

domain:ui#5 · zhuangjianguo · session_01QsR2jQjvWkZ7is6GXbC6DB

席位核对 — ACCEPT · domain:ui#5 · head d0ee5c76 · 2026-11-02T14:13Z

• 形态: draft PR on main, titled with !. The body opens with Part of #25921, a partial item with no closing keyword, so the umbrella card stays open. 6 files, +71/−107, Tier S: check-governed-merges --pr 29388 finds no governed path and 178 lines, under the 3000 limit. The body carries 维护者速读 and both declaration lines (Clause-②: yes (narrowing), Metadata-protocol: none), matching claim 6576816381.
• 前提实测: landing 6542309053 listed these names as still open after refactor(ui)!: retire zero-consumer exports from @object-ui/collaboration and @object-ui/mobile #28385. The dev re-ran the census on current main with refactor(ui)!: retire zero-consumer exports from @object-ui/collaboration and @object-ui/mobile #28385's method: per-name git grep -w -F across the whole tree minus the package, plus the brief's cut and hotcrm. Positive controls: ObjectGantt 108 files, SchemaRenderer 1157. 24 gantt names and 15 designer names were zero-hit. Under refactor(ui)!: retire zero-consumer exports from @object-ui/collaboration and @object-ui/mobile #28385's keep rule 3, the 5 gantt types a kept signature needs stay, and 19 leave. The 9 designer Props of kept components stay, and 6 values leave. useConfirmDialog and VersionHistory stay, since a test comment names their modules (os-dev report 6579032346).
• diff read: only the two src/index.tsx entry barrels and the two READMEs change. No module file is deleted, because each is still imported inside its package, and no test imports a removed name through an entry. There is one ! changeset per package, with fresh filenames distinct from the 25921-* notes Version Packages already holds, plus the FROM → TO "no replacement" table and the ADR-0087 marker.
• 门禁 check-runs (pinned to head d0ee5c769a6f6bb1ef6c0d69021050f415f2d79c): 44 success, 11 skipped, 0 failed, 0 pending. mergeable_state clean. Dev local gates: 28 derived, all exit 0. Type-check gantt/designer/app-shell/console 74/74. Package suites 156 files, 1248 tests. Dogfood battery for the ! title: VERDICT 0.
• Contract review: director CONTRACT_REVIEW_TIER review, VERDICT PASS, comment 6579108918 on this head (claim 6579051345). It has no must-fix and no should-fix. It independently re-censused all 25 names (0 outside hits; control ObjectGantt 136 files) and confirmed the five kept types are exactly right.
• 接受的偏差: check:readme-exports could not measure packages/cli locally (unbuilt prerequisite), and CI's run is green. The barrels are index.tsx, where the claim said .ts: same files.
• 范围外去向: carried on retire the zero-consumer surface: nine dead modules (2,211 LOC), plugin-ai, runner's publish flag, test-only reset helpers on public barrels, the providers trio, 122 dead i18n keys, root orphans #25921 and recorded in its landing note: the designers' collaboration branches and the CollaborationProvider bodies are now dead code, and useConfirmDialog and VersionHistory are zero-hit once tests are excluded.
• 落地条款: narrowing with a same-head PASS. Land by pr_ready + automerge_enable. After it lands, the card returns to pm:queue with a landing note listing what is still open.