u-foo 2.5.6 → 2.5.8

package/src/code/nativeRunner.js

@@ -102,104 +102,6 @@ function clipText(value = "", maxChars = 6000) {
   return `${text.slice(0, maxChars)}\n...[truncated]`;
 }
 
-function summarizeFileSnippet(file = "", content = "") {
-  const target = String(file || "").trim();
-  const body = String(content || "").trim();
-  if (!body) return `${target}: empty`;
-
-  if (target.toLowerCase().endsWith("package.json")) {
-    try {
-      const parsed = JSON.parse(body);
-      const name = String(parsed.name || "").trim() || "(unknown)";
-      const version = String(parsed.version || "").trim() || "(unknown)";
-      const scripts = parsed.scripts && typeof parsed.scripts === "object"
-        ? Object.keys(parsed.scripts).slice(0, 4)
-        : [];
-      const scriptText = scripts.length > 0 ? ` scripts=${scripts.join(",")}` : "";
-      return `${target}: name=${name} version=${version}${scriptText}`;
-    } catch {
-      // fall through
-    }
-  }
-
-  const lines = body
-    .split(/\r?\n/)
-    .map((line) => line.trim())
-    .filter(Boolean)
-    .slice(0, 3)
-    .map((line) => (line.length > 120 ? `${line.slice(0, 120)}...` : line));
-
-  if (lines.length === 0) return `${target}: empty`;
-  return `${target}: ${lines.join(" | ")}`;
-}
-
-function extractPreflightEvidence(systemPrompt = "") {
-  const source = String(systemPrompt || "");
-  if (!source) return [];
-
-  const results = [];
-  const fileRegex = /File:\s*([^\n]+)\n([\s\S]*?)(?=\n---\n(?:File|Command):|$)/g;
-  let match = fileRegex.exec(source);
-  while (match) {
-    const file = String(match[1] || "").trim();
-    const content = String(match[2] || "").trim();
-    if (file) {
-      results.push({ kind: "file", label: file, summary: summarizeFileSnippet(file, content) });
-    }
-    match = fileRegex.exec(source);
-  }
-
-  const cmdRegex = /Command:\s*([^\n]+)\n([\s\S]*?)(?=\n---\n(?:File|Command):|$)/g;
-  match = cmdRegex.exec(source);
-  while (match) {
-    const command = String(match[1] || "").trim();
-    const output = String(match[2] || "").trim();
-    if (command) {
-      const clipped = clipText(output, 300).replace(/\s+/g, " ").trim();
-      results.push({ kind: "command", label: command, summary: `${command}: ${clipped || "(no output)"}` });
-    }
-    match = cmdRegex.exec(source);
-  }
-
-  return results;
-}
-
-function isAnalysisPrompt(text = "") {
-  return /(?:analy[sz]e|analysis|review|audit|status|architecture|codebase|repo|project|现状|架构|审查|分析|项目|代码库)/i.test(text);
-}
-
-function parseReadIntent(prompt = "") {
-  const text = String(prompt || "");
-  const patterns = [
-    /(?:\bread\b|\bcat\b|查看|读取)\s+([A-Za-z0-9_./\\-]+(?:\.[A-Za-z0-9._-]+)?)/i,
-    /([A-Za-z0-9_./\\-]+\.(?:md|txt|json|js|ts|jsx|tsx|yml|yaml|toml|sh))/i,
-  ];
-  for (const re of patterns) {
-    const match = text.match(re);
-    if (!match || !match[1]) continue;
-    const candidate = String(match[1]).trim().replace(/[),.;:]+$/, "");
-    if (!candidate) continue;
-    if (candidate.length > 260) continue;
-    return candidate;
-  }
-  return "";
-}
-
-function parseBashIntent(prompt = "") {
-  const text = String(prompt || "").trim();
-  if (!text) return "";
-  if (
-    /\b(ls|dir|tree)\b/i.test(text)
-    || /\b(list|show)\s+(files|dirs|directories|folders)\b/i.test(text)
-    || /列出|目录|文件列表/.test(text)
-  ) {
-    return "ls -la";
-  }
-  const cmdMatch = text.match(/(?:运行|执行|run|exec(?:ute)?)\s+`([^`]+)`/i);
-  if (cmdMatch && cmdMatch[1]) return String(cmdMatch[1]).trim();
-  return "";
-}
-
 function normalizeProvider(value = "") {
   const text = String(value || "").trim().toLowerCase();
   if (!text) return "";
@@ -317,6 +219,11 @@ function buildCoreToolSpecs() {
           properties: {
             path: { type: "string" },
             content: { type: "string" },
+            mode: {
+              type: "string",
+              enum: ["overwrite", "append"],
+              description: 'Write mode: "overwrite" replaces the file (default), "append" adds to its end.',
+            },
             append: { type: "boolean" },
           },
           required: ["path", "content"],
@@ -530,34 +437,22 @@ function emitPhase(callback, event = {}) {
   }
 }
 
-async function runOpenAiLikeTurn({
+// Shared SSE transport skeleton: POST the payload, then read the stream as
+// SSE blocks, dispatch each non-[DONE] block to onEvent, and stop after the
+// batch that carried [DONE]. Timeout/cancel translation and request cleanup
+// live here so each protocol turn only declares its event handling.
+async function runSseRequest({
   url = "",
-  apiKey = "",
-  model = "",
-  messages = [],
-  onTextDelta = null,
-  onThinkingDelta = null,
-  onPhase = null,
+  headers = {},
+  payload = {},
   signal = null,
   timeoutMs = 300000,
+  onPhase = null,
+  onNonStream,
+  onEvent,
+  onTail = null,
+  buildResult,
 } = {}) {
-  const payload = {
-    model,
-    max_tokens: resolveMaxTokens(DEFAULT_OPENAI_MAX_TOKENS),
-    messages,
-    tools: buildCoreToolSpecs(),
-    tool_choice: "auto",
-    stream: true,
-    temperature: 0,
-  };
-
-  const headers = {
-    "content-type": "application/json",
-  };
-  if (apiKey) {
-    headers.authorization = `Bearer ${apiKey}`;
-  }
-
   const request = createRequestController({ signal, timeoutMs });
 
   emitPhase(onPhase, { type: "request_start" });
@@ -577,28 +472,13 @@ async function runOpenAiLikeTurn({
 
     if (!response.body || typeof response.body.getReader !== "function") {
       const data = await response.json();
-      const message = data && data.choices && data.choices[0] && data.choices[0].message
-        ? data.choices[0].message
-        : {};
-      const text = typeof message.content === "string" ? message.content : "";
-      const toolCalls = Array.isArray(message.tool_calls) ? message.tool_calls : [];
-      if (text && typeof onTextDelta === "function") {
-        onTextDelta(text);
-      }
-      return {
-        text,
-        toolCalls,
-      };
+      return onNonStream(data);
     }
 
     const reader = response.body.getReader();
     const decoder = new TextDecoder();
-    const toolCallMap = new Map();
     let rawBuffer = "";
-    let responseText = "";
-    const announcedToolNames = new Set();
-    let nextSyntheticIndex = 0;
-    let lastSyntheticIndex = -1;
+    let sawDone = false;
 
     while (true) {
       const { done, value } = await reader.read();
@@ -609,91 +489,178 @@ async function runOpenAiLikeTurn({
       rawBuffer = parsed.rest;
 
       for (const block of parsed.blocks) {
-        const payloadText = parseSseDataBlock(block);
-        if (!payloadText) continue;
-        if (payloadText === "[DONE]") {
-          rawBuffer = "";
-          break;
+        const { event, data } = parseSseEventBlock(block);
+        if (!data) continue;
+        if (data === "[DONE]") {
+          // Stop reading after this batch instead of waiting for the server
+          // to close the connection, but keep the buffered tail and finish
+          // the blocks already parsed alongside [DONE] instead of silently
+          // dropping them.
+          sawDone = true;
+          continue;
         }
 
-        const chunk = parseJsonSafe(payloadText, null);
-        if (!chunk || typeof chunk !== "object") continue;
+        onEvent({ event, data });
+      }
 
-        const choice = chunk.choices && chunk.choices[0] ? chunk.choices[0] : null;
-        if (!choice || typeof choice !== "object") continue;
+      if (sawDone) break;
+    }
 
-        const delta = choice.delta && typeof choice.delta === "object" ? choice.delta : {};
+    if (typeof onTail === "function") {
+      onTail(rawBuffer);
+    }
 
-        const reasoningChunk = typeof delta.reasoning_content === "string"
-          ? delta.reasoning_content
-          : (typeof delta.reasoning === "string" ? delta.reasoning : "");
-        if (reasoningChunk) {
-          emitPhase(onPhase, { type: "thinking_delta", text: reasoningChunk });
-          if (typeof onThinkingDelta === "function") {
-            onThinkingDelta(reasoningChunk);
-          }
+    return buildResult();
+  } catch (err) {
+    if (request.timedOut()) {
+      const timeoutError = new Error(`CLI timeout (${normalizeTimeoutMs(timeoutMs)}ms)`);
+      timeoutError.code = "timeout";
+      throw timeoutError;
+    }
+    if (signal && typeof signal === "object" && signal.aborted) {
+      const cancelError = new Error("CLI cancelled");
+      cancelError.code = "cancelled";
+      throw cancelError;
+    }
+    throw err;
+  } finally {
+    request.cleanup();
+  }
+}
+
+async function runOpenAiLikeTurn({
+  url = "",
+  apiKey = "",
+  model = "",
+  messages = [],
+  onTextDelta = null,
+  onThinkingDelta = null,
+  onPhase = null,
+  signal = null,
+  timeoutMs = 300000,
+} = {}) {
+  const payload = {
+    model,
+    max_tokens: resolveMaxTokens(DEFAULT_OPENAI_MAX_TOKENS),
+    messages,
+    tools: buildCoreToolSpecs(),
+    tool_choice: "auto",
+    stream: true,
+    temperature: 0,
+  };
+
+  const headers = {
+    "content-type": "application/json",
+  };
+  if (apiKey) {
+    headers.authorization = `Bearer ${apiKey}`;
+  }
+
+  const toolCallMap = new Map();
+  const announcedToolNames = new Set();
+  let responseText = "";
+  let nextSyntheticIndex = 0;
+  let lastSyntheticIndex = -1;
+
+  return runSseRequest({
+    url,
+    headers,
+    payload,
+    signal,
+    timeoutMs,
+    onPhase,
+    onNonStream: (data) => {
+      const message = data && data.choices && data.choices[0] && data.choices[0].message
+        ? data.choices[0].message
+        : {};
+      const text = typeof message.content === "string" ? message.content : "";
+      const toolCalls = Array.isArray(message.tool_calls) ? message.tool_calls : [];
+      if (text && typeof onTextDelta === "function") {
+        onTextDelta(text);
+      }
+      return {
+        text,
+        toolCalls,
+      };
+    },
+    onEvent: ({ data }) => {
+      const chunk = parseJsonSafe(data, null);
+      if (!chunk || typeof chunk !== "object") return;
+
+      const choice = chunk.choices && chunk.choices[0] ? chunk.choices[0] : null;
+      if (!choice || typeof choice !== "object") return;
+
+      const delta = choice.delta && typeof choice.delta === "object" ? choice.delta : {};
+
+      const reasoningChunk = typeof delta.reasoning_content === "string"
+        ? delta.reasoning_content
+        : (typeof delta.reasoning === "string" ? delta.reasoning : "");
+      if (reasoningChunk) {
+        emitPhase(onPhase, { type: "thinking_delta", text: reasoningChunk });
+        if (typeof onThinkingDelta === "function") {
+          onThinkingDelta(reasoningChunk);
         }
+      }
 
-        if (typeof delta.content === "string" && delta.content) {
-          responseText += delta.content;
-          emitPhase(onPhase, { type: "text_delta", text: delta.content });
-          if (typeof onTextDelta === "function") {
-            onTextDelta(delta.content);
-          }
+      if (typeof delta.content === "string" && delta.content) {
+        responseText += delta.content;
+        emitPhase(onPhase, { type: "text_delta", text: delta.content });
+        if (typeof onTextDelta === "function") {
+          onTextDelta(delta.content);
         }
+      }
 
-        if (Array.isArray(delta.tool_calls)) {
-          for (const callPart of delta.tool_calls) {
-            let index;
-            if (Number.isFinite(callPart.index)) {
-              index = callPart.index;
-            } else if (typeof callPart.id === "string" && callPart.id) {
-              // Provider omitted index: a chunk carrying an id starts a new
-              // call, so give it its own synthetic index instead of
-              // collapsing every call into slot 0.
-              while (toolCallMap.has(nextSyntheticIndex)) nextSyntheticIndex += 1;
-              index = nextSyntheticIndex;
-              nextSyntheticIndex += 1;
-              lastSyntheticIndex = index;
-            } else if (lastSyntheticIndex >= 0) {
-              // No index and no id: continuation of the latest synthetic call.
-              index = lastSyntheticIndex;
-            } else {
-              index = 0;
-            }
-            const previous = toolCallMap.get(index) || {
-              id: "",
-              type: "function",
-              function: {
-                name: "",
-                arguments: "",
-              },
-            };
+      if (Array.isArray(delta.tool_calls)) {
+        for (const callPart of delta.tool_calls) {
+          let index;
+          if (Number.isFinite(callPart.index)) {
+            index = callPart.index;
+          } else if (typeof callPart.id === "string" && callPart.id) {
+            // Provider omitted index: a chunk carrying an id starts a new
+            // call, so give it its own synthetic index instead of
+            // collapsing every call into slot 0.
+            while (toolCallMap.has(nextSyntheticIndex)) nextSyntheticIndex += 1;
+            index = nextSyntheticIndex;
+            nextSyntheticIndex += 1;
+            lastSyntheticIndex = index;
+          } else if (lastSyntheticIndex >= 0) {
+            // No index and no id: continuation of the latest synthetic call.
+            index = lastSyntheticIndex;
+          } else {
+            index = 0;
+          }
+          const previous = toolCallMap.get(index) || {
+            id: "",
+            type: "function",
+            function: {
+              name: "",
+              arguments: "",
+            },
+          };
 
-            if (typeof callPart.id === "string" && callPart.id) previous.id = callPart.id;
-            if (callPart.function && typeof callPart.function === "object") {
-              if (typeof callPart.function.name === "string" && callPart.function.name) {
-                previous.function.name = callPart.function.name;
-              }
-              if (typeof callPart.function.arguments === "string" && callPart.function.arguments) {
-                previous.function.arguments += callPart.function.arguments;
-              }
+          if (typeof callPart.id === "string" && callPart.id) previous.id = callPart.id;
+          if (callPart.function && typeof callPart.function === "object") {
+            if (typeof callPart.function.name === "string" && callPart.function.name) {
+              previous.function.name = callPart.function.name;
+            }
+            if (typeof callPart.function.arguments === "string" && callPart.function.arguments) {
+              previous.function.arguments += callPart.function.arguments;
             }
+          }
 
-            toolCallMap.set(index, previous);
+          toolCallMap.set(index, previous);
 
-            const toolName = previous.function.name;
-            const announceKey = `${index}:${toolName}`;
-            if (toolName && !announcedToolNames.has(announceKey)) {
-              announcedToolNames.add(announceKey);
-              emitPhase(onPhase, { type: "tool_request", name: toolName });
-            }
+          const toolName = previous.function.name;
+          const announceKey = `${index}:${toolName}`;
+          if (toolName && !announcedToolNames.has(announceKey)) {
+            announcedToolNames.add(announceKey);
+            emitPhase(onPhase, { type: "tool_request", name: toolName });
           }
         }
       }
-    }
-
-    if (rawBuffer.trim()) {
+    },
+    onTail: (rawBuffer) => {
+      if (!rawBuffer.trim()) return;
       const fallbackBlock = parseSseDataBlock(rawBuffer);
       if (fallbackBlock && fallbackBlock !== "[DONE]") {
         const chunk = parseJsonSafe(fallbackBlock, null);
@@ -705,29 +672,14 @@ async function runOpenAiLikeTurn({
           }
         }
       }
-    }
-
-    return {
+    },
+    buildResult: () => ({
       text: responseText,
       toolCalls: Array.from(toolCallMap.entries())
         .sort((a, b) => a[0] - b[0])
         .map((entry) => entry[1]),
-    };
-  } catch (err) {
-    if (request.timedOut()) {
-      const timeoutError = new Error(`CLI timeout (${normalizeTimeoutMs(timeoutMs)}ms)`);
-      timeoutError.code = "timeout";
-      throw timeoutError;
-    }
-    if (signal && typeof signal === "object" && signal.aborted) {
-      const cancelError = new Error("CLI cancelled");
-      cancelError.code = "cancelled";
-      throw cancelError;
-    }
-    throw err;
-  } finally {
-    request.cleanup();
-  }
+    }),
+  });
 }
 
 function normalizeAnthropicMessageContent(raw = []) {
@@ -800,25 +752,19 @@ async function runAnthropicTurn({
     headers["x-api-key"] = apiKey;
   }
 
-  const request = createRequestController({ signal, timeoutMs });
-
-  emitPhase(onPhase, { type: "request_start" });
-
-  try {
-    const response = await fetch(url, {
-      method: "POST",
-      headers,
-      body: JSON.stringify(payload),
-      signal: request.signal,
-    });
-
-    if (!response.ok) {
-      const body = await response.text().catch(() => "");
-      throw new Error(`provider request failed (${response.status}): ${clipText(body, 500)}`);
-    }
-
-    if (!response.body || typeof response.body.getReader !== "function") {
-      const data = await response.json();
+  const blockMap = new Map();
+  let responseText = "";
+  let nextSyntheticBlockIndex = 0;
+  let lastBlockIndex = -1;
+
+  return runSseRequest({
+    url,
+    headers,
+    payload,
+    signal,
+    timeoutMs,
+    onPhase,
+    onNonStream: (data) => {
       const content = normalizeAnthropicMessageContent(data && data.content);
       const text = content
         .filter((item) => item.type === "text")
@@ -832,240 +778,181 @@ async function runAnthropicTurn({
         assistantContent: content,
         toolCalls: extractAnthropicToolCalls(content),
       };
-    }
-
-    const reader = response.body.getReader();
-    const decoder = new TextDecoder();
-    const blockMap = new Map();
-    let rawBuffer = "";
-    let responseText = "";
-
-    while (true) {
-      const { done, value } = await reader.read();
-      if (done) break;
-
-      rawBuffer += decoder.decode(value, { stream: true });
-      const parsed = parseSseBlocks(rawBuffer);
-      rawBuffer = parsed.rest;
-
-      for (const rawBlock of parsed.blocks) {
-        const { event, data } = parseSseEventBlock(rawBlock);
-        if (!data || data === "[DONE]") continue;
-
-        const payloadChunk = parseJsonSafe(data, null);
-        if (!payloadChunk || typeof payloadChunk !== "object") continue;
+    },
+    onEvent: ({ event, data }) => {
+      const payloadChunk = parseJsonSafe(data, null);
+      if (!payloadChunk || typeof payloadChunk !== "object") return;
+
+      if (event === "error") {
+        const errMsg = payloadChunk.error && payloadChunk.error.message
+          ? String(payloadChunk.error.message)
+          : "anthropic stream error";
+        throw new Error(errMsg);
+      }
 
-        if (event === "error") {
-          const errMsg = payloadChunk.error && payloadChunk.error.message
-            ? String(payloadChunk.error.message)
-            : "anthropic stream error";
-          throw new Error(errMsg);
+      if (event === "content_block_start") {
+        let index;
+        if (Number.isFinite(payloadChunk.index)) {
+          index = payloadChunk.index;
+        } else {
+          // Provider omitted index: each start opens a new block, so give
+          // it its own synthetic index instead of collapsing every block
+          // into slot 0.
+          while (blockMap.has(nextSyntheticBlockIndex)) nextSyntheticBlockIndex += 1;
+          index = nextSyntheticBlockIndex;
+          nextSyntheticBlockIndex += 1;
         }
-
-        if (event === "content_block_start") {
-          const index = Number.isFinite(payloadChunk.index) ? payloadChunk.index : 0;
-          const contentBlock = payloadChunk.content_block && typeof payloadChunk.content_block === "object"
-            ? payloadChunk.content_block
-            : {};
-
-          if (contentBlock.type === "text") {
-            blockMap.set(index, {
-              order: index,
-              type: "text",
-              text: String(contentBlock.text || ""),
-            });
-          } else if (contentBlock.type === "thinking") {
-            blockMap.set(index, {
-              order: index,
-              type: "thinking",
-              text: String(contentBlock.thinking || ""),
-            });
-          } else if (contentBlock.type === "tool_use") {
-            blockMap.set(index, {
-              order: index,
-              type: "tool_use",
-              id: String(contentBlock.id || ""),
-              name: String(contentBlock.name || ""),
-              input: contentBlock.input && typeof contentBlock.input === "object" && !Array.isArray(contentBlock.input)
-                ? { ...contentBlock.input }
-                : {},
-              inputJson: "",
-            });
-            const toolName = String(contentBlock.name || "");
-            if (toolName) {
-              emitPhase(onPhase, { type: "tool_request", name: toolName });
-            }
+        lastBlockIndex = index;
+        const contentBlock = payloadChunk.content_block && typeof payloadChunk.content_block === "object"
+          ? payloadChunk.content_block
+          : {};
+
+        if (contentBlock.type === "text") {
+          blockMap.set(index, {
+            order: index,
+            type: "text",
+            text: String(contentBlock.text || ""),
+          });
+        } else if (contentBlock.type === "thinking") {
+          blockMap.set(index, {
+            order: index,
+            type: "thinking",
+            text: String(contentBlock.thinking || ""),
+          });
+        } else if (contentBlock.type === "tool_use") {
+          blockMap.set(index, {
+            order: index,
+            type: "tool_use",
+            id: String(contentBlock.id || ""),
+            name: String(contentBlock.name || ""),
+            input: contentBlock.input && typeof contentBlock.input === "object" && !Array.isArray(contentBlock.input)
+              ? { ...contentBlock.input }
+              : {},
+            inputJson: "",
+          });
+          const toolName = String(contentBlock.name || "");
+          if (toolName) {
+            emitPhase(onPhase, { type: "tool_request", name: toolName });
           }
-          continue;
         }
+        return;
+      }
 
-        if (event === "content_block_delta") {
-          const index = Number.isFinite(payloadChunk.index) ? payloadChunk.index : 0;
-          const delta = payloadChunk.delta && typeof payloadChunk.delta === "object"
-            ? payloadChunk.delta
-            : {};
-          const current = blockMap.get(index) || { order: index, type: "text", text: "" };
-
-          if (delta.type === "text_delta") {
-            const deltaText = String(delta.text || "");
-            current.type = "text";
-            current.text = `${String(current.text || "")}${deltaText}`;
-            blockMap.set(index, current);
-            if (deltaText) {
-              responseText += deltaText;
-              emitPhase(onPhase, { type: "text_delta", text: deltaText });
-              if (typeof onTextDelta === "function") {
-                onTextDelta(deltaText);
-              }
+      if (event === "content_block_delta") {
+        let index;
+        if (Number.isFinite(payloadChunk.index)) {
+          index = payloadChunk.index;
+        } else if (lastBlockIndex >= 0) {
+          // No index: continuation of the most recently started block.
+          index = lastBlockIndex;
+        } else {
+          index = 0;
+        }
+        const delta = payloadChunk.delta && typeof payloadChunk.delta === "object"
+          ? payloadChunk.delta
+          : {};
+        const current = blockMap.get(index) || { order: index, type: "text", text: "" };
+
+        if (delta.type === "text_delta") {
+          const deltaText = String(delta.text || "");
+          current.type = "text";
+          current.text = `${String(current.text || "")}${deltaText}`;
+          blockMap.set(index, current);
+          if (deltaText) {
+            responseText += deltaText;
+            emitPhase(onPhase, { type: "text_delta", text: deltaText });
+            if (typeof onTextDelta === "function") {
+              onTextDelta(deltaText);
             }
-            continue;
           }
+          return;
+        }
 
-          if (delta.type === "thinking_delta") {
-            const deltaText = String(delta.thinking || "");
-            current.type = "thinking";
-            current.text = `${String(current.text || "")}${deltaText}`;
-            blockMap.set(index, current);
-            if (deltaText) {
-              emitPhase(onPhase, { type: "thinking_delta", text: deltaText });
-              if (typeof onThinkingDelta === "function") {
-                onThinkingDelta(deltaText);
-              }
+        if (delta.type === "thinking_delta") {
+          const deltaText = String(delta.thinking || "");
+          current.type = "thinking";
+          current.text = `${String(current.text || "")}${deltaText}`;
+          blockMap.set(index, current);
+          if (deltaText) {
+            emitPhase(onPhase, { type: "thinking_delta", text: deltaText });
+            if (typeof onThinkingDelta === "function") {
+              onThinkingDelta(deltaText);
             }
-            continue;
           }
+          return;
+        }
 
-          if (delta.type === "input_json_delta") {
-            current.type = "tool_use";
-            current.inputJson = `${String(current.inputJson || "")}${String(delta.partial_json || "")}`;
-            blockMap.set(index, current);
-            continue;
-          }
+        if (delta.type === "input_json_delta") {
+          current.type = "tool_use";
+          current.inputJson = `${String(current.inputJson || "")}${String(delta.partial_json || "")}`;
+          blockMap.set(index, current);
+          return;
         }
       }
-    }
+    },
+    buildResult: () => {
+      const assistantContent = Array.from(blockMap.values())
+        .sort((a, b) => a.order - b.order)
+        .filter((item) => item.type !== "thinking")
+        .map((item) => {
+          if (item.type === "text") {
+            return {
+              type: "text",
+              text: String(item.text || ""),
+            };
+          }
 
-    const assistantContent = Array.from(blockMap.values())
-      .sort((a, b) => a.order - b.order)
-      .filter((item) => item.type !== "thinking")
-      .map((item) => {
-        if (item.type === "text") {
+          const inputFromDelta = normalizeToolCallArgs(item.inputJson || "");
+          const mergedInput = {
+            ...(item.input && typeof item.input === "object" ? item.input : {}),
+            ...(inputFromDelta && typeof inputFromDelta === "object" ? inputFromDelta : {}),
+          };
           return {
-            type: "text",
-            text: String(item.text || ""),
+            type: "tool_use",
+            id: String(item.id || `tool_${randomUUID()}`),
+            name: String(item.name || ""),
+            input: mergedInput,
           };
-        }
-
-        const inputFromDelta = normalizeToolCallArgs(item.inputJson || "");
-        const mergedInput = {
-          ...(item.input && typeof item.input === "object" ? item.input : {}),
-          ...(inputFromDelta && typeof inputFromDelta === "object" ? inputFromDelta : {}),
-        };
-        return {
-          type: "tool_use",
-          id: String(item.id || `tool_${randomUUID()}`),
-          name: String(item.name || ""),
-          input: mergedInput,
-        };
-      });
+        });
 
-    if (!responseText) {
-      responseText = assistantContent
-        .filter((item) => item.type === "text")
-        .map((item) => item.text)
-        .join("");
-    }
+      if (!responseText) {
+        responseText = assistantContent
+          .filter((item) => item.type === "text")
+          .map((item) => item.text)
+          .join("");
+      }
 
-    return {
-      text: responseText,
-      assistantContent,
-      toolCalls: extractAnthropicToolCalls(assistantContent),
-    };
-  } catch (err) {
-    if (request.timedOut()) {
-      const timeoutError = new Error(`CLI timeout (${normalizeTimeoutMs(timeoutMs)}ms)`);
-      timeoutError.code = "timeout";
-      throw timeoutError;
-    }
-    if (signal && typeof signal === "object" && signal.aborted) {
-      const cancelError = new Error("CLI cancelled");
-      cancelError.code = "cancelled";
-      throw cancelError;
-    }
-    throw err;
-  } finally {
-    request.cleanup();
-  }
+      return {
+        text: responseText,
+        assistantContent,
+        toolCalls: extractAnthropicToolCalls(assistantContent),
+      };
+    },
+  });
 }
 
-async function runNativeLoopOpenAi({
-  workspaceRoot = process.cwd(),
-  prompt = "",
-  systemPrompt = "",
-  historyMessages = [],
-  model = "",
-  baseUrl = "",
-  apiKey = "",
-  timeoutMs = 300000,
-  onStreamDelta = null,
-  onThinkingDelta = null,
-  onPhase = null,
-  onToolEvent = null,
-  signal = null,
-  guards,
-} = {}) {
-  const requestModel = String(model || "").trim();
-  if (!requestModel) {
-    throw new Error("ucode model is not configured");
-  }
-
-  const requestUrl = resolveCompletionUrl(baseUrl);
-  if (!requestUrl) {
-    throw new Error("ucode baseUrl is not configured");
-  }
-
-  const messages = cloneMessageList(historyMessages);
-  const systemText = String(systemPrompt || "").trim();
-  const hasSystem = messages.some((entry) => String(entry.role || "").trim() === "system");
-  if (systemText && !hasSystem) {
-    messages.unshift({ role: "system", content: systemText });
-  }
-  messages.push({ role: "user", content: String(prompt || "") });
-
-  let aggregated = "";
-  let streamed = false;
-  let toolCallsExecuted = 0;
-  let toolErrors = 0;
-  const toolBudget = resolveNativeToolBudget();
-
-  while (true) {
-    guards.ensureActive();
-
-    const turnResult = await runOpenAiLikeTurn({
-      url: requestUrl,
-      apiKey,
-      model: requestModel,
-      messages,
-      signal,
-      timeoutMs,
-      onPhase,
-      onThinkingDelta,
-      onTextDelta: (chunk) => {
-        const text = String(chunk || "");
-        if (!text) return;
-        aggregated += text;
-        if (typeof onStreamDelta === "function") {
-          streamed = true;
-          onStreamDelta(text);
-        }
-      },
-    });
-
-    const toolCalls = Array.isArray(turnResult.toolCalls)
-      ? turnResult.toolCalls.filter((call) => call && call.function && typeof call.function === "object")
-      : [];
-
-    if (toolCalls.length === 0) {
+// Transport descriptors: everything the shared native loop needs that differs
+// between the OpenAI chat-completions and Anthropic messages protocols —
+// request URL resolution, initial message shaping, turn execution, and
+// assistant/tool-result message formatting.
+const TRANSPORTS = {
+  "openai-chat": {
+    resolveUrl: resolveCompletionUrl,
+    prepareMessages({ messages, systemPrompt, prompt }) {
+      const systemText = String(systemPrompt || "").trim();
+      const hasSystem = messages.some((entry) => String(entry.role || "").trim() === "system");
+      if (systemText && !hasSystem) {
+        messages.unshift({ role: "system", content: systemText });
+      }
+      messages.push({ role: "user", content: String(prompt || "") });
+    },
+    runTurn: runOpenAiLikeTurn,
+    getToolCalls(turnResult) {
+      return Array.isArray(turnResult.toolCalls)
+        ? turnResult.toolCalls.filter((call) => call && call.function && typeof call.function === "object")
+        : [];
+    },
+    appendFinalAssistantMessage({ messages, turnResult }) {
       const text = String(turnResult.text || "").trim();
       if (text) {
         messages.push({
@@ -1073,78 +960,114 @@ async function runNativeLoopOpenAi({
           content: text,
         });
       }
-      if (!aggregated.trim() && text) {
-        aggregated = text;
+    },
+    prepareToolCalls({ messages, toolCalls }) {
+      const assistantToolCalls = [];
+      for (const call of toolCalls) {
+        const callId = String(call.id || `call_${randomUUID()}`);
+        const name = normalizeToolName(call.function.name || "");
+        const args = normalizeToolCallArgs(call.function.arguments || "");
+
+        assistantToolCalls.push({
+          id: callId,
+          type: "function",
+          function: {
+            name: name || String(call.function.name || ""),
+            arguments: toJsonString(args),
+          },
+        });
       }
-      return {
-        text: aggregated,
-        streamed,
-        toolCallsExecuted,
-        messages,
-      };
-    }
 
-    const assistantToolCalls = [];
-    for (const call of toolCalls) {
-      const callId = String(call.id || `call_${randomUUID()}`);
-      const name = normalizeToolName(call.function.name || "");
-      const args = normalizeToolCallArgs(call.function.arguments || "");
-
-      assistantToolCalls.push({
-        id: callId,
-        type: "function",
-        function: {
-          name: name || String(call.function.name || ""),
-          arguments: toJsonString(args),
-        },
-      });
-    }
+      if (assistantToolCalls.length === 0) return null;
 
-    if (assistantToolCalls.length === 0) {
-      return {
-        text: aggregated,
-        streamed,
-        toolCallsExecuted,
-        messages,
-      };
-    }
-
-    messages.push({
-      role: "assistant",
-      content: null,
-      tool_calls: assistantToolCalls,
-    });
+      messages.push({
+        role: "assistant",
+        content: null,
+        tool_calls: assistantToolCalls,
+      });
 
-    for (const toolCall of assistantToolCalls) {
-      const toolResult = runCoreTool({
-        tool: toolCall.function.name,
+      return assistantToolCalls.map((toolCall) => ({
+        name: toolCall.function.name,
         args: normalizeToolCallArgs(toolCall.function.arguments),
-        workspaceRoot,
-        onToolEvent,
-      });
-      toolCallsExecuted += 1;
-      if (!toolResult || toolResult.ok === false) {
-        toolErrors += 1;
-      }
-      enforceNativeToolBudget({
-        toolCallsExecuted,
-        toolErrors,
-        maxToolCalls: toolBudget.maxToolCalls,
-        maxToolErrors: toolBudget.maxToolErrors,
-        lastTool: toolCall.function.name,
-        lastError: toolResult && toolResult.error ? String(toolResult.error) : "",
-      });
+        source: toolCall,
+      }));
+    },
+    appendToolResult({ messages, call, toolResult }) {
       messages.push({
         role: "tool",
-        tool_call_id: toolCall.id,
+        tool_call_id: call.source.id,
         content: clipText(toJsonString(toolResult), 12000),
       });
-    }
-  }
+    },
+  },
+  "anthropic-messages": {
+    resolveUrl: resolveAnthropicMessagesUrl,
+    prepareMessages({ messages, prompt }) {
+      messages.push({
+        role: "user",
+        content: String(prompt || ""),
+      });
+    },
+    runTurn: runAnthropicTurn,
+    getToolCalls(turnResult) {
+      return Array.isArray(turnResult.toolCalls) ? turnResult.toolCalls : [];
+    },
+    appendFinalAssistantMessage({ messages, turnResult }) {
+      const assistantContent = Array.isArray(turnResult.assistantContent)
+        ? turnResult.assistantContent
+        : [];
+      if (assistantContent.length > 0) {
+        messages.push({
+          role: "assistant",
+          content: assistantContent,
+        });
+      } else if (String(turnResult.text || "").trim()) {
+        messages.push({
+          role: "assistant",
+          content: [
+            {
+              type: "text",
+              text: String(turnResult.text || ""),
+            },
+          ],
+        });
+      }
+    },
+    prepareToolCalls({ messages, turnResult, toolCalls }) {
+      const assistantContent = Array.isArray(turnResult.assistantContent)
+        ? turnResult.assistantContent
+        : [];
 
-}
+      messages.push({
+        role: "assistant",
+        content: assistantContent,
+      });
+
+      return toolCalls.map((call) => ({
+        name: call.name,
+        args: call.args,
+        source: call,
+      }));
+    },
+    appendToolResult({ collected, call, toolResult }) {
+      collected.push({
+        type: "tool_result",
+        tool_use_id: String(call.source.id || ""),
+        content: clipText(toJsonString(toolResult), 12000),
+        is_error: Boolean(!toolResult || toolResult.ok === false),
+      });
+    },
+    flushToolResults({ messages, collected }) {
+      messages.push({
+        role: "user",
+        content: collected,
+      });
+    },
+  },
+};
 
-async function runNativeLoopAnthropic({
+async function runNativeLoop({
+  transport,
   workspaceRoot = process.cwd(),
   prompt = "",
   systemPrompt = "",
@@ -1165,16 +1088,13 @@ async function runNativeLoopAnthropic({
     throw new Error("ucode model is not configured");
   }
 
-  const requestUrl = resolveAnthropicMessagesUrl(baseUrl);
+  const requestUrl = transport.resolveUrl(baseUrl);
   if (!requestUrl) {
     throw new Error("ucode baseUrl is not configured");
   }
 
   const messages = cloneMessageList(historyMessages);
-  messages.push({
-    role: "user",
-    content: String(prompt || ""),
-  });
+  transport.prepareMessages({ messages, systemPrompt, prompt });
 
   let aggregated = "";
   let streamed = false;
@@ -1185,7 +1105,7 @@ async function runNativeLoopAnthropic({
   while (true) {
     guards.ensureActive();
 
-    const turnResult = await runAnthropicTurn({
+    const turnResult = await transport.runTurn({
       url: requestUrl,
       apiKey,
       model: requestModel,
@@ -1206,28 +1126,10 @@ async function runNativeLoopAnthropic({
       },
     });
 
-    const toolCalls = Array.isArray(turnResult.toolCalls) ? turnResult.toolCalls : [];
+    const toolCalls = transport.getToolCalls(turnResult);
 
     if (toolCalls.length === 0) {
-      const assistantContent = Array.isArray(turnResult.assistantContent)
-        ? turnResult.assistantContent
-        : [];
-      if (assistantContent.length > 0) {
-        messages.push({
-          role: "assistant",
-          content: assistantContent,
-        });
-      } else if (String(turnResult.text || "").trim()) {
-        messages.push({
-          role: "assistant",
-          content: [
-            {
-              type: "text",
-              text: String(turnResult.text || ""),
-            },
-          ],
-        });
-      }
+      transport.appendFinalAssistantMessage({ messages, turnResult });
       const text = String(turnResult.text || "").trim();
       if (!aggregated.trim() && text) {
         aggregated = text;
@@ -1240,20 +1142,21 @@ async function runNativeLoopAnthropic({
       };
     }
 
-    const assistantContent = Array.isArray(turnResult.assistantContent)
-      ? turnResult.assistantContent
-      : [];
-
-    messages.push({
-      role: "assistant",
-      content: assistantContent,
-    });
+    const pendingCalls = transport.prepareToolCalls({ messages, turnResult, toolCalls });
+    if (!pendingCalls) {
+      return {
+        text: aggregated,
+        streamed,
+        toolCallsExecuted,
+        messages,
+      };
+    }
 
-    const toolResults = [];
-    for (const call of toolCalls) {
+    const collectedResults = [];
+    for (const pending of pendingCalls) {
       const toolResult = runCoreTool({
-        tool: call.name,
-        args: call.args,
+        tool: pending.name,
+        args: pending.args,
         workspaceRoot,
         onToolEvent,
       });
@@ -1266,23 +1169,21 @@ async function runNativeLoopAnthropic({
         toolErrors,
         maxToolCalls: toolBudget.maxToolCalls,
         maxToolErrors: toolBudget.maxToolErrors,
-        lastTool: call.name,
+        lastTool: pending.name,
         lastError: toolResult && toolResult.error ? String(toolResult.error) : "",
       });
-      toolResults.push({
-        type: "tool_result",
-        tool_use_id: String(call.id || ""),
-        content: clipText(toJsonString(toolResult), 12000),
-        is_error: Boolean(!toolResult || toolResult.ok === false),
+      transport.appendToolResult({
+        messages,
+        collected: collectedResults,
+        call: pending,
+        toolResult,
       });
     }
 
-    messages.push({
-      role: "user",
-      content: toolResults,
-    });
+    if (typeof transport.flushToolResults === "function") {
+      transport.flushToolResults({ messages, collected: collectedResults });
+    }
   }
-
 }
 
 async function runNativeAgentTask({
@@ -1303,6 +1204,17 @@ async function runNativeAgentTask({
   const guards = createGuards({ signal, timeoutMs });
   const nextSessionId = String(sessionId || "").trim() || `native-${randomUUID()}`;
   const promptText = String(prompt || "").trim();
+  // Track every text delta so the error path can return the partial output
+  // the model already produced instead of discarding it.
+  let partialOutput = "";
+  const trackingStreamDelta = (chunk) => {
+    const text = String(chunk || "");
+    if (!text) return;
+    partialOutput += text;
+    if (typeof onStreamDelta === "function") {
+      onStreamDelta(chunk);
+    }
+  };
 
   try {
     guards.ensureActive();
@@ -1323,11 +1235,10 @@ async function runNativeAgentTask({
       model,
     });
 
-    const loopRunner = runtime.transport === "anthropic-messages"
-      ? runNativeLoopAnthropic
-      : runNativeLoopOpenAi;
+    const transport = TRANSPORTS[runtime.transport] || TRANSPORTS["openai-chat"];
 
-    const runResult = await loopRunner({
+    const runResult = await runNativeLoop({
+      transport,
       workspaceRoot,
       prompt: promptText,
       systemPrompt,
@@ -1336,7 +1247,7 @@ async function runNativeAgentTask({
       baseUrl: runtime.baseUrl,
       apiKey: runtime.apiKey,
       timeoutMs,
-      onStreamDelta,
+      onStreamDelta: trackingStreamDelta,
       onThinkingDelta,
       onPhase,
       onToolEvent,
@@ -1356,14 +1267,16 @@ async function runNativeAgentTask({
       output: outputText,
       messages: cloneMessageList(runResult.messages),
       sessionId: nextSessionId,
-      streamed: Boolean(runResult.streamed),
+      // The loop marks streamed=true whenever it receives a stream callback;
+      // only report it when the caller actually registered one.
+      streamed: Boolean(runResult.streamed) && typeof onStreamDelta === "function",
     };
   } catch (err) {
     const message = err && err.message ? err.message : "native runner failed";
     return {
       ok: false,
       error: message,
-      output: "",
+      output: partialOutput.trim(),
       sessionId: nextSessionId,
       streamed: false,
     };
@@ -1372,9 +1285,6 @@ async function runNativeAgentTask({
 
 module.exports = {
   runNativeAgentTask,
-  parseReadIntent,
-  parseBashIntent,
-  extractPreflightEvidence,
   resolveRuntimeConfig,
   resolveCompletionUrl,
   resolveAnthropicMessagesUrl,