@diff-review-system/drs 3.3.1 → 4.0.1

package/dist/lib/review-artifact.js

@@ -0,0 +1,171 @@
+import { createIssueFingerprint } from './comment-manager.js';
+function createReviewId(date) {
+    const timestamp = date.toISOString().replace(/[-:.TZ]/g, '');
+    const random = Math.random().toString(36).slice(2, 8);
+    return `rev_${timestamp}_${random}`;
+}
+function getSourceString(source) {
+    return source?.name;
+}
+function getSourceProject(source) {
+    const projectId = source?.context.projectId;
+    return typeof projectId === 'string' ? projectId : undefined;
+}
+function getSourcePullRequest(source) {
+    const pullRequest = source?.context.pullRequest;
+    if (!pullRequest || typeof pullRequest !== 'object') {
+        return {};
+    }
+    const record = pullRequest;
+    return {
+        headSha: typeof record.headSha === 'string' ? record.headSha : undefined,
+        sourceBranch: typeof record.sourceBranch === 'string' ? record.sourceBranch : undefined,
+        targetBranch: typeof record.targetBranch === 'string' ? record.targetBranch : undefined,
+    };
+}
+function findingId(index) {
+    return `F${String(index + 1).padStart(3, '0')}`;
+}
+function nextFindingId(findings) {
+    const max = findings.reduce((currentMax, finding) => {
+        const match = /^F(\d+)$/.exec(finding.id);
+        return match ? Math.max(currentMax, Number.parseInt(match[1] ?? '0', 10)) : currentMax;
+    }, 0);
+    return findingId(max);
+}
+function selectorMatches(finding, selector) {
+    const hasIds = selector.ids !== undefined && selector.ids.length > 0;
+    const hasFingerprints = selector.fingerprints !== undefined && selector.fingerprints.length > 0;
+    const hasSeverity = selector.severity !== undefined && selector.severity !== '';
+    if (!hasIds && !hasFingerprints && !hasSeverity) {
+        return true;
+    }
+    return ((hasIds && (selector.ids?.includes(finding.id) ?? false)) ||
+        (hasFingerprints && (selector.fingerprints?.includes(finding.fingerprint) ?? false)) ||
+        (hasSeverity && finding.issue.severity === selector.severity));
+}
+export function createReviewArtifactPayload(review, source, date = new Date()) {
+    const now = date.toISOString();
+    const pullRequest = getSourcePullRequest(source);
+    const findings = review.issues.map((issue, index) => ({
+        id: findingId(index),
+        fingerprint: createIssueFingerprint(issue),
+        issue,
+        state: 'open',
+        disposition: 'confirmed',
+        source: 'agent',
+        createdAt: now,
+        updatedAt: now,
+    }));
+    return {
+        schemaVersion: 1,
+        reviewId: createReviewId(date),
+        reviewedAt: now,
+        reviewedSha: pullRequest.headSha,
+        baseBranch: pullRequest.targetBranch,
+        headBranch: pullRequest.sourceBranch,
+        summary: review.summary,
+        findings,
+        usage: review.usage,
+        metadata: {
+            source: getSourceString(source),
+            project: getSourceProject(source),
+            branch: pullRequest.sourceBranch || pullRequest.targetBranch
+                ? {
+                    source: pullRequest.sourceBranch,
+                    target: pullRequest.targetBranch,
+                }
+                : undefined,
+        },
+    };
+}
+export function isReviewArtifactPayload(value) {
+    if (!value || typeof value !== 'object') {
+        return false;
+    }
+    const candidate = value;
+    return (candidate.schemaVersion === 1 &&
+        typeof candidate.reviewId === 'string' &&
+        typeof candidate.reviewedAt === 'string' &&
+        typeof candidate.summary === 'object' &&
+        Array.isArray(candidate.findings));
+}
+export function getReviewArtifactStatus(artifact) {
+    const byState = { open: 0, attempted: 0, resolved: 0 };
+    const byDisposition = {
+        confirmed: 0,
+        uncertain: 0,
+        pre_existing: 0,
+        partial: 0,
+        still_open: 0,
+        regression: 0,
+        resolved: 0,
+    };
+    for (const finding of artifact.findings) {
+        byState[finding.state] += 1;
+        byDisposition[finding.disposition] += 1;
+    }
+    return {
+        reviewId: artifact.reviewId,
+        reviewedAt: artifact.reviewedAt,
+        reviewedSha: artifact.reviewedSha,
+        totalFindings: artifact.findings.length,
+        byState,
+        byDisposition,
+        bySeverity: artifact.summary.bySeverity,
+        openFindings: byState.open,
+    };
+}
+export function addReviewArtifactFinding(artifact, issue, source = 'manual', date = new Date()) {
+    const now = date.toISOString();
+    const finding = {
+        id: nextFindingId(artifact.findings),
+        fingerprint: createIssueFingerprint(issue),
+        issue,
+        state: 'open',
+        disposition: 'confirmed',
+        source,
+        createdAt: now,
+        updatedAt: now,
+    };
+    return {
+        ...artifact,
+        findings: [...artifact.findings, finding],
+        summary: {
+            ...artifact.summary,
+            issuesFound: artifact.summary.issuesFound + 1,
+            bySeverity: {
+                ...artifact.summary.bySeverity,
+                [issue.severity]: (artifact.summary.bySeverity[issue.severity] ?? 0) + 1,
+            },
+            byCategory: {
+                ...artifact.summary.byCategory,
+                [issue.category]: (artifact.summary.byCategory[issue.category] ?? 0) + 1,
+            },
+        },
+    };
+}
+export function updateReviewArtifactFindings(artifact, options, date = new Date()) {
+    const now = date.toISOString();
+    const updatedIds = [];
+    const findings = artifact.findings.map((finding) => {
+        if (!selectorMatches(finding, options)) {
+            return finding;
+        }
+        updatedIds.push(finding.id);
+        return {
+            ...finding,
+            state: options.state ?? finding.state,
+            disposition: options.disposition ?? finding.disposition,
+            updatedAt: now,
+        };
+    });
+    return {
+        artifact: {
+            ...artifact,
+            findings,
+        },
+        updatedIds,
+    };
+}
+//# sourceMappingURL=review-artifact.js.map

