Skip to content
This repository was archived by the owner on Dec 11, 2025. It is now read-only.

feat: Add support for claude-sonnet-4.5 model#106

Merged
r0ymanesco merged 5 commits into
mainfrom
ENG-3520_add-support-claude-sonnet-4.5
Oct 17, 2025
Merged

feat: Add support for claude-sonnet-4.5 model#106
r0ymanesco merged 5 commits into
mainfrom
ENG-3520_add-support-claude-sonnet-4.5

Conversation

@dragos199993

Copy link
Copy Markdown
Member
  • Updated settings.py to include claude-sonnet-4.5 in the PROVIDERS list.
  • Modified providers.py to define the new model in the NDLLMProviders enum.
  • Added test cases in test_anthropic.py to validate functionality for claude-sonnet-4.5 with tool calling.

- Updated settings.py to include claude-sonnet-4.5 in the PROVIDERS list.
- Modified providers.py to define the new model in the NDLLMProviders enum.
- Added test cases in test_anthropic.py to validate functionality for claude-sonnet-4.5 with tool calling.
@github-actions

github-actions Bot commented Oct 10, 2025

Copy link
Copy Markdown

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  notdiamond/llms
  providers.py
  tests/test_llm_calls
  test_anthropic.py
Project Total  

This report was generated by python-coverage-comment-action

CLAUDE_OPUS_4_20250514 (NDLLMProvider): refers to 'claude-opus-4-20250514' model by Anthropic
CLAUDE_OPUS_4_1_20250805 (NDLLMProvider): refers to 'claude-opus-4-1-20250805' model by Anthropic
CLAUDE_SONNET_4_20250514 (NDLLMProvider): refers to 'claude-sonnet-4-20250514' model by Anthropic
CLAUDE_SONNET_4_5 (NDLLMProvider): refers to 'claude-sonnet-4-5' model by Anthropic

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.

You should also add the dated versino

Comment thread notdiamond/settings.py
"claude-opus-4-20250514",
"claude-opus-4-1-20250805",
"claude-sonnet-4-20250514",
"claude-sonnet-4-5",

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.

You need to add the dated one here as well

@r0ymanesco r0ymanesco merged commit 850eebc into main Oct 17, 2025
2 of 3 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants