diffprism 0.16.0 → 0.17.1

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-4WN4FIY4.js";
+} from "./chunk-4VXA6GCO.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.16.0" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.17.1" : "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-4WN4FIY4.js → chunk-4VXA6GCO.js}

@@ -1447,6 +1447,7 @@ import open3 from "open";
 import { WebSocketServer as WebSocketServer3, WebSocket as WebSocket3 } from "ws";
 var sessions2 = /* @__PURE__ */ new Map();
 var clientSessions = /* @__PURE__ */ new Map();
+var reopenBrowserIfNeeded = null;
 function toSummary(session) {
   const { payload } = session;
   const fileCount = payload.diffSet.files.length;
@@ -1558,6 +1559,7 @@ async function handleApiRequest(req, res) {
         type: "session:added",
         payload: toSummary(session)
       });
+      reopenBrowserIfNeeded?.();
       jsonResponse(res, 201, { sessionId });
     } catch {
       jsonResponse(res, 400, { error: "Invalid request body" });
@@ -1770,6 +1772,18 @@ Waiting for reviews...
   }
   const uiUrl = `http://localhost:${uiPort}?wsPort=${wsPort}&serverMode=true`;
   await open3(uiUrl);
+  function hasConnectedClients() {
+    if (!wss) return false;
+    for (const client of wss.clients) {
+      if (client.readyState === WebSocket3.OPEN) return true;
+    }
+    return false;
+  }
+  reopenBrowserIfNeeded = () => {
+    if (!hasConnectedClients()) {
+      open3(uiUrl);
+    }
+  };
   async function stop() {
     if (wss) {
       for (const client of wss.clients) {
@@ -1780,6 +1794,7 @@ Waiting for reviews...
     }
     clientSessions.clear();
     sessions2.clear();
+    reopenBrowserIfNeeded = null;
     await new Promise((resolve) => {
       httpServer.close(() => resolve());
     });

package/dist/mcp-server.js

@@ -7,7 +7,7 @@ import {
   readReviewResult,
   readWatchFile,
   startReview
-} from "./chunk-4WN4FIY4.js";
+} from "./chunk-4VXA6GCO.js";
 
 // packages/mcp-server/src/index.ts
 import fs from "fs";
@@ -76,7 +76,7 @@ async function reviewViaGlobalServer(serverInfo, diffRef, options) {
 async function startMcpServer() {
   const server = new McpServer({
     name: "diffprism",
-    version: true ? "0.16.0" : "0.0.0-dev"
+    version: true ? "0.17.1" : "0.0.0-dev"
   });
   server.tool(
     "open_review",

package/package.json

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