Skip to content

chore(providers/openai): re-introduce logprobs as providerMetadata#6049

Merged
samdenty merged 7 commits intov5from
v5-openai-logprobs
Apr 30, 2025
Merged

chore(providers/openai): re-introduce logprobs as providerMetadata#6049
samdenty merged 7 commits intov5from
v5-openai-logprobs

Conversation

@samdenty
Copy link
Copy Markdown
Contributor

@samdenty samdenty commented Apr 29, 2025 •
edited by lgrammel
Loading

Background

In #5896 we removed logprobs to simplify models.

Summary

Re-introduce logprobs as provider-specific functionality for openai.

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
lgrammel reviewed Apr 30, 2025
View reviewed changes
lgrammel
lgrammel reviewed Apr 30, 2025
View reviewed changes
@lgrammel
Copy link
Copy Markdown
Collaborator

lgrammel commented Apr 30, 2025

Please add a section to the OpenAI provider docs.

lgrammel
lgrammel reviewed Apr 30, 2025
View reviewed changes
lgrammel
lgrammel reviewed Apr 30, 2025
View reviewed changes
lgrammel
lgrammel reviewed Apr 30, 2025
View reviewed changes
lgrammel
lgrammel reviewed Apr 30, 2025
View reviewed changes
lgrammel
lgrammel reviewed Apr 30, 2025
View reviewed changes
@samdenty samdenty requested a review from lgrammel April 30, 2025 14:53
@samdenty samdenty closed this Apr 30, 2025
@samdenty samdenty reopened this Apr 30, 2025
@samdenty samdenty merged commit 136819b into v5 Apr 30, 2025
8 checks passed
@samdenty samdenty deleted the v5-openai-logprobs branch April 30, 2025 17:13
gr2m added a commit that referenced this pull request May 1, 2025
@gr2m
test (openai): update snapshot
8d9dfc0 
Follow up to #6049
@gr2m gr2m mentioned this pull request May 1, 2025
Merged
lgrammel pushed a commit that referenced this pull request May 1, 2025
@gr2m
test (openai): update snapshot (#6086)
b8239db 
Follow up to #6049
This was referenced May 1, 2025
Merged
Closed
samdenty added a commit that referenced this pull request May 2, 2025
@samdenty
fix(providers/openai): logprobs for stream alongside completion model (…
928fadf 
…#6091)

Followup to #6049, we were missing
streaming logprobs it was commented for the completion model but
accidentally missed this one
iteratetograceness pushed a commit to iteratetograceness/ai that referenced this pull request May 9, 2025
@samdenty @iteratetograceness
chore(providers/openai): re-introduce logprobs as providerMetadata (v…
5053fc2 
…ercel#6049)

## Background

In vercel#5896 we removed logprobs to
simplify models.

## Summary

Re-introduce logprobs as provider-specific functionality for openai.
iteratetograceness pushed a commit to iteratetograceness/ai that referenced this pull request May 9, 2025
@gr2m @iteratetograceness
test (openai): update snapshot (vercel#6086)
13d8103 
Follow up to vercel#6049
iteratetograceness pushed a commit to iteratetograceness/ai that referenced this pull request May 9, 2025
@samdenty @iteratetograceness
fix(providers/openai): logprobs for stream alongside completion model (…
39288b7 
…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

Reviewers

@lgrammel lgrammel lgrammel approved these changes

Assignees

@samdenty samdenty

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

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@samdenty @lgrammel @gr2m