diffprism 0.15.0 → 0.17.0

package/README.md

@@ -15,6 +15,8 @@ DiffPrism gives you a visual review step for AI-written code — stage your chan
 - **Keyboard shortcuts** — `j`/`k` navigate files, `s` cycles file status
 - **Three-way decisions** — approve, request changes, or approve with comments
 - **Branch display** — current git branch shown in the review header
+- **Global server mode** — `diffprism server` runs a persistent multi-session review server, multiple agents post reviews to one browser tab
+- **Multi-session UI** — session list with status badges, branch info, file counts, and change stats when running in server mode
 
 ## Quick Start
 
@@ -85,6 +87,32 @@ When `diffprism watch` is running:
 
 Stop the watcher with `Ctrl+C`.
 
+### Global Server Mode (multi-session)
+
+Run a persistent server that accepts reviews from multiple Claude Code sessions and displays them in one browser tab:
+
+```bash
+# Start the global server (auto-runs global setup if needed)
+diffprism server
+
+# Check status and list active sessions
+diffprism server status
+
+# Stop the server
+diffprism server stop
+```
+
+When the global server is running, MCP tools automatically detect it and route reviews there instead of opening ephemeral browser tabs. Each review appears as a session in the multi-session UI — click to switch between them.
+
+**Global setup** (optional, `diffprism server` runs this automatically):
+
+```bash
+# Configure skill + permissions globally (no git repo required)
+diffprism setup --global
+```
+
+Per-project MCP registration (`.mcp.json`) is still needed via `diffprism setup` in each project.
+
 ## MCP Tool Reference
 
 The MCP server exposes three tools:
@@ -162,12 +190,12 @@ npx tsc --noEmit -p packages/core/tsconfig.json # Type-check a package
 ### Project Structure
 
 ```
-packages/core       — Shared types, pipeline orchestrator, WebSocket bridge
+packages/core       — Shared types, pipeline orchestrator, WebSocket bridge, global server
 packages/git        — Git diff extraction + unified diff parser
 packages/analysis   — Deterministic review briefing (complexity, test gaps, patterns)
-packages/ui         — React 19 + Vite 6 + Tailwind + Zustand diff viewer
-packages/mcp-server — MCP tool server (open_review)
-cli/                — Commander CLI entry point
+packages/ui         — React 19 + Vite 6 + Tailwind + Zustand diff viewer + session list
+packages/mcp-server — MCP tool server, auto-routes to global server when available
+cli/                — Commander CLI (review, serve, setup, server commands)
 ```
 
 ### Requirements

package/dist/bin.js

@@ -6,7 +6,7 @@ import {
   startGlobalServer,
   startReview,
   startWatch
-} from "./chunk-NGHUHDAM.js";
+} from "./chunk-4WN4FIY4.js";
 
 // cli/src/index.ts
 import { Command } from "commander";
