-
Notifications
You must be signed in to change notification settings - Fork 424
refactor(plugins/googleai): migrate to v2 API #3479
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
3126f71
to
df51ca8
Compare
(Giving this another self-review and some manual testing) |
d74f7f4
to
f66475b
Compare
async () => { | ||
return genkitPluginV2({ | ||
name: 'googleai', | ||
async init() { |
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.
@ifielker can we drop init function? Is there anything here that can't be dynamically resolved by the resolver? and/or not returned by list?
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.
Before I review any further... why are we refactoring an essentially deprecated plugin?
ah, I thought it was plugins/google-genai/googleai... but either way, we might as well, so that we don't have loose ends w.r.t. to the v2 plugin api migration. We haven't done much planning on exactly how we're going to deprecate this plugin... |
This PR migrates the googleai plugin to the v2 API
Resolves #3443