Skip to content

fix: Fix reasoning_content for openai compatible responses - #2524

Open
Luca Forstner (lforst) wants to merge 1 commit into
mainfrom
lforst/dum-e/milan-933dfae6be
Open

Luca Forstner (lforst) wants to merge 1 commit into
mainfrom
lforst/dum-e/milan-933dfae6be

Conversation

@lforst

Copy link
Copy Markdown
Member
  • Fix missing reasoning text in traces from streamed OpenAI-compatible responses. Previously, reasoning_content fragments were dropped; they are now combined separately from the final answer for each response choice.
  • Preserve empty and null reasoning values without allowing later null fragments to erase text already received. This applies to both manual wrapping and automatic instrumentation.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant