diffprism 0.31.1 → 0.33.0

package/dist/bin.js

@@ -1,12 +1,19 @@
 #!/usr/bin/env node
 import {
+  createGitHubClient,
+  fetchPullRequest,
+  fetchPullRequestDiff,
   isServerAlive,
+  normalizePr,
+  parsePrRef,
   readServerFile,
   readWatchFile,
+  resolveGitHubToken,
   startGlobalServer,
   startReview,
-  startWatch
-} from "./chunk-OJ723D6Z.js";
+  startWatch,
+  submitGitHubReview
+} from "./chunk-ZLIUNVTW.js";
 
 // cli/src/index.ts
 import { Command } from "commander";
@@ -39,6 +46,111 @@ async function review(ref, flags) {
   }
 }
 
+// cli/src/commands/review-pr.ts
+import readline from "readline";
+async function reviewPr(pr, flags) {
+  try {
+    const token = resolveGitHubToken();
+    const { owner, repo, number } = parsePrRef(pr);
+    console.log(`Fetching PR #${number} from ${owner}/${repo}...`);
+    const client = createGitHubClient(token);
+    const [prMetadata, rawDiff] = await Promise.all([
+      fetchPullRequest(client, owner, repo, number),
+      fetchPullRequestDiff(client, owner, repo, number)
+    ]);
+    if (!rawDiff.trim()) {
+      console.log("PR has no changes to review.");
+      return;
+    }
+    const { payload, diffSet, briefing, metadata } = normalizePr(rawDiff, prMetadata, {
+      title: flags.title,
+      reasoning: flags.reasoning
+    });
+    console.log(
+      `${diffSet.files.length} files, +${diffSet.files.reduce((s, f) => s + f.additions, 0)} -${diffSet.files.reduce((s, f) => s + f.deletions, 0)}`
+    );
+    let result;
+    const serverInfo = await isServerAlive();
+    if (serverInfo) {
+      const createResponse = await fetch(
+        `http://localhost:${serverInfo.httpPort}/api/reviews`,
+        {
+          method: "POST",
+          headers: { "Content-Type": "application/json" },
+          body: JSON.stringify({
+            payload,
+            projectPath: `github:${owner}/${repo}`,
+            diffRef: `PR #${number}`
+          })
+        }
+      );
+      if (!createResponse.ok) {
+        throw new Error(`Global server returned ${createResponse.status}`);
+      }
+      const { sessionId } = await createResponse.json();
+      console.log(`Review session created: ${sessionId}`);
+      console.log("Waiting for review submission...");
+      result = await pollForResult(serverInfo.httpPort, sessionId);
+    } else {
+      result = await startReview({
+        diffRef: `PR #${number}`,
+        title: metadata.title,
+        description: metadata.description,
+        reasoning: metadata.reasoning,
+        cwd: process.cwd(),
+        dev: flags.dev,
+        injectedPayload: payload
+      });
+    }
+    console.log(JSON.stringify(result, null, 2));
+    if (flags.postToGithub || result.decision !== "dismissed" && await promptPostToGithub()) {
+      console.log("Posting review to GitHub...");
+      const posted = await submitGitHubReview(client, owner, repo, number, result);
+      if (posted) {
+        console.log(`Review posted: ${prMetadata.url}#pullrequestreview-${posted.reviewId}`);
+      }
+    }
+    process.exit(0);
+  } catch (err) {
+    const message = err instanceof Error ? err.message : String(err);
+    console.error(`Error: ${message}`);
+    process.exit(1);
+  }
+}
+async function pollForResult(httpPort, sessionId) {
+  const pollIntervalMs = 2e3;
+  const maxWaitMs = 600 * 1e3;
+  const start2 = Date.now();
+  while (Date.now() - start2 < maxWaitMs) {
+    const response = await fetch(
+      `http://localhost:${httpPort}/api/reviews/${sessionId}/result`
+    );
+    if (response.ok) {
+      const data = await response.json();
+      if (data.result) {
+        return data.result;
+      }
+    }
+    await new Promise((resolve) => setTimeout(resolve, pollIntervalMs));
+  }
+  throw new Error("Review timed out waiting for submission.");
+}
+async function promptPostToGithub() {
+  if (!process.stdin.isTTY) {
+    return false;
+  }
+  const rl = readline.createInterface({
+    input: process.stdin,
+    output: process.stdout
+  });
+  return new Promise((resolve) => {
+    rl.question("Post this review to GitHub? (y/N) ", (answer) => {
+      rl.close();
+      resolve(answer.toLowerCase() === "y");
+    });
+  });
+}
+
 // cli/src/commands/serve.ts
 async function serve() {
   const { startMcpServer } = await import("./mcp-server.js");
@@ -49,7 +161,7 @@ async function serve() {
 import fs from "fs";
 import path from "path";
 import os from "os";
-import readline from "readline";
+import readline2 from "readline";
 
 // cli/src/templates/skill.ts
 var skillContent = `---
@@ -244,7 +356,8 @@ function setupClaudeSettings(baseDir, force) {
     "mcp__diffprism__analyze_diff",
     "mcp__diffprism__add_annotation",
     "mcp__diffprism__get_review_state",
-    "mcp__diffprism__flag_for_attention"
+    "mcp__diffprism__flag_for_attention",
+    "mcp__diffprism__review_pr"
   ];
   const allPresent = toolNames.every((t) => allow.includes(t));
   if (allPresent && !force) {
@@ -344,7 +457,7 @@ function setupStopHook(gitRoot, force) {
   return { action, filePath: filePath + " (Stop hook)" };
 }
 async function promptUser(question) {
-  const rl = readline.createInterface({
+  const rl = readline2.createInterface({
     input: process.stdin,
     output: process.stdout
   });
@@ -479,7 +592,8 @@ function isGlobalSetupDone() {
     "mcp__diffprism__analyze_diff",
     "mcp__diffprism__add_annotation",
     "mcp__diffprism__get_review_state",
-    "mcp__diffprism__flag_for_attention"
+    "mcp__diffprism__flag_for_attention",
+    "mcp__diffprism__review_pr"
   ];
   return toolNames.every((t) => allow.includes(t));
 }
@@ -887,8 +1001,9 @@ 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.31.1" : "0.0.0-dev");
+program.name("diffprism").description("Local-first code review tool for agent-generated changes").version(true ? "0.33.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("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);