diffprism 1.2.0 → 1.3.1

package/README.md

@@ -218,6 +218,26 @@ pnpm run build
 pnpm cli review --staged            # Run CLI from source
 ```
 
+### Dogfooding a checkout
+
+To try unreleased changes in another project without publishing, point your global `diffprism` at a checkout:
+
+```bash
+git clone https://github.com/CodeJonesW/diffprism.git diffprism-dogfood
+cd diffprism-dogfood && pnpm install && pnpm build && npm link
+diffprism --version                 # 1.2.0 (dev build — /path/to/diffprism-dogfood)
+```
+
+Then, for each change you want to try:
+
+```bash
+git fetch && git checkout <branch> && git pull && pnpm build
+```
+
+The next `diffprism` command — a review, the commit gate — replaces the running server with the new build, unless a review is open in it. The dashboard picks up a rebuilt UI on reload. Go back to the release with `npm install -g diffprism@latest`.
+
+A server is only ever replaced by a **newer** build. Claude Code keeps `diffprism serve` running on the build it started with until you restart it, so restart Claude Code to put its MCP tools on the new build too.
+
 ### Project Structure
 
 ```

package/dist/bin.js

@@ -5,7 +5,7 @@ import {
 } from "./chunk-EPU4F7WT.js";
 import {
   demo
-} from "./chunk-FONA4U6B.js";
+} from "./chunk-PJYDCLBP.js";
 import {
   COMMIT_GATE_DIFF_REF,
   DEFAULT_DIFF_REF,
@@ -26,7 +26,7 @@ import {
   recordError,
   startGlobalServer,
   submitReviewToServer
-} from "./chunk-F6O5YP3I.js";
+} from "./chunk-35YVBTBH.js";
 import {
   getDiff
 } from "./chunk-3GMPE2ZR.js";
@@ -79,6 +79,9 @@ ${REPORT_HINT}`);
   }
   const where = data.annotation ? ` on ${data.annotation.file}:${data.annotation.line}` : "";
   console.log(`Replied${where}. It shows in the review now.`);
+  console.log(
+    "The review is still open. Go back to waiting for the decision now: run the same command that opened it again (`git commit` or `diffprism review`). It returns the decision, or the reviewer's next question. Don't ask them in the terminal."
+  );
 }
 function fail(message2) {
   console.error(message2);
@@ -136,7 +139,7 @@ ${REPORT_HINT}`);
     if (err instanceof ReviewerAskedError) {
       printQuestions(err.sessionId, err.threads);
       fail2(
-        "Commit blocked: the reviewer asked you something before deciding. Answer each question with the command under it \u2014 change the code too if that's what they asked for \u2014 then run git commit again. The review stays open and the decision still comes."
+        "Commit blocked: the reviewer asked you something before deciding. Answer each question with the command under it \u2014 change the code too if that's what they asked for \u2014 then run git commit again straight away to keep waiting. The review stays open, the reviewer may ask more, and the decision still comes. Don't stop to ask them in the terminal."
       );
       return;
     }
