@synergenius/flow-weaver-pack-weaver 0.9.152 → 0.9.154

package/src/ui/task-detail-view.tsx

@@ -24,7 +24,7 @@ import type { HistoricalRun } from './trace-to-timeline';
 // Types
 // ---------------------------------------------------------------------------
 
-type TaskStatus = 'pending' | 'in-progress' | 'blocked' | 'done' | 'failed' | 'cancelled';
+type TaskStatus = 'open' | 'pending' | 'in-progress' | 'blocked' | 'done' | 'cancelled';
 
 interface TaskContext {
   files: string[];
@@ -89,19 +89,19 @@ interface TaskDetailViewProps {
 // ---------------------------------------------------------------------------
 
 const statusToIcon: Record<TaskStatus, string> = {
+  'open': 'pending',
   'pending': 'pending',
   'in-progress': 'running',
   'done': 'completed',
-  'failed': 'failed',
   'blocked': 'pending',
   'cancelled': 'failed',
 };
 
 const statusToLabel: Record<TaskStatus, string> = {
-  'pending': 'Pending',
-  'in-progress': 'In Progress',
+  'open': 'Open',
+  'pending': 'Queued',
+  'in-progress': 'Running',
   'done': 'Done',
-  'failed': 'Failed',
   'blocked': 'Blocked',
   'cancelled': 'Cancelled',
 };
@@ -223,6 +223,27 @@ function TaskDetailView({ taskId, onBack, onEdit }: TaskDetailViewProps) {
       const t = (data.task ?? data) as Task;
       setTask(t);
 
+      // Populate run history from task's runSummaries (RunStore may be empty for swarm tasks)
+      const rs = (t as unknown as Record<string, unknown>).context as Record<string, unknown> | undefined;
+      const summaries = rs?.runSummaries as Array<Record<string, unknown>> | undefined;
+      if (summaries?.length) {
+        setHistory(prev => {
+          if (prev.length > 0) return prev; // Don't overwrite if RunStore data exists
+          return summaries.map((s) => ({
+            id: s.runId as string,
+            runId: s.runId as string,
+            outcome: s.outcome as string,
+            summary: s.summary as string,
+            report: s.report as string | undefined,
+            filesModified: s.filesModified as string[],
+            durationMs: s.durationMs as number,
+            cost: s.cost as number,
+            tokensUsed: s.tokensUsed as number,
+            error: s.error as string | undefined,
+          } as unknown as HistoricalRun));
+        });
+      }
+
       // Subtasks from response or fetch separately
       if (data.subtasks && Array.isArray(data.subtasks)) {
         setSubtasks(data.subtasks as Subtask[]);
@@ -242,13 +263,14 @@ function TaskDetailView({ taskId, onBack, onEdit }: TaskDetailViewProps) {
     try {
       const raw = await callTool('fw_weaver_history', { taskId });
       const data = typeof raw === 'string' ? JSON.parse(raw) : raw;
-      if (Array.isArray(data)) {
+      if (Array.isArray(data) && data.length > 0) {
         setHistory(
           data.map((r: Record<string, unknown>) => {
             const costObj = r.cost && typeof r.cost === 'object' ? (r.cost as Record<string, unknown>) : undefined;
             return { ...r, costDetail: costObj, cost: costObj?.totalCost } as unknown as HistoricalRun;
           }),
         );
+        return;
       }
     } catch {
       // Non-fatal
@@ -261,6 +283,7 @@ function TaskDetailView({ taskId, onBack, onEdit }: TaskDetailViewProps) {
     Promise.all([fetchTask(), fetchHistory()]).finally(() => setLoading(false));
   }, [fetchTask, fetchHistory]);
 
+
   // Poll while in-progress (every 5s)
   useEffect(() => {
     if (!task || task.status !== 'in-progress') return;
@@ -545,10 +568,10 @@ function TaskDetailView({ taskId, onBack, onEdit }: TaskDetailViewProps) {
       // Meta row: status tag, assigned profile, priority, attempts
       React.createElement(Flex, { variant: 'row-center-start-wrap-6' },
         React.createElement(Chip, {
-          label: statusToLabel[task.status] || task.status || 'pending',
+          label: statusToLabel[task.status as TaskStatus] || task.status || 'open',
           size: 'small',
           color: task.status === 'done' ? 'color-status-positive'
-            : task.status === 'failed' ? 'color-status-negative'
+            : task.status === 'cancelled' ? 'color-status-negative'
             : task.status === 'in-progress' ? 'color-status-info'
             : task.status === 'blocked' ? 'color-status-caution'
             : 'color-brand-alt',
@@ -560,10 +583,10 @@ function TaskDetailView({ taskId, onBack, onEdit }: TaskDetailViewProps) {
           label: `P${task.priority}`, size: 'small', color: task.priority >= 3 ? 'color-status-caution' : 'color-status-info',
         }),
 
-        (task.attempt != null && task.maxAttempts != null) && React.createElement(Typography, {
+        task.maxAttempts != null && task.maxAttempts > 1 && React.createElement(Typography, {
           variant: 'smallCaption-regular',
           color: 'color-text-subtle',
-        }, `Attempt ${task.attempt}/${task.maxAttempts}`),
+        }, `${task.runs?.length ?? 0}/${task.maxAttempts} runs`),
       ),
 
       // Description
@@ -673,14 +696,14 @@ function TaskDetailView({ taskId, onBack, onEdit }: TaskDetailViewProps) {
           React.createElement(Flex, { variant: 'column-stretch-start-nowrap-8' },
             React.createElement(Typography, { variant: 'caption-thick', color: 'color-text-medium' }, 'Status'),
             React.createElement(Flex, { variant: 'row-center-start-nowrap-6' },
-              task.status === 'failed' && React.createElement(Button, {
+              (task.status === 'open' && (task.attempt ?? 0) > 0) && React.createElement(Button, {
                 size: 'xs', variant: 'fill', color: 'primary',
                 onClick: handleRetry,
                 loading: actionLoading === 'retry',
                 disabled: !!actionLoading,
               }, 'Retry Task'),
 
-              (task.status === 'pending' || task.status === 'in-progress' || task.status === 'blocked') && React.createElement(Button, {
+              (task.status === 'open' || task.status === 'pending' || task.status === 'in-progress' || task.status === 'blocked') && React.createElement(Button, {
                 size: 'xs', variant: 'outlined', color: 'danger',
                 onClick: handleCancel,
                 loading: actionLoading === 'cancel',

package/src/ui/task-editor.tsx

@@ -29,7 +29,7 @@ interface TaskEditorProps {
   onDelete?: () => void;
 }
 
-type TaskStatus = 'pending' | 'in-progress' | 'blocked' | 'done' | 'failed' | 'cancelled';
+type TaskStatus = 'open' | 'pending' | 'in-progress' | 'blocked' | 'done' | 'cancelled';
 
 interface TaskData {
   id: string;
@@ -77,11 +77,11 @@ const COMPLEXITY_OPTIONS = [
 ];
 
 const STATUS_COLOR: Record<string, string> = {
+  'open': 'color-brand-alt',
   'pending': 'color-brand-alt',
   'in-progress': 'color-status-info',
   'blocked': 'color-status-caution',
   'done': 'color-status-positive',
-  'failed': 'color-status-negative',
   'cancelled': 'color-status-negative',
 };
 

package/src/ui/task-pool-list.tsx

@@ -11,7 +11,7 @@ const { Flex, Typography, Icon, StatusIcon, Chip, ScrollArea, Badge, EmptyState
 
 // --- Types ---
 
-type TaskStatus = 'pending' | 'in-progress' | 'blocked' | 'done' | 'failed' | 'cancelled';
+type TaskStatus = 'open' | 'pending' | 'in-progress' | 'blocked' | 'done' | 'cancelled';
 
 interface Task {
   id: string;
@@ -32,10 +32,10 @@ interface TaskPoolListProps {
 // --- Status mapping ---
 
 const statusToIcon: Record<TaskStatus, string> = {
+  'open': 'pending',
   'pending': 'pending',
   'in-progress': 'running',
   'done': 'completed',
-  'failed': 'failed',
   'blocked': 'pending',
   'cancelled': 'failed',
 };

package/src/workflows/weaver-bot.ts

@@ -423,7 +423,7 @@ class GeneratedExecutionContext {
 
 /**
  * @flowWeaver workflow
- * @node cfg weaverLoadConfig [color: "teal"] [icon: "settings"] [position: 200 200]
+ * @node cfg weaverLoadConfig [color: "teal"] [icon: "settings"] [suppress: "AGENT_UNGUARDED_TOOL_EXECUTOR"] [position: 200 200]
  * @node detect weaverDetectProvider [color: "cyan"] [icon: "search"] [suppress: "OBJECT_TYPE_MISMATCH", "ANNOTATION_SIGNATURE_TYPE_MISMATCH"] [position: 400 200]
  * @node receive weaverReceiveTask [color: "blue"] [icon: "send"] [position: 600 200]
  * @node route weaverRouteTask [color: "purple"] [icon: "flow"] [position: 800 200]
@@ -441,7 +441,7 @@ class GeneratedExecutionContext {
  * @node resolveReviewModel weaverResolveModel [label: "Review Model"] [expr: phaseName="'review'"] [minimized] [color: "green"] [icon: "tune"] [position: 1900 50]
  * @node review weaverReviewResult [color: "green"] [icon: "spellcheck"] [position: 1950 100]
  * @node mergeReview weaverCtxMerge [label: "Merge Review"] [minimized] [color: "teal"] [icon: "callMerge"] [position: 2050 200]
- * @node gitOps weaverGitOps [color: "green"] [icon: "code"] [position: 2150 100]
+ * @node gitOps weaverGitOps [color: "green"] [icon: "code"] [suppress: "UNUSED_OUTPUT_PORT"] [position: 2150 100]
  * @node notify weaverSendNotify [color: "yellow"] [icon: "send"] [suppress: "UNUSED_OUTPUT_PORT"] [position: 2050 400]
  * @node report weaverBotReport [color: "green"] [icon: "description"] [suppress: "UNUSED_OUTPUT_PORT", "DESIGN_ASYNC_NO_ERROR_PATH"] [position: 2350 200]
  * @path Start -> cfg -> detect -> receive -> route -> context -> gatePlan -> resolvePlanModel -> plan -> mergePlan -> approve -> resolveExecModel -> execRetry -> gateReview -> resolveReviewModel -> review -> mergeReview -> gitOps -> report -> Exit
@@ -465,11 +465,12 @@ class GeneratedExecutionContext {
  * @connect readWf.onSuccess -> report.execute
  * @connect abort.onSuccess -> report.execute
  * @connect notify.onSuccess -> report.execute
- * @connect gitOps.ctx -> report.mainCtx
+ * @connect mergePlan.ctx -> report.mainCtx
  * @connect readWf.ctx -> report.readCtx
  * @connect abort.ctx -> report.abortCtx
- * @connect execRetry.ctx -> report.failCtx
+ * @connect plan.ctx -> report.failCtx
  * @connect report.summary -> Exit.summary
+ * @connect review.onFailure -> report.execute
  * @param execute [order:-1] - Execute
  * @param taskJson [order:0] - TaskJson
  * @param projectDir [order:1] - ProjectDir
@@ -2116,15 +2117,15 @@ export async function weaverBot(
       });
 
       try {
-        const report_execute = (readWfIdx !== undefined ? await ctx.getVariable({ id: 'readWf', portName: 'onSuccess', executionIndex: readWfIdx }) as boolean : false) || (abortIdx !== undefined ? await ctx.getVariable({ id: 'abort', portName: 'onSuccess', executionIndex: abortIdx }) as boolean : false) || (notifyIdx !== undefined ? await ctx.getVariable({ id: 'notify', portName: 'onSuccess', executionIndex: notifyIdx }) as boolean : false) || (gitOpsIdx !== undefined ? await ctx.getVariable({ id: 'gitOps', portName: 'onSuccess', executionIndex: gitOpsIdx }) as boolean : false) || (receiveIdx !== undefined ? await ctx.getVariable({ id: 'receive', portName: 'onFailure', executionIndex: receiveIdx }) as boolean : false) || (planIdx !== undefined ? await ctx.getVariable({ id: 'plan', portName: 'onFailure', executionIndex: planIdx }) as boolean : false) || (execRetryIdx !== undefined ? await ctx.getVariable({ id: 'execRetry', portName: 'onFailure', executionIndex: execRetryIdx }) as boolean : false);
+        const report_execute = (readWfIdx !== undefined ? await ctx.getVariable({ id: 'readWf', portName: 'onSuccess', executionIndex: readWfIdx }) as boolean : false) || (abortIdx !== undefined ? await ctx.getVariable({ id: 'abort', portName: 'onSuccess', executionIndex: abortIdx }) as boolean : false) || (notifyIdx !== undefined ? await ctx.getVariable({ id: 'notify', portName: 'onSuccess', executionIndex: notifyIdx }) as boolean : false) || (gitOpsIdx !== undefined ? await ctx.getVariable({ id: 'gitOps', portName: 'onSuccess', executionIndex: gitOpsIdx }) as boolean : false) || (receiveIdx !== undefined ? await ctx.getVariable({ id: 'receive', portName: 'onFailure', executionIndex: receiveIdx }) as boolean : false) || (planIdx !== undefined ? await ctx.getVariable({ id: 'plan', portName: 'onFailure', executionIndex: planIdx }) as boolean : false) || (execRetryIdx !== undefined ? await ctx.getVariable({ id: 'execRetry', portName: 'onFailure', executionIndex: execRetryIdx }) as boolean : false) || (reviewIdx !== undefined ? await ctx.getVariable({ id: 'review', portName: 'onFailure', executionIndex: reviewIdx }) as boolean : false);
         await ctx.setVariable({ id: 'report', portName: 'execute', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, report_execute);
-        const report_mainCtx = gitOpsIdx !== undefined ? await ctx.getVariable({ id: 'gitOps', portName: 'ctx', executionIndex: gitOpsIdx }) as string : undefined;
+        const report_mainCtx = mergePlanIdx !== undefined ? await ctx.getVariable({ id: 'mergePlan', portName: 'ctx', executionIndex: mergePlanIdx }) as string : undefined;
         await ctx.setVariable({ id: 'report', portName: 'mainCtx', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, report_mainCtx);
         const report_readCtx = readWfIdx !== undefined ? await ctx.getVariable({ id: 'readWf', portName: 'ctx', executionIndex: readWfIdx }) as string : undefined;
         await ctx.setVariable({ id: 'report', portName: 'readCtx', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, report_readCtx);
         const report_abortCtx = abortIdx !== undefined ? await ctx.getVariable({ id: 'abort', portName: 'ctx', executionIndex: abortIdx }) as string : undefined;
         await ctx.setVariable({ id: 'report', portName: 'abortCtx', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, report_abortCtx);
-        const report_failCtx = execRetryIdx !== undefined ? await ctx.getVariable({ id: 'execRetry', portName: 'ctx', executionIndex: execRetryIdx }) as string : undefined;
+        const report_failCtx = planIdx !== undefined ? await ctx.getVariable({ id: 'plan', portName: 'ctx', executionIndex: planIdx }) as string : undefined;
         await ctx.setVariable({ id: 'report', portName: 'failCtx', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, report_failCtx);
         const reportResult = await weaverBotReport(report_execute, report_mainCtx, report_readCtx, report_abortCtx, report_failCtx);
         await ctx.setVariable({ id: 'report', portName: 'summary', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, reportResult.summary);