@heyhuynhgiabuu/pi-diff 0.6.4 → 0.6.6

package/dist/core/resolve-lines.js

@@ -1,204 +0,0 @@
-/**
- * resolve-lines.ts — Hunk-based line number resolution
- *
- * Matches an `existing_code` snippet from an LLM review comment against
- * parsed diff hunks to backfill `startLine`/`endLine`. Ported from
- * alibaba/open-code-review's internal/diff/resolver.go three-tier approach.
- *
- * Tiers:
- *   1. New-side match (context + added lines with new-file line numbers)
- *   2. Old-side match (context + deleted lines with old-file line numbers)
- *   3. File-content scan (consecutive match in full new-file content)
- */
-import { parsePatchFiles } from "./diff.js";
-// ---------------------------------------------------------------------------
-// Normalization
-// ---------------------------------------------------------------------------
-/**
- * Normalize a single code line for comparison.
- * - Trims leading/trailing whitespace
- * - Strips leading `+` or `-` diff markers
- * - Re-trims after stripping
- */
-function normalizeLine(s) {
-    s = s.trim();
-    s = s.startsWith("+") || s.startsWith("-") ? s.slice(1).trim() : s.trim();
-    return s;
-}
-/**
- * Split `existing_code` into normalized non-empty lines.
- * Empty lines after normalization are dropped (mirrors OCR behavior).
- */
-function splitAndNormalize(code) {
-    const raw = code.split("\n");
-    const result = [];
-    for (const line of raw) {
-        const n = normalizeLine(line);
-        if (n === "")
-            continue;
-        result.push(n);
-    }
-    return result;
-}
-/**
- * Reconstruct hunks from a flat ParsedDiff.
- * Each `sep` marker with HunkMeta starts a new hunk. Context lines
- * appear on both sides; `add` lines on new-side only; `del` lines on
- * old-side only.
- */
-function reconstructHunks(parsed) {
-    const hunks = [];
-    let currentHunk = null;
-    let oldNum = 0;
-    let newNum = 0;
-    for (const line of parsed.lines) {
-        if (line.type === "sep" && line.hunkMeta) {
-            // Start a new hunk (or the first one)
-            currentHunk = { newSide: [], oldSide: [] };
-            hunks.push(currentHunk);
-            oldNum = line.hunkMeta.oldStart;
-            newNum = line.hunkMeta.newStart;
-            continue;
-        }
-        if (!currentHunk)
-            continue; // lines before first sep — shouldn't happen
-        switch (line.type) {
-            case "ctx":
-                currentHunk.newSide.push({ lineNum: newNum++, content: normalizeLine(line.content) });
-                currentHunk.oldSide.push({ lineNum: oldNum++, content: normalizeLine(line.content) });
-                break;
-            case "add":
-                currentHunk.newSide.push({ lineNum: newNum++, content: normalizeLine(line.content) });
-                break;
-            case "del":
-                currentHunk.oldSide.push({ lineNum: oldNum++, content: normalizeLine(line.content) });
-                break;
-            // "sep" without hunkMeta (e.g., multi-edit separators) — skip
-        }
-    }
-    return hunks;
-}
-// ---------------------------------------------------------------------------
-// Sliding-window match
-// ---------------------------------------------------------------------------
-/**
- * Scan `sideLines` for a consecutive run matching all `targetLines`.
- * Returns the matching file line range, or undefined.
- */
-function matchConsecutive(sideLines, targetLines) {
-    if (targetLines.length === 0 || sideLines.length < targetLines.length)
-        return undefined;
-    for (let i = 0; i <= sideLines.length - targetLines.length; i++) {
-        let matched = true;
-        for (let j = 0; j < targetLines.length; j++) {
-            if (sideLines[i + j].content !== targetLines[j]) {
-                matched = false;
-                break;
-            }
-        }
-        if (matched) {
-            return {
-                startLine: sideLines[i].lineNum,
-                endLine: sideLines[i + targetLines.length - 1].lineNum,
-            };
-        }
-    }
-    return undefined;
-}
-// ---------------------------------------------------------------------------
-// File-content fallback
-// ---------------------------------------------------------------------------
-/**
- * Scan full file content line-by-line for a consecutive match of target lines.
- * Returns 1-based line numbers.
- */
-function matchInFileContent(fileContent, targetLines) {
-    if (!fileContent || targetLines.length === 0)
-        return undefined;
-    const fileLines = fileContent.split("\n");
-    if (fileLines.length < targetLines.length)
-        return undefined;
-    for (let i = 0; i <= fileLines.length - targetLines.length; i++) {
-        let matched = true;
-        for (let j = 0; j < targetLines.length; j++) {
-            const fileLine = fileLines[i + j].replace(/\r$/, "").trim();
-            if (fileLine !== targetLines[j]) {
-                matched = false;
-                break;
-            }
-        }
-        if (matched) {
-            return { startLine: i + 1, endLine: i + targetLines.length };
-        }
-    }
-    return undefined;
-}
-// ---------------------------------------------------------------------------
-// Public API
-// ---------------------------------------------------------------------------
-/**
- * Resolve line numbers for an `existing_code` snippet against a parsed diff.
- *
- * Tries three tiers:
- *   1. New-side hunk match (context + added lines)
- *   2. Old-side hunk match (context + deleted lines)
- *   3. Full file-content scan (if `fileContent` is provided)
- *
- * @param existingCode - The code snippet from the LLM comment
- * @param parsedDiff   - Parsed diff for the file (from parsePatchFiles or parseDiff)
- * @param fileContent  - Optional full new-file content for fallback scan
- * @returns Resolved line numbers or Unresolved
- */
-export function resolveLines(existingCode, parsedDiff, fileContent) {
-    if (!existingCode || !parsedDiff || parsedDiff.lines.length === 0) {
-        return { unresolved: true };
-    }
-    const targetLines = splitAndNormalize(existingCode);
-    if (targetLines.length === 0) {
-        return { unresolved: true };
-    }
-    const hunks = reconstructHunks(parsedDiff);
-    if (hunks.length === 0) {
-        return { unresolved: true };
-    }
-    // Tier 1: New-side match (context + added → new-file line numbers)
-    for (const hunk of hunks) {
-        const result = matchConsecutive(hunk.newSide, targetLines);
-        if (result)
-            return result;
-    }
-    // Tier 2: Old-side match (context + deleted → old-file line numbers)
-    for (const hunk of hunks) {
-        const result = matchConsecutive(hunk.oldSide, targetLines);
-        if (result)
-            return result;
-    }
-    // Tier 3: File-content fallback
-    if (fileContent) {
-        const result = matchInFileContent(fileContent, targetLines);
-        if (result)
-            return result;
-    }
-    return { unresolved: true };
-}
-/**
- * Convenience wrapper: resolve from a raw unified diff string instead of
- * a pre-parsed ParsedDiff. Parses the diff internally.
- *
- * @param existingCode - The code snippet from the LLM comment
- * @param unifiedDiff  - Raw unified diff text (single file)
- * @param fileContent  - Optional full new-file content for fallback scan
- */
-export function resolveLinesFromPatch(existingCode, unifiedDiff, fileContent) {
-    const parsed = parsePatchFiles(unifiedDiff);
-    if (parsed.length === 0)
-        return { unresolved: true };
-    // For multi-file patches, try each file's diff
-    for (const pd of parsed) {
-        const result = resolveLines(existingCode, pd, fileContent);
-        if (!("unresolved" in result))
-            return result;
-    }
-    return { unresolved: true };
-}
-//# sourceMappingURL=resolve-lines.js.map

