diffprism 0.37.0 → 0.37.1

package/README.md

@@ -1,14 +1,18 @@
 # DiffPrism
 
-Browser-based code review for agent-generated changes. Review what your AI wrote before it ships.
+Your AI writes code — you need to review it before it ships. But terminal diffs
+are hard to read, and there's no way to leave structured feedback that the agent
+actually acts on.
 
-## Try It
+DiffPrism opens a code review UI in your browser with syntax-highlighted diffs,
+inline commenting, and structured decisions (approve / request changes) that
+Claude reads and responds to.
 
-```bash
-npx diffprism demo
-```
+## How It Works
 
-A browser window opens with a sample code review. Click around — leave comments, approve, see how it works. No git repo needed.
+1. Type `/review` in Claude Code — your browser opens with the diff
+2. Review the changes, leave inline comments, approve or request changes
+3. Claude reads your decision and acts on it (commits, opens a PR, or fixes what you flagged)
 
 ## Setup for Claude Code
 
@@ -16,18 +20,12 @@ A browser window opens with a sample code review. Click around — leave comment
 npx diffprism setup
 ```
 
-Walks you through configuration. After restarting Claude Code, type `/review` to review your agent's changes.
-
-**Two ways to trigger a review:**
-
-1. **Type `/review`** — Claude opens your current changes in DiffPrism's browser UI, waits for your decision, and acts on it (e.g., commits if you approve).
-
-2. **Ask Claude to review** — Say "review my changes" or "open a review" and Claude will call the tool.
-
-The review blocks Claude until you submit your decision in the browser. If you request changes, Claude reads your comments and fixes them. If you approve via the quick action menu, Claude commits or opens a PR automatically.
+Configures everything and opens a demo review so you can see it in action. Restart Claude Code afterward to load the MCP server.
 
 ## Use from the CLI
 
+No setup needed — just run:
+
 ```bash
 diffprism review                    # Review all changes (staged + unstaged)
 diffprism review --staged           # Staged changes only

package/dist/bin.js

@@ -258,67 +258,6 @@ function setupSkill(gitRoot, global, force) {
   fs.writeFileSync(filePath, skillContent);
   return { action, filePath };
 }
