Lee-W commented on code in PR #74695: URL: https://github.com/apache/airflow/pull/74695#discussion_r4521032881
########## providers/common/ai/docs/changelog.rst: ########## @@ -34,17 +34,16 @@ Changelog The worker cannot register a model class for XCom deserialization when ``output_type`` is a list of output types such as ``[Summary, int]``, an output marker such as ``ToolOutput(Summary)``, an output function, a ``TypeAliasType`` or a ``NewType``. For these - output types, ``AgentOperator`` and ``@task.agent`` now push each model in the output as a - JSON-compatible dict (``model_dump(mode="json")``), including models inside a list, tuple or - dict. On Airflow < 3.3, the worker registers no model class at all, so a model inside a list, - tuple or dict is now pushed as a JSON-compatible dict for every ``output_type``. A top-level - model is still dumped with plain ``model_dump()`` when ``serialize_output=True`` is set or on - Airflow < 3.3. + output types, ``LLMOperator``, ``@task.llm``, ``LLMFileAnalysisOperator``, ``AgentOperator`` and Review Comment: This PR's changes are in a new note above 1.1.0, in two parts: 1. output serialization, including the agent's `[M, int]` changes (`full_name` for a `serialize_by_alias` model, a tuple becomes a list, a `date` becomes a string) 2. the MCP `tool_prefix` default. The MCP part is described as a signature change because the behavior is the same -- 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]
