fix(i18n): declare the walk's return-shape change and pin the dashboard filter emitter

The changeset graded `@objectstack/cli` and `@objectstack/platform-objects`
`patch` while the diff moves each package's `src/**` and the PR declares
clause ②; the maintainer ruling of 2026-09-04 (decision batch #35, on #15294)
puts a purely additive widening of a published surface at `minor` or above, so
both are raised. `walkAddressedPageComponents` is published and its return
value changed shape, so the changeset now carries the `**BREAKING**` banner and
exactly one ADR-0087 disposition, which are the only breaking-ness carriers
during the launch window the level number cannot express.

`collectExpectedEntries` gained a `globalFilters` emitter with no test:
`check:i18n-walk-parity` measures at top-level group granularity and
`dashboards` was already walked, so the sub-group could drift green. The new
pin holds the `name`-keyed and `field`-keyed spellings, the option keys, the
unkeyed filter that is passed over, and the `optionsFrom` filter whose label
stays addressable while its fetched rows do not.

Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_018rzQyhLGC5iVs11V3TzRs5

Author: claude

.changeset/i18n-slotted-pages-and-global-filters.md

@@ -1,11 +1,13 @@
 ---
 "@objectstack/spec": minor
-"@objectstack/cli": patch
-"@objectstack/platform-objects": patch
+"@objectstack/cli": minor
+"@objectstack/platform-objects": minor
 ---
 
 Two surfaces the console renders that no translation bundle could address — a `kind: 'slotted'` page's components and a dashboard's global-filter bar — are now addressable (#16772).
 
+**BREAKING** (return shape) — `walkAddressedPageComponents` is a published export of `@objectstack/spec` and its return value is now the rebuilt roots pair `{ regions?, slots? }` where it used to be the regions array alone. A caller that only enumerates components through the visitor and ignores the return value is unaffected. A caller that reads the return value binds `const { regions } = walkAddressedPageComponents(doc, visit)` and reads `regions` exactly as it did before; `slots` is the other half of the same rebuild and is present exactly when the input page authors slots. The bump stays `minor` because the launch-window convention `scripts/check-changeset-no-major.mjs` enforces refuses a `major` while the fixed group is in lockstep — during that window the version number carries nothing about breaking-ness, so this banner and the disposition below are the carriers.
+
 **`walkAddressedPageComponents` widens in both dimensions.** The shared page walk behind `translatePage` and the CLI extractor (`os i18n extract` / `os i18n coverage`) rooted at `regions[].components[]` only and descended `properties.children` only. A slotted record page authors `regions: []` and puts everything under `slots.<slot>`, so the walk visited nothing on it and `pages.<name>` carried exactly two addressable keys however many components the page authored; a `page:tabs` / `page:accordion` keeps its panels' components under `properties.items[].children`, one level deeper than the descended slot, so a related list inside a tab was unreachable on any page kind. The walk now roots at `regions[].components[]` **and** `slots.<slot>` (one component or an array per slot, regions first, then slots in authored order — both root level for the collision arbitration and for the page-name `page:header` route, so a slotted page's `slots.header` is translated as the page's header), and descends `properties.children` **and** `properties.items[].children` (matched by shape, so a custom container speaking the same vocabulary is walked too; `body` / `footer` remain undescended — a renderer back-compat fallback, not an authorable spelling). The depth cap, the cycle guard and the ruled id arbitration are unchanged.
 
 - Signature: the parameter is `AddressedPageRoots` (= `Pick<PageLike, 'regions' | 'slots'>`) instead of `Pick<PageLike, 'regions'>`, and the walk returns the rebuilt roots pair `{ regions?, slots? }` (each key present exactly when present on the input) instead of the regions array alone. `PageLike` gains `slots`. An enumeration-only consumer that ignores the return value needs no change; a consumer reading the returned regions destructures `{ regions }`.
@@ -16,3 +18,7 @@ Two surfaces the console renders that no translation bundle could address — a
 **`@objectstack/cli`:** `os i18n extract` offers `dashboards.<name>.globalFilters.<key>.label` / `.options.<value>` for every static filter, and `pages.<name>.title` / `.subtitle` for a `page:header` at any root (a slotted page's `slots.header` included) — the component keys under `slots` and tab panels follow from the shared walk with no extractor change.
 
 **`@objectstack/platform-objects`:** the shipped Setup bundles (`en`, `zh-CN`, `ja-JP`, `es-ES`) carry the new `dashboards.<name>.globalFilters.created_at.label` entry for the system-overview dashboard's date-range filter, which authors no `name` and is therefore keyed by its `field`.
+
+**Why no ADR-0087 ledger entry.** Nothing an author writes moves. The authorable side is purely additive — `dashboards.<name>.globalFilters.<key>` is a new optional group and every bundle that was valid before is valid unchanged — no spec key is retired, no stored `sys_metadata` shape changes, and no conversion or migration id is touched, so `objectstack migrate meta` has nothing to act on. The one incompatible surface is a published function's TypeScript return type, which reaches every affected consumer through the compiler.
+
+<!-- adr-0087: not-required (no-migration-prescription) nothing authorable moves: the bundle group is additive, no spec key is retired and no stored metadata shape changes; the one incompatible surface is a published function's TypeScript return type, which the compiler reports and no ledger entry can act on -->

packages/cli/test/i18n-extract.test.ts

@@ -275,6 +275,96 @@ describe('collectExpectedEntries', () => {
   });
 });
 
+describe('collectExpectedEntries — dashboard global filters (#16772)', () => {
+  // The extractor's `globalFilters` emitter is the CLI half of the bundle
+  // group `dashboards.<name>.globalFilters.<key>`. `check:i18n-walk-parity`
+  // measures at TOP-LEVEL group granularity and `dashboards` was already
+  // walked before this group existed, so nothing that gate sees changes when
+  // a sub-group emitter drifts — this pin is what holds it.
+  const dashboardConfig: any = {
+    dashboards: [
+      {
+        name: 'ops',
+        label: 'Operations',
+        globalFilters: [
+          {
+            // `name` present: it is the key, and `field` is NOT.
+            name: 'dept',
+            field: 'department',
+            label: 'Requesting Department',
+            options: [
+              { value: 'eng', label: 'Engineering' },
+              { value: 'sales', label: 'Sales' },
+            ],
+          },
+          {
+            // No `name`: keyed by `field`. Not a lenient fallback —
+            // `GlobalFilterSchema.name` is declared as defaulting to `field`,
+            // so this IS the filter's key everywhere the platform reads it.
+            field: 'created_at',
+            label: 'Date Range',
+          },
+          {
+            // `optionsFrom` rows are fetched at runtime, so no option of this
+            // filter is addressable from a bundle. Its own `label` is authored
+            // text and stays addressable — the two halves are pinned apart.
+            name: 'owner',
+            field: 'owner_id',
+            label: 'Owner',
+            optionsFrom: { object: 'user', valueField: 'id', labelField: 'name' },
+          },
+          {
+            // Neither key: `globalFilterKey` is undefined and the filter is
+            // passed over rather than offered under a made-up key.
+            label: 'Unkeyed',
+            options: [{ value: 'x', label: 'X' }],
+          },
+        ],
+      },
+    ],
+  };
+
+  const filterPaths = () =>
+    collectExpectedEntries(dashboardConfig)
+      .map((e) => e.path.join('.'))
+      .filter((p) => p.startsWith('dashboards.ops.globalFilters.'))
+      .sort();
+
+  it('offers the filter label and every static option, keyed by `name` else `field`', () => {
+    expect(filterPaths()).toEqual([
+      'dashboards.ops.globalFilters.created_at.label',
+      'dashboards.ops.globalFilters.dept.label',
+      'dashboards.ops.globalFilters.dept.options.eng',
+      'dashboards.ops.globalFilters.dept.options.sales',
+      'dashboards.ops.globalFilters.owner.label',
+    ]);
+  });
+
+  it('carries the authored source values, and keys an option by its `value`', () => {
+    const byPath = Object.fromEntries(
+      collectExpectedEntries(dashboardConfig).map((e) => [e.path.join('.'), e.sourceValue]),
+    );
+    expect(byPath['dashboards.ops.globalFilters.dept.label']).toBe('Requesting Department');
+    expect(byPath['dashboards.ops.globalFilters.dept.options.eng']).toBe('Engineering');
+    expect(byPath['dashboards.ops.globalFilters.dept.options.sales']).toBe('Sales');
+    expect(byPath['dashboards.ops.globalFilters.created_at.label']).toBe('Date Range');
+  });
+
+  it('offers nothing under a `field` a named filter overrode, and nothing for an unkeyed one', () => {
+    const paths = filterPaths();
+    // `name: 'dept'` won, so the field spelling addresses nothing.
+    expect(paths).not.toContain('dashboards.ops.globalFilters.department.label');
+    // The unkeyed filter contributes no entry at all, under any spelling.
+    expect(paths.some((p) => p.endsWith('.options.x'))).toBe(false);
+  });
+
+  it('offers no option key for an `optionsFrom` filter — the rows are fetched, not authored', () => {
+    const paths = filterPaths();
+    expect(paths).toContain('dashboards.ops.globalFilters.owner.label');
+    expect(paths.filter((p) => p.startsWith('dashboards.ops.globalFilters.owner.options.'))).toEqual([]);
+  });
+});
+
 describe('extractTranslations', () => {
   it('fills the default locale from schema and emits empty strings for other locales', () => {
     const { bundles, counts } = extractTranslations(config, {