nothumanallowed 13.5.126 → 13.5.128

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "13.5.126",
+  "version": "13.5.128",
   "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": {
@@ -57,9 +57,11 @@
     "check-bundle": "node check-bundle.mjs"
   },
   "dependencies": {
-    "better-sqlite3": "^12.9.0",
     "imapflow": "^1.3.3",
     "mailparser": "^3.9.8",
     "ws": "^8.18.0"
+  },
+  "optionalDependencies": {
+    "better-sqlite3": "^12.9.0"
   }
 }

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 = '13.5.126';
+export const VERSION = '13.5.128';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/services/email-db.mjs

@@ -387,6 +387,18 @@ export function getSystemLabel(accountId, systemType) {
   return getDb().prepare('SELECT * FROM email_labels WHERE account_id = ? AND system_type = ?').get(accountId, systemType);
 }
 
+/** Idempotent — seeds any missing system labels for a single account. */
+export function ensureLabelsForAccount(accountId) {
+  const db = getDb();
+  const insert = db.prepare(`
+    INSERT OR IGNORE INTO email_labels (id, account_id, name, system_type, is_system, color, icon, sort_order, path)
+    VALUES (?, ?, ?, ?, 1, ?, ?, ?, ?)
+  `);
+  for (const lbl of SYSTEM_LABELS) {
+    insert.run(randomUUID(), accountId, lbl.name, lbl.system_type, lbl.color, lbl.icon, lbl.sort_order, lbl.system_type);
+  }
+}
+
 export function createLabel(accountId, name, color, parentId) {
   const db = getDb();
   const id = randomUUID();
@@ -503,8 +515,16 @@ export function listMessages(accountId, labelId, limit, offset, search) {
   const params = [accountId];
 
   if (labelId) {
-    where += ' AND EXISTS (SELECT 1 FROM email_message_labels j WHERE j.message_id = m.id AND j.label_id = ?)';
-    params.push(labelId);
+    // Also include messages saved with matching imap_folder_path in case label link was missing
+    const lbl = db.prepare('SELECT system_type FROM email_labels WHERE id = ?').get(labelId);
+    const folderFallback = lbl?.system_type ? lbl.system_type.charAt(0).toUpperCase() + lbl.system_type.slice(1) : null;
+    if (folderFallback) {
+      where += ' AND (EXISTS (SELECT 1 FROM email_message_labels j WHERE j.message_id = m.id AND j.label_id = ?) OR m.imap_folder_path = ?)';
+      params.push(labelId, folderFallback);
+    } else {
+      where += ' AND EXISTS (SELECT 1 FROM email_message_labels j WHERE j.message_id = m.id AND j.label_id = ?)';
+      params.push(labelId);
+    }
   }
   if (search) {
     where += ' AND (m.subject LIKE ? OR m.from_address LIKE ? OR m.from_name LIKE ? OR m.body_preview LIKE ?)';

package/src/services/email-smtp.mjs

@@ -9,7 +9,7 @@ import { createTransport } from 'nodemailer';
 import { randomUUID } from 'crypto';
 import {
   getAccountCredentials, insertMessage, addMessageToLabel, getSystemLabel,
-  insertAttachments, getDb,
+  ensureLabelsForAccount, insertAttachments, getDb,
 } from './email-db.mjs';
 import { createHash } from 'crypto';
 
@@ -97,6 +97,8 @@ export async function sendEmail(accountId, opts) {
   // ── Save to local DB as "sent" ────────────────────────────────────────
   const now = new Date().toISOString();
   const tid = threadId(opts.inReplyTo || msgId);
+  // Ensure system labels exist for this account (first send may precede first sync)
+  ensureLabelsForAccount(accountId);
   const sentLabel = getSystemLabel(accountId, 'sent');
 
   const dbId = insertMessage({

package/src/services/tool-executor.mjs

@@ -988,15 +988,19 @@ export async function executeTool(action, params, config) {
     case 'imap_send': {
       if (!params.accountId || !params.to || !params.subject) return 'accountId, to, subject required.';
       const { sendEmail: imapSendEmail } = await import('./email-smtp.mjs');
-      const result = await imapSendEmail(params.accountId, {
-        to: params.to,
-        cc: params.cc || null,
-        subject: params.subject,
-        bodyHtml: params.bodyHtml || params.body || '',
-        bodyText: params.bodyText || null,
-        inReplyTo: params.inReplyTo || null,
-      });
-      return `Email sent successfully. Message-ID: ${result.messageId}`;
+      try {
+        const result = await imapSendEmail(params.accountId, {
+          to: params.to,
+          cc: params.cc || null,
+          subject: params.subject,
+          bodyHtml: params.bodyHtml || params.body || '',
+          bodyText: params.bodyText || null,
+          inReplyTo: params.inReplyTo || null,
+        });
+        return `✅ Email sent and saved to Sent folder. Message-ID: ${result.messageId}`;
+      } catch (e) {
+        return `❌ SEND FAILED — the email was NOT delivered. Error: ${e.message}. Tell the user the send failed and show the exact error.`;
+      }
     }
 
     case 'imap_mark_read': {
@@ -1016,15 +1020,19 @@ export async function executeTool(action, params, config) {
       let refs = [];
       try { refs = JSON.parse(orig.references_list || '[]'); } catch {}
       if (orig.message_id) refs.push(orig.message_id);
-      const result = await imapSendReply(params.accountId, {
-        to: orig.from_address,
-        cc: params.cc || null,
-        subject: replySubject,
-        bodyHtml: params.bodyHtml,
-        inReplyTo: orig.message_id || null,
-        references: refs,
-      });
-      return `Reply sent to ${orig.from_address}. Message-ID: ${result.messageId}`;
+      try {
+        const result = await imapSendReply(params.accountId, {
+          to: orig.from_address,
+          cc: params.cc || null,
+          subject: replySubject,
+          bodyHtml: params.bodyHtml,
+          inReplyTo: orig.message_id || null,
+          references: refs,
+        });
+        return `✅ Reply sent to ${orig.from_address} and saved to Sent folder. Message-ID: ${result.messageId}`;
+      } catch (e) {
+        return `❌ SEND FAILED — reply was NOT delivered. Error: ${e.message}. Tell the user the send failed and show the exact error.`;
+      }
     }
 
     case 'imap_thread': {
@@ -1089,8 +1097,12 @@ export async function executeTool(action, params, config) {
       const applyVars = (str, vars) => Object.entries(vars).reduce((s, [k, v]) => s.split('[' + k + ']').join(v || ''), str);
       const subject = applyVars(tpl.subject, params.vars);
       const html = applyVars(tpl.html, params.vars);
-      const result = await imapSendTpl(params.accountId, { to: params.to, subject, bodyHtml: html });
-      return `Template email "${params.templateId}" sent to ${params.to}. Message-ID: ${result.messageId}`;
+      try {
+        const result = await imapSendTpl(params.accountId, { to: params.to, subject, bodyHtml: html });
+        return `✅ Template email "${params.templateId}" sent to ${params.to} and saved to Sent folder. Message-ID: ${result.messageId}`;
+      } catch (e) {
+        return `❌ SEND FAILED — email was NOT delivered. Error: ${e.message}. Tell the user the send failed and show the exact error.`;
+      }
     }
 
     case 'imap_bulk_send': {

package/src/services/web-ui.mjs

@@ -1639,7 +1639,11 @@ function emailSend() {
   }).then(function(r) {
     if (r.ok) {
       if (status) { status.textContent = 'Sent!'; status.style.color = 'var(--green)'; }
-      setTimeout(emailCloseCompose, 800);
+      setTimeout(function() {
+        emailCloseCompose();
+        // Reload current label to show the sent message if we are on Sent
+        emailLoadMessages();
+      }, 800);
     } else {
       if (status) { status.textContent = r.error || 'Error'; status.style.color = 'var(--red)'; }
     }