@sireai/optimus 0.1.42 → 0.1.44

package/dist/cli/optimus.js

@@ -26,9 +26,11 @@ import { buildTaskExecutionMetrics, formatExecutionMetricsCompact } from "../tas
 import { FeishuNotifier } from "../task-environment/delivery/feishu-notifier.js";
 import { TaskDeliveryDispatcher } from "../task-environment/delivery/task-delivery-dispatcher.js";
 import { TaskDeliveryService } from "../task-environment/delivery/task-delivery-service.js";
+import { formatDeliveryWarningsWithDescriptions } from "../task-environment/delivery/delivery-warning-copy.js";
 import { FeishuAnalysisDocService } from "../task-environment/delivery/feishu-analysis-doc-service.js";
 import { TaskPublicationService } from "../task-environment/delivery/task-publication-service.js";
 import { FeishuClient } from "../integrations/feishu/feishu-client.js";
+import { FeishuDocumentReader, mergeFeishuReferenceMaterials } from "../integrations/feishu/feishu-document-reader.js";
 import { FeishuUserService } from "../integrations/feishu/feishu-user-service.js";
 import { createFeedbackReport } from "./feedback.js";
 import { DEFAULT_FEEDBACK_RECIPIENT_EMAIL, FeishuFeedbackDeliveryService, persistFeedbackDeliveryResult, resolveDefaultFeedbackRecipientFallbackOpenId } from "./feedback-delivery.js";
