@yemi33/minions 0.1.1573 → 0.1.1574

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # 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

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/package.json

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