@humanbased/crosscheck 1.3.1-beta.7 → 1.6.0-beta.11

package/dist/lib/no-verdict.js

@@ -0,0 +1,262 @@
+import { fmtTokensRaw } from './board.js';
+import { shaCovers } from './pr-workflow-state.js';
+/**
+ * The verdict that still governs the PR: the newest review or recheck that
+ * actually carries one.
+ *
+ * A review can run, post its findings, and record no verdict at all — that is
+ * the `ran_no_verdict` disposition, and it says nothing about the `BLOCK` that
+ * was standing before it. Taking the newest review-or-recheck record
+ * unconditionally would let one malformed review erase a verdict that still
+ * gates the PR, and the report would go quiet about it in exactly the state
+ * that most needs saying out loud.
+ */
+export function selectStandingVerdict(history) {
+    const judged = history.filter(r => isVerdictStep(r.type) && r.verdict).at(-1);
+    if (!judged?.verdict)
+        return undefined;
+    return { verdict: judged.verdict, ...(judged.sha !== undefined && { sha: judged.sha }) };
+}
+function isVerdictStep(type) {
+    return type === 'review' || type === 'recheck';
+}
+/**
+ * Classifies every verdict-capable step in the workflow. When none is
+ * configured, returns a single synthetic `review` entry — the absence is itself
+ * the answer, and callers should not have to special-case an empty array.
+ */
+export function describeVerdictSteps(workflowSteps, outcomes) {
+    const steps = workflowSteps.filter(s => isVerdictStep(s.type));
+    if (steps.length === 0)
+        return [{ step: 'review', disposition: 'not_configured' }];
+    const ran = new Set(outcomes?.ran ?? []);
+    const skipped = new Map((outcomes?.skipped ?? []).map(s => [s.step, s.reason]));
+    return steps.map(({ name }) => {
+        if (ran.has(name)) {
+            // `verdict: null` means the step ran and its output had no parseable
+            // verdict. A missing key means the step recorded no verdict field at all,
+            // which for a dispatched verdict step means it ran normally.
+            const detail = outcomes?.ranDetail?.[name];
+            return detail && 'verdict' in detail && detail.verdict === null
+                ? { step: name, disposition: 'ran_no_verdict' }
+                : { step: name, disposition: 'ran' };
+        }
+        const reason = skipped.get(name);
+        if (reason !== undefined)
+            return { step: name, disposition: 'skipped', reason };
+        return { step: name, disposition: 'not_dispatched' };
+    });
+}
+// Guards that stop the run deliberately. Re-running the same step walks into the
+// same cap, so these get an explanation instead of a command.
+const DELIBERATE_STOPS = new Set(['max_rounds', 'commit_limit_reached', 'vendor_limit']);
+// Reasons that mean no vendor was available to do the work — a config gap, not
+// a per-run accident.
+const ROUTING_GAPS = new Set(['no_vendor', 'no_reviewer', 'unsupported_vendor', 'human_origin']);
+function describeSkipReason(reason, step, prUrl) {
+    if (ROUTING_GAPS.has(reason)) {
+        return {
+            next: [
+                { text: 'set routing.fallback_reviewer in ~/.crosscheck/config.yml', recommended: true },
+                { text: `ck run ${prUrl} --reviewer <claude|codex>`, recommended: false },
+            ],
+            notes: ['No vendor was available to judge this PR — nothing about the code caused this.'],
+        };
+    }
+    if (DELIBERATE_STOPS.has(reason)) {
+        return {
+            next: [],
+            notes: [`${step} stopped on \`${reason}\` — crosscheck gave up on purpose, so this needs a human.`],
+        };
+    }
+    if (reason === 'no_change_since_review') {
+        return {
+            next: [{ text: `ck run ${prUrl} --steps review`, recommended: false }],
+            notes: ['Nothing has changed since the last review, so there was nothing new to judge.'],
+        };
+    }
+    return { next: [{ text: `ck run ${prUrl} --steps ${step}`, recommended: true }], notes: [] };
+}
+function shortSha(sha) {
+    return sha === undefined ? undefined : sha.slice(0, 7);
+}
+function formatRanDetail(detail) {
+    const parts = [];
+    if (detail.vendor)
+        parts.push(detail.vendor);
+    if (detail.tokensUsed !== undefined)
+        parts.push(`${fmtTokensRaw(detail.tokensUsed)} tokens`);
+    if (detail.appliedCount === 0)
+        parts.push('no changes applied');
+    else if (detail.appliedCount !== undefined)
+        parts.push(`${detail.appliedCount} change${detail.appliedCount === 1 ? '' : 's'} applied`);
+    if (detail.verdict === null)
+        parts.push('no verdict parsed');
+    return parts.join(' · ');
+}
+/**
+ * Explains an empty verdict and says what to do about it. Pure — every input it
+ * needs is already in memory at the reporting site.
+ */
+export function buildNoVerdictReport(input) {
+    const { workflowSteps, outcomes, prUrl } = input;
+    const verdictSteps = describeVerdictSteps(workflowSteps, outcomes);
+    const ran = (outcomes?.ran ?? []).map(step => ({
+        step,
+        detail: outcomes?.ranDetail?.[step] ? formatRanDetail(outcomes.ranDetail[step]) : '',
+    }));
+    const why = [];
+    let next = [];
+    const notes = [];
+    // Precedence: a step that ran and produced nothing outranks one that skipped,
+    // which outranks one that never ran. The most specific thing that happened is
+    // the thing worth reporting.
+    // Fix steps by type: conflict-resolve records `applied_count: 0` too, and a
+    // branch with no conflicts to resolve says nothing about a review's findings.
+    const fixStepNames = new Set(workflowSteps.filter(s => s.type === 'fix').map(s => s.name));
+    const noopFixStep = (outcomes?.ran ?? []).find(step => fixStepNames.has(step) && outcomes?.ranDetail?.[step]?.appliedCount === 0);
+    // True when every step this run actually ran was a fix that changed nothing —
+    // i.e. the run's whole output was nothing at all.
+    const scopedFixDidNothing = noopFixStep !== undefined
+        && (outcomes?.ran ?? []).every(step => outcomes?.ranDetail?.[step]?.appliedCount === 0)
+        ? noopFixStep
+        : undefined;
+    const ranNoVerdict = verdictSteps.find(v => v.disposition === 'ran_no_verdict');
+    const skippedStep = verdictSteps.find(v => v.disposition === 'skipped');
+    const allNotConfigured = verdictSteps.every(v => v.disposition === 'not_configured');
+    const allNotDispatched = verdictSteps.every(v => v.disposition === 'not_dispatched');
+    let cause;
+    let expected;
+    if (input.strategySkipped) {
+        cause = 'strategy_skip';
+        expected = true;
+        why.push(`PR classified ${input.strategySkipped} — the strategy skipped it, so no step produced a verdict.`);
+    }
+    else if (allNotConfigured) {
+        cause = 'not_configured';
+        expected = true;
+        why.push('This workflow has no review or recheck step, so it produces no verdict by design.');
+    }
+    else if (ranNoVerdict) {
+        cause = 'ran_no_verdict';
+        expected = false;
+        why.push(`${ranNoVerdict.step} ran and posted, but its output carried no parseable VERDICT: line.`);
+        why.push('The findings are in the comment; the workflow has no verdict to gate on.');
+        next = [{ text: `ck run ${prUrl} --steps ${ranNoVerdict.step}`, recommended: true }];
+        notes.push('Malformed vendor output is usually transient — a re-run normally settles it.');
+    }
+    else if (skippedStep) {
+        cause = 'skipped';
+        expected = false;
+        why.push('Only review and recheck produce verdicts, and neither judged this commit.');
+        const advice = describeSkipReason(skippedStep.reason ?? 'unknown', skippedStep.step, prUrl);
+        next = advice.next;
+        notes.push(...advice.notes);
+    }
+    else if (scopedFixDidNothing) {
+        // `ck fix` asks for exactly one step, so a missing verdict is expected and the
+        // run would otherwise close green — the same green a fix that rewrote ten files
+        // prints. The absent verdict is indeed expected; the absent *change* is not,
+        // and it is the only thing the invocation was for.
+        cause = 'fix_noop';
+        expected = false;
+        why.push(`This run was scoped to ${scopedFixDidNothing}, which ran and applied no changes.`);
+        why.push('Nothing was written to the PR — the review it read may carry no actionable findings, or they may already be resolved in HEAD.');
+        next = [{ text: `ck run ${prUrl} --steps review`, recommended: true }];
+        notes.push('A review that could not resolve the diff produces a findings-free body that no fix can act on — check the review comment the fix step read.');
+    }
+    else {
+        cause = 'not_dispatched';
+        // A run that asked for one step got what it asked for. An unscoped run that
+        // dispatched no verdict step did not.
+        expected = allNotDispatched && input.stepsExplicitlyScoped === true;
+        why.push(expected
+            ? 'This run was scoped to steps that do not produce verdicts.'
+            : 'Only review and recheck produce verdicts, and neither was dispatched.');
+        if (!expected) {
+            next = [{ text: `ck run ${prUrl} --steps review`, recommended: true }];
+        }
+    }
+    // Context that holds regardless of cause: what the PR still says, and whether
+    // it still describes the code under review.
+    if (!expected && input.standingVerdict?.verdict) {
+        const sha = shortSha(input.standingVerdict.sha);
+        const head = shortSha(input.headSha);
+        // shaCovers is the codebase's one definition of "the same commit" — it
+        // handles the short/full forms annotations carry either of. A verdict whose
+        // SHA is unknown proves nothing about HEAD, so it is reported without the
+        // staleness claim rather than guessed at.
+        const covers = input.headSha !== undefined && shaCovers(input.standingVerdict.sha, input.headSha);
+        why.push(sha !== undefined && head !== undefined && !covers
+            ? `The PR's standing verdict is ${input.standingVerdict.verdict} on ${sha}, which is not HEAD (${head}).`
+            : `The PR's standing verdict is still ${input.standingVerdict.verdict}${sha ? ` on ${sha}` : ''}.`);
+    }
+    // A fix that applied nothing is the strongest hint available that the findings
+    // are already resolved — which is precisely what a recheck confirms. Not repeated
+    // under `fix_noop`, where it is already the headline.
+    if (!expected && noopFixStep && cause !== 'ran_no_verdict' && cause !== 'fix_noop') {
+        why.push(`The ${noopFixStep} step applied no changes — the findings may already be resolved in HEAD.`);
+    }
+    // Offer the other verdict step as an alternative: recheck judges against the
+    // original findings, review sets them aside. Not for malformed output, where
+    // the action is to re-run the step that misbehaved and a second command only
+    // blurs that.
+    const recommendedStep = next.find(n => n.recommended)?.text.match(/--steps (\S+)/)?.[1];
+    const alternative = verdictSteps.find(v => v.step !== recommendedStep && v.disposition !== 'not_configured');
+    if (cause !== 'ran_no_verdict' && recommendedStep && alternative
+        && !next.some(n => n.text.includes(`--steps ${alternative.step}`))) {
+        next.push({ text: `ck run ${prUrl} --steps ${alternative.step}`, recommended: false });
+    }
+    // Skipped steps of any type bear on the outcome, so all of them are listed. A
+    // step that was never dispatched earns a line only when it produces verdicts
+    // AND its absence is the problem: a conflict-resolve that sat out explains
+    // nothing about a missing verdict, and on a run that was never going to be
+    // judged, `why` has already said so in a sentence.
+    const order = new Map(workflowSteps.map((s, i) => [s.name, i]));
+    const didNotRun = [
+        ...(outcomes?.skipped ?? []).map(s => ({ step: s.step, note: `skipped — ${s.reason}` })),
+        ...(expected ? [] : verdictSteps
+            .filter(v => v.disposition === 'not_dispatched')
+            .map(v => ({ step: v.step, note: 'not dispatched' }))),
+    ].sort((a, b) => (order.get(a.step) ?? Infinity) - (order.get(b.step) ?? Infinity));
+    return { expected, cause, verdictSteps, ran, didNotRun, why, next, notes };
+}
+function pad(entries) {
+    return entries.reduce((w, e) => Math.max(w, e.step.length), 0);
+}
+/** Renders the report as plain lines, unindented and uncoloured. The caller
+ *  owns presentation; this owns what is said and in what order. */
+export function renderNoVerdictReport(report) {
+    const lines = ['no verdict this run'];
+    const width = Math.max(pad(report.ran), pad(report.didNotRun));
+    if (report.ran.length > 0) {
+        lines.push('', 'ran');
+        for (const { step, detail } of report.ran) {
+            lines.push(detail ? `  ${step.padEnd(width)}  ${detail}` : `  ${step}`);
+        }
+    }
+    if (report.didNotRun.length > 0) {
+        lines.push('', 'did not run');
+        for (const { step, note } of report.didNotRun) {
+            lines.push(`  ${step.padEnd(width)}  ${note}`);
+        }
+    }
+    if (report.why.length > 0) {
+        lines.push('', 'why');
+        for (const line of report.why)
+            lines.push(`  ${line}`);
+    }
+    if (report.next.length > 0) {
+        lines.push('', 'next');
+        for (const { text, recommended } of report.next) {
+            lines.push(recommended ? `→ ${text}` : `  ${text}`);
+        }
+    }
+    if (report.notes.length > 0) {
+        lines.push('');
+        for (const note of report.notes)
+            lines.push(`  ${note}`);
+    }
+    return lines;
+}
+//# sourceMappingURL=no-verdict.js.map

