Skip to content

adapters send temperature unconditionally (forward-looking 400 risk) #22

Description

@ernestprovo23

Forward-looking — do not fix speculatively. temperature is sent unconditionally in all three adapters (openai_compat.py:79, anthropic.py:83, gemini.py:85). Some current models (e.g. certain OpenAI reasoning models) reject a non-default temperature with a 400. Not a bug for today's listed defaults, but a foreseeable failure as #5 adds providers; it surfaces as an opaque per-member error.

Acceptance (defer until a concrete provider needs it)

  • Allow temperature=None to omit the param from the request body.
  • Note in adapter docs.

Source: pre-dev planning audit 2026-06-08 (finding D-11).

Metadata

Metadata

Assignees

No one assigned

    Labels

    tech-debtRefactor, debt paydown, internal quality

    Type

    No type

    Fields

    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions