Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bump ai dependency to v4? #470

Open
tmcw opened this issue Dec 3, 2024 · 3 comments
Open

Bump ai dependency to v4? #470

tmcw opened this issue Dec 3, 2024 · 3 comments

Comments

@tmcw
Copy link

tmcw commented Dec 3, 2024

Hi! We use both this module as well as the vercel ai module directly in our application. This module pins ai at 4, which means that if we upgraded to 4 in our application, we'd still have a v3 version of ai sticking around as a transitive dependency. Could this module bump its dependency on ai?

@ankrgyl
Copy link
Contributor

ankrgyl commented Dec 3, 2024

CC @ibolmo

@ibolmo
Copy link
Contributor

ibolmo commented Dec 3, 2024

Thanks for reporting this @tmcw. We're currently deciding whether to keep the ai-sdk integration as part of the SDK (perhaps move to an optional peer dependency) or if we'll have a separate package i.e. @braintrust/ai-sdk

In either case, we'll have to consider backward compatibility requirements to not affect current users. Curious on your take, though, if you'd have a preference.

@tmcw
Copy link
Author

tmcw commented Dec 3, 2024

Moving them to a peer dependency would be ideal, I think - we don't use the integrations and they seem like they could be a constantly-expanding part of the API surface, if you want to support anthropic's, for example, as well. Also it kind of looks like (didn't thoroughly read through) the ai import was only used for types.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants