@shareworker/code-review-mcp 0.1.0

package/dist/reflect.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"reflect.d.ts","sourceRoot":"","sources":["../src/reflect.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAe,YAAY,EAAE,aAAa,EAAE,MAAM,YAAY,CAAC;AAE3E;;;;;;;;;;;;;GAaG;AACH,wBAAsB,cAAc,CAClC,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,YAAY,GAClB,OAAO,CAAC,aAAa,CAAC,CAkGxB"}

package/dist/reflect.js

@@ -0,0 +1,208 @@
+import { getDiffForFile, getFileContent } from "./git.js";
+import { parseFileDiffs, getAddedLineNumbers, splitAndNormalize, normalizeLine } from "./diff-parser.js";
+/**
+ * Deterministic validation of a positioned comment.
+ * Returns keep or drop. Does not call LLM.
+ *
+ * Three checks:
+ * 1. line_in_hunk: Are start_line/end_line within the diff hunk's changed line range?
+ * 2. existing_code_found: Does the existing_code snippet actually exist in the file?
+ *    (not applicable when existing_code is empty/undefined — passes by default)
+ * 3. existing_code_in_diff: Is at least one line of existing_code within the diff's changed lines?
+ *    (not applicable when existing_code is empty/undefined — passes by default)
+ *
+ * Verdict: any check fails → drop; all pass → keep.
+ * Semantic-level reflection is the host LLM's responsibility.
+ */
+export async function reflectComment(repo, input) {
+    const path = input.path.replace(/\\/g, "/");
+    const diffRef = input.diffRef ?? "HEAD";
+    const checks = [
+        { name: "line_in_hunk", passed: false },
+        { name: "existing_code_found", passed: false },
+        { name: "existing_code_in_diff", passed: false },
+    ];
+    // Read the diff for this file.
+    let diffText = "";
+    let hasDiff = true;
+    try {
+        diffText = await getDiffForFile(repo, diffRef, path);
+    }
+    catch {
+        hasDiff = false;
+    }
+    // Read the file content (try ref first, then worktree).
+    // Note: `git show <range>:<path>` returns empty string for range refs, so
+    // we treat empty content as "not found at ref" and fall back to worktree.
+    let fileContent = null;
+    try {
+        const atRef = await getFileContent(repo, diffRef, path);
+        fileContent = atRef && atRef.length > 0 ? atRef : null;
+    }
+    catch {
+        fileContent = null;
+    }
+    if (fileContent === null) {
+        try {
+            const atWorktree = await getFileContent(repo, "WORKTREE", path);
+            fileContent = atWorktree && atWorktree.length > 0 ? atWorktree : null;
+        }
+        catch {
+            fileContent = null;
+        }
+    }
+    // File not found → all checks fail, drop.
+    if (fileContent === null) {
+        return {
+            verdict: "drop",
+            reason: "file not found",
+            checks: [
+                { name: "line_in_hunk", passed: false },
+                { name: "existing_code_found", passed: false },
+                { name: "existing_code_in_diff", passed: false },
+            ],
+        };
+    }
+    // Check 1: line_in_hunk
+    // For new files (untracked / full-file-add), all lines are "changed" → passes vacuously.
+    // For positioning failure (0,0), fails.
+    // For workspace mode untracked files: `git diff HEAD -- <path>` returns empty string
+    // (file not tracked), but the file exists in worktree → treat as new file (all lines changed).
+    const isUntracked = hasDiff && !diffText && fileContent !== null && fileContent.length > 0;
+    let lineInHunk;
+    if (isUntracked) {
+        // Untracked file: all lines are "changed" → any valid line range passes.
+        lineInHunk = input.startLine > 0 && input.endLine >= input.startLine;
+    }
+    else {
+        lineInHunk = checkLineInHunk(input.startLine, input.endLine, diffText, hasDiff);
+    }
+    checks[0] = { name: "line_in_hunk", passed: lineInHunk };
+    // Determine if existing_code is applicable.
+    const hasExistingCode = !!input.existingCode && input.existingCode.trim() !== "";
+    // Check 2: existing_code_found
+    if (!hasExistingCode) {
+        checks[1] = { name: "existing_code_found", passed: true }; // not applicable
+    }
+    else {
+        const found = checkExistingCodeFound(input.existingCode, fileContent);
+        checks[1] = { name: "existing_code_found", passed: found };
+    }
+    // Check 3: existing_code_in_diff
+    if (!hasExistingCode) {
+        checks[2] = { name: "existing_code_in_diff", passed: true }; // not applicable
+    }
+    else if (isUntracked) {
+        // Untracked file: all code is "new" → all existing_code is in the diff.
+        checks[2] = { name: "existing_code_in_diff", passed: true };
+    }
+    else if (!hasDiff || !diffText) {
+        // No diff available (e.g., invalid ref) → can't verify → fail.
+        checks[2] = { name: "existing_code_in_diff", passed: false };
+    }
+    else {
+        const inDiff = checkExistingCodeInDiff(input.existingCode, diffText);
+        checks[2] = { name: "existing_code_in_diff", passed: inDiff };
+    }
+    // Verdict: any check fails → drop.
+    const allPass = checks.every((c) => c.passed);
+    return {
+        verdict: allPass ? "keep" : "drop",
+        reason: allPass ? "passed all checks" : describeFailure(checks),
+        checks,
+    };
+}
+/**
+ * Check if start_line/end_line fall within the diff's changed line range.
+ * - For new files (full-file-add, all lines added), all lines are "changed" → passes.
+ * - For positioning failure (0,0), fails.
+ * - For empty/invalid diff, fails.
+ */
+function checkLineInHunk(startLine, endLine, diffText, hasDiff) {
+    if (startLine <= 0 || endLine < startLine)
+        return false;
+    if (!hasDiff || !diffText)
+        return false;
+    const fileDiffs = parseFileDiffs(diffText);
+    const fileDiff = fileDiffs[0];
+    if (!fileDiff)
+        return false;
+    // New file: all lines are "changed" → any line is in hunk.
+    if (fileDiff.isNew)
+        return true;
+    // Collect all added line numbers across hunks.
+    const addedLines = getAddedLineNumbers(fileDiff.hunks);
+    if (addedLines.size === 0)
+        return false;
+    // The comment's line range must overlap with at least one added line.
+    for (let line = startLine; line <= endLine; line++) {
+        if (addedLines.has(line))
+            return true;
+    }
+    return false;
+}
+/**
+ * Check if existing_code exists in the file content.
+ * Normalizes both sides (trim, strip diff markers) and checks if all
+ * non-blank target lines appear consecutively in the file.
+ */
+function checkExistingCodeFound(existingCode, fileContent) {
+    const targetLines = splitAndNormalize(existingCode);
+    if (targetLines.length === 0)
+        return true;
+    const fileLines = fileContent.split("\n").map((l) => normalizeLine(l));
+    // Check for consecutive match (same logic as matchInFileContent but simpler — just existence).
+    for (let i = 0; i <= fileLines.length - targetLines.length; i++) {
+        let matched = true;
+        for (let j = 0; j < targetLines.length; j++) {
+            if (fileLines[i + j] !== targetLines[j]) {
+                matched = false;
+                break;
+            }
+        }
+        if (matched)
+            return true;
+    }
+    return false;
+}
+/**
+ * Check if at least one line of existing_code falls within the diff's changed lines.
+ * "Changed lines" = added lines (new-side) + deleted lines (old-side content).
+ * We check if any normalized target line matches any normalized added/deleted line.
+ */
+function checkExistingCodeInDiff(existingCode, diffText) {
+    const targetLines = splitAndNormalize(existingCode);
+    if (targetLines.length === 0)
+        return true;
+    const fileDiffs = parseFileDiffs(diffText);
+    const fileDiff = fileDiffs[0];
+    if (!fileDiff)
+        return false;
+    // Collect normalized content of all changed lines (added + deleted).
+    const changedContents = new Set();
+    for (const hunk of fileDiff.hunks) {
+        for (const line of hunk.lines) {
+            if (line.type === "added" || line.type === "deleted") {
+                const n = normalizeLine(line.content);
+                if (n !== "")
+                    changedContents.add(n);
+            }
+        }
+    }
+    // At least one target line must match a changed line.
+    for (const target of targetLines) {
+        if (changedContents.has(target))
+            return true;
+    }
+    return false;
+}
+/**
+ * Describe which check(s) failed for the drop reason.
+ */
+function describeFailure(checks) {
+    const failed = checks.filter((c) => !c.passed).map((c) => c.name);
+    if (failed.length === 0)
+        return "passed all checks";
+    return `failed: ${failed.join(", ")}`;
+}
+//# sourceMappingURL=reflect.js.map

