diffprism 0.45.0 → 0.47.0

package/README.md

@@ -102,6 +102,41 @@ diffprism review main..feature      # Branch diff
 
 Running multiple Claude Code sessions? All reviews appear in one browser dashboard with status badges, branch info, and desktop notifications.
 
+## Commit Gate
+
+Review is only reliable if something other than memory triggers it. `diffprism hook`
+wires the review into your pre-commit hook, so a substantial change opens a review
+before it can land.
+
+```bash
+diffprism hook install              # Add the gate to this repo's pre-commit hook
+diffprism hook uninstall            # Remove it
+```
+
+By default a staged diff of **120+ changed lines** opens a review; anything smaller
+commits untouched. A gate that stops every commit is one you learn to skip with
+`--no-verify`, and a skipped gate is worse than none. Tune it per repo:
+
+```bash
+git config diffprism.gate-lines 200
+```
+
+Approve and the commit proceeds. Request changes and the commit is blocked — with
+your comments printed in the output, so the agent that ran `git commit` can read
+what you asked for and fix it without another round trip:
+
+```
+140 staged lines (gate at 120) — opening DiffPrism review...
+
+  feature.ts:12  [must_fix]  these should be a single exported record
+  feature.ts:88  [question]  is this range meant to be inclusive?
+
+Commit blocked: the review requested changes.
+```
+
+Install adds one line between markers, so uninstall removes exactly that and leaves
+the rest of your hook alone. Repos using `core.hooksPath` are handled.
+
 ## Features
 
 - **AI-powered PR review** — Your AI gets full codebase context via 14 MCP tools
@@ -124,6 +159,8 @@ diffprism setup --global            # Global setup (no git repo needed)
 diffprism server                    # Start the background server
 diffprism server status             # Check server status
 diffprism server stop               # Stop the server
