@telora/daemon 0.22.239 → 0.22.289

package/dist/completion/review-exit/review-defect-detector.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-defect-detector.d.ts","sourceRoot":"","sources":["../../../src/completion/review-exit/review-defect-detector.ts"],"names":[],"mappings":"AAQA;;;;;;;;;;;;;;;;;GAiBG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAG3D,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,oDAAoD,CAAC;AAI7F;;;GAGG;AACH,eAAO,MAAM,wBAAwB,EAAE,MACU,CAAC;AAYlD,wBAAgB,oBAAoB,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAE5D;AAED,wBAAgB,0BAA0B,CAAC,OAAO,EAAE,MAAM,GAAG,MAAM,CAIlE;AAED,wBAAgB,sBAAsB,CAAC,OAAO,EAAE,MAAM,GAAG,IAAI,CAE5D;AAED,wCAAwC;AACxC,wBAAgB,iCAAiC,IAAI,IAAI,CAExD;AAID;;;;;;;GAOG;AACH,wBAAgB,kBAAkB,CAChC,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,iBAAiB,GAAG,WAAW,GAAG,QAAQ,CAAC,EACvE,iBAAiB,EAAE,MAAM,GAAG,IAAI,GAC/B,OAAO,CAMT;AAED;;;GAGG;AACH,wBAAgB,uBAAuB,CACrC,MAAM,EAAE,cAAc,EAAE,EACxB,iBAAiB,EAAE,MAAM,GAAG,IAAI,GAC/B,cAAc,EAAE,CAElB;AAED;;;GAGG;AACH,wBAAgB,sBAAsB,CACpC,MAAM,EAAE,cAAc,EAAE,EACxB,iBAAiB,EAAE,MAAM,GAAG,IAAI,GAC/B,OAAO,CAUT;AAeD;;;;;;;;GAQG;AACH,wBAAgB,aAAa,CAAC,QAAQ,EAAE,MAAM,GAAG,IAAI,GAAG,SAAS,GAAG,OAAO,CAG1E;AAED;;;;GAIG;AACH,MAAM,MAAM,oBAAoB,GAAG,WAAW,CAAC,MAAM,EAAE,kBAAkB,CAAC,CAAC;AAE3E,sFAAsF;AACtF,eAAO,MAAM,mBAAmB,EAAE,oBAAgC,CAAC;AAEnE;;;;;;;;;;;;;;;;;;;GAmBG;AACH,wBAAgB,uBAAuB,CACrC,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,IAAI,GAAG,UAAU,CAAC,EAC9C,cAAc,EAAE,oBAAoB,GACnC,OAAO,CAIT;AAED;;;;;;;;;;GAUG;AACH,wBAAgB,sBAAsB,CACpC,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,IAAI,GAAG,iBAAiB,GAAG,WAAW,GAAG,QAAQ,GAAG,UAAU,CAAC,EAC3F,iBAAiB,EAAE,MAAM,GAAG,IAAI,EAChC,cAAc,EAAE,oBAAoB,GACnC,OAAO,CAET;AAED;;;;;GAKG;AACH,wBAAgB,2BAA2B,CACzC,MAAM,EAAE,cAAc,EAAE,EACxB,iBAAiB,EAAE,MAAM,GAAG,IAAI,EAChC,cAAc,EAAE,oBAAoB,GACnC,cAAc,EAAE,CAElB;AAED;;;;;;;GAOG;AACH,wBAAgB,8BAA8B,CAC5C,MAAM,EAAE,cAAc,EAAE,EACxB,iBAAiB,EAAE,MAAM,GAAG,IAAI,EAChC,cAAc,EAAE,oBAAoB,GACnC,OAAO,CAIT;AAmBD;;;GAGG;AACH,wBAAgB,iBAAiB,CAC/B,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,IAAI,GAAG,QAAQ,CAAC,EAC5C,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,GAClC,OAAO,CAET;AAED,mFAAmF;AACnF,wBAAgB,qBAAqB,CACnC,KAAK,EAAE,IAAI,CAAC,cAAc,EAAE,IAAI,GAAG,iBAAiB,GAAG,WAAW,GAAG,QAAQ,CAAC,EAC9E,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,EACnC,iBAAiB,EAAE,MAAM,GAAG,IAAI,GAC/B,OAAO,CAET;AAED,0DAA0D;AAC1D,wBAAgB,8BAA8B,CAC5C,MAAM,EAAE,cAAc,EAAE,EACxB,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,EACnC,iBAAiB,EAAE,MAAM,GAAG,IAAI,GAC/B,OAAO,CAET;AAED,kEAAkE;AAClE,wBAAgB,sCAAsC,CACpD,MAAM,EAAE,cAAc,EAAE,EACxB,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,EACnC,iBAAiB,EAAE,MAAM,GAAG,IAAI,EAChC,cAAc,EAAE,oBAAoB,GACnC,OAAO,CAIT;AAED;;;;;;;GAOG;AACH,wBAAgB,iCAAiC,CAC/C,MAAM,EAAE,cAAc,EAAE,EACxB,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,EACnC,iBAAiB,EAAE,MAAM,GAAG,IAAI,EAChC,cAAc,EAAE,oBAAoB,GACnC,cAAc,EAAE,CAIlB;AAED,+DAA+D;AAC/D,wBAAgB,mCAAmC,CACjD,MAAM,EAAE,cAAc,EAAE,EACxB,cAAc,EAAE,WAAW,CAAC,MAAM,CAAC,EACnC,iBAAiB,EAAE,MAAM,GAAG,IAAI,EAChC,cAAc,EAAE,oBAAoB,GACnC,cAAc,EAAE,CAIlB"}

package/dist/completion/review-exit/review-defect-detector.js

