From 2f8c9b23ca47c1a8ff9631debe42c452e8260a16 Mon Sep 17 00:00:00 2001 From: nightcityblade Date: Mon, 13 Jul 2026 23:08:28 +0800 Subject: [PATCH] fix: handle null response output in parse_response --- src/openai/lib/_parsing/_responses.py | 2 +- tests/lib/responses/test_responses.py | 27 +++++++++++++++++++++++++++ 2 files changed, 28 insertions(+), 1 deletion(-) diff --git a/src/openai/lib/_parsing/_responses.py b/src/openai/lib/_parsing/_responses.py index 5676eb0b63..0704e37026 100644 --- a/src/openai/lib/_parsing/_responses.py +++ b/src/openai/lib/_parsing/_responses.py @@ -58,7 +58,7 @@ def parse_response( ) -> ParsedResponse[TextFormatT]: output_list: List[ParsedResponseOutputItem[TextFormatT]] = [] - for output in response.output: + for output in response.output or []: if output.type == "message": content_list: List[ParsedContent[TextFormatT]] = [] for item in output.content: diff --git a/tests/lib/responses/test_responses.py b/tests/lib/responses/test_responses.py index 8e5f16df95..f4b0b3c948 100644 --- a/tests/lib/responses/test_responses.py +++ b/tests/lib/responses/test_responses.py @@ -7,6 +7,9 @@ from inline_snapshot import snapshot from openai import OpenAI, AsyncOpenAI +from openai._types import NOT_GIVEN +from openai.lib._parsing._responses import parse_response +from openai.types.responses import Response from openai._utils import assert_signatures_in_sync from ...conftest import base_url @@ -61,3 +64,27 @@ def test_parse_method_definition_in_sync(sync: bool, client: OpenAI, async_clien checking_client.responses.parse, exclude_params={"tools"}, ) + + +def test_parse_response_handles_null_output() -> None: + response = Response.model_construct( + id="resp_123", + object="response", + created_at=0, + status="completed", + error=None, + incomplete_details=None, + instructions=None, + max_output_tokens=None, + model="gpt-4o-mini", + output=None, + parallel_tool_calls=True, + temperature=1.0, + tool_choice="auto", + tools=[], + top_p=1.0, + ) + + parsed = parse_response(text_format=NOT_GIVEN, input_tools=None, response=response) + + assert parsed.output == []