package/dist/lib/review-artifact.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-artifact.js","sourceRoot":"","sources":["../../src/lib/review-artifact.ts"],"names":[],"mappings":"AAGA,OAAO,EAAE,sBAAsB,EAAE,MAAM,sBAAsB,CAAC;AAyE9D,SAAS,cAAc,CAAC,IAAU;IAChC,MAAM,SAAS,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC,OAAO,CAAC,UAAU,EAAE,EAAE,CAAC,CAAC;IAC7D,MAAM,MAAM,GAAG,IAAI,CAAC,MAAM,EAAE,CAAC,QAAQ,CAAC,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC;IACtD,OAAO,OAAO,SAAS,IAAI,MAAM,EAAE,CAAC;AACtC,CAAC;AAED,SAAS,eAAe,CAAC,MAAgC;IACvD,OAAO,MAAM,EAAE,IAAI,CAAC;AACtB,CAAC;AAED,SAAS,gBAAgB,CAAC,MAAgC;IACxD,MAAM,SAAS,GAAG,MAAM,EAAE,OAAO,CAAC,SAAS,CAAC;IAC5C,OAAO,OAAO,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC;AAC/D,CAAC;AAED,SAAS,oBAAoB,CAAC,MAAgC;IAK5D,MAAM,WAAW,GAAG,MAAM,EAAE,OAAO,CAAC,WAAW,CAAC;IAChD,IAAI,CAAC,WAAW,IAAI,OAAO,WAAW,KAAK,QAAQ,EAAE,CAAC;QACpD,OAAO,EAAE,CAAC;IACZ,CAAC;IACD,MAAM,MAAM,GAAG,WAAsC,CAAC;IACtD,OAAO;QACL,OAAO,EAAE,OAAO,MAAM,CAAC,OAAO,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS;QACxE,YAAY,EAAE,OAAO,MAAM,CAAC,YAAY,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,SAAS;QACvF,YAAY,EAAE,OAAO,MAAM,CAAC,YAAY,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,YAAY,CAAC,CAAC,CAAC,SAAS;KACxF,CAAC;AACJ,CAAC;AAED,SAAS,SAAS,CAAC,KAAa;IAC9B,OAAO,IAAI,MAAM,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAAC;AAClD,CAAC;AAED,SAAS,aAAa,CAAC,QAAyB;IAC9C,MAAM,GAAG,GAAG,QAAQ,CAAC,MAAM,CAAC,CAAC,UAAU,EAAE,OAAO,EAAE,EAAE;QAClD,MAAM,KAAK,GAAG,UAAU,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC1C,OAAO,KAAK,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,UAAU,EAAE,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,CAAC,CAAC,IAAI,GAAG,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC;IACzF,CAAC,EAAE,CAAC,CAAC,CAAC;IACN,OAAO,SAAS,CAAC,GAAG,CAAC,CAAC;AACxB,CAAC;AAED,SAAS,eAAe,CAAC,OAAsB,EAAE,QAA+B;IAC9E,MAAM,MAAM,GAAG,QAAQ,CAAC,GAAG,KAAK,SAAS,IAAI,QAAQ,CAAC,GAAG,CAAC,MAAM,GAAG,CAAC,CAAC;IACrE,MAAM,eAAe,GAAG,QAAQ,CAAC,YAAY,KAAK,SAAS,IAAI,QAAQ,CAAC,YAAY,CAAC,MAAM,GAAG,CAAC,CAAC;IAChG,MAAM,WAAW,GAAG,QAAQ,CAAC,QAAQ,KAAK,SAAS,IAAI,QAAQ,CAAC,QAAQ,KAAK,EAAE,CAAC;IAEhF,IAAI,CAAC,MAAM,IAAI,CAAC,eAAe,IAAI,CAAC,WAAW,EAAE,CAAC;QAChD,OAAO,IAAI,CAAC;IACd,CAAC;IAED,OAAO,CACL,CAAC,MAAM,IAAI,CAAC,QAAQ,CAAC,GAAG,EAAE,QAAQ,CAAC,OAAO,CAAC,EAAE,CAAC,IAAI,KAAK,CAAC,CAAC;QACzD,CAAC,eAAe,IAAI,CAAC,QAAQ,CAAC,YAAY,EAAE,QAAQ,CAAC,OAAO,CAAC,WAAW,CAAC,IAAI,KAAK,CAAC,CAAC;QACpF,CAAC,WAAW,IAAI,OAAO,CAAC,KAAK,CAAC,QAAQ,KAAK,QAAQ,CAAC,QAAQ,CAAC,CAC9D,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,2BAA2B,CACzC,MAAoB,EACpB,MAAqB,EACrB,OAAa,IAAI,IAAI,EAAE;IAEvB,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IAC/B,MAAM,WAAW,GAAG,oBAAoB,CAAC,MAAM,CAAC,CAAC;IACjD,MAAM,QAAQ,GAAG,MAAM,CAAC,MAAM,CAAC,GAAG,CAChC,CAAC,KAAK,EAAE,KAAK,EAAiB,EAAE,CAAC,CAAC;QAChC,EAAE,EAAE,SAAS,CAAC,KAAK,CAAC;QACpB,WAAW,EAAE,sBAAsB,CAAC,KAAK,CAAC;QAC1C,KAAK;QACL,KAAK,EAAE,MAAM;QACb,WAAW,EAAE,WAAW;QACxB,MAAM,EAAE,OAAO;QACf,SAAS,EAAE,GAAG;QACd,SAAS,EAAE,GAAG;KACf,CAAC,CACH,CAAC;IAEF,OAAO;QACL,aAAa,EAAE,CAAC;QAChB,QAAQ,EAAE,cAAc,CAAC,IAAI,CAAC;QAC9B,UAAU,EAAE,GAAG;QACf,WAAW,EAAE,WAAW,CAAC,OAAO;QAChC,UAAU,EAAE,WAAW,CAAC,YAAY;QACpC,UAAU,EAAE,WAAW,CAAC,YAAY;QACpC,OAAO,EAAE,MAAM,CAAC,OAAO;QACvB,QAAQ;QACR,KAAK,EAAE,MAAM,CAAC,KAAK;QACnB,QAAQ,EAAE;YACR,MAAM,EAAE,eAAe,CAAC,MAAM,CAAC;YAC/B,OAAO,EAAE,gBAAgB,CAAC,MAAM,CAAC;YACjC,MAAM,EACJ,WAAW,CAAC,YAAY,IAAI,WAAW,CAAC,YAAY;gBAClD,CAAC,CAAC;oBACE,MAAM,EAAE,WAAW,CAAC,YAAY;oBAChC,MAAM,EAAE,WAAW,CAAC,YAAY;iBACjC;gBACH,CAAC,CAAC,SAAS;SAChB;KACF,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,uBAAuB,CAAC,KAAc;IACpD,IAAI,CAAC,KAAK,IAAI,OAAO,KAAK,KAAK,QAAQ,EAAE,CAAC;QACxC,OAAO,KAAK,CAAC;IACf,CAAC;IACD,MAAM,SAAS,GAAG,KAAuC,CAAC;IAC1D,OAAO,CACL,SAAS,CAAC,aAAa,KAAK,CAAC;QAC7B,OAAO,SAAS,CAAC,QAAQ,KAAK,QAAQ;QACtC,OAAO,SAAS,CAAC,UAAU,KAAK,QAAQ;QACxC,OAAO,SAAS,CAAC,OAAO,KAAK,QAAQ;QACrC,KAAK,CAAC,OAAO,CAAC,SAAS,CAAC,QAAQ,CAAC,CAClC,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,uBAAuB,CAAC,QAA+B;IACrE,MAAM,OAAO,GAAuC,EAAE,IAAI,EAAE,CAAC,EAAE,SAAS,EAAE,CAAC,EAAE,QAAQ,EAAE,CAAC,EAAE,CAAC;IAC3F,MAAM,aAAa,GAA6C;QAC9D,SAAS,EAAE,CAAC;QACZ,SAAS,EAAE,CAAC;QACZ,YAAY,EAAE,CAAC;QACf,OAAO,EAAE,CAAC;QACV,UAAU,EAAE,CAAC;QACb,UAAU,EAAE,CAAC;QACb,QAAQ,EAAE,CAAC;KACZ,CAAC;IAEF,KAAK,MAAM,OAAO,IAAI,QAAQ,CAAC,QAAQ,EAAE,CAAC;QACxC,OAAO,CAAC,OAAO,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC;QAC5B,aAAa,CAAC,OAAO,CAAC,WAAW,CAAC,IAAI,CAAC,CAAC;IAC1C,CAAC;IAED,OAAO;QACL,QAAQ,EAAE,QAAQ,CAAC,QAAQ;QAC3B,UAAU,EAAE,QAAQ,CAAC,UAAU;QAC/B,WAAW,EAAE,QAAQ,CAAC,WAAW;QACjC,aAAa,EAAE,QAAQ,CAAC,QAAQ,CAAC,MAAM;QACvC,OAAO;QACP,aAAa;QACb,UAAU,EAAE,QAAQ,CAAC,OAAO,CAAC,UAAU;QACvC,YAAY,EAAE,OAAO,CAAC,IAAI;KAC3B,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,wBAAwB,CACtC,QAA+B,EAC/B,KAAkB,EAClB,SAA8B,QAAQ,EACtC,OAAa,IAAI,IAAI,EAAE;IAEvB,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IAC/B,MAAM,OAAO,GAAkB;QAC7B,EAAE,EAAE,aAAa,CAAC,QAAQ,CAAC,QAAQ,CAAC;QACpC,WAAW,EAAE,sBAAsB,CAAC,KAAK,CAAC;QAC1C,KAAK;QACL,KAAK,EAAE,MAAM;QACb,WAAW,EAAE,WAAW;QACxB,MAAM;QACN,SAAS,EAAE,GAAG;QACd,SAAS,EAAE,GAAG;KACf,CAAC;IAEF,OAAO;QACL,GAAG,QAAQ;QACX,QAAQ,EAAE,CAAC,GAAG,QAAQ,CAAC,QAAQ,EAAE,OAAO,CAAC;QACzC,OAAO,EAAE;YACP,GAAG,QAAQ,CAAC,OAAO;YACnB,WAAW,EAAE,QAAQ,CAAC,OAAO,CAAC,WAAW,GAAG,CAAC;YAC7C,UAAU,EAAE;gBACV,GAAG,QAAQ,CAAC,OAAO,CAAC,UAAU;gBAC9B,CAAC,KAAK,CAAC,QAAQ,CAAC,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,UAAU,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC;aACzE;YACD,UAAU,EAAE;gBACV,GAAG,QAAQ,CAAC,OAAO,CAAC,UAAU;gBAC9B,CAAC,KAAK,CAAC,QAAQ,CAAC,EAAE,CAAC,QAAQ,CAAC,OAAO,CAAC,UAAU,CAAC,KAAK,CAAC,QAAQ,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC;aACzE;SACF;KACF,CAAC;AACJ,CAAC;AAED,MAAM,UAAU,4BAA4B,CAC1C,QAA+B,EAC/B,OAAoC,EACpC,OAAa,IAAI,IAAI,EAAE;IAEvB,MAAM,GAAG,GAAG,IAAI,CAAC,WAAW,EAAE,CAAC;IAC/B,MAAM,UAAU,GAAa,EAAE,CAAC;IAChC,MAAM,QAAQ,GAAG,QAAQ,CAAC,QAAQ,CAAC,GAAG,CAAC,CAAC,OAAO,EAAE,EAAE;QACjD,IAAI,CAAC,eAAe,CAAC,OAAO,EAAE,OAAO,CAAC,EAAE,CAAC;YACvC,OAAO,OAAO,CAAC;QACjB,CAAC;QACD,UAAU,CAAC,IAAI,CAAC,OAAO,CAAC,EAAE,CAAC,CAAC;QAC5B,OAAO;YACL,GAAG,OAAO;YACV,KAAK,EAAE,OAAO,CAAC,KAAK,IAAI,OAAO,CAAC,KAAK;YACrC,WAAW,EAAE,OAAO,CAAC,WAAW,IAAI,OAAO,CAAC,WAAW;YACvD,SAAS,EAAE,GAAG;SACf,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,OAAO;QACL,QAAQ,EAAE;YACR,GAAG,QAAQ;YACX,QAAQ;SACT;QACD,UAAU;KACX,CAAC;AACJ,CAAC"}

package/dist/lib/review-core.d.ts

@@ -1,13 +1,13 @@
 /**
  * Core review execution logic shared between local and platform reviews
  *
- * This module contains the common agent execution logic to avoid duplication
- * between review-orchestrator.ts (local) and unified-review-executor.ts (platform).
+ * This module contains the common agent execution logic used by the review orchestrator.
  */
 import type { DRSConfig } from './config.js';
 import { calculateSummary, type ReviewIssue } from './comment-formatter.js';
 import type { ChangeSummary } from './change-summary.js';
 import type { RuntimeClient } from '../runtime/client.js';
+import type { ReviewVerificationContext, ReviewVerificationResult } from './review-orchestrator.js';
 import { type AgentUsageSummary, type ReviewUsageSummary } from './review-usage.js';
 /**
  * File with optional diff content
@@ -34,11 +34,14 @@ export interface AgentReviewResult {
     agentResults: AgentResult[];
     /** Token usage and cost details for the review run */
     usage?: ReviewUsageSummary;
+    /** Explicit verification verdicts for existing review findings. */
+    verification?: ReviewVerificationResult;
 }
 export interface AgentResult {
     agentType: string;
     success: boolean;
     issues: ReviewIssue[];
+    verification?: ReviewVerificationResult;
     usage?: AgentUsageSummary;
 }
 /**
@@ -48,8 +51,7 @@ export interface AgentResult {
  * @param files - List of files with optional diff content
  * @param diffCommand - Fallback git diff command hint (used when diff not provided)
  */
