codeep 1.0.28 → 1.0.29

package/dist/utils/agent.js

@@ -343,7 +343,7 @@ async function agentChatFallback(messages, systemPrompt, onChunk, abortSignal, d
                 ],
                 stream: Boolean(onChunk),
                 temperature: config.get('temperature'),
-                max_tokens: config.get('maxTokens'),
+                max_tokens: Math.max(config.get('maxTokens'), 16384), // Ensure enough tokens for large file generation
             };
         }
         else {
@@ -357,7 +357,7 @@ async function agentChatFallback(messages, systemPrompt, onChunk, abortSignal, d
                 ],
                 stream: Boolean(onChunk),
                 temperature: config.get('temperature'),
-                max_tokens: config.get('maxTokens'),
+                max_tokens: Math.max(config.get('maxTokens'), 16384), // Ensure enough tokens for large file generation
             };
         }
         const response = await fetch(endpoint, {

package/dist/utils/tools.js

@@ -226,7 +226,12 @@ export function parseOpenAIToolCalls(toolCalls) {
             }
         }
         // Validate required parameters for specific tools
-        if (toolName === 'write_file' && (!parameters.path || parameters.content === undefined)) {
+        // For write_file, we need at least a path (content can be empty string or placeholder)
+        if (toolName === 'write_file' && !parameters.path) {
+            // Log for debugging write_file issues
+            if (process.env.CODEEP_DEBUG) {
+                console.error(`[WARN] write_file missing path, raw args: ${rawArgs.substring(0, 200)}`);
+            }
             continue;
         }
         if (toolName === 'read_file' && !parameters.path) {
@@ -252,21 +257,29 @@ function extractPartialToolParams(toolName, rawArgs) {
         // For write_file, try to extract path and content
         if (toolName === 'write_file') {
             const pathMatch = rawArgs.match(/"path"\s*:\s*"([^"]+)"/);
-            const contentMatch = rawArgs.match(/"content"\s*:\s*"([\s\S]*?)(?:"|$)/);
-            if (pathMatch && contentMatch) {
-                // Unescape the content
-                let content = contentMatch[1];
-                content = content
-                    .replace(/\\n/g, '\n')
-                    .replace(/\\t/g, '\t')
-                    .replace(/\\r/g, '\r')
-                    .replace(/\\"/g, '"')
-                    .replace(/\\\\/g, '\\');
-                // If content appears truncated (doesn't end properly), add a comment
-                if (!content.endsWith('\n') && !content.endsWith('}') && !content.endsWith(';') && !content.endsWith('>')) {
-                    content += '\n<!-- Content may be truncated -->\n';
-                }
-                return { path: pathMatch[1], content };
+            if (pathMatch) {
+                // Try to extract content - it may be truncated
+                const contentMatch = rawArgs.match(/"content"\s*:\s*"([\s\S]*?)(?:"|$)/);
+                if (contentMatch) {
+                    // Unescape the content
+                    let content = contentMatch[1];
+                    content = content
+                        .replace(/\\n/g, '\n')
+                        .replace(/\\t/g, '\t')
+                        .replace(/\\r/g, '\r')
+                        .replace(/\\"/g, '"')
+                        .replace(/\\\\/g, '\\');
+                    // If content appears truncated (doesn't end properly), add a comment
+                    if (!content.endsWith('\n') && !content.endsWith('}') && !content.endsWith(';') && !content.endsWith('>')) {
+                        content += '\n<!-- Content may be truncated -->\n';
+                    }
+                    return { path: pathMatch[1], content };
+                }
+                else {
+                    // Path found but content completely missing or malformed
+                    // Return with empty content placeholder so the file is at least created
+                    return { path: pathMatch[1], content: '<!-- Content was truncated by API -->\n' };
+                }
             }
         }
         // For read_file, just need path
@@ -553,7 +566,16 @@ function tryParseToolCall(str) {
  * Validate path is within project
  */
 function validatePath(path, projectRoot) {
-    const absolutePath = isAbsolute(path) ? path : resolve(projectRoot, path);
+    // If path is absolute but starts with projectRoot, convert to relative
+    let normalizedPath = path;
+    if (isAbsolute(path) && path.startsWith(projectRoot)) {
+        normalizedPath = relative(projectRoot, path);
+    }
+    // If still absolute and doesn't match projectRoot, reject it
+    if (isAbsolute(normalizedPath)) {
+        return { valid: false, absolutePath: normalizedPath, error: `Absolute path '${path}' not allowed. Use relative paths.` };
+    }
+    const absolutePath = resolve(projectRoot, normalizedPath);
     const relativePath = relative(projectRoot, absolutePath);
     if (relativePath.startsWith('..')) {
         return { valid: false, absolutePath, error: `Path '${path}' is outside project directory` };
@@ -594,12 +616,13 @@ export function executeTool(toolCall, projectRoot) {
             }
             case 'write_file': {
                 const path = parameters.path;
-                const content = parameters.content;
+                let content = parameters.content;
                 if (!path) {
                     return { success: false, output: '', error: 'Missing required parameter: path', tool, parameters };
                 }
-                if (content === undefined) {
-                    return { success: false, output: '', error: 'Missing required parameter: content', tool, parameters };
+                // Allow empty content or provide placeholder for truncated responses
+                if (content === undefined || content === null) {
+                    content = '<!-- Content was not provided -->\n';
                 }
                 const validation = validatePath(path, projectRoot);
                 if (!validation.valid) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codeep",
-  "version": "1.0.28",
+  "version": "1.0.29",
   "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",