Skip to content

chore(firestore-translate-text/deps): switch from googleai and vertexai Genkit plugins to google-genai#2745

Merged
CorieW merged 3 commits intonextfrom
@invertase/ftt-use-googlegenai
Apr 9, 2026
Merged

chore(firestore-translate-text/deps): switch from googleai and vertexai Genkit plugins to google-genai#2745
CorieW merged 3 commits intonextfrom
@invertase/ftt-use-googlegenai

Conversation

@CorieW
Copy link
Copy Markdown
Member

@CorieW CorieW commented Apr 8, 2026

Description:

Fixes the issue #2573 when merged alongside #2574. From testing, Genkit's googleai plugin seems to do some sanitization. I believe this is likely not an issue in the plugin's code, as I couldn't find any sanitization there. This problem likely occurs in the version of the GoogleAI SDK used in the googleai plugin. This PR introduces google-genai plugin to fix this sanitization problem. Switching to using google-genai plugin is also recommended, as it's more up-to-date.

Testing:

image

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request migrates the Genkit plugins from the separate @genkit-ai/googleai and @genkit-ai/vertexai packages to the unified @genkit-ai/google-genai package and updates the genkit dependency to version 1.31.0. A review comment suggests updating the changelog entry to use a colon instead of a dash for better alignment with conventional commit formatting.

Co-authored-by: gemini-code-assist[bot] <176961590+gemini-code-assist[bot]@users.noreply.github.com>
Copy link
Copy Markdown
Contributor

@cabljac cabljac left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

@CorieW CorieW merged commit d772812 into next Apr 9, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants