openfox 1.6.13 → 1.6.15

package/dist/{chunk-POJO7A5H.js → chunk-3WHZ47PY.js}

@@ -13,12 +13,12 @@ import {
 } from "./chunk-574HZVLE.js";
 import {
   streamWithSegments
-} from "./chunk-XFXOSPYH.js";
+} from "./chunk-55N6FAAZ.js";
 import {
   getContextMessages,
   getCurrentContextWindowId,
   getEventStore
-} from "./chunk-ZRDRYL6W.js";
+} from "./chunk-6L3X7T4K.js";
 import {
   createChatDoneMessage,
   createChatMessageMessage,
@@ -31,7 +31,7 @@ import {
 import {
   AskUserInterrupt,
   askUserTool
-} from "./chunk-5LDTVERY.js";
+} from "./chunk-22CTURMH.js";
 import {
   SETTINGS_KEYS,
   getSetting,
@@ -375,6 +375,31 @@ You may use available tools to read files and verify changes if needed.`;
 var FORMAT_CORRECTION_PROMPT = `IMPORTANT: You MUST use the JSON function calling API. Do NOT output XML tags like <tool_call>, <function=>, or <parameter=>. Your previous attempt was stopped because you used the wrong format. Use the proper tool_calls format.`;
 
 // src/server/chat/request-context.ts
+function minimalMessagesToRequestContextMessages(messages, source = "history") {
+  return messages.map((message) => minimalMessageToRequestContextMessage(message, source));
+}
+function spreadMessageProps(message) {
+  return {
+    ...message.toolCalls ? { toolCalls: message.toolCalls } : {},
+    ...message.toolCallId ? { toolCallId: message.toolCallId } : {},
+    ...message.attachments ? { attachments: message.attachments } : {}
+  };
+}
+function minimalMessageToRequestContextMessage(message, source = "history") {
+  return {
+    role: message.role,
+    content: message.content,
+    source,
+    ...spreadMessageProps(message)
+  };
+}
+function messageToMinimal(message) {
+  return {
+    role: message.role,
+    content: message.content,
+    ...spreadMessageProps(message)
+  };
+}
 function getTriggerUserMessage(messages) {
   const stripRuntimeReminders = (content) => {
     return content.replace(/\n*<system-reminder>[\s\S]*<\/system-reminder>\s*/gi, "").trim();
@@ -396,12 +421,8 @@ function createPromptContext(input) {
     injectedFiles: input.injectedFiles,
     userMessage: input.userMessage ?? getTriggerUserMessage(input.messages),
     messages: input.messages.map((message) => ({
-      role: message.role,
-      content: message.content,
-      source: message.source,
-      ...message.toolCalls ? { toolCalls: message.toolCalls } : {},
-      ...message.toolCallId ? { toolCallId: message.toolCallId } : {},
-      ...message.attachments ? { attachments: message.attachments } : {}
+      ...messageToMinimal(message),
+      source: message.source
     })),
     tools: input.requestTools.map((tool) => ({
       name: tool.function.name,
@@ -419,19 +440,25 @@ function createAssemblyResult(input) {
   const messagesForLLM = input.messages.filter((m) => m.source !== "runtime");
   return {
     systemPrompt: input.systemPrompt,
-    messages: messagesForLLM.map((message) => ({
-      role: message.role,
-      content: message.content,
-      ...message.toolCalls ? { toolCalls: message.toolCalls } : {},
-      ...message.toolCallId ? { toolCallId: message.toolCallId } : {},
-      ...message.attachments ? { attachments: message.attachments } : {}
-    })),
+    messages: messagesForLLM.map((message) => messageToMinimal(message)),
     promptContext: createPromptContext({
       ...input,
       userMessage: triggerUserMessage
     })
   };
 }
+function buildAssemblyInput(systemPrompt, baseInput) {
+  return createAssemblyResult({
+    systemPrompt,
+    messages: baseInput.messages,
+    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 } : {}
+  });
+}
 function assembleAgentRequest(input) {
   const { agentDef, subAgentDefs, ...baseInput } = input;
   if (agentDef.metadata.subagent) {
@@ -440,14 +467,7 @@ function assembleAgentRequest(input) {
       agentDef,
       baseInput.skills
     );
-    return createAssemblyResult({
-      systemPrompt: systemPrompt2,
-      messages: baseInput.messages,
-      injectedFiles: baseInput.injectedFiles,
-      requestTools: baseInput.requestTools ?? baseInput.promptTools,
-      toolChoice: baseInput.toolChoice ?? "auto",
-      disableThinking: baseInput.disableThinking ?? false
-    });
+    return buildAssemblyInput(systemPrompt2, baseInput);
   }
   const systemPrompt = buildTopLevelSystemPrompt(
     baseInput.workdir,
@@ -455,14 +475,25 @@ function assembleAgentRequest(input) {
     baseInput.skills,
     subAgentDefs
   );
-  return createAssemblyResult({
-    systemPrompt,
-    messages: baseInput.messages,
-    injectedFiles: baseInput.injectedFiles,
-    requestTools: baseInput.requestTools ?? baseInput.promptTools,
-    toolChoice: baseInput.toolChoice ?? "auto",
-    disableThinking: baseInput.disableThinking ?? false
-  });
+  return buildAssemblyInput(systemPrompt, baseInput);
+}
+
+// src/server/chat/stream-utils.ts
+function buildStreamRequestObject(params) {
+  const { messages, tools, toolChoice, disableThinking, signal, onVisionFallbackStart, onVisionFallbackDone } = params;
+  const streamRequest = {
+    messages,
+    ...tools && { tools },
+    ...toolChoice && { toolChoice },
+    disableThinking: disableThinking ?? false,
+    ...signal && { signal }
+  };
+  if (onVisionFallbackStart) streamRequest.onVisionFallbackStart = onVisionFallbackStart;
+  if (onVisionFallbackDone) streamRequest.onVisionFallbackDone = onVisionFallbackDone;
+  return streamRequest;
+}
+function buildStreamRequest(client, options) {
+  return streamWithSegments(client, buildStreamRequestObject(options));
 }
 
 // src/server/chat/stats.ts
@@ -483,19 +514,29 @@ function computeAggregatedStats(input) {
 }
 
 // src/server/chat/stream-pure.ts
+function createEmptyStreamResult(aborted, xmlFormatError) {
+  return {
+    content: "",
+    toolCalls: [],
+    segments: [],
+    usage: { promptTokens: 0, completionTokens: 0 },
+    timing: { ttft: 0, completionTime: 0, tps: 0, prefillTps: 0 },
+    aborted,
+    xmlFormatError
+  };
+}
 async function* streamLLMPure(options) {
   const { messageId, systemPrompt, llmClient, messages, tools, toolChoice, signal, disableThinking, onVisionFallbackStart, onVisionFallbackDone } = options;
   const llmMessages = [{ role: "system", content: systemPrompt }, ...messages];
-  const streamRequest = {
+  const stream = buildStreamRequest(llmClient, {
     messages: llmMessages,
-    ...tools && { tools },
-    ...tools && { toolChoice: toolChoice ?? "auto" },
+    tools,
+    toolChoice,
     disableThinking: disableThinking ?? false,
-    ...signal && { signal }
-  };
-  if (onVisionFallbackStart) streamRequest.onVisionFallbackStart = onVisionFallbackStart;
-  if (onVisionFallbackDone) streamRequest.onVisionFallbackDone = onVisionFallbackDone;
-  const stream = streamWithSegments(llmClient, streamRequest);
+    signal,
+    onVisionFallbackStart,
+    onVisionFallbackDone
+  });
   const seenToolIndices = /* @__PURE__ */ new Set();
   const toolNames = /* @__PURE__ */ new Map();
   const toolIds = /* @__PURE__ */ new Map();
@@ -594,15 +635,7 @@ async function* streamLLMPure(options) {
     }
   }
   if (!result) {
-    return {
-      content: "",
-      toolCalls: [],
-      segments: [],
-      usage: { promptTokens: 0, completionTokens: 0 },
-      timing: { ttft: 0, completionTime: 0, tps: 0, prefillTps: 0 },
-      aborted,
-      xmlFormatError
-    };
+    return createEmptyStreamResult(aborted, xmlFormatError);
   }
   const baseResult = {
     content: result.content,
@@ -768,15 +801,7 @@ async function consumeStreamWithToolLoop(options) {
   let iterations = 0;
   for (; ; ) {
     if (signal?.aborted) {
-      return {
-        content: "",
-        toolCalls: [],
-        segments: [],
-        usage: { promptTokens: 0, completionTokens: 0 },
-        timing: { ttft: 0, completionTime: 0, tps: 0, prefillTps: 0 },
-        aborted: true,
-        xmlFormatError: false
-      };
+      return createEmptyStreamResult(true, false);
     }
     if (++iterations > MAX_TOOL_LOOP_ITERATIONS) {
       throw new Error("Max tool loop iterations exceeded during compaction");
@@ -844,7 +869,7 @@ async function consumeStreamWithToolLoop(options) {
 }
 async function executeToolBatchWithContext(assistantMsgId, toolCalls, ctx, onEvent) {
   const toolMessages = [];
-  let criteriaChanged = false;
+  const criteriaChanged = false;
   for (const toolCall of toolCalls) {
     if (ctx.signal?.aborted) {
       throw new Error("Aborted");
@@ -884,17 +909,48 @@ Error: ${toolResult.error}` : `Error: ${toolResult.error}`;
 }
 
 // src/server/agents/registry.ts
-import { readdir, readFile as readFile2, writeFile, copyFile, mkdir, access as access2, unlink } from "fs/promises";
-import { join as join2, dirname as dirname2 } from "path";
+import { readdir, readFile as readFile3, writeFile, copyFile, mkdir, access as access2, unlink } from "fs/promises";
+import { join as join3, dirname as dirname2 } from "path";
 import { constants as constants2 } from "fs";
 import { fileURLToPath } from "url";
 import matter from "gray-matter";
+
+// src/server/utils/defaults.ts
+import { readFile as readFile2 } from "fs/promises";
+import { join as join2 } from "path";
+async function findModifiedDefaultFiles(defaultIds, extension, bundledDirs, userDir) {
+  const [primaryDir, altDir] = bundledDirs;
+  const modified = [];
+  for (const id of defaultIds) {
+    const filename = `${id}${extension}`;
+    const userPath = join2(userDir, filename);
+    let bundledContent = null;
+    for (const dir of [primaryDir, altDir]) {
+      try {
+        bundledContent = await readFile2(join2(dir, filename), "utf-8");
+        break;
+      } catch {
+      }
+    }
+    if (!bundledContent) continue;
+    try {
+      const userContent = await readFile2(userPath, "utf-8");
+      if (userContent !== bundledContent) {
+        modified.push(id);
+      }
+    } catch {
+    }
+  }
+  return modified;
+}
+
+// src/server/agents/registry.ts
 var __bundleDir = dirname2(fileURLToPath(import.meta.url));
-var DEFAULTS_DIR = join2(__bundleDir, "defaults");
-var DEFAULTS_DIR_ALT = join2(__bundleDir, "agent-defaults");
+var DEFAULTS_DIR = join3(__bundleDir, "defaults");
+var DEFAULTS_DIR_ALT = join3(__bundleDir, "agent-defaults");
 var AGENT_EXTENSION = ".agent.md";
 function getAgentsDir(configDir) {
-  return join2(configDir, "agents");
+  return join3(configDir, "agents");
 }
 async function pathExists(path) {
   try {
@@ -924,10 +980,10 @@ async function ensureDefaultAgents(configDir) {
     }
   }
   for (const file of defaultFiles) {
-    const targetPath = join2(agentsDir, file);
+    const targetPath = join3(agentsDir, file);
     if (!await pathExists(targetPath)) {
       try {
-        await copyFile(join2(sourceDir, file), targetPath);
+        await copyFile(join3(sourceDir, file), targetPath);
         logger.info("Installed default agent", { file });
       } catch (err) {
         logger.error("Failed to copy default agent", { file, error: err instanceof Error ? err.message : String(err) });
@@ -966,7 +1022,7 @@ async function loadAgentsFromDir(dir) {
   const agents = [];
   for (const file of files) {
     try {
-      const raw = await readFile2(join2(dir, file), "utf-8");
+      const raw = await readFile3(join3(dir, file), "utf-8");
       const agent = parseAgentFile(raw, file);
       if (agent) {
         agents.push(agent);
@@ -1017,7 +1073,7 @@ async function getDefaultAgentIds() {
 async function restoreDefaultAgent(configDir, agentId) {
   const defaultIds = await getDefaultAgentIds();
   if (!defaultIds.includes(agentId)) return false;
-  const userPath = join2(getAgentsDir(configDir), `${agentId}${AGENT_EXTENSION}`);
+  const userPath = join3(getAgentsDir(configDir), `${agentId}${AGENT_EXTENSION}`);
   try {
     await unlink(userPath);
   } catch {
@@ -1026,28 +1082,7 @@ async function restoreDefaultAgent(configDir, agentId) {
 }
 async function getModifiedDefaultAgentIds(configDir) {
   const defaultIds = await getDefaultAgentIds();
-  const modified = [];
-  for (const id of defaultIds) {
-    const filename = `${id}${AGENT_EXTENSION}`;
-    const userPath = join2(getAgentsDir(configDir), filename);
-    let bundledContent = null;
-    for (const dir of [DEFAULTS_DIR, DEFAULTS_DIR_ALT]) {
-      try {
-        bundledContent = await readFile2(join2(dir, filename), "utf-8");
-        break;
-      } catch {
-      }
-    }
-    if (!bundledContent) continue;
-    try {
-      const userContent = await readFile2(userPath, "utf-8");
-      if (userContent !== bundledContent) {
-        modified.push(id);
-      }
-    } catch {
-    }
-  }
-  return modified;
+  return findModifiedDefaultFiles(defaultIds, AGENT_EXTENSION, [DEFAULTS_DIR, DEFAULTS_DIR_ALT], getAgentsDir(configDir));
 }
 async function restoreAllDefaultAgents(configDir) {
   const ids = await getDefaultAgentIds();
@@ -1057,6 +1092,13 @@ async function restoreAllDefaultAgents(configDir) {
   }
   return count;
 }
+function getAgentFilePaths(agentsDir, agentId) {
+  const hyphenated = agentId.replace(/_/g, "-");
+  return [
+    join3(agentsDir, `${agentId}${AGENT_EXTENSION}`),
+    join3(agentsDir, `${hyphenated}${AGENT_EXTENSION}`)
+  ];
+}
 function findAgentById(agentId, agents) {
   return agents.find((a) => a.metadata.id === agentId);
 }
@@ -1065,11 +1107,7 @@ function getSubAgents(agents) {
 }
 async function agentExists(configDir, agentId) {
   const agentsDir = getAgentsDir(configDir);
-  const hyphenated = agentId.replace(/_/g, "-");
-  const paths = [
-    join2(agentsDir, `${agentId}${AGENT_EXTENSION}`),
-    join2(agentsDir, `${hyphenated}${AGENT_EXTENSION}`)
-  ];
+  const paths = getAgentFilePaths(agentsDir, agentId);
   for (const filePath of paths) {
     if (await pathExists(filePath)) {
       return true;
@@ -1082,17 +1120,13 @@ async function saveAgent(configDir, agent) {
   if (!await pathExists(agentsDir)) {
     await mkdir(agentsDir, { recursive: true });
   }
-  const filePath = join2(agentsDir, `${agent.metadata.id}${AGENT_EXTENSION}`);
+  const filePath = join3(agentsDir, `${agent.metadata.id}${AGENT_EXTENSION}`);
   const content = matter.stringify(agent.prompt, agent.metadata);
   await writeFile(filePath, content, "utf-8");
 }
 async function deleteAgent(configDir, agentId) {
   const agentsDir = getAgentsDir(configDir);
-  const hyphenated = agentId.replace(/_/g, "-");
-  const paths = [
-    join2(agentsDir, `${agentId}${AGENT_EXTENSION}`),
-    join2(agentsDir, `${hyphenated}${AGENT_EXTENSION}`)
-  ];
+  const paths = getAgentFilePaths(agentsDir, agentId);
   for (const filePath of paths) {
     try {
       await unlink(filePath);
@@ -1105,7 +1139,7 @@ async function deleteAgent(configDir, agentId) {
 }
 
 // src/server/tools/read.ts
-import { readFile as readFile4, stat as stat2 } from "fs/promises";
+import { readFile as readFile5, stat as stat2 } from "fs/promises";
 import { extname } from "path";
 
 // src/server/tools/types.ts
@@ -1136,7 +1170,7 @@ import { resolve as resolve2, isAbsolute } from "path";
 
 // src/server/tools/path-security.ts
 import { realpath } from "fs/promises";
-import { resolve, normalize, join as join3, basename, sep } from "path";
+import { resolve, normalize, join as join4, basename, sep } from "path";
 import { homedir, tmpdir } from "os";
 var SAFE_PATHS = /* @__PURE__ */ new Set([
   "/dev/null",
@@ -1271,7 +1305,7 @@ function extractAbsolutePathsFromCommand(command) {
       }
     } else if (content.startsWith("~")) {
       const pathPart = content.slice(1);
-      const fullPath = join3(home, pathPart);
+      const fullPath = join4(home, pathPart);
       const resolved = normalize(resolve(fullPath));
       if (!isSafePath(resolved)) {
         paths.push(resolved);
@@ -1416,9 +1450,6 @@ var PathAccessDeniedError = class extends Error {
     this.reason = reason;
     this.name = "PathAccessDeniedError";
   }
-  paths;
-  tool;
-  reason;
 };
 var pendingConfirmations = /* @__PURE__ */ new Map();
 function registerPathConfirmation(callId, paths, sessionId) {
@@ -1464,6 +1495,39 @@ function cancelPathConfirmationsForSession(sessionId, reason) {
 }
 
 // src/server/tools/tool-helpers.ts
+function validateAction(action, allowed, startTime) {
+  if (!action || !allowed.includes(action)) {
+    return {
+      success: false,
+      error: `Invalid action: ${action}. Must be one of: ${allowed.join(", ")}`,
+      durationMs: Date.now() - startTime,
+      truncated: false
+    };
+  }
+  return void 0;
+}
+function checkActionPermission(action, permittedActions, startTime) {
+  if (action && permittedActions && !permittedActions.includes(action)) {
+    return {
+      success: false,
+      error: `Action '${action}' not allowed. Available: ${permittedActions.join(", ")}`,
+      durationMs: Date.now() - startTime,
+      truncated: false
+    };
+  }
+  return void 0;
+}
+function requireSession(sessionManager, sessionId) {
+  return sessionManager.requireSession(sessionId);
+}
+function validateActionWithPermission(action, allowedActions, toolName, permittedActions, startTime) {
+  const actionError = validateAction(action, allowedActions, startTime ?? Date.now());
+  if (actionError) return actionError;
+  const permittedToolActions = permittedActions?.[toolName];
+  const permissionError = checkActionPermission(action, permittedToolActions, startTime ?? Date.now());
+  if (permissionError) return permissionError;
+  return void 0;
+}
 function createTool(name, definition, handler2) {
   return {
     name,
@@ -1519,7 +1583,7 @@ function createTool(name, definition, handler2) {
 
 // src/server/tools/file-tracker.ts
 import { createHash } from "crypto";
-import { readFile as readFile3 } from "fs/promises";
+import { readFile as readFile4 } from "fs/promises";
 import { resolve as resolve3 } from "path";
 var FileNotReadError = class extends Error {
   constructor(path) {
@@ -1535,7 +1599,7 @@ var FileChangedExternallyError = class extends Error {
 };
 async function computeFileHash(filePath) {
   try {
-    const content = await readFile3(filePath);
+    const content = await readFile4(filePath);
     const hash = createHash("sha256");
     hash.update(content);
     return hash.digest("hex");
@@ -1668,7 +1732,7 @@ var readFileTool = createTool(
     } catch {
       return helpers.error(`File not found: ${args.path}`);
     }
-    const rawBuffer = await readFile4(fullPath);
+    const rawBuffer = await readFile5(fullPath);
     const mimeType = detectImageType(rawBuffer, args.path);
     if (mimeType) {
       const base64Data = rawBuffer.toString("base64");
@@ -1802,7 +1866,7 @@ var writeFileTool = createTool(
 );
 
 // src/server/tools/edit.ts
-import { readFile as readFile5, writeFile as writeFile3 } from "fs/promises";
+import { readFile as readFile6, writeFile as writeFile3 } from "fs/promises";
 
 // src/server/tools/edit-context.ts
 var CONTEXT_LINES = 4;
@@ -1958,7 +2022,7 @@ var editFileTool = createTool(
     }
     let content;
     try {
-      content = await readFile5(fullPath, "utf-8");
+      content = await readFile6(fullPath, "utf-8");
     } catch {
       return helpers.error(`File not found: ${args.path}`);
     }
@@ -2024,11 +2088,26 @@ Make sure whitespace and indentation match exactly.`
 );
 
 // src/server/tools/shell.ts
-import { spawn as spawn2 } from "child_process";
+import "child_process";
 import { resolve as resolve4, isAbsolute as isAbsolute2 } from "path";
 
-// src/server/utils/process-tree.ts
+// src/server/utils/shell.ts
 import { spawn } from "child_process";
+function checkAborted(signal) {
+  return !!signal?.aborted;
+}
+function spawnShellProcess(command, cwd, signal, detached = false) {
+  const shell = getPlatformShell();
+  return spawn(shell.command, [...shell.args, command], {
+    cwd,
+    env: { ...process.env, FORCE_COLOR: "0" },
+    stdio: ["ignore", "pipe", "pipe"],
+    ...detached ? { detached: true } : {}
+  });
+}
+
+// src/server/utils/process-tree.ts
+import { spawn as spawn2 } from "child_process";
 import { setTimeout as sleep } from "timers/promises";
 var SIGKILL_TIMEOUT_MS = 200;
 async function terminateProcessTree(proc, options) {
@@ -2038,7 +2117,7 @@ async function terminateProcessTree(proc, options) {
   }
   if (process.platform === "win32") {
     await new Promise((resolve6) => {
-      const killer = spawn("taskkill", ["/pid", String(pid), "/f", "/t"], {
+      const killer = spawn2("taskkill", ["/pid", String(pid), "/f", "/t"], {
         stdio: "ignore",
         windowsHide: true
       });
@@ -2142,18 +2221,11 @@ ${result.stderr}`;
 );
 function executeCommand(command, cwd, timeout, signal, onProgress) {
   return new Promise((resolve6, reject) => {
-    if (signal?.aborted) {
+    if (checkAborted(signal)) {
       reject(new Error("Command aborted before execution"));
       return;
     }
-    const shell = getPlatformShell();
-    const proc = spawn2(shell.command, [...shell.args, command], {
-      cwd,
-      env: { ...process.env, FORCE_COLOR: "0" },
-      stdio: ["ignore", "pipe", "pipe"],
-      detached: true
-      // Create new process group so we can kill all children
-    });
+    const proc = spawnShellProcess(command, cwd, signal, true);
     let stdout = "";
     let stderr = "";
     let killed = false;
@@ -2171,12 +2243,12 @@ function executeCommand(command, cwd, timeout, signal, onProgress) {
       }
     };
     signal?.addEventListener("abort", onAbort);
-    proc.stdout.on("data", (data) => {
+    proc.stdout?.on("data", (data) => {
       const chunk = data.toString();
       stdout += chunk;
       onProgress?.(`[stdout] ${chunk}`);
     });
-    proc.stderr.on("data", (data) => {
+    proc.stderr?.on("data", (data) => {
       const chunk = data.toString();
       stderr += chunk;
       onProgress?.(`[stderr] ${chunk}`);
@@ -2219,10 +2291,27 @@ function formatCriteriaList(criteria) {
   if (criteria.length === 0) return "No criteria defined.";
   return criteria.map((c, i) => `${i + 1}. [${c.id}] ${c.description}`).join("\n");
 }
-var criterionTool = {
-  name: "criterion",
-  permittedActions: ["get", "add", "update", "remove", "complete", "pass", "fail"],
-  definition: {
+function completeCriterion(session, context, id, statusType, reason) {
+  const criterion = session.criteria.find((c) => c.id === id);
+  if (!criterion) {
+    return { success: false, output: `Criterion "${id}" not found` };
+  }
+  const status = statusType === "passed" ? { type: "passed", verifiedAt: (/* @__PURE__ */ new Date()).toISOString(), ...reason && { reason } } : { type: "failed", failedAt: (/* @__PURE__ */ new Date()).toISOString(), reason };
+  context.sessionManager.updateCriterionStatus(context.sessionId, id, status);
+  context.sessionManager.addCriterionAttempt(context.sessionId, id, {
+    attemptNumber: criterion.attempts.length + 1,
+    status: statusType,
+    timestamp: (/* @__PURE__ */ new Date()).toISOString(),
+    ...reason ? { details: reason } : {}
+  });
+  return {
+    success: true,
+    output: statusType === "passed" ? `Criterion "${id}" verified as PASSED.${reason ? ` Verification: ${reason}` : ""}` : `Criterion "${id}" marked as FAILED. Reason: ${reason}`
+  };
+}
+var criterionTool = createTool(
+  "criterion",
+  {
     type: "function",
     function: {
       name: "criterion",
@@ -2252,267 +2341,90 @@ var criterionTool = {
       }
     }
   },
-  async execute(args, context) {
-    const startTime = Date.now();
-    try {
-      const action = args["action"];
-      const id = args["id"];
-      const description = args["description"];
-      const reason = args["reason"];
-      if (!action || !["get", "add", "update", "remove", "complete", "pass", "fail"].includes(action)) {
-        return {
-          success: false,
-          error: `Invalid action: ${action}. Must be one of: get, add, update, remove, complete, pass, fail`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      const permittedActions = context.permittedActions?.["criterion"];
-      if (permittedActions && !permittedActions.includes(action)) {
-        return {
-          success: false,
-          error: `Action '${action}' not allowed. Available: ${permittedActions.join(", ")}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      const session = context.sessionManager.requireSession(context.sessionId);
-      if (action === "get") {
-        const criteria = session.criteria;
-        return {
-          success: true,
-          output: criteria.length === 0 ? "No criteria defined yet." : JSON.stringify(criteria.map((c) => ({
-            id: c.id,
-            description: c.description
-          })), null, 2),
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "add") {
-        if (!description) {
-          return {
-            success: false,
-            error: "Missing required field: description",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const criterion = {
-          id: id || "",
-          description,
-          status: { type: "pending" },
-          attempts: []
-        };
-        const result = context.sessionManager.addCriterion(context.sessionId, criterion);
-        if ("error" in result) {
-          return { success: false, error: result.error, durationMs: Date.now() - startTime, truncated: false };
-        }
-        return {
-          success: true,
-          output: `Added criterion "${result.actualId}". Current criteria:
-${formatCriteriaList(result.criteria)}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "update") {
-        if (!id) {
-          return {
-            success: false,
-            error: "Missing required field: id",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (!description) {
-          return {
-            success: false,
-            error: "Missing required field: description",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (!session.criteria.find((c) => c.id === id)) {
-          return {
-            success: false,
-            error: `Criterion "${id}" not found`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const criteria = context.sessionManager.updateCriterionFull(context.sessionId, id, { description });
-        return {
-          success: true,
-          output: `Updated criterion "${id}". Current criteria:
-${formatCriteriaList(criteria)}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "remove") {
-        if (!id) {
-          return {
-            success: false,
-            error: "Missing required field: id",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (!session.criteria.find((c) => c.id === id)) {
-          return {
-            success: false,
-            error: `Criterion "${id}" not found`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const criteria = context.sessionManager.removeCriterion(context.sessionId, id);
-        return {
-          success: true,
-          output: criteria.length === 0 ? `Removed criterion "${id}". No criteria remaining.` : `Removed criterion "${id}". Current criteria:
-${formatCriteriaList(criteria)}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "complete") {
-        if (!id) {
-          return {
-            success: false,
-            error: "Missing required field: id",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const criterion = session.criteria.find((c) => c.id === id);
-        if (!criterion) {
-          return {
-            success: false,
-            error: `Criterion "${id}" not found. Available: ${session.criteria.map((c) => c.id).join(", ")}`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const status = {
-          type: "completed",
-          completedAt: (/* @__PURE__ */ new Date()).toISOString(),
-          ...reason ? { reason } : {}
-        };
-        context.sessionManager.updateCriterionStatus(context.sessionId, id, status);
-        return {
-          success: true,
-          output: `Criterion "${id}" marked as completed.${reason ? ` Reason: ${reason}` : ""}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "pass") {
-        if (!id) {
-          return {
-            success: false,
-            error: "Missing required field: id",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const criterion = session.criteria.find((c) => c.id === id);
-        if (!criterion) {
-          return {
-            success: false,
-            error: `Criterion "${id}" not found`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const status = {
-          type: "passed",
-          verifiedAt: (/* @__PURE__ */ new Date()).toISOString(),
-          ...reason ? { reason } : {}
-        };
-        context.sessionManager.updateCriterionStatus(context.sessionId, id, status);
-        context.sessionManager.addCriterionAttempt(context.sessionId, id, {
-          attemptNumber: criterion.attempts.length + 1,
-          status: "passed",
-          timestamp: (/* @__PURE__ */ new Date()).toISOString(),
-          ...reason && { details: reason }
-        });
-        return {
-          success: true,
-          output: `Criterion "${id}" verified as PASSED.${reason ? ` Verification: ${reason}` : ""}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "fail") {
-        if (!id) {
-          return {
-            success: false,
-            error: "Missing required field: id",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (!reason) {
-          return {
-            success: false,
-            error: "Missing required field: reason",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const criterion = session.criteria.find((c) => c.id === id);
-        if (!criterion) {
-          return {
-            success: false,
-            error: `Criterion "${id}" not found`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const status = {
-          type: "failed",
-          failedAt: (/* @__PURE__ */ new Date()).toISOString(),
-          reason
-        };
-        context.sessionManager.updateCriterionStatus(context.sessionId, id, status);
-        context.sessionManager.addCriterionAttempt(context.sessionId, id, {
-          attemptNumber: criterion.attempts.length + 1,
-          status: "failed",
-          timestamp: (/* @__PURE__ */ new Date()).toISOString(),
-          details: reason
-        });
-        return {
-          success: true,
-          output: `Criterion "${id}" marked as FAILED. Reason: ${reason}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      return {
-        success: false,
-        error: "Unexpected error",
-        durationMs: Date.now() - startTime,
-        truncated: false
-      };
-    } catch (error) {
-      return {
-        success: false,
-        error: error instanceof Error ? error.message : "Unknown error",
-        durationMs: Date.now() - startTime,
-        truncated: false
+  async (args, context, helpers) => {
+    const actionError = validateActionWithPermission(args.action, ["get", "add", "update", "remove", "complete", "pass", "fail"], "criterion", context.permittedActions);
+    if (actionError) return actionError;
+    const session = requireSession(context.sessionManager, context.sessionId);
+    if (args.action === "get") {
+      return helpers.success(
+        session.criteria.length === 0 ? "No criteria defined yet." : JSON.stringify(session.criteria.map((c) => ({ id: c.id, description: c.description })), null, 2)
+      );
+    }
+    if (args.action === "add") {
+      if (!args.description) return helpers.error("Missing required field: description");
+      const criterion = {
+        id: args.id || "",
+        description: args.description,
+        status: { type: "pending" },
+        attempts: []
       };
+      const result = context.sessionManager.addCriterion(context.sessionId, criterion);
+      if ("error" in result) return helpers.error(result.error);
+      return helpers.success(`Added criterion "${result.actualId}". Current criteria:
+${formatCriteriaList(result.criteria)}`);
+    }
+    if (args.action === "update") {
+      if (!args.id) return helpers.error("Missing required field: id");
+      if (!args.description) return helpers.error("Missing required field: description");
+      if (!session.criteria.find((c) => c.id === args.id)) return helpers.error(`Criterion "${args.id}" not found`);
+      const criteria = context.sessionManager.updateCriterionFull(context.sessionId, args.id, { description: args.description });
+      return helpers.success(`Updated criterion "${args.id}". Current criteria:
+${formatCriteriaList(criteria)}`);
+    }
+    if (args.action === "remove") {
+      if (!args.id) return helpers.error("Missing required field: id");
+      if (!session.criteria.find((c) => c.id === args.id)) return helpers.error(`Criterion "${args.id}" not found`);
+      const criteria = context.sessionManager.removeCriterion(context.sessionId, args.id);
+      return helpers.success(
+        criteria.length === 0 ? `Removed criterion "${args.id}". No criteria remaining.` : `Removed criterion "${args.id}". Current criteria:
+${formatCriteriaList(criteria)}`
+      );
+    }
+    if (args.action === "complete") {
+      if (!args.id) return helpers.error("Missing required field: id");
+      const criterion = session.criteria.find((c) => c.id === args.id);
+      if (!criterion) return helpers.error(`Criterion "${args.id}" not found. Available: ${session.criteria.map((c) => c.id).join(", ")}`);
+      context.sessionManager.updateCriterionStatus(context.sessionId, args.id, {
+        type: "completed",
+        completedAt: (/* @__PURE__ */ new Date()).toISOString(),
+        ...args.reason ? { reason: args.reason } : {}
+      });
+      return helpers.success(`Criterion "${args.id}" marked as completed.${args.reason ? ` Reason: ${args.reason}` : ""}`);
+    }
+    if (args.action === "pass") {
+      if (!args.id) return helpers.error("Missing required field: id");
+      const result = completeCriterion(session, context, args.id, "passed", args.reason);
+      return result.success ? helpers.success(result.output) : helpers.error(result.output);
     }
+    if (args.action === "fail") {
+      if (!args.id) return helpers.error("Missing required field: id");
+      if (!args.reason) return helpers.error("Missing required field: reason");
+      const result = completeCriterion(session, context, args.id, "failed", args.reason);
+      return result.success ? helpers.success(result.output) : helpers.error(result.output);
+    }
+    return helpers.error("Unexpected error");
   }
-};
+);
 
 // src/server/tools/todo.ts
 var sessionTodos = /* @__PURE__ */ new Map();
 var onTodoUpdate = null;
-var todoTool = {
-  name: "todo",
-  permittedActions: ["list", "write", "add", "update", "remove"],
-  definition: {
+function buildTaskListSummary(todosList) {
+  const pending = todosList.filter((t) => t.status === "pending").length;
+  const inProgress = todosList.filter((t) => t.status === "in_progress").length;
+  const completed = todosList.filter((t) => t.status === "completed").length;
+  return `${completed} completed, ${inProgress} in progress, ${pending} pending`;
+}
+function validateIndex(index, todosList, helpers) {
+  if (index === void 0) return helpers.error("Missing required field: index");
+  if (index < 0 || index >= todosList.length) {
+    return helpers.error(`Index out of range: ${index}. Valid range: 0-${todosList.length - 1}`);
+  }
+  return todosList[index];
+}
+var todoTool = createTool(
+  "todo",
+  {
     type: "function",
     function: {
       name: "todo",
@@ -2555,235 +2467,65 @@ var todoTool = {
       }
     }
   },
-  async execute(args, context) {
-    const startTime = Date.now();
-    try {
-      const action = args["action"];
-      const todos = args["todos"];
-      const content = args["content"];
-      const index = args["index"];
-      const status = args["status"];
-      if (!action || !["list", "write", "add", "update", "remove"].includes(action)) {
-        return {
-          success: false,
-          error: `Invalid action: ${action}. Must be one of: list, write, add, update, remove`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      const permittedActions = context.permittedActions?.["todo"];
-      if (permittedActions && !permittedActions.includes(action)) {
-        return {
-          success: false,
-          error: `Action '${action}' not allowed. Available: ${permittedActions.join(", ")}`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "list") {
-        const todosList = sessionTodos.get(context.sessionId) ?? [];
-        return {
-          success: true,
-          output: todosList.length === 0 ? "No tasks defined yet." : JSON.stringify(todosList, null, 2),
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "write") {
-        if (!todos) {
-          return {
-            success: false,
-            error: "Missing required field: todos",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (!Array.isArray(todos)) {
-          return {
-            success: false,
-            error: "todos must be an array",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        for (const todo of todos) {
-          if (!todo.content || !todo.status) {
-            return {
-              success: false,
-              error: "Each todo must have content and status",
-              durationMs: Date.now() - startTime,
-              truncated: false
-            };
-          }
-          if (!["pending", "in_progress", "completed"].includes(todo.status)) {
-            return {
-              success: false,
-              error: `Invalid status: ${todo.status}. Must be pending, in_progress, or completed`,
-              durationMs: Date.now() - startTime,
-              truncated: false
-            };
-          }
-        }
-        sessionTodos.set(context.sessionId, todos);
-        if (onTodoUpdate) {
-          onTodoUpdate(context.sessionId, todos);
-        }
-        const pending = todos.filter((t) => t.status === "pending").length;
-        const inProgress = todos.filter((t) => t.status === "in_progress").length;
-        const completed = todos.filter((t) => t.status === "completed").length;
-        return {
-          success: true,
-          output: `Task list updated: ${completed} completed, ${inProgress} in progress, ${pending} pending`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "add") {
-        if (!content) {
-          return {
-            success: false,
-            error: "Missing required field: content",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const todosList = sessionTodos.get(context.sessionId) ?? [];
-        const newTodo = { content, status: "pending" };
-        todosList.push(newTodo);
-        sessionTodos.set(context.sessionId, todosList);
-        if (onTodoUpdate) {
-          onTodoUpdate(context.sessionId, todosList);
-        }
-        const pending = todosList.filter((t) => t.status === "pending").length;
-        const inProgress = todosList.filter((t) => t.status === "in_progress").length;
-        const completed = todosList.filter((t) => t.status === "completed").length;
-        return {
-          success: true,
-          output: `Added task "${content}". Task list: ${completed} completed, ${inProgress} in progress, ${pending} pending`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
-      }
-      if (action === "update") {
-        if (index === void 0) {
-          return {
-            success: false,
-            error: "Missing required field: index",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (!content && !status) {
-          return {
-            success: false,
-            error: "update requires content or status",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (status && !["pending", "in_progress", "completed"].includes(status)) {
-          return {
-            success: false,
-            error: `Invalid status: ${status}. Must be pending, in_progress, or completed`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const todosList = sessionTodos.get(context.sessionId) ?? [];
-        if (index < 0 || index >= todosList.length) {
-          return {
-            success: false,
-            error: `Index out of range: ${index}. Valid range: 0-${todosList.length - 1}`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const todo = todosList[index];
-        if (!todo) {
-          return {
-            success: false,
-            error: `Index out of range: ${index}. Valid range: 0-${todosList.length - 1}`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        if (content) {
-          todo.content = content;
-        }
-        if (status) {
-          todo.status = status;
-        }
-        sessionTodos.set(context.sessionId, todosList);
-        if (onTodoUpdate) {
-          onTodoUpdate(context.sessionId, todosList);
+  async (args, context, helpers) => {
+    const actionError = validateActionWithPermission(args.action, ["list", "write", "add", "update", "remove"], "todo", context.permittedActions);
+    if (actionError) return actionError;
+    if (args.action === "list") {
+      const todosList = sessionTodos.get(context.sessionId) ?? [];
+      return helpers.success(
+        todosList.length === 0 ? "No tasks defined yet." : JSON.stringify(todosList, null, 2)
+      );
+    }
+    if (args.action === "write") {
+      if (!args.todos) return helpers.error("Missing required field: todos");
+      if (!Array.isArray(args.todos)) return helpers.error("todos must be an array");
+      for (const todo of args.todos) {
+        if (!todo.content || !todo.status) return helpers.error("Each todo must have content and status");
+        if (!["pending", "in_progress", "completed"].includes(todo.status)) {
+          return helpers.error(`Invalid status: ${todo.status}. Must be pending, in_progress, or completed`);
         }
-        const pending = todosList.filter((t) => t.status === "pending").length;
-        const inProgress = todosList.filter((t) => t.status === "in_progress").length;
-        const completed = todosList.filter((t) => t.status === "completed").length;
-        return {
-          success: true,
-          output: `Updated task ${index}. Task list: ${completed} completed, ${inProgress} in progress, ${pending} pending`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
       }
-      if (action === "remove") {
-        if (index === void 0) {
-          return {
-            success: false,
-            error: "Missing required field: index",
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const todosList = sessionTodos.get(context.sessionId) ?? [];
-        if (index < 0 || index >= todosList.length) {
-          return {
-            success: false,
-            error: `Index out of range: ${index}. Valid range: 0-${todosList.length - 1}`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const removed = todosList.splice(index, 1)[0];
-        sessionTodos.set(context.sessionId, todosList);
-        if (onTodoUpdate) {
-          onTodoUpdate(context.sessionId, todosList);
-        }
-        if (todosList.length === 0) {
-          return {
-            success: true,
-            output: `Removed task "${removed.content}". No tasks remaining.`,
-            durationMs: Date.now() - startTime,
-            truncated: false
-          };
-        }
-        const pending = todosList.filter((t) => t.status === "pending").length;
-        const inProgress = todosList.filter((t) => t.status === "in_progress").length;
-        const completed = todosList.filter((t) => t.status === "completed").length;
-        return {
-          success: true,
-          output: `Removed task "${removed.content}". Task list: ${completed} completed, ${inProgress} in progress, ${pending} pending`,
-          durationMs: Date.now() - startTime,
-          truncated: false
-        };
+      sessionTodos.set(context.sessionId, args.todos);
+      if (onTodoUpdate) onTodoUpdate(context.sessionId, args.todos);
+      return helpers.success(`${args.todos.filter((t) => t.status === "completed").length} completed, ${args.todos.filter((t) => t.status === "in_progress").length} in progress, ${args.todos.filter((t) => t.status === "pending").length} pending`);
+    }
+    if (args.action === "add") {
+      if (!args.content) return helpers.error("Missing required field: content");
+      const todosList = sessionTodos.get(context.sessionId) ?? [];
+      todosList.push({ content: args.content, status: "pending" });
+      sessionTodos.set(context.sessionId, todosList);
+      if (onTodoUpdate) onTodoUpdate(context.sessionId, todosList);
+      return helpers.success(`Added task "${args.content}". Task list: ${buildTaskListSummary(todosList)}`);
+    }
+    if (args.action === "update") {
+      if (!args.content && !args.status) return helpers.error("update requires content or status");
+      if (args.status && !["pending", "in_progress", "completed"].includes(args.status)) {
+        return helpers.error(`Invalid status: ${args.status}. Must be pending, in_progress, or completed`);
       }
-      return {
-        success: false,
-        error: "Unexpected error",
-        durationMs: Date.now() - startTime,
-        truncated: false
-      };
-    } catch (error) {
-      return {
-        success: false,
-        error: error instanceof Error ? error.message : "Unknown error",
-        durationMs: Date.now() - startTime,
-        truncated: false
-      };
+      const todosList = sessionTodos.get(context.sessionId) ?? [];
+      const indexResult = validateIndex(args.index, todosList, helpers);
+      if ("success" in indexResult && !indexResult.success) return indexResult;
+      const todo = indexResult;
+      if (args.content) todo.content = args.content;
+      if (args.status) todo.status = args.status;
+      sessionTodos.set(context.sessionId, todosList);
+      if (onTodoUpdate) onTodoUpdate(context.sessionId, todosList);
+      return helpers.success(`Updated task ${args.index}. Task list: ${buildTaskListSummary(todosList)}`);
+    }
+    if (args.action === "remove") {
+      const todosList = sessionTodos.get(context.sessionId) ?? [];
+      const indexResult = validateIndex(args.index, todosList, helpers);
+      if ("success" in indexResult && !indexResult.success) return indexResult;
+      const removed = todosList.splice(args.index, 1)[0];
+      sessionTodos.set(context.sessionId, todosList);
+      if (onTodoUpdate) onTodoUpdate(context.sessionId, todosList);
+      return helpers.success(
+        todosList.length === 0 ? `Removed task "${removed.content}". No tasks remaining.` : `Removed task "${removed.content}". Task list: ${buildTaskListSummary(todosList)}`
+      );
     }
+    return helpers.error("Unexpected error");
   }
-};
+);
 
 // src/server/chat/tool-streaming.ts
 function parseProgressMessage(message) {
@@ -2803,18 +2545,17 @@ function createToolProgressHandler(messageId, callId, onMessage) {
 }
 
 // src/server/skills/registry.ts
-import { readdir as readdir2, readFile as readFile6, writeFile as writeFile4, copyFile as copyFile2, mkdir as mkdir3, access as access3, unlink as unlink2 } from "fs/promises";
-import { join as join4, dirname as dirname4 } from "path";
+import { readdir as readdir3 } from "fs/promises";
+import { join as join6 } from "path";
+
+// src/server/commands/registry-utils.ts
+import { readdir as readdir2, readFile as readFile7, writeFile as writeFile4, copyFile as copyFile2, mkdir as mkdir3, access as access3, unlink as unlink2 } from "fs/promises";
+import { join as join5, dirname as dirname4 } from "path";
 import { constants as constants3 } from "fs";
 import { fileURLToPath as fileURLToPath2 } from "url";
 import matter2 from "gray-matter";
-var __bundleDir2 = dirname4(fileURLToPath2(import.meta.url));
-var DEFAULTS_DIR2 = join4(__bundleDir2, "defaults");
-var DEFAULTS_DIR_ALT2 = join4(__bundleDir2, "skill-defaults");
-var SKILL_EXTENSION = ".skill.md";
-var SKILL_SETTING_PREFIX = "skill.enabled.";
-function getSkillsDir(configDir) {
-  return join4(configDir, "skills");
+function getBundleDir() {
+  return dirname4(fileURLToPath2(import.meta.url));
 }
 async function dirExists(path) {
   try {
@@ -2824,64 +2565,92 @@ async function dirExists(path) {
     return false;
   }
 }
-async function ensureDefaultSkills(configDir) {
-  const skillsDir = getSkillsDir(configDir);
-  if (!await dirExists(skillsDir)) {
-    await mkdir3(skillsDir, { recursive: true });
-  }
-  let defaultFiles;
-  let sourceDir;
-  try {
-    defaultFiles = (await readdir2(DEFAULTS_DIR2)).filter((f) => f.endsWith(SKILL_EXTENSION));
-    sourceDir = DEFAULTS_DIR2;
-  } catch {
-    try {
-      defaultFiles = (await readdir2(DEFAULTS_DIR_ALT2)).filter((f) => f.endsWith(SKILL_EXTENSION));
-      sourceDir = DEFAULTS_DIR_ALT2;
-    } catch {
-      logger.warn("No bundled skill defaults found", { dir: DEFAULTS_DIR2 });
-      return;
-    }
-  }
-  for (const file of defaultFiles) {
-    const targetPath = join4(skillsDir, file);
-    if (!await dirExists(targetPath)) {
-      try {
-        await copyFile2(join4(sourceDir, file), targetPath);
-        logger.info("Installed default skill", { file });
-      } catch (err) {
-        logger.error("Failed to copy default skill", { file, error: err instanceof Error ? err.message : String(err) });
-      }
-    }
+async function ensureDir(path) {
+  if (!await dirExists(path)) {
+    await mkdir3(path, { recursive: true });
   }
 }
-async function loadAllSkills(configDir) {
-  const skillsDir = getSkillsDir(configDir);
-  if (!await dirExists(skillsDir)) {
-    return [];
-  }
+async function loadItems(dir, extension) {
+  if (!await dirExists(dir)) return [];
   let files;
   try {
-    files = (await readdir2(skillsDir)).filter((f) => f.endsWith(SKILL_EXTENSION));
+    files = (await readdir2(dir)).filter((f) => f.endsWith(extension));
   } catch {
     return [];
   }
-  const skills = [];
+  const items = [];
   for (const file of files) {
     try {
-      const raw = await readFile6(join4(skillsDir, file), "utf-8");
+      const raw = await readFile7(join5(dir, file), "utf-8");
       const { data, content } = matter2(raw);
-      const metadata = data;
-      if (metadata.id && content.trim()) {
-        skills.push({ metadata, prompt: content.trim() });
+      if (data.id && content.trim()) {
+        items.push({ metadata: data, prompt: content.trim() });
       } else {
-        logger.warn("Skipping invalid skill file", { file });
+        logger.warn(`Skipping invalid ${extension} file`, { file });
       }
     } catch (err) {
-      logger.warn("Failed to parse skill file", { file, error: err instanceof Error ? err.message : String(err) });
+      logger.warn(`Failed to parse ${extension} file`, { file, error: err instanceof Error ? err.message : String(err) });
+    }
+  }
+  return items;
+}
+async function saveItem(dir, id, extension, item) {
+  await ensureDir(dir);
+  const filePath = join5(dir, `${id}${extension}`);
+  const content = matter2.stringify(item.prompt, item.metadata);
+  await writeFile4(filePath, content, "utf-8");
+}
+async function deleteItem(dir, id, extension) {
+  const filePath = join5(dir, `${id}${extension}`);
+  try {
+    await unlink2(filePath);
+    return true;
+  } catch {
+    return false;
+  }
+}
+function findById(id, items) {
+  return items.find((i) => i.metadata.id === id);
+}
+async function getDefaultIds(bundleDir, defaultsDir, extension) {
+  try {
+    const files = (await readdir2(join5(bundleDir, defaultsDir))).filter((f) => f.endsWith(extension));
+    return files.map((f) => f.replace(extension, ""));
+  } catch {
+    return [];
+  }
+}
+async function restoreDefault(itemDir, bundleDir, defaultsDir, id, extension) {
+  const filename = `${id}${extension}`;
+  const src = join5(bundleDir, defaultsDir, filename);
+  if (await dirExists(src)) {
+    await copyFile2(src, join5(itemDir, filename));
+    return true;
+  }
+  return false;
+}
+
+// src/server/skills/registry.ts
+var DEFAULTS_DIR2 = "defaults";
+var DEFAULTS_DIR_ALT2 = "skill-defaults";
+var SKILL_EXTENSION = ".skill.md";
+var SKILL_SETTING_PREFIX = "skill.enabled.";
+function getSkillsDir(configDir) {
+  return join6(configDir, "skills");
+}
+async function ensureDefaultSkills(configDir) {
+  const bundleDir = getBundleDir();
+  const skillsDir = getSkillsDir(configDir);
+  await ensureDir(skillsDir);
+  const ids = await getDefaultIds(bundleDir, DEFAULTS_DIR2, SKILL_EXTENSION);
+  for (const id of ids) {
+    if (!await restoreDefault(skillsDir, bundleDir, DEFAULTS_DIR2, id, SKILL_EXTENSION)) {
+      await restoreDefault(skillsDir, bundleDir, DEFAULTS_DIR_ALT2, id, SKILL_EXTENSION);
     }
   }
-  return skills;
+}
+async function loadAllSkills(configDir) {
+  return loadItems(getSkillsDir(configDir), SKILL_EXTENSION);
 }
 async function getEnabledSkills(configDir) {
   const all = await loadAllSkills(configDir);
@@ -2900,51 +2669,21 @@ function setSkillEnabled(skillId, enabled) {
   setSetting(`${SKILL_SETTING_PREFIX}${skillId}`, String(enabled));
 }
 async function getDefaultSkillIds() {
-  for (const dir of [DEFAULTS_DIR2, DEFAULTS_DIR_ALT2]) {
-    try {
-      const files = (await readdir2(dir)).filter((f) => f.endsWith(SKILL_EXTENSION));
-      return files.map((f) => f.replace(SKILL_EXTENSION, ""));
-    } catch {
-    }
-  }
-  return [];
+  const bundleDir = getBundleDir();
+  const ids = await getDefaultIds(bundleDir, DEFAULTS_DIR2, SKILL_EXTENSION);
+  if (ids.length) return ids;
+  return getDefaultIds(bundleDir, DEFAULTS_DIR_ALT2, SKILL_EXTENSION);
 }
 async function restoreDefaultSkill(configDir, skillId) {
-  const filename = `${skillId}${SKILL_EXTENSION}`;
-  for (const dir of [DEFAULTS_DIR2, DEFAULTS_DIR_ALT2]) {
-    const sourcePath = join4(dir, filename);
-    if (await dirExists(sourcePath)) {
-      const targetPath = join4(getSkillsDir(configDir), filename);
-      await copyFile2(sourcePath, targetPath);
-      return true;
-    }
-  }
-  return false;
+  const bundleDir = getBundleDir();
+  const skillsDir = getSkillsDir(configDir);
+  return restoreDefault(skillsDir, bundleDir, DEFAULTS_DIR2, skillId, SKILL_EXTENSION) || restoreDefault(skillsDir, bundleDir, DEFAULTS_DIR_ALT2, skillId, SKILL_EXTENSION);
 }
 async function getModifiedDefaultSkillIds(configDir) {
-  const defaultIds = await getDefaultSkillIds();
-  const modified = [];
-  for (const id of defaultIds) {
-    const filename = `${id}${SKILL_EXTENSION}`;
-    const userPath = join4(getSkillsDir(configDir), filename);
-    let bundledContent = null;
-    for (const dir of [DEFAULTS_DIR2, DEFAULTS_DIR_ALT2]) {
-      try {
-        bundledContent = await readFile6(join4(dir, filename), "utf-8");
-        break;
-      } catch {
-      }
-    }
-    if (!bundledContent) continue;
-    try {
-      const userContent = await readFile6(userPath, "utf-8");
-      if (userContent !== bundledContent) {
-        modified.push(id);
-      }
-    } catch {
-    }
-  }
-  return modified;
+  const bundleDir = getBundleDir();
+  const ids = await getDefaultSkillIds();
+  const files = (await readdir3(join6(bundleDir, DEFAULTS_DIR2))).filter((f) => f.endsWith(SKILL_EXTENSION));
+  return files.map((f) => f.replace(SKILL_EXTENSION, "")).filter((id) => ids.includes(id));
 }
 async function restoreAllDefaultSkills(configDir) {
   const ids = await getDefaultSkillIds();
@@ -2955,34 +2694,34 @@ async function restoreAllDefaultSkills(configDir) {
   return count;
 }
 function findSkillById(skillId, skills) {
-  return skills.find((s) => s.metadata.id === skillId);
+  return findById(skillId, skills);
 }
 async function skillExists(configDir, skillId) {
-  const filePath = join4(getSkillsDir(configDir), `${skillId}${SKILL_EXTENSION}`);
-  return dirExists(filePath);
+  return dirExists(join6(getSkillsDir(configDir), `${skillId}${SKILL_EXTENSION}`));
 }
 async function saveSkill(configDir, skill) {
-  const skillsDir = getSkillsDir(configDir);
-  if (!await dirExists(skillsDir)) {
-    await mkdir3(skillsDir, { recursive: true });
-  }
-  const filePath = join4(skillsDir, `${skill.metadata.id}${SKILL_EXTENSION}`);
-  const content = matter2.stringify(skill.prompt, skill.metadata);
-  await writeFile4(filePath, content, "utf-8");
+  return saveItem(getSkillsDir(configDir), skill.metadata.id, SKILL_EXTENSION, skill);
 }
 async function deleteSkill(configDir, skillId) {
-  const filePath = join4(getSkillsDir(configDir), `${skillId}${SKILL_EXTENSION}`);
-  try {
-    await unlink2(filePath);
+  const deleted = await deleteItem(getSkillsDir(configDir), skillId, SKILL_EXTENSION);
+  if (deleted) {
     const { deleteSetting } = await import("./settings-NCLN4JDZ.js");
     deleteSetting(`${SKILL_SETTING_PREFIX}${skillId}`);
-    return true;
-  } catch {
-    return false;
   }
+  return deleted;
 }
 
 // src/server/chat/agent-loop.ts
+import stripAnsi from "strip-ansi";
+function emitPartialDoneEvents(sessionId, assistantMsgId, statsIdentity, mode, turnMetrics, promptContext, eventStore) {
+  const stats = turnMetrics.buildStats(statsIdentity, mode);
+  eventStore.append(sessionId, createMessageDoneEvent(assistantMsgId, {
+    stats,
+    partial: true,
+    promptContext
+  }));
+  eventStore.append(sessionId, createChatDoneEvent(assistantMsgId, "stopped", stats));
+}
 function getCurrentWindowMessageOptions(sessionId) {
   const contextWindowId = getCurrentContextWindowId(sessionId);
   return contextWindowId ? { contextWindowId } : void 0;
@@ -3048,7 +2787,7 @@ async function executeToolBatch(assistantMsgId, toolCalls, ctx) {
           type: "chat.ask_user",
           data: { callId: error.callId, question: error.question }
         });
-        const { awaitAnswer } = await import("./ask-JQQNGB5K.js");
+        const { awaitAnswer } = await import("./ask-3RK5YJZE.js");
         const answerPromise = awaitAnswer(error.callId);
         if (!answerPromise) {
           throw new Error(`No pending question found for callId: ${error.callId}`);
@@ -3073,9 +2812,9 @@ async function executeToolBatch(assistantMsgId, toolCalls, ctx) {
     }
     toolMessages.push({
       role: "tool",
-      content: toolResult.success ? toolResult.output ?? "Success" : toolResult.output ? `${toolResult.output}
+      content: stripAnsi(toolResult.success ? toolResult.output ?? "Success" : toolResult.output ? `${toolResult.output}
 
-Error: ${toolResult.error}` : `Error: ${toolResult.error}`,
+Error: ${toolResult.error}` : `Error: ${toolResult.error}`),
       source: "history",
       toolCallId: toolCall.id
     });
@@ -3213,13 +2952,7 @@ async function runTopLevelAgentLoop(config, turnMetrics) {
       }
     }
     if (result.aborted) {
-      const stats2 = turnMetrics.buildStats(statsIdentity, mode);
-      eventStore.append(sessionId, createMessageDoneEvent(assistantMsgId, {
-        stats: stats2,
-        partial: true,
-        promptContext: assembledRequest.promptContext
-      }));
-      eventStore.append(sessionId, createChatDoneEvent(assistantMsgId, "stopped", stats2));
+      emitPartialDoneEvents(sessionId, assistantMsgId, statsIdentity, mode, turnMetrics, assembledRequest.promptContext, eventStore);
       throw new Error("Aborted");
     }
     turnMetrics.addLLMCall(result.timing, result.usage.promptTokens, result.usage.completionTokens);
@@ -3254,25 +2987,13 @@ async function runTopLevelAgentLoop(config, turnMetrics) {
         }
       } catch (error) {
         if (error instanceof Error && error.message === "Aborted") {
-          const stats2 = turnMetrics.buildStats(statsIdentity, mode);
-          eventStore.append(sessionId, createMessageDoneEvent(assistantMsgId, {
-            stats: stats2,
-            partial: true,
-            promptContext: assembledRequest.promptContext
-          }));
-          eventStore.append(sessionId, createChatDoneEvent(assistantMsgId, "stopped", stats2));
+          emitPartialDoneEvents(sessionId, assistantMsgId, statsIdentity, mode, turnMetrics, assembledRequest.promptContext, eventStore);
           throw error;
         }
         throw error;
       }
       if (signal?.aborted) {
-        const stats2 = turnMetrics.buildStats(statsIdentity, mode);
-        eventStore.append(sessionId, createMessageDoneEvent(assistantMsgId, {
-          stats: stats2,
-          partial: true,
-          promptContext: assembledRequest.promptContext
-        }));
-        eventStore.append(sessionId, createChatDoneEvent(assistantMsgId, "stopped", stats2));
+        emitPartialDoneEvents(sessionId, assistantMsgId, statsIdentity, mode, turnMetrics, assembledRequest.promptContext, eventStore);
         throw new Error("Aborted");
       }
       const asapMessages = sessionManager.drainAsapMessages(sessionId);
@@ -3310,12 +3031,32 @@ async function runTopLevelAgentLoop(config, turnMetrics) {
   };
 }
 
+// src/server/context/nudge-helpers.ts
+function appendNudgeMessage(eventStore, sessionId, content, currentWindowMessageOptions, options) {
+  const msgId = crypto.randomUUID();
+  eventStore.append(sessionId, createMessageStartEvent(msgId, "user", content, {
+    ...currentWindowMessageOptions ?? {},
+    isSystemGenerated: true,
+    messageKind: "correction",
+    subAgentId: options.subAgentId,
+    subAgentType: options.subAgentType
+  }));
+  eventStore.append(sessionId, { type: "message.done", data: { messageId: msgId } });
+}
+
 // src/server/sub-agents/manager.ts
 var RETURN_VALUE_INSTRUCTION = `
 
 ## RETURN VALUE
 As the very last thing you do, call \`return_value\` ONCE with a structured summary of your work. This is how your findings get passed back to the calling agent. Do not finish without calling return_value.`;
 var RETURN_VALUE_NUDGE = "You must call return_value with a summary of your findings before finishing. Call return_value now.";
+function buildSubAgentResult(returnValueContent, returnValueResult, subAgentType, failed, remaining) {
+  return {
+    content: returnValueContent ?? "",
+    ...returnValueResult !== null && returnValueResult !== void 0 ? { result: returnValueResult } : {},
+    ...subAgentType === "verifier" ? { allPassed: failed.length === 0 && remaining.length === 0, failed } : {}
+  };
+}
 async function resolveAgentDef(agentId) {
   const allAgents = await loadAllAgentsDefault();
   const def = findAgentById(agentId, allAgents);
@@ -3485,19 +3226,11 @@ async function executeSubAgent(options) {
           if (consecutiveEmptyStops < nudgeConfig.maxConsecutiveNudges) {
             consecutiveEmptyStops += 1;
             const nudgeContent = nudgeConfig.buildNudgeContent(criteriaAwaiting);
-            const nudgeMsgId = crypto.randomUUID();
             eventStore.append(sessionId, createMessageDoneEvent(assistantMsgId, {
               segments: result.segments,
               promptContext
             }));
-            eventStore.append(sessionId, createMessageStartEvent(nudgeMsgId, "user", nudgeContent, {
-              ...currentWindowMessageOptions ?? {},
-              isSystemGenerated: true,
-              messageKind: "correction",
-              subAgentId,
-              subAgentType
-            }));
-            eventStore.append(sessionId, { type: "message.done", data: { messageId: nudgeMsgId } });
+            appendNudgeMessage(eventStore, sessionId, nudgeContent, currentWindowMessageOptions, { subAgentId, subAgentType });
             customMessages = [...customMessages, { role: "user", content: nudgeContent, source: "runtime" }];
             continue;
           }
@@ -3514,19 +3247,11 @@ async function executeSubAgent(options) {
       }
       if (!returnValueContent && !returnValueNudged) {
         returnValueNudged = true;
-        const nudgeMsgId = crypto.randomUUID();
         eventStore.append(sessionId, createMessageDoneEvent(assistantMsgId, {
           segments: result.segments,
           promptContext
         }));
-        eventStore.append(sessionId, createMessageStartEvent(nudgeMsgId, "user", RETURN_VALUE_NUDGE, {
-          ...currentWindowMessageOptions ?? {},
-          isSystemGenerated: true,
-          messageKind: "correction",
-          subAgentId,
-          subAgentType
-        }));
-        eventStore.append(sessionId, { type: "message.done", data: { messageId: nudgeMsgId } });
+        appendNudgeMessage(eventStore, sessionId, RETURN_VALUE_NUDGE, currentWindowMessageOptions, { subAgentId, subAgentType });
         customMessages = [...customMessages, { role: "user", content: RETURN_VALUE_NUDGE, source: "runtime" }];
         continue;
       }
@@ -3582,21 +3307,9 @@ async function executeSubAgent(options) {
     subAgentId,
     resultLength: finalContent.length
   });
-  if (subAgentType === "verifier") {
-    session = sessionManager.requireSession(sessionId);
-    const remaining = nudgeConfig?.getCriteriaAwaiting(session.criteria) ?? [];
-    const failed = session.criteria.filter((c) => c.status.type === "failed").map((c) => ({
-      id: c.id,
-      reason: c.status.type === "failed" ? c.status.reason : "unknown"
-    }));
-    return {
-      content: returnValueContent ?? finalContent,
-      ...returnValueResult ? { result: returnValueResult } : {},
-      allPassed: failed.length === 0 && remaining.length === 0,
-      failed
-    };
-  }
-  return { content: returnValueContent ?? finalContent, ...returnValueResult ? { result: returnValueResult } : { result: "success" } };
+  const failed = session.criteria.filter((c) => c.status.type === "failed").map((c) => ({ id: c.id, reason: c.status.reason ?? "unknown" }));
+  const remaining = nudgeConfig?.getCriteriaAwaiting(session.criteria) ?? [];
+  return buildSubAgentResult(returnValueContent ?? void 0, returnValueResult ?? (subAgentType !== "verifier" ? "success" : void 0), subAgentType, failed, remaining);
 }
 
 // src/server/tools/sub-agent.ts
@@ -3664,7 +3377,7 @@ var callSubAgentTool = {
       };
     }
     try {
-      const { getToolRegistryForAgent: getToolRegistryForAgent2 } = await import("./tools-JWAF3MAR.js");
+      const { getToolRegistryForAgent: getToolRegistryForAgent2 } = await import("./tools-VOWNOP6V.js");
       const toolRegistry = getToolRegistryForAgent2(agentDef);
       const turnMetrics = new TurnMetrics();
       const result = await executeSubAgent({
@@ -3816,7 +3529,7 @@ function convertHTMLToMarkdown(html) {
   return turndown.turndown(html);
 }
 function stripHTMLTags(html) {
-  let text = html.replace(/<(script|style|noscript|iframe|object|embed)[^>]*>[\s\S]*?<\/\1>/gi, "").replace(/<[^>]+>/g, " ").replace(/&amp;/g, "&").replace(/&lt;/g, "<").replace(/&gt;/g, ">").replace(/&quot;/g, '"').replace(/&#39;/g, "'").replace(/&nbsp;/g, " ").replace(/\s+/g, " ").trim();
+  const text = html.replace(/<(script|style|noscript|iframe|object|embed)[^>]*>[\s\S]*?<\/\1>/gi, "").replace(/<[^>]+>/g, " ").replace(/&amp;/g, "&").replace(/&lt;/g, "<").replace(/&gt;/g, ">").replace(/&quot;/g, '"').replace(/&#39;/g, "'").replace(/&nbsp;/g, " ").replace(/\s+/g, " ").trim();
   return text;
 }
 var webFetchTool = createTool(
@@ -3918,9 +3631,9 @@ var webFetchTool = createTool(
 );
 
 // src/server/dev-server/manager.ts
-import { spawn as spawn3 } from "child_process";
-import { readFile as readFile7, writeFile as writeFile5, mkdir as mkdir4 } from "fs/promises";
-import { resolve as resolve5, join as join5 } from "path";
+import { spawn as spawn4 } from "child_process";
+import { readFile as readFile8, writeFile as writeFile5, mkdir as mkdir4 } from "fs/promises";
+import { resolve as resolve5, join as join7 } from "path";
 var MAX_LOG_LINES = 2e3;
 var MAX_LOG_BYTES = 1e5;
 var CONFIG_PATH = ".openfox/dev.json";
@@ -3976,8 +3689,8 @@ var DevServerManager = class {
   }
   async loadConfig(workdir) {
     try {
-      const configPath = join5(this.resolveWorkdir(workdir), CONFIG_PATH);
-      const raw = await readFile7(configPath, "utf-8");
+      const configPath = join7(this.resolveWorkdir(workdir), CONFIG_PATH);
+      const raw = await readFile8(configPath, "utf-8");
       const parsed = JSON.parse(raw);
       if (!parsed.command || !parsed.url) return null;
       return {
@@ -3991,9 +3704,9 @@ var DevServerManager = class {
   }
   async saveConfig(workdir, config) {
     const resolved = this.resolveWorkdir(workdir);
-    const dirPath = join5(resolved, ".openfox");
+    const dirPath = join7(resolved, ".openfox");
     await mkdir4(dirPath, { recursive: true });
-    const configPath = join5(resolved, CONFIG_PATH);
+    const configPath = join7(resolved, CONFIG_PATH);
     await writeFile5(configPath, JSON.stringify(config, null, 2) + "\n", "utf-8");
   }
   async start(workdir) {
@@ -4012,7 +3725,7 @@ var DevServerManager = class {
     instance.exited = false;
     const resolved = this.resolveWorkdir(workdir);
     const shell = getPlatformShell();
-    const proc = spawn3(shell.command, [...shell.args, config.command], {
+    const proc = spawn4(shell.command, [...shell.args, config.command], {
       cwd: resolved,
       env: { ...process.env, FORCE_COLOR: "1" },
       stdio: ["ignore", "pipe", "pipe"],
@@ -4584,14 +4297,7 @@ function getCurrentWindowMessageOptions3(sessionId) {
   return contextWindowId ? { contextWindowId } : void 0;
 }
 function toRequestContextMessages2(messages) {
-  return messages.map((message) => ({
-    role: message.role,
-    content: message.content,
-    source: "history",
-    ...message.toolCalls ? { toolCalls: message.toolCalls } : {},
-    ...message.toolCallId ? { toolCallId: message.toolCallId } : {},
-    ...message.attachments ? { attachments: message.attachments } : {}
-  }));
+  return minimalMessagesToRequestContextMessages(messages, "history");
 }
 async function maybeAutoCompactContext(options) {
   const config = getRuntimeConfig();
@@ -4746,6 +4452,9 @@ export {
   PathAccessDeniedError,
   providePathConfirmation,
   cancelPathConfirmationsForSession,
+  checkAborted,
+  spawnShellProcess,
+  findModifiedDefaultFiles,
   ensureDefaultAgents,
   loadAllAgents,
   loadAllAgentsDefault,
@@ -4769,6 +4478,15 @@ export {
   createChatDoneEvent,
   getAllInstructions,
   assembleAgentRequest,
+  getBundleDir,
+  dirExists,
+  ensureDir,
+  loadItems,
+  saveItem,
+  deleteItem,
+  findById,
+  getDefaultIds,
+  restoreDefault,
   ensureDefaultSkills,
   loadAllSkills,
   getEnabledSkillMetadata,
@@ -4797,4 +4515,4 @@ export {
   getToolRegistryForAgent,
   createToolRegistry
 };
-//# sourceMappingURL=chunk-POJO7A5H.js.map
+//# sourceMappingURL=chunk-3WHZ47PY.js.map