fix(table): address pr review (drop dead workflowNameById prop, reset didDragRef on dragend, align sidebar width)

Author: TheodoreSpeaks

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

@@ -14,7 +14,6 @@ interface CellContentProps {
   initialCharacter?: string | null
   onSave: (value: unknown, reason: SaveReason) => void
   onCancel: () => void
-  workflowNameById?: Record<string, string>
   /**
    * Human-readable labels for unmet deps on this row+group, used to render a
    * "Waiting" pill when the cell hasn't run because something it depends on

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

@@ -7,4 +7,4 @@ export const COL_WIDTH = 160
 
 /** Column config sidebar width in pixels — drives both the sidebar's own width
  *  and the table's reserved padding-right while a sidebar is open. */
-export const COLUMN_SIDEBAR_WIDTH = 480
+export const COLUMN_SIDEBAR_WIDTH = 400

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/components/table-grid/headers/workflow-group-meta-cell.tsx

@@ -318,6 +318,7 @@ export function WorkflowGroupMetaCell({
   )
 
   const handleDragEnd = useCallback(() => {
+    didDragRef.current = false
     onDragEnd?.()
   }, [onDragEnd])
 

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

@@ -289,7 +289,6 @@ export function TableGrid({
     tableWorkflowGroups,
     workflowStates,
     columnSourceInfo,
-    workflowNameById,
   } = useTable({ workspaceId, tableId, queryOptions })
 
   const fetchNextPageRef = useRef(fetchNextPage)
@@ -3034,7 +3033,6 @@ export function TableGrid({
                         isLargeRowCountTable={isLargeRowCountTable}
                         onStopRow={onStopRow}
                         onRunRow={handleRunRow}
-                        workflowNameById={workflowNameById}
                         workflowGroups={tableWorkflowGroups}
                       />
                     ))}
@@ -3161,8 +3159,6 @@ interface DataRowProps {
   isLargeRowCountTable: boolean
   onStopRow: (rowId: string) => void
   onRunRow: (rowId: string) => void
-  /** Lookup from workflow id → human-readable name, used to label running cells. */
-  workflowNameById: Record<string, string>
   /**
    * The table's workflow groups, used to compute per-row "Waiting on …" labels
    * for empty workflow-output cells whose group has unmet dependencies.
@@ -3223,7 +3219,6 @@ function dataRowPropsAreEqual(prev: DataRowProps, next: DataRowProps): boolean {
     prev.isLargeRowCountTable !== next.isLargeRowCountTable ||
     prev.onStopRow !== next.onStopRow ||
     prev.onRunRow !== next.onRunRow ||
-    prev.workflowNameById !== next.workflowNameById ||
     prev.workflowGroups !== next.workflowGroups
   ) {
     return false
@@ -3266,7 +3261,6 @@ const DataRow = React.memo(function DataRow({
   isLargeRowCountTable,
   onStopRow,
   onRunRow,
-  workflowNameById,
   workflowGroups,
 }: DataRowProps) {
   const sel = normalizedSelection
@@ -3422,7 +3416,6 @@ const DataRow = React.memo(function DataRow({
                 initialCharacter={isEditing ? initialCharacter : undefined}
                 onSave={(value, reason) => onSave(row.id, column.name, value, reason)}
                 onCancel={onCancel}
-                workflowNameById={workflowNameById}
                 waitingOnLabels={
                   column.workflowGroupId
                     ? (waitingByGroupId?.get(column.workflowGroupId) ?? undefined)

apps/sim/app/workspace/[workspaceId]/tables/[tableId]/hooks/use-table.ts

@@ -52,8 +52,6 @@ export interface UseTableReturn {
   /** Pre-resolved icon + block-name info per output column name. Headers read
    *  from this map instead of each subscribing to its own workflow-state query. */
   columnSourceInfo: Map<string, ColumnSourceInfo>
-  /** `workflowId → workflow.name` lookup for cell labels and execution-detail copy. */
-  workflowNameById: Record<string, string>
 }
 
 /**
@@ -137,14 +135,6 @@ export function useTable({ workspaceId, tableId, queryOptions }: UseTableParams)
     return map
   }, [tableWorkflowGroups, workflowStates])
 
-  const workflowNameById = useMemo(() => {
-    const map: Record<string, string> = {}
-    for (const wf of workflows ?? []) {
-      map[wf.id] = wf.name
-    }
-    return map
-  }, [workflows])
-
   return {
     tableData,
     isLoadingTable,
@@ -159,6 +149,5 @@ export function useTable({ workspaceId, tableId, queryOptions }: UseTableParams)
     tableWorkflowGroups,
     workflowStates,
     columnSourceInfo,
-    workflowNameById,
   }
 }