agent-reviews 0.5.2 → 0.6.0

package/bin/agent-reviews.js

@@ -169,7 +169,7 @@ ${colors.bright}Options:${colors.reset}
   -v, --version      Show version
 
 ${colors.bright}Watch Mode:${colors.reset}
-  -w, --watch        Poll for new comments periodically
+  -w, --watch        Poll for new comments (exits on detection)
   -i, --interval     Poll interval in seconds (default: 30)
   --timeout          Exit after N seconds of inactivity (default: 600)
 
@@ -276,17 +276,48 @@ async function watchForComments(context, options) {
     const newComments = filtered.filter((c) => !seenIds.has(c.id));
 
     if (newComments.length > 0) {
-      lastActivityTime = Date.now();
+      for (const comment of newComments) {
+        seenIds.add(comment.id);
+      }
 
       console.log(
         `\n${colors.green}=== NEW COMMENTS DETECTED [${formatTimestamp()}] ===${colors.reset}`
       );
       console.log(
-        `${colors.bright}Found ${newComments.length} new comment${newComments.length === 1 ? "" : "s"}${colors.reset}\n`
+        `${colors.bright}Found ${newComments.length} new comment${newComments.length === 1 ? "" : "s"}${colors.reset}`
       );
 
-      for (const comment of newComments) {
+      // Brief grace period to catch any stragglers from the same bot batch
+      console.log(
+        `${colors.dim}Waiting 5s for additional comments...${colors.reset}`
+      );
+      await sleep(5);
+
+      // Re-fetch to catch any comments posted during the grace period
+      const graceData = await fetchPRComments(
+        owner,
+        repo,
+        prNumber,
+        token,
+        proxyFetch
+      );
+      const graceProcessed = processComments(graceData);
+      const graceFiltered = filterComments(graceProcessed, options);
+      const lateComments = graceFiltered.filter((c) => !seenIds.has(c.id));
+
+      for (const comment of lateComments) {
         seenIds.add(comment.id);
+        newComments.push(comment);
+      }
+
+      if (lateComments.length > 0) {
+        console.log(
+          `${colors.bright}Caught ${lateComments.length} additional comment${lateComments.length === 1 ? "" : "s"}${colors.reset}`
+        );
+      }
+
+      console.log("");
+      for (const comment of newComments) {
         console.log(formatComment(comment));
         console.log("");
       }
@@ -294,7 +325,16 @@ async function watchForComments(context, options) {
       // JSON output for AI agent parsing
       console.log(`${colors.dim}--- JSON for processing ---${colors.reset}`);
       console.log(JSON.stringify(newComments, null, 2));
-      console.log(`${colors.dim}--- end JSON ---${colors.reset}\n`);
+      console.log(`${colors.dim}--- end JSON ---${colors.reset}`);
+
+      // Exit immediately so the caller can process and restart if needed
+      console.log(
+        `\n${colors.green}=== WATCH: EXITING WITH NEW COMMENTS ===${colors.reset}`
+      );
+      console.log(
+        `${colors.dim}Restart watcher after processing to catch further comments.${colors.reset}`
+      );
+      return;
     } else {
       const inactiveSeconds = Math.round(
         (Date.now() - lastActivityTime) / 1000

package/package.json

@@ -1,12 +1,12 @@
 {
   "name": "agent-reviews",
-  "version": "0.5.2",
+  "version": "0.6.0",
   "description": "CLI and Claude Code skill for managing GitHub PR review comments. List, filter, reply, and watch for bot findings.",
   "license": "MIT",
   "author": "Paul Bakaus",
   "repository": {
     "type": "git",
-    "url": "https://github.com/pbakaus/agent-reviews.git"
+    "url": "git+https://github.com/pbakaus/agent-reviews.git"
   },
   "keywords": [
     "github",

package/skills/agent-reviews/SKILL.md

@@ -6,7 +6,7 @@ compatibility: Requires git and gh (GitHub CLI) installed. Designed for Claude C
 allowed-tools: Bash(node scripts/agent-reviews.js *) Bash(gh pr view *)
 metadata:
   author: pbakaus
-  version: "0.5.2"
+  version: "0.6.0"
   homepage: https://github.com/pbakaus/agent-reviews
 ---
 
@@ -110,27 +110,29 @@ Run `scripts/agent-reviews.js --reply <comment_id> "Skipped per user request"`
 
 ---
 
-## Phase 2: POLL FOR NEW COMMENTS (10-minute inactivity timeout)
+## Phase 2: POLL FOR NEW COMMENTS (loop until quiet)
 
-### Step 6: Start Watcher
+The watcher exits immediately when new comments are found (after a 5s grace period to catch batch posts). This means you run it in a loop: start watcher, process any comments it returns, restart watcher, repeat until the watcher times out with no new comments.
 
-Launch the watcher in the background. It polls every 30 seconds and exits after 10 minutes of inactivity (no new comments):
+### Step 6: Start Watcher Loop
 
-Run `scripts/agent-reviews.js --watch --bots-only` as a background task.
+Repeat the following until the watcher exits with no new comments:
 
-**CRITICAL: DO NOT cancel the background task early. Let it complete its full cycle.**
+**6a.** Launch the watcher in the background:
 
-### Step 7: Wait for Results
+Run `scripts/agent-reviews.js --watch --bots-only` as a background task.
 
-Use `TaskOutput` to wait for the watcher to complete (blocks up to 12 minutes).
+**6b.** Use `TaskOutput` to wait for the watcher to complete (blocks up to 12 minutes).
 
-### Step 8: Process New Comments (if any)
+**6c.** Check the output:
 
-If the watcher found new comments:
-1. Use `--detail <id>` to read each new comment's full detail
-2. Process them exactly as in Phase 1, Steps 3-5
+- **If new comments were found** (output contains `EXITING WITH NEW COMMENTS`):
+  1. Use `--detail <id>` to read each new comment's full detail
+  2. Process them exactly as in Phase 1, Steps 3-5 (evaluate, fix, commit, push, reply)
+  3. **Go back to Step 6a** to restart the watcher
 
-If no new comments were found, move to the summary.
+- **If no new comments** (output contains `WATCH COMPLETE`):
+  Stop looping and move to the Summary Report.
 
 ---
 

package/skills/agent-reviews/scripts/agent-reviews.js

@@ -169,7 +169,7 @@ ${colors.bright}Options:${colors.reset}
   -v, --version      Show version
 
 ${colors.bright}Watch Mode:${colors.reset}
-  -w, --watch        Poll for new comments periodically
+  -w, --watch        Poll for new comments (exits on detection)
   -i, --interval     Poll interval in seconds (default: 30)
   --timeout          Exit after N seconds of inactivity (default: 600)
 
@@ -276,17 +276,48 @@ async function watchForComments(context, options) {
     const newComments = filtered.filter((c) => !seenIds.has(c.id));
 
     if (newComments.length > 0) {
-      lastActivityTime = Date.now();
+      for (const comment of newComments) {
+        seenIds.add(comment.id);
+      }
 
       console.log(
         `\n${colors.green}=== NEW COMMENTS DETECTED [${formatTimestamp()}] ===${colors.reset}`
       );
       console.log(
-        `${colors.bright}Found ${newComments.length} new comment${newComments.length === 1 ? "" : "s"}${colors.reset}\n`
+        `${colors.bright}Found ${newComments.length} new comment${newComments.length === 1 ? "" : "s"}${colors.reset}`
       );
 
-      for (const comment of newComments) {
+      // Brief grace period to catch any stragglers from the same bot batch
+      console.log(
+        `${colors.dim}Waiting 5s for additional comments...${colors.reset}`
+      );
+      await sleep(5);
+
+      // Re-fetch to catch any comments posted during the grace period
+      const graceData = await fetchPRComments(
+        owner,
+        repo,
+        prNumber,
+        token,
+        proxyFetch
+      );
+      const graceProcessed = processComments(graceData);
+      const graceFiltered = filterComments(graceProcessed, options);
+      const lateComments = graceFiltered.filter((c) => !seenIds.has(c.id));
+
+      for (const comment of lateComments) {
         seenIds.add(comment.id);
+        newComments.push(comment);
+      }
+
+      if (lateComments.length > 0) {
+        console.log(
+          `${colors.bright}Caught ${lateComments.length} additional comment${lateComments.length === 1 ? "" : "s"}${colors.reset}`
+        );
+      }
+
+      console.log("");
+      for (const comment of newComments) {
         console.log(formatComment(comment));
         console.log("");
       }
@@ -294,7 +325,16 @@ async function watchForComments(context, options) {
       // JSON output for AI agent parsing
       console.log(`${colors.dim}--- JSON for processing ---${colors.reset}`);
       console.log(JSON.stringify(newComments, null, 2));
-      console.log(`${colors.dim}--- end JSON ---${colors.reset}\n`);
+      console.log(`${colors.dim}--- end JSON ---${colors.reset}`);
+
+      // Exit immediately so the caller can process and restart if needed
+      console.log(
+        `\n${colors.green}=== WATCH: EXITING WITH NEW COMMENTS ===${colors.reset}`
+      );
+      console.log(
+        `${colors.dim}Restart watcher after processing to catch further comments.${colors.reset}`
+      );
+      return;
     } else {
       const inactiveSeconds = Math.round(
         (Date.now() - lastActivityTime) / 1000
@@ -328,7 +368,7 @@ async function main() {
   const options = parseArgs();
 
   if (options.version) {
-    console.log("0.5.2");
+    console.log("0.6.0");
     process.exit(0);
   }