diffprism 0.32.0 → 0.34.0

package/dist/mcp-server.js

@@ -1,14 +1,21 @@
 import {
   analyze,
   consumeReviewResult,
+  createGitHubClient,
   detectWorktree,
+  fetchPullRequest,
+  fetchPullRequestDiff,
   getCurrentBranch,
   getDiff,
   isServerAlive,
+  normalizePr,
+  parsePrRef,
   readReviewResult,
   readWatchFile,
-  startReview
-} from "./chunk-G4W5QBT4.js";
+  resolveGitHubToken,
+  startReview,
+  submitGitHubReview
+} from "./chunk-NJKYNMAQ.js";
 
 // packages/mcp-server/src/index.ts
 import fs from "fs";
@@ -106,11 +113,11 @@ async function reviewViaGlobalServer(serverInfo, diffRef, options) {
 async function startMcpServer() {
   const server = new McpServer({
     name: "diffprism",
-    version: true ? "0.32.0" : "0.0.0-dev"
+    version: true ? "0.34.0" : "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"'
@@ -772,6 +779,133 @@ 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. 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"'
+      ),
+      title: z.string().optional().describe("Override review title"),
+      reasoning: z.string().optional().describe("Agent reasoning about the PR changes"),
+      post_to_github: z.boolean().optional().describe("Post the review back to GitHub after submission (default: false)")
+    },
+    async ({ pr, title, reasoning, post_to_github }) => {
+      try {
+        const token = resolveGitHubToken();
+        const { owner, repo, number } = parsePrRef(pr);
+        const client = createGitHubClient(token);
+        const [prMetadata, rawDiff] = await Promise.all([
+          fetchPullRequest(client, owner, repo, number),
+          fetchPullRequestDiff(client, owner, repo, number)
+        ]);
+        if (!rawDiff.trim()) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: JSON.stringify({
+                  decision: "approved",
+                  comments: [],
+                  summary: "PR has no changes to review."
+                }, null, 2)
+              }
+            ]
+          };
+        }
+        const { payload } = normalizePr(rawDiff, prMetadata, { title, reasoning });
+        const serverInfo = await isServerAlive();
+        let result;
+        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();
+          lastGlobalSessionId = sessionId;
+          lastGlobalServerInfo = serverInfo;
+          const pollIntervalMs = 2e3;
+          const maxWaitMs = 600 * 1e3;
+          const start = Date.now();
+          while (Date.now() - start < maxWaitMs) {
+            const resultResponse = await fetch(
+              `http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}/result`
+            );
+            if (resultResponse.ok) {
+              const data = await resultResponse.json();
+              if (data.result) {
+                result = data.result;
+                break;
+              }
+            }
+            await new Promise((resolve) => setTimeout(resolve, pollIntervalMs));
+          }
+          result ??= { decision: "dismissed", comments: [], summary: "Review timed out." };
+        } else {
+          const isDev = fs.existsSync(
+            path.join(process.cwd(), "packages", "ui", "src", "App.tsx")
+          );
+          result = await startReview({
+            diffRef: `PR #${number}`,
+            title: payload.metadata.title,
+            description: payload.metadata.description,
+            reasoning: payload.metadata.reasoning,
+            cwd: process.cwd(),
+            silent: true,
+            dev: isDev,
+            injectedPayload: payload
+          });
+        }
+        if ((post_to_github || result.postToGithub) && result.decision !== "dismissed") {
+          const posted = await submitGitHubReview(client, owner, repo, number, result);
+          if (posted) {
+            return {
+              content: [
+                {
+                  type: "text",
+                  text: JSON.stringify({
+                    ...result,
+                    githubReviewId: posted.reviewId,
+                    githubReviewUrl: `${prMetadata.url}#pullrequestreview-${posted.reviewId}`
+                  }, null, 2)
+                }
+              ]
+            };
+          }
+        }
+        return {
+          content: [
+            {
+              type: "text",
+              text: JSON.stringify(result, null, 2)
+            }
+          ]
+        };
+      } catch (err) {
+        const message = err instanceof Error ? err.message : String(err);
+        return {
+          content: [
+            {
+              type: "text",
+              text: `Error: ${message}`
+            }
+          ],
+          isError: true
+        };
+      }
+    }
+  );
   const transport = new StdioServerTransport();
   await server.connect(transport);
 }

package/package.json

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