fix(tables): make runWithoutRecording async-aware so undoRedoInProgress covers full undo execution

Author: waleedlatif1

apps/sim/hooks/use-table-undo.test.ts

@@ -50,7 +50,7 @@ const storeState = {
 
 vi.mock('@/stores/table/store', () => ({
   useTableUndoStore: vi.fn((selector: (s: typeof storeState) => unknown) => selector(storeState)),
-  runWithoutRecording: (fn: () => void) => fn(),
+  runWithoutRecording: (fn: () => unknown) => Promise.resolve(fn()),
 }))
 
 import { useTableUndo } from '@/hooks/use-table-undo'

apps/sim/hooks/use-table-undo.ts

@@ -355,19 +355,13 @@ export function useTableUndo({
   const undo = useCallback(() => {
     const entry = popUndo(tableId)
     if (!entry) return
-
-    runWithoutRecording(() => {
-      void executeAction(entry.action, 'undo')
-    })
+    void runWithoutRecording(() => executeAction(entry.action, 'undo'))
   }, [popUndo, tableId, executeAction])
 
   const redo = useCallback(() => {
     const entry = popRedo(tableId)
     if (!entry) return
-
-    runWithoutRecording(() => {
-      void executeAction(entry.action, 'redo')
-    })
+    void runWithoutRecording(() => executeAction(entry.action, 'redo'))
   }, [popRedo, tableId, executeAction])
 
   return { pushUndo, undo, redo, canUndo, canRedo }

apps/sim/stores/table/store.ts

@@ -81,13 +81,14 @@ function patchRowIdInEntry(entry: UndoEntry, oldRowId: string, newRowId: string)
 }
 
 /**
- * Run a function without recording undo entries.
- * Used by the hook when executing undo/redo mutations to prevent recursive recording.
+ * Run a function without recording undo entries. Supports async functions —
+ * `undoRedoInProgress` stays true until the returned Promise settles, so
+ * mutations inside `executeAction` don't accidentally push new undo entries.
  */
-export function runWithoutRecording<T>(fn: () => T): T {
+export async function runWithoutRecording<T>(fn: () => T | Promise<T>): Promise<T> {
   undoRedoInProgress = true
   try {
-    return fn()
+    return await fn()
   } finally {
     undoRedoInProgress = false
   }