Skip to content

Commit f23f234

Browse files
fix(table): per-row Run fires autoRun=false groups regardless of deps
handleRunRow filtered out every group whose deps weren't satisfied, which silently dropped autoRun=false groups (since their deps usually aren't satisfied — that's the whole point of autoRun=false). Click Run row, the autoRun=false group's cells stayed empty. Mirror the scheduler's semantics: autoRun=false bypasses the dep check, autoRun=true still requires deps.
1 parent 3c87a07 commit f23f234

1 file changed

Lines changed: 5 additions & 5 deletions

File tree

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

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

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -2712,12 +2712,12 @@ export function TableGrid({
27122712
if (tableWorkflowGroups.length === 0) return
27132713
const target = rowsRef.current.find((r) => r.id === rowId)
27142714
if (!target) return
2715-
// Only fire groups whose deps are already satisfied for THIS row. The
2716-
// cascade picks up downstream groups: when an upstream group completes,
2717-
// `scheduleWorkflowGroupRuns` evaluates eligibility and enqueues the
2718-
// newly-ready successors automatically.
2715+
// Fire each group that's runnable on this row. autoRun=false groups
2716+
// run regardless of deps (user model: no autoRun = no deps).
2717+
// autoRun=true groups only fire when deps are satisfied — others stay
2718+
// as "Waiting" and the cascade picks them up after upstream finishes.
27192719
for (const group of tableWorkflowGroups) {
2720-
if (!areRowDepsSatisfied(group, target)) continue
2720+
if (group.autoRun !== false && !areRowDepsSatisfied(group, target)) continue
27212721
void runWorkflowGroup({
27222722
tableId,
27232723
rowId,

0 commit comments

Comments
 (0)