@lightcone-ai/daemon 0.15.63 → 0.15.65

package/package.json

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

package/src/mcp-config.js

@@ -62,25 +62,40 @@ function resolveSkillArg(arg, config) {
   return arg;
 }
 
+// MCP servers that talk to the lightcone server at runtime — either a full
+// server-backed MCP (mysql / publisher / platform / domain-data services) or a
+// thin-proxy (`daemon/mcp-servers/_thin-proxy/forward.js`) whose tools forward
+// to /internal/agent/:agentId/mcp/:serverId/:toolName. All of these need the
+// SERVER_URL / MACHINE_API_KEY / AGENT_ID triple injected, or the proxy throws
+// "thin-proxy missing SERVER_URL / MACHINE_API_KEY / AGENT_ID env".
+const SERVER_BACKED_MCP_SERVERS = new Set([
+  'mysql',
+  'publisher',
+  'platform',
+  'research-fetch',
+  'wechat-mp-fetch',
+  'market-data-query',
+  'company-fundamentals',
+  'industry-report',
+  'risk-metrics',
+  'compliance-check',
+  'portfolio-read',
+  'portfolio-analysis',
+  // thin-proxy MCP services migrated to the server (roadmap §4) — every one of
+  // these uses startThinProxy() and requires the SERVER_URL/MACHINE_API_KEY/AGENT_ID triple.
+  'video-narration-planner',
+  'page-understanding',
+  'platform-policy-db',
+  'keyword-research',
+  'audience-research',
+  'hook-pattern-library',
+]);
+
 function baseEnvForServer(serverKey, { serverUrl, authToken, agentId, workspaceId, workspaceDir }) {
   if (serverKey === 'workspace-migrate') {
     return { SERVER_URL: serverUrl, MACHINE_API_KEY: authToken, AGENT_ID: agentId };
   }
-  if (
-    serverKey === 'mysql'
-    || serverKey === 'publisher'
-    || serverKey === 'platform'
-    || serverKey === 'research-fetch'
-    || serverKey === 'wechat-mp-fetch'
-    || serverKey === 'market-data-query'
-    || serverKey === 'company-fundamentals'
-    || serverKey === 'industry-report'
-    || serverKey === 'risk-metrics'
-    || serverKey === 'compliance-check'
-    || serverKey === 'portfolio-read'
-    || serverKey === 'portfolio-analysis'
-    || serverKey === 'video-narration-planner'
-  ) {
+  if (SERVER_BACKED_MCP_SERVERS.has(serverKey)) {
     return {
       SERVER_URL: serverUrl,
       MACHINE_API_KEY: authToken,

package/src/tools/compose-video-v2.js

@@ -1,5 +1,6 @@
 import path from 'path';
 import os from 'os';
+import fs from 'fs';
 import { randomUUID } from 'crypto';
 import { composeVideoV2 } from '../_vendor/video/composer-v2/index.js';
 
@@ -11,11 +12,22 @@ function toolError(text) {
   return { isError: true, content: [{ type: 'text', text }] };
 }
 
+// A real screenshot / rendered card / video frame is at least tens of KB. A
+// few-KB image is almost certainly a blank or broken capture (e.g. the page
+// hadn't finished loading when screenshotted). Composing from those produces a
+// near-blank video, so reject up front rather than emit garbage.
+const MIN_IMAGE_BYTES = 4 * 1024;
+
+function statSizeOrNull(p) {
+  try { return fs.statSync(p).size; } catch { return null; }
+}
+
 export async function runComposeVideoV2Tool({ segments, outro_paths, format, resolution, output_path, workspaceDir }) {
   if (!Array.isArray(segments) || segments.length === 0) {
     return toolError('segments must be a non-empty array.');
   }
 
+  const imagePaths = [];
   for (let i = 0; i < segments.length; i++) {
     const seg = segments[i];
     const kind = seg.visual_kind;
@@ -30,6 +42,30 @@ export async function runComposeVideoV2Tool({ segments, outro_paths, format, res
     if (kind !== 'carousel' && !seg.visual_path) {
       return toolError(`segments[${i}]: visual_path required for kind=${kind}.`);
     }
+    if (kind === 'image' && seg.visual_path) imagePaths.push(seg.visual_path);
+    if (kind === 'carousel' && Array.isArray(seg.visual_paths)) imagePaths.push(...seg.visual_paths.filter(Boolean));
+  }
+
+  // Garbage-in guard: refuse to compose from blank/broken source images.
+  for (const p of new Set(imagePaths)) {
+    const size = statSizeOrNull(p);
+    if (size === null) {
+      return toolError(`Source image not found: ${p}. Re-capture/render it before composing.`);
+    }
+    if (size < MIN_IMAGE_BYTES) {
+      return toolError(
+        `Source image ${p} is only ${size} bytes — almost certainly a blank or broken capture `
+        + `(e.g. the page hadn't finished loading when screenshotted). Re-capture with the page fully loaded, then retry.`
+      );
+    }
+  }
+  // Heuristic warning: a multi-segment image video that reuses one single image
+  // will look near-static — usually a sign the source page didn't render and the
+  // agent fell back to one blank screenshot.
+  let warning = null;
+  if (imagePaths.length >= 2 && new Set(imagePaths).size === 1) {
+    warning = `WARNING: all ${imagePaths.length} image segments reuse the same file (${imagePaths[0]}). `
+      + 'The output will be near-static — verify the source page actually rendered before submitting this video.';
   }
 
   const outDir = workspaceDir
@@ -46,13 +82,15 @@ export async function runComposeVideoV2Tool({ segments, outro_paths, format, res
       output_path: outPath,
     });
 
-    return toolText([
+    const lines = [
       'compose_video_v2 completed.',
       `path=${result.path}`,
       `duration_ms=${result.duration_ms}`,
       `segments=${segments.length}`,
       `outro_clips=${(outro_paths ?? []).length}`,
-    ].join('\n'));
+    ];
+    if (warning) lines.push(warning);
+    return toolText(lines.join('\n'));
   } catch (error) {
     return toolError(`compose_video_v2 failed: ${error.message}`);
   }