Skip to content

fix(ai-gateway): prefer Vertex for Claude models - #6048

Merged
chrarnoldus merged 4 commits into
mainfrom
chrarnoldus/prefer-vertex-for-claude
Sep 10, 2026
Merged

fix(ai-gateway): prefer Vertex for Claude models#6048
chrarnoldus merged 4 commits into
mainfrom
chrarnoldus/prefer-vertex-for-claude

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • prefer Google Vertex before Bedrock and Anthropic for Claude models
  • apply the preferred provider order to Claude Fable models
  • update provider-routing unit coverage

Testing

  • pnpm --filter web test --runInBand src/lib/ai-gateway/providers/apply-provider-specific-logic.test.ts
  • pnpm --filter web lint
  • pnpm --filter web typecheck
  • git diff --check

@chrarnoldus chrarnoldus self-assigned this Sep 10, 2026
@kilo-code-bot

kilo-code-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.ts
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.test.ts
Previous Review Summaries (3 snapshots, latest commit 8bb10e8)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 8bb10e8)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.ts
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.test.ts

Previous review (commit 01ea1f5)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.ts
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.test.ts

Previous review (commit e481dd2)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.ts
  • apps/web/src/lib/ai-gateway/providers/apply-provider-specific-logic.test.ts

Reviewed by grok-4.6 · Input: 37.8K · Output: 5.8K · Cached: 180.6K

Review guidance: REVIEW.md from base branch main

@chrarnoldus
chrarnoldus force-pushed the chrarnoldus/prefer-vertex-for-claude branch from 8bb10e8 to 01ea1f5 Compare September 10, 2026 10:54
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