@@ -0,0 +1,221 @@
+// PINNED-BY-BEGIN (generated by gen:pinned-by -- do not hand-edit; checked by lint:pinned-by)
+// Pinned by: packages/daemon/src/completion/review-exit/review-defect-loop.test.ts,
+//   packages/daemon/src/completion/review-exit/review-deferrable-classification.test.ts,
+//   packages/daemon/src/completion/review-exit/review-exit-foreign-findings.test.ts,
+//   packages/daemon/src/completion/review-exit/review-exit-foreign-focus-regression.test.ts,
+//   packages/daemon/src/completion/review-exit/review-finding-blocking-predicate.test.ts
+// PINNED-BY-END
+import { OPEN_ISSUE_STATUSES } from '../../types/issue-status.js';
+import { loadEnvConfig } from '@telora/daemon-core';
+// ── Configuration ────────────────────────────────────────────────────────
+/**
+ * Cap on remediation cycles before escalation. Sourced from the typed
+ * env-config (TELORA_REVIEW_REMEDIATION_LIMIT, validated at startup; default 3).
+ */
+export const REVIEW_REMEDIATION_LIMIT = loadEnvConfig().TELORA_REVIEW_REMEDIATION_LIMIT;
+// ── Per-focus cycle counter ──────────────────────────────────────────
+/**
+ * In-memory map of focus ID -> consecutive remediation cycles. Cycle
+ * counts are not persisted -- a daemon restart resets the counter, which
+ * is acceptable because the bound exists to prevent runaway loops within
+ * a single daemon lifetime, not across restarts.
+ */
+const remediationCycles = new Map();
+export function getRemediationCycles(focusId) {
+    return remediationCycles.get(focusId) ?? 0;
+}
+export function incrementRemediationCycles(focusId) {
+    const next = (remediationCycles.get(focusId) ?? 0) + 1;
+    remediationCycles.set(focusId, next);
+    return next;
+}
+export function resetRemediationCycles(focusId) {
+    remediationCycles.delete(focusId);
+}
+/** Test-only: clear all cycle state. */
+export function _resetAllRemediationCyclesForTest() {
+    remediationCycles.clear();
+}
+// ── Issue classification ────────────────────────────────────────────────
+/**
+ * Returns true when an issue was filed during the most recent review pass.
+ *
+ * An issue is "review-filed" when ALL of:
+ *   - It has a non-null created_by_role_id (excludes human/UI creates).
+ *   - Its created_at is later than the focus's review_requested_at.
+ *   - Its status is open (To Do / In Progress / Blocked).
+ */
+export function isReviewFiledIssue(issue, reviewRequestedAt) {
+    if (!issue.createdByRoleId)
+        return false;
+    if (!OPEN_ISSUE_STATUSES.has(issue.status))
+        return false;
+    if (!reviewRequestedAt)
+        return false;
+    if (!issue.createdAt)
+        return false;
+    return new Date(issue.createdAt).getTime() >= new Date(reviewRequestedAt).getTime();
+}
+/**
+ * Filter a focus's issues to only those that were filed during the most
+ * recent review pass and are still open.
+ */
+export function filterReviewFiledIssues(issues, reviewRequestedAt) {
+    return issues.filter(i => isReviewFiledIssue(i, reviewRequestedAt));
+}
+/**
+ * Returns true when there are any review-filed issues (open or closed)
+ * since the most recent review pass.
+ */
+export function reviewFiledIssueExists(issues, reviewRequestedAt) {
+    if (!reviewRequestedAt)
+        return false;
+    for (const i of issues) {
+        if (!i.createdByRoleId)
+            continue;
+        if (!i.createdAt)
+            continue;
+        if (new Date(i.createdAt).getTime() >= new Date(reviewRequestedAt).getTime()) {
+            return true;
+        }
+    }
+    return false;
+}
+// ── Deferrability classification ──────────────────────────────────────────
+/**
+ * Priority labels that mark a review-filed note as DEFERRABLE when NO finding
+ * class is recorded for it (the fallback inside isBlockingReviewFinding -- a
+ * recorded blocking/concern fact always wins). The blocking set is everything
+ * else -- P0/P1 AND a null/unset priority. The asymmetry is deliberate: the
+ * control loop moves reality toward intent and never lowers the bar, so an
+ * unprioritized note is treated as BLOCKING (route to verify_failed), not
+ * silently converged. Only an EXPLICIT low priority makes a note deferrable.
+ */
+const DEFERRABLE_PRIORITIES = new Set(['P2', 'P3']);
+/**
+ * Whether a priority label is an explicit, low (deferrable) severity. Null,
+ * empty, P0, and P1 are all NOT deferrable.
+ *
+ * This is the FALLBACK of isBlockingReviewFinding only, for a finding with no
+ * recorded class. Do not call it to decide whether a review finding blocks --
+ * call isBlockingReviewFinding (pinned by review-finding-blocking-predicate.test.ts,
+ * which fails on any non-test caller outside this module).
+ */
+export function isLowPriority(priority) {
+    if (!priority)
+        return false;
+    return DEFERRABLE_PRIORITIES.has(priority.trim().toUpperCase());
+}
+/** No recorded finding classes: every finding is classified by its priority alone. */
+export const NO_RECORDED_CLASSES = new Map();
+/**
+ * THE blocking predicate for a review-filed finding -- the ONE place the daemon
+ * decides whether a review finding may hold its delivery (route it to
+ * verify_failed, target it with a gaps_found verdict, keep the review-cycle
+ * count climbing). Every such decision -- the review exit's evidence
+ * (review-exit-evidence.ts), the verify routing it tags for the reconciler
+ * (delivery-reconciler.ts `deferrable`), the review-cycle cap's "open blocking
+ * finding" (review-cycle-cap.ts, fed openBlockingFindings), delivery
+ * attribution (review-exit-attribution.ts) and foreign-finding location
+ * (review-exit-foreign-findings.ts) -- reads it; none reads priority directly.
+ *
+ *   - a RECORDED fact wins, whatever the priority: `blocking` blocks, `concern`
+ *     does not. A finding blocks by falsifying an agreed criterion, not by the
+ *     severity a reviewer typed.
+ *   - NO recorded fact -> today's priority rule: only an explicit P2/P3 is
+ *     non-blocking; P0/P1 and unset block (the bar is never lowered by absence).
+ *
+ * It classifies severity only; whether the issue is review-filed and open is
+ * the caller's filter (isReviewFiledIssue / isReviewFiledComplete).
+ */
+export function isBlockingReviewFinding(issue, findingClasses) {
+    const recorded = findingClasses.get(issue.id);
+    if (recorded !== undefined)
+        return recorded === 'blocking';
+    return !isLowPriority(issue.priority);
+}
+/**
+ * Whether an issue is a DEFERRABLE review note: a review-filed, still-open
+ * issue (per isReviewFiledIssue) that is NOT a blocking finding
+ * (isBlockingReviewFinding -- a recorded `concern`, or, with no recorded fact,
+ * an explicit P2/P3). A deferrable note must NOT route a verify delivery to
+ * verify_failed -- the review should converge (approve) and leave the note
+ * tracked.
+ *
+ * Any open work that is NOT review-filed (e.g. pre-existing coding work) is
+ * blocking. Deferrability is a property of review findings only.
+ */
+export function isDeferrableReviewNote(issue, reviewRequestedAt, findingClasses) {
+    return isReviewFiledIssue(issue, reviewRequestedAt) && !isBlockingReviewFinding(issue, findingClasses);
+}
+/**
+ * Filter a focus's issues to the open, review-filed notes that are deferrable
+ * (not blocking). The caller uses the ids to mark the matching reconciler
+ * issues as `deferrable` so the verify-routing reconciler does not treat them
+ * as blocking open work.
+ */
+export function filterDeferrableReviewNotes(issues, reviewRequestedAt, findingClasses) {
+    return issues.filter(i => isDeferrableReviewNote(i, reviewRequestedAt, findingClasses));
+}
+/**
+ * Returns true when there is at least one BLOCKING review-filed issue since the
+ * most recent review pass -- a review-filed, still-open issue that is a
+ * blocking finding (isBlockingReviewFinding). This is the gate for the
+ * review_missing_report notice: a review that filed ONLY deferrable notes
+ * converged cleanly and must not raise the "review skipped its outcome"
+ * escalation.
+ */
+export function blockingReviewFiledIssueExists(issues, reviewRequestedAt, findingClasses) {
+    return issues.some(i => isReviewFiledIssue(i, reviewRequestedAt) && isBlockingReviewFinding(i, findingClasses));
+}
+// ── Non-lossy (complete-set) classification ───────────────────────────────
+//
+// The functions above decide "was this issue filed by a review" from timing
+// alone (created_at >= review_requested_at). That is a lossy proxy on two axes:
+//   - TIME: review_requested_at is cleared at each review exit and re-stamped
+//     with a fresh now() at the next auto-review, so a note genuinely filed on
+//     pass 1 falls outside the window from pass 2 onward and reads as absent.
+//   - (the SCOPE axis is a review-exit-phase concern: reading focus-wide issues
+//     vs verify-scoped, handled at the call site.)
+//
+// The non-lossy source is `focus_reviews.created_issue_ids`, surfaced as the
+// focus-wide union `reviewFiledIds`. The `*Complete` variants below treat an
+// issue as review-filed when its id is in that set OR the legacy time-window
+// heuristic matches -- a strict SUPERSET of the old behaviour, so nothing the
+// window used to catch is lost, while the id-set makes the classification
+// stable across passes (the fix for the deferrable-note-re-blocked loop).
+/**
+ * Time-independent review-filed test: the issue is a member of the focus's
+ * review-filed id union AND is still open.
+ */
+export function isReviewFiledById(issue, reviewFiledIds) {
+    return reviewFiledIds.has(issue.id) && OPEN_ISSUE_STATUSES.has(issue.status);
+}
+/** Review-filed via the non-lossy id set OR the legacy review-window heuristic. */
+export function isReviewFiledComplete(issue, reviewFiledIds, reviewRequestedAt) {
+    return isReviewFiledById(issue, reviewFiledIds) || isReviewFiledIssue(issue, reviewRequestedAt);
+}
+/** Complete-set counterpart of reviewFiledIssueExists. */
+export function reviewFiledIssueExistsComplete(issues, reviewFiledIds, reviewRequestedAt) {
+    return issues.some(i => isReviewFiledComplete(i, reviewFiledIds, reviewRequestedAt));
+}
+/** Complete-set counterpart of blockingReviewFiledIssueExists. */
+export function blockingReviewFiledIssueExistsComplete(issues, reviewFiledIds, reviewRequestedAt, findingClasses) {
+    return issues.some(i => isReviewFiledComplete(i, reviewFiledIds, reviewRequestedAt) && isBlockingReviewFinding(i, findingClasses));
+}
+/**
+ * Complete-set counterpart of blockingReviewFiledIssueExists returning the LIST
+ * (not just existence): the OPEN, BLOCKING review-filed issues on the focus. Used
+ * by the hand-off breaker to ENUMERATE the outstanding findings in its escalation
+ * -- so an operator can tell whether the focus carries outstanding review
+ * findings without opening the daemon log, rather than reading the breaker's old
+ * untested "there was no work left to route" assertion.
+ */
+export function filterBlockingReviewFiledComplete(issues, reviewFiledIds, reviewRequestedAt, findingClasses) {
+    return issues.filter(i => isReviewFiledComplete(i, reviewFiledIds, reviewRequestedAt) && isBlockingReviewFinding(i, findingClasses));
+}
+/** Complete-set counterpart of filterDeferrableReviewNotes. */
+export function filterDeferrableReviewNotesComplete(issues, reviewFiledIds, reviewRequestedAt, findingClasses) {
+    return issues.filter(i => isReviewFiledComplete(i, reviewFiledIds, reviewRequestedAt) && !isBlockingReviewFinding(i, findingClasses));
+}
+//# sourceMappingURL=review-defect-detector.js.map

