@humanbased/crosscheck 1.3.0-beta.91 → 1.3.0-beta.93

package/dist/lib/adoption.js

@@ -0,0 +1,165 @@
+// Adoption metrics answer "is crosscheck actually being used, and does it reach a
+// verdict fast enough to matter" — a different question from `impact`, which prices
+// the value of reviews that already happened.
+//
+// Every number here is derived from the local NDJSON logs in ~/.crosscheck/logs.
+// Nothing is transmitted: there is no endpoint, no account, and no network call in
+// this file. See docs/metrics.md for the field-by-field inventory.
+// A verdict-bearing latency is only meaningful for a review that produced one; a
+// verdictless review (from `crosscheck review`) has nothing to time to.
+export function prOpenToVerdictMs(createdAt, verdict, now = Date.now()) {
+    if (verdict === null || createdAt === undefined)
+        return undefined;
+    const opened = new Date(createdAt).getTime();
+    if (Number.isNaN(opened))
+        return undefined;
+    const elapsed = now - opened;
+    // A negative elapsed time means the clocks disagree, not that the PR was
+    // reviewed before it existed. Drop it rather than publish a nonsense number.
+    return elapsed >= 0 ? elapsed : undefined;
+}
+export function isoWeekMonday(ts) {
+    const d = new Date(ts);
+    if (Number.isNaN(d.getTime()))
+        return null;
+    const day = d.getUTCDay();
+    const diff = day === 0 ? -6 : 1 - day;
+    return new Date(Date.UTC(d.getUTCFullYear(), d.getUTCMonth(), d.getUTCDate() + diff))
+        .toISOString().slice(0, 10);
+}
+export function percentile(sorted, fraction) {
+    if (sorted.length === 0)
+        return 0;
+    // Nearest-rank on the sorted sample: no interpolation, so every reported value
+    // is one that actually occurred.
+    const rank = Math.ceil(fraction * sorted.length) - 1;
+    return sorted[Math.min(Math.max(rank, 0), sorted.length - 1)];
+}
+export function buildAdoptionReport(lines, period) {
+    const weekly = new Map();
+    const activeRepos = new Set();
+    const latencies = [];
+    let unmeasured = 0;
+    const counts = {
+        onboard_started: 0,
+        onboard_completed: 0,
+        reviews_started: 0,
+        reviews_completed: 0,
+        rechecks_completed: 0,
+        blocking: 0,
+        fixes: 0,
+    };
+    for (const line of lines) {
+        switch (line.event) {
+            case 'onboard_started':
+                counts.onboard_started++;
+                break;
+            case 'onboard_completed':
+                // The event is emitted for both outcomes so an abandoned setup is
+                // visible; only a success counts as completed.
+                if (line.outcome === 'completed')
+                    counts.onboard_completed++;
+                break;
+            case 'review_started':
+                counts.reviews_started++;
+                break;
+            case 'review_complete': {
+                counts.reviews_completed++;
+                if (line.step_type === 'recheck')
+                    counts.rechecks_completed++;
+                if (line.repo) {
+                    activeRepos.add(line.repo);
+                    const week = isoWeekMonday(line.ts);
+                    if (week) {
+                        const bucket = weekly.get(week) ?? { repos: new Set(), reviews: 0 };
+                        bucket.repos.add(line.repo);
+                        bucket.reviews++;
+                        weekly.set(week, bucket);
+                    }
+                }
+                if (typeof line.open_to_verdict_ms === 'number')
+                    latencies.push(line.open_to_verdict_ms);
+                else if (line.verdict)
+                    unmeasured++;
+                break;
+            }
+            case 'blocking_finding_posted':
+                counts.blocking++;
+                break;
+            case 'fix_complete':
+                // delivery: 'comment' still counts — a suggestion the author applied is
+                // adoption. A no-op fix (applied_count 0) is not.
+                if ((line.applied_count ?? 0) > 0)
+                    counts.fixes++;
+                break;
+        }
+    }
+    latencies.sort((a, b) => a - b);
+    return {
+        period,
+        onboarding: {
+            started: counts.onboard_started,
+            completed: counts.onboard_completed,
+            abandoned: Math.max(0, counts.onboard_started - counts.onboard_completed),
+        },
+        activity: {
+            reviews_started: counts.reviews_started,
+            reviews_completed: counts.reviews_completed,
+            rechecks_completed: counts.rechecks_completed,
+            blocking_findings_posted: counts.blocking,
+            fixes_applied: counts.fixes,
+            active_repos: activeRepos.size,
+        },
+        weekly: [...weekly.entries()]
+            .sort(([a], [b]) => a.localeCompare(b))
+            .map(([week, { repos, reviews }]) => ({ week, active_repos: repos.size, reviews })),
+        open_to_verdict: {
+            count: latencies.length,
+            p50_ms: percentile(latencies, 0.5),
+            p90_ms: percentile(latencies, 0.9),
+            max_ms: latencies.at(-1) ?? 0,
+            unmeasured,
+        },
+        first_run_failures: firstRunFailures(lines),
+    };
+}
+// "First run" is a session that never completed a review. Its first error is the
+// one that stopped a new install from reaching a verdict — the category worth
+// fixing. Sessions that did complete a review are excluded even if they later
+// errored: those are operational failures, not activation failures.
+export function firstRunFailures(lines) {
+    const sessions = [];
+    let current = null;
+    const close = () => {
+        if (current)
+            sessions.push(current);
+        current = null;
+    };
+    for (const line of lines) {
+        if (line.event === 'session_start') {
+            close();
+            current = { completedReview: false, firstErrorCategory: null };
+            continue;
+        }
+        // Logs from before session events existed, or a truncated file, would
+        // otherwise drop every entry on the floor.
+        if (current === null)
+            current = { completedReview: false, firstErrorCategory: null };
+        if (line.event === 'review_complete')
+            current.completedReview = true;
+        if (line.event === 'error' && current.firstErrorCategory === null) {
+            current.firstErrorCategory = line.category ?? 'unknown';
+        }
+        if (line.event === 'session_end')
+            close();
+    }
+    close();
+    const failures = {};
+    for (const session of sessions) {
+        if (session.completedReview || session.firstErrorCategory === null)
+            continue;
+        failures[session.firstErrorCategory] = (failures[session.firstErrorCategory] ?? 0) + 1;
+    }
+    return failures;
+}
+//# sourceMappingURL=adoption.js.map

