Skip to content

fix: register vertex provider in CLI menus - #68249

Closed
zmack12344321 wants to merge 1 commit into
NousResearch:mainfrom
zmack12344321:fix/register-vertex-provider
Closed

fix: register vertex provider in CLI menus#68249
zmack12344321 wants to merge 1 commit into
NousResearch:mainfrom
zmack12344321:fix/register-vertex-provider

Conversation

@zmack12344321

Copy link
Copy Markdown

Summary

The vertex provider plugin (plugins/model-providers/vertex/) works at the engine level, but is invisible to the CLI's /model menu and hermes doctor because the auto-extension loop in hermes_cli/auth.py explicitly filters out non-API-key providers.

Changes

  • Updated the PROVIDER_REGISTRY auto-extension loop in hermes_cli/auth.py to allow the vertex auth type.
  • Added the vertex overlay and human-friendly aliases to hermes_cli/providers.py.

Verification

  • Verified that vertex now appears in the /model menu.
  • Verified that hermes doctor now correctly resolves Vertex credentials via ADC.
  • Verified live request routing to Vertex AI endpoints with current GCP project ID.

- Update PROVIDER_REGISTRY auto-extension to allow 'vertex' auth_type
- Add 'vertex' overlay and aliases to providers.py
- Fixes issue where Vertex AI is working at runtime but invisible in /model
@alt-glitch alt-glitch added type/bug Something isn't working comp/cli CLI entry point, hermes_cli/, setup wizard provider/gemini Google Gemini (AI Studio, Cloud Code) area/auth Authentication, OAuth, credential pools sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades P3 Low — cosmetic, nice to have duplicate This issue or pull request already exists labels Jul 20, 2026
@alt-glitch

Copy link
Copy Markdown
Collaborator

This was generated by AI during triage.

Duplicate of #56688: it already provides the Vertex registry and overlay wiring in a broader, tested implementation. This PR also repeats overlay and alias entries within its own diff; please consolidate any wanted changes into the canonical PR.

@zmack12344321

Copy link
Copy Markdown
Author

Closing as duplicate of #56688 which provides a more comprehensive fix with tests. Consolidated changes locally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/auth Authentication, OAuth, credential pools comp/cli CLI entry point, hermes_cli/, setup wizard duplicate This issue or pull request already exists P3 Low — cosmetic, nice to have provider/gemini Google Gemini (AI Studio, Cloud Code) sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants