devez-vibe 1.9.36 → 1.9.38

package/bridge/claude-agent-sdk-bridge.mjs

@@ -1485,41 +1485,11 @@ async function createSession(params, resumeId) {
     session.model = visibleModel(capabilities.value);
   }
   session.effort = supportedEffort(capabilities, params.effort);
-  watchMcpStartup(session);
   const account = initialization.account || await safeAccount(agentQuery);
   const usage = await safeUsage(agentQuery);
   return { session, initialization, account, usage };
 }
 
-// Claude connects MCP servers in the background and reports a failure only
-// through mcpServerStatus(). Once nothing is pending, each failed or signed-out
-// server goes to the host as Codex's startup notification, so both providers
-// warn the same way. The first look waits a second so the host already knows
-// this thread when the notice arrives.
-async function watchMcpStartup(session, attempts = 30, delay = 1000) {
-  for (let attempt = 0; attempt < attempts; attempt += 1) {
-    await new Promise((resolve) => setTimeout(resolve, delay));
-    if (sessions.get(session.id) !== session) return;
-    let statuses;
-    try { statuses = await session.query.mcpServerStatus(); } catch { return; }
-    if (!Array.isArray(statuses)) return;
-    if (statuses.some((server) => server?.status === "pending")) continue;
-    for (const server of statuses) {
-      if (server?.status !== "failed" && server?.status !== "needs-auth") continue;
-      const source = server.source || server.scope;
-      notify("mcpServer/startupStatus/updated", {
-        threadId: session.id,
-        name: String(server.name || "MCP"),
-        status: "failed",
-        ...(server.status === "needs-auth" ? { failureReason: "authenticationRequired" } : {}),
-        ...(server.error ? { error: String(server.error) } : {}),
-        ...(source ? { source: String(source) } : {}),
-      });
-    }
-    return;
-  }
-}
-
 async function safeAccount(agentQuery) {
   try { return await agentQuery.accountInfo(); } catch { return null; }
 }
@@ -2865,7 +2835,12 @@ async function processResult(session, message) {
   const detail = message.errors?.join("\n") || message.result || message.stop_reason || "Claude 실행 실패";
   const guide = STARTUP_FAILURE_GUIDES[message.startup_failure_reason];
   let error = message.is_error && !interrupted
-    ? { message: guide ? `${guide}\n${detail}` : detail }
+    ? {
+        message: guide ? `${guide}\n${detail}` : detail,
+        // An expired login gets the host's sign-in guide instead of the raw
+        // `API Error: 401 {...}`; Codex tags the same failure `unauthorized`.
+        ...(session.turn?.assistantError === "authentication_failed" ? { codexErrorInfo: "unauthorized" } : {}),
+      }
     : null;
   if (!error && !interrupted) {
     const outcome = abnormalStopOutcome(message.stop_reason);
@@ -2965,7 +2940,12 @@ function finishTurn(session, error, durationMs) {
   clearForegroundSubagents(session);
   const turn = { id: session.turn.id, status: error ? "failed" : session.turn.interruptRequested ? "interrupted" : "completed" };
   if (turn.status === "completed") completeLingeringTasks(session, Date.now());
-  if (error) turn.error = { message: error instanceof Error ? error.message : error.message || String(error) };
+  if (error) {
+    turn.error = {
+      message: error instanceof Error ? error.message : error.message || String(error),
+      ...(error.codexErrorInfo ? { codexErrorInfo: error.codexErrorInfo } : {}),
+    };
+  }
   if (durationMs != null) turn.durationMs = durationMs;
   notify("turn/completed", { threadId: session.id, turn });
   session.turn = null;
@@ -4282,39 +4262,6 @@ async function runSelfTest() {
   if (reconnected.join(",") !== "connected,disabled") {
     throw new Error(`Claude MCP reconnect self-test failed: ${reconnected.join(",")}`);
   }
-  // Startup failures wait out pending servers, then go out once each.
-  const startupPolls = [
-    [{ name: "slow", status: "pending" }],
-    [
-      { name: "ok", status: "connected" },
-      { name: "broken", status: "failed", error: "HTTP 401: token_expired" },
-      { name: "claude.ai Notion", status: "needs-auth", scope: "claudeai" },
-    ],
-  ];
-  const startupSession = {
-    id: "mcp-startup-self-test",
-    query: { mcpServerStatus: async () => startupPolls.shift() || [] },
-  };
-  sessions.set(startupSession.id, startupSession);
-  const startupNotices = [];
-  const startupWrite = process.stdout.write;
-  process.stdout.write = (chunk) => { startupNotices.push(JSON.parse(String(chunk))); return true; };
-  try {
-    await watchMcpStartup(startupSession, 3, 0);
-  } finally {
-    process.stdout.write = startupWrite;
-    sessions.delete(startupSession.id);
-  }
-  const startupNames = startupNotices.map((notice) => notice.params?.name).join(",");
-  if (startupNames !== "broken,claude.ai Notion"
-    || startupNotices.some((notice) => notice.method !== "mcpServer/startupStatus/updated"
-      || notice.params?.threadId !== startupSession.id
-      || notice.params?.status !== "failed")
-    || startupNotices[0].params.error !== "HTTP 401: token_expired"
-    || startupNotices[1].params.failureReason !== "authenticationRequired"
-    || startupNotices[1].params.source !== "claudeai") {
-    throw new Error(`Claude MCP startup self-test failed: ${JSON.stringify(startupNotices)}`);
-  }
   const user = (uuid, text) => ({
     type: "user",
     uuid,
@@ -4869,6 +4816,21 @@ async function runSelfTest() {
     await consumeMessage(limitSession, { type: "assistant", error: "server_error", message: { content: [] } });
     await consumeMessage(limitSession, failedResult);
     if (limitSession.usageLimitWait) throw new Error("Unrelated server error started a usage wait");
+    // An expired login reaches the host tagged like Codex's `unauthorized`.
+    const authNotices = [];
+    const authWrite = process.stdout.write;
+    process.stdout.write = (chunk) => { authNotices.push(JSON.parse(String(chunk))); return true; };
+    try {
+      beginTurn(limitSession);
+      await consumeMessage(limitSession, { type: "assistant", error: "authentication_failed", message: { content: [] } });
+      await consumeMessage(limitSession, { ...failedResult, errors: ["API Error: 401 {\"type\":\"error\"}"] });
+    } finally {
+      process.stdout.write = authWrite;
+    }
+    const authTurn = authNotices.find((notice) => notice.method === "turn/completed")?.params?.turn;
+    if (authTurn?.error?.codexErrorInfo !== "unauthorized" || !authTurn.error.message.includes("401")) {
+      throw new Error(`Claude login failure lost its tag: ${JSON.stringify(authTurn)}`);
+    }
     beginTurn(limitSession);
     for (let attempt = 0; attempt < 4; attempt++) {
       await rejectLimit();

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "devez-vibe",
-  "version": "1.9.36",
+  "version": "1.9.38",
   "type": "module",
   "description": "Stable terminal UI for Codex and Claude Agent SDK",
   "keywords": [