From bc4587d354b72348b2c9018eedad76c7931fed5d Mon Sep 17 00:00:00 2001 From: Emmanuel Ferdman Date: Fri, 6 Jun 2025 10:51:44 -0700 Subject: [PATCH 1/2] Python: Throw exception on image error Signed-off-by: Emmanuel Ferdman --- .../agents/open_ai/responses_agent_thread_actions.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py b/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py index 2fb09e3f66b2..f6a1a210ccc5 100644 --- a/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py +++ b/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py @@ -699,7 +699,7 @@ def _prepare_chat_history_for_request( image_url = str(content.uri) if not image_url: - ValueError("ImageContent must have either a data_uri or uri set to be used in the request.") + raise ValueError("ImageContent must have either a data_uri or uri set to be used in the request.") contents.append({"type": "input_image", "image_url": image_url}) response_inputs.append({"role": original_role, "content": contents}) From 6d0c44cb9a98ebb13b53b41b45b26fd0d3fcf851 Mon Sep 17 00:00:00 2001 From: Emmanuel Ferdman Date: Mon, 9 Jun 2025 03:54:55 -0700 Subject: [PATCH 2/2] Python: Throw exception on image error Signed-off-by: Emmanuel Ferdman --- .../agents/open_ai/responses_agent_thread_actions.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py b/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py index f6a1a210ccc5..527ffa5aa8eb 100644 --- a/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py +++ b/python/semantic_kernel/agents/open_ai/responses_agent_thread_actions.py @@ -699,7 +699,9 @@ def _prepare_chat_history_for_request( image_url = str(content.uri) if not image_url: - raise ValueError("ImageContent must have either a data_uri or uri set to be used in the request.") + raise ValueError( + "ImageContent must have either a data_uri or uri set to be used in the request." + ) contents.append({"type": "input_image", "image_url": image_url}) response_inputs.append({"role": original_role, "content": contents})