Repository navigation
fix(models): keep adaptive thinking for Claude 5 and default Haiku to 5.5 - #1580
Merged
Merged
Conversation
… 5.5 Claude 5.x and Opus 4.7/4.8 reject budget_tokens with a 400, so CLAUDE_CODE_DISABLE_ADAPTIVE_THINKING no longer falls back to a manual budget for them, and provider capability lists that omit required_thinking keep adaptive-only models adaptive. Non-Claude Anthropic-protocol models are unchanged. Add Haiku 5.5: canonical name, 1M context, 128K output, all effort levels (medium default), adaptive thinking with the disabled-thinking effort cap, prompt-length tiered pricing in both cost tables, and make it the first-party Haiku default (third-party providers keep Haiku 4.5).
PR quality triageChanged areas: area:cli-core, area:desktop, area:server CLI core policy: CLI-core change authored by a CODEOWNER; no override label needed. Missing-test policy: No missing-test policy block detected. Coverage baseline policy: No coverage-baseline policy block detected. CLI core files:
Coverage policy files:
Required check plan:
Test coverage signals:
Risk notes:
Hard merge gates come from the deterministic GitHub Actions contract lanes above. |
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Requests to
claude-opus-5-5through Modelflare failed with400 claude-opus-5-5 requires adaptive thinking. WithCLAUDE_CODE_DISABLE_ADAPTIVE_THINKING=1set, the CLI sentthinking: {type: "enabled", budget_tokens}. Two things combined to cause this: the provider capability string for Claude models omitsrequired_thinking, and the env var fell back to a manual budget. Some relays rewrite the field (AruHub), but strict ones forward it and the API rejects it.Per the official thinking table,
budget_tokensreturns 400 on every Claude 5.x model and on Opus 4.7/4.8. The official CLI removed this env var in 2.1.260.adaptive_thinkingbut leaves outrequired_thinking, adaptive-only models still stay adaptive. A provider can still opt out by leaving outadaptive_thinking.claude-haiku)mediumhighFeature Quality Contract
haiku55.test.ts: identity, limits, effort, thinking, prompt-length pricing in both tables, alias defaultscheck:coverage)Verification
Run on the rebased HEAD (target
origin/mainb6388b4):bun run check:server: 616 files, 7405 passed, 0 failedbun run check:provider-contract: 36 suites passedgit diff --check: cleancheck:agent-e2e,check:persistence-upgrade,check:coverage,bun run verify