marketing-shibata50/9router
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

fix: strip reasoning_content from non-streaming responses (closes #509) (#517)

Browse source
This commit is contained in:
Anurag Saxena 2026-04-06 22:46:28 -04:00 • committed by GitHub
parent fca829aa2a
commit a53ccf1343
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
2 changed files with 15 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

8
open-sse/handlers/chatCore/nonStreamingHandler.js
View file

@ -185,6 +185,14 @@ export async function handleNonStreamingResponse({ providerResponse, provider, m
translatedResponse.usage = filterUsageForFormat(addBufferToUsage(translatedResponse.usage), sourceFormat);
}
// Strip reasoning_content — some clients (e.g. Firecrawl AI SDK) have JSON parsers that
// break on this non-standard field, even though OpenAI allows it in extensions.
if (translatedResponse?.choices) {
for (const choice of translatedResponse.choices) {
if (choice?.message) delete choice.message.reasoning_content;
}
}
reqLogger.logConvertedResponse(translatedResponse);
const totalLatency = Date.now() - requestStartTime;

7
open-sse/handlers/chatCore/sseToJsonHandler.js
View file

@ -210,6 +210,13 @@ export async function handleForcedSSEToJson({ providerResponse, sourceFormat, pr
status: "success"
}, { endpoint: clientRawRequest?.endpoint || null })).catch(() => {});
// Strip reasoning_content — breaks JSON parsers in some clients (e.g. Firecrawl AI SDK)
if (parsed?.choices) {
for (const choice of parsed.choices) {
if (choice?.message) delete choice.message.reasoning_content;
}
}
return { success: true, response: new Response(JSON.stringify(parsed), { headers: { "Content-Type": "application/json", "Access-Control-Allow-Origin": "*" } }) };
} catch (err) {
console.error("[ChatCore] Chat Completions SSE→JSON failed:", err);