package/dist/lib/adoption.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"adoption.js","sourceRoot":"","sources":["../../src/lib/adoption.ts"],"names":[],"mappings":"AAAA,kFAAkF;AAClF,oFAAoF;AACpF,8CAA8C;AAC9C,EAAE;AACF,iFAAiF;AACjF,mFAAmF;AACnF,mEAAmE;AAoDnE,iFAAiF;AACjF,wEAAwE;AACxE,MAAM,UAAU,iBAAiB,CAC/B,SAA6B,EAC7B,OAAsB,EACtB,MAAc,IAAI,CAAC,GAAG,EAAE;IAExB,IAAI,OAAO,KAAK,IAAI,IAAI,SAAS,KAAK,SAAS;QAAE,OAAO,SAAS,CAAA;IACjE,MAAM,MAAM,GAAG,IAAI,IAAI,CAAC,SAAS,CAAC,CAAC,OAAO,EAAE,CAAA;IAC5C,IAAI,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC;QAAE,OAAO,SAAS,CAAA;IAC1C,MAAM,OAAO,GAAG,GAAG,GAAG,MAAM,CAAA;IAC5B,yEAAyE;IACzE,6EAA6E;IAC7E,OAAO,OAAO,IAAI,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,CAAC,CAAC,SAAS,CAAA;AAC3C,CAAC;AAED,MAAM,UAAU,aAAa,CAAC,EAAU;IACtC,MAAM,CAAC,GAAG,IAAI,IAAI,CAAC,EAAE,CAAC,CAAA;IACtB,IAAI,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,EAAE,CAAC;QAAE,OAAO,IAAI,CAAA;IAC1C,MAAM,GAAG,GAAG,CAAC,CAAC,SAAS,EAAE,CAAA;IACzB,MAAM,IAAI,GAAG,GAAG,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,GAAG,GAAG,CAAA;IACrC,OAAO,IAAI,IAAI,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC,WAAW,EAAE,EAAE,CAAC,CAAC,UAAU,EAAE,GAAG,IAAI,CAAC,CAAC;SAClF,WAAW,EAAE,CAAC,KAAK,CAAC,CAAC,EAAE,EAAE,CAAC,CAAA;AAC/B,CAAC;AAED,MAAM,UAAU,UAAU,CAAC,MAAgB,EAAE,QAAgB;IAC3D,IAAI,MAAM,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,CAAC,CAAA;IACjC,+EAA+E;IAC/E,iCAAiC;IACjC,MAAM,IAAI,GAAG,IAAI,CAAC,IAAI,CAAC,QAAQ,GAAG,MAAM,CAAC,MAAM,CAAC,GAAG,CAAC,CAAA;IACpD,OAAO,MAAM,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,CAAC,GAAG,CAAC,IAAI,EAAE,CAAC,CAAC,EAAE,MAAM,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAA;AAC/D,CAAC;AAED,MAAM,UAAU,mBAAmB,CACjC,KAAwB,EACxB,MAAuD;IAEvD,MAAM,MAAM,GAAG,IAAI,GAAG,EAAmD,CAAA;IACzE,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAA;IACrC,MAAM,SAAS,GAAa,EAAE,CAAA;IAC9B,IAAI,UAAU,GAAG,CAAC,CAAA;IAElB,MAAM,MAAM,GAAG;QACb,eAAe,EAAE,CAAC;QAClB,iBAAiB,EAAE,CAAC;QACpB,eAAe,EAAE,CAAC;QAClB,iBAAiB,EAAE,CAAC;QACpB,kBAAkB,EAAE,CAAC;QACrB,QAAQ,EAAE,CAAC;QACX,KAAK,EAAE,CAAC;KACT,CAAA;IAED,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,QAAQ,IAAI,CAAC,KAAK,EAAE,CAAC;YACnB,KAAK,iBAAiB;gBACpB,MAAM,CAAC,eAAe,EAAE,CAAA;gBACxB,MAAK;YACP,KAAK,mBAAmB;gBACtB,kEAAkE;gBAClE,+CAA+C;gBAC/C,IAAI,IAAI,CAAC,OAAO,KAAK,WAAW;oBAAE,MAAM,CAAC,iBAAiB,EAAE,CAAA;gBAC5D,MAAK;YACP,KAAK,gBAAgB;gBACnB,MAAM,CAAC,eAAe,EAAE,CAAA;gBACxB,MAAK;YACP,KAAK,iBAAiB,CAAC,CAAC,CAAC;gBACvB,MAAM,CAAC,iBAAiB,EAAE,CAAA;gBAC1B,IAAI,IAAI,CAAC,SAAS,KAAK,SAAS;oBAAE,MAAM,CAAC,kBAAkB,EAAE,CAAA;gBAC7D,IAAI,IAAI,CAAC,IAAI,EAAE,CAAC;oBACd,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;oBAC1B,MAAM,IAAI,GAAG,aAAa,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;oBACnC,IAAI,IAAI,EAAE,CAAC;wBACT,MAAM,MAAM,GAAG,MAAM,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,IAAI,GAAG,EAAU,EAAE,OAAO,EAAE,CAAC,EAAE,CAAA;wBAC3E,MAAM,CAAC,KAAK,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,CAAC,CAAA;wBAC3B,MAAM,CAAC,OAAO,EAAE,CAAA;wBAChB,MAAM,CAAC,GAAG,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;oBAC1B,CAAC;gBACH,CAAC;gBACD,IAAI,OAAO,IAAI,CAAC,kBAAkB,KAAK,QAAQ;oBAAE,SAAS,CAAC,IAAI,CAAC,IAAI,CAAC,kBAAkB,CAAC,CAAA;qBACnF,IAAI,IAAI,CAAC,OAAO;oBAAE,UAAU,EAAE,CAAA;gBACnC,MAAK;YACP,CAAC;YACD,KAAK,yBAAyB;gBAC5B,MAAM,CAAC,QAAQ,EAAE,CAAA;gBACjB,MAAK;YACP,KAAK,cAAc;gBACjB,wEAAwE;gBACxE,kDAAkD;gBAClD,IAAI,CAAC,IAAI,CAAC,aAAa,IAAI,CAAC,CAAC,GAAG,CAAC;oBAAE,MAAM,CAAC,KAAK,EAAE,CAAA;gBACjD,MAAK;QACT,CAAC;IACH,CAAC;IAED,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAA;IAE/B,OAAO;QACL,MAAM;QACN,UAAU,EAAE;YACV,OAAO,EAAE,MAAM,CAAC,eAAe;YAC/B,SAAS,EAAE,MAAM,CAAC,iBAAiB;YACnC,SAAS,EAAE,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,MAAM,CAAC,eAAe,GAAG,MAAM,CAAC,iBAAiB,CAAC;SAC1E;QACD,QAAQ,EAAE;YACR,eAAe,EAAE,MAAM,CAAC,eAAe;YACvC,iBAAiB,EAAE,MAAM,CAAC,iBAAiB;YAC3C,kBAAkB,EAAE,MAAM,CAAC,kBAAkB;YAC7C,wBAAwB,EAAE,MAAM,CAAC,QAAQ;YACzC,aAAa,EAAE,MAAM,CAAC,KAAK;YAC3B,YAAY,EAAE,WAAW,CAAC,IAAI;SAC/B;QACD,MAAM,EAAE,CAAC,GAAG,MAAM,CAAC,OAAO,EAAE,CAAC;aAC1B,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,aAAa,CAAC,CAAC,CAAC,CAAC;aACtC,GAAG,CAAC,CAAC,CAAC,IAAI,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,YAAY,EAAE,KAAK,CAAC,IAAI,EAAE,OAAO,EAAE,CAAC,CAAC;QACrF,eAAe,EAAE;YACf,KAAK,EAAE,SAAS,CAAC,MAAM;YACvB,MAAM,EAAE,UAAU,CAAC,SAAS,EAAE,GAAG,CAAC;YAClC,MAAM,EAAE,UAAU,CAAC,SAAS,EAAE,GAAG,CAAC;YAClC,MAAM,EAAE,SAAS,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC;YAC7B,UAAU;SACX;QACD,kBAAkB,EAAE,gBAAgB,CAAC,KAAK,CAAC;KAC5C,CAAA;AACH,CAAC;AAED,iFAAiF;AACjF,8EAA8E;AAC9E,8EAA8E;AAC9E,oEAAoE;AACpE,MAAM,UAAU,gBAAgB,CAAC,KAAwB;IACvD,MAAM,QAAQ,GAA2E,EAAE,CAAA;IAC3F,IAAI,OAAO,GAA2E,IAAI,CAAA;IAE1F,MAAM,KAAK,GAAG,GAAG,EAAE;QACjB,IAAI,OAAO;YAAE,QAAQ,CAAC,IAAI,CAAC,OAAO,CAAC,CAAA;QACnC,OAAO,GAAG,IAAI,CAAA;IAChB,CAAC,CAAA;IAED,KAAK,MAAM,IAAI,IAAI,KAAK,EAAE,CAAC;QACzB,IAAI,IAAI,CAAC,KAAK,KAAK,eAAe,EAAE,CAAC;YACnC,KAAK,EAAE,CAAA;YACP,OAAO,GAAG,EAAE,eAAe,EAAE,KAAK,EAAE,kBAAkB,EAAE,IAAI,EAAE,CAAA;YAC9D,SAAQ;QACV,CAAC;QACD,sEAAsE;QACtE,2CAA2C;QAC3C,IAAI,OAAO,KAAK,IAAI;YAAE,OAAO,GAAG,EAAE,eAAe,EAAE,KAAK,EAAE,kBAAkB,EAAE,IAAI,EAAE,CAAA;QACpF,IAAI,IAAI,CAAC,KAAK,KAAK,iBAAiB;YAAE,OAAO,CAAC,eAAe,GAAG,IAAI,CAAA;QACpE,IAAI,IAAI,CAAC,KAAK,KAAK,OAAO,IAAI,OAAO,CAAC,kBAAkB,KAAK,IAAI,EAAE,CAAC;YAClE,OAAO,CAAC,kBAAkB,GAAG,IAAI,CAAC,QAAQ,IAAI,SAAS,CAAA;QACzD,CAAC;QACD,IAAI,IAAI,CAAC,KAAK,KAAK,aAAa;YAAE,KAAK,EAAE,CAAA;IAC3C,CAAC;IACD,KAAK,EAAE,CAAA;IAEP,MAAM,QAAQ,GAA2B,EAAE,CAAA;IAC3C,KAAK,MAAM,OAAO,IAAI,QAAQ,EAAE,CAAC;QAC/B,IAAI,OAAO,CAAC,eAAe,IAAI,OAAO,CAAC,kBAAkB,KAAK,IAAI;YAAE,SAAQ;QAC5E,QAAQ,CAAC,OAAO,CAAC,kBAAkB,CAAC,GAAG,CAAC,QAAQ,CAAC,OAAO,CAAC,kBAAkB,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAA;IACxF,CAAC;IACD,OAAO,QAAQ,CAAA;AACjB,CAAC"}

