@lzehrung/codegraph 2.1.0

package/dist/review/types.d.ts

@@ -0,0 +1,146 @@
+import type { MarkdownLinkCheckResult } from "../documentLinks/check.js";
+import type { CandidateTestFile } from "../impact/context.js";
+import type { CallCompatibilityHint, FileChange, MemberResolutionCoverage } from "../impact/types.js";
+import type { AnalysisSummary } from "../analysisSummary.js";
+import type { BuildReport, IncrementalBuildOptions, ProjectIndex } from "../indexer/types.js";
+import type { DuplicatePreparedAnalysis } from "../duplicates.js";
+import type { SqlReviewContext } from "../sql/review.js";
+import type { Edge, Range } from "../types.js";
+import type { ProjectFileInfo } from "../util/projectFiles.js";
+export type ReviewFileSummary = {
+    file: string;
+    status: "updated" | "deleted" | "missing";
+    oldFile?: string;
+    similarityIndex?: number;
+    /** True when Git reported a binary diff and symbol analysis was skipped. */
+    isBinary?: true;
+    /** Unchanged re-exports included only as API context for a changed export. */
+    apiContext?: ReviewSymbolSummary[];
+    symbols: ReviewSymbolSummary[];
+};
+export type ReviewSymbolCallsite = {
+    file: string;
+    range: Range;
+};
+export type ReviewSymbolSummary = {
+    name: string;
+    kind: string;
+    handle: string;
+    exported: boolean;
+    callCompatibility?: CallCompatibilityHint[];
+    definitionSnippet?: string;
+    diffSnippets?: string[];
+    callsites?: ReviewSymbolCallsite[];
+};
+export type ReviewChangedFileSummaries = {
+    summaries: ReviewFileSummary[];
+    changedSymbolIds: string[];
+    exportedChangedCount: number;
+    riskRelevantParseFailures: number;
+};
+export type ReviewReport = {
+    schemaVersion: number;
+    status: "ok" | "no_changes";
+    base?: string;
+    head?: string;
+    analysis?: AnalysisSummary;
+    projectFiles?: ProjectFileInfo[];
+    summary: {
+        filesChanged: number;
+        symbolsChanged: number;
+        candidateTests: number;
+    };
+    riskSummary: ReviewRiskSummary;
+    reviewTasks: ReviewTask[];
+    changedFiles: ReviewFileSummary[];
+    graphDelta: Edge[];
+    candidateTests: CandidateTestFile[];
+    sqlContext?: SqlReviewContext;
+    markdownLinks?: MarkdownLinkCheckResult;
+    diagnostics?: ReviewDiagnostics;
+};
+export type ReviewOptions = IncrementalBuildOptions & {
+    reviewDepth?: ReviewDepth;
+    maxCandidates?: number;
+    includeSymbolDetails?: boolean;
+    maxCallsites?: number;
+    includeDiffContext?: boolean;
+    diffContextLines?: number;
+    diffText?: string;
+    testPatterns?: string[];
+    referenceConcurrency?: number;
+    /** When false, skip prepareDuplicateAnalysis / duplicate review tasks entirely. Default true. */
+    duplicateTasks?: boolean;
+    report?: ReviewBuildReport;
+};
+export type ReviewDepth = "minimal" | "standard" | "deep";
+export type ReviewRiskLevel = "low" | "medium" | "high";
+export type ReviewRiskSummary = {
+    level: ReviewRiskLevel;
+    score: number;
+    signals: string[];
+};
+export type ReviewTaskPriority = "low" | "medium" | "high";
+export type ReviewTask = {
+    id: string;
+    title: string;
+    description: string;
+    priority: ReviewTaskPriority;
+    reason: string;
+};
+export type ReviewDiagnostics = {
+    missingFiles: string[];
+    symbolMappingParseFailures: string[];
+    memberResolutionCoverage?: MemberResolutionCoverage;
+};
+export type ReviewTimingReport = {
+    totalMs?: number;
+    changesMs?: number;
+    diffMs?: number;
+    indexMs?: number;
+    referencesMs?: number;
+    candidatesMs?: number;
+    duplicateAnalysisMs?: number;
+};
+export type ReviewBuildReport = {
+    timings: ReviewTimingReport;
+    indexReport?: BuildReport;
+    index?: ProjectIndex;
+    duplicateAnalysis?: DuplicatePreparedAnalysis;
+};
+export type ReviewDiffMetadata = Pick<ReviewFileSummary, "oldFile" | "similarityIndex">;
+export type ReviewDiffChange = FileChange;
+/**
+ * Per-collection caps applied only when a `ReviewReport` crosses a bounded
+ * transport (currently the MCP `review` tool) — see
+ * `boundReviewReportForTransport`.
+ */
+export type ReviewTransportLimits = {
+    projectFiles: number;
+    changedFiles: number;
+    symbolsPerFile: number;
+    graphDelta: number;
+    candidateTests: number;
+};
+export declare const DEFAULT_REVIEW_TRANSPORT_LIMITS: ReviewTransportLimits;
+export type ReviewTransportOmittedCounts = {
+    projectFiles: number;
+    changedFiles: number;
+    symbols: number;
+    graphDelta: number;
+    candidateTests: number;
+};
+export type ReviewReportForTransport = ReviewReport & {
+    limits: ReviewTransportLimits;
+    omittedCounts: ReviewTransportOmittedCounts;
+};
+/**
+ * Caps each top-level (and per-file nested `symbols`) collection in a
+ * `ReviewReport` for wire transports with hard payload/context limits, per
+ * finding #45. `summary` counts are left untouched so the numeric totals
+ * stay accurate even when the detailed listings are capped. Library callers
+ * that need the complete, unbounded report should call `buildReviewReport`
+ * directly instead of going through a bounded transport.
+ */
+export declare function boundReviewReportForTransport(report: ReviewReport, limits?: ReviewTransportLimits): ReviewReportForTransport;
+//# sourceMappingURL=types.d.ts.map

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

@@ -0,0 +1 @@
+{"version":3,"file":"types.d.ts","sourceRoot":"","sources":["../../src/review/types.ts"],"names":[],"mappings":"AAAA,OAAO,KAAK,EAAE,uBAAuB,EAAE,MAAM,2BAA2B,CAAC;AACzE,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,sBAAsB,CAAC;AAC9D,OAAO,KAAK,EAAE,qBAAqB,EAAE,UAAU,EAAE,wBAAwB,EAAE,MAAM,oBAAoB,CAAC;AACtG,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,uBAAuB,CAAC;AAC7D,OAAO,KAAK,EAAE,WAAW,EAAE,uBAAuB,EAAE,YAAY,EAAE,MAAM,qBAAqB,CAAC;AAC9F,OAAO,KAAK,EAAE,yBAAyB,EAAE,MAAM,kBAAkB,CAAC;AAClE,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,kBAAkB,CAAC;AACzD,OAAO,KAAK,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,aAAa,CAAC;AAC/C,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,yBAAyB,CAAC;AAE/D,MAAM,MAAM,iBAAiB,GAAG;IAC9B,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,SAAS,GAAG,SAAS,GAAG,SAAS,CAAC;IAC1C,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,eAAe,CAAC,EAAE,MAAM,CAAC;IACzB,4EAA4E;IAC5E,QAAQ,CAAC,EAAE,IAAI,CAAC;IAChB,8EAA8E;IAC9E,UAAU,CAAC,EAAE,mBAAmB,EAAE,CAAC;IACnC,OAAO,EAAE,mBAAmB,EAAE,CAAC;CAChC,CAAC;AAEF,MAAM,MAAM,oBAAoB,GAAG;IACjC,IAAI,EAAE,MAAM,CAAC;IACb,KAAK,EAAE,KAAK,CAAC;CACd,CAAC;AAEF,MAAM,MAAM,mBAAmB,GAAG;IAChC,IAAI,EAAE,MAAM,CAAC;IACb,IAAI,EAAE,MAAM,CAAC;IACb,MAAM,EAAE,MAAM,CAAC;IACf,QAAQ,EAAE,OAAO,CAAC;IAClB,iBAAiB,CAAC,EAAE,qBAAqB,EAAE,CAAC;IAC5C,iBAAiB,CAAC,EAAE,MAAM,CAAC;IAC3B,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,SAAS,CAAC,EAAE,oBAAoB,EAAE,CAAC;CACpC,CAAC;AAEF,MAAM,MAAM,0BAA0B,GAAG;IACvC,SAAS,EAAE,iBAAiB,EAAE,CAAC;IAC/B,gBAAgB,EAAE,MAAM,EAAE,CAAC;IAC3B,oBAAoB,EAAE,MAAM,CAAC;IAC7B,yBAAyB,EAAE,MAAM,CAAC;CACnC,CAAC;AAEF,MAAM,MAAM,YAAY,GAAG;IACzB,aAAa,EAAE,MAAM,CAAC;IACtB,MAAM,EAAE,IAAI,GAAG,YAAY,CAAC;IAC5B,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,IAAI,CAAC,EAAE,MAAM,CAAC;IACd,QAAQ,CAAC,EAAE,eAAe,CAAC;IAC3B,YAAY,CAAC,EAAE,eAAe,EAAE,CAAC;IACjC,OAAO,EAAE;QACP,YAAY,EAAE,MAAM,CAAC;QACrB,cAAc,EAAE,MAAM,CAAC;QACvB,cAAc,EAAE,MAAM,CAAC;KACxB,CAAC;IACF,WAAW,EAAE,iBAAiB,CAAC;IAC/B,WAAW,EAAE,UAAU,EAAE,CAAC;IAC1B,YAAY,EAAE,iBAAiB,EAAE,CAAC;IAClC,UAAU,EAAE,IAAI,EAAE,CAAC;IACnB,cAAc,EAAE,iBAAiB,EAAE,CAAC;IACpC,UAAU,CAAC,EAAE,gBAAgB,CAAC;IAC9B,aAAa,CAAC,EAAE,uBAAuB,CAAC;IACxC,WAAW,CAAC,EAAE,iBAAiB,CAAC;CACjC,CAAC;AAEF,MAAM,MAAM,aAAa,GAAG,uBAAuB,GAAG;IACpD,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,aAAa,CAAC,EAAE,MAAM,CAAC;IACvB,oBAAoB,CAAC,EAAE,OAAO,CAAC;IAC/B,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,kBAAkB,CAAC,EAAE,OAAO,CAAC;IAC7B,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B,QAAQ,CAAC,EAAE,MAAM,CAAC;IAClB,YAAY,CAAC,EAAE,MAAM,EAAE,CAAC;IACxB,oBAAoB,CAAC,EAAE,MAAM,CAAC;IAC9B,iGAAiG;IACjG,cAAc,CAAC,EAAE,OAAO,CAAC;IACzB,MAAM,CAAC,EAAE,iBAAiB,CAAC;CAC5B,CAAC;AAEF,MAAM,MAAM,WAAW,GAAG,SAAS,GAAG,UAAU,GAAG,MAAM,CAAC;AAE1D,MAAM,MAAM,eAAe,GAAG,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC;AAExD,MAAM,MAAM,iBAAiB,GAAG;IAC9B,KAAK,EAAE,eAAe,CAAC;IACvB,KAAK,EAAE,MAAM,CAAC;IACd,OAAO,EAAE,MAAM,EAAE,CAAC;CACnB,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAAG,KAAK,GAAG,QAAQ,GAAG,MAAM,CAAC;AAE3D,MAAM,MAAM,UAAU,GAAG;IACvB,EAAE,EAAE,MAAM,CAAC;IACX,KAAK,EAAE,MAAM,CAAC;IACd,WAAW,EAAE,MAAM,CAAC;IACpB,QAAQ,EAAE,kBAAkB,CAAC;IAC7B,MAAM,EAAE,MAAM,CAAC;CAChB,CAAC;AAEF,MAAM,MAAM,iBAAiB,GAAG;IAC9B,YAAY,EAAE,MAAM,EAAE,CAAC;IACvB,0BAA0B,EAAE,MAAM,EAAE,CAAC;IACrC,wBAAwB,CAAC,EAAE,wBAAwB,CAAC;CACrD,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAAG;IAC/B,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,OAAO,CAAC,EAAE,MAAM,CAAC;IACjB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,YAAY,CAAC,EAAE,MAAM,CAAC;IACtB,mBAAmB,CAAC,EAAE,MAAM,CAAC;CAC9B,CAAC;AAEF,MAAM,MAAM,iBAAiB,GAAG;IAC9B,OAAO,EAAE,kBAAkB,CAAC;IAC5B,WAAW,CAAC,EAAE,WAAW,CAAC;IAC1B,KAAK,CAAC,EAAE,YAAY,CAAC;IACrB,iBAAiB,CAAC,EAAE,yBAAyB,CAAC;CAC/C,CAAC;AAEF,MAAM,MAAM,kBAAkB,GAAG,IAAI,CAAC,iBAAiB,EAAE,SAAS,GAAG,iBAAiB,CAAC,CAAC;AAExF,MAAM,MAAM,gBAAgB,GAAG,UAAU,CAAC;AAE1C;;;;GAIG;AACH,MAAM,MAAM,qBAAqB,GAAG;IAClC,YAAY,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,MAAM,CAAC;IACrB,cAAc,EAAE,MAAM,CAAC;IACvB,UAAU,EAAE,MAAM,CAAC;IACnB,cAAc,EAAE,MAAM,CAAC;CACxB,CAAC;AAEF,eAAO,MAAM,+BAA+B,EAAE,qBAM7C,CAAC;AAEF,MAAM,MAAM,4BAA4B,GAAG;IACzC,YAAY,EAAE,MAAM,CAAC;IACrB,YAAY,EAAE,MAAM,CAAC;IACrB,OAAO,EAAE,MAAM,CAAC;IAChB,UAAU,EAAE,MAAM,CAAC;IACnB,cAAc,EAAE,MAAM,CAAC;CACxB,CAAC;AAEF,MAAM,MAAM,wBAAwB,GAAG,YAAY,GAAG;IACpD,MAAM,EAAE,qBAAqB,CAAC;IAC9B,aAAa,EAAE,4BAA4B,CAAC;CAC7C,CAAC;AAEF;;;;;;;GAOG;AACH,wBAAgB,6BAA6B,CAC3C,MAAM,EAAE,YAAY,EACpB,MAAM,GAAE,qBAAuD,GAC9D,wBAAwB,CA4B1B"}

package/dist/review/types.js

@@ -0,0 +1,43 @@
+export const DEFAULT_REVIEW_TRANSPORT_LIMITS = {
+    projectFiles: 300,
+    changedFiles: 50,
+    symbolsPerFile: 20,
+    graphDelta: 300,
+    candidateTests: 50,
+};
+/**
+ * Caps each top-level (and per-file nested `symbols`) collection in a
+ * `ReviewReport` for wire transports with hard payload/context limits, per
+ * finding #45. `summary` counts are left untouched so the numeric totals
+ * stay accurate even when the detailed listings are capped. Library callers
+ * that need the complete, unbounded report should call `buildReviewReport`
+ * directly instead of going through a bounded transport.
+ */
+export function boundReviewReportForTransport(report, limits = DEFAULT_REVIEW_TRANSPORT_LIMITS) {
+    const projectFilesOmitted = report.projectFiles ? Math.max(0, report.projectFiles.length - limits.projectFiles) : 0;
+    const changedFilesOmitted = Math.max(0, report.changedFiles.length - limits.changedFiles);
+    const graphDeltaOmitted = Math.max(0, report.graphDelta.length - limits.graphDelta);
+    const candidateTestsOmitted = Math.max(0, report.candidateTests.length - limits.candidateTests);
+    let symbolsOmitted = 0;
+    const changedFiles = report.changedFiles.slice(0, limits.changedFiles).map((file) => {
+        const omitted = Math.max(0, file.symbols.length - limits.symbolsPerFile);
+        symbolsOmitted += omitted;
+        return omitted ? { ...file, symbols: file.symbols.slice(0, limits.symbolsPerFile) } : file;
+    });
+    return {
+        ...report,
+        ...(report.projectFiles ? { projectFiles: report.projectFiles.slice(0, limits.projectFiles) } : {}),
+        changedFiles,
+        graphDelta: report.graphDelta.slice(0, limits.graphDelta),
+        candidateTests: report.candidateTests.slice(0, limits.candidateTests),
+        limits,
+        omittedCounts: {
+            projectFiles: projectFilesOmitted,
+            changedFiles: changedFilesOmitted,
+            symbols: symbolsOmitted,
+            graphDelta: graphDeltaOmitted,
+            candidateTests: candidateTestsOmitted,
+        },
+    };
+}
+//# sourceMappingURL=types.js.map

