@hasna/terminal 1.5.0 → 1.6.1

package/dist/ai.js

@@ -119,10 +119,19 @@ function detectProjectContext() {
         // Top-level dirs
         const topLevel = execSync("ls -1", { cwd, encoding: "utf8", timeout: 2000 }).trim();
         parts.push(`Top-level: ${topLevel.split("\n").join(", ")}`);
+        // Detect monorepo (packages/ or workspaces in package.json)
+        const isMonorepo = existsSync(join(cwd, "packages")) || existsSync(join(cwd, "apps"));
+        if (isMonorepo) {
+            const pkgDirs = execSync(`ls -d packages/*/src 2>/dev/null || ls -d apps/*/src 2>/dev/null || echo ""`, { cwd, encoding: "utf8", timeout: 2000 }).trim();
+            if (pkgDirs) {
+                parts.push(`MONOREPO: Source is in packages/*/src/, NOT src/. Search packages/ not src/.`);
+                parts.push(`Package sources:\n${pkgDirs}`);
+            }
+        }
         // src/ structure — include FILES so AI knows exact filenames + extensions
-        for (const srcDir of ["src", "lib", "app"]) {
+        for (const srcDir of isMonorepo ? ["packages"] : ["src", "lib", "app"]) {
             if (existsSync(join(cwd, srcDir))) {
-                const tree = execSync(`find ${srcDir} -maxdepth 3 -not -path '*/node_modules/*' -not -path '*/dist/*' -not -name '*.test.*' -not -name '*.spec.*' 2>/dev/null | sort | head -60`, { cwd, encoding: "utf8", timeout: 2000 }).trim();
+                const tree = execSync(`find ${srcDir} -maxdepth ${isMonorepo ? 4 : 3} -not -path '*/node_modules/*' -not -path '*/dist/*' -not -name '*.test.*' -not -name '*.spec.*' 2>/dev/null | sort | head -80`, { cwd, encoding: "utf8", timeout: 3000 }).trim();
                 if (tree)
                     parts.push(`Files in ${srcDir}/:\n${tree}`);
                 break;
@@ -203,6 +212,8 @@ SEMANTIC MAPPING: When the user references a concept, search the file tree for R
 - When uncertain: grep -rn "keyword" src/ --include="*.ts" -l (list matching files)
 
 ACTION vs CONCEPTUAL: If the prompt starts with "run", "execute", "check", "test", "build", "show output of" — ALWAYS generate an executable command. NEVER read README for action requests. Only read docs for "explain why", "what does X mean", "how was X designed".
+
+MONOREPO: If the project context says "MONOREPO", search packages/ or apps/ NOT src/. Use: grep -rn "pattern" packages/ --include="*.ts". For specific packages, use packages/PKGNAME/src/.
 cwd: ${process.cwd()}
 shell: zsh / macOS${projectContext}${restrictionBlock}${contextBlock}`;
 }

package/dist/cli.js

@@ -467,9 +467,27 @@ else if (args.length > 0) {
             process.exit(1);
         }
     }
+    // Step 2: Validate command before executing
+    const { validateCommand } = await import("./command-validator.js");
+    const validation = validateCommand(command, process.cwd());
+    if (!validation.valid) {
+        // Auto-retry: re-translate with simpler constraints
+        console.error(`[open-terminal] invalid command detected: ${validation.issues.join(", ")}`);
+        try {
+            const retryCommand = await translateToCommand(`${prompt} (IMPORTANT: keep it simple. Use basic grep/find/cat/ls/wc commands. No complex awk/sed pipelines. No GNU flags. Verify file paths from the project context.)`, perms, []);
+            if (retryCommand && retryCommand !== command) {
+                const retryValidation = validateCommand(retryCommand, process.cwd());
+                if (retryValidation.valid || retryValidation.issues.length < validation.issues.length) {
+                    command = retryCommand;
+                    console.error(`[open-terminal] retried: $ ${command}`);
+                }
+            }
+        }
+        catch { }
+    }
     // Show what we're running
     console.error(`$ ${command}`);
-    // Step 2: Rewrite for optimization
+    // Step 3: Rewrite for optimization
     const rw = rewriteCommand(command);
     const actualCmd = rw.changed ? rw.rewritten : command;
     if (rw.changed)
@@ -529,6 +547,23 @@ else if (args.length > 0) {
             console.log(`No results found for: ${prompt}`);
             process.exit(0);
         }
+        // Auto-retry: if command failed (exit 2+), ask AI for a simpler alternative
+        if (e.status >= 2 && !actualCmd.includes("(retry)")) {
+            try {
+                const retryCmd = await translateToCommand(`${prompt} (The previous command failed with: ${errStderr.slice(0, 200)}. Try a SIMPLER approach. Use basic commands only.)`, perms, []);
+                if (retryCmd && !isIrreversible(retryCmd) && !checkPermissions(retryCmd, perms)) {
+                    console.error(`[open-terminal] retrying: $ ${retryCmd}`);
+                    const retryResult = execSync(retryCmd + " #(retry)", { encoding: "utf8", maxBuffer: 10 * 1024 * 1024, cwd: process.cwd() });
+                    const retryClean = stripNoise(stripAnsi(retryResult)).cleaned;
+                    if (retryClean.length > 5) {
+                        const processed = await processOutput(retryCmd, retryClean, prompt);
+                        console.log(processed.aiProcessed ? processed.summary : retryClean);
+                        process.exit(0);
+                    }
+                }
+            }
+            catch { /* retry also failed, fall through */ }
+        }
         // Combine stdout+stderr and try AI answer framing (for audit/lint/test commands)
         const combined = errStderr && errStdout.includes(errStderr.trim()) ? errStdout : errStdout + errStderr;
         const errorClean = stripNoise(stripAnsi(combined)).cleaned;

package/dist/command-validator.js

@@ -0,0 +1,77 @@
+// Command validator — catch invalid commands BEFORE executing
+// Prevents shell errors from hallucinated flags, wrong paths, bad syntax
+import { existsSync } from "fs";
+import { join } from "path";
+/** Extract file paths referenced in a command */
+function extractPaths(command) {
+    const paths = [];
+    // Match quoted paths
+    const quoted = command.match(/["']([^"']+\.\w+)["']/g);
+    if (quoted)
+        paths.push(...quoted.map(q => q.replace(/["']/g, "")));
+    // Match unquoted paths with extensions or directory separators
+    const tokens = command.split(/\s+/);
+    for (const t of tokens) {
+        if (t.includes("/") && !t.startsWith("-") && !t.startsWith("|") && !t.startsWith("&")) {
+            // Clean shell operators from end
+            const clean = t.replace(/[;|&>]+$/, "");
+            if (clean && !clean.startsWith("-"))
+                paths.push(clean);
+        }
+    }
+    return [...new Set(paths)];
+}
+/** Check for obviously broken shell syntax */
+function checkSyntax(command) {
+    const issues = [];
+    // Unmatched quotes
+    const singleQuotes = (command.match(/'/g) || []).length;
+    const doubleQuotes = (command.match(/"/g) || []).length;
+    if (singleQuotes % 2 !== 0)
+        issues.push("unmatched single quote");
+    if (doubleQuotes % 2 !== 0)
+        issues.push("unmatched double quote");
+    // Unmatched parentheses
+    const openParens = (command.match(/\(/g) || []).length;
+    const closeParens = (command.match(/\)/g) || []).length;
+    if (openParens !== closeParens)
+        issues.push("unmatched parentheses");
+    // Empty pipe targets
+    if (/\|\s*$/.test(command))
+        issues.push("pipe with no target");
+    if (/^\s*\|/.test(command))
+        issues.push("pipe with no source");
+    return issues;
+}
+/** Validate a command before execution */
+export function validateCommand(command, cwd) {
+    const issues = [];
+    // Check syntax
+    issues.push(...checkSyntax(command));
+    // Check file paths exist
+    const paths = extractPaths(command);
+    for (const p of paths) {
+        const fullPath = p.startsWith("/") ? p : join(cwd, p);
+        if (p.includes("*") || p.includes("?"))
+            continue; // skip globs
+        if (p.startsWith("-"))
+            continue; // skip flags
+        if ([".", "..", "/", "~"].includes(p))
+            continue; // skip special
+        if (!existsSync(fullPath) && !existsSync(p)) {
+            // Only flag source file paths, not output paths
+            if (/\.(ts|tsx|js|jsx|json|md|yaml|yml|py|go|rs)$/.test(p)) {
+                issues.push(`file not found: ${p}`);
+            }
+        }
+    }
+    // Check for common GNU flags on macOS
+    const gnuFlags = command.match(/--max-depth|--color=|--sort=|--field-type|--no-deps/g);
+    if (gnuFlags) {
+        issues.push(`GNU flag on macOS: ${gnuFlags.join(", ")}`);
+    }
+    return {
+        valid: issues.length === 0,
+        issues,
+    };
+}

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hasna/terminal",
-  "version": "1.5.0",
+  "version": "1.6.1",
   "description": "Smart terminal wrapper for AI agents and humans — structured output, token compression, MCP server, natural language",
   "type": "module",
   "bin": {

package/src/ai.ts

@@ -148,12 +148,25 @@ function detectProjectContext(): string {
     const topLevel = execSync("ls -1", { cwd, encoding: "utf8", timeout: 2000 }).trim();
     parts.push(`Top-level: ${topLevel.split("\n").join(", ")}`);
 
+    // Detect monorepo (packages/ or workspaces in package.json)
+    const isMonorepo = existsSync(join(cwd, "packages")) || existsSync(join(cwd, "apps"));
+    if (isMonorepo) {
+      const pkgDirs = execSync(
+        `ls -d packages/*/src 2>/dev/null || ls -d apps/*/src 2>/dev/null || echo ""`,
+        { cwd, encoding: "utf8", timeout: 2000 }
+      ).trim();
+      if (pkgDirs) {
+        parts.push(`MONOREPO: Source is in packages/*/src/, NOT src/. Search packages/ not src/.`);
+        parts.push(`Package sources:\n${pkgDirs}`);
+      }
+    }
+
     // src/ structure — include FILES so AI knows exact filenames + extensions
-    for (const srcDir of ["src", "lib", "app"]) {
+    for (const srcDir of isMonorepo ? ["packages"] : ["src", "lib", "app"]) {
       if (existsSync(join(cwd, srcDir))) {
         const tree = execSync(
-          `find ${srcDir} -maxdepth 3 -not -path '*/node_modules/*' -not -path '*/dist/*' -not -name '*.test.*' -not -name '*.spec.*' 2>/dev/null | sort | head -60`,
-          { cwd, encoding: "utf8", timeout: 2000 }
+          `find ${srcDir} -maxdepth ${isMonorepo ? 4 : 3} -not -path '*/node_modules/*' -not -path '*/dist/*' -not -name '*.test.*' -not -name '*.spec.*' 2>/dev/null | sort | head -80`,
+          { cwd, encoding: "utf8", timeout: 3000 }
         ).trim();
         if (tree) parts.push(`Files in ${srcDir}/:\n${tree}`);
         break;
@@ -238,6 +251,8 @@ SEMANTIC MAPPING: When the user references a concept, search the file tree for R
 - When uncertain: grep -rn "keyword" src/ --include="*.ts" -l (list matching files)
 
 ACTION vs CONCEPTUAL: If the prompt starts with "run", "execute", "check", "test", "build", "show output of" — ALWAYS generate an executable command. NEVER read README for action requests. Only read docs for "explain why", "what does X mean", "how was X designed".
+
+MONOREPO: If the project context says "MONOREPO", search packages/ or apps/ NOT src/. Use: grep -rn "pattern" packages/ --include="*.ts". For specific packages, use packages/PKGNAME/src/.
 cwd: ${process.cwd()}
 shell: zsh / macOS${projectContext}${restrictionBlock}${contextBlock}`;
 }

package/src/cli.tsx

@@ -448,10 +448,31 @@ else if (args.length > 0) {
     }
   }
 
+  // Step 2: Validate command before executing
+  const { validateCommand } = await import("./command-validator.js");
+  const validation = validateCommand(command, process.cwd());
+  if (!validation.valid) {
+    // Auto-retry: re-translate with simpler constraints
+    console.error(`[open-terminal] invalid command detected: ${validation.issues.join(", ")}`);
+    try {
+      const retryCommand = await translateToCommand(
+        `${prompt} (IMPORTANT: keep it simple. Use basic grep/find/cat/ls/wc commands. No complex awk/sed pipelines. No GNU flags. Verify file paths from the project context.)`,
+        perms, []
+      );
+      if (retryCommand && retryCommand !== command) {
+        const retryValidation = validateCommand(retryCommand, process.cwd());
+        if (retryValidation.valid || retryValidation.issues.length < validation.issues.length) {
+          command = retryCommand;
+          console.error(`[open-terminal] retried: $ ${command}`);
+        }
+      }
+    } catch {}
+  }
+
   // Show what we're running
   console.error(`$ ${command}`);
 
-  // Step 2: Rewrite for optimization
+  // Step 3: Rewrite for optimization
   const rw = rewriteCommand(command);
   const actualCmd = rw.changed ? rw.rewritten : command;
   if (rw.changed) console.error(`[open-terminal] optimized: ${actualCmd}`);
@@ -509,6 +530,27 @@ else if (args.length > 0) {
       console.log(`No results found for: ${prompt}`);
       process.exit(0);
     }
+
+    // Auto-retry: if command failed (exit 2+), ask AI for a simpler alternative
+    if (e.status >= 2 && !actualCmd.includes("(retry)")) {
+      try {
+        const retryCmd = await translateToCommand(
+          `${prompt} (The previous command failed with: ${errStderr.slice(0, 200)}. Try a SIMPLER approach. Use basic commands only.)`,
+          perms, []
+        );
+        if (retryCmd && !isIrreversible(retryCmd) && !checkPermissions(retryCmd, perms)) {
+          console.error(`[open-terminal] retrying: $ ${retryCmd}`);
+          const retryResult = execSync(retryCmd + " #(retry)", { encoding: "utf8", maxBuffer: 10 * 1024 * 1024, cwd: process.cwd() });
+          const retryClean = stripNoise(stripAnsi(retryResult)).cleaned;
+          if (retryClean.length > 5) {
+            const processed = await processOutput(retryCmd, retryClean, prompt);
+            console.log(processed.aiProcessed ? processed.summary : retryClean);
+            process.exit(0);
+          }
+        }
+      } catch { /* retry also failed, fall through */ }
+    }
+
     // Combine stdout+stderr and try AI answer framing (for audit/lint/test commands)
     const combined = errStderr && errStdout.includes(errStderr.trim()) ? errStdout : errStdout + errStderr;
     const errorClean = stripNoise(stripAnsi(combined)).cleaned;

package/src/command-validator.ts

@@ -0,0 +1,85 @@
+// Command validator — catch invalid commands BEFORE executing
+// Prevents shell errors from hallucinated flags, wrong paths, bad syntax
+
+import { existsSync } from "fs";
+import { join } from "path";
+
+export interface ValidationResult {
+  valid: boolean;
+  issues: string[];
+  fixedCommand?: string;
+}
+
+/** Extract file paths referenced in a command */
+function extractPaths(command: string): string[] {
+  const paths: string[] = [];
+  // Match quoted paths
+  const quoted = command.match(/["']([^"']+\.\w+)["']/g);
+  if (quoted) paths.push(...quoted.map(q => q.replace(/["']/g, "")));
+  // Match unquoted paths with extensions or directory separators
+  const tokens = command.split(/\s+/);
+  for (const t of tokens) {
+    if (t.includes("/") && !t.startsWith("-") && !t.startsWith("|") && !t.startsWith("&")) {
+      // Clean shell operators from end
+      const clean = t.replace(/[;|&>]+$/, "");
+      if (clean && !clean.startsWith("-")) paths.push(clean);
+    }
+  }
+  return [...new Set(paths)];
+}
+
+/** Check for obviously broken shell syntax */
+function checkSyntax(command: string): string[] {
+  const issues: string[] = [];
+
+  // Unmatched quotes
+  const singleQuotes = (command.match(/'/g) || []).length;
+  const doubleQuotes = (command.match(/"/g) || []).length;
+  if (singleQuotes % 2 !== 0) issues.push("unmatched single quote");
+  if (doubleQuotes % 2 !== 0) issues.push("unmatched double quote");
+
+  // Unmatched parentheses
+  const openParens = (command.match(/\(/g) || []).length;
+  const closeParens = (command.match(/\)/g) || []).length;
+  if (openParens !== closeParens) issues.push("unmatched parentheses");
+
+  // Empty pipe targets
+  if (/\|\s*$/.test(command)) issues.push("pipe with no target");
+  if (/^\s*\|/.test(command)) issues.push("pipe with no source");
+
+  return issues;
+}
+
+/** Validate a command before execution */
+export function validateCommand(command: string, cwd: string): ValidationResult {
+  const issues: string[] = [];
+
+  // Check syntax
+  issues.push(...checkSyntax(command));
+
+  // Check file paths exist
+  const paths = extractPaths(command);
+  for (const p of paths) {
+    const fullPath = p.startsWith("/") ? p : join(cwd, p);
+    if (p.includes("*") || p.includes("?")) continue; // skip globs
+    if (p.startsWith("-")) continue; // skip flags
+    if ([".", "..", "/", "~"].includes(p)) continue; // skip special
+    if (!existsSync(fullPath) && !existsSync(p)) {
+      // Only flag source file paths, not output paths
+      if (/\.(ts|tsx|js|jsx|json|md|yaml|yml|py|go|rs)$/.test(p)) {
+        issues.push(`file not found: ${p}`);
+      }
+    }
+  }
+
+  // Check for common GNU flags on macOS
+  const gnuFlags = command.match(/--max-depth|--color=|--sort=|--field-type|--no-deps/g);
+  if (gnuFlags) {
+    issues.push(`GNU flag on macOS: ${gnuFlags.join(", ")}`);
+  }
+
+  return {
+    valid: issues.length === 0,
+    issues,
+  };
+}