neuro-cli 4.1.1 → 4.1.2

package/dist/core/approval.d.ts

@@ -1,3 +1,4 @@
+import { Interface as ReadLineInterface } from 'readline';
 export type PermissionMode = 'manual' | 'auto' | 'plan' | 'yolo';
 export interface ApprovalResult {
     approved: boolean;
@@ -39,6 +40,7 @@ export declare class ApprovalSystem {
     private persistDecisions;
     private consecutiveAutoApproves;
     private rl;
+    private mainRl;
     private stats;
     private pendingBatch;
     private batchTimer;
@@ -65,6 +67,11 @@ export declare class ApprovalSystem {
     private isWhitelisted;
     private getPattern;
     private getAlwaysKey;
+    /**
+     * Set the main readline interface from index.ts
+     * This prevents creating a second readline on the same stdin
+     */
+    setMainReadline(rl: ReadLineInterface): void;
     private readline;
     close(): void;
     addAutoApprove(toolName: string): void;

package/dist/core/approval.js

@@ -23,6 +23,7 @@ export class ApprovalSystem {
     persistDecisions = true;
     consecutiveAutoApproves = 0;
     rl = null;
+    mainRl = null;
     stats = new Map();
     pendingBatch = [];
     batchTimer = null;
@@ -333,10 +334,38 @@ export class ApprovalSystem {
     getAlwaysKey(toolName, args) {
         return `${toolName}:${this.getPattern(args)}`;
     }
+    /**
+     * Set the main readline interface from index.ts
+     * This prevents creating a second readline on the same stdin
+     */
+    setMainReadline(rl) {
+        this.mainRl = rl;
+    }
     readline(prompt) {
-        if (!this.rl)
-            this.rl = createInterface({ input: process.stdin, output: process.stdout });
-        return new Promise((resolve) => { this.rl.question(prompt, (answer) => resolve(answer)); });
+        // Pause the main readline to prevent it from also reading stdin input
+        // This is the root cause of double character input (YY instead of Y)
+        const mainInputStream = this.mainRl?.input;
+        const wasMainPaused = mainInputStream?.isPaused?.();
+        if (this.mainRl && mainInputStream && mainInputStream.isPaused()) {
+            // Already paused, skip
+        }
+        else if (this.mainRl) {
+            this.mainRl.pause();
+        }
+        return new Promise((resolve) => {
+            // Reuse the main readline if available, otherwise create a temporary one
+            const rl = this.mainRl || this.rl || createInterface({ input: process.stdin, output: process.stdout });
+            if (!this.rl && !this.mainRl)
+                this.rl = rl;
+            rl.question(prompt, (answer) => {
+                // Resume the main readline after getting the answer
+                if (this.mainRl) {
+                    this.mainRl.resume();
+                    this.mainRl.prompt();
+                }
+                resolve(answer);
+            });
+        });
     }
     close() {
         if (this.rl) {

package/dist/core/diff-preview.js

@@ -3,6 +3,7 @@
 // Shows file changes before applying them
 // ============================================================
 import { readFileSync, existsSync } from 'fs';
+import { createInterface } from 'readline';
 import chalk from 'chalk';
 export class DiffPreview {
     /**
@@ -128,7 +129,7 @@ export class DiffPreview {
      */
     static async confirmDiff(diff) {
         DiffPreview.renderDiff(diff);
-        const rl = require('readline').createInterface({ input: process.stdin, output: process.stdout });
+        const rl = createInterface({ input: process.stdin, output: process.stdout });
         return new Promise((resolve) => {
             rl.question(chalk.cyan('  Apply these changes? [y/n]: '), (answer) => {
                 rl.close();

package/dist/core/engine.js

@@ -1,5 +1,5 @@
 // ============================================================
-// NeuroCLI - NeuroEngine v4.1.1
+// NeuroCLI - NeuroEngine v4.1.2
 // The main engine that ties everything together
 // Now with: Sandbox, Plugin SDK, Enhanced MCP, Enhanced Approval,
 // Model Router, Prompt Cache, Undo/Redo, Output Styles,
@@ -483,7 +483,7 @@ export class NeuroEngine {
         this.gitWorktree = new GitWorktreeManager(process.cwd());
         // Auto-Updater
         this.updater = new AutoUpdater({
-            currentVersion: '4.1.1',
+            currentVersion: '4.1.2',
             autoCheck: true,
             autoUpdate: false,
         });

package/dist/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 // ============================================================
 // NeuroCLI - Advanced AI Terminal Coding Assistant
-// Main Entry Point - v4.1.1 with cross-platform path fix
+// Main Entry Point - v4.1.2 with cross-platform path fix
 // ============================================================
 import { Command } from 'commander';
 import { createInterface } from 'readline';
@@ -16,7 +16,7 @@ import { HeadlessMode } from './core/headless.js';
 import { ShellCompletionGenerator } from './core/shell-completion.js';
 import chalk from 'chalk';
 import { AutoUpdater } from './core/updater.js';
-const VERSION = '4.1.1';
+const VERSION = '4.1.2';
 // ---- CLI Setup ----
 const program = new Command();
 program
@@ -430,14 +430,22 @@ async function startInteractive(options) {
         completer: completionEngine.complete,
     });
     rl.prompt();
+    // Wire the main readline to the approval system to prevent dual-readline bug
+    // (double character input when approval prompt and main rl both read stdin)
+    engine.approval.setMainReadline(rl);
     let currentMode = 'auto';
     let currentAgent;
+    let isProcessing = false;
     rl.on('line', async (line) => {
         const input = line.trim();
         if (!input) {
             rl.prompt();
             return;
         }
+        // Prevent input while processing (approval prompt may be active)
+        if (isProcessing) {
+            return;
+        }
         // Add to history
         completionEngine.addHistory(input);
         // Handle slash commands
@@ -909,7 +917,7 @@ async function startInteractive(options) {
                     }
                     break;
                 case 'doctor':
-                    console.log(chalk.bold('\nNeuroCLI v4.1.1 Health Check:\n'));
+                    console.log(chalk.bold('\nNeuroCLI v4.1.2 Health Check:\n'));
                     console.log(`  API Key: ${config.apiKey ? chalk.green('configured') : chalk.red('MISSING')}`);
                     console.log(`  Default Model: ${chalk.cyan(config.defaultModel)} ${MODELS[config.defaultModel] ? chalk.green('valid') : chalk.red('INVALID')}`);
                     console.log(`  MCP Servers: ${chalk.cyan(String(engine.mcpClient.listServers().length))}`);
@@ -1112,11 +1120,15 @@ async function startInteractive(options) {
         }
         // Process message with the engine
         try {
+            isProcessing = true;
             await engine.processMessage(input, currentMode, currentAgent);
         }
         catch (error) {
             engine.ui.error(error instanceof Error ? error.message : String(error));
         }
+        finally {
+            isProcessing = false;
+        }
         rl.prompt();
     });
     rl.on('close', () => {
@@ -1128,7 +1140,7 @@ async function startInteractive(options) {
 }
 function printHelp(engine) {
     const t = engine.ui.theme;
-    console.log(`\n  ${t.bold('NeuroCLI v4.1.1 Commands:')}\n`);
+    console.log(`\n  ${t.bold('NeuroCLI v4.1.2 Commands:')}\n`);
     console.log(`  ${t.tool('/help')}            Show this help message`);
     console.log(`  ${t.tool('/model [id]')}      Switch or list models`);
     console.log(`  ${t.tool('/agent [name]')}    Switch or list agents`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "neuro-cli",
-  "version": "4.1.1",
+  "version": "4.1.2",
   "description": "Advanced AI-powered terminal coding assistant with multi-agent orchestration, sub-agent spawning, ACP protocol, OS-level sandboxing, spec-driven development, smart monitoring, multi-model routing, MCP Apps, auto-updater, and 23+ free models",
   "main": "dist/index.js",
   "bin": {