Bugfix/reset run issues

[srijanpatel]

This pull request includes several updates to the useWorkflowExecution hook and a small change to the flowSlice reducer. The main focus is on improving the handling of status intervals during workflow execution and ensuring proper cleanup of intervals.

Improvements to useWorkflowExecution hook:

• frontend/src/hooks/useWorkflowExecution.ts: Added useRef to manage multiple status intervals and ensure they are cleared appropriately.
• frontend/src/hooks/useWorkflowExecution.ts: Replaced the single currentStatusInterval with an array statusIntervals to track all intervals and clear them when necessary. [1] [2] [3]

Update to flowSlice reducer:

• frontend/src/store/flowSlice.ts: Modified the resetRun action to also reset the error field in each node's data.

---

Important

Improves interval management in useWorkflowExecution and resets error field in resetRun action of flowSlice.

• Behavior:
  • useWorkflowExecution hook now uses useRef to manage multiple status intervals, ensuring they are cleared properly.
  • Replaces currentStatusInterval with statusIntervals array to track and clear all intervals in useWorkflowExecution.ts.
  • resetRun action in flowSlice.ts now resets the error field in each node's data.
• Components:
  • Adds NodeOutputDisplay to InputNode.tsx to display node output when data.run is present.

^{This description was created by for 2526a12. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 2526a12 in 38 seconds

More details

• Looked at 118 lines of code in 3 files
• Skipped 0 files when reviewing.
• Skipped posting 4 drafted comments based on config settings.

1. frontend/src/components/nodes/InputNode.tsx:32

• Draft comment:
  The showOutput state is defined but never used. Consider removing it if it's not needed.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The showOutput state is defined but never used in the component. This is likely an oversight and should be removed if not needed.

2. frontend/src/components/nodes/InputNode.tsx:349

• Draft comment:
  The addition of NodeOutputDisplay for displaying data.run is a good enhancement for visualizing node output.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The addition of NodeOutputDisplay is correct and aligns with the PR description. No issues here.

3. frontend/src/hooks/useWorkflowExecution.ts:30

• Draft comment:
  The use of useRef to manage intervals is a good practice for ensuring proper cleanup. The logic for clearing intervals is correctly implemented.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The use of useRef to manage intervals is a good practice to ensure proper cleanup. The logic for clearing intervals is correctly implemented.

4. frontend/src/store/flowSlice.ts:662

• Draft comment:
  The resetRun action now correctly resets the error field, which aligns with the PR description and intent.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The resetRun action now correctly resets the error field, which aligns with the PR description and intent.

Workflow ID: wflow_WWDB9RTJyAbfWGaG

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.