Repository navigation
chore: move API requests from claude-sonnet-4-5 to claude-sonnet-5-5 - #95
Open
jayypatel18 wants to merge 1 commit into
Open
jayypatel18 wants to merge 1 commit into
jayypatel18 wants to merge 1 commit into
Conversation
This branch has not been deployed
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.
What
This moves the API requests that send
claude-sonnet-4-5toclaude-sonnet-5-5.src/lib/analyzer.ts:DEFAULT_ANALYZER_MODELgoes fromclaude-sonnet-4-5-20250929toclaude-sonnet-5-5, because the analyzer sends this default as its request model and the old id is retiring.src/lib/env-check.ts: the model in theclient.messages.createcall insidecheckApiKeychanges fromclaude-sonnet-4-5-20250929toclaude-sonnet-5-5, so the key check runs against a model that stays available.src/commands/analyze.ts: the--modelhelp text now says the default isclaude-sonnet-5-5, so it matches the newDEFAULT_ANALYZER_MODEL.src/commands/run.ts: the--analyzer-modelhelp text and theoasis config set default-modelhint now nameclaude-sonnet-5-5instead ofclaude-sonnet-4-5-20250929, so users are pointed at the model that is still served.What changes for you
thinkingfield ran without thinking on Claude Sonnet 4.5 and runs with adaptive thinking on Claude Sonnet 5.5. Thinking tokens are billed as output tokens, andmax_tokenscovers thinking plus text.max_tokensis not changed here.max_tokens. What these requests send today, unchanged:src/lib/env-check.ts(max_tokens: 1).src/commands/providers.ts(line 20),src/lib/providers.ts(line 19),tests/unit/report.test.ts(lines 14, 73, 116).Why
Claude Sonnet 4.5 (
claude-sonnet-4-5-20250929) is deprecated and retires on November 30, 2026. Requests to a retired model fail. The recommended replacement isclaude-sonnet-5-5.How
Only the places that send a request, or tell a user which model is the default, were changed. Entries that describe the old model (the provider list and the test fixtures) are left as they are.
src/commands/run.tsline 21 still showsclaude-sonnet-4-5-20250929in a help example; say if you want that changed too.Checklist
npm run typecheckpassesnpm testpassesnpm run buildsucceedsWhat I checked
In a clean Node 20 container, on
main(22b57a4) and then on this branch, with install scripts off:mainnpm installnpm testtsc --noEmitnpm run typechecknpm run buildNot checked: No request was sent to the API, so the model's answers, cost and latency were not compared. The app was not run.