Llm Context Window Engineering: Diff-Scoped Context Windows That Catch Real Bugs

// ci/ai-review/context.ts
import { execSync } from "child_process";

export function changedFiles(base: string): string[] {
  return execSync(`git diff --name-only ${base}...HEAD`, { encoding: "utf8" })
    .trim()
    .split("\n")
    .filter((f) => /\.(ts|tsx|js|py)$/.test(f));
}

export function unifiedDiff(base: string, file: string): string {
  // ✅ Prefer unified diff hunks over whole files
  return execSync(`git diff -U5 ${base}...HEAD -- ${file}`, { encoding: "utf8" });
}

// ❌ Dumping entire files "so the model has context"
const context = files.map((f) => readFileSync(f, "utf8")).join("\n\n");

export type SymbolHit = { file: string; name: string; kind: "fn" | "class" };

export function neighborContext(hits: SymbolHit[], maxHops = 1, tokenBudget = 6000): string {
  const parts: string[] = [];
  let used = 0;
  for (const h of hits) {
    const callees = lookupCallees(h).slice(0, 5); // ✅ hard cap
    const callers = lookupCallers(h).slice(0, 5);
    for (const n of [...callees, ...callers]) {
      const snippet = readSymbolSource(n);
      if (used + snippet.length > tokenBudget) return parts.join("\n\n");
      parts.push(`// neighbor ${n.file}::${n.name}\n${snippet}`);
      used += snippet.length;
    }
  }
  return parts.join("\n\n");
}

You are a senior PR reviewer. Given DIFF hunks, NEIGHBORS, and TESTS:
- Find nullability, authz, tenancy, and schema/contract breaks
- Cite file:line and severity (blocker/major/nit)
- Ignore formatting, import order, and naming already enforced by eslint
- If unsure, say uncertain — do not invent APIs
Return JSON: { findings: [{ severity, file, line, title, why }] }

type Finding = {
  severity: "blocker" | "major" | "nit";
  file: string;
  line: number;
  title: string;
  why: string;
};

export function filterFindings(raw: Finding[]): Finding[] {
  // ✅ Drop nits; keep blockers/majors for human attention
  return raw.filter((f) => f.severity !== "nit" && f.line > 0);
}

# ✅ Idempotent review marker
gh api repos/$REPO/commits/$SHA/comments -f body="$(cat review.md)"