[objectstack-fleet]

Requested by Steedos · project thread

Fixes #28680
Fixes #28645

Clause-②: no
Metadata-protocol: none

维护者速读

• 改了什么:页面或仪表盘在控制台被创建(首次发布)或删除时,由该处直接写入应用导航;NavigationSyncEffect 不再在每个应用页读取整张 /meta/page 和 /meta/dashboard 列表做差异比较。
• 为什么改:这是 console: on an app page, answer /meta/view, /meta/page and /meta/dashboard from the app bundle (#28321) instead of three list reads; page/pricing 24 to ~21 calls #28645 的第二步,按维护者裁定 A([Decision] console: write app navigation when a page or dashboard is created or deleted, instead of diffing the whole page and dashboard lists on every app page #28680)。去掉这两次列表读取后,页面路由 26 → 24 次请求,对象列表路由 33 → 31。
• 风险与回滚:在本标签页之外创建或删除的无包页面/仪表盘(服务端智能体、另一个标签页、直接调用 REST)不再自动进入导航,只能由其写入方处理;包安装和 CLI 部署的条目带 _packageId,原本就不经过差异比较。回滚即还原本 PR,无数据或协议变更。
• 先看:ui/packages/app-shell/src/hooks/useNavigationSync.ts(useNavigationWrites)、ui/packages/app-shell/src/views/metadata-admin/ResourceEditPage.tsx(doPublish、runReset)、ui/packages/app-shell/src/preview/usePublishAllDrafts.ts。
• 两分钟验证:cd ui && pnpm exec vitest run packages/app-shell/src/views/metadata-admin/ResourceEditPage.navigationWrites-28680.test.tsx packages/app-shell/src/hooks/__tests__/useNavigationSync.test.tsx,预期 22 个用例全绿。
• 需要你做的:无,席位落地。仪表盘路由仍读 /meta/dashboard(见 Deviations),可另开卡处理。

What changes

NavigationSyncEffect read the whole /meta/page and /meta/dashboard lists on every app page, after the first paint, only to diff their name sets and add or remove navigation entries. Maintainer ruling A on #28680 retires it. The console's sites now write navigation themselves, through a new door in useNavigationSync.ts:

• useNavigationWrites() returns created(type, name, item?) and deleted(type, name, item?). They call the existing sync…CreatedAllApps / sync…DeletedAllApps writers and keep the diff's rules: only a page or a dashboard; never a platform artifact (isPlatformArtifact: a sys_ name, or _packageId / _provenance: 'package'); nothing in draft preview (ADR-0037, where apps may be draft bodies); only into apps that may be written (isNavigationSyncableApp). Entries are written label-less, as before (objectui#9868).
• The metadata editor's Publish (ResourceEditPage doPublish): when the item held no layer before the publish (its draft was its only layer, /layers 404) and the editor is not bound to a package, the page or dashboard is created and joins navigation. A later publish of the same item is an update and writes nothing.
• The metadata editor's Delete (runReset with no baseline): an unbound real delete removes the item from navigation. A reset to a packaged baseline writes nothing.
• Publish all (usePublishAllDrafts): before publishing each package-less page or dashboard draft, it asks the item's layers. The ones that had none are created and join navigation after the batch. A failed layers read is not taken for a create.
• A create no longer duplicates an entry. syncPageCreated / syncDashboardCreated skip an app whose navigation already references the target at any depth (navigationReferences). addNavigationItem itself still appends.
• NavigationSyncEffect stays exported, @deprecated, and renders nothing. AppContent no longer mounts it.

No rename site exists in the console today, so the sync…RenamedAllApps writers keep no caller.

Who relied on the diff (the claim's pre-code check)

Nothing created outside the console. The diff dropped every platform artifact, and package installs and CLI deploys carry _packageId (checked on the running showcase: showcase_start_here's layers answer _packageId: 'com.example.showcase', _provenance: 'package'). Studio always saves with a package. The diff acted only on package-less pages or dashboards that appeared or vanished in this tab's lists mid-session, and its baseline re-seeded on every load.

What no longer reaches navigation: a package-less page or dashboard created or deleted outside this tab (a server-side agent, another tab, a raw REST call). It now reaches navigation only through its own writer. The changeset says the same.

Tests

• New ResourceEditPage.navigationWrites-28680.test.tsx (5 cases, the real page): the first publish adds the page to the app once and a second publish adds no second entry; publishing an item that already had a layer writes nothing; a package-bound publish writes nothing; a real delete removes the entry; a reset leaves navigation alone.
• New usePublishAllDrafts.navigationWrites-28680.test.tsx (2 cases, real MetadataClient): new package-less page and dashboard join; an updated page, a packaged page and a flow do not; layers are asked before each publish; a failed layers read writes nothing.
• useNavigationSync.test.tsx: the diff's cases are replaced by useNavigationWrites cases (platform artifacts in both directions, other types, draft preview, locked apps, a second create of a referenced page or dashboard), navigationReferences, and a pin that NavigationSyncEffect reads neither list and writes nothing.
• labelInherits-9868, standardEntryLocalizes-11201 and chromeReads.afterFirstPaint-12980 now drive the new door, or pin that the retired effect reads nothing.
• Ablations, each one red: always-first-publish, publish ignoring the package binding, no create write, no delete write, delete on reset too, publish-all without the layers check, publish-all without the write, writer without dedupe, no preview skip, no platform skip, no type filter (11 of 11).

Acceptance notes

Measured on a production console build (pnpm build, pnpm console:build, pnpm dev -- --fresh), showcase_app, admin, API calls of a warm reload, two runs each with the same count:

Route	Base 2678600d	PR 1 (#28754)	This PR	Difference from PR 1
page/showcase_start_here	26	26	24	/meta/page −1, /meta/dashboard −1
dashboard/showcase_ops_dashboard	35	35	34	/meta/page −1
showcase_task (object list)	33	33	31	/meta/page −1, /meta/dashboard −1

On the running server, a page saved as a draft answers /layers 404 and its first publish turns it into a layer with resettable: false, which is the editor's create and delete verdict this PR reads. The probe page was deleted afterwards.

Deviations:

• The dashboard route is 34, not the expected 33. Its own route gate in AppContent waits for the dashboard list (routeReads), and the dashboard view reads its definition from it. The effect shared that one request, so removing the effect saves only /meta/page there. Reading the dashboard route bundle-first, as PR 1 did for the page route, would remove it. That is AppContent's route gate and the dashboard view, outside this claim's surface, so it is left for a follow-up card.
• Publish all from the draft-preview bar writes no navigation. The door writes nothing in preview, where apps may be draft bodies. The diff wrote nothing there either.
• AppContent.tsx changes only to drop the mount. useNavTargetLabel.ts keeps a comment that says the console's NavigationSyncEffect loads the dashboard list; it is outside the surface and left as it is.

Verification

Derived gates (34)

Run at f31794f6 (this PR merged with origin/main at cd0ec76b). dispatch-gates --ran: 34 derived, 34 run, 0 NOT-MEASURED.

node scripts/check-adr-0087-registration.mjs --base origin/main :: exit 0
node scripts/check-adr-0087-registration.mjs --self-test :: exit 0
node scripts/check-changeset-no-major.mjs --base origin/main :: 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
node scripts/check-empty-changeset.mjs --base origin/main :: exit 0
node scripts/check-empty-changeset.mjs --self-test :: exit 0
node scripts/check-sdui-manifest.mjs :: exit 0
node scripts/check-sdui-manifest.mjs --self-test :: exit 0
node scripts/pm/release-rehearsal-clone.mjs --self-test :: exit 0
node scripts/release-pending-publish.mjs --self-test :: exit 0
pnpm --dir ui exec vitest run scripts/ :: exit 0
pnpm --filter @objectstack/spec run check:duration-unit-keys :: exit 0
pnpm check:changeset-gate-self-tests :: exit 0
pnpm check:closed-ceremony-text :: exit 0
pnpm check:cross-package-test-inputs :: exit 0
pnpm check:dispatcher-error-vocabulary :: exit 0
pnpm check:driver-memory-census :: exit 0
pnpm check:engine-double-contract :: exit 0
pnpm check:gitlink-declared :: exit 0
pnpm check:nul-bytes :: exit 0
pnpm check:pm-changeset-deadline-census :: exit 0
pnpm check:published-files :: exit 0
pnpm check:query-options-erasure :: exit 0
pnpm check:refd-timer-probe :: exit 0
pnpm check:skill-identifier-liveness :: exit 0
pnpm check:test-source-alias :: exit 0
pnpm check:tier-file-adoption :: exit 0
pnpm check:type-check-coverage :: exit 0
pnpm check:type-check-debt :: exit 0
pnpm check:undeclared-dep-imports :: exit 0
pnpm check:watch-hint-literal :: exit 0
pnpm check:where-matcher :: exit 0

Tests and type-check

• 295 test files naming the touched modules (useNavigationSync, ResourceEditPage, usePublishAllDrafts, AppContent, DraftPreviewBar, HomePage), 5 locked batches at --maxWorkers=3: 295 passed, 2493 tests. The full app-shell suite is left to CI's Test UI shards.
• pnpm turbo run type-check --filter=@object-ui/app-shell: 65/65 tasks, exit 0.
• eslint --quiet over the changed files: exit 0.
• Merged origin/main at cd0ec76b with no overlap on touched files and no parallel fix found.

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; pnpm lint passes. (run as eslint --quiet over the changed ui files: 0 errors; the whole-workspace lint is left to CI)
• 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. (except the 维护者速读 block the dispatch rules require)
• The Before Submitting list in CONTRIBUTING.md is done.

🤖 Generated with Claude Code

https://claude.ai/code/session_01QsR2jUjXYpqdaAkEWBHbqG

[objectstack-fleet]

domain:ui#1 · zhuangjianguo · session_01QsR2jUJ19MstFvFhjgiYTu

席位核对 · ACCEPT · 2026-11-01T20:28Z

Head-sha: afd4ebd6

• Shape: draft → main, first content lines Fixes #28680 and Fixes #28645 (PR 2 of 2; PR 1 perf(console): answer an app page's views, page and dashboard reads from the app bundle first #28754 landed as 706ac52d). 20 files, +798 / −450 (1248 lines ≤ 3000), no governed path (check-governed-merges --pr 28783: 0 of 20). Clause-②: no / Metadata-protocol: none line-initial in body and changeset, as claimed (6558711281, amended 6562895040). @object-ui/app-shell is private, so deleting NavigationSyncEffect narrows nothing published.
• Read in the diff (ruling A, 13:37Z): NavigationSyncEffect and its whole-list diff are deleted and AppContent no longer mounts them, so app routes stop reading /meta/page and /meta/dashboard for navigation. useNavigationWrites() keeps the diff's rules (page and dashboard only, no platform artifact, not in preview, writable apps) and writes navigation at ResourceEditPage's first publish (no layer before) and real delete (no baseline), and in Publish all for package-less page/dashboard drafts whose layers answered 404. syncPageCreated / syncDashboardCreated skip an app that already references the target; a pin shows a second publish adds no second nav item. No rename site exists in the console, so the rename writers stay uncalled.
• Body corrections (the seat does not patch dev bodies): the body says the effect "stays exported, @deprecated"; at this head it is deleted (commit 21428086, the seat's review ask), with the comments that named it reworded. The dashboard route's remaining /meta/dashboard read (34, not 33) is filed as console: the dashboard route reads its dashboard from the app bundle, so an app's dashboard open stops reading the /meta/dashboard list #28784.
• Lost under ruling A, stated in the body and changeset: a package-less page or dashboard created or deleted mid-session outside this tab (another tab, a server-side agent, raw REST) reaches navigation only through its own writer. Package installs, CLI deploys and Studio saves carry a package and never reached navigation through the diff.
• Measured (dev, warm reload, production build, showcase_app): page 26 → 24, object list 33 → 31, dashboard 35 → 34. With PR 1 this answers console: on an app page, answer /meta/view, /meta/page and /meta/dashboard from the app bundle (#28321) instead of three list reads; page/pricing 24 to ~21 calls #28645's page open: the three list reads are gone from it; the /packages +1 stays as recorded at PR 1.
• Gates (check-runs on this head): 44 success, 11 skipped, 0 failed or pending. mergeable_state clean.
• Tests (dev): 58 affected files / 367 tests at this head, 295 files / 2493 tests before the review push; 11 of 11 ablations red then restored; type-check app-shell + react 66/66; derived gates 34/34 exit 0.
• Out of scope / NOT MEASURED: no live Console run by the seat.
• Landing: ordinary queue landing, pr_ready then automerge_enable. The two Fixes close [Decision] console: write app navigation when a page or dashboard is created or deleted, instead of diffing the whole page and dashboard lists on every app page #28680 and console: on an app page, answer /meta/view, /meta/page and /meta/dashboard from the app bundle (#28321) instead of three list reads; page/pricing 24 to ~21 calls #28645 on merge.