diffprism 1.3.2 → 1.4.0

package/README.md

@@ -89,7 +89,7 @@ Reviews are **one per repo**: opening a review for a repo that already has one u
 
 In a local review, **Ask agent now** in a line's comment form asks the agent while you're still reviewing: the agent's wait for your decision ends with your question, it answers in the thread, and goes back to waiting.
 
-PR reviews are opened with `diffprism review <PR URL>` or the dashboard — not by `open_review` — and your AI then works inside them with the tools above. Click a line to ask the agent about it; an agent listening with `wait_for_comments` answers in the thread.
+PR reviews are opened with `diffprism review <PR URL>` or the dashboard — not by `open_review` — and your AI then works inside them with the tools above. Click a line to ask the agent about it; an agent listening with `wait_for_comments` answers in the thread. Nothing starts an agent for you: if none has picked a question up, the thread says so and names the session to ask Claude Code about.
 
 When you're done, **Approve**, **Request changes** or **Comment** from the bar at the bottom: DiffPrism posts it to GitHub as a pull request review. Your threads are a conversation with the agent, so none of them goes to GitHub unless you tick it; a ticked thread posts your opening message as an inline review comment. The token comes from `GITHUB_TOKEN`, `gh auth token`, or `~/.diffprism/config.json`.
 

package/dist/bin.js

@@ -2,10 +2,10 @@
 import {
   isPrRef,
   parsePrRef
-} from "./chunk-EPU4F7WT.js";
+} from "./chunk-RFDSC6MC.js";
 import {
   demo
-} from "./chunk-XGQRRG2H.js";
+} from "./chunk-6P6QJPWT.js";
 import {
   COMMIT_GATE_DIFF_REF,
   DEFAULT_DIFF_REF,
@@ -26,10 +26,10 @@ import {
   recordError,
   startGlobalServer,
   submitReviewToServer
-} from "./chunk-USOWTWJD.js";
+} from "./chunk-SCQFIDIK.js";
 import {
   getDiff
-} from "./chunk-3GMPE2ZR.js";
+} from "./chunk-GGK6TIQZ.js";
 import "./chunk-DHCVZGHE.js";
 import "./chunk-JSBRDJBE.js";
 
@@ -518,6 +518,8 @@ The reviewer can also ask you questions on lines of the PR. Hold that conversati
 2. \`mcp__diffprism__reply\` \u2014 answer each thread, passing its \`annotation_id\`.
 3. Wait again. On \`timed_out\`, nothing new was said \u2014 keep waiting until the user tells you to stop.
 
+When no agent has read a question, the dashboard tells the reviewer so, and to ask you to answer their DiffPrism comments on a session id. When they do, run this loop with that \`session_id\`.
+
 A PR review and a working-copy review can be open for the same clone at once. If a tool reports more than one session, pass the \`session_id\` of the one you mean.
 
 ## Tool Reference
@@ -719,7 +721,7 @@ async function setupInteractive(flags) {
 }
 async function runDemo(dev) {
   console.log("");
-  const { demo: demo2 } = await import("./demo-M3O63OIX.js");
+  const { demo: demo2 } = await import("./demo-GHOE7YNU.js");
   await demo2({ dev });
 }
 async function setupBatch(flags) {

package/dist/{chunk-XGQRRG2H.js → chunk-6P6QJPWT.js}

@@ -1,10 +1,10 @@
 import {
   ensureServer,
   submitReviewToServer
-} from "./chunk-USOWTWJD.js";
+} from "./chunk-SCQFIDIK.js";
 import {
   parseDiff
-} from "./chunk-3GMPE2ZR.js";
+} from "./chunk-GGK6TIQZ.js";
 import {
   analyze
 } from "./chunk-DHCVZGHE.js";

package/dist/{chunk-3GMPE2ZR.js → chunk-GGK6TIQZ.js}

@@ -210,6 +210,27 @@ function getRepoRoot(options) {
     return null;
   }
 }
+function getGitHubRemotes(options) {
+  const cwd = options?.cwd ?? process.cwd();
+  let output;
+  try {
+    output = execSync("git remote -v", {
+      cwd,
+      encoding: "utf-8",
+      stdio: ["pipe", "pipe", "pipe"]
+    });
+  } catch {
+    return [];
+  }
+  const repos = /* @__PURE__ */ new Set();
+  for (const line of output.split("\n")) {
+    const match = /github\.com[:/]([^/\s]+)\/([^/\s]+?)(?:\.git)?\s/i.exec(line);
+    if (match) {
+      repos.add(`${match[1]}/${match[2]}`.toLowerCase());
+    }
+  }
+  return Array.from(repos);
+}
 
 // packages/git/src/parser.ts
 import path2 from "path";
@@ -456,6 +477,7 @@ export {
   listCommits,
   detectWorktree,
   getRepoRoot,
+  getGitHubRemotes,
   parseDiff,
   getDiff
 };

package/dist/{chunk-EPU4F7WT.js → chunk-RFDSC6MC.js}

@@ -1,6 +1,6 @@
 import {
   parseDiff
-} from "./chunk-3GMPE2ZR.js";
+} from "./chunk-GGK6TIQZ.js";
 import {
   analyze
 } from "./chunk-DHCVZGHE.js";

package/dist/{chunk-USOWTWJD.js → chunk-SCQFIDIK.js}

@@ -1,10 +1,11 @@
 import {
   getCurrentBranch,
   getDiff,
+  getGitHubRemotes,
   getRepoRoot,
   listBranches,
   listCommits
-} from "./chunk-3GMPE2ZR.js";
+} from "./chunk-GGK6TIQZ.js";
 import {
   analyze
 } from "./chunk-DHCVZGHE.js";
@@ -79,12 +80,28 @@ async function isServerAlive() {
   }
 }
 
+// packages/core/src/threads.ts
+function lastAuthor(annotation) {
+  const replies = annotation.replies ?? [];
+  return replies.length > 0 ? replies[replies.length - 1].author : annotation.author ?? "agent";
+}
+function awaitingAgent(annotation) {
+  return !annotation.dismissed && lastAuthor(annotation) === "reviewer";
+}
+function lastMessageAt(annotation) {
+  const replies = annotation.replies ?? [];
+  return replies.length > 0 ? replies[replies.length - 1].createdAt : annotation.createdAt;
+}
+function pickedUpByAgent(annotation, agentReadAt) {
+  return agentReadAt !== void 0 && agentReadAt > lastMessageAt(annotation);
+}
+
 // packages/core/src/build-info.ts
 import fs2 from "fs";
 import path2 from "path";
 import { fileURLToPath } from "url";
 function builtAt() {
-  return true ? 1789610217504 : null;
+  return true ? 1789619773143 : null;
 }
 function getBuildInfo() {
   let dir = path2.dirname(fileURLToPath(import.meta.url));
@@ -122,7 +139,7 @@ import path3 from "path";
 var ISSUES_NEW_URL = "https://github.com/CodeJonesW/diffprism/issues/new";
 var MAX_ERROR_CHARS = 1500;
 function currentVersion() {
-  return true ? "1.3.2" : "0.0.0-dev";
+  return true ? "1.4.0" : "0.0.0-dev";
 }
 function describeEnvironment() {
   return {
@@ -211,15 +228,6 @@ ${errorText}
   return `${ISSUES_NEW_URL}?${params.toString()}`;
 }
 
-// packages/core/src/threads.ts
-function lastAuthor(annotation) {
-  const replies = annotation.replies ?? [];
-  return replies.length > 0 ? replies[replies.length - 1].author : annotation.author ?? "agent";
-}
-function awaitingAgent(annotation) {
-  return !annotation.dismissed && lastAuthor(annotation) === "reviewer";
-}
-
 // packages/core/src/server-client.ts
 import { spawn } from "child_process";
 import fs4 from "fs";
@@ -393,7 +401,7 @@ async function waitForDecision(serverInfo, sessionId, maxWaitMs, pollIntervalMs
     if (result) {
       return result;
     }
-    const annotationsResponse = await fetch(`${base}/annotations`);
+    const annotationsResponse = await fetch(`${base}/annotations?reader=agent`);
     if (!annotationsResponse.ok) {
       throw new Error(`Could not read threads for ${sessionId}: server returned ${annotationsResponse.status}`);
     }
@@ -413,7 +421,7 @@ async function submitReviewToServer(serverInfo, diffRef, options = {}) {
   if (options.injectedPayload) {
     payload = options.injectedPayload;
   } else {
-    const { getDiff: getDiff2, getCurrentBranch: getCurrentBranch2, detectWorktree } = await import("./src-OZQ32NDN.js");
+    const { getDiff: getDiff2, getCurrentBranch: getCurrentBranch2, detectWorktree } = await import("./src-XT4XP6TC.js");
     const { analyze: analyze2 } = await import("./src-JMPTSU3P.js");
     const { diffSet, rawDiff } = getDiff2(diffRef, { cwd });
     if (diffSet.files.length === 0) {
@@ -800,6 +808,14 @@ var IDLE_SESSION_TTL_MS = 24 * 60 * 60 * 1e3;
 function touch(session) {
   session.lastActivityAt = Date.now();
 }
+function recordAgentRead(session) {
+  const previous = session.agentReadAt;
+  session.agentReadAt = Date.now();
+  touch(session);
+  if (session.annotations.some((a) => awaitingAgent(a) && !pickedUpByAgent(a, previous))) {
+    broadcastSessionUpdate(session);
+  }
+}
 var sessions = /* @__PURE__ */ new Map();
 function listedSummaries() {
   return Array.from(sessions.values()).filter((session) => session.closedAt === void 0).map(toSummary);
@@ -943,7 +959,8 @@ function toSummary(session) {
     hasNewChanges: session.hasNewChanges,
     needsAttention: needsAttention(session),
     diffRef: session.diffRef,
-    source: session.source
+    source: session.source,
+    agentReadAt: session.agentReadAt
   };
 }
 function readBody(req) {
@@ -1260,7 +1277,7 @@ async function handleApiRequest(req, res) {
         fetchPullRequest,
         fetchPullRequestDiff,
         normalizePr
-      } = await import("./src-DNNNQ2RD.js");
+      } = await import("./src-LLACFIIT.js");
       if (!isPrRef(prUrl)) {
         jsonResponse(res, 400, {
           error: "Invalid PR URL. Expected https://github.com/owner/repo/pull/123 or owner/repo#123"
@@ -1283,20 +1300,7 @@ async function handleApiRequest(req, res) {
         fetchPullRequestDiff(client, owner, repo, prNumber)
       ]);
       const normalized = normalizePr(rawDiff, prMetadata);
-      let localRepoPath = null;
-      try {
-        const { execSync } = await import("child_process");
-        const remoteOutput = execSync("git remote -v", {
-          cwd: process.cwd(),
-          encoding: "utf-8",
-          stdio: ["pipe", "pipe", "pipe"]
-        });
-        const repoPattern = new RegExp(`github\\.com[:/]${owner}/${repo}(\\.git)?\\s`, "i");
-        if (repoPattern.test(remoteOutput)) {
-          localRepoPath = process.cwd();
-        }
-      } catch {
-      }
+      const localRepoPath = getGitHubRemotes({ cwd: process.cwd() }).includes(`${owner}/${repo}`.toLowerCase()) ? process.cwd() : null;
       const { session, reused } = openSession({
         key: `pr:${owner}/${repo}#${prNumber}`.toLowerCase(),
         repoRoot: localRepoPath ? getRepoRoot({ cwd: localRepoPath }) ?? localRepoPath : null,