package/dist/completion/review-exit/review-defect-detector.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-defect-detector.js","sourceRoot":"","sources":["../../../src/completion/review-exit/review-defect-detector.ts"],"names":[],"mappings":"AAAA,8FAA8F;AAC9F,oFAAoF;AACpF,yFAAyF;AACzF,qFAAqF;AACrF,6FAA6F;AAC7F,yFAAyF;AACzF,gBAAgB;AAsBhB,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AAGpD,4EAA4E;AAE5E;;;GAGG;AACH,MAAM,CAAC,MAAM,wBAAwB,GACnC,aAAa,EAAE,CAAC,+BAA+B,CAAC;AAElD,wEAAwE;AAExE;;;;;GAKG;AACH,MAAM,iBAAiB,GAAG,IAAI,GAAG,EAAkB,CAAC;AAEpD,MAAM,UAAU,oBAAoB,CAAC,OAAe;IAClD,OAAO,iBAAiB,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;AAC7C,CAAC;AAED,MAAM,UAAU,0BAA0B,CAAC,OAAe;IACxD,MAAM,IAAI,GAAG,CAAC,iBAAiB,CAAC,GAAG,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC;IACvD,iBAAiB,CAAC,GAAG,CAAC,OAAO,EAAE,IAAI,CAAC,CAAC;IACrC,OAAO,IAAI,CAAC;AACd,CAAC;AAED,MAAM,UAAU,sBAAsB,CAAC,OAAe;IACpD,iBAAiB,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC;AACpC,CAAC;AAED,wCAAwC;AACxC,MAAM,UAAU,iCAAiC;IAC/C,iBAAiB,CAAC,KAAK,EAAE,CAAC;AAC5B,CAAC;AAED,2EAA2E;AAE3E;;;;;;;GAOG;AACH,MAAM,UAAU,kBAAkB,CAChC,KAAuE,EACvE,iBAAgC;IAEhC,IAAI,CAAC,KAAK,CAAC,eAAe;QAAE,OAAO,KAAK,CAAC;IACzC,IAAI,CAAC,mBAAmB,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC;QAAE,OAAO,KAAK,CAAC;IACzD,IAAI,CAAC,iBAAiB;QAAE,OAAO,KAAK,CAAC;IACrC,IAAI,CAAC,KAAK,CAAC,SAAS;QAAE,OAAO,KAAK,CAAC;IACnC,OAAO,IAAI,IAAI,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC,OAAO,EAAE,IAAI,IAAI,IAAI,CAAC,iBAAiB,CAAC,CAAC,OAAO,EAAE,CAAC;AACtF,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,uBAAuB,CACrC,MAAwB,EACxB,iBAAgC;IAEhC,OAAO,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,kBAAkB,CAAC,CAAC,EAAE,iBAAiB,CAAC,CAAC,CAAC;AACtE,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,sBAAsB,CACpC,MAAwB,EACxB,iBAAgC;IAEhC,IAAI,CAAC,iBAAiB;QAAE,OAAO,KAAK,CAAC;IACrC,KAAK,MAAM,CAAC,IAAI,MAAM,EAAE,CAAC;QACvB,IAAI,CAAC,CAAC,CAAC,eAAe;YAAE,SAAS;QACjC,IAAI,CAAC,CAAC,CAAC,SAAS;YAAE,SAAS;QAC3B,IAAI,IAAI,IAAI,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,OAAO,EAAE,IAAI,IAAI,IAAI,CAAC,iBAAiB,CAAC,CAAC,OAAO,EAAE,EAAE,CAAC;YAC7E,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED,6EAA6E;AAE7E;;;;;;;;GAQG;AACH,MAAM,qBAAqB,GAAG,IAAI,GAAG,CAAC,CAAC,IAAI,EAAE,IAAI,CAAC,CAAC,CAAC;AAEpD;;;;;;;;GAQG;AACH,MAAM,UAAU,aAAa,CAAC,QAAmC;IAC/D,IAAI,CAAC,QAAQ;QAAE,OAAO,KAAK,CAAC;IAC5B,OAAO,qBAAqB,CAAC,GAAG,CAAC,QAAQ,CAAC,IAAI,EAAE,CAAC,WAAW,EAAE,CAAC,CAAC;AAClE,CAAC;AASD,sFAAsF;AACtF,MAAM,CAAC,MAAM,mBAAmB,GAAyB,IAAI,GAAG,EAAE,CAAC;AAEnE;;;;;;;;;;;;;;;;;;;GAmBG;AACH,MAAM,UAAU,uBAAuB,CACrC,KAA8C,EAC9C,cAAoC;IAEpC,MAAM,QAAQ,GAAG,cAAc,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;IAC9C,IAAI,QAAQ,KAAK,SAAS;QAAE,OAAO,QAAQ,KAAK,UAAU,CAAC;IAC3D,OAAO,CAAC,aAAa,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;AACxC,CAAC;AAED;;;;;;;;;;GAUG;AACH,MAAM,UAAU,sBAAsB,CACpC,KAA2F,EAC3F,iBAAgC,EAChC,cAAoC;IAEpC,OAAO,kBAAkB,CAAC,KAAK,EAAE,iBAAiB,CAAC,IAAI,CAAC,uBAAuB,CAAC,KAAK,EAAE,cAAc,CAAC,CAAC;AACzG,CAAC;AAED;;;;;GAKG;AACH,MAAM,UAAU,2BAA2B,CACzC,MAAwB,EACxB,iBAAgC,EAChC,cAAoC;IAEpC,OAAO,MAAM,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,CAAC,sBAAsB,CAAC,CAAC,EAAE,iBAAiB,EAAE,cAAc,CAAC,CAAC,CAAC;AAC1F,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,8BAA8B,CAC5C,MAAwB,EACxB,iBAAgC,EAChC,cAAoC;IAEpC,OAAO,MAAM,CAAC,IAAI,CAChB,CAAC,CAAC,EAAE,CAAC,kBAAkB,CAAC,CAAC,EAAE,iBAAiB,CAAC,IAAI,uBAAuB,CAAC,CAAC,EAAE,cAAc,CAAC,CAC5F,CAAC;AACJ,CAAC;AAED,6EAA6E;AAC7E,EAAE;AACF,4EAA4E;AAC5E,gFAAgF;AAChF,8EAA8E;AAC9E,+EAA+E;AAC/E,8EAA8E;AAC9E,gFAAgF;AAChF,mDAAmD;AACnD,EAAE;AACF,6EAA6E;AAC7E,6EAA6E;AAC7E,6EAA6E;AAC7E,8EAA8E;AAC9E,0EAA0E;AAC1E,0EAA0E;AAE1E;;;GAGG;AACH,MAAM,UAAU,iBAAiB,CAC/B,KAA4C,EAC5C,cAAmC;IAEnC,OAAO,cAAc,CAAC,GAAG,CAAC,KAAK,CAAC,EAAE,CAAC,IAAI,mBAAmB,CAAC,GAAG,CAAC,KAAK,CAAC,MAAM,CAAC,CAAC;AAC/E,CAAC;AAED,mFAAmF;AACnF,MAAM,UAAU,qBAAqB,CACnC,KAA8E,EAC9E,cAAmC,EACnC,iBAAgC;IAEhC,OAAO,iBAAiB,CAAC,KAAK,EAAE,cAAc,CAAC,IAAI,kBAAkB,CAAC,KAAK,EAAE,iBAAiB,CAAC,CAAC;AAClG,CAAC;AAED,0DAA0D;AAC1D,MAAM,UAAU,8BAA8B,CAC5C,MAAwB,EACxB,cAAmC,EACnC,iBAAgC;IAEhC,OAAO,MAAM,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,qBAAqB,CAAC,CAAC,EAAE,cAAc,EAAE,iBAAiB,CAAC,CAAC,CAAC;AACvF,CAAC;AAED,kEAAkE;AAClE,MAAM,UAAU,sCAAsC,CACpD,MAAwB,EACxB,cAAmC,EACnC,iBAAgC,EAChC,cAAoC;IAEpC,OAAO,MAAM,CAAC,IAAI,CAChB,CAAC,CAAC,EAAE,CAAC,qBAAqB,CAAC,CAAC,EAAE,cAAc,EAAE,iBAAiB,CAAC,IAAI,uBAAuB,CAAC,CAAC,EAAE,cAAc,CAAC,CAC/G,CAAC;AACJ,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,iCAAiC,CAC/C,MAAwB,EACxB,cAAmC,EACnC,iBAAgC,EAChC,cAAoC;IAEpC,OAAO,MAAM,CAAC,MAAM,CAClB,CAAC,CAAC,EAAE,CAAC,qBAAqB,CAAC,CAAC,EAAE,cAAc,EAAE,iBAAiB,CAAC,IAAI,uBAAuB,CAAC,CAAC,EAAE,cAAc,CAAC,CAC/G,CAAC;AACJ,CAAC;AAED,+DAA+D;AAC/D,MAAM,UAAU,mCAAmC,CACjD,MAAwB,EACxB,cAAmC,EACnC,iBAAgC,EAChC,cAAoC;IAEpC,OAAO,MAAM,CAAC,MAAM,CAClB,CAAC,CAAC,EAAE,CAAC,qBAAqB,CAAC,CAAC,EAAE,cAAc,EAAE,iBAAiB,CAAC,IAAI,CAAC,uBAAuB,CAAC,CAAC,EAAE,cAAc,CAAC,CAChH,CAAC;AACJ,CAAC"}

