zack3d/big-agi
1
0
Fork 0
mirror of https://github.com/enricoros/big-AGI.git synced 2026-05-10 21:50:14 -07:00
Code Issues Packages Projects Releases Wiki Activity

AIX: OpenRouter: don't log on empty reasoning

Browse Source
This commit is contained in:
Enrico Ros
2026-02-13 12:01:52 -08:00
parent af919be2ac
commit 73f88d4715
2 changed files with 12 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files
src/modules/aix/server/dispatch/chatGenerate/parsers/openai.parser.ts
+10 -5
View File
@@ -206,9 +206,12 @@ export function createOpenAIChatCompletionsChunkParser(): ChatGenerateParseFunct
for (const reasoningDetail of delta.reasoning_details) {
// Extract text from reasoning blocks based on type
if (reasoningDetail.type === 'reasoning.text' && typeof reasoningDetail.text === 'string') {
pt.appendReasoningText(reasoningDetail.text);
deltaHasReasoning = true;
if (reasoningDetail.type === 'reasoning.text') {
if (typeof reasoningDetail.text === 'string') {
pt.appendReasoningText(reasoningDetail.text);
deltaHasReasoning = true;
}
// else: empty reasoning chunk, e.g. "{ type: 'reasoning.text' }", skip
}
// Summaries can also be shown as reasoning
else if (reasoningDetail.type === 'reasoning.summary' && typeof reasoningDetail.summary === 'string') {
@@ -474,8 +477,10 @@ export function createOpenAIChatCompletionsParserNS(): ChatGenerateParseFunction
// [OpenRouter, 2025-01-20] Handle structured reasoning_details
if (Array.isArray(message.reasoning_details)) {
for (const reasoningDetail of message.reasoning_details) {
if (reasoningDetail.type === 'reasoning.text' && typeof reasoningDetail.text === 'string') {
pt.appendReasoningText(reasoningDetail.text);
if (reasoningDetail.type === 'reasoning.text') {
if (typeof reasoningDetail.text === 'string')
pt.appendReasoningText(reasoningDetail.text);
// else: empty reasoning chunk, e.g. "{ type: 'reasoning.text' }", skip
} else if (reasoningDetail.type === 'reasoning.summary' && typeof reasoningDetail.summary === 'string') {
// pt.appendReasoningText(`[Summary] ${reasoningDetail.summary}`);
pt.appendReasoningText(reasoningDetail.summary);
src/modules/aix/server/dispatch/wiretypes/openai.wiretypes.ts
+2 -2
View File
@@ -115,8 +115,8 @@ export namespace OpenAIWire_ContentParts {
z.enum(['reasoning.summary', 'reasoning.text', 'reasoning.encrypted']),
z.string(),
]),
text: z.string().optional(), // Actual reasoning text (for 'text' type)
summary: z.string().optional(), // Summary of reasoning (for 'summary' type)
text: z.string().nullish(), // Actual reasoning text (for 'text' type)
summary: z.string().nullish(), // Summary of reasoning (for 'summary' type)
// we don't use these for now:
// signature: z.string().nullable().optional(), // Signature verification (for 'text' type)
// // 'encrypted' type has 'data' field - indicates reasoning happened but not returned