package/dist/lib/no-verdict.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"no-verdict.js","sourceRoot":"","sources":["../../src/lib/no-verdict.ts"],"names":[],"mappings":"AACA,OAAO,EAAE,YAAY,EAAE,MAAM,YAAY,CAAA;AACzC,OAAO,EAAE,SAAS,EAAE,MAAM,wBAAwB,CAAA;AAwDlD;;;;;;;;;;GAUG;AACH,MAAM,UAAU,qBAAqB,CACnC,OAAqC;IAErC,MAAM,MAAM,GAAG,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,OAAO,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,CAAA;IAC7E,IAAI,CAAC,MAAM,EAAE,OAAO;QAAE,OAAO,SAAS,CAAA;IACtC,OAAO,EAAE,OAAO,EAAE,MAAM,CAAC,OAAO,EAAE,GAAG,CAAC,MAAM,CAAC,GAAG,KAAK,SAAS,IAAI,EAAE,GAAG,EAAE,MAAM,CAAC,GAAG,EAAE,CAAC,EAAE,CAAA;AAC1F,CAAC;AAiCD,SAAS,aAAa,CAAC,IAAY;IACjC,OAAO,IAAI,KAAK,QAAQ,IAAI,IAAI,KAAK,SAAS,CAAA;AAChD,CAAC;AAED;;;;GAIG;AACH,MAAM,UAAU,oBAAoB,CAClC,aAA0C,EAC1C,QAAkC;IAElC,MAAM,KAAK,GAAG,aAAa,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,aAAa,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAA;IAC9D,IAAI,KAAK,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,CAAC,EAAE,IAAI,EAAE,QAAQ,EAAE,WAAW,EAAE,gBAAgB,EAAE,CAAC,CAAA;IAElF,MAAM,GAAG,GAAG,IAAI,GAAG,CAAC,QAAQ,EAAE,GAAG,IAAI,EAAE,CAAC,CAAA;IACxC,MAAM,OAAO,GAAG,IAAI,GAAG,CAAC,CAAC,QAAQ,EAAE,OAAO,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,CAAA;IAE/E,OAAO,KAAK,CAAC,GAAG,CAAC,CAAC,EAAE,IAAI,EAAE,EAAqB,EAAE;QAC/C,IAAI,GAAG,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC;YAClB,qEAAqE;YACrE,0EAA0E;YAC1E,6DAA6D;YAC7D,MAAM,MAAM,GAAG,QAAQ,EAAE,SAAS,EAAE,CAAC,IAAI,CAAC,CAAA;YAC1C,OAAO,MAAM,IAAI,SAAS,IAAI,MAAM,IAAI,MAAM,CAAC,OAAO,KAAK,IAAI;gBAC7D,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,gBAAgB,EAAE;gBAC/C,CAAC,CAAC,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,KAAK,EAAE,CAAA;QACxC,CAAC;QACD,MAAM,MAAM,GAAG,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QAChC,IAAI,MAAM,KAAK,SAAS;YAAE,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,SAAS,EAAE,MAAM,EAAE,CAAA;QAC/E,OAAO,EAAE,IAAI,EAAE,IAAI,EAAE,WAAW,EAAE,gBAAgB,EAAE,CAAA;IACtD,CAAC,CAAC,CAAA;AACJ,CAAC;AAED,iFAAiF;AACjF,8DAA8D;AAC9D,MAAM,gBAAgB,GAAG,IAAI,GAAG,CAAC,CAAC,YAAY,EAAE,sBAAsB,EAAE,cAAc,CAAC,CAAC,CAAA;AACxF,+EAA+E;AAC/E,sBAAsB;AACtB,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,CAAC,WAAW,EAAE,aAAa,EAAE,oBAAoB,EAAE,cAAc,CAAC,CAAC,CAAA;AAEhG,SAAS,kBAAkB,CAAC,MAAc,EAAE,IAAY,EAAE,KAAa;IAIrE,IAAI,YAAY,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;QAC7B,OAAO;YACL,IAAI,EAAE;gBACJ,EAAE,IAAI,EAAE,2DAA2D,EAAE,WAAW,EAAE,IAAI,EAAE;gBACxF,EAAE,IAAI,EAAE,UAAU,KAAK,4BAA4B,EAAE,WAAW,EAAE,KAAK,EAAE;aAC1E;YACD,KAAK,EAAE,CAAC,gFAAgF,CAAC;SAC1F,CAAA;IACH,CAAC;IACD,IAAI,gBAAgB,CAAC,GAAG,CAAC,MAAM,CAAC,EAAE,CAAC;QACjC,OAAO;YACL,IAAI,EAAE,EAAE;YACR,KAAK,EAAE,CAAC,GAAG,IAAI,iBAAiB,MAAM,4DAA4D,CAAC;SACpG,CAAA;IACH,CAAC;IACD,IAAI,MAAM,KAAK,wBAAwB,EAAE,CAAC;QACxC,OAAO;YACL,IAAI,EAAE,CAAC,EAAE,IAAI,EAAE,UAAU,KAAK,iBAAiB,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC;YACtE,KAAK,EAAE,CAAC,+EAA+E,CAAC;SACzF,CAAA;IACH,CAAC;IACD,OAAO,EAAE,IAAI,EAAE,CAAC,EAAE,IAAI,EAAE,UAAU,KAAK,YAAY,IAAI,EAAE,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC,EAAE,KAAK,EAAE,EAAE,EAAE,CAAA;AAC9F,CAAC;AAED,SAAS,QAAQ,CAAC,GAAY;IAC5B,OAAO,GAAG,KAAK,SAAS,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,EAAE,CAAC,CAAC,CAAA;AACxD,CAAC;AAED,SAAS,eAAe,CAAC,MAAqB;IAC5C,MAAM,KAAK,GAAa,EAAE,CAAA;IAC1B,IAAI,MAAM,CAAC,MAAM;QAAE,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,MAAM,CAAC,CAAA;IAC5C,IAAI,MAAM,CAAC,UAAU,KAAK,SAAS;QAAE,KAAK,CAAC,IAAI,CAAC,GAAG,YAAY,CAAC,MAAM,CAAC,UAAU,CAAC,SAAS,CAAC,CAAA;IAC5F,IAAI,MAAM,CAAC,YAAY,KAAK,CAAC;QAAE,KAAK,CAAC,IAAI,CAAC,oBAAoB,CAAC,CAAA;SAC1D,IAAI,MAAM,CAAC,YAAY,KAAK,SAAS;QAAE,KAAK,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,YAAY,UAAU,MAAM,CAAC,YAAY,KAAK,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,GAAG,UAAU,CAAC,CAAA;IACtI,IAAI,MAAM,CAAC,OAAO,KAAK,IAAI;QAAE,KAAK,CAAC,IAAI,CAAC,mBAAmB,CAAC,CAAA;IAC5D,OAAO,KAAK,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;AAC1B,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,oBAAoB,CAAC,KAAqB;IACxD,MAAM,EAAE,aAAa,EAAE,QAAQ,EAAE,KAAK,EAAE,GAAG,KAAK,CAAA;IAChD,MAAM,YAAY,GAAG,oBAAoB,CAAC,aAAa,EAAE,QAAQ,CAAC,CAAA;IAElE,MAAM,GAAG,GAAG,CAAC,QAAQ,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,IAAI,CAAC,EAAE,CAAC,CAAC;QAC7C,IAAI;QACJ,MAAM,EAAE,QAAQ,EAAE,SAAS,EAAE,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,eAAe,CAAC,QAAQ,CAAC,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,CAAC,CAAC,EAAE;KACrF,CAAC,CAAC,CAAA;IAEH,MAAM,GAAG,GAAa,EAAE,CAAA;IACxB,IAAI,IAAI,GAA6C,EAAE,CAAA;IACvD,MAAM,KAAK,GAAa,EAAE,CAAA;IAE1B,8EAA8E;IAC9E,8EAA8E;IAC9E,6BAA6B;IAC7B,4EAA4E;IAC5E,8EAA8E;IAC9E,MAAM,YAAY,GAAG,IAAI,GAAG,CAAC,aAAa,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,KAAK,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,CAAA;IAC1F,MAAM,WAAW,GAAG,CAAC,QAAQ,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,EAAE,CACpD,YAAY,CAAC,GAAG,CAAC,IAAI,CAAC,IAAI,QAAQ,EAAE,SAAS,EAAE,CAAC,IAAI,CAAC,EAAE,YAAY,KAAK,CAAC,CAAC,CAAA;IAC5E,8EAA8E;IAC9E,kDAAkD;IAClD,MAAM,mBAAmB,GAAG,WAAW,KAAK,SAAS;WAChD,CAAC,QAAQ,EAAE,GAAG,IAAI,EAAE,CAAC,CAAC,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,QAAQ,EAAE,SAAS,EAAE,CAAC,IAAI,CAAC,EAAE,YAAY,KAAK,CAAC,CAAC;QACvF,CAAC,CAAC,WAAW;QACb,CAAC,CAAC,SAAS,CAAA;IAEb,MAAM,YAAY,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,KAAK,gBAAgB,CAAC,CAAA;IAC/E,MAAM,WAAW,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,KAAK,SAAS,CAAC,CAAA;IACvE,MAAM,gBAAgB,GAAG,YAAY,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,KAAK,gBAAgB,CAAC,CAAA;IACpF,MAAM,gBAAgB,GAAG,YAAY,CAAC,KAAK,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,KAAK,gBAAgB,CAAC,CAAA;IAEpF,IAAI,KAAqB,CAAA;IACzB,IAAI,QAAiB,CAAA;IAErB,IAAI,KAAK,CAAC,eAAe,EAAE,CAAC;QAC1B,KAAK,GAAG,eAAe,CAAA;QACvB,QAAQ,GAAG,IAAI,CAAA;QACf,GAAG,CAAC,IAAI,CAAC,iBAAiB,KAAK,CAAC,eAAe,4DAA4D,CAAC,CAAA;IAC9G,CAAC;SAAM,IAAI,gBAAgB,EAAE,CAAC;QAC5B,KAAK,GAAG,gBAAgB,CAAA;QACxB,QAAQ,GAAG,IAAI,CAAA;QACf,GAAG,CAAC,IAAI,CAAC,mFAAmF,CAAC,CAAA;IAC/F,CAAC;SAAM,IAAI,YAAY,EAAE,CAAC;QACxB,KAAK,GAAG,gBAAgB,CAAA;QACxB,QAAQ,GAAG,KAAK,CAAA;QAChB,GAAG,CAAC,IAAI,CAAC,GAAG,YAAY,CAAC,IAAI,qEAAqE,CAAC,CAAA;QACnG,GAAG,CAAC,IAAI,CAAC,0EAA0E,CAAC,CAAA;QACpF,IAAI,GAAG,CAAC,EAAE,IAAI,EAAE,UAAU,KAAK,YAAY,YAAY,CAAC,IAAI,EAAE,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC,CAAA;QACpF,KAAK,CAAC,IAAI,CAAC,8EAA8E,CAAC,CAAA;IAC5F,CAAC;SAAM,IAAI,WAAW,EAAE,CAAC;QACvB,KAAK,GAAG,SAAS,CAAA;QACjB,QAAQ,GAAG,KAAK,CAAA;QAChB,GAAG,CAAC,IAAI,CAAC,2EAA2E,CAAC,CAAA;QACrF,MAAM,MAAM,GAAG,kBAAkB,CAAC,WAAW,CAAC,MAAM,IAAI,SAAS,EAAE,WAAW,CAAC,IAAI,EAAE,KAAK,CAAC,CAAA;QAC3F,IAAI,GAAG,MAAM,CAAC,IAAI,CAAA;QAClB,KAAK,CAAC,IAAI,CAAC,GAAG,MAAM,CAAC,KAAK,CAAC,CAAA;IAC7B,CAAC;SAAM,IAAI,mBAAmB,EAAE,CAAC;QAC/B,+EAA+E;QAC/E,gFAAgF;QAChF,6EAA6E;QAC7E,mDAAmD;QACnD,KAAK,GAAG,UAAU,CAAA;QAClB,QAAQ,GAAG,KAAK,CAAA;QAChB,GAAG,CAAC,IAAI,CAAC,0BAA0B,mBAAmB,qCAAqC,CAAC,CAAA;QAC5F,GAAG,CAAC,IAAI,CAAC,+HAA+H,CAAC,CAAA;QACzI,IAAI,GAAG,CAAC,EAAE,IAAI,EAAE,UAAU,KAAK,iBAAiB,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC,CAAA;QACtE,KAAK,CAAC,IAAI,CAAC,6IAA6I,CAAC,CAAA;IAC3J,CAAC;SAAM,CAAC;QACN,KAAK,GAAG,gBAAgB,CAAA;QACxB,4EAA4E;QAC5E,sCAAsC;QACtC,QAAQ,GAAG,gBAAgB,IAAI,KAAK,CAAC,qBAAqB,KAAK,IAAI,CAAA;QACnE,GAAG,CAAC,IAAI,CAAC,QAAQ;YACf,CAAC,CAAC,4DAA4D;YAC9D,CAAC,CAAC,uEAAuE,CAAC,CAAA;QAC5E,IAAI,CAAC,QAAQ,EAAE,CAAC;YACd,IAAI,GAAG,CAAC,EAAE,IAAI,EAAE,UAAU,KAAK,iBAAiB,EAAE,WAAW,EAAE,IAAI,EAAE,CAAC,CAAA;QACxE,CAAC;IACH,CAAC;IAED,8EAA8E;IAC9E,4CAA4C;IAC5C,IAAI,CAAC,QAAQ,IAAI,KAAK,CAAC,eAAe,EAAE,OAAO,EAAE,CAAC;QAChD,MAAM,GAAG,GAAG,QAAQ,CAAC,KAAK,CAAC,eAAe,CAAC,GAAG,CAAC,CAAA;QAC/C,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAK,CAAC,OAAO,CAAC,CAAA;QACpC,uEAAuE;QACvE,4EAA4E;QAC5E,0EAA0E;QAC1E,0CAA0C;QAC1C,MAAM,MAAM,GAAG,KAAK,CAAC,OAAO,KAAK,SAAS,IAAI,SAAS,CAAC,KAAK,CAAC,eAAe,CAAC,GAAG,EAAE,KAAK,CAAC,OAAO,CAAC,CAAA;QACjG,GAAG,CAAC,IAAI,CAAC,GAAG,KAAK,SAAS,IAAI,IAAI,KAAK,SAAS,IAAI,CAAC,MAAM;YACzD,CAAC,CAAC,gCAAgC,KAAK,CAAC,eAAe,CAAC,OAAO,OAAO,GAAG,wBAAwB,IAAI,IAAI;YACzG,CAAC,CAAC,sCAAsC,KAAK,CAAC,eAAe,CAAC,OAAO,GAAG,GAAG,CAAC,CAAC,CAAC,OAAO,GAAG,EAAE,CAAC,CAAC,CAAC,EAAE,GAAG,CAAC,CAAA;IACvG,CAAC;IAED,+EAA+E;IAC/E,kFAAkF;IAClF,sDAAsD;IACtD,IAAI,CAAC,QAAQ,IAAI,WAAW,IAAI,KAAK,KAAK,gBAAgB,IAAI,KAAK,KAAK,UAAU,EAAE,CAAC;QACnF,GAAG,CAAC,IAAI,CAAC,OAAO,WAAW,0EAA0E,CAAC,CAAA;IACxG,CAAC;IAED,6EAA6E;IAC7E,6EAA6E;IAC7E,6EAA6E;IAC7E,cAAc;IACd,MAAM,eAAe,GAAG,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,CAAC,EAAE,IAAI,CAAC,KAAK,CAAC,eAAe,CAAC,EAAE,CAAC,CAAC,CAAC,CAAA;IACvF,MAAM,WAAW,GAAG,YAAY,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,KAAK,eAAe,IAAI,CAAC,CAAC,WAAW,KAAK,gBAAgB,CAAC,CAAA;IAC5G,IAAI,KAAK,KAAK,gBAAgB,IAAI,eAAe,IAAI,WAAW;WAC3D,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,WAAW,WAAW,CAAC,IAAI,EAAE,CAAC,CAAC,EAAE,CAAC;QACrE,IAAI,CAAC,IAAI,CAAC,EAAE,IAAI,EAAE,UAAU,KAAK,YAAY,WAAW,CAAC,IAAI,EAAE,EAAE,WAAW,EAAE,KAAK,EAAE,CAAC,CAAA;IACxF,CAAC;IAED,8EAA8E;IAC9E,6EAA6E;IAC7E,2EAA2E;IAC3E,2EAA2E;IAC3E,mDAAmD;IACnD,MAAM,KAAK,GAAG,IAAI,GAAG,CAAC,aAAa,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC,CAAA;IAC/D,MAAM,SAAS,GAAG;QAChB,GAAG,CAAC,QAAQ,EAAE,OAAO,IAAI,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,aAAa,CAAC,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;QACxF,GAAG,CAAC,QAAQ,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,YAAY;aAC7B,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,WAAW,KAAK,gBAAgB,CAAC;aAC/C,GAAG,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,IAAI,EAAE,gBAAgB,EAAE,CAAC,CAAC,CAAC;KACzD,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,QAAQ,CAAC,CAAC,CAAA;IAEnF,OAAO,EAAE,QAAQ,EAAE,KAAK,EAAE,YAAY,EAAE,GAAG,EAAE,SAAS,EAAE,GAAG,EAAE,IAAI,EAAE,KAAK,EAAE,CAAA;AAC5E,CAAC;AAED,SAAS,GAAG,CAAC,OAA2B;IACtC,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,EAAE,CAAC,CAAC,IAAI,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAA;AAChE,CAAC;AAED;mEACmE;AACnE,MAAM,UAAU,qBAAqB,CAAC,MAAuB;IAC3D,MAAM,KAAK,GAAa,CAAC,qBAAqB,CAAC,CAAA;IAC/C,MAAM,KAAK,GAAG,IAAI,CAAC,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,GAAG,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAA;IAE9D,IAAI,MAAM,CAAC,GAAG,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC1B,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,KAAK,CAAC,CAAA;QACrB,KAAK,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,IAAI,MAAM,CAAC,GAAG,EAAE,CAAC;YAC1C,KAAK,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,KAAK,MAAM,EAAE,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAA;QACzE,CAAC;IACH,CAAC;IACD,IAAI,MAAM,CAAC,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAChC,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,aAAa,CAAC,CAAA;QAC7B,KAAK,MAAM,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,MAAM,CAAC,SAAS,EAAE,CAAC;YAC9C,KAAK,CAAC,IAAI,CAAC,KAAK,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,KAAK,IAAI,EAAE,CAAC,CAAA;QAChD,CAAC;IACH,CAAC;IACD,IAAI,MAAM,CAAC,GAAG,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC1B,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,KAAK,CAAC,CAAA;QACrB,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,GAAG;YAAE,KAAK,CAAC,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC,CAAA;IACxD,CAAC;IACD,IAAI,MAAM,CAAC,IAAI,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC3B,KAAK,CAAC,IAAI,CAAC,EAAE,EAAE,MAAM,CAAC,CAAA;QACtB,KAAK,MAAM,EAAE,IAAI,EAAE,WAAW,EAAE,IAAI,MAAM,CAAC,IAAI,EAAE,CAAC;YAChD,KAAK,CAAC,IAAI,CAAC,WAAW,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,CAAC,KAAK,IAAI,EAAE,CAAC,CAAA;QACrD,CAAC;IACH,CAAC;IACD,IAAI,MAAM,CAAC,KAAK,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC5B,KAAK,CAAC,IAAI,CAAC,EAAE,CAAC,CAAA;QACd,KAAK,MAAM,IAAI,IAAI,MAAM,CAAC,KAAK;YAAE,KAAK,CAAC,IAAI,CAAC,KAAK,IAAI,EAAE,CAAC,CAAA;IAC1D,CAAC;IACD,OAAO,KAAK,CAAA;AACd,CAAC"}

package/dist/lib/pr-lock.d.ts

@@ -1,4 +1,40 @@
+export declare const INSTANCE_ID: string;
+/** What a holder writes into its lock file so other instances can reason about it. */
+interface LockRecord {
+    instance: string;
+    host: string;
+    pid: number;
+    /** Head SHA being worked on. Diagnostic — the lock itself is per-PR. */
+    sha?: string;
+    startedAt: number;
+    heartbeatAt: number;
+}
 export declare function handleLockSignal(signal: NodeJS.Signals): void;
-export declare function acquirePRLock(owner: string, repo: string, pr: number, _sha?: string): boolean;
+/** Why a lock held by someone else was taken over. Exported for tests and logging. */
+export type StealReason = 'holder_dead' | 'heartbeat_stale' | 'legacy_lock_expired';
+export declare function evaluateExistingLock(record: LockRecord | null, mtimeMs: number, now: number): {
+    stealable: true;
+    reason: StealReason;
+} | {
+    stealable: false;
+};
+/**
+ * Take the local lock for a PR. Returns false when another live process holds it.
+ *
+ * The lock records who holds it, so a peer can tell a live holder from a crashed
+ * one instead of guessing from a fixed timeout.
+ */
+export declare function acquirePRLock(owner: string, repo: string, pr: number, sha?: string): boolean;
+/** Describe the current holder of a PR's lock, for diagnostics. Null when free. */
+export declare function inspectPRLock(owner: string, repo: string, pr: number): LockRecord | null;
+/**
+ * Release the local lock — but only if this process still owns it.
+ *
+ * The ownership check is what stops a late release from clearing a lock that a
+ * peer has since taken over: without it, an instance whose lock was stolen as
+ * stale would delete the new holder's lock on its way out, and a third instance
+ * would walk straight in on a PR two others were already working.
+ */
 export declare function releasePRLock(owner: string, repo: string, pr: number, _sha?: string): void;