package/dist/reflect.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"reflect.js","sourceRoot":"","sources":["../src/reflect.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,cAAc,EAAE,MAAM,UAAU,CAAC;AAC1D,OAAO,EAAE,cAAc,EAAE,mBAAmB,EAAE,iBAAiB,EAAE,aAAa,EAAE,MAAM,kBAAkB,CAAC;AAGzG;;;;;;;;;;;;;GAaG;AACH,MAAM,CAAC,KAAK,UAAU,cAAc,CAClC,IAAY,EACZ,KAAmB;IAEnB,MAAM,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;IAC5C,MAAM,OAAO,GAAG,KAAK,CAAC,OAAO,IAAI,MAAM,CAAC;IAExC,MAAM,MAAM,GAAkB;QAC5B,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,KAAK,EAAE;QACvC,EAAE,IAAI,EAAE,qBAAqB,EAAE,MAAM,EAAE,KAAK,EAAE;QAC9C,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,KAAK,EAAE;KACjD,CAAC;IAEF,+BAA+B;IAC/B,IAAI,QAAQ,GAAG,EAAE,CAAC;IAClB,IAAI,OAAO,GAAG,IAAI,CAAC;IACnB,IAAI,CAAC;QACH,QAAQ,GAAG,MAAM,cAAc,CAAC,IAAI,EAAE,OAAO,EAAE,IAAI,CAAC,CAAC;IACvD,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,GAAG,KAAK,CAAC;IAClB,CAAC;IAED,wDAAwD;IACxD,0EAA0E;IAC1E,0EAA0E;IAC1E,IAAI,WAAW,GAAkB,IAAI,CAAC;IACtC,IAAI,CAAC;QACH,MAAM,KAAK,GAAG,MAAM,cAAc,CAAC,IAAI,EAAE,OAAO,EAAE,IAAI,CAAC,CAAC;QACxD,WAAW,GAAG,KAAK,IAAI,KAAK,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC;IACzD,CAAC;IAAC,MAAM,CAAC;QACP,WAAW,GAAG,IAAI,CAAC;IACrB,CAAC;IACD,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;QACzB,IAAI,CAAC;YACH,MAAM,UAAU,GAAG,MAAM,cAAc,CAAC,IAAI,EAAE,UAAU,EAAE,IAAI,CAAC,CAAC;YAChE,WAAW,GAAG,UAAU,IAAI,UAAU,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,IAAI,CAAC;QACxE,CAAC;QAAC,MAAM,CAAC;YACP,WAAW,GAAG,IAAI,CAAC;QACrB,CAAC;IACH,CAAC;IAED,0CAA0C;IAC1C,IAAI,WAAW,KAAK,IAAI,EAAE,CAAC;QACzB,OAAO;YACL,OAAO,EAAE,MAAM;YACf,MAAM,EAAE,gBAAgB;YACxB,MAAM,EAAE;gBACN,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,KAAK,EAAE;gBACvC,EAAE,IAAI,EAAE,qBAAqB,EAAE,MAAM,EAAE,KAAK,EAAE;gBAC9C,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,KAAK,EAAE;aACjD;SACF,CAAC;IACJ,CAAC;IAED,wBAAwB;IACxB,yFAAyF;IACzF,wCAAwC;IACxC,qFAAqF;IACrF,+FAA+F;IAC/F,MAAM,WAAW,GAAG,OAAO,IAAI,CAAC,QAAQ,IAAI,WAAW,KAAK,IAAI,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC;IAC3F,IAAI,UAAmB,CAAC;IACxB,IAAI,WAAW,EAAE,CAAC;QAChB,yEAAyE;QACzE,UAAU,GAAG,KAAK,CAAC,SAAS,GAAG,CAAC,IAAI,KAAK,CAAC,OAAO,IAAI,KAAK,CAAC,SAAS,CAAC;IACvE,CAAC;SAAM,CAAC;QACN,UAAU,GAAG,eAAe,CAAC,KAAK,CAAC,SAAS,EAAE,KAAK,CAAC,OAAO,EAAE,QAAQ,EAAE,OAAO,CAAC,CAAC;IAClF,CAAC;IACD,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,cAAc,EAAE,MAAM,EAAE,UAAU,EAAE,CAAC;IAEzD,4CAA4C;IAC5C,MAAM,eAAe,GAAG,CAAC,CAAC,KAAK,CAAC,YAAY,IAAI,KAAK,CAAC,YAAY,CAAC,IAAI,EAAE,KAAK,EAAE,CAAC;IAEjF,+BAA+B;IAC/B,IAAI,CAAC,eAAe,EAAE,CAAC;QACrB,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,qBAAqB,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,iBAAiB;IAC9E,CAAC;SAAM,CAAC;QACN,MAAM,KAAK,GAAG,sBAAsB,CAAC,KAAK,CAAC,YAAa,EAAE,WAAW,CAAC,CAAC;QACvE,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,qBAAqB,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC;IAC7D,CAAC;IAED,iCAAiC;IACjC,IAAI,CAAC,eAAe,EAAE,CAAC;QACrB,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC,CAAC,iBAAiB;IAChF,CAAC;SAAM,IAAI,WAAW,EAAE,CAAC;QACvB,wEAAwE;QACxE,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,IAAI,EAAE,CAAC;IAC9D,CAAC;SAAM,IAAI,CAAC,OAAO,IAAI,CAAC,QAAQ,EAAE,CAAC;QACjC,+DAA+D;QAC/D,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC;IAC/D,CAAC;SAAM,CAAC;QACN,MAAM,MAAM,GAAG,uBAAuB,CAAC,KAAK,CAAC,YAAa,EAAE,QAAQ,CAAC,CAAC;QACtE,MAAM,CAAC,CAAC,CAAC,GAAG,EAAE,IAAI,EAAE,uBAAuB,EAAE,MAAM,EAAE,MAAM,EAAE,CAAC;IAChE,CAAC;IAED,mCAAmC;IACnC,MAAM,OAAO,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IAC9C,OAAO;QACL,OAAO,EAAE,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,MAAM;QAClC,MAAM,EAAE,OAAO,CAAC,CAAC,CAAC,mBAAmB,CAAC,CAAC,CAAC,eAAe,CAAC,MAAM,CAAC;QAC/D,MAAM;KACP,CAAC;AACJ,CAAC;AAED;;;;;GAKG;AACH,SAAS,eAAe,CAAC,SAAiB,EAAE,OAAe,EAAE,QAAgB,EAAE,OAAgB;IAC7F,IAAI,SAAS,IAAI,CAAC,IAAI,OAAO,GAAG,SAAS;QAAE,OAAO,KAAK,CAAC;IACxD,IAAI,CAAC,OAAO,IAAI,CAAC,QAAQ;QAAE,OAAO,KAAK,CAAC;IAExC,MAAM,SAAS,GAAG,cAAc,CAAC,QAAQ,CAAC,CAAC;IAC3C,MAAM,QAAQ,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;IAC9B,IAAI,CAAC,QAAQ;QAAE,OAAO,KAAK,CAAC;IAE5B,2DAA2D;IAC3D,IAAI,QAAQ,CAAC,KAAK;QAAE,OAAO,IAAI,CAAC;IAEhC,+CAA+C;IAC/C,MAAM,UAAU,GAAG,mBAAmB,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC;IACvD,IAAI,UAAU,CAAC,IAAI,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IAExC,sEAAsE;IACtE,KAAK,IAAI,IAAI,GAAG,SAAS,EAAE,IAAI,IAAI,OAAO,EAAE,IAAI,EAAE,EAAE,CAAC;QACnD,IAAI,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,OAAO,IAAI,CAAC;IACxC,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;GAIG;AACH,SAAS,sBAAsB,CAAC,YAAoB,EAAE,WAAmB;IACvE,MAAM,WAAW,GAAG,iBAAiB,CAAC,YAAY,CAAC,CAAC;IACpD,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,IAAI,CAAC;IAE1C,MAAM,SAAS,GAAG,WAAW,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC,CAAC;IACvE,+FAA+F;IAC/F,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,SAAS,CAAC,MAAM,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QAChE,IAAI,OAAO,GAAG,IAAI,CAAC;QACnB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YAC5C,IAAI,SAAS,CAAC,CAAC,GAAG,CAAC,CAAC,KAAK,WAAW,CAAC,CAAC,CAAC,EAAE,CAAC;gBACxC,OAAO,GAAG,KAAK,CAAC;gBAChB,MAAM;YACR,CAAC;QACH,CAAC;QACD,IAAI,OAAO;YAAE,OAAO,IAAI,CAAC;IAC3B,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;GAIG;AACH,SAAS,uBAAuB,CAAC,YAAoB,EAAE,QAAgB;IACrE,MAAM,WAAW,GAAG,iBAAiB,CAAC,YAAY,CAAC,CAAC;IACpD,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,IAAI,CAAC;IAE1C,MAAM,SAAS,GAAG,cAAc,CAAC,QAAQ,CAAC,CAAC;IAC3C,MAAM,QAAQ,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;IAC9B,IAAI,CAAC,QAAQ;QAAE,OAAO,KAAK,CAAC;IAE5B,qEAAqE;IACrE,MAAM,eAAe,GAAG,IAAI,GAAG,EAAU,CAAC;IAC1C,KAAK,MAAM,IAAI,IAAI,QAAQ,CAAC,KAAK,EAAE,CAAC;QAClC,KAAK,MAAM,IAAI,IAAI,IAAI,CAAC,KAAK,EAAE,CAAC;YAC9B,IAAI,IAAI,CAAC,IAAI,KAAK,OAAO,IAAI,IAAI,CAAC,IAAI,KAAK,SAAS,EAAE,CAAC;gBACrD,MAAM,CAAC,GAAG,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;gBACtC,IAAI,CAAC,KAAK,EAAE;oBAAE,eAAe,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC;YACvC,CAAC;QACH,CAAC;IACH,CAAC;IAED,sDAAsD;IACtD,KAAK,MAAM,MAAM,IAAI,WAAW,EAAE,CAAC;QACjC,IAAI,eAAe,CAAC,GAAG,CAAC,MAAM,CAAC;YAAE,OAAO,IAAI,CAAC;IAC/C,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;GAEG;AACH,SAAS,eAAe,CAAC,MAAqB;IAC5C,MAAM,MAAM,GAAG,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC;IAClE,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,mBAAmB,CAAC;IACpD,OAAO,WAAW,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CAAC;AACxC,CAAC"}

