-
Notifications
You must be signed in to change notification settings - Fork 864
Add Anthropic Claude Opus 5 API support #1026
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -6,7 +6,12 @@ import { getConversationPairs } from '../../utils/get-conversation-pairs.mjs' | |
| import { getModelValue } from '../../utils/model-name-convert.mjs' | ||
|
|
||
| function shouldOmitTemperature(model) { | ||
| return model === 'claude-opus-4-7' || model === 'claude-opus-4-8' || model === 'claude-sonnet-5' | ||
| return ( | ||
| model === 'claude-opus-4-7' || | ||
| model === 'claude-opus-4-8' || | ||
| model === 'claude-sonnet-5' || | ||
| model === 'claude-opus-5' | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more.
When Opus 5's always-on adaptive thinking consumes Useful? React with 👍 / 👎.
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. This is valid, but the handling is not specific to Opus 5. It affects Claude streaming responses generally and requires broader stop-reason and error-propagation coverage. It will be addressed in a focused follow-up change rather than expanding this model-registration PR. |
||
| ) | ||
| } | ||
|
|
||
| function shouldDisableDefaultThinking(model) { | ||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
When any of the 12 non-English locales is active, this new key is absent from that locale's catalog and only renders through the English fallback. Add the label to each existing locale as required for newly introduced English strings.
AGENTS.md reference: AGENTS.md:L209-L215
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No change needed. Commit
1c976a3intentionally removed redundant model labels from non-English locales because i18next falls back to English. Only labels with actual localization differences are kept, so the English entry is sufficient here.