From 57f64347ac49707c6518d47d5ca29d1f2040755b Mon Sep 17 00:00:00 2001 From: artboy <80608452+itxaiohanglover@users.noreply.github.com> Date: Tue, 23 Jun 2026 14:26:45 +0800 Subject: [PATCH] fix(llms): skip reasoning_effort patch for *-chat-latest models normalizeModelName reduces gpt-5.2-chat-latest to gpt-52-chat-latest, which matches the gpt-52 prefix and gets reasoning_effort='none'. But chat-latest models don't support reasoning_effort, causing HTTP 400. Fix: check for 'chat-latest' before applying reasoning_effort patches. Fixes #563 --- packages/llms/src/utils.ts | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/packages/llms/src/utils.ts b/packages/llms/src/utils.ts index d3839d3..462906c 100644 --- a/packages/llms/src/utils.ts +++ b/packages/llms/src/utils.ts @@ -79,7 +79,10 @@ export function modelPatch(body: Record) { debug('Applying GPT patch: set verbosity to low') body.verbosity = 'low' - if (modelName.startsWith('gpt-52')) { + // *-chat-latest models don't support reasoning_effort — skip patches that set it + if (modelName.includes('chat-latest')) { + debug('Skipping reasoning_effort patch for chat-latest model') + } else if (modelName.startsWith('gpt-52')) { debug('Applying GPT-52 patch: disable reasoning') body.reasoning_effort = 'none' } else if (modelName.startsWith('gpt-51')) {