package/dist/rules.d.ts

@@ -0,0 +1,14 @@
+import type { MatchRulesResult } from "./types.js";
+/**
+ * Built-in generic default rule, aligned with open-code-review's default.md.
+ * Covers correctness, security, performance, maintainability, test coverage.
+ */
+export declare const BUILT_IN_DEFAULT_RULE = "#### Correctness\nIs the logic correct? Are there missing boundary conditions?\nAre exceptions handled properly?\nIs it thread-safe in concurrent scenarios?\n\n#### Security\nAre there security vulnerabilities such as SQL injection or XSS?\nIs sensitive information handled correctly?\nIs permission validation complete?\n\n#### Performance\nAre there obvious performance issues (e.g., N+1 queries, unnecessary loops)?\nAre resources properly released?\n\n#### Maintainability\nIs the code clear and easy to understand?\nDo names accurately express intent?\nDoes it follow the project's existing code style and architecture patterns?\n\n#### Test Coverage\nDo critical logic paths have corresponding test cases?\nDo test cases cover boundary conditions?";
+/**
+ * Match review rules for a file path, returning a ready-to-use prompt_section.
+ *
+ * Priority: repo .code-review/rules.json > home ~/.code-review/rules.json > built-in default.
+ * First match wins at each layer; the first layer with a match takes precedence.
+ */
+export declare function matchRules(repo: string, path: string): Promise<MatchRulesResult>;
+//# sourceMappingURL=rules.d.ts.map

