Skip to content

fix(providers/openai): logprobs for stream alongside completion model#6091

Merged
samdenty merged 3 commits intov5from
v5-openai-logprobs2
May 2, 2025
Merged

fix(providers/openai): logprobs for stream alongside completion model#6091
samdenty merged 3 commits intov5from
v5-openai-logprobs2

Conversation

@samdenty
Copy link
Copy Markdown
Contributor

@samdenty samdenty commented May 1, 2025

Background

Followup to #6049, we were missing streaming logprobs it was commented for the completion model but accidentally missed this one

Summary

Verification

Tasks

  • Tests have been added / updated (for bug fixes / features)
  • Documentation has been added / updated (for bug fixes / features)
  • A patch changeset for relevant packages has been added (for bug fixes / features - run pnpm changeset in the project root)
  • Formatting issues have been fixed (run pnpm prettier-fix in the project root)

Future Work

Related Issues

@lgrammel
Copy link
Copy Markdown
Collaborator

lgrammel commented May 2, 2025

needs changeset (the task has been checked off?)

@samdenty
Copy link
Copy Markdown
Contributor Author

samdenty commented May 2, 2025

needs changeset oops yeah i thought because we already had one in the repo one wasn't required, but i see new additions need one

@samdenty samdenty merged commit 928fadf into v5 May 2, 2025
8 checks passed
@samdenty samdenty deleted the v5-openai-logprobs2 branch May 2, 2025 13:13
samdenty pushed a commit that referenced this pull request May 2, 2025
* origin/v5:
  fix(providers/openai): logprobs for stream alongside completion model (#6091)
  chore (ai): move maxSteps into UseChatOptions (#6113)
  docs (ai): add notes on schema .optional() (#6111)
  chore: update references to use ai-sdk.dev (#6076) (#6107)
  Version Packages (canary) (#6069)
  chore (provider-utils): rename TestServerCall.requestBody to requestBodyJson (#6106)
  chore (ai): rename CoreMessage to ModelMessage (#6105)
  feat (ai): add filename to file ui parts (#6104)
  feat (docs): add Dify community provider (#4084) (#6067) (#6103)
  feat (docs): add Sarvam community provider (#6079) (#6096)
  fix(providers/openai): zod parse error with function (#6074) (#6095)
  feat(docs): add Weave observability provider (#6029) (#6094)
  core (ai/mcp): update experimental MCP client documentation for Streamable HTTP transport usage (#5972) (#6093)
  fix (docs): update formatting in useChat reference docs  (#6100)
  chore (ai): replace useChat attachments with file ui parts (#6071)
  feat (ui/react): support resuming an ongoing stream (#6053)
  test (openai): update snapshot (#6086)
samdenty pushed a commit that referenced this pull request May 2, 2025
…call

* origin/v5: (43 commits)
  chore (provider-utils): return IdGenerator interface (#6118)
  chore (provider): improve jsdoc comments (#6117)
  chore (ai): remove outdated typecasts (#6116)
  chore (ai): remove useChat keepLastMessageOnError (#6114)
  fix(providers/openai): logprobs for stream alongside completion model (#6091)
  chore (ai): move maxSteps into UseChatOptions (#6113)
  docs (ai): add notes on schema .optional() (#6111)
  chore: update references to use ai-sdk.dev (#6076) (#6107)
  Version Packages (canary) (#6069)
  chore (provider-utils): rename TestServerCall.requestBody to requestBodyJson (#6106)
  chore (ai): rename CoreMessage to ModelMessage (#6105)
  feat (ai): add filename to file ui parts (#6104)
  feat (docs): add Dify community provider (#4084) (#6067) (#6103)
  feat (docs): add Sarvam community provider (#6079) (#6096)
  fix(providers/openai): zod parse error with function (#6074) (#6095)
  feat(docs): add Weave observability provider (#6029) (#6094)
  core (ai/mcp): update experimental MCP client documentation for Streamable HTTP transport usage (#5972) (#6093)
  fix (docs): update formatting in useChat reference docs  (#6100)
  chore (ai): replace useChat attachments with file ui parts (#6071)
  feat (ui/react): support resuming an ongoing stream (#6053)
  ...
samdenty pushed a commit that referenced this pull request May 5, 2025
* origin/v5: (23 commits)
  chore (ai): remove StreamData and mergeStreams (#6150)
  chore (build): fix typecheck setup (#6151)
  chore (ai): remove IdGenerator duplication (#6147)
  chore (ai): remove content from ui messages (#6145)
  Version Packages (canary) (#6115)
  feat (provider): support reasoning tokens, cached input tokens, total token in usage information (#6140)
  test (ai): use `mock-provider` instead of `openai` to match used `MockImageModelV2` (#6132)
  fix(providers): always use optional instead of mix of nullish for providerOptions (#6122)
  chore (ai): remove sendExtraMessageFields (#6120)
  chore(provider-utils): move ToolResultContent to provider-utils (#6121)
  feat(embed-many): respect supportsParallelCalls & concurrency (#6108)
  chore (provider-utils): return IdGenerator interface (#6118)
  chore (provider): improve jsdoc comments (#6117)
  chore (ai): remove outdated typecasts (#6116)
  chore (ai): remove useChat keepLastMessageOnError (#6114)
  fix(providers/openai): logprobs for stream alongside completion model (#6091)
  chore (ai): move maxSteps into UseChatOptions (#6113)
  docs (ai): add notes on schema .optional() (#6111)
  chore: update references to use ai-sdk.dev (#6076) (#6107)
  Version Packages (canary) (#6069)
  ...
iteratetograceness pushed a commit to iteratetograceness/ai that referenced this pull request May 9, 2025
…vercel#6091)

Followup to vercel#6049, we were missing
streaming logprobs it was commented for the completion model but
accidentally missed this one
@gr2m gr2m added ai/provider related to a provider package. Must be assigned together with at least one `provider/*` label provider/openai Issues related to the @ai-sdk/openai provider labels Oct 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai/provider related to a provider package. Must be assigned together with at least one `provider/*` label provider/openai Issues related to the @ai-sdk/openai provider

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants