Conversation
There was a problem hiding this comment.
Pull request overview
This PR migrates from beta versions of the AI SDK packages to stable releases, addressing issue #19. The changes update the core AI SDK dependencies to specific stable versions and include updates to various development dependencies.
- Replaces beta AI SDK dependencies with stable versions (^1.0.29, ^2.0.0, ^3.0.19)
- Adds the 'ai' package as a direct dependency (^5.0.113)
- Updates development dependencies including vitest (^4.0.15), eslint (^9.39.2), and @types/node (^25.0.2)
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | Updates AI SDK dependencies from "beta" to stable versions, adds "ai" package, and updates devDependencies to newer versions |
| package-lock.json | Reflects the dependency version updates with complete resolution tree for all new stable package versions |
| src/model-id.ts | Removes several model IDs (ai21-jamba-1.5-mini, jais-30b-chat, mistral-large-2411, mistral-nemo, cohere-embed-v3 variants) from the type definitions |
| examples/ai/package.json | Updates "ai" package from "beta" to ^5.0.68 for the example application |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "dependencies": { | ||
| "@github/models": "file:../../", | ||
| "ai": "beta", | ||
| "ai": "^5.0.68", |
There was a problem hiding this comment.
The 'ai' package version in the example (^5.0.68) is lower than the version in the main package.json (^5.0.113). These versions should be aligned to ensure consistent behavior across the project and avoid potential compatibility issues.
| "ai": "^5.0.68", | |
| "ai": "^5.0.113", |
| "@ai-sdk/provider-utils": "beta" | ||
| "@ai-sdk/openai-compatible": "^1.0.29", | ||
| "@ai-sdk/provider": "^2.0.0", | ||
| "@ai-sdk/provider-utils": "^3.0.19", |
There was a problem hiding this comment.
The 'ai' package is added as a direct dependency, but it appears to be used by consumers of this SDK rather than by the SDK itself internally. Consider whether this should be a peerDependency instead to allow consumers to manage their own version of the 'ai' package and avoid potential version conflicts.
| "@ai-sdk/provider-utils": "^3.0.19", | |
| "@ai-sdk/provider-utils": "^3.0.19" | |
| }, | |
| "peerDependencies": { |
| "dependencies": { | ||
| "@github/models": "file:../../", | ||
| "ai": "beta", | ||
| "ai": "^5.0.68", |
There was a problem hiding this comment.
ai sdk has now released v6.0 stable version
should resolve #19 but I'm not entirely sure what the intent of using the beta SDK was. also that issue could just be a skill issue on my end.