package/dist/rules.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"rules.d.ts","sourceRoot":"","sources":["../src/rules.ts"],"names":[],"mappings":"AAIA,OAAO,KAAK,EAAE,gBAAgB,EAAyB,MAAM,YAAY,CAAC;AAE1E;;;GAGG;AACH,eAAO,MAAM,qBAAqB,svBAqBO,CAAC;AA0C1C;;;;;GAKG;AACH,wBAAsB,UAAU,CAC9B,IAAI,EAAE,MAAM,EACZ,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,gBAAgB,CAAC,CAsC3B"}

package/dist/rules.js

@@ -0,0 +1,123 @@
+import { minimatch } from "minimatch";
+import { readFile } from "node:fs/promises";
+import { join } from "node:path";
+import { homedir } from "node:os";
+/**
+ * Built-in generic default rule, aligned with open-code-review's default.md.
+ * Covers correctness, security, performance, maintainability, test coverage.
+ */
+export const BUILT_IN_DEFAULT_RULE = `#### Correctness
+Is the logic correct? Are there missing boundary conditions?
+Are exceptions handled properly?
+Is it thread-safe in concurrent scenarios?
+
+#### Security
+Are there security vulnerabilities such as SQL injection or XSS?
+Is sensitive information handled correctly?
+Is permission validation complete?
+
+#### Performance
+Are there obvious performance issues (e.g., N+1 queries, unnecessary loops)?
+Are resources properly released?
+
+#### Maintainability
+Is the code clear and easy to understand?
+Do names accurately express intent?
+Does it follow the project's existing code style and architecture patterns?
+
+#### Test Coverage
+Do critical logic paths have corresponding test cases?
+Do test cases cover boundary conditions?`;
+/**
+ * Load a rules.json file, returning null if missing or unparseable.
+ * Parse failures log a warning and return null (caller falls back to default).
+ * Accepts both `path` and `pattern` as the glob field key (spec uses `path`).
+ */
+async function loadRulesFile(dir) {
+    const filePath = join(dir, ".code-review", "rules.json");
+    try {
+        const content = await readFile(filePath, "utf8");
+        const raw = JSON.parse(content);
+        // Normalize: rules entries may use `path` or `pattern` — map to `pattern`.
+        if (raw?.rules && Array.isArray(raw.rules)) {
+            raw.rules = raw.rules.map((r) => ({
+                pattern: r.pattern ?? r.path,
+                rule: r.rule,
+            }));
+        }
+        return raw;
+    }
+    catch (err) {
+        if (err?.code === "ENOENT")
+            return null; // missing: silent fallback
+        // Parse error or other failure: warn and fall back.
+        console.warn(`[code-review-mcp] Failed to parse ${filePath}: ${err?.message ?? err}. Falling back to default rules.`);
+        return null;
+    }
+}
+/**
+ * Match a file path against a list of path rules (first match wins).
+ * Case-insensitive, supports glob patterns including ** and brace expansion.
+ */
+function matchPathRules(path, rules) {
+    const lowerPath = path.toLowerCase();
+    for (const rule of rules) {
+        if (minimatch(lowerPath, rule.pattern.toLowerCase())) {
+            return rule;
+        }
+    }
+    return null;
+}
+/**
+ * Match review rules for a file path, returning a ready-to-use prompt_section.
+ *
+ * Priority: repo .code-review/rules.json > home ~/.code-review/rules.json > built-in default.
+ * First match wins at each layer; the first layer with a match takes precedence.
+ */
+export async function matchRules(repo, path) {
+    const normalizedPath = path.replace(/\\/g, "/");
+    // Layer 1: repo rules.
+    const repoConfig = await loadRulesFile(repo);
+    if (repoConfig?.rules) {
+        const matched = matchPathRules(normalizedPath, repoConfig.rules);
+        if (matched) {
+            return {
+                path: normalizedPath,
+                matchedRules: [matched],
+                promptSection: formatPromptSection([matched.rule]),
+                usedDefault: false,
+            };
+        }
+    }
+    // Layer 2: home rules.
+    const homeConfig = await loadRulesFile(homedir());
+    if (homeConfig?.rules) {
+        const matched = matchPathRules(normalizedPath, homeConfig.rules);
+        if (matched) {
+            return {
+                path: normalizedPath,
+                matchedRules: [matched],
+                promptSection: formatPromptSection([matched.rule]),
+                usedDefault: false,
+            };
+        }
+    }
+    // Layer 3: built-in default.
+    return {
+        path: normalizedPath,
+        matchedRules: [],
+        promptSection: formatPromptSection([BUILT_IN_DEFAULT_RULE]),
+        usedDefault: true,
+    };
+}
+/**
+ * Format rule texts into a single prompt_section string.
+ */
+function formatPromptSection(rules) {
+    if (rules.length === 0)
+        return "";
+    if (rules.length === 1)
+        return rules[0];
+    return rules.map((r) => `- ${r}`).join("\n");
+}
+//# sourceMappingURL=rules.js.map

