fix(tables): use text input for number cells; idle poll backoff; csv error toast; column-cut drain

Author: waleedlatif1

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

@@ -198,17 +198,13 @@ function InlineTextEditor({
   return (
     <input
       ref={inputRef}
-      type={isNumber ? 'number' : 'text'}
-      step={isNumber ? 'any' : undefined}
+      type='text'
+      inputMode={isNumber ? 'decimal' : undefined}
       value={draft ?? ''}
       onChange={(e) => setDraft(e.target.value)}
       onKeyDown={handleKeyDown}
       onBlur={() => doSave('blur')}
-      className={cn(
-        'w-full min-w-0 select-text border-none bg-transparent p-0 text-[var(--text-primary)] text-small outline-none',
-        isNumber &&
-          '[appearance:textfield] [&::-webkit-inner-spin-button]:appearance-none [&::-webkit-outer-spin-button]:appearance-none'
-      )}
+      className='w-full min-w-0 select-text border-none bg-transparent p-0 text-[var(--text-primary)] text-small outline-none'
     />
   )
 }

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

@@ -2078,6 +2078,65 @@ export function TableGrid({
       if (!sel) return
 
       e.preventDefault()
+
+      if (isColumnSelectionRef.current) {
+        // Column-header cut spans all rows — drain pages first, then use async
+        // clipboard so we don't block the event before the drain completes.
+        void (async () => {
+          const allRows = await ensureAllRowsLoadedRef.current()
+          const lines: string[] = []
+          const undoCells: Array<{ rowId: string; data: Record<string, unknown> }> = []
+          const batchUpdates: Array<{ rowId: string; data: Record<string, unknown> }> = []
+          for (const row of allRows) {
+            const cells: string[] = []
+            const updates: Record<string, unknown> = {}
+            const previousData: Record<string, unknown> = {}
+            for (let c = sel.startCol; c <= sel.endCol; c++) {
+              const colName = cols[c]?.name
+              if (!colName) continue
+              const value: unknown = row.data[colName]
+              cells.push(
+                value === null || value === undefined
+                  ? ''
+                  : typeof value === 'object'
+                    ? JSON.stringify(value)
+                    : String(value)
+              )
+              previousData[colName] = row.data[colName] ?? null
+              updates[colName] = null
+            }
+            lines.push(cells.join('\t'))
+            undoCells.push({ rowId: row.id, data: previousData })
+            batchUpdates.push({ rowId: row.id, data: updates })
+          }
+          if (!navigator.clipboard) {
+            toast.error('Clipboard access is unavailable in this context')
+            return
+          }
+          try {
+            await navigator.clipboard.writeText(lines.join('\n'))
+          } catch (err) {
+            if (err instanceof DOMException && err.name === 'NotAllowedError') {
+              toast.error(
+                'Clipboard permission expired — press Cmd+X again immediately after selecting'
+              )
+              return
+            }
+            throw err
+          }
+          if (undoCells.length > 0) {
+            pushUndoRef.current({ type: 'clear-cells', cells: undoCells })
+          }
+          if (batchUpdates.length > 0) {
+            await chunkBatchUpdates(batchUpdates, batchUpdateAsyncRef.current)
+          }
+        })().catch((error) => {
+          logger.error('Failed to cut column cells', { error })
+          toast.error('Failed to cut — please try again')
+        })
+        return
+      }
+
       const lines: string[] = []
       const undoCells: Array<{ rowId: string; data: Record<string, unknown> }> = []
       const batchUpdates: Array<{ rowId: string; data: Record<string, unknown> }> = []

apps/sim/hooks/queries/tables.ts

@@ -79,6 +79,7 @@ import {
 const logger = createLogger('TableQueries')
 
 const ROWS_POLL_INTERVAL_WHILE_RUNNING_MS = 2_000
+const ROWS_POLL_INTERVAL_IDLE_MS = 30_000
 
 type TableQueryScope = 'active' | 'archived' | 'all'
 
@@ -341,6 +342,7 @@ export function useInfiniteTableRows({
     let timeoutId: ReturnType<typeof setTimeout> | null = null
     const tick = async () => {
       if (cancelled) return
+      let hasDirty = false
       if (queryClient.isMutating() === 0) {
         const data = queryClient.getQueryData<InfiniteData<TableRowsResponse, number>>(queryKey)
         const dirty: number[] = []
@@ -351,7 +353,8 @@ export function useInfiniteTableRows({
             }
           }
         }
-        if (dirty.length > 0) {
+        hasDirty = dirty.length > 0
+        if (hasDirty) {
           await Promise.all(
             dirty.map(async (offset) => {
               try {
@@ -388,8 +391,12 @@ export function useInfiniteTableRows({
       if (cancelled) return
       // Recursive setTimeout instead of setInterval so a slow tick can't
       // overlap the next one — out-of-order responses would otherwise let
-      // stale data overwrite fresh.
-      timeoutId = setTimeout(() => void tick(), ROWS_POLL_INTERVAL_WHILE_RUNNING_MS)
+      // stale data overwrite fresh. Use a long interval when idle so tables
+      // with no running executions don't burn CPU on constant cache reads.
+      timeoutId = setTimeout(
+        () => void tick(),
+        hasDirty ? ROWS_POLL_INTERVAL_WHILE_RUNNING_MS : ROWS_POLL_INTERVAL_IDLE_MS
+      )
     }
     timeoutId = setTimeout(() => void tick(), ROWS_POLL_INTERVAL_WHILE_RUNNING_MS)
     return () => {
@@ -1036,7 +1043,6 @@ export function useUploadCsvToTable() {
       return response.json()
     },
     onError: (error) => {
-      if (isValidationError(error)) return
       logger.error('Failed to upload CSV:', error)
       toast.error(error.message, { duration: 5000 })
     },