@@ -369,7 +372,7 @@ async function reviewLocalFlow(diffRef, flags) {
     if (err instanceof ReviewerAskedError) {
       printQuestions(err.sessionId, err.threads);
       console.error(
-        "The reviewer asked something before deciding. Answer each question with the command under it, then run diffprism review again \u2014 the review stays open."
+        "The reviewer asked something before deciding. Answer each question with the command under it, then run diffprism review again straight away to keep waiting \u2014 the review stays open, and the reviewer may ask more. Don't stop to ask them in the terminal."
       );
       process.exit(1);
     }
@@ -716,7 +719,7 @@ async function setupInteractive(flags) {
 }
 async function runDemo(dev) {
   console.log("");
-  const { demo: demo2 } = await import("./demo-VNWYEDZ3.js");
+  const { demo: demo2 } = await import("./demo-WIP5MSCH.js");
   await demo2({ dev });
 }
 async function setupBatch(flags) {

package/dist/{chunk-F6O5YP3I.js → chunk-35YVBTBH.js}

@@ -79,15 +79,13 @@ async function isServerAlive() {
   }
 }
 
-// packages/core/src/diff-scope.ts
-var DEFAULT_DIFF_REF = "working-copy";
-var COMMIT_GATE_DIFF_REF = "staged";
-var DIFF_REF_DESCRIPTION = 'Which changes to review. "working-copy" (the default): everything not yet committed, staged and unstaged shown as separate groups. "staged": only what the next commit would contain. "unstaged": only edits not yet staged. Or a ref range such as "HEAD~3..HEAD" or "main..feature".';
-
 // packages/core/src/build-info.ts
 import fs2 from "fs";
 import path2 from "path";
 import { fileURLToPath } from "url";
+function builtAt() {
+  return true ? 1789610123399 : null;
+}
 function getBuildInfo() {
   let dir = path2.dirname(fileURLToPath(import.meta.url));
   while (dir !== path2.dirname(dir)) {
@@ -112,6 +110,11 @@ function describeVersion(version) {
   return info.dev ? `${version} (dev build \u2014 ${info.root})` : version;
 }
 
+// packages/core/src/diff-scope.ts
+var DEFAULT_DIFF_REF = "working-copy";
+var COMMIT_GATE_DIFF_REF = "staged";
+var DIFF_REF_DESCRIPTION = 'Which changes to review. "working-copy" (the default): everything not yet committed, staged and unstaged shown as separate groups. "staged": only what the next commit would contain. "unstaged": only edits not yet staged. Or a ref range such as "HEAD~3..HEAD" or "main..feature".';
+
 // packages/core/src/feedback.ts
 import fs3 from "fs";
 import os2 from "os";
@@ -119,7 +122,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.2.0" : "0.0.0-dev";
+  return true ? "1.3.1" : "0.0.0-dev";
 }
 function describeEnvironment() {
   return {
@@ -226,7 +229,22 @@ import { fileURLToPath as fileURLToPath2 } from "url";
 async function ensureServer(options = {}) {
   const existing = await isServerAlive();
   if (existing) {
-    return existing;
+    const decision = await decideOnRunningServer(existing, builtAt());
+    if (decision.action === "keep") {
+      return existing;
+    }
+    if (decision.action === "keep-busy") {
+      if (!options.silent) {
+        console.error(
+          `The DiffPrism server (PID ${existing.pid}) runs an older build, but ${decision.openReviews} review${decision.openReviews === 1 ? " is" : "s are"} open in it, so it stays up. Finish or close ${decision.openReviews === 1 ? "it" : "them"} and the next command starts the new build \u2014 or run \`diffprism server stop\`.`
+        );
+      }
+      return existing;
+    }
+    if (!options.silent) {
+      console.error(`Replacing the DiffPrism server (PID ${existing.pid}): it runs an older build than this command.`);
+    }
+    await stopServer(existing);
   }
   const spawnArgs = options.spawnCommand ?? buildDefaultSpawnCommand(options);
   const logDir = path4.join(os3.homedir(), ".diffprism");
@@ -256,6 +274,44 @@ async function ensureServer(options = {}) {
     `DiffPrism server failed to start within ${timeoutMs / 1e3}s. Check logs at ${logPath}`
   );
 }
+async function decideOnRunningServer(server, ownBuiltAt) {
+  if (ownBuiltAt === null) return { action: "keep" };
+  if (server.builtAt !== void 0 && server.builtAt >= ownBuiltAt) return { action: "keep" };
+  const response = await fetch(`http://localhost:${server.httpPort}/api/reviews`);
+  if (!response.ok) {
+    throw new Error(`Could not list reviews on the running DiffPrism server: it returned ${response.status}`);
+  }
+  const { sessions: sessions2 } = await response.json();
+  const openReviews = sessions2.filter((s) => s.status === "pending" || s.status === "in_review").length;
+  return openReviews > 0 ? { action: "keep-busy", openReviews } : { action: "replace" };
+}
+async function stopServer(server, timeoutMs = 5e3) {
+  try {
+    process.kill(server.pid, "SIGTERM");
+  } catch {
+  }
+  const start = Date.now();
+  while (Date.now() - start < timeoutMs) {
+    if (!isProcessRunning(server.pid)) {
+      if (readServerFile()?.pid === server.pid) {
+        removeServerFile();
+      }
+      return;
+    }
+    await new Promise((resolve) => setTimeout(resolve, 100));
+  }
+  throw new Error(
+    `The old DiffPrism server (PID ${server.pid}) did not stop within ${timeoutMs / 1e3}s. Stop it with \`diffprism server stop\` and try again.`
+  );
+}
+function isProcessRunning(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
 function buildDefaultSpawnCommand(options) {
   const thisFile = fileURLToPath2(import.meta.url);
   const thisDir = path4.dirname(thisFile);
@@ -861,6 +917,7 @@ var clientSessions = /* @__PURE__ */ new Map();
 var sessionWatchers = /* @__PURE__ */ new Map();
 var watchSchedule = DEFAULT_WATCH_SCHEDULE;
 var reopenBrowserIfNeeded = null;
+var DASHBOARD_RECONNECT_GRACE_MS = 3e3;
 var serverUiUrl = null;
 function toSummary(session) {
   const { payload } = session;
@@ -1741,6 +1798,8 @@ async function startGlobalServer(options = {}) {
   const {
     httpPort: preferredHttpPort = 24680,
     wsPort: preferredWsPort = 24681,
+    uiPort: preferredUiPort = 24682,
+    reconnectGraceMs = DASHBOARD_RECONNECT_GRACE_MS,
     silent = false,
     dev = false,
     pollInterval = DEFAULT_WATCH_SCHEDULE.viewedMs,
@@ -1763,11 +1822,11 @@ async function startGlobalServer(options = {}) {
   let uiHttpServer = null;
   let viteServer = null;
   if (dev) {
-    uiPort = await getPort();
+    uiPort = await getPort({ port: preferredUiPort });
     const uiRoot = resolveUiRoot();
     viteServer = await startViteDevServer(uiRoot, uiPort, silent);
   } else {
-    uiPort = await getPort();
+    uiPort = await getPort({ port: preferredUiPort });
     const uiDist = resolveUiDist();
     uiHttpServer = await createStaticServer(uiDist, uiPort);
   }
@@ -1913,6 +1972,10 @@ async function startGlobalServer(options = {}) {
     pid: process.pid,
     startedAt: Date.now()
   };
+  const serverBuiltAt = builtAt();
+  if (serverBuiltAt !== null) {
+    serverInfo.builtAt = serverBuiltAt;
+  }
   writeServerFile(serverInfo);
   if (!silent) {
     console.log(`
@@ -1930,9 +1993,23 @@ Waiting for reviews...
   if (openBrowser) {
     await open(uiUrl);
   }
+  const serverStartedAt = Date.now();
+  const pendingOpens = /* @__PURE__ */ new Set();
   reopenBrowserIfNeeded = () => {
-    if (!hasConnectedClients()) {
-      open(uiUrl);
+    const openIfUnwatched = () => {
+      if (reopenBrowserIfNeeded && !hasConnectedClients()) {
+        open(uiUrl);
+      }
+    };
+    const wait = serverStartedAt + reconnectGraceMs - Date.now();
+    if (wait > 0) {
+      const timer = setTimeout(() => {
+        pendingOpens.delete(timer);
+        openIfUnwatched();
+      }, wait);
+      pendingOpens.add(timer);
+    } else {
+      openIfUnwatched();
     }
   };
   async function stop() {
@@ -1947,6 +2024,8 @@ Waiting for reviews...
     }
     clientSessions.clear();
     sessions.clear();
+    for (const timer of pendingOpens) clearTimeout(timer);
+    pendingOpens.clear();
     reopenBrowserIfNeeded = null;
     serverUiUrl = null;
     await new Promise((resolve) => {
@@ -1993,11 +2072,11 @@ function mcpToolPermission(name) {
 export {
   readServerFile,
   isServerAlive,
+  getBuildInfo,
+  describeVersion,
   DEFAULT_DIFF_REF,
   COMMIT_GATE_DIFF_REF,
   DIFF_REF_DESCRIPTION,
-  getBuildInfo,
-  describeVersion,
   currentVersion,
   recordError,
   readLastError,

package/dist/{chunk-FONA4U6B.js → chunk-PJYDCLBP.js}

@@ -1,7 +1,7 @@
 import {
   ensureServer,
   submitReviewToServer
-} from "./chunk-F6O5YP3I.js";
+} from "./chunk-35YVBTBH.js";
 import {
   parseDiff
 } from "./chunk-3GMPE2ZR.js";

package/dist/{demo-VNWYEDZ3.js → demo-WIP5MSCH.js}

@@ -1,7 +1,7 @@
 import {
   demo
-} from "./chunk-FONA4U6B.js";
-import "./chunk-F6O5YP3I.js";
+} from "./chunk-PJYDCLBP.js";
+import "./chunk-35YVBTBH.js";
 import "./chunk-3GMPE2ZR.js";
 import "./chunk-DHCVZGHE.js";
 import "./chunk-JSBRDJBE.js";

package/dist/mcp-server.js

@@ -14,7 +14,7 @@ import {
   recordError,
   submitReviewToServer,
   waitForDecision
-} from "./chunk-F6O5YP3I.js";
+} from "./chunk-35YVBTBH.js";
 import {
   getDiff
 } from "./chunk-3GMPE2ZR.js";

package/package.json

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