package/dist/rules.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"rules.js","sourceRoot":"","sources":["../src/rules.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,SAAS,EAAE,MAAM,WAAW,CAAC;AACtC,OAAO,EAAE,QAAQ,EAAE,MAAM,kBAAkB,CAAC;AAC5C,OAAO,EAAE,IAAI,EAAE,MAAM,WAAW,CAAC;AACjC,OAAO,EAAE,OAAO,EAAE,MAAM,SAAS,CAAC;AAGlC;;;GAGG;AACH,MAAM,CAAC,MAAM,qBAAqB,GAAG;;;;;;;;;;;;;;;;;;;;;yCAqBI,CAAC;AAE1C;;;;GAIG;AACH,KAAK,UAAU,aAAa,CAAC,GAAW;IACtC,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,EAAE,cAAc,EAAE,YAAY,CAAC,CAAC;IACzD,IAAI,CAAC;QACH,MAAM,OAAO,GAAG,MAAM,QAAQ,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC;QACjD,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,OAAO,CAAQ,CAAC;QACvC,2EAA2E;QAC3E,IAAI,GAAG,EAAE,KAAK,IAAI,KAAK,CAAC,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC;YAC3C,GAAG,CAAC,KAAK,GAAG,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAM,EAAE,EAAE,CAAC,CAAC;gBACrC,OAAO,EAAE,CAAC,CAAC,OAAO,IAAI,CAAC,CAAC,IAAI;gBAC5B,IAAI,EAAE,CAAC,CAAC,IAAI;aACb,CAAC,CAAC,CAAC;QACN,CAAC;QACD,OAAO,GAAkB,CAAC;IAC5B,CAAC;IAAC,OAAO,GAAQ,EAAE,CAAC;QAClB,IAAI,GAAG,EAAE,IAAI,KAAK,QAAQ;YAAE,OAAO,IAAI,CAAC,CAAC,2BAA2B;QACpE,oDAAoD;QACpD,OAAO,CAAC,IAAI,CAAC,qCAAqC,QAAQ,KAAK,GAAG,EAAE,OAAO,IAAI,GAAG,kCAAkC,CAAC,CAAC;QACtH,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,SAAS,cAAc,CAAC,IAAY,EAAE,KAAiB;IACrD,MAAM,SAAS,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IACrC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,SAAS,CAAC,SAAS,EAAE,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,CAAC,EAAE,CAAC;YACrD,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;;GAKG;AACH,MAAM,CAAC,KAAK,UAAU,UAAU,CAC9B,IAAY,EACZ,IAAY;IAEZ,MAAM,cAAc,GAAG,IAAI,CAAC,OAAO,CAAC,KAAK,EAAE,GAAG,CAAC,CAAC;IAEhD,uBAAuB;IACvB,MAAM,UAAU,GAAG,MAAM,aAAa,CAAC,IAAI,CAAC,CAAC;IAC7C,IAAI,UAAU,EAAE,KAAK,EAAE,CAAC;QACtB,MAAM,OAAO,GAAG,cAAc,CAAC,cAAc,EAAE,UAAU,CAAC,KAAK,CAAC,CAAC;QACjE,IAAI,OAAO,EAAE,CAAC;YACZ,OAAO;gBACL,IAAI,EAAE,cAAc;gBACpB,YAAY,EAAE,CAAC,OAAO,CAAC;gBACvB,aAAa,EAAE,mBAAmB,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;gBAClD,WAAW,EAAE,KAAK;aACnB,CAAC;QACJ,CAAC;IACH,CAAC;IAED,uBAAuB;IACvB,MAAM,UAAU,GAAG,MAAM,aAAa,CAAC,OAAO,EAAE,CAAC,CAAC;IAClD,IAAI,UAAU,EAAE,KAAK,EAAE,CAAC;QACtB,MAAM,OAAO,GAAG,cAAc,CAAC,cAAc,EAAE,UAAU,CAAC,KAAK,CAAC,CAAC;QACjE,IAAI,OAAO,EAAE,CAAC;YACZ,OAAO;gBACL,IAAI,EAAE,cAAc;gBACpB,YAAY,EAAE,CAAC,OAAO,CAAC;gBACvB,aAAa,EAAE,mBAAmB,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;gBAClD,WAAW,EAAE,KAAK;aACnB,CAAC;QACJ,CAAC;IACH,CAAC;IAED,6BAA6B;IAC7B,OAAO;QACL,IAAI,EAAE,cAAc;QACpB,YAAY,EAAE,EAAE;QAChB,aAAa,EAAE,mBAAmB,CAAC,CAAC,qBAAqB,CAAC,CAAC;QAC3D,WAAW,EAAE,IAAI;KAClB,CAAC;AACJ,CAAC;AAED;;GAEG;AACH,SAAS,mBAAmB,CAAC,KAAe;IAC1C,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAClC,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC,CAAC,CAAC,CAAC;IACxC,OAAO,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC/C,CAAC"}

