opencode-orchestrator 1.0.31 → 1.0.34

package/README.md

@@ -66,38 +66,42 @@ MSVP is a structured verification process that decouples implementation from qua
               [User Task Input]
                      │
            ┌─────────▼─────────┐
-           │     COMMANDER     │ (Orchestration context)
+           │     COMMANDER     │  (Orchestration & Discovery)
            └─────────┬─────────┘
                      │
            ┌─────────▼─────────┐
-           │      PLANNER      │ (Create Hierarchical Plan)
+           │      PLANNER      │  (Create TODO.md)
            └─────────┬─────────┘
                      │
     ┌────────────────▼────────────────┐
-    │  Phase 1: Parallel Execution    │ (HPFA Implementation Grid)
+    │   COMMANDER: Parallel Workers   │
     └──────┬─────────┬─────────┬──────┘
            │         │         │
     ┌──────▼──┐ ┌────▼───┐ ┌───▼────┐
-    │ WORKER  │ │ WORKER │ │ WORKER │ (Fractal Spawning)
-    └──────┬──┘ └────┬───┘ └───┬────┘
-           │         │         │
-    ┌──────▼──┐ ┌────▼───┐ ┌───▼────┐
-    │ REVIEWER│ │ REVIEWER│ │ REVIEWER│ (Stage 1: Unit Verification)
-    └──────┬──┘ └────┬───┘ └───┬────┘
-           │         │         │
-    ───────▼─────────▼─────────▼───────
-    │          Sync Barrier           │ (Wait for all units)
-    ─────────────────┬─────────────────
-                     │
-           ┌─────────▼─────────┐
-           │  MASTER REVIEWER  │ (Stage 2: e2e Verification)
-           └─────────┬─────────┘
-                     │
-           ┌─────────▼─────────┐
-           │  Mission Sealed?  │
-           └─────────┬─────────┘
-                No ↙   ↘ Yes
-             [Loop]   [Complete]
+    │ WORKER A│ │ WORKER B│ │ WORKER C│  (Implementation)
+    └──────┬──┘ └────┬───┘ └────┬────┘
+           │         │          │
+    ╔══════▼═════════▼══════════▼══════╗
+    ║   COMMANDER: Parallel Reviewers  ║
+    ╚══════╤═════════╤══════════╤══════╝
+           │         │          │
+    ┌──────▼──┐ ┌────▼───┐ ┌────▼────┐
+    │REVIEWER │ │REVIEWER │ │REVIEWER │  (Unit Verification)
+    └──────┬──┘ └────┬───┘ └────┬────┘
+           │         │          │
+    ═══════▼═════════▼══════════▼══════
+    │          SYNC BARRIER           │
+    ════════════════╤══════════════════
+                    │
+          ┌─────────▼─────────┐
+          │  MASTER REVIEWER  │  (E2E Verification)
+          └─────────┬─────────┘
+                    │
+          ┌─────────▼─────────┐
+          │  Mission Sealed?  │
+          └─────────┬─────────┘
+               No ↙   ↘ Yes
+            [Loop]   [Complete]
 ```
 
 ---

package/dist/agents/prompts/planner/forbidden.d.ts

@@ -2,5 +2,6 @@
  * Planner Forbidden Actions
  *
  * Stay in your lane - research and plan, never implement.
+ * Planner is a TERMINAL node and cannot spawn other agents.
  */
 export declare const PLANNER_FORBIDDEN: string;

package/dist/index.js

@@ -334,8 +334,24 @@ var PARALLEL_TASK = {
   // Task lifecycle (24 hours for long tasks)
   TTL_MS: 24 * TIME.HOUR,
   CLEANUP_DELAY_MS: 10 * TIME.MINUTE,
-  MAX_DEPTH: 8,
-  // Titan-Class depth for complex fractal missions
+  /**
+   * Maximum recursion depth for parallel task spawning
+   * 
+   * Depth hierarchy:
+   * - Depth 0: Commander (can spawn Planner/Worker/Reviewer)
+   * - Depth 1: Planner (can spawn Worker)
+   * - Depth 2: Worker/Reviewer (TERMINAL - cannot spawn sub-agents)
+   * - Depth 3: BLOCKED
+   * 
+   * This prevents infinite recursion and ensures Master Reviewer
+   * doesn't wait indefinitely for deeply nested sub-tasks.
+   */
+  MAX_DEPTH: 3,
+  /**
+   * Terminal node depth - agents at this depth cannot spawn sub-agents.
+   * Worker and Reviewer are terminal nodes by design.
+   */
+  TERMINAL_DEPTH: 2,
   // Concurrency limits (Aggressive for intense processing)
   DEFAULT_CONCURRENCY: 10,
   MAX_CONCURRENCY: 50,
@@ -935,15 +951,16 @@ var PHASE_1_THINK_ANALYSIS = `### 1.1 ANALYZE & SCOPE (INPUT)
 ### 1.4 RISK ASSESSMENT
 - Identify high-risk parts and fallback plans.
 - If agent fails \u2192 See RECOVERY section.`;
-var PHASE_5_MSVP = `1. **STAGE 1 (Unit)**: Workers MUST trigger ${AGENT_NAMES.REVIEWER} for unit verification immediately upon completion.
-2. **STAGE 2 (Integration Master)**: The Master ${AGENT_NAMES.REVIEWER} directly reads all modified files and validates cross-module consistency.
-3. **E2E VALIDATION**: Run full system tests and build pass check.
+var PHASE_5_MSVP = `1. **STAGE 1 (Unit)**: As Workers complete, Commander spawns ${AGENT_NAMES.REVIEWER} for each to verify.
+2. **STAGE 2 (Integration Master)**: Once all Units pass, Commander spawns Master ${AGENT_NAMES.REVIEWER} for cross-module validation.
+3. **E2E VALIDATION**: Master Reviewer runs full system tests and build pass check.
 4. **SEAL GATE**: No SEALED output until ${PATHS.TODO} is all [x] and zero issues remain.`;
-var HPFA_RULES = `1. **Fractal Spawning**: Workers spawn sub-workers for complex files/modules.
-2. **Speculative Racing**: Launch multiple strategies in parallel (mode: race) for uncertainty.
-3. **Asynchronous Batching**: Group non-dependent tool calls to trigger host-side parallelism.
-4. **Barrier-Sync Pipeline**: Reviewers start Unit Review (Stage 1) while other workers still run.
-5. **Real-time Brain Sync**: Parallel sessions share public interfaces via shared task logs.`;
+var HPFA_RULES = `1. **Commander Parallel Workers**: Commander spawns multiple Workers in parallel for independent modules.
+2. **Commander Parallel Reviewers**: As Workers complete, Commander spawns Unit Reviewers for each.
+3. **Speculative Racing**: Launch multiple strategies in parallel (mode: race) for uncertainty.
+4. **Asynchronous Batching**: Group non-dependent tool calls to trigger host-side parallelism.
+5. **Barrier-Sync Pipeline**: Commander waits at sync barrier before launching Master Reviewer.
+6. **Terminal Nodes**: Workers and Reviewers are TERMINAL - they cannot spawn any sub-agents.`;
 
 // src/shared/prompt/constants/scouts.ts
 var SCOUT_LABEL = "Scout";
@@ -14278,12 +14295,15 @@ ${PROMPT_TAGS.ROLE.close}`;
 var PLANNER_FORBIDDEN = `${PROMPT_TAGS.FORBIDDEN_ACTIONS.open}
 **PLANNER FORBIDDEN ACTIONS**
 
