Fix: Improve serialization of completions/responses in Agents SDK instrumentation #845
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Fix: Improve serialization of completions/responses in Agents SDK instrumentation
This PR fixes the serialization of completions/responses in the Agents SDK OpenTelemetry instrumentation. Instead of truncating or printing the string representation of the object, it now extracts specific fields referenced in the semconv definitions.
Changes
model_as_dict()
helper function to safely convert response objects to dictionaries_export_span()
method to extract specific fields from response objects:Benefits
Link to Devin run: https://app.devin.ai/sessions/480cc142d3b94f70828c01815f3622ef
This is a draft PR for exploring potential solutions to the problem.