banana-code 1.3.1 → 1.4.1

package/lib/contextBuilder.js

@@ -40,10 +40,15 @@ class ContextBuilder {
   constructor(fileManager, ignorePatterns = []) {
     this.fileManager = fileManager;
     this.projectDir = fileManager.projectDir;
-    this.customIgnores = ignorePatterns;
+    this.customIgnores = [...ignorePatterns];
+    this.activeIgnores = [...this.customIgnores];
     this.loadedFiles = new Map(); // path -> content
   }
 
+  refreshIgnorePatterns() {
+    this.activeIgnores = [...new Set([...this.customIgnores, ...this.loadGitignore()])];
+  }
+
   /**
    * Load .gitignore patterns
    */
@@ -79,7 +84,7 @@ class ContextBuilder {
     if (IGNORED_FILES.includes(basename)) return true;
 
     // Check custom patterns (simple matching)
-    for (const pattern of this.customIgnores) {
+    for (const pattern of this.activeIgnores) {
       if (pattern.endsWith('/') || pattern.endsWith('\\')) {
         // Directory pattern
         const dirName = pattern.slice(0, -1);
@@ -105,6 +110,9 @@ class ContextBuilder {
    * @param {object} counter - Shared counter across recursive calls
    */
   scanDirectory(dir = this.projectDir, depth = 0, maxDepth = 6, maxFiles = 5000, counter = { count: 0 }) {
+    if (depth === 0) {
+      this.refreshIgnorePatterns();
+    }
     if (depth > maxDepth) return [];
     if (counter.count >= maxFiles) return [];
 
@@ -262,8 +270,7 @@ class ContextBuilder {
    * Build the full context string for AI
    */
   buildContext() {
-    // Load gitignore patterns
-    this.customIgnores = [...this.customIgnores, ...this.loadGitignore()];
+    this.refreshIgnorePatterns();
 
     // Load priority files first
     this.loadPriorityFiles();

package/lib/fileManager.js

@@ -4,6 +4,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { ensureDirSync, atomicWriteFileSync } = require('./fsUtils');
 
 // File extensions to always skip
 const BINARY_EXTENSIONS = [
@@ -29,9 +30,7 @@ class FileManager {
    * Ensure backup directory exists
    */
   ensureBackupDir() {
-    if (!fs.existsSync(this.backupDir)) {
-      fs.mkdirSync(this.backupDir, { recursive: true });
-    }
+    ensureDirSync(this.backupDir);
   }
 
   /**
@@ -85,8 +84,9 @@ class FileManager {
       : path.join(this.projectDir, filePath);
 
     try {
+      const existed = fs.existsSync(fullPath);
       // Create backup if file exists
-      if (fs.existsSync(fullPath)) {
+      if (existed) {
         this.ensureBackupDir();
         const timestamp = Date.now();
         const relativePath = path.relative(this.projectDir, fullPath);
@@ -97,15 +97,12 @@ class FileManager {
       }
 
       // Ensure directory exists
-      const dir = path.dirname(fullPath);
-      if (!fs.existsSync(dir)) {
-        fs.mkdirSync(dir, { recursive: true });
-      }
+      ensureDirSync(path.dirname(fullPath));
 
       // Write the file
-      fs.writeFileSync(fullPath, content, 'utf-8');
+      atomicWriteFileSync(fullPath, content);
 
-      return { success: true, path: fullPath, isNew: !fs.existsSync(fullPath) };
+      return { success: true, path: fullPath, isNew: !existed };
     } catch (error) {
       return { success: false, error: error.message };
     }
@@ -187,8 +184,9 @@ class FileManager {
       const targetPath = path.isAbsolute(filePath)
         ? filePath
         : path.join(this.projectDir, filePath);
+      ensureDirSync(path.dirname(targetPath));
 
-      fs.writeFileSync(targetPath, content, 'utf-8');
+      atomicWriteFileSync(targetPath, content);
 
       return { success: true, restored: latestBackup.name };
     } catch (error) {

package/lib/fsUtils.js

@@ -0,0 +1,30 @@
+const fs = require('fs');
+const path = require('path');
+const crypto = require('crypto');
+
+function ensureDirSync(dirPath) {
+  if (!fs.existsSync(dirPath)) {
+    fs.mkdirSync(dirPath, { recursive: true });
+  }
+}
+
+function atomicWriteFileSync(targetPath, content, encoding = 'utf-8') {
+  const dir = path.dirname(targetPath);
+  ensureDirSync(dir);
+  const tempPath = path.join(
+    dir,
+    `.${path.basename(targetPath)}.${process.pid}.${Date.now()}.${crypto.randomBytes(4).toString('hex')}.tmp`
+  );
+  fs.writeFileSync(tempPath, content, encoding);
+  fs.renameSync(tempPath, targetPath);
+}
+
+function atomicWriteJsonSync(targetPath, value) {
+  atomicWriteFileSync(targetPath, JSON.stringify(value, null, 2));
+}
+
+module.exports = {
+  ensureDirSync,
+  atomicWriteFileSync,
+  atomicWriteJsonSync
+};

package/lib/historyManager.js

@@ -4,6 +4,7 @@
 
 const fs = require('fs');
 const path = require('path');
+const { ensureDirSync, atomicWriteFileSync } = require('./fsUtils');
 
 class HistoryManager {
   constructor(bananaDir, maxHistory = 100) {
@@ -44,11 +45,8 @@ class HistoryManager {
 
   save() {
     try {
-      const dir = path.dirname(this.historyFile);
-      if (!fs.existsSync(dir)) {
-        fs.mkdirSync(dir, { recursive: true });
-      }
-      fs.writeFileSync(this.historyFile, this.history.join('\n'));
+      ensureDirSync(path.dirname(this.historyFile));
+      atomicWriteFileSync(this.historyFile, this.history.join('\n'));
     } catch {
       // Ignore save errors
     }

package/lib/interactivePicker.js

@@ -74,10 +74,10 @@ function pick(items, options = {}) {
 
       if (isSel) {
         const lead = showVisionIndicator ? `${pointer}${marker} ${vision} ` : `${pointer}${marker} `;
-        return `${lead}${INVERSE} ${YELLOW}${key}${RESET}${INVERSE}  ${label} ${RESET}${tags}`;
+        return `${lead}${INVERSE} ${YELLOW}${key}${RESET}${INVERSE}  ${label} ${RESET}${desc}${tags}`;
       }
       const lead = showVisionIndicator ? `${pointer}${marker} ${vision} ` : `${pointer}${marker} `;
-      return `${lead}${YELLOW}${key}${RESET}  ${DIM}${label}${RESET}${tags}`;
+      return `${lead}${YELLOW}${key}${RESET}  ${DIM}${label}${RESET}${desc}${tags}`;
     }
 
     // Total lines we render (title + items + footer)

package/lib/modelRegistry.js

@@ -4,6 +4,7 @@
  */
 
 const fs = require('fs');
+const { atomicWriteJsonSync } = require('./fsUtils');
 
 class ModelRegistry {
   constructor(registryPath, lmStudio, providerStore = null) {
@@ -34,7 +35,7 @@ class ModelRegistry {
   }
 
   save() {
-    fs.writeFileSync(this.registryPath, JSON.stringify(this.localRegistry, null, 2));
+    atomicWriteJsonSync(this.registryPath, this.localRegistry);
   }
 
   _normalizeLocalModels() {
@@ -68,7 +69,7 @@ class ModelRegistry {
           },
           tags: remoteModel.tags || ['remote', remoteModel.provider],
           tier: 'remote',
-          description: `${remoteModel.name} via ${remoteModel.provider}`,
+          description: remoteModel.name,
           remote: true
         };
       }

package/lib/providerManager.js

@@ -18,7 +18,8 @@ const PROVIDER_LABELS = {
   anthropic: 'Anthropic',
   openai: 'OpenAI',
   openrouter: 'OpenRouter',
-  monkey: 'Monkey Models'
+  monkey: 'Monkey Models',
+  'claude-code': 'Claude Code'
 };
 
 class ProviderManager {
@@ -115,6 +116,11 @@ class ProviderManager {
       });
     }
 
+    if (provider === 'claude-code') {
+      const { ClaudeCodeClient } = require('./claudeCodeProvider');
+      return new ClaudeCodeClient();
+    }
+
     throw new Error(`Unknown provider: ${provider}`);
   }
 

package/lib/providerStore.js

@@ -4,7 +4,7 @@ const path = require('path');
 
 const DEFAULT_PATH = path.join(os.homedir(), '.banana', 'providers.json');
 
-const PROVIDERS = ['monkey', 'anthropic', 'openai', 'openrouter'];
+const PROVIDERS = ['monkey', 'anthropic', 'openai', 'openrouter', 'claude-code'];
 
 const DEFAULT_PROVIDER_MODELS = {
   monkey: {},
@@ -42,6 +42,29 @@ const DEFAULT_PROVIDER_MODELS = {
       prompt: 'code-agent'
     }
   },
+  'claude-code': {
+    'opus': {
+      name: 'Claude Opus (Claude Code)',
+      id: 'opus',
+      contextLimit: 200000,
+      supportsThinking: false,
+      prompt: 'code-agent'
+    },
+    'sonnet': {
+      name: 'Claude Sonnet (Claude Code)',
+      id: 'sonnet',
+      contextLimit: 200000,
+      supportsThinking: false,
+      prompt: 'code-agent'
+    },
+    'haiku': {
+      name: 'Claude Haiku (Claude Code)',
+      id: 'haiku',
+      contextLimit: 200000,
+      supportsThinking: false,
+      prompt: 'code-agent'
+    }
+  },
   openrouter: {
     'claude-opus-4.6': {
       name: 'Claude Opus 4.6 (OpenRouter)',
@@ -91,7 +114,8 @@ function createDefaultData() {
     providers: {
       anthropic: defaultProviderRecord('anthropic'),
       openai: defaultProviderRecord('openai'),
-      openrouter: defaultProviderRecord('openrouter')
+      openrouter: defaultProviderRecord('openrouter'),
+      'claude-code': defaultProviderRecord('claude-code')
     }
   };
 }
@@ -328,6 +352,16 @@ class ProviderStore {
     this.save();
   }
 
+  /**
+   * Connect Claude Code provider (no API key needed, uses CLI auth).
+   */
+  connectClaudeCode() {
+    this._touch('claude-code');
+    this.data.providers['claude-code'].connected = true;
+    this.data.providers['claude-code'].auth = { type: 'cli' };
+    this.save();
+  }
+
   connectOpenAI(auth) {
     this._touch('openai');
     this.data.providers.openai.connected = true;
@@ -396,8 +430,8 @@ class ProviderStore {
           contextLimit: model.contextLimit || 128000,
           supportsThinking: model.supportsThinking !== false,
           prompt: model.prompt || 'code-agent',
-          tags: provider === 'openrouter'
-            ? ['remote', provider, 'tool-calling']
+          tags: (provider === 'openrouter' || provider === 'claude-code')
+            ? ['remote', provider]
             : ['remote', provider, 'tool-calling', 'vision']
         });
       }

package/lib/streamHandler.js

@@ -36,6 +36,9 @@ class StreamHandler {
     // Buffer output until we confirm there's no orphan </think> coming
     this.orphanBuffer = '';
     this.orphanCheckComplete = false; // Once true, stop buffering for orphan check
+    this.completionStatus = 'completed';
+    this.completionWarning = null;
+    this.doneSignalReceived = false;
   }
 
   /**
@@ -176,7 +179,7 @@ class StreamHandler {
   async handleStream(response) {
     const reader = response.body.getReader();
     const decoder = new TextDecoder();
-    const IDLE_TIMEOUT = 30000; // 30 seconds with no data = assume done (LM Studio needs time after prompt processing)
+    const IDLE_TIMEOUT = 30000;
 
     try {
       while (true) {
@@ -193,7 +196,8 @@ class StreamHandler {
           value = result.value;
         } catch (e) {
           if (e.message === 'IDLE_TIMEOUT') {
-            // No data for 5 seconds, assume stream is complete
+            this.completionStatus = 'incomplete_stream';
+            this.completionWarning = `Stream stalled for ${IDLE_TIMEOUT}ms before an explicit completion signal.`;
             break;
           }
           throw e;
@@ -218,7 +222,9 @@ class StreamHandler {
 
             if (data === '[DONE]') {
               // Stream complete signal - exit the loop
-              this.onComplete(this.fullResponse);
+              this.doneSignalReceived = true;
+              this.completionStatus = 'completed';
+              this.onComplete(this.fullResponse, this.getResult());
               return this.fullResponse;
             }
 
@@ -251,6 +257,11 @@ class StreamHandler {
         }
       }
 
+      if (!this.doneSignalReceived && this.completionStatus === 'completed') {
+        this.completionStatus = 'incomplete_stream';
+        this.completionWarning = 'Stream ended without an explicit completion signal.';
+      }
+
       // Process any remaining buffer
       if (this.buffer) {
         const lines = this.buffer.split('\n');
@@ -280,7 +291,7 @@ class StreamHandler {
         }
       }
 
-      this.onComplete(this.fullResponse);
+      this.onComplete(this.fullResponse, this.getResult());
       return this.fullResponse;
 
     } catch (error) {
@@ -292,6 +303,16 @@ class StreamHandler {
   getFullResponse() {
     return this.fullResponse;
   }
+
+  getResult() {
+    return {
+      content: this.fullResponse,
+      completed: this.completionStatus === 'completed',
+      status: this.completionStatus,
+      warning: this.completionWarning,
+      doneSignalReceived: this.doneSignalReceived
+    };
+  }
 }
 
 // Non-streaming fallback with simulated streaming effect

package/package.json

@@ -1,43 +1,48 @@
-{
-  "name": "banana-code",
-  "version": "1.3.1",
-  "description": "AI coding agent CLI powered by Monkey Models and remote providers (Anthropic, OpenAI, OpenRouter)",
-  "main": "banana.js",
-  "bin": {
-    "banana": "banana.js"
-  },
-  "files": [
-    "banana.js",
-    "lib/",
-    "prompts/",
-    "models.json",
-    "package.json"
-  ],
-  "scripts": {
-    "postinstall": "node scripts/createBananaDir.js || echo banana: skipped dir creation"
-  },
-  "keywords": [
-    "ai",
-    "coding",
-    "agent",
-    "cli",
-    "monkey-models",
-    "banana-code",
-    "code-assistant"
-  ],
-  "author": "Matt Johnston",
-  "license": "MIT",
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/mrchevyceleb/banana-code.git"
-  },
-  "homepage": "https://github.com/mrchevyceleb/banana-code",
-  "engines": {
-    "node": ">=18"
-  },
-  "dependencies": {
-    "diff": "^8.0.3",
-    "glob": "^13.0.6",
-    "update-notifier": "^7.3.1"
-  }
-}
+{
+  "name": "banana-code",
+  "version": "1.4.1",
+  "description": "AI coding agent CLI powered by Monkey Models and remote providers (Anthropic, OpenAI, OpenRouter)",
+  "main": "banana.js",
+  "bin": {
+    "banana": "banana.js"
+  },
+  "files": [
+    "banana.js",
+    "lib/",
+    "prompts/",
+    "models.json",
+    "package.json"
+  ],
+  "scripts": {
+    "postinstall": "node scripts/createBananaDir.js || echo banana: skipped dir creation",
+    "test:tool-batch": "node scripts/regression-tool-batch-sanitizer.js",
+    "test:command-verification": "node scripts/regression-command-verification.js",
+    "test:stream-handler": "node scripts/regression-stream-handler.js",
+    "test:persistence-context": "node scripts/regression-persistence-context.js",
+    "test:reliability": "npm run test:tool-batch && npm run test:command-verification && npm run test:stream-handler && npm run test:persistence-context"
+  },
+  "keywords": [
+    "ai",
+    "coding",
+    "agent",
+    "cli",
+    "monkey-models",
+    "banana-code",
+    "code-assistant"
+  ],
+  "author": "Matt Johnston",
+  "license": "MIT",
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/mrchevyceleb/banana-code.git"
+  },
+  "homepage": "https://github.com/mrchevyceleb/banana-code",
+  "engines": {
+    "node": ">=18"
+  },
+  "dependencies": {
+    "diff": "^8.0.3",
+    "glob": "^13.0.6",
+    "update-notifier": "^7.3.1"
+  }
+}

package/prompts/base.md

@@ -1,23 +1,33 @@
-You are Banana, a sharp, *proactive* Executive assistant *AND* senior full-stack engineer. You are just as good at assisting people on multi-tool tasks and giving advice as you are a brilliant coder.
-
-## Personality
-- You are always kind.
-- You are *tenacious*, you never give up
-- You are supportive, enthusiastic, and make people feel good about themselves.
-- Your knowledge of coding and software development is *second to none*
-
-## General Knowledge
-
-You're a general-purpose assistant, not just a code tool. If the user asks about general knowledge topics, answer directly. Only use file/code tools when the question is actually about code or the project.
-
-## OS
-
-Windows. Use PowerShell/cmd syntax for shell commands. No bash, no `ls`, no `grep`.
-
-## File Operations
-
-Use the `create_file` and `edit_file` tools to write files. Use `run_command` for shell commands (PowerShell syntax only). Never dump file content as a markdown code block.
-
-## Self-Awareness
-
-You ARE Banana Code. When users ask about your features, how to do something, or need guidance on setup, use the `banana_help` tool to look up accurate documentation. Topics: overview, hooks, models, commands, project-instructions, agents. Don't guess at feature details. Look them up, then guide the user through it naturally.
+You are Banana, the AI inside Banana Code. You are a chill, supportive genius. Playful but relentless. You talk like a best friend who happens to be a world-class engineer. You are built for vibe coders who have big ideas but don't write code themselves.
+
+## Personality
+
+- Always positive and encouraging. Match the user's energy.
+- Never talk down to people. Explain things in a way that makes people feel smarter, not smaller.
+- Brief, real, occasionally witty. No sycophancy. No "Certainly!" No walls of text.
+- Tenacious. You never give up. You don't wait for approval unless genuinely required.
+- When you make a mistake, own it with humor and fix it immediately.
+
+## General Knowledge
+
+You're a general-purpose assistant, not just a code tool. If the user asks about general knowledge topics, answer directly. Only use file/code tools when the question is actually about code or the project.
+
+## OS
+
+Windows. Use cmd.exe syntax for shell commands. No bash (grep, cat, ls). If you need PowerShell, wrap it: `powershell -Command "your command here"`.
+
+## File Operations
+
+Use the `create_file` and `edit_file` tools to write files. Use `run_command` for shell commands. Never dump file content as a markdown code block.
+
+## Completeness
+
+Never use placeholders. Every function is complete. Every component renders. If the feature needs 5 files, write all 5. If a vague prompt comes in, go all out and let the user steer from there.
+
+## Design Philosophy
+
+Every frontend should feel like a toy: click easter eggs, floating animations, tactile hover effects, playful micro-interactions. Reject generic templates. Use distinctive typography, cohesive color palettes, orchestrated motion, and atmospheric backgrounds. Every project should look different.
+
+## Self-Awareness
+
+You ARE Banana Code. When users ask about your features, how to do something, or need guidance on setup, use the `banana_help` tool to look up accurate documentation. Don't guess at feature details.

package/prompts/code-agent-qwen.md

@@ -5,6 +5,7 @@
 - Always provide complete, valid JSON in tool call arguments. Never use partial or malformed JSON.
 - When calling tools, provide ALL required arguments. Missing arguments cause failures.
 - After receiving a tool result, analyze it before deciding your next step.
+- If a command changes git state or the filesystem, verify the effect with a separate read-only check before you tell the user it is done.
 - Do not call the same tool with the same arguments twice in a row.
 - If a tool returns an error, try a different approach or explain the issue.
 - `run_command` uses cmd.exe on Windows, NOT PowerShell. Use cmd syntax (findstr, type, dir). For PowerShell, wrap it: `powershell -Command "..."`.