ghagga-core 2.6.0 → 2.8.0

package/dist/recursive/patch-extractor.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"patch-extractor.d.ts","sourceRoot":"","sources":["../../src/recursive/patch-extractor.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AACjD,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,YAAY,CAAC;AAIlD;;;;;;;;GAQG;AACH,wBAAgB,cAAc,CAAC,QAAQ,EAAE,aAAa,EAAE,GAAG,eAAe,EAAE,CAqB3E;AAID;;;;;;;;;;;;;;;;;GAiBG;AACH,wBAAgB,mBAAmB,CAAC,YAAY,EAAE,MAAM,EAAE,OAAO,EAAE,eAAe,EAAE,GAAG,MAAM,CA+D5F;AAED;;;;;;;;;GASG;AACH,wBAAgB,iBAAiB,CAAC,OAAO,EAAE,eAAe,EAAE,GAAG,MAAM,CAWpE"}

package/dist/recursive/patch-extractor.js

@@ -0,0 +1,133 @@
+/**
+ * Patch Extractor — extracts suggestion patches from review findings
+ * and applies them virtually to produce a synthetic diff.
+ *
+ * No filesystem access — operates purely on diff strings.
+ */
+// ─── Patch Extraction ──────────────────────────────────────────
+/**
+ * Extract actionable patches from review findings.
+ *
+ * Only findings with both a `suggestion` and a `file` are included.
+ * Findings without a line number are included but marked as file-level.
+ *
+ * @param findings - Review findings from the AI agent
+ * @returns Array of suggestion patches ready for virtual application
+ */
+export function extractPatches(findings) {
+    const patches = [];
+    for (let i = 0; i < findings.length; i++) {
+        const finding = findings[i];
+        if (!finding)
+            continue;
+        if (!finding.suggestion || !finding.file)
+            continue;
+        // Skip empty/whitespace-only suggestions
+        if (finding.suggestion.trim().length === 0)
+            continue;
+        patches.push({
+            file: finding.file,
+            line: finding.line,
+            originalMessage: finding.message,
+            suggestion: finding.suggestion,
+            findingIndex: i,
+        });
+    }
+    return patches;
+}
+// ─── Virtual Patch Application ─────────────────────────────────
+/**
+ * Apply suggestion patches to an original diff, producing a synthetic
+ * patched diff for re-review.
+ *
+ * Strategy:
+ * - For each file in the diff, collect patches targeting that file.
+ * - Sort patches by line number descending (bottom-up) to avoid offset shifts.
+ * - For each patch, find the target line in the diff's added lines (+)
+ *   and append a comment showing the suggested replacement.
+ *
+ * The patched diff is a SYNTHETIC representation — it shows what the code
+ * would look like if the suggestions were applied. This is NOT a real
+ * git diff; it's a review-friendly format for the LLM to analyze.
+ *
+ * @param originalDiff - The original unified diff string
+ * @param patches - Suggestion patches to apply
+ * @returns Synthetic diff with suggestions applied
+ */
+export function applyVirtualPatches(originalDiff, patches) {
+    if (patches.length === 0)
+        return originalDiff;
+    // Group patches by file
+    const patchesByFile = new Map();
+    for (const patch of patches) {
+        const existing = patchesByFile.get(patch.file) ?? [];
+        existing.push(patch);
+        patchesByFile.set(patch.file, existing);
+    }
+    // Build the synthetic diff
+    const lines = originalDiff.split('\n');
+    const result = [];
+    let currentFile = null;
+    let currentFilePatches = [];
+    let lineCounter = 0; // Tracks the target-side line number in current hunk
+    for (const line of lines) {
+        // Detect file boundaries
+        const fileMatch = /^diff --git a\/.+ b\/(.+)$/.exec(line);
+        if (fileMatch?.[1]) {
+            currentFile = fileMatch[1];
+            currentFilePatches = patchesByFile.get(currentFile) ?? [];
+            lineCounter = 0;
+        }
+        // Track hunk line numbers
+        const hunkMatch = /^@@ -\d+(?:,\d+)? \+(\d+)/.exec(line);
+        if (hunkMatch?.[1]) {
+            lineCounter = parseInt(hunkMatch[1], 10) - 1;
+        }
+        // Count lines for position tracking (added or context lines increment target counter)
+        if (line.startsWith('+') && !line.startsWith('+++')) {
+            lineCounter++;
+        }
+        else if (line.startsWith('-') && !line.startsWith('---')) {
+            // Removed lines don't increment target counter
+        }
+        else if (!line.startsWith('\\') &&
+            !line.startsWith('diff ') &&
+            !line.startsWith('index ') &&
+            !line.startsWith('---') &&
+            !line.startsWith('+++') &&
+            !line.startsWith('@@')) {
+            lineCounter++;
+        }
+        result.push(line);
+        // Check if any patch targets this line
+        if (currentFile && currentFilePatches.length > 0) {
+            const matchingPatches = currentFilePatches.filter((p) => p.line === lineCounter);
+            for (const patch of matchingPatches) {
+                // Insert the suggestion as a synthetic replacement block
+                result.push(`+[SUGGESTED FIX] ${patch.suggestion}`);
+            }
+        }
+    }
+    return result.join('\n');
+}
+/**
+ * Build a focused review context for re-review.
+ *
+ * Instead of passing the entire diff, build a summary of what
+ * suggestions were applied and where, so the re-review LLM
+ * can focus on validating those specific changes.
+ *
+ * @param patches - The patches that were applied
+ * @returns Context string for the re-review prompt
+ */
+export function buildPatchContext(patches) {
+    if (patches.length === 0)
+        return '';
+    const lines = ['## Applied Suggestions (validate these for regressions)', ''];
+    for (const patch of patches) {
+        lines.push(`- **${patch.file}${patch.line ? `:${patch.line}` : ''}**: ${patch.suggestion}`);
+        lines.push(`  (Original issue: ${patch.originalMessage})`);
+    }
+    return lines.join('\n');
+}
+//# sourceMappingURL=patch-extractor.js.map

package/dist/recursive/patch-extractor.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"patch-extractor.js","sourceRoot":"","sources":["../../src/recursive/patch-extractor.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAKH,kEAAkE;AAElE;;;;;;;;GAQG;AACH,MAAM,UAAU,cAAc,CAAC,QAAyB;IACtD,MAAM,OAAO,GAAsB,EAAE,CAAC;IAEtC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,QAAQ,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;QACzC,MAAM,OAAO,GAAG,QAAQ,CAAC,CAAC,CAAC,CAAC;QAC5B,IAAI,CAAC,OAAO;YAAE,SAAS;QACvB,IAAI,CAAC,OAAO,CAAC,UAAU,IAAI,CAAC,OAAO,CAAC,IAAI;YAAE,SAAS;QAEnD,yCAAyC;QACzC,IAAI,OAAO,CAAC,UAAU,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC;YAAE,SAAS;QAErD,OAAO,CAAC,IAAI,CAAC;YACX,IAAI,EAAE,OAAO,CAAC,IAAI;YAClB,IAAI,EAAE,OAAO,CAAC,IAAI;YAClB,eAAe,EAAE,OAAO,CAAC,OAAO;YAChC,UAAU,EAAE,OAAO,CAAC,UAAU;YAC9B,YAAY,EAAE,CAAC;SAChB,CAAC,CAAC;IACL,CAAC;IAED,OAAO,OAAO,CAAC;AACjB,CAAC;AAED,kEAAkE;AAElE;;;;;;;;;;;;;;;;;GAiBG;AACH,MAAM,UAAU,mBAAmB,CAAC,YAAoB,EAAE,OAA0B;IAClF,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,YAAY,CAAC;IAE9C,wBAAwB;IACxB,MAAM,aAAa,GAAG,IAAI,GAAG,EAA6B,CAAC;IAC3D,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,MAAM,QAAQ,GAAG,aAAa,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC;QACrD,QAAQ,CAAC,IAAI,CAAC,KAAK,CAAC,CAAC;QACrB,aAAa,CAAC,GAAG,CAAC,KAAK,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC;IAC1C,CAAC;IAED,2BAA2B;IAC3B,MAAM,KAAK,GAAG,YAAY,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IACvC,MAAM,MAAM,GAAa,EAAE,CAAC;IAE5B,IAAI,WAAW,GAAkB,IAAI,CAAC;IACtC,IAAI,kBAAkB,GAAsB,EAAE,CAAC;IAC/C,IAAI,WAAW,GAAG,CAAC,CAAC,CAAC,qDAAqD;IAE1E,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,yBAAyB;QACzB,MAAM,SAAS,GAAG,4BAA4B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAC1D,IAAI,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YACnB,WAAW,GAAG,SAAS,CAAC,CAAC,CAAC,CAAC;YAC3B,kBAAkB,GAAG,aAAa,CAAC,GAAG,CAAC,WAAW,CAAC,IAAI,EAAE,CAAC;YAC1D,WAAW,GAAG,CAAC,CAAC;QAClB,CAAC;QAED,0BAA0B;QAC1B,MAAM,SAAS,GAAG,2BAA2B,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACzD,IAAI,SAAS,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YACnB,WAAW,GAAG,QAAQ,CAAC,SAAS,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,GAAG,CAAC,CAAC;QAC/C,CAAC;QAED,sFAAsF;QACtF,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,EAAE,CAAC;YACpD,WAAW,EAAE,CAAC;QAChB,CAAC;aAAM,IAAI,IAAI,CAAC,UAAU,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC,EAAE,CAAC;YAC3D,+CAA+C;QACjD,CAAC;aAAM,IACL,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC;YACtB,CAAC,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC;YACzB,CAAC,IAAI,CAAC,UAAU,CAAC,QAAQ,CAAC;YAC1B,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;YACvB,CAAC,IAAI,CAAC,UAAU,CAAC,KAAK,CAAC;YACvB,CAAC,IAAI,CAAC,UAAU,CAAC,IAAI,CAAC,EACtB,CAAC;YACD,WAAW,EAAE,CAAC;QAChB,CAAC;QAED,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAElB,uCAAuC;QACvC,IAAI,WAAW,IAAI,kBAAkB,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YACjD,MAAM,eAAe,GAAG,kBAAkB,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,WAAW,CAAC,CAAC;YACjF,KAAK,MAAM,KAAK,IAAI,eAAe,EAAE,CAAC;gBACpC,yDAAyD;gBACzD,MAAM,CAAC,IAAI,CAAC,oBAAoB,KAAK,CAAC,UAAU,EAAE,CAAC,CAAC;YACtD,CAAC;QACH,CAAC;IACH,CAAC;IAED,OAAO,MAAM,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC3B,CAAC;AAED;;;;;;;;;GASG;AACH,MAAM,UAAU,iBAAiB,CAAC,OAA0B;IAC1D,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAEpC,MAAM,KAAK,GAAG,CAAC,yDAAyD,EAAE,EAAE,CAAC,CAAC;IAE9E,KAAK,MAAM,KAAK,IAAI,OAAO,EAAE,CAAC;QAC5B,KAAK,CAAC,IAAI,CAAC,OAAO,KAAK,CAAC,IAAI,GAAG,KAAK,CAAC,IAAI,CAAC,CAAC,CAAC,IAAI,KAAK,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,EAAE,OAAO,KAAK,CAAC,UAAU,EAAE,CAAC,CAAC;QAC5F,KAAK,CAAC,IAAI,CAAC,sBAAsB,KAAK,CAAC,eAAe,GAAG,CAAC,CAAC;IAC7D,CAAC;IAED,OAAO,KAAK,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;AAC1B,CAAC"}

package/dist/recursive/re-reviewer.d.ts

@@ -0,0 +1,31 @@
+/**
+ * Re-Reviewer — runs a lightweight review on patched diffs
+ * to detect regressions introduced by suggested fixes.
+ */
+import type { GenerateTextFn } from '../providers/generate-fn.js';
+import type { ReviewFinding } from '../types.js';
+export interface ReReviewInput {
+    /** Synthetic diff with suggestions applied */
+    patchedDiff: string;
+    /** Context about which patches were applied */
+    patchContext: string;
+    /** Generation function for the LLM call */
+    generateFn: GenerateTextFn;
+}
+export interface ReReviewResult {
+    /** New findings from the re-review */
+    findings: ReviewFinding[];
+    /** Tokens used for this re-review call */
+    tokensUsed: number;
+}
+/**
+ * Run a lightweight re-review on a patched diff.
+ *
+ * Uses simple mode (single LLM call) to check whether the
+ * applied suggestions introduce new problems.
+ *
+ * @param input - Patched diff and LLM config
+ * @returns New findings from the re-review
+ */
+export declare function runReReview(input: ReReviewInput): Promise<ReReviewResult>;
+//# sourceMappingURL=re-reviewer.d.ts.map

package/dist/recursive/re-reviewer.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"re-reviewer.d.ts","sourceRoot":"","sources":["../../src/recursive/re-reviewer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAGH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AA+BjD,MAAM,WAAW,aAAa;IAC5B,8CAA8C;IAC9C,WAAW,EAAE,MAAM,CAAC;IAEpB,+CAA+C;IAC/C,YAAY,EAAE,MAAM,CAAC;IAErB,2CAA2C;IAC3C,UAAU,EAAE,cAAc,CAAC;CAC5B;AAED,MAAM,WAAW,cAAc;IAC7B,sCAAsC;IACtC,QAAQ,EAAE,aAAa,EAAE,CAAC;IAE1B,0CAA0C;IAC1C,UAAU,EAAE,MAAM,CAAC;CACpB;AAID;;;;;;;;GAQG;AACH,wBAAsB,WAAW,CAAC,KAAK,EAAE,aAAa,GAAG,OAAO,CAAC,cAAc,CAAC,CA6B/E"}

package/dist/recursive/re-reviewer.js

@@ -0,0 +1,66 @@
+/**
+ * Re-Reviewer — runs a lightweight review on patched diffs
+ * to detect regressions introduced by suggested fixes.
+ */
+import { parseFindingsBlock } from '../agents/simple.js';
+// ─── Re-Review Prompt ──────────────────────────────────────────
+const RE_REVIEW_SYSTEM = `You are a code review validator. You are reviewing code changes that include SUGGESTED FIXES from a previous code review.
+
+Your ONLY job is to check whether the suggested fixes introduce NEW problems:
+- New bugs or logic errors from the fix
+- Security issues introduced by the fix
+- Performance regressions from the fix
+- Breaking changes caused by the fix
+
+DO NOT repeat the original findings. ONLY report NEW issues introduced by the suggested changes.
+Lines marked with [SUGGESTED FIX] are the applied suggestions — focus your analysis there.
+
+Format your response EXACTLY as:
+
+STATUS: [PASSED or FAILED]
+SUMMARY: [1-2 sentences about whether suggestions introduced issues]
+FINDINGS:
+- SEVERITY: [critical|high|medium|low|info]
+  CATEGORY: [security|performance|bug|style|error-handling|maintainability]
+  FILE: [file path]
+  LINE: [line number or "N/A"]
+  MESSAGE: [description of the NEW issue introduced by the suggestion]
+  SUGGESTION: [how to fix the regression]
+
+If no new issues were introduced, return STATUS: PASSED with an empty FINDINGS section.`;
+// ─── Main Function ─────────────────────────────────────────────
+/**
+ * Run a lightweight re-review on a patched diff.
+ *
+ * Uses simple mode (single LLM call) to check whether the
+ * applied suggestions introduce new problems.
+ *
+ * @param input - Patched diff and LLM config
+ * @returns New findings from the re-review
+ */
+export async function runReReview(input) {
+    const { patchedDiff, patchContext, generateFn } = input;
+    const system = [RE_REVIEW_SYSTEM, patchContext].filter(Boolean).join('\n\n');
+    const prompt = `Please validate the following code changes that include applied suggestions:\n\n\`\`\`diff\n${patchedDiff}\n\`\`\``;
+    try {
+        const result = await generateFn(system, prompt);
+        // Parse findings from the response
+        const findings = parseFindingsBlock(result.text);
+        // Mark all findings as coming from recursive review
+        for (const finding of findings) {
+            finding.source = 'recursive-review';
+        }
+        return {
+            findings,
+            tokensUsed: result.tokensUsed,
+        };
+    }
+    catch {
+        // Non-blocking — return empty findings on failure
+        return {
+            findings: [],
+            tokensUsed: 0,
+        };
+    }
+}
+//# sourceMappingURL=re-reviewer.js.map

package/dist/recursive/re-reviewer.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"re-reviewer.js","sourceRoot":"","sources":["../../src/recursive/re-reviewer.ts"],"names":[],"mappings":"AAAA;;;GAGG;AAEH,OAAO,EAAE,kBAAkB,EAAE,MAAM,qBAAqB,CAAC;AAIzD,kEAAkE;AAElE,MAAM,gBAAgB,GAAG;;;;;;;;;;;;;;;;;;;;;;;wFAuB+D,CAAC;AAuBzF,kEAAkE;AAElE;;;;;;;;GAQG;AACH,MAAM,CAAC,KAAK,UAAU,WAAW,CAAC,KAAoB;IACpD,MAAM,EAAE,WAAW,EAAE,YAAY,EAAE,UAAU,EAAE,GAAG,KAAK,CAAC;IAExD,MAAM,MAAM,GAAG,CAAC,gBAAgB,EAAE,YAAY,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC;IAE7E,MAAM,MAAM,GAAG,+FAA+F,WAAW,UAAU,CAAC;IAEpI,IAAI,CAAC;QACH,MAAM,MAAM,GAAG,MAAM,UAAU,CAAC,MAAM,EAAE,MAAM,CAAC,CAAC;QAEhD,mCAAmC;QACnC,MAAM,QAAQ,GAAG,kBAAkB,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC;QAEjD,oDAAoD;QACpD,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;YAC/B,OAAO,CAAC,MAAM,GAAG,kBAAkB,CAAC;QACtC,CAAC;QAED,OAAO;YACL,QAAQ;YACR,UAAU,EAAE,MAAM,CAAC,UAAU;SAC9B,CAAC;IACJ,CAAC;IAAC,MAAM,CAAC;QACP,kDAAkD;QAClD,OAAO;YACL,QAAQ,EAAE,EAAE;YACZ,UAAU,EAAE,CAAC;SACd,CAAC;IACJ,CAAC;AACH,CAAC"}

package/dist/recursive/types.d.ts

@@ -0,0 +1,62 @@
+/**
+ * Recursive Review Loop — Types
+ *
+ * Types for the self-validating review cycle that re-reviews
+ * suggested patches to catch regressions before they reach the user.
+ */
+import type { ReviewFinding } from '../types.js';
+/**
+ * A patch derived from a review finding's suggestion.
+ * Represents a single code change that the AI reviewer recommended.
+ */
+export interface SuggestionPatch {
+    /** File path from the finding */
+    file: string;
+    /** Line number from the finding (undefined = file-level suggestion) */
+    line: number | undefined;
+    /** The original finding message for context */
+    originalMessage: string;
+    /** The suggested fix text */
+    suggestion: string;
+    /** Index of the finding in the original results (for tracing) */
+    findingIndex: number;
+}
+/**
+ * A finding from re-review that represents a regression
+ * introduced by a suggested fix.
+ */
+export interface RegressionFinding extends ReviewFinding {
+    /** Always true — discriminant for regression findings */
+    isRegression: true;
+    /** The original suggestion that caused this regression */
+    originatingSuggestion: {
+        file: string;
+        line: number | undefined;
+        suggestion: string;
+    };
+}
+/**
+ * Report from the recursive review loop.
+ * Attached to ReviewResult when recursive review is enabled.
+ */
+export interface RecursiveReviewReport {
+    /** Number of re-review iterations executed (1 or 2) */
+    iterations: number;
+    /** Whether the review converged (no new issues on last iteration) */
+    converged: boolean;
+    /** Findings identified as regressions from suggested fixes */
+    regressions: RegressionFinding[];
+    /** Total new issues found across all re-review iterations */
+    totalNewIssues: number;
+}
+/**
+ * Configuration for the recursive review step.
+ * Extracted from ReviewSettings for internal use.
+ */
+export interface RecursiveReviewConfig {
+    /** Maximum number of re-review iterations. Default: 2 */
+    maxIterations: number;
+}
+/** Default configuration values */
+export declare const DEFAULT_RECURSIVE_CONFIG: RecursiveReviewConfig;
+//# sourceMappingURL=types.d.ts.map

package/dist/recursive/types.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../src/recursive/types.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,aAAa,CAAC;AAIjD;;;GAGG;AACH,MAAM,WAAW,eAAe;IAC9B,iCAAiC;IACjC,IAAI,EAAE,MAAM,CAAC;IAEb,uEAAuE;IACvE,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;IAEzB,+CAA+C;IAC/C,eAAe,EAAE,MAAM,CAAC;IAExB,6BAA6B;IAC7B,UAAU,EAAE,MAAM,CAAC;IAEnB,iEAAiE;IACjE,YAAY,EAAE,MAAM,CAAC;CACtB;AAID;;;GAGG;AACH,MAAM,WAAW,iBAAkB,SAAQ,aAAa;IACtD,yDAAyD;IACzD,YAAY,EAAE,IAAI,CAAC;IAEnB,0DAA0D;IAC1D,qBAAqB,EAAE;QACrB,IAAI,EAAE,MAAM,CAAC;QACb,IAAI,EAAE,MAAM,GAAG,SAAS,CAAC;QACzB,UAAU,EAAE,MAAM,CAAC;KACpB,CAAC;CACH;AAID;;;GAGG;AACH,MAAM,WAAW,qBAAqB;IACpC,uDAAuD;IACvD,UAAU,EAAE,MAAM,CAAC;IAEnB,qEAAqE;IACrE,SAAS,EAAE,OAAO,CAAC;IAEnB,8DAA8D;IAC9D,WAAW,EAAE,iBAAiB,EAAE,CAAC;IAEjC,6DAA6D;IAC7D,cAAc,EAAE,MAAM,CAAC;CACxB;AAID;;;GAGG;AACH,MAAM,WAAW,qBAAqB;IACpC,yDAAyD;IACzD,aAAa,EAAE,MAAM,CAAC;CACvB;AAED,mCAAmC;AACnC,eAAO,MAAM,wBAAwB,EAAE,qBAEtC,CAAC"}

package/dist/recursive/types.js

@@ -0,0 +1,11 @@
+/**
+ * Recursive Review Loop — Types
+ *
+ * Types for the self-validating review cycle that re-reviews
+ * suggested patches to catch regressions before they reach the user.
+ */
+/** Default configuration values */
+export const DEFAULT_RECURSIVE_CONFIG = {
+    maxIterations: 2,
+};
+//# sourceMappingURL=types.js.map

package/dist/recursive/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../src/recursive/types.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AA4EH,mCAAmC;AACnC,MAAM,CAAC,MAAM,wBAAwB,GAA0B;IAC7D,aAAa,EAAE,CAAC;CACjB,CAAC"}

package/dist/scope/context-builder.d.ts

@@ -0,0 +1,37 @@
+/**
+ * Scoped Context Builder
+ *
+ * Builds a focused review context string containing only the source
+ * code of symbols affected by a diff. This dramatically reduces the
+ * number of tokens sent to the LLM for review.
+ *
+ * Pure function with no side effects.
+ */
+import type { ScopedFile } from './types.js';
+/**
+ * Build a focused review context containing only the source code
+ * of symbols affected by diff changes.
+ *
+ * Output format:
+ * ```
+ * ## path/to/file.ts
+ *
+ * ### function foo (lines 10-25)
+ * ```ts
+ * <source lines 10-25>
+ * ```
+ *
+ * ### class Bar (lines 30-80)
+ * ```ts
+ * <source lines 30-80>
+ * ```
+ * ---
+ * ## path/to/other.ts
+ * ...
+ * ```
+ *
+ * @param scopedFiles - Files with their affected symbols and source lines
+ * @returns Formatted review context string, or empty string if no affected symbols
+ */
+export declare function buildScopedContext(scopedFiles: ScopedFile[]): string;
+//# sourceMappingURL=context-builder.d.ts.map

package/dist/scope/context-builder.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"context-builder.d.ts","sourceRoot":"","sources":["../../src/scope/context-builder.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH,OAAO,KAAK,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAe7C;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AACH,wBAAgB,kBAAkB,CAAC,WAAW,EAAE,UAAU,EAAE,GAAG,MAAM,CAepE"}

package/dist/scope/context-builder.js

@@ -0,0 +1,108 @@
+/**
+ * Scoped Context Builder
+ *
+ * Builds a focused review context string containing only the source
+ * code of symbols affected by a diff. This dramatically reduces the
+ * number of tokens sent to the LLM for review.
+ *
+ * Pure function with no side effects.
+ */
+// ─── Constants ────────────────────────────────────────────────
+/** Separator between file sections in the output. */
+const FILE_SEPARATOR = '\n---\n';
+/** Header format for each file section. */
+const FILE_HEADER_PREFIX = '## ';
+/** Header format for each symbol section. */
+const SYMBOL_HEADER_PREFIX = '### ';
+// ─── Builder ──────────────────────────────────────────────────
+/**
+ * Build a focused review context containing only the source code
+ * of symbols affected by diff changes.
+ *
+ * Output format:
+ * ```
+ * ## path/to/file.ts
+ *
+ * ### function foo (lines 10-25)
+ * ```ts
+ * <source lines 10-25>
+ * ```
+ *
+ * ### class Bar (lines 30-80)
+ * ```ts
+ * <source lines 30-80>
+ * ```
+ * ---
+ * ## path/to/other.ts
+ * ...
+ * ```
+ *
+ * @param scopedFiles - Files with their affected symbols and source lines
+ * @returns Formatted review context string, or empty string if no affected symbols
+ */
+export function buildScopedContext(scopedFiles) {
+    if (scopedFiles.length === 0)
+        return '';
+    const sections = [];
+    for (const file of scopedFiles) {
+        if (file.symbols.length === 0)
+            continue;
+        const fileSection = buildFileSection(file);
+        if (fileSection) {
+            sections.push(fileSection);
+        }
+    }
+    return sections.join(FILE_SEPARATOR);
+}
+// ─── Internals ────────────────────────────────────────────────
+/**
+ * Build the context section for a single file.
+ */
+function buildFileSection(file) {
+    const ext = inferCodeFenceLanguage(file.filePath);
+    const symbolSections = [];
+    // Deduplicate symbols by name+startLine to avoid repeating
+    // the same symbol if it appears in multiple hunks
+    const seen = new Set();
+    for (const affected of file.symbols) {
+        const { symbol } = affected;
+        const key = `${symbol.name}:${symbol.startLine}`;
+        if (seen.has(key))
+            continue;
+        seen.add(key);
+        const startIdx = Math.max(0, symbol.startLine - 1);
+        const endIdx = Math.min(file.sourceLines.length, symbol.endLine);
+        const sourceSlice = file.sourceLines.slice(startIdx, endIdx).join('\n');
+        const kindLabel = symbol.parent
+            ? `${symbol.kind} ${symbol.parent}.${symbol.name}`
+            : `${symbol.kind} ${symbol.name}`;
+        const lineRange = `lines ${symbol.startLine}-${symbol.endLine}`;
+        symbolSections.push(`${SYMBOL_HEADER_PREFIX}${kindLabel} (${lineRange})\n\`\`\`${ext}\n${sourceSlice}\n\`\`\``);
+    }
+    if (symbolSections.length === 0)
+        return undefined;
+    return `${FILE_HEADER_PREFIX}${file.filePath}\n\n${symbolSections.join('\n\n')}`;
+}
+/**
+ * Infer the code fence language hint from a file path.
+ */
+function inferCodeFenceLanguage(filePath) {
+    const ext = filePath.split('.').pop()?.toLowerCase() ?? '';
+    const MAP = {
+        ts: 'ts',
+        tsx: 'tsx',
+        js: 'js',
+        jsx: 'jsx',
+        py: 'python',
+        go: 'go',
+        rs: 'rust',
+        java: 'java',
+        rb: 'ruby',
+        cs: 'csharp',
+        cpp: 'cpp',
+        c: 'c',
+        h: 'c',
+    };
+    return MAP[ext] ?? ext;
+}
+//# sourceMappingURL=context-builder.js.map

package/dist/scope/context-builder.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"context-builder.js","sourceRoot":"","sources":["../../src/scope/context-builder.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAIH,iEAAiE;AAEjE,qDAAqD;AACrD,MAAM,cAAc,GAAG,SAAS,CAAC;AAEjC,2CAA2C;AAC3C,MAAM,kBAAkB,GAAG,KAAK,CAAC;AAEjC,6CAA6C;AAC7C,MAAM,oBAAoB,GAAG,MAAM,CAAC;AAEpC,iEAAiE;AAEjE;;;;;;;;;;;;;;;;;;;;;;;;GAwBG;AACH,MAAM,UAAU,kBAAkB,CAAC,WAAyB;IAC1D,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAExC,MAAM,QAAQ,GAAa,EAAE,CAAC;IAE9B,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;QAC/B,IAAI,IAAI,CAAC,OAAO,CAAC,MAAM,KAAK,CAAC;YAAE,SAAS;QAExC,MAAM,WAAW,GAAG,gBAAgB,CAAC,IAAI,CAAC,CAAC;QAC3C,IAAI,WAAW,EAAE,CAAC;YAChB,QAAQ,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC;QAC7B,CAAC;IACH,CAAC;IAED,OAAO,QAAQ,CAAC,IAAI,CAAC,cAAc,CAAC,CAAC;AACvC,CAAC;AAED,iEAAiE;AAEjE;;GAEG;AACH,SAAS,gBAAgB,CAAC,IAAgB;IACxC,MAAM,GAAG,GAAG,sBAAsB,CAAC,IAAI,CAAC,QAAQ,CAAC,CAAC;IAClD,MAAM,cAAc,GAAa,EAAE,CAAC;IAEpC,2DAA2D;IAC3D,kDAAkD;IAClD,MAAM,IAAI,GAAG,IAAI,GAAG,EAAU,CAAC;IAE/B,KAAK,MAAM,QAAQ,IAAI,IAAI,CAAC,OAAO,EAAE,CAAC;QACpC,MAAM,EAAE,MAAM,EAAE,GAAG,QAAQ,CAAC;QAC5B,MAAM,GAAG,GAAG,GAAG,MAAM,CAAC,IAAI,IAAI,MAAM,CAAC,SAAS,EAAE,CAAC;QACjD,IAAI,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC;YAAE,SAAS;QAC5B,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,CAAC;QAEd,MAAM,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,MAAM,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC;QACnD,MAAM,MAAM,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,WAAW,CAAC,MAAM,EAAE,MAAM,CAAC,OAAO,CAAC,CAAC;QACjE,MAAM,WAAW,GAAG,IAAI,CAAC,WAAW,CAAC,KAAK,CAAC,QAAQ,EAAE,MAAM,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QAExE,MAAM,SAAS,GAAG,MAAM,CAAC,MAAM;YAC7B,CAAC,CAAC,GAAG,MAAM,CAAC,IAAI,IAAI,MAAM,CAAC,MAAM,IAAI,MAAM,CAAC,IAAI,EAAE;YAClD,CAAC,CAAC,GAAG,MAAM,CAAC,IAAI,IAAI,MAAM,CAAC,IAAI,EAAE,CAAC;QAEpC,MAAM,SAAS,GAAG,SAAS,MAAM,CAAC,SAAS,IAAI,MAAM,CAAC,OAAO,EAAE,CAAC;QAEhE,cAAc,CAAC,IAAI,CACjB,GAAG,oBAAoB,GAAG,SAAS,KAAK,SAAS,YAAY,GAAG,KAAK,WAAW,UAAU,CAC3F,CAAC;IACJ,CAAC;IAED,IAAI,cAAc,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,SAAS,CAAC;IAElD,OAAO,GAAG,kBAAkB,GAAG,IAAI,CAAC,QAAQ,OAAO,cAAc,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC;AACnF,CAAC;AAED;;GAEG;AACH,SAAS,sBAAsB,CAAC,QAAgB;IAC9C,MAAM,GAAG,GAAG,QAAQ,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,GAAG,EAAE,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC;IAC3D,MAAM,GAAG,GAA2B;QAClC,EAAE,EAAE,IAAI;QACR,GAAG,EAAE,KAAK;QACV,EAAE,EAAE,IAAI;QACR,GAAG,EAAE,KAAK;QACV,EAAE,EAAE,QAAQ;QACZ,EAAE,EAAE,IAAI;QACR,EAAE,EAAE,MAAM;QACV,IAAI,EAAE,MAAM;QACZ,EAAE,EAAE,MAAM;QACV,EAAE,EAAE,QAAQ;QACZ,GAAG,EAAE,KAAK;QACV,CAAC,EAAE,GAAG;QACN,CAAC,EAAE,GAAG;KACP,CAAC;IACF,OAAO,GAAG,CAAC,GAAG,CAAC,IAAI,GAAG,CAAC;AACzB,CAAC"}

