@yemi33/minions 0.1.631 → 0.1.632

package/CHANGELOG.md

@@ -1,6 +1,9 @@
 # Changelog
 
-## 0.1.631 (2026-04-08)
+## 0.1.632 (2026-04-08)
+
+### Features
+-  store _prdFilename on work item for deterministic completion check
 
 ### Fixes
 -  PRD existence check scans by source_plan, not filename

package/engine/lifecycle.js

@@ -1349,18 +1349,23 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
 
   // Detect plan-to-prd tasks that completed without creating a PRD file
   if (effectiveSuccess && type === WORK_TYPE.PLAN_TO_PRD && meta?.item?.planFile) {
-    // PRD filename may differ from plan filename — scan prd/ for any file referencing this plan
-    const planFile = meta.item.planFile;
+    // Check by stored filename first (deterministic), fall back to source_plan scan
     let prdFound = false;
-    try {
-      for (const f of fs.readdirSync(PRD_DIR)) {
-        if (!f.endsWith('.json')) continue;
-        try {
-          const prd = safeJson(path.join(PRD_DIR, f));
-          if (prd && prd.source_plan === planFile) { prdFound = true; break; }
-        } catch {}
-      }
-    } catch {}
+    const expectedFile = meta.item._prdFilename;
+    if (expectedFile) {
+      prdFound = fs.existsSync(path.join(PRD_DIR, expectedFile));
+    }
+    if (!prdFound) {
+      try {
+        for (const f of fs.readdirSync(PRD_DIR)) {
+          if (!f.endsWith('.json')) continue;
+          try {
+            const prd = safeJson(path.join(PRD_DIR, f));
+            if (prd && prd.source_plan === meta.item.planFile) { prdFound = true; break; }
+          } catch {}
+        }
+      } catch {}
+    }
     if (!prdFound) {
       const hasOutput = stdout && stdout.length > 500;
       const wiPath = resolveWorkItemPath(meta);

package/engine.js

@@ -2305,6 +2305,7 @@ function discoverCentralWorkItems(config) {
         let prdCounter = 2;
         while (prdExisting.has(prdFilename)) { prdFilename = prdBase + '-' + prdCounter + '.json'; prdCounter++; }
         vars.prd_filename = prdFilename;
+        mutations.set(item.id, Object.assign(mutations.get(item.id) || {}, { _prdFilename: prdFilename }));
         vars.branch_strategy_hint = item.branchStrategy
           ? `The user requested **${item.branchStrategy}** strategy. Use this unless the analysis strongly suggests otherwise.`
           : 'Choose the best strategy based on your analysis of item dependencies.';

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.631",
+  "version": "0.1.632",
   "description": "Multi-agent AI dev team that runs from ~/.minions/ — five autonomous agents share a single engine, dashboard, and knowledge base",
   "bin": {
     "minions": "bin/minions.js"