banana-code 1.2.0 → 1.3.0

package/banana.js

@@ -21,6 +21,7 @@ const readline = require('readline');
 const path = require('path');
 const fs = require('fs');
 const os = require('os');
+const updateNotifier = require('update-notifier');
 
 const FileManager = require('./lib/fileManager');
 const ContextBuilder = require('./lib/contextBuilder');
@@ -257,6 +258,19 @@ function logSessionEnd(reason, detail = '') {
   appendDebugLog(`=== session_end ${new Date().toISOString()} reason=${reason}${detail} ===\n`);
 }
 
+function notifyIfUpdateAvailable() {
+  if (!process.stdout.isTTY) return;
+  try {
+    const notifier = updateNotifier({
+      pkg: { name: 'banana-code', version: VERSION },
+      updateCheckInterval: 1000 * 60 * 60 * 12
+    });
+    notifier.notify({ isGlobal: true, defer: false });
+  } catch {
+    // Update checks are best-effort; never fail startup.
+  }
+}
+
 // =============================================================================
 // ASCII ART & UI
 // =============================================================================
@@ -391,6 +405,8 @@ ${P}${c.yellow}/run <cmd>${c.reset}          Run a shell command
 ${P}${c.yellow}/undo${c.reset}               Show recent backups
 ${P}${c.yellow}/restore <path>${c.reset}     Restore file from backup
 ${P}${c.yellow}/commands${c.reset}           List custom commands (~/.banana/commands/)
+${P}${c.yellow}/update${c.reset}             Update Banana via npm
+
 ${P}${c.yellow}/version${c.reset}            Show version
 ${P}${c.yellow}/help${c.reset}               Show this help
 ${P}${c.yellow}/exit${c.reset}               Exit Banana
