Skip to content

feat: Vertex AI credentials support and LangChain migration#79

Merged
lfnovo merged 1 commit intomainfrom
vertex-credentials-and-langchain-migration
Feb 7, 2026
Merged

feat: Vertex AI credentials support and LangChain migration#79
lfnovo merged 1 commit intomainfrom
vertex-credentials-and-langchain-migration

Conversation

@lfnovo
Copy link
Owner

@lfnovo lfnovo commented Feb 7, 2026

Summary

  • Add credentials_file parameter to Vertex AI provider with full credentials priority chain (explicit file → GOOGLE_APPLICATION_CREDENTIALS → ADC → gcloud CLI fallback)
  • Migrate to_langchain() from deprecated ChatVertexAI (langchain-google-vertexai) to ChatGoogleGenerativeAI (langchain-google-genai)
  • Remove langchain-google-vertexai dev dependency
  • Bump version to 2.19.0

Test plan

  • All 44 vertex provider tests pass (including 12 new tests)
  • Full test suite passes with no regressions (992 passing)
  • No remaining references to langchain-google-vertexai in source code
  • uv sync --all-extras succeeds with updated lock file

Copy link

@cubic-dev-ai cubic-dev-ai bot left a comment

Choose a reason for hiding this comment

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

1 issue found across 7 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="src/esperanto/providers/llm/vertex.py">

<violation number="1" location="src/esperanto/providers/llm/vertex.py:103">
P1: When `credentials_file` is explicitly provided, loading failures (missing `google-auth` or invalid file) should raise an error instead of silently falling back to other methods. This prevents using incorrect identities (e.g., personal gcloud credentials) when the intended service account fails to load.</violation>
</file>

Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.

- Add credentials_file parameter with priority chain: explicit file,
  GOOGLE_APPLICATION_CREDENTIALS env var, ADC, gcloud CLI fallback
- Migrate to_langchain() from deprecated ChatVertexAI to
  ChatGoogleGenerativeAI (langchain-google-genai)
- Remove langchain-google-vertexai dependency
- Add comprehensive tests for credential loading and LangChain migration
- Bump version to 2.19.0
@lfnovo lfnovo force-pushed the vertex-credentials-and-langchain-migration branch from 5cb4cc8 to e393d1d Compare February 7, 2026 14:15
@lfnovo lfnovo merged commit c0cbe2b into main Feb 7, 2026
4 checks passed
@lfnovo lfnovo deleted the vertex-credentials-and-langchain-migration branch February 7, 2026 14:22
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.

1 participant