expose workflow failure reason to api

[LawyZheng]

Important

Adds failure_reason to WorkflowRunStatusResponse and updates build_workflow_run_status_response to include it.

• Behavior:
  • Adds failure_reason field to WorkflowRunStatusResponse in workflow.py to expose workflow failure reasons.
  • Updates build_workflow_run_status_response in service.py to include failure_reason from workflow_run.
• Models:
  • Adds failure_reason to WorkflowRunStatusResponse in workflow.py.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to c937364 in 10 seconds

More details

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

1. skyvern/forge/sdk/workflow/models/workflow.py:117

• Draft comment:
  Ensure that failure_reason is properly set and validated in all instances where WorkflowRunStatusResponse is used. This includes checking for nullability and ensuring that the failure reason is meaningful and secure to expose.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The addition of failure_reason to the WorkflowRunStatusResponse model is consistent with the changes in the service layer where failure_reason is being passed. This ensures that the failure reason is exposed through the API as intended.

Workflow ID: wflow_di1CG5avgBzQKZRY

---

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