diffprism 0.31.1 → 0.33.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-OJ723D6Z.js";
+  resolveGitHubToken,
+  startReview,
+  submitGitHubReview
+} from "./chunk-ZLIUNVTW.js";
 
 // packages/mcp-server/src/index.ts
 import fs from "fs";
@@ -63,6 +70,29 @@ async function reviewViaGlobalServer(serverInfo, diffRef, options) {
   const { sessionId } = await createResponse.json();
   lastGlobalSessionId = sessionId;
   lastGlobalServerInfo = serverInfo;
+  if (options.annotations?.length) {
+    for (const ann of options.annotations) {
+      await fetch(
+        `http://localhost:${serverInfo.httpPort}/api/reviews/${sessionId}/annotations`,
+        {
+          method: "POST",
+          headers: { "Content-Type": "application/json" },
+          body: JSON.stringify({
+            file: ann.file,
+            line: ann.line,
+            body: ann.body,
+            type: ann.type,
+            confidence: ann.confidence ?? 1,
+            category: ann.category ?? "other",
+            source: {
+              agent: ann.source_agent ?? "unknown",
+              tool: "open_review"
+            }
+          })
+        }
+      );
+    }
+  }
   const pollIntervalMs = 2e3;
   const maxWaitMs = 600 * 1e3;
   const start = Date.now();
@@ -83,7 +113,7 @@ async function reviewViaGlobalServer(serverInfo, diffRef, options) {
 async function startMcpServer() {
   const server = new McpServer({
     name: "diffprism",
-    version: true ? "0.31.1" : "0.0.0-dev"
+    version: true ? "0.33.0" : "0.0.0-dev"
   });
   server.tool(
     "open_review",
@@ -94,9 +124,29 @@ async function startMcpServer() {
       ),
       title: z.string().optional().describe("Title for the review"),
       description: z.string().optional().describe("Description of the changes"),
-      reasoning: z.string().optional().describe("Agent reasoning about why these changes were made")
+      reasoning: z.string().optional().describe("Agent reasoning about why these changes were made"),
+      annotations: z.array(
+        z.object({
+          file: z.string().describe("File path within the diff to annotate"),
+          line: z.number().describe("Line number to annotate"),
+          body: z.string().describe("The annotation text"),
+          type: z.enum(["finding", "suggestion", "question", "warning"]).describe("Type of annotation"),
+          confidence: z.number().min(0).max(1).optional().describe("Confidence in the finding (0-1, defaults to 1)"),
+          category: z.enum([
+            "security",
+            "performance",
+            "convention",
+            "correctness",
+            "complexity",
+            "test-coverage",
+            "documentation",
+            "other"
+          ]).optional().describe("Category of the finding (defaults to 'other')"),
+          source_agent: z.string().optional().describe("Agent identifier (e.g., 'security-reviewer')")
+        })
+      ).optional().describe("Initial annotations to attach to the review")
     },
-    async ({ diff_ref, title, description, reasoning }) => {
+    async ({ diff_ref, title, description, reasoning, annotations }) => {
       try {
         const serverInfo = await isServerAlive();
         if (serverInfo) {
@@ -104,7 +154,8 @@ async function startMcpServer() {
             title,
             description,
             reasoning,
-            cwd: process.cwd()
+            cwd: process.cwd(),
+            annotations
           });
           return {
             content: [
@@ -728,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.",
+    {
+      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.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.31.1",
+  "version": "0.33.0",
   "type": "module",
   "description": "Local-first code review tool for agent-generated code changes",
   "bin": {