package/dist/review/types.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"types.js","sourceRoot":"","sources":["../../src/review/types.ts"],"names":[],"mappings":"AA8IA,MAAM,CAAC,MAAM,+BAA+B,GAA0B;IACpE,YAAY,EAAE,GAAG;IACjB,YAAY,EAAE,EAAE;IAChB,cAAc,EAAE,EAAE;IAClB,UAAU,EAAE,GAAG;IACf,cAAc,EAAE,EAAE;CACnB,CAAC;AAeF;;;;;;;GAOG;AACH,MAAM,UAAU,6BAA6B,CAC3C,MAAoB,EACpB,SAAgC,+BAA+B;IAE/D,MAAM,mBAAmB,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,MAAM,CAAC,YAAY,CAAC,MAAM,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC;IACpH,MAAM,mBAAmB,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,MAAM,CAAC,YAAY,CAAC,MAAM,GAAG,MAAM,CAAC,YAAY,CAAC,CAAC;IAC1F,MAAM,iBAAiB,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC,MAAM,GAAG,MAAM,CAAC,UAAU,CAAC,CAAC;IACpF,MAAM,qBAAqB,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,MAAM,CAAC,cAAc,CAAC,MAAM,GAAG,MAAM,CAAC,cAAc,CAAC,CAAC;IAEhG,IAAI,cAAc,GAAG,CAAC,CAAC;IACvB,MAAM,YAAY,GAAG,MAAM,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,YAAY,CAAC,CAAC,GAAG,CAAC,CAAC,IAAI,EAAE,EAAE;QAClF,MAAM,OAAO,GAAG,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,CAAC,OAAO,CAAC,MAAM,GAAG,MAAM,CAAC,cAAc,CAAC,CAAC;QACzE,cAAc,IAAI,OAAO,CAAC;QAC1B,OAAO,OAAO,CAAC,CAAC,CAAC,EAAE,GAAG,IAAI,EAAE,OAAO,EAAE,IAAI,CAAC,OAAO,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,cAAc,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC;IAC7F,CAAC,CAAC,CAAC;IAEH,OAAO;QACL,GAAG,MAAM;QACT,GAAG,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,YAAY,EAAE,MAAM,CAAC,YAAY,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,YAAY,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;QACnG,YAAY;QACZ,UAAU,EAAE,MAAM,CAAC,UAAU,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,UAAU,CAAC;QACzD,cAAc,EAAE,MAAM,CAAC,cAAc,CAAC,KAAK,CAAC,CAAC,EAAE,MAAM,CAAC,cAAc,CAAC;QACrE,MAAM;QACN,aAAa,EAAE;YACb,YAAY,EAAE,mBAAmB;YACjC,YAAY,EAAE,mBAAmB;YACjC,OAAO,EAAE,cAAc;YACvB,UAAU,EAAE,iBAAiB;YAC7B,cAAc,EAAE,qBAAqB;SACtC;KACF,CAAC;AACJ,CAAC"}

package/dist/review.d.ts

@@ -0,0 +1,19 @@
+import { type DuplicatePreparedAnalysis } from "./duplicates.js";
+import { type ProjectIndex } from "./indexer/types.js";
+import type { ReviewOptions, ReviewReport } from "./review/types.js";
+export type { ReviewBuildReport, ReviewChangedFileSummaries, ReviewDepth, ReviewDiagnostics, ReviewFileSummary, ReviewOptions, ReviewReport, ReviewRiskLevel, ReviewRiskSummary, ReviewSymbolCallsite, ReviewSymbolSummary, ReviewTask, ReviewTaskPriority, ReviewTimingReport, } from "./review/types.js";
+/**
+ * Build the structured review report used by programmatic review agents.
+ *
+ * The report keeps changed files, changed symbols, graph deltas, candidate tests,
+ * risk signals, review tasks, diagnostics, and optional snippets as data instead
+ * of terminal prose. Prefer this API over CLI summary output when composing
+ * deterministic model context or review file packs.
+ */
+export declare function buildReviewReport(projectRoot: string, opts?: ReviewOptions, cached?: {
+    index?: ProjectIndex;
+    loadIndex?: () => Promise<ProjectIndex>;
+    duplicateAnalysis?: DuplicatePreparedAnalysis;
+    loadDuplicateAnalysis?: () => Promise<DuplicatePreparedAnalysis>;
+}): Promise<ReviewReport>;
+//# sourceMappingURL=review.d.ts.map

