Skip to content

fix(ai-gateway): expose Sakana Fugu Ultra - #4866

Open
chrarnoldus wants to merge 2 commits into
mainfrom
pickle-question
Open

fix(ai-gateway): expose Sakana Fugu Ultra#4866
chrarnoldus wants to merge 2 commits into
mainfrom
pickle-question

Conversation

@chrarnoldus

@chrarnoldus chrarnoldus commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Summary

  • remove the hard-coded OpenRouter filter for sakana/fugu

Verification

  • 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.ts
  • git diff --check

@chrarnoldus chrarnoldus self-assigned this Jul 29, 2026
!isForbiddenFreeModel(model.id) &&
!model.id.endsWith(':batch') &&
!unavailableModels.some(unavailableId => model.id.includes(unavailableId))
!model.id.endsWith(':batch')

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

intentional

@kilo-code-bot

kilo-code-bot Bot commented Jul 29, 2026

Copy link
Copy Markdown
Contributor

Code Review Summary

Status: 1 Issue Found | Recommendation: Address before merge

Executive Summary

The latest commit removes the only test that verified sakana/fugu is now exposed in the public model list, leaving the PR's core behavior change unverified.

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts (removed) Deleted the 'includes Sakana Fugu Ultra in the public model list' test, removing the only assertion that the EU-restriction removal actually exposes sakana/fugu; the PR's stated fix is now unverified by tests
Files Reviewed (1 file)
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts - 1 issue

Note: the previously flagged EU-availability filter removal (index.ts line 151) was confirmed intentional by the author in the review thread and is not carried forward as an open issue.

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 Summary

Removing the sakana/fugu EU-unavailability filter in openrouter/index.ts may expose the model to EU users without confirmation that the underlying regional restriction has been lifted.

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 0
Issue Details (click to expand)

WARNING

File Line Issue
apps/web/src/lib/ai-gateway/providers/openrouter/index.ts 151 Removed the unavailableModels EU-restriction filter (originally added because sakana/fugu "is not available in the EU") without confirming the underlying restriction is resolved
Files Reviewed (2 files)
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.ts - 1 issue
  • apps/web/src/lib/ai-gateway/providers/openrouter/index.test.ts - 0 issues

Fix these issues in Kilo Cloud


Reviewed by claude-sonnet-5 · Input: 22 · Output: 4.5K · Cached: 377.8K

Review guidance: REVIEW.md from base branch main

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.

1 participant