@yemi33/minions 0.1.974 → 0.1.976

package/CHANGELOG.md

@@ -1,11 +1,12 @@
 # Changelog
 
-## 0.1.974 (2026-04-14)
+## 0.1.976 (2026-04-14)
 
 ### Features
 -  flush queued CC messages as single combined request
 -  add quality standard reminder to all agent and CC prompts
 - surface in-flight tool calls in lastAction (#1064)
+- reassign tasks when preferred agent is busy too long
 - wire agentBusyReassignMs into settings UI and persist
 -  ADO throttle detection, poll guards, and dashboard banner (#1051)
 - gate auto-fix conflict dispatch behind autoFixConflicts flag
@@ -22,12 +23,13 @@
 - CC tab unread dot + reopened badge on work items
 - fix dep re-merge failure on retry with existing commits (#977)
 - audit and harden log buffering implementation (#971)
-- replace magic string 'active' with PR_STATUS.ACTIVE in lifecycle.js (#969)
 
 ### Fixes
+-  move review verdict check before updateWorkItemStatus(DONE)
 -  skip isAlreadyDispatched in needsReReview to allow re-review within 1hr
 -  loop queue flush so messages queued during combined send aren't orphaned
 -  set lastPushedAt on fix completion to unblock re-review without poller
+-  restore ADO throttle test functions deleted during merge conflict resolution
 -  trigger second-pass re-review after fix agent completes
 -  update ccInFlightAborts when retry LLM replaces original
 -  skip orphaned LLM retry when client disconnected during CC stream
@@ -40,11 +42,9 @@
 - enforce --timeout 1 on all azureauth calls to prevent agent orphans (#1063)
 - cancel steering kill watcher on resume spawn (#1052) (#1062)
 - suppress warn for optional template variables (#1061)
+- skip SessionStart hook settings test on CI
+- fix boolean settings test and add agentBusyReassignMs to dashboard
 - fix ENGINE_DEFAULTS ref and derive boolean settings from defaults
-- add autoFixConflicts to boolean settings persist list
-- tighten build query guard to require mergeCommitId
-- restore sourceVersion filter after refs/pull merge ref switch
-- use refs/pull/{id}/merge for build status scoping
 
 ### Other
 - refactor(dashboard): extract _releaseCCTab helper and CC_LOCK_WAIT_MS constant

package/dashboard.js

@@ -3789,7 +3789,7 @@ What would you like to discuss or change? When you're happy, say "approve" and I
           versionCheckInterval: [60000],
           maxBuildFixAttempts: [1, 10],
           adoPollStatusEvery: [1], adoPollCommentsEvery: [1],
-          agentBusyReassignMs: [60000],
+          agentBusyReassignMs: [0],
         };
         for (const [key, [min, max]] of Object.entries(numericFields)) {
           if (e[key] !== undefined) {

package/engine/lifecycle.js

@@ -940,8 +940,6 @@ function updatePrAfterFix(pr, project, source) {
     if (!target) return prs;
     // Never downgrade from approved — fix was dispatched but PR is already approved
     if (target.reviewStatus !== 'approved') target.reviewStatus = 'waiting';
-    // Advance lastPushedAt so alreadyReviewed resets immediately, without waiting for the poller
-    target.lastPushedAt = ts();
     // Always clear pendingFix — a fix dispatch (regardless of source) addresses all pending feedback
     if (target.humanFeedback) target.humanFeedback.pendingFix = false;
     if (source === 'pr-human-feedback') {
@@ -1599,6 +1597,41 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
     // If decomposition produced nothing, fall through to mark parent as done
   }
 
+  // Verify review work items include a verdict — must run BEFORE updateWorkItemStatus(DONE),
+  // same pattern as plan-to-prd (#893): updateWorkItemStatus deletes _retryCount, so the check
+  // must read/increment it before that happens. Also sets skipDoneStatus so completedAt isn't
+  // written and then left dangling when status is reset to pending for retry.
+  if (effectiveSuccess && type === WORK_TYPE.REVIEW && meta?.item?.id) {
+    const verdict = parseReviewVerdict(resultSummary);
+    if (!verdict) {
+      skipDoneStatus = true;
+      const wiPath = resolveWorkItemPath(meta);
+      if (wiPath) {
+        try {
+          mutateJsonFileLocked(wiPath, data => {
+            if (!Array.isArray(data)) return data;
+            const w = data.find(i => i.id === meta.item.id);
+            if (!w) return data;
+            const retries = w._retryCount || 0;
+            if (retries < ENGINE_DEFAULTS.maxRetries) {
+              w.status = WI_STATUS.PENDING;
+              w._retryCount = retries + 1;
+              delete w.dispatched_at;
+              delete w.completedAt;
+              log('warn', `Review ${meta.item.id} completed without verdict — auto-retry ${retries + 1}/${ENGINE_DEFAULTS.maxRetries}`);
+            } else {
+              w.status = WI_STATUS.FAILED;
+              w.failReason = 'No review verdict after ' + ENGINE_DEFAULTS.maxRetries + ' attempts';
+              w.failedAt = ts();
+              log('warn', `Review ${meta.item.id} failed — no verdict after ${ENGINE_DEFAULTS.maxRetries} retries`);
+            }
+            return data;
+          });
+        } catch (err) { log('warn', `review verdict check: ${err.message}`); }
+      }
+    }
+  }
+
   // Verify plan-to-prd actually created the PRD file before marking done (#893)
   // Must run BEFORE updateWorkItemStatus(DONE) — otherwise _retryCount is deleted and retries never advance
   if (effectiveSuccess && type === WORK_TYPE.PLAN_TO_PRD && meta?.item?.id) {
@@ -1772,35 +1805,7 @@ async function runPostCompletionHooks(dispatchItem, agentId, code, stdout, confi
 
   // Old plan-to-prd PRD check removed — moved before updateWorkItemStatus(DONE) to fix #893
   // (retryCount was being deleted by done-marking before the check could read it)
-
-  // Verify review work items include a verdict — mirrors the PRD file existence check pattern.
-  // If agent completed (exit 0) but output has no VERDICT: APPROVE/REQUEST_CHANGES, the review
-  // is incomplete. Auto-retry up to maxRetries, then fail.
-  if (effectiveSuccess && type === WORK_TYPE.REVIEW && meta?.item?.id) {
-    const verdict = parseReviewVerdict(resultSummary);
-    if (!verdict) {
-      const wiPath = resolveWorkItemPath(meta);
-      if (wiPath) {
-        mutateJsonFileLocked(wiPath, data => {
-          if (!Array.isArray(data)) return data;
-          const w = data.find(i => i.id === meta.item.id);
-          if (!w) return data;
-          const retries = w._retryCount || 0;
-          if (retries < ENGINE_DEFAULTS.maxRetries) {
-            w.status = WI_STATUS.PENDING;
-            w._retryCount = retries + 1;
-            delete w.dispatched_at;
-            log('warn', `Review ${meta.item.id} completed without verdict — auto-retry ${retries + 1}/${ENGINE_DEFAULTS.maxRetries}`);
-          } else {
-            w.status = WI_STATUS.FAILED;
-            w.failReason = 'No review verdict after ' + ENGINE_DEFAULTS.maxRetries + ' attempts';
-            log('warn', `Review ${meta.item.id} failed — no verdict after ${ENGINE_DEFAULTS.maxRetries} retries`);
-          }
-          return data;
-        });
-      }
-    }
-  }
+  // Review verdict check similarly moved before updateWorkItemStatus(DONE) — same root cause.
 
   if (type === WORK_TYPE.REVIEW) await updatePrAfterReview(agentId, meta?.pr, meta?.project, config, resultSummary);
   if (type === WORK_TYPE.FIX) {

package/package.json

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