fix(dashboards): split diagnostics, switch panel query modes, return create revision

- One diagnostics entry per parse error instead of a newline-joined string.
- A panel choosing aggregate or columns drops the other mode's inherited
  dashboard fields, so shared defaults serve table and aggregate panels.
- The v2 create response returns the revision it produced, like the replace and
  edit responses.
- The analytics route test uses the shared createMockRequest helper.

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

Author: TheodoreSpeaks

apps/docs/openapi-v2-files-audit.json

@@ -4116,6 +4116,10 @@
             "format": "date-time",
             "examples": ["2026-01-16T09:00:00Z"]
           },
+          "revision": {
+            "description": "Opaque token for the content this write produced. Send it back as `expectedRevision` on the next write. Absent for a file with no recorded content version.",
+            "type": "string"
+          },
           "diagnostics": {
             "description": "For a dashboard file, the YAML parse errors in the written content. Empty when it parses; the file is saved either way. Absent for other file types.",
             "type": "array",
@@ -4139,7 +4143,7 @@
         ],
         "additionalProperties": false,
         "title": "Created file",
-        "description": "A newly created workspace file, with any dashboard parse errors."
+        "description": "A newly created workspace file, with the revision it produced and any dashboard parse errors."
       },
       "V2CreatedFileResponse": {
         "type": "object",
@@ -4152,7 +4156,7 @@
         "required": ["data"],
         "additionalProperties": false,
         "title": "Created file response",
-        "description": "A newly created workspace file, with any dashboard parse errors.",
+        "description": "A newly created workspace file, with the revision it produced and any dashboard parse errors.",
         "examples": [
           {
             "data": {

apps/sim/app/api/table/[tableId]/analytics/route.test.ts

@@ -1,6 +1,6 @@
 import { authMockFns } from '@sim/testing/mocks/auth.mock'
 import { rateLimiterMock, rateLimiterMockFns } from '@sim/testing/mocks/rate-limiter.mock'
-import { NextRequest } from 'next/server'
+import { createMockRequest } from '@sim/testing/mocks/request.mock'
 import { beforeEach, describe, expect, it, vi } from 'vitest'
 import { POST } from '@/app/api/table/[tableId]/analytics/route'
 
@@ -24,11 +24,7 @@ const body = {
   },
 }
 const request = (value: unknown) =>
-  new NextRequest('http://localhost/api/table/tbl_test/analytics', {
-    method: 'POST',
-    headers: { 'content-type': 'application/json' },
-    body: JSON.stringify(value),
-  })
+  createMockRequest({ method: 'POST', url: '/api/table/tbl_test/analytics', body: value })
 beforeEach(() => {
   mocks.session.mockResolvedValue({ user: { id: 'viewer' }, session: { id: 'session' } })
   mocks.limit.mockResolvedValue(null)

apps/sim/app/api/v2/files/route.ts

@@ -9,6 +9,7 @@ import { defineV2JsonRoute, v2ApiKeyAuth, v2RateLimits } from '@/lib/api/server/
 import { getFileExtension, getMimeTypeFromExtension } from '@/lib/uploads/utils/file-utils'
 import { v2FileErrorPolicies } from '@/lib/workspace-files/api'
 import { createWorkspaceFile } from '@/lib/workspace-files/application/create-workspace-file'
+import { workspaceFileRevisionField } from '@/lib/workspace-files/application/file-revision'
 import { queryWorkspaceFilePage } from '@/lib/workspace-files/application/list-workspace-files'
 import { fileOperations } from '@/lib/workspace-files/application/operations'
 import { MAX_WORKSPACE_FILE_INLINE_BODY_BYTES } from '@/lib/workspace-files/orchestration'
@@ -94,6 +95,10 @@ export const POST = defineV2JsonRoute({
   }),
   useCase: createWorkspaceFile,
   present: async ({ file, diagnostics }) => ({
-    data: { ...(await toV2File(file)), ...(diagnostics ? { diagnostics } : {}) },
+    data: {
+      ...(await toV2File(file)),
+      ...workspaceFileRevisionField(file),
+      ...(diagnostics ? { diagnostics } : {}),
+    },
   }),
 })

apps/sim/lib/api/contracts/v2/files.ts

@@ -767,11 +767,12 @@ const fileDiagnosticsSchema = z
   )
 
 export const v2CreatedFileSchema = v2FileSchema
-  .extend({ diagnostics: fileDiagnosticsSchema })
+  .extend({ revision: writtenFileRevisionSchema, diagnostics: fileDiagnosticsSchema })
   .meta({
     id: 'V2CreatedFile',
     title: 'Created file',
-    description: 'A newly created workspace file, with any dashboard parse errors.',
+    description:
+      'A newly created workspace file, with the revision it produced and any dashboard parse errors.',
   })
 
 export const v2CreateFileContract = defineRouteContract({

apps/sim/lib/api/contracts/v2/openapi/files-audit.ts

@@ -210,7 +210,7 @@ const declaredRoutes = [
         v2CreateFileContract.response.schema,
         'V2CreatedFileResponse',
         'Created file response',
-        'A newly created workspace file, with any dashboard parse errors.',
+        'A newly created workspace file, with the revision it produced and any dashboard parse errors.',
         [{ data: FILE_EXAMPLE }]
       ),
     }

apps/sim/lib/dashboards/diagnostics.ts

@@ -12,5 +12,5 @@ export function dashboardDiagnostics(
 ): string[] | undefined {
   if (contentType !== DASHBOARD_CONTENT_TYPE) return undefined
   const { error } = parseDashboardSpec(content.toString())
-  return error ? [error] : []
+  return error ? error.split('\n') : []
 }

apps/sim/lib/dashboards/spec.test.ts

@@ -111,6 +111,19 @@ describe('UTC ranges', () => {
   )
 })
 
+describe('panel query modes', () => {
+  it('lets a panel switch between detail columns and aggregates inherited from the dashboard', () => {
+    const detailDefault = parseDashboardSpec(
+      'title: T\nsource: {tableId: tbl_1, columns: [status]}\nblocks:\n  - stat: Total\n    source: {aggregate: {n: {op: count}}}\n'
+    )
+    expect(detailDefault.error).toBeUndefined()
+    const aggregateDefault = parseDashboardSpec(
+      'title: T\nsource: {tableId: tbl_1, groupBy: [status], aggregate: {n: {op: count}}}\nblocks:\n  - table: Rows\n    source: {columns: [status]}\n'
+    )
+    expect(aggregateDefault.error).toBeUndefined()
+  })
+})
+
 describe('dashboard parse errors', () => {
   it('names the block path, the allowed block kinds and any unknown key', () => {
     expect(parseDashboardSpec('title: Probe\nblocks:\n  - bogus: 1\n').error).toBe(

apps/sim/lib/dashboards/spec.ts

@@ -1,4 +1,5 @@
 import { getErrorMessage } from '@sim/utils/errors'
+import { omit } from '@sim/utils/object'
 import { JSON_SCHEMA, load } from 'js-yaml'
 import { z } from 'zod'
 import { parseChartSpec } from '@/lib/charts/spec'
@@ -88,11 +89,20 @@ const dashboardSchema: z.ZodType<DashboardSpec> = z
   })
   .strict()
 
+/**
+ * A panel's source shallowly overrides the dashboard's. Choosing a query mode (`aggregate` or
+ * `columns`) drops the other mode's inherited fields, so shared defaults serve both panel kinds.
+ */
 export function resolveDashboardSource(
   defaults: DashboardSource | undefined,
   source: DashboardSource | undefined
 ): ResolvedDashboardSource {
-  const merged = { ...defaults, ...source }
+  const inherited = source?.columns
+    ? omit(defaults ?? {}, ['aggregate', 'groupBy', 'bucket'])
+    : source?.aggregate
+      ? omit(defaults ?? {}, ['columns'])
+      : defaults
+  const merged = { ...inherited, ...source }
   if (!merged.tableId)
     throw new Error('A data panel requires source.tableId, on the dashboard or on the panel')
   return { ...merged, tableId: merged.tableId }

packages/sim-cli/src/generated/v2-api.ts

@@ -1702,6 +1702,7 @@ type CreateFileResponseRef0 = {
   uploadedAt: string
   updatedAt: string
   deletedAt: string | null
+  revision?: string
   diagnostics?: Array<string>
 }