-
+## \u26D4 NEVER Spawn or Delegate (CRITICAL)
+- NEVER use \`${TOOL_NAMES.DELEGATE_TASK}\` to spawn agents
+- NEVER use \`${TOOL_NAMES.CALL_AGENT}\` to create sessions
+- You are a TERMINAL node - create plans, don't execute them
+- ${AGENT_NAMES.COMMANDER} is the ONLY agent who can spawn Workers/Reviewers
 
 ## Never Implement
 - NEVER write actual code \u2192 Only plan and research
 - NEVER execute build/test commands \u2192 That's ${AGENT_NAMES.WORKER}/${AGENT_NAMES.REVIEWER}'s job
-
 - NEVER modify source files \u2192 Only ${PATHS.TODO} and ${PATHS.DOCS}/
 
 ## Never Assume
@@ -14585,7 +14605,12 @@ ${PROMPT_TAGS.ROLE.close}`;
 var WORKER_FORBIDDEN = `${PROMPT_TAGS.FORBIDDEN_ACTIONS.open}
 **FORBIDDEN ACTIONS (Adapt to Project Conventions)**
 
-
+## \u26D4 NEVER Spawn or Delegate (CRITICAL)
+- NEVER use \`${TOOL_NAMES.DELEGATE_TASK}\` to spawn sub-workers
+- NEVER use \`${TOOL_NAMES.CALL_AGENT}\` to create additional sessions
+- You are a TERMINAL node - complete your assigned file directly
+- If task is too complex, REPORT BACK to ${AGENT_NAMES.COMMANDER} with specific blockers
+- Violating this rule causes infinite recursion and system failure
 
 ## Never Assume
 - NEVER guess API syntax \u2192 CHECK ${PATHS.DOCS}/ or research first
@@ -14595,7 +14620,6 @@ var WORKER_FORBIDDEN = `${PROMPT_TAGS.FORBIDDEN_ACTIONS.open}
 ## Never Skip
 - NEVER skip error handling \u2192 Follow project's error handling patterns
 - NEVER skip ${TOOL_NAMES.LSP_DIAGNOSTICS} \u2192 Always verify code compiles
-
 - NEVER skip verification \u2192 Test before claiming done
 
 ## Never Shortcut
@@ -15009,7 +15033,12 @@ ${PROMPT_TAGS.ROLE.close}`;
 var REVIEWER_FORBIDDEN = `${PROMPT_TAGS.FORBIDDEN_ACTIONS.open}
 **FORBIDDEN ACTIONS**
 
-
+## \u26D4 NEVER Spawn or Delegate (CRITICAL)
+- NEVER use \`${TOOL_NAMES.DELEGATE_TASK}\` to spawn additional reviewers
+- NEVER use \`${TOOL_NAMES.CALL_AGENT}\` to create sub-sessions
+- You are a TERMINAL node - verify your assigned file directly
+- If verification scope is too large, REPORT BACK to ${AGENT_NAMES.COMMANDER}
+- Violating this rule causes infinite recursion and blocks Master Review
 
 ## Never Approve Without Verification
 - NEVER approve without actually running the project's test command
@@ -15026,7 +15055,6 @@ var REVIEWER_FORBIDDEN = `${PROMPT_TAGS.FORBIDDEN_ACTIONS.open}
 - NEVER make architecture changes \u2192 Escalate to ${AGENT_NAMES.COMMANDER}
 - NEVER implement fixes yourself \u2192 Send back to ${AGENT_NAMES.WORKER} with clear feedback
 
-
 ## Adaptive Verification
 - READ ${PATHS.CONTEXT} to know the correct build/test commands
 - COMPARE with existing code patterns for consistency
