@lightcone-ai/daemon 0.15.9 → 0.15.10

package/mcp-servers/publisher/index.js

@@ -457,16 +457,19 @@ server.tool(
         const adapter = await getAdapter(platform);
         return adapter.checkLoginStatus();
       });
+      // Always close session after login check so the Chrome profile is free
+      // for the daemon's publish job (which runs in a separate process)
+      closeSession(platform);
       if (loggedIn) {
         return { content: [{ type: 'text', text: `✓ ${label} 已登录，可以发布内容。` }] };
       } else {
-        closeSession(platform);
         return {
           content: [{ type: 'text', text: `${label} 未登录或登录已过期。\n调试信息: url=${url}\nprofileDir=${profileDir}\n请在「连接外部账号」中重新扫码连接。` }],
           isError: true,
         };
       }
     } catch (err) {
+      closeSession(platform);
       return { content: [{ type: 'text', text: `检查失败: ${err.message}` }], isError: true };
     }
   }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lightcone-ai/daemon",
-  "version": "0.15.9",
+  "version": "0.15.10",
   "type": "module",
   "main": "src/index.js",
   "bin": {

package/src/chat-bridge.js

@@ -22,6 +22,12 @@ import { runRecordUrlNarrationTool } from './record-url-narration-tool.js';
 import { runSubmitToLibraryTool } from './submit-to-library-tool.js';
 import { isLeaseInvalidated, clearInvalidatedLease } from './governance-state.js';
 import { classifyLeaseWindow } from './lease-window.js';
+import {
+  buildDirectApiHttpError,
+  buildDirectApiTransportError,
+  shouldEmitToolCallFailed,
+  toolCallFailedReason,
+} from './tool-call-failure.js';
 
 const cliArgs = process.argv.slice(2);
 function getArg(name) {
@@ -528,17 +534,22 @@ function enqueueBundleEvent(eventType, payload = {}) {
 
 async function directApi(method, apiPath, body) {
   const url = `${SERVER_URL}/internal/agent/${AGENT_ID}${apiPath}`;
-  const res = await fetch(url, {
-    method,
-    headers: {
-      'Content-Type': 'application/json',
-      'Authorization': `Bearer ${MACHINE_API_KEY}`,
-    },
-    body: body != null ? JSON.stringify(body) : undefined,
-  });
+  let res;
+  try {
+    res = await fetch(url, {
+      method,
+      headers: {
+        'Content-Type': 'application/json',
+        'Authorization': `Bearer ${MACHINE_API_KEY}`,
+      },
+      body: body != null ? JSON.stringify(body) : undefined,
+    });
+  } catch (error) {
+    throw buildDirectApiTransportError({ method, apiPath, error });
+  }
   if (!res.ok) {
     const text = await res.text();
-    throw new Error(`API ${method} ${apiPath} → ${res.status}: ${text}`);
+    throw buildDirectApiHttpError({ method, apiPath, status: res.status, text });
   }
   return res.json();
 }
@@ -555,16 +566,21 @@ async function directApiVideoUpload(apiPath, {
   };
   if (filename) headers['X-File-Name'] = filename;
 
-  const res = await fetch(url, {
-    method: 'POST',
-    headers,
-    body: createReadStream(localPath),
-    duplex: 'half',
-  });
+  let res;
+  try {
+    res = await fetch(url, {
+      method: 'POST',
+      headers,
+      body: createReadStream(localPath),
+      duplex: 'half',
+    });
+  } catch (error) {
+    throw buildDirectApiTransportError({ method: 'POST', apiPath, error });
+  }
 
   if (!res.ok) {
     const text = await res.text();
-    throw new Error(`API POST ${apiPath} → ${res.status}: ${text}`);
+    throw buildDirectApiHttpError({ method: 'POST', apiPath, status: res.status, text });
   }
   return res.json();
 }
@@ -701,12 +717,14 @@ async function runMandatoryLocalTool({ toolName, toolInput = {}, executor }) {
     });
     return result;
   } catch (error) {
-    enqueueBundleEvent('tool_call_failed', {
-      trace_id: traceId,
-      tool_name: toolName,
-      tool_classification: classification,
-      reason: error?.code ?? error?.message ?? 'unknown_error',
-    });
+    if (shouldEmitToolCallFailed(error)) {
+      enqueueBundleEvent('tool_call_failed', {
+        trace_id: traceId,
+        tool_name: toolName,
+        tool_classification: classification,
+        reason: toolCallFailedReason(error),
+      });
+    }
     throw error;
   }
 }
@@ -854,12 +872,14 @@ async function api(method, apiPath, body) {
     });
     return data;
   } catch (err) {
-    enqueueBundleEvent('tool_call_failed', {
-      trace_id: traceId,
-      tool_name: toolName,
-      tool_classification: classification,
-      reason: err?.code ?? err?.message ?? 'unknown_error',
-    });
+    if (shouldEmitToolCallFailed(err)) {
+      enqueueBundleEvent('tool_call_failed', {
+        trace_id: traceId,
+        tool_name: toolName,
+        tool_classification: classification,
+        reason: toolCallFailedReason(err),
+      });
+    }
     throw err;
   }
 }

package/src/tool-call-failure.js

@@ -0,0 +1,26 @@
+export function buildDirectApiTransportError({ method, apiPath, error }) {
+  const wrapped = new Error(`API ${method} ${apiPath} transport_error: ${error?.message ?? 'unknown_error'}`);
+  wrapped.code = error?.name === 'AbortError' ? 'timeout' : 'server_error';
+  wrapped.cause = error;
+  return wrapped;
+}
+
+export function buildDirectApiHttpError({ method, apiPath, status, text }) {
+  const error = new Error(`API ${method} ${apiPath} → ${status}: ${text}`);
+  error.status = status;
+  error.code = status >= 500 ? 'server_error' : 'api_client_error';
+  return error;
+}
+
+export function shouldEmitToolCallFailed(error) {
+  const status = Number(error?.status);
+  if (Number.isFinite(status) && status >= 400 && status < 500) return false;
+  if (error?.code === 'api_client_error') return false;
+  return true;
+}
+
+export function toolCallFailedReason(error) {
+  if (typeof error?.code === 'string' && error.code.trim()) return error.code.trim();
+  if (typeof error?.message === 'string' && error.message.trim()) return error.message.trim();
+  return 'unknown_error';
+}