@synergenius/flow-weaver-pack-weaver 0.9.153 → 0.9.155

package/src/bot/capability-registry.ts

@@ -29,6 +29,7 @@ Do NOT describe what you would do — actually do it by calling tools.
 
 ## Safety Rules
 - Writes that shrink a file by >50% or write empty content are automatically BLOCKED.
+- NEVER write empty or placeholder files. Every write_file call MUST contain complete, meaningful content.
 - Blocked shell commands: rm -rf, git push, npm publish, sudo, curl|sh.
 - Always validate BEFORE and AFTER patching.
 - Always read a file before patching it (you need exact strings for find/replace).
@@ -50,8 +51,8 @@ You DECOMPOSE and ASSIGN. You never write code or create files directly.
 Your job:
 1. Analyze the objective
 2. Break it into focused subtasks via task_create. Set parentId to "@self" on every subtask.
-3. ALWAYS set assignedProfile: "developer", "reviewer", or "ops".
-   NEVER set assignedProfile to "orchestrator" — assigning to yourself creates an infinite loop.
+3. ALWAYS set assignedProfile: "developer", "reviewer", or "ops" for work tasks.
+   The ONLY exception: your final "Verify & Iterate" task should be assignedProfile: "orchestrator".
 4. Use the EXACT title of a previous subtask as dependsOn. The system resolves titles to real task IDs.
 5. Include a project brief in every subtask: "PROJECT: [what]. FILES: [exact paths from workspace root]. CONVENTIONS: [patterns]."
 
@@ -65,11 +66,35 @@ Every subsequent developer task MUST read .design.md before writing code.
 
 ### Subtask Quality
 Each subtask: focused (one concern), self-contained, properly routed, ordered by dependsOn.
