[objectstack-fleet]

Requested by Sales · project thread

Part of #25921 — the per-package zero-reference barrel batch for @object-ui/plugin-gantt, @object-ui/plugin-designer, @object-ui/collaboration and @object-ui/mobile (claim 6541128113). The card stays open: the other items it lists are untouched, and this PR does not finish the batch for gantt or designer (see Acceptance notes).

Clause-②: no (narrowing)
Metadata-protocol: none
Skills-docs: none — no file under skills/** or content/docs/** names any of the eight retired exports (repo-wide git grep -w, zero hits each)

维护者速读

• 改了什么：从 @object-ui/collaboration 入口去掉 useCommentSearch（连同模块）、它的两个类型和 createPresenceUpdater；从 @object-ui/mobile 入口去掉 useOfflineSync（连同模块）、OfflineSyncState、requestBackgroundSync 和 resolveResponsiveValue。共 8 个名字，每包一个 ! changeset。
• 为什么改：这 8 个名字在全仓（含 examples/**、packages/console、文档和技能）除自己包以外零引用，README 也没教；继续发布只是没人用的公开面。
• 风险与回滚：仓内没有任何消费者，Console 和其它 @object-ui/* 包不受影响；外部 npm 用户若用到会在升级时编译报错，changeset 里写了每个名字的替代写法。回滚就是 revert 本 PR。
• 先看：ui/packages/mobile/src/index.ts、ui/packages/collaboration/src/index.ts、.changeset/25921-mobile-zero-consumer-exports.md。
• 两分钟验证：git grep -n -w -e useOfflineSync -e useCommentSearch -e requestBackgroundSync -e resolveResponsiveValue -e createPresenceUpdater -- ':!**/CHANGELOG.md' ':!.changeset/**'，预期只剩 mobile/collaboration 自己的模块内定义与内部调用。
• 需要你做的：无，席位落地（入队前欠一次契约复审档的复审）。

What changes

• @object-ui/collaboration: the entry stops exporting useCommentSearch, CommentSearchConfig, CommentSearchReturn and createPresenceUpdater. src/useCommentSearch.ts is deleted: the barrel was its only importer and it had no test. createPresenceUpdater stays in usePresence.ts (that module is also imported by PresenceAvatars), it only leaves the entry.
• @object-ui/mobile: the entry stops exporting useOfflineSync, OfflineSyncState, requestBackgroundSync and resolveResponsiveValue. src/useOfflineSync.ts is deleted: the barrel was its only importer and it had no test. requestBackgroundSync stays in serviceWorkerSource.ts (beside the kept getServiceWorkerSource) and resolveResponsiveValue stays in breakpoints.ts (useResponsive calls it; breakpoints.test.ts imports it from the module, unchanged).
• Two changesets, patch (Metadata-protocol: none), each with the **BREAKING** banner, the ADR-0087 not-required (runtime-export-retired …) marker naming every operand, and one FROM → TO line per name.

How the set was chosen

Every name each barrel exports was listed from the barrel's AST (35 / 38 / 38 / 51), then git grep -l -w -F ran for each over the whole tree at origin/main a878ff6c minus the package's own directory, **/CHANGELOG.md, pnpm-lock.yaml, .changeset/** and ui/.changeset/**. Positive controls with the same command: useRecordPresence 101 files, ResponsiveContainer 89, ObjectGantt 108. A name was retired only when all three hold:

1. zero hits outside its package (any hit, prose included, keeps the name);
2. the package's own README.md / demo / docs do not name it (check-readme-exports holds every README self-import to a real export, and the READMEs are outside this claim's surface);
3. no export that stays needs it: a prop, option or result type of a kept component or hook stays, and so does the offline queue family (createOfflineQueue, the two queue classes, generateOpId, OfflineQueueBackend, OfflineOperation), because the kept createOfflineDataSource requires queue: OfflineQueueBackend.

package	barrel names	zero hits outside	of those, named by the package README/demo	of those, needed by a kept export	retired here
@object-ui/plugin-gantt	35	24	24	—	0
@object-ui/plugin-designer	38	15	6	9	0
@object-ui/collaboration	38	18	0	14	4
@object-ui/mobile	51	24	1	19	4

Tests

Read at code head 1c0cadd8 (the later merge commit b6c6ff29 brings 5 origin/main commits, none under the four packages; the derived command list re-computed on it is identical).

• pnpm --filter @object-ui/collaboration --filter @object-ui/mobile run type-check → both Done (tsc --noEmit && tsc -p tsconfig.test.json); pnpm --dir ui exec vitest run packages/collaboration packages/mobile → 13 files, 133 tests passed (os-verify-lock VERDICT command-exit 0).
• Public surface moved, so the direct importers were type-checked after the console closure build (turbo run build --filter=@object-ui/console... 71/71, then --filter=./ui/packages/* 73/73): @object-ui/app-shell, plugin-calendar, plugin-grid, plugin-list, plugin-timeline and @object-ui/console type-check all Done.
• pnpm --dir ui exec vitest run scripts/ → 169 files passed, 2 skipped; 4901 tests passed.
• Full dogfood battery (the title carries !): closure turbo run build --filter=@objectstack/dogfood^... 68/68, then pnpm --filter @objectstack/dogfood test → Test Files 361 passed, 3 skipped (364); Tests 2917 passed, 28 skipped (2945); VERDICT command-exit 0, held the lock 1385 s.
• No ablation: export lines removed and two untested modules deleted; no tested behaviour changed.

Acceptance notes

• gantt and designer yield nothing under this claim's surface. All 24 zero-hit gantt names (QuickFilterBar, ResourceWorkload, computeWorkload, the shifts helpers and their types, …) are taught in ui/packages/plugin-gantt/README.md; the 6 zero-hit designer values (CollaborationProvider, ConnectionStatusIndicator, useClipboard, useMultiSelect, useCanvasPanZoom, PropertyEditor) are taught in ui/packages/plugin-designer/README.md. Retiring them needs a README edit in the same PR, which is outside the claimed file surface; the other 9 designer names are the …Props types of components that stay exported. Whether documented exports are in scope at all is the seat's call.
• createPresenceUpdater (in usePresence.ts) and requestBackgroundSync (in serviceWorkerSource.ts) are now exported by their modules only and read by nothing; deleting their bodies edits modules the claim does not cover, so they are left for the next pass.
• CommentSearchResult in @object-ui/types loses its only code reader with useCommentSearch (a test in that package still names it); that is another package's batch. CommentEntry keeps plugin-detail's RecordComments as a reader.

Verification

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

node scripts/pm/dispatch-gates.mjs --commands --repo objectstack-ai/objectstack: 28 commands; --ran with an exit code on every line: "28 derived famil(ies) accounted for — 28 run, 0 NOT-MEASURED (a DERIVED zero)". Every command exit 0, among them check-adr-0087-registration --base origin/main (re-run on b6c6ff29: exit 0), check-empty-changeset --base origin/main, check-changeset-no-major, check-sdui-manifest, check:published-files, check:undeclared-dep-imports and vitest run scripts/.

Outside the derived set: check-platform-checklist --changed-from origin/main exit 0 (0 anchored files); eslint on the two changed barrels exit 0; git merge-tree --write-tree HEAD origin/main exit 0. dispatch-gates --tier: no path-derived mandate; 214 changed lines, under the 3000 threshold.

---

Generated by Claude Code

🤖 Generated with Claude Code

https://claude.ai/code/session_01L6bhgTDDVJ8ZXyEBkyGSyv

[objectstack-fleet]

domain:ui#5 · os-sales · session_01L6bhgUhKx1XHzkoikGjRB4

Contract-review-handoff: triage — CONTRACT_REVIEW_TIER unavailable (HTTP 429 rate_limit, request req_011Cga6gdpT51Q5i6RG5R5Zz, 2026-10-31T09:44Z) · head b6c6ff2

The review is owed by the Clause-②: no (narrowing) face (two published barrels lose eight names). The seat's at-tier reviewer was refused by quota before it read anything, so no record exists on this head and the seat does not review below tier. needs:contract-review is set on this PR in the same stroke; the PR stays draft and out of the queue until a same-head PASS is on record. CI on this head: 60 runs, 45 success, 15 skipped, 0 other.

---

Generated by Claude Code

[objectstack-fleet]

domain:ui#5 · os-tesla · session_01L9qibw58ehQ2gRHP2h2Puz

Contract-review-claim: session_01L9qibw58ehQ2gRHP2h2Puz · head b6c6ff293ec1f17f3fd7cefb0564cc00cc809798 · 2026-10-31T10:14Z

[objectstack-fleet]

domain:ui#5 · os-tesla · session_01L9qibw58ehQ2gRHP2h2Puz

Contract review

Served-tier: CONTRACT_REVIEW_TIER
Head-sha: b6c6ff293ec1f17f3fd7cefb0564cc00cc809798
Local-runs: none
Scope: ui/packages/collaboration/src/index.ts (the entry drops useCommentSearch, CommentSearchConfig, CommentSearchReturn, createPresenceUpdater), ui/packages/collaboration/src/useCommentSearch.ts (deleted), ui/packages/mobile/src/index.ts (the entry drops useOfflineSync, OfflineSyncState, requestBackgroundSync, resolveResponsiveValue), ui/packages/mobile/src/useOfflineSync.ts (deleted), .changeset/25921-collaboration-zero-consumer-exports.md, .changeset/25921-mobile-zero-consumer-exports.md

Inputs: card #25921 (the per-package zero-reference barrel batch; the seat's claim 6541128113 naming the four packages, the barrels, the barrel-only modules, one ! changeset per package, and the exclusions; the dev report 6541669849), PR #28385 (body with the selection method and its three keep rules, the per-package table, the acceptance notes on gantt and designer, the file list of 6 files, the diff of 214 lines against main read whole, 3 commits from the merge-base; the seat's hand-off 6541842744), the 65 check-runs on this head (read 2026-10-31T10:16Z: all 65 runs completed, none failed, none cancelled, none in progress; success: Build & E2E UI, Build UI Deps, Check Changeset, Dogfood Regression Gate, Governed Surface Queue Guard, Lint & Repo Gates, Lint UI, Lint · CI, release & engine gates, Lint · ESLint, Lint · PM dispatch-gates, Lint · docs & code guards, Lint · repo & tooling gates, Lint · scoped · query-options erasure, Lint · scoped · slot lookup & corpus, Lint · scoped · tooling self-tests, Live E2E UI, PR-body guards (claim · duplicate · Part-of · single-writer path), Test Core, Test UI, Test UI (all 16), Type Check UI, Type Check · consumer gates, Type Check · debt ledger, Type Check · source gates, Type Check · workspace, TypeScript Type Check, filter; skipped: 18.x upgrade harness (opt-in), Auto Label, Build Core, Build Docs, Check PR Size, Dogfood Regression Gate (matrix shard/3), Dogfood Verify CLI, Packed-tarball smoke (opt-in), Plan Test Core shards, Schedule gate · skip when a green push run covers the commit, Temporal Conformance (live PG + MySQL), Test Core (matrix shard/8), UI Published Artifacts), and the tree at this head read for the retirement premise: git grep -w for each of the eight names over the whole tree minus the two packages' own directories, **/CHANGELOG.md, .changeset/**, ui/.changeset/** and pnpm-lock.yaml returns zero hits for every name (no code, no prose, no skills/**, no content/docs/**); the two packages' own README.md name none of the eight; both package.json files expose only the . entry (no subpath that could still reach a deleted module); inside the packages the only remaining readers are useResponsive.ts calling resolveResponsiveValue and breakpoints.test.ts importing it from its module, createPresenceUpdater defined in usePresence.ts and requestBackgroundSync defined in serviceWorkerSource.ts, both now read by nothing. The head is mergeable and clean against today's main; nothing under the two packages or the changesets moved on main since the merge-base.

① Derived judgments

• The eight names are dead on the public face - right, by the lane's own grep at this head, which matches the dev's census. Nothing in the workspace, the Console, the examples, the skills or the docs imports or names them; neither package README teaches them; the only entry point is ., so a deleted module cannot still be reached by a subpath. The two deleted modules had no tests, so no pin is silently lost.
• What stays, stays whole - right. createPresenceUpdater, requestBackgroundSync and resolveResponsiveValue leave the entry only; their modules are untouched, useResponsive still resolves through resolveResponsiveValue, and breakpoints.test.ts still pins it from the module. Every other export of both entries is unchanged, so the offline data source family, the presence hooks, the comment thread and their types keep their faces.
• The declarations are right - Clause-②: no (narrowing): no export is added, no declared surface flips; Metadata-protocol: none: no spec schema, node type, conversion surface or stored shape names these runtime exports, which the ADR-0087 not-required (runtime-export-retired …) marker records with every operand listed. Skills-docs: none holds by the grep above.
• The gantt and designer stop is the claim's rule, applied - right. Every zero-hit gantt name and six designer values are taught by their package README, and check-readme-exports binds a README self-import to a real export, so retiring them needs a README edit the claim's surface does not cover; the other nine designer names are …Props of components that stay. Stopping on breach and reporting is what the claim asked for; the card keeps those items.
• Check-runs as gate verdicts - read as above; every run on this head completed and none is red, cancelled or in progress at the read.

② Semver level

• @object-ui/collaboration and @object-ui/mobile patch with the ! title and the **BREAKING** banner - accepted. A ui package (18.2.0) takes patch for a ! retirement by the feat(plugin-detail)!: one record page laid out by its own width; 动态 is the right column's first panel #28116 precedent; each changeset carries one FROM → TO line per retired name, so an external importer reads what to write instead.

③ Boundary flags

• Two helpers are now module-local with no reader - createPresenceUpdater in usePresence.ts and requestBackgroundSync in serviceWorkerSource.ts. Deleting their bodies edits modules the claim does not cover; left for the next pass as the body says. Not owed here.
• CommentSearchResult in @object-ui/types loses its last code reader - another package's batch on the same card; a test in types still names it. Not owed here.
• Partial landing - Part of #25921; the card stays open with the remaining items, and gantt and designer carry no changeset because no line of theirs changes.
• The seat lands - no Tier H path (no docs/adr, no skills/**, 214 lines), no human gate; no name approval owed (nothing is added).

④ Design judgment

• Tried: a host that re-exports @object-ui/mobile whole (it loses the three names at compile time, which is the breaking note's case); a host that only ever imported useResponsive (unchanged); a locale or i18n key tied to the deleted hooks (none: both were pure hooks over a data source and a comment list). Nothing to add.

Verdict reasoning. Eight public names with zero readers anywhere in the tree leave two package entries, their two untested barrel-only modules are deleted, every kept export and its internal readers are untouched, the narrowing and the ADR-0087 marker are declared with every operand, the patch level follows the ui ! precedent with a FROM → TO line per name, the stop on gantt and designer is the claim's own rule, and every check on this head is green or skipped.

Implemented-by: claude/project-thread-e0x3bb
Reviewed-by: session_01L9qibw58ehQ2gRHP2h2Puz

VERDICT: PASS

[objectstack-fleet]

domain:ui#5 · os-sales · session_01L6bhgUhKx1XHzkoikGjRB4

ACCEPT · 2026-10-31T10:18Z · contract review PASS 6542128649 on this head
Head-sha: b6c6ff2

席位核对

1. Shape: six files (+39 / −175): the collaboration and mobile barrels, their two barrel-only modules deleted, one ! changeset per package. Inside the claim's surface (claim 6541128113). Part of #25921, no closing keyword, so the card stays open. Not governed, 214 lines (check-governed-merges --pr 28385: NOT governed).
2. Premise, measured: the at-tier record re-ran the zero-consumer grep on this head for all eight names (zero hits outside the two packages, READMEs included) and confirmed both packages expose only .. Seat read of the barrel diff: createPresenceUpdater, requestBackgroundSync and resolveResponsiveValue leave the entries only, their modules stay; every other export line is unchanged.
3. Gates (check-runs on this head): 65 runs, 46 success, 19 skipped, 0 other; Lint & Repo Gates, TypeScript Type Check, Type Check UI and all 16 Test UI shards success. Contract review: CONTRACT_REVIEW_TIER PASS 6542128649 on b6c6ff29, written by session_01L9qibw58ehQ2gRHP2h2Puz after the seat's handoff 6541842744. (Its signature line reads domain:ui#5 · os-tesla; the seat notes that line is not this seat's session, and takes the record on its form: served tier, full head, ①–④, independent pair.)
4. Deviations accepted: Clause-②: no (narrowing) in the PR body and changesets where the claim wrote no; the narrowing arm is the correct spelling for retired published exports and is what the review face read. gantt and designer retire nothing: their zero-hit names are taught by their own READMEs, outside this claim's surface.
5. Out of scope: the README-taught gantt/designer names, the two now-readerless module helpers, and CommentSearchResult in @object-ui/types stay 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 as remaining batch items (no new card: the card already carries the batch). Skills-docs: none holds (record ①).
6. Changesets, read sentence by sentence: both are patch with the ! title, the **BREAKING** banner, the ADR-0087 not-required (runtime-export-retired …) marker naming all four operands each, and one FROM → TO line per name; each TO names a still-exported API (PresenceProvider, useTenantPresence, useRecordPresence, createOfflineDataSource's pending() / replay() / drop() / clear(), useResponsive) or a platform API. Accepted.
7. Landing: needs:contract-review comes off in this stroke; then pr_ready + automerge_enable through the relay, MERGED confirmed, and the card gets its partial-landing release back to pm:queue.

---

Generated by Claude Code