@vincemakes/kiso-provider-openai 0.1.2 → 0.1.4

package/dist/index.js

@@ -294,30 +294,43 @@ function toOpenAIMessages(messages, systemPrompt) {
     if (systemPrompt !== undefined) {
         out.push({ role: "system", content: systemPrompt });
     }
-    for (const msg of messages) {
+    // 自举 P1/P2: thinking mode is detected by the presence of ANY
+    // reasoning in the projection (real OpenAI never emits thinking events,
+    // so its requests never see the field). In thinking mode, ONLY the
+    // CURRENT turn's assistant messages (after the last user message) carry
+    // reasoning_content — their own, or "" when the step produced no
+    // thinking (the field must still be present, or DeepSeek 400s); OLD
+    // turns' CoT is never echoed (DeepSeek does not need it, and echoing
+    // it is token waste).
+    const thinkingMode = messages.some((m) => m.role === "assistant" && m.reasoning !== undefined);
+    let lastUser = -1;
+    for (const [i, m] of messages.entries()) {
+        if (m.role === "user")
+            lastUser = i;
+    }
+    for (const [i, msg] of messages.entries()) {
         if (msg.role === "user") {
             out.push({ role: "user", content: toOpenAIContent(msg.content) });
         }
         else if (msg.role === "assistant") {
+            // OpenAI-compat APIs reject an EMPTY tool_calls array with 400
+            // ("expected an array with minimum length 1") — when the turn
+            // made no tool calls, the field is omitted entirely.
+            const toolCalls = msg.blocks
+                .filter((b) => b.type === "tool_use")
+                .map((b) => {
+                const t = b;
+                return {
+                    id: t.callId,
+                    type: "function",
+                    function: { name: t.name, arguments: JSON.stringify(t.input) },
+                };
+            });
             out.push({
                 role: "assistant",
                 content: msg.blocks.filter((b) => b.type === "text").map((b) => b.text).join(""),
-                tool_calls: msg.blocks
-                    .filter((b) => b.type === "tool_use")
-                    .map((b) => {
-                    const t = b;
-                    return {
-                        id: t.callId,
-                        type: "function",
-                        function: { name: t.name, arguments: JSON.stringify(t.input) },
-                    };
-                }),
-                // 自举 P1: DeepSeek's thinking mode REQUIRES the turn's
-                // reasoning back on follow-up requests (else 400) — the
-                // projection derived it from the thinking events; the compat
-                // wire field is reasoning_content. Real OpenAI never emits
-                // thinking events, so it never sees this field.
-                ...(msg.reasoning !== undefined ? { reasoning_content: msg.reasoning } : {}),
+                ...(toolCalls.length ? { tool_calls: toolCalls } : {}),
+                ...(thinkingMode && i > lastUser ? { reasoning_content: msg.reasoning ?? "" } : {}),
             });
         }
         else {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vincemakes/kiso-provider-openai",
-  "version": "0.1.2",
+  "version": "0.1.4",
   "description": "kiso OpenAI-compatible adapter \u2014 OpenAI + the compat family (GLM, Kimi, DeepSeek, OpenRouter) via base_url swap, reasoning dialects digested.",
   "type": "module",
   "license": "MIT",
@@ -21,7 +21,7 @@
     "test": "vitest run"
   },
   "dependencies": {
-    "@vincemakes/kiso-core": "0.1.2",
+    "@vincemakes/kiso-core": "0.1.4",
     "openai": "^7.3.0"
   },
   "devDependencies": {