fix(dashboards): carry dashboard chat context ids, label the chart readout, isolate repository tests (#8499)

* fix(dashboards): carry dashboard chat context ids, label the chart readout, isolate repository tests

- User message contexts keep a dashboard mention's dashboardId, both in the
  optimistic message and when a persisted message is reopened, matching the
  context the server stores.
- The time-series readout row has role="group", so its aria-label is exposed to
  assistive technology.
- The revision test in the dashboard repository suite seeds its own workspace
  instead of depending on the previous test's row.

* fix(dashboards): name the chart readout group only when it has values

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/home/hooks/use-chat.ts

@@ -3423,6 +3423,7 @@ export function useChat(
           ? { viewId: (c.currentView ? c.currentView.viewId : c.viewId) ?? undefined }
           : {}),
         ...('fileId' in c && c.fileId ? { fileId: c.fileId } : {}),
+        ...(c.kind === 'dashboard' ? { dashboardId: c.dashboardId } : {}),
         ...('folderId' in c && c.folderId ? { folderId: c.folderId } : {}),
         ...(c.kind === 'skill' && 'skillId' in c ? { skillId: c.skillId } : {}),
         ...(c.kind === 'integration' && 'blockType' in c ? { blockType: c.blockType } : {}),

apps/sim/components/charts/time-series-chart.tsx

@@ -19,11 +19,13 @@ export function TimeSeriesChart({ label, option, ...config }: TimeSeriesChartPro
   const valuesRef = useRef<HTMLDivElement>(null)
   const edges = useScrollEdges(valuesRef, { axis: 'x' })
   const [readout, setReadout] = useState<ChartReadout | null>(null)
+  const hasValues = Boolean(readout?.values.length)
   return (
     <div className='h-full min-w-0'>
       <div
+        role={hasValues ? 'group' : undefined}
         className='mb-2 flex h-8 min-w-0 items-center justify-between gap-4 text-sm tabular-nums'
-        aria-label={`${label} values`}
+        aria-label={hasValues ? `${label} values` : undefined}
       >
         <div
           ref={valuesRef}

apps/sim/lib/dashboards/repository.integration.ts

@@ -53,14 +53,14 @@ describe('dashboard repository in PostgreSQL', () => {
   })
 
   it('updates only at the expected revision and advances it', async () => {
-    const current = (await getWorkspaceDashboard('ws-a'))!
+    const current = (await insertWorkspaceDashboard('ws-c', 'first', 'user-1'))!
     const updated = await updateDashboardContent(current.id, 'edited', 'user-2', current.revision)
     expect(updated).toMatchObject({
       content: 'edited',
       revision: current.revision + 1,
       updatedBy: 'user-2',
     })
     expect(await updateDashboardContent(current.id, 'stale', 'user-3', current.revision)).toBeNull()
-    expect((await getWorkspaceDashboard('ws-a'))?.content).toBe('edited')
+    expect((await getWorkspaceDashboard('ws-c'))?.content).toBe('edited')
   })
 })

apps/sim/lib/mothership/chat/display-message.test.ts

@@ -208,6 +208,20 @@ describe('display-message', () => {
     ])
   })
 
+  it('keeps the dashboard id of a reopened dashboard mention', () => {
+    const display = toDisplayMessage({
+      id: 'msg-dashboard',
+      role: 'user',
+      content: '@Dashboard',
+      timestamp: '2024-01-01T00:00:00.000Z',
+      contexts: [{ kind: 'dashboard', label: 'Dashboard', dashboardId: 'dashboard-1' }],
+    })
+
+    expect(display.contexts).toEqual([
+      { kind: 'dashboard', label: 'Dashboard', dashboardId: 'dashboard-1' },
+    ])
+  })
+
   it.each(['pending', 'executing', 'awaiting_approval'])(
     'shows a %s row of a stored message as interrupted, not running',
     (state) => {

apps/sim/lib/mothership/chat/display-message.ts

@@ -149,6 +149,7 @@ function toDisplayContexts(
     ...(c.tableId ? { tableId: c.tableId } : {}),
     ...(c.viewId ? { viewId: c.viewId } : {}),
     ...(c.fileId ? { fileId: c.fileId } : {}),
+    ...(c.dashboardId ? { dashboardId: c.dashboardId } : {}),
     ...(c.folderId ? { folderId: c.folderId } : {}),
     ...(c.chatId ? { chatId: c.chatId } : {}),
     ...(c.blockType ? { blockType: c.blockType } : {}),