feat(table): model wire keying and actor attribution on row write use cases

The row write use cases assumed every caller speaks column names. That holds
for /api/v2, /api/v1 and the Copilot tools, but not for the first-party grid
or the internal /api/table routes, which address cells by stable storage id.
Feeding id-keyed data through the name remap drops every key it does not
recognise — a storage id names no column name — so the write would store
nothing and still report success.

Make the wire an explicit, required property of the input rather than an
assumption. `dataKeying: 'names' | 'ids'` sits alongside `strictWrite` and is
required for the same reason: a new write surface must state which contract it
publishes. Strictness now means the same thing on either wire — an unknown
column id is refused exactly as an unknown column name already was.

Single-row writes also gain optional actor attribution, so the acting tab can
skip refetching its own write. It is optional and absent by default, so every
existing caller keeps broadcasting to all subscribers as before. Only the
single-row create, update and delete paths accept it; a batch write is not
reconciled locally by the actor and must still refetch.

The attribution pin moves with the behaviour: what selects the actor-scoped
signal is no longer which file calls it but which surface supplies an actor,
so that is now what the test pins.

Verified to fail: ignoring the keying discriminator, and dropping actor
attribution, each turn the corresponding tests red.

Author: waleedlatif1

apps/sim/app/api/v2/tables/[tableId]/rows/[rowId]/route.test.ts

@@ -132,6 +132,7 @@ describe('/api/v2/tables/[tableId]/rows/[rowId]', () => {
         assertedWorkspaceId: WORKSPACE_ID,
         data: { name: 'Ada' },
         strictWrite: true,
+        dataKeying: 'names',
       },
       request: req,
     })

apps/sim/app/api/v2/tables/[tableId]/rows/[rowId]/route.ts

@@ -42,6 +42,7 @@ export const PATCH = defineV2JsonRoute({
     assertedWorkspaceId: body.workspaceId,
     data: body.data,
     strictWrite: true,
+    dataKeying: 'names' as const,
   }),
   useCase: updateTableRow,
   present: ({ table, row }) => ({

apps/sim/app/api/v2/tables/[tableId]/rows/route.test.ts

@@ -190,6 +190,7 @@ describe('/api/v2/tables/[tableId]/rows', () => {
         // or a value the column cannot hold is a 400, not a dropped key or a
         // nulled cell. Every first-party surface leaves this unset.
         strictWrite: true,
+        dataKeying: 'names',
       },
       request: single,
     })
@@ -207,6 +208,7 @@ describe('/api/v2/tables/[tableId]/rows', () => {
         assertedWorkspaceId: WORKSPACE_ID,
         rows: [{ name: 'Ada' }],
         strictWrite: true,
+        dataKeying: 'names',
       },
       request: batch,
     })

apps/sim/app/api/v2/tables/[tableId]/rows/route.ts

@@ -71,6 +71,7 @@ export const POST = defineV2JsonRoute({
           assertedWorkspaceId: body.workspaceId,
           rows: body.rows,
           strictWrite: true,
+          dataKeying: 'names' as const,
         }
       : {
           kind: 'single' as const,
@@ -80,6 +81,7 @@ export const POST = defineV2JsonRoute({
           afterRowId: body.afterRowId,
           beforeRowId: body.beforeRowId,
           strictWrite: true,
+          dataKeying: 'names' as const,
         },
   useCase: createTableRows,
   present: (result) => {
@@ -108,6 +110,7 @@ export const PATCH = defineV2JsonRoute({
     data: body.data,
     limit: body.limit,
     strictWrite: true,
+    dataKeying: 'names' as const,
   }),
   useCase: updateTableRows,
   present: ({ affectedCount, affectedRowIds }) => ({

apps/sim/app/api/v2/tables/[tableId]/rows/upsert/route.test.ts

@@ -103,6 +103,7 @@ describe('POST /api/v2/tables/[tableId]/rows/upsert', () => {
         data: { email: 'ada@example.com' },
         conflictTarget: 'email',
         strictWrite: true,
+        dataKeying: 'names',
       },
       request,
     })

apps/sim/app/api/v2/tables/[tableId]/rows/upsert/route.ts

@@ -21,6 +21,7 @@ export const POST = defineV2JsonRoute({
     data: body.data,
     conflictTarget: body.conflictTarget,
     strictWrite: true,
+    dataKeying: 'names' as const,
   }),
   useCase: upsertTableRow,
   present: ({ table, row, operation }) => ({

apps/sim/lib/copilot/tools/server/table/user-table.ts

@@ -286,6 +286,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
               tableId: args.tableId,
               assertedWorkspaceId: workspaceId,
               strictWrite: false,
+              dataKeying: 'names',
               data: args.data,
               position: args.position as number | undefined,
               secretProvenance: createExactEmptyTableRowSecretProvenance(args.data),
@@ -329,6 +330,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
               tableId: args.tableId,
               assertedWorkspaceId: workspaceId,
               strictWrite: false,
+              dataKeying: 'names',
               rows: sourceRows,
               secretProvenance: sourceRows.map(createExactEmptyTableRowSecretProvenance),
             },
@@ -476,6 +478,7 @@ export const userTableServerTool: BaseServerTool<UserTableArgs, UserTableResult>
               tableId: args.tableId,
               assertedWorkspaceId: workspaceId,
               strictWrite: false,
+              dataKeying: 'names',
               rowId: args.rowId,
               data: args.data,
               secretProvenance: createExactEmptyTableRowSecretProvenance(args.data),