nothumanallowed 15.1.50 → 15.1.51

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "15.1.50",
+  "version": "15.1.51",
   "description": "NotHumanAllowed — 38 AI agents, 80 tools, Studio (visual agentic workflows). Email, calendar, browser automation, screen capture, canvas, cron/heartbeat, Alexandria E2E messaging, GitHub, Notion, Slack, voice chat, free AI (Liara), 28 languages. Zero-dependency CLI.",
   "type": "module",
   "bin": {

package/src/constants.mjs

@@ -5,7 +5,7 @@ import { fileURLToPath } from 'url';
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 
-export const VERSION = '15.1.50';
+export const VERSION = '15.1.51';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/server/routes/chat.mjs

@@ -425,6 +425,69 @@ export function register(router) {
         }
       }
 
+      // ── Auto-chain: search → read → attachment_read ──────────────────────
+      // When the user wants to read an attachment of a specific email, the
+      // model often emits the first search but then gets stuck saying "Let
+      // me read the full email" without actually emitting the follow-up
+      // tool. We chain them server-side: parse the imap_search results,
+      // pick the best subject match, fetch it with imap_read, then read
+      // the first attachment if wantsReadAttachment is on.
+      if (wantsReadAttachment) {
+        try {
+          // Collect candidate messages from all imap_search results we already
+          // executed in this turn. Format of imap_search output:
+          //   [id_xxx] [UNREAD] From: name | Subject | date\n  preview...
+          const candidates = new Map(); // id → { id, subject }
+          for (const tr of toolResults) {
+            if (tr.action !== 'imap_search' || typeof tr.result !== 'string') continue;
+            const lineRe = /\[([a-zA-Z0-9_-]{6,})\][^\n]*?\|\s*([^|]{2,200})\s*\|/g;
+            let m;
+            while ((m = lineRe.exec(tr.result)) !== null) {
+              if (!candidates.has(m[1])) candidates.set(m[1], { id: m[1], subject: m[2].trim() });
+            }
+          }
+          if (candidates.size > 0) {
+            // Rank by token-overlap with the identifiers we extracted from
+            // the user message.
+            const norm = (s) => String(s || '').toLowerCase()
+              .normalize('NFD').replace(/[̀-ͯ]/g, '')
+              .replace(/[^a-z0-9\s./_-]/g, ' ')
+              .split(/\s+/).filter(t => t.length > 2);
+            const userTokens = norm(msg);
+            const ranked = [...candidates.values()].map(c => {
+              const subjTokens = new Set(norm(c.subject));
+              const score = userTokens.filter(t => subjTokens.has(t)).length;
+              return { ...c, score };
+            }).sort((a, b) => b.score - a.score);
+            const best = ranked[0];
+            if (best && best.score >= 1) {
+              sse('tool', { action: 'imap_read', status: 'executing' });
+              try {
+                const readResult = await executeTool('imap_read', { messageId: best.id }, config);
+                toolResults.push({ action: 'imap_read', result: readResult });
+                sse('tool', { action: 'imap_read', status: 'done', result: String(readResult).slice(0, 500) });
+                // If the read result lists at least one attachment, fetch it.
+                const attMatch = String(readResult).match(/ATTACHMENTS \(\d+\)/);
+                if (attMatch) {
+                  sse('tool', { action: 'imap_attachment_read', status: 'executing' });
+                  try {
+                    const attResult = await executeTool('imap_attachment_read', { messageId: best.id, index: 1 }, config);
+                    toolResults.push({ action: 'imap_attachment_read', result: attResult });
+                    sse('tool', { action: 'imap_attachment_read', status: 'done', result: String(attResult).slice(0, 500) });
+                  } catch (e) {
+                    toolResults.push({ action: 'imap_attachment_read', result: `Error: ${e.message}` });
+                    sse('tool', { action: 'imap_attachment_read', status: 'error', error: e.message });
+                  }
+                }
+              } catch (e) {
+                toolResults.push({ action: 'imap_read', result: `Error: ${e.message}` });
+                sse('tool', { action: 'imap_read', status: 'error', error: e.message });
+              }
+            }
+          }
+        } catch { /* chain best-effort — fall through to synthesis */ }
+      }
+
       // Capture the pre-synthesis prose (what the model said in Round 1).
       // We will combine this with the synthesis output so the user keeps
       // BOTH — currently the UI was overwriting Round 1 with Round 2 alone,
@@ -458,8 +521,8 @@ export function register(router) {
           }
         };
         const toolContext = toolResults.map(t => `[${t.action} result]:\n${cleanResult(t.action, t.result)}`).join('\n\n---\n\n');
-        const synthesisPrompt = `${enrichedPrompt}\n\n## DATA FROM TOOLS:\n${toolContext}\n\n## STRICT OUTPUT RULES:\n- Write ONLY plain prose or markdown (headers, bullets, bold)\n- NEVER use \`\`\`json, \`\`\`data, or any fenced code block containing data\n- NEVER output raw JSON, arrays, or objects\n- Format numbers/prices as plain text (e.g. "Bitcoin: $103,000")\n- Be concise and human-readable`;
-        const synthesisMsg = `${effectiveMsg}\n\nAnswer using ONLY the data above. Plain text/markdown only — zero JSON, zero code blocks.`;
+        const synthesisPrompt = `${enrichedPrompt}\n\n## DATA FROM TOOLS (already executed — do NOT plan to call them again):\n${toolContext}\n\n## STRICT OUTPUT RULES:\n- The tools above HAVE ALREADY RUN. Their output is the ground truth.\n- NEVER say "Let me read…", "I'll search…", "I will fetch…", "leggerò", "cercherò", "ti dirò" — those tools are already done.\n- Answer the user's question DIRECTLY using the data above. If a specific detail (delivery date, item, total, etc.) is in the data, quote it verbatim.\n- If the data does not contain the answer, state plainly what is missing — do not announce intent to look further.\n- Write ONLY plain prose or markdown (headers, bullets, bold). NEVER use \`\`\`json or any fenced code block.\n- Format numbers/prices as plain text. Be concise and human-readable.`;
+        const synthesisMsg = `${effectiveMsg}\n\nThe tools have already been executed and their results are in the system prompt. Answer the question DIRECTLY using that data. Do NOT announce further actions ("Let me…", "I'll…", "leggerò", "cercherò"). Plain prose/markdown — zero JSON, zero code blocks.`;
         sse('tool_synthesis', {});
         // Keep the pre-synthesis prose around. If the synthesis call returns
         // empty (provider error, content filter, model bailed), we fall back