Change rerun workflow behavior

[ykeremy]

🚀	This description was created by Ellipsis for commit 5fceca5

Summary:

Updated rerun workflow behavior and enhanced logging for better traceability.

Key points:

• Updated rerun workflow behavior by removing mutation.
• Passed parameters via navigation state in WorkflowRun and WorkflowRunParameters.
• Removed runWorkflowMutation from WorkflowRun.tsx.
• Updated WorkflowRun to navigate with state data.
• Modified WorkflowRunParameters to use location.state.data.
• Adjusted imports in WorkflowRun and WorkflowRunParameters.
• Enhanced logging in create_agent_task to include organization_id and task_id.
• Improved traceability and debugging capabilities.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 5d399c0 in 13 seconds

More details

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

1. skyvern-frontend/src/routes/workflows/WorkflowRun.tsx:4

• Draft comment:
  There are duplicate imports for Button, Input, Label, and pagination components. Please remove the duplicates to clean up the code.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The import statements in WorkflowRun.tsx are not organized properly. There are duplicate imports for Button, Input, Label, and pagination components. This can lead to confusion and unnecessary code bloat.

2. skyvern-frontend/src/routes/workflows/WorkflowRunParameters.tsx:44

• Draft comment:
  The initialValues logic can be simplified by using a default value for location.state.data to avoid duplicating the reduce logic.
• Reason this comment was not posted:
  Confidence changes required: 50%
  In WorkflowRunParameters.tsx, the initialValues logic is duplicated. The code first checks if location.state.data is available and uses it, otherwise it falls back to reducing workflowParameters. This logic can be simplified by using a default value for location.state.data.

3. skyvern/forge/sdk/routes/agent_protocol.py:127

• Draft comment:
  Ensure that the logging format is consistent throughout the file for better readability and maintainability.
• Reason this comment was not posted:
  Confidence changes required: 33%
  In agent_protocol.py, the logging statement for overriding max steps per run was changed to a single line. This change is fine, but it should be consistent with other logging practices in the file. Ensure that the logging format is consistent throughout the file for better readability and maintainability.

Workflow ID: wflow_erA1RlEflZ48AtE8

---

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

[ellipsis-dev]

👍 Looks good to me! Incremental review on 5d399c0 in 15 seconds

More details

• Looked at 192 lines of code in 2 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. skyvern-frontend/src/routes/workflows/WorkflowRunParameters.tsx:44

• Draft comment:
  Consider using a default value for location.state.data to simplify the logic for initialValues. This can make the code cleaner and easier to read.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The code in WorkflowRunParameters.tsx uses location.state?.data to determine initialValues. However, if location.state.data is not available, it falls back to reducing workflowParameters. This logic is correct, but it could be more concise by using a default value for location.state.data. This would make the code cleaner and easier to read.

Workflow ID: wflow_FA0s9TwXvAUA9yO6

---

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

[ellipsis-dev]

👍 Looks good to me! Incremental review on 5fceca5 in 17 seconds

More details

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

Workflow ID: wflow_AtUVJuG3f5oB36DT

---

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