From 319cae3be6f2f70a6361357036f7180e80f39004 Mon Sep 17 00:00:00 2001 From: Tao Chen Date: Wed, 6 May 2026 17:25:20 -0700 Subject: [PATCH] Address comments --- .../agent_framework_foundry_hosting/_responses.py | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/python/packages/foundry_hosting/agent_framework_foundry_hosting/_responses.py b/python/packages/foundry_hosting/agent_framework_foundry_hosting/_responses.py index 10d89d198b..1645fcec2e 100644 --- a/python/packages/foundry_hosting/agent_framework_foundry_hosting/_responses.py +++ b/python/packages/foundry_hosting/agent_framework_foundry_hosting/_responses.py @@ -1477,6 +1477,11 @@ async def _to_outputs( await approval_storage.save_approval_request(approval_request_id, content) request_saved = True yield event + if approval_storage is not None and not request_saved: + logger.warning( + "Approval request was not saved to approval storage because the approval request ID " + "could not be extracted from the stream event." + ) else: # Log a warning for unsupported content types instead of raising an error to avoid breaking the response stream. logger.warning(f"Content type '{content.type}' is not supported yet. This is usually safe to ignore.")