@hasna/terminal 1.2.2 → 1.3.1

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));
@@ -196,8 +198,27 @@ export async function translateToCommand(nl, perms, sessionEntries, onToken) {
     }
     if (text.startsWith("BLOCKED:"))
         throw new Error(text);
-    cacheSet(nl, text);
-    return text;
+    // Strip AI reasoning — extract only the shell command
+    // AI sometimes prefixes with "Based on..." or wraps in backticks
+    let cleaned = text.trim();
+    // Remove markdown code blocks
+    cleaned = cleaned.replace(/^```(?:bash|sh|shell)?\n?/m, "").replace(/\n?```$/m, "");
+    // Remove lines that look like AI reasoning (start with capital letter, contain "I ", "Based on", etc.)
+    const lines = cleaned.split("\n");
+    const commandLines = lines.filter(l => {
+        const t = l.trim();
+        if (!t)
+            return false;
+        // Skip obvious reasoning lines
+        if (/^(Based on|I |This |The |Let me|Here|Note:|Since|Looking|To )/.test(t))
+            return false;
+        if (/^[A-Z][a-z].*\.$/.test(t))
+            return false; // English sentence ending with period
+        return true;
+    });
+    cleaned = commandLines.join("\n").trim() || cleaned;
+    cacheSet(nl, cleaned);
+    return cleaned;
 }
 // ── prefetch ──────────────────────────────────────────────────────────────────
 export function prefetchNext(lastNl, perms, sessionEntries) {

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.2",
+  "version": "1.3.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

@@ -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 {
@@ -237,8 +239,26 @@ export async function translateToCommand(
   }
 
   if (text.startsWith("BLOCKED:")) throw new Error(text);
-  cacheSet(nl, text);
-  return text;
+
+  // Strip AI reasoning — extract only the shell command
+  // AI sometimes prefixes with "Based on..." or wraps in backticks
+  let cleaned = text.trim();
+  // Remove markdown code blocks
+  cleaned = cleaned.replace(/^```(?:bash|sh|shell)?\n?/m, "").replace(/\n?```$/m, "");
+  // Remove lines that look like AI reasoning (start with capital letter, contain "I ", "Based on", etc.)
+  const lines = cleaned.split("\n");
+  const commandLines = lines.filter(l => {
+    const t = l.trim();
+    if (!t) return false;
+    // Skip obvious reasoning lines
+    if (/^(Based on|I |This |The |Let me|Here|Note:|Since|Looking|To )/.test(t)) return false;
+    if (/^[A-Z][a-z].*\.$/.test(t)) return false; // English sentence ending with period
+    return true;
+  });
+  cleaned = commandLines.join("\n").trim() || cleaned;
+
+  cacheSet(nl, cleaned);
+  return cleaned;
 }
 
 // ── prefetch ──────────────────────────────────────────────────────────────────

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,