package/dist/types.d.ts

@@ -0,0 +1,119 @@
+/** A single line within a diff hunk. */
+export type HunkLineType = "context" | "added" | "deleted";
+export interface HunkLine {
+    type: HunkLineType;
+    /** Content without the leading +/-/space diff marker. */
+    content: string;
+}
+/** One @@ ... @@ block in a unified diff. */
+export interface Hunk {
+    /** Starting line in the old file (1-indexed). */
+    oldStart: number;
+    oldCount: number;
+    /** Starting line in the new file (1-indexed). */
+    newStart: number;
+    newCount: number;
+    lines: HunkLine[];
+}
+/** A parsed diff for a single file. */
+export interface FileDiff {
+    oldPath: string;
+    newPath: string;
+    /** Raw unified diff text for this file (including headers). */
+    diff: string;
+    hunks: Hunk[];
+    isBinary: boolean;
+    isNew: boolean;
+    isDeleted: boolean;
+    isRenamed: boolean;
+    insertions: number;
+    deletions: number;
+}
+/** A file selected for review. */
+export interface ReviewTarget {
+    path: string;
+    diff: string;
+    additions: number;
+    deletions: number;
+    status: "added" | "modified" | "deleted" | "renamed";
+}
+/** A path -> rule entry. */
+export interface PathRule {
+    pattern: string;
+    rule: string;
+}
+/** Filter configuration from .code-review/rules.json. */
+export interface FilterConfig {
+    include: string[];
+    exclude: string[];
+}
+/** Rules configuration from .code-review/rules.json. */
+export interface RulesConfig {
+    filters?: FilterConfig;
+    rules?: PathRule[];
+}
+/** A bundle of related files reviewed together. */
+export interface BundleFile {
+    path: string;
+    diff: string;
+    additions: number;
+    deletions: number;
+}
+export type BundleReason = "test_source_pair" | "i18n_variants" | "single_file";
+export interface FileBundle {
+    id: string;
+    files: BundleFile[];
+    totalChars: number;
+    bundleReason: BundleReason;
+}
+/** Input for position_comment. */
+export interface PositionInput {
+    path: string;
+    content: string;
+    existingCode?: string;
+    suggestionCode?: string;
+    hintLine?: number;
+    diffRef?: string;
+    repo?: string;
+}
+export type LocatedBy = "text_match" | "hunk_align" | "failed";
+export interface PositionResult {
+    path: string;
+    startLine: number;
+    endLine: number;
+    locatedBy: LocatedBy;
+}
+/** Input for reflect_comment. */
+export interface ReflectInput {
+    path: string;
+    content: string;
+    startLine: number;
+    endLine: number;
+    existingCode?: string;
+    diffRef?: string;
+    repo?: string;
+}
+export interface CheckResult {
+    name: "line_in_hunk" | "existing_code_found" | "existing_code_in_diff";
+    passed: boolean;
+}
+export interface ReflectResult {
+    verdict: "keep" | "drop";
+    reason: string;
+    checks: CheckResult[];
+}
+/** Output of get_review_targets. */
+export interface ReviewTargetsResult {
+    diffRef: string;
+    files: ReviewTarget[];
+    totalFiles: number;
+    filteredOut: number;
+}
+/** Output of match_rules. */
+export interface MatchRulesResult {
+    path: string;
+    matchedRules: PathRule[];
+    promptSection: string;
+    usedDefault: boolean;
+}
+//# sourceMappingURL=types.d.ts.map

