Skip to content

Fix Claude provider: stop sending temperature - #37

Open
Yoosseph wants to merge 1 commit into
mainfrom
fix/claude-temperature
Open

Yoosseph wants to merge 1 commit into
mainfrom
fix/claude-temperature

Conversation

@Yoosseph

@Yoosseph Yoosseph commented Nov 7, 2026

Copy link
Copy Markdown
Owner

Summary

The Claude provider added in #36 sends temperature with anthropic-version: 2026-10-21. That API version rejects temperature, top_p and top_k with a 400 on every model, so every Claude debrief fails. The 400 maps to unavailable, so users see "Claude (Anthropic) is unreachable right now."

This removes temperature from the Claude request body. Claude now samples at its default (1.0) instead of 0 to 0.3. Other providers are unchanged.

Evidence

  • The Add OpenAI, Claude, and DeepSeek API key support #36 tests mocked fetch and asserted temperature: 0 was sent. The test now asserts the exact body without it.
  • npm test (149 worker + 91 CLI) and npm run typecheck pass.
  • Not yet checked with a real Anthropic key. After deploy, one ownit debrief with a Claude key confirms it.

Merge Danger

Door: two-way. Worker only, no CLI release needed. Merging triggers the production Deploy workflow.

🤖 Generated with Claude Code

anthropic-version 2026-10-21 rejects temperature, top_p and top_k with a
400 on every model, so every Claude debrief failed and surfaced as
"Claude (Anthropic) is unreachable right now." Claude now samples at its
default temperature.

This branch has not been deployed

No deployments
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