nothumanallowed 16.0.48 → 16.0.50

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "16.0.48",
+  "version": "16.0.50",
   "description": "Local AI assistant: 80 tools (Gmail, Calendar, Drive, GitHub, Slack, browser, code, files), 38 agents, visual workflows (Studio, AWF, WebCraft). Install with `npm i -g nothumanallowed`, run with `nha ui`. Free tier built-in (Liara), no API key required. Your data stays on your PC — OAuth tokens local, no cloud. Open-source MIT.",
   "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 = '16.0.48';
+export const VERSION = '16.0.50';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/server/routes/webcraft.mjs

@@ -1286,7 +1286,9 @@ RULES:
         return 'OK — edit applied (fuzzy match)';
       }
       emit({ type: 'tool', op: 'edit', path: relPath, result: 'old_not_found' });
-      return 'Error: old_text not found in file. Use read_file to see the EXACT current content, copy the exact lines, and retry.';
+      // Include the actual file content in the error response so the LLM can
+      // produce a correct old_text on retry without needing a separate read_file.
+      return `Error: old_text not found in file.\n\nCURRENT CONTENT OF ${relPath}:\n\`\`\`\n${src.slice(0, 16000)}\n\`\`\`\n\nPick the EXACT lines you want to replace from above, copy them as old_text, and retry edit_file.`;
     }
 
     if (toolName === 'create_file') {
@@ -1562,8 +1564,18 @@ After ALL fixes are done, emit <done/> on its own line.
             toolResults.push({ op: 'edit', path: relPath, result: 'ok' });
             emit({ type: 'tool', op: 'edit', path: relPath, result: 'ok', oldSnippet: oldStr.slice(0, 2000), newSnippet: newStr?.slice(0, 2000) });
           } else {
-            // No match — return error, let the agent retry with correct text
-            toolResults.push({ op: 'edit', path: relPath, result: 'old_not_found — your old text does not match the file. Use read tool to see the EXACT current content, then copy-paste the exact lines and retry.' });
+            // No match — auto-read the file and INCLUDE its content in the
+            // feedback. Without this, the LLM at the next step is blind and
+            // can't correct the old_text. Critical fix for text-based mode
+            // where the LLM doesn't see tool results between calls.
+            const fileContentForRetry = src.slice(0, 16000);
+            toolResults.push({
+              op: 'edit',
+              path: relPath,
+              result: 'old_not_found',
+              hint: 'Your old_text did NOT match the file. The CURRENT file content is included below — copy the EXACT lines you want to replace and retry edit with that exact text as old.',
+              content: fileContentForRetry,
+            });
             emit({ type: 'tool', op: 'edit', path: relPath, result: 'old_not_found', oldSnippet: oldStr.slice(0, 200) });
           }
         }
@@ -1825,11 +1837,16 @@ After ALL fixes are done, emit <done/> on its own line.
       break;
     }
 
-    // Build tool results feedback for next iteration
+    // Build tool results feedback for next iteration.
+    // CRITICAL: when edit fails with old_not_found, include the CURRENT
+    // file content so the LLM can produce a correct old_text on retry.
+    // Without this, text-based mode loops forever on the same wrong old_text.
     const feedbackParts = toolResults.map((r) => {
       let msg = `[${r.op}] ${r.path || ''}: ${r.result}`;
-      if (r.op === 'read' && r.content) msg += `\n\`\`\`\n${r.content}\n\`\`\``;
-      if (r.hint) msg += ` — ${r.hint}`;
+      if (r.hint) msg += `\n  HINT: ${r.hint}`;
+      if (r.content) {
+        msg += `\n\nCURRENT CONTENT OF ${r.path}:\n\`\`\`\n${r.content}\n\`\`\`\n\nTo fix: pick the exact lines you want to replace from above, use them as "old", and retry the edit.`;
+      }
       return msg;
     });
 
@@ -4313,19 +4330,30 @@ OUTPUT: ONLY the complete extended CSS file content. No markdown fences, no expl
     `- Hover/focus/transition states for buttons, links, cards\n` +
     `- Color contrast must pass WCAG AA: text-on-bg >= 4.5:1`;
 
-  if (emit) emit({ type: 'status', msg: `CSS coverage ${(analysis.coverage * 100).toFixed(0)}% (${analysis.missing.length} selectors missing). Auto-extending ${targetRel} via LLM (timeout 60s)...` });
+  const provider = config?.llm?.provider || 'unknown';
+  const model = config?.llm?.model || config?.llm?.[provider]?.model || 'default';
+  if (emit) emit({ type: 'status', msg: `CSS coverage ${(analysis.coverage * 100).toFixed(0)}% (${analysis.missing.length} selectors missing). Auto-extending ${targetRel} via ${provider}:${model} (timeout 60s)...` });
 