package/dist/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAEA,wCAAwC;AACxC,MAAM,MAAM,YAAY,GAAG,SAAS,GAAG,OAAO,GAAG,SAAS,CAAC;AAE3D,MAAM,WAAW,QAAQ;IACvB,IAAI,EAAE,YAAY,CAAC;IACnB,yDAAyD;IACzD,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,6CAA6C;AAC7C,MAAM,WAAW,IAAI;IACnB,iDAAiD;IACjD,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,iDAAiD;IACjD,QAAQ,EAAE,MAAM,CAAC;IACjB,QAAQ,EAAE,MAAM,CAAC;IACjB,KAAK,EAAE,QAAQ,EAAE,CAAC;CACnB;AAED,uCAAuC;AACvC,MAAM,WAAW,QAAQ;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;IAChB,+DAA+D;IAC/D,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,IAAI,EAAE,CAAC;IACd,QAAQ,EAAE,OAAO,CAAC;IAClB,KAAK,EAAE,OAAO,CAAC;IACf,SAAS,EAAE,OAAO,CAAC;IACnB,SAAS,EAAE,OAAO,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,kCAAkC;AAClC,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;IAClB,MAAM,EAAE,OAAO,GAAG,UAAU,GAAG,SAAS,GAAG,SAAS,CAAC;CACtD;AAED,4BAA4B;AAC5B,MAAM,WAAW,QAAQ;IACvB,OAAO,EAAE,MAAM,CAAC;IAChB,IAAI,EAAE,MAAM,CAAC;CACd;AAED,yDAAyD;AACzD,MAAM,WAAW,YAAY;IAC3B,OAAO,EAAE,MAAM,EAAE,CAAC;IAClB,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB;AAED,wDAAwD;AACxD,MAAM,WAAW,WAAW;IAC1B,OAAO,CAAC,EAAE,YAAY,CAAC;IACvB,KAAK,CAAC,EAAE,QAAQ,EAAE,CAAC;CACpB;AAED,mDAAmD;AACnD,MAAM,WAAW,UAAU;IACzB,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,SAAS,EAAE,MAAM,CAAC;CACnB;AAED,MAAM,MAAM,YAAY,GAAG,kBAAkB,GAAG,eAAe,GAAG,aAAa,CAAC;AAEhF,MAAM,WAAW,UAAU;IACzB,EAAE,EAAE,MAAM,CAAC;IACX,KAAK,EAAE,UAAU,EAAE,CAAC;IACpB,UAAU,EAAE,MAAM,CAAC;IACnB,YAAY,EAAE,YAAY,CAAC;CAC5B;AAED,kCAAkC;AAClC,MAAM,WAAW,aAAa;IAC5B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,cAAc,CAAC,EAAE,MAAM,CAAC;IACxB,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,MAAM,SAAS,GAAG,YAAY,GAAG,YAAY,GAAG,QAAQ,CAAC;AAE/D,MAAM,WAAW,cAAc;IAC7B,IAAI,EAAE,MAAM,CAAC;IACb,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,SAAS,CAAC;CACtB;AAED,iCAAiC;AACjC,MAAM,WAAW,YAAY;IAC3B,IAAI,EAAE,MAAM,CAAC;IACb,OAAO,EAAE,MAAM,CAAC;IAChB,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,MAAM,CAAC;IAChB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,IAAI,CAAC,EAAE,MAAM,CAAC;CACf;AAED,MAAM,WAAW,WAAW;IAC1B,IAAI,EAAE,cAAc,GAAG,qBAAqB,GAAG,uBAAuB,CAAC;IACvE,MAAM,EAAE,OAAO,CAAC;CACjB;AAED,MAAM,WAAW,aAAa;IAC5B,OAAO,EAAE,MAAM,GAAG,MAAM,CAAC;IACzB,MAAM,EAAE,MAAM,CAAC;IACf,MAAM,EAAE,WAAW,EAAE,CAAC;CACvB;AAED,oCAAoC;AACpC,MAAM,WAAW,mBAAmB;IAClC,OAAO,EAAE,MAAM,CAAC;IAChB,KAAK,EAAE,YAAY,EAAE,CAAC;IACtB,UAAU,EAAE,MAAM,CAAC;IACnB,WAAW,EAAE,MAAM,CAAC;CACrB;AAED,6BAA6B;AAC7B,MAAM,WAAW,gBAAgB;IAC/B,IAAI,EAAE,MAAM,CAAC;IACb,YAAY,EAAE,QAAQ,EAAE,CAAC;IACzB,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,OAAO,CAAC;CACtB"}

