Comment review tool

entry

@@ -121638,7 +121638,6 @@ function computeModes() {
 3. Fetch review comments using ${ghPullfrogMcpName}/get_review_comments with \`pull_number\` and \`review_id\` from EVENT DATA. This returns \`commentsPath\` - read that file for full comment details with diff context. If EVENT DATA contains a \`triggerer\` field (indicating who requested fixes), you can pass \`approved_by\` to filter to only comments they approved with \u{1F44D}.
 
 4. Review the feedback provided. Understand each review comment and what changes are being requested.    
-  - **If there are no actionable comments** (e.g., the review is an approval with no specific feedback to address), do NOT post a progress comment. Simply exit without taking action.
 
 5. If the request requires understanding the codebase structure or conventions, gather relevant context. Read AGENTS.md if it exists.
 

mcp/reviewComments.ts

@@ -6,6 +6,8 @@ import type { ToolContext } from "./server.ts";
 import { execute, tool } from "./shared.ts";
 
 // fragment for nested replyTo (5 levels deep covers most threads)
+// because in_reply_to_id generally points to the top-level comment
+// this doesn't actually work as expected
 const REPLY_TO_FRAGMENT = `
   replyTo {
     databaseId

modes.ts

@@ -76,7 +76,6 @@ export function computeModes(): Mode[] {
 3. Fetch review comments using ${ghPullfrogMcpName}/get_review_comments with \`pull_number\` and \`review_id\` from EVENT DATA. This returns \`commentsPath\` - read that file for full comment details with diff context. If EVENT DATA contains a \`triggerer\` field (indicating who requested fixes), you can pass \`approved_by\` to filter to only comments they approved with 👍.
 
 4. Review the feedback provided. Understand each review comment and what changes are being requested.    
-  - **If there are no actionable comments** (e.g., the review is an approval with no specific feedback to address), do NOT post a progress comment. Simply exit without taking action.
 
 5. If the request requires understanding the codebase structure or conventions, gather relevant context. Read AGENTS.md if it exists.
 

scratch.ts

@@ -1,15 +0,0 @@
-import { spawnSync } from "child_process";
-import { existsSync } from "fs";
-
-function findCliPath(name: string): string | null {
-  const result = spawnSync("which", [name], { encoding: "utf-8" });
-  if (result.status === 0 && result.stdout) {
-    const cliPath = result.stdout.trim();
-    if (cliPath && existsSync(cliPath)) {
-      return cliPath;
-    }
-  }
-  return null;
-}
-
-console.log(findCliPath("codei"));