fix(table): show Stop button on optimistic-pending row cells

isExecInFlight required a jobId for `pending` status, gating it as "real
backend pending" vs "optimistic flag only." The row-gutter Stop button
keyed on this — so a freshly clicked Play sat as `pending` (no jobId) and
the user couldn't cancel it until the server-side `queued` stamp arrived
via SSE. With the dispatcher pre-batch stamping cells as `queued` (not
`pending`) and no per-cell jobIds under batchTriggerAndWait, the gap was
worse.

Drop the jobId requirement. `pending` now counts as in-flight everywhere.
Cancel writes `cancelled` to the cell exec authoritatively whether or not
a real trigger.dev run exists yet — cancelling an optimistic cell means
"don't run this," which is correct.

Also collapse isOptimisticInFlight into isExecInFlight since the two
helpers are now identical.

Author: TheodoreSpeaks

apps/sim/hooks/queries/tables.ts

@@ -69,7 +69,11 @@ import type {
   WorkflowGroupDependencies,
   WorkflowGroupOutput,
 } from '@/lib/table'
-import { areOutputsFilled, optimisticallyScheduleNewlyEligibleGroups } from '@/lib/table/deps'
+import {
+  areOutputsFilled,
+  isExecInFlight,
+  optimisticallyScheduleNewlyEligibleGroups,
+} from '@/lib/table/deps'
 
 const logger = createLogger('TableQueries')
 
@@ -869,7 +873,7 @@ export function useCancelTableRuns({ workspaceId, tableId }: RowMutationContext)
         const nextExecutions: RowExecutions = { ...executions }
         for (const gid in executions) {
           const exec = executions[gid]
-          if (!isOptimisticInFlight(exec)) continue
+          if (!isExecInFlight(exec)) continue
           // Preserve blockErrors so cells that already errored keep their
           // Error rendering after the stop — only cells without a value or
           // error should flip to "Cancelled".
@@ -1251,14 +1255,6 @@ function buildPendingExec(
   }
 }
 
-/** Broader sibling of `isExecInFlight` from `lib/table/deps`: treats any
- *  `pending` (with or without a jobId) as in-flight. The optimistic-patch
- *  context uses this to avoid re-marking a cell we just flipped optimistically.
- *  The eligibility predicate uses the stricter version. */
-function isOptimisticInFlight(exec: RowExecutionMetadata | undefined): boolean {
-  return exec?.status === 'running' || exec?.status === 'queued' || exec?.status === 'pending'
-}
-
 /**
  * The single canonical run mutation. Every UI gesture (single cell, per-row
  * Play, action-bar Play/Refresh, column-header menu) maps to a `groupIds` +
@@ -1295,7 +1291,7 @@ export function useRunColumn({ workspaceId, tableId }: RowMutationContext) {
         const nextData = { ...r.data }
         for (const groupId of targetGroupIds) {
           const exec = executions[groupId] as RowExecutionMetadata | undefined
-          if (isOptimisticInFlight(exec)) continue
+          if (isExecInFlight(exec)) continue
           // Mirror server eligibility for `mode: 'incomplete'`: skip cells whose
           // outputs are filled, regardless of exec status. A cancelled/error
           // cell with a leftover value from a prior run was rendering as filled

apps/sim/lib/table/deps.ts

@@ -10,18 +10,18 @@ import type { RowData, RowExecutionMetadata, RowExecutions, TableRow, WorkflowGr
 const logger = createLogger('OptimisticCascade')
 
 /**
- * True when the cell has a worker actively reserved — `queued` / `running`,
- * or `pending` after the scheduler stamped a jobId. Single source of truth
- * for the "is this exec in flight" classification across the eligibility
- * predicate, optimistic patches, status counters, and renderer. `pending`
- * without a jobId is the optimistic-flag-only state, not in-flight.
+ * True when the cell is `pending` / `queued` / `running`. Single source of
+ * truth for the "is this exec in flight" classification across the
+ * eligibility predicate, optimistic patches, status counters, and renderer.
+ * `pending` counts even without a jobId so the row-gutter Stop button is
+ * available the moment the user clicks Play — the cancel path writes
+ * `cancelled` authoritatively whether or not a real trigger.dev run exists
+ * yet, which is correct: cancel means "don't run this."
  */
 export function isExecInFlight(exec: RowExecutionMetadata | undefined): boolean {
   if (!exec) return false
   const s = exec.status
-  if (s === 'queued' || s === 'running') return true
-  if (s === 'pending' && exec.jobId) return true
-  return false
+  return s === 'queued' || s === 'running' || s === 'pending'
 }
 
 /**