@@ -608,7 +624,8 @@ async function getActiveClient() {
 }
 
 async function syncLocalModelToLmStudio(activeModel) {
-  if (!activeModel || (activeModel.provider || 'local') !== 'local' || !activeModel.id) return;
+  if (!activeModel || activeModel.dynamic) return; // Dynamic LM Studio models are user-managed
+  if ((activeModel.provider || 'local') !== 'local' || !activeModel.id) return;
   try {
     const loaded = await lmStudio.getLoadedInstances();
     const activeLoaded = loaded.some(inst => (inst.key || inst.id || '').includes(activeModel.id));
@@ -638,14 +655,13 @@ async function checkConnection() {
   const activeModel = modelRegistry.getCurrentModel();
 
   const lmConnected = await lmStudio.isConnected();
+  await modelRegistry.refreshLmStudio();
   if (lmConnected) {
-    console.log(`${PAD}${c.green}✓${c.reset} LM Studio: Connected (${lmStudio.baseUrl})`);
-    const discovery = await modelRegistry.discover();
-    if (discovery.matched > 0) {
-      console.log(`${PAD}${c.green}✓${c.reset} Models: ${discovery.matched} matched from ${discovery.total} loaded`);
-    }
-    if (activeProvider === 'local') {
-      await syncLocalModelToLmStudio(activeModel);
+    const lmModel = modelRegistry.get('lmstudio');
+    if (lmModel && lmModel.id) {
+      console.log(`${PAD}${c.green}✓${c.reset} LM Studio: ${lmModel.name} (${lmStudio.baseUrl})`);
+    } else {
+      console.log(`${PAD}${c.yellow}○${c.reset} LM Studio: Connected, no model loaded (${lmStudio.baseUrl})`);
     }
   } else {
     const prefix = activeProvider === 'local' ? c.red : c.yellow;
@@ -1234,29 +1250,14 @@ async function switchModel(modelKey, options = {}) {
 
   const provider = switched.provider || 'local';
   if (provider === 'local') {
-    if (switched.id && (!oldModel || switched.id !== oldModel.id || (oldModel.provider || 'local') !== 'local')) {
-      try {
-        const loaded = await lmStudio.getLoadedInstances();
-        for (const inst of loaded) {
-          try {
-            await lmStudio.unloadModel(inst.instanceId);
-            if (!silent) console.log(`${PAD}${c.dim}Unloaded ${inst.displayName || inst.key}${c.reset}`);
-          } catch (err) {
-            // 404 = model already ejected by LM Studio (normal during swap)
-            if (!silent && !err.message.includes('404')) {
-              console.log(`${PAD}${c.yellow}⚠ Could not unload ${inst.key}: ${err.message}${c.reset}`);
-            }
-          }
-        }
-        const ctxLen = switched.contextLimit || 32768;
-        if (!silent) console.log(`${PAD}${c.dim}Loading ${switched.name} in LM Studio (ctx: ${(ctxLen / 1024).toFixed(0)}K)...${c.reset}`);
-        const result = await lmStudio.loadModel(switched.id, { contextLength: ctxLen });
-        if (!silent) console.log(`${PAD}${c.green}✓ Loaded in ${result.load_time_seconds?.toFixed(1)}s${c.reset}`);
-      } catch (err) {
-        if (!silent) {
-          console.log(`${PAD}${c.yellow}⚠ Could not auto-load: ${err.message}${c.reset}`);
-          console.log(`${PAD}${c.dim}Load it manually in LM Studio${c.reset}`);
-        }
+    if (switched.dynamic) {
+      // Dynamic LM Studio model - refresh to get latest loaded model info
+      await modelRegistry.refreshLmStudio();
+      const refreshed = modelRegistry.get('lmstudio');
+      if (refreshed && refreshed.id) {
+        if (!silent) console.log(`${PAD}${c.green}✓ Using ${refreshed.name}${c.reset}`);
+      } else {
+        if (!silent) console.log(`${PAD}${c.yellow}⚠ LM Studio connected but no model loaded${c.reset}`);
       }
     }
     if (switched.tags?.includes('vision') && !silent) {
@@ -1372,7 +1373,7 @@ async function handleCommand(input) {
       console.log(`\n${PAD}${c.cyan}Checking for updates...${c.reset}`);
       const { execSync } = require('child_process');
       try {
-        const result = execSync('npm install -g mrchevyceleb/banana-code', {
+        const result = execSync('npm install -g banana-code@latest', {
           encoding: 'utf-8',
           timeout: 60000,
           stdio: ['pipe', 'pipe', 'pipe']
@@ -2004,7 +2005,8 @@ async function handleCommand(input) {
       const modelArg = modelArgRaw.toLowerCase();
 
       if (!modelArg) {
-        // Interactive model picker
+        // Refresh LM Studio state before showing picker
+        await modelRegistry.refreshLmStudio();
         const models = modelRegistry.list();
         const current = modelRegistry.getCurrent();
         const pickerItems = models.map(m => ({
@@ -2793,12 +2795,19 @@ async function sendStreamingMessage(message, images = [], rawMessage = '') {
       if (_renderingWorkPrompt) return; // prevent re-entrancy cascade
       _renderingWorkPrompt = true;
       try {
-        // rl.prompt(false) redraws the prompt + whatever is already in rl.line.
-        // Do NOT call rl.write(savedLine) here - that would re-inject the text
-        // through readline's input path, doubling it in the buffer and on screen.
+        // rl.prompt(false) redraws the prompt + whatever is already in rl.line,
+        // but always places cursor at end. Save/restore cursor position so the
+        // user can keep typing mid-line without the cursor jumping.
+        const savedCursor = rl.cursor;
         process.stdout.write(`${fitToTerminal(getStreamSpinnerText())}\n\n`);
         rl.setPrompt(buildPromptPrefix());
         rl.prompt(false);
+        if (savedCursor < rl.line.length) {
+          // Move cursor back from end-of-line to saved position
+          const moveBack = rl.line.length - savedCursor;
+          process.stdout.write(`\x1b[${moveBack}D`);
+          rl.cursor = savedCursor;
+        }
         if (statusBar) statusBar.render();
       } finally {
         _renderingWorkPrompt = false;
@@ -2900,6 +2909,11 @@ async function sendStreamingMessage(message, images = [], rawMessage = '') {
   startThinking();
   if (statusBar) statusBar.startTiming();
 
+  // Refresh LM Studio model info if active (picks up model swaps between messages)
+  if (modelRegistry.getCurrentModel()?.dynamic) {
+    await modelRegistry.refreshLmStudio();
+  }
+
   const streamInferenceSettings = modelRegistry.getInferenceSettings();
   const activeClient = await getActiveClient();
   const streamModelMeta = modelRegistry.getCurrentModel();
@@ -3557,12 +3571,18 @@ async function sendAgenticMessage(message, images = [], rawMessage = '') {
       if (_renderingWorkPrompt) return; // prevent re-entrancy cascade
       _renderingWorkPrompt = true;
       try {
-        // rl.prompt(false) redraws the prompt + whatever is already in rl.line.
-        // Do NOT call rl.write(savedLine) here - that would re-inject the text
-        // through readline's input path, doubling it in the buffer and on screen.
+        // rl.prompt(false) redraws the prompt + whatever is already in rl.line,
+        // but always places cursor at end. Save/restore cursor position so the
+        // user can keep typing mid-line without the cursor jumping.
+        const savedCursor = rl.cursor;
         process.stdout.write(`${fitToTerminal(getSpinnerText())}\n\n`);
         rl.setPrompt(buildPromptPrefix());
         rl.prompt(false);
+        if (savedCursor < rl.line.length) {
+          const moveBack = rl.line.length - savedCursor;
+          process.stdout.write(`\x1b[${moveBack}D`);
+          rl.cursor = savedCursor;
+        }
         if (statusBar) statusBar.render();
       } finally {
         _renderingWorkPrompt = false;
@@ -3923,11 +3943,16 @@ async function sendAgenticMessage(message, images = [], rawMessage = '') {
             if (_renderingWorkPrompt) return; // prevent re-entrancy cascade
             _renderingWorkPrompt = true;
             try {
-              // rl.prompt(false) redraws the prompt + whatever is already in rl.line.
-              // Do NOT call rl.write(savedLine) here - that would re-inject the text
-              // through readline's input path, doubling it in the buffer and on screen.
+              // rl.prompt(false) redraws the prompt + whatever is already in rl.line,
+              // but always places cursor at end. Save/restore cursor position.
+              const savedCursor = rl.cursor;
               process.stdout.write(`${fitToTerminal(getSpinnerText())}\n`);
               rl.prompt(false);
+              if (savedCursor < rl.line.length) {
+                const moveBack = rl.line.length - savedCursor;
+                process.stdout.write(`\x1b[${moveBack}D`);
+                rl.cursor = savedCursor;
+              }
               if (statusBar) statusBar.render();
             } finally {
               _renderingWorkPrompt = false;
@@ -4005,6 +4030,11 @@ async function sendAgenticMessage(message, images = [], rawMessage = '') {
     });
     currentRunner = runner;
 
+    // Refresh LM Studio model info if active (picks up model swaps between messages)
+    if (modelRegistry.getCurrentModel()?.dynamic) {
+      await modelRegistry.refreshLmStudio();
+    }
+
     // Use model-specific inference settings
     currentAbortController = new AbortController();
     const inferenceSettings = modelRegistry.getInferenceSettings();
@@ -4949,6 +4979,7 @@ Examples:
   // Show banner and initialize
   await showBanner();
   initProject();
+  notifyIfUpdateAvailable();
   if (runtimeLogPath) {
     console.log(`${PAD}${c.dim}Logs: ${runtimeLogPath}${c.reset}`);
   }

package/lib/agenticRunner.js

@@ -1304,6 +1304,16 @@ class AgenticRunner {
     const failedMcpTools = new Set(); // Track MCP tools that returned "Unknown tool" errors
     let readOnlyStreak = 0; // Consecutive iterations with only read-only tool calls
     let loopWarningCount = 0; // How many times loop detection has fired
+
+    // Model-tier-aware read-only thresholds: smarter models get more research leeway
+    // options.model is the raw model ID (e.g. "claude-sonnet-4-6-20250514", "gpt-4o", "silverback")
+    const modelId = (options.model || '').toLowerCase();
+    const isFrontier = /claude|gpt-4|gpt-5|o[1-9]-|o[1-9]$|chatgpt/.test(modelId) || modelId.includes('silverback');
+    const readOnlyThresholds = isFrontier
+      ? { nudge: 8, escalate: 10, hardStop: 12 }
+      : (modelId.includes('tamarin'))
+        ? { nudge: 3, escalate: 5, hardStop: 6 }
+        : /* mandrill, local, openrouter, or unknown */ { nudge: 5, escalate: 7, hardStop: 10 };
     const writtenFiles = [];
     this._lastWrittenFiles = null;
     this._pendingSteers = [];
@@ -1473,6 +1483,14 @@ class AgenticRunner {
       // Some models use finish_reason "tool_calls", others use "stop" or "function_call"
       // but still include tool_calls in the message. Check for the array itself.
       if (assistantMessage.tool_calls && assistantMessage.tool_calls.length > 0) {
+        // Cap tool calls per response to prevent runaway models spamming dozens of calls
+        const MAX_TOOL_CALLS_PER_RESPONSE = 8;
+        if (assistantMessage.tool_calls.length > MAX_TOOL_CALLS_PER_RESPONSE) {
+          appendDebugLog(`  [TOOL CALL CAP] Model returned ${assistantMessage.tool_calls.length} tool calls, capping to ${MAX_TOOL_CALLS_PER_RESPONSE}\n`);
+          this.onWarning(`Model tried to make ${assistantMessage.tool_calls.length} tool calls at once. Capping to ${MAX_TOOL_CALLS_PER_RESPONSE}.`);
+          assistantMessage.tool_calls = assistantMessage.tool_calls.slice(0, MAX_TOOL_CALLS_PER_RESPONSE);
+        }
+
         // Add assistant message to history, preserving the reasoning field
         // GPT-OSS requires the reasoning/CoT from prior tool calls to be passed back.
         // Other models ignore it harmlessly.
@@ -1663,24 +1681,36 @@ class AgenticRunner {
         }
 
         // Track read-only streaks (iterations with no writes or commands)
+        // Skip streak tracking in plan mode - plan mode is inherently read-only
         const thisIterToolNames = assistantMessage.tool_calls.map(t => t.function.name);
-        const hadWriteAction = thisIterToolNames.some(n => WRITE_TOOL_NAMES.has(n));
-        if (hadWriteAction) {
-          readOnlyStreak = 0;
-        } else {
-          readOnlyStreak++;
+        if (!options.readOnly) {
+          const hadWriteAction = thisIterToolNames.some(n => WRITE_TOOL_NAMES.has(n));
+          if (hadWriteAction) {
+            readOnlyStreak = 0;
+          } else {
+            readOnlyStreak++;
+          }
         }
 
-        // Read-only streak nudge: after 5 consecutive read-only iterations, nudge to act
-        if (readOnlyStreak === 5) {
-          appendDebugLog(`  [READ-ONLY NUDGE] ${readOnlyStreak} consecutive read-only iterations\n`);
-          this.onWarning('5 consecutive read-only iterations. Nudging to start implementing.');
+        // Read-only streak nudges: escalating pressure, thresholds vary by model tier
+        if (readOnlyStreak === readOnlyThresholds.nudge) {
+          appendDebugLog(`  [READ-ONLY NUDGE] ${readOnlyStreak} consecutive read-only iterations (threshold: ${readOnlyThresholds.nudge})\n`);
+          this.onWarning(`${readOnlyStreak} consecutive read-only iterations. Nudging to start implementing.`);
           messages.push({
             role: 'system',
-            content: 'NOTICE: You have done 5 consecutive read/search iterations without writing any code or running commands. ' +
+            content: `NOTICE: You have done ${readOnlyStreak} consecutive read/search iterations without writing any code or running commands. ` +
               'You likely have enough context now. Start implementing changes. ' +
               'If you are genuinely blocked, use ask_human to tell the user what you need.'
           });
+        } else if (readOnlyStreak === readOnlyThresholds.escalate) {
+          appendDebugLog(`  [READ-ONLY ESCALATION] ${readOnlyStreak} consecutive read-only iterations (threshold: ${readOnlyThresholds.escalate})\n`);
+          this.onWarning(`${readOnlyStreak} consecutive read-only iterations. You MUST start writing code now.`);
+          messages.push({
+            role: 'system',
+            content: `URGENT: You have done ${readOnlyStreak} consecutive read/search iterations. You are over-researching. ` +
+              'STOP reading files. STOP searching. Write code NOW using create_file or edit_file. ' +
+              'If you cannot proceed, use ask_human immediately. Do NOT call read_file or search_code again.'
+          });
         }
 
         // Loop detection: inject nudge or hard-break after all tool results
@@ -1738,8 +1768,8 @@ class AgenticRunner {
           }
         }
 
-        // Hard break if 8+ consecutive read-only iterations with no progress
-        if (readOnlyStreak >= 8) {
+        // Hard break based on model tier threshold
+        if (readOnlyStreak >= readOnlyThresholds.hardStop) {
           appendDebugLog(`  [NO-PROGRESS BREAKER] ${readOnlyStreak} consecutive read-only iterations\n`);
           this.onWarning(`No write actions for ${readOnlyStreak} consecutive iterations. Stopping.`);
           messages.push({

package/lib/interactivePicker.js

@@ -1,254 +1,258 @@
-/**
- * Interactive arrow-key picker for Banana Code
- *
- * Renders a navigable list in the terminal. Arrow keys move highlight,
- * Enter selects, Escape cancels. Temporarily takes over stdin.
- */
-
-const readline = require('readline');
-
-const ESC = '\x1b';
-const HIDE_CURSOR = `${ESC}[?25l`;
-const SHOW_CURSOR = `${ESC}[?25h`;
-const CLEAR_LINE = `${ESC}[2K`;
-const MOVE_UP = (n) => n > 0 ? `${ESC}[${n}A` : '';
-const MOVE_COL0 = `${ESC}[0G`;
-
-// 256-color codes matching diffViewer.js palette
-const DIM = `${ESC}[2m`;
-const RESET = `${ESC}[0m`;
-const GREEN = `${ESC}[38;5;120m`;
-const YELLOW = `${ESC}[38;5;226m`;
-const CYAN = `${ESC}[38;5;51m`;
-const GRAY = `${ESC}[38;5;245m`;
-const INVERSE = `${ESC}[7m`;
-
-// Flag indicating a picker UI is active. When true, the global keypress
-// handler in banana.js should not process keys (arrow keys, escape, etc.)
-// because the picker owns stdin during its lifetime.
-let _pickerActive = false;
-function isPickerActive() { return _pickerActive; }
-
-/**
- * Show an interactive picker menu.
- *
- * @param {Array} items - Array of { key, label, description, active, tags }\r\n * @param {Object} options\r\n * @param {string} options.title - Header text\r\n * @param {number} options.selected - Initial selected index (default: active item or 0)\r\n * @returns {Promise<<ObjectObject|null>} - Selected item, or null if cancelled\r\n */
-function pick(items, options = {}) {
-  return new Promise((resolve) => {
-    if (!items || items.length === 0) {
-      resolve(null);
-      return;
-    }
-
-    let selected = options.selected ?? items.findIndex(i => i.active);
-    if (selected <<  0) selected = 0;
-
-    const out = process.stdout;
-    const title = options.title || 'Select:';
-    const showVisionIndicator = options.showVisionIndicator === true;
-    const footerText = showVisionIndicator
-      ? `${DIM}  ↑↓ navigate  Enter select  Esc cancel  V=vision${RESET}`
-      : `${DIM}  ↑↓ navigate  Enter select  Esc cancel${RESET}`;
-
-    // Pad key names to align descriptions
-    const maxKeyLen = Math.max(...items.map(i => (i.key || '').length));
-
-    function formatItem(item, index) {
-      const isSel = index === selected;
-      const isActive = item.active;
-      const marker = isActive ? `${GREEN}●${RESET}` : `${GRAY}○${RESET}`;
-      const pointer = isSel ? `${CYAN}▸${RESET} ` : '  ';
-      const hasVision = Array.isArray(item.tags) && item.tags.includes('vision');
-      const vision = showVisionIndicator
-        ? (hasVision ? `${CYAN}V${RESET}` : `${DIM}.${RESET}`)
-        : '';
-      const key = (item.key || '').padEnd(maxKeyLen);
-      const tags = item.tags?.length ? ` ${DIM}[${item.tags.join(', ')}]${RESET}` : '';
-      const desc = item.description ? ` ${DIM}- ${item.description}${RESET}` : '';
-      const label = item.label || item.name || item.key;
-
-      if (isSel) {
-        const lead = showVisionIndicator ? `${pointer}${marker} ${vision} ` : `${pointer}${marker} `;
-        return `${lead}${INVERSE} ${YELLOW}${key}${RESET}${INVERSE}  ${label} ${RESET}${tags}`;
-      }
-      const lead = showVisionIndicator ? `${pointer}${marker} ${vision} ` : `${pointer}${marker} `;
-      return `${lead}${YELLOW}${key}${RESET}  ${DIM}${label}${RESET}${tags}`;
-    }
-
-    // Total lines we render (title + items + footer)
-    const totalLines = 1 + items.length + 1;
-
-    function render() {
-      _pickerActive = true;
-      let buf = HIDE_CURSOR;
-      buf += `\n${CYAN}  ${title}${RESET}\n`;
-      for (let i = 0; i << items items.length; i++) {
-        buf += `  ${formatItem(items[i], i)}\n`;
-      }
-      buf += `${footerText}\n`;
-      out.write(buf);
-    }
-
-    function redraw() {
-      // Move up to the start of the picker area
-      let buf = MOVE_UP(totalLines) + MOVE_COL0;
-      // Clear all lines in the area
-      for (let i = 0; i << total totalLines; i++) {
-        buf += CLEAR_LINE + '\n';
-      }
-      // Move back up to the start to re-render
-      buf += MOVE_UP(totalLines) + MOVE_COL0;
-      buf += `${CYAN}  ${title}${RESET}\n`;
-      for (let i = 0; i << items items.length; i++) {
-        buf += `  ${formatItem(items[i], i)}\n`;
-      }
-      buf += `${footerText}\n`;
-      out.write(buf);
-    }
-
-    function cleanup() {
-      _pickerActive = false;
-      process.stdin.removeListener('keypress', onKey);
-      out.write(SHOW_CURSOR);
-    }
-
-    function onKey(str, key) {
-      if (!key) return;
-      try {
-        if (key.name === 'up') {
-          selected = (selected - 1 + items.length) % items.length;
-          redraw();
-        } else if (key.name === 'down') {
-          selected = (selected + 1) % items.length;
-          redraw();
-        } else if (key.name === 'return') {
-          cleanup();
-          resolve(items[selected]);
-        } else if (key.name === 'escape') {
-          cleanup();
-          resolve(null);
-        }
-      } catch (err) {
-        cleanup();
-        resolve(null);
-      }
-    }
-
-    // Ensure keypress events are flowing
-    if (!process.stdin.listenerCount('keypress')) {
-      readline.emitKeypressEvents(process.stdin);
-    }
-
-    render();
-    process.stdin.on('keypress', onKey);
-  });
-}
-
-/**
- * Interactive toggle list. Items can be toggled on/off with Space.
- * Arrow keys navigate, Space toggles, Escape exits.
- *
- * @param {Array} items - Array of { key, label, description, enabled, meta }\r\n * @param {Object} options\r\n * @param {string} options.title - Header text\r\n * @param {Function} options.onToggle - Called with (item, newEnabled) when toggled. Should return true if toggle succeeded.\r\n * @returns {Promise<<voidvoid>} - Resolves when user presses Escape\r\n */
-function pickToggle(items, options = {}) {
-  return new Promise((resolve) => {
-    if (!items || items.length === 0) {
-      resolve();
-      return;
-    }
-
-    let selected = 0;
-    const out = process.stdout;
-    const title = options.title || 'Toggle:';
-    const onToggle = options.onToggle || (() => true);
-    const RED = `${ESC}[38;5;203m`;
-    const footerText = `${DIM}  ↑↓ navigate  Space toggle  Enter/Esc done${RESET}`;
-
-    const maxKeyLen = Math.max(...items.map(i => (i.key || '').length));
-
-    function formatItem(item, index) {
-      const isSel = index === selected;
-      const pointer = isSel ? `${CYAN}▸${RESET} ` : '  ';
-      const toggle = item.enabled ? `${GREEN}on ${RESET}` : `${RED}off${RESET}`;
-      const key = (item.key || '').padEnd(maxKeyLen);
-      const desc = item.description ? ` ${DIM}${item.description}${RESET}` : '';
-      const meta = item.meta ? ` ${DIM}[${item.meta}]${RESET}` : '';
-
-      if (isSel) {
-        return `${pointer}[${toggle}] ${INVERSE} ${YELLOW}${key}${RESET}${INVERSE} ${RESET}${desc}${meta}`;
-      }
-      return `${pointer}[${toggle}] ${YELLOW}${key}${RESET}${desc}${meta}`;
-    }
-
-    const totalLines = 1 + items.length + 1;
-
-    function render() {
-      _pickerActive = true;
-      let buf = HIDE_CURSOR;
-      buf += `\n${CYAN}  ${title}${RESET}\n`;
-      for (let i = 0; i << items items.length; i++) {
-        buf += `  ${formatItem(items[i], i)}\n`;
-      }
-      buf += `${footerText}\n`;
-      out.write(buf);
-    }
-
-    function redraw() {
-      // Move up to the start of the toggle area
-      let buf = MOVE_UP(totalLines) + MOVE_COL0;
-      // Clear all lines in the area
-      for (let i = 0; i << total totalLines; i++) {
-        buf += CLEAR_LINE + '\n';
-      }
-      // Move back up to the start to re-render
-      buf += MOVE_UP(totalLines) + MOVE_COL0;
-      buf += `${CYAN}  ${title}${RESET}\n`;
-      for (let i = 0; i << items items.length; i++) {
-        buf += `  ${formatItem(items[i], i)}\n`;
-      }
-      buf += `${footerText}\n`;
-      out.write(buf);
-    }
-
-    function cleanup() {
-      _pickerActive = false;
-      process.stdin.removeListener('keypress', onKey);
-      out.write(SHOW_CURSOR);
-    }
-
-    function onKey(str, key) {
-      if (!key) return;
-      try {
-        if (key.name === 'up') {
-          selected = (selected - 1 + items.length) % items.length;
-          redraw();
-        } else if (key.name === 'down') {
-          selected = (selected + 1) % items.length;
-          redraw();
-        } else if (key.name === 'space') {
-          const item = items[selected];
-          const newState = !item.enabled;
-          const success = onToggle(item, newState);
-          if (success !== false) {
-            item.enabled = newState;
-          }
-          redraw();
-        } else if (key.name === 'escape' || key.name === 'return') {
-          cleanup();
-          resolve();
-        }
-      } catch (err) {
-        cleanup();
-        resolve();
-      }
-    }
-
-    if (!process.stdin.listenerCount('keypress')) {
-      readline.emitKeypressEvents(process.stdin);
-    }
-
-    render();
-    process.stdin.on('keypress', onKey);
-  });
-}
-
-module.exports = { pick, pickToggle, isPickerActive };
+/**
+ * Interactive arrow-key picker for Banana Code
+ *
+ * Renders a navigable list in the terminal. Arrow keys move highlight,
+ * Enter selects, Escape cancels. Temporarily takes over stdin.
+ */
+
+const readline = require('readline');
+
+const ESC = '\x1b';
+const HIDE_CURSOR = `${ESC}[?25l`;
+const SHOW_CURSOR = `${ESC}[?25h`;
+const CLEAR_LINE = `${ESC}[2K`;
+const MOVE_UP = (n) => n > 0 ? `${ESC}[${n}A` : '';
+const MOVE_COL0 = `${ESC}[0G`;
+
+// 256-color codes matching diffViewer.js palette
+const DIM = `${ESC}[2m`;
+const RESET = `${ESC}[0m`;
+const GREEN = `${ESC}[38;5;120m`;
+const YELLOW = `${ESC}[38;5;226m`;
+const CYAN = `${ESC}[38;5;51m`;
+const GRAY = `${ESC}[38;5;245m`;
+const INVERSE = `${ESC}[7m`;
+
+// Flag indicating a picker UI is active. When true, the global keypress
+// handler in banana.js should not process keys (arrow keys, escape, etc.)
+// because the picker owns stdin during its lifetime.
+let _pickerActive = false;
+function isPickerActive() { return _pickerActive; }
+
+/**
+ * Show an interactive picker menu.
+ *
+ * @param {Array} items - Array of { key, label, description, active, tags }
+ * @param {Object} options
+ * @param {string} options.title - Header text
+ * @param {number} options.selected - Initial selected index (default: active item or 0)
+ * @returns {Promise<Object|null>} - Selected item, or null if cancelled
+ */
+function pick(items, options = {}) {
+  return new Promise((resolve) => {
+    if (!items || items.length === 0) {
+      resolve(null);
+      return;
+    }
+
+    let selected = options.selected ?? items.findIndex(i => i.active);
+    if (selected < 0) selected = 0;
+
+    const out = process.stdout;
+    const title = options.title || 'Select:';
+    const showVisionIndicator = options.showVisionIndicator === true;
+    const footerText = showVisionIndicator
+      ? `${DIM}  ↑↓ navigate  Enter select  Esc cancel  V=vision${RESET}`
+      : `${DIM}  ↑↓ navigate  Enter select  Esc cancel${RESET}`;
+
+    // Pad key names to align descriptions
+    const maxKeyLen = Math.max(...items.map(i => (i.key || '').length));
+
+    function formatItem(item, index) {
+      const isSel = index === selected;
+      const isActive = item.active;
+      const marker = isActive ? `${GREEN}●${RESET}` : `${GRAY}○${RESET}`;
+      const pointer = isSel ? `${CYAN}▸${RESET} ` : '  ';
+      const hasVision = Array.isArray(item.tags) && item.tags.includes('vision');
+      const vision = showVisionIndicator
+        ? (hasVision ? `${CYAN}V${RESET}` : `${DIM}.${RESET}`)
+        : '';
+      const key = (item.key || '').padEnd(maxKeyLen);
+      const tags = item.tags?.length ? ` ${DIM}[${item.tags.join(', ')}]${RESET}` : '';
+      const desc = item.description ? ` ${DIM}- ${item.description}${RESET}` : '';
+      const label = item.label || item.name || item.key;
+
+      if (isSel) {
+        const lead = showVisionIndicator ? `${pointer}${marker} ${vision} ` : `${pointer}${marker} `;
+        return `${lead}${INVERSE} ${YELLOW}${key}${RESET}${INVERSE}  ${label} ${RESET}${tags}`;
+      }
+      const lead = showVisionIndicator ? `${pointer}${marker} ${vision} ` : `${pointer}${marker} `;
+      return `${lead}${YELLOW}${key}${RESET}  ${DIM}${label}${RESET}${tags}`;
+    }
+
+    // Total lines we render (title + items + footer)
+    const totalLines = 1 + items.length + 1;
+
+    function render() {
+      _pickerActive = true;
+      let buf = HIDE_CURSOR;
+      buf += `\n${CYAN}  ${title}${RESET}\n`;
+      for (let i = 0; i < items.length; i++) {
+        buf += `  ${formatItem(items[i], i)}\n`;
+      }
+      buf += `${footerText}\n`;
+      out.write(buf);
+    }
+
+    function redraw() {
+      let buf = MOVE_UP(totalLines) + MOVE_COL0;
+      for (let i = 0; i < totalLines; i++) {
+        buf += CLEAR_LINE + '\n';
+      }
+      buf += MOVE_UP(totalLines) + MOVE_COL0;
+      buf += `${CYAN}  ${title}${RESET}\n`;
+      for (let i = 0; i < items.length; i++) {
+        buf += `  ${formatItem(items[i], i)}\n`;
+      }
+      buf += `${footerText}\n`;
+      out.write(buf);
+    }
+
+    function cleanup() {
+      _pickerActive = false;
+      process.stdin.removeListener('keypress', onKey);
+      out.write(SHOW_CURSOR);
+    }
+
+    function onKey(str, key) {
+      if (!key) return;
+      try {
+        if (key.name === 'up') {
+          selected = (selected - 1 + items.length) % items.length;
+          redraw();
+        } else if (key.name === 'down') {
+          selected = (selected + 1) % items.length;
+          redraw();
+        } else if (key.name === 'return') {
+          cleanup();
+          resolve(items[selected]);
+        } else if (key.name === 'escape') {
+          cleanup();
+          resolve(null);
+        }
+      } catch (err) {
+        cleanup();
+        resolve(null);
+      }
+    }
+
+    // Ensure keypress events are flowing
+    if (!process.stdin.listenerCount('keypress')) {
+      readline.emitKeypressEvents(process.stdin);
+    }
+
+    render();
+    process.stdin.on('keypress', onKey);
+  });
+}
+
+/**
+ * Interactive toggle list. Items can be toggled on/off with Space.
+ * Arrow keys navigate, Space toggles, Escape exits.
+ *
+ * @param {Array} items - Array of { key, label, description, enabled, meta }
+ * @param {Object} options
+ * @param {string} options.title - Header text
+ * @param {Function} options.onToggle - Called with (item, newEnabled) when toggled. Should return true if toggle succeeded.
+ * @returns {Promise<void>} - Resolves when user presses Escape
+ */
+function pickToggle(items, options = {}) {
+  return new Promise((resolve) => {
+    if (!items || items.length === 0) {
+      resolve();
+      return;
+    }
+
+    let selected = 0;
+    const out = process.stdout;
+    const title = options.title || 'Toggle:';
+    const onToggle = options.onToggle || (() => true);
+    const RED = `${ESC}[38;5;203m`;
+    const footerText = `${DIM}  ↑↓ navigate  Space toggle  Enter/Esc done${RESET}`;
+
+    const maxKeyLen = Math.max(...items.map(i => (i.key || '').length));
+
+    function formatItem(item, index) {
+      const isSel = index === selected;
+      const pointer = isSel ? `${CYAN}▸${RESET} ` : '  ';
+      const toggle = item.enabled ? `${GREEN}on ${RESET}` : `${RED}off${RESET}`;
+      const key = (item.key || '').padEnd(maxKeyLen);
+      const desc = item.description ? ` ${DIM}${item.description}${RESET}` : '';
+      const meta = item.meta ? ` ${DIM}[${item.meta}]${RESET}` : '';
+
+      if (isSel) {
+        return `${pointer}[${toggle}] ${INVERSE} ${YELLOW}${key}${RESET}${INVERSE} ${RESET}${desc}${meta}`;
+      }
+      return `${pointer}[${toggle}] ${YELLOW}${key}${RESET}${desc}${meta}`;
+    }
+
+    const totalLines = 1 + items.length + 1;
+
+    function render() {
+      _pickerActive = true;
+      let buf = HIDE_CURSOR;
+      buf += `\n${CYAN}  ${title}${RESET}\n`;
+      for (let i = 0; i < items.length; i++) {
+        buf += `  ${formatItem(items[i], i)}\n`;
+      }
+      buf += `${footerText}\n`;
+      out.write(buf);
+    }
+
+    function redraw() {
+      let buf = MOVE_UP(totalLines) + MOVE_COL0;
+      for (let i = 0; i < totalLines; i++) {
+        buf += CLEAR_LINE + '\n';
+      }
+      buf += MOVE_UP(totalLines) + MOVE_COL0;
+      buf += `${CYAN}  ${title}${RESET}\n`;
+      for (let i = 0; i < items.length; i++) {
+        buf += `  ${formatItem(items[i], i)}\n`;
+      }
+      buf += `${footerText}\n`;
+      out.write(buf);
+    }
+
+    function cleanup() {
+      _pickerActive = false;
+      process.stdin.removeListener('keypress', onKey);
+      out.write(SHOW_CURSOR);
+    }
+
+    function onKey(str, key) {
+      if (!key) return;
+      try {
+        if (key.name === 'up') {
+          selected = (selected - 1 + items.length) % items.length;
+          redraw();
+        } else if (key.name === 'down') {
+          selected = (selected + 1) % items.length;
+          redraw();
+        } else if (key.name === 'space') {
+          const item = items[selected];
+          const newState = !item.enabled;
+          const success = onToggle(item, newState);
+          if (success !== false) {
+            item.enabled = newState;
+          }
+          redraw();
+        } else if (key.name === 'escape' || key.name === 'return') {
+          cleanup();
+          resolve();
+        }
+      } catch (err) {
+        cleanup();
+        resolve();
+      }
+    }
+
+    if (!process.stdin.listenerCount('keypress')) {
+      readline.emitKeypressEvents(process.stdin);
+    }
+
+    render();
+    process.stdin.on('keypress', onKey);
+  });
+}
+
+module.exports = { pick, pickToggle, isPickerActive };

package/lib/lmStudio.js

@@ -28,6 +28,7 @@ class LmStudio {
     if (options.tools) {
       body.tools = options.tools;
       body.tool_choice = options.toolChoice ?? 'auto';
+      body.parallel_tool_calls = false; // Prevent local models from spamming dozens of tool calls
     }
     if (typeof options.thinking === 'number' && options.thinking > 0) {
       body.thinking = { type: 'enabled', budget_tokens: options.thinking };
@@ -143,7 +144,7 @@ class LmStudio {
 
   /**
    * Get all loaded model instances from LM Studio REST API.
-   * Returns array of { key, instanceId, displayName }
+   * Returns array of { key, instanceId, displayName, contextLength }
    */
   async getLoadedInstances() {
     try {
@@ -155,7 +156,12 @@ class LmStudio {
       const instances = [];
       for (const model of data.models || []) {
         for (const inst of model.loaded_instances || []) {
-          instances.push({ key: model.key, instanceId: inst.id, displayName: model.display_name });
+          instances.push({
+            key: model.key,
+            instanceId: inst.id,
+            displayName: model.display_name || model.key,
+            contextLength: inst.context_length || null
+          });
         }
       }
       return instances;

package/lib/mcpClient.js

@@ -76,7 +76,7 @@ class McpClient {
           params: {
             protocolVersion: '2024-11-05',
             capabilities: {},
-            clientInfo: { name: 'banana', version: '1.2.0' }
+            clientInfo: { name: 'banana', version: '1.3.0' }
           }
         }),
         signal: AbortSignal.timeout(this.timeout)

package/lib/modelRegistry.js

@@ -215,66 +215,52 @@ class ModelRegistry {
   }
 
   /**
-   * Auto-discover local model IDs from LM Studio and update local registry
+   * Probe LM Studio and inject/update a dynamic "lmstudio" entry in the registry.
+   * If LM Studio is not connected or has no model loaded, removes the entry.
    */
-  async discover() {
-    if (!this.lmStudio) return { matched: 0, total: 0 };
+  async refreshLmStudio() {
+    if (!this.lmStudio) return;
 
-    const lmModels = await this.lmStudio.listModels();
-    if (lmModels.length === 0) return { matched: 0, total: 0 };
-
-    let matched = 0;
-    const usedLmIds = new Set();
-    for (const [key, model] of Object.entries(this.localRegistry.models)) {
-      if ((model.provider || 'local') !== 'local') continue;
-
-      // Do not overwrite explicit model IDs from models.json.
-      const existingId = String(model.id || '').trim();
-      if (existingId) continue;
-
-      const keywords = this._getMatchKeywords(key, model.name);
-      for (const lmModel of lmModels) {
-        const lmId = (lmModel.id || '').toLowerCase();
-        if (!lmId || usedLmIds.has(lmId)) continue;
-        if (keywords.some(kw => lmId.includes(kw))) {
-          model.id = lmModel.id;
-          usedLmIds.add(lmId);
-          matched++;
-          break;
-        }
-      }
+    const connected = await this.lmStudio.isConnected();
+    if (!connected) {
+      delete this.registry.models['lmstudio'];
+      return;
     }
 
-    if (matched > 0) {
-      this.save();
-      this.refreshRemoteModels();
+    const instances = await this.lmStudio.getLoadedInstances();
+    const first = instances[0];
+
+    if (first && first.key) {
+      this.registry.models['lmstudio'] = {
+        name: `LM Studio: ${first.displayName}`,
+        id: first.key,
+        provider: 'local',
+        contextLimit: first.contextLength || 32768,
+        maxOutputTokens: Math.min(first.contextLength || 32768, 32768),
+        supportsThinking: false,
+        prompt: 'code-agent',
+        inferenceSettings: { temperature: 0.7, topP: 0.9 },
+        tags: ['local', 'lm-studio'],
+        tier: 'local',
+        description: `${first.displayName} via LM Studio`,
+        dynamic: true
+      };
+    } else {
+      // Connected but no model loaded
+      this.registry.models['lmstudio'] = {
+        name: 'LM Studio (no model loaded)',
+        id: '',
+        provider: 'local',
+        contextLimit: 32768,
+        supportsThinking: false,
+        prompt: 'code-agent',
+        inferenceSettings: { temperature: 0.7, topP: 0.9 },
+        tags: ['local', 'lm-studio'],
+        tier: 'local',
+        description: 'Load a model in LM Studio to use',
+        dynamic: true
+      };
     }
-
-    return { matched, total: lmModels.length };
-  }
-
-  /**
-   * Generate match keywords from local model key and name
-   */
-  _getMatchKeywords(key, name) {
-    const keywords = [];
-    const keyMap = {
-      'silverback': ['silverback', 'gpt_oss'],
-      'qwen35': ['qwen3.5-35b', 'qwen3.5-35'],
-      'nemotron': ['nemotron'],
-      'coder': ['qwen3-coder-30b', 'qwen3-coder-30'],
-      'glm': ['glm-4.7', 'glm-4-flash', 'glm4'],
-      'max': ['qwen3-coder-next', 'coder-next'],
-      'mistral': ['mistral-small']
-    };
-    if (keyMap[key]) keywords.push(...keyMap[key]);
-
-    if (typeof name === 'string' && name.trim()) {
-      const parts = name.toLowerCase().split(/[^a-z0-9]+/).filter(Boolean);
-      keywords.push(...parts.slice(0, 3));
-    }
-
-    return [...new Set(keywords)];
   }
 
   /**

package/models.json

@@ -27,19 +27,6 @@
       "description": "Balanced. Everyday coding, solid quality + speed.",
       "provider": "monkey"
     },
-    "gibbon": {
-      "name": "Gibbon",
-      "id": "gibbon",
-      "contextLimit": 262000,
-      "maxOutputTokens": 65536,
-      "supportsThinking": false,
-      "prompt": "code-agent",
-      "inferenceSettings": { "temperature": 0.7, "topP": 0.9 },
-      "tags": ["fast", "quick-fixes", "tool-calling", "vision"],
-      "tier": "gibbon",
-      "description": "Fast. Quick fixes, autocomplete, rapid iteration.",
-      "provider": "monkey"
-    },
     "tamarin": {
       "name": "Tamarin",
       "id": "tamarin",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "banana-code",
-  "version": "1.2.0",
+  "version": "1.3.0",
   "description": "AI coding agent CLI powered by Monkey Models and remote providers (Anthropic, OpenAI, OpenRouter)",
   "main": "banana.js",
   "bin": {
@@ -37,6 +37,7 @@
   },
   "dependencies": {
     "diff": "^8.0.3",
-    "glob": "^13.0.6"
+    "glob": "^13.0.6",
+    "update-notifier": "^7.3.1"
   }
 }