@yemi33/minions 0.1.1572 → 0.1.1574

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.1.1574 (2026-04-27)
+
+### Fixes
+- oneShot flag prevents one-off reviews from eval loop (#1776)
+
+## 0.1.1573 (2026-04-27)
+
+### Fixes
+- default existing_prd_json to empty so plan-to-prd dispatch doesn't warn
+
 ## 0.1.1572 (2026-04-27)
 
 ### Fixes

package/dashboard.js

@@ -912,6 +912,10 @@ async function executeCCActions(actions) {
           const project = action.project || '';
           const targetProject = project ? PROJECTS.find(p => p.name?.toLowerCase() === project.toLowerCase()) : PROJECTS[0];
           const wiPath = targetProject ? shared.projectWorkItemsPath(targetProject) : path.join(MINIONS_DIR, 'work-items.json');
+          // Issue #1772: CC review/explore/test are human-initiated one-offs.
+          // Mark oneShot so any discovered PR is tagged _contextOnly (skips eval loop).
+          const ccOneShotTypes = new Set(['review', 'explore', 'test']);
+          const isOneShot = action.oneShot === true || (action.oneShot !== false && ccOneShotTypes.has(workType));
           shared.mutateJsonFileLocked(wiPath, items => {
             if (!Array.isArray(items)) items = [];
             items.push({
@@ -920,6 +924,7 @@ async function executeCCActions(actions) {
               status: WI_STATUS.PENDING, created: new Date().toISOString(),
               createdBy: 'command-center', project,
               ...(action.agents?.length ? { preferred_agent: action.agents[0] } : {}),
+              ...(isOneShot ? { oneShot: true } : {}),
             });
             return items;
           }, { defaultValue: [] });
@@ -1978,6 +1983,7 @@ const server = http.createServer(async (req, res) => {
       if (body.references) item.references = body.references;
       if (body.acceptanceCriteria) item.acceptanceCriteria = body.acceptanceCriteria;
       if (body.skipPr) item.skipPr = true;
+      if (body.oneShot) item.oneShot = true;
       let dupId = null;
       mutateJsonFileLocked(wiPath, (items) => {
         if (!Array.isArray(items)) items = [];

package/engine/lifecycle.js

@@ -800,23 +800,25 @@ function syncPrsFromOutput(output, agentId, meta, config) {
     }
 
     if (!newPrsByPath.has(prPath)) newPrsByPath.set(prPath, { name: targetName, project: targetProject, entries: [] });
-    newPrsByPath.get(prPath).entries.push({
-      prId, fullId,
-      entry: {
-        id: fullId,
-        prNumber: parseInt(prId, 10) || prId,
-        title: (title || `PR created by ${agentName}`).slice(0, 120),
-        agent: agentName,
-        branch: meta?.branch || '',
-        reviewStatus: 'pending',
-        status: PR_STATUS.ACTIVE,
-        created: ts(),
-        url: prUrl,
-        prdItems: meta?.item?.id ? [meta.item.id] : [],
-        sourcePlan: meta?.item?.sourcePlan || '',
-        itemType: meta?.item?.itemType || ''
-      }
-    });
+    const entry = {
+      id: fullId,
+      prNumber: parseInt(prId, 10) || prId,
+      title: (title || `PR created by ${agentName}`).slice(0, 120),
+      agent: agentName,
+      branch: meta?.branch || '',
+      reviewStatus: 'pending',
+      status: PR_STATUS.ACTIVE,
+      created: ts(),
+      url: prUrl,
+      prdItems: meta?.item?.id ? [meta.item.id] : [],
+      sourcePlan: meta?.item?.sourcePlan || '',
+      itemType: meta?.item?.itemType || ''
+    };
+    // Issue #1772: one-off dispatches (e.g. human-initiated "review this PR" via CC)
+    // must not enroll the discovered PR into the auto eval loop. Tag _contextOnly so
+    // discoverFromPrs skips it for review/fix dispatch (still polled for status/comments).
+    if (meta?.item?.oneShot) entry._contextOnly = true;
+    newPrsByPath.get(prPath).entries.push({ prId, fullId, entry });
   }
 
   const entryBranch = meta?.branch || '';

package/engine.js

@@ -2982,6 +2982,11 @@ function discoverCentralWorkItems(config) {
         vars.plan_summary = (item.title || item.planFile).substring(0, 80);
         vars.plan_file = item.planFile || '';
         vars.project_name_lower = (firstProject?.name || 'project').toLowerCase();
+        // Default empty string so the {{existing_prd_json}} token always resolves —
+        // playbook treats empty as "no existing PRD, fresh run". Without this default
+        // the renderPlaybook pass logs an "unresolved template variables" warning
+        // every time a fresh plan-to-prd dispatches.
+        vars.existing_prd_json = '';
         // Check if a PRD already exists for this plan — reuse its filename to avoid duplicates (#884)
         let prdFilename = null;
         const prdFiles = safeReadDir(PRD_DIR).filter(f => f.endsWith('.json'));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yemi33/minions",
-  "version": "0.1.1572",
+  "version": "0.1.1574",
   "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"