nothumanallowed 14.2.4 → 14.2.6

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nothumanallowed",
-  "version": "14.2.4",
+  "version": "14.2.6",
   "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 = '14.2.4';
+export const VERSION = '14.2.6';
 export const BASE_URL = 'https://nothumanallowed.com/cli';
 export const API_BASE = 'https://nothumanallowed.com/api/v1';
 

package/src/server/routes/config.mjs

@@ -106,7 +106,7 @@ export function register(router) {
       // Restart daemon (Telegram/Discord) + self-restart UI server
       setTimeout(async () => {
         try {
-          const { spawn, execSync } = await import('child_process');
+          const { execSync } = await import('child_process');
 
           // Restart ops daemon if running (so Telegram/Discord reconnect with new code)
           try {
@@ -117,22 +117,20 @@ export function register(router) {
             }
           } catch { /* daemon restart is best-effort */ }
 
-          // Find the nha binary in PATH (points to the newly installed version)
-          let nhaBin = 'nha';
-          try {
-            nhaBin = execSync('which nha', { encoding: 'utf-8', timeout: 3000 }).trim() || 'nha';
-          } catch { /* fallback to bare 'nha' */ }
-
-          const child = spawn(nhaBin, ['ui'], {
+          // Self-restart: launch a shell script that waits for us to die, then starts nha ui.
+          // This avoids the port-in-use race condition.
+          // Detect current port from the server's address
+          let port = '3847';
+          try { port = String(req.socket?.localPort || 3847); } catch {}
+          const { spawn } = await import('child_process');
+          const restartScript = `sleep 2 && nha ui --port ${port}`;
+          const child = spawn('sh', ['-c', restartScript], {
             detached: true,
             stdio: 'ignore',
             env: { ...process.env, NHA_RESTARTED: '1' },
-            shell: true,
           });
           child.unref();
-          // Give the child process time to bind the port before we die
-          await new Promise((r) => setTimeout(r, 2000));
-        } catch { /* ignore spawn errors */ }
+        } catch { /* ignore errors */ }
         process.exit(0);
       }, 1500);
     } catch (e) {

package/src/server/routes/webcraft.mjs

@@ -848,20 +848,17 @@ function isFileTruncated(content, filename) {
   const trimmed = content.trimEnd();
   const ext = filename.split('.').pop()?.toLowerCase();
 
-  // Common truncation signs: ends mid-statement without closing bracket/brace
   if (ext === 'js' || ext === 'mjs' || ext === 'ts') {
-    // Balanced braces check (fast approximation)
     const open = (trimmed.match(/\{/g) || []).length;
     const close = (trimmed.match(/\}/g) || []).length;
-    if (open > close + 2) return true;
-    // Last meaningful line should not end with an operator or comma
+    if (open > close) return true; // ANY unbalanced brace = truncated
     const lastLine = trimmed.split('\n').pop()?.trim() ?? '';
     if (/[,({=+\-*/<>|&]$/.test(lastLine)) return true;
   }
   if (ext === 'css') {
     const open = (trimmed.match(/\{/g) || []).length;
     const close = (trimmed.match(/\}/g) || []).length;
-    if (open > close + 1) return true;
+    if (open > close) return true;
   }
   if (ext === 'html') {
     if (!trimmed.includes('</html>') && !trimmed.includes('</body>')) return true;
@@ -872,6 +869,69 @@ function isFileTruncated(content, filename) {
   return false;
 }
 
+/**
+ * Deterministic repair of common truncation artifacts.
+ * Adds missing closing braces, tags, etc. WITHOUT calling the LLM.
+ */
+function repairTruncation(content, filename) {
+  if (!content) return content;
+  const ext = filename.split('.').pop()?.toLowerCase();
+  let result = content.trimEnd();
+
+  if (ext === 'js' || ext === 'mjs' || ext === 'ts') {
+    // Balance curly braces
+    const open = (result.match(/\{/g) || []).length;
+    const close = (result.match(/\}/g) || []).length;
+    const missing = open - close;
+    if (missing > 0 && missing <= 10) {
+      result += '\n' + '}\n'.repeat(missing);
+    }
+    // Balance parentheses
+    const openP = (result.match(/\(/g) || []).length;
+    const closeP = (result.match(/\)/g) || []).length;
+    const missingP = openP - closeP;
+    if (missingP > 0 && missingP <= 5) {
+      // Find last line and append
+      const lines = result.split('\n');
+      const lastIdx = lines.length - 1;
+      lines[lastIdx] = lines[lastIdx] + ')'.repeat(missingP) + ';';
+      result = lines.join('\n');
+    }
+  }
+
+  if (ext === 'css') {
+    const open = (result.match(/\{/g) || []).length;
+    const close = (result.match(/\}/g) || []).length;
+    const missing = open - close;
+    if (missing > 0 && missing <= 10) {
+      result += '\n' + '}\n'.repeat(missing);
+    }
+  }
+
+  if (ext === 'html' || ext === 'htm') {
+    if (!result.includes('</body>')) result += '\n</body>';
+    if (!result.includes('</html>')) result += '\n</html>';
+  }
+
+  if (ext === 'json') {
+    // Try to fix unclosed JSON
+    try { JSON.parse(result); } catch {
+      // Count brackets
+      const openB = (result.match(/\[/g) || []).length;
+      const closeB = (result.match(/\]/g) || []).length;
+      const openC = (result.match(/\{/g) || []).length;
+      const closeC = (result.match(/\}/g) || []).length;
+      // Remove trailing comma if present
+      result = result.replace(/,\s*$/, '');
+      // Add missing closers
+      if (openB > closeB) result += '\n' + ']'.repeat(openB - closeB);
+      if (openC > closeC) result += '\n' + '}'.repeat(openC - closeC);
+    }
+  }
+
+  return result;
+}
+
 async function runGenerate(config, projectName, description, blocks, authFields, emit) {
   const blocksDesc = Object.entries(blocks)
     .filter(([, enabled]) => enabled)
@@ -1058,7 +1118,8 @@ Continue from here:`;
           }
         }
 
-        fileContent = rawOutput;
+        // Deterministic repair of truncation artifacts (missing braces, tags)
+        fileContent = repairTruncation(rawOutput, fileSpec.name);
         syntaxError = null;
 
         const fileTokensOut = countTokens(fileContent);
@@ -1109,10 +1170,27 @@ Continue from here:`;
     return isFileTruncated(f.content, f.name);
   });
 
-  if (brokenFiles.length > 0 && brokenFiles.length <= 10) {
+  if (brokenFiles.length > 0) {
     emit({ type: 'phase', phase: 'autofix', msg: `Post-generation fix: ${brokenFiles.length} file(s) need repair...` });
     for (const broken of brokenFiles) {
       try {
+        // Step 1: Try deterministic repair first (fast, no LLM call)
+        const deterministicFix = repairTruncation(broken.content, broken.name);
+        let isFixed = false;
+        if (deterministicFix !== broken.content) {
+          // Verify the fix actually resolved the issue
+          if (!isFileTruncated(deterministicFix, broken.name)) {
+            emit({ type: 'status', msg: `Auto-fixed ${broken.name} (deterministic repair)` });
+            broken.content = deterministicFix;
+            const abs = path.join(projectDir, broken.name);
+            fs.writeFileSync(abs, deterministicFix, 'utf-8');
+            isFixed = true;
+          }
+        }
+        if (isFixed) continue;
+
+        // Step 2: If deterministic repair wasn't enough, regenerate with LLM
+        if (brokenFiles.length > 15) continue; // don't LLM-regenerate too many files
         emit({ type: 'status', msg: `Regenerating ${broken.name}...` });
         let fixedContent = '';
         const fixPrompt = `Regenerate this file COMPLETELY. It was truncated or has errors.\n\nFile: ${broken.name}\nProject: ${projectName}\nDescription: ${description}\nFull file list: ${allFileNames}\n\nOutput the COMPLETE file content only, no explanation.`;
@@ -1121,7 +1199,8 @@ Continue from here:`;
         }, { max_tokens: 16384 });
         fixedContent = fixedContent
           .replace(/^```[\w]*\n/, '').replace(/\n```$/, '').trim();
-        if (fixedContent.length > broken.content.length) {
+        fixedContent = repairTruncation(fixedContent, broken.name); // repair the regenerated content too
+        if (fixedContent.length > 50) {
           broken.content = fixedContent;
           const abs = path.join(projectDir, broken.name);
           fs.writeFileSync(abs, fixedContent, 'utf-8');