@@ -17997,6 +18025,17 @@ var createDelegateTaskTool = (manager, client) => tool({
     const parentTask = allTasks.find((t) => t.sessionID === ctx.sessionID);
     const parentDepth = parentTask?.depth ?? 0;
     log(`${PARALLEL_LOG.DELEGATE_TASK} execute() called`, { agent, description, background, resume, mode, groupID, parentSession: ctx.sessionID, depth: parentDepth });
+    if (parentDepth >= PARALLEL_TASK.TERMINAL_DEPTH) {
+      log(`${PARALLEL_LOG.DELEGATE_TASK} Terminal node guard triggered`, { parentDepth, TERMINAL_DEPTH: PARALLEL_TASK.TERMINAL_DEPTH });
+      return `${OUTPUT_LABEL.ERROR} Delegation blocked: You are a terminal node (depth ${parentDepth}).
+
+**${AGENT_NAMES.WORKER} and ${AGENT_NAMES.REVIEWER} cannot spawn sub-agents.** This prevents infinite recursion.
+
+If your task is too complex, please:
+1. Report back to ${AGENT_NAMES.COMMANDER} with specific blockers
+2. Request task decomposition at the ${AGENT_NAMES.PLANNER} level
+3. Complete your assigned file directly without delegation`;
+    }
     const sessionClient = client;
     if (background === void 0) {
       return `${OUTPUT_LABEL.ERROR} 'background' parameter is REQUIRED.`;

package/dist/shared/prompt/constants/mandates.d.ts

@@ -17,9 +17,15 @@ export declare const PHASE_0_DIRECT_DISCOVERY: string;
 export declare const PHASE_1_THINK_ANALYSIS: string;
 /**
  * Phase 5: MSVP (Multi-Stage Verification Pipeline)
+ *
+ * NOTE: Commander spawns both Workers AND Reviewers.
+ * Workers are terminal nodes and do not spawn Reviewers themselves.
  */
 export declare const PHASE_5_MSVP: string;
 /**
  * HPFA (Hyper-Parallel Fractal Architecture) Rules
+ *
+ * All spawning happens at Commander level only.
+ * Workers and Reviewers are terminal nodes and cannot spawn sub-agents.
  */
-export declare const HPFA_RULES = "1. **Fractal Spawning**: Workers spawn sub-workers for complex files/modules.\n2. **Speculative Racing**: Launch multiple strategies in parallel (mode: race) for uncertainty.\n3. **Asynchronous Batching**: Group non-dependent tool calls to trigger host-side parallelism.\n4. **Barrier-Sync Pipeline**: Reviewers start Unit Review (Stage 1) while other workers still run.\n5. **Real-time Brain Sync**: Parallel sessions share public interfaces via shared task logs.";
+export declare const HPFA_RULES = "1. **Commander Parallel Workers**: Commander spawns multiple Workers in parallel for independent modules.\n2. **Commander Parallel Reviewers**: As Workers complete, Commander spawns Unit Reviewers for each.\n3. **Speculative Racing**: Launch multiple strategies in parallel (mode: race) for uncertainty.\n4. **Asynchronous Batching**: Group non-dependent tool calls to trigger host-side parallelism.\n5. **Barrier-Sync Pipeline**: Commander waits at sync barrier before launching Master Reviewer.\n6. **Terminal Nodes**: Workers and Reviewers are TERMINAL - they cannot spawn any sub-agents.";

package/dist/shared/task/constants/parallel-task.d.ts

@@ -9,7 +9,24 @@ export declare const TASK_MODE: {
 export declare const PARALLEL_TASK: {
     readonly TTL_MS: number;
     readonly CLEANUP_DELAY_MS: number;
-    readonly MAX_DEPTH: 8;
+    /**
+     * Maximum recursion depth for parallel task spawning
+     *
+     * Depth hierarchy:
+     * - Depth 0: Commander (can spawn Planner/Worker/Reviewer)
+     * - Depth 1: Planner (can spawn Worker)
+     * - Depth 2: Worker/Reviewer (TERMINAL - cannot spawn sub-agents)
+     * - Depth 3: BLOCKED
+     *
+     * This prevents infinite recursion and ensures Master Reviewer
+     * doesn't wait indefinitely for deeply nested sub-tasks.
+     */
+    readonly MAX_DEPTH: 3;
+    /**
+     * Terminal node depth - agents at this depth cannot spawn sub-agents.
+     * Worker and Reviewer are terminal nodes by design.
+     */
+    readonly TERMINAL_DEPTH: 2;
     readonly DEFAULT_CONCURRENCY: 10;
     readonly MAX_CONCURRENCY: 50;
     readonly SYNC_TIMEOUT_MS: number;

package/package.json

@@ -2,7 +2,7 @@
   "name": "opencode-orchestrator",
   "displayName": "OpenCode Orchestrator",
   "description": "Distributed Cognitive Architecture for OpenCode. Turns simple prompts into specialized multi-agent workflows (Planner, Coder, Reviewer).",
-  "version": "1.0.31",
+  "version": "1.0.34",
   "author": "agnusdei1207",
   "license": "MIT",
   "repository": {