chore(tables): remove stale comments and dead defensive code

- Fix chunkBatchUpdates JSDoc to reflect parallel dispatch (was "sequentially")
- Inline CHUNK_CONCURRENCY=3, single-use constant needs no abstraction
- Drop stale "Polls while any cell is in flight" from useTableRows JSDoc
- Remove two generic "Validation errors surfaced by caller" comments
- Remove ASCII separator line from workflow group mutations section
- Remove dead `if (!variables) return` guard in useImportCsvIntoTable onSettled
  (TanStack v5 always provides variables to onSettled)

Author: waleedlatif1

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/table-grid.tsx

@@ -193,13 +193,10 @@ interface TableGridProps {
 }
 
 /**
- * Split updates into chunks that fit within the server's batch-size limit,
- * running each chunk sequentially. Throws on first failure so callers see
- * an all-or-nothing result and partial success cannot leave the table in an
- * ambiguous half-cleared state.
+ * Split updates into chunks bounded by the server batch-size limit, dispatching
+ * up to 3 chunks concurrently. Throws on first failure — `Promise.all` rejects
+ * immediately, so partial success cannot leave the table in an ambiguous state.
  */
-const CHUNK_CONCURRENCY = 3
-
 async function chunkBatchUpdates(
   updates: Array<{ rowId: string; data: Record<string, unknown> }>,
   mutateAsync: (args: {
@@ -213,7 +210,7 @@ async function chunkBatchUpdates(
   }
   let cursor = 0
   await Promise.all(
-    Array.from({ length: Math.min(CHUNK_CONCURRENCY, chunks.length) }, async () => {
+    Array.from({ length: Math.min(3, chunks.length) }, async () => {
       while (cursor < chunks.length) {
         await mutateAsync({ updates: chunks[cursor++]! })
       }

apps/sim/hooks/queries/tables.ts

@@ -210,11 +210,6 @@ interface InfiniteTableRowsParams {
   enabled?: boolean
 }
 
-/**
- * Fetch a single page of rows for a table with pagination/filter/sort. Polls
- * while any cell is in flight so cells reach their terminal state without a
- * manual refresh.
- */
 export function useTableRows({
   workspaceId,
   tableId,
@@ -416,7 +411,6 @@ export function useCreateTableRow({ workspaceId, tableId }: RowMutationContext)
       reconcileCreatedRow(queryClient, tableId, row)
     },
     onError: (error) => {
-      // Validation errors are surfaced inline by the caller (see useUpdateColumn).
       if (isValidationError(error)) return
       toast.error(error.message, { duration: 5000 })
     },
@@ -771,7 +765,6 @@ export function useUpdateColumn({ workspaceId, tableId }: RowMutationContext) {
           queryClient.setQueryData(key, data)
         }
       }
-      // Validation errors are surfaced as inline FieldErrors by the caller.
       if (isValidationError(error)) return
       toast.error(error.message, { duration: 5000 })
     },
@@ -1011,7 +1004,6 @@ export function useImportCsvIntoTable() {
       logger.error('Failed to import CSV into table:', error)
     },
     onSettled: (_data, _error, variables) => {
-      if (!variables) return
       invalidateRowCount(queryClient, variables.tableId)
     },
   })
@@ -1291,8 +1283,6 @@ export function useRunColumn({ workspaceId, tableId }: RowMutationContext) {
   })
 }
 
-// ───────────────────────── Workflow group mutations ─────────────────────────
-
 interface AddWorkflowGroupVariables {
   group: WorkflowGroup
   outputColumns: AddWorkflowGroupBodyInput['outputColumns']