fix(ai-gateway): expose Sakana Fugu Ultra - #4866
Conversation
| !isForbiddenFreeModel(model.id) && | ||
| !model.id.endsWith(':batch') && | ||
| !unavailableModels.some(unavailableId => model.id.includes(unavailableId)) | ||
| !model.id.endsWith(':batch') |
There was a problem hiding this comment.
WARNING: Removed EU-availability filter without a replacement check
This change removes the unavailableModels filter, whose sole entry (sakana/fugu) was excluded with the comment "this model is not available in the EU". Deleting the filter exposes the model to all users, including those in the EU, with no indication in the PR that the underlying unavailability has been resolved on the OpenRouter side. If the model is still geo-restricted, EU users selecting it from the catalog will now hit request failures instead of never seeing it offered.
Please confirm the EU availability restriction no longer applies (e.g., via OpenRouter/Sakana), or reintroduce equivalent handling if it does.
Reply with @kilocode-bot fix it to have Kilo Code address this issue.
Code Review SummaryStatus: 1 Issue Found | Recommendation: Address before merge Executive SummaryThe latest commit removes the only test that verified Overview
Issue Details (click to expand)WARNING
Files Reviewed (1 file)
Note: the previously flagged EU-availability filter removal ( Fix these issues in Kilo Cloud Previous Review Summary (commit 643d1cd)Current summary above is authoritative. Previous snapshots are kept for context only. Previous review (commit 643d1cd)Status: 1 Issue Found | Recommendation: Address before merge Executive SummaryRemoving the Overview
Issue Details (click to expand)WARNING
Files Reviewed (2 files)
Reviewed by claude-sonnet-5 · Input: 22 · Output: 4.5K · Cached: 377.8K Review guidance: REVIEW.md from base branch |
Summary
sakana/fuguVerification
pnpm exec oxlint --config .oxlintrc.json apps/web/src/lib/ai-gateway/providers/openrouter/index.ts apps/web/src/lib/ai-gateway/providers/openrouter/index.test.tsgit diff --check