package/dist/review.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review.d.ts","sourceRoot":"","sources":["../src/review.ts"],"names":[],"mappings":"AAEA,OAAO,EAIL,KAAK,yBAAyB,EAE/B,MAAM,iBAAiB,CAAC;AAIzB,OAAO,EAAE,KAAK,YAAY,EAAkB,MAAM,oBAAoB,CAAC;AAkBvE,OAAO,KAAK,EAKV,aAAa,EACb,YAAY,EAGb,MAAM,mBAAmB,CAAC;AAE3B,YAAY,EACV,iBAAiB,EACjB,0BAA0B,EAC1B,WAAW,EACX,iBAAiB,EACjB,iBAAiB,EACjB,aAAa,EACb,YAAY,EACZ,eAAe,EACf,iBAAiB,EACjB,oBAAoB,EACpB,mBAAmB,EACnB,UAAU,EACV,kBAAkB,EAClB,kBAAkB,GACnB,MAAM,mBAAmB,CAAC;AA8K3B;;;;;;;GAOG;AACH,wBAAsB,iBAAiB,CACrC,WAAW,EAAE,MAAM,EACnB,IAAI,GAAE,aAAkB,EACxB,MAAM,CAAC,EAAE;IACP,KAAK,CAAC,EAAE,YAAY,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,OAAO,CAAC,YAAY,CAAC,CAAC;IACxC,iBAAiB,CAAC,EAAE,yBAAyB,CAAC;IAC9C,qBAAqB,CAAC,EAAE,MAAM,OAAO,CAAC,yBAAyB,CAAC,CAAC;CAClE,GACA,OAAO,CAAC,YAAY,CAAC,CAiKvB"}

package/dist/review.js

@@ -0,0 +1,442 @@
+import { checkMarkdownLinksInFiles } from "./documentLinks/check.js";
+import { performance } from "node:perf_hooks";
+import { findDuplicateContextsWithPreparedAnalysis, prepareDuplicateAnalysis, } from "./duplicates.js";
+import { loadCurrentProjectIndex } from "./indexer/load-current-index.js";
+import { summarizeAnalysis } from "./analysisSummary.js";
+import {} from "./indexer/types.js";
+import { symbolId } from "./indexer/symbols.js";
+import { normalizePath, toProjectDisplayPath } from "./util/paths.js";
+import { fileExists } from "./util/workspace.js";
+import { discoverProjectFiles } from "./util/projectFiles.js";
+import { collectReviewCandidateTests } from "./review/candidates.js";
+import { collectReviewChanges } from "./review/changes.js";
+import { buildDeletedFileSnapshots } from "./review/deleted.js";
+import { assembleReviewReport, collectReviewGraphDelta, collectReviewSqlContext, REVIEW_SCHEMA_VERSION, } from "./review/report.js";
+import { buildReviewTasks, computeRiskSummary } from "./review/risk.js";
+import { summarizeChangedFiles } from "./review/summaries.js";
+const REVIEW_PRESETS = {
+    minimal: {
+        includeSymbolDetails: false,
+        maxCallsites: 0,
+        maxCandidates: 10,
+        graph: { fast: true },
+    },
+    standard: {
+        includeSymbolDetails: true,
+        maxCallsites: 2,
+        maxCandidates: 25,
+        graph: { fast: false },
+    },
+    deep: {
+        includeSymbolDetails: true,
+        maxCallsites: 10,
+        maxCandidates: 50,
+        graph: { fast: false },
+    },
+};
+const REVIEW_DUPLICATE_TASK_LIMIT = 5;
+const REVIEW_DUPLICATE_MAX_PAIRS = 20_000;
+function mergeGraphOptions(base, override) {
+    if (!base)
+        return override;
+    if (!override)
+        return base;
+    return { ...base, ...override };
+}
+function applyReviewPresetOptions(opts) {
+    if (!opts.reviewDepth)
+        return opts;
+    const preset = REVIEW_PRESETS[opts.reviewDepth];
+    const mergedGraph = mergeGraphOptions(preset.graph, opts.graph);
+    return {
+        ...opts,
+        includeSymbolDetails: opts.includeSymbolDetails ?? preset.includeSymbolDetails,
+        maxCallsites: opts.maxCallsites ?? preset.maxCallsites,
+        maxCandidates: opts.maxCandidates ?? preset.maxCandidates,
+        ...(mergedGraph ? { graph: mergedGraph } : {}),
+    };
+}
+function comparePaths(left, right) {
+    return left.localeCompare(right);
+}
+async function buildReviewIndex(input) {
+    const { projectRoot, appliedOptions, changedFileList, diffKindsByFile, diffChangesByFile, reviewReport, reviewTimings, providedIndex, loadProvidedIndex, } = input;
+    const fastGraphRequested = appliedOptions.graph?.fast ?? false;
+    const graphOptions = appliedOptions.graph ? { ...appliedOptions.graph, fast: fastGraphRequested } : { fast: false };
+    const existenceChecks = await Promise.all(changedFileList.map(async (file) => ({
+        file,
+        exists: await fileExists(file),
+    })));
+    const existenceByFile = new Map(existenceChecks.map((entry) => [entry.file, entry.exists]));
+    const deletedFiles = changedFileList.filter((file) => diffKindsByFile.get(file) === "deleted");
+    const deletedSnapshots = await buildDeletedFileSnapshots(projectRoot, deletedFiles, {
+        ...((appliedOptions.gitBase ?? appliedOptions.changedSince)
+            ? { revision: appliedOptions.gitBase ?? appliedOptions.changedSince }
+            : {}),
+        diffChangesByFile,
+        graphOptions,
+    });
+    const indexStart = performance.now();
+    const indexReport = reviewReport?.indexReport ?? (reviewReport ? { timings: {} } : undefined);
+    if (reviewReport && !reviewReport.indexReport && indexReport) {
+        reviewReport.indexReport = indexReport;
+    }
+    // Review range selection describes the diff, not the freshness scope of the
+    // current-project index: the shared loader strips those inputs and lets incremental
+    // indexing reconcile the whole project, unioning review targets outside discovery.
+    // A caller-provided index (e.g. an already-loaded agent session snapshot) is reused
+    // as-is when the review does not request additional files beyond normal project scope.
+    let index;
+    if (providedIndex && !appliedOptions.files?.length) {
+        index = providedIndex;
+    }
+    else if (loadProvidedIndex && !appliedOptions.files?.length) {
+        index = await loadProvidedIndex();
+    }
+    else {
+        index = await loadCurrentProjectIndex({
+            root: projectRoot,
+            scope: {
+                kind: "project",
+                ...(appliedOptions.files?.length ? { additionalFiles: appliedOptions.files } : {}),
+            },
+            options: {
+                ...appliedOptions,
+                graph: graphOptions,
+                keepParsed: true,
+                ...(indexReport ? { report: indexReport } : {}),
+            },
+        });
+    }
+    if (reviewReport) {
+        Object.defineProperty(reviewReport, "index", {
+            value: index,
+            enumerable: false,
+            configurable: true,
+        });
+    }
+    if (reviewTimings) {
+        reviewTimings.indexMs = Math.round(performance.now() - indexStart);
+    }
+    return {
+        index,
+        existenceByFile,
+        deletedFiles,
+        deletedSnapshots,
+        graphOptions,
+    };
+}
+/**
+ * Build the structured review report used by programmatic review agents.
+ *
+ * The report keeps changed files, changed symbols, graph deltas, candidate tests,
+ * risk signals, review tasks, diagnostics, and optional snippets as data instead
+ * of terminal prose. Prefer this API over CLI summary output when composing
+ * deterministic model context or review file packs.
+ */
+export async function buildReviewReport(projectRoot, opts = {}, cached) {
+    const appliedOptions = applyReviewPresetOptions(opts);
+    const reviewReport = appliedOptions.report;
+    const reviewTimings = reviewReport?.timings;
+    if (reviewTimings)
+        reviewTimings.duplicateAnalysisMs = 0;
+    if (reviewReport && appliedOptions.duplicateTasks === false) {
+        delete reviewReport.duplicateAnalysis;
+    }
+    const totalStart = performance.now();
+    const { changedFiles, explicitFiles, diffHunksByFile, diffKindsByFile, diffChangesByFile } = await collectReviewChanges(projectRoot, appliedOptions, reviewTimings);
+    if (changedFiles.size === 0) {
+        const riskSummary = computeRiskSummary({
+            filesChanged: 0,
+            symbolsChanged: 0,
+            exportedChanged: 0,
+            missingFiles: 0,
+            parseFailures: 0,
+        });
+        const projectFiles = await discoverProjectFiles(projectRoot);
+        const report = {
+            schemaVersion: REVIEW_SCHEMA_VERSION,
+            status: "no_changes",
+            ...(reviewReport?.indexReport
+                ? { analysis: summarizeAnalysis({ nativeMode: appliedOptions.native, report: reviewReport.indexReport }) }
+                : {}),
+            projectFiles,
+            summary: { filesChanged: 0, symbolsChanged: 0, candidateTests: 0 },
+            riskSummary,
+            reviewTasks: buildReviewTasks({
+                filesChanged: 0,
+                symbolsChanged: 0,
+                exportedChanged: 0,
+                candidateTests: 0,
+                missingFiles: 0,
+                parseFailures: 0,
+            }),
+            changedFiles: [],
+            graphDelta: [],
+            candidateTests: [],
+        };
+        if (appliedOptions.gitBase !== undefined)
+            report.base = appliedOptions.gitBase;
+        report.head = appliedOptions.gitHead ?? "HEAD";
+        if (reviewTimings)
+            reviewTimings.totalMs = Math.round(performance.now() - totalStart);
+        return report;
+    }
+    const changedFileList = Array.from(changedFiles).sort(comparePaths);
+    const diagnostics = {
+        missingFiles: [],
+        symbolMappingParseFailures: [],
+    };
+    const includeSymbolDetails = appliedOptions.includeSymbolDetails ?? false;
+    const diffContextLines = typeof appliedOptions.diffContextLines === "number" && appliedOptions.diffContextLines >= 0
+        ? appliedOptions.diffContextLines
+        : 2;
+    const maxCallsites = typeof appliedOptions.maxCallsites === "number" && appliedOptions.maxCallsites >= 0
+        ? appliedOptions.maxCallsites
+        : 5;
+    const referenceConcurrency = typeof appliedOptions.referenceConcurrency === "number" && appliedOptions.referenceConcurrency > 0
+        ? appliedOptions.referenceConcurrency
+        : 8;
+    const { index, existenceByFile, deletedFiles, deletedSnapshots } = await buildReviewIndex({
+        projectRoot,
+        appliedOptions,
+        changedFileList,
+        diffKindsByFile,
+        diffChangesByFile,
+        ...(reviewReport ? { reviewReport } : {}),
+        ...(reviewTimings ? { reviewTimings } : {}),
+        ...(cached?.index ? { providedIndex: cached.index } : {}),
+        ...(cached?.loadIndex ? { loadProvidedIndex: cached.loadIndex } : {}),
+    });
+    const includeDiffContext = appliedOptions.includeDiffContext ?? (includeSymbolDetails && diffHunksByFile.size > 0);
+    const { summaries, changedSymbolIds, exportedChangedCount, riskRelevantParseFailures } = await summarizeChangedFiles({
+        projectRoot,
+        index,
+        changedFileList,
+        diffHunksByFile,
+        diffKindsByFile,
+        diffChangesByFile,
+        explicitFiles,
+        existenceByFile,
+        deletedSnapshots,
+        includeSymbolDetails,
+        includeDiffContext,
+        diffContextLines,
+        maxCallsites,
+        referenceConcurrency,
+        diagnostics,
+        ...(reviewTimings ? { reviewTimings } : {}),
+    });
+    const graphDelta = await collectReviewGraphDelta({
+        projectRoot,
+        index,
+        changedFiles,
+        deletedFiles,
+        deletedSnapshots,
+    });
+    const candidateTests = await collectReviewCandidateTests({
+        projectRoot,
+        index,
+        changedFileList,
+        changedSymbolIds,
+        deletedFiles,
+        appliedOptions,
+        ...(reviewTimings ? { reviewTimings } : {}),
+    });
+    const projectFiles = index.projectFiles ?? (await discoverProjectFiles(projectRoot));
+    const markdownLinks = await checkMarkdownLinksInFiles(projectRoot, index.byFile.keys());
+    const sqlContext = await collectReviewSqlContext({ projectRoot, index, changedFileList });
+    const report = assembleReviewReport({
+        appliedOptions,
+        projectFiles,
+        summaries,
+        changedSymbolIds,
+        candidateTests,
+        graphDelta,
+        analysis: summarizeAnalysis({
+            index,
+            ...(reviewReport?.indexReport ? { report: reviewReport.indexReport } : {}),
+        }),
+        ...(sqlContext ? { sqlContext } : {}),
+        markdownLinks,
+        diagnostics,
+        riskRelevantParseFailures,
+        exportedChangedCount,
+    });
+    const duplicateAnalysisStart = performance.now();
+    if (appliedOptions.duplicateTasks !== false) {
+        const duplicateReview = await collectReviewDuplicateTasks({
+            projectRoot,
+            index,
+            summaries,
+            changedSymbolIds,
+            diffHunksByFile,
+            ...(cached?.duplicateAnalysis ? { providedAnalysis: cached.duplicateAnalysis } : {}),
+            ...(cached?.loadDuplicateAnalysis ? { loadProvidedAnalysis: cached.loadDuplicateAnalysis } : {}),
+        });
+        report.reviewTasks.push(...duplicateReview.tasks);
+        if (reviewReport && duplicateReview.preparedAnalysis) {
+            Object.defineProperty(reviewReport, "duplicateAnalysis", {
+                value: duplicateReview.preparedAnalysis,
+                enumerable: false,
+                configurable: true,
+            });
+        }
+    }
+    if (reviewTimings && appliedOptions.duplicateTasks !== false) {
+        reviewTimings.duplicateAnalysisMs = Math.round(performance.now() - duplicateAnalysisStart);
+    }
+    if (reviewTimings)
+        reviewTimings.totalMs = Math.round(performance.now() - totalStart);
+    return report;
+}
+function buildSymbolDefLookup(index) {
+    const lookup = new Map();
+    for (const mod of index.byFile.values()) {
+        for (const local of mod.locals) {
+            lookup.set(symbolId(local), local);
+        }
+    }
+    return lookup;
+}
+function relativeReviewPath(projectRoot, file) {
+    const normalizedRoot = normalizePath(projectRoot);
+    const normalizedFile = normalizePath(file);
+    return toProjectDisplayPath(normalizedRoot, normalizedFile) || normalizedFile;
+}
+function changedLinesForHunks(hunks) {
+    const changedLines = [];
+    for (const hunk of hunks) {
+        let nextLine = hunk.newStart;
+        let recordedLine = false;
+        for (const line of hunk.lines) {
+            const prefix = line[0];
+            if (prefix === "+") {
+                changedLines.push(nextLine);
+                nextLine++;
+                recordedLine = true;
+                continue;
+            }
+            if (prefix === " ") {
+                nextLine++;
+            }
+        }
+        if (!recordedLine && hunk.newStart > 0) {
+            changedLines.push(hunk.newStart);
+        }
+    }
+    return changedLines;
+}
+function compactChangedLineTargets(file, lines) {
+    if (!lines.length)
+        return [];
+    const uniqueLines = Array.from(new Set(lines)).sort((left, right) => left - right);
+    const targets = [];
+    let startLine = uniqueLines[0];
+    let endLine = startLine;
+    for (const line of uniqueLines.slice(1)) {
+        if (line === endLine + 1) {
+            endLine = line;
+            continue;
+        }
+        targets.push({ file, startLine, endLine });
+        startLine = line;
+        endLine = line;
+    }
+    targets.push({ file, startLine, endLine });
+    return targets;
+}
+function duplicateUnitOverlapsTarget(unit, target) {
+    if (unit.file !== target.file)
+        return false;
+    if (target.startLine === undefined)
+        return true;
+    const targetEndLine = target.endLine ?? target.startLine;
+    return unit.startLine <= targetEndLine && target.startLine <= unit.endLine;
+}
+function duplicateSiblingForTarget(group, target) {
+    if (duplicateUnitOverlapsTarget(group.primaryLeft, target))
+        return group.primaryRight;
+    if (duplicateUnitOverlapsTarget(group.primaryRight, target))
+        return group.primaryLeft;
+    for (const variant of group.variants) {
+        if (duplicateUnitOverlapsTarget(variant.left, target))
+            return variant.right;
+        if (duplicateUnitOverlapsTarget(variant.right, target))
+            return variant.left;
+    }
+    return group.primaryLeft;
+}
+function duplicateReviewTask(group, target) {
+    const sibling = duplicateSiblingForTarget(group, target);
+    const targetLabel = target.startLine === undefined
+        ? target.file
+        : `${target.file}:${target.startLine}-${target.endLine ?? target.startLine}`;
+    const siblingLabel = `${sibling.file}:${sibling.startLine}-${sibling.endLine}`;
+    return {
+        id: `duplicate-sibling-check:${group.id}`,
+        title: "Check related duplicate implementation",
+        description: `${targetLabel} overlaps a ${group.confidence}-confidence ${group.cloneType} duplicate group. ` +
+            `Check sibling ${siblingLabel} for drift before changing only one side.`,
+        priority: group.confidence === "high" ? "high" : "medium",
+        reason: "duplicate-sibling",
+    };
+}
+async function collectReviewDuplicateTasks(input) {
+    const diffHunksByDisplayFile = new Map(Array.from(input.diffHunksByFile, ([file, hunks]) => [relativeReviewPath(input.projectRoot, file), hunks]));
+    const symbolLookup = buildSymbolDefLookup(input.index);
+    const targets = input.changedSymbolIds
+        .map((id) => symbolLookup.get(id))
+        .filter((def) => def !== undefined)
+        .map((def) => ({
+        file: relativeReviewPath(input.projectRoot, def.file),
+        startLine: def.range.start.line,
+        endLine: def.range.end.line,
+    }));
+    for (const summary of input.summaries) {
+        if (summary.status !== "updated")
+            continue;
+        const hunks = diffHunksByDisplayFile.get(summary.file);
+        if (!hunks?.length) {
+            targets.push({ file: summary.file });
+            continue;
+        }
+        const symbolTargets = targets.filter((target) => target.file === summary.file && target.startLine !== undefined);
+        const uncoveredChangedLines = changedLinesForHunks(hunks).filter((line) => {
+            return !symbolTargets.some((target) => {
+                const targetEndLine = target.endLine ?? target.startLine;
+                return target.startLine <= line && line <= targetEndLine;
+            });
+        });
+        if (!uncoveredChangedLines.length)
+            continue;
+        targets.push(...compactChangedLineTargets(summary.file, uncoveredChangedLines));
+    }
+    if (!targets.length)
+        return { tasks: [] };
+    const preparedAnalysis = input.providedAnalysis ??
+        (input.loadProvidedAnalysis ? await input.loadProvidedAnalysis() : undefined) ??
+        (await prepareDuplicateAnalysis(input.index, {
+            projectRoot: input.projectRoot,
+        }));
+    const contexts = await findDuplicateContextsWithPreparedAnalysis(preparedAnalysis, targets, {
+        projectRoot: input.projectRoot,
+        minConfidence: "high",
+        includeSameFile: true,
+        limit: REVIEW_DUPLICATE_TASK_LIMIT,
+        maxPairs: REVIEW_DUPLICATE_MAX_PAIRS,
+    });
+    const tasks = new Map();
+    for (const context of contexts) {
+        for (const group of context.groups) {
+            if (tasks.size >= REVIEW_DUPLICATE_TASK_LIMIT) {
+                return { tasks: [...tasks.values()], preparedAnalysis };
+            }
+            const task = duplicateReviewTask(group, context.target);
+            if (!tasks.has(task.id))
+                tasks.set(task.id, task);
+        }
+    }
+    return { tasks: [...tasks.values()], preparedAnalysis };
+}
+//# sourceMappingURL=review.js.map