pr-shepherd 0.26.1 → 0.26.2

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pr-shepherd",
   "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
-  "version": "0.26.1",
+  "version": "0.26.2",
   "author": {
     "name": "Jonathan Ong",
     "email": "jonathanrichardong@gmail.com"

package/bin/checks/classify.mjs

@@ -20,7 +20,7 @@ export function classifyChecks(checks) {
     const config = loadConfig();
     const relevantEvents = new Set(config.checks.ciTriggerEvents);
     const isIgnored = buildIgnoreMatcher(config.ignoreChecks ?? []);
-    return checks.filter((c) => !isIgnored(c.name)).map((c) => classify(c, relevantEvents));
+    return checks.map((c) => isIgnored(c.name) ? { ...c, category: "ignored" } : classify(c, relevantEvents));
 }
 function buildIgnoreMatcher(patterns) {
     if (patterns.length === 0)
@@ -51,7 +51,7 @@ function classify(check, relevantEvents) {
 }
 /** Compute a high-level CI verdict from a list of classified checks. */
 export function getCiVerdict(classified) {
-    const relevant = classified.filter((c) => c.category !== "filtered" && c.category !== "skipped");
+    const relevant = classified.filter((c) => c.category !== "filtered" && c.category !== "skipped" && c.category !== "ignored");
     const anyInProgress = relevant.some((c) => c.category === "in_progress");
     const anyFailing = relevant.some((c) => c.category === "failing");
     // When there are no relevant checks (e.g. docs-only PR where all checks are filtered/skipped),
@@ -59,5 +59,6 @@ export function getCiVerdict(classified) {
     const allPassed = !anyInProgress && !anyFailing;
     const hasChecks = relevant.length > 0;
     const filteredNames = classified.filter((c) => c.category === "filtered").map((c) => c.name);
-    return { allPassed, hasChecks, anyInProgress, anyFailing, filteredNames };
+    const ignoredNames = Array.from(new Set(classified.filter((c) => c.category === "ignored").map((c) => c.name)));
+    return { allPassed, hasChecks, anyInProgress, anyFailing, filteredNames, ignoredNames };
 }

package/bin/cli/iterate-formatter.mjs

@@ -112,6 +112,10 @@ export function formatIterateResult(result, opts) {
     const headerLines = [heading, "", baseLine, summaryLine];
     if (requiredLine)
         headerLines.push(requiredLine);
+    if (result.ignoredNames && result.ignoredNames.length > 0) {
+        const names = result.ignoredNames.map((n) => "`" + n + "`").join(", ");
+        headerLines.push(`**ignored** ${names}`);
+    }
     const activityLine = formatActivityLine(result);
     if (activityLine)
         headerLines.push(activityLine);
@@ -133,6 +137,10 @@ export function formatIterateResult(result, opts) {
             const cancelHeaderLines = [`${heading} — ${result.reason}`, "", baseLine, summaryLine];
             if (requiredLine)
                 cancelHeaderLines.push(requiredLine);
+            if (result.ignoredNames && result.ignoredNames.length > 0) {
+                const ignoredStr = result.ignoredNames.map((n) => "`" + n + "`").join(", ");
+                cancelHeaderLines.push(`**ignored** ${ignoredStr}`);
+            }
             if (activityLine)
                 cancelHeaderLines.push(activityLine);
             return joinSections([

package/bin/cli/iterate-lean.mjs

@@ -55,6 +55,9 @@ export function projectIterateLean(result, opts) {
         ...((result.inProgressChecks?.length ?? 0) > 0 && {
             inProgressChecks: result.inProgressChecks,
         }),
+        ...((result.ignoredNames?.length ?? 0) > 0 && {
+            ignoredNames: result.ignoredNames,
+        }),
     };
     switch (result.action) {
         case "wait":

package/bin/commands/check-status.mjs

@@ -13,13 +13,18 @@ export function computeStatus(verdict, unresolvedThreads, unresolvedComments, me
     // is BLOCKED (review pending, insufficient approvals, branch-protection rule, etc.).
     // Requires hasChecks so that a PR with zero relevant checks (CI never started, or all
     // filtered/skipped) doesn't prematurely trigger READY before any check has reported.
+    // Exception: UNSTABLE with ignored checks — UNSTABLE means only non-required checks are
+    // pending/failing, and if those are all ignored the handoff is safe even with no other checks.
+    // BLOCKED is excluded from the ignoredNames extension: BLOCKED can mean required checks haven't
+    // started, and handing off prematurely there risks a broken merge attempt.
     // blockingBotReviewInProgress is still excluded — a bot review is shepherd's problem, not a hand-off.
+    const hasRelevantPassingChecks = verdict.hasChecks || (mergeStatus.status === "UNSTABLE" && verdict.ignoredNames.length > 0);
     if (verdict.allPassed &&
-        verdict.hasChecks &&
+        hasRelevantPassingChecks &&
         unresolvedThreads === 0 &&
         unresolvedComments === 0 &&
         changesRequestedReviews === 0 &&
-        mergeStatus.status === "BLOCKED" &&
+        (mergeStatus.status === "BLOCKED" || mergeStatus.status === "UNSTABLE") &&
         !mergeStatus.blockingBotReviewInProgress) {
         return "READY";
     }

package/bin/commands/check.mjs

@@ -124,6 +124,7 @@ export async function runCheck(opts) {
             filtered,
             filteredNames: verdict.filteredNames,
             blockedByFilteredCheck,
+            ...(verdict.ignoredNames.length > 0 && { ignoredNames: verdict.ignoredNames }),
         },
         threads: {
             actionable: threadVisibility.activeThreads,

package/bin/commands/iterate/index.mjs

@@ -50,6 +50,7 @@ export async function runIterate(opts) {
             branchProtection: report.branchProtection,
             checks: buildRelevantChecks(report),
             inProgressChecks: buildActiveChecks(report),
+            ...(report.checks.ignoredNames?.length ? { ignoredNames: report.checks.ignoredNames } : {}),
             activity: report.activity,
             action: "cancel",
             reason: report.mergeStatus.state === "MERGED" ? "merged" : "closed",
@@ -93,6 +94,7 @@ export async function runIterate(opts) {
         branchProtection: report.branchProtection,
         checks: buildRelevantChecks(report),
         inProgressChecks: buildActiveChecks(report),
+        ...(report.checks.ignoredNames?.length ? { ignoredNames: report.checks.ignoredNames } : {}),
         activity: report.activity,
     };
     if (readyState.shouldCancel) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.26.1",
+  "version": "0.26.2",
   "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
   "license": "MIT",
   "author": "Jonathan Ong",

package/plugins/pr-shepherd/.codex-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.26.1",
+  "version": "0.26.2",
   "description": "Autonomous PR CI monitor and review-comment resolver for Codex.",
   "author": {
     "name": "Jonathan Ong",