pr-shepherd 0.10.3 → 0.12.0

package/bin/reporters/text.mjs

@@ -1,7 +1,7 @@
 import { buildCheckInstructions } from "./check-instructions.mjs";
-import { renderThreadBullet, renderCommentBullet, renderReviewBullet, renderFirstLookStatusTag, } from "../cli/list-formatters.mjs";
+import { renderThreadBullet, renderCommentBullet, renderFirstLookStatusTag, renderReviewListSection, renderThreadResolutionStatusTag, } from "../cli/list-formatters.mjs";
 import { joinSections } from "../util/markdown.mjs";
-export function formatText(report) {
+export function formatText(report, opts) {
     const header = [
         `# PR #${report.pr} [CHECK] — ${report.repo}`,
         `Status: ${report.status}`,
@@ -60,8 +60,11 @@ export function formatText(report) {
         ciSubsections.push(lines.join("\n"));
     }
     const ciSection = `## CI Checks\n\n${ciSubsections.join("\n\n")}`;
-    const { actionable: actionableThreads, autoResolved, autoResolveErrors, firstLook: firstLookThreads, } = report.threads;
-    const hasThreadSection = autoResolved.length > 0 || autoResolveErrors.length > 0 || actionableThreads.length > 0;
+    const { actionable: actionableThreads, resolutionOnly: resolutionOnlyThreads, autoResolved, autoResolveErrors, firstLook: firstLookThreads, } = report.threads;
+    const hasThreadSection = autoResolved.length > 0 ||
+        autoResolveErrors.length > 0 ||
+        actionableThreads.length > 0 ||
+        resolutionOnlyThreads.length > 0;
     let threadsSection = null;
     if (hasThreadSection) {
         const subparts = [];
@@ -86,6 +89,13 @@ export function formatText(report) {
             }
             subparts.push(lines.join("\n"));
         }
+        if (resolutionOnlyThreads.length > 0) {
+            const lines = [`### Needs resolution only (${resolutionOnlyThreads.length})`, ""];
+            for (const t of resolutionOnlyThreads) {
+                lines.push(renderThreadBullet(t, { statusTag: renderThreadResolutionStatusTag(t) }));
+            }
+            subparts.push(lines.join("\n"));
+        }
         threadsSection = `## Review Threads\n\n${subparts.join("\n\n")}`;
     }
     const { actionable: actionableComments, firstLook: firstLookComments } = report.comments;
@@ -97,15 +107,15 @@ export function formatText(report) {
         }
         commentsSection = `## PR Comments\n\n${lines.join("\n")}`;
     }
-    const changesRequestedSection = reviewListSection("CHANGES_REQUESTED Reviews", report.changesRequestedReviews);
+    const changesRequestedSection = renderReviewListSection("CHANGES_REQUESTED Reviews", report.changesRequestedReviews);
     const allSummaries = [...report.firstLookSummaries, ...report.reviewSummaries];
-    const reviewSummariesSection = reviewListSection("Review Summaries", allSummaries);
-    const approvedReviewsSection = reviewListSection("Approved Reviews", report.approvedReviews);
+    const reviewSummariesSection = renderReviewListSection("Review Summaries", allSummaries);
+    const approvedReviewsSection = renderReviewListSection("Approved Reviews", report.approvedReviews);
     const firstLookTotal = firstLookThreads.length + firstLookComments.length;
     let firstLookSection = null;
     if (firstLookTotal > 0) {
         const lines = [
-            `## First-look items (${firstLookTotal}) — already closed on GitHub; acknowledge only`,
+            `## First-look items (${firstLookTotal}) — acknowledge status before acting`,
             "",
         ];
         for (const t of firstLookThreads) {
@@ -117,15 +127,18 @@ export function formatText(report) {
         }
         firstLookSection = lines.join("\n");
     }
-    const totalActionable = actionableThreads.length + actionableComments.length + report.changesRequestedReviews.length;
+    const totalActionable = actionableThreads.length +
+        resolutionOnlyThreads.length +
+        actionableComments.length +
+        report.changesRequestedReviews.length;
     const counts = [];
     if (totalActionable > 0)
         counts.push(`${totalActionable} actionable`);
     if (firstLookTotal > 0)
         counts.push(`${firstLookTotal} first-look`);
-    const summaryLine = counts.join(", ") || "0 actionable — all threads resolved/minimized";
+    const summaryLine = counts.join(", ") || "0 actionable — no unresolved review items";
     const summarySection = `## Summary\n\n${summaryLine}`;
-    const instructions = buildCheckInstructions(report);
+    const instructions = buildCheckInstructions(report, opts);
     const instructionsSection = `## Instructions\n\n${instructions.map((step, i) => `${i + 1}. ${step}`).join("\n")}`;
     return joinSections([
         header,
@@ -141,8 +154,3 @@ export function formatText(report) {
         instructionsSection,
     ]);
 }
-function reviewListSection(heading, items) {
-    if (items.length === 0)
-        return null;
-    return `## ${heading}\n\n${items.map((r) => renderReviewBullet(r, { includeBody: true })).join("\n")}`;
-}

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "pr-shepherd",
-  "version": "0.10.3",
-  "description": "Autonomous PR CI monitor and review-comment resolver for Claude Code",
+  "version": "0.12.0",
+  "description": "Autonomous PR CI monitor and review-comment resolver for agentic coding tools",
   "license": "MIT",
   "author": "Jonathan Ong",
   "type": "module",
@@ -49,6 +49,7 @@
     "ci",
     "code-review",
     "automation",
+    "codex",
     "claude",
     "claude-code"
   ],