Misc fixes

[creatorrr]

• fix(agents-api): Allow url etc to be overridden by arguments
• fix(agents-api): base64 encode http content
• feat(agents-api): Extend stdlib
• fix(agents-api): Fix interceptors and simplify get/set steps
• fix(agents-api): Fix the bug where execution.output was not being set

---

Important

This PR enhances API call execution, extends the standard library, improves error handling, refines execution transitions, and updates workflow execution in the agents-api module.

• API Call Execution:
  • execute_api_call in excecute_api_call.py now supports overriding url and headers via RequestArgs.
  • HTTP content is base64 encoded before being returned.
• Standard Library Extension:
  • utils.py extended with classes for re, json, yaml, time, random, itertools, functools, base64, urllib, string, zoneinfo, datetime, math, and statistics.
• Error Handling:
  • Updated is_non_retryable_error in exceptions/tasks.py to accept BaseException.
  • CustomActivityInterceptor and CustomWorkflowInterceptor in interceptors.py now handle BaseException.
• Execution Transition:
  • Fixed bug in create_execution_transition.py where execution.output was not set correctly for non-error transitions.
• Workflow Execution:
  • Removed user state management from TaskExecutionWorkflow in task_execution/__init__.py.
  • Updated continue_as_child in helpers.py to handle user state via workflow memo.
• Testing:
  • Updated tests in test_execution_workflow.py to cover new functionalities and ensure correct behavior.

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

[ellipsis-dev]

❌ Changes requested. Reviewed everything up to 2b78559 in 45 seconds

More details

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

1. agents-api/agents_api/common/exceptions/tasks.py:23

• Draft comment:
  The change from list to tuple for NON_RETRYABLE_ERROR_TYPES is appropriate as it is used in isinstance, which requires a tuple. This change is correct and improves the code.
• Reason this comment was not posted:
  Confidence changes required: 0%
  The change from list to tuple for NON_RETRYABLE_ERROR_TYPES is appropriate as it is used in isinstance, which requires a tuple. This change is correct and improves the code.

2. agents-api/agents_api/models/execution/create_execution_transition.py:179

• Draft comment:
  The query construction logic here is complex and could benefit from additional comments or refactoring for improved readability and maintainability.
• Reason this comment was not posted:
  Confidence changes required: 50%
  The create_execution_transition function has a complex query construction logic. It would be beneficial to add comments or refactor the code to improve readability and maintainability.

Workflow ID: wflow_nRTiFjpd8JCPjsg0

---

Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.