diffprism 0.33.0 → 0.34.1

package/dist/bin.js

@@ -13,7 +13,7 @@ import {
   startReview,
   startWatch,
   submitGitHubReview
-} from "./chunk-ZLIUNVTW.js";
+} from "./chunk-VASCXEMN.js";
 
 // cli/src/index.ts
 import { Command } from "commander";
@@ -1001,7 +1001,7 @@ 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.33.0" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.34.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("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);

package/dist/{chunk-ZLIUNVTW.js → chunk-VASCXEMN.js}

@@ -1155,6 +1155,45 @@ function consumeReviewResult(cwd) {
 import getPort from "get-port";
 import open from "open";
 
+// packages/core/src/review-history.ts
+import fs2 from "fs";
+import path4 from "path";
+import { randomUUID } from "crypto";
+function generateEntryId() {
+  return randomUUID();
+}
+function getHistoryPath(projectDir) {
+  return path4.join(projectDir, ".diffprism", "history", "reviews.json");
+}
+function readHistory(projectDir) {
+  const filePath = getHistoryPath(projectDir);
+  if (!fs2.existsSync(filePath)) {
+    return { version: 1, entries: [] };
+  }
+  try {
+    const raw = fs2.readFileSync(filePath, "utf-8");
+    const parsed = JSON.parse(raw);
+    return parsed;
+  } catch {
+    return { version: 1, entries: [] };
+  }
+}
+function appendHistory(projectDir, entry) {
+  const filePath = getHistoryPath(projectDir);
+  const dir = path4.dirname(filePath);
+  if (!fs2.existsSync(dir)) {
+    fs2.mkdirSync(dir, { recursive: true });
+  }
+  const history = readHistory(projectDir);
+  history.entries.push(entry);
+  history.entries.sort((a, b) => a.timestamp - b.timestamp);
+  fs2.writeFileSync(filePath, JSON.stringify(history, null, 2) + "\n");
+}
+function getRecentHistory(projectDir, limit = 50) {
+  const history = readHistory(projectDir);
+  return history.entries.slice(-limit);
+}
+
 // packages/core/src/watch-bridge.ts
 import http from "http";
 import { WebSocketServer, WebSocket } from "ws";
@@ -1478,8 +1517,8 @@ function updateSession(id, update) {
 
 // packages/core/src/ui-server.ts
 import http2 from "http";
-import fs2 from "fs";
-import path4 from "path";
+import fs3 from "fs";
+import path5 from "path";
 import { fileURLToPath } from "url";
 var MIME_TYPES = {
   ".html": "text/html",
@@ -1494,14 +1533,14 @@ var MIME_TYPES = {
 };
 function resolveUiDist() {
   const thisFile = fileURLToPath(import.meta.url);
-  const thisDir = path4.dirname(thisFile);
-  const publishedUiDist = path4.resolve(thisDir, "..", "ui-dist");
-  if (fs2.existsSync(path4.join(publishedUiDist, "index.html"))) {
+  const thisDir = path5.dirname(thisFile);
+  const publishedUiDist = path5.resolve(thisDir, "..", "ui-dist");
+  if (fs3.existsSync(path5.join(publishedUiDist, "index.html"))) {
     return publishedUiDist;
   }
-  const workspaceRoot = path4.resolve(thisDir, "..", "..", "..");
-  const devUiDist = path4.join(workspaceRoot, "packages", "ui", "dist");
-  if (fs2.existsSync(path4.join(devUiDist, "index.html"))) {
+  const workspaceRoot = path5.resolve(thisDir, "..", "..", "..");
+  const devUiDist = path5.join(workspaceRoot, "packages", "ui", "dist");
+  if (fs3.existsSync(path5.join(devUiDist, "index.html"))) {
     return devUiDist;
   }
   throw new Error(
@@ -1510,10 +1549,10 @@ function resolveUiDist() {
 }
 function resolveUiRoot() {
   const thisFile = fileURLToPath(import.meta.url);
-  const thisDir = path4.dirname(thisFile);
-  const workspaceRoot = path4.resolve(thisDir, "..", "..", "..");
-  const uiRoot = path4.join(workspaceRoot, "packages", "ui");
-  if (fs2.existsSync(path4.join(uiRoot, "index.html"))) {
+  const thisDir = path5.dirname(thisFile);
+  const workspaceRoot = path5.resolve(thisDir, "..", "..", "..");
+  const uiRoot = path5.join(workspaceRoot, "packages", "ui");
+  if (fs3.existsSync(path5.join(uiRoot, "index.html"))) {
     return uiRoot;
   }
   throw new Error(
@@ -1533,14 +1572,14 @@ async function startViteDevServer(uiRoot, port, silent) {
 function createStaticServer(distPath, port) {
   const server = http2.createServer((req, res) => {
     const urlPath = req.url?.split("?")[0] ?? "/";
-    let filePath = path4.join(distPath, urlPath === "/" ? "index.html" : urlPath);
-    if (!fs2.existsSync(filePath)) {
-      filePath = path4.join(distPath, "index.html");
+    let filePath = path5.join(distPath, urlPath === "/" ? "index.html" : urlPath);
+    if (!fs3.existsSync(filePath)) {
+      filePath = path5.join(distPath, "index.html");
     }
-    const ext = path4.extname(filePath);
+    const ext = path5.extname(filePath);
     const contentType = MIME_TYPES[ext] ?? "application/octet-stream";
     try {
-      const content = fs2.readFileSync(filePath);
+      const content = fs3.readFileSync(filePath);
       res.writeHead(200, { "Content-Type": contentType });
       res.end(content);
     } catch {
@@ -1737,6 +1776,24 @@ DiffPrism Review: ${title ?? briefing.summary}`);
     const result = await bridge.waitForResult();
     poller?.stop();
     updateSession(session.id, { status: "completed", result });
+    try {
+      const projectDir = cwd ?? process.cwd();
+      const entry = {
+        id: generateEntryId(),
+        timestamp: Date.now(),
+        diffRef,
+        decision: result.decision,
+        filesReviewed: diffSet.files.length,
+        additions: diffSet.files.reduce((sum, f) => sum + f.additions, 0),
+        deletions: diffSet.files.reduce((sum, f) => sum + f.deletions, 0),
+        commentCount: result.comments.length,
+        branch: metadata.currentBranch,
+        title: metadata.title,
+        summary: result.summary ?? briefing.summary
+      };
+      appendHistory(projectDir, entry);
+    } catch {
+    }
     return result;
   } finally {
     poller?.stop();
@@ -1752,14 +1809,14 @@ DiffPrism Review: ${title ?? briefing.summary}`);
 }
 
 // packages/core/src/server-file.ts
-import fs3 from "fs";
-import path5 from "path";
+import fs4 from "fs";
+import path6 from "path";
 import os from "os";
 function serverDir() {
-  return path5.join(os.homedir(), ".diffprism");
+  return path6.join(os.homedir(), ".diffprism");
 }
 function serverFilePath() {
-  return path5.join(serverDir(), "server.json");
+  return path6.join(serverDir(), "server.json");
 }
 function isPidAlive2(pid) {
   try {
@@ -1771,21 +1828,21 @@ function isPidAlive2(pid) {
 }
 function writeServerFile(info) {
   const dir = serverDir();
-  if (!fs3.existsSync(dir)) {
-    fs3.mkdirSync(dir, { recursive: true });
+  if (!fs4.existsSync(dir)) {
+    fs4.mkdirSync(dir, { recursive: true });
   }
-  fs3.writeFileSync(serverFilePath(), JSON.stringify(info, null, 2) + "\n");
+  fs4.writeFileSync(serverFilePath(), JSON.stringify(info, null, 2) + "\n");
 }
 function readServerFile() {
   const filePath = serverFilePath();
-  if (!fs3.existsSync(filePath)) {
+  if (!fs4.existsSync(filePath)) {
     return null;
   }
   try {
-    const raw = fs3.readFileSync(filePath, "utf-8");
+    const raw = fs4.readFileSync(filePath, "utf-8");
     const info = JSON.parse(raw);
     if (!isPidAlive2(info.pid)) {
-      fs3.unlinkSync(filePath);
+      fs4.unlinkSync(filePath);
       return null;
     }
     return info;
@@ -1796,8 +1853,8 @@ function readServerFile() {
 function removeServerFile() {
   try {
     const filePath = serverFilePath();
-    if (fs3.existsSync(filePath)) {
-      fs3.unlinkSync(filePath);
+    if (fs4.existsSync(filePath)) {
+      fs4.unlinkSync(filePath);
     }
   } catch {
   }
@@ -1998,7 +2055,7 @@ Review submitted: ${result.decision}`);
 
 // packages/core/src/global-server.ts
 import http3 from "http";
-import { randomUUID } from "crypto";
+import { randomUUID as randomUUID2 } from "crypto";
 import getPort3 from "get-port";
 import open3 from "open";
 import { WebSocketServer as WebSocketServer2, WebSocket as WebSocket2 } from "ws";
@@ -2171,6 +2228,27 @@ function broadcastSessionList() {
   const summaries = Array.from(sessions2.values()).map(toSummary);
   broadcastToAll({ type: "session:list", payload: summaries });
 }
+function recordReviewHistory(session, result) {
+  if (session.projectPath.startsWith("github:")) return;
+  try {
+    const { payload } = session;
+    const entry = {
+      id: generateEntryId(),
+      timestamp: Date.now(),
+      diffRef: session.diffRef ?? "unknown",
+      decision: result.decision,
+      filesReviewed: payload.diffSet.files.length,
+      additions: payload.diffSet.files.reduce((sum, f) => sum + f.additions, 0),
+      deletions: payload.diffSet.files.reduce((sum, f) => sum + f.deletions, 0),
+      commentCount: result.comments.length,
+      branch: payload.metadata.currentBranch,
+      title: payload.metadata.title,
+      summary: result.summary ?? payload.briefing.summary
+    };
+    appendHistory(session.projectPath, entry);
+  } catch {
+  }
+}
 async function handleApiRequest(req, res) {
   const method = req.method ?? "GET";
   const url = (req.url ?? "/").split("?")[0];
@@ -2234,7 +2312,7 @@ async function handleApiRequest(req, res) {
         reopenBrowserIfNeeded?.();
         jsonResponse(res, 200, { sessionId });
       } else {
-        const sessionId = `session-${randomUUID().slice(0, 8)}`;
+        const sessionId = `session-${randomUUID2().slice(0, 8)}`;
         payload.reviewId = sessionId;
         if (diffRef) {
           payload.watchMode = true;
@@ -2295,6 +2373,7 @@ async function handleApiRequest(req, res) {
       const result = JSON.parse(body);
       session.result = result;
       session.status = "submitted";
+      recordReviewHistory(session, result);
       if (result.decision === "dismissed") {
         broadcastSessionRemoved(postResultParams.id);
       } else {
@@ -2360,7 +2439,7 @@ async function handleApiRequest(req, res) {
       const body = await readBody(req);
       const { file, line, body: annotationBody, type, confidence, category, source } = JSON.parse(body);
       const annotation = {
-        id: randomUUID(),
+        id: randomUUID2(),
         sessionId: session.id,
         file,
         line,
@@ -2405,6 +2484,10 @@ async function handleApiRequest(req, res) {
       return true;
     }
     annotation.dismissed = true;
+    sendToSessionClients(dismissAnnotationParams.id, {
+      type: "annotation:dismissed",
+      payload: { annotationId: dismissAnnotationParams.annotationId }
+    });
     jsonResponse(res, 200, { ok: true });
     return true;
   }
@@ -2426,6 +2509,10 @@ async function handleApiRequest(req, res) {
       jsonResponse(res, 404, { error: "Session not found" });
       return true;
     }
+    if (session.projectPath.startsWith("github:")) {
+      jsonResponse(res, 400, { error: "Ref listing not available for GitHub PRs" });
+      return true;
+    }
     try {
       const branches = listBranches({ cwd: session.projectPath });
       const commits = listCommits({ cwd: session.projectPath });
@@ -2443,6 +2530,10 @@ async function handleApiRequest(req, res) {
       jsonResponse(res, 404, { error: "Session not found" });
       return true;
     }
+    if (session.projectPath.startsWith("github:")) {
+      jsonResponse(res, 400, { error: "Ref comparison not available for GitHub PRs" });
+      return true;
+    }
     try {
       const body = await readBody(req);
       const { ref } = JSON.parse(body);
@@ -2484,6 +2575,38 @@ async function handleApiRequest(req, res) {
     }
     return true;
   }
+  const getSessionHistoryParams = matchRoute(method, url, "GET", "/api/reviews/:id/history");
+  if (getSessionHistoryParams) {
+    const session = sessions2.get(getSessionHistoryParams.id);
+    if (!session) {
+      jsonResponse(res, 404, { error: "Session not found" });
+      return true;
+    }
+    if (session.projectPath.startsWith("github:")) {
+      jsonResponse(res, 200, { history: [] });
+      return true;
+    }
+    const history = getRecentHistory(session.projectPath);
+    jsonResponse(res, 200, { history });
+    return true;
+  }
+  if (method === "GET" && req.url) {
+    const parsedUrl = new URL(req.url, "http://localhost");
+    if (parsedUrl.pathname === "/api/history") {
+      const projectPath = parsedUrl.searchParams.get("project");
+      if (!projectPath) {
+        jsonResponse(res, 400, { error: "Missing required query parameter: project" });
+        return true;
+      }
+      if (projectPath.startsWith("github:")) {
+        jsonResponse(res, 200, { history: [] });
+        return true;
+      }
+      const history = getRecentHistory(projectPath);
+      jsonResponse(res, 200, { history });
+      return true;
+    }
+  }
   jsonResponse(res, 404, { error: "Not found" });
   return true;
 }
@@ -2580,6 +2703,7 @@ async function startGlobalServer(options = {}) {
             if (session) {
               session.result = msg.payload;
               session.status = "submitted";
+              recordReviewHistory(session, msg.payload);
               if (msg.payload.decision === "dismissed") {
                 broadcastSessionRemoved(sid);
               } else {
@@ -2738,11 +2862,6 @@ Waiting for reviews...
   return { httpPort, wsPort, stop };
 }
 
-// packages/core/src/review-history.ts
-import fs4 from "fs";
-import path6 from "path";
-import { randomUUID as randomUUID2 } from "crypto";
-
 // packages/github/src/auth.ts
 import { execSync as execSync3 } from "child_process";
 import fs5 from "fs";

package/dist/mcp-server.js

@@ -15,7 +15,7 @@ import {
   resolveGitHubToken,
   startReview,
   submitGitHubReview
-} from "./chunk-ZLIUNVTW.js";
+} from "./chunk-VASCXEMN.js";
 
 // packages/mcp-server/src/index.ts
 import fs from "fs";
@@ -113,11 +113,11 @@ async function reviewViaGlobalServer(serverInfo, diffRef, options) {
 async function startMcpServer() {
   const server = new McpServer({
     name: "diffprism",
-    version: true ? "0.33.0" : "0.0.0-dev"
+    version: true ? "0.34.1" : "0.0.0-dev"
   });
   server.tool(
     "open_review",
-    "Open a browser-based code review for local git changes. Blocks until the engineer submits their review decision.",
+    "Open a browser-based code review for local git changes. Blocks until the engineer submits their review decision. The result may include a `postReviewAction` field ('commit' or 'commit_and_pr') if the reviewer requested a post-review action.",
     {
       diff_ref: z.string().describe(
         'Git diff reference: "staged", "unstaged", "working-copy" (staged+unstaged grouped), or a ref range like "HEAD~3..HEAD"'
@@ -781,7 +781,7 @@ async function startMcpServer() {
   );
   server.tool(
     "review_pr",
-    "Open a browser-based code review for a GitHub pull request. Fetches the PR diff, runs DiffPrism analysis, and opens the review UI. Blocks until the engineer submits their review decision. Optionally posts the review back to GitHub.",
+    "Open a browser-based code review for a GitHub pull request. Fetches the PR diff, runs DiffPrism analysis, and opens the review UI. Blocks until the engineer submits their review decision. Optionally posts the review back to GitHub. The result may include a `postReviewAction` field ('commit' or 'commit_and_pr') if the reviewer requested a post-review action.",
     {
       pr: z.string().describe(
         'GitHub PR reference: "owner/repo#123" or "https://github.com/owner/repo/pull/123"'
@@ -867,7 +867,7 @@ async function startMcpServer() {
             injectedPayload: payload
           });
         }
-        if (post_to_github && result.decision !== "dismissed") {
+        if ((post_to_github || result.postToGithub) && result.decision !== "dismissed") {
           const posted = await submitGitHubReview(client, owner, repo, number, result);
           if (posted) {
             return {

package/package.json

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