@@ -111,7 +111,13 @@ The tool blocks until the user submits their review in the browser. When it retu
 ### 5. Error Handling
 
 If the \`mcp__diffprism__open_review\` tool is not available:
-- Tell the user: "The DiffPrism MCP server isn't configured. Run \`npx diffprism start\` to set it up, then restart Claude Code."
+- Tell the user: "The DiffPrism MCP server isn't configured. Run \`npx diffprism setup\` to set it up, then restart Claude Code."
+
+## Global Server Mode
+
+When a global DiffPrism server is running (\`diffprism server\`), the MCP tools automatically detect it and route reviews there instead of opening a new browser tab each time. The review appears in the server's multi-session UI at the existing browser tab.
+
+This is transparent \u2014 the same \`open_review\`, \`update_review_context\`, and \`get_review_result\` tools work the same way. No changes to the workflow are needed.
 
 ## Watch Mode: Waiting for Review Feedback
 
@@ -179,8 +185,8 @@ function setupMcpJson(gitRoot, force) {
   writeJsonFile(filePath, { ...existing, mcpServers: servers });
   return { action, filePath };
 }
-function setupClaudeSettings(gitRoot, force) {
-  const filePath = path.join(gitRoot, ".claude", "settings.json");
+function setupClaudeSettings(baseDir, force) {
+  const filePath = path.join(baseDir, ".claude", "settings.json");
   const existing = readJsonFile(filePath);
   const permissions = existing.permissions ?? {};
   const allow = permissions.allow ?? [];
@@ -324,54 +330,59 @@ async function setupGitignore(gitRoot) {
   return { action: "created", filePath };
 }
 async function setup(flags) {
+  const force = flags.force ?? false;
+  const global = flags.global ?? false;
+  const quiet = flags.quiet ?? false;
+  const result = { created: [], updated: [], skipped: [] };
+  const home = os.homedir();
+  if (global) {
+    if (!quiet) {
+      console.log("Setting up DiffPrism globally...\n");
+    }
+    const skill2 = setupSkill("", true, force);
+    result[skill2.action].push(skill2.filePath);
+    const settings2 = setupClaudeSettings(home, force);
+    result[settings2.action].push(settings2.filePath);
+    if (!quiet) {
+      printSummary(result, home);
+      console.log("\n\u2713 DiffPrism configured globally.\n");
+      console.log("Next steps:");
+      console.log("  1. Run `diffprism server` to start the global review server");
+      console.log("  2. In each project, run `diffprism setup` to register the MCP server");
+      console.log("  3. Use /review in Claude Code to review your changes\n");
+    }
+    return result;
+  }
   const gitRoot = findGitRoot(process.cwd());
   if (!gitRoot) {
     console.error(
       "Error: Not in a git repository. Run this command from inside a git project."
     );
+    console.error(
+      "Tip: Use `diffprism setup --global` to configure DiffPrism globally without a git repo."
+    );
     process.exit(1);
     return { created: [], updated: [], skipped: [] };
   }
-  const force = flags.force ?? false;
-  const global = flags.global ?? false;
-  const quiet = flags.quiet ?? false;
   if (!quiet) {
     console.log("Setting up DiffPrism for Claude Code...\n");
   }
-  const result = { created: [], updated: [], skipped: [] };
   const gitignore = await setupGitignore(gitRoot);
   result[gitignore.action].push(gitignore.filePath);
   const mcp = setupMcpJson(gitRoot, force);
-  result[mcp.action === "skipped" ? "skipped" : mcp.action === "created" ? "created" : "updated"].push(mcp.filePath);
+  result[mcp.action].push(mcp.filePath);
   const settings = setupClaudeSettings(gitRoot, force);
-  result[settings.action === "skipped" ? "skipped" : settings.action === "created" ? "created" : "updated"].push(settings.filePath);
+  result[settings.action].push(settings.filePath);
   const cleaned = cleanDiffprismHooks(gitRoot);
   if (cleaned.removed > 0 && !quiet) {
     console.log(`  Cleaned ${cleaned.removed} stale hook(s)`);
   }
   const hook = setupStopHook(gitRoot, force);
-  result[hook.action === "skipped" ? "skipped" : hook.action === "created" ? "created" : "updated"].push(hook.filePath);
-  const skill = setupSkill(gitRoot, global, force);
-  result[skill.action === "skipped" ? "skipped" : skill.action === "created" ? "created" : "updated"].push(skill.filePath);
+  result[hook.action].push(hook.filePath);
+  const skill = setupSkill(gitRoot, false, force);
+  result[skill.action].push(skill.filePath);
   if (!quiet) {
-    if (result.created.length > 0) {
-      console.log("Created:");
-      for (const f of result.created) {
-        console.log(`  + ${path.relative(gitRoot, f)}`);
-      }
-    }
-    if (result.updated.length > 0) {
-      console.log("Updated:");
-      for (const f of result.updated) {
-        console.log(`  ~ ${path.relative(gitRoot, f)}`);
-      }
-    }
-    if (result.skipped.length > 0) {
-      console.log("Skipped (already configured):");
-      for (const f of result.skipped) {
-        console.log(`  - ${path.relative(gitRoot, f)}`);
-      }
-    }
+    printSummary(result, gitRoot);
     console.log("\n\u2713 DiffPrism configured for Claude Code.\n");
     console.log("Next steps:");
     console.log("  1. Restart Claude Code to pick up the MCP configuration");
@@ -380,6 +391,41 @@ async function setup(flags) {
   }
   return result;
 }
+function printSummary(result, baseDir) {
+  if (result.created.length > 0) {
+    console.log("Created:");
+    for (const f of result.created) {
+      console.log(`  + ${path.relative(baseDir, f) || f}`);
+    }
+  }
+  if (result.updated.length > 0) {
+    console.log("Updated:");
+    for (const f of result.updated) {
+      console.log(`  ~ ${path.relative(baseDir, f) || f}`);
+    }
+  }
+  if (result.skipped.length > 0) {
+    console.log("Skipped (already configured):");
+    for (const f of result.skipped) {
+      console.log(`  - ${path.relative(baseDir, f) || f}`);
+    }
+  }
+}
+function isGlobalSetupDone() {
+  const home = os.homedir();
+  const skillPath = path.join(home, ".claude", "skills", "review", "SKILL.md");
+  const settingsPath = path.join(home, ".claude", "settings.json");
+  if (!fs.existsSync(skillPath)) return false;
+  const settings = readJsonFile(settingsPath);
+  const permissions = settings.permissions ?? {};
+  const allow = permissions.allow ?? [];
+  const toolNames = [
+    "mcp__diffprism__open_review",
+    "mcp__diffprism__update_review_context",
+    "mcp__diffprism__get_review_result"
+  ];
+  return toolNames.every((t) => allow.includes(t));
+}
 
 // cli/src/commands/start.ts
 async function start(ref, flags) {
@@ -502,6 +548,11 @@ async function server(flags) {
     process.exit(1);
     return;
   }
+  if (!isGlobalSetupDone()) {
+    console.log("Running global setup...\n");
+    await setup({ global: true, quiet: false });
+    console.log("");
+  }
   const httpPort = flags.port ? parseInt(flags.port, 10) : void 0;
   const wsPort = flags.wsPort ? parseInt(flags.wsPort, 10) : void 0;
   try {
@@ -578,13 +629,13 @@ async function serverStop() {
 
 // cli/src/index.ts
 var program = new Command();
-program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.15.0" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.17.0" : "0.0.0-dev");
 program.command("review [ref]").description("Open a browser-based diff review").option("--staged", "Review staged changes").option("--unstaged", "Review unstaged changes").option("-t, --title <title>", "Review title").option("--dev", "Use Vite dev server with HMR instead of static files").action(review);
 program.command("start [ref]").description("Set up DiffPrism and start watching for changes").option("--staged", "Watch staged changes").option("--unstaged", "Watch unstaged changes").option("-t, --title <title>", "Review title").option("--interval <ms>", "Poll interval in milliseconds (default: 1000)").option("--dev", "Use Vite dev server with HMR instead of static files").option("--global", "Install skill globally (~/.claude/skills/)").option("--force", "Overwrite existing configuration files").action(start);
 program.command("watch [ref]").description("Start a persistent diff watcher with live-updating browser UI").option("--staged", "Watch staged changes").option("--unstaged", "Watch unstaged changes").option("-t, --title <title>", "Review title").option("--interval <ms>", "Poll interval in milliseconds (default: 1000)").option("--dev", "Use Vite dev server with HMR instead of static files").action(watch);
 program.command("notify-stop").description("Signal the watch server to refresh (used by Claude Code hooks)").action(notifyStop);
 program.command("serve").description("Start the MCP server for Claude Code integration").action(serve);
-program.command("setup").description("Configure DiffPrism for Claude Code integration").option("--global", "Install skill globally (~/.claude/skills/)").option("--force", "Overwrite existing configuration files").action((flags) => {
+program.command("setup").description("Configure DiffPrism for Claude Code integration").option("--global", "Configure globally (skill + permissions, no git repo required)").option("--force", "Overwrite existing configuration files").action((flags) => {
   setup(flags);
 });
 var serverCmd = program.command("server").description("Start the global DiffPrism server for multi-session reviews").option("-p, --port <port>", "HTTP API port (default: 24680)").option("--ws-port <port>", "WebSocket port (default: 24681)").option("--dev", "Use Vite dev server with HMR instead of static files").action(server);

package/dist/{chunk-NGHUHDAM.js → chunk-4WN4FIY4.js}

@@ -1,7 +1,3 @@
-// packages/core/src/pipeline.ts
-import getPort from "get-port";
-import open from "open";
-
 // packages/git/src/local.ts
 import { execSync } from "child_process";
 import { readFileSync } from "fs";
@@ -717,6 +713,10 @@ function analyze(diffSet) {
   };
 }
 
+// packages/core/src/pipeline.ts
+import getPort from "get-port";
+import open from "open";
+
 // packages/core/src/ws-bridge.ts
 import { WebSocketServer, WebSocket } from "ws";
 function createWsBridge(port) {
@@ -1061,6 +1061,76 @@ function consumeReviewResult(cwd) {
   }
 }
 
+// packages/core/src/server-file.ts
+import fs3 from "fs";
+import path5 from "path";
+import os from "os";
+function serverDir() {
+  return path5.join(os.homedir(), ".diffprism");
+}
+function serverFilePath() {
+  return path5.join(serverDir(), "server.json");
+}
+function isPidAlive2(pid) {
+  try {
+    process.kill(pid, 0);
+    return true;
+  } catch {
+    return false;
+  }
+}
+function writeServerFile(info) {
+  const dir = serverDir();
+  if (!fs3.existsSync(dir)) {
+    fs3.mkdirSync(dir, { recursive: true });
+  }
+  fs3.writeFileSync(serverFilePath(), JSON.stringify(info, null, 2) + "\n");
+}
+function readServerFile() {
+  const filePath = serverFilePath();
+  if (!fs3.existsSync(filePath)) {
+    return null;
+  }
+  try {
+    const raw = fs3.readFileSync(filePath, "utf-8");
+    const info = JSON.parse(raw);
+    if (!isPidAlive2(info.pid)) {
+      fs3.unlinkSync(filePath);
+      return null;
+    }
+    return info;
+  } catch {
+    return null;
+  }
+}
+function removeServerFile() {
+  try {
+    const filePath = serverFilePath();
+    if (fs3.existsSync(filePath)) {
+      fs3.unlinkSync(filePath);
+    }
+  } catch {
+  }
+}
+async function isServerAlive() {
+  const info = readServerFile();
+  if (!info) {
+    return null;
+  }
+  try {
+    const response = await fetch(`http://localhost:${info.httpPort}/api/status`, {
+      signal: AbortSignal.timeout(2e3)
+    });
+    if (response.ok) {
+      return info;
+    }
+    return null;
+  } catch {
+    removeServerFile();
+    return null;
+  }
+}
+
 // packages/core/src/watch.ts
 import { createHash } from "crypto";
 import getPort2 from "get-port";
