v1.8.31: Continuation rotation, cwd validation, directory deletion fixes

Fix 1: Continuation budget triggers context rotation instead of abort
- When continuation hits budget limit, now rotates context with summary
- Prevents mid-generation cutoffs that caused naked code blocks

Fix 2: run_command validates cwd parameter
- Rejects wildcards, empty strings, and invalid paths
- Returns proper error instead of false success

Fix 3: delete_file handles directories
- Uses fs.rm with recursive:true for directories
- Single tool works for both files and directories

Fix 4: Preamble strengthened
- Added verification requirement even after prior failures
- Added directory empty claim requires list_directory
- Updated delete_file instructions

Author: Brendan Gray

main/agenticChat.js

@@ -1394,9 +1394,6 @@ function register(ctx) {
       let _hasUnclosedToolFence = _fenceIdx !== -1 &&
         !_afterFence.match(/```(?:json|tool_call|tool)\b[\s\S]*?\n```/) &&
         !_afterFence.match(/```(?:json|tool_call|tool)\b[\s\S]*?[^`]```\s*$/);
-      // Also detect any unclosed code fence (html, js, css, etc.) for higher continuation budget
-      const _anyCodeFenceIdx = _stitchedForMcp.search(/```(?:html?|css|javascript|js|typescript|ts|python|py|json|jsx|tsx|java|c|cpp|csharp|ruby|go|rust|php|sql|xml|yaml|sh|bash|markdown|md)\b/);
-      const _hasUnclosedCodeFence = _anyCodeFenceIdx !== -1 && !_stitchedForMcp.slice(_anyCodeFenceIdx).match(/```[^\n]*\n[\s\S]*?\n```/);
 
       // If the unclosed fence contains a complete JSON tool call, don't treat as truncated
       if (_hasUnclosedToolFence) {
@@ -1430,16 +1427,59 @@ function register(ctx) {
           contContextPct = contUsed / totalCtx;
         } catch (_) {}
 
-        // Higher budget limit for unclosed code fences to allow large code blocks to complete
-        const budgetLimit = _hasUnclosedToolFence ? 0.92 : (_hasUnclosedCodeFence ? 0.90 : 0.88);
+        const budgetLimit = _hasUnclosedToolFence ? 0.92 : 0.70;
         if (contContextPct > budgetLimit) {
-          console.log(`[AI Chat] Continuation aborted: context at ${Math.round(contContextPct * 100)}% (limit=${Math.round(budgetLimit * 100)}%)`);
+          // Context budget exceeded — trigger rotation instead of aborting
+          // This allows large content generation (HTML, code files) to complete
+          console.log(`[AI Chat] Continuation budget rotation: context at ${Math.round(contContextPct * 100)}% (limit=${Math.round(budgetLimit * 100)}%)`);
           continuationCount = 0;
-          // Try salvage
+          
+          // Attempt context rotation to continue the task
+          if (contextRotations < MAX_CONTEXT_ROTATIONS) {
+            console.log(`[AI Chat] Budget-triggered rotation (${contextRotations + 1}/${MAX_CONTEXT_ROTATIONS})`);
+            contextRotations++;
+            try {
+              summarizer.markRotation();
+              const convSummary = summarizer.generateSummary({
+                maxTokens: Math.min(Math.floor(totalCtx * 0.25), 3000),
+                activeTodos: mcpToolServer?._todos || [],
+              });
+              
+              // Store partial output for context
+              const partialOutput = fullResponseText.slice(-Math.min(fullResponseText.length, 2000));
+              await llmEngine.resetSession(true);
+              await ensureLlmChat(llmEngine, getNodeLlamaCppPath);
+              if (mainWindow && !mainWindow.isDestroyed()) {
+                mainWindow.webContents.send('llm-thinking-token', '\n[Context rotated to continue generation]\n');
+              }
+              
+              // Build continuation prompt with summary and partial output
+              const incrementalHint = summarizer.incrementalTask
+                ? `\n**INCREMENTAL TASK: ${summarizer.incrementalTask.current}/${summarizer.incrementalTask.target} ${summarizer.incrementalTask.type} completed.**`
+                : '';
+              const fileProgressHint = Object.keys(summarizer.fileProgress).length > 0
+                ? `\n**FILES IN PROGRESS:** ${Object.entries(summarizer.fileProgress).map(([f, p]) => `${f} (${p.writtenLines} lines)`).join(', ')}`
+                : '';
+              
+              currentPrompt = {
+                systemContext: buildStaticPrompt(),
+                userMessage: buildDynamicContext() + '\n\n' + convSummary +
+                  `\n\n## CONTINUE FROM HERE\n---\n${partialOutput}\n---` +
+                  incrementalHint + fileProgressHint +
+                  `\n\n**CRITICAL: DO NOT REFUSE. DO NOT SAY "I cannot continue."**` +
+                  `\nUse append_to_file to add more content. Call a tool NOW to make progress.`,
+              };
+              sessionJustRotated = true;
+              continue;
+            } catch (rotErr) {
+              console.error('[AI Chat] Budget-triggered rotation failed:', rotErr.message);
+            }
+          }
+          
+          // Rotation failed or exhausted — try salvage as fallback
           if (_hasUnclosedToolFence && _stitchedForMcp) {
             const salvageResult = salvagePartialToolCall(_stitchedForMcp, _fenceIdx);
             if (salvageResult) {
-              // Replace responseText with reconstructed tool call
               fullResponseText = fullResponseText.slice(0, fullResponseText.length - (result.text || '').length) + salvageResult;
             }
           }

main/constants.js

@@ -105,8 +105,10 @@ const DEFAULT_COMPACT_PREAMBLE = `You are a helpful, knowledgeable AI assistant.
 ## Rules
 - **Never output full file content as code blocks in chat** — always use write_file, edit_file, or append_to_file. Code blocks are only for brief snippets or explanations.
 - **For new files: call write_file immediately.** Do not describe what the file would contain — create it.
-- **When the user asks for confirmation or verification, ALWAYS call list_directory or read_file to verify.** NEVER say "I can confirm" without actually checking. NEVER refuse a verification request — you MUST call the tool.
-- **Path awareness:** All relative paths are relative to the project root. Use paths like "file.html" for root files, "subfolder/file.html" for nested files. To delete directories, use run_command with "Remove-Item -Recurse -Force path".
+- **When the user asks for confirmation or verification, ALWAYS call list_directory or read_file to verify.** NEVER say "I can confirm" without actually checking. NEVER refuse a verification request — you MUST call the tool. Even if previous operations failed, you MUST still verify when asked.
+- **Never claim a directory is empty without calling list_directory.** If list_directory returns items, report them exactly as returned.
+- **Path awareness:** All relative paths are relative to the project root. Use paths like "file.html" for root files, "subfolder/file.html" for nested files.
+- **delete_file works on BOTH files AND directories.** Use delete_file for any deletion — it handles recursive directory removal automatically.
 - When calling tools, format tool calls as valid JSON with properly quoted string values. Never use backtick template literals in tool call JSON.
 - Tools execute in the live environment. Call them — do not describe what you would do.
 - Never say you did something unless you called the tool that did it.

main/mcpToolServer.js

@@ -553,9 +553,9 @@ class MCPToolServer {
       },
       {
         name: 'delete_file',
-        description: 'Delete a file from the project.',
+        description: 'Delete a file OR directory from the project. Directories are removed recursively.',
         parameters: {
-          filePath: { type: 'string', description: 'Path of the file to delete', required: true },
+          filePath: { type: 'string', description: 'Path of the file or directory to delete', required: true },
         },
       },
       {
@@ -1492,8 +1492,15 @@ class MCPToolServer {
   async _deleteFile(filePath) {
     const fullPath = path.isAbsolute(filePath) ? filePath : path.join(this.projectPath || '', filePath);
     try {
-      await fs.unlink(fullPath);
-      return { success: true, path: fullPath, message: `File deleted: ${fullPath}` };
+      const stats = await fs.stat(fullPath);
+      if (stats.isDirectory()) {
+        // Recursively delete directory
+        await fs.rm(fullPath, { recursive: true, force: true });
+        return { success: true, path: fullPath, message: `Directory deleted: ${fullPath}` };
+      } else {
+        await fs.unlink(fullPath);
+        return { success: true, path: fullPath, message: `File deleted: ${fullPath}` };
+      }
     } catch (error) {
       return { success: false, error: error.message };
     }
@@ -1586,13 +1593,44 @@ class MCPToolServer {
     }
 
     let workDir = this.projectPath || process.cwd();
-    if (cwd && path.isAbsolute(cwd)) {
-      const cwdNorm = cwd.replace(/\\/g, '/').toLowerCase();
-      const projNorm = (this.projectPath || '').replace(/\\/g, '/').toLowerCase();
-      if (projNorm && cwdNorm.startsWith(projNorm)) {
-        workDir = cwd;
+    if (cwd) {
+      // Validate cwd parameter
+      const cwdStr = String(cwd).trim();
+      
+      // Block obvious invalid values (wildcards, single chars, empty after trim)
+      if (!cwdStr || cwdStr === '*' || cwdStr === '?' || /^[*?]+$/.test(cwdStr)) {
+        console.log(`[MCPToolServer] Blocked invalid cwd "${cwd}"`);
+        return { success: false, error: `Invalid cwd parameter: "${cwd}". Use a valid directory path or omit cwd to use project root.` };
+      }
+      
+      if (path.isAbsolute(cwdStr)) {
+        const cwdNorm = cwdStr.replace(/\\/g, '/').toLowerCase();
+        const projNorm = (this.projectPath || '').replace(/\\/g, '/').toLowerCase();
+        if (projNorm && cwdNorm.startsWith(projNorm)) {
+          workDir = cwdStr;
+        } else {
+          console.log(`[MCPToolServer] Ignoring hallucinated cwd "${cwd}", using project path`);
+        }
       } else {
-        console.log(`[MCPToolServer] Ignoring hallucinated cwd "${cwd}", using project path`);
+        // Relative path — resolve relative to project
+        const resolved = path.resolve(this.projectPath || process.cwd(), cwdStr);
+        const resolvedNorm = resolved.replace(/\\/g, '/').toLowerCase();
+        const projNorm = (this.projectPath || '').replace(/\\/g, '/').toLowerCase();
+        if (projNorm && resolvedNorm.startsWith(projNorm)) {
+          // Check if directory exists
+          try {
+            const stats = fsSync.statSync(resolved);
+            if (stats.isDirectory()) {
+              workDir = resolved;
+            } else {
+              console.log(`[MCPToolServer] cwd "${cwd}" is not a directory, using project path`);
+            }
+          } catch (e) {
+            console.log(`[MCPToolServer] cwd "${cwd}" does not exist, using project path`);
+          }
+        } else {
+          console.log(`[MCPToolServer] Ignoring cwd "${cwd}" — resolves outside project`);
+        }
       }
     }
     const timeoutMs = Math.min(Math.max(timeout || 60000, 5000), 300000);

package.json

@@ -1,6 +1,6 @@
 {
   "name": "guide-ide",
-  "version": "1.8.30",
+  "version": "1.8.31",
   "description": "guIDE - AI-Powered Offline IDE with local LLM, RAG, MCP tools, browser automation, and integrated terminal",
   "author": {
     "name": "Brendan Gray",

website/src/app/download/page.tsx

@@ -2,7 +2,7 @@
 
 // Single source of truth for the displayed release version.
 // Updated automatically by: npm run release:deploy (from IDE root)
-const CURRENT_VERSION = '1.8.28';
+const CURRENT_VERSION = '1.8.31';
 
 import Link from 'next/link';
 import { useState } from 'react';