-  // Call LLM with timeout + progress heartbeat. Otherwise a slow/stuck Liara
-  // leaves the user staring at "Auto-extending..." forever with no feedback.
+  // Call LLM with timeout + progress heartbeat + early warning when no bytes arrive.
+  // Different providers have different latencies: Claude is fast (~5-15s),
+  // OpenAI similar, Liara/Qwen3 can be 20-40s under load, Gemini sometimes
+  // stuck on long prompts. Adapt max_tokens to 4096 (down from 8192) to keep
+  // Liara responsive.
   let body = '';
   let lastChunkAt = Date.now();
   const startedAt = Date.now();
   const timeoutMs = 60_000;
+  let earlyWarningEmitted = false;
   const heartbeatInterval = setInterval(() => {
     if (!emit) return;
     const elapsed = ((Date.now() - startedAt) / 1000).toFixed(0);
     const sinceLast = ((Date.now() - lastChunkAt) / 1000).toFixed(0);
     emit({ type: 'status', msg: `LLM extend: ${elapsed}s elapsed, ${body.length} bytes received (${sinceLast}s since last chunk)` });
+    // Early warning if no bytes at all after 15s — provider is likely stuck or rate-limited
+    if (!earlyWarningEmitted && body.length === 0 && Date.now() - startedAt > 15_000) {
+      earlyWarningEmitted = true;
+      emit({ type: 'warn', msg: `Provider ${provider} hasn't sent any data in 15s. If this is Liara, the free tier may be under load — try switching to Anthropic/OpenAI in Settings, or check your API key.` });
+    }
   }, 5_000);
 
   try {
@@ -4333,13 +4361,19 @@ OUTPUT: ONLY the complete extended CSS file content. No markdown fences, no expl
       callLLMStream(config, sys, user, (chunk) => {
         body += chunk;
         lastChunkAt = Date.now();
-      }, { max_tokens: 8192 }),
-      new Promise((_, reject) => setTimeout(() => reject(new Error('LLM timeout after ' + (timeoutMs / 1000) + 's')), timeoutMs)),
+      }, { max_tokens: 4096 }),
+      new Promise((_, reject) => setTimeout(() => reject(new Error('LLM timeout after ' + (timeoutMs / 1000) + 's — provider ' + provider + ' did not respond')), timeoutMs)),
     ]);
   } catch (e) {
     clearInterval(heartbeatInterval);
-    if (emit) emit({ type: 'warn', msg: `CSS extend failed: ${(e.message || e).slice(0, 200)}. Sandbox continues with current CSS — open chat to extend manually.` });
-    return { extended: false, reason: 'llm_failed', error: e.message, partialBytes: body.length };
+    const errMsg = (e.message || String(e)).slice(0, 200);
+    if (emit) {
+      emit({ type: 'warn', msg: `CSS extend failed: ${errMsg}` });
+      if (errMsg.includes('timeout') || body.length === 0) {
+        emit({ type: 'warn', msg: `Provider ${provider} unresponsive. Open NHA Settings and switch to a different provider (Anthropic Claude is fastest), or check that your API key is valid. Sandbox continues with current CSS.` });
+      }
+    }
+    return { extended: false, reason: 'llm_failed', error: errMsg, partialBytes: body.length };
   }
   clearInterval(heartbeatInterval);
 
@@ -5021,6 +5055,7 @@ export const _SHIMMED_MODULES = new Set([
   'dotenv', 'cors', 'morgan', 'body-parser', 'cookie-parser',
   'compression', 'express-rate-limit', 'jsonwebtoken', 'bcryptjs', 'bcrypt',
   'uuid', 'lodash', 'debug', 'chalk', 'multer', 'axios', 'express',
+  'marked', 'markdown-it',
 ]);
 
 /** Read declared dependencies from package.json (deps + devDeps + peer). */
@@ -5148,9 +5183,16 @@ export function _classifyInstallError(err) {
     return { reason: 'offline (npm registry unreachable)', offlineFallback: true,
       hint: 'Check VM network bridge/NAT, DNS, or corporate proxy (HTTP_PROXY/HTTPS_PROXY). Activating shim fallback.' };
   }
