Fix reasoning data lost when restoring conversation history

[Cbrad24]

Originally introduced in PR #203.

This fixes #271, by ensuring that reasoning_id and reasoning_summary are replayed from database conversations to OpenAI reasoning models.

[mohali-id]

Its works very well for me

[joelclermont]

I can also confirm that this fixes the issue. We have run into this issue with reasoning models and this solves it.

Any timeline on when it could be merged?

[mohali-id]

I can also confirm that this fixes the issue. We have run into this issue with reasoning models and this solves it.

Any timeline on when it could be merged?

It still error if function call more then one in one message. Fixed in PR 461

[nikitaku11]

This doesn’t fix the issue, because all tool calls are still merged into a single message, while every provider requires two messages per tool call: #402

[mohali-id]

This doesn’t fix the issue, because all tool calls are still merged into a single message, while every provider requires two messages per tool call: #402

This PR #461 actually fix the issue

[Cbrad24]

This doesn’t fix the issue, because all tool calls are still merged into a single message, while every provider requires two messages per tool call: #402

Yes this PR is a month old now, and since then this package has pulled in its own gateway functionality into the package natively, but at the time I also lodged prism-php/prism#971 upstream to fix the other issue in the Prism gateway.

Been running both fixes using cweagans/composer-patches without issue since then, but the other PR might be the go now.