@rigour-labs/mcp 2.19.2 → 2.20.0

package/dist/index.js

@@ -67,6 +67,35 @@ async function logStudioEvent(cwd, event) {
         // Silent fail - Studio logging is non-blocking and zero-telemetry
     }
 }
+// Helper to parse diff and get modified lines per file
+function parseDiff(diff) {
+    const lines = diff.split('\n');
+    const mapping = {};
+    let currentFile = "";
+    let currentLine = 0;
+    for (const line of lines) {
+        if (line.startsWith('+++ b/')) {
+            currentFile = line.slice(6);
+            mapping[currentFile] = new Set();
+        }
+        else if (line.startsWith('@@')) {
+            const match = line.match(/\+(\d+)/);
+            if (match) {
+                currentLine = parseInt(match[1], 10);
+            }
+        }
+        else if (line.startsWith('+') && !line.startsWith('+++')) {
+            if (currentFile) {
+                mapping[currentFile].add(currentLine);
+            }
+            currentLine++;
+        }
+        else if (!line.startsWith('-')) {
+            currentLine++;
+        }
+    }
+    return mapping;
+}
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
         tools: [
@@ -430,6 +459,37 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     },
                     required: ["cwd", "handoffId", "agentId"],
                 },
+            },
+            {
+                name: "rigour_review",
+                description: "Perform a high-fidelity code review on a pull request diff. Analyzes changed files using all active quality gates.",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        cwd: {
+                            type: "string",
+                            description: "Absolute path to the project root.",
+                        },
+                        repository: {
+                            type: "string",
+                            description: "Full repository name (e.g., 'owner/repo').",
+                        },
+                        branch: {
+                            type: "string",
+                            description: "The branch containing the changes.",
+                        },
+                        diff: {
+                            type: "string",
+                            description: "The git diff content to analyze.",
+                        },
+                        files: {
+                            type: "array",
+                            items: { type: "string" },
+                            description: "List of filenames that were changed.",
+                        },
+                    },
+                    required: ["cwd", "diff"],
+                },
             }
         ],
     };
@@ -1154,6 +1214,52 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 };
                 break;
             }
+            case "rigour_review": {
+                const { diff, files: changedFiles } = args;
+                // 1. Map diff to line numbers for filtering
+                const diffMapping = parseDiff(diff);
+                const targetFiles = changedFiles || Object.keys(diffMapping);
+                // 2. Run high-fidelity analysis on changed files
+                const report = await runner.run(cwd, targetFiles);
+                // 3. Filter failures to only those on changed lines (or global gate failures)
+                const filteredFailures = report.failures.filter(failure => {
+                    // Global failures (no file associated) are always reported
+                    if (!failure.files || failure.files.length === 0)
+                        return true;
+                    return failure.files.some(file => {
+                        const fileModifiedLines = diffMapping[file];
+                        // If we can't find the file in the diff, assume it's a side-effect or skip
+                        if (!fileModifiedLines)
+                            return false;
+                        // If failure has line info, check if it's in modifiedLines
+                        if (failure.line !== undefined) {
+                            return fileModifiedLines.has(failure.line);
+                        }
+                        // Fallback: if file is changed and no specific line is given, report it
+                        return true;
+                    });
+                });
+                result = {
+                    content: [
+                        {
+                            type: "text",
+                            text: JSON.stringify({
+                                status: filteredFailures.length > 0 ? "FAIL" : "PASS",
+                                failures: filteredFailures.map(f => ({
+                                    id: f.id,
+                                    gate: f.title,
+                                    severity: "FAIL", // Rigour failures are FAIL by default
+                                    message: f.details,
+                                    file: f.files?.[0] || "",
+                                    line: f.line || 1,
+                                    suggestion: f.hint
+                                }))
+                            }),
+                        },
+                    ],
+                };
+                break;
+            }
             default:
                 throw new Error(`Unknown tool: ${name}`);
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rigour-labs/mcp",
-  "version": "2.19.2",
+  "version": "2.20.0",
   "type": "module",
   "mcpName": "io.github.rigour-labs/rigour",
   "description": "Quality gates for AI-generated code. Forces AI agents to meet strict engineering standards with PASS/FAIL enforcement.",
@@ -20,7 +20,7 @@
     "execa": "^8.0.1",
     "fs-extra": "^11.2.0",
     "yaml": "^2.8.2",
