-
Notifications
You must be signed in to change notification settings - Fork 5
Gemini #100
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?
Gemini #100
Conversation
"@langchain/anthropic": "^0.3.22", | ||
"@langchain/community": "^0.3.46", | ||
"@langchain/core": "^0.3.58", | ||
"@langchain/google-genai": "^0.2.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.
The latest version of @langchain/google-genai
seems to depend on newer releases of @langchain/core
, so bumping the other langchain dependencies here.
tsconfig.json
Outdated
@@ -19,6 +19,7 @@ | |||
"resolveJsonModule": true, | |||
"outDir": "lib", | |||
"rootDir": "src", | |||
"skipLibCheck": true, |
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.
Need to investigate the following TypeScript build issue:
node_modules/zod/v3/index.d.ts:1:15 - error TS2846: A declaration file cannot be imported without 'import type'. Did you mean to import an implementation file '../dist/types/v3/index.js' instead?
1 export * from "../dist/types/v3/index.d.ts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
node_modules/zod/v4/core/index.d.ts:1:15 - error TS2846: A declaration file cannot be imported without 'import type'. Did you mean to import an implementation file '../../dist/types/v4/core/index.js' instead?
1 export * from "../../dist/types/v4/core/index.d.ts";
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
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.
This seems to be fixed colinhacks/zod#4545
Working fine with resolutions
pining it to "^3.25.56"
.
"resolutions": {
"zod": "^3.25.56"
},
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.
Oh cool, thanks!
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 opened langchain-ai/langchainjs#8388
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.
Actually my fix was useless, and this dependency issue is already fixed by langchain-ai/langchainjs#8380, not yet released AFAIK.
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.
Looks like the latest release may have these changes now: https://github.com/langchain-ai/langchainjs/releases/tag/langchain%3D%3D0.3.29
Looks like there is now an issue when running the |
The failing tests seem to be related to the upgrade of I don't know how to fix it, but we have the same issue in #110 |
CI now passes after #113. Did a bit of testing locally. The chat works fine. Completions often give a 429 with a lot of requests being made to the Gemini API. So not sure yet if it's because of the API key being used, an issue with the langchain wrapper, or a missing option when instantiating the completer. |
Was curious if it would be quick and easy to add Gemini to the list of default providers.
Fixes #99
jupyterlite-ai-gemini.mp4