fix(tables): use current frozenColumns on delete-column redo, not stale snapshot

Author: waleedlatif1

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

@@ -495,6 +495,10 @@ export function TableGrid({
     frozenColumnsRef.current = frozen
   }
 
+  function getFrozenColumns() {
+    return frozenColumnsRef.current
+  }
+
   const handleFreezeToggle = useCallback((columnName: string) => {
     const col = columnsRef.current.find((c) => c.name === columnName)
     const siblings: string[] = col?.workflowGroupId
@@ -539,6 +543,7 @@ export function TableGrid({
     onColumnRename: handleColumnRename,
     onColumnWidthsChange: handleColumnWidthsChange,
     onFrozenColumnsChange: handleFrozenColumnsChange,
+    getFrozenColumns,
     getColumnWidths,
   })
   const undoRef = useRef(undo)

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

@@ -32,6 +32,7 @@ interface UseTableUndoProps {
   onColumnRename?: (oldName: string, newName: string) => void
   onColumnWidthsChange?: (widths: Record<string, number>) => void
   onFrozenColumnsChange?: (frozen: string[]) => void
+  getFrozenColumns?: () => string[]
   getColumnWidths?: () => Record<string, number>
 }
 
@@ -42,6 +43,7 @@ export function useTableUndo({
   onColumnRename,
   onColumnWidthsChange,
   onFrozenColumnsChange,
+  getFrozenColumns,
   getColumnWidths,
 }: UseTableUndoProps) {
   const push = useTableUndoStore((s) => s.push)
@@ -73,6 +75,8 @@ export function useTableUndo({
   onColumnWidthsChangeRef.current = onColumnWidthsChange
   const onFrozenColumnsChangeRef = useRef(onFrozenColumnsChange)
   onFrozenColumnsChangeRef.current = onFrozenColumnsChange
+  const getFrozenColumnsRef = useRef(getFrozenColumns)
+  getFrozenColumnsRef.current = getFrozenColumns
   const getColumnWidthsRef = useRef(getColumnWidths)
   getColumnWidthsRef.current = getColumnWidths
 
@@ -303,9 +307,8 @@ export function useTableUndo({
                     onColumnWidthsChangeRef.current?.(rest)
                   }
                   if (action.previousFrozenColumns !== null) {
-                    const newFrozen = action.previousFrozenColumns.filter(
-                      (n) => n !== action.columnName
-                    )
+                    const currentFrozen = getFrozenColumnsRef.current?.() ?? []
+                    const newFrozen = currentFrozen.filter((n) => n !== action.columnName)
                     onFrozenColumnsChangeRef.current?.(newFrozen)
                     metadata.frozenColumns = newFrozen
                   }