-function cleanDiffprismHooks(gitRoot) {
-  const filePath = path.join(gitRoot, ".claude", "settings.json");
-  const existing = readJsonFile(filePath);
-  const hooks = existing.hooks;
-  if (!hooks) return { removed: 0 };
-  const stopHooks = hooks.Stop;
-  if (!Array.isArray(stopHooks) || stopHooks.length === 0) {
-    return { removed: 0 };
-  }
-  const filtered = stopHooks.filter((entry) => {
-    const innerHooks = entry.hooks;
-    if (!Array.isArray(innerHooks)) return true;
-    return !innerHooks.some((h) => {
-      const cmd = h.command;
-      return typeof cmd === "string" && cmd.includes("diffprism") && cmd.includes("notify-stop");
-    });
-  });
-  const removed = stopHooks.length - filtered.length;
-  if (removed > 0) {
-    if (filtered.length > 0) {
-      hooks.Stop = filtered;
-    } else {
-      delete hooks.Stop;
-    }
-    writeJsonFile(filePath, { ...existing, hooks });
-  }
-  return { removed };
-}
-function setupStopHook(gitRoot, force) {
-  const filePath = path.join(gitRoot, ".claude", "settings.json");
-  const existing = readJsonFile(filePath);
-  const hooks = existing.hooks ?? {};
-  const stopHooks = hooks.Stop;
-  const hookCommand = "npx diffprism@latest notify-stop";
-  if (stopHooks && !force) {
-    const hasHook = stopHooks.some((entry) => {
-      const innerHooks = entry.hooks;
-      return innerHooks?.some((h) => h.command === hookCommand);
-    });
-    if (hasHook) {
-      return { action: "skipped", filePath };
-    }
-  }
-  const hookEntry = {
-    matcher: "",
-    hooks: [
-      {
-        type: "command",
-        command: hookCommand
-      }
-    ]
-  };
-  if (stopHooks && !force) {
-    stopHooks.push(hookEntry);
-  } else {
-    hooks.Stop = [hookEntry];
-  }
-  const action = fs.existsSync(filePath) ? "updated" : "created";
-  writeJsonFile(filePath, { ...existing, hooks });
-  return { action, filePath: filePath + " (Stop hook)" };
-}
 async function promptUser(question) {
   const rl = readline2.createInterface({
     input: process.stdin,
@@ -357,26 +296,6 @@ async function setupGitignore(gitRoot) {
   fs.writeFileSync(filePath, GITIGNORE_ENTRIES.map((e) => e + "\n").join(""));
   return { action: "created", filePath };
 }
-async function promptChoice(question, options) {
-  const rl = readline2.createInterface({
-    input: process.stdin,
-    output: process.stdout
-  });
-  for (let i = 0; i < options.length; i++) {
-    console.log(`  ${i + 1}. ${options[i]}`);
-  }
-  return new Promise((resolve) => {
-    rl.question(question, (answer) => {
-      rl.close();
-      const num = parseInt(answer.trim(), 10);
-      if (num >= 1 && num <= options.length) {
-        resolve(num - 1);
-      } else {
-        resolve(0);
-      }
-    });
-  });
-}
 async function setup(flags) {
   const force = flags.force ?? false;
   const global = flags.global ?? false;
@@ -389,50 +308,32 @@ async function setup(flags) {
 }
 async function setupInteractive(flags) {
   const dev = flags.dev;
+  const demo2 = flags.demo !== false;
   const gitRoot = findGitRoot(process.cwd());
   console.log("\n  Welcome to DiffPrism");
   console.log("  Browser-based code review for agent-generated changes.\n");
-  const modeChoice = await promptChoice("\nHow will you use DiffPrism? ", [
-    "With Claude Code (recommended)",
-    "From the CLI only"
-  ]);
-  if (modeChoice === 0) {
-    if (!gitRoot) {
-      console.log("\n  Not in a git repository \u2014 configuring globally.\n");
-      const outcome2 = await setupBatch({ global: true, quiet: true });
-      console.log("  Setting up for Claude Code...");
-      console.log("  \u2713 Installed /review skill");
-      console.log("  \u2713 Added tool permissions");
-      console.log("\n  Restart Claude Code, then type /review to start a review.\n");
-      await offerDemo(dev);
-      return outcome2;
-    }
-    console.log("\n  Setting up for Claude Code...");
-    const outcome = await setupBatch({ quiet: true });
+  let outcome;
+  if (!gitRoot) {
+    outcome = await setupBatch({ global: true, quiet: true });
+    console.log("  \u2713 Configured for Claude Code (global).");
+  } else {
+    outcome = await setupBatch({ quiet: true });
     if (outcome.created.length > 0 || outcome.updated.length > 0) {
-      console.log("  \u2713 Registered MCP server (.mcp.json)");
-      console.log("  \u2713 Added tool permissions (.claude/settings.json)");
-      console.log("  \u2713 Installed /review skill");
-      console.log("  \u2713 Added .diffprism to .gitignore");
+      console.log("  \u2713 Configured for Claude Code.");
     } else {
-      console.log("  \u2713 Already configured");
+      console.log("  \u2713 Already configured.");
     }
-    console.log("\n  Restart Claude Code, then type /review to start a review.\n");
-    await offerDemo(dev);
-    return outcome;
-  }
-  console.log("\n  DiffPrism is ready to use.");
-  console.log("  Run `diffprism review` in any git repo to review changes.\n");
-  await offerDemo(dev);
-  return { created: [], updated: [], skipped: [] };
-}
-async function offerDemo(dev) {
-  const wantsDemo = await promptUser("Try a demo review now? (Y/n) ");
-  if (wantsDemo) {
-    console.log("");
-    const { demo: demo2 } = await import("./demo-JH5YOKTZ.js");
-    await demo2({ dev });
   }
+  console.log("  Restart Claude Code, then type /review to start a review.\n");
+  if (demo2) {
+    await runDemo(dev);
+  }
+  return outcome;
+}
+async function runDemo(dev) {
+  console.log("");
+  const { demo: demo2 } = await import("./demo-JH5YOKTZ.js");
+  await demo2({ dev });
 }
 async function setupBatch(flags) {
   const force = flags.force ?? false;
@@ -478,21 +379,11 @@ async function setupBatch(flags) {
   result[mcp.action].push(mcp.filePath);
   const settings = setupClaudeSettings(gitRoot, force);
   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].push(hook.filePath);
   const skill = setupSkill(gitRoot, false, force);
   result[skill.action].push(skill.filePath);
   if (!quiet) {
-    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");
-    console.log("  2. Use /review in Claude Code to review your changes\n");
-    console.log("Tip: Run `diffprism start` to combine setup + live watch mode.");
+    console.log("Next: Restart Claude Code, then type /review to review your changes.\n");
   }
   return result;
 }
@@ -600,20 +491,6 @@ function teardownClaudePermissions(baseDir) {
   writeJsonFile(filePath, existing);
   return { action: "removed", filePath };
 }
-function teardownHooks(gitRoot) {
-  const filePath = path2.join(gitRoot, ".claude", "settings.json");
-  const result = cleanDiffprismHooks(gitRoot);
-  if (result.removed > 0) {
-    const existing = readJsonFile(filePath);
-    const hooks = existing.hooks;
-    if (hooks && Object.keys(hooks).length === 0) {
-      delete existing.hooks;
-      writeJsonFile(filePath, existing);
-    }
-    return { action: "removed", filePath: filePath + " (hooks)" };
-  }
-  return { action: "skipped", filePath: filePath + " (hooks)" };
-}
 function cleanupSettingsFile(baseDir) {
   const filePath = path2.join(baseDir, ".claude", "settings.json");
   if (!fs2.existsSync(filePath)) return;
@@ -709,8 +586,6 @@ async function teardown(flags) {
   result[mcp.action].push(mcp.filePath);
   const perms = teardownClaudePermissions(gitRoot);
   result[perms.action].push(perms.filePath);
-  const hooks = teardownHooks(gitRoot);
-  result[hooks.action].push(hooks.filePath);
   cleanupSettingsFile(gitRoot);
   const skill = teardownSkill(gitRoot, false);
   result[skill.action].push(skill.filePath);
@@ -739,106 +614,6 @@ function printTeardownSummary(result, baseDir) {
   }
 }
 
-// cli/src/commands/start.ts
-async function start(ref, flags) {
-  const outcome = await setup({
-    global: flags.global,
-    force: flags.force,
-    quiet: true
-  });
-  const hasChanges = outcome.created.length > 0 || outcome.updated.length > 0;
-  if (hasChanges) {
-    console.log("DiffPrism configured for Claude Code.");
-  }
-  let diffRef;
-  if (flags.staged) {
-    diffRef = "staged";
-  } else if (flags.unstaged) {
-    diffRef = "unstaged";
-  } else if (ref) {
-    diffRef = ref;
-  } else {
-    diffRef = "working-copy";
-  }
-  try {
-    const serverInfo = await ensureServer({ dev: flags.dev });
-    console.log(
-      `DiffPrism server at http://localhost:${serverInfo.httpPort}`
-    );
-    if (hasChanges) {
-      console.log(
-        "If this is your first time, restart Claude Code first to load the MCP server."
-      );
-    }
-    const { result } = await submitReviewToServer(serverInfo, diffRef, {
-      title: flags.title,
-      cwd: process.cwd(),
-      diffRef
-    });
-    console.log(JSON.stringify(result, null, 2));
-    process.exit(0);
-  } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(`Error: ${message}`);
-    process.exit(1);
-  }
-}
-
-// cli/src/commands/watch.ts
-async function watch(ref, flags) {
-  let diffRef;
-  if (flags.staged) {
-    diffRef = "staged";
-  } else if (flags.unstaged) {
-    diffRef = "unstaged";
-  } else if (ref) {
-    diffRef = ref;
-  } else {
-    diffRef = "working-copy";
-  }
-  try {
-    const serverInfo = await ensureServer({ dev: flags.dev });
-    console.log(
-      `DiffPrism server at http://localhost:${serverInfo.httpPort}`
-    );
-    console.log("Submitting review session...");
-    const { result } = await submitReviewToServer(serverInfo, diffRef, {
-      title: flags.title,
-      cwd: process.cwd(),
-      diffRef
-    });
-    console.log(JSON.stringify(result, null, 2));
-    process.exit(0);
-  } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(`Error: ${message}`);
-    process.exit(1);
-  }
-}
-
-// cli/src/commands/notify-stop.ts
-async function notifyStop() {
-  try {
-    const serverInfo = await isServerAlive();
-    if (!serverInfo) {
-      process.exit(0);
-      return;
-    }
-    const controller = new AbortController();
-    const timeout = setTimeout(() => controller.abort(), 2e3);
-    try {
-      await fetch(`http://localhost:${serverInfo.httpPort}/api/refresh`, {
-        method: "POST",
-        signal: controller.signal
-      });
-    } finally {
-      clearTimeout(timeout);
-    }
-  } catch {
-  }
-  process.exit(0);
-}
-
 // cli/src/commands/server.ts
 import { spawn } from "child_process";
 import fs3 from "fs";
@@ -986,15 +761,12 @@ 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.37.0" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.37.1" : "0.0.0-dev");
 program.command("demo").description("Open a sample review to see DiffPrism in action").option("--dev", "Use Vite dev server").action(demo);
 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("review-pr <pr>").description("Review a GitHub pull request in DiffPrism").option("-t, --title <title>", "Override review title").option("--reasoning <text>", "Agent reasoning about the PR").option("--dev", "Use Vite dev server with HMR instead of static files").option("--post-to-github", "Automatically post review back to GitHub without prompting").action(reviewPr);
-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", "Configure globally (skill + permissions, no git repo required)").option("--force", "Overwrite existing configuration files").option("--dev", "Use Vite dev server").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").option("--dev", "Use Vite dev server").option("--no-demo", "Skip the demo review after setup").action((flags) => {
   setup(flags);
 });
 program.command("teardown").description("Remove DiffPrism configuration from the current project").option("--global", "Remove global configuration (skill + permissions at ~/.claude/)").option("-q, --quiet", "Suppress output").action((flags) => {

package/dist/mcp-server.js

@@ -29,7 +29,7 @@ var lastGlobalServerInfo = null;
 async function startMcpServer() {
   const server = new McpServer({
     name: "diffprism",
-    version: true ? "0.37.0" : "0.0.0-dev"
+    version: true ? "0.37.1" : "0.0.0-dev"
   });
   server.tool(
     "open_review",

package/package.json

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