@shareworker/code-review-mcp 0.1.0

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 shareworker
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/README

@@ -0,0 +1,155 @@
+# @shareworker/code-review-mcp
+
+An MCP server that exposes the deterministic engineering layer of code review as
+tools, callable by host agents (Claude Code, Codex, Devin) within their own LLM
+loops. The server never calls an LLM — all reasoning happens in the host session.
+
+## Why
+
+General-purpose agents doing code review via Skills suffer from: incomplete
+coverage (skipping files on large changesets), position drift (line numbers
+don't match actual code), and unstable quality (minor prompt variations cause
+large quality swings). This server enforces hard constraints on the review
+process — file selection, smart bundling, rule matching, comment positioning,
+and comment reflection — so quality is stable regardless of host model variation.
+
+## Tools
+
+| Tool | Purpose |
+|------|---------|
+| `get_review_targets` | Git diff → file filtering → `diff_ref` + file list |
+| `get_file_bundle` | Smart bundling (test/source + i18n) with 20000 char cap |
+| `match_rules` | Path-based rule matching → `prompt_section` for host LLM |
+| `position_comment` | Text match + hunk align → precise line numbers |
+| `reflect_comment` | Deterministic validation (keep/drop) — no LLM |
+
+## Installation
+
+### Claude Code
+
+Add to `.claude/mcp.json`:
+
+```json
+{
+  "mcpServers": {
+    "code-review": {
+      "command": "npx",
+      "args": ["-y", "@shareworker/code-review-mcp"]
+    }
+  }
+}
+```
+
+Copy the skill to `.claude/skills/`:
+
+```bash
+mkdir -p .claude/skills/code-review
+cp skills/code-review/SKILL.md .claude/skills/code-review/SKILL.md
+```
+
+### Codex
+
+Add to your Codex MCP configuration:
+
+```json
+{
+  "mcpServers": {
+    "code-review": {
+      "command": "npx",
+      "args": ["-y", "@shareworker/code-review-mcp"]
+    }
+  }
+}
+```
+
+Copy the skill to Codex's skill directory:
+
+```bash
+cp skills/code-review/SKILL.md <codex-skills-dir>/code-review/SKILL.md
+```
+
+### Devin
+
+Add to `.devin/config.json`:
+
+```json
+{
+  "mcpServers": {
+    "code-review": {
+      "command": "npx",
+      "args": ["-y", "@shareworker/code-review-mcp"]
+    }
+  }
+}
+```
+
+Copy the skill to `.devin/skills/`:
+
+```bash
+mkdir -p .devin/skills/code-review
+cp skills/code-review/SKILL.md .devin/skills/code-review/SKILL.md
+```
+
+## Configuration
+
+Create `.code-review/rules.json` in your repo (or `~/.code-review/rules.json`
+for global config):
+
+```json
+{
+  "filters": {
+    "exclude": ["**/*.lock", "**/*.min.js", "**/*.map"],
+    "include": ["**/*.ts", "**/*.js"]
+  },
+  "rules": [
+    {
+      "path": "**/*.ts",
+      "rule": "Check for any types and proper null handling"
+    },
+    {
+      "path": "**/*mapper*.xml",
+      "rule": "Check SQL for injection risks and missing closing tags"
+    }
+  ]
+}
+```
+
+### Configuration Resolution Priority
+
+1. `--rule <path>` flag (highest) — not exposed via MCP, reserved for future CLI
+2. `<repo>/.code-review/rules.json` — project-level
+3. `~/.code-review/rules.json` — global/user-level
+4. Built-in defaults (lowest) — covers correctness, security, performance,
+   maintainability, test coverage
+
+For MVP, only layers 2-4 are active (no `--rule` flag since there's no CLI).
+The first matching user rule replaces the built-in system rule at the same layer.
+
+### `filters`
+
+- `exclude`: glob patterns for files to exclude (merged with built-in defaults)
+- `include`: glob patterns — when present, only matching files are reviewed
+
+### `rules`
+
+Array of `{ "path": "<glob>", "rule": "<text>" }`. First match wins. The `rule`
+text is returned as `prompt_section` for the host LLM to inject into its review
+prompt.
+
+## Development
+
+```bash
+npm install
+npm run build      # compile TypeScript
+npm test           # run unit tests
+npm run dev        # watch mode
+```
+
+## Architecture
+
+See `docs/superpowers/specs/2026-07-02-code-review-mcp-design.md` for the full
+design spec and `docs/adr/` for architecture decision records.
+
+## License
+
+MIT

package/README.md

@@ -0,0 +1,155 @@
+# @shareworker/code-review-mcp
+
+An MCP server that exposes the deterministic engineering layer of code review as
+tools, callable by host agents (Claude Code, Codex, Devin) within their own LLM
+loops. The server never calls an LLM — all reasoning happens in the host session.
+
+## Why
+
+General-purpose agents doing code review via Skills suffer from: incomplete
+coverage (skipping files on large changesets), position drift (line numbers
+don't match actual code), and unstable quality (minor prompt variations cause
+large quality swings). This server enforces hard constraints on the review
+process — file selection, smart bundling, rule matching, comment positioning,
+and comment reflection — so quality is stable regardless of host model variation.
+
+## Tools
+
+| Tool | Purpose |
+|------|---------|
+| `get_review_targets` | Git diff → file filtering → `diff_ref` + file list |
+| `get_file_bundle` | Smart bundling (test/source + i18n) with 20000 char cap |
+| `match_rules` | Path-based rule matching → `prompt_section` for host LLM |
+| `position_comment` | Text match + hunk align → precise line numbers |
+| `reflect_comment` | Deterministic validation (keep/drop) — no LLM |
+
+## Installation
+
+### Claude Code
+
+Add to `.claude/mcp.json`:
+
+```json
+{
+  "mcpServers": {
+    "code-review": {
+      "command": "npx",
+      "args": ["-y", "@shareworker/code-review-mcp"]
+    }
+  }
+}
+```
+
+Copy the skill to `.claude/skills/`:
+
+```bash
+mkdir -p .claude/skills/code-review
+cp skills/code-review/SKILL.md .claude/skills/code-review/SKILL.md
+```
+
+### Codex
+
+Add to your Codex MCP configuration:
+
+```json
+{
+  "mcpServers": {
+    "code-review": {
+      "command": "npx",
+      "args": ["-y", "@shareworker/code-review-mcp"]
+    }
+  }
+}
+```
+
+Copy the skill to Codex's skill directory:
+
+```bash
+cp skills/code-review/SKILL.md <codex-skills-dir>/code-review/SKILL.md
+```
+
+### Devin
+
+Add to `.devin/config.json`:
+
+```json
+{
+  "mcpServers": {
+    "code-review": {
+      "command": "npx",
+      "args": ["-y", "@shareworker/code-review-mcp"]
+    }
+  }
+}
+```
+
+Copy the skill to `.devin/skills/`:
+
+```bash
+mkdir -p .devin/skills/code-review
+cp skills/code-review/SKILL.md .devin/skills/code-review/SKILL.md
+```
+
+## Configuration
+
+Create `.code-review/rules.json` in your repo (or `~/.code-review/rules.json`
+for global config):
+
+```json
+{
+  "filters": {
+    "exclude": ["**/*.lock", "**/*.min.js", "**/*.map"],
+    "include": ["**/*.ts", "**/*.js"]
+  },
+  "rules": [
+    {
+      "path": "**/*.ts",
+      "rule": "Check for any types and proper null handling"
+    },
+    {
+      "path": "**/*mapper*.xml",
+      "rule": "Check SQL for injection risks and missing closing tags"
+    }
+  ]
+}
+```
+
+### Configuration Resolution Priority
+
+1. `--rule <path>` flag (highest) — not exposed via MCP, reserved for future CLI
+2. `<repo>/.code-review/rules.json` — project-level
+3. `~/.code-review/rules.json` — global/user-level
+4. Built-in defaults (lowest) — covers correctness, security, performance,
+   maintainability, test coverage
+
+For MVP, only layers 2-4 are active (no `--rule` flag since there's no CLI).
+The first matching user rule replaces the built-in system rule at the same layer.
+
+### `filters`
+
+- `exclude`: glob patterns for files to exclude (merged with built-in defaults)
+- `include`: glob patterns — when present, only matching files are reviewed
+
+### `rules`
+
+Array of `{ "path": "<glob>", "rule": "<text>" }`. First match wins. The `rule`
+text is returned as `prompt_section` for the host LLM to inject into its review
+prompt.
+
+## Development
+
+```bash
+npm install
+npm run build      # compile TypeScript
+npm test           # run unit tests
+npm run dev        # watch mode
+```
+
+## Architecture
+
+See `docs/superpowers/specs/2026-07-02-code-review-mcp-design.md` for the full
+design spec and `docs/adr/` for architecture decision records.
+
+## License
+
+MIT

package/dist/bundler.d.ts

@@ -0,0 +1,27 @@
+import type { FileBundle } from "./types.js";
+/** Character cap per bundle (counting diff text only). */
+export declare const BUNDLE_CHAR_CAP = 20000;
+/**
+ * Identify the test counterpart of a source file, or vice versa.
+ * Returns the paired path or null.
+ *
+ * Conventions:
+ *   foo.test.ts     <-> foo.ts
+ *   foo.spec.ts     <-> foo.ts
+ *   foo_test.go     <-> foo.go
+ *   foo_spec.go     <-> foo.go
+ *   TestFoo.java    <-> Foo.java
+ *   FooTest.java    <-> Foo.java
+ */
+export declare function findTestSourcePair(path: string, allFiles: string[]): string | null;
+export declare function findI18nVariants(path: string, allFiles: string[]): string[];
+/**
+ * Bundle files by test/source pairing and i18n variants, with a 20000 char cap.
+ *
+ * Algorithm:
+ * 1. Build pairing groups (test/source pairs, i18n variant groups, singletons).
+ * 2. For each group, read diffs and pack into bundles respecting the char cap.
+ * 3. A single file whose diff alone exceeds the cap gets its own over-cap bundle.
+ */
+export declare function bundleFiles(repo: string, files: string[], diffRef: string): Promise<FileBundle[]>;
+//# sourceMappingURL=bundler.d.ts.map

package/dist/bundler.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"bundler.d.ts","sourceRoot":"","sources":["../src/bundler.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAA4B,UAAU,EAAE,MAAM,YAAY,CAAC;AAEvE,0DAA0D;AAC1D,eAAO,MAAM,eAAe,QAAQ,CAAC;AAErC;;;;;;;;;;;GAWG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,MAAM,GAAG,IAAI,CAoClF;AASD,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,EAAE,GAAG,MAAM,EAAE,CAa3E;AAuCD;;;;;;;GAOG;AACH,wBAAsB,WAAW,CAC/B,IAAI,EAAE,MAAM,EACZ,KAAK,EAAE,MAAM,EAAE,EACf,OAAO,EAAE,MAAM,GACd,OAAO,CAAC,UAAU,EAAE,CAAC,CAkCvB"}

package/dist/bundler.js

@@ -0,0 +1,203 @@
+import { getDiffForFile } from "./git.js";
+import { parseFileDiffs } from "./diff-parser.js";
+/** Character cap per bundle (counting diff text only). */
+export const BUNDLE_CHAR_CAP = 20000;
+/**
+ * Identify the test counterpart of a source file, or vice versa.
+ * Returns the paired path or null.
+ *
+ * Conventions:
+ *   foo.test.ts     <-> foo.ts
+ *   foo.spec.ts     <-> foo.ts
+ *   foo_test.go     <-> foo.go
+ *   foo_spec.go     <-> foo.go
+ *   TestFoo.java    <-> Foo.java
+ *   FooTest.java    <-> Foo.java
+ */
+export function findTestSourcePair(path, allFiles) {
+    const base = stripExt(path);
+    const ext = getExt(path);
+    // Test file -> source file.
+    const testMatch = base.match(/^(.+)\.(test|spec)$/i) || base.match(/^(.+)[._](test|spec)$/i);
+    if (testMatch) {
+        const sourceBase = testMatch[1];
+        const sourcePath = `${sourceBase}.${ext}`;
+        if (allFiles.includes(sourcePath))
+            return sourcePath;
+    }
+    // Java: TestFoo.java <-> Foo.java, FooTest.java <-> Foo.java
+    const javaTestMatch = base.match(/^Test(.+)$/);
+    if (javaTestMatch && ext === "java") {
+        const sourcePath = `${javaTestMatch[1]}.java`;
+        if (allFiles.includes(sourcePath))
+            return sourcePath;
+    }
+    const javaTestSuffixMatch = base.match(/^(.+)Test$/);
+    if (javaTestSuffixMatch && ext === "java") {
+        const sourcePath = `${javaTestSuffixMatch[1]}.java`;
+        if (allFiles.includes(sourcePath))
+            return sourcePath;
+    }
+    // Source file -> test file.
+    for (const candidate of allFiles) {
+        if (candidate === path)
+            continue;
+        const cBase = stripExt(candidate);
+        const cExt = getExt(candidate);
+        if (cExt !== ext)
+            continue;
+        const cTestMatch = cBase.match(/^(.+)\.(test|spec)$/i) || cBase.match(/^(.+)[._](test|spec)$/i);
+        if (cTestMatch && cTestMatch[1] === base)
+            return candidate;
+        if (ext === "java") {
+            if (cBase === `Test${base}` || cBase === `${base}Test`)
+                return candidate;
+        }
+    }
+    return null;
+}
+/**
+ * Identify i18n variant files: same base name with different locale suffixes.
+ * Locale suffixes: _en, _zh, _ja, _ko, _fr, _de, _es, _pt, _ru, _ar, etc.
+ * The suffix sits before the file extension, e.g. messages_en.ts.
+ */
+const LOCALE_SUFFIX_RE = /[_-](en|zh|ja|ko|fr|de|es|pt|ru|ar|it|nl|pl|tr|vi|th|id|hi|bn|mx|tw|cn|hk|us|uk|br)(?:[_-][A-Z]{2})?(?=\.|$)/i;
+export function findI18nVariants(path, allFiles) {
+    const base = path.replace(LOCALE_SUFFIX_RE, "");
+    if (base === path)
+        return []; // no locale suffix, not an i18n file
+    const ext = getExt(path);
+    const result = [];
+    for (const candidate of allFiles) {
+        if (candidate === path)
+            continue;
+        const cBase = candidate.replace(LOCALE_SUFFIX_RE, "");
+        if (cBase === base && getExt(candidate) === ext) {
+            result.push(candidate);
+        }
+    }
+    return result;
+}
+function stripExt(path) {
+    const dot = path.lastIndexOf(".");
+    const slash = Math.max(path.lastIndexOf("/"), path.lastIndexOf("\\"));
+    if (dot <= slash)
+        return path;
+    return path.slice(0, dot);
+}
+function getExt(path) {
+    const dot = path.lastIndexOf(".");
+    const slash = Math.max(path.lastIndexOf("/"), path.lastIndexOf("\\"));
+    if (dot <= slash)
+        return "";
+    return path.slice(dot + 1);
+}
+/**
+ * Read a single file's diff and build a BundleFile.
+ * Returns null if the diff can't be read.
+ */
+async function buildBundleFile(repo, path, diffRef) {
+    try {
+        const diff = await getDiffForFile(repo, diffRef, path);
+        const parsed = parseFileDiffs(diff)[0];
+        return {
+            path,
+            diff,
+            additions: parsed?.insertions ?? 0,
+            deletions: parsed?.deletions ?? 0,
+        };
+    }
+    catch {
+        return null;
+    }
+}
+/**
+ * Bundle files by test/source pairing and i18n variants, with a 20000 char cap.
+ *
+ * Algorithm:
+ * 1. Build pairing groups (test/source pairs, i18n variant groups, singletons).
+ * 2. For each group, read diffs and pack into bundles respecting the char cap.
+ * 3. A single file whose diff alone exceeds the cap gets its own over-cap bundle.
+ */
+export async function bundleFiles(repo, files, diffRef) {
+    if (files.length === 0)
+        return [];
+    const groups = buildPairingGroups(files);
+    const bundles = [];
+    let bundleIndex = 0;
+    for (const group of groups) {
+        const bundleFiles = [];
+        for (const path of group.files) {
+            const bf = await buildBundleFile(repo, path, diffRef);
+            if (bf)
+                bundleFiles.push(bf);
+        }
+        if (bundleFiles.length === 0)
+            continue;
+        // Pack into bundles respecting the char cap.
+        let current = [];
+        let currentChars = 0;
+        for (const bf of bundleFiles) {
+            const bfChars = bf.diff.length;
+            if (current.length > 0 && currentChars + bfChars > BUNDLE_CHAR_CAP) {
+                bundles.push(makeBundle(bundleIndex++, current, group.reason));
+                current = [];
+                currentChars = 0;
+            }
+            current.push(bf);
+            currentChars += bfChars;
+        }
+        if (current.length > 0) {
+            bundles.push(makeBundle(bundleIndex++, current, group.reason));
+        }
+    }
+    return bundles;
+}
+/**
+ * Build pairing groups from the file list.
+ * Each file appears in exactly one group.
+ */
+function buildPairingGroups(files) {
+    const remaining = new Set(files);
+    const groups = [];
+    // Pass 1: test/source pairs.
+    for (const file of files) {
+        if (!remaining.has(file))
+            continue;
+        const pair = findTestSourcePair(file, files);
+        if (pair && remaining.has(pair)) {
+            remaining.delete(file);
+            remaining.delete(pair);
+            groups.push({ files: [file, pair], reason: "test_source_pair" });
+        }
+    }
+    // Pass 2: i18n variants.
+    for (const file of files) {
+        if (!remaining.has(file))
+            continue;
+        const variants = findI18nVariants(file, files).filter((v) => remaining.has(v));
+        if (variants.length > 0) {
+            remaining.delete(file);
+            for (const v of variants)
+                remaining.delete(v);
+            groups.push({ files: [file, ...variants], reason: "i18n_variants" });
+        }
+    }
+    // Pass 3: singletons.
+    for (const file of files) {
+        if (!remaining.has(file))
+            continue;
+        remaining.delete(file);
+        groups.push({ files: [file], reason: "single_file" });
+    }
+    return groups;
+}
+function makeBundle(id, files, reason) {
+    return {
+        id: `bundle-${id}`,
+        files,
+        totalChars: files.reduce((sum, f) => sum + f.diff.length, 0),
+        bundleReason: reason,
+    };
+}
+//# sourceMappingURL=bundler.js.map

package/dist/bundler.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"bundler.js","sourceRoot":"","sources":["../src/bundler.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,cAAc,EAAE,MAAM,UAAU,CAAC;AAC1C,OAAO,EAAE,cAAc,EAAE,MAAM,kBAAkB,CAAC;AAGlD,0DAA0D;AAC1D,MAAM,CAAC,MAAM,eAAe,GAAG,KAAK,CAAC;AAErC;;;;;;;;;;;GAWG;AACH,MAAM,UAAU,kBAAkB,CAAC,IAAY,EAAE,QAAkB;IACjE,MAAM,IAAI,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC;IAC5B,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;IAEzB,4BAA4B;IAC5B,MAAM,SAAS,GAAG,IAAI,CAAC,KAAK,CAAC,sBAAsB,CAAC,IAAI,IAAI,CAAC,KAAK,CAAC,wBAAwB,CAAC,CAAC;IAC7F,IAAI,SAAS,EAAE,CAAC;QACd,MAAM,UAAU,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;QAChC,MAAM,UAAU,GAAG,GAAG,UAAU,IAAI,GAAG,EAAE,CAAC;QAC1C,IAAI,QAAQ,CAAC,QAAQ,CAAC,UAAU,CAAC;YAAE,OAAO,UAAU,CAAC;IACvD,CAAC;IACD,6DAA6D;IAC7D,MAAM,aAAa,GAAG,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC;IAC/C,IAAI,aAAa,IAAI,GAAG,KAAK,MAAM,EAAE,CAAC;QACpC,MAAM,UAAU,GAAG,GAAG,aAAa,CAAC,CAAC,CAAC,OAAO,CAAC;QAC9C,IAAI,QAAQ,CAAC,QAAQ,CAAC,UAAU,CAAC;YAAE,OAAO,UAAU,CAAC;IACvD,CAAC;IACD,MAAM,mBAAmB,GAAG,IAAI,CAAC,KAAK,CAAC,YAAY,CAAC,CAAC;IACrD,IAAI,mBAAmB,IAAI,GAAG,KAAK,MAAM,EAAE,CAAC;QAC1C,MAAM,UAAU,GAAG,GAAG,mBAAmB,CAAC,CAAC,CAAC,OAAO,CAAC;QACpD,IAAI,QAAQ,CAAC,QAAQ,CAAC,UAAU,CAAC;YAAE,OAAO,UAAU,CAAC;IACvD,CAAC;IAED,4BAA4B;IAC5B,KAAK,MAAM,SAAS,IAAI,QAAQ,EAAE,CAAC;QACjC,IAAI,SAAS,KAAK,IAAI;YAAE,SAAS;QACjC,MAAM,KAAK,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC;QAClC,MAAM,IAAI,GAAG,MAAM,CAAC,SAAS,CAAC,CAAC;QAC/B,IAAI,IAAI,KAAK,GAAG;YAAE,SAAS;QAC3B,MAAM,UAAU,GAAG,KAAK,CAAC,KAAK,CAAC,sBAAsB,CAAC,IAAI,KAAK,CAAC,KAAK,CAAC,wBAAwB,CAAC,CAAC;QAChG,IAAI,UAAU,IAAI,UAAU,CAAC,CAAC,CAAC,KAAK,IAAI;YAAE,OAAO,SAAS,CAAC;QAC3D,IAAI,GAAG,KAAK,MAAM,EAAE,CAAC;YACnB,IAAI,KAAK,KAAK,OAAO,IAAI,EAAE,IAAI,KAAK,KAAK,GAAG,IAAI,MAAM;gBAAE,OAAO,SAAS,CAAC;QAC3E,CAAC;IACH,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;;;GAIG;AACH,MAAM,gBAAgB,GAAG,+GAA+G,CAAC;AAEzI,MAAM,UAAU,gBAAgB,CAAC,IAAY,EAAE,QAAkB;IAC/D,MAAM,IAAI,GAAG,IAAI,CAAC,OAAO,CAAC,gBAAgB,EAAE,EAAE,CAAC,CAAC;IAChD,IAAI,IAAI,KAAK,IAAI;QAAE,OAAO,EAAE,CAAC,CAAC,qCAAqC;IACnE,MAAM,GAAG,GAAG,MAAM,CAAC,IAAI,CAAC,CAAC;IACzB,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,KAAK,MAAM,SAAS,IAAI,QAAQ,EAAE,CAAC;QACjC,IAAI,SAAS,KAAK,IAAI;YAAE,SAAS;QACjC,MAAM,KAAK,GAAG,SAAS,CAAC,OAAO,CAAC,gBAAgB,EAAE,EAAE,CAAC,CAAC;QACtD,IAAI,KAAK,KAAK,IAAI,IAAI,MAAM,CAAC,SAAS,CAAC,KAAK,GAAG,EAAE,CAAC;YAChD,MAAM,CAAC,IAAI,CAAC,SAAS,CAAC,CAAC;QACzB,CAAC;IACH,CAAC;IACD,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,QAAQ,CAAC,IAAY;IAC5B,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAClC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,EAAE,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;IACtE,IAAI,GAAG,IAAI,KAAK;QAAE,OAAO,IAAI,CAAC;IAC9B,OAAO,IAAI,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,CAAC;AAC5B,CAAC;AAED,SAAS,MAAM,CAAC,IAAY;IAC1B,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,CAAC;IAClC,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,WAAW,CAAC,GAAG,CAAC,EAAE,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC,CAAC;IACtE,IAAI,GAAG,IAAI,KAAK;QAAE,OAAO,EAAE,CAAC;IAC5B,OAAO,IAAI,CAAC,KAAK,CAAC,GAAG,GAAG,CAAC,CAAC,CAAC;AAC7B,CAAC;AAED;;;GAGG;AACH,KAAK,UAAU,eAAe,CAC5B,IAAY,EACZ,IAAY,EACZ,OAAe;IAEf,IAAI,CAAC;QACH,MAAM,IAAI,GAAG,MAAM,cAAc,CAAC,IAAI,EAAE,OAAO,EAAE,IAAI,CAAC,CAAC;QACvD,MAAM,MAAM,GAAG,cAAc,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC;QACvC,OAAO;YACL,IAAI;YACJ,IAAI;YACJ,SAAS,EAAE,MAAM,EAAE,UAAU,IAAI,CAAC;YAClC,SAAS,EAAE,MAAM,EAAE,SAAS,IAAI,CAAC;SAClC,CAAC;IACJ,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAC;IACd,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAC/B,IAAY,EACZ,KAAe,EACf,OAAe;IAEf,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAElC,MAAM,MAAM,GAAG,kBAAkB,CAAC,KAAK,CAAC,CAAC;IACzC,MAAM,OAAO,GAAiB,EAAE,CAAC;IACjC,IAAI,WAAW,GAAG,CAAC,CAAC;IAEpB,KAAK,MAAM,KAAK,IAAI,MAAM,EAAE,CAAC;QAC3B,MAAM,WAAW,GAAiB,EAAE,CAAC;QACrC,KAAK,MAAM,IAAI,IAAI,KAAK,CAAC,KAAK,EAAE,CAAC;YAC/B,MAAM,EAAE,GAAG,MAAM,eAAe,CAAC,IAAI,EAAE,IAAI,EAAE,OAAO,CAAC,CAAC;YACtD,IAAI,EAAE;gBAAE,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC/B,CAAC;QACD,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC;YAAE,SAAS;QAEvC,6CAA6C;QAC7C,IAAI,OAAO,GAAiB,EAAE,CAAC;QAC/B,IAAI,YAAY,GAAG,CAAC,CAAC;QACrB,KAAK,MAAM,EAAE,IAAI,WAAW,EAAE,CAAC;YAC7B,MAAM,OAAO,GAAG,EAAE,CAAC,IAAI,CAAC,MAAM,CAAC;YAC/B,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,IAAI,YAAY,GAAG,OAAO,GAAG,eAAe,EAAE,CAAC;gBACnE,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,WAAW,EAAE,EAAE,OAAO,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC;gBAC/D,OAAO,GAAG,EAAE,CAAC;gBACb,YAAY,GAAG,CAAC,CAAC;YACnB,CAAC;YACD,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;YACjB,YAAY,IAAI,OAAO,CAAC;QAC1B,CAAC;QACD,IAAI,OAAO,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACvB,OAAO,CAAC,IAAI,CAAC,UAAU,CAAC,WAAW,EAAE,EAAE,OAAO,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,CAAC;QACjE,CAAC;IACH,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAOD;;;GAGG;AACH,SAAS,kBAAkB,CAAC,KAAe;IACzC,MAAM,SAAS,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,CAAC;IACjC,MAAM,MAAM,GAAmB,EAAE,CAAC;IAElC,6BAA6B;IAC7B,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,SAAS;QACnC,MAAM,IAAI,GAAG,kBAAkB,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC;QAC7C,IAAI,IAAI,IAAI,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;YAChC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACvB,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACvB,MAAM,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,CAAC,IAAI,EAAE,IAAI,CAAC,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC,CAAC;QACnE,CAAC;IACH,CAAC;IAED,yBAAyB;IACzB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,SAAS;QACnC,MAAM,QAAQ,GAAG,gBAAgB,CAAC,IAAI,EAAE,KAAK,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC;QAC/E,IAAI,QAAQ,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACxB,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;YACvB,KAAK,MAAM,CAAC,IAAI,QAAQ;gBAAE,SAAS,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC;YAC9C,MAAM,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,CAAC,IAAI,EAAE,GAAG,QAAQ,CAAC,EAAE,MAAM,EAAE,eAAe,EAAE,CAAC,CAAC;QACvE,CAAC;IACH,CAAC;IAED,sBAAsB;IACtB,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,CAAC,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC;YAAE,SAAS;QACnC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;QACvB,MAAM,CAAC,IAAI,CAAC,EAAE,KAAK,EAAE,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,aAAa,EAAE,CAAC,CAAC;IACxD,CAAC;IAED,OAAO,MAAM,CAAC;AAChB,CAAC;AAED,SAAS,UAAU,CAAC,EAAU,EAAE,KAAmB,EAAE,MAAoB;IACvE,OAAO;QACL,EAAE,EAAE,UAAU,EAAE,EAAE;QAClB,KAAK;QACL,UAAU,EAAE,KAAK,CAAC,MAAM,CAAC,CAAC,GAAG,EAAE,CAAC,EAAE,EAAE,CAAC,GAAG,GAAG,CAAC,CAAC,IAAI,CAAC,MAAM,EAAE,CAAC,CAAC;QAC5D,YAAY,EAAE,MAAM;KACrB,CAAC;AACJ,CAAC"}

package/dist/diff-parser.d.ts

@@ -0,0 +1,46 @@
+import type { FileDiff, Hunk } from "./types.js";
+/**
+ * Normalize a diff line: trim whitespace, strip leading +/- diff markers, trim \r.
+ * Mirrors open-code-review's normalizeLine.
+ */
+export declare function normalizeLine(line: string): string;
+/**
+ * Split code text into normalized non-blank lines.
+ */
+export declare function splitAndNormalize(code: string): string[];
+/**
+ * Parse @@ ... @@ blocks from raw diff text into Hunk[].
+ * Lines before the first @@ header (file-level headers) are ignored.
+ */
+export declare function parseHunks(rawDiffText: string): Hunk[];
+/**
+ * Parse a full unified diff (possibly multiple files) into FileDiff[].
+ * Uses the `diff` library's parsePatch for hunk line splitting, and
+ * raw-text scanning for git edge cases (rename/binary/new-file/deleted-file).
+ */
+export declare function parseFileDiffs(diffText: string): FileDiff[];
+/**
+ * Extract one side of a hunk as indexed lines (line number + normalized content).
+ * When newSide=true: context + added lines with new-file line numbers.
+ * When newSide=false: context + deleted lines with old-file line numbers.
+ * Mirrors open-code-review's extractSideLines.
+ */
+export interface IndexedLine {
+    lineNum: number;
+    content: string;
+}
+export declare function extractSideLines(hunk: Hunk, newSide: boolean): IndexedLine[];
+/**
+ * Scan sideLines for a consecutive run matching all targetLines.
+ * Returns the start/end line numbers of the match, or null.
+ * Mirrors open-code-review's matchConsecutive.
+ */
+export declare function matchConsecutive(sideLines: IndexedLine[], targetLines: string[]): {
+    start: number;
+    end: number;
+} | null;
+/**
+ * Get the set of changed (added) line numbers in a hunk (new-file line numbers).
+ */
+export declare function getAddedLineNumbers(hunks: Hunk[]): Set<number>;
+//# sourceMappingURL=diff-parser.d.ts.map

package/dist/diff-parser.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-parser.d.ts","sourceRoot":"","sources":["../src/diff-parser.ts"],"names":[],"mappings":"AACA,OAAO,KAAK,EAAE,QAAQ,EAAE,IAAI,EAA0B,MAAM,YAAY,CAAC;AAIzE;;;GAGG;AACH,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,CAMlD;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,EAAE,CAKxD;AAwBD;;;GAGG;AACH,wBAAgB,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,IAAI,EAAE,CAsCtD;AAoFD;;;;GAIG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,MAAM,GAAG,QAAQ,EAAE,CAuD3D;AAwBD;;;;;GAKG;AACH,MAAM,WAAW,WAAW;IAC1B,OAAO,EAAE,MAAM,CAAC;IAChB,OAAO,EAAE,MAAM,CAAC;CACjB;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,IAAI,EAAE,OAAO,EAAE,OAAO,GAAG,WAAW,EAAE,CA8B5E;AAED;;;;GAIG;AACH,wBAAgB,gBAAgB,CAC9B,SAAS,EAAE,WAAW,EAAE,EACxB,WAAW,EAAE,MAAM,EAAE,GACpB;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,GAAG,EAAE,MAAM,CAAA;CAAE,GAAG,IAAI,CAoBvC;AAED;;GAEG;AACH,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,IAAI,EAAE,GAAG,GAAG,CAAC,MAAM,CAAC,CAU9D"}