-export declare function buildBaseInstructions(label: string, files: FileWithDiff[], diffCommand?: string, compressionSummary?: string): string;
-export declare function runUnifiedReviewAgent(runtime: RuntimeClient, config: DRSConfig, baseInstructions: string, reviewLabel: string, filteredFiles: string[], additionalContext?: Record<string, unknown>, workingDir?: string, debug?: boolean): Promise<AgentReviewResult>;
+export declare function buildBaseInstructions(label: string, files: FileWithDiff[], diffCommand?: string, compressionSummary?: string, verificationContext?: ReviewVerificationContext): string;
 export declare function runReviewAgents(runtime: RuntimeClient, config: DRSConfig, baseInstructions: string, reviewLabel: string, filteredFiles: string[], additionalContext?: Record<string, unknown>, workingDir?: string, debug?: boolean): Promise<AgentReviewResult>;
 export declare function runReviewPipeline(runtime: RuntimeClient, config: DRSConfig, baseInstructions: string, reviewLabel: string, filteredFiles: string[], additionalContext?: Record<string, unknown>, workingDir?: string, debug?: boolean): Promise<AgentReviewResult>;
 /**

package/dist/lib/review-core.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"review-core.d.ts","sourceRoot":"","sources":["../../src/lib/review-core.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAW7C,OAAO,EAAE,gBAAgB,EAAE,KAAK,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAC5E,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AAG1D,OAAO,EAIL,KAAK,iBAAiB,EACtB,KAAK,kBAAkB,EACxB,MAAM,mBAAmB,CAAC;AAG3B;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,gBAAgB;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,sDAAsD;IACtD,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,wCAAwC;IACxC,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,oCAAoC;IACpC,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;IAC7C,+CAA+C;IAC/C,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,wCAAwC;IACxC,aAAa,EAAE,MAAM,CAAC;IACtB,8BAA8B;IAC9B,YAAY,EAAE,WAAW,EAAE,CAAC;IAC5B,sDAAsD;IACtD,KAAK,CAAC,EAAE,kBAAkB,CAAC;CAC5B;AAED,MAAM,WAAW,WAAW;IAC1B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,KAAK,CAAC,EAAE,iBAAiB,CAAC;CAC3B;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,YAAY,EAAE,EACrB,WAAW,CAAC,EAAE,MAAM,EACpB,kBAAkB,CAAC,EAAE,MAAM,GAC1B,MAAM,CAoIR;AAqJD,wBAAsB,qBAAqB,CACzC,OAAO,EAAE,aAAa,EACtB,MAAM,EAAE,SAAS,EACjB,gBAAgB,EAAE,MAAM,EACxB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EAAE,EACvB,iBAAiB,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,EAC/C,UAAU,GAAE,MAAsB,EAClC,KAAK,UAAQ,GACZ,OAAO,CAAC,iBAAiB,CAAC,CA+J5B;AAuID,wBAAsB,eAAe,CACnC,OAAO,EAAE,aAAa,EACtB,MAAM,EAAE,SAAS,EACjB,gBAAgB,EAAE,MAAM,EACxB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EAAE,EACvB,iBAAiB,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,EAC/C,UAAU,GAAE,MAAsB,EAClC,KAAK,UAAQ,GACZ,OAAO,CAAC,iBAAiB,CAAC,CAgF5B;AAED,wBAAsB,iBAAiB,CACrC,OAAO,EAAE,aAAa,EACtB,MAAM,EAAE,SAAS,EACjB,gBAAgB,EAAE,MAAM,EACxB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EAAE,EACvB,iBAAiB,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,EAC/C,UAAU,GAAE,MAAsB,EAClC,KAAK,UAAQ,GACZ,OAAO,CAAC,iBAAiB,CAAC,CAW5B;AAED;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,MAAM,EAAE;IAC3C,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;IAC7C,aAAa,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,KAAK,CAAC,EAAE,kBAAkB,CAAC;CAC5B,GAAG,IAAI,CAkDP;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE;IACxC,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;CAC9C,GAAG,OAAO,CAEV"}
+{"version":3,"file":"review-core.d.ts","sourceRoot":"","sources":["../../src/lib/review-core.ts"],"names":[],"mappings":"AAAA;;;;GAIG;AAGH,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,aAAa,CAAC;AAU7C,OAAO,EAAE,gBAAgB,EAAE,KAAK,WAAW,EAAE,MAAM,wBAAwB,CAAC;AAC5E,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AACzD,OAAO,KAAK,EAAE,aAAa,EAAE,MAAM,sBAAsB,CAAC;AAG1D,OAAO,KAAK,EACV,yBAAyB,EAGzB,wBAAwB,EACzB,MAAM,0BAA0B,CAAC;AAClC,OAAO,EAML,KAAK,iBAAiB,EACtB,KAAK,kBAAkB,EACxB,MAAM,mBAAmB,CAAC;AAG3B;;GAEG;AACH,MAAM,WAAW,YAAY;IAC3B,gBAAgB;IAChB,QAAQ,EAAE,MAAM,CAAC;IACjB,sDAAsD;IACtD,KAAK,CAAC,EAAE,MAAM,CAAC;CAChB;AAED;;GAEG;AACH,MAAM,WAAW,iBAAiB;IAChC,wCAAwC;IACxC,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,oCAAoC;IACpC,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;IAC7C,+CAA+C;IAC/C,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,wCAAwC;IACxC,aAAa,EAAE,MAAM,CAAC;IACtB,8BAA8B;IAC9B,YAAY,EAAE,WAAW,EAAE,CAAC;IAC5B,sDAAsD;IACtD,KAAK,CAAC,EAAE,kBAAkB,CAAC;IAC3B,mEAAmE;IACnE,YAAY,CAAC,EAAE,wBAAwB,CAAC;CACzC;AAED,MAAM,WAAW,WAAW;IAC1B,SAAS,EAAE,MAAM,CAAC;IAClB,OAAO,EAAE,OAAO,CAAC;IACjB,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,YAAY,CAAC,EAAE,wBAAwB,CAAC;IACxC,KAAK,CAAC,EAAE,iBAAiB,CAAC;CAC3B;AAED;;;;;;GAMG;AACH,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,MAAM,EACb,KAAK,EAAE,YAAY,EAAE,EACrB,WAAW,CAAC,EAAE,MAAM,EACpB,kBAAkB,CAAC,EAAE,MAAM,EAC3B,mBAAmB,CAAC,EAAE,yBAAyB,GAC9C,MAAM,CAqIR;AAuSD,wBAAsB,eAAe,CACnC,OAAO,EAAE,aAAa,EACtB,MAAM,EAAE,SAAS,EACjB,gBAAgB,EAAE,MAAM,EACxB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EAAE,EACvB,iBAAiB,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,EAC/C,UAAU,GAAE,MAAsB,EAClC,KAAK,UAAQ,GACZ,OAAO,CAAC,iBAAiB,CAAC,CAwF5B;AAED,wBAAsB,iBAAiB,CACrC,OAAO,EAAE,aAAa,EACtB,MAAM,EAAE,SAAS,EACjB,gBAAgB,EAAE,MAAM,EACxB,WAAW,EAAE,MAAM,EACnB,aAAa,EAAE,MAAM,EAAE,EACvB,iBAAiB,GAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAM,EAC/C,UAAU,GAAE,MAAsB,EAClC,KAAK,UAAQ,GACZ,OAAO,CAAC,iBAAiB,CAAC,CAW5B;AA0CD;;GAEG;AACH,wBAAgB,oBAAoB,CAAC,MAAM,EAAE;IAC3C,MAAM,EAAE,WAAW,EAAE,CAAC;IACtB,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;IAC7C,aAAa,EAAE,MAAM,CAAC;IACtB,aAAa,CAAC,EAAE,aAAa,CAAC;IAC9B,KAAK,CAAC,EAAE,kBAAkB,CAAC;CAC5B,GAAG,IAAI,CAkDP;AAED;;GAEG;AACH,wBAAgB,iBAAiB,CAAC,MAAM,EAAE;IACxC,OAAO,EAAE,UAAU,CAAC,OAAO,gBAAgB,CAAC,CAAC;CAC9C,GAAG,OAAO,CAEV"}

package/dist/lib/review-core.js

@@ -1,18 +1,17 @@
 /**
  * Core review execution logic shared between local and platform reviews
  *
- * This module contains the common agent execution logic to avoid duplication
- * between review-orchestrator.ts (local) and unified-review-executor.ts (platform).
+ * This module contains the common agent execution logic used by the review orchestrator.
  */
 import chalk from 'chalk';
