jacob/5890 #5956
jacob/5890 #5956
Conversation
…ation feat: add mixedbread ai integration
dosubot
bot
added
the
size:XL
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
dosubot
bot
added
auto:documentation
![dosubot[bot]](https://avatars.githubusercontent.com/in/324583?s=60&v=4){kind=small}
| @@ -0,0 +1,89 @@ | |||
| { | |||
There was a problem hiding this comment.
Hey there! 👋 This PR includes changes to the peer and dev dependencies of the project, as well as an update to the minimum required Node.js version. I've flagged this for your review to ensure the changes align with the project's requirements. Keep up the great work! 🚀
| @@ -0,0 +1,179 @@ | |||
| import { getEnvironmentVariable } from "@langchain/core/utils/env"; | |||
There was a problem hiding this comment.
Hey there! I noticed that the recent code changes explicitly add, access, read, or require an environment variable via process.env or getEnvironmentVariable. I've flagged this for the maintainers to review and ensure that the handling of environment variables aligns with best practices. Great work on the PR!
| @@ -0,0 +1,198 @@ | |||
| import { DocumentInterface } from "@langchain/core/documents"; | |||
There was a problem hiding this comment.
Hey there! I've reviewed the code and it looks great. I've left a comment flagging the use of environment variables for the maintainers to review. Keep up the good work!
|
@jacoblee93 Looks good to me! Please merge from here :) |
|
Thank you! Live here: https://www.npmjs.com/package/@langchain/mixedbread-ai |
|
🙏 |
@juliuslipp