Skip to content

feat(models): declare reasoning efforts for sakana fugu-ultra - #3083

Merged
steebchen merged 1 commit into
theopenco:mainfrom
AmineAce:feat/reasoning-sakana
Jul 16, 2026
Merged

steebchen merged 1 commit into
theopenco:mainfrom
AmineAce:feat/reasoning-sakana

Conversation

@AmineAce

@AmineAce AmineAce commented Jul 16, 2026 •

Copy link
Copy Markdown
Contributor

Summary

Declares reasoningEfforts for the providerId: "sakana" mapping (fugu-ultra) in packages/models/src/models/sakana.ts.

Changes

  • fugu-ultra → ["high", "xhigh"]

Sources:

  • Sakana AI / On Note, which states Fugu models accept two reasoning effort levels, high and xhigh, set via the reasoning.effort parameter; any other value is rejected. max is also accepted but is explicitly a compatibility alias that currently maps to the same level as xhigh, not a distinct effort level.
  • The same page's Codex model catalog config (fugu.json) independently lists supported_reasoning_levels for fugu-ultra as exactly high and xhigh, confirming the same two values from a second source on the same page.

max is excluded from reasoningEfforts since it doesn't produce behavior distinct from xhigh, same convention used for DeepSeek's xhigh→max aliasing. none, low, and medium are not documented as accepted values for this model and are excluded.

Testing

  • pnpm format, only sakana.ts changed, 1 line.

@coderabbitai

coderabbitai Bot commented Jul 16, 2026 •

Copy link
Copy Markdown
Contributor

Review Change Stack

Walkthrough

The fugu-ultra model definition now declares "high" and "xhigh" as supported reasoning effort levels.

Changes

Sakana model metadata

Layer / File(s) Summary
Configure fugu-ultra reasoning efforts
packages/models/src/models/sakana.ts
The fugu-ultra configuration adds reasoningEfforts: ["high", "xhigh"].

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

Suggested reviewers: steebchen

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the main change: adding reasoning-effort declarations for Sakana's fugu-ultra model.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@steebchen
steebchen merged commit f3dacdc into theopenco:main Jul 16, 2026
12 checks passed
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