@@ -1375,78 +1445,6 @@ import { randomUUID } from "crypto";
 import getPort3 from "get-port";
 import open3 from "open";
 import { WebSocketServer as WebSocketServer3, WebSocket as WebSocket3 } from "ws";
-
-// packages/core/src/server-file.ts
-import fs3 from "fs";
-import path5 from "path";
-import os from "os";
-function serverDir() {
-  return path5.join(os.homedir(), ".diffprism");
-}
-function serverFilePath() {
-  return path5.join(serverDir(), "server.json");
-}
-function isPidAlive2(pid) {
-  try {
-    process.kill(pid, 0);
-    return true;
-  } catch {
-    return false;
-  }
-}
-function writeServerFile(info) {
-  const dir = serverDir();
-  if (!fs3.existsSync(dir)) {
-    fs3.mkdirSync(dir, { recursive: true });
-  }
-  fs3.writeFileSync(serverFilePath(), JSON.stringify(info, null, 2) + "\n");
-}
-function readServerFile() {
-  const filePath = serverFilePath();
-  if (!fs3.existsSync(filePath)) {
-    return null;
-  }
-  try {
-    const raw = fs3.readFileSync(filePath, "utf-8");
-    const info = JSON.parse(raw);
-    if (!isPidAlive2(info.pid)) {
-      fs3.unlinkSync(filePath);
-      return null;
-    }
-    return info;
-  } catch {
-    return null;
-  }
-}
-function removeServerFile() {
-  try {
-    const filePath = serverFilePath();
-    if (fs3.existsSync(filePath)) {
-      fs3.unlinkSync(filePath);
-    }
-  } catch {
-  }
-}
-async function isServerAlive() {
-  const info = readServerFile();
-  if (!info) {
-    return null;
-  }
-  try {
-    const response = await fetch(`http://localhost:${info.httpPort}/api/status`, {
-      signal: AbortSignal.timeout(2e3)
-    });
-    if (response.ok) {
-      return info;
-    }
-    return null;
-  } catch {
-    removeServerFile();
-    return null;
-  }
-}
-
-// packages/core/src/global-server.ts
 var sessions2 = /* @__PURE__ */ new Map();
 var clientSessions = /* @__PURE__ */ new Map();
 function toSummary(session) {
@@ -1699,6 +1697,24 @@ async function startGlobalServer(options = {}) {
         };
         ws.send(JSON.stringify(msg));
       }
+    } else {
+      const summaries = Array.from(sessions2.values()).map(toSummary);
+      const msg = {
+        type: "session:list",
+        payload: summaries
+      };
+      ws.send(JSON.stringify(msg));
+      if (summaries.length === 1) {
+        const session = sessions2.get(summaries[0].id);
+        if (session) {
+          clientSessions.set(ws, session.id);
+          session.status = "in_review";
+          ws.send(JSON.stringify({
+            type: "review:init",
+            payload: session.payload
+          }));
+        }
+      }
     }
     ws.on("message", (data) => {
       try {
@@ -1712,6 +1728,16 @@ async function startGlobalServer(options = {}) {
               session.status = "submitted";
             }
           }
+        } else if (msg.type === "session:select") {
+          const session = sessions2.get(msg.payload.sessionId);
+          if (session) {
+            clientSessions.set(ws, session.id);
+            session.status = "in_review";
+            ws.send(JSON.stringify({
+              type: "review:init",
+              payload: session.payload
+            }));
+          }
         }
       } catch {
       }
@@ -1769,6 +1795,9 @@ Waiting for reviews...
 }
 
 export {
+  getCurrentBranch,
+  getDiff,
+  analyze,
   startReview,
   readWatchFile,
   readReviewResult,