fix(agents-api): Fix bug in task-execution workflow and uuid-int-list-to-str fn

[creatorrr]

Signed-off-by: Diwank Tomer [email protected]

---

🚀	This description was created by Ellipsis for commit 061d728

Summary:

This PR fixes a bug in task execution workflow and enhances type safety in uuid_int_list_to_uuid4, along with improving error handling across several modules.

Key points:

• Added beartype decorator to uuid_int_list_to_uuid4 in agents-api/agents_api/common/utils/cozo.py for type checking.
• Modified fix_uuid in agents-api/agents_api/models/utils.py to handle cases where attributes are not lists.
• Updated get_execution in agents-api/agents_api/models/execution/get_execution.py to handle AssertionError with a 404 status code.
• Simplified get_execution_details in agents-api/agents_api/routers/tasks/get_execution_details.py by removing exception handling.
• Refactored TaskExecutionWorkflow.run in agents-api/agents_api/workflows/task_execution.py to improve error handling and transition logic.

---

Generated with ❤️ by ellipsis.dev

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 061d728 in 19 seconds

More details

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

1. agents-api/agents_api/common/utils/cozo.py:25

• Draft comment:
  Ensure that the input list data only contains integers in the range 0-255 to avoid OverflowError when converting to bytes.
• Reason this comment was not posted:
  Decided after close inspection that this draft comment was likely wrong and/or not actionable:
  The comment is speculative and suggests a check that is not implemented in the code. The function assumes the input is correct as per the type hint, and the comment does not point out a change in the diff. It is more of a suggestion rather than a necessary code change.
  The comment could be seen as a useful reminder to handle potential errors, but it doesn't align with the rules for review comments as it doesn't point out a definite issue in the code.
  While the comment might be informative, it doesn't align with the rules for actionable comments that require a code change.
  The comment should be removed as it doesn't point out a necessary code change and is speculative.

2. agents-api/agents_api/models/utils.py:33

• Draft comment:
  Ensure that the lists being converted to UUIDs contain integers only in the range 0-255 to prevent OverflowError. This applies to all uses of uuid_int_list_to_uuid4.
• Reason this comment was not posted:
  Marked as duplicate.

Workflow ID: wflow_AeDluLAD6OoOIHpo

---

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