-
Notifications
You must be signed in to change notification settings - Fork 48.1k
feat(vertex): route Claude models through the AnthropicVertex SDK #66522
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
Open
nickkpoon
wants to merge
26
commits into
NousResearch:main
Choose a base branch
from
nickkpoon:feat/claude-on-vertex
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
bcee1ba
feat(vertex): route Claude models through the AnthropicVertex SDK
nickkpoon ed47881
fix(vertex): send x-goog-user-project header for user-ADC quota billing
nickkpoon c5e1085
chore(vertex): refresh Claude picker suggestions to current lineup
nickkpoon 58c5b59
fix(vertex): retry transient MaaS concurrency 429s with backoff
nickkpoon cb3dfff
fix(vertex): add curated model list so /model picker enumerates verte…
nickkpoon f43b9b1
feat(vertex): route Claude models through AnthropicVertex in auxiliar…
nickkpoon a88cb95
fix(aux): preserve Anthropic cache/native usage fields through _Anthr…
nickkpoon 1d2c0a3
fix(vertex): show Google Vertex AI in the /model picker when ADC/SA c…
nickkpoon c535671
fix(vertex): centralize Claude client construction; model-aware api_m…
nickkpoon 462feb5
fix(vertex): route per-request/fallback/refresh client builds through…
nickkpoon 6a1ee48
fix(vertex): detect Claude-on-Vertex in the fallback-activation api_m…
nickkpoon 6ba697f
fix(model-switch): add 'fable' short alias so /model fable resolves t…
nickkpoon 4d75581
fix(vertex): recover auxiliary Gemini/openapi clients from ~1h OAuth …
nickkpoon 412159f
feat(vertex): add claude-opus-4-8 to the curated Vertex model catalog
nickkpoon 55a121b
fix: /branch HTTP 400 (whitespace placeholder) + duplicated pre-branc…
nickkpoon 085f4e2
fix(vertex): repair merge fallout from upstream/main integration
nickkpoon 33a65bf
chore: map nick.poon@irrigreen.com to nickkpoon for attribution check
nickkpoon f56fa88
fix(context): claude-opus-5 missing from 1M context tables
nickkpoon f5f642e
fix(vertex): keep one Claude generation per family in the curated cat…
nickkpoon b1a0757
test(vertex): sync _StubAgent with upstream's LM Studio switch-path c…
nickkpoon e99210f
test(vertex): mock refresh_vertex_credentials, not the retired get_ve…
nickkpoon 39202d0
local: pricing keys for opus-5/fable-5, Claude-on-Vertex billing rout…
nickkpoon 89ca050
test(vertex): sync _StubAgent with upstream's per-request Anthropic c…
nickkpoon cfb4c9e
feat(vertex): add google/gemini-3.7-flash (curated catalog, setup lis…
nickkpoon bf97a45
fix(vertex): route xai/grok-* billing to provider=xai + add Grok pric…
nickkpoon ffd8315
feat(vertex): support custom base_url for AnthropicVertex corporate p…
nickkpoon File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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 branch can be reached by
anthropic/claude-*, because the new classifier accepts that alias, but the primary model normalizer does not strip ananthropic/prefix for providervertex(hermes_cli/model_normalize.py:412-417,453-467). Normalize the model to the bare Vertex publisher ID before constructing the client, and add a main-agent alias regression test; the auxiliary branch already performs that strip.