+export {};
 //# sourceMappingURL=pr-lock.d.ts.map

package/dist/lib/pr-lock.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pr-lock.d.ts","sourceRoot":"","sources":["../../src/lib/pr-lock.ts"],"names":[],"mappings":"AAwBA,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,MAAM,CAAC,OAAO,GAAG,IAAI,CAS7D;AAsBD,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,MAAM,GAAG,OAAO,CAsB7F;AAED,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,MAAM,GAAG,IAAI,CAI1F"}
+{"version":3,"file":"pr-lock.d.ts","sourceRoot":"","sources":["../../src/lib/pr-lock.ts"],"names":[],"mappings":"AAwBA,eAAO,MAAM,WAAW,QAAmE,CAAA;AAE3F,sFAAsF;AACtF,UAAU,UAAU;IAClB,QAAQ,EAAE,MAAM,CAAA;IAChB,IAAI,EAAE,MAAM,CAAA;IACZ,GAAG,EAAE,MAAM,CAAA;IACX,wEAAwE;IACxE,GAAG,CAAC,EAAE,MAAM,CAAA;IACZ,SAAS,EAAE,MAAM,CAAA;IACjB,WAAW,EAAE,MAAM,CAAA;CACpB;AAmBD,wBAAgB,gBAAgB,CAAC,MAAM,EAAE,MAAM,CAAC,OAAO,GAAG,IAAI,CAU7D;AA8CD,sFAAsF;AACtF,MAAM,MAAM,WAAW,GAAG,aAAa,GAAG,iBAAiB,GAAG,qBAAqB,CAAA;AAMnF,wBAAgB,oBAAoB,CAClC,MAAM,EAAE,UAAU,GAAG,IAAI,EACzB,OAAO,EAAE,MAAM,EACf,GAAG,EAAE,MAAM,GACV;IAAE,SAAS,EAAE,IAAI,CAAC;IAAC,MAAM,EAAE,WAAW,CAAA;CAAE,GAAG;IAAE,SAAS,EAAE,KAAK,CAAA;CAAE,CAgBjE;AA6BD;;;;;GAKG;AACH,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,GAAG,CAAC,EAAE,MAAM,GAAG,OAAO,CA2C5F;AAED,mFAAmF;AACnF,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,GAAG,UAAU,GAAG,IAAI,CAExF;AAED;;;;;;;GAOG;AACH,wBAAgB,aAAa,CAAC,KAAK,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,EAAE,IAAI,CAAC,EAAE,MAAM,GAAG,IAAI,CAW1F"}