-import { getAgentNames, getDefaultSkills, getModelOverrides, getUnifiedModelOverride, normalizeAgentConfig, } from './config.js';
+import { getModelOverrides, getReviewAgentIds, getUnifiedModelOverride, resolveAgentSkills, } from './config.js';
 import { buildReviewPrompt } from './context-loader.js';
 import { parseReviewIssues } from './issue-parser.js';
 import { parseReviewOutput } from './review-parser.js';
 import { calculateSummary } from './comment-formatter.js';
-import { loadReviewAgents } from '../runtime/agent-loader.js';
+import { loadAgents } from '../runtime/agent-loader.js';
 import { getLogger } from './logger.js';
-import { aggregateAgentUsage, applyUsageMessage, createAgentUsageSummary, } from './review-usage.js';
+import { aggregateAgentUsage, applySkillCall, applyToolCall, applyUsageMessage, createAgentUsageSummary, } from './review-usage.js';
 import { estimatePromptBudget, formatPromptBudgetEstimate } from './prompt-budget.js';
 /**
  * Build base review instructions for agents
@@ -21,7 +20,7 @@ import { estimatePromptBudget, formatPromptBudgetEstimate } from './prompt-budge
  * @param files - List of files with optional diff content
  * @param diffCommand - Fallback git diff command hint (used when diff not provided)
  */
