openfox 2.0.0-beta.8 → 2.0.0

package/README.md

@@ -22,6 +22,17 @@ openfox
 
 On first run, OpenFox automatically detects your local LLM backend (vLLM, sglang, ollama, llamacpp) and configures itself.
 
+## What's New in 2.0
+
+- **Multi-Turn Agent Engine** — Completely rewritten agent loop with EventStore as single source of truth. All modes (builder, planner, verifier, sub-agents, compaction) run through the same unified loop.
+- **Provider Dialog** — Comprehensive provider configuration UI with thinking mode (`reasoningEffort`), editable kwargs, profile defaults, and preset management.
+- **Auto-Retry Patterns** — Replace the old XML protection toggle with configurable pattern matching. Define your own retry triggers in settings.
+- **Unified Image Handling** — Automatic vision model fallback for non-vision models. Images are described before each turn so any model can "see" them.
+- **Session Metadata** — Unified `session_metadata` tool replaces separate criterion/todo tools. Interactive criteria editor with CRUD and agent badges.
+- **Workflow Sub-Groups** — Run individual workflow steps in isolation. New code review phase in the build-verify pipeline.
+- **Parallel `edit_file` Safety** — Per-file mutex prevents race conditions when editing multiple files simultaneously.
+- **Firefox Support** — Custom thin scrollbars with hover behavior, cross-browser compatible.
+
 ## CLI Commands
 
 ```bash

package/dist/{chat-handler-XIAY7O5X.js → chat-handler-LJMW4OK3.js}

@@ -5,18 +5,19 @@ import {
   generateSessionName,
   getSessionMessageCount,
   needsNameGenerationCheck
-} from "./chunk-CDDMHSZF.js";
+} from "./chunk-CK6LGE7G.js";
 import {
   runChatTurn
-} from "./chunk-DFITQ75T.js";
-import "./chunk-QN5ST54E.js";
+} from "./chunk-LEDG5WAN.js";
+import "./chunk-32GQUDLN.js";
 import "./chunk-DL6ZILAF.js";
 import "./chunk-PBGOZMVY.js";
 import "./chunk-VRGRAQDG.js";
+import "./chunk-XAMAYRDA.js";
 import {
   getEventStore
-} from "./chunk-NPHYEUYE.js";
-import "./chunk-XJEOP6XU.js";
+} from "./chunk-XEK3KII6.js";
+import "./chunk-ITWVFGFV.js";
 import {
   createChatMessageMessage,
   createPhaseChangedMessage,
@@ -24,14 +25,13 @@ import {
 } from "./chunk-7TTEGAO6.js";
 import "./chunk-BJYPTN5S.js";
 import "./chunk-RFNEDBVO.js";
-import "./chunk-XAMAYRDA.js";
 import "./chunk-FBGWG4N6.js";
 import "./chunk-BVHFMAVN.js";
-import "./chunk-7FDY3EOH.js";
+import "./chunk-CQGTEGKL.js";
+import "./chunk-FFEAEPJB.js";
 import {
   logger
 } from "./chunk-K44MW7JJ.js";
-import "./chunk-CQGTEGKL.js";
 
 // src/server/session/chat-handler.ts
 var activeAgents = /* @__PURE__ */ new Map();
@@ -161,4 +161,4 @@ export {
   startChatSession,
   stopSessionExecution
 };
-//# sourceMappingURL=chat-handler-XIAY7O5X.js.map
+//# sourceMappingURL=chat-handler-LJMW4OK3.js.map

package/dist/{chunk-QN5ST54E.js → chunk-32GQUDLN.js}

@@ -14,12 +14,15 @@ import {
   getMaxPerSession,
   getSessionProcessCount
 } from "./chunk-VRGRAQDG.js";
+import {
+  getProject
+} from "./chunk-XAMAYRDA.js";
 import {
   getCurrentContextWindowId,
   getCurrentWindowMessageOptions,
   getEventStore,
   getRuntimeConfig
-} from "./chunk-NPHYEUYE.js";
+} from "./chunk-XEK3KII6.js";
 import {
   buildContextMessagesFromEventHistory,
   foldContextState,
@@ -28,7 +31,7 @@ import {
   handleToolCall,
   handleToolResult,
   stripOrphanedToolCalls
-} from "./chunk-XJEOP6XU.js";
+} from "./chunk-ITWVFGFV.js";
 import {
   createChatDoneMessage,
   createChatMessageMessage,
@@ -46,21 +49,18 @@ import {
   getSetting
 } from "./chunk-RFNEDBVO.js";
 import {
-  getProject
-} from "./chunk-XAMAYRDA.js";
+  getGlobalConfigDir
+} from "./chunk-CQGTEGKL.js";
 import {
   buildModelParams,
   getBackendCapabilities,
   getModelProfile,
   modelSupportsVision,
   streamWithSegments
-} from "./chunk-7FDY3EOH.js";
+} from "./chunk-FFEAEPJB.js";
 import {
   logger
 } from "./chunk-K44MW7JJ.js";
-import {
-  getGlobalConfigDir
-} from "./chunk-CQGTEGKL.js";
 
 // src/server/tools/read.ts
 import { readFile as readFile2, stat, readdir } from "fs/promises";
@@ -954,6 +954,21 @@ function buildRegion(edits, contextStart, contextEnd, lines) {
 }
 
 // src/server/tools/edit.ts
+var fileLocks = /* @__PURE__ */ new Map();
+async function withFileLock(filePath, fn) {
+  const prev = fileLocks.get(filePath) ?? Promise.resolve();
+  const next = prev.then(fn, fn);
+  fileLocks.set(
+    filePath,
+    next.then(
+      () => {
+      },
+      () => {
+      }
+    )
+  );
+  return next;
+}
 function detectLineEnding(content) {
   if (content.includes("\r\n")) return "crlf";
   if (content.includes("\n")) return "lf";
@@ -995,95 +1010,97 @@ var editFileTool = createTool(
     }
   },
   async (args, context, helpers) => {
-    const replaceAll = args.replace_all ?? false;
     const fullPath = helpers.resolvePath(args.path);
     await helpers.checkPathAccess([fullPath]);
-    const readFiles = context.sessionManager.getReadFiles(context.sessionId);
-    const validation = await validateFileForWrite(fullPath, readFiles);
-    if (!validation.valid) {
-      return helpers.error(validation.error?.message ?? "File validation failed");
-    }
-    let content;
-    try {
-      content = await readFile3(fullPath, "utf-8");
-    } catch {
-      return helpers.error(`File not found: ${args.path}`);
-    }
-    const fileLineEnding = detectLineEnding(content);
-    const normalizedContent = normalizeToLF(content);
-    const normalizedOldString = normalizeToLF(args.old_string);
-    const occurrences = normalizedContent.split(normalizedOldString).length - 1;
-    if (occurrences === 0) {
-      const preview = args.old_string.length > 100 ? args.old_string.slice(0, 100) + "..." : args.old_string;
-      return helpers.error(
-        `old_string not found in file.
+    return withFileLock(fullPath, async () => {
+      const replaceAll = args.replace_all ?? false;
+      const readFiles = context.sessionManager.getReadFiles(context.sessionId);
+      const validation = await validateFileForWrite(fullPath, readFiles);
+      if (!validation.valid) {
+        return helpers.error(validation.error?.message ?? "File validation failed");
+      }
+      let content;
+      try {
+        content = await readFile3(fullPath, "utf-8");
+      } catch {
+        return helpers.error(`File not found: ${args.path}`);
+      }
+      const fileLineEnding = detectLineEnding(content);
+      const normalizedContent = normalizeToLF(content);
+      const normalizedOldString = normalizeToLF(args.old_string);
+      const occurrences = normalizedContent.split(normalizedOldString).length - 1;
+      if (occurrences === 0) {
+        const preview = args.old_string.length > 100 ? args.old_string.slice(0, 100) + "..." : args.old_string;
+        return helpers.error(
+          `old_string not found in file.
 
 Searched for:
 ${preview}
 
 Make sure whitespace and indentation match exactly.`
+        );
+      }
+      if (occurrences > 1 && !replaceAll) {
+        return helpers.error(
+          `Found ${occurrences} matches for old_string. Use replace_all: true to replace all, or provide more context to make the match unique.`
+        );
+      }
+      const contextResult = extractEditContext(
+        normalizedContent,
+        normalizedOldString,
+        normalizeToLF(args.new_string),
+        replaceAll
       );
-    }
-    if (occurrences > 1 && !replaceAll) {
-      return helpers.error(
-        `Found ${occurrences} matches for old_string. Use replace_all: true to replace all, or provide more context to make the match unique.`
+      const editContextRegions = contextResult.regions.map((region) => ({
+        beforeContext: region.beforeContext.map((line) => ({
+          lineNumber: line.lineNumber,
+          content: line.content
+        })),
+        afterContext: region.afterContext.map((line) => ({
+          lineNumber: line.lineNumber,
+          content: line.content
+        })),
+        startLine: region.startLine,
+        endLine: region.endLine,
+        oldContent: region.oldContent,
+        newContent: region.newContent,
+        edits: region.edits.map((edit) => ({
+          startLine: edit.startLine,
+          endLine: edit.endLine,
+          oldContent: edit.oldContent,
+          newContent: edit.newContent
+        }))
+      }));
+      const normalizedNewString = normalizeToLF(args.new_string);
+      let replacedContent;
+      if (replaceAll) {
+        replacedContent = normalizedContent.replaceAll(normalizedOldString, normalizedNewString);
+      } else {
+        const index = normalizedContent.indexOf(normalizedOldString);
+        if (index === -1) {
+          return helpers.error("old_string not found in file (unexpected)");
+        }
+        replacedContent = normalizedContent.slice(0, index) + normalizedNewString + normalizedContent.slice(index + normalizedOldString.length);
+      }
+      const newContent = replacedContent.replace(
+        /\n/g,
+        fileLineEnding === "crlf" ? "\r\n" : fileLineEnding === "cr" ? "\r" : "\n"
       );
-    }
-    const contextResult = extractEditContext(
-      normalizedContent,
-      normalizedOldString,
-      normalizeToLF(args.new_string),
-      replaceAll
-    );
-    const editContextRegions = contextResult.regions.map((region) => ({
-      beforeContext: region.beforeContext.map((line) => ({
-        lineNumber: line.lineNumber,
-        content: line.content
-      })),
-      afterContext: region.afterContext.map((line) => ({
-        lineNumber: line.lineNumber,
-        content: line.content
-      })),
-      startLine: region.startLine,
-      endLine: region.endLine,
-      oldContent: region.oldContent,
-      newContent: region.newContent,
-      edits: region.edits.map((edit) => ({
-        startLine: edit.startLine,
-        endLine: edit.endLine,
-        oldContent: edit.oldContent,
-        newContent: edit.newContent
-      }))
-    }));
-    const normalizedNewString = normalizeToLF(args.new_string);
-    let replacedContent;
-    if (replaceAll) {
-      replacedContent = normalizedContent.replaceAll(normalizedOldString, normalizedNewString);
-    } else {
-      const index = normalizedContent.indexOf(normalizedOldString);
-      if (index === -1) {
-        return helpers.error("old_string not found in file (unexpected)");
+      await writeFile2(fullPath, newContent, "utf-8");
+      let output = `Successfully replaced ${replaceAll ? occurrences : 1} occurrence(s) in ${args.path}`;
+      let diagnostics = [];
+      if (context.lspManager) {
+        diagnostics = await context.lspManager.notifyFileChange(fullPath, newContent);
+        output += formatDiagnosticsForLLM(diagnostics);
       }
-      replacedContent = normalizedContent.slice(0, index) + normalizedNewString + normalizedContent.slice(index + normalizedOldString.length);
-    }
-    const newContent = replacedContent.replace(
-      /\n/g,
-      fileLineEnding === "crlf" ? "\r\n" : fileLineEnding === "cr" ? "\r" : "\n"
-    );
-    await writeFile2(fullPath, newContent, "utf-8");
-    let output = `Successfully replaced ${replaceAll ? occurrences : 1} occurrence(s) in ${args.path}`;
-    let diagnostics = [];
-    if (context.lspManager) {
-      diagnostics = await context.lspManager.notifyFileChange(fullPath, newContent);
-      output += formatDiagnosticsForLLM(diagnostics);
-    }
-    const newHash = await computeFileHash(fullPath);
-    if (newHash) {
-      context.sessionManager.updateFileHash(context.sessionId, fullPath, newHash);
-    }
-    return helpers.success(output, false, {
-      ...diagnostics.length > 0 && { diagnostics },
-      ...editContextRegions.length > 0 && { editContext: { regions: editContextRegions } }
+      const newHash = await computeFileHash(fullPath);
+      if (newHash) {
+        context.sessionManager.updateFileHash(context.sessionId, fullPath, newHash);
+      }
+      return helpers.success(output, false, {
+        ...diagnostics.length > 0 && { diagnostics },
+        ...editContextRegions.length > 0 && { editContext: { regions: editContextRegions } }
+      });
     });
   }
 );
@@ -1938,6 +1955,11 @@ function buildAgentReminder(agentDef) {
 ${agentDef.prompt}${toolPermissions}
 </system-reminder>`;
 }
+function buildAgentSmallReminder(name) {
+  return `<system-reminder>
+Reminder: you are in '${name}' mode.
+</system-reminder>`;
+}
 var WORKFLOW_KICKOFF_PROMPT = (criteriaCount) => `Implement the task and make sure you fulfil the ${criteriaCount} criteria.`;
 var COMPACTION_PROMPT = `You are a helpful AI assistant tasked with summarizing conversations for continuation.
 
@@ -1979,12 +2001,12 @@ function matchRetryPatterns(content, thinking, patterns) {
 
 // src/server/chat/stream-utils.ts
 function buildStreamRequestObject(params) {
-  const { messages, tools, toolChoice, disableThinking, signal, modelSettings } = params;
+  const { messages, tools, toolChoice, reasoningEffort, signal, modelSettings } = params;
   return {
     messages,
     ...tools && { tools },
     ...toolChoice && { toolChoice },
-    disableThinking: disableThinking ?? false,
+    ...reasoningEffort && { reasoningEffort },
     ...signal && { signal },
     ...modelSettings && { modelSettings }
   };
@@ -2038,7 +2060,7 @@ function createEmptyStreamResult(aborted, modelParams, patternMatch) {
   };
 }
 async function* streamLLMPure(options) {
-  const { messageId, systemPrompt, llmClient, messages, tools, toolChoice, signal, disableThinking, retryPatterns } = options;
+  const { messageId, systemPrompt, llmClient, messages, tools, toolChoice, signal, reasoningEffort, retryPatterns } = options;
   const llmMessages = [{ role: "system", content: systemPrompt }, ...messages];
   const profile = getModelProfile(llmClient.getModel());
   const backend = getBackendCapabilities(llmClient.getBackend());
@@ -2071,7 +2093,7 @@ async function* streamLLMPure(options) {
     messages: llmMessages,
     tools,
     toolChoice,
-    disableThinking: disableThinking ?? false,
+    reasoningEffort,
     signal: combinedSignal,
     modelSettings: options.modelSettings
   });
@@ -3083,7 +3105,7 @@ ${COMPACTION_PROMPT}`,
       });
       append(createMessageDoneEvent(assistantMsgId, { stats: turnMetrics.buildStats(statsIdentity, mode) }));
       append(createChatDoneEvent(assistantMsgId, "complete"));
-      config.injectModeReminder?.();
+      config.injectAgentReminder?.();
       compacting = false;
       continue;
     }
@@ -3167,7 +3189,6 @@ function buildAssemblyInput(systemPrompt, baseInput) {
     injectedFiles: baseInput.injectedFiles,
     requestTools: baseInput.requestTools ?? baseInput.promptTools,
     toolChoice: baseInput.toolChoice ?? "auto",
-    disableThinking: baseInput.disableThinking ?? false,
     ...baseInput.customInstructions ? { customInstructions: baseInput.customInstructions } : {},
     ...baseInput.skills && baseInput.skills.length > 0 ? { skills: baseInput.skills } : {}
   });
@@ -3259,7 +3280,7 @@ async function describeImageFromDataUrl(dataUrl, visionModel, options) {
 import { createHash as createHash2 } from "crypto";
 async function loadVisionModelFromGlobalConfig() {
   try {
-    const { loadGlobalConfig, getVisionFallback } = await import("./config-QUT7YZQ7.js");
+    const { loadGlobalConfig, getVisionFallback } = await import("./config-BU66P4KX.js");
     const runtimeConfig = getRuntimeConfig();
     const mode = runtimeConfig.mode ?? "production";
     const globalConfig = await loadGlobalConfig(mode);
@@ -3622,7 +3643,6 @@ async function executeSubAgent(options) {
         injectedFiles: input.injectedFiles,
         requestTools: input.promptTools,
         toolChoice: input.toolChoice,
-        disableThinking: true,
         ...instructionContent ? { customInstructions: instructionContent } : {},
         ...skills.length > 0 ? { skills } : {}
       }),
@@ -3713,7 +3733,7 @@ var callSubAgentTool = {
       };
     }
     try {
-      const { getToolRegistryForAgent: getToolRegistryForAgent2 } = await import("./tools-SDXWIYSA.js");
+      const { getToolRegistryForAgent: getToolRegistryForAgent2 } = await import("./tools-BOE5T3KC.js");
       const toolRegistry = getToolRegistryForAgent2(agentDef);
       const turnMetrics = new TurnMetrics();
       const result = await executeSubAgent({
@@ -4721,6 +4741,7 @@ export {
   buildBasePrompt,
   buildTopLevelSystemPrompt,
   buildAgentReminder,
+  buildAgentSmallReminder,
   WORKFLOW_KICKOFF_PROMPT,
   COMPACTION_PROMPT,
   streamLLMPure,
@@ -4764,4 +4785,4 @@ export {
   getToolRegistryForAgent,
   createToolRegistry
 };
-//# sourceMappingURL=chunk-QN5ST54E.js.map
+//# sourceMappingURL=chunk-32GQUDLN.js.map