pr-shepherd 1.1.1 → 1.1.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": "1.1.1",
+  "version": "1.1.2",
   "author": {
     "name": "Jonathan Ong",
     "email": "jonathanrichardong@gmail.com"

package/bin/commands/check-annotations.d.mts

@@ -14,14 +14,12 @@ export declare function hasCheckDrivenActionableWork(checks: {
     passing: ClassifiedCheck[];
     skipped: ClassifiedCheck[];
     filtered: ClassifiedCheck[];
-    ignored?: TriagedCheck[];
 }, mergeStatusValue: string): boolean;
 export declare function attachAndMergeCheckAnnotations(buckets: {
     passing: ClassifiedCheck[];
     failing: TriagedCheck[];
     skipped: ClassifiedCheck[];
     filtered: ClassifiedCheck[];
-    ignored: ClassifiedCheck[];
 }, seenMap: Map<string, {
     seenAt: number;
 }>, prNumber: number, cacheOpts?: AnnotationCacheOptions): Promise<{
@@ -29,6 +27,5 @@ export declare function attachAndMergeCheckAnnotations(buckets: {
     failing: TriagedCheck[];
     skipped: ClassifiedCheck[];
     filtered: ClassifiedCheck[];
-    ignored: ClassifiedCheck[];
 }>;
 export declare function annotationMarkerBody(a: CheckAnnotation): string;

package/bin/commands/check-annotations.mjs

@@ -12,7 +12,6 @@ export function checksWithActionableAnnotations(report) {
         ...report.checks.passing,
         ...report.checks.skipped,
         ...report.checks.filtered,
-        ...(report.checks.ignored ?? []),
     ].filter(hasActionableAnnotation);
 }
 /**
@@ -25,13 +24,7 @@ export function checksWithActionableAnnotations(report) {
  */
 export function hasCheckDrivenActionableWork(checks, mergeStatusValue) {
     return (checks.failing.length > 0 ||
-        [
-            ...checks.failing,
-            ...checks.passing,
-            ...checks.skipped,
-            ...checks.filtered,
-            ...(checks.ignored ?? []),
-        ].some(hasActionableAnnotation) ||
+        [...checks.failing, ...checks.passing, ...checks.skipped, ...checks.filtered].some(hasActionableAnnotation) ||
         mergeStatusValue === "CONFLICTS");
 }
 export async function attachAndMergeCheckAnnotations(buckets, seenMap, prNumber, cacheOpts) {
@@ -40,7 +33,6 @@ export async function attachAndMergeCheckAnnotations(buckets, seenMap, prNumber,
         ...buckets.passing,
         ...buckets.skipped,
         ...buckets.filtered,
-        ...buckets.ignored,
     ].filter(shouldFetchCheckAnnotations);
     const expectedCounts = new Map(candidates.flatMap((c) => c.id != null && c.annotationCount ? [[c.id, c.annotationCount]] : []));
     const annotated = await attachUnseenCheckAnnotations(candidates, seenMap, prNumber, cacheOpts && { ...cacheOpts, expectedCounts });
@@ -57,7 +49,6 @@ export async function attachAndMergeCheckAnnotations(buckets, seenMap, prNumber,
         failing: apply(buckets.failing),
         skipped: apply(buckets.skipped),
         filtered: apply(buckets.filtered),
-        ignored: apply(buckets.ignored),
     };
 }
 async function attachUnseenCheckAnnotations(checks, seenMap, prNumber, cacheOpts) {

package/bin/commands/check-unreported.mjs

@@ -110,7 +110,7 @@ function reportChecks(report) {
         ...report.checks.inProgress,
         ...report.checks.skipped,
         ...report.checks.filtered,
-        ...(report.checks.ignored ?? []),
+        ...(report.checks.ignoredNames ?? []).map((name) => ({ name })),
         ...(report.checks.supersededNames ?? []).map((name) => ({ name })),
     ];
 }

package/bin/commands/check.mjs

@@ -186,7 +186,6 @@ async function runScopedCheck(opts, context, repo, prNumber, stateKey) {
     const inProgress = classifiedChecks.filter((c) => c.category === "in_progress");
     const skipped = classifiedChecks.filter((c) => c.category === "skipped");
     const filtered = classifiedChecks.filter((c) => c.category === "filtered");
-    const ignored = classifiedChecks.filter((c) => c.category === "ignored");
     const triagedBase = failing.length > 0 && !opts.skipTriage
         ? await triageFailingChecks(failing, repo, stateKey, triageBudget, classifiedChecks.filter((c) => c.category !== "failing"))
         : failing;
@@ -199,8 +198,7 @@ async function runScopedCheck(opts, context, repo, prNumber, stateKey) {
     const ruleSet = await loadClassifyRules(getEffectiveCwd(), config.standardRules);
     const classifyIndex = buildClassifyIndex(ruleSet, batchData);
     const partition = partitionBatch(classifyIndex, batchData);
-    const merged = await attachAndMergeCheckAnnotations({ passing, failing: triagedBase, skipped, filtered, ignored }, seenMap, prNumber, { stateKey, headSha: batchData.headRefOid });
-    const ignoredAnnotated = merged.ignored.filter((c) => (c.annotations?.length ?? 0) > 0);
+    const merged = await attachAndMergeCheckAnnotations({ passing, failing: triagedBase, skipped, filtered }, seenMap, prNumber, { stateKey, headSha: batchData.headRefOid });
     const minimizedCommentCandidates = batchData.comments.filter((c) => c.isMinimized && !partition.suppressedCommentIds.has(c.id));
     const deniedRuleAutoResolveCommentIds = new Set(partition.ruleAutoResolveCommentIds.filter((id) => !canGenerateItemMutation(id, batchData.comments.find((comment) => comment.id === id)?.body ?? "", batchData.comments.find((comment) => comment.id === id)?.viewerCanMinimize, "minimize")));
     const visibleCommentClassification = classifyVisibleComments(batchData.comments.filter((c) => !partition.suppressedCommentIds.has(c.id) || deniedRuleAutoResolveCommentIds.has(c.id)), seenMap, config.iterate["minimize-comments"], botUsernames, new Set(batchData.comments
@@ -301,7 +299,6 @@ async function runScopedCheck(opts, context, repo, prNumber, stateKey) {
             passing: merged.passing,
             skipped: merged.skipped,
             filtered: merged.filtered,
-            ignored: merged.ignored,
         }, mergeStatus.status);
     // A deferred tick shows no review items, so it is never capped; its seen markers stay unwritten.
     const { buckets: budgeted, rollup: reviewItems, heldThreadReviewIds, } = applyReviewOutputBudget({
@@ -314,12 +311,7 @@ async function runScopedCheck(opts, context, repo, prNumber, stateKey) {
         editedComments,
     }, deferWhileQueued ? 0 : (opts.reviewOutputMaxChars ?? 0));
     if (opts.persistSeen !== false) {
-        const successfulAnnotations = [
-            ...merged.passing,
-            ...merged.skipped,
-            ...merged.filtered,
-            ...merged.ignored,
-        ]
+        const successfulAnnotations = [...merged.passing, ...merged.skipped, ...merged.filtered]
             .filter((check) => check.conclusion === "SUCCESS")
             .flatMap((check) => check.annotations ?? []);
         const deferredMarkSeen = deferWhileQueued
@@ -410,7 +402,6 @@ async function runScopedCheck(opts, context, repo, prNumber, stateKey) {
             inProgress: inProgress,
             skipped: merged.skipped,
             filtered: merged.filtered,
-            ...(ignoredAnnotated.length > 0 && { ignored: ignoredAnnotated }),
             filteredNames: verdict.filteredNames,
             blockedByFilteredCheck,
             ...(verdict.ignoredNames.length > 0 && {

package/bin/commands/iterate/conflicting-head-ci.d.mts

@@ -12,7 +12,7 @@ export declare function countReportedChecks(checks: {
     inProgress: readonly unknown[];
     skipped: readonly unknown[];
     filtered: readonly unknown[];
-    ignored?: readonly unknown[];
+    ignoredNames?: readonly unknown[];
 }): number;
 /** Place the missing-CI note with the conflict step, ahead of later repair lines. */
 export declare function insertConflictingHeadCiNote(instructions: string[], note: string | undefined): void;

package/bin/commands/iterate/conflicting-head-ci.mjs

@@ -18,7 +18,7 @@ export function countReportedChecks(checks) {
         checks.inProgress.length +
         checks.skipped.length +
         checks.filtered.length +
-        (checks.ignored?.length ?? 0));
+        (checks.ignoredNames?.length ?? 0));
 }
 /** Place the missing-CI note with the conflict step, ahead of later repair lines. */
 export function insertConflictingHeadCiNote(instructions, note) {

package/bin/github/poll-summary-annotations.d.mts

@@ -4,7 +4,7 @@ import type { RawSummaryPr } from "./poll-summary-raw.mts";
 type Routed = Pick<PollSummaryItem, "action" | "reasons">;
 /**
  * The one-PR check treats unseen annotations on any non-SUCCESS check (failing, skipped,
- * filtered, ignored or neutral) as actionable work. The compact summary only has annotation
+ * filtered or neutral) as actionable work; `checks.ignore` checks never gate, annotations included. The compact summary only has annotation
  * totals, and only for a row that otherwise claims readiness, so it loads them then and reads
  * the annotations of just the non-SUCCESS checks that have any. A row with unseen annotations,
  * or whose annotations cannot be read, goes to its one-PR `pollCommand` instead of

package/bin/github/poll-summary-annotations.mjs

@@ -11,7 +11,7 @@ const READY_CLAIMS = new Set([
 ]);
 /**
  * The one-PR check treats unseen annotations on any non-SUCCESS check (failing, skipped,
- * filtered, ignored or neutral) as actionable work. The compact summary only has annotation
+ * filtered or neutral) as actionable work; `checks.ignore` checks never gate, annotations included. The compact summary only has annotation
  * totals, and only for a row that otherwise claims readiness, so it loads them then and reads
  * the annotations of just the non-SUCCESS checks that have any. A row with unseen annotations,
  * or whose annotations cannot be read, goes to its one-PR `pollCommand` instead of
@@ -44,7 +44,7 @@ function nonSuccessCheckTotals(raw, required, totalsKnown) {
     for (const check of classifiedSummaryChecks(raw, required)) {
         if (check.scope || check.id == null || !head.has(check.id))
             continue;
-        if (check.category === "in_progress" || check.category === "superseded")
+        if (["in_progress", "superseded", "ignored"].includes(check.category))
             continue;
         if (check.status !== "COMPLETED" || check.conclusion === "SUCCESS")
             continue;

package/bin/types/report.d.mts

@@ -69,8 +69,6 @@ export interface ShepherdReport {
         skipped: ClassifiedCheck[];
         /** Checks filtered out because they were triggered by a non-PR event (push, schedule, etc.). */
         filtered: ClassifiedCheck[];
-        /** Ignored checks with unseen annotations; omitted when empty. */
-        ignored?: ClassifiedCheck[];
         filteredNames: string[];
         blockedByFilteredCheck: boolean;
         ignoredNames?: string[];

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "1.1.1",
+  "version": "1.1.2",
   "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
   "keywords": [
     "automation",

package/plugins/pr-shepherd/.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": "1.1.1",
+  "version": "1.1.2",
   "author": {
     "name": "Jonathan Ong",
     "email": "jonathanrichardong@gmail.com"

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

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

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

@@ -2,7 +2,7 @@
   "mcpServers": {
     "pr-shepherd": {
       "command": "npx",
-      "args": ["--yes", "--package", "pr-shepherd@1.1.1", "pr-shepherd-mcp"]
+      "args": ["--yes", "--package", "pr-shepherd@1.1.2", "pr-shepherd-mcp"]
     }
   }
 }

package/plugins/pr-shepherd/.mcp.json

@@ -1,6 +1,6 @@
 {
   "pr-shepherd": {
     "command": "npx",
-    "args": ["--yes", "--package", "pr-shepherd@1.1.1", "pr-shepherd-mcp"]
+    "args": ["--yes", "--package", "pr-shepherd@1.1.2", "pr-shepherd-mcp"]
   }
 }