Change order of information in timeline info section

[wintonzheng]

Important

Reorders tabs and changes default tab logic in WorkflowRunTimelineItemInfoSection based on item status.

• Behavior:
  • Changes default tab logic in WorkflowRunTimelineItemInfoSection to prioritize extracted_information if status is Completed, otherwise failure_reason if status is a failure type, else navigation_goal.
  • Reorders TabsTrigger for navigation_goal to appear after failure_reason and extracted_information.
• UI:
  • Adjusts Tabs component in WorkflowRunTimelineItemInfoSection to reflect new tab order and default selection logic.

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

[ellipsis-dev]

👍 Looks good to me! Incremental review on 8c993fd in 7 seconds

More details

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

1. skyvern-frontend/src/routes/workflows/workflowRun/WorkflowRunTimelineItemInfoSection.tsx:59

• Draft comment:
  Consider extracting the logic for determining defaultTab into a separate function to improve readability and maintainability.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The logic for determining the default tab is repeated in multiple places. It would be more maintainable to extract this logic into a separate function.

Workflow ID: wflow_94hlK9wZzjImQsni

---

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 8c993fd in 21 seconds

More details

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

1. skyvern-frontend/src/routes/workflows/workflowRun/WorkflowRunTimelineItemInfoSection.tsx:53

• Draft comment:
  hasExtractedInformation is imported but never used. Consider removing it to clean up the code.
• Reason this comment was not posted:
  Confidence changes required: 10%
  The import statement for hasExtractedInformation is unnecessary as it is not used in the code.

Workflow ID: wflow_SFdGjJ1WbIavZm0h

---

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