-  if (/e404|notarget|not found in the npm registry/.test(msg)) {
+  // Distinguish "true E404" (package doesn't exist) from "offline cache miss"
+  // (--prefer-offline can produce 404-looking errors when registry is unreachable).
+  // True E404 also mentions "not in registry"; cache miss mentions "not in cache" or "ENETUNREACH".
+  if (/not in.*(cache|local)|ENETUNREACH|prefer.?offline.*not.*found/i.test(msg)) {
+    return { reason: 'offline cache miss (registry not reached, package not cached)', offlineFallback: true,
+      hint: 'npm --prefer-offline could not find the package in local cache and registry is unreachable. Activating shim fallback.' };
+  }
+  if (/e404|"npm error code e404"|notarget|404 not found|not found in the npm registry|package.*does not exist/i.test(msg)) {
     return { reason: 'package does not exist on npm', offlineFallback: false,
-      hint: 'The package name from the LLM-generated code is likely a hallucination. Tier 2 LLM-rewrite will rename it.' };
+      hint: 'The package name from the LLM-generated code is likely a hallucination, or you are offline. Tier 2 LLM-rewrite will rename it, or shim layer will take over if available.' };
   }
   if (/eacces|eperm|permission denied/.test(msg)) {
     return { reason: 'permissions denied', offlineFallback: false,
@@ -5826,6 +5868,74 @@ express.static = function (root, opts) {
 express.Router = function () { const r = createApp(); return r; };
 module.exports = express;
 module.exports.default = express;
+`;
+
+  // marked — minimal Markdown → HTML parser. Real `marked` is 200KB+ but we
+  // only need basic parsing. Covers: headings, bold/italic, links, code blocks,
+  // lists, paragraphs, line breaks. Good enough for blog-style content.
+  const markedShim = `
+function escapeHtml(s) {
+  return String(s).replace(/&/g, '&amp;').replace(/</g, '&lt;').replace(/>/g, '&gt;').replace(/"/g, '&quot;');
+}
+function parseMarkdown(md) {
+  if (!md) return '';
+  let html = String(md);
+  // Code blocks (must come first)
+  html = html.replace(/\\\`\\\`\\\`(\\\\w+)?\\n([\\s\\S]*?)\\n\\\`\\\`\\\`/g, (_, lang, code) => '<pre><code' + (lang ? ' class="language-' + lang + '"' : '') + '>' + escapeHtml(code) + '</code></pre>');
+  // Inline code
+  html = html.replace(/\\\`([^\\\`\\n]+)\\\`/g, '<code>$1</code>');
+  // Headings
+  html = html.replace(/^######\\s+(.+)$/gm, '<h6>$1</h6>');
+  html = html.replace(/^#####\\s+(.+)$/gm, '<h5>$1</h5>');
+  html = html.replace(/^####\\s+(.+)$/gm, '<h4>$1</h4>');
+  html = html.replace(/^###\\s+(.+)$/gm, '<h3>$1</h3>');
+  html = html.replace(/^##\\s+(.+)$/gm, '<h2>$1</h2>');
+  html = html.replace(/^#\\s+(.+)$/gm, '<h1>$1</h1>');
+  // Bold + italic
+  html = html.replace(/\\*\\*\\*([^*]+)\\*\\*\\*/g, '<strong><em>$1</em></strong>');
+  html = html.replace(/\\*\\*([^*]+)\\*\\*/g, '<strong>$1</strong>');
+  html = html.replace(/\\*([^*]+)\\*/g, '<em>$1</em>');
+  html = html.replace(/__([^_]+)__/g, '<strong>$1</strong>');
+  html = html.replace(/_([^_]+)_/g, '<em>$1</em>');
+  // Links + images
+  html = html.replace(/!\\[([^\\]]*)\\]\\(([^)]+)\\)/g, '<img src="$2" alt="$1">');
+  html = html.replace(/\\[([^\\]]+)\\]\\(([^)]+)\\)/g, '<a href="$2">$1</a>');
+  // Blockquotes
+  html = html.replace(/^>\\s+(.+)$/gm, '<blockquote>$1</blockquote>');
+  // Horizontal rules
+  html = html.replace(/^---+$/gm, '<hr>');
+  // Lists (simple)
+  html = html.replace(/^[\\*\\-]\\s+(.+)$/gm, '<li>$1</li>');
+  html = html.replace(/(<li>[\\s\\S]*?<\\/li>\\n?)+/g, (m) => '<ul>' + m.replace(/\\n/g, '') + '</ul>');
+  html = html.replace(/^\\d+\\.\\s+(.+)$/gm, '<li>$1</li>');
+  // Paragraphs (lines not in other elements)
+  const lines = html.split(/\\n\\n+/);
+  html = lines.map(line => {
+    line = line.trim();
+    if (!line) return '';
+    if (/^<(h[1-6]|ul|ol|pre|blockquote|hr|p|div)/.test(line)) return line;
+    return '<p>' + line + '</p>';
+  }).join('\\n');
+  return html;
+}
+function marked(md, opts) {
+  return parseMarkdown(md);
+}
+marked.parse = parseMarkdown;
+marked.setOptions = function () { return marked; };
+marked.use = function () { return marked; };
+marked.Renderer = function () {};
+marked.parseInline = function (md) {
+  let html = String(md || '');
+  html = html.replace(/\\*\\*([^*]+)\\*\\*/g, '<strong>$1</strong>');
+  html = html.replace(/\\*([^*]+)\\*/g, '<em>$1</em>');
+  html = html.replace(/\\\`([^\\\`]+)\\\`/g, '<code>$1</code>');
+  html = html.replace(/\\[([^\\]]+)\\]\\(([^)]+)\\)/g, '<a href="$2">$1</a>');
+  return html;
+};
+module.exports = marked;
+module.exports.marked = marked;
+module.exports.default = marked;
 `;
 
   // axios — minimal fetch-based replacement
@@ -5903,6 +6013,8 @@ module.exports.default = proxy;
     'multer.js': multerShim,
     'axios.js': axiosShim,
     'express.js': expressShim,
+    'marked.js': markedShim,
+    'markdown-it.js': markedShim,
     'noop.js': noopShim,
   };
   for (const [name, content] of Object.entries(shimFiles)) {