fix(ai): remove itemId from providerOptions for openai models#886
fix(ai): remove itemId from providerOptions for openai models#886bhuvaneshprasad wants to merge 1 commit intovercel:mainfrom
Conversation
|
|
@bhuvaneshprasad is attempting to deploy a commit to the Vercel Labs Team on Vercel. A member of the Team first needs to authorize it. |
5f6a49f to
daddac9
Compare
Fix DurableAgent + OpenAI Responses API failing on tool calls due to missing required reasoning item. The issue occurred because tool calls were forwarding itemId in providerMetadata, which requires a paired reasoning item that DurableAgent doesn't provide. Changes: - Remove itemId from tool call providerMetadata - Add previousResponseId tracking for proper conversation continuity - Extract responseId from finish chunk and pass it in subsequent requests The previousResponseId approach is sufficient for conversation continuity and avoids the duplicate item error that occurs when both previousResponseId and itemId are present. Fixes vercel#880 Signed-off-by: bhuvaneshprasad <98113536+bhuvaneshprasad@users.noreply.github.com>
daddac9 to
fbcd78a
Compare
|
Hi @bhuvaneshprasad, thanks for contributing! We're just now catching up on community PRs. Could you:
and then this should be good to merge |
|
@bhuvaneshprasad Was this PR succeeded by #889 and now works for you as intended? Let me know if not, otherwise I'll close this PR after some time |
|
Hi @VaguelySerious , the #889 should work. We can close this PR. Thanks |
Fix DurableAgent + OpenAI Responses API failing on tool calls due to missing required reasoning item. The issue occurred because tool calls were forwarding itemId in providerMetadata, which requires a paired reasoning item that DurableAgent doesn't provide.
Changes:
The previousResponseId approach is sufficient for conversation continuity and avoids the duplicate item error that occurs when both previousResponseId and itemId are present.
Fixes #880