package/dist/types.js

@@ -0,0 +1,3 @@
+// Shared types for the code-review MCP server.
+export {};
+//# sourceMappingURL=types.js.map

package/dist/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../src/types.ts"],"names":[],"mappings":"AAAA,+CAA+C"}

package/package.json

@@ -0,0 +1,58 @@
+{
+  "name": "@shareworker/code-review-mcp",
+  "version": "0.1.0",
+  "description": "MCP server exposing the deterministic engineering layer of code review as tools for host agents (Claude Code, Codex, Devin). The server never calls an LLM.",
+  "type": "module",
+  "bin": {
+    "code-review-mcp": "dist/index.js"
+  },
+  "files": [
+    "dist",
+    "skills",
+    "README.md",
+    "LICENSE"
+  ],
+  "scripts": {
+    "build": "tsc",
+    "dev": "tsc --watch",
+    "test": "vitest run",
+    "test:watch": "vitest",
+    "start": "node dist/index.js",
+    "prepublishOnly": "tsc && vitest run"
+  },
+  "keywords": [
+    "mcp",
+    "code-review",
+    "model-context-protocol",
+    "claude",
+    "codex",
+    "devin",
+    "ai",
+    "llm"
+  ],
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/shareworker/code-review-mcp.git"
+  },
+  "homepage": "https://github.com/shareworker/code-review-mcp#readme",
+  "bugs": {
+    "url": "https://github.com/shareworker/code-review-mcp/issues"
+  },
+  "author": "shareworker",
+  "license": "MIT",
+  "dependencies": {
+    "@modelcontextprotocol/sdk": "^1.0.4",
+    "diff": "^7.0.0",
+    "minimatch": "^9.0.9",
+    "simple-git": "^3.27.0"
+  },
+  "devDependencies": {
+    "@types/diff": "^6.0.0",
+    "@types/node": "^22.10.0",
+    "typescript": "^5.7.0",
+    "vitest": "^2.1.0"
+  },
+  "engines": {
+    "node": ">=20"
+  }
+}