codeep 1.0.21 → 1.0.23

package/dist/utils/agent.js

@@ -62,7 +62,7 @@ import { getProviderBaseUrl, getProviderAuthHeader, supportsNativeTools } from '
 import { startSession, endSession, undoLastAction, undoAllActions, getCurrentSession, getRecentSessions, formatSession } from './history.js';
 import { runAllVerifications, formatErrorsForAgent, hasVerificationErrors, getVerificationSummary } from './verify.js';
 import { gatherSmartContext, formatSmartContext, extractTargetFile } from './smartContext.js';
-import { planTasks, getNextTask, formatTaskPlan } from './taskPlanner.js';
+import { planTasks, formatTaskPlan } from './taskPlanner.js';
 const DEFAULT_OPTIONS = {
     maxIterations: 100, // Increased for large tasks
     maxDuration: 20 * 60 * 1000, // 20 minutes
@@ -636,83 +636,13 @@ export async function runAgent(prompt, projectContext, options = {}) {
                     toolCalls = textToolCalls;
                 }
             }
-            // If no tool calls, check if agent is really done
+            // If no tool calls, agent is done - simple and clean like Claude Code/Aider
             if (toolCalls.length === 0) {
-                console.error(`[DEBUG] No tool calls at iteration ${iteration}, content length: ${content.length}, actions so far: ${actions.length}`);
-                console.error(`[DEBUG] Response preview:`, content.substring(0, 200));
+                console.error(`[DEBUG] No tool calls at iteration ${iteration}, agent finished`);
                 // Remove <think>...</think> tags from response (some models include thinking)
                 finalResponse = content.replace(/<think>[\s\S]*?<\/think>/gi, '').trim();
-                // CRITICAL: Don't stop prematurely if we haven't done much work yet
-                // Check if prompt contains "create", "build", "make", "implement", "add" etc.
-                const taskKeywords = ['create', 'build', 'make', 'implement', 'add', 'generate', 'write', 'setup', 'develop', 'kreiraj', 'napravi', 'dodaj', 'website', 'app', 'feature'];
-                const promptLowerCase = prompt.toLowerCase();
-                const isCreationTask = taskKeywords.some(kw => promptLowerCase.includes(kw));
-                // Only count actual FILE writes, not directories
-                const hasCreatedFiles = actions.some(a => a.type === 'write');
-                const writeFileCount = actions.filter(a => a.type === 'write').length;
-                const hasMkdir = actions.some(a => a.type === 'mkdir');
-                const isVeryEarlyIteration = iteration <= 5; // Extended to 5 iterations
-                console.error(`[DEBUG] Task check: isCreationTask=${isCreationTask}, hasCreatedFiles=${hasCreatedFiles}, writeFileCount=${writeFileCount}, hasMkdir=${hasMkdir}, iteration=${iteration}`);
-                console.error(`[DEBUG] Actions breakdown:`, actions.map(a => `${a.type}:${a.target}`).join(', '));
-                // STRICT RULE: If it's a creation task, agent MUST create FILES (not just directories)
-                // Creating a directory is not enough - we need actual file content
-                if (isCreationTask && !hasCreatedFiles && iteration <= 10) {
-                    console.error(`[DEBUG] BLOCKING early stop - creation task detected but NO files created yet (iteration ${iteration}, ${actions.length} actions)`);
-                    // Force agent to create files
-                    messages.push({ role: 'assistant', content: finalResponse });
-                    messages.push({
-                        role: 'user',
-                        content: `❌ STOP. You have NOT created any files yet!\n\nYou MUST use write_file and create_directory tools to create the actual files I asked for.\n\nDo NOT respond with explanations. Execute the tools NOW:\n1. Use create_directory if needed\n2. Use write_file to create EACH file\n3. Only after creating ALL files, respond with a summary.`
-                    });
-                    finalResponse = ''; // Reset
-                    continue;
-                }
-                // If creation task but not enough files created, keep pushing
-                if (isCreationTask && writeFileCount < 2 && iteration <= 10) {
-                    console.error(`[DEBUG] BLOCKING early stop - only ${writeFileCount} files created, need more (iteration ${iteration})`);
-                    messages.push({ role: 'assistant', content: finalResponse });
-                    messages.push({
-                        role: 'user',
-                        content: `You've only created ${writeFileCount} file(s). The task requires creating multiple files. Continue creating the remaining files now.`
-                    });
-                    finalResponse = ''; // Reset
-                    continue;
-                }
-                // General safety: very few actions in early iterations
-                if (isVeryEarlyIteration && actions.length < 2) {
-                    console.error(`[DEBUG] BLOCKING early stop - very few actions (iteration ${iteration}, only ${actions.length} actions)`);
-                    messages.push({ role: 'assistant', content: finalResponse });
-                    messages.push({
-                        role: 'user',
-                        content: `Continue working on the task. Use the available tools to complete what I asked for.`
-                    });
-                    finalResponse = ''; // Reset
-                    continue;
-                }
-                // Check if we're using task planning and there are more tasks
-                if (taskPlan && taskPlan.tasks.some(t => t.status === 'pending')) {
-                    const nextTask = getNextTask(taskPlan.tasks);
-                    if (nextTask) {
-                        // Mark previous task as completed
-                        const prevTask = taskPlan.tasks.find(t => t.status === 'in_progress' && t.id !== nextTask.id);
-                        if (prevTask) {
-                            prevTask.status = 'completed';
-                        }
-                        // Move to next task
-                        nextTask.status = 'in_progress';
-                        opts.onTaskUpdate?.(nextTask);
-                        // Build progress summary
-                        const completedTasks = taskPlan.tasks.filter(t => t.status === 'completed');
-                        const progressSummary = completedTasks.map(t => `✓ ${t.id}. ${t.description}`).join('\n');
-                        messages.push({ role: 'assistant', content: finalResponse });
-                        messages.push({
-                            role: 'user',
-                            content: `Good! Task ${prevTask?.id} done.\n\nCompleted:\n${progressSummary}\n\nNEXT TASK (do it now):\n${nextTask.id}. ${nextTask.description}\n\nUse your tools immediately to complete this task. Do NOT ask for permission or confirmation.`
-                        });
-                        finalResponse = ''; // Reset for next task
-                        continue;
-                    }
-                }
+                // Trust the model - if it says it's done, it's done
+                // No forcing, no blocking, no artificial requirements
                 break;
             }
             // Add assistant response to history

