FrankYang0529 commented on code in PR #74729: URL: https://github.com/apache/airflow/pull/74729#discussion_r4321416289
########## providers/common/ai/docs/structured_output.rst: ########## @@ -50,6 +50,15 @@ loaded Dag's tasks at startup and registers each declared class, so it also work for mapped producers (``.expand(...)``) and for workers that load Dags from a cache that bypasses operator construction. +The worker finds no class to register 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`` push +each model in the output as a JSON-compatible dict, including models inside a Review Comment: Added two paragraphs. The first paragraph gives examples of values that change type in JSON mode. It also covers HITL review. The second paragraph says that a top-level model is still dumped with plain `model_dump()` in either of two cases: `serialize_output=True` is set, or Airflow is older than 3.3. -- 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]