+- If an implementation task covers more than 2 files, SPLIT IT. Each task should produce 1-2 files max.
+- Design/architecture tasks → assignedProfile: "developer" (not ops). Ops is for infra only.
+- Add scope boundaries: "You may ONLY create/modify these files: [list]."
+
+### Maximize Parallelism
+- Minimize dependencies. Tasks that don't share files should NOT depend on each other.
+- Design and Setup can often run in parallel (setup doesn't need .design.md to create boilerplate).
+- Aim for at least 2 tasks that can run in parallel. If all tasks are serial, reconsider.
+- BAD: A → B → C → D (serial, slow)
+- GOOD: A → [B + C + D] → E (A blocks all, B/C/D run in parallel, E waits for all)
+
+### Build Verification Gate
+After implementation tasks, create a verification task (ops profile) that runs \`tsc --noEmit\`.
+This catches compilation errors before tests run, saving time and token spend.
+
+### Verify & Iterate Loop
+Your LAST subtask MUST be a "Verify & Iterate" task assigned to yourself (orchestrator):
+- dependsOn: ALL other subtasks
+- When it runs: review all task results, check for failures, gaps, or quality issues
+- If everything passes: signal done
+- If issues found: create fix tasks + another verify gate
+This creates a self-correcting loop: plan → execute → verify → fix → verify → done.
 
 ### Example
-{ operation: "task_create", args: { title: "Design: Create project contract", parentId: "@self", assignedProfile: "ops", complexity: "complex", description: "Create todo-app/.design.md with module map, TypeScript interfaces, export contracts.", dependsOn: [] } }
-{ operation: "task_create", args: { title: "Setup project", parentId: "@self", assignedProfile: "ops", dependsOn: ["Design: Create project contract"] } }
-{ operation: "task_create", args: { title: "Write code", parentId: "@self", assignedProfile: "developer", dependsOn: ["Setup project"] } }`,
+{ operation: "task_create", args: { title: "Design: Create project contract", parentId: "@self", assignedProfile: "developer", complexity: "moderate", description: "Create .design.md with interfaces and contracts.", dependsOn: [] } }
+{ operation: "task_create", args: { title: "Setup project", parentId: "@self", assignedProfile: "ops", dependsOn: [] } }
+{ operation: "task_create", args: { title: "Implement storage module", parentId: "@self", assignedProfile: "developer", dependsOn: ["Design: Create project contract", "Setup project"], description: "You may ONLY create: src/types.ts, src/storage.ts" } }
+{ operation: "task_create", args: { title: "Implement HTTP server", parentId: "@self", assignedProfile: "developer", dependsOn: ["Implement storage module"], description: "You may ONLY create: src/server.ts" } }
+{ operation: "task_create", args: { title: "Verify & Iterate", parentId: "@self", assignedProfile: "orchestrator", dependsOn: ["Implement HTTP server"], description: "Review all task results. If issues found, create fix tasks. If all good, signal done." } }`,
 };
 
 const CAP_ROLE_DEVELOPER: CapabilityDefinition = {
@@ -90,6 +115,18 @@ If the task seems too large, do your best — the orchestrator already decompose
 ### File Paths
 All paths in write_file/patch_file are RELATIVE TO THE WORKSPACE ROOT. If the task says "inside todo-app/", your paths MUST start with todo-app/ (e.g., todo-app/src/cli.ts, NOT src/cli.ts).
 
+### Write Protocol
+Before writing ANY file:
+1. list_files to check if the file already exists
+2. If it exists → read_file, then patch_file with targeted changes
+3. If it does NOT exist → write_file with COMPLETE content
+Never call write_file on a file that already exists — the shrink guard will block you and waste a tool call.
+
+### Sibling Awareness
+Your context may include files modified by sibling tasks. Before writing a file:
+- Check "Previous Task Completions" → if a sibling already created it, READ it first, then patch_file
+- Never blindly overwrite files your siblings created
+
 ### Code Quality
 - Write COMPLETE, WORKING code. No TODOs, no placeholders, no empty function bodies, no "// implement later".
 - Every function must be fully implemented with real logic.
@@ -156,7 +193,14 @@ const CAP_FILE_OPS: CapabilityDefinition = {
 PREFER patch_file over write_file for modifying existing files (surgical edits, no truncation risk).
 Use read_file to understand a file before modifying it.
 Use list_files to discover project structure.
-Writes that shrink a file by >50% or write empty content are automatically BLOCKED.`,
+
+## Write Protocol
+Before writing ANY file:
+1. Use list_files to check if the file already exists
+2. If it EXISTS → read_file first, then patch_file with targeted changes
+3. If it does NOT exist → write_file with COMPLETE content
+NEVER call write_file on a file that already exists — use patch_file instead.
+Empty content and writes that shrink an existing file by >50% are automatically BLOCKED and waste a tool call.`,
 };
 
 const CAP_SHELL: CapabilityDefinition = {
@@ -285,8 +329,9 @@ const CAP_CODE_REVIEW: CapabilityDefinition = {
   tools: [OP_READ_FILE, OP_PATCH_FILE, OP_RUN_SHELL],
   prompt: `## Code Review Checklist
 
-### 1. Correctness
+### 1. Correctness & Contract Compliance
 - Does the code do what the task asked?
+- If .design.md exists, verify: exported functions match contracts, interface shapes match, error behavior matches spec
 - Edge cases handled (empty input, null, invalid types)?
 - Error paths covered (try/catch, validation)?
 - Return types match function signature?
@@ -298,16 +343,19 @@ const CAP_CODE_REVIEW: CapabilityDefinition = {
 - User input validated and sanitized before use
 - File paths validated (no ../ traversal)
 
-### 3. Style
+### 3. Style & Dead Code
 - Naming is clear and consistent with project conventions
-- No dead code (unused variables, unreachable branches)
+- No dead code (unused variables, unreachable branches, duplicated functions across files)
+- No duplicated logic — if two files define the same function, flag it
 - No debug statements left in (console.log, debugger)
-- Imports organized, no duplicates
+- Imports organized, no duplicates, no unused imports
 
 ### 4. Testing
 - Unit tests exist for new/changed functions
 - Tests cover happy path AND edge cases
 - Error cases have tests
+- Test isolation: state reset between tests, server handles returned and closed in afterEach
+- No order-dependent tests (each test must pass in isolation)
 - Code coverage adequate (aim for 80%+ of changed code)
 
 ### 5. Performance
@@ -328,11 +376,18 @@ const CAP_WEB: CapabilityDefinition = {
 
 const CAP_CONTEXT: CapabilityDefinition = {
   name: 'context',
-  description: 'Project file listings, directory structure, and workspace context.',
+  description: 'Project file listings, directory structure, workspace context, and sibling task awareness.',
   prompt: `## Project Context
 
 Use list_files to understand the project structure before making changes.
-The context bundle (when available) provides a snapshot of the workspace.`,
+The context bundle (when available) provides a snapshot of the workspace.
+
+## Sibling Awareness
+Your context includes files modified by sibling tasks (in "Previous Task Completions").
+Before writing a file:
+- Check if it appears in previous task completions → if yes, read_file first, then patch_file
+- NEVER blindly overwrite files your siblings created
+- If you need to extend a sibling's work, READ their output first and build on it`,
 };
 
 // ---------------------------------------------------------------------------

package/src/bot/profile-store.ts

@@ -251,17 +251,23 @@ const DEFAULT_PROFILES: Record<string, Array<Omit<CreateProfileInput, 'botId'>>>
 
 ## Protocol
 1. ANALYZE: Understand the objective. What files, features, and concerns are involved?
-2. DECOMPOSE: Break into subtasks. Each subtask should be completable by a single bot.
+2. DECOMPOSE: Break into subtasks. Each subtask should produce 1-2 files max. Split large tasks.
 3. ASSIGN: Set assignedProfile per task (developer, reviewer, ops). Set complexity and priority.
-4. DEPENDENCIES: Use dependsOn to model blocking relationships. Minimize deps for parallelism.
+4. DEPENDENCIES: Minimize deps for parallelism. Tasks that don't share files should NOT depend on each other.
+5. VERIFY: Your LAST subtask must be a "Verify & Iterate" task assigned to "orchestrator" that reviews results.
 
 ## Rules
 - You do NOT write code yourself. You create tasks for other bots.
-- CRITICAL: Every subtask MUST have assignedProfile set to "developer", "reviewer", or "ops". NEVER leave it empty or set it to "orchestrator".
-- Every subtask needs: title, description, assignedProfile, complexity.
-- Code/file tasks → assignedProfile: "developer". Setup/infra → assignedProfile: "ops". Review → assignedProfile: "reviewer".
-- Structure as: setup → independent implementations → integration → review.
-- If a task is simple enough (single file, clear scope), don't decompose further.`,
+- Every work subtask MUST have assignedProfile: "developer", "reviewer", or "ops".
+- The ONLY exception: your final "Verify & Iterate" task uses assignedProfile: "orchestrator".
+- Add scope boundaries: "You may ONLY create/modify these files: [list]."
+- Design/architecture tasks → "developer" (not ops). Ops is for infra/config only.
+- Aim for at least 2 tasks that can run in parallel. If all tasks are serial, reconsider.
+
+## Verify & Iterate Loop
+Your LAST subtask MUST be:
+{ title: "Verify & Iterate", assignedProfile: "orchestrator", dependsOn: [all other tasks] }
+When this runs: review results, check for failures/gaps, create fix tasks if needed, or signal done.`,
         behavior: {
           capabilities: PROFILE_CAPABILITIES.orchestrator,
           phases: {

package/src/bot/system-prompt.ts

@@ -208,6 +208,8 @@ USE TOOLS to complete tasks. Do NOT describe what you would do — actually do i
 
 ### Knowledge tools
 - learn(key, value), recall(query)
+- When you discover project facts (test runner, build tool, conventions, common patterns), use learn() to persist them for future runs
+- Previous knowledge is auto-loaded into your context — avoid re-discovering what's already known
 
 ### Web & interaction tools
 - web_fetch(url), ask_user(question)

package/src/bot/tool-registry.ts

@@ -294,7 +294,7 @@ export const ALL_TOOLS: WeaverTool[] = [
   },
   {
     name: 'write_file',
-    description: 'Write content to a file (creates or overwrites). Use patch_file instead for modifications to existing files.',
+    description: 'Write content to a NEW file. ONLY for files that do not exist yet — use patch_file for existing files. Empty content and writes that shrink an existing file by >50% are automatically BLOCKED.',
     inputSchema: {
       type: 'object',
       properties: {

package/src/bot/weaver-tools.ts

@@ -185,6 +185,13 @@ export function createWeaverExecutor(projectDir: string) {
         break;
     }
 
+    // Ignore Claude Code built-in tools that leak through when disallowed list is incomplete.
+    // These are no-ops — the bot should use the MCP bridge tools instead.
+    const IGNORED_BUILTINS = new Set(['TodoWrite', 'TodoRead', 'Agent', 'WebFetch', 'WebSearch', 'MultiTool', 'Skill']);
+    if (IGNORED_BUILTINS.has(name) || IGNORED_BUILTINS.has(rawName)) {
+      return { result: `Tool "${name}" is not available. Use the provided tools instead.`, isError: true };
+    }
+
     // Existing step-executor-based tools
     const operation = OPERATION_MAP[name];
     if (!operation) {

package/src/node-types/build-context.ts

@@ -45,6 +45,17 @@ export function weaverBuildContext(ctx: string): { ctx: string } {
     }
   } catch { /* non-fatal — memory is best-effort */ }
 
+  // Auto-recall learned knowledge from previous bot runs
+  try {
+    const { KnowledgeStore } = require('../bot/knowledge-store.js') as typeof import('../bot/knowledge-store.js');
+    const knowledge = new KnowledgeStore(projectDir);
+    const entries = knowledge.list();
+    if (entries.length > 0) {
+      const knowledgeLines = entries.map((e: { key: string; value: string }) => `- **${e.key}**: ${e.value}`);
+      sections.push(`## Learned Knowledge\n\nFacts discovered by previous runs — use these instead of re-discovering:\n${knowledgeLines.join('\n')}`);
+    }
+  } catch { /* non-fatal — knowledge recall is best-effort */ }
+
   // Include sibling context from hierarchy events (previous task completions in same hierarchy)
   try {
     const parsedTask = JSON.parse(context.taskJson!) as { parentId?: string };

package/src/ui/chat-task-result.tsx

@@ -18,16 +18,18 @@ const { Flex, Typography, StatusIcon, Button } = require('@fw/plugin-ui-kit');
 interface TaskData {
   id: string;
   title: string;
-  status: 'pending' | 'in-progress' | 'blocked' | 'done' | 'failed' | 'cancelled';
+  status: 'open' | 'pending' | 'in-progress' | 'blocked' | 'done' | 'cancelled';
   isParent: boolean;
   currentBotId?: string;
   assignedProfile?: string;
+  attempt?: number;
+  maxAttempts?: number;
 }
 
 interface SubtaskData {
   id: string;
   title: string;
-  status: 'pending' | 'in-progress' | 'blocked' | 'done' | 'failed' | 'cancelled';
+  status: 'open' | 'pending' | 'in-progress' | 'blocked' | 'done' | 'cancelled';
 }
 
 interface ChatTaskResultProps {
@@ -59,6 +61,7 @@ type StatusKind = 'running' | 'completed' | 'failed' | 'pending';
 
 function statusToIcon(status: TaskData['status']): StatusKind {
   switch (status) {
+    case 'open':
     case 'pending':
     case 'blocked':
       return 'pending';
@@ -66,7 +69,6 @@ function statusToIcon(status: TaskData['status']): StatusKind {
       return 'running';
     case 'done':
       return 'completed';
-    case 'failed':
     case 'cancelled':
       return 'failed';
     default:
@@ -76,18 +78,18 @@ function statusToIcon(status: TaskData['status']): StatusKind {
 
 function statusLabel(status: TaskData['status']): string {
   switch (status) {
-    case 'pending': return 'Pending';
-    case 'in-progress': return 'In Progress';
+    case 'open': return 'Open';
+    case 'pending': return 'Queued';
+    case 'in-progress': return 'Running';
     case 'blocked': return 'Blocked';
     case 'done': return 'Done';
-    case 'failed': return 'Failed';
     case 'cancelled': return 'Cancelled';
     default: return status;
   }
 }
 
 function isTerminal(status: TaskData['status']): boolean {
-  return status === 'done' || status === 'failed' || status === 'cancelled';
+  return status === 'done' || status === 'cancelled';
 }
 
 // ---------------------------------------------------------------------------
@@ -204,7 +206,7 @@ function ChatTaskResult(props: ChatTaskResultProps | null) {
         React.createElement(Typography, {
           variant: 'smallCaption-regular',
           color: task.status === 'done' ? 'color-text-positive' :
-                 task.status === 'failed' ? 'color-text-negative' :
+                 task.status === 'cancelled' ? 'color-text-negative' :
                  task.status === 'in-progress' ? 'color-text-info' :
                  'color-text-medium',
         }, statusLabel(task.status)),

package/src/ui/swarm-dashboard.tsx

@@ -77,7 +77,7 @@ interface SwarmStatus {
   packVersion?: string;
 }
 
-type TaskStatus = 'pending' | 'in-progress' | 'blocked' | 'done' | 'failed' | 'cancelled';
+type TaskStatus = 'open' | 'pending' | 'in-progress' | 'blocked' | 'done' | 'cancelled';
 
 interface PoolTask {
   id: string;

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',
 };
@@ -568,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',
@@ -583,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
@@ -696,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' || (task.status === 'open' && task.attempt >= task.maxAttempts)) && 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
@@ -470,6 +470,7 @@ class GeneratedExecutionContext {
  * @connect abort.ctx -> report.abortCtx
  * @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,7 +2117,7 @@ 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 = 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);
@@ -2128,7 +2129,6 @@ export async function weaverBot(
         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);
-        await ctx.setVariable({ id: 'report', portName: 'report', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, reportResult.report);
         await ctx.setVariable({ id: 'report', portName: 'reportJson', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, reportResult.reportJson);
         await ctx.setVariable({ id: 'report', portName: 'onFailure', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, reportResult.onFailure);
         await ctx.setVariable({ id: 'report', portName: 'onSuccess', executionIndex: reportIdx, nodeTypeName: 'weaverBotReport' }, reportResult.onSuccess);