package/dist/lib/pr-lock.js

@@ -1,10 +1,27 @@
-import { openSync, closeSync, rmSync, mkdirSync, statSync } from 'fs';
+import { openSync, closeSync, writeSync, readFileSync, rmSync, mkdirSync, statSync, utimesSync } from 'fs';
+import { hostname } from 'os';
+import { randomBytes } from 'crypto';
 import { homedir } from 'os';
 import { join } from 'path';
 const LOCK_DIR = join(homedir(), '.crosscheck', 'locks');
-const STALE_MS = 20 * 60 * 1000;
-// Track active lock paths so signal handlers can clean them up
-const activeLocks = new Set();
+// A lock is abandoned when its holder has not heartbeated within this window.
+// The holder refreshes every HEARTBEAT_INTERVAL_MS, so this only trips on a
+// process that died without running its cleanup — a SIGKILL, an OOM, a power cut.
+//
+// It used to be 20 minutes measured from the lock file's CREATION time, which is
+// the wrong clock: a review that legitimately ran longer than 20 minutes (routine
+// on a large PR — the thorough tier alone allows 1200s per reviewer call, before
+// clone, fix and recheck) had its lock declared stale and stolen while it was
+// still running. Measuring from the last heartbeat means a live review is never
+// stale no matter how long it takes, which in turn lets this window be short.
+const STALE_MS = 5 * 60 * 1000;
+const HEARTBEAT_INTERVAL_MS = 60 * 1000;
+// Identifies this process for the lifetime of the process. The random suffix
+// matters: pids are recycled, so host+pid alone can make a dead holder look alive
+// once the OS hands its pid to something else.
+export const INSTANCE_ID = `${hostname()}:${process.pid}:${randomBytes(4).toString('hex')}`;
+// Locks this process holds, by path, with the heartbeat timer keeping each fresh.
+const heldLocks = new Map();
 let signalHandlersRegistered = false;
 // Exported for unit testing. Cleans up local lock files on signal, then —
 // when no other listener will drive termination — restores the default
