Skip to content

fix: preserve Fable fallback on Vercel#4578

Merged
chrarnoldus merged 1 commit into
mainfrom
fix/fable-vercel-provider-fallback
Jul 16, 2026
Merged

fix: preserve Fable fallback on Vercel#4578
chrarnoldus merged 1 commit into
mainfrom
fix/fable-vercel-provider-fallback

Conversation

@chrarnoldus

Copy link
Copy Markdown
Contributor

Summary

  • avoid forcing Fable requests through Amazon Bedrock on Vercel
  • preserve the configured Claude Opus fallback
  • add regression coverage for Fable provider ordering

Testing

  • pnpm --filter web test -- --runInBand apps/web/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 Jul 16, 2026
@kilo-code-bot

kilo-code-bot Bot commented Jul 16, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

Reviewed the Fable/Vercel provider-order fix and its regression test; the guard correctly excludes Fable from the Bedrock/Anthropic preferred-provider order without affecting the existing Opus fallback logic, and no high-confidence bugs were found in the changed lines.

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 claude-sonnet-5 · Input: 28 · Output: 5.5K · Cached: 540.1K

Review guidance: REVIEW.md from base branch main

@chrarnoldus chrarnoldus merged commit 9c335f7 into main Jul 16, 2026
15 checks passed
@chrarnoldus chrarnoldus deleted the fix/fable-vercel-provider-fallback branch July 16, 2026 09:56
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