-    "@rigour-labs/core": "2.19.2"
+    "@rigour-labs/core": "2.20.0"
   },
   "devDependencies": {
     "@types/node": "^25.0.3",

package/src/index.ts

@@ -93,6 +93,34 @@ async function logStudioEvent(cwd: string, event: any) {
     }
 }
 
+// Helper to parse diff and get modified lines per file
+function parseDiff(diff: string): Record<string, Set<number>> {
+    const lines = diff.split('\n');
+    const mapping: Record<string, Set<number>> = {};
+    let currentFile = "";
+    let currentLine = 0;
+
+    for (const line of lines) {
+        if (line.startsWith('+++ b/')) {
+            currentFile = line.slice(6);
+            mapping[currentFile] = new Set();
+        } else if (line.startsWith('@@')) {
+            const match = line.match(/\+(\d+)/);
+            if (match) {
+                currentLine = parseInt(match[1], 10);
+            }
+        } else if (line.startsWith('+') && !line.startsWith('+++')) {
+            if (currentFile) {
+                mapping[currentFile].add(currentLine);
+            }
+            currentLine++;
+        } else if (!line.startsWith('-')) {
+            currentLine++;
+        }
+    }
+    return mapping;
+}
+
 server.setRequestHandler(ListToolsRequestSchema, async () => {
     return {
         tools: [
@@ -456,6 +484,37 @@ server.setRequestHandler(ListToolsRequestSchema, async () => {
                     },
                     required: ["cwd", "handoffId", "agentId"],
                 },
+            },
+            {
+                name: "rigour_review",
+                description: "Perform a high-fidelity code review on a pull request diff. Analyzes changed files using all active quality gates.",
+                inputSchema: {
+                    type: "object",
+                    properties: {
+                        cwd: {
+                            type: "string",
+                            description: "Absolute path to the project root.",
+                        },
+                        repository: {
+                            type: "string",
+                            description: "Full repository name (e.g., 'owner/repo').",
+                        },
+                        branch: {
+                            type: "string",
+                            description: "The branch containing the changes.",
+                        },
+                        diff: {
+                            type: "string",
+                            description: "The git diff content to analyze.",
+                        },
+                        files: {
+                            type: "array",
+                            items: { type: "string" },
+                            description: "List of filenames that were changed.",
+                        },
+                    },
+                    required: ["cwd", "diff"],
+                },
             }
         ],
     };
@@ -1270,6 +1329,58 @@ server.setRequestHandler(CallToolRequestSchema, async (request) => {
                 break;
             }
 
+            case "rigour_review": {
+                const { diff, files: changedFiles } = args as any;
+
+                // 1. Map diff to line numbers for filtering
+                const diffMapping = parseDiff(diff);
+                const targetFiles = changedFiles || Object.keys(diffMapping);
+
+                // 2. Run high-fidelity analysis on changed files
+                const report = await runner.run(cwd, targetFiles);
+
+                // 3. Filter failures to only those on changed lines (or global gate failures)
+                const filteredFailures = report.failures.filter(failure => {
+                    // Global failures (no file associated) are always reported
+                    if (!failure.files || failure.files.length === 0) return true;
+
+                    return failure.files.some(file => {
+                        const fileModifiedLines = diffMapping[file];
+                        // If we can't find the file in the diff, assume it's a side-effect or skip
+                        if (!fileModifiedLines) return false;
+
+                        // If failure has line info, check if it's in modifiedLines
+                        if (failure.line !== undefined) {
+                            return fileModifiedLines.has(failure.line);
+                        }
+
+                        // Fallback: if file is changed and no specific line is given, report it
+                        return true;
+                    });
+                });
+
+                result = {
+                    content: [
+                        {
+                            type: "text",
+                            text: JSON.stringify({
+                                status: filteredFailures.length > 0 ? "FAIL" : "PASS",
+                                failures: filteredFailures.map(f => ({
+                                    id: f.id,
+                                    gate: f.title,
+                                    severity: "FAIL", // Rigour failures are FAIL by default
+                                    message: f.details,
+                                    file: f.files?.[0] || "",
+                                    line: f.line || 1,
+                                    suggestion: f.hint
+                                }))
+                            }),
+                        },
+                    ],
+                };
+                break;
+            }
+
             default:
                 throw new Error(`Unknown tool: ${name}`);
         }