package/dist/completion/review-exit/review-exit-attribution.d.ts

@@ -0,0 +1,38 @@
+/**
+ * Delivery attribution for a gaps_found review verdict.
+ *
+ * A focus-level gaps_found review is NOT an undifferentiated per-delivery
+ * instruction: it judges only the deliveries that own its filed gap issues
+ * (focus_reviews.created_issue_ids -> each issue's delivery). This module turns
+ * that session's created-issue ids into the verdict's TARGET SET and classifies,
+ * per delivery, whether the verdict targets it.
+ *
+ * Pure and dependency-free (beyond the issue classifier + open-status set) so the
+ * decision is unit-testable and the handler stays under the file-size limit.
+ * Sibling of review-exit-shortfall.ts / review-exit-escalations.ts, same seam.
+ *
+ * @module completion/review-exit/review-exit-attribution
+ */
+import type { FocusIssueInfo } from '../../types/index.js';
+import { type ReviewFindingClasses } from './review-defect-detector.js';
+/** How a gaps_found verdict relates to one delivery. See ReconcilerReviewState.gapsAttribution. */
+export type GapsAttribution = 'targeted' | 'not_targeted' | 'unresolvable';
+/**
+ * The deliveries a gaps_found verdict actually judges: those that own an OPEN,
+ * BLOCKING gap issue the review filed this session (created_issue_ids mapped to
+ * each issue's delivery). A closed or non-blocking note does not target its
+ * delivery -- only an actionable open gap does. Blocking is
+ * isBlockingReviewFinding over the findings' recorded classes (a recorded
+ * concern never targets, a recorded blocking always does; no fact -> priority).
+ */
+export declare function resolveAttributedDeliveryIds(focusIssues: FocusIssueInfo[], createdIssueIds: ReadonlySet<string>, findingClasses: ReviewFindingClasses): Set<string>;
+/**
+ * Classify one delivery against a gaps_found verdict:
+ *   'targeted'     -> this delivery owns a filed gap -> route back.
+ *   'not_targeted' -> a sibling owns the gap -> this delivery converges on merit.
+ *   'unresolvable' -> the verdict resolved to no delivery at all -> hold (absence
+ *                     of attribution is not a pass).
+ * Returns undefined when the verdict is not gaps_found (attribution is moot).
+ */
+export declare function resolveGapsAttribution(deliveryId: string, isGapsFound: boolean, attributedDeliveryIds: ReadonlySet<string>, attributionResolvable: boolean): GapsAttribution | undefined;
+//# sourceMappingURL=review-exit-attribution.d.ts.map

package/dist/completion/review-exit/review-exit-attribution.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-exit-attribution.d.ts","sourceRoot":"","sources":["../../../src/completion/review-exit/review-exit-attribution.ts"],"names":[],"mappings":"AAIA;;;;;;;;;;;;;;GAcG;AAEH,OAAO,KAAK,EAAE,cAAc,EAAE,MAAM,sBAAsB,CAAC;AAE3D,OAAO,EAA2B,KAAK,oBAAoB,EAAE,MAAM,6BAA6B,CAAC;AAEjG,mGAAmG;AACnG,MAAM,MAAM,eAAe,GAAG,UAAU,GAAG,cAAc,GAAG,cAAc,CAAC;AAE3E;;;;;;;GAOG;AACH,wBAAgB,4BAA4B,CAC1C,WAAW,EAAE,cAAc,EAAE,EAC7B,eAAe,EAAE,WAAW,CAAC,MAAM,CAAC,EACpC,cAAc,EAAE,oBAAoB,GACnC,GAAG,CAAC,MAAM,CAAC,CASb;AAED;;;;;;;GAOG;AACH,wBAAgB,sBAAsB,CACpC,UAAU,EAAE,MAAM,EAClB,WAAW,EAAE,OAAO,EACpB,qBAAqB,EAAE,WAAW,CAAC,MAAM,CAAC,EAC1C,qBAAqB,EAAE,OAAO,GAC7B,eAAe,GAAG,SAAS,CAI7B"}

package/dist/completion/review-exit/review-exit-attribution.js

@@ -0,0 +1,43 @@
+// PINNED-BY-BEGIN (generated by gen:pinned-by -- do not hand-edit; checked by lint:pinned-by)
+// Pinned by: packages/daemon/src/completion/review-exit/review-finding-blocking-predicate.test.ts
+// PINNED-BY-END
+import { OPEN_ISSUE_STATUSES } from '../../types/issue-status.js';
+import { isBlockingReviewFinding } from './review-defect-detector.js';
+/**
+ * The deliveries a gaps_found verdict actually judges: those that own an OPEN,
+ * BLOCKING gap issue the review filed this session (created_issue_ids mapped to
+ * each issue's delivery). A closed or non-blocking note does not target its
+ * delivery -- only an actionable open gap does. Blocking is
+ * isBlockingReviewFinding over the findings' recorded classes (a recorded
+ * concern never targets, a recorded blocking always does; no fact -> priority).
+ */
+export function resolveAttributedDeliveryIds(focusIssues, createdIssueIds, findingClasses) {
+    const attributed = new Set();
+    for (const iss of focusIssues) {
+        if (!createdIssueIds.has(iss.id))
+            continue;
+        if (!OPEN_ISSUE_STATUSES.has(iss.status))
+            continue; // an already-closed gap targets nothing
+        if (!isBlockingReviewFinding(iss, findingClasses))
+            continue; // non-blocking notes are not gaps
+        if (iss.deliveryId)
+            attributed.add(iss.deliveryId);
+    }
+    return attributed;
+}
+/**
+ * Classify one delivery against a gaps_found verdict:
+ *   'targeted'     -> this delivery owns a filed gap -> route back.
+ *   'not_targeted' -> a sibling owns the gap -> this delivery converges on merit.
+ *   'unresolvable' -> the verdict resolved to no delivery at all -> hold (absence
+ *                     of attribution is not a pass).
+ * Returns undefined when the verdict is not gaps_found (attribution is moot).
+ */
+export function resolveGapsAttribution(deliveryId, isGapsFound, attributedDeliveryIds, attributionResolvable) {
+    if (!isGapsFound)
+        return undefined;
+    if (!attributionResolvable)
+        return 'unresolvable';
+    return attributedDeliveryIds.has(deliveryId) ? 'targeted' : 'not_targeted';
+}
+//# sourceMappingURL=review-exit-attribution.js.map

package/dist/completion/review-exit/review-exit-attribution.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-exit-attribution.js","sourceRoot":"","sources":["../../../src/completion/review-exit/review-exit-attribution.ts"],"names":[],"mappings":"AAAA,8FAA8F;AAC9F,kGAAkG;AAClG,gBAAgB;AAmBhB,OAAO,EAAE,mBAAmB,EAAE,MAAM,6BAA6B,CAAC;AAClE,OAAO,EAAE,uBAAuB,EAA6B,MAAM,6BAA6B,CAAC;AAKjG;;;;;;;GAOG;AACH,MAAM,UAAU,4BAA4B,CAC1C,WAA6B,EAC7B,eAAoC,EACpC,cAAoC;IAEpC,MAAM,UAAU,GAAG,IAAI,GAAG,EAAU,CAAC;IACrC,KAAK,MAAM,GAAG,IAAI,WAAW,EAAE,CAAC;QAC9B,IAAI,CAAC,eAAe,CAAC,GAAG,CAAC,GAAG,CAAC,EAAE,CAAC;YAAE,SAAS;QAC3C,IAAI,CAAC,mBAAmB,CAAC,GAAG,CAAC,GAAG,CAAC,MAAM,CAAC;YAAE,SAAS,CAAC,wCAAwC;QAC5F,IAAI,CAAC,uBAAuB,CAAC,GAAG,EAAE,cAAc,CAAC;YAAE,SAAS,CAAC,kCAAkC;QAC/F,IAAI,GAAG,CAAC,UAAU;YAAE,UAAU,CAAC,GAAG,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC;IACrD,CAAC;IACD,OAAO,UAAU,CAAC;AACpB,CAAC;AAED;;;;;;;GAOG;AACH,MAAM,UAAU,sBAAsB,CACpC,UAAkB,EAClB,WAAoB,EACpB,qBAA0C,EAC1C,qBAA8B;IAE9B,IAAI,CAAC,WAAW;QAAE,OAAO,SAAS,CAAC;IACnC,IAAI,CAAC,qBAAqB;QAAE,OAAO,cAAc,CAAC;IAClD,OAAO,qBAAqB,CAAC,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,cAAc,CAAC;AAC7E,CAAC"}

package/dist/completion/review-exit/review-exit-deps.d.ts

@@ -0,0 +1,93 @@
+/**
+ * Dependency contract for the review-exit phase.
+ *
+ * review-exit-phase.ts holds the review-exit routing decision tree and takes
+ * this context as a REQUIRED parameter. The production binding -- which real
+ * query/directive/loop implementation each member is -- is built once, at the
+ * daemon composition root (`composition/review-exit-context.ts`), never here and
+ * never at a call site.
+ *
+ * History: docs/daemon/history-team-lifecycle.md (completion/review-exit/review-exit-deps.ts).
+ */
+import type { fetchEffectiveWorkflow } from '../../queries/workflow-transitions/workflows.js';
+import type { getIssueDelivery } from '../../queries/deliveries/deliveries.js';
+import type { AutoApproveDelivery } from '../team-outcome/delivery-advance.js';
+import type { DeliveryStatusWriter } from '../team-outcome/delivery-advance-types.js';
+import type { ReviewWindowCommit } from '../../git/review-window-commits.js';
+import type { getActiveFocuses } from '../../queries/focuses/focus-discovery.js';
+import type { hasPendingSpawnDirective } from '../../directive/directive-queue.js';
+import type { emitLoopTrigger } from '../../loops/index.js';
+import type { getFocusDeliveries, getFocusIssues, clearReviewRequestedAt, releaseFocusArm, setReviewCycleCount, setReviewHandoffCount, getFocusReviewReportForSession } from '../../queries/focuses/focuses.js';
+import type { getFocusReviewFindingFacts } from '../../queries/focus-review/review-finding-facts.js';
+import type { resolveReviewCycleCap, resolveReviewHandoffCap } from '../../supervision/stall-detector-config.js';
+import type { getDeliveryIssues, rehomeIssueToDelivery } from '../../queries/deliveries/issues.js';
+import type { hasOpenFocusEscalation, hasOpenEscalationForDelivery } from '../../queries/escalations/escalations.js';
+import type { createEscalation } from '../../supervision/escalation-filing.js';
+import type { findOpenOperatorHoldForDelivery } from '../../supervision/holds.js';
+import type { OpenShortfallProposalLister } from '../../drift/open-shortfall-proposals.js';
+import type { FailedGateVerdict } from '../review-exit-holds/review-exit-verdict-gate.js';
+import type { CommitGateSeams } from '../../verify/gate-config/commit-gate-configured.js';
+import type { ReviewAutoApproval } from '../../merge/review-auto-approval-record.js';
+import type { ReviewCensus } from './review-census-record.js';
+/**
+ * Dependencies for runReviewExitHandler. Every member is required: production
+ * wires them at the composition root, and a test names every collaborator it
+ * runs against (testing/fixtures/review-exit-deps.ts builds an inert set).
+ */
+export interface ReviewExitDeps {
+    getActiveFocuses: typeof getActiveFocuses;
+    getFocusDeliveries: typeof getFocusDeliveries;
+    getFocusIssues: typeof getFocusIssues;
+    getDeliveryIssues: typeof getDeliveryIssues;
+    fetchEffectiveWorkflow: typeof fetchEffectiveWorkflow;
+    updateDeliveryStatus: DeliveryStatusWriter;
+    markDeliveryAutoApproved: AutoApproveDelivery;
+    recordReviewAutoApproval: (approval: ReviewAutoApproval) => boolean;
+    recordReviewCensus: (census: ReviewCensus) => boolean;
+    clearReviewRequestedAt: typeof clearReviewRequestedAt;
+    fileReworkForFailedVerdict: (focusId: string, verdict: FailedGateVerdict) => Promise<number>;
+    clearPendingReviewDirective: (focusId: string) => boolean;
+    /**
+     * Record that this review pass covered a commit. The PRODUCTION binding is
+     * `stampReviewCoverage` (review-coverage-shas.ts), not the bare query: it
+     * writes the watermark AND the delivery merge commits git proves that commit
+     * covers, in one call.
+     *
+     * Declared at the ARITY THE PHASE USES (focusId, sha) rather than as
+     * `typeof setReviewedThroughSha`: what the exit knows is "this review covered
+     * this commit", and how much of the range the binding manages to record with
+     * it is the binding's business, not a decision the routing tree makes. Both
+     * the bare query and the coverage stamp satisfy it, as does a two-argument
+     * test stub.
+     */
+    setReviewedThroughSha: (focusId: string, sha: string) => Promise<void>;
+    setReviewCycleCount: typeof setReviewCycleCount;
+    resolveReviewCycleCap: typeof resolveReviewCycleCap;
+    releaseFocusArm: typeof releaseFocusArm;
+    isShuttingDown: () => boolean;
+    setReviewHandoffCount: typeof setReviewHandoffCount;
+    resolveReviewHandoffCap: typeof resolveReviewHandoffCap;
+    resolveHandoffExhaustedHold: (focusId: string, notes: string) => Promise<number>;
+    resolveConvergedFocusEscalations: (focusId: string, notes: string) => Promise<number>;
+    hasOpenFocusEscalation: typeof hasOpenFocusEscalation;
+    listOpenShortfallProposalIds: OpenShortfallProposalLister;
+    findOpenOperatorHoldForDelivery: typeof findOpenOperatorHoldForDelivery;
+    resolveCommitGate: CommitGateSeams['resolveCommitGate'];
+    resolveJudgedCommit: CommitGateSeams['resolveJudgedCommit'];
+    readCommitVerdict: CommitGateSeams['readCommitVerdict'];
+    readDeliveryGateOutcome: CommitGateSeams['readDeliveryGateOutcome'];
+    hasPendingGateRun: CommitGateSeams['hasPendingGateRun'];
+    rehomeIssueToDelivery: typeof rehomeIssueToDelivery;
+    listReviewWindowCommits: (focusId: string, base: string, tip: string) => readonly ReviewWindowCommit[] | null;
+    blameLines: (focusId: string, tip: string, file: string, start: number, end: number) => readonly string[] | null;
+    resolveDeliveryFocusId: (deliveryId: string) => Promise<string | null>;
+    getIssueDelivery: typeof getIssueDelivery;
+    hasOpenEscalationForDelivery: typeof hasOpenEscalationForDelivery;
+    hasPendingSpawnDirective: typeof hasPendingSpawnDirective;
+    emitLoopTrigger: typeof emitLoopTrigger;
+    getFocusReviewReportForSession: typeof getFocusReviewReportForSession;
+    getFocusReviewFindingFacts: typeof getFocusReviewFindingFacts;
+    createEscalation: typeof createEscalation;
+    now: () => string;
+}
+//# sourceMappingURL=review-exit-deps.d.ts.map

