Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -1405,7 +1405,7 @@ def _prepare_content_for_openai(
else "auto",
}
file_id = content.additional_properties.get("file_id") if content.additional_properties else None
if file_id:
if file_id is not None:
result["file_id"] = file_id
return result
if content.has_top_level_media_type("audio"):
Expand Down
11 changes: 11 additions & 0 deletions python/packages/openai/tests/openai/test_openai_chat_client.py
Original file line number Diff line number Diff line change
Expand Up @@ -2975,6 +2975,17 @@ def test_prepare_content_for_openai_image_content() -> None:
assert result["detail"] == "auto"
assert "file_id" not in result

# Test image content with additional_properties present but no file_id key
image_content_detail_only = Content.from_uri(
uri="https://example.com/basic.png",
media_type="image/png",
additional_properties={"detail": "high"},
)
result = client._prepare_content_for_openai("user", image_content_detail_only)
assert result["type"] == "input_image"
assert result["detail"] == "high"
assert "file_id" not in result


def test_prepare_content_for_openai_audio_content() -> None:
"""Test _prepare_content_for_openai with audio content variations."""
Expand Down
Loading