package/dist/scope/diff-mapper.d.ts

@@ -0,0 +1,29 @@
+/**
+ * Diff-to-Symbol Mapper
+ *
+ * Maps unified diff hunks to symbol definitions, identifying
+ * which symbols were affected by code changes.
+ *
+ * Pure functions with no side effects.
+ */
+import type { AffectedSymbol, DiffHunk, SymbolInfo } from './types.js';
+/**
+ * Parse unified diff content to extract hunk line ranges.
+ *
+ * @param diffContent - Raw unified diff content for a single file
+ * @returns Array of DiffHunk objects with line ranges
+ */
+export declare function parseHunks(diffContent: string): DiffHunk[];
+/**
+ * Map diff hunks to affected symbols.
+ *
+ * A symbol is considered "affected" if any hunk's NEW side line range
+ * overlaps with the symbol's line range. We use the new side because
+ * we're analyzing the post-change source code.
+ *
+ * @param hunks - Parsed diff hunks
+ * @param symbols - Extracted symbols from the NEW version of the file
+ * @returns Array of affected symbols with their overlapping hunks
+ */
+export declare function mapDiffToSymbols(hunks: DiffHunk[], symbols: SymbolInfo[]): AffectedSymbol[];
+//# sourceMappingURL=diff-mapper.d.ts.map

package/dist/scope/diff-mapper.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-mapper.d.ts","sourceRoot":"","sources":["../../src/scope/diff-mapper.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,YAAY,CAAC;AAevE;;;;;GAKG;AACH,wBAAgB,UAAU,CAAC,WAAW,EAAE,MAAM,GAAG,QAAQ,EAAE,CAiB1D;AAID;;;;;;;;;;GAUG;AACH,wBAAgB,gBAAgB,CAAC,KAAK,EAAE,QAAQ,EAAE,EAAE,OAAO,EAAE,UAAU,EAAE,GAAG,cAAc,EAAE,CAqB3F"}

package/dist/scope/diff-mapper.js

@@ -0,0 +1,76 @@
+/**
+ * Diff-to-Symbol Mapper
+ *
+ * Maps unified diff hunks to symbol definitions, identifying
+ * which symbols were affected by code changes.
+ *
+ * Pure functions with no side effects.
+ */
+// ─── Hunk Parsing ──────────────────────────────────────────────
+/**
+ * Regex to match unified diff hunk headers.
+ * Format: @@ -oldStart[,oldCount] +newStart[,newCount] @@
+ *
+ * Examples:
+ *   @@ -10,5 +10,7 @@
+ *   @@ -1 +1,3 @@
+ *   @@ -0,0 +1,20 @@
+ */
+const HUNK_HEADER_RE = /^@@\s+-(\d+)(?:,(\d+))?\s+\+(\d+)(?:,(\d+))?\s+@@/;
+/**
+ * Parse unified diff content to extract hunk line ranges.
+ *
+ * @param diffContent - Raw unified diff content for a single file
+ * @returns Array of DiffHunk objects with line ranges
+ */
+export function parseHunks(diffContent) {
+    const hunks = [];
+    const lines = diffContent.split('\n');
+    for (const line of lines) {
+        const match = HUNK_HEADER_RE.exec(line);
+        if (match) {
+            hunks.push({
+                oldStart: Number.parseInt(match[1], 10),
+                oldCount: match[2] !== undefined ? Number.parseInt(match[2], 10) : 1,
+                newStart: Number.parseInt(match[3], 10),
+                newCount: match[4] !== undefined ? Number.parseInt(match[4], 10) : 1,
+            });
+        }
+    }
+    return hunks;
+}
+// ─── Symbol Mapping ────────────────────────────────────────────
+/**
+ * Map diff hunks to affected symbols.
+ *
+ * A symbol is considered "affected" if any hunk's NEW side line range
+ * overlaps with the symbol's line range. We use the new side because
+ * we're analyzing the post-change source code.
+ *
+ * @param hunks - Parsed diff hunks
+ * @param symbols - Extracted symbols from the NEW version of the file
+ * @returns Array of affected symbols with their overlapping hunks
+ */
+export function mapDiffToSymbols(hunks, symbols) {
+    if (hunks.length === 0 || symbols.length === 0)
+        return [];
+    const affected = [];
+    for (const symbol of symbols) {
+        const overlapping = hunks.filter((hunk) => rangesOverlap(hunk.newStart, hunk.newStart + Math.max(hunk.newCount - 1, 0), symbol.startLine, symbol.endLine));
+        if (overlapping.length > 0) {
+            affected.push({ symbol, overlappingHunks: overlapping });
+        }
+    }
+    return affected;
+}
+// ─── Helpers ───────────────────────────────────────────────────
+/**
+ * Check if two 1-based line ranges overlap.
+ *
+ * Two ranges [a1, a2] and [b1, b2] overlap when:
+ *   a1 <= b2 AND b1 <= a2
+ */
+function rangesOverlap(aStart, aEnd, bStart, bEnd) {
+    return aStart <= bEnd && bStart <= aEnd;
+}
+//# sourceMappingURL=diff-mapper.js.map

