Skip to content

feat(models): add Claude Sonnet 5 support - #55848

Closed
vortexopenclaw wants to merge 1 commit into
NousResearch:mainfrom
vortexopenclaw:feat/claude-sonnet-5-models
Closed

feat(models): add Claude Sonnet 5 support#55848
vortexopenclaw wants to merge 1 commit into
NousResearch:mainfrom
vortexopenclaw:feat/claude-sonnet-5-models

Conversation

@vortexopenclaw

Copy link
Copy Markdown

Summary

  • add Claude Sonnet 5 to OpenRouter/Nous/native Anthropic/OpenCode/KiloCode/Bedrock curated model lists
  • add 1M context, 128k output, timeout, Bedrock context, and pricing metadata
  • cover Sonnet 5 visibility in the Anthropic picker and output-token limit tests

Testing

  • uv run --extra dev pytest tests/hermes_cli/test_anthropic_picker_curated.py tests/agent/test_minimax_provider.py -q
  • uv run --extra dev pytest tests/hermes_cli/test_anthropic_picker_curated.py tests/hermes_cli/test_copilot_in_model_list.py tests/hermes_cli/test_model_switch_variant_tags.py tests/agent/test_reasoning_stale_timeout_floor.py tests/agent/test_minimax_provider.py -q

Source

@alt-glitch alt-glitch added type/feature New feature or request comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard provider/anthropic Anthropic native Messages API provider/bedrock AWS Bedrock (boto3, IAM) P3 Low — cosmetic, nice to have labels Jun 30, 2026

@teknium1 teknium1 left a comment

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.

Thanks for the broad Sonnet 5 coverage. Current main already added the OpenRouter and Nous curated visibility in 76a468e51315e7e822990257b284dc8ee938ff38, so this needs a conflict-aware salvage rather than applying its catalog edits directly.

Problems

  • agent/usage_pricing.py:174 records the standard $3/$15 rate. Anthropic’s current pricing page specifies introductory Sonnet 5 pricing through 2026-08-31: $2 input, $10 output, $0.20 cache reads, and $2.50 5-minute cache writes per MTok. The matching Bedrock entry has the same issue.
  • tests/hermes_cli/test_anthropic_picker_curated.py:23 and :30 add model-name snapshots. The repository rubric requires behavioral invariants instead of tests that fail whenever curated catalog data changes.

Suggested changes

  • Use the currently effective introductory pricing for both billing routes and add targeted native-Anthropic and normalized-Bedrock pricing tests.
  • Keep the existing merge/order invariant test without pinning Sonnet 5 by name; add behavior coverage for the Bedrock context fallback instead.

This is an automated hermes-sweeper review.

Comment thread agent/usage_pricing.py
): PricingEntry(
input_cost_per_million=Decimal("3.00"),
output_cost_per_million=Decimal("15.00"),
cache_read_cost_per_million=Decimal("0.30"),

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.

Anthropic currently lists Sonnet 5 introductory pricing through 2026-08-31 as $2 input / $10 output / $0.20 cache read / $2.50 5-minute cache write per MTok. Please use those effective rates here and in the matching Bedrock entry; $3/$15 is the rate starting 2026-09-01.

@teknium1 teknium1 added sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform labels Jul 15, 2026
teknium1 added a commit that referenced this pull request Jul 20, 2026
PRs #55848 and #55853 both added claude-sonnet-5 to the anthropic and
gmi curated lists; keep one entry each (newest-sonnet-first ordering
under claude-fable-5, matching the existing list convention).
teknium1 added a commit that referenced this pull request Jul 20, 2026
PR #55848 and #60410 both added an (anthropic, claude-sonnet-5) pricing
key; the later duplicate (/$15 standard rate) would silently win in
the dict literal. Keep the intro pricing entry ($2/$10 through
2026-08-31 per Anthropic docs) which carries the reversion note.
@teknium1

Copy link
Copy Markdown
Contributor

Merged via PR #67932 with your commit cherry-picked onto current main — your authorship is preserved in git history (rebase merge). You were the earliest and most complete submission in the Sonnet 5 cluster, so your commit was used as the base; thanks for the thorough coverage (lists, pricing, metadata, timeouts, tests).

One note: the duplicate (anthropic, claude-sonnet-5) standard-rate pricing entry was dropped in favor of #60410's introductory-pricing entry ($2/$10 through 2026-08-31), which matches Anthropic's current docs.

#67932

randlee pushed a commit to randlee/hermes-agent that referenced this pull request Aug 11, 2026
PRs NousResearch#55848 and NousResearch#55853 both added claude-sonnet-5 to the anthropic and
gmi curated lists; keep one entry each (newest-sonnet-first ordering
under claude-fable-5, matching the existing list convention).
randlee pushed a commit to randlee/hermes-agent that referenced this pull request Aug 11, 2026
PR NousResearch#55848 and NousResearch#60410 both added an (anthropic, claude-sonnet-5) pricing
key; the later duplicate (/$15 standard rate) would silently win in
the dict literal. Keep the intro pricing entry ($2/$10 through
2026-08-31 per Anthropic docs) which carries the reversion note.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp/agent Core agent runtime: loop, agent_init, prompt builder, context-compression, responses endpoint comp/cli CLI entry point, hermes_cli/, setup wizard P3 Low — cosmetic, nice to have provider/anthropic Anthropic native Messages API provider/bedrock AWS Bedrock (boto3, IAM) sweeper:blast-moderate Sweeper blast radius: moderate — a subsystem or single platform sweeper:risk-compatibility Sweeper risk: may break existing users, config, migrations, defaults, or upgrades type/feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants