kaxil commented on PR #64199: URL: https://github.com/apache/airflow/pull/64199#issuecomment-4128189156
Addressed both review comments in 35d5833: **HITL + durable** — Added `ValueError` in `__init__` if both `durable=True` and `enable_hitl_review=True`. HITL regeneration builds a fresh agent without the CachingModel wrapper, so model calls during review would bypass caching. Making them mutually exclusive for now. **BinaryContent from MCP tools** — Changed `save_tool_result` to catch `TypeError` and log a warning instead of failing the task. The tool call succeeds, but that step won't be cached (it'll re-execute on retry). Added a note in the docs about the JSON-serializable requirement. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