package/dist/completion/review-exit/review-exit-deps.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-exit-deps.d.ts","sourceRoot":"","sources":["../../../src/completion/review-exit/review-exit-deps.ts"],"names":[],"mappings":"AAIA;;;;;;;;;;GAUG;AAEH,OAAO,KAAK,EAAE,sBAAsB,EAAE,MAAM,iDAAiD,CAAC;AAC9F,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAC/E,OAAO,KAAK,EAAE,mBAAmB,EAAE,MAAM,qCAAqC,CAAC;AAC/E,OAAO,KAAK,EAAE,oBAAoB,EAAE,MAAM,2CAA2C,CAAC;AACtF,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,oCAAoC,CAAC;AAC7E,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,0CAA0C,CAAC;AACjF,OAAO,KAAK,EAAE,wBAAwB,EAAE,MAAM,oCAAoC,CAAC;AACnF,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,sBAAsB,CAAC;AAC5D,OAAO,KAAK,EAAE,kBAAkB,EAAE,cAAc,EAAE,sBAAsB,EAAE,eAAe,EAAE,mBAAmB,EAAE,qBAAqB,EAAE,8BAA8B,EAAE,MAAM,kCAAkC,CAAC;AAChN,OAAO,KAAK,EAAE,0BAA0B,EAAE,MAAM,oDAAoD,CAAC;AACrG,OAAO,KAAK,EAAE,qBAAqB,EAAE,uBAAuB,EAAE,MAAM,4CAA4C,CAAC;AACjH,OAAO,KAAK,EAAE,iBAAiB,EAAE,qBAAqB,EAAE,MAAM,oCAAoC,CAAC;AACnG,OAAO,KAAK,EAAE,sBAAsB,EAAE,4BAA4B,EAAE,MAAM,0CAA0C,CAAC;AACrH,OAAO,KAAK,EAAE,gBAAgB,EAAE,MAAM,wCAAwC,CAAC;AAC/E,OAAO,KAAK,EAAE,+BAA+B,EAAE,MAAM,4BAA4B,CAAC;AAClF,OAAO,KAAK,EAAE,2BAA2B,EAAE,MAAM,yCAAyC,CAAC;AAC3F,OAAO,KAAK,EAAE,iBAAiB,EAAE,MAAM,kDAAkD,CAAC;AAC1F,OAAO,KAAK,EAAE,eAAe,EAAE,MAAM,oDAAoD,CAAC;AAC1F,OAAO,KAAK,EAAE,kBAAkB,EAAE,MAAM,4CAA4C,CAAC;AACrF,OAAO,KAAK,EAAE,YAAY,EAAE,MAAM,2BAA2B,CAAC;AAE9D;;;;GAIG;AACH,MAAM,WAAW,cAAc;IAC7B,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAC1C,kBAAkB,EAAE,OAAO,kBAAkB,CAAC;IAC9C,cAAc,EAAE,OAAO,cAAc,CAAC;IACtC,iBAAiB,EAAE,OAAO,iBAAiB,CAAC;IAC5C,sBAAsB,EAAE,OAAO,sBAAsB,CAAC;IAGtD,oBAAoB,EAAE,oBAAoB,CAAC;IAC3C,wBAAwB,EAAE,mBAAmB,CAAC;IAK9C,wBAAwB,EAAE,CAAC,QAAQ,EAAE,kBAAkB,KAAK,OAAO,CAAC;IAGpE,kBAAkB,EAAE,CAAC,MAAM,EAAE,YAAY,KAAK,OAAO,CAAC;IACtD,sBAAsB,EAAE,OAAO,sBAAsB,CAAC;IAGtD,0BAA0B,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,OAAO,EAAE,iBAAiB,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IAE7F,2BAA2B,EAAE,CAAC,OAAO,EAAE,MAAM,KAAK,OAAO,CAAC;IAC1D;;;;;;;;;;;;OAYG;IACH,qBAAqB,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,KAAK,OAAO,CAAC,IAAI,CAAC,CAAC;IAEvE,mBAAmB,EAAE,OAAO,mBAAmB,CAAC;IAChD,qBAAqB,EAAE,OAAO,qBAAqB,CAAC;IAGpD,eAAe,EAAE,OAAO,eAAe,CAAC;IAIxC,cAAc,EAAE,MAAM,OAAO,CAAC;IAE9B,qBAAqB,EAAE,OAAO,qBAAqB,CAAC;IACpD,uBAAuB,EAAE,OAAO,uBAAuB,CAAC;IAIxD,2BAA2B,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IAIjF,gCAAgC,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,CAAC,CAAC;IAEtF,sBAAsB,EAAE,OAAO,sBAAsB,CAAC;IAEtD,4BAA4B,EAAE,2BAA2B,CAAC;IAG1D,+BAA+B,EAAE,OAAO,+BAA+B,CAAC;IAKxE,iBAAiB,EAAE,eAAe,CAAC,mBAAmB,CAAC,CAAC;IACxD,mBAAmB,EAAE,eAAe,CAAC,qBAAqB,CAAC,CAAC;IAC5D,iBAAiB,EAAE,eAAe,CAAC,mBAAmB,CAAC,CAAC;IAExD,uBAAuB,EAAE,eAAe,CAAC,yBAAyB,CAAC,CAAC;IAGpE,iBAAiB,EAAE,eAAe,CAAC,mBAAmB,CAAC,CAAC;IAGxD,qBAAqB,EAAE,OAAO,qBAAqB,CAAC;IAQpD,uBAAuB,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,KAAK,SAAS,kBAAkB,EAAE,GAAG,IAAI,CAAC;IAC9G,UAAU,EAAE,CAAC,OAAO,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,EAAE,IAAI,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAAE,GAAG,EAAE,MAAM,KAAK,SAAS,MAAM,EAAE,GAAG,IAAI,CAAC;IACjH,sBAAsB,EAAE,CAAC,UAAU,EAAE,MAAM,KAAK,OAAO,CAAC,MAAM,GAAG,IAAI,CAAC,CAAC;IACvE,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAC1C,4BAA4B,EAAE,OAAO,4BAA4B,CAAC;IAClE,wBAAwB,EAAE,OAAO,wBAAwB,CAAC;IAC1D,eAAe,EAAE,OAAO,eAAe,CAAC;IACxC,8BAA8B,EAAE,OAAO,8BAA8B,CAAC;IAKtE,0BAA0B,EAAE,OAAO,0BAA0B,CAAC;IAC9D,gBAAgB,EAAE,OAAO,gBAAgB,CAAC;IAE1C,GAAG,EAAE,MAAM,MAAM,CAAC;CACnB"}

package/dist/completion/review-exit/review-exit-deps.js

@@ -0,0 +1,5 @@
+// PINNED-BY-BEGIN (generated by gen:pinned-by -- do not hand-edit; checked by lint:pinned-by)
+// Pinned by: 25 tests -- run: npm run gen:pinned-by -- --list packages/daemon/src/completion/review-exit/review-exit-deps.ts
+// PINNED-BY-END
+export {};
+//# sourceMappingURL=review-exit-deps.js.map

package/dist/completion/review-exit/review-exit-deps.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-exit-deps.js","sourceRoot":"","sources":["../../../src/completion/review-exit/review-exit-deps.ts"],"names":[],"mappings":"AAAA,8FAA8F;AAC9F,6HAA6H;AAC7H,gBAAgB"}

package/dist/completion/review-exit/review-exit-escalations.d.ts

