OpenAI Responses API Instrumentation

[TimPansino]

Overview

• Instrument the newer openai.responses APIs.
• Add a new set of tests that mirrors the original chat_completions tests.

[github-actions]

✅MegaLinter analysis: Success

Descriptor	Linter	Files	Fixed	Errors	Max errors	Warnings	Elapsed time
✅ ACTION	actionlint	11		0		0	1.05s
✅ MARKDOWN	markdownlint	8	0	0		0	1.2s
✅ PYTHON	ruff	1141	0	0		0	1.15s
✅ PYTHON	ruff-format	1141	0	0		0	0.26s
✅ YAML	prettier	26	0	0		0	1.23s
✅ YAML	v8r	26		0		0	6.12s
✅ YAML	yamllint	26		0		0	0.72s

See detailed reports in MegaLinter artifacts

Show us your support by starring ⭐ the repository

[codecov-commenter]

Codecov Report

❌ Patch coverage is 87.93103% with 14 lines in your changes missing coverage. Please review.
✅ Project coverage is 81.70%. Comparing base (e3e51a1) to head (740fa7d).

Files with missing lines	Patch %	Lines
newrelic/hooks/mlmodel_openai.py	87.82%	2 Missing and 12 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1813      +/-   ##
==========================================
- Coverage   81.75%   81.70%   -0.05%     
==========================================
  Files         223      223              
  Lines       28013    28116     +103     
  Branches     4433     4456      +23     
==========================================
+ Hits        22901    22972      +71     
- Misses       3560     3575      +15     
- Partials     1552     1569      +17     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[umaannamalai]

For chat completions, we deliberately suppress content-less tool call messages (on line 595 by checking "tool_call" in finish_reason). Since we don't want to report every tool call as an LlmChatCompletionMessage with content: "", do we want to mirror that suppression behavior here if possible?

[umaannamalai]

Or is that why "role" is being checked for explicitly?

[umaannamalai]

Are we capturing errors surfaced via response.failed somewhere? From my scan of the codebase, it seems like APIErrors are raised when the raw payload has a top-level error key. response.failed events nest errors inside response.error and the stream just ends normally afterwards. I just want to make sure we are capturing errors in this case also.