Skip to content

fix(anthropic): skip thinking params for Haiku models#1130

Merged
teknium1 merged 1 commit into
mainfrom
hermes/hermes-c877bdeb
Mar 13, 2026
Merged

fix(anthropic): skip thinking params for Haiku models#1130
teknium1 merged 1 commit into
mainfrom
hermes/hermes-c877bdeb

Conversation

@teknium1

Copy link
Copy Markdown
Contributor

Haiku models don't support extended thinking. Without this guard, claude-haiku-4-5-20251001 gets thinking: {type: enabled, budget_tokens: ...} and returns 400.

Adds "haiku" not in model.lower() guard on top of #1128's adaptive thinking refactor. Incorporates the fix from PR #1127 by @frizynn.

Verified live:

  • claude-opus-4-6 → adaptive thinking ✓
  • claude-haiku-4-5-20251001 → no thinking params ✓
  • claude-sonnet-4-20250514 → enabled thinking ✓

Haiku models don't support extended thinking at all. Without this
guard, claude-haiku-4-5-20251001 would receive type=enabled +
budget_tokens and return a 400 error.

Incorporates the fix from PR #1127 (by frizynn) on top of #1128's
adaptive thinking refactor.

Verified live with Claude Code OAuth:
  claude-opus-4-6       → adaptive thinking ✓
  claude-haiku-4-5      → no thinking params ✓
  claude-sonnet-4       → enabled thinking ✓
@teknium1 teknium1 merged commit 6ceae61 into main Mar 13, 2026
1 check passed
angelburgosrosado pushed a commit to angelburgosrosado/hermes-agent that referenced this pull request Apr 27, 2026
…c877bdeb

fix(anthropic): skip thinking params for Haiku models
02356abc pushed a commit to 02356abc/hermes-agent that referenced this pull request May 14, 2026
…c877bdeb

fix(anthropic): skip thinking params for Haiku models
olympus-terminal pushed a commit to olympus-terminal/hermes-agent that referenced this pull request May 16, 2026
…c877bdeb

fix(anthropic): skip thinking params for Haiku models
Egavasyug pushed a commit to Egavasyug/hermes-agent that referenced this pull request Jun 10, 2026
…c877bdeb

fix(anthropic): skip thinking params for Haiku models
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