nothumanallowed 13.5.140 → 13.5.142

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "13.5.140",
+  "version": "13.5.142",
   "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

@@ -5706,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;
@@ -5718,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;
           }
@@ -5756,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.140';
+export const VERSION = '13.5.142';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/services/llm.mjs

@@ -609,7 +609,7 @@ function buildRequestBody(provider, model, systemPrompt, userMessage, stream) {
   };
   const req = {
     model: model || modelDefaults[provider] || 'gpt-4o',
-    max_tokens: provider === 'nha' ? 4096 : 8192,
+    max_tokens: opts.max_tokens || (provider === 'nha' ? 8192 : 8192),
     messages: [
       { role: 'system', content: systemPrompt },
       { role: 'user', content: userMessage },