-
Notifications
You must be signed in to change notification settings - Fork 434
Update vercel ai sdk to v4 #65
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
base: main
Are you sure you want to change the base?
Conversation
@deepu105 is attempting to deploy a commit to the LangChain Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
couple nits, need to try out locally but from a first glance, lgtm!
} else if (message.getType() === "tool") { | ||
return { | ||
content: message.content, | ||
role: "system", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this should be role: tool
?
role: "system", | |
role: "tool", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Vercel Message
has bound the type to role: 'system' | 'user' | 'assistant' | 'data';
so I thought system was the closest to tool
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
hmm weird. id need to read the docs then but i dont think system would be the right role for this
Hey Guys, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I tried it out locally, and when compared to the deployed version I'm seeing some differences. Specifically around rendering tool calls. See the screenshots below. The screenshot which shows the tool call dropdown is from the deployed version, and the one without is your branch.

.env | ||
|
||
.vscode |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we dont want to gitignore this
.vscode |
Updated to latest Vercel AI SDK and also updated LangChain versions