u-foo 2.3.12 → 2.3.13

package/src/agent/ufooAgent.js

@@ -1,7 +1,5 @@
 const fs = require("fs");
 const path = require("path");
-const { runCliAgent } = require("./cliRunner");
-const { normalizeCliOutput } = require("./normalizeOutput");
 const { buildStatus } = require("../daemon/status");
 const { getUfooPaths } = require("../ufoo/paths");
 const { normalizeGateRouterResult } = require("../controller/gateRouter");
@@ -633,7 +631,6 @@ async function runUfooAgent({
   loopRuntime = null,
   controllerMode = null,
 }) {
-  const state = loadSessionState(projectRoot);
   const mode = String(routingMode || (routingContext && routingContext.mode) || "").trim().toLowerCase();
   const resolvedControllerMode = String(
     controllerMode
@@ -660,7 +657,6 @@ async function runUfooAgent({
   let res;
 
   const useDirectProvider = shouldUseDirectProvider(provider);
-  let usedDirectProvider = false;
 
   if (useDirectProvider) {
     res = await runNativeRouterCall({
@@ -671,47 +667,22 @@ async function runUfooAgent({
       model,
     });
     if (!res.ok) {
+      // eslint-disable-next-line no-console
+      console.error(`[ufoo-agent] native provider failed: ${res.error || "unknown error"}`);
       return { ok: false, error: res.error };
     } else {
-      usedDirectProvider = true;
       res = { ok: true, output: res.output, sessionId: "", provider: res.provider, model: res.model };
     }
   }
 
   if (!useDirectProvider) {
-    res = await runCliAgent({
-      provider,
-      model,
-      prompt: fullPrompt,
-      systemPrompt,
-      sessionId: state.data?.sessionId,
-      disableSession: provider === "claude-cli",
-      cwd: projectRoot,
-    });
-
-    if (!res.ok) {
-      const msg = (res.error || "").toLowerCase();
-      if (msg.includes("session id") || msg.includes("session-id") || msg.includes("already in use")) {
-        res = await runCliAgent({
-          provider,
-          model,
-          prompt: fullPrompt,
-          systemPrompt,
-          sessionId: undefined,
-          disableSession: provider === "claude-cli",
-          cwd: projectRoot,
-        });
-      }
-    }
-
-    if (!res.ok) {
-      return { ok: false, error: res.error };
-    }
+    const error = `unsupported ufoo-agent provider "${provider || ""}"; cliRunner fallback has been removed`;
+    // eslint-disable-next-line no-console
+    console.error(`[ufoo-agent] ${error}`);
+    return { ok: false, error };
   }
 
-  const rawText = usedDirectProvider
-    ? String(res.output || "").trim()
-    : normalizeCliOutput(res.output);
+  const rawText = String(res.output || "").trim();
   const text = stripMarkdownFence(rawText);
   let payload = null;
   try {

package/src/chat/daemonMessageRouter.js

@@ -442,7 +442,9 @@ function createDaemonMessageRouter(options = {}) {
   }
 
   function handleErrorMessage(msg) {
-    resolveStatusLine(`{gray-fg}✗{/gray-fg} Error: ${msg.error}`);
+    const error = String(msg.error || "unknown error");
+    resolveStatusLine(`{gray-fg}✗{/gray-fg} Error: ${error}`);
+    logMessage("error", `{white-fg}✗{/white-fg} ${escapeBlessed(error)}`);
     renderScreen();
     return false;
   }

package/src/chat/dashboardKeyController.js

@@ -1,4 +1,4 @@
-const DEFAULT_MODE_OPTIONS = ["auto", "host", "terminal", "tmux", "internal"];
+const DEFAULT_MODE_OPTIONS = ["auto", "host", "terminal", "tmux", "internal-pty", "internal"];
 
 function createDashboardKeyController(options = {}) {
   const {

package/src/chat/dashboardView.js

@@ -1,6 +1,6 @@
 const { clampAgentWindowWithSelection } = require("./agentDirectory");
 
-const DEFAULT_MODE_OPTIONS = ["auto", "host", "terminal", "tmux", "internal"];
+const DEFAULT_MODE_OPTIONS = ["auto", "host", "terminal", "tmux", "internal-pty", "internal"];
 
 function providerLabel(value) {
   if (value === "claude-cli") return "claude";

package/src/chat/index.js

@@ -65,7 +65,7 @@ const {
   pruneTransientAgentStates,
 } = require("./transientAgentState");
 
-const MODE_OPTIONS = ["auto", "host", "terminal", "tmux", "internal"];
+const MODE_OPTIONS = ["auto", "host", "terminal", "tmux", "internal-pty", "internal"];
 
 async function runChat(projectRoot, options = {}) {
   const globalMode = options && options.globalMode === true;

package/src/config.js

@@ -29,6 +29,7 @@ const DEFAULT_UCODE_CONFIG = {
 function normalizeLaunchMode(value) {
   if (value === "auto") return "auto";
   if (value === "internal") return "internal";
+  if (value === "internal-pty") return "internal-pty";
   if (value === "tmux") return "tmux";
   if (value === "terminal") return "terminal";
   if (value === "host") return "host";

package/src/daemon/ops.js

@@ -14,6 +14,7 @@ const {
 const {
   createSession: createHostSession,
 } = require("../terminal/adapters/hostAdapter");
+const { resolveDefaultManualBootstrap } = require("../agent/defaultBootstrap");
 
 function normalizeLaunchAgent(agent = "") {
   const value = String(agent || "").trim().toLowerCase();
@@ -44,6 +45,43 @@ function toTmuxBinary(agent = "") {
   return "";
 }
 
+function applyDefaultManagedBootstrap(projectRoot, normalizedAgent, args = [], extraEnv = {}) {
+  const agentType = toBusAgentType(normalizedAgent);
+  if (!agentType || agentType === "ufoo-code") {
+    return {
+      args: Array.isArray(args) ? args.slice() : [],
+      extraEnv: extraEnv && typeof extraEnv === "object" ? { ...extraEnv } : {},
+      applied: false,
+    };
+  }
+
+  const currentArgs = Array.isArray(args) ? args.slice() : [];
+  const currentExtraEnv = extraEnv && typeof extraEnv === "object" ? { ...extraEnv } : {};
+  const resolved = resolveDefaultManualBootstrap({
+    projectRoot,
+    agentType,
+    args: currentArgs,
+    env: {
+      ...process.env,
+      ...currentExtraEnv,
+    },
+  });
+
+  if (!resolved || resolved.mode === "skip") {
+    return { args: currentArgs, extraEnv: currentExtraEnv, applied: false };
+  }
+
+  return {
+    args: Array.isArray(resolved.args) ? resolved.args : currentArgs,
+    extraEnv: {
+      ...currentExtraEnv,
+      ...(resolved.env && typeof resolved.env === "object" ? resolved.env : {}),
+      UFOO_SKIP_DEFAULT_BOOTSTRAP: "1",
+    },
+    applied: true,
+  };
+}
+
 function resolveUfooRunnerPath() {
   return path.resolve(__dirname, "../../bin/ufoo.js");
 }
@@ -116,7 +154,7 @@ function resolveHostLaunchContext(options = {}) {
 
 function resolveConfiguredLaunchMode(configuredMode = "", options = {}) {
   const mode = normalizeOptionalString(configuredMode);
-  if (mode === "internal" || mode === "tmux" || mode === "terminal" || mode === "host") {
+  if (mode === "internal" || mode === "internal-pty" || mode === "tmux" || mode === "terminal" || mode === "host") {
     return mode;
   }
   const hostContext = resolveHostLaunchContext(options);
@@ -518,7 +556,9 @@ async function spawnManagedHostAgent(
     subscriberId = "";
   }
 
-  const args = Array.isArray(extraArgs) ? extraArgs : [];
+  const hostBootstrap = applyDefaultManagedBootstrap(projectRoot, normalizedAgent, extraArgs, extraEnv);
+  const args = hostBootstrap.args;
+  const hostExtraEnv = hostBootstrap.extraEnv;
   const argText = args.length > 0 ? ` ${args.map(shellEscape).join(" ")}` : "";
 
   const titleCmd = buildTitleCmd(nickname);
@@ -538,8 +578,8 @@ async function spawnManagedHostAgent(
     env.UFOO_NICKNAME = nickname;
   }
   // Parse extraEnv string (e.g., "UFOO_UCODE_BOOTSTRAP_FILE=/path/to/file") and add to env
-  if (extraEnv && typeof extraEnv === "object") {
-    for (const [key, value] of Object.entries(extraEnv)) {
+  if (hostExtraEnv && typeof hostExtraEnv === "object") {
+    for (const [key, value] of Object.entries(hostExtraEnv)) {
       if (/^[A-Za-z_][A-Za-z0-9_]*$/.test(String(key || ""))) {
         env[String(key)] = String(value ?? "");
       }
@@ -645,7 +685,9 @@ async function spawnInternalAgent(
       ? (count > 1 ? `${nickname}-${i + 1}` : nickname)
       : "";
     const providerSessionId = typeof options.providerSessionId === "string" ? options.providerSessionId.trim() : "";
-    const usePty = process.env.UFOO_INTERNAL_PTY !== "0";
+    const usePty = typeof options.usePty === "boolean"
+      ? options.usePty
+      : process.env.UFOO_INTERNAL_PTY !== "0";
     const launchMode = usePty ? "internal-pty" : "internal";
 
     // 传递 launch_mode 和 parent PID 到 join
@@ -657,8 +699,9 @@ async function spawnInternalAgent(
     const finalNickname = joinResult.nickname || requestedNickname || "";
     bus.saveBusData();
 
+    const managedBootstrap = applyDefaultManagedBootstrap(projectRoot, normalizedAgent, extraArgs, extraEnv);
     const runnerCmd = usePty ? "agent-pty-runner" : "agent-runner";
-    const args = Array.isArray(extraArgs) ? extraArgs : [];
+    const args = managedBootstrap.args;
     const child = spawn(process.execPath, [runner, runnerCmd, agent, ...args], {
       // 关键改动：不使用 detached，daemon 作为父进程
       detached: false,
@@ -666,7 +709,7 @@ async function spawnInternalAgent(
       cwd: projectRoot,
       env: {
         ...process.env,
-        ...(extraEnv && typeof extraEnv === "object" ? extraEnv : {}),
+        ...(managedBootstrap.extraEnv && typeof managedBootstrap.extraEnv === "object" ? managedBootstrap.extraEnv : {}),
         UFOO_INTERNAL_AGENT: "1",
         UFOO_INTERNAL_PTY: usePty ? "1" : "0",
         UFOO_SUBSCRIBER_ID: subscriberId,  // 直接传递 subscriber ID
@@ -888,7 +931,8 @@ async function launchAgent(projectRoot, agent, count = 1, nickname = "", process
     throw new Error(`unsupported agent type: ${agent}`);
   }
 
-  if (mode === "internal") {
+  if (mode === "internal" || mode === "internal-pty") {
+    const usePty = mode === "internal-pty";
     const result = await spawnInternalAgent(
       projectRoot,
       normalizedAgent,
@@ -896,9 +940,10 @@ async function launchAgent(projectRoot, agent, count = 1, nickname = "", process
       nickname,
       processManager,
       launchEnvObject,
-      extraArgs
+      extraArgs,
+      { usePty }
     );
-    return { mode: "internal", launchScope, subscriberIds: result.subscriberIds };
+    return { mode, launchScope, subscriberIds: result.subscriberIds };
   }
   if (mode === "tmux") {
     // Check if tmux is available
@@ -1129,7 +1174,7 @@ async function resumeAgents(projectRoot, target = "", processManager = null) {
     const args = buildResumeArgs(item.agent, sessionId);
     let reused = false;
     let resumedId = item.id;
-    if (mode === "internal") {
+    if (mode === "internal" || mode === "internal-pty") {
       // Internal agents have no terminal/pane to reattach. Start a fresh
       // daemon-managed runner and replace the old recoverable registration.
       // The provider session is still reused via the normal provider args.
@@ -1142,7 +1187,7 @@ async function resumeAgents(projectRoot, target = "", processManager = null) {
         processManager,
         { UFOO_SKIP_SESSION_PROBE: "1" },
         args,
-        { replaceAgentId: item.id, providerSessionId: sessionId }
+        { replaceAgentId: item.id, providerSessionId: sessionId, usePty: mode === "internal-pty" }
       );
       resumedId = launchResult.subscriberIds && launchResult.subscriberIds[0]
         ? launchResult.subscriberIds[0]