@yemi33/minions 0.1.1627 → 0.1.1628

package/CHANGELOG.md

@@ -1,6 +1,9 @@
 # Changelog
 
-## 0.1.1627 (2026-04-29)
+## 0.1.1628 (2026-04-29)
+
+### Features
+- prevent stale PR redispatch
 
 ### Fixes
 - ADO PR helpers prefer az CLI with MCP fallback (#1833)

package/engine/copilot-models.json

@@ -1,5 +1,5 @@
 {
   "runtime": "copilot",
   "models": null,
-  "cachedAt": "2026-04-29T18:05:40.260Z"
+  "cachedAt": "2026-04-29T18:06:23.658Z"
 }

package/engine/dispatch.js

@@ -12,7 +12,7 @@ const { setCooldownFailure } = require('./cooldown');
 const { safeJson, safeWrite, safeReadDir, mutateJsonFileLocked, mutateWorkItems,
   mutatePullRequests, getProjects, projectWorkItemsPath, projectPrPath, log, ts, dateStamp,
   sidecarDispatchPrompt, deleteDispatchPromptSidecar,
-  WI_STATUS, DISPATCH_RESULT, ENGINE_DEFAULTS, AGENT_STATUS, FAILURE_CLASS } = shared;
+  WI_STATUS, DISPATCH_RESULT, ENGINE_DEFAULTS, AGENT_STATUS, FAILURE_CLASS, PR_STATUS } = shared;
 const { getConfig, getDispatch, DISPATCH_PATH, INBOX_DIR } = queries;
 
 const MINIONS_DIR = shared.MINIONS_DIR;
@@ -91,6 +91,64 @@ function addToDispatch(item) {
   return item.id;
 }
 
+function _resolveDispatchProject(projectRef, config) {
+  if (!projectRef) return null;
+  const projects = getProjects(config);
+  if (projectRef.name) {
+    const byName = projects.find(p => p.name === projectRef.name);
+    if (byName) return byName;
+  }
+  if (projectRef.localPath) {
+    const refPath = path.resolve(projectRef.localPath);
+    const byPath = projects.find(p => p.localPath && path.resolve(p.localPath) === refPath);
+    if (byPath) return byPath;
+  }
+  return projectRef;
+}
+
+function _isPrBackedDispatch(entry) {
+  return !!(entry?.meta?.pr && entry.meta?.project);
+}
+
+function getStalePrDispatchReason(entry, config) {
+  if (!_isPrBackedDispatch(entry)) return '';
+  const project = _resolveDispatchProject(entry.meta.project, config);
+  if (!project) return 'missing project metadata';
+
+  const tracked = shared.findPrRecord(queries.getPrs(project), entry.meta.pr, project);
+  const prLabel = entry.meta.pr?.id || entry.meta.pr?.url || entry.id;
+  if (!tracked) return `PR ${prLabel} is no longer tracked`;
+  if (tracked.status !== PR_STATUS.ACTIVE) return `PR ${tracked.id || prLabel} is ${tracked.status || 'missing status'}`;
+  if (tracked._contextOnly) return `PR ${tracked.id || prLabel} is context-only`;
+
+  const queuedBranch = entry.meta.branch || entry.meta.pr?.branch || '';
+  const trackedBranch = tracked.branch || '';
+  if (queuedBranch && trackedBranch && shared.sanitizeBranch(queuedBranch) !== shared.sanitizeBranch(trackedBranch)) {
+    return `PR ${tracked.id || prLabel} branch changed from ${queuedBranch} to ${trackedBranch}`;
+  }
+
+  return '';
+}
+
+function pruneStalePrDispatches(config = queries.getConfig()) {
+  const removed = [];
+  mutateDispatch((dispatch) => {
+    dispatch.pending = (dispatch.pending || []).filter(entry => {
+      const reason = getStalePrDispatchReason(entry, config);
+      if (!reason) return true;
+      removed.push({ entry, reason });
+      return false;
+    });
+    return dispatch;
+  });
+
+  for (const { entry, reason } of removed) {
+    try { deleteDispatchPromptSidecar(entry); } catch { /* cleanup best-effort */ }
+    log('info', `Dropped stale PR dispatch ${entry.id}: ${reason}`);
+  }
+  return removed.length;
+}
+
 // ─── Retryable Failure Classification ────────────────────────────────────────
 
 function isRetryableFailureReason(reason = '', failureClass = '') {
@@ -253,11 +311,16 @@ function completeDispatch(id, result = DISPATCH_RESULT.SUCCESS, reason = '', res
       if (prId && project) {
         try {
           const prsPath = projectPrPath(project);
+          let restored = false;
           mutatePullRequests(prsPath, prs => {
             const target = shared.findPrRecord(prs, { id: prId }, project);
-            if (target?.humanFeedback) target.humanFeedback.pendingFix = true;
+            if (target?.humanFeedback) {
+              target.humanFeedback.pendingFix = true;
+              restored = true;
+            }
           });
-          log('info', `Restored pendingFix=true on ${prId} after failed human-feedback fix`);
+          if (restored) log('info', `Restored pendingFix=true on ${prId} after failed human-feedback fix`);
+          else log('info', `Skipped pendingFix restore for ${prId} — PR is no longer tracked`);
         } catch (e) { log('warn', `restore pendingFix: ${e.message}`); }
       }
       // Clear completed dispatch entry so dedup doesn't block re-dispatch
@@ -424,6 +487,8 @@ module.exports = {
   completeDispatch,
   writeInboxAlert,
   updateAgentStatus,
+  getStalePrDispatchReason,
+  pruneStalePrDispatches,
   cancelPendingDispatchesForPr,
   cleanDispatchEntries,
   cancelPendingWorkItems,

package/engine.js

@@ -102,7 +102,7 @@ const withFileLock = shared.withFileLock;
 // ─── Dispatch Management (extracted to engine/dispatch.js) ───────────────────
 
 const { mutateDispatch, addToDispatch, isRetryableFailureReason, completeDispatch,
-  writeInboxAlert, updateAgentStatus } = require('./engine/dispatch');
+  writeInboxAlert, updateAgentStatus, pruneStalePrDispatches } = require('./engine/dispatch');
 
 // ─── Timeout / Steering / Idle (extracted to engine/timeout.js) ──────────────
 
@@ -3639,6 +3639,7 @@ async function tickInner() {
     const maxC = config.engine?.maxConcurrent ?? ENGINE_DEFAULTS.maxConcurrent;
     setTempBudget(Math.max(0, maxC - activeCountPre));
   }
+  try { pruneStalePrDispatches(config); } catch (e) { log('warn', 'prune stale PR dispatches: ' + e.message); }
   let discoveryOk = true;
   try { await discoverWork(config); } catch (e) { log('warn', 'discoverWork: ' + e.message); discoveryOk = false; }
 
@@ -3915,7 +3916,7 @@ module.exports = {
   validateConfig,
 
   // Dispatch management (re-exported from engine/dispatch.js)
-  mutateDispatch, addToDispatch, isRetryableFailureReason, completeDispatch, writeInboxAlert, updateAgentStatus,
+  mutateDispatch, addToDispatch, isRetryableFailureReason, completeDispatch, writeInboxAlert, updateAgentStatus, pruneStalePrDispatches,
   activeProcesses, realActivityMap, engineRestartGraceExempt,
   get engineRestartGraceUntil() { return engineRestartGraceUntil; },
   set engineRestartGraceUntil(v) { engineRestartGraceUntil = v; },

package/package.json

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