@hasna/terminal 1.2.1 → 1.3.0

package/dist/ai.js

@@ -43,6 +43,8 @@ const IRREVERSIBLE_PATTERNS = [
     // Code modification / package installation (security risk)
     /\bnpx\s+\S+/, /\bnpm\s+install\b/, /\bbun\s+add\b/, /\bpip\s+install\b/,
     /\bcodemod\b/, /\bsed\s+-i\b/, /\bawk\s.*>/, /\bperl\s+-[pi]\b/,
+    // File creation/modification (READ-ONLY terminal)
+    /\btouch\b/, /\bmkdir\b/, /\becho\s.*>/, /\btee\b/, /\bcp\b/, /\bmv\b/,
 ];
 export function isIrreversible(command) {
     return IRREVERSIBLE_PATTERNS.some((r) => r.test(command));
@@ -109,12 +111,12 @@ function detectProjectContext() {
         // Top-level dirs
         const topLevel = execSync("ls -1", { cwd, encoding: "utf8", timeout: 2000 }).trim();
         parts.push(`Top-level: ${topLevel.split("\n").join(", ")}`);
-        // src/ structure (2 levels deep, most important for path resolution)
+        // src/ structure — include FILES so AI knows exact filenames + extensions
         for (const srcDir of ["src", "lib", "app"]) {
             if (existsSync(join(cwd, srcDir))) {
-                const tree = execSync(`find ${srcDir} -maxdepth 2 -type d -not -path '*/node_modules/*' 2>/dev/null | head -30`, { cwd, encoding: "utf8", timeout: 2000 }).trim();
+                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();
                 if (tree)
-                    parts.push(`Directories in ${srcDir}/:\n${tree}`);
+                    parts.push(`Files in ${srcDir}/:\n${tree}`);
                 break;
             }
         }
@@ -166,6 +168,8 @@ RULES:
 - NEVER install packages (npx, npm install, pip install, brew install). This is a READ-ONLY terminal.
 - NEVER modify source code (sed -i, codemod, awk with redirect). Only observe, never change.
 - Search src/ directory, NOT dist/ or node_modules/ for code queries.
+- For compound questions ("how many X and are they Y"), prefer ONE command that captures all info. Do NOT chain with &&.
+- Use exact file paths from the project context below. Do NOT guess paths.
 cwd: ${process.cwd()}
 shell: zsh / macOS${projectContext}${restrictionBlock}${contextBlock}`;
 }

package/dist/cli.js

@@ -471,17 +471,20 @@ else if (args.length > 0) {
             console.log(JSON.stringify(lazy, null, 2));
             process.exit(0);
         }
-        // AI summary for medium-large output
-        if (shouldProcess(clean)) {
+        // AI answer framing — ALWAYS use in NL mode (even for small output)
+        // The AI needs to ANSWER the question, not just pass through data
+        if (clean.length > 10) {
             const processed = await processOutput(actualCmd, clean, prompt);
-            if (processed.aiProcessed && processed.tokensSaved > 30) {
-                recordSaving("compressed", processed.tokensSaved);
+            if (processed.aiProcessed) {
+                if (processed.tokensSaved > 0)
+                    recordSaving("compressed", processed.tokensSaved);
                 console.log(processed.summary);
-                console.error(`[open-terminal] ${rawTokens} → ${rawTokens - processed.tokensSaved} tokens (saved ${processed.tokensSaved})`);
+                if (processed.tokensSaved > 10)
+                    console.error(`[open-terminal] ${rawTokens} → ${rawTokens - processed.tokensSaved} tokens (saved ${processed.tokensSaved})`);
                 process.exit(0);
             }
         }
-        // Small output — pass through clean
+        // Fallback: AI unavailable — pass through clean
         console.log(clean);
         const saved = rawTokens - estimateTokens(clean);
         if (saved > 10) {
@@ -490,9 +493,14 @@ else if (args.length > 0) {
         }
     }
     catch (e) {
-        const stderr = e.stderr?.toString() ?? "";
-        const stdout = e.stdout?.toString() ?? "";
-        const combined = stderr && stdout.includes(stderr.trim()) ? stdout : stdout + stderr;
+        // Empty result (grep exit 1 = no matches) — not a real error
+        const errStdout = e.stdout?.toString() ?? "";
+        const errStderr = e.stderr?.toString() ?? "";
+        if (e.status === 1 && !errStdout.trim() && !errStderr.trim()) {
+            console.log(`No results found for: ${prompt}`);
+            process.exit(0);
+        }
+        const combined = errStderr && errStdout.includes(errStderr.trim()) ? errStdout : errStdout + errStderr;
         console.log(stripNoise(stripAnsi(combined)).cleaned);
         process.exit(e.status ?? 1);
     }

package/dist/output-processor.js

@@ -22,8 +22,8 @@ RULES:
  */
 export async function processOutput(command, output, originalPrompt) {
     const lines = output.split("\n");
-    // Short output — pass through, no AI needed
-    if (lines.length <= MIN_LINES_TO_PROCESS) {
+    // Short output — skip AI UNLESS we have an original prompt (NL mode needs answer framing)
+    if (lines.length <= MIN_LINES_TO_PROCESS && !originalPrompt) {
         return {
             summary: output,
             full: output,

package/package.json

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

package/src/ai.ts

@@ -51,6 +51,8 @@ const IRREVERSIBLE_PATTERNS = [
   // Code modification / package installation (security risk)
   /\bnpx\s+\S+/, /\bnpm\s+install\b/, /\bbun\s+add\b/, /\bpip\s+install\b/,
   /\bcodemod\b/, /\bsed\s+-i\b/, /\bawk\s.*>/, /\bperl\s+-[pi]\b/,
+  // File creation/modification (READ-ONLY terminal)
+  /\btouch\b/, /\bmkdir\b/, /\becho\s.*>/, /\btee\b/, /\bcp\b/, /\bmv\b/,
 ];
 
 export function isIrreversible(command: string): boolean {
@@ -138,14 +140,14 @@ function detectProjectContext(): string {
     const topLevel = execSync("ls -1", { cwd, encoding: "utf8", timeout: 2000 }).trim();
     parts.push(`Top-level: ${topLevel.split("\n").join(", ")}`);
 
-    // src/ structure (2 levels deep, most important for path resolution)
+    // src/ structure — include FILES so AI knows exact filenames + extensions
     for (const srcDir of ["src", "lib", "app"]) {
       if (existsSync(join(cwd, srcDir))) {
         const tree = execSync(
-          `find ${srcDir} -maxdepth 2 -type d -not -path '*/node_modules/*' 2>/dev/null | head -30`,
+          `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();
-        if (tree) parts.push(`Directories in ${srcDir}/:\n${tree}`);
+        if (tree) parts.push(`Files in ${srcDir}/:\n${tree}`);
         break;
       }
     }