@@ -1373,7 +1377,13 @@ async function handleApiRequest(req, res) {
       return true;
     }
     const repoRoot = getRepoRoot({ cwd: rawPath }) ?? path7.resolve(rawPath);
-    const matches = Array.from(sessions.values()).filter((session) => session.closedAt === void 0 && session.repoRoot === repoRoot).map(toSummary);
+    const remotes = getGitHubRemotes({ cwd: repoRoot });
+    const matches = Array.from(sessions.values()).filter((session) => {
+      if (session.closedAt !== void 0) return false;
+      if (session.repoRoot === repoRoot) return true;
+      const pr = session.payload.metadata.githubPr;
+      return pr !== void 0 && remotes.includes(`${pr.owner}/${pr.repo}`.toLowerCase());
+    }).map(toSummary);
     jsonResponse(res, 200, { repoRoot, sessions: matches });
     return true;
   }
@@ -1527,6 +1537,9 @@ async function handleApiRequest(req, res) {
       jsonResponse(res, 404, { error: "Session not found" });
       return true;
     }
+    if (new URL(req.url ?? "/", "http://localhost").searchParams.get("reader") === "agent") {
+      recordAgentRead(session);
+    }
     jsonResponse(res, 200, { annotations: session.annotations });
     return true;
   }
@@ -1592,7 +1605,7 @@ async function handleApiRequest(req, res) {
       jsonResponse(res, 400, { error: built.error });
       return true;
     }
