fix(tables): merge column widths on delete-column undo, try/finally for auto-resize

- Delete-column undo now reads current column widths via getColumnWidths
  callback and merges the restored column's width into the full map,
  preventing other columns' widths from being wiped
- Auto-resize measurement span is now wrapped in try/finally to ensure
  DOM cleanup if an exception occurs during measurement

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: waleedlatif1

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

@@ -270,11 +270,14 @@ export function Table({
     })
   }, [])
 
+  const getColumnWidths = useCallback(() => columnWidthsRef.current, [])
+
   const { pushUndo, undo, redo } = useTableUndo({
     workspaceId,
     tableId,
     onColumnOrderChange: handleColumnOrderChange,
     onColumnRename: handleColumnRename,
+    getColumnWidths,
   })
   const undoRef = useRef(undo)
   undoRef.current = undo
@@ -757,20 +760,22 @@ export function Table({
     measure.style.cssText = 'position:absolute;visibility:hidden;white-space:nowrap;font:inherit'
     document.body.appendChild(measure)
 
-    measure.className = 'font-medium text-small'
-    measure.textContent = columnName
-    maxWidth = Math.max(maxWidth, measure.offsetWidth + 36)
-
-    measure.className = 'text-small'
-    for (const row of currentRows) {
-      const val = row.data[columnName]
-      if (val == null) continue
-      measure.textContent = String(val)
-      maxWidth = Math.max(maxWidth, measure.offsetWidth + 17)
+    try {
+      measure.className = 'font-medium text-small'
+      measure.textContent = columnName
+      maxWidth = Math.max(maxWidth, measure.offsetWidth + 36)
+
+      measure.className = 'text-small'
+      for (const row of currentRows) {
+        const val = row.data[columnName]
+        if (val == null) continue
+        measure.textContent = String(val)
+        maxWidth = Math.max(maxWidth, measure.offsetWidth + 17)
+      }
+    } finally {
+      document.body.removeChild(measure)
     }
 
-    document.body.removeChild(measure)
-
     const newWidth = Math.min(Math.ceil(maxWidth), 600)
     setColumnWidths((prev) => ({ ...prev, [columnName]: newWidth }))
     const updated = { ...columnWidthsRef.current, [columnName]: newWidth }

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

@@ -36,13 +36,15 @@ interface UseTableUndoProps {
   tableId: string
   onColumnOrderChange?: (order: string[]) => void
   onColumnRename?: (oldName: string, newName: string) => void
+  getColumnWidths?: () => Record<string, number>
 }
 
 export function useTableUndo({
   workspaceId,
   tableId,
   onColumnOrderChange,
   onColumnRename,
+  getColumnWidths,
 }: UseTableUndoProps) {
   const push = useTableUndoStore((s) => s.push)
   const popUndo = useTableUndoStore((s) => s.popUndo)
@@ -69,6 +71,8 @@ export function useTableUndo({
   onColumnOrderChangeRef.current = onColumnOrderChange
   const onColumnRenameRef = useRef(onColumnRename)
   onColumnRenameRef.current = onColumnRename
+  const getColumnWidthsRef = useRef(getColumnWidths)
+  getColumnWidthsRef.current = getColumnWidths
 
   useEffect(() => {
     return () => clear(tableId)
@@ -230,7 +234,7 @@ export function useTableUndo({
                         ...(action.previousWidth !== null
                           ? {
                               columnWidths: {
-                                ...({} as Record<string, number>),
+                                ...(getColumnWidthsRef.current?.() ?? {}),
                                 [action.columnName]: action.previousWidth,
                               },
                             }