@@ -2172,6 +2174,8 @@ function mergeReplayBundle(rebuilt, existing) {
             ...existing.summary,
             ...rebuilt.summary,
             ...(rebuilt.summary.analysisDocUrl ? {} : existing.summary.analysisDocUrl ? { analysisDocUrl: existing.summary.analysisDocUrl } : {}),
+            ...(rebuilt.summary.prototypePreviewUrl ? {} : existing.summary.prototypePreviewUrl ? { prototypePreviewUrl: existing.summary.prototypePreviewUrl } : {}),
+            ...(rebuilt.summary.prototypeDownloadUrl ? {} : existing.summary.prototypeDownloadUrl ? { prototypeDownloadUrl: existing.summary.prototypeDownloadUrl } : {}),
             ...(rebuilt.summary.repo ? {} : existing.summary.repo ? { repo: existing.summary.repo } : {}),
             ...(rebuilt.summary.sourceSystem ? {} : existing.summary.sourceSystem ? { sourceSystem: existing.summary.sourceSystem } : {}),
             ...(rebuilt.summary.sourceRef ? {} : existing.summary.sourceRef ? { sourceRef: existing.summary.sourceRef } : {}),
@@ -2226,6 +2230,7 @@ const TASK_EVENT_EXPLANATIONS = {
     "execution.completed": "任务执行已结束并完成收口。",
     "execution.failed": "任务执行已失败并完成收口。",
     "execution.canceled": "任务执行已取消并完成收口。",
+    "execution.review_subagent.skipped": "独立 reviewer subagent 调用失败，本轮已跳过 review 并继续按现有产物收口。",
     "task.needs_human": "任务需要人工补充信息或接手处理。",
     "task.delivery_bundle.warning": "交付产物存在需要关注的告警，通常表示结果文档缺少关键标准字段。",
 };
@@ -2245,6 +2250,7 @@ const DOCTOR_IMPORTANT_EVENT_TYPES = [
     "task.failed",
     "task.canceled",
     "task.timed_out",
+    "task.review_subagent.skipped",
     "task.fallback",
     "task.needs_human",
     "task.retry_enqueued",
@@ -2543,46 +2549,60 @@ async function resolveTaskTextFromUrl(url) {
     if (!isFeishuDocumentUrl(trimmedUrl)) {
         throw new Error("submit currently supports document URLs from Feishu/Lark only.");
     }
-    let stdout;
+    const reader = new FeishuDocumentReader();
     try {
-        const result = await execFileAsync("feishu", ["fetch", trimmedUrl], {
-            maxBuffer: 10 * 1024 * 1024
-        });
-        stdout = result.stdout;
+        const resolved = await reader.readFromUrl(trimmedUrl);
+        const referenceMaterials = mergeFeishuReferenceMaterials(extractFeishuReferenceMaterials(resolved.content, undefined, trimmedUrl), resolved.referenceMaterials);
+        return {
+            content: resolved.content,
+            ...(resolved.title ? { title: resolved.title } : {}),
+            sourceType: resolved.sourceType,
+            ...(resolved.sourceDocumentType ? { sourceDocumentType: resolved.sourceDocumentType } : {}),
+            ...(referenceMaterials ? { referenceMaterials } : {})
+        };
     }
-    catch (error) {
-        const execError = error;
-        if (execError?.code === "ENOENT") {
-            throw new Error("submit could not fetch the Feishu document because the `feishu` CLI is not installed or not on PATH.");
+    catch (builtinError) {
+        let stdout;
+        try {
+            const result = await execFileAsync("feishu", ["fetch", trimmedUrl], {
+                maxBuffer: 10 * 1024 * 1024
+            });
+            stdout = result.stdout;
         }
-        const detail = execError?.stderr?.trim() || execError?.stdout?.trim() || execError?.message || "unknown error";
-        throw new Error(`submit could not fetch the Feishu document: ${detail}`);
-    }
-    let payload;
-    try {
-        payload = JSON.parse(stdout);
-    }
-    catch {
-        throw new Error("submit received non-JSON output from `feishu fetch`.");
-    }
-    if (!payload || typeof payload !== "object" || Array.isArray(payload)) {
-        throw new Error("submit received an invalid response from `feishu fetch`.");
-    }
-    const record = payload;
-    const markdown = typeof record.markdown === "string" ? record.markdown.trim() : "";
-    const title = (typeof record.title === "string" ? record.title.trim() : "") || extractTitleFromMarkdown(markdown);
-    const sourceDocumentType = typeof record.type === "string" ? record.type.trim() : undefined;
-    const referenceMaterials = extractFeishuReferenceMaterials(markdown, record.media, trimmedUrl);
-    if (!markdown) {
-        throw new Error("submit fetched the document URL, but it did not return readable markdown content.");
+        catch (error) {
+            const execError = error;
+            const cliDetail = execError?.code === "ENOENT"
+                ? "the `feishu` CLI is not installed or not on PATH"
+                : execError?.stderr?.trim() || execError?.stdout?.trim() || execError?.message || "unknown error";
+            const builtinDetail = builtinError instanceof Error ? builtinError.message : String(builtinError);
+            throw new Error(`submit could not fetch the Feishu document via built-in app (${builtinDetail}) or via \`feishu fetch\` (${cliDetail}).`);
+        }
+        let payload;
+        try {
+            payload = JSON.parse(stdout);
+        }
+        catch {
+            throw new Error("submit received non-JSON output from `feishu fetch`.");
+        }
+        if (!payload || typeof payload !== "object" || Array.isArray(payload)) {
+            throw new Error("submit received an invalid response from `feishu fetch`.");
+        }
+        const record = payload;
+        const markdown = typeof record.markdown === "string" ? record.markdown.trim() : "";
+        const title = (typeof record.title === "string" ? record.title.trim() : "") || extractTitleFromMarkdown(markdown);
+        const sourceDocumentType = typeof record.type === "string" ? record.type.trim() : undefined;
+        const referenceMaterials = extractFeishuReferenceMaterials(markdown, record.media, trimmedUrl);
+        if (!markdown) {
+            throw new Error("submit fetched the document URL, but it did not return readable markdown content.");
+        }
+        return {
+            content: markdown,
+            ...(title ? { title } : {}),
+            sourceType: "feishu_doc",
+            ...(sourceDocumentType ? { sourceDocumentType } : {}),
+            ...(referenceMaterials ? { referenceMaterials } : {})
+        };
     }
-    return {
-        content: markdown,
-        ...(title ? { title } : {}),
-        sourceType: "feishu_doc",
-        ...(sourceDocumentType ? { sourceDocumentType } : {}),
-        ...(referenceMaterials ? { referenceMaterials } : {})
-    };
 }
 async function main() {
     const argv = process.argv.slice(2);
@@ -2799,7 +2819,7 @@ async function main() {
         }
         else {
             try {
-                const remote = await resolveTaskTextFromUrl(url);
+                const remote = await runWithTerminalLoading("Resolving remote requirement document", async () => await resolveTaskTextFromUrl(url));
                 description = remote.content;
                 resolvedTitle = resolvedTitle || remote.title;
                 remoteSourceType = remote.sourceType;
@@ -4031,7 +4051,7 @@ function renderTaskResultReport(input) {
             lines.push(`Metrics: ${metrics}`);
         }
         if (input.deliveryBundle.warnings?.length) {
-            lines.push(`Warnings: ${input.deliveryBundle.warnings.join(", ")}`);
+            lines.push(`Warnings: ${formatDeliveryWarningsWithDescriptions(input.deliveryBundle.warnings)}`);
         }
         if (input.deliveryBundle.publication) {
             lines.push(`Publication: ${input.deliveryBundle.publication.action}${input.deliveryBundle.publication.reason ? ` (${input.deliveryBundle.publication.reason})` : ""}`);
@@ -4144,7 +4164,7 @@ function renderDeliveryStatusReport(input) {
         lines.push(`Decision: ${input.deliveryBundle.summary.decision}`);
         lines.push(`Created At: ${input.deliveryBundle.createdAt}`);
         if (input.deliveryBundle.warnings?.length) {
-            lines.push(`Warnings: ${input.deliveryBundle.warnings.join(", ")}`);
+            lines.push(`Warnings: ${formatDeliveryWarningsWithDescriptions(input.deliveryBundle.warnings)}`);
         }
     }
     else {