package/dist/utils/tools.d.ts

@@ -196,9 +196,6 @@ export declare function getOpenAITools(): OpenAITool[];
  * Get tools in Anthropic Tool Use format
  */
 export declare function getAnthropicTools(): AnthropicTool[];
-/**
- * Parse tool calls from OpenAI response
- */
 export declare function parseOpenAIToolCalls(toolCalls: any[]): ToolCall[];
 /**
  * Parse tool calls from Anthropic response

package/dist/utils/tools.js

@@ -174,6 +174,33 @@ export function getAnthropicTools() {
 /**
  * Parse tool calls from OpenAI response
  */
+/**
+ * Normalize tool name to lowercase with underscores
+ */
+function normalizeToolName(name) {
+    const toolNameMap = {
+        'executecommand': 'execute_command',
+        'execute_command': 'execute_command',
+        'readfile': 'read_file',
+        'read_file': 'read_file',
+        'writefile': 'write_file',
+        'write_file': 'write_file',
+        'editfile': 'edit_file',
+        'edit_file': 'edit_file',
+        'deletefile': 'delete_file',
+        'delete_file': 'delete_file',
+        'listfiles': 'list_files',
+        'list_files': 'list_files',
+        'searchcode': 'search_code',
+        'search_code': 'search_code',
+        'createdirectory': 'create_directory',
+        'create_directory': 'create_directory',
+        'fetchurl': 'fetch_url',
+        'fetch_url': 'fetch_url',
+    };
+    const lower = name.toLowerCase().replace(/-/g, '_');
+    return toolNameMap[lower] || lower;
+}
 export function parseOpenAIToolCalls(toolCalls) {
     if (!toolCalls || !Array.isArray(toolCalls))
         return [];
@@ -186,7 +213,7 @@ export function parseOpenAIToolCalls(toolCalls) {
             parameters = {};
         }
         return {
-            tool: tc.function?.name || '',
+            tool: normalizeToolName(tc.function?.name || ''),
             parameters,
             id: tc.id,
         };
@@ -201,7 +228,7 @@ export function parseAnthropicToolCalls(content) {
     return content
         .filter(block => block.type === 'tool_use')
         .map(block => ({
-        tool: block.name || '',
+        tool: normalizeToolName(block.name || ''),
         parameters: block.input || {},
         id: block.id,
     }))
@@ -386,7 +413,7 @@ function tryParseToolCall(str) {
         const parsed = JSON.parse(cleaned);
         if (parsed.tool && typeof parsed.tool === 'string') {
             return {
-                tool: parsed.tool,
+                tool: normalizeToolName(parsed.tool),
                 parameters: parsed.parameters || {},
                 id: parsed.id,
             };
@@ -394,9 +421,9 @@ function tryParseToolCall(str) {
     }
     catch {
         // Try to extract tool name and parameters manually for malformed JSON
-        const toolMatch = str.match(/"tool"\s*:\s*"([^"]+)"/);
+        const toolMatch = str.match(/"tool"\s*:\s*"([^"]+)"/i);
         if (toolMatch) {
-            const tool = toolMatch[1];
+            const tool = normalizeToolName(toolMatch[1]);
             const params = {};
             // Extract simple string parameters
             const paramMatches = str.matchAll(/"(\w+)"\s*:\s*"([^"]*)"/g);
@@ -432,7 +459,9 @@ function validatePath(path, projectRoot) {
  * Execute a tool call
  */
 export function executeTool(toolCall, projectRoot) {
-    const { tool, parameters } = toolCall;
+    // Normalize tool name to handle case variations (WRITE_FILE -> write_file)
+    const tool = normalizeToolName(toolCall.tool);
+    const parameters = toolCall.parameters;
     try {
         switch (tool) {
             case 'read_file': {
@@ -705,6 +734,8 @@ function listDirectory(dir, projectRoot, recursive, prefix = '') {
  * Create action log from tool result
  */
 export function createActionLog(toolCall, result) {
+    // Normalize tool name to handle case variations
+    const normalizedTool = normalizeToolName(toolCall.tool);
     const typeMap = {
         read_file: 'read',
         write_file: 'write',
@@ -722,7 +753,7 @@ export function createActionLog(toolCall, result) {
         toolCall.parameters.url ||
         'unknown';
     return {
-        type: typeMap[toolCall.tool] || 'command',
+        type: typeMap[normalizedTool] || 'command',
         target,
         result: result.success ? 'success' : 'error',
         details: result.success ? result.output.slice(0, 200) : result.error,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeep",
-  "version": "1.0.21",
+  "version": "1.0.23",
   "description": "AI-powered coding assistant built for the terminal. Multiple LLM providers, project-aware context, and a seamless development workflow.",
   "type": "module",
   "main": "dist/index.js",