Better handling of failed to summarize failure reason

[wintonzheng]

Important

Improve logging and error handling in summary_failure_reason_for_max_steps() in agent.py.

• Logging:
  • Change log level from exception to warning in summary_failure_reason_for_max_steps() in agent.py.
  • Add task_id and step_id to log message for better traceability.
• Error Handling:
  • Add fallback return value in summary_failure_reason_for_max_steps() to return last step's action results if an exception occurs.

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

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 9e5c56f in 13 seconds

More details

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

1. skyvern/forge/agent.py:1745

• Draft comment:
  Typo in log message: 'summary' should be 'summarize'.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The log message in the exception block should use 'summarize' instead of 'summary' to match the function name and context.

Workflow ID: wflow_3R8lS4uEpsJ2uYCl

---

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