@yemi33/minions 0.1.2290 → 0.1.2291

package/dashboard.js

@@ -6804,13 +6804,13 @@ const server = http.createServer(async (req, res) => {
       if (originWi) item._originWi = originWi;
       await copyWorkItemPrFields(item, body, null, { project: targetProject });
       // W-mq5wfh1v000e0da9 — Auto-enroll the PR into pull-requests.json when
-      // this is a `type: fix` WI carrying a structured PR pointer and the PR
-      // isn't tracked yet. Without this, the engine's `pr_not_found` gate
-      // skips the fix forever (engine.js dispatch loop). Idempotent — safe to
+      // this is a `type: fix/review/test` WI carrying a structured PR pointer and
+      // the PR isn't tracked yet. Without this, the engine's `pr_not_found` gate
+      // skips the WI forever (engine.js dispatch loop). Idempotent — safe to
       // call on every WI create. Failures are swallowed (best-effort belt;
       // engine.js#dispatch loop also calls this as defense-in-depth).
       try {
-        shared.autoEnrollPrFromFixWorkItem(item, targetProject, MINIONS_DIR);
+        shared.autoEnrollPrFromWorkItem(item, targetProject, MINIONS_DIR);
       } catch (e) {
         shared.log('warn', `auto-enroll PR for ${item.id}: ${e.message}`);
       }

package/engine/live-checkout.js

@@ -101,7 +101,6 @@ const shared = require('./shared');
 // still bounded. Threaded through `baseOpts` so every git call inherits it; the
 // injected `_git` mock in unit tests ignores opts so the seam is unaffected.
 const LIVE_CHECKOUT_GIT_MAX_BUFFER = 50 * 1024 * 1024;
-
 // PL-live-checkout-reliability-hardening — partial-clone / GVFS blob-fetch
 // signature match. On a Scalar/GVFS-managed ADO repo (blobless partial clone),
 // switching the working tree onto a branch whose tree differs from HEAD must

package/engine/shared.js

@@ -7245,11 +7245,12 @@ function classifyPrRefForVerification(prRef, project = null) {
 // per the "structured-vs-loose split" — enrollment must be intentional.
 //
 // Returns:
-//   { skipped: true, reason }            — not a fix / no ref / no URL / upsert error
+//   { skipped: true, reason }            — not a pr-requiring type / no ref / no URL / upsert error
 //   { alreadyEnrolled: true, id }        — PR already in pull-requests.json
 //   { enrolled: true, id, prPath }       — newly enrolled
-function autoEnrollPrFromFixWorkItem(item, project, minionsDir) {
-  if (!item || item.type !== WORK_TYPE.FIX) return { skipped: true, reason: 'not-fix' };
+const _PR_REQUIRING_TYPES = new Set([WORK_TYPE.FIX, WORK_TYPE.REVIEW, WORK_TYPE.TEST]);
+function autoEnrollPrFromWorkItem(item, project, minionsDir) {
+  if (!item || !_PR_REQUIRING_TYPES.has(item.type)) return { skipped: true, reason: 'not-pr-requiring-type' };
   const prRef = extractStructuredWorkItemPrRef(item);
   if (!prRef) return { skipped: true, reason: 'no-structured-ref' };
   const url = deriveUrlForPrRef(prRef, project);
@@ -7280,10 +7281,12 @@ function autoEnrollPrFromFixWorkItem(item, project, minionsDir) {
       ? { enrolled: true, id: result.id, prPath }
       : { alreadyEnrolled: true, id: result.id };
   } catch (e) {
-    log('warn', `autoEnrollPrFromFixWorkItem ${item.id}: ${e.message}`);
+    log('warn', `autoEnrollPrFromWorkItem ${item.id}: ${e.message}`);
     return { skipped: true, reason: 'upsert-error', error: e.message };
   }
 }
+// Backward-compat alias — external callers that reference autoEnrollPrFromFixWorkItem keep working.
+const autoEnrollPrFromFixWorkItem = autoEnrollPrFromWorkItem;
 
 // ─── Cross-Platform Process Kill Helpers ─────────────────────────────────────
 
@@ -9015,6 +9018,7 @@ module.exports = {
   isContextOnlyPrRecord,
   upsertPullRequestRecord,
   isAutoManagedPrRecord, // W-mq5s5ttx000j7ab8-a — exported for engine + watch-plugin gate consolidation
+  autoEnrollPrFromWorkItem,
   autoEnrollPrFromFixWorkItem,
   deriveUrlForPrRef, // exported for testing
   classifyPrRefForVerification, // issue #246 — host routing for loose PR-ref verification

package/engine.js

@@ -8096,16 +8096,15 @@ function discoverFromWorkItems(config, project) {
       skipped.noAgent++; continue;
     }
 
-    // W-mq5wfh1v000e0da9 — defense-in-depth: auto-enroll the PR for fix WIs
+    // W-mq5wfh1v000e0da9 — defense-in-depth: auto-enroll the PR for fix/review/test WIs
     // carrying a structured PR pointer. Primary enrollment runs in the
     // dashboard `POST /api/work-items` handler, but WIs created via CLI / restored
     // from disk / older code paths might land here without the PR record.
-    // No-op if the PR is already tracked. Failures swallowed — the gate below
-    // still trips on missing PR records and surfaces `pr_not_found`.
+    // No-op if the PR is already tracked or if the type doesn't require a PR.
+    // Failures swallowed — the gate below still trips on missing PR records
+    // and surfaces `pr_not_found`.
     try {
-      if (item.type === WORK_TYPE.FIX) {
-        shared.autoEnrollPrFromFixWorkItem(item, project, MINIONS_DIR);
-      }
+      shared.autoEnrollPrFromWorkItem(item, project, MINIONS_DIR);
     } catch (e) { log('warn', `auto-enroll PR for ${item.id}: ${e.message}`); }
 
     const linkedPr = resolveWorkItemPrRecord(item, project);

package/package.json

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