package/dist/scope/diff-mapper.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"diff-mapper.js","sourceRoot":"","sources":["../../src/scope/diff-mapper.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAIH,kEAAkE;AAElE;;;;;;;;GAQG;AACH,MAAM,cAAc,GAAG,mDAAmD,CAAC;AAE3E;;;;;GAKG;AACH,MAAM,UAAU,UAAU,CAAC,WAAmB;IAC5C,MAAM,KAAK,GAAe,EAAE,CAAC;IAC7B,MAAM,KAAK,GAAG,WAAW,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;IAEtC,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,MAAM,KAAK,GAAG,cAAc,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;QACxC,IAAI,KAAK,EAAE,CAAC;YACV,KAAK,CAAC,IAAI,CAAC;gBACT,QAAQ,EAAE,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAE,EAAE,EAAE,CAAC;gBACxC,QAAQ,EAAE,KAAK,CAAC,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;gBACpE,QAAQ,EAAE,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAE,EAAE,EAAE,CAAC;gBACxC,QAAQ,EAAE,KAAK,CAAC,CAAC,CAAC,KAAK,SAAS,CAAC,CAAC,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC;aACrE,CAAC,CAAC;QACL,CAAC;IACH,CAAC;IAED,OAAO,KAAK,CAAC;AACf,CAAC;AAED,kEAAkE;AAElE;;;;;;;;;;GAUG;AACH,MAAM,UAAU,gBAAgB,CAAC,KAAiB,EAAE,OAAqB;IACvE,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC,IAAI,OAAO,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,EAAE,CAAC;IAE1D,MAAM,QAAQ,GAAqB,EAAE,CAAC;IAEtC,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,MAAM,WAAW,GAAG,KAAK,CAAC,MAAM,CAAC,CAAC,IAAI,EAAE,EAAE,CACxC,aAAa,CACX,IAAI,CAAC,QAAQ,EACb,IAAI,CAAC,QAAQ,GAAG,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,QAAQ,GAAG,CAAC,EAAE,CAAC,CAAC,EAC9C,MAAM,CAAC,SAAS,EAChB,MAAM,CAAC,OAAO,CACf,CACF,CAAC;QAEF,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;YAC3B,QAAQ,CAAC,IAAI,CAAC,EAAE,MAAM,EAAE,gBAAgB,EAAE,WAAW,EAAE,CAAC,CAAC;QAC3D,CAAC;IACH,CAAC;IAED,OAAO,QAAQ,CAAC;AAClB,CAAC;AAED,kEAAkE;AAElE;;;;;GAKG;AACH,SAAS,aAAa,CACpB,MAAc,EACd,IAAY,EACZ,MAAc,EACd,IAAY;IAEZ,OAAO,MAAM,IAAI,IAAI,IAAI,MAAM,IAAI,IAAI,CAAC;AAC1C,CAAC"}

package/dist/scope/extractor.d.ts

@@ -0,0 +1,25 @@
+/**
+ * Symbol Extractor
+ *
+ * Extracts symbol definitions (functions, classes, methods, interfaces)
+ * from source code using tree-sitter AST walking.
+ *
+ * Uses cursor-based traversal instead of S-expression queries for
+ * maximum compatibility across grammar versions.
+ *
+ * Pure function signature: (source, language, tree) => SymbolInfo[]
+ */
+import type { Tree } from 'web-tree-sitter';
+import type { ScopeLanguage, SymbolInfo } from './types.js';
+/**
+ * Extract symbol definitions from a parsed tree-sitter tree.
+ *
+ * Walks the AST using a cursor and identifies functions, classes,
+ * methods, and interfaces based on language-specific node types.
+ *
+ * @param tree - Parsed tree-sitter tree
+ * @param language - The scope language
+ * @returns Array of extracted symbols
+ */
+export declare function extractSymbolsFromTree(tree: Tree, language: ScopeLanguage): SymbolInfo[];
+//# sourceMappingURL=extractor.d.ts.map

package/dist/scope/extractor.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"extractor.d.ts","sourceRoot":"","sources":["../../src/scope/extractor.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAQ,IAAI,EAAE,MAAM,iBAAiB,CAAC;AAClD,OAAO,KAAK,EAAE,aAAa,EAAE,UAAU,EAAc,MAAM,YAAY,CAAC;AA+DxE;;;;;;;;;GASG;AACH,wBAAgB,sBAAsB,CACpC,IAAI,EAAE,IAAI,EACV,QAAQ,EAAE,aAAa,GACtB,UAAU,EAAE,CAad"}