@yemi33/minions 0.1.2098 → 0.1.2099

package/engine/shared.js

@@ -5202,6 +5202,14 @@ function getPrFixAutomationCause({ dispatchKey = '', source = '', task = '' } =
   return PR_FIX_CAUSE.UNKNOWN;
 }
 
+// Source-branch head SHA, normalized across hosts. GitHub PRs carry
+// `headSha`/`headRefOid` (engine/github.js:718-742 keeps both in sync); ADO PRs
+// carry `_adoSourceCommit`/`headRefOid` (engine/ado.js:1083-1129) and a legacy
+// `_adoHeadCommit`. Mirrors engine/lifecycle.js:1849 getPrFixBaselineHead.
+function _prHeadSha(pr) {
+  return String(pr?.headRefOid || pr?.headSha || pr?._adoSourceCommit || pr?._adoHeadCommit || '').trim();
+}
+
 function prFixEvidenceFingerprint(pr, cause = PR_FIX_CAUSE.UNKNOWN) {
   const review = pr?.minionsReview || {};
   const feedback = pr?.humanFeedback || {};
@@ -5214,6 +5222,13 @@ function prFixEvidenceFingerprint(pr, cause = PR_FIX_CAUSE.UNKNOWN) {
     evidence.buildFailReason = pr?.buildFailReason || '';
     evidence.buildErrorLog = pr?.buildErrorLog || '';
     evidence.buildStatusDetail = pr?._buildStatusDetail || '';
+    // #2979 — head SHA + lastPushedAt are the only fingerprint components that
+    // change across a rebase + force-push. Without them, a no-op-fix pause was
+    // sticky forever because failing pipeline name / fail reason / error log
+    // are unchanged across the push. Existing paused records re-fingerprint on
+    // the next poll and clear naturally when the head moves.
+    evidence.headRefOid = _prHeadSha(pr);
+    evidence.lastPushedAt = pr?.lastPushedAt || '';
   } else if (cause === PR_FIX_CAUSE.MERGE_CONFLICT) {
     evidence.mergeConflict = !!pr?._mergeConflict;
     evidence.mergeStatus = pr?.mergeStatus || '';
@@ -5223,6 +5238,13 @@ function prFixEvidenceFingerprint(pr, cause = PR_FIX_CAUSE.UNKNOWN) {
     evidence.lastReviewedAt = pr?.lastReviewedAt || '';
     evidence.reviewedAt = review.reviewedAt || '';
     evidence.reviewNote = review.note || pr?.reviewNote || '';
+    // #2979 — same rationale as BUILD_FAILURE: review feedback fingerprints
+    // were sticky across force-push because reviewStatus / reviewedAt /
+    // reviewNote don't change when the author rebases. Adding the head SHA
+    // gives REVIEW_FEEDBACK the same natural-unsticking property HUMAN_FEEDBACK
+    // already has via lastProcessedCommentDate.
+    evidence.headRefOid = _prHeadSha(pr);
+    evidence.lastPushedAt = pr?.lastPushedAt || '';
   }
   return crypto.createHash('sha1').update(JSON.stringify(evidence)).digest('hex').slice(0, 16);
 }

package/package.json

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