package/dist/core/resolve-lines.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"resolve-lines.js","sourceRoot":"","sources":["../../src/core/resolve-lines.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAGH,OAAO,EAAE,eAAe,EAAE,MAAM,WAAW,CAAC;AAiB5C,8EAA8E;AAC9E,gBAAgB;AAChB,8EAA8E;AAE9E;;;;;GAKG;AACH,SAAS,aAAa,CAAC,CAAS;IAC/B,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,CAAC;IACb,CAAC,GAAG,CAAC,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,CAAC,UAAU,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC;IAC1E,OAAO,CAAC,CAAC;AACV,CAAC;AAED;;;GAGG;AACH,SAAS,iBAAiB,CAAC,IAAY;IACtC,MAAM,GAAG,GAAG,IAAI,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC7B,MAAM,MAAM,GAAa,EAAE,CAAC;IAC5B,KAAK,MAAM,IAAI,IAAI,GAAG,EAAE,CAAC;QACxB,MAAM,CAAC,GAAG,aAAa,CAAC,IAAI,CAAC,CAAC;QAC9B,IAAI,CAAC,KAAK,EAAE;YAAE,SAAS;QACvB,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC;IAChB,CAAC;IACD,OAAO,MAAM,CAAC;AACf,CAAC;AAsBD;;;;;GAKG;AACH,SAAS,gBAAgB,CAAC,MAAkB;IAC3C,MAAM,KAAK,GAAkB,EAAE,CAAC;IAChC,IAAI,WAAW,GAAuB,IAAI,CAAC;IAC3C,IAAI,MAAM,GAAG,CAAC,CAAC;IACf,IAAI,MAAM,GAAG,CAAC,CAAC;IAEf,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,KAAK,EAAE,CAAC;QACjC,IAAI,IAAI,CAAC,IAAI,KAAK,KAAK,IAAI,IAAI,CAAC,QAAQ,EAAE,CAAC;YAC1C,sCAAsC;YACtC,WAAW,GAAG,EAAE,OAAO,EAAE,EAAE,EAAE,OAAO,EAAE,EAAE,EAAE,CAAC;YAC3C,KAAK,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;YACxB,MAAM,GAAG,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC;YAChC,MAAM,GAAG,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC;YAChC,SAAS;QACV,CAAC;QAED,IAAI,CAAC,WAAW;YAAE,SAAS,CAAC,4CAA4C;QAExE,QAAQ,IAAI,CAAC,IAAI,EAAE,CAAC;YACnB,KAAK,KAAK;gBACT,WAAW,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,OAAO,EAAE,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;gBACtF,WAAW,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,OAAO,EAAE,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;gBACtF,MAAM;YACP,KAAK,KAAK;gBACT,WAAW,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,OAAO,EAAE,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;gBACtF,MAAM;YACP,KAAK,KAAK;gBACT,WAAW,CAAC,OAAO,CAAC,IAAI,CAAC,EAAE,OAAO,EAAE,MAAM,EAAE,EAAE,OAAO,EAAE,aAAa,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;gBACtF,MAAM;YACP,8DAA8D;QAC/D,CAAC;IACF,CAAC;IAED,OAAO,KAAK,CAAC;AACd,CAAC;AAED,8EAA8E;AAC9E,uBAAuB;AACvB,8EAA8E;AAE9E;;;GAGG;AACH,SAAS,gBAAgB,CACxB,SAAwB,EACxB,WAAqB;IAErB,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,IAAI,SAAS,CAAC,MAAM,GAAG,WAAW,CAAC,MAAM;QAAE,OAAO,SAAS,CAAC;IAExF,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,SAAS,CAAC,MAAM,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACjE,IAAI,OAAO,GAAG,IAAI,CAAC;QACnB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YAC7C,IAAI,SAAS,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,OAAO,KAAK,WAAW,CAAC,CAAC,CAAC,EAAE,CAAC;gBACjD,OAAO,GAAG,KAAK,CAAC;gBAChB,MAAM;YACP,CAAC;QACF,CAAC;QACD,IAAI,OAAO,EAAE,CAAC;YACb,OAAO;gBACN,SAAS,EAAE,SAAS,CAAC,CAAC,CAAC,CAAC,OAAO;gBAC/B,OAAO,EAAE,SAAS,CAAC,CAAC,GAAG,WAAW,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,OAAO;aACtD,CAAC;QACH,CAAC;IACF,CAAC;IACD,OAAO,SAAS,CAAC;AAClB,CAAC;AAED,8EAA8E;AAC9E,wBAAwB;AACxB,8EAA8E;AAE9E;;;GAGG;AACH,SAAS,kBAAkB,CAC1B,WAAmB,EACnB,WAAqB;IAErB,IAAI,CAAC,WAAW,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,SAAS,CAAC;IAE/D,MAAM,SAAS,GAAG,WAAW,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAC1C,IAAI,SAAS,CAAC,MAAM,GAAG,WAAW,CAAC,MAAM;QAAE,OAAO,SAAS,CAAC;IAE5D,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,IAAI,SAAS,CAAC,MAAM,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACjE,IAAI,OAAO,GAAG,IAAI,CAAC;QACnB,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YAC7C,MAAM,QAAQ,GAAG,SAAS,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,OAAO,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,IAAI,EAAE,CAAC;YAC5D,IAAI,QAAQ,KAAK,WAAW,CAAC,CAAC,CAAC,EAAE,CAAC;gBACjC,OAAO,GAAG,KAAK,CAAC;gBAChB,MAAM;YACP,CAAC;QACF,CAAC;QACD,IAAI,OAAO,EAAE,CAAC;YACb,OAAO,EAAE,SAAS,EAAE,CAAC,GAAG,CAAC,EAAE,OAAO,EAAE,CAAC,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC;QAC9D,CAAC;IACF,CAAC;IACD,OAAO,SAAS,CAAC;AAClB,CAAC;AAED,8EAA8E;AAC9E,aAAa;AACb,8EAA8E;AAE9E;;;;;;;;;;;;GAYG;AACH,MAAM,UAAU,YAAY,CAC3B,YAAoB,EACpB,UAAsB,EACtB,WAAoB;IAEpB,IAAI,CAAC,YAAY,IAAI,CAAC,UAAU,IAAI,UAAU,CAAC,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACnE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC;IAC7B,CAAC;IAED,MAAM,WAAW,GAAG,iBAAiB,CAAC,YAAY,CAAC,CAAC;IACpD,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QAC9B,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC;IAC7B,CAAC;IAED,MAAM,KAAK,GAAG,gBAAgB,CAAC,UAAU,CAAC,CAAC;IAC3C,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACxB,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC;IAC7B,CAAC;IAED,mEAAmE;IACnE,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QAC1B,MAAM,MAAM,GAAG,gBAAgB,CAAC,IAAI,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;QAC3D,IAAI,MAAM;YAAE,OAAO,MAAM,CAAC;IAC3B,CAAC;IAED,qEAAqE;IACrE,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QAC1B,MAAM,MAAM,GAAG,gBAAgB,CAAC,IAAI,CAAC,OAAO,EAAE,WAAW,CAAC,CAAC;QAC3D,IAAI,MAAM;YAAE,OAAO,MAAM,CAAC;IAC3B,CAAC;IAED,gCAAgC;IAChC,IAAI,WAAW,EAAE,CAAC;QACjB,MAAM,MAAM,GAAG,kBAAkB,CAAC,WAAW,EAAE,WAAW,CAAC,CAAC;QAC5D,IAAI,MAAM;YAAE,OAAO,MAAM,CAAC;IAC3B,CAAC;IAED,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC;AAC7B,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,qBAAqB,CACpC,YAAoB,EACpB,WAAmB,EACnB,WAAoB;IAEpB,MAAM,MAAM,GAAG,eAAe,CAAC,WAAW,CAAC,CAAC;IAC5C,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC;IACrD,+CAA+C;IAC/C,KAAK,MAAM,EAAE,IAAI,MAAM,EAAE,CAAC;QACzB,MAAM,MAAM,GAAG,YAAY,CAAC,YAAY,EAAE,EAAE,EAAE,WAAW,CAAC,CAAC;QAC3D,IAAI,CAAC,CAAC,YAAY,IAAI,MAAM,CAAC;YAAE,OAAO,MAAM,CAAC;IAC9C,CAAC;IACD,OAAO,EAAE,UAAU,EAAE,IAAI,EAAE,CAAC;AAC7B,CAAC"}

package/dist/review/command.d.ts

@@ -1,3 +0,0 @@
-import type { ExtensionAPI } from "@earendil-works/pi-coding-agent";
-export declare function registerReviewDiffCommand(pi: ExtensionAPI, cwd: string): void;
-//# sourceMappingURL=command.d.ts.map

package/dist/review/command.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"command.d.ts","sourceRoot":"","sources":["../../src/review/command.ts"],"names":[],"mappings":"AAGA,OAAO,KAAK,EAAE,YAAY,EAA4J,MAAM,iCAAiC,CAAC;AAmH9N,wBAAgB,yBAAyB,CAAC,EAAE,EAAE,YAAY,EAAE,GAAG,EAAE,MAAM,GAAG,IAAI,CAmF7E"}