iriai-build 0.4.2 → 0.4.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "iriai-build",
-  "version": "0.4.2",
+  "version": "0.4.4",
   "description": "Iriai Build tool — AI agent orchestration CLI",
   "type": "module",
   "bin": {

package/v3/orchestrator.js

@@ -2132,12 +2132,12 @@ export class Orchestrator {
         }
 
         // Handle [DECISION] blocks — render as separate decision prompts.
-        // Skip any that duplicate a deferred decision (already queued by _requestPhaseReview).
+        // Skip ALL Operator-emitted decisions when a deferred decision exists for this feature.
+        // The orchestrator already manages phase review / plan-approval decisions via
+        // _requestPhaseReview → _postDeferredDecision. Operator [DECISION] blocks would
+        // duplicate those, even if the IDs don't match exactly.
         for (const decision of parsed.decisions) {
-          if (this._deferredDecisions[feature.id]) {
-            const deferred = this._deferredDecisions[feature.id].decision;
-            if (deferred.id === decision.id) continue;
-          }
+          if (this._deferredDecisions[feature.id]) continue;
           const options = decision.options.map(o => ({
             id: o.id,
             label: o.label,
@@ -2649,11 +2649,16 @@ export class Orchestrator {
       return;
     }
 
+    // Re-read feature from DB — the file watcher's _handlePlanningDone may have
+    // updated phase/role between the time the exit event was queued and now.
+    const freshFeature = queries.getFeatureById(feature.id);
+    if (!freshFeature) return;
+
     // Check if the file watcher already processed .done (it deletes the file).
     // If review gate is posted or agent already marked done, don't retry.
     const meta = queries.getFeatureMetadata(feature.id);
     const agent = queries.getAgentById(agentId);
-    if (meta.awaiting_phase_review && meta.phase_review_role === roleName) {
+    if (meta.awaiting_phase_review) {
       console.log(`[orchestrator] ${roleName} exit: review gate already posted, skipping retry`);
       queries.updateAgentStatus(agentId, "done");
       return;
@@ -2662,9 +2667,15 @@ export class Orchestrator {
       console.log(`[orchestrator] ${roleName} exit: already marked done, skipping retry`);
       return;
     }
-    // Also skip if phase has already advanced past this role
-    if (feature.active_planning_role && feature.active_planning_role !== roleName) {
-      console.log(`[orchestrator] ${roleName} exit: phase already advanced to ${feature.active_planning_role}, skipping retry`);
+    // Skip if feature has left planning phase (e.g. plan-compiler PASS → plan-approval)
+    if (freshFeature.phase !== "planning") {
+      console.log(`[orchestrator] ${roleName} exit: feature phase is ${freshFeature.phase}, not planning — skipping retry`);
+      queries.updateAgentStatus(agentId, "done");
+      return;
+    }
+    // Skip if phase has already advanced past this role (null means role was cleared)
+    if (freshFeature.active_planning_role !== roleName) {
+      console.log(`[orchestrator] ${roleName} exit: active role is now ${freshFeature.active_planning_role || "none"}, skipping retry`);
       queries.updateAgentStatus(agentId, "done");
       return;
     }

package/v3/recovery.js

@@ -100,8 +100,10 @@ export class Recovery {
       };
     }
 
-    // Emit stale signals
+    // Emit stale signals — only for the active planning role
+    const activeRole = feature.active_planning_role;
     for (const [role, dir] of Object.entries(planningTree)) {
+      if (role !== activeRole) continue;
       for (const [sig, eventType] of [
         [SIGNAL.DONE, "planning:done"],
         [SIGNAL.AGENT_RESPONSE, "planning:response"],
@@ -281,8 +283,12 @@ export class Recovery {
         };
       }
 
-      // Emit stale signals
+      // Emit stale signals — only for the active planning role to avoid
+      // re-processing signals from already-completed roles (which would cause
+      // duplicate artifact uploads and decision re-posts).
+      const activeRole = feature.active_planning_role;
       for (const [role, dir] of Object.entries(planningTree)) {
+        if (role !== activeRole) continue;
         for (const [sig, eventType] of [
           [SIGNAL.DONE, "planning:done"],
           [SIGNAL.AGENT_RESPONSE, "planning:response"],