nothumanallowed 13.5.139 → 13.5.141

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "13.5.139",
+  "version": "13.5.141",
   "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/commands/ui.mjs

@@ -5145,6 +5145,7 @@ module.exports.notFoundHandler = notFoundHandler;
           // Always overwrite — older shim versions may be missing exports like notFoundHandler
           fs.writeFileSync(path.join(sandboxDir, 'server', 'middleware', 'errors.js'), errorsShim, 'utf8');
 
+
           // Models shim — LLM often generates require('../models/User') etc. that don't exist
           // Create a generic User model shim backed by the in-memory DB shim
           const userModelShim = `
@@ -5705,11 +5706,23 @@ module.exports = { validateEmail, sanitizeText, validatePassword, validateUserna
             return abs.startsWith(sandboxDir) ? abs.slice(sandboxDir.length + 1) : abs.split('/').slice(-3).join('/');
           }));
 
+          // For autofix: also detect "Cannot find module './redis'" style errors and extract the requiring file
+          // Stack trace line: "at Object.<anonymous> (/path/server/middleware/security.js:3:X)"
+          // We want to guarantee the file that contains the bad require() is always in context.
+          const moduleNotFoundMatch = autofix ? message.match(/Cannot find module '([^']+)'[\s\S]*?at Object[^(]*\(([^)]+\.js):\d+:\d+\)/) : null;
+          if (moduleNotFoundMatch) {
+            const requirerAbs = moduleNotFoundMatch[2];
+            const requirerRel = requirerAbs.startsWith(sandboxDir) ? requirerAbs.slice(sandboxDir.length + 1) : requirerAbs.split('/').slice(-3).join('/');
+            stackFiles.add(requirerRel);
+          }
+
+          const isStackFile = (f) => stackFiles.size > 0 && [...stackFiles].some(sf => f.name.includes(sf) || sf.includes(f.name));
+
           const relevantFiles = allFiles.filter(f => {
             const nameLower = f.name.toLowerCase();
             if (alwaysInclude.some(a => nameLower.endsWith(a))) return true;
-            // Always include files mentioned in stack trace (autofix)
-            if (stackFiles.size > 0 && [...stackFiles].some(sf => f.name.includes(sf) || sf.includes(f.name))) return true;
+            // Always include files mentioned in stack trace (autofix) — highest priority
+            if (isStackFile(f)) return true;
             // Include if file name or path fragment mentioned in message
             const parts = f.name.split('/');
             if (parts.some(p => msgLower.includes(p.replace(/\.[^.]+$/, '').toLowerCase()))) return true;
@@ -5717,11 +5730,19 @@ module.exports = { validateEmail, sanitizeText, validatePassword, validateUserna
             if (/errore|error|fix|crash|module|require|import/i.test(message) && f.name.startsWith('server/') && f.name.endsWith('.js')) return true;
             return false;
           });
+          // Sort: stack trace files first (guaranteed to be in context regardless of budget)
+          relevantFiles.sort((a, b) => {
+            const aStack = isStackFile(a) ? 0 : 1;
+            const bStack = isStackFile(b) ? 0 : 1;
+            return aStack - bStack;
+          });
           // Cap total context at ~24KB to stay within 7B model limits
+          // Stack trace files are always included even if over budget
           let contextBudget = 24 * 1024;
           const selectedFiles = [];
           for (const f of relevantFiles) {
-            if (contextBudget <= 0) break;
+            const isRequired = isStackFile(f) || alwaysInclude.some(a => f.name.toLowerCase().endsWith(a));
+            if (contextBudget <= 0 && !isRequired) break;
             selectedFiles.push(f);
             contextBudget -= f.content.length;
           }
@@ -5755,7 +5776,7 @@ Per leggere un file (se hai bisogno di piu contesto):
 
 REGOLE CRITICHE:
 - Spiega SEMPRE in linguaggio naturale cosa stai facendo PRIMA dei blocchi tool
-- ${autofix ? 'MODALITA AUTO-FIX: usa "write" per riscrivere i file problematici nella loro interezza — è più affidabile di "edit" quando il file ha già subito modifiche. Includi il contenuto COMPLETO del file nel campo "content".' : 'Usa "edit" (old/new) quando possibile, "write" solo per file nuovi o riscritture complete'}
+- ${autofix ? 'MODALITA AUTO-FIX: il file incriminato è già incluso qui sotto. Riscrivilo COMPLETAMENTE con "write" — è più affidabile di "edit" quando il file ha già subito modifiche. REGOLE CRITICHE PER IL FIX: (1) SyntaxError "Unexpected token \';\'" in helmet/CSP = stai usando punto-e-virgola invece di virgola negli oggetti JS — usa VIRGOLE. (2) "Cannot find module \'./redis\'" = rimuovi il require e usa solo i moduli in package.json. (3) NON aggiungere require() di moduli non in package.json. Includi il contenuto COMPLETO del file nel campo "content".' : 'Usa "edit" (old/new) quando possibile, "write" solo per file nuovi o riscritture complete'}
 - old_string deve essere ESATTO come appare nel file (copy-paste)
 - Non inventare moduli npm: usa solo quelli in package.json o standard Node.js
 - Dopo ogni fix spiega brevemente cosa hai cambiato e perche

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

package/src/services/web-ui.mjs

@@ -8545,7 +8545,7 @@ function wcDiffPanelHtml() {
   var items = _wcDiffQueue.map(function(d, di) {
     var addedLines = (d.after||'').split(String.fromCharCode(10)).length - (d.before||'').split(String.fromCharCode(10)).length;
     var sign = addedLines >= 0 ? '+' : '';
-    return '<details style="border:1px solid var(--border);border-radius:6px;margin-bottom:6px;background:var(--bg3)">' +
+    return '<details open style="border:1px solid var(--border);border-radius:6px;margin-bottom:6px;background:var(--bg3)">' +
       '<summary style="padding:7px 10px;cursor:pointer;font-size:11px;font-family:var(--mono);color:var(--text);list-style:none;display:flex;align-items:center;gap:8px">' +
         '<span style="color:var(--green);font-size:10px">&#9650;</span>' +
         '<span style="flex:1">' + wcEsc(d.file) + '</span>' +