Skip to content

fix: update anthropic default to claude sonnet 5.5 - #145

Open
Anandb71 wants to merge 1 commit into
EfficientAI-tech:mainfrom
Anandb71:arbor/retire-models
Open

Anandb71 wants to merge 1 commit into
EfficientAI-tech:mainfrom
Anandb71:arbor/retire-models

Conversation

@Anandb71

@Anandb71 Anandb71 commented Nov 9, 2026

Copy link
Copy Markdown

What Changed?

as discussed over email, the anthropic service default moves from claude-sonnet-4-5-20250929 to claude-sonnet-5-5.

Why?

sonnet 4.5 retires on nov 30, 2026. per the migration guide, sonnet 5.5 rejects temperature, top_p and top_k, so for that model i drop them from the stream request (including overrides from extra). the non streaming path now joins text blocks instead of reading content[0], since a thinking block can come first.

one thing i left alone on purpose: sonnet 5.5 thinks adaptively by default, and tool continuations normally replay the signed thinking block. the streaming loop doesn't keep those yet. happy to add that in a follow up if you want, it touches the context aggregator so i didn't want to guess at your design here.

How to Test?

PYTHONPATH=src pytest tests/test_services/test_anthropic_llm.py --disable-socket --allow-unix-socket -q (6 passed, python 3.11, mocked sdk, no network). black and ruff pass on the touched files.

Release Label

  • fix

Checklist

  • i have read the CONTRIBUTING.md guide.
  • my changes follow the project's style guidelines.
  • i have added tests that prove my fix is effective.
  • i have updated documentation where needed.

@greptile-apps

greptile-apps Bot commented Nov 9, 2026 •

Copy link
Copy Markdown

RetriggerView in GreptileConfidence Score: 5/5

[Medium impact] The reviewed changes appear safe to merge.

Summary

Updates the Anthropic service default to claude-sonnet-5-5.

  • Anthropic requests now default to Sonnet 5.5 and drop its sampling fields.
  • One-shot Anthropic replies now return text from every text block.

Anandb71 explicitly deferred preserving signed thinking blocks during tool continuations because it touches the context aggregator.

Reviews (1) · Last reviewed commit: "fix: update anthropic default to claude ..." · Reviewed by Greptile

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.

1 participant