@@ -0,0 +1,200 @@
+/**
+ * Escalation tail of the review-exit gate.
+ *
+ * Every notice in this module fires AFTER routing has already been decided --
+ * none of them gate a delivery transition. The delivery has moved toward intent
+ * (or been held) by the time any of these file; what they carry is the residue
+ * the routing decider or an agent reconciles: a hand-off that keeps routing
+ * nothing, a review whose coverage keeps falling behind the branch, a gaps_found
+ * verdict with nothing actionable filed, an agent that skipped review_complete.
+ * (The review-cycle cap, which DOES stop the loop, lives in review-cycle-cap.ts
+ * and files through fileDedupedFocusEscalation here.)
+ *
+ * Kept apart from review-exit-phase.ts to hold that file under the daemon
+ * file-size limit -- sibling of review-exit-shortfall.ts, same seam. The filers
+ * share one deduped filer, `fileDedupedFocusEscalation`.
+ *
+ * History: docs/daemon/history-team-lifecycle.md (completion/review-exit/review-exit-escalations.ts).
+ *
+ * @module completion/review-exit/review-exit-escalations
+ */
+import { ESCALATION_REASONS } from '@telora/daemon-core';
+import type { hasOpenFocusEscalation } from '../../queries/escalations/escalations.js';
+import type { createEscalation } from '../../supervision/escalation-filing.js';
+import type { clearReviewRequestedAt, setReviewHandoffCount } from '../../queries/focuses/focuses.js';
+import type { resolveReviewHandoffCap } from '../../supervision/stall-detector-config.js';
+import { type HumanAct } from '../../queries/escalations/filing-observation.js';
+/** The slice of ReviewExitDeps this module needs (structural, so no import cycle). */
+export interface ReviewExitEscalationDeps {
+    createEscalation: typeof createEscalation;
+    hasOpenFocusEscalation: typeof hasOpenFocusEscalation;
+}
+/** Focus/session anchor shared by every review-exit escalation. */
+export interface ReviewExitEscalationContext {
+    organizationId: string;
+    focusId: string;
+    focusName: string;
+    sessionId: string;
+    /**
+     * The focus-branch head this review pass covered (reviewSpawnTip, else the exit
+     * tip), or null when unknown. The hand-off breaker records it as the head it
+     * crossed on; the hold ends when the focus branch moves past it
+     * (review-breaker-hold.ts). Optional: a context without it records no head.
+     */
+    coveredSha?: string | null;
+}
+/**
+ * A compact summary of one open, blocking, review-filed finding on a focus,
+ * enumerated in the hand-off-exhausted escalation and the review-cycle cap
+ * observation so whoever routes them sees the outstanding findings without
+ * opening the daemon log.
+ */
+export interface ReviewFiledFindingSummary {
+    /** The issue id -- carried so the re-homing pass can move the issue. */
+    id: string;
+    key: string;
+    title: string;
+    deliveryId: string | null;
+    /** The issue's current description -- preserved when the issue is re-homed. */
+    description: string | null;
+}
+/**
+ * File a focus-anchored needs_operator escalation, at most one open per
+ * (focus, kind). Each caller decides `humanAct`: a named act only a human can
+ * perform reaches the operator; NO_HUMAN_ACT files an observation the focus's
+ * next agent session decides (supervision/filing-route.ts).
+ *
+ * Best-effort throughout: a dedup-probe failure files anyway (a duplicate
+ * notice is cheaper than a swallowed one), and a create failure is logged, not
+ * thrown -- none of these notices may abort an exit handler that has already
+ * written delivery state. Returns whether a row of the kind is now open (filed,
+ * or already open): the review-cycle cap (review-cycle-cap.ts) resets its count
+ * only once its observation exists.
+ *
+ * `label` names the site in the warn lines so the pre-extraction log text
+ * survives verbatim.
+ */
+export declare function fileDedupedFocusEscalation(deps: ReviewExitEscalationDeps, ctx: ReviewExitEscalationContext, args: {
+    escalationKind: string;
+    label: string;
+    description: string;
+    metadata: Record<string, unknown>;
+    /** Required so every site states its routing: a named act, or NO_HUMAN_ACT. */
+    humanAct: HumanAct | null;
+    reasonType?: typeof ESCALATION_REASONS[keyof typeof ESCALATION_REASONS];
+    onFiled?: () => void;
+}): Promise<boolean>;
+/**
+ * Drift persisted after the full-pass respawn -- an operator must look.
+ *
+ * Raised only on the SECOND consecutive below-coverage clean exit: the focus
+ * branch advanced DURING each review pass, so a clean review never covered the
+ * exact state it would have approved.
+ */
+export declare function fileCoverageIncompleteEscalation(deps: ReviewExitEscalationDeps, ctx: ReviewExitEscalationContext, args: {
+    coveredSha: string | null;
+    currentBranchTip: string | null;
+    streak: number;
+}): Promise<void>;
+/** Deps for the bounded hand-off decision: the escalation deps plus the two focus writers it drives. */
+export interface HandoffBoundDeps extends ReviewExitEscalationDeps {
+    clearReviewRequestedAt: typeof clearReviewRequestedAt;
+    setReviewHandoffCount: typeof setReviewHandoffCount;
+    resolveReviewHandoffCap: typeof resolveReviewHandoffCap;
+    /**
+     * Resolve any open review_handoff_exhausted escalation for the focus -- the
+     * durable breaker HOLD. Called on a real delta so a focus that resumes progress
+     * lifts its own hold (returns the count resolved). The state cascade also ends
+     * it when the focus branch moves past the head it crossed on
+     * (review-breaker-hold.ts), and a resolve still clears it.
+     */
+    resolveHandoffExhaustedHold: (focusId: string, notes: string) => Promise<number>;
+}
+/**
+ * Bound the review-exit hand-off. Called ONLY when a review exit routed nothing
+ * AND a spawn directive is pending -- the branch that used to leave
+ * review_requested_at set unconditionally "for the incoming team", re-making an
+ * unverified promise forever.
+ *
+ * Counts consecutive routes-nothing hand-offs in a DURABLE focus counter (so the
+ * bound survives a daemon restart, which would reset an in-memory counter and
+ * make the loop look fresh each cycle). Under the cap: record the incremented
+ * count and leave review_requested_at set (the legitimate hand-off). AT the cap:
+ * stop re-asserting -- close the review window (clear review_requested_at,
+ * removing the precondition that re-fires the branch) and file ONE
+ * review_handoff_exhausted escalation instead, so a review exit can no longer
+ * re-create the condition that produced it. All writes best-effort.
+ */
+export declare function recordHandoffAndMaybeTerminate(deps: HandoffBoundDeps, ctx: ReviewExitEscalationContext, currentHandoffCount: number, opts?: {
+    /**
+     * What produced this no-progress cycle, for accurate logging. Two callers
+     * share the ONE durable counter (both are no-progress review exits):
+     *   - 'pending_spawn_handoff' (default): nothing routed, a spawn directive is
+     *     pending, so the flag is left set "for the incoming team".
+     *   - 'no_delta_cycle': a status was routed but focus state did not change
+     *     (the empty verify->verify_failed round trip).
+     */
+    trigger?: 'pending_spawn_handoff' | 'no_delta_cycle';
+    /**
+     * The EXITING session's type. The hand-off breaker measures ONLY review-pass
+     * exits: a coding pass cannot approve a review, so its routes-nothing exit
+     * was never a routing candidate and must not advance the cap (2026-08-27:
+     * two CODING exits exhausted the cap without one review exit having failed,
+     * then the breaker filed an escalation asserting the focus was done while a
+     * P1 finding was open). When present and NOT 'review', this is a no-op --
+     * the counter does not advance and no escalation is filed. Absent preserves
+     * the legacy count-any-exit behavior for callers/tests that predate the gate.
+     */
+    sessionType?: 'coding' | 'review';
+    /**
+     * Open, blocking, review-filed findings on the focus, resolved by the
+     * handler. Enumerated in the exhausted escalation (and its metadata) so the
+     * escalation states what it MEASURED instead of asserting "nothing to route".
+     */
+    openBlockingFindings?: ReviewFiledFindingSummary[];
+}): Promise<void>;
+/**
+ * Pure: who owns an exhausted review hand-off, decided from what was MEASURED.
+ *
+ * Open blocking findings -> NO_HUMAN_ACT: routing or re-homing them onto a
+ * delivery is Telora planning work an agent session performs. Zero findings ->
+ * the focus reads as done, and closing it is the completion act humans own (the
+ * breaker hold is kind-keyed, so either route keeps the review window held).
+ */
+export declare function handoffExhaustedHumanAct(focusName: string, openBlockingFindingCount: number): HumanAct | null;
+/**
+ * The review hand-off has terminated. The review-exit hand-off branch re-asserts
+ * an unverified promise -- "the incoming team will route it" -- and re-makes it
+ * unchanged when it fails. After TELORA_REVIEW_HANDOFF_CAP consecutive
+ * routes-nothing hand-offs the daemon stops re-asserting: it closes the review
+ * window (clears review_requested_at, removing the precondition that re-fires the
+ * branch) and files THIS escalation instead. One open row per focus.
+ */
+export declare function fileHandoffExhaustedEscalation(deps: ReviewExitEscalationDeps, ctx: ReviewExitEscalationContext, args: {
+    handoffCount: number;
+    cap: number;
+    trigger?: 'pending_spawn_handoff' | 'no_delta_cycle';
+    openBlockingFindings?: ReviewFiledFindingSummary[];
+}): Promise<void>;
+/**
+ * The routing inconsistency the review-exit gate exists to catch: the review
+ * stamped a gaps_found VERDICT but no verify delivery carried blocking open
+ * work, so the coding team has nothing concrete to close.
+ *
+ * The verdict has already routed the delivery to verify_failed (toward intent,
+ * never toward done) by the time this files. Mutually exclusive with the
+ * missing-report notice by construction: gaps_found implies reportExists=true,
+ * and that notice requires !reportExists.
+ */
+export declare function fileGapsWithoutIssueEscalation(deps: ReviewExitEscalationDeps, ctx: ReviewExitEscalationContext): Promise<void>;
+/**
+ * Surface that the agent skipped review_complete even though the loop healed.
+ *
+ * Fires only when routing happened via BLOCKING issue evidence: a review that
+ * filed only deferrable low-severity notes converged cleanly (auto-approve), so
+ * it neither re-coded nor "skipped" a needed outcome -- raising the notice there
+ * would be operator noise. Deliberately NOT deduped: it is a per-exit hygiene
+ * notice about one session's behavior, not a standing focus condition.
+ */
+export declare function fileMissingReportNotice(deps: ReviewExitEscalationDeps, ctx: ReviewExitEscalationContext): Promise<void>;
+//# sourceMappingURL=review-exit-escalations.d.ts.map