Add clause to avoid reporting empty content in openai. (#1517)

umaannamalai · umaannamalai · commit 6c342fe209ba · 2025-09-23T18:39:05.000-07:00
diff --git a/newrelic/hooks/mlmodel_openai.py b/newrelic/hooks/mlmodel_openai.py
@@ -489,6 +489,8 @@ def _record_completion_success(transaction, linking_metadata, completion_id, kwa
                     choices[0].get("message") or {"content": choices[0].get("text"), "role": "assistant"}
                 ]
                 finish_reason = choices[0].get("finish_reason")
+            if "tool_calls" in output_message_list[0] and not output_message_list[0].get("content"):
+                output_message_list = []
         else:
             response_model = kwargs.get("response.model")
             response_id = kwargs.get("id")

Original file line number	Diff line number	Diff line change
`@@ -489,6 +489,8 @@ def _record_completion_success(transaction, linking_metadata, completion_id, kwa`
`489`	`489`	`choices[0].get("message") or {"content": choices[0].get("text"), "role": "assistant"}`
`490`	`490`	`]`
`491`	`491`	`finish_reason = choices[0].get("finish_reason")`
	`492`	`+ if "tool_calls" in output_message_list[0] and not output_message_list[0].get("content"):`
	`493`	`+ output_message_list = []`
`492`	`494`	`else:`
`493`	`495`	`response_model = kwargs.get("response.model")`
`494`	`496`	`response_id = kwargs.get("id")`