neuro-cli 4.1.3 → 4.2.0

package/dist/agents/base.js

@@ -192,7 +192,7 @@ export class BaseAgent {
         }
         if (iteration > maxIter && execution.status === 'running') {
             execution.status = 'completed';
-            execution.result = 'Max iterations reached';
+            execution.result = `Max iterations reached (${maxIter}). The task may not be fully completed. Consider using a higher effort level or switching to /orchestrate mode for complex tasks.`;
         }
         execution.endTime = Date.now();
         execution.iterations = iteration;

package/dist/config/config.js

@@ -19,7 +19,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.7,
             maxTokens: 4096,
             tools: ['read_file', 'search_files', 'list_directory'],
-            maxIterations: 3,
+            maxIterations: 10,
         },
         coder: {
             name: 'Coder',
@@ -29,7 +29,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.4,
             maxTokens: 16384,
             tools: ['read_file', 'write_file', 'edit_file', 'search_files', 'list_directory', 'run_command', 'apply_diff'],
-            maxIterations: 15,
+            maxIterations: 50,
         },
         reviewer: {
             name: 'Reviewer',
@@ -39,7 +39,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.3,
             maxTokens: 8192,
             tools: ['read_file', 'search_files', 'list_directory'],
-            maxIterations: 5,
+            maxIterations: 15,
         },
         researcher: {
             name: 'Researcher',
@@ -49,7 +49,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.5,
             maxTokens: 8192,
             tools: ['read_file', 'search_files', 'list_directory', 'web_search', 'web_fetch'],
-            maxIterations: 8,
+            maxIterations: 20,
         },
         tester: {
             name: 'Tester',
@@ -59,7 +59,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.3,
             maxTokens: 8192,
             tools: ['read_file', 'write_file', 'search_files', 'run_command', 'list_directory'],
-            maxIterations: 10,
+            maxIterations: 25,
         },
         debugger: {
             name: 'Debugger',
@@ -69,7 +69,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.2,
             maxTokens: 8192,
             tools: ['read_file', 'edit_file', 'search_files', 'run_command', 'list_directory', 'apply_diff'],
-            maxIterations: 12,
+            maxIterations: 20,
         },
         architect: {
             name: 'Architect',
@@ -79,7 +79,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.7,
             maxTokens: 8192,
             tools: ['read_file', 'search_files', 'list_directory'],
-            maxIterations: 5,
+            maxIterations: 15,
         },
         devops: {
             name: 'DevOps',
@@ -89,7 +89,7 @@ export const DEFAULT_CONFIG = {
             temperature: 0.3,
             maxTokens: 8192,
             tools: ['read_file', 'write_file', 'edit_file', 'run_command', 'search_files', 'list_directory'],
-            maxIterations: 10,
+            maxIterations: 25,
         },
     },
     tools: {

package/dist/core/approval.js

@@ -344,27 +344,36 @@ export class ApprovalSystem {
     readline(prompt) {
         // 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();
+        if (this.mainRl) {
+            const mainInputStream = this.mainRl?.input;
+            if (mainInputStream && !mainInputStream.isPaused()) {
+                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);
-            });
+            // Use the main readline's question method directly - it handles pause/resume internally
+            if (this.mainRl) {
+                this.mainRl.question(prompt, (answer) => {
+                    // Resume the main readline after getting the answer
+                    if (this.mainRl) {
+                        this.mainRl.resume();
+                        this.mainRl.prompt();
+                    }
+                    resolve(answer);
+                });
+            }
+            else if (this.rl) {
+                this.rl.question(prompt, (answer) => {
+                    resolve(answer);
+                });
+            }
+            else {
+                const tempRl = createInterface({ input: process.stdin, output: process.stdout });
+                this.rl = tempRl;
+                tempRl.question(prompt, (answer) => {
+                    resolve(answer);
+                });
+            }
         });
     }
     close() {

package/dist/core/engine.d.ts

@@ -157,6 +157,10 @@ export declare class NeuroEngine {
      * Now delegates to ModelRouter for more sophisticated analysis
      */
     private assessComplexity;
+    /**
+     * Select the best agent for a given task category
+     */
+    private selectAgentForCategory;
     /**
      * Switch the active model
      */

package/dist/core/engine.js

@@ -537,7 +537,7 @@ Always consider the strengths of each agent when delegating:
             temperature: 0.7,
             maxTokens: 4096,
             tools: [],
-            maxIterations: 5,
+            maxIterations: 20,
         };
         this.orchestrator = new Orchestrator(orchestratorConfig, this.client, this.registry, process.cwd(), this.sessionManager.getCurrent()?.id || 'default');
         // Register all agents with orchestrator
@@ -622,7 +622,7 @@ Always consider the strengths of each agent when delegating:
         const cwd = process.cwd();
         // Built-in agents
         for (const [key, agentConfig] of Object.entries(this.config.agents)) {
-            const overrideConfig = { ...agentConfig, model: this.config.defaultModel };
+            const overrideConfig = { ...agentConfig };
             const agent = new BaseAgent(overrideConfig, this.client, this.registry, cwd, sessionId);
             this.agents.set(agentConfig.name, agent);
         }
@@ -750,9 +750,13 @@ Always consider the strengths of each agent when delegating:
         }
         else {
             const complexity = routeDecision?.complexity || this.assessComplexity(message);
+            const category = routeDecision?.category || this.modelRouter.getCategory(message);
             if (complexity === 'simple') {
-                const agent = this.agents.get('Coder');
+                // Route to the best agent based on task category
+                const targetAgentName = this.selectAgentForCategory(category);
+                const agent = this.agents.get(targetAgentName);
                 if (agent) {
+                    this.ui.info(`Using ${targetAgentName} agent (${category} task)`);
                     this.ui.startStreaming();
                     try {
                         result = await agent.run(message, callbacks);
@@ -762,16 +766,28 @@ Always consider the strengths of each agent when delegating:
                     }
                 }
                 else {
-                    // Fallback to orchestration instead of crashing
-                    this.ui.warning('Coder agent not initialized, using orchestration mode');
-                    const orchestrateResult = await this.orchestrator.orchestrate(message, callbacks);
-                    result = {
-                        content: orchestrateResult.content,
-                        toolCallsMade: 0,
-                        iterations: orchestrateResult.execution.iterations,
-                        usage: orchestrateResult.totalUsage,
-                        execution: orchestrateResult.execution,
-                    };
+                    // Fallback to Coder
+                    const coderAgent = this.agents.get('Coder');
+                    if (coderAgent) {
+                        this.ui.startStreaming();
+                        try {
+                            result = await coderAgent.run(message, callbacks);
+                        }
+                        finally {
+                            this.ui.endStreaming();
+                        }
+                    }
+                    else {
+                        this.ui.warning('No agents initialized, using orchestration mode');
+                        const orchestrateResult = await this.orchestrator.orchestrate(message, callbacks);
+                        result = {
+                            content: orchestrateResult.content,
+                            toolCallsMade: 0,
+                            iterations: orchestrateResult.execution.iterations,
+                            usage: orchestrateResult.totalUsage,
+                            execution: orchestrateResult.execution,
+                        };
+                    }
                 }
             }
             else {
@@ -905,6 +921,22 @@ Always consider the strengths of each agent when delegating:
         const decision = this.modelRouter.route(message);
         return decision.complexity;
     }
+    /**
+     * Select the best agent for a given task category
+     */
+    selectAgentForCategory(category) {
+        const categoryAgentMap = {
+            code: 'Coder',
+            reasoning: 'Architect',
+            creative: 'Coder',
+            analysis: 'Researcher',
+            conversation: 'Coder',
+            debugging: 'Debugger',
+            review: 'Reviewer',
+            refactoring: 'Coder',
+        };
+        return categoryAgentMap[category] || 'Coder';
+    }
     /**
      * Switch the active model
      */

package/dist/core/model-router.js

@@ -72,7 +72,7 @@ const COMPLEXITY_SIGNALS = {
             /\b(hello|hi|thanks|yes|no|ok|sure|done|correct|right|please)\b/i,
         ],
         lengthThreshold: 80,
-        maxSteps: 1,
+        maxSteps: 5,
     },
     moderate: {
         patterns: [
@@ -83,7 +83,7 @@ const COMPLEXITY_SIGNALS = {
             /\b(refactor|clean|simplify|optimize|restructure)\b/i,
         ],
         lengthThreshold: 300,
-        maxSteps: 3,
+        maxSteps: 15,
     },
     complex: {
         patterns: [
@@ -97,7 +97,7 @@ const COMPLEXITY_SIGNALS = {
             /\b(security\s+(audit|review|hardening)|performance\s+(optim|tuning|profiling)|scalab(ility|le))\b/i,
         ],
         lengthThreshold: 500,
-        maxSteps: 6,
+        maxSteps: 30,
     },
 };
 // ---------------------------------------------------------------------------

package/dist/index.js

@@ -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.3';
+const VERSION = '4.2.0';
 // ---- Global Error Handlers (prevent crashes) ----
 process.on('unhandledRejection', (reason) => {
     console.error(chalk.red('\n⚠️ Unhandled promise rejection:'), reason);
@@ -462,10 +462,6 @@ async function startInteractive(options) {
                 rl.prompt();
                 return;
             }
-            // Prevent input while processing (approval prompt may be active)
-            if (isProcessing) {
-                return;
-            }
             // Add to history
             completionEngine.addHistory(input);
             // Handle slash commands
@@ -937,7 +933,7 @@ async function startInteractive(options) {
                         }
                         break;
                     case 'doctor':
-                        console.log(chalk.bold('\nNeuroCLI v4.1.2 Health Check:\n'));
+                        console.log(chalk.bold('\nNeuroCLI v' + VERSION + ' 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))}`);
@@ -1139,6 +1135,11 @@ async function startInteractive(options) {
                 return;
             }
             // Process message with the engine
+            if (isProcessing) {
+                engine.ui.warning('Still processing previous request. Please wait...');
+                rl.prompt();
+                return;
+            }
             try {
                 isProcessing = true;
                 await engine.processMessage(input, currentMode, currentAgent);
@@ -1177,7 +1178,7 @@ async function startInteractive(options) {
 }
 function printHelp(engine) {
     const t = engine.ui.theme;
-    console.log(`\n  ${t.bold('NeuroCLI v4.1.2 Commands:')}\n`);
+    console.log(`\n  ${t.bold(`NeuroCLI v${VERSION} 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.3",
+  "version": "4.2.0",
   "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": {