chore(deps): upgrade AI SDK family to v4/v7 (supersedes #255, #249, #253, #250) - #275
Merged
Conversation
…i/react) Bumps ai 6.0.206 -> 7.0.37, @ai-sdk/google 3.0.82 -> 4.0.24, @ai-sdk/openai 3.0.71 -> 4.0.20, @ai-sdk/react 3.0.201 -> 4.0.40 as one coordinated unit, superseding Dependabot PRs #255, #249, #253, #250. Root cause for not merging them individually: @ai-sdk/google@4 emits the new LanguageModelV4 spec, but the `ai` package's LanguageModel type only recognizes it from v7 onward -- upgrading @ai-sdk/google alone (as #255 proposed) left services/ai/providerFactory.ts with a real tsc error (LanguageModelV4 not assignable to LanguageModelV2 | LanguageModelV3). Bumping the whole family together resolves it with no application code changes needed. Verified: pnpm run typecheck (clean), pnpm run lint (clean), and the full AI-provider/completion-fetch test suites (171 tests across aiProviderService.test.ts, geminiService.test.ts, providerFactory.test.ts, useWorldScriptAI.test.ts, worldScriptCompletionFetch.test.ts) all passing. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe AI SDK dependencies were upgraded to ChangesAI SDK upgrade
Estimated code review effort: 1 (Trivial) | ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
🏁 CodeAnt Quality Gate ResultsCommit: ✅ Overall Status: PASSEDQuality Gate Details
|
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Supersedes and closes the need for #255, #249, #253, #250 — those four Dependabot PRs each bump one package in the same AI SDK family, but only merging all four together actually works.
Root cause for why they couldn't merge individually:
@ai-sdk/google@4(proposed by #255) emits the newLanguageModelV4spec, but theaipackage'sLanguageModeltype only recognizesLanguageModelV4from v7 onward. Merging #255 alone leftservices/ai/providerFactory.tswith a realtscerror:Bumping
ai6→7,@ai-sdk/google3→4,@ai-sdk/openai3→4, and@ai-sdk/react3→4 together resolves this with no application code changes —streamText/toTextStreamResponse/onFinish'susageshape (services/ai/worldScriptCompletionFetch.ts) andcreateLanguageModelForWorldScript'sLanguageModelreturn type (services/ai/providerFactory.ts) are unaffected by the major bump.Test plan
pnpm run typecheck— clean (noLanguageModelV4mismatch)pnpm run lint— cleanaiProviderService.test.ts(52),geminiService.test.ts(28),providerFactory.test.ts(13),useWorldScriptAI.test.ts(7),worldScriptCompletionFetch.test.ts(19), plus the 52 in aiProviderService again post-bump for a clean re-runai/@ai-sdk/reactdirectly (providerFactory.ts,worldScriptCompletionFetch.ts,useWorldScriptAI.ts) — all covered by the aboveFollow-up
Once this merges, I'll close #255, #249, #253, #250 as superseded by this PR.
🤖 Generated with Claude Code
Summary by CodeRabbit