+diffprism hook install              # Gate commits on a review
+diffprism hook uninstall            # Remove the gate
 diffprism teardown                  # Remove configuration
 ```
 

package/dist/bin.js

@@ -5,15 +5,17 @@ import {
 } from "./chunk-24B33UN6.js";
 import {
   demo
-} from "./chunk-RVI74JF5.js";
+} from "./chunk-TZ5SGWHP.js";
 import {
   ensureServer,
   isServerAlive,
   readServerFile,
   startGlobalServer,
   submitReviewToServer
-} from "./chunk-RICVPPTE.js";
-import "./chunk-QGWYCEJN.js";
+} from "./chunk-PQWU5NW4.js";
+import {
+  getDiff
+} from "./chunk-QGWYCEJN.js";
 import "./chunk-DHCVZGHE.js";
 import "./chunk-JSBRDJBE.js";
 
@@ -39,21 +41,26 @@ async function review(ref, flags) {
       await reviewLocalFlow(diffRef, flags);
     }
   } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(`Error: ${message}`);
+    const message2 = err instanceof Error ? err.message : String(err);
+    console.error(`Error: ${message2}`);
     process.exit(1);
   }
 }
 async function reviewLocalFlow(diffRef, flags) {
   const serverInfo = await ensureServer({ dev: flags.dev });
-  console.log("Opening review in browser...");
+  console.error("Opening review in browser...");
   const { result } = await submitReviewToServer(serverInfo, diffRef, {
     title: flags.title,
     cwd: process.cwd(),
     diffRef
   });
   console.log(JSON.stringify(result, null, 2));
-  process.exit(0);
+  if (!result) {
+    console.error("No review result was returned.");
+    process.exit(1);
+  }
+  const approved = result.decision === "approved" || result.decision === "approved_with_comments";
+  process.exit(approved ? 0 : 1);
 }
 async function reviewPrFlow(pr, flags) {
   const { owner, repo, number } = parsePrRef(pr);
@@ -370,7 +377,7 @@ async function setupInteractive(flags) {
 }
 async function runDemo(dev) {
   console.log("");
-  const { demo: demo2 } = await import("./demo-WVNNI6CE.js");
+  const { demo: demo2 } = await import("./demo-H2NAMKCT.js");
   await demo2({ dev });
 }
 async function setupBatch(flags) {
@@ -703,9 +710,9 @@ async function server(flags) {
     await new Promise(() => {
     });
   } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
+    const message2 = err instanceof Error ? err.message : String(err);
     if (!isDaemon) {
-      console.error(`Error starting server: ${message}`);
+      console.error(`Error starting server: ${message2}`);
     }
     process.exit(1);
   }
@@ -778,8 +785,8 @@ async function serverStatus() {
       }
     }
   } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(`Error checking server status: ${message}`);
+    const message2 = err instanceof Error ? err.message : String(err);
+    console.error(`Error checking server status: ${message2}`);
     process.exit(1);
   }
 }
@@ -813,7 +820,7 @@ async function fetchStatus(httpPort) {
   return await response.json();
 }
 function printStatus(status, httpPort) {
-  const version = true ? "0.45.0" : "0.0.0-dev";
+  const version = true ? "0.47.0" : "0.0.0-dev";
   console.log(`
 DiffPrism v${version}
 `);
@@ -845,19 +852,211 @@ async function defaultAction() {
       await open(status.uiUrl);
     }
   } catch (err) {
-    const message = err instanceof Error ? err.message : String(err);
-    console.error(`Error fetching server status: ${message}`);
+    const message2 = err instanceof Error ? err.message : String(err);
+    console.error(`Error fetching server status: ${message2}`);
     process.exit(1);
   }
 }
 
+// cli/src/commands/hook.ts
+import { execFileSync } from "child_process";
+import fs4 from "fs";
+import path4 from "path";
+var DEFAULT_MIN_LINES = 120;
+var MARKER_START = "# >>> diffprism >>>";
+var MARKER_END = "# <<< diffprism <<<";
+var HOOK_LINE = "diffprism hook pre-commit || exit 1";
+async function preCommitHook(flags = {}) {
+  const cwd = process.cwd();
+  const minLines = resolveMinLines(flags, cwd);
+  let changedLines;
+  try {
+    const { diffSet } = getDiff("staged", { cwd });
+    changedLines = diffSet.files.reduce(
+      (total, file) => total + file.additions + file.deletions,
+      0
+    );
+  } catch (err) {
+    fail(`Could not read the staged diff: ${message(err)}`);
+    return;
+  }
+  if (changedLines === 0) {
+    process.exit(0);
+  }
+  if (changedLines < minLines) {
+    process.exit(0);
+  }
+  console.error(
+    `${changedLines} staged lines (gate at ${minLines}) \u2014 opening DiffPrism review...`
+  );
+  let review2 = null;
+  try {
+    const serverInfo = await ensureServer({ dev: flags.dev });
+    const { result } = await submitReviewToServer(serverInfo, "staged", {
+      cwd,
+      diffRef: "staged",
+      title: "Pre-commit review"
+    });
+    review2 = result;
+  } catch (err) {
+    fail(`DiffPrism could not run the review: ${message(err)}`);
+    return;
+  }
+  const decision = review2?.decision;
+  if (decision === "approved" || decision === "approved_with_comments") {
+    console.error(`Review: ${decision} \u2014 proceeding.`);
+    process.exit(0);
+  }
+  if (decision === "changes_requested") {
+    printComments(review2?.comments ?? []);
+    fail("Commit blocked: the review requested changes.");
+    return;
+  }
+  if (decision === "dismissed") {
+    fail("Commit blocked: the review was dismissed without a decision.");
+    return;
+  }
+  fail(
+    `Commit blocked: no review decision was returned (got ${String(decision)}).`
+  );
+}
+function printComments(comments) {
+  if (comments.length === 0) {
+    return;
+  }
+  console.error("");
+  for (const c of comments) {
+    console.error(`  ${c.file}:${c.line}  [${c.type}]  ${c.body}`);
+  }
+  console.error("");
+}
+function installHook() {
+  const hookPath = resolveHookPath(process.cwd());
+  const existing = fs4.existsSync(hookPath) ? fs4.readFileSync(hookPath, "utf8") : "";
+  if (existing.includes(MARKER_START)) {
+    console.log(`Already installed in ${hookPath}`);
+    return;
+  }
+  const block = `${MARKER_START}
+${HOOK_LINE}
+${MARKER_END}
+`;
+  const base = existing === "" ? "#!/bin/sh\n" : ensureTrailingNewline(existing);
+  const created = existing === "";
+  fs4.mkdirSync(path4.dirname(hookPath), { recursive: true });
+  fs4.writeFileSync(hookPath, `${base}
+${block}`);
+  fs4.chmodSync(hookPath, 493);
+  console.log(`${created ? "Created" : "Updated"} ${hookPath}`);
+  console.log(
+    `Staged changes of ${DEFAULT_MIN_LINES}+ lines now open a review before the commit lands.`
+  );
+  console.log("Tune with:  git config diffprism.gate-lines <n>");
+  console.log("Remove with: diffprism hook uninstall");
+}
+function uninstallHook() {
+  const hookPath = resolveHookPath(process.cwd());
+  if (!fs4.existsSync(hookPath)) {
+    console.log("Nothing to remove \u2014 no hook file.");
+    return;
+  }
+  const existing = fs4.readFileSync(hookPath, "utf8");
+  if (!existing.includes(MARKER_START)) {
+    console.log(`Nothing to remove \u2014 no diffprism block in ${hookPath}`);
+    return;
+  }
+  const cleaned = removeMarkedBlock(existing);
+  if (/^\s*(#![^\n]*)?\s*$/.test(cleaned)) {
+    fs4.rmSync(hookPath);
+    console.log(`Removed ${hookPath} (it contained nothing else).`);
+    return;
+  }
+  fs4.writeFileSync(hookPath, cleaned);
+  console.log(`Removed the diffprism block from ${hookPath}`);
+}
+function removeMarkedBlock(contents) {
+  const lines = contents.split("\n");
+  const out = [];
+  let inside = false;
+  for (const line of lines) {
+    if (line.trim() === MARKER_START) {
+      inside = true;
+      continue;
+    }
+    if (line.trim() === MARKER_END) {
+      inside = false;
+      continue;
+    }
+    if (!inside) {
+      out.push(line);
+    }
+  }
+  return ensureTrailingNewline(out.join("\n").replace(/\n{3,}/g, "\n\n"));
+}
+function resolveMinLines(flags, cwd) {
+  const fromFlag = flags.minLines ? Number.parseInt(flags.minLines, 10) : NaN;
+  if (Number.isFinite(fromFlag) && fromFlag > 0) {
+    return fromFlag;
+  }
+  const configured = Number.parseInt(
+    git(["config", "--get", "diffprism.gate-lines"], cwd) ?? "",
+    10
+  );
+  if (Number.isFinite(configured) && configured > 0) {
+    return configured;
+  }
+  return DEFAULT_MIN_LINES;
+}
+function resolveHookPath(cwd) {
+  const configured = git(["config", "--get", "core.hooksPath"], cwd);
+  if (configured) {
+    return path4.resolve(cwd, configured, "pre-commit");
+  }
+  const hooksDir = git(["rev-parse", "--git-path", "hooks"], cwd) ?? ".git/hooks";
+  return path4.resolve(cwd, hooksDir, "pre-commit");
+}
+function git(args, cwd) {
+  try {
+    const out = execFileSync("git", args, {
+      cwd,
+      encoding: "utf8",
+      stdio: ["ignore", "pipe", "ignore"]
+    });
+    const trimmed = out.trim();
+    return trimmed === "" ? null : trimmed;
+  } catch {
+    return null;
+  }
+}
+function ensureTrailingNewline(text) {
+  return text.endsWith("\n") ? text : `${text}
+`;
+}
+function message(err) {
+  return err instanceof Error ? err.message : String(err);
+}
+function fail(text) {
+  console.error(text);
+  process.exit(1);
+}
+
 // cli/src/index.ts
 var program = new Command();
-program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.45.0" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.47.0" : "0.0.0-dev");
 program.action(defaultAction);
 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 (local git ref or GitHub PR ref like owner/repo#123)").option("--staged", "Review staged changes").option("--unstaged", "Review unstaged changes").option("-t, --title <title>", "Review title").option("--reasoning <text>", "Agent reasoning about the changes").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(review);
 program.command("review-pr <pr>", { hidden: true }).action((pr, flags) => review(pr, flags));
+var hookCmd = program.command("hook").description("Git hook integration \u2014 gate commits on a human review");
+hookCmd.command("pre-commit").description("Run the review gate for the staged changes (called from a git hook)").option("--min-lines <n>", "Staged lines required to trigger a review (default: 120)").option("--dev", "Use Vite dev server").action((flags) => {
+  void preCommitHook(flags);
+});
+hookCmd.command("install").description("Add the diffprism gate to this repo's pre-commit hook").action(() => {
+  installHook();
+});
+hookCmd.command("uninstall").description("Remove the diffprism gate from this repo's pre-commit hook").action(() => {
+  uninstallHook();
+});
 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").option("--no-demo", "Skip the demo review after setup").action((flags) => {
   setup(flags);

package/dist/{chunk-RICVPPTE.js → chunk-PQWU5NW4.js}

@@ -122,30 +122,45 @@ function buildDefaultSpawnCommand(options) {
   const thisDir = path2.dirname(thisFile);
   const workspaceRoot = path2.resolve(thisDir, "..", "..", "..");
   const devBin = path2.join(workspaceRoot, "cli", "bin", "diffprism.mjs");
-  let binPath = "diffprism";
   if (fs2.existsSync(devBin)) {
-    binPath = devBin;
-  } else {
-    let searchDir = thisDir;
-    while (searchDir !== path2.dirname(searchDir)) {
-      const candidate = path2.join(
-        searchDir,
-        "node_modules",
-        ".bin",
-        "diffprism"
+    return withDevFlag([process.execPath, devBin, "server", "--_daemon"], options);
+  }
+  let searchDir = thisDir;
+  while (searchDir !== path2.dirname(searchDir)) {
+    const ownBin = readOwnBinPath(searchDir);
+    if (ownBin) {
+      return withDevFlag(
+        [process.execPath, ownBin, "server", "--_daemon"],
+        options
       );
-      if (fs2.existsSync(candidate)) {
-        binPath = candidate;
-        break;
-      }
-      searchDir = path2.dirname(searchDir);
     }
+    const shim = path2.join(searchDir, "node_modules", ".bin", "diffprism");
+    if (fs2.existsSync(shim)) {
+      return withDevFlag([shim, "server", "--_daemon"], options);
+    }
+    searchDir = path2.dirname(searchDir);
+  }
+  return withDevFlag(["diffprism", "server", "--_daemon"], options);
+}
+function withDevFlag(args, options) {
+  return options.dev ? [...args, "--dev"] : args;
+}
+function readOwnBinPath(dir) {
+  const manifestPath = path2.join(dir, "package.json");
+  if (!fs2.existsSync(manifestPath)) {
+    return null;
   }
-  const args = [process.execPath, binPath, "server", "--_daemon"];
-  if (options.dev) {
-    args.push("--dev");
+  try {
+    const manifest = JSON.parse(fs2.readFileSync(manifestPath, "utf8"));
+    const entry = typeof manifest.bin === "string" ? manifest.bin : manifest.bin?.diffprism;
+    if (!entry) {
+      return null;
+    }
+    const resolved = path2.resolve(dir, entry);
+    return fs2.existsSync(resolved) ? resolved : null;
+  } catch {
+    return null;
   }
-  return args;
 }
 async function submitReviewToServer(serverInfo, diffRef, options = {}) {
   const cwd = options.cwd ?? process.cwd();

package/dist/{chunk-RVI74JF5.js → chunk-TZ5SGWHP.js}

@@ -1,7 +1,7 @@
 import {
   ensureServer,
   submitReviewToServer
-} from "./chunk-RICVPPTE.js";
+} from "./chunk-PQWU5NW4.js";
 import {
   parseDiff
 } from "./chunk-QGWYCEJN.js";

package/dist/{demo-WVNNI6CE.js → demo-H2NAMKCT.js}

@@ -1,7 +1,7 @@
 import {
   demo
-} from "./chunk-RVI74JF5.js";
-import "./chunk-RICVPPTE.js";
+} from "./chunk-TZ5SGWHP.js";
+import "./chunk-PQWU5NW4.js";
 import "./chunk-QGWYCEJN.js";
 import "./chunk-DHCVZGHE.js";
 import "./chunk-JSBRDJBE.js";

package/dist/mcp-server.js

@@ -6,7 +6,7 @@ import {
   ensureServer,
   isServerAlive,
   submitReviewToServer
-} from "./chunk-RICVPPTE.js";
+} from "./chunk-PQWU5NW4.js";
 import {
   getDiff
 } from "./chunk-QGWYCEJN.js";
@@ -144,7 +144,7 @@ async function handlePrReview(pr, options) {
 async function startMcpServer() {
   const server = new McpServer({
     name: "diffprism",
-    version: true ? "0.45.0" : "0.0.0-dev"
+    version: true ? "0.47.0" : "0.0.0-dev"
   });
   server.tool(
     "open_review",

package/package.json

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