@space3-npm/cybersoul-client 1.4.25 → 1.4.26

package/dist/client.js

@@ -809,21 +809,49 @@ Note: Always include "isEndTurn". If "imageParams", "voiceArgs", "triggerEvent",
                         "\n\nReturn only valid JSON matching the schema. Escape newlines inside JSON strings with \\n. Keep imageParams values in ENGLISH and use the provided enums.",
                 },
             ];
-            // 3. Local Execute LLM
-            const rawLlmResponse = await this.llm.generate(promptMessages, 15000, 0.7);
-            // console.debug("[CyberSoulClient] Raw LLM Response:", rawLlmResponse);
-            let parsedIntent;
-            try {
-                parsedIntent = robustJsonParse(rawLlmResponse, "Dispatcher fallback", { textResponse: "", actionText: "", isEndTurn: false });
+            // 3. Local Execute LLM (retry on non-actionable parse).
+            //    A "non-actionable" parse yields no usable `textResponse` AND no
+            //    media intent — i.e. the LLM hiccuped (empty body, truncated or
+            //    garbled JSON, or an empty textResponse field). Retrying the
+            //    generation is far better than silently echoing the user's own
+            //    message back as the character's reply (the historical
+            //    `: params.userMessage` fallback below), which downstream
+            //    consumers correctly reject as a non-actionable response.
+            const MAX_DISPATCH_ATTEMPTS = 3;
+            let parsedIntent = { textResponse: "" };
+            for (let attempt = 1; attempt <= MAX_DISPATCH_ATTEMPTS; attempt++) {
+                const rawLlmResponse = await this.llm.generate(promptMessages, 15000, 0.7);
+                // console.debug("[CyberSoulClient] Raw LLM Response:", rawLlmResponse);
+                try {
+                    parsedIntent = robustJsonParse(rawLlmResponse, "Dispatcher fallback", { textResponse: "", actionText: "", isEndTurn: false });
+                }
+                catch (e) {
+                    console.warn("[CyberSoulClient] JSON parse failed, falling back to raw text:", e);
+                    // Fallback robust mode - just text if completely broken
+                    parsedIntent = {
+                        textResponse: rawLlmResponse.replace(/^[\`\s]+|[\`\s]+$/g, "").trim(),
+                    };
+                }
+                // console.debug("[CyberSoulClient] Parsed Intent:", parsedIntent);
+                const hasUsableText = typeof parsedIntent.textResponse === "string" &&
+                    parsedIntent.textResponse.trim().length > 0;
+                const hasMediaIntent = !!parsedIntent.imageParams || !!parsedIntent.voiceArgs;
+                if (hasUsableText || hasMediaIntent) {
+                    break;
+                }
+                console.warn(`[CyberSoulClient] interact produced a non-actionable intent (attempt ${attempt}/${MAX_DISPATCH_ATTEMPTS}); ${attempt < MAX_DISPATCH_ATTEMPTS ? "retrying" : "giving up"}.`);
             }
-            catch (e) {
-                console.warn("[CyberSoulClient] JSON parse failed, falling back to raw text:", e);
-                // Fallback robust mode - just text if completely broken
-                parsedIntent = {
-                    textResponse: rawLlmResponse.replace(/^[\`\s]+|[\`\s]+$/g, "").trim(),
-                };
+            // After exhausting retries, fail fast instead of echoing the user's
+            // own message back as the character's reply. The `: params.userMessage`
+            // fallback used below is only ever reached for media-only turns now.
+            {
+                const finalHasUsableText = typeof parsedIntent.textResponse === "string" &&
+                    parsedIntent.textResponse.trim().length > 0;
+                const finalHasMediaIntent = !!parsedIntent.imageParams || !!parsedIntent.voiceArgs;
+                if (!finalHasUsableText && !finalHasMediaIntent) {
+                    throw new Error("LLM returned a non-actionable response after retries (no usable textResponse and no media intent). Check LLM template/provider alignment.");
+                }
             }
-            // console.debug("[CyberSoulClient] Parsed Intent:", parsedIntent);
             // 4. Update Backend State async (in parallel with media generation
             //    below). We keep the promise so we can resolve the
             //    server-authoritative `temperature` / `relationshipStage` and

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@space3-npm/cybersoul-client",
-  "version": "1.4.25",
+  "version": "1.4.26",
   "type": "module",
   "main": "dist/index.js",
   "module": "dist/index.js",