refactor(app-shell): move ObjectView's view-options family into objectViewOptions.ts (slice 2) (#26233)

<!-- claude-projects-attribution: {"github_login":"os-steve"} -->
_Requested by **Steve** · [project
thread](https://claude.ai/code/project/chan_01FdL1XeUyvVNxLreh3F4Qi9?thread=cmsg_01FdL1XeUyvVNxLreh3F4Qi9VtKf3F9dzJPJM4RRtABaEc)_

Part of #25992 (slice 2 of the move-only split; slice 1 landed as
#26152, and the card stays open for any later slice).

Clause-②: no
Metadata-protocol: none

## What moved

Base `d00215f6` (slice 1 merged). This slice moves the view-options
family and the view-patch dispatcher, verbatim, out of
`ui/packages/app-shell/src/views/ObjectView.tsx` into a new sibling
module `ui/packages/app-shell/src/views/objectViewOptions.ts`:

- `timelineViewOptions`, `calendarViewOptions`, `ganttViewOptions`,
`galleryViewOptions`, `kanbanViewOptions` (with the shared docblocks
above them)
- `storedLegacyOptions`
- `isSameOptionsValue` (was file-local; it gains the `export` keyword
because the component still calls it, and `ObjectView.tsx` does NOT
re-export it, so no new name is published from there)
- `dispatchViewPatches`

`ObjectView.tsx` imports all eight back and re-exports the seven names
it exported before, from the same path. The package entry is untouched.
`detectStatusField` is imported by the new module instead of
`ObjectView.tsx` (its only reader moved). `ObjectView.tsx` goes from
3689 to 3254 lines.

Move-only proof: the multiset of non-blank lines of the base
`ObjectView.tsx` equals new `ObjectView.tsx` plus
`objectViewOptions.ts`, after normalising the one added `export`
keyword. Nothing is missing; the only added lines are the new module's
header comment, the import and re-export blocks, and their one-line
comment.

## Test edits (all forced by the move, as the claim allows)

1. `ObjectView.calendarBinding-7029.test.tsx`, the "no invented calendar
field name survives in the source" scan
2. `ObjectView.galleryBinding-7547.test.tsx`, the "no invented gallery
cover name survives in the source" scan
3. `ObjectView.ganttBinding-7070.test.tsx`, the "no invented gantt date
name survives in the source" scan
4. `ObjectView.titleFieldConvergence.test.tsx`, the "the seven seams
share ONE expression shape" scan

Each `readFileSync` of `ObjectView.tsx` now reads `ObjectView.tsx` and
`objectViewOptions.ts` and joins them. Reading both, not only the new
module, keeps every scan over exactly the lines it read before: the
titleFieldConvergence count ("exactly six") spans four seams that moved
and two (map, tree) that stay in the component, and the
calendar/gantt/gallery tripwires keep covering the component's own
relay. No assertion changed.

5. `writeRefusalCensus-11583.test.ts`: the `LEDGER` row for
`dispatchViewPatches` (callee `updateView`, kind `relays`) is keyed
`file: 'objectViewOptions.ts'` instead of `'ObjectView.tsx'`. The
`handleSetDefaultView` row that calls it stays on `ObjectView.tsx`.

No other test file changed.

## Verification

See the os-dev-report comment on the card for the full readings. In
short: every test file under `ui/packages/app-shell/src/views/` plus the
7 source-reading tests outside it passed (1160 files); app-shell
`type-check`; eslint on the 7 touched files with 0 errors; the 35
derived dispatch gates; a Console build after merging the latest main.

## Acceptance notes

- Some moved prose still says "below" or "above" about the relay in
`ObjectView.tsx` (for example the `storedLegacyOptions` docblock's "The
relay below"). It is kept verbatim because the slice is move-only; the
relay is now in the sibling file.
- `ObjectView.unboundAdapterCall.test.ts` still scans only
`ObjectView.tsx` for a detached adapter write. `dispatchViewPatches`
itself calls through the object, and the census row above still pins it.

Changeset: `.changeset/25992-objectview-options-split.md` (patch,
`@object-ui/app-shell`).

---
_Generated by [Claude
Code](https://claude.ai/code/session_01QciHBvFJXnhZqXmdqpDtpz)_

Co-authored-by: Claude <noreply@anthropic.com>

Author: objectstack-fleet[bot]

.changeset/25992-objectview-options-split.md

@@ -0,0 +1,12 @@
+---
+'@object-ui/app-shell': patch
+---
+
+refactor(app-shell): the Console `ObjectView`'s view-options family moves into a sibling module
+
+Clause-②: no
+Metadata-protocol: none
+
+The per-kind `options` blocks `views/ObjectView.tsx` hands `ListView` (`timelineViewOptions`, `calendarViewOptions`, `ganttViewOptions`, `galleryViewOptions`, `kanbanViewOptions`), `storedLegacyOptions`, the by-value options comparison and `dispatchViewPatches` move verbatim into `objectViewOptions.ts` beside it. No logic, name or rendered output changes. `ObjectView.tsx` re-exports every name it exported before, and the package entry is unchanged.
+
+What to do: nothing.

ui/packages/app-shell/src/views/ObjectView.calendarBinding-7029.test.tsx

@@ -106,10 +106,11 @@ describe('calendarViewOptions — the object page forwards, it does not invent (
 });
 
 describe('no invented calendar field name survives in the source (objectui#7029)', () => {
-  const SOURCE = readFileSync(
-    path.join(path.dirname(fileURLToPath(import.meta.url)), 'ObjectView.tsx'),
-    'utf8',
-  );
+  // This face spans `ObjectView.tsx` and the view-options family it imports
+  // from `objectViewOptions.ts` (objectstack#25992 slice 2), so the scan reads both.
+  const SOURCE = ['ObjectView.tsx', 'objectViewOptions.ts']
+    .map((file) => readFileSync(path.join(path.dirname(fileURLToPath(import.meta.url)), file), 'utf8'))
+    .join('\n');
 
   /**
    * Executable lines only. The prose above this file's own seams names

ui/packages/app-shell/src/views/ObjectView.galleryBinding-7547.test.tsx

@@ -152,10 +152,11 @@ describe('galleryViewOptions — the object page forwards, it does not invent (o
 });
 
 describe('no invented gallery cover name survives in the source (objectui#7547)', () => {
-  const SOURCE = readFileSync(
-    path.join(path.dirname(fileURLToPath(import.meta.url)), 'ObjectView.tsx'),
-    'utf8',
-  );
+  // This face spans `ObjectView.tsx` and the view-options family it imports
+  // from `objectViewOptions.ts` (objectstack#25992 slice 2), so the scan reads both.
+  const SOURCE = ['ObjectView.tsx', 'objectViewOptions.ts']
+    .map((file) => readFileSync(path.join(path.dirname(fileURLToPath(import.meta.url)), file), 'utf8'))
+    .join('\n');
 
   /**
    * Executable lines only. The prose above this file's own seams names

ui/packages/app-shell/src/views/ObjectView.ganttBinding-7070.test.tsx

@@ -116,10 +116,11 @@ describe('ganttViewOptions — the object page forwards, it does not invent (obj
 });
 
 describe('no invented gantt date name survives in the source (objectui#7070)', () => {
-  const SOURCE = readFileSync(
-    path.join(path.dirname(fileURLToPath(import.meta.url)), 'ObjectView.tsx'),
-    'utf8',
-  );
+  // This face spans `ObjectView.tsx` and the view-options family it imports
+  // from `objectViewOptions.ts` (objectstack#25992 slice 2), so the scan reads both.
+  const SOURCE = ['ObjectView.tsx', 'objectViewOptions.ts']
+    .map((file) => readFileSync(path.join(path.dirname(fileURLToPath(import.meta.url)), file), 'utf8'))
+    .join('\n');
 
   /**
    * Executable lines only. The prose above this file's own seams names

ui/packages/app-shell/src/views/ObjectView.titleFieldConvergence.test.tsx

@@ -312,10 +312,11 @@ describe('ObjectView view-config `titleField` — the middle rung is gone (objec
 });
 
 describe('the seven seams share ONE expression shape (objectui#6557)', () => {
-  const SOURCE = readFileSync(
-    path.join(path.dirname(fileURLToPath(import.meta.url)), 'ObjectView.tsx'),
-    'utf8',
-  );
+  // This face spans `ObjectView.tsx` and the view-options family it imports
+  // from `objectViewOptions.ts` (objectstack#25992 slice 2), so the scan reads both.
+  const SOURCE = ['ObjectView.tsx', 'objectViewOptions.ts']
+    .map((file) => readFileSync(path.join(path.dirname(fileURLToPath(import.meta.url)), file), 'utf8'))
+    .join('\n');
 
   /** Every `titleField:` / `labelField:` assignment in the file. */
   const seamLines = SOURCE.split('\n').filter((l) => /^\s*(titleField|labelField):/.test(l));

ui/packages/app-shell/src/views/ObjectView.tsx

@@ -83,7 +83,7 @@ const LEDGER: Row[] = [
     reason: '"Save as view": the refusal is a toast with the door\'s message, and the dialog stays open (objectui#11578).',
   },
   {
-    file: 'ObjectView.tsx', symbol: 'dispatchViewPatches', callee: 'updateView', calls: 1,
+    file: 'objectViewOptions.ts', symbol: 'dispatchViewPatches', callee: 'updateView', calls: 1,
     kind: 'relays',
     reason: 'Issues one `updateView` per patch and hands every promise back; set-as-default and reorder await them and answer for the refusal.',
   },