Skip to content

fix: omit unset temperature from Responses API requests 🩷#12977

Open
kingrubic wants to merge 1 commit into
continuedev:mainfrom
kingrubic:fix/12972-optional-temperature
Open

fix: omit unset temperature from Responses API requests 🩷#12977
kingrubic wants to merge 1 commit into
continuedev:mainfrom
kingrubic:fix/12972-optional-temperature

Conversation

@kingrubic

Copy link
Copy Markdown

Description

Responses API requests no longer include a temperature property when the option is unset. Explicit values, including 0, are preserved. This avoids failures from OpenAI-compatible endpoints that reject the parameter.

Closes #12972

AI Code Review

  • Team members only: AI review runs automatically when PR is opened or marked ready for review
  • Team members can also trigger a review by commenting @continue-review

Checklist

  • I have read the contributing guide
  • No documentation update is needed for this request serialization fix
  • The relevant regression test was added

Screen recording or screenshot

Not applicable; this change affects request serialization only.

Tests

  • npm test -- --runInBand llm/llms/OpenAI.test.ts (3 passed)
  • npm run tsc:check
  • Prettier check on both changed files

@kingrubic
kingrubic requested a review from a team as a code owner July 12, 2026 00:04
@kingrubic
kingrubic requested review from sestinj and removed request for a team July 12, 2026 00:04
@github-actions

github-actions Bot commented Jul 12, 2026

Copy link
Copy Markdown
Contributor

All contributors have signed the CLA ✍️ ✅
Posted by the CLA Assistant Lite bot.

@dosubot dosubot Bot added the size:XS This PR changes 0-9 lines, ignoring generated files. label Jul 12, 2026
@kingrubic kingrubic changed the title fix: omit unset temperature from Responses API requests fix: omit unset temperature from Responses API requests 🩷 Jul 12, 2026
@kingrubic

Copy link
Copy Markdown
Author

I have read the CLA Document and I hereby sign the CLA

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size:XS This PR changes 0-9 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

The temperature field will be automatically injected even if it is not set.

1 participant