@diff-review-system/drs 4.0.0-rc.4 → 4.1.0

package/dist/lib/review-artifact.js

@@ -0,0 +1,172 @@
+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' &&
+        !!candidate.summary &&
+        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,CAAC,CAAC,SAAS,CAAC,OAAO;QACnB,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,15 @@ 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[];
+    error?: string;
+    verification?: ReviewVerificationResult;
     usage?: AgentUsageSummary;
 }
 /**
@@ -48,8 +52,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;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,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;AA8ID,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,KAAK,CAAC,EAAE,MAAM,CAAC;IACf,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,CA6HR;AAySD,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,CA6F5B;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,8 +1,7 @@
 /**
  * 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 { getModelOverrides, getReviewAgentIds, getUnifiedModelOverride, resolveAgentSkills, } from './config.js';
@@ -12,7 +11,7 @@ import { parseReviewOutput } from './review-parser.js';
 import { calculateSummary } from './comment-formatter.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;
@@ -42,12 +41,8 @@ The following shows exactly what changed in this PR/MR:
 ${diffContent}
 
 ${compressionSummary ? `${compressionSummary}\n\n` : ''}Output requirements:
-- You MUST call the write_json_output tool with:
-  - outputType: "review_output"
-  - payload: the JSON object
-  - After calling the tool, return only the JSON pointer returned by the tool
-    (e.g. {"outputType":"review_output","outputPath":".drs/review-output.json"})
-- Do not return raw JSON directly.
+- Return only the raw JSON object.
+- Do not call write_json_output.
 - Do not include markdown, code fences, or extra text.
 - Follow this exact schema:
 {
@@ -81,18 +76,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}:
@@ -100,12 +96,8 @@ ${compressionSummary ? `${compressionSummary}\n\n` : ''}Output requirements:
 ${fileList}
 
 Output requirements:
-- You MUST call the write_json_output tool with:
-  - outputType: "review_output"
-  - payload: the JSON object
-  - After calling the tool, return only the JSON pointer returned by the tool
-    (e.g. {"outputType":"review_output","outputPath":".drs/review-output.json"})
-- Do not return raw JSON directly.
+- Return only the raw JSON object.
+- Do not call write_json_output.
 - Do not include markdown, code fences, or extra text.
 - Follow this exact schema:
 {
@@ -139,17 +131,30 @@ 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
  */
@@ -255,143 +260,7 @@ function extractSkillNameFromToolOutput(content) {
     }
     return undefined;
 }
-export async function runUnifiedReviewAgent(runtime, config, baseInstructions, reviewLabel, filteredFiles, additionalContext = {}, workingDir = process.cwd(), debug = false) {
-    const agentType = 'review/unified-reviewer';
-    const agentName = agentType;
-    console.log(chalk.bold('🎯 Selected Agents: review/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), agentType);
-            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(`✗ review/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) {
+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);
@@ -400,7 +269,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,
@@ -430,6 +299,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)
@@ -437,6 +307,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;
                 }
@@ -467,6 +338,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)`));
@@ -475,6 +347,7 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
             agentType,
             success: true,
             issues: agentIssues,
+            verification,
             usage: {
                 ...agentUsage,
                 success: true,
@@ -482,11 +355,13 @@ async function executeSingleAgent(runtime, config, baseInstructions, reviewLabel
         };
     }
     catch (error) {
-        console.error(chalk.red(`✗ ${agentType} agent failed: ${error}`));
+        const message = error instanceof Error ? error.message : String(error);
+        console.error(chalk.red(`✗ ${agentType} agent failed: ${message}`));
         return {
             agentType,
             success: false,
             issues: [],
+            error: message,
             usage: {
                 ...agentUsage,
                 success: false,
@@ -512,18 +387,24 @@ export async function runReviewAgents(runtime, config, baseInstructions, reviewL
     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);
     if (successfulAgents.length === 0) {
+        const failureDetails = failedAgents
+            .map((result) => `${result.agentType}: ${result.error ?? 'unknown error'}`)
+            .join('; ');
         console.error(chalk.red('\n✗ All review agents failed!\n'));
         console.error(chalk.yellow('This usually means:\n' +
             '  1. Model configuration is incorrect or missing\n' +
             '  2. API credentials are invalid or missing\n' +
             '  3. Models are not accessible or timed out\n' +
             '  4. Agents cannot find files to review\n'));
-        throw new Error('All review agents failed');
+        throw new Error(failureDetails ? `All review agents failed: ${failureDetails}` : 'All review agents failed');
     }
     if (failedAgents.length > 0) {
         console.log(chalk.yellow(`\n⚠️  ${failedAgents.length} of ${agentResults.length} agents failed: ${failedAgents.map((r) => r.agentType).join(', ')}\n`));
@@ -532,17 +413,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
  */