@scottsus/mdreview-mcp 0.1.0 → 0.4.0

package/dist/api-client.d.ts

@@ -33,26 +33,25 @@ interface CommentResponse {
     authorName: string | null;
     createdAt: string;
 }
-interface WaitResponse {
+interface SubmitDecisionResponse {
     id: string;
     status: string;
     decisionMessage: string | null;
     decidedAt: string | null;
-    threads: ThreadResponse[];
-    summary: {
-        totalThreads: number;
-        resolvedThreads: number;
-        unresolvedThreads: number;
-        totalComments: number;
-    };
+}
+interface ResolveThreadResponse {
+    id: string;
+    resolved: boolean;
+    resolvedAt: string | null;
 }
 export declare class ApiClient {
     private baseUrl;
     constructor(baseUrl?: string);
-    createReview(content: string, title?: string): Promise<CreateReviewResponse>;
-    getReview(reviewId: string): Promise<ReviewResponse>;
-    waitForReview(reviewId: string, timeoutSeconds?: number): Promise<WaitResponse>;
+    createReview(content: string): Promise<CreateReviewResponse>;
+    getReview(reviewId: string, includeContent?: boolean): Promise<ReviewResponse>;
     addComment(threadId: string, body: string): Promise<CommentResponse>;
+    submitDecision(reviewId: string, decision: "approved" | "rejected" | "changes_requested", message?: string): Promise<SubmitDecisionResponse>;
+    resolveThread(threadId: string): Promise<ResolveThreadResponse>;
 }
 export {};
 //# sourceMappingURL=api-client.d.ts.map

package/dist/api-client.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"api-client.d.ts","sourceRoot":"","sources":["../src/api-client.ts"],"names":[],"mappings":"AAEA,UAAU,oBAAoB;IAC5B,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,UAAU,cAAc;IACtB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,OAAO,EAAE,cAAc,EAAE,CAAC;IAC1B,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,UAAU,cAAc;IACtB,EAAE,EAAE,MAAM,CAAC;IACX,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,QAAQ,EAAE,OAAO,CAAC;IAClB,QAAQ,EAAE,eAAe,EAAE,CAAC;CAC7B;AAED,UAAU,eAAe;IACvB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,UAAU,YAAY;IACpB,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,OAAO,EAAE,cAAc,EAAE,CAAC;IAC1B,OAAO,EAAE;QACP,YAAY,EAAE,MAAM,CAAC;QACrB,eAAe,EAAE,MAAM,CAAC;QACxB,iBAAiB,EAAE,MAAM,CAAC;QAC1B,aAAa,EAAE,MAAM,CAAC;KACvB,CAAC;CACH;AAED,qBAAa,SAAS;IACpB,OAAO,CAAC,OAAO,CAAS;gBAEZ,OAAO,CAAC,EAAE,MAAM;IAItB,YAAY,CAChB,OAAO,EAAE,MAAM,EACf,KAAK,CAAC,EAAE,MAAM,GACb,OAAO,CAAC,oBAAoB,CAAC;IAmB1B,SAAS,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,cAAc,CAAC;IAWpD,aAAa,CACjB,QAAQ,EAAE,MAAM,EAChB,cAAc,SAAM,GACnB,OAAO,CAAC,YAAY,CAAC;IAalB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,CAAC;CAqB3E"}
+{"version":3,"file":"api-client.d.ts","sourceRoot":"","sources":["../src/api-client.ts"],"names":[],"mappings":"AAEA,UAAU,oBAAoB;IAC5B,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,GAAG,EAAE,MAAM,CAAC;IACZ,MAAM,EAAE,MAAM,CAAC;IACf,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,UAAU,cAAc;IACtB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,GAAG,EAAE,MAAM,CAAC;IACZ,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,MAAM,GAAG,IAAI,CAAC;IACrB,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;IACzB,OAAO,EAAE,cAAc,EAAE,CAAC;IAC1B,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,UAAU,cAAc;IACtB,EAAE,EAAE,MAAM,CAAC;IACX,WAAW,EAAE,MAAM,CAAC;IACpB,SAAS,EAAE,MAAM,CAAC;IAClB,YAAY,EAAE,MAAM,CAAC;IACrB,QAAQ,EAAE,OAAO,CAAC;IAClB,QAAQ,EAAE,eAAe,EAAE,CAAC;CAC7B;AAED,UAAU,eAAe;IACvB,EAAE,EAAE,MAAM,CAAC;IACX,IAAI,EAAE,MAAM,CAAC;IACb,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;IAC1B,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,UAAU,sBAAsB;IAC9B,EAAE,EAAE,MAAM,CAAC;IACX,MAAM,EAAE,MAAM,CAAC;IACf,eAAe,EAAE,MAAM,GAAG,IAAI,CAAC;IAC/B,SAAS,EAAE,MAAM,GAAG,IAAI,CAAC;CAC1B;AAED,UAAU,qBAAqB;IAC7B,EAAE,EAAE,MAAM,CAAC;IACX,QAAQ,EAAE,OAAO,CAAC;IAClB,UAAU,EAAE,MAAM,GAAG,IAAI,CAAC;CAC3B;AAED,qBAAa,SAAS;IACpB,OAAO,CAAC,OAAO,CAAS;gBAEZ,OAAO,CAAC,EAAE,MAAM;IAItB,YAAY,CAAC,OAAO,EAAE,MAAM,GAAG,OAAO,CAAC,oBAAoB,CAAC;IAkB5D,SAAS,CACb,QAAQ,EAAE,MAAM,EAChB,cAAc,UAAQ,GACrB,OAAO,CAAC,cAAc,CAAC;IAcpB,UAAU,CAAC,QAAQ,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,GAAG,OAAO,CAAC,eAAe,CAAC;IAsBpE,cAAc,CAClB,QAAQ,EAAE,MAAM,EAChB,QAAQ,EAAE,UAAU,GAAG,UAAU,GAAG,mBAAmB,EACvD,OAAO,CAAC,EAAE,MAAM,GACf,OAAO,CAAC,sBAAsB,CAAC;IAkB5B,aAAa,CAAC,QAAQ,EAAE,MAAM,GAAG,OAAO,CAAC,qBAAqB,CAAC;CActE"}

package/dist/api-client.js

@@ -4,13 +4,12 @@ export class ApiClient {
     constructor(baseUrl) {
         this.baseUrl = baseUrl || BASE_URL;
     }
-    async createReview(content, title) {
+    async createReview(content) {
         const response = await fetch(`${this.baseUrl}/api/reviews`, {
             method: "POST",
             headers: { "Content-Type": "application/json" },
             body: JSON.stringify({
                 content,
-                title,
                 source: "agent",
             }),
         });
@@ -20,22 +19,17 @@ export class ApiClient {
         }
         return response.json();
     }
-    async getReview(reviewId) {
-        const response = await fetch(`${this.baseUrl}/api/reviews/${reviewId}`);
+    async getReview(reviewId, includeContent = false) {
+        const url = includeContent
+            ? `${this.baseUrl}/api/reviews/${reviewId}?includeContent=true`
+            : `${this.baseUrl}/api/reviews/${reviewId}`;
+        const response = await fetch(url);
         if (!response.ok) {
             const error = await response.json();
             throw new Error(error.message || "Failed to get review");
         }
         return response.json();
     }
-    async waitForReview(reviewId, timeoutSeconds = 300) {
-        const response = await fetch(`${this.baseUrl}/api/reviews/${reviewId}/wait?timeout=${timeoutSeconds}`);
-        if (!response.ok) {
-            const error = await response.json();
-            throw new Error(error.message || "Failed to wait for review");
-        }
-        return response.json();
-    }
     async addComment(threadId, body) {
         const response = await fetch(`${this.baseUrl}/api/threads/${threadId}/replies`, {
             method: "POST",
@@ -52,4 +46,28 @@ export class ApiClient {
         }
         return response.json();
     }
+    async submitDecision(reviewId, decision, message) {
+        const response = await fetch(`${this.baseUrl}/api/reviews/${reviewId}/submit`, {
+            method: "POST",
+            headers: { "Content-Type": "application/json" },
+            body: JSON.stringify({ decision, message }),
+        });
+        if (!response.ok) {
+            const error = await response.json();
+            throw new Error(error.message || "Failed to submit decision");
+        }
+        return response.json();
+    }
+    async resolveThread(threadId) {
+        const response = await fetch(`${this.baseUrl}/api/threads/${threadId}`, {
+            method: "PATCH",
+            headers: { "Content-Type": "application/json" },
+            body: JSON.stringify({ resolved: true }),
+        });
+        if (!response.ok) {
+            const error = await response.json();
+            throw new Error(error.message || "Failed to resolve thread");
+        }
+        return response.json();
+    }
 }

package/dist/index.js

@@ -1,11 +1,12 @@
 #!/usr/bin/env node
+import fs from "fs";
 import { McpServer } from "@modelcontextprotocol/sdk/server/mcp.js";
 import { StdioServerTransport } from "@modelcontextprotocol/sdk/server/stdio.js";
 import { z } from "zod";
 import { ApiClient } from "./api-client.js";
 const server = new McpServer({
     name: "mdreview",
-    version: "0.1.0",
+    version: "0.4.0",
 });
 const apiClient = new ApiClient(process.env.MDREVIEW_BASE_URL);
 // Tool: request_review
@@ -13,67 +14,38 @@ server.registerTool("request_review", {
     title: "Request Review",
     description: "Create a markdown document review and get a shareable URL. Share this URL with the reviewer and they can add inline comments and approve/reject the document.",
     inputSchema: {
-        content: z.string().describe("The markdown content to review"),
-        title: z.string().optional().describe("Optional title for the review"),
+        filePath: z.string().describe("The path to the markdown file to review"),
     },
-}, async ({ content, title }) => {
-    const result = await apiClient.createReview(content, title);
-    return {
-        content: [
-            {
-                type: "text",
-                text: `Review created successfully!\n\nReview URL: ${result.url}\n\nShare this URL with your reviewer. They can:\n- Add inline comments by selecting text\n- Approve, reject, or request changes\n\nUse 'wait_for_review' with reviewId "${result.id}" to wait for the review to complete.`,
-            },
-        ],
-        structuredContent: {
-            reviewId: result.id,
-            url: result.url,
-            status: result.status,
-        },
-    };
-});
-// Tool: wait_for_review
-server.registerTool("wait_for_review", {
-    title: "Wait for Review",
-    description: "Wait for a review to be completed (approved, rejected, or changes requested). This will block until the reviewer makes a decision or the timeout is reached.",
-    inputSchema: {
-        reviewId: z.string().describe("The review ID to wait for"),
-        timeoutSeconds: z
-            .number()
-            .min(1)
-            .max(300)
-            .default(300)
-            .describe("Maximum time to wait in seconds (default: 300)"),
-    },
-}, async ({ reviewId, timeoutSeconds }) => {
-    const result = await apiClient.waitForReview(reviewId, timeoutSeconds);
-    if (result.status === "pending") {
+}, async ({ filePath }) => {
+    let content;
+    try {
+        content = fs.readFileSync(filePath, "utf-8");
+    }
+    catch (error) {
+        const message = error instanceof Error ? error.message : "Unknown error reading file";
         return {
             content: [
                 {
                     type: "text",
-                    text: `Review is still pending after ${timeoutSeconds} seconds. You can call wait_for_review again to continue waiting.`,
+                    text: `Error: Failed to read file at "${filePath}"\n\n${message}`,
                 },
             ],
-            structuredContent: { status: "pending", timedOut: true },
+            isError: true,
         };
     }
-    const commentsText = result.threads
-        .map((thread) => {
-        const comments = thread.comments
-            .map((c) => `  - ${c.authorType}: ${c.body}`)
-            .join("\n");
-        return `[${thread.resolved ? "RESOLVED" : "UNRESOLVED"}] "${thread.selectedText}"\n${comments}`;
-    })
-        .join("\n\n");
+    const result = await apiClient.createReview(content);
     return {
         content: [
             {
                 type: "text",
-                text: `Review completed!\n\nStatus: ${result.status.toUpperCase()}\nMessage: ${result.decisionMessage || "(none)"}\n\nSummary:\n- Total threads: ${result.summary.totalThreads}\n- Resolved: ${result.summary.resolvedThreads}\n- Unresolved: ${result.summary.unresolvedThreads}\n- Total comments: ${result.summary.totalComments}\n\n${commentsText ? `Comments:\n${commentsText}` : "No comments."}`,
+                text: `Review created successfully!\n\nReview URL: ${result.url}\n\nShare this URL with your reviewer. They can:\n- Add inline comments by selecting text\n- Approve, reject, or request changes\n\nUse 'get_review_status' with reviewId "${result.id}" to check the review status.`,
             },
         ],
-        structuredContent: { ...result },
+        structuredContent: {
+            reviewId: result.id,
+            url: result.url,
+            status: result.status,
+        },
     };
 });
 // Tool: get_review_status
@@ -82,21 +54,45 @@ server.registerTool("get_review_status", {
     description: "Get the current status of a review without waiting. Use this to check if a review has been completed.",
     inputSchema: {
         reviewId: z.string().describe("The review ID to check"),
+        includeContent: z
+            .boolean()
+            .default(false)
+            .describe("Include the full markdown content in the response"),
     },
-}, async ({ reviewId }) => {
-    const result = await apiClient.getReview(reviewId);
+}, async ({ reviewId, includeContent }) => {
+    const result = await apiClient.getReview(reviewId, includeContent);
+    const commentsText = result.threads
+        .map((thread) => {
+        const comments = thread.comments
+            .map((c) => `  - ${c.authorType}: ${c.body}`)
+            .join("\n");
+        return `[${thread.resolved ? "RESOLVED" : "UNRESOLVED"}] Thread: ${thread.id}\n"${thread.selectedText}"\n${comments}`;
+    })
+        .join("\n\n");
+    const summary = {
+        totalThreads: result.threads.length,
+        resolvedThreads: result.threads.filter((t) => t.resolved).length,
+        unresolvedThreads: result.threads.filter((t) => !t.resolved).length,
+        totalComments: result.threads.reduce((sum, t) => sum + t.comments.length, 0),
+    };
+    const structuredContent = {
+        status: result.status,
+        decisionMessage: result.decisionMessage,
+        decidedAt: result.decidedAt,
+        threads: result.threads,
+        summary,
+    };
+    if (includeContent) {
+        structuredContent.content = result.content;
+    }
     return {
         content: [
             {
                 type: "text",
-                text: `Review Status: ${result.status.toUpperCase()}\nTitle: ${result.title || "(untitled)"}\nThreads: ${result.threads.length}\nURL: ${result.url}`,
+                text: `Review Status: ${result.status.toUpperCase()}\nTitle: ${result.title || "(untitled)"}\nMessage: ${result.decisionMessage || "(none)"}\nURL: ${result.url}\n\nSummary:\n- Total threads: ${summary.totalThreads}\n- Resolved: ${summary.resolvedThreads}\n- Unresolved: ${summary.unresolvedThreads}\n- Total comments: ${summary.totalComments}\n\n${commentsText ? `Comments:\n${commentsText}` : "No comments."}`,
             },
         ],
-        structuredContent: {
-            status: result.status,
-            threadCount: result.threads.length,
-            commentCount: result.threads.reduce((sum, t) => sum + t.comments.length, 0),
-        },
+        structuredContent,
     };
 });
 // Tool: add_comment
@@ -122,6 +118,60 @@ server.registerTool("add_comment", {
         },
     };
 });
+// Tool: resolve_thread
+server.registerTool("resolve_thread", {
+    title: "Resolve Thread",
+    description: "Mark a comment thread as resolved. Use this after addressing reviewer feedback.",
+    inputSchema: {
+        threadId: z.string().describe("The thread ID to resolve"),
+    },
+}, async ({ threadId }) => {
+    const result = await apiClient.resolveThread(threadId);
+    return {
+        content: [
+            {
+                type: "text",
+                text: `Thread resolved successfully!\n\nThread ID: ${result.id}\nResolved at: ${result.resolvedAt}`,
+            },
+        ],
+        structuredContent: {
+            threadId: result.id,
+            resolved: result.resolved,
+            resolvedAt: result.resolvedAt,
+        },
+    };
+});
+// Tool: submit_decision
+server.registerTool("submit_decision", {
+    title: "Submit Decision",
+    description: "Submit a review decision (approve, reject, or request changes). Use this to finalize a review.",
+    inputSchema: {
+        reviewId: z.string().describe("The review ID"),
+        decision: z
+            .enum(["approved", "rejected", "changes_requested"])
+            .describe("The decision"),
+        message: z
+            .string()
+            .optional()
+            .describe("Optional message explaining the decision"),
+    },
+}, async ({ reviewId, decision, message }) => {
+    const result = await apiClient.submitDecision(reviewId, decision, message);
+    return {
+        content: [
+            {
+                type: "text",
+                text: `Decision submitted successfully!\n\nReview ID: ${result.id}\nDecision: ${result.status.toUpperCase()}\nMessage: ${result.decisionMessage || "(none)"}`,
+            },
+        ],
+        structuredContent: {
+            reviewId: result.id,
+            status: result.status,
+            decisionMessage: result.decisionMessage,
+            decidedAt: result.decidedAt,
+        },
+    };
+});
 async function main() {
     const transport = new StdioServerTransport();
     await server.connect(transport);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scottsus/mdreview-mcp",
-  "version": "0.1.0",
+  "version": "0.4.0",
   "description": "MCP server for MDReview - markdown document review tool",
   "type": "module",
   "bin": {