@@ -201,6 +203,8 @@ RULES:
 - NEVER install packages (npx, npm install, pip install, brew install). This is a READ-ONLY terminal.
 - NEVER modify source code (sed -i, codemod, awk with redirect). Only observe, never change.
 - Search src/ directory, NOT dist/ or node_modules/ for code queries.
+- For compound questions ("how many X and are they Y"), prefer ONE command that captures all info. Do NOT chain with &&.
+- Use exact file paths from the project context below. Do NOT guess paths.
 cwd: ${process.cwd()}
 shell: zsh / macOS${projectContext}${restrictionBlock}${contextBlock}`;
 }

package/src/cli.tsx

@@ -455,25 +455,31 @@ else if (args.length > 0) {
       process.exit(0);
     }
 
-    // AI summary for medium-large output
-    if (shouldProcess(clean)) {
+    // AI answer framing — ALWAYS use in NL mode (even for small output)
+    // The AI needs to ANSWER the question, not just pass through data
+    if (clean.length > 10) {
       const processed = await processOutput(actualCmd, clean, prompt);
-      if (processed.aiProcessed && processed.tokensSaved > 30) {
-        recordSaving("compressed", processed.tokensSaved);
+      if (processed.aiProcessed) {
+        if (processed.tokensSaved > 0) recordSaving("compressed", processed.tokensSaved);
         console.log(processed.summary);
-        console.error(`[open-terminal] ${rawTokens} → ${rawTokens - processed.tokensSaved} tokens (saved ${processed.tokensSaved})`);
+        if (processed.tokensSaved > 10) console.error(`[open-terminal] ${rawTokens} → ${rawTokens - processed.tokensSaved} tokens (saved ${processed.tokensSaved})`);
         process.exit(0);
       }
     }
 
-    // Small output — pass through clean
+    // Fallback: AI unavailable — pass through clean
     console.log(clean);
     const saved = rawTokens - estimateTokens(clean);
     if (saved > 10) { recordSaving("compressed", saved); console.error(`[open-terminal] saved ${saved} tokens`); }
   } catch (e: any) {
-    const stderr = e.stderr?.toString() ?? "";
-    const stdout = e.stdout?.toString() ?? "";
-    const combined = stderr && stdout.includes(stderr.trim()) ? stdout : stdout + stderr;
+    // Empty result (grep exit 1 = no matches) — not a real error
+    const errStdout = e.stdout?.toString() ?? "";
+    const errStderr = e.stderr?.toString() ?? "";
+    if (e.status === 1 && !errStdout.trim() && !errStderr.trim()) {
+      console.log(`No results found for: ${prompt}`);
+      process.exit(0);
+    }
+    const combined = errStderr && errStdout.includes(errStderr.trim()) ? errStdout : errStdout + errStderr;
     console.log(stripNoise(stripAnsi(combined)).cleaned);
     process.exit(e.status ?? 1);
   }

package/src/output-processor.ts

@@ -52,8 +52,8 @@ export async function processOutput(
 ): Promise<ProcessedOutput> {
   const lines = output.split("\n");
 
-  // Short output — pass through, no AI needed
-  if (lines.length <= MIN_LINES_TO_PROCESS) {
+  // Short output — skip AI UNLESS we have an original prompt (NL mode needs answer framing)
+  if (lines.length <= MIN_LINES_TO_PROCESS && !originalPrompt) {
     return {
       summary: output,
       full: output,