-    const { resolveGitHubToken, createGitHubClient, submitGitHubReview } = await import("./src-DNNNQ2RD.js");
+    const { resolveGitHubToken, createGitHubClient, submitGitHubReview } = await import("./src-LLACFIIT.js");
     let token;
     try {
       token = resolveGitHubToken();
@@ -2072,6 +2085,7 @@ function mcpToolPermission(name) {
 export {
   readServerFile,
   isServerAlive,
+  awaitingAgent,
   getBuildInfo,
   describeVersion,
   DEFAULT_DIFF_REF,
@@ -2083,7 +2097,6 @@ export {
   REPORT_HINT,
   buildFeedbackUrl,
   startGlobalServer,
-  awaitingAgent,
   ensureServer,
   ReviewTimeoutError,
   ReviewerAskedError,

package/dist/{demo-M3O63OIX.js → demo-GHOE7YNU.js}

@@ -1,8 +1,8 @@
 import {
   demo
-} from "./chunk-XGQRRG2H.js";
-import "./chunk-USOWTWJD.js";
-import "./chunk-3GMPE2ZR.js";
+} from "./chunk-6P6QJPWT.js";
+import "./chunk-SCQFIDIK.js";
+import "./chunk-GGK6TIQZ.js";
 import "./chunk-DHCVZGHE.js";
 import "./chunk-JSBRDJBE.js";
 export {

package/dist/mcp-server.js

@@ -1,6 +1,6 @@
 import {
   isPrRef
-} from "./chunk-EPU4F7WT.js";
+} from "./chunk-RFDSC6MC.js";
 import {
   DEFAULT_DIFF_REF,
   DIFF_REF_DESCRIPTION,
@@ -14,10 +14,10 @@ import {
   recordError,
   submitReviewToServer,
   waitForDecision
-} from "./chunk-USOWTWJD.js";
+} from "./chunk-SCQFIDIK.js";
 import {
   getDiff
-} from "./chunk-3GMPE2ZR.js";
+} from "./chunk-GGK6TIQZ.js";
 import {
   analyze
 } from "./chunk-DHCVZGHE.js";
@@ -73,7 +73,7 @@ ${candidates}`
 }
 async function readThreads(serverInfo, sessionId) {
   const response = await fetch(
-    `http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}/annotations`
+    `http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}/annotations?reader=agent`
   );
   if (!response.ok) {
     return null;
@@ -354,7 +354,7 @@ ${REPORT_HINT}`);
     async ({ session_id, repo_path }) => withSession({ session_id, repo_path }, async ({ serverInfo, sessionId }) => {
       const [sessionResponse, annotationsResponse] = await Promise.all([
         fetch(`http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}`),
-        fetch(`http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}/annotations`)
+        fetch(`http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}/annotations?reader=agent`)
       ]);
       if (!sessionResponse.ok) {
         return toolError(`Session not found: ${sessionId}`);

package/dist/{src-DNNNQ2RD.js → src-LLACFIIT.js}

@@ -7,8 +7,8 @@ import {
   parsePrRef,
   resolveGitHubToken,
   submitGitHubReview
-} from "./chunk-EPU4F7WT.js";
-import "./chunk-3GMPE2ZR.js";
+} from "./chunk-RFDSC6MC.js";
+import "./chunk-GGK6TIQZ.js";
 import "./chunk-DHCVZGHE.js";
 import "./chunk-JSBRDJBE.js";
 export {

package/dist/{src-OZQ32NDN.js → src-XT4XP6TC.js}

@@ -3,17 +3,19 @@ import {
   getCurrentBranch,
   getDiff,
   getGitDiff,
+  getGitHubRemotes,
   getRepoRoot,
   listBranches,
   listCommits,
   parseDiff
-} from "./chunk-3GMPE2ZR.js";
+} from "./chunk-GGK6TIQZ.js";
 import "./chunk-JSBRDJBE.js";
 export {
   detectWorktree,
   getCurrentBranch,
   getDiff,
   getGitDiff,
+  getGitHubRemotes,
   getRepoRoot,
   listBranches,
   listCommits,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "diffprism",
-  "version": "1.3.2",
+  "version": "1.4.0",
   "type": "module",
   "description": "Local-first code review tool for agent-generated code changes",
   "bin": {