test(app-shell): stub useRecordPresence as the array the hook returns

.changeset/8083-record-presence-stub-array-shape.md

@@ -0,0 +1,12 @@
+---
+---
+
+Test-only change. Thirteen `RecordDetailView` test files stubbed
+`useRecordPresence` as `{ viewers: [], others: [] }`, while the real hook
+returns `PresenceUser[]` and its only consumer reads `recordPresence.length > 0`.
+An object has no `length`, so the presence row was structurally unrenderable in
+those files: they exercised the no-presence branch through a shape mismatch
+rather than through the empty array the branch is about, and no value of the
+object stub could ever have made the row appear. All thirteen now stub `[]`,
+matching the nine files that already did. No published package changes — the
+edits are confined to `*.test.tsx` files, which no package's `files[]` ships.

packages/app-shell/src/views/RecordDetailView.activityMapIdentity-5878.test.tsx

@@ -84,7 +84,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.activityUnmappedType-5896.test.tsx

@@ -111,7 +111,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.builtinActionSessionUserPredicate.test.tsx

@@ -72,7 +72,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.expandFls-7230.test.tsx

@@ -93,7 +93,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 }));
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 vi.mock('sonner', () => ({

packages/app-shell/src/views/RecordDetailView.feedLoading.test.tsx

@@ -51,7 +51,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.feedRecordScope.test.tsx

@@ -47,7 +47,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.headerActionLookupPredicate.test.tsx

@@ -76,7 +76,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.headerRefresh.test.tsx

@@ -45,7 +45,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.lookupDependsOn-7190.test.tsx

@@ -74,7 +74,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 }));
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 vi.mock('sonner', () => ({

packages/app-shell/src/views/RecordDetailView.relatedListFilter-4664.test.tsx

@@ -76,7 +76,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.relatedListInheritedSort-5795.test.tsx

@@ -74,7 +74,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 

packages/app-shell/src/views/RecordDetailView.sectionHeadingsRenderPath-6190.test.tsx

@@ -72,7 +72,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 }));
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));
 vi.mock('sonner', () => ({

packages/app-shell/src/views/RecordDetailView.userActionPredicates.test.tsx

@@ -70,7 +70,7 @@ vi.mock('@object-ui/auth', async (importOriginal) => ({
 
 vi.mock('@object-ui/collaboration', async (importOriginal) => ({
   ...(await importOriginal<Record<string, unknown>>()),
-  useRecordPresence: () => ({ viewers: [], others: [] }),
+  useRecordPresence: () => [],
   PresenceAvatars: () => null,
 }));