-
Notifications
You must be signed in to change notification settings - Fork 12
Update langgraph deps #262
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?
Update langgraph deps #262
Conversation
"@langchain/langgraph-cli": "^0.0.17", | ||
"@langchain/langgraph-sdk": "^0.0.57", | ||
"@langchain/openai": "^0.3.14", |
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.
can we try to check if upgrading examples' @langchain dependency is breaking or non-breaking as well?
e.g.
"@langchain/core": "^0.3.72",
"@langchain/langgraph": "^0.4.6",
"@langchain/langgraph-checkpoint": "0.0.18",
"@langchain/langgraph-cli": "^0.0.60",
"@langchain/langgraph-sdk": "^0.0.109"
This is a good start though if we can get by w/o breaking changes to the core, but upgrading examples' dependency will also be useful. If we have to modify typings, consumption etc. let's see how much the impact..and change if we can.
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.
@priley86 updated
@@ -17,7 +17,7 @@ | |||
"dependencies": { | |||
"@auth0/ai-langchain": "*", | |||
"@langchain/openai": "^0.3.17", | |||
"@openfga/sdk": "^0.8.0", | |||
"@openfga/sdk": "0.8.1", | |||
"dotenv": "^16.4.7", | |||
"langchain": "^0.3.12" |
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.
is "langchain" still used? Seems it is so slowly being moved to the core @langchain/*
. Something to consider, but certainly not required to remove.
looks mostly good here @PavelKorobchuk 👍 👍 tweaked a bit further in: #264 today. Thank you for verifying commits 🙇 |
fix: fixed build
31fbd20
to
981e203
Compare
Updated langgraph deps to the latest versions