@@ -20,13 +37,14 @@ let signalHandlersRegistered = false;
 // watch/serve graceful shutdown handlers referenced in the original
 // comment), we let them drive termination via their own finally blocks.
 export function handleLockSignal(signal) {
-    for (const p of activeLocks) {
+    for (const [path, held] of heldLocks) {
+        clearInterval(held.timer);
         try {
-            rmSync(p);
+            rmSync(path);
         }
         catch { /* ignore */ }
     }
-    activeLocks.clear();
+    heldLocks.clear();
     if (process.listenerCount(signal) <= 1) {
         process.removeListener(signal, handleLockSignal);
         process.kill(process.pid, signal);
@@ -42,47 +60,176 @@ function registerSignalCleanup() {
 function lockPath(owner, repo, pr) {
     return join(LOCK_DIR, `${owner}-${repo}-${pr}.lock`);
 }
-function isStale(path) {
+function readRecord(path) {
     try {
-        const age = Date.now() - statSync(path).mtimeMs;
-        return age > STALE_MS;
+        const raw = readFileSync(path, 'utf8').trim();
+        // A lock written by an older crosscheck is an empty file. It carries no owner
+        // and no heartbeat, so the only honest reading is the file's mtime.
+        if (raw === '')
+            return null;
+        const parsed = JSON.parse(raw);
+        if (typeof parsed.instance !== 'string' || typeof parsed.pid !== 'number')
+            return null;
+        return {
+            instance: parsed.instance,
+            host: typeof parsed.host === 'string' ? parsed.host : '',
+            pid: parsed.pid,
+            ...(typeof parsed.sha === 'string' && { sha: parsed.sha }),
+            startedAt: typeof parsed.startedAt === 'number' ? parsed.startedAt : 0,
+            heartbeatAt: typeof parsed.heartbeatAt === 'number' ? parsed.heartbeatAt : 0,
+        };
     }
     catch {
-        return false;
+        return null;
     }
 }
-export function acquirePRLock(owner, repo, pr, _sha) {
-    mkdirSync(LOCK_DIR, { recursive: true });
-    registerSignalCleanup();
-    const path = lockPath(owner, repo, pr);
+// Whether a process is still running on THIS host. Cross-host liveness is
+// unknowable from here, which is what the heartbeat is for.
+function pidAlive(pid) {
     try {
-        const fd = openSync(path, 'wx');
-        closeSync(fd);
-        activeLocks.add(path);
+        process.kill(pid, 0);
         return true;
     }
     catch (e) {
-        if (e.code !== 'EEXIST')
-            throw e;
-        if (isStale(path)) {
+        // EPERM means the pid exists but belongs to another user — alive, not ours.
+        return e.code === 'EPERM';
+    }
+}
+// Decide whether an existing lock may be taken over. Ownership is proven two ways,
+// in order of strength: a same-host holder whose pid is gone is definitively dead;
+// otherwise the heartbeat is the only evidence, and only silence beyond STALE_MS
+// counts. A lock held by a live peer is never stolen, however long it has run.
+export function evaluateExistingLock(record, mtimeMs, now) {
+    if (record === null) {
+        // Legacy empty lock file: mtime is all there is, and it is never refreshed,
+        // so fall back to the old generous window rather than stealing from a run
+        // that may still be going.
+        return now - mtimeMs > 20 * 60 * 1000
+            ? { stealable: true, reason: 'legacy_lock_expired' }
+            : { stealable: false };
+    }
+    if (record.host === hostname() && !pidAlive(record.pid)) {
+        return { stealable: true, reason: 'holder_dead' };
+    }
+    if (now - record.heartbeatAt > STALE_MS) {
+        return { stealable: true, reason: 'heartbeat_stale' };
+    }
+    return { stealable: false };
+}
+function writeRecord(path, record) {
+    const fd = openSync(path, 'wx');
+    try {
+        writeSync(fd, JSON.stringify(record));
+    }
+    finally {
+        closeSync(fd);
+    }
+}
+function startHeartbeat(path, record) {
+    const timer = setInterval(() => {
+        const held = heldLocks.get(path);
+        if (!held)
+            return;
+        held.record.heartbeatAt = Date.now();
+        try {
+            // Rewrite rather than touch: the timestamp inside the file is what a peer
+            // on another host reads, and mtime alone would not reach it.
+            const fd = openSync(path, 'w');
             try {
-                rmSync(path);
+                writeSync(fd, JSON.stringify(held.record));
             }
-            catch { /* already gone */ }
-            try {
-                const fd = openSync(path, 'wx');
+            finally {
                 closeSync(fd);
-                activeLocks.add(path);
-                return true;
             }
-            catch { /* lost the race after clearing stale lock */ }
+            utimesSync(path, new Date(), new Date());
         }
+        catch { /* best-effort — a failed beat is covered by the next one */ }
+    }, HEARTBEAT_INTERVAL_MS);
+    // Never hold the event loop open on the heartbeat alone.
+    timer.unref?.();
+    heldLocks.set(path, { record, timer });
+}
+/**
+ * Take the local lock for a PR. Returns false when another live process holds it.
+ *
+ * The lock records who holds it, so a peer can tell a live holder from a crashed
+ * one instead of guessing from a fixed timeout.
+ */
+export function acquirePRLock(owner, repo, pr, sha) {
+    mkdirSync(LOCK_DIR, { recursive: true });
+    registerSignalCleanup();
+    const path = lockPath(owner, repo, pr);
+    const now = Date.now();
+    const record = {
+        instance: INSTANCE_ID,
+        host: hostname(),
+        pid: process.pid,
+        ...(sha !== undefined && { sha }),
+        startedAt: now,
+        heartbeatAt: now,
+    };
+    try {
+        writeRecord(path, record);
+        startHeartbeat(path, record);
+        return true;
+    }
+    catch (e) {
+        if (e.code !== 'EEXIST')
+            throw e;
+    }
+    // Re-entering our own lock is not a collision: the same process asking twice for
+    // the same PR means a second event arrived for it, and the caller's own inFlight
+    // guard decides that. Report it as taken so the caller's skip path runs.
+    const existing = readRecord(path);
+    if (existing?.instance === INSTANCE_ID)
+        return false;
+    let mtimeMs = 0;
+    try {
+        mtimeMs = statSync(path).mtimeMs;
+    }
+    catch {
+        return false;
+    }
+    const verdict = evaluateExistingLock(existing, mtimeMs, now);
+    if (!verdict.stealable)
+        return false;
+    try {
+        rmSync(path);
+    }
+    catch { /* already gone */ }
+    try {
+        writeRecord(path, record);
+        startHeartbeat(path, record);
+        return true;
+    }
+    catch {
+        // Lost the race to another instance that cleared the same stale lock.
         return false;
     }
 }
+/** Describe the current holder of a PR's lock, for diagnostics. Null when free. */
+export function inspectPRLock(owner, repo, pr) {
+    return readRecord(lockPath(owner, repo, pr));
+}
+/**
+ * Release the local lock — but only if this process still owns it.
+ *
+ * The ownership check is what stops a late release from clearing a lock that a
+ * peer has since taken over: without it, an instance whose lock was stolen as
+ * stale would delete the new holder's lock on its way out, and a third instance
+ * would walk straight in on a PR two others were already working.
+ */
 export function releasePRLock(owner, repo, pr, _sha) {
     const path = lockPath(owner, repo, pr);
-    activeLocks.delete(path);
+    const held = heldLocks.get(path);
+    if (held)
+        clearInterval(held.timer);
+    heldLocks.delete(path);
+    const existing = readRecord(path);
+    // An unreadable or legacy record cannot prove another owner. Removing it matches
+    // the pre-ownership behaviour and keeps an upgrade from stranding old locks.
+    if (existing !== null && existing.instance !== INSTANCE_ID)
+        return;
     try {
         rmSync(path);
     }

package/dist/lib/pr-lock.js.map

@@ -1 +1 @@
-{"version":3,"file":"pr-lock.js","sourceRoot":"","sources":["../../src/lib/pr-lock.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,IAAI,CAAA;AACrE,OAAO,EAAE,OAAO,EAAE,MAAM,IAAI,CAAA;AAC5B,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAA;AAE3B,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,OAAO,CAAC,CAAA;AACxD,MAAM,QAAQ,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI,CAAA;AAE/B,+DAA+D;AAC/D,MAAM,WAAW,GAAG,IAAI,GAAG,EAAU,CAAA;AACrC,IAAI,wBAAwB,GAAG,KAAK,CAAA;AAEpC,0EAA0E;AAC1E,uEAAuE;AACvE,0CAA0C;AAC1C,EAAE;AACF,2EAA2E;AAC3E,wEAAwE;AACxE,yEAAyE;AACzE,wEAAwE;AACxE,qEAAqE;AACrE,qEAAqE;AACrE,oEAAoE;AACpE,oEAAoE;AACpE,wEAAwE;AACxE,MAAM,UAAU,gBAAgB,CAAC,MAAsB;IACrD,KAAK,MAAM,CAAC,IAAI,WAAW,EAAE,CAAC;QAC5B,IAAI,CAAC;YAAC,MAAM,CAAC,CAAC,CAAC,CAAA;QAAC,CAAC;QAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;IAC1C,CAAC;IACD,WAAW,CAAC,KAAK,EAAE,CAAA;IACnB,IAAI,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;QACvC,OAAO,CAAC,cAAc,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAA;QAChD,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,MAAM,CAAC,CAAA;IACnC,CAAC;AACH,CAAC;AAED,SAAS,qBAAqB;IAC5B,IAAI,wBAAwB;QAAE,OAAM;IACpC,wBAAwB,GAAG,IAAI,CAAA;IAC/B,OAAO,CAAC,EAAE,CAAC,SAAS,EAAE,gBAAgB,CAAC,CAAA;IACvC,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAA;AACxC,CAAC;AAED,SAAS,QAAQ,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU;IACvD,OAAO,IAAI,CAAC,QAAQ,EAAE,GAAG,KAAK,IAAI,IAAI,IAAI,EAAE,OAAO,CAAC,CAAA;AACtD,CAAC;AAED,SAAS,OAAO,CAAC,IAAY;IAC3B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,OAAO,CAAA;QAC/C,OAAO,GAAG,GAAG,QAAQ,CAAA;IACvB,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAA;IACd,CAAC;AACH,CAAC;AAED,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU,EAAE,IAAa;IAClF,SAAS,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IACxC,qBAAqB,EAAE,CAAA;IACvB,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,CAAC,CAAA;IACtC,IAAI,CAAC;QACH,MAAM,EAAE,GAAG,QAAQ,CAAC,IAAI,EAAE,IAAI,CAAC,CAAA;QAC/B,SAAS,CAAC,EAAE,CAAC,CAAA;QACb,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QACrB,OAAO,IAAI,CAAA;IACb,CAAC;IAAC,OAAO,CAAU,EAAE,CAAC;QACpB,IAAK,CAA2B,CAAC,IAAI,KAAK,QAAQ;YAAE,MAAM,CAAC,CAAA;QAC3D,IAAI,OAAO,CAAC,IAAI,CAAC,EAAE,CAAC;YAClB,IAAI,CAAC;gBAAC,MAAM,CAAC,IAAI,CAAC,CAAA;YAAC,CAAC;YAAC,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC;YACjD,IAAI,CAAC;gBACH,MAAM,EAAE,GAAG,QAAQ,CAAC,IAAI,EAAE,IAAI,CAAC,CAAA;gBAC/B,SAAS,CAAC,EAAE,CAAC,CAAA;gBACb,WAAW,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;gBACrB,OAAO,IAAI,CAAA;YACb,CAAC;YAAC,MAAM,CAAC,CAAC,6CAA6C,CAAC,CAAC;QAC3D,CAAC;QACD,OAAO,KAAK,CAAA;IACd,CAAC;AACH,CAAC;AAED,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU,EAAE,IAAa;IAClF,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,CAAC,CAAA;IACtC,WAAW,CAAC,MAAM,CAAC,IAAI,CAAC,CAAA;IACxB,IAAI,CAAC;QAAC,MAAM,CAAC,IAAI,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC;AACnD,CAAC"}
+{"version":3,"file":"pr-lock.js","sourceRoot":"","sources":["../../src/lib/pr-lock.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,QAAQ,EAAE,SAAS,EAAE,SAAS,EAAE,YAAY,EAAE,MAAM,EAAE,SAAS,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,IAAI,CAAA;AAC1G,OAAO,EAAE,QAAQ,EAAE,MAAM,IAAI,CAAA;AAC7B,OAAO,EAAE,WAAW,EAAE,MAAM,QAAQ,CAAA;AACpC,OAAO,EAAE,OAAO,EAAE,MAAM,IAAI,CAAA;AAC5B,OAAO,EAAE,IAAI,EAAE,MAAM,MAAM,CAAA;AAE3B,MAAM,QAAQ,GAAG,IAAI,CAAC,OAAO,EAAE,EAAE,aAAa,EAAE,OAAO,CAAC,CAAA;AAExD,8EAA8E;AAC9E,4EAA4E;AAC5E,kFAAkF;AAClF,EAAE;AACF,iFAAiF;AACjF,kFAAkF;AAClF,iFAAiF;AACjF,8EAA8E;AAC9E,gFAAgF;AAChF,8EAA8E;AAC9E,MAAM,QAAQ,GAAG,CAAC,GAAG,EAAE,GAAG,IAAI,CAAA;AAC9B,MAAM,qBAAqB,GAAG,EAAE,GAAG,IAAI,CAAA;AAEvC,6EAA6E;AAC7E,kFAAkF;AAClF,+CAA+C;AAC/C,MAAM,CAAC,MAAM,WAAW,GAAG,GAAG,QAAQ,EAAE,IAAI,OAAO,CAAC,GAAG,IAAI,WAAW,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,KAAK,CAAC,EAAE,CAAA;AAa3F,kFAAkF;AAClF,MAAM,SAAS,GAAG,IAAI,GAAG,EAAyE,CAAA;AAClG,IAAI,wBAAwB,GAAG,KAAK,CAAA;AAEpC,0EAA0E;AAC1E,uEAAuE;AACvE,0CAA0C;AAC1C,EAAE;AACF,2EAA2E;AAC3E,wEAAwE;AACxE,yEAAyE;AACzE,wEAAwE;AACxE,qEAAqE;AACrE,qEAAqE;AACrE,oEAAoE;AACpE,oEAAoE;AACpE,wEAAwE;AACxE,MAAM,UAAU,gBAAgB,CAAC,MAAsB;IACrD,KAAK,MAAM,CAAC,IAAI,EAAE,IAAI,CAAC,IAAI,SAAS,EAAE,CAAC;QACrC,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;QACzB,IAAI,CAAC;YAAC,MAAM,CAAC,IAAI,CAAC,CAAA;QAAC,CAAC;QAAC,MAAM,CAAC,CAAC,YAAY,CAAC,CAAC;IAC7C,CAAC;IACD,SAAS,CAAC,KAAK,EAAE,CAAA;IACjB,IAAI,OAAO,CAAC,aAAa,CAAC,MAAM,CAAC,IAAI,CAAC,EAAE,CAAC;QACvC,OAAO,CAAC,cAAc,CAAC,MAAM,EAAE,gBAAgB,CAAC,CAAA;QAChD,OAAO,CAAC,IAAI,CAAC,OAAO,CAAC,GAAG,EAAE,MAAM,CAAC,CAAA;IACnC,CAAC;AACH,CAAC;AAED,SAAS,qBAAqB;IAC5B,IAAI,wBAAwB;QAAE,OAAM;IACpC,wBAAwB,GAAG,IAAI,CAAA;IAC/B,OAAO,CAAC,EAAE,CAAC,SAAS,EAAE,gBAAgB,CAAC,CAAA;IACvC,OAAO,CAAC,EAAE,CAAC,QAAQ,EAAE,gBAAgB,CAAC,CAAA;AACxC,CAAC;AAED,SAAS,QAAQ,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU;IACvD,OAAO,IAAI,CAAC,QAAQ,EAAE,GAAG,KAAK,IAAI,IAAI,IAAI,EAAE,OAAO,CAAC,CAAA;AACtD,CAAC;AAED,SAAS,UAAU,CAAC,IAAY;IAC9B,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,YAAY,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC,IAAI,EAAE,CAAA;QAC7C,8EAA8E;QAC9E,oEAAoE;QACpE,IAAI,GAAG,KAAK,EAAE;YAAE,OAAO,IAAI,CAAA;QAC3B,MAAM,MAAM,GAAG,IAAI,CAAC,KAAK,CAAC,GAAG,CAAwB,CAAA;QACrD,IAAI,OAAO,MAAM,CAAC,QAAQ,KAAK,QAAQ,IAAI,OAAO,MAAM,CAAC,GAAG,KAAK,QAAQ;YAAE,OAAO,IAAI,CAAA;QACtF,OAAO;YACL,QAAQ,EAAE,MAAM,CAAC,QAAQ;YACzB,IAAI,EAAE,OAAO,MAAM,CAAC,IAAI,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE;YACxD,GAAG,EAAE,MAAM,CAAC,GAAG;YACf,GAAG,CAAC,OAAO,MAAM,CAAC,GAAG,KAAK,QAAQ,IAAI,EAAE,GAAG,EAAE,MAAM,CAAC,GAAG,EAAE,CAAC;YAC1D,SAAS,EAAE,OAAO,MAAM,CAAC,SAAS,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC;YACtE,WAAW,EAAE,OAAO,MAAM,CAAC,WAAW,KAAK,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC,WAAW,CAAC,CAAC,CAAC,CAAC;SAC7E,CAAA;IACH,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,IAAI,CAAA;IACb,CAAC;AACH,CAAC;AAED,0EAA0E;AAC1E,4DAA4D;AAC5D,SAAS,QAAQ,CAAC,GAAW;IAC3B,IAAI,CAAC;QACH,OAAO,CAAC,IAAI,CAAC,GAAG,EAAE,CAAC,CAAC,CAAA;QACpB,OAAO,IAAI,CAAA;IACb,CAAC;IAAC,OAAO,CAAU,EAAE,CAAC;QACpB,4EAA4E;QAC5E,OAAQ,CAA2B,CAAC,IAAI,KAAK,OAAO,CAAA;IACtD,CAAC;AACH,CAAC;AAKD,mFAAmF;AACnF,mFAAmF;AACnF,iFAAiF;AACjF,+EAA+E;AAC/E,MAAM,UAAU,oBAAoB,CAClC,MAAyB,EACzB,OAAe,EACf,GAAW;IAEX,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;QACpB,4EAA4E;QAC5E,0EAA0E;QAC1E,2BAA2B;QAC3B,OAAO,GAAG,GAAG,OAAO,GAAG,EAAE,GAAG,EAAE,GAAG,IAAI;YACnC,CAAC,CAAC,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,qBAAqB,EAAE;YACpD,CAAC,CAAC,EAAE,SAAS,EAAE,KAAK,EAAE,CAAA;IAC1B,CAAC;IACD,IAAI,MAAM,CAAC,IAAI,KAAK,QAAQ,EAAE,IAAI,CAAC,QAAQ,CAAC,MAAM,CAAC,GAAG,CAAC,EAAE,CAAC;QACxD,OAAO,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,aAAa,EAAE,CAAA;IACnD,CAAC;IACD,IAAI,GAAG,GAAG,MAAM,CAAC,WAAW,GAAG,QAAQ,EAAE,CAAC;QACxC,OAAO,EAAE,SAAS,EAAE,IAAI,EAAE,MAAM,EAAE,iBAAiB,EAAE,CAAA;IACvD,CAAC;IACD,OAAO,EAAE,SAAS,EAAE,KAAK,EAAE,CAAA;AAC7B,CAAC;AAED,SAAS,WAAW,CAAC,IAAY,EAAE,MAAkB;IACnD,MAAM,EAAE,GAAG,QAAQ,CAAC,IAAI,EAAE,IAAI,CAAC,CAAA;IAC/B,IAAI,CAAC;QACH,SAAS,CAAC,EAAE,EAAE,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,CAAC,CAAA;IACvC,CAAC;YAAS,CAAC;QACT,SAAS,CAAC,EAAE,CAAC,CAAA;IACf,CAAC;AACH,CAAC;AAED,SAAS,cAAc,CAAC,IAAY,EAAE,MAAkB;IACtD,MAAM,KAAK,GAAG,WAAW,CAAC,GAAG,EAAE;QAC7B,MAAM,IAAI,GAAG,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;QAChC,IAAI,CAAC,IAAI;YAAE,OAAM;QACjB,IAAI,CAAC,MAAM,CAAC,WAAW,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;QACpC,IAAI,CAAC;YACH,0EAA0E;YAC1E,6DAA6D;YAC7D,MAAM,EAAE,GAAG,QAAQ,CAAC,IAAI,EAAE,GAAG,CAAC,CAAA;YAC9B,IAAI,CAAC;gBAAC,SAAS,CAAC,EAAE,EAAE,IAAI,CAAC,SAAS,CAAC,IAAI,CAAC,MAAM,CAAC,CAAC,CAAA;YAAC,CAAC;oBAAS,CAAC;gBAAC,SAAS,CAAC,EAAE,CAAC,CAAA;YAAC,CAAC;YAC5E,UAAU,CAAC,IAAI,EAAE,IAAI,IAAI,EAAE,EAAE,IAAI,IAAI,EAAE,CAAC,CAAA;QAC1C,CAAC;QAAC,MAAM,CAAC,CAAC,4DAA4D,CAAC,CAAC;IAC1E,CAAC,EAAE,qBAAqB,CAAC,CAAA;IACzB,yDAAyD;IACzD,KAAK,CAAC,KAAK,EAAE,EAAE,CAAA;IACf,SAAS,CAAC,GAAG,CAAC,IAAI,EAAE,EAAE,MAAM,EAAE,KAAK,EAAE,CAAC,CAAA;AACxC,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU,EAAE,GAAY;IACjF,SAAS,CAAC,QAAQ,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAA;IACxC,qBAAqB,EAAE,CAAA;IACvB,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,CAAC,CAAA;IACtC,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,EAAE,CAAA;IACtB,MAAM,MAAM,GAAe;QACzB,QAAQ,EAAE,WAAW;QACrB,IAAI,EAAE,QAAQ,EAAE;QAChB,GAAG,EAAE,OAAO,CAAC,GAAG;QAChB,GAAG,CAAC,GAAG,KAAK,SAAS,IAAI,EAAE,GAAG,EAAE,CAAC;QACjC,SAAS,EAAE,GAAG;QACd,WAAW,EAAE,GAAG;KACjB,CAAA;IAED,IAAI,CAAC;QACH,WAAW,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;QACzB,cAAc,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;QAC5B,OAAO,IAAI,CAAA;IACb,CAAC;IAAC,OAAO,CAAU,EAAE,CAAC;QACpB,IAAK,CAA2B,CAAC,IAAI,KAAK,QAAQ;YAAE,MAAM,CAAC,CAAA;IAC7D,CAAC;IAED,iFAAiF;IACjF,iFAAiF;IACjF,yEAAyE;IACzE,MAAM,QAAQ,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;IACjC,IAAI,QAAQ,EAAE,QAAQ,KAAK,WAAW;QAAE,OAAO,KAAK,CAAA;IAEpD,IAAI,OAAO,GAAG,CAAC,CAAA;IACf,IAAI,CAAC;QAAC,OAAO,GAAG,QAAQ,CAAC,IAAI,CAAC,CAAC,OAAO,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC;QAAC,OAAO,KAAK,CAAA;IAAC,CAAC;IAE/D,MAAM,OAAO,GAAG,oBAAoB,CAAC,QAAQ,EAAE,OAAO,EAAE,GAAG,CAAC,CAAA;IAC5D,IAAI,CAAC,OAAO,CAAC,SAAS;QAAE,OAAO,KAAK,CAAA;IAEpC,IAAI,CAAC;QAAC,MAAM,CAAC,IAAI,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC;IACjD,IAAI,CAAC;QACH,WAAW,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;QACzB,cAAc,CAAC,IAAI,EAAE,MAAM,CAAC,CAAA;QAC5B,OAAO,IAAI,CAAA;IACb,CAAC;IAAC,MAAM,CAAC;QACP,sEAAsE;QACtE,OAAO,KAAK,CAAA;IACd,CAAC;AACH,CAAC;AAED,mFAAmF;AACnF,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU;IACnE,OAAO,UAAU,CAAC,QAAQ,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,CAAC,CAAC,CAAA;AAC9C,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,aAAa,CAAC,KAAa,EAAE,IAAY,EAAE,EAAU,EAAE,IAAa;IAClF,MAAM,IAAI,GAAG,QAAQ,CAAC,KAAK,EAAE,IAAI,EAAE,EAAE,CAAC,CAAA;IACtC,MAAM,IAAI,GAAG,SAAS,CAAC,GAAG,CAAC,IAAI,CAAC,CAAA;IAChC,IAAI,IAAI;QAAE,aAAa,CAAC,IAAI,CAAC,KAAK,CAAC,CAAA;IACnC,SAAS,CAAC,MAAM,CAAC,IAAI,CAAC,CAAA;IAEtB,MAAM,QAAQ,GAAG,UAAU,CAAC,IAAI,CAAC,CAAA;IACjC,iFAAiF;IACjF,6EAA6E;IAC7E,IAAI,QAAQ,KAAK,IAAI,IAAI,QAAQ,CAAC,QAAQ,KAAK,WAAW;QAAE,OAAM;IAClE,IAAI,CAAC;QAAC,MAAM,CAAC,IAAI,CAAC,CAAA;IAAC,CAAC;IAAC,MAAM,CAAC,CAAC,kBAAkB,CAAC,CAAC;AACnD,CAAC"}

package/dist/lib/pr-spec.d.ts

@@ -4,6 +4,13 @@ export interface PRRef {
     number: number;
     /** Canonical https URL for the PR. */
     url: string;
+    /**
+     * The `#issuecomment-<id>` anchor, when the token carried one. Callers that act
+     * on a specific review comment use it to target exactly the comment the user
+     * pasted; before this existed the fragment was silently dropped and `ck fix`
+     * ran against whatever the latest review happened to be.
+     */
+    commentId?: number;
 }
 export interface ParsePRSpecOptions {
     /** Hard cap on the number of PRs a single spec may expand to. */

package/dist/lib/pr-spec.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"pr-spec.d.ts","sourceRoot":"","sources":["../../src/lib/pr-spec.ts"],"names":[],"mappings":"AAkBA,MAAM,WAAW,KAAK;IACpB,KAAK,EAAE,MAAM,CAAA;IACb,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,MAAM,CAAA;IACd,sCAAsC;IACtC,GAAG,EAAE,MAAM,CAAA;CACZ;AAED,MAAM,WAAW,kBAAkB;IACjC,iEAAiE;IACjE,MAAM,CAAC,EAAE,MAAM,CAAA;CAChB;AAED,eAAO,MAAM,eAAe,MAAM,CAAA;AAyBlC,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,GAAE,kBAAuB,GAAG,KAAK,EAAE,CA6CnF"}
+{"version":3,"file":"pr-spec.d.ts","sourceRoot":"","sources":["../../src/lib/pr-spec.ts"],"names":[],"mappings":"AAkBA,MAAM,WAAW,KAAK;IACpB,KAAK,EAAE,MAAM,CAAA;IACb,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,MAAM,CAAA;IACd,sCAAsC;IACtC,GAAG,EAAE,MAAM,CAAA;IACX;;;;;OAKG;IACH,SAAS,CAAC,EAAE,MAAM,CAAA;CACnB;AAED,MAAM,WAAW,kBAAkB;IACjC,iEAAiE;IACjE,MAAM,CAAC,EAAE,MAAM,CAAA;CAChB;AAED,eAAO,MAAM,eAAe,MAAM,CAAA;AA2BlC,wBAAgB,WAAW,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,GAAE,kBAAuB,GAAG,KAAK,EAAE,CAsDnF"}