package/dist/lib/runner.d.ts

@@ -98,13 +98,37 @@ export interface WorkflowResult {
         id?: number;
         body: string;
     };
+    /** What each dispatched step actually did. Lets a caller tell "ran and found
+     *  nothing" apart from "never ran", which the verdict alone cannot express —
+     *  every step of a conflict-resolve run can skip and still leave verdict null,
+     *  exactly like a review that approved nothing. */
+    stepOutcomes?: StepOutcomes;
 }
+export interface StepOutcomes {
+    /** Steps that executed. A dispatched step with no recorded result counts as
+     *  ran: only an explicit skip is evidence that nothing happened. */
+    ran: string[];
+    /** Steps dispatched but skipped, each with the reason recorded on its
+     *  step_skipped log entry. */
+    skipped: {
+        step: string;
+        reason: string;
+    }[];
+}
+export declare function summariseStepOutcomes(stepsRun: readonly string[], results: Record<string, StepResult>): StepOutcomes;
+export declare function mergeStepOutcomes(base: StepOutcomes | undefined, next: StepOutcomes | undefined): StepOutcomes | undefined;
 export declare function fixCommitSubject(appliedCount: number, vendor: Vendor): string;
 export declare function fixPRCommitSubject(prNumber: number, vendor: Vendor): string;
 export declare function conflictResolveCommitSubject(conflictCount: number, vendor: Vendor): string;
 export declare function resolveFixVendor(stepReviewer: string, origin: PROrigin, config: Config, fallback?: 'claude' | 'codex'): {
     vendor: 'claude' | 'codex' | null;
     usedHumanFallback: boolean;
+    substitutedOriginVendor?: 'claude' | 'codex';
+};
+export declare function resolveConflictResolveVendor(stepReviewer: string, origin: PROrigin, config: Config, fallback?: 'claude' | 'codex'): {
+    vendor: 'claude' | 'codex' | null;
+    usedHumanFallback: boolean;
+    substitutedOriginVendor?: 'claude' | 'codex';
 };
 /**
  * Builds the input the review strategy classifies on, from the already-cloned

package/dist/lib/runner.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,MAAM,EAAoB,MAAM,qBAAqB,CAAA;AAEnE,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,sBAAsB,CAAA;AACnD,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAA;AACrD,OAAO,EAAqB,KAAK,MAAM,EAAE,MAAM,kBAAkB,CAAA;AAQjE,OAAO,EAAiC,KAAK,kBAAkB,EAAE,MAAM,uBAAuB,CAAA;AAQ9F,OAAO,EAAuE,KAAK,SAAS,EAAE,KAAK,gBAAgB,EAAE,MAAM,sBAAsB,CAAA;AAIjJ,OAAO,EAAuE,KAAK,UAAU,EAAE,MAAM,oBAAoB,CAAA;AACzH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,iBAAiB,CAAA;AAsB9C,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAGzD;AAaD,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,YAAY,EAAE,OAAO,GAAG,MAAM,CAEpF;AAWD,MAAM,WAAW,qBAAqB;IACpC,KAAK,EAAE,MAAM,CAAA;IACb;;;;OAIG;IACH,MAAM,EAAE,OAAO,CAAA;CAChB;AAED,wBAAgB,mCAAmC,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,qBAAqB,CAqB1G;AAED,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAEnF;AAOD,MAAM,MAAM,eAAe,GAAG,KAAK,GAAG,SAAS,GAAG,OAAO,GAAG,OAAO,GAAG,WAAW,GAAG,SAAS,CAAA;AAE7F,MAAM,WAAW,sBAAsB;IACrC,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,MAAM,CAAA;IAChB,QAAQ,EAAE,MAAM,CAAA;IAChB,UAAU,EAAE,MAAM,CAAA;IAClB,aAAa,EAAE,MAAM,CAAA;IACrB,QAAQ,EAAE,MAAM,EAAE,CAAA;IAClB,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAA;IACnC,cAAc,EAAE,OAAO,CAAA;IACvB,KAAK,CAAC,EAAE,MAAM,CAAA;IACd,OAAO,CAAC,EAAE,eAAe,CAAA;IACzB,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,aAAa,CAAC,EAAE,OAAO,CAAA;IACvB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,GAAG,CAAC,EAAE,MAAM,CAAA;CACb;AAOD,MAAM,MAAM,mBAAmB,GAC3B,WAAW,GACX,OAAO,GACP,eAAe,GACf,gBAAgB,GAChB,cAAc,CAAA;AAElB,wBAAgB,0BAA0B,CACxC,MAAM,EAAE,sBAAsB,GAC7B,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAsCzB;AAKD,wBAAgB,gBAAgB,CAC9B,aAAa,EAAE,MAAM,EACrB,gBAAgB,EAAE,MAAM,EACxB,SAAS,EAAE,MAAM,EACjB,KAAK,EAAE,MAAM,GAAG,SAAS,GACxB,OAAO,CAOT;AAKD,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,GAAG,OAAO,CAE1E;AAED,MAAM,MAAM,eAAe,GAAG,cAAc,GAAG,QAAQ,GAAG,SAAS,CAAA;AAanE,MAAM,MAAM,UAAU,GAAG,QAAQ,GAAG,yBAAyB,GAAG,SAAS,CAAA;AACzE,wBAAgB,iBAAiB,CAAC,YAAY,EAAE,eAAe,GAAG,UAAU,CAM3E;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,CAAC,EAAE,OAAO,CAAA;IACf,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;IACvB,YAAY,CAAC,EAAE,MAAM,CAAA;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;IAC9B,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,aAAa,CAAC,EAAE,MAAM,CAAA;IACtB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,eAAe,CAAC,EAAE,MAAM,CAAA;IACxB,WAAW,CAAC,EAAE,MAAM,CAAA;CACrB;AAED,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,MAAM,CAAA;IAChB,QAAQ,EAAE,MAAM,CAAA;IAChB,EAAE,EAAE,OAAO,CAAC,cAAc,CAAC,CAAA;IAC3B,MAAM,EAAE,MAAM,CAAA;IACd,KAAK,EAAE,MAAM,CAAA;IACb,MAAM,EAAE,MAAM,CAAA;IACd,MAAM,EAAE,QAAQ,CAAA;IAChB,WAAW,EAAE,MAAM,CAAA;IACnB,GAAG,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAA;IAC1B,aAAa,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,WAAW,KAAK,IAAI,CAAA;IAE1D,cAAc,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAG3B,YAAY,CAAC,EAAE,OAAO,CAAA;IAEtB,KAAK,CAAC,EAAE,MAAM,CAAA;IAGd,MAAM,CAAC,EAAE,OAAO,CAAA;IAGhB,UAAU,CAAC,EAAE,kBAAkB,GAAG,IAAI,CAAA;IAGtC,KAAK,CAAC,EAAE,OAAO,eAAe,EAAE,YAAY,EAAE,CAAA;IAG9C,mBAAmB,CAAC,EAAE,QAAQ,GAAG,OAAO,CAAA;IAMxC,UAAU,CAAC,EAAE,MAAM,EAAE,CAAA;IAIrB,oBAAoB,CAAC,EAAE;QACrB,EAAE,CAAC,EAAE,MAAM,CAAA;QACX,IAAI,EAAE,MAAM,CAAA;KACb,CAAA;IAID,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAG1B,SAAS,CAAC,EAAE,OAAO,GAAG,WAAW,CAAA;IAGjC,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAE1B,OAAO,CAAC,EAAE,eAAe,CAAA;IAIzB,YAAY,CAAC,EAAE,MAAM,CAAA;IAIrB,aAAa,CAAC,EAAE,CAAC,YAAY,EAAE,MAAM,EAAE,cAAc,EAAE,MAAM,GAAG,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAA;CAChH;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IACtB;uEACmE;IACnE,eAAe,CAAC,EAAE,MAAM,CAAA;IAGxB,eAAe,CAAC,EAAE,MAAM,CAAA;IAIxB,aAAa,CAAC,EAAE,MAAM,CAAA;IAItB,mBAAmB,CAAC,EAAE;QACpB,EAAE,CAAC,EAAE,MAAM,CAAA;QACX,IAAI,EAAE,MAAM,CAAA;KACb,CAAA;CACF;AAkDD,wBAAgB,gBAAgB,CAAC,YAAY,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE7E;AAED,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE3E;AAED,wBAAgB,4BAA4B,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE1F;AAUD,wBAAgB,gBAAgB,CAC9B,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,QAAQ,EAChB,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,QAAQ,GAAG,OAAO,GAC5B;IAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,GAAG,IAAI,CAAC;IAAC,iBAAiB,EAAE,OAAO,CAAA;CAAE,CAenE;AAgCD;;;;;;;GAOG;AACH,wBAAgB,cAAc,CAAC,GAAG,EAAE,eAAe,GAAG,SAAS,GAAG,IAAI,CAsCrE;AAED;;;;;;;GAOG;AACH,wBAAgB,eAAe,CAC7B,OAAO,EAAE,eAAe,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,EAC7C,QAAQ,EAAE,gBAAgB,GAAG,IAAI,GAChC,eAAe,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,CAGtC;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,cAAc,CAAC,CAAC,SAAS;IAAE,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,EAC1D,MAAM,EAAE,CAAC,EACT,QAAQ,EAAE,gBAAgB,GAAG,IAAI,EACjC,QAAQ,EAAE,SAAS,MAAM,EAAE,GAC1B,CAAC,CAKH;AASD;;;;;;;;;;;GAWG;AACH,wBAAgB,uBAAuB,CACrC,MAAM,EAAE;IAAE,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,EACjC,QAAQ,EAAE,gBAAgB,GAAG,IAAI,EACjC,aAAa,CAAC,EAAE,MAAM,GACrB,OAAO,CAIT;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,GAAG,EAAE,eAAe,GAAG,gBAAgB,GAAG,IAAI,CAKlF;AAED,MAAM,WAAW,cAAc;IAC7B,oEAAoE;IACpE,QAAQ,EAAE,gBAAgB,GAAG,IAAI,CAAA;IACjC,OAAO,EAAE,MAAM,CAAC,SAAS,CAAC,CAAA;IAC1B,YAAY,EAAE,MAAM,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,CAAA;IACzC,WAAW,EAAE,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,CAAA;IACvC,iFAAiF;IACjF,WAAW,EAAE,MAAM,CAAA;IACnB,SAAS,EAAE,OAAO,CAAA;CACnB;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,MAAM,EACd,QAAQ,EAAE,gBAAgB,GAAG,IAAI,EACjC,KAAK,EAAE,MAAM,GACZ,cAAc,CA4ChB;AAuHD,wBAAsB,WAAW,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,cAAc,CAAC,CAkpC/E"}
+{"version":3,"file":"runner.d.ts","sourceRoot":"","sources":["../../src/lib/runner.ts"],"names":[],"mappings":"AAKA,OAAO,KAAK,EAAE,MAAM,EAAoB,MAAM,qBAAqB,CAAA;AAEnE,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,sBAAsB,CAAA;AACnD,OAAO,KAAK,EAAE,QAAQ,EAAE,MAAM,uBAAuB,CAAA;AACrD,OAAO,EAAqB,KAAK,MAAM,EAAE,MAAM,kBAAkB,CAAA;AAQjE,OAAO,EAAiC,KAAK,kBAAkB,EAAE,MAAM,uBAAuB,CAAA;AAQ9F,OAAO,EAAuE,KAAK,SAAS,EAAE,KAAK,gBAAgB,EAAE,MAAM,sBAAsB,CAAA;AAKjJ,OAAO,EAAuE,KAAK,UAAU,EAAE,MAAM,oBAAoB,CAAA;AACzH,OAAO,KAAK,EAAE,OAAO,EAAE,MAAM,iBAAiB,CAAA;AAsB9C,wBAAgB,mBAAmB,CAAC,GAAG,EAAE,OAAO,GAAG,OAAO,CAGzD;AAaD,wBAAgB,oBAAoB,CAAC,QAAQ,EAAE,MAAM,EAAE,YAAY,EAAE,OAAO,GAAG,MAAM,CAEpF;AAWD,MAAM,WAAW,qBAAqB;IACpC,KAAK,EAAE,MAAM,CAAA;IACb;;;;OAIG;IACH,MAAM,EAAE,OAAO,CAAA;CAChB;AAED,wBAAgB,mCAAmC,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,qBAAqB,CAqB1G;AAED,wBAAgB,2BAA2B,CAAC,MAAM,EAAE,MAAM,EAAE,OAAO,EAAE,MAAM,GAAG,MAAM,CAEnF;AAOD,MAAM,MAAM,eAAe,GAAG,KAAK,GAAG,SAAS,GAAG,OAAO,GAAG,OAAO,GAAG,WAAW,GAAG,SAAS,CAAA;AAE7F,MAAM,WAAW,sBAAsB;IACrC,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,MAAM,CAAA;IAChB,QAAQ,EAAE,MAAM,CAAA;IAChB,UAAU,EAAE,MAAM,CAAA;IAClB,aAAa,EAAE,MAAM,CAAA;IACrB,QAAQ,EAAE,MAAM,EAAE,CAAA;IAClB,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,CAAA;IACnC,cAAc,EAAE,OAAO,CAAA;IACvB,KAAK,CAAC,EAAE,MAAM,CAAA;IACd,OAAO,CAAC,EAAE,eAAe,CAAA;IACzB,WAAW,CAAC,EAAE,MAAM,CAAA;IACpB,aAAa,CAAC,EAAE,OAAO,CAAA;IACvB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,GAAG,CAAC,EAAE,MAAM,CAAA;CACb;AAOD,MAAM,MAAM,mBAAmB,GAC3B,WAAW,GACX,OAAO,GACP,eAAe,GACf,gBAAgB,GAChB,cAAc,CAAA;AAElB,wBAAgB,0BAA0B,CACxC,MAAM,EAAE,sBAAsB,GAC7B,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAsCzB;AAKD,wBAAgB,gBAAgB,CAC9B,aAAa,EAAE,MAAM,EACrB,gBAAgB,EAAE,MAAM,EACxB,SAAS,EAAE,MAAM,EACjB,KAAK,EAAE,MAAM,GAAG,SAAS,GACxB,OAAO,CAOT;AAKD,wBAAgB,aAAa,CAAC,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,GAAG,OAAO,CAE1E;AAED,MAAM,MAAM,eAAe,GAAG,cAAc,GAAG,QAAQ,GAAG,SAAS,CAAA;AAanE,MAAM,MAAM,UAAU,GAAG,QAAQ,GAAG,yBAAyB,GAAG,SAAS,CAAA;AACzE,wBAAgB,iBAAiB,CAAC,YAAY,EAAE,eAAe,GAAG,UAAU,CAM3E;AAED,MAAM,WAAW,WAAW;IAC1B,KAAK,CAAC,EAAE,OAAO,CAAA;IACf,OAAO,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;IACvB,YAAY,CAAC,EAAE,MAAM,CAAA;IACrB,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,cAAc,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;IAC9B,QAAQ,CAAC,EAAE,MAAM,CAAA;IACjB,aAAa,CAAC,EAAE,MAAM,CAAA;IACtB,SAAS,CAAC,EAAE,MAAM,CAAA;IAClB,UAAU,CAAC,EAAE,MAAM,CAAA;IACnB,eAAe,CAAC,EAAE,MAAM,CAAA;IACxB,WAAW,CAAC,EAAE,MAAM,CAAA;CACrB;AAED,MAAM,WAAW,eAAe;IAC9B,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,MAAM,CAAA;IAChB,QAAQ,EAAE,MAAM,CAAA;IAChB,EAAE,EAAE,OAAO,CAAC,cAAc,CAAC,CAAA;IAC3B,MAAM,EAAE,MAAM,CAAA;IACd,KAAK,EAAE,MAAM,CAAA;IACb,MAAM,EAAE,MAAM,CAAA;IACd,MAAM,EAAE,QAAQ,CAAA;IAChB,WAAW,EAAE,MAAM,CAAA;IACnB,GAAG,EAAE,CAAC,GAAG,EAAE,MAAM,KAAK,IAAI,CAAA;IAC1B,aAAa,EAAE,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,WAAW,KAAK,IAAI,CAAA;IAE1D,cAAc,EAAE,GAAG,CAAC,MAAM,CAAC,CAAA;IAG3B,YAAY,CAAC,EAAE,OAAO,CAAA;IAEtB,KAAK,CAAC,EAAE,MAAM,CAAA;IAGd,MAAM,CAAC,EAAE,OAAO,CAAA;IAGhB,UAAU,CAAC,EAAE,kBAAkB,GAAG,IAAI,CAAA;IAGtC,KAAK,CAAC,EAAE,OAAO,eAAe,EAAE,YAAY,EAAE,CAAA;IAG9C,mBAAmB,CAAC,EAAE,QAAQ,GAAG,OAAO,CAAA;IAMxC,UAAU,CAAC,EAAE,MAAM,EAAE,CAAA;IAIrB,oBAAoB,CAAC,EAAE;QACrB,EAAE,CAAC,EAAE,MAAM,CAAA;QACX,IAAI,EAAE,MAAM,CAAA;KACb,CAAA;IAID,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAG1B,SAAS,CAAC,EAAE,OAAO,GAAG,WAAW,CAAA;IAGjC,iBAAiB,CAAC,EAAE,MAAM,CAAA;IAE1B,OAAO,CAAC,EAAE,eAAe,CAAA;IAIzB,YAAY,CAAC,EAAE,MAAM,CAAA;IAIrB,aAAa,CAAC,EAAE,CAAC,YAAY,EAAE,MAAM,EAAE,cAAc,EAAE,MAAM,GAAG,IAAI,EAAE,MAAM,EAAE,MAAM,EAAE,QAAQ,EAAE,MAAM,KAAK,IAAI,CAAA;CAChH;AAED,MAAM,WAAW,cAAc;IAC7B,OAAO,EAAE,MAAM,GAAG,IAAI,CAAA;IACtB;uEACmE;IACnE,eAAe,CAAC,EAAE,MAAM,CAAA;IAGxB,eAAe,CAAC,EAAE,MAAM,CAAA;IAIxB,aAAa,CAAC,EAAE,MAAM,CAAA;IAItB,mBAAmB,CAAC,EAAE;QACpB,EAAE,CAAC,EAAE,MAAM,CAAA;QACX,IAAI,EAAE,MAAM,CAAA;KACb,CAAA;IACD;;;uDAGmD;IACnD,YAAY,CAAC,EAAE,YAAY,CAAA;CAC5B;AAED,MAAM,WAAW,YAAY;IAC3B;wEACoE;IACpE,GAAG,EAAE,MAAM,EAAE,CAAA;IACb;kCAC8B;IAC9B,OAAO,EAAE;QAAE,IAAI,EAAE,MAAM,CAAC;QAAC,MAAM,EAAE,MAAM,CAAA;KAAE,EAAE,CAAA;CAC5C;AAKD,wBAAgB,qBAAqB,CACnC,QAAQ,EAAE,SAAS,MAAM,EAAE,EAC3B,OAAO,EAAE,MAAM,CAAC,MAAM,EAAE,UAAU,CAAC,GAClC,YAAY,CAQd;AAWD,wBAAgB,iBAAiB,CAC/B,IAAI,EAAE,YAAY,GAAG,SAAS,EAC9B,IAAI,EAAE,YAAY,GAAG,SAAS,GAC7B,YAAY,GAAG,SAAS,CAY1B;AAkDD,wBAAgB,gBAAgB,CAAC,YAAY,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE7E;AAED,wBAAgB,kBAAkB,CAAC,QAAQ,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE3E;AAED,wBAAgB,4BAA4B,CAAC,aAAa,EAAE,MAAM,EAAE,MAAM,EAAE,MAAM,GAAG,MAAM,CAE1F;AA8DD,wBAAgB,gBAAgB,CAC9B,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,QAAQ,EAChB,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,QAAQ,GAAG,OAAO,GAC5B;IAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,GAAG,IAAI,CAAC;IAAC,iBAAiB,EAAE,OAAO,CAAC;IAAC,uBAAuB,CAAC,EAAE,QAAQ,GAAG,OAAO,CAAA;CAAE,CAEjH;AAKD,wBAAgB,4BAA4B,CAC1C,YAAY,EAAE,MAAM,EACpB,MAAM,EAAE,QAAQ,EAChB,MAAM,EAAE,MAAM,EACd,QAAQ,CAAC,EAAE,QAAQ,GAAG,OAAO,GAC5B;IAAE,MAAM,EAAE,QAAQ,GAAG,OAAO,GAAG,IAAI,CAAC;IAAC,iBAAiB,EAAE,OAAO,CAAC;IAAC,uBAAuB,CAAC,EAAE,QAAQ,GAAG,OAAO,CAAA;CAAE,CAEjH;AAgCD;;;;;;;GAOG;AACH,wBAAgB,cAAc,CAAC,GAAG,EAAE,eAAe,GAAG,SAAS,GAAG,IAAI,CAsCrE;AAED;;;;;;;GAOG;AACH,wBAAgB,eAAe,CAC7B,OAAO,EAAE,eAAe,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,EAC7C,QAAQ,EAAE,gBAAgB,GAAG,IAAI,GAChC,eAAe,CAAC,QAAQ,CAAC,CAAC,SAAS,CAAC,CAGtC;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,cAAc,CAAC,CAAC,SAAS;IAAE,MAAM,CAAC,EAAE,MAAM,CAAA;CAAE,EAC1D,MAAM,EAAE,CAAC,EACT,QAAQ,EAAE,gBAAgB,GAAG,IAAI,EACjC,QAAQ,EAAE,SAAS,MAAM,EAAE,GAC1B,CAAC,CAKH;AASD;;;;;;;;;;;GAWG;AACH,wBAAgB,uBAAuB,CACrC,MAAM,EAAE;IAAE,KAAK,CAAC,EAAE,MAAM,GAAG,IAAI,CAAA;CAAE,EACjC,QAAQ,EAAE,gBAAgB,GAAG,IAAI,EACjC,aAAa,CAAC,EAAE,MAAM,GACrB,OAAO,CAIT;AAED;;;GAGG;AACH,wBAAgB,oBAAoB,CAAC,GAAG,EAAE,eAAe,GAAG,gBAAgB,GAAG,IAAI,CAKlF;AAED,MAAM,WAAW,cAAc;IAC7B,oEAAoE;IACpE,QAAQ,EAAE,gBAAgB,GAAG,IAAI,CAAA;IACjC,OAAO,EAAE,MAAM,CAAC,SAAS,CAAC,CAAA;IAC1B,YAAY,EAAE,MAAM,CAAC,SAAS,CAAC,CAAC,QAAQ,CAAC,CAAA;IACzC,WAAW,EAAE,MAAM,CAAC,SAAS,CAAC,CAAC,OAAO,CAAC,CAAA;IACvC,iFAAiF;IACjF,WAAW,EAAE,MAAM,CAAA;IACnB,SAAS,EAAE,OAAO,CAAA;CACnB;AAED;;;;;;;;;;;;GAYG;AACH,wBAAgB,qBAAqB,CACnC,MAAM,EAAE,MAAM,EACd,QAAQ,EAAE,gBAAgB,GAAG,IAAI,EACjC,KAAK,EAAE,MAAM,GACZ,cAAc,CA4ChB;AAuHD,wBAAsB,WAAW,CAAC,GAAG,EAAE,eAAe,GAAG,OAAO,CAAC,cAAc,CAAC,CA2qC/E"}

package/dist/lib/runner.js

@@ -23,6 +23,7 @@ import { resolveClaudeModel, resolveCodexModel } from '../lib/review-models.js';
 import { resolveReviewStrategy, escalate, clampToLevels } from './review-strategy.js';
 import { CLAUDE_EFFORT_LEVELS, CODEX_EFFORT_LEVELS } from '../config/schema.js';
 import { buildStepIdentityFields } from '../lib/event-fields.js';
+import { prOpenToVerdictMs } from '../lib/adoption.js';
 import { buildAttributionFooter, buildFixAppliedCommentBody, buildFixFailedCommentBody, buildConflictResolvedCommentBody, buildRetriedReviewBanner } from '../lib/comment-bodies.js';
 import { linearWritePossible, loadWorkflow, loadHarnessSection, evaluateWhen } from '../lib/workflow.js';
 import { isSubscriptionLimitError, isVendorUnavailableError } from '../lib/smart-switch.js';
@@ -147,6 +148,45 @@ export function resolveFixLanding(deliveryMode) {
         case 'pull_request': return 'branch-then-separate-pr';
     }
 }
+// stepsRun holds every step the runner dispatched, skips included; results holds
+// what each one did. Split them so callers can report a run where nothing
+// happened without re-deriving the reasons from the log file.
+export function summariseStepOutcomes(stepsRun, results) {
+    const outcomes = { ran: [], skipped: [] };
+    for (const name of new Set(stepsRun)) {
+        const result = results[name];
+        if (result?.skipped)
+            outcomes.skipped.push({ step: name, reason: result.skipReason ?? 'unknown' });
+        else
+            outcomes.ran.push(name);
+    }
+    return outcomes;
+}
+// Accumulates outcomes across the fix→recheck rounds of one invocation, so the
+// completion line reports whether the run as a whole did work: a first round
+// that skips everything followed by a round that applies a fix is not a
+// "no step ran" run.
+//
+// Running once wins over skipping any number of times — the step demonstrably
+// happened. A step skipped in several rounds is reported once, carrying its
+// latest reason: that is the state the run ended in, and listing the same step
+// several times reads as several distinct problems.
+export function mergeStepOutcomes(base, next) {
+    if (!base)
+        return next;
+    if (!next)
+        return base;
+    const ran = [...new Set([...base.ran, ...next.ran])];
+    const ranSet = new Set(ran);
+    // Map.set on an existing key overwrites the reason but keeps the original
+    // position, so order stays first-seen while the reason stays last-seen.
+    const skipped = new Map();
+    for (const entry of [...base.skipped, ...next.skipped]) {
+        if (!ranSet.has(entry.step))
+            skipped.set(entry.step, entry.reason);
+    }
+    return { ran, skipped: [...skipped].map(([step, reason]) => ({ step, reason })) };
+}
 function countComments(reviewText) {
     const bullets = (reviewText.match(/^[-*•]\s/gm) ?? []).length;
     const numbered = (reviewText.match(/^\d+\.\s/gm) ?? []).length;
@@ -203,16 +243,38 @@ export function fixPRCommitSubject(prNumber, vendor) {
 export function conflictResolveCommitSubject(conflictCount, vendor) {
     return `[crosscheck] resolve: resolve ${conflictCount} conflict${conflictCount !== 1 ? 's' : ''} — by ${vendorDisplayName(vendor)}`;
 }
-// Extends resolveReviewer with a human-origin fallback for the fix step.
+// Extends resolveReviewer with a human-origin fallback for the steps that write
+// code (fix, conflict-resolve).
 // Scoped to reviewer: 'origin' only — other reviewer types (claude, codex, auto)
 // already encode explicit vendor intent and need no fallback.
 // When origin is 'human' and no vendor resolved, honours routing.fallback_reviewer
-// so the fix step respects the same routing intent as the review step.
-// 'auto' mirrors resolveReviewer's auto path (config-enabled check, codex-first)
-// without async auth calls. null disables the fallback entirely.
-// Exported so callers can detect when the fallback was applied (e.g. for logging).
-export function resolveFixVendor(stepReviewer, origin, config, fallback) {
+// so the step respects the same routing intent as the review step.
+// 'auto' resolves against the vendors that can actually run stepType, so a step
+// only one vendor supports doesn't fall back to the other and skip a line later.
+// An explicit 'claude'/'codex' is an operator decision and is honoured as written
+// even when that vendor cannot run the step — the caller then reports a precise
+// unsupported-step skip instead of silently substituting a different vendor.
+// null disables the fallback entirely.
+function resolveStepVendor(stepType, stepReviewer, origin, config, fallback) {
     const vendor = resolveReviewer(stepReviewer, origin, config, fallback);
+    // Origin detection can assign a vendor that cannot run the step: conflict
+    // resolution is Claude-only, so a Codex-origin PR (reviewer: 'origin', origin:
+    // 'codex') resolves to 'codex' and the dispatch skips it as unsupported — the
+    // conflicts never get resolved. (#284's `Crosscheck-Reviewer: codex` detection
+    // introduced this: these crosscheck-authored fix PRs used to detect as 'human'
+    // and the auto fallback picked Claude.) Substitute a capable, enabled vendor.
+    // Scoped to origin-derived assignment only — an explicit reviewer: claude|codex,
+    // reviewer: auto, or routing.fallback_reviewer is an operator decision, left as
+    // written so the caller can report the precise unsupported-step skip.
+    if (stepReviewer === 'origin' &&
+        (origin === 'claude' || origin === 'codex') &&
+        vendor !== null &&
+        !supportsStep(vendor, stepType)) {
+        const capable = vendor === 'claude' ? 'codex' : 'claude';
+        if (config.vendors[capable].enabled && supportsStep(capable, stepType)) {
+            return { vendor: capable, usedHumanFallback: false, substitutedOriginVendor: vendor };
+        }
+    }
     if (vendor !== null || origin !== 'human' || stepReviewer !== 'origin') {
         return { vendor, usedHumanFallback: false };
     }
@@ -223,13 +285,25 @@ export function resolveFixVendor(stepReviewer, origin, config, fallback) {
     else if (fb === 'codex')
         humanFallback = config.vendors.codex.enabled ? 'codex' : null;
     else if (fb !== null) {
-        // 'auto': prefer codex then claude, same as resolveReviewer's auto path
-        humanFallback = config.vendors.codex.enabled ? 'codex' : config.vendors.claude.enabled ? 'claude' : null;
+        // 'auto': prefer codex then claude, same as resolveReviewer's auto path,
+        // narrowed to vendors that support this step type.
+        const usable = (v) => config.vendors[v].enabled && supportsStep(v, stepType);
+        humanFallback = usable('codex') ? 'codex' : usable('claude') ? 'claude' : null;
     }
     if (!humanFallback)
         return { vendor: null, usedHumanFallback: false };
     return { vendor: humanFallback, usedHumanFallback: true };
 }
+// Exported so callers can detect when the fallback was applied (e.g. for logging).
+export function resolveFixVendor(stepReviewer, origin, config, fallback) {
+    return resolveStepVendor('fix', stepReviewer, origin, config, fallback);
+}
+// The default workflow gives conflict-resolve `reviewer: origin`, so every PR
+// crosscheck cannot attribute resolved to null here and skipped with 'no_vendor'
+// — the fix step honoured routing.fallback_reviewer, this one did not.
+export function resolveConflictResolveVendor(stepReviewer, origin, config, fallback) {
+    return resolveStepVendor('conflict-resolve', stepReviewer, origin, config, fallback);
+}
 // ─── pr_complexity helpers ────────────────────────────────────────────────────
 const EXT_LANG = {
     ts: 'typescript', tsx: 'typescript', js: 'javascript', jsx: 'javascript',
@@ -717,7 +791,7 @@ export async function runWorkflow(ctx) {
             const effectiveType = getEffectiveStepType(step.type, ctx.isRecheckRun === true);
             if (exceedsMaxRounds(effectiveType, step.type, ctx.overrideMaxRounds ?? step.max_rounds, ctx.round)) {
                 fileLog({ level: 'info', event: 'step_skipped', repo: `${owner}/${repoName}`, pr: prNumber, step: step.name, reason: 'max_rounds' });
-                results[step.name] = { skipped: true };
+                results[step.name] = { skipped: true, skipReason: 'max_rounds' };
                 if (effectiveType === 'fix')
                     onPhaseChange('', { phase: 'fixed', fixCount: 0 });
                 else if (effectiveType === 'recheck')
@@ -729,7 +803,7 @@ export async function runWorkflow(ctx) {
             // Evaluate when condition — skip step if false
             if (step.when && !evaluateWhen(step.when, results)) {
                 fileLog({ level: 'info', event: 'step_skipped', repo: `${owner}/${repoName}`, pr: prNumber, step: step.name, reason: 'when_condition' });
-                results[step.name] = { skipped: true };
+                results[step.name] = { skipped: true, skipReason: 'when_condition' };
                 if (effectiveType === 'fix')
                     onPhaseChange('', { phase: 'fixed', fixCount: 0 });
                 else if (effectiveType === 'recheck')
@@ -743,7 +817,7 @@ export async function runWorkflow(ctx) {
             // review coerced to a recheck (isRecheckRun) is never blocked here.
             if (step.type === 'recheck' && reviewRanThisSession && !anyFixApplied(results)) {
                 fileLog({ level: 'info', event: 'step_skipped', repo: `${owner}/${repoName}`, pr: prNumber, step: step.name, reason: 'no_change_since_review' });
-                results[step.name] = { skipped: true };
+                results[step.name] = { skipped: true, skipReason: 'no_change_since_review' };
                 onPhaseChange('', { phase: 'rechecked' });
                 continue;
             }
@@ -752,7 +826,7 @@ export async function runWorkflow(ctx) {
                 let reviewer = resolveReviewer(step.reviewer, origin, config, ctx.smartSwitchFallback);
                 if (!reviewer) {
                     fileLog({ level: 'info', event: 'step_skipped', repo: `${owner}/${repoName}`, pr: prNumber, step: step.name, reason: 'no_reviewer' });
-                    results[step.name] = { skipped: true };
+                    results[step.name] = { skipped: true, skipReason: 'no_reviewer' };
                     continue;
                 }
                 // The recheck step is confirmed to run — clear the pending-recheck guard
@@ -876,7 +950,12 @@ export async function runWorkflow(ctx) {
                     ? `${buildRetriedReviewBanner(retried.timeoutMs, retried.delayMs)}\n\n${baseBody}`
                     : baseBody;
                 const commentCount = countComments(rawReview);
-                fileLog({ level: 'info', event: 'review_complete', repo: `${owner}/${repoName}`, pr: prNumber, reviewer, model, ...stepIdentity, verdict, duration_ms: Date.now() - stepStart, tokens_used: tokensUsed, skills_activated: activatedSkills.map(skill => skill.name), ...(inputTokens !== undefined && { input_tokens: inputTokens }), ...(outputTokens !== undefined && { output_tokens: outputTokens }), ...(ctx.round !== undefined && { round: ctx.round }), ...(ctx.roundMode && { mode: ctx.roundMode }), ...triggerField });
+                // How long the PR waited for a verdict, measured from when its author opened
+                // it — the number a team feels, as distinct from duration_ms (how long the
+                // reviewer ran). Omitted rather than guessed when the PR event carried no
+                // created_at, so the metric never mixes real latencies with invented ones.
+                const openToVerdictMs = prOpenToVerdictMs(pr.created_at, verdict);
+                fileLog({ level: 'info', event: 'review_complete', repo: `${owner}/${repoName}`, pr: prNumber, reviewer, model, ...stepIdentity, verdict, duration_ms: Date.now() - stepStart, ...(openToVerdictMs !== undefined && { open_to_verdict_ms: openToVerdictMs }), tokens_used: tokensUsed, skills_activated: activatedSkills.map(skill => skill.name), ...(inputTokens !== undefined && { input_tokens: inputTokens }), ...(outputTokens !== undefined && { output_tokens: outputTokens }), ...(ctx.round !== undefined && { round: ctx.round }), ...(ctx.roundMode && { mode: ctx.roundMode }), ...triggerField });
                 // Recheck verdict is stored separately to preserve the original review's commentCount on the board
                 const phaseUpdate = isRecheck
                     ? { recheckVerdict: verdict, phase: donePhase, recheckTokens: tokensUsed, recheckReviewer: reviewer, qualityTier: quality.tier }
@@ -946,6 +1025,16 @@ export async function runWorkflow(ctx) {
                         : undefined);
                     const commentUrl = `github.com/${owner}/${repoName}/pull/${prNumber}`;
                     fileLog({ level: 'info', event: 'comment_posted', repo: `${owner}/${repoName}`, pr: prNumber, url: `https://${commentUrl}` });
+                    // A posted verdict that blocks the merge is the product's whole reason to
+                    // exist, so it gets its own event rather than being re-derived downstream.
+                    // BLOCK blocks by definition; a NEEDS WORK that reached here survived the
+                    // severity gate, which only lets it through when a Critical/High/Medium
+                    // finding backs it — a nit-only review was already downgraded to APPROVE.
+                    // Logged only here, after the comment actually posted, so dry runs and
+                    // failed postReviewComment calls are never counted as posted findings.
+                    if (verdict === 'BLOCK' || verdict === 'NEEDS WORK') {
+                        fileLog({ level: 'info', event: 'blocking_finding_posted', repo: `${owner}/${repoName}`, pr: prNumber, reviewer, model, ...stepIdentity, verdict, ...(ctx.round !== undefined && { round: ctx.round }), ...triggerField });
+                    }
                     // Mirror the verdict onto the PR's Linear issue. `run` and `watch` both
                     // land here, so this is the path that matters — reviews posted from
                     // commands/review.ts are the exception, not the rule.
@@ -985,7 +1074,7 @@ export async function runWorkflow(ctx) {
                 const skipFix = (reason) => {
                     lastFixSkipReason = reason;
                     onPhaseChange('', { phase: 'fixed', fixCount: 0 });
-                    results[step.name] = { skipped: true };
+                    results[step.name] = { skipped: true, skipReason: reason };
                     fileLog({ level: 'info', event: 'step_skipped', repo: `${owner}/${repoName}`, pr: prNumber, step: step.name, reason });
                 };
                 if (ctx.dryRun) {
@@ -1380,7 +1469,7 @@ export async function runWorkflow(ctx) {
             else if (effectiveType === 'conflict-resolve') {
                 const skipConflictResolve = (reason) => {
                     onPhaseChange('', { phase: 'fixed', fixCount: 0 });
-                    results[step.name] = { skipped: true };
+                    results[step.name] = { skipped: true, skipReason: reason };
                     fileLog({ level: 'info', event: 'step_skipped', repo: `${owner}/${repoName}`, pr: prNumber, step: step.name, reason });
                 };
                 if (ctx.dryRun) {
@@ -1427,7 +1516,16 @@ export async function runWorkflow(ctx) {
                     skipConflictResolve('no_conflicts');
                     continue;
                 }
-                const vendor = resolveReviewer(step.reviewer, origin, config, ctx.smartSwitchFallback);
+                // resolveConflictResolveVendor extends resolveReviewer with the same human-origin
+                // fallback the fix step uses, so a PR crosscheck cannot attribute still gets its
+                // conflicts resolved instead of skipping with 'no_vendor'.
+                const { vendor, usedHumanFallback, substitutedOriginVendor } = resolveConflictResolveVendor(step.reviewer, origin, config, ctx.smartSwitchFallback);
+                if (usedHumanFallback && vendor) {
+                    fileLog({ level: 'info', event: 'conflict_resolve_vendor_fallback', repo: `${owner}/${repoName}`, pr: prNumber, from: 'none', to: vendor, reason: 'human_origin' });
+                }
+                else if (substitutedOriginVendor && vendor) {
+                    fileLog({ level: 'info', event: 'conflict_resolve_vendor_fallback', repo: `${owner}/${repoName}`, pr: prNumber, from: substitutedOriginVendor, to: vendor, reason: 'unsupported_vendor' });
+                }
                 if (!vendor) {
                     try {
                         execSync('git merge --abort', { cwd: tmpDir });
@@ -1633,6 +1731,7 @@ export async function runWorkflow(ctx) {
         return {
             verdict: verdict ?? null,
             fixAppliedCount,
+            stepOutcomes: summariseStepOutcomes(stepsRun, results),
             ...(fixAppliedCount === undefined && lastFixSkipReason !== undefined && { fixSkipReason: lastFixSkipReason }),
             ...(latestReviewResult?.commentBody && {
                 latestReviewComment: {