Skip to content

fix(receipts): send no temperature to the AI service - #183

Merged
codebyarne merged 1 commit into
mainfrom
claude/fout-vanaf-mijn-pi-hnmhv2
Nov 13, 2026
Merged

codebyarne merged 1 commit into
mainfrom
claude/fout-vanaf-mijn-pi-hnmhv2

Conversation

@codebyarne

Copy link
Copy Markdown
Owner

Wat er misging

Bij "Bonnetjes lezen met AI" kon je Anthropic (Claude Haiku 5.5) niet opslaan: "Anthropic gaf een antwoord dat Huismus niet begrijpt." Huismus stuurde temperature: 0 mee, en nieuwere modellen (Claude Haiku 5.5, Sonnet 5.5, Opus 5.5) weigeren dat met een 400. Die 400 viel onder "antwoord dat Huismus niet begrijpt", en wat de dienst zei, kwam nergens in het log.

Wat er verandert

  • Het verzoek stuurt geen temperature meer; de dienst gebruikt zijn eigen standaard.
  • Mislukt de test van de dienst (bij opslaan of "Test"), dan staat het antwoord van de dienst in het log (nooit de sleutel).
  • De nep-dienst in de tests weigert temperature, top_p en top_k bij elk verzoek, zoals deze modellen doen: zonder de fix vallen 51 tests om.
  • Regel in BUGS-BY-HAND.md.

Elk platform: werkt overal hetzelfde (Docker, Mac-app, Windows-app).

./scripts/verify.sh: groen.


Generated by Claude Code

Newer models (Anthropic's Claude Haiku 5.5 among them) refuse sampling
settings with a 400, so saving Anthropic in "Bonnetjes lezen met AI"
failed with "gaf een antwoord dat Huismus niet begrijpt". The request
now leaves them to the service's default, the test fake refuses them
like those models do, and a failed test logs what the service said.

Playbook: n.v.t. — bugfix in Huismus zelf, geen werkwijze

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01W6o7JHCnFJVV7ub63VfxGH
@codebyarne
codebyarne merged commit ed2d9bc into main Nov 13, 2026
2 checks passed
@codebyarne
codebyarne deleted the claude/fout-vanaf-mijn-pi-hnmhv2 branch November 13, 2026 21:53
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.

2 participants