wingbot 3.75.9-alpha.2 → 3.75.9-alpha.4

package/jsconfig.json

@@ -1,7 +1,7 @@
 {
     "compilerOptions": {
         "module": "commonjs",
-        "lib": ["es2019", "es2020.promise", "es2020.bigint", "es2020.string", "dom"],
+        "lib": ["es2019", "es2020.promise", "es2020.bigint", "es2020.string", "es2022.error", "dom"],
         "target": "ESNext",
         "allowSyntheticDefaultImports": true,
         "checkJs": true,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "wingbot",
-  "version": "3.75.9-alpha.2",
+  "version": "3.75.9-alpha.4",
   "description": "Enterprise Messaging Bot Conversation Engine",
   "main": "index.js",
   "type": "commonjs",

package/src/ChatGpt.js

@@ -373,7 +373,8 @@ class ChatGpt {
                     type: 'json_schema',
                     json_schema: {
                         name,
-                        schema
+                        schema,
+                        strict: true
                     }
                 };
             } else if (typeof responseFormat === 'string') {

package/src/GlobalIntents.js

@@ -166,20 +166,44 @@ class GlobalIntents {
                     id, matcher = null, action: intentPath, local, title,
                     entitiesSetState = {}, usedEntities, meta = {},
                     classification = null, router = null,
-                    keepUserInInteractionsWithBounceAllowed
-
+                    keepUserInInteractionsWithBounceAllowed,
+                    localPath: prevLocalPath
                 } = gi;
                 const action = intentPath === '/*'
                     ? pathContext.path
                     : `${pathContext.path}${intentPath}`.replace(/^\/\*/, '');
 
+                let localPath;
+
+                if (prevLocalPath === '/') {
+                    localPath = `${pathContext.path}`;
+                } else if (prevLocalPath) {
+                    localPath = `${pathContext.path}${prevLocalPath}`;
+                } else {
+                    localPath = action === pathContext.path
+                        ? action.replace(/\/[^/]+$/, '')
+                        : pathContext.path.replace(/\/[^/]+$/, '');
+
+                    if (!localPath) {
+                        localPath = '/';
+                    }
+                }
+
+                // console.log('merge', {
+                //     action,
+                //     pathContextPath: pathContext.path,
+                //     intentPath,
+                //     prevLocalPath,
+                //     localPath
+                // });
+
                 target.set(id, {
                     id,
                     matcher,
                     usedEntities,
                     entitiesSetState,
                     action,
-                    localPath: pathContext.path,
+                    localPath,
                     local,
                     title,
                     classification,

package/src/LLMDispatcher.js

@@ -496,8 +496,10 @@ class LLMDispatcher {
             }
         }
 
+        const llmRouting = LLMRouter.prepareRouting(globalIntents);
+
         return {
-            ...LLMRouter.prepareRouting(globalIntents),
+            ...llmRouting,
             nlpMatchers: Array.from(giMatchersByAction.values()),
             giByAction
         };

package/src/LLMRouter.js

@@ -143,7 +143,7 @@ class LLMRouter {
                 .object({
                     action: LLMType.string()
                         .description('recommended action')
-                }, 'conversation routing information')
+                }, 'conversation_routing_information')
                 .toJSON();
         }
         return LLMRouter._cachedStructuredOutput;
@@ -159,12 +159,34 @@ class LLMRouter {
     static defaultRoute (prompt) {
         /** @type {RouterResolver} */
         const route = async (llm, req, routing) => {
-            const res = await llm.session('routing')
+            const session = llm.session('routing')
                 .setData(routing)
-                .systemPrompt(prompt)
-                .generateStructured(LLMRouter.structuredOutput());
+                .systemPrompt(prompt);
 
-            if (!routing.byAction.has(LLMRouter._normByAction(res.action))) {
+            const userText = typeof req?.text === 'function' ? req.text() : null;
+            if (userText) {
+                session.user(userText);
+            }
+
+            let res;
+            try {
+                res = await session.generateStructured(LLMRouter.structuredOutput());
+            } catch (e) {
+                // Fail closed: any error in routing classification is treated
+                // as "no context change" so the user's current flow continues.
+                // Surfaces to Sentry via the host-configured logger.
+                llm.log.error('LLMRouter classification failed', e, {
+                    // userText,
+                    candidateActions: routing?.byAction
+                        ? Array.from(routing.byAction.keys())
+                        : [],
+                    senderId: req?.senderId ?? null,
+                    pageId: req?.pageId ?? null
+                });
+                return null;
+            }
+
+            if (!res || !routing.byAction.has(LLMRouter._normByAction(res.action))) {
                 return null;
             }
 
@@ -226,6 +248,10 @@ class LLMRouter {
         });
     }
 
+    /**
+     * @param {string} act
+     * @returns {string}
+     */
     static _normByAction (act) {
         return act.replace(/^\/|\/$/g, '');
     }

package/src/LLMSession.js

@@ -842,7 +842,14 @@ class LLMSession {
                 responseFormat
             }, logOptions);
 
-            return JSON.parse(result.content);
+            try {
+                return JSON.parse(result.content);
+            } catch (e) {
+                throw new Error(
+                    `LLM structured output is not valid JSON: ${e.message}. Raw content: ${JSON.stringify(result.content)}`,
+                    { cause: e }
+                );
+            }
         });
         return this;
     }

package/.claude/settings.local.json

@@ -1,10 +0,0 @@
-{
-  "permissions": {
-    "allow": [
-      "Bash(npx mocha:*)",
-      "Bash(npx tsc *)",
-      "Bash(npm test *)",
-      "Bash(node -e \"const p = require\\('./bot/plugins/BTMLLM'\\); console.log\\('factory type:', typeof p\\); console.log\\('factory name:', p.name\\);\")"
-    ]
-  }
-}