Skip to content

fix: reasoning streaming#751

Merged
iBotPeaches merged 1 commit intoopenai-php:mainfrom
Infomaniak:fix/reasoning
Mar 27, 2026
Merged

fix: reasoning streaming#751
iBotPeaches merged 1 commit intoopenai-php:mainfrom
Infomaniak:fix/reasoning

Conversation

@L3o-pold
Copy link
Copy Markdown
Contributor

What:

  • Bug Fix
  • New Feature

Description:

Using streaming response, reasoning was not handled.

Copy link
Copy Markdown
Collaborator

@iBotPeaches iBotPeaches left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@L3o-pold
Copy link
Copy Markdown
Contributor Author

@iBotPeaches I think it's more for OpenAI compilant server like VLLM

Copy link
Copy Markdown
Collaborator

@iBotPeaches iBotPeaches left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't love growing these for non-openai, but Chat is pretty much dead for pure OpenAI usages as I'm guessing most moved to Responses. So I think its fine to expand Chat as long as all properties are nullable like done here.

@iBotPeaches iBotPeaches merged commit 9c0b8cc into openai-php:main Mar 27, 2026
14 checks passed
@iBotPeaches iBotPeaches added this to the v0.20.0 milestone Mar 27, 2026
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.

2 participants