improvement(workflow): narrow zustand selectors and optimize log tree builds

[waleedlatif1]

Summary

• Add useIsCurrentWorkflowExecuting selector; swap broad useCurrentWorkflowExecution in action-bar/chat (was re-rendering on every iteration update)
• Use useShallow for execution-state subset in useWorkflowExecution
• Drop useCallback-wrapped object selectors in diff-controls and use-block-state — primitive selectors instead (the useCallback did nothing for Zustand reference equality, which was the bug)
• Stable EMPTY_SUBBLOCK_VALUES module constant for empty-workflow selector results in api-info-modal, mcp deploy, tag-dropdown
• O(n) Map lookups in buildEntryTree (was O(n²) filter scans — relevant for 100+ iteration parallel-loop logs)
• Skip expanded-paths reset in structured output when data content is unchanged (preserves user expansions during stream refreshes)
• Extract ReactFlow constants out of workflow.tsx into workflow-constants.ts

Type of Change

• Improvement

Testing

Tested manually. Behavior is preserved across all changes — selectors return the same values, useShallow/primitive selectors read the same fields, Map iteration preserves insertion order, JSON short-circuit only narrows when the existing reset would have fired.

Checklist

• Code follows project style guidelines
• Self-reviewed my changes
• Tests added/updated and passing
• No new warnings introduced
• I confirm that I have read and agree to the terms outlined in the Contributor License Agreement (CLA)

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated (UTC)
docs	Ready	Preview, Comment	May 1, 2026 6:59pm

[cursor]

PR Summary

Medium Risk
Mostly performance-oriented changes, but they touch workflow execution state selection and terminal log tree construction, so regressions could impact run/stop UI state and console rendering in large loop/parallel executions.

Overview
Reduces unnecessary React re-renders across the workflow UI by narrowing Zustand selectors (including a new useIsCurrentWorkflowExecuting hook), removing ineffective useCallback object selectors, and using stable empty fallbacks for subblock value selectors.

Improves terminal/output performance by optimizing buildEntryTree lookups for nested iterations, and by preventing StructuredOutput from resetting expanded paths when streamed data is referentially new but content-equivalent.

Extracts ReactFlow configuration constants from workflow.tsx into a dedicated workflow-constants.ts module, and updates the API validation contract baseline to account for one additional route.

^{Reviewed by Cursor Bugbot for commit c7dab39. Configure here.}

[greptile-apps]

Greptile Summary

This PR improves rendering performance in several areas: it introduces useIsCurrentWorkflowExecuting to narrow Zustand subscriptions in action-bar and chat, applies useShallow in useWorkflowExecution, removes no-op useCallback-wrapped selectors, and exports a stable EMPTY_SUBBLOCK_VALUES constant to prevent false-positive re-renders in three components. The buildEntryTree function is optimised from O(n²) filter scans to O(n) Map lookups for parallel-loop logs, and StructuredOutput now skips expansion resets when streamed data content is unchanged. ReactFlow constants are extracted into workflow-constants.ts.

Confidence Score: 5/5

This PR is safe to merge — all changes are purely additive optimisations with no behavioral regressions.

No P0 or P1 findings. Every change either narrows a Zustand selector, replaces a no-op pattern with a correct one, or improves algorithmic complexity. The structured-output null-sentinel fix was applied correctly, and the Map-based buildEntryTree optimisation preserves insertion-order semantics.

No files require special attention.

Important Files Changed

Filename	Overview
apps/sim/stores/execution/store.ts	Adds useIsCurrentWorkflowExecuting — a focused boolean selector that avoids re-renders on every iteration update. Pattern mirrors existing useIsBlockActive and is exported correctly.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/terminal/utils.ts	O(n²) → O(n) optimization in buildEntryTree: pre-builds Maps keyed by iterationContainerId and iterationCurrent so each subflow/iteration lookup is O(1). Semantics are preserved.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/terminal/components/output-panel/components/structured-output.tsx	Adds null-sentinel prevDataJsonRef to skip expansion-state resets when streamed data is referentially new but content-identical. The null initialisation correctly forces a lazy stringify on first comparison.
apps/sim/stores/workflows/subblock/store.ts	Exports EMPTY_SUBBLOCK_VALUES as a stable module-level constant so selectors returning an empty fallback don't create a new object reference on every call.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/hooks/use-workflow-execution.ts	Replaces useCurrentWorkflowExecution with an inline useShallow selector that reads only 5 needed fields, preventing re-renders from other execution-state changes (e.g., lastRunPath).
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow-constants.ts	New file extracting ReactFlow node/edge type maps and config constants out of workflow.tsx. All values are module-level and stable — correct for ReactFlow's requirement that nodeTypes/edgeTypes references don't change each render.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/components/diff-controls/diff-controls.tsx	Drops useCallback-wrapped object selectors for four primitive fields. Correct fix: useCallback has no effect on Zustand selector equality, so separate primitive selectors are both simpler and correct.
apps/sim/app/workspace/[workspaceId]/w/[workflowId]/workflow.tsx	Removes ~30 lines of inlined ReactFlow constants and component imports that are now in workflow-constants.ts. No behavioral change.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
subgraph Execution Store
A[workflowExecutions Map]
B[useCurrentWorkflowExecution\nreturns full WorkflowExecutionState]
C[useIsCurrentWorkflowExecuting\nreturns boolean only]
end
subgraph Consumers
E[action-bar.tsx]
F[chat.tsx]
G[use-workflow-execution.ts\nuseShallow 5-field slice]
end
A -->|"full object — re-renders on\nevery iteration update"| B
A -->|"boolean — re-renders only\nwhen isExecuting flips"| C
A -->|"useShallow — re-renders only\nwhen any of 5 fields changes"| G
C --> E
C --> F

Loading

_{Reviews (3): Last reviewed commit: "chore(ci): bump api-validation route bas..." | Re-trigger Greptile}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit b152392. Configure here.}

[waleedlatif1]

@greptile

[waleedlatif1]

@cursor review

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit c7dab39. Configure here.}