pr-shepherd 0.10.1 → 0.10.2

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pr-shepherd",
   "description": "Autonomous PR CI monitor and review-comment resolver for Claude Code",
-  "version": "0.10.1",
+  "version": "0.10.2",
   "author": {
     "name": "Jonathan Ong",
     "email": "jonathanrichardong@gmail.com"

package/README.md

@@ -29,12 +29,12 @@ At a high level, to start the monitor, the skill/command invokes a CLI that retu
 
 # PR #123 [MONITOR]
 
-Loop tag: `# pr-shepherd-loop:pr=123`
-Loop args: `4m --max-turns 50 --expires 8h`
+Loop tag: `#pr-shepherd-loop:pr=123:`
+Loop args: `4m`
 
 ## Loop prompt
 
-# pr-shepherd-loop:pr=123
+#pr-shepherd-loop:pr=123:
 
 **IMPORTANT — recurrence rules:** Do not call ScheduleWakeup or /loop. End the turn
 after completing the actions below. The cron job handles the next fire.
@@ -125,7 +125,9 @@ Some other workflow improvements:
 
 Recommendations:
 
-- Run `pr-shepherd` on all your PRs before you go to sleep so that you wake up to reviewable PRs. As it uses `/loop`, it will continue working when your rate limit window is reset. There is a default timeout of 8 hours and 50 loops before exiting automatically.
+- Run `pr-shepherd` on all your PRs before you go to sleep so that you wake up to reviewable PRs. As it uses `/loop`, it will continue working when your rate limit window is reset. The loop cancels automatically when the PR is merged, closed, or after the ready-delay elapses.
+- Instruct your agents to write comments in a single review (comment, changes requested, or approved). This allows the review's comments/threads to be minimized or resolved together, keeping your pull request history clean. If you write inline comments outside of a review, each comment would still show up in the pull request history and take up space.
+- Avoid sticky comments as they will continue to be hidden. Instead, just make a new comment, especially on reviews. If you really want sticky comments, instruct your agent to unhide/unminimize them when updating them.
 
 ## Design Principles
 

package/bin/commands/monitor.mjs

@@ -6,16 +6,10 @@ export async function runMonitor(opts) {
     if (prNumber === null) {
         throw new Error("No open PR found for current branch. Pass a PR number explicitly.");
     }
-    const { interval, maxTurns, expiresHours } = config.watch;
+    const { interval } = config.watch;
     if (typeof interval !== "string" || !/^\d+[smhd]$/.test(interval)) {
         throw new Error(`Invalid config: watch.interval must be a duration string like "4m" or "1h", got ${JSON.stringify(interval)}`);
     }
-    if (!Number.isFinite(expiresHours) || expiresHours <= 0 || !Number.isInteger(expiresHours)) {
-        throw new Error(`Invalid config: watch.expiresHours must be a positive integer, got ${JSON.stringify(expiresHours)}`);
-    }
-    if (!Number.isFinite(maxTurns) || maxTurns <= 0 || !Number.isInteger(maxTurns)) {
-        throw new Error(`Invalid config: watch.maxTurns must be a positive integer, got ${JSON.stringify(maxTurns)}`);
-    }
     // No space after `#` — `# text` is a CommonMark ATX heading; `#text` is not.
     // Trailing `:` prevents substring false positives: without it, the dedup grep
     // for pr=1 would match a cron prompt for pr=135. Both the CronList check in
@@ -23,7 +17,7 @@ export async function runMonitor(opts) {
     // block depend on this exact string — don't change the format.
     const loopTag = `#pr-shepherd-loop:pr=${prNumber}:`;
     const loopPrompt = buildLoopPrompt(prNumber, loopTag, opts.readyDelaySuffix);
-    const loopArgs = `${interval} --max-turns ${maxTurns} --expires ${expiresHours}h`;
+    const loopArgs = interval;
     return { prNumber, loopTag, loopArgs, loopPrompt };
 }
 // ---------------------------------------------------------------------------
@@ -44,7 +38,7 @@ export function formatMonitorResult(result) {
         "## Instructions",
         "",
         `1. Run \`CronList\`. If any job's prompt contains \`${loopTag}\`, run the \`## Loop prompt\` body once inline (as if it were a cron tick) then stop — do not create a duplicate loop.`,
-        `2. Otherwise, invoke the \`/loop\` skill via the Skill tool. Build the \`args\` parameter as: only the value inside the backticks on the \`Loop args\` line above (the interval/flags string — not the \`Loop args:\` label), then a blank line, then the full \`## Loop prompt\` body.`,
+        `2. Otherwise, invoke the \`/loop\` skill via the Skill tool. Build the \`args\` parameter as: only the value inside the backticks on the \`Loop args\` line above (the interval — not the \`Loop args:\` label), then a blank line, then the full \`## Loop prompt\` body.`,
     ].join("\n");
 }
 // ---------------------------------------------------------------------------

package/bin/config.json

@@ -7,9 +7,7 @@
   },
   "watch": {
     "interval": "4m",
-    "readyDelayMinutes": 10,
-    "expiresHours": 8,
-    "maxTurns": 50
+    "readyDelayMinutes": 10
   },
   "resolve": {
     "shaPoll": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pr-shepherd",
-  "version": "0.10.1",
+  "version": "0.10.2",
   "description": "Autonomous PR CI monitor and review-comment resolver for Claude Code",
   "license": "MIT",
   "author": "Jonathan Ong",