neuro-cli 4.1.0 → 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.0
+// 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.0',
+            currentVersion: '4.1.2',
             autoCheck: true,
             autoUpdate: false,
         });

package/dist/core/sandbox.js

@@ -6,6 +6,7 @@
 import { join, resolve, relative, normalize } from 'path';
 import { existsSync, mkdirSync, readFileSync, writeFileSync, copyFileSync } from 'fs';
 import chalk from 'chalk';
+import { normalizeCrossPlatformPath, isWindowsAbsolutePath } from '../utils/crosspath.js';
 export const DEFAULT_SANDBOX_CONFIG = {
     enabled: false,
     rootDir: process.cwd(),
@@ -312,11 +313,21 @@ export class Sandbox {
     }
     // --- Private Helpers ---
     resolvePath(filePath) {
+        // Expand home directory
+        if (filePath.startsWith('~/') || filePath === '~') {
+            const homeDir = process.env.HOME || process.env.USERPROFILE || require('os').homedir();
+            const expanded = filePath === '~' ? homeDir : join(homeDir, filePath.slice(2));
+            return normalize(expanded);
+        }
+        // Windows absolute path (C:\... or C:/...)
+        if (isWindowsAbsolutePath(filePath)) {
+            return normalizeCrossPlatformPath(filePath);
+        }
+        // POSIX absolute path
         if (filePath.startsWith('/'))
             return normalize(filePath);
-        if (filePath.startsWith('~/'))
-            return normalize(join(require('os').homedir(), filePath.slice(2)));
-        return normalize(resolve(this.config.rootDir, filePath));
+        // Relative path - resolve against rootDir
+        return normalize(resolve(this.config.rootDir, normalizeCrossPlatformPath(filePath)));
     }
     isUnderRootDir(absPath) {
         const relativePath = relative(this.config.rootDir, absPath);

package/dist/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 // ============================================================
 // NeuroCLI - Advanced AI Terminal Coding Assistant
-// Main Entry Point - v4.1.0 with auto-updater
+// 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.0';
+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.0 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.0 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/dist/tools/file.js

@@ -5,6 +5,7 @@
 import { readFileSync, writeFileSync, existsSync, mkdirSync, unlinkSync, statSync, readdirSync } from 'fs';
 import { join, relative, extname } from 'path';
 import { execSync } from 'child_process';
+import { resolvePath } from '../utils/crosspath.js';
 const MAX_FILE_SIZE = 1024 * 1024; // 1MB
 const MAX_OUTPUT_LENGTH = 50000;
 function truncateOutput(output, maxLength = MAX_OUTPUT_LENGTH) {
@@ -32,7 +33,7 @@ export const readFileTool = {
     definition: readFileDef,
     risk: 'low',
     async execute(args, context) {
-        const filePath = join(context.workingDirectory, args.path);
+        const filePath = resolvePath(context.workingDirectory, args.path);
         if (!existsSync(filePath))
             return `Error: File not found: ${args.path}`;
         const stat = statSync(filePath);
@@ -78,7 +79,7 @@ export const writeFileTool = {
         };
     },
     async execute(args, context) {
-        const filePath = join(context.workingDirectory, args.path);
+        const filePath = resolvePath(context.workingDirectory, args.path);
         const dir = join(filePath, '..');
         if (!existsSync(dir)) {
             mkdirSync(dir, { recursive: true });
@@ -114,7 +115,7 @@ export const editFileTool = {
         };
     },
     async execute(args, context) {
-        const filePath = join(context.workingDirectory, args.path);
+        const filePath = resolvePath(context.workingDirectory, args.path);
         if (!existsSync(filePath))
             return `Error: File not found: ${args.path}`;
         let content = readFileSync(filePath, 'utf-8');
@@ -157,7 +158,7 @@ export const deleteFileTool = {
         };
     },
     async execute(args, context) {
-        const filePath = join(context.workingDirectory, args.path);
+        const filePath = resolvePath(context.workingDirectory, args.path);
         if (!existsSync(filePath))
             return `Error: File not found: ${args.path}`;
         unlinkSync(filePath);
@@ -183,7 +184,7 @@ export const listDirectoryTool = {
     definition: listDirDef,
     risk: 'low',
     async execute(args, context) {
-        const dirPath = join(context.workingDirectory, args.path || '.');
+        const dirPath = resolvePath(context.workingDirectory, args.path || '.');
         if (!existsSync(dirPath))
             return `Error: Directory not found: ${args.path}`;
         const stat = statSync(dirPath);
@@ -259,7 +260,7 @@ export const searchFilesTool = {
     risk: 'low',
     async execute(args, context) {
         const pattern = args.pattern;
-        const searchPath = join(context.workingDirectory, args.path || '.');
+        const searchPath = resolvePath(context.workingDirectory, args.path || '.');
         const fileType = args.file_type;
         const maxResults = args.max_results || 50;
         const caseInsensitive = args.case_insensitive || false;
@@ -332,7 +333,7 @@ export const applyDiffTool = {
         };
     },
     async execute(args, context) {
-        const filePath = join(context.workingDirectory, args.path);
+        const filePath = resolvePath(context.workingDirectory, args.path);
         if (!existsSync(filePath))
             return `Error: File not found: ${args.path}`;
         try {

package/dist/utils/crosspath.d.ts

@@ -0,0 +1,28 @@
+/**
+ * Check if a path looks like a Windows absolute path (drive letter)
+ * Matches: C:\, C:/, D:\foo, etc.
+ */
+export declare function isWindowsAbsolutePath(p: string): boolean;
+/**
+ * Normalize a path for cross-platform compatibility:
+ * 1. Convert backslashes to forward slashes
+ * 2. Detect Windows absolute paths (C:\...) and treat them as absolute
+ * 3. Resolve ~ to home directory
+ * 4. Handle relative paths correctly
+ */
+export declare function normalizeCrossPlatformPath(inputPath: string): string;
+/**
+ * Resolve a path relative to a working directory with cross-platform support.
+ * This is the main function to use in file tools.
+ *
+ * - Absolute paths (POSIX or Windows) are used as-is after normalization
+ * - Relative paths are resolved against workingDirectory
+ * - Home directory (~) is expanded
+ * - Backslashes are normalized to forward slashes
+ */
+export declare function resolvePath(workingDirectory: string, inputPath: string): string;
+/**
+ * Check if a path is absolute on any platform (POSIX or Windows)
+ */
+export declare function isAnyAbsolutePath(p: string): boolean;
+//# sourceMappingURL=crosspath.d.ts.map

package/dist/utils/crosspath.js

@@ -0,0 +1,81 @@
+// ============================================================
+// NeuroCLI - Cross-Platform Path Utility
+// Handles Windows, macOS, and Linux path differences
+// ============================================================
+import { join, resolve, isAbsolute, normalize } from 'path';
+/**
+ * Check if a path looks like a Windows absolute path (drive letter)
+ * Matches: C:\, C:/, D:\foo, etc.
+ */
+export function isWindowsAbsolutePath(p) {
+    return /^[A-Za-z]:[/\\]/.test(p);
+}
+/**
+ * Normalize a path for cross-platform compatibility:
+ * 1. Convert backslashes to forward slashes
+ * 2. Detect Windows absolute paths (C:\...) and treat them as absolute
+ * 3. Resolve ~ to home directory
+ * 4. Handle relative paths correctly
+ */
+export function normalizeCrossPlatformPath(inputPath) {
+    let p = inputPath;
+    // Step 1: Expand home directory
+    if (p.startsWith('~/') || p === '~') {
+        const homeDir = process.env.HOME || process.env.USERPROFILE || '';
+        p = p === '~' ? homeDir : join(homeDir, p.slice(2));
+        return normalize(p);
+    }
+    // Step 2: Detect Windows absolute path with drive letter
+    if (isWindowsAbsolutePath(p)) {
+        // Normalize backslashes to forward slashes, then let Node normalize
+        p = p.replace(/\\/g, '/');
+        // On Windows, Node's path.resolve handles drive letters natively
+        // On non-Windows, we still need to handle this for the case where
+        // the LLM sends a Windows path from a user's context
+        if (process.platform === 'win32') {
+            return normalize(p);
+        }
+        // On non-Windows systems, a Windows path like C:/Users/foo
+        // won't resolve. This likely means the LLM is confused about
+        // the OS. We still normalize it so it can be used as-is.
+        return normalize(p);
+    }
+    // Step 3: Convert backslashes to forward slashes for mixed paths
+    // (handles cases where LLM or user provides backslash paths on any OS)
+    p = p.replace(/\\/g, '/');
+    // Step 4: If already absolute (POSIX), normalize directly
+    if (isAbsolute(p)) {
+        return normalize(p);
+    }
+    // Step 5: Relative path - return as-is (caller will resolve against working dir)
+    return normalize(p);
+}
+/**
+ * Resolve a path relative to a working directory with cross-platform support.
+ * This is the main function to use in file tools.
+ *
+ * - Absolute paths (POSIX or Windows) are used as-is after normalization
+ * - Relative paths are resolved against workingDirectory
+ * - Home directory (~) is expanded
+ * - Backslashes are normalized to forward slashes
+ */
+export function resolvePath(workingDirectory, inputPath) {
+    const normalized = normalizeCrossPlatformPath(inputPath);
+    // If it's already an absolute path after normalization, use it directly
+    if (isAbsolute(normalized)) {
+        return normalized;
+    }
+    // Windows absolute paths that became normalized (C:/...) 
+    if (isWindowsAbsolutePath(normalized)) {
+        return normalized;
+    }
+    // Relative path: resolve against working directory
+    return resolve(workingDirectory, normalized);
+}
+/**
+ * Check if a path is absolute on any platform (POSIX or Windows)
+ */
+export function isAnyAbsolutePath(p) {
+    return isAbsolute(p) || isWindowsAbsolutePath(p);
+}
+//# sourceMappingURL=crosspath.js.map

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "neuro-cli",
-  "version": "4.1.0",
+  "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": {