-export function buildBaseInstructions(label, files, diffCommand, compressionSummary) {
+export function buildBaseInstructions(label, files, diffCommand, compressionSummary, verificationContext) {
     // Check if we have actual diff content
     const filesWithDiffs = files.filter((f) => f.patch);
     const hasDiffs = filesWithDiffs.length > 0;
@@ -81,18 +80,19 @@ ${compressionSummary ? `${compressionSummary}\n\n` : ''}Output requirements:
       "references": ["https://link1", "https://link2"],
       "agent": "security" | "quality" | "style" | "performance" | "documentation" | "unified"
     }
-  ]
+  ]${verificationContext ? buildVerificationSchemaSnippet() : ''}
 }
 
 **Analysis approach:**
 1. First, use Grep or Read to quickly understand the project's existing patterns relevant to the changed files (e.g., existing validation, error handling, auth patterns, naming conventions).
 2. Then analyze the diff content above against those established patterns.
+3. If the prompt says a diff was omitted or summarized, call git_diff for that file before making file-specific claims.
 
 **Review rules:**
-3. **IMPORTANT: Only report issues on lines that were actually changed or added (lines starting with + in the diff).** Do not report issues on unchanged code.
-4. Only flag deviations or new risks introduced by the changes.
-5. Populate summary counts based on the issues you report (use 0 when none).
-6. Focus on the changes - only report issues for newly added or modified lines (lines with + prefix in the diff).`;
+4. **IMPORTANT: Only report issues on lines that were actually changed or added (lines starting with + in the diff).** Do not report issues on unchanged code.
+5. Only flag deviations or new risks introduced by the changes.
+6. Populate summary counts based on the issues you report (use 0 when none).
+7. Focus on the changes - only report issues for newly added or modified lines (lines with + prefix in the diff).`;
     }
     // No diff content available - instruct agent to read files directly
     return `Review the following changed files from ${label}:
@@ -139,40 +139,52 @@ Output requirements:
       "references": ["https://link1", "https://link2"],
       "agent": "security" | "quality" | "style" | "performance" | "documentation" | "unified"
     }
-  ]
+  ]${verificationContext ? buildVerificationSchemaSnippet() : ''}
 }
 
 **Instructions:**
-1. The diffs for these files were omitted due to size constraints. Use the Read tool to examine the changed files listed above.
-2. Focus your review on newly added or modified code patterns.
+1. The diffs for these files were omitted due to size constraints. Use git_diff to inspect the file diffs, then use Read/Grep for surrounding context as needed.
+2. Focus your review on newly added or modified code patterns from the git_diff output.
 3. **IMPORTANT: Only report issues on lines that were actually changed or added.** Do not report issues on existing code that was not modified.
 4. Analyze the changed code for issues in your specialty area
 5. Populate summary counts based on the issues you report (use 0 when none).
 6. Focus on the changes - only report issues for newly added or modified lines.`;
 }
