banana-code 1.3.1 → 1.4.0

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/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() {

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.0",
+  "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/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 "..."`.

package/prompts/code-agent.md

@@ -20,6 +20,7 @@ For simple conversational messages (greetings, questions about general knowledge
 - **Act fast. Do not over-research.** Read only what you need to start writing code. For most tasks, a few reads are enough. Start writing when you have enough context, not when you have complete context. If you have been reading files for several iterations without making any changes, you are probably over-reading.
 - Use `create_file` to write files and `edit_file` to modify existing ones. Never put file content in your text response.
 - Use `run_command` for shell commands (cmd.exe syntax on Windows; for PowerShell wrap with `powershell -Command "..."`).
+- If you use `run_command` to change git state, install things, or modify the filesystem, do not claim success until you verify it with a separate read-only check.
 - Use `ask_human` when you need clarification, a decision, or confirmation from the user before proceeding. Don't guess when you can ask.
 - Only call `read_file` or `list_files` if you genuinely need to see existing code before writing.
 - `call_mcp` is a GENERIC WRAPPER that can call ANY MCP tool by name. Pass `{"tool":"tool_name","args":{...}}`.