+function buildVerificationSchemaSnippet() {
+    return `,
+  "verification": {
+    "findings": [
+      {
+        "id": "F001",
+        "disposition": "resolved",
+        "rationale": "short explanation",
+        "issue": null
+      }
+    ]
+  }`;
+}
 /**
  * Get information about configured review agents
  */
 function getConfiguredAgentInfo(config, workingDir, cachedAgents) {
-    const configuredNames = getAgentNames(config);
-    const allAgents = cachedAgents ?? loadReviewAgents(workingDir, config);
+    const configuredNames = getReviewAgentIds(config);
+    const allAgents = cachedAgents ?? loadAgents(workingDir, config);
     return configuredNames
-        .map((name) => {
-        const fullName = `review/${name}`;
-        const agent = allAgents.find((a) => a.name === fullName);
+        .map((agentId) => {
+        const agent = allAgents.find((a) => a.id === agentId);
         return {
-            name,
-            description: agent?.description ?? `${name} review agent`,
+            name: agentId,
+            description: agent?.description ?? `${agentId} review agent`,
         };
     })
         .filter((a) => a !== null);
 }
 function validateConfiguredReviewAgents(config, workingDir, cachedAgents) {
-    const configuredNames = getAgentNames(config);
-    const availableAgents = new Set((cachedAgents ?? loadReviewAgents(workingDir, config))
-        .filter((agent) => agent.name.startsWith('review/'))
-        .map((agent) => agent.name.replace(/^review\//, '')));
-    const missingAgents = configuredNames.filter((name) => !availableAgents.has(name));
+    const configuredAgentIds = getReviewAgentIds(config);
+    const availableAgents = new Set((cachedAgents ?? loadAgents(workingDir, config))
+        .filter((agent) => agent.namespace === 'review')
+        .map((agent) => agent.id));
+    const missingAgents = configuredAgentIds.filter((agentId) => !availableAgents.has(agentId));
     if (missingAgents.length === 0) {
         return;
     }
@@ -192,11 +204,7 @@ function summarizeRunUsage(agentResults) {
     return aggregateAgentUsage(agentUsage);
 }
 function getConfiguredSkillsForAgent(config, agentType) {
-    const defaultSkills = getDefaultSkills(config);
-    const normalizedAgents = normalizeAgentConfig(config.review.agents);
-    const agentConfig = normalizedAgents.find((agent) => agent.name === agentType || agent.name === `review/${agentType}`);
-    const agentSkills = (agentConfig?.skills ?? []).map(String).filter((skill) => skill.length > 0);
-    return [...new Set([...defaultSkills, ...agentSkills])];
+    return resolveAgentSkills(config, agentType);
 }
 function resolveReviewModelOverrides(config) {
     return {
@@ -260,144 +268,8 @@ function extractSkillNameFromToolOutput(content) {
     }
     return undefined;
 }
-export async function runUnifiedReviewAgent(runtime, config, baseInstructions, reviewLabel, filteredFiles, additionalContext = {}, workingDir = process.cwd(), debug = false) {
-    const agentType = 'unified-reviewer';
-    const agentName = `review/${agentType}`;
-    console.log(chalk.bold('🎯 Selected Agents: unified-reviewer\n'));
-    console.log(chalk.gray('Running unified review...\n'));
-    let agentUsage = createAgentUsageSummary(agentType);
-    const configuredSkills = getConfiguredSkillsForAgent(config, agentType);
-    const reviewModelOverrides = resolveReviewModelOverrides(config);
-    const modelId = reviewModelOverrides[agentName];
-    let sawSkillToolCall = false;
-    const describeSummary = typeof additionalContext.describeSummary === 'string'
-        ? additionalContext.describeSummary
-        : undefined;
-    try {
-        const reviewPrompt = buildReviewPrompt(agentType, baseInstructions, reviewLabel, filteredFiles, workingDir, config, describeSummary);
-        logPromptInputBudget({
-            runtime,
-            config,
-            agentType,
-            prompt: reviewPrompt,
-            modelId,
-        });
-        const logger = getLogger();
-        if (debug) {
-            console.log(chalk.gray('┌── DEBUG: Message sent to review agent'));
-            console.log(chalk.gray(`│ Agent: ${agentName}`));
-            console.log(chalk.gray('│ Prompt:'));
-            console.log(chalk.gray('─'.repeat(60)));
-            console.log(reviewPrompt);
-            console.log(chalk.gray('─'.repeat(60)));
-            console.log(chalk.gray(`└── End message for ${agentName}\n`));
-        }
-        else {
-            logger.agentInput(agentType, reviewPrompt);
-        }
-        const session = await runtime.createSession({
-            agent: agentName,
-            message: reviewPrompt,
-        });
-        const agentIssues = [];
-        let fullResponse = '';
-        for await (const message of runtime.streamMessages(session.id)) {
-            if (message.role === 'tool') {
-                // Detect skill loading: legacy 'skill' tool or Pi-native read of SKILL.md
-                const skillFromLegacy = message.toolName === 'skill'
-                    ? extractSkillNameFromToolOutput(message.content)
-                    : undefined;
-                const skillFromRead = extractSkillNameFromReadToolOutput(message.toolName, message.content);
-                const skillName = skillFromLegacy ?? skillFromRead;
-                if (skillName) {
-                    logger.skillLoaded(skillName, agentType);
-                    sawSkillToolCall = true;
-                }
-                else {
-                    logger.toolOutput(message.toolName ?? 'unknown', agentType, message.content);
-                }
-                continue;
-            }
-            if (message.role === 'assistant') {
-                agentUsage = applyUsageMessage(agentUsage, message);
-                if (!message.content.trim()) {
-                    continue;
-                }
-                fullResponse += message.content;
-                if (debug) {
-                    console.log(chalk.gray(`┌── DEBUG: Full response from ${agentName}`));
-                    console.log(message.content);
-                    console.log(chalk.gray(`└── End response for ${agentName}\n`));
-                }
-                else {
-                    logger.agentMessage(agentType, message.content);
-                }
-            }
-        }
-        if (configuredSkills.length > 0 && !sawSkillToolCall) {
-            logger.noSkillCalls(agentType);
-        }
-        await runtime.closeSession(session.id);
-        try {
-            const reviewOutput = await parseReviewOutput(workingDir, debug, fullResponse);
-            const parsedIssues = parseReviewIssues(JSON.stringify(reviewOutput), 'unified');
-            if (parsedIssues.length > 0) {
-                agentIssues.push(...parsedIssues);
-                console.log(chalk.green(`✓ [unified] Found ${parsedIssues.length} issue(s)`));
-            }
-        }
-        catch (parseError) {
-            console.error(chalk.red('Failed to parse unified review output'));
-            if (debug) {
-                console.log(chalk.dim('Unified agent output:'), fullResponse);
-            }
-            const reason = parseError instanceof Error ? `: ${parseError.message}` : '';
-            throw new Error(`Unified review agent did not return valid JSON output${reason}`);
-        }
-        const summary = calculateSummary(filteredFiles.length, agentIssues);
-        const agentResults = [
-            {
-                agentType,
-                success: true,
-                issues: agentIssues,
-                usage: {
-                    ...agentUsage,
-                    success: true,
-                },
-            },
-        ];
-        return {
-            issues: agentIssues,
-            summary,
-            filesReviewed: filteredFiles.length,
-            agentResults,
-            usage: summarizeRunUsage(agentResults),
-        };
-    }
-    catch (error) {
-        console.error(chalk.red(`✗ unified-reviewer agent failed: ${error}`));
-        const agentResults = [
-            {
-                agentType,
-                success: false,
-                issues: [],
-                usage: {
-                    ...agentUsage,
-                    success: false,
-                },
-            },
-        ];
-        return {
-            issues: [],
-            summary: calculateSummary(filteredFiles.length, []),
-            filesReviewed: filteredFiles.length,
-            agentResults,
-            usage: summarizeRunUsage(agentResults),
-        };
-    }
-}
-async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel, filteredFiles, workingDir, debug, agentType, reviewModelOverrides, describeSummary) {
-    const agentName = `review/${agentType}`;
+async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel, filteredFiles, workingDir, debug, agentType, reviewModelOverrides, describeSummary, verificationContext) {
+    const agentName = agentType;
     console.log(chalk.gray(`Running ${agentType} review...\n`));
     let agentUsage = createAgentUsageSummary(agentType);
     const configuredSkills = getConfiguredSkillsForAgent(config, agentType);
@@ -405,7 +277,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
     let sawSkillToolCall = false;
     try {
         // Build prompt with global and agent-specific context
-        const reviewPrompt = buildReviewPrompt(agentType, baseInstructions, reviewLabel, filteredFiles, workingDir, config, describeSummary);
+        const reviewPrompt = buildReviewPrompt(agentType, baseInstructions, reviewLabel, filteredFiles, workingDir, config, describeSummary, verificationContext);
         logPromptInputBudget({
             runtime,
             config,
@@ -435,6 +307,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
         // Collect results from this agent
         for await (const message of runtime.streamMessages(session.id)) {
             if (message.role === 'tool') {
+                agentUsage = applyToolCall(agentUsage, message.toolName);
                 // Detect skill loading: legacy 'skill' tool or Pi-native read of SKILL.md
                 const skillFromLegacy = message.toolName === 'skill'
                     ? extractSkillNameFromToolOutput(message.content)
@@ -442,6 +315,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
                 const skillFromRead = extractSkillNameFromReadToolOutput(message.toolName, message.content);
                 const skillName = skillFromLegacy ?? skillFromRead;
                 if (skillName) {
+                    agentUsage = applySkillCall(agentUsage, skillName);
                     logger.skillLoaded(skillName, agentType);
                     sawSkillToolCall = true;
                 }
@@ -472,6 +346,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
         await runtime.closeSession(session.id);
         const reviewOutput = await parseReviewOutput(workingDir, debug, fullResponse);
         const parsedIssues = parseReviewIssues(JSON.stringify(reviewOutput), agentType);
+        const verification = parseReviewVerification(reviewOutput);
         if (parsedIssues.length > 0) {
             agentIssues.push(...parsedIssues);
             console.log(chalk.green(`✓ [${agentType}] Found ${parsedIssues.length} issue(s)`));
@@ -480,6 +355,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
             agentType,
             success: true,
             issues: agentIssues,
+            verification,
             usage: {
                 ...agentUsage,
                 success: true,
@@ -501,7 +377,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
 }
 export async function runReviewAgents(runtime, config, baseInstructions, reviewLabel, filteredFiles, additionalContext = {}, workingDir = process.cwd(), debug = false) {
     console.log(chalk.gray('Starting code analysis...\n'));
-    const allAgents = loadReviewAgents(workingDir, config);
+    const allAgents = loadAgents(workingDir, config);
     validateConfiguredReviewAgents(config, workingDir, allAgents);
     const configuredAgentInfo = getConfiguredAgentInfo(config, workingDir, allAgents);
     if (configuredAgentInfo.length > 0) {
@@ -511,13 +387,16 @@ export async function runReviewAgents(runtime, config, baseInstructions, reviewL
         });
         console.log('');
     }
-    const agentNames = getAgentNames(config);
+    const agentNames = getReviewAgentIds(config);
     console.log(chalk.bold(`🎯 Selected Agents: ${agentNames.join(', ') || 'None'}\n`));
     const reviewModelOverrides = resolveReviewModelOverrides(config);
     const describeSummary = typeof additionalContext.describeSummary === 'string'
         ? additionalContext.describeSummary
         : undefined;
-    const agentResults = await Promise.all(agentNames.map((agentType) => executeSingleAgent(runtime, config, baseInstructions, reviewLabel, filteredFiles, workingDir, debug, agentType, reviewModelOverrides, describeSummary)));
+    const verificationContext = isReviewVerificationContext(additionalContext.verificationContext)
+        ? additionalContext.verificationContext
+        : undefined;
+    const agentResults = await Promise.all(agentNames.map((agentType) => executeSingleAgent(runtime, config, baseInstructions, reviewLabel, filteredFiles, workingDir, debug, agentType, reviewModelOverrides, describeSummary, verificationContext)));
     // Check agent results
     const successfulAgents = agentResults.filter((r) => r.success);
     const failedAgents = agentResults.filter((r) => !r.success);
@@ -537,17 +416,53 @@ export async function runReviewAgents(runtime, config, baseInstructions, reviewL
     const issues = [];
     agentResults.forEach((result) => issues.push(...result.issues));
     const summary = calculateSummary(filteredFiles.length, issues);
+    const verificationFindings = agentResults.flatMap((result) => result.verification?.findings ?? []);
     return {
         issues,
         summary,
         filesReviewed: filteredFiles.length,
         agentResults,
+        verification: verificationFindings.length > 0 ? { findings: verificationFindings } : undefined,
         usage: summarizeRunUsage(agentResults),
     };
 }
 export async function runReviewPipeline(runtime, config, baseInstructions, reviewLabel, filteredFiles, additionalContext = {}, workingDir = process.cwd(), debug = false) {
     return runReviewAgents(runtime, config, baseInstructions, reviewLabel, filteredFiles, additionalContext, workingDir, debug);
 }
+function isReviewVerificationContext(value) {
+    if (!value || typeof value !== 'object') {
+        return false;
+    }
+    const candidate = value;
+    return !!candidate.artifact && Array.isArray(candidate.artifact.findings);
+}
+function parseReviewVerification(value) {
+    if (!value || typeof value !== 'object') {
+        return undefined;
+    }
+    const verification = value.verification;
+    if (!verification || typeof verification !== 'object') {
+        return undefined;
+    }
+    const findings = verification.findings;
+    if (!Array.isArray(findings)) {
+        return undefined;
+    }
+    const parsed = findings.filter(isReviewVerificationFinding);
+    return parsed.length > 0 ? { findings: parsed } : undefined;
+}
+function isReviewVerificationFinding(value) {
+    if (!value || typeof value !== 'object') {
+        return false;
+    }
+    const candidate = value;
+    return (typeof candidate.id === 'string' &&
+        isReviewVerificationDisposition(candidate.disposition) &&
+        (candidate.rationale === undefined || typeof candidate.rationale === 'string'));
+}
+function isReviewVerificationDisposition(value) {
+    return value === 'resolved' || value === 'still_open' || value === 'partial';
+}
 /**
  * Display review summary to terminal
  */