@mmnto/totem 1.73.2 → 1.75.0

package/dist/spine/extract.js

@@ -33,38 +33,123 @@
 // lesson-markdown is the DSL *syntax* (ADR-058 Pipeline 1/3 target), NOT a
 // Pipeline-1 trust class: every draft body is `unverified` and Stage-4-gated by
 // the slice-4 compiler, never a manual-rule trust bypass.
+import { z } from 'zod';
 import { ProvenanceRecordSchema } from '../compiler-schema.js';
 import { TotemParseError } from '../errors.js';
 import { extractManualPattern } from '../lesson-pattern.js';
-import { isBotIdentity } from './selection-rule.js';
+import { NoDraftCauseSchema } from './ledgers.js';
+import { isBotIdentity, reviewBotIdentity } from './selection-rule.js';
+/**
+ * The `DraftExtractor` port's return: the zero-or-more draft bodies PLUS, when the
+ * list is empty, WHY (`noDraftCause`). The cause is the extract-stage twin of the
+ * classifier's `dispositionSource` (a non-FM Tenet-19 diagnostic) — a bare `[]`
+ * conflated ≥6 causes (model declined / parser rejected a valid draft / transient
+ * invoke failure) the funnel could not tell apart. INVARIANT (refined): a cause is
+ * present IFF `drafts` is empty — a non-empty result carries drafts and no cause;
+ * an empty result MUST name its cause. Parsed at the core boundary so a
+ * contract-violating port (cause-without-empty, or empty-without-cause) fails loud
+ * before the drop ledger, exactly as `ClassifierResultSchema.parse` guards classify.
+ */
+export const DraftResultSchema = z
+    .object({
+    drafts: z.array(z.string()),
+    noDraftCause: NoDraftCauseSchema.optional(),
+})
+    .refine((r) => (r.drafts.length === 0) === (r.noDraftCause !== undefined), {
+    message: 'noDraftCause must be present iff drafts is empty (the extract-stage diagnostic invariant)',
+    path: ['noDraftCause'],
+});
 // ── Helpers ──────────────────────────────────────────────────────────────────
 /**
- * Count HUMAN review comments (fold 5): bot comments (CodeRabbit / Greptile /
- * Renovate / dependabot, via the shared `isBotIdentity`) and empty/whitespace
- * bodies do NOT count toward §6's "≥1 review comment" threshold — a bot-only or
- * empty thread is content-thin and must take the loud-drop path, never seed a
- * hallucinated draft.
+ * Classify a comment author (slice β, strategy#709). `'bot'` iff it is a recognized
+ * review-FINDING bot (`reviewBotIdentity` allowlist — gemini/CR); `'human'`
+ * otherwise. The SINGLE classification home: the CLI mapping boundary stamps each
+ * comment's `authorKind` via this, and the count + source-tag read that field.
+ */
+export function classifyAuthorKind(author) {
+    return reviewBotIdentity(author) ? 'bot' : 'human';
+}
+/**
+ * Is this comment SUBSTANTIVE mining substrate (slice β)? Counts toward §6's
+ * "≥1 review comment" completeness threshold iff its body is non-empty AND it is
+ * either a recognized review-finding bot (`authorKind === 'bot'`) OR a human.
+ *
+ * The slice-β substrate flip (panel OQ-β2, ALLOWLIST): for this bot-reviewed cert
+ * corpus, gemini/CR review comments ARE legitimate substrate — they count. But an
+ * UNRECOGNIZED `[bot]` automation account (renovate / dependabot) is still excluded
+ * via the `isBotIdentity` denylist, so future automation noise can never launder
+ * itself in as a substantive reviewer. Empty/whitespace bodies never count.
+ */
+function isSubstantiveComment(comment) {
+    // Gate on what the extractor ACTUALLY consumes (greptile #2242): for a review
+    // bot that is the de-chromed `normalizedBody`, so a badge-ONLY comment (non-empty
+    // raw `body`, but `normalizedBody === ''` after the strip) is correctly thin —
+    // it would otherwise clear the gate yet hand the extractor an empty body and
+    // mislead a `no-draft` drop where `truncated` is the truth. Human bodies are
+    // never chrome-stripped, so `normalizedBody === body` for them.
+    const effectiveBody = comment.authorKind === 'bot' ? comment.normalizedBody : comment.body;
+    if (effectiveBody.trim().length === 0)
+        return false;
+    if (comment.authorKind === 'bot')
+        return true;
+    return !isBotIdentity(comment.author);
+}
+/**
+ * Count SUBSTANTIVE review comments across threads (slice β — replaces the old
+ * `humanCommentCount`). A thread set with zero substantive comments is content-thin
+ * and must take the loud-drop path, never seed a hallucinated draft.
  */
-function humanCommentCount(threads) {
+function substantiveCommentCount(threads) {
     let count = 0;
     for (const thread of threads) {
         for (const comment of thread.comments) {
-            if (comment.body.trim().length > 0 && !isBotIdentity(comment.author))
+            if (isSubstantiveComment(comment))
                 count++;
         }
     }
     return count;
 }
 /**
- * The resolution-eligibility gate (slice 5a, mmnto-ai/totem#2201). A thread is
- * INELIGIBLE if the author resolved it OR its diff hunk went outdated — either
- * marks it as superseded review discussion, contamination the miner must not
- * draft from. The adapter SURFACES `isResolved`/`isOutdated` (it never
- * pre-filters); core decides here so the rejection is ledgered (§8). Returns the
+ * The substrate provenance of a thread set (slice β, panel OQ-β4): `human` /
+ * `bot` / `mixed` over its SUBSTANTIVE comments only (recognized review-bot vs
+ * human; unrecognized-bot noise is excluded, exactly as the count excludes it).
+ * Called only when ≥1 substantive comment survives the gate, so at least one axis
+ * is non-zero. A non-FM Tenet-19 diagnostic carried onto each draft + the
+ * zero-draft drop.
+ */
+function computeSourceKind(threads) {
+    let human = 0;
+    let bot = 0;
+    for (const thread of threads) {
+        for (const comment of thread.comments) {
+            if (!isSubstantiveComment(comment))
+                continue;
+            if (comment.authorKind === 'bot')
+                bot++;
+            else
+                human++;
+        }
+    }
+    if (bot > 0 && human > 0)
+        return 'mixed';
+    return bot > 0 ? 'bot' : 'human';
+}
+/**
+ * The eligibility gate. Slice γ (strategy#709) NARROWS this from slice-5a's
+ * `!isResolved && !isOutdated` to `!isOutdated` — RESOLVED threads are now ADMITTED.
+ *
+ * The slice-5a rationale (resolved == superseded contamination) was REVERSED by the
+ * Gate-1 cert finding: a RESOLVED thread is the highest-signal LEGITIMACY marker —
+ * a defect a reviewer raised AND the author confirmed real by fixing — so excluding
+ * it discarded the very evidence the miner wants (exhibit: lc#532's fail-open-on-
+ * non-finite, dropped solely for being resolved). Only OUTDATED stays excluded: an
+ * outdated thread's diff hunk no longer matches HEAD, so its invariant may have been
+ * refactored away — that IS stale. The adapter SURFACES both flags (it never
+ * pre-filters); core decides here so every rejection is ledgered (§8). Returns the
  * eligible (surviving) threads only.
  */
 function eligibleThreads(threads) {
-    return threads.filter((t) => !t.isResolved && !t.isOutdated);
+    return threads.filter((t) => !t.isOutdated);
 }
 /**
  * Syntactic preflight (fold 4): a draft is a usable lesson-markdown DSL body iff
@@ -120,21 +205,27 @@ function buildProvenance(pr, content) {
  * CI-locked with a fixture extractor + a strict-spy fetch source.
  *
  * Per train PR (and ONLY train PRs): log the fetch → fetch → on unreachable /
- * unparseable-at-source, loud-drop → resolution-eligibility gate (slice 5a: drop
- * `resolved-rejected` when the resolution gate empties an otherwise-human thread,
- * else `truncated` when thin to begin with) → completeness-check (≥1 human
- * comment on the survivors) → build provenance → draft zero-or-more bodies from
- * the SURVIVING threads only → preflight each → carry a `DraftCandidate` or
- * loud-drop. Every train PR ends with at least one draft or one drop (FM i,
- * slice-2 half).
+ * unparseable-at-source, loud-drop → eligibility gate (slice γ: drop
+ * `outdated-rejected` when the outdated filter empties an otherwise-substantive
+ * thread, else `truncated` when thin to begin with) → completeness-check (≥1
+ * substantive comment on the survivors) → build provenance → draft zero-or-more
+ * bodies from the SURVIVING threads only → preflight each → carry a
+ * `DraftCandidate` or loud-drop. Every train PR ends with at least one draft or
+ * one drop (FM i, slice-2 half).
  */
 export async function runExtractStage(split, deps) {
     const trainSet = new Set(split.trainPrs);
     const drafts = [];
     const dropEntries = [];
     const apiEntries = [];
-    const drop = (sourcePr, reasonCode, detail) => {
-        dropEntries.push({ sourcePr, reasonCode, detail });
+    const drop = (sourcePr, reasonCode, detail, noDraftCause, sourceKind) => {
+        dropEntries.push({
+            sourcePr,
+            reasonCode,
+            detail,
+            ...(noDraftCause ? { noDraftCause } : {}),
+            ...(sourceKind ? { sourceKind } : {}),
+        });
     };
     // Iterate the TRAIN slice ONLY — held-out / control / excluded PRs are never
     // fetched (§6 / FM h). Deterministic ascending order.
@@ -161,26 +252,26 @@ export async function runExtractStage(split, deps) {
             drop(pr, 'incomplete-provenance', `fetched content PR #${content.pr} does not match requested train PR #${pr}`);
             continue;
         }
-        // Resolution-eligibility gate (slice 5a, mmnto-ai/totem#2201) — BEFORE the
-        // completeness check. The adapter surfaced per-thread `isResolved`/`isOutdated`
-        // (it never pre-filters); core decides + ledgers here so every resolution
-        // rejection is auditable (§8). Filter to eligible (non-resolved, non-outdated)
-        // threads and recount human comments on the SURVIVORS only.
-        const preFilterHumanCount = humanCommentCount(content.threads);
+        // Eligibility gate (slice γ) — BEFORE the completeness check. The adapter
+        // surfaced per-thread `isOutdated` (it never pre-filters); core decides +
+        // ledgers here so every rejection is auditable (§8). Filter to eligible
+        // (non-outdated; RESOLVED is now admitted) threads and recount SUBSTANTIVE
+        // comments (slice β: human + recognized review-bot) on the SURVIVORS only.
+        const preFilterSubstantiveCount = substantiveCommentCount(content.threads);
         const survivingThreads = eligibleThreads(content.threads);
-        const survivorHumanCount = humanCommentCount(survivingThreads);
-        if (survivorHumanCount < 1) {
-            if (preFilterHumanCount >= 1) {
-                // The thread carried human content, but the resolution gate is what
-                // emptied it → `resolved-rejected` (an eligibility rejection, not thin
-                // content). Carry the concrete resolution evidence in the detail.
+        const survivorSubstantiveCount = substantiveCommentCount(survivingThreads);
+        if (survivorSubstantiveCount < 1) {
+            if (preFilterSubstantiveCount >= 1) {
+                // The thread carried substantive content, but the OUTDATED filter is what
+                // emptied it → `outdated-rejected` (an eligibility rejection, not thin
+                // content). Carry the concrete outdated evidence in the detail.
                 const ineligible = content.threads.length - survivingThreads.length;
-                drop(pr, 'resolved-rejected', `${ineligible} of ${content.threads.length} threads resolved/outdated; ${survivorHumanCount} eligible human comments remain`);
+                drop(pr, 'outdated-rejected', `${ineligible} of ${content.threads.length} threads outdated; ${survivorSubstantiveCount} eligible substantive comments remain`);
             }
             else {
-                // Thin to begin with (0 human comments BEFORE the resolution gate) — the
-                // existing `truncated` path, NOT a resolution rejection.
-                drop(pr, 'truncated', 'no non-empty human review comment after bot filtering');
+                // Thin to begin with (0 substantive comments BEFORE the gate) — the
+                // existing `truncated` path, NOT an eligibility rejection.
+                drop(pr, 'truncated', 'no non-empty substantive review comment after bot filtering');
             }
             continue;
         }
@@ -198,11 +289,27 @@ export async function runExtractStage(split, deps) {
         // catches its own LLM/network errors) — so the core needs no swallowing catch
         // (Tenet 4). An empty list is a loud drop below, not a silent skip.
         const eligibleContent = { ...content, threads: survivingThreads };
-        const draftBodies = await deps.extractor.draft(eligibleContent);
+        // The substrate provenance (slice β, Tenet-19 diagnostic) of THIS PR's eligible
+        // threads — carried onto every draft + the zero-draft drop (panel OQ-β4).
+        const sourceKind = computeSourceKind(survivingThreads);
+        // Parse the port result at the boundary (mirrors ClassifierResultSchema.parse in
+        // classify.ts): a contract-violating DraftResult (empty-without-cause or
+        // cause-without-empty from a buggy adapter) fails loud HERE, before the ledger.
+        const draftResult = DraftResultSchema.parse(await deps.extractor.draft(eligibleContent));
+        const draftBodies = draftResult.drafts;
         if (draftBodies.length === 0) {
             // A complete thread that yields no draft is a loud drop (keeps the train PR
-            // creditable under FM i), not a silent skip.
-            drop(pr, 'unparseable', 'extractor produced no draft from a complete thread');
+            // creditable under FM i), not a silent skip. Reason code `no-draft` (slice β,
+            // strategy β-watch): the slice-α empty-draft drop reused `unparseable`, which
+            // was semantically wrong for a legitimate model decline (`none-sentinel`) —
+            // the coarse code now NAMES the no-draft case while `noDraftCause` (Tenet-19
+            // diagnostic) carries the precise sub-reason, so a parser/format/transient
+            // failure is never conflated with the model judging nothing mintable.
+            drop(pr, 'no-draft', 
+            // The boundary parse above + the "cause iff empty" refine guarantee
+            // `noDraftCause` is present in this empty-drafts branch — assert it rather
+            // than defend with a `?? 'unknown'` fallback that can never fire.
+            `extractor produced no draft from a complete thread (cause: ${draftResult.noDraftCause})`, draftResult.noDraftCause, sourceKind);
             continue;
         }
         for (const body of draftBodies) {
@@ -210,7 +317,7 @@ export async function runExtractStage(split, deps) {
                 drop(pr, 'unparseable', 'draft is empty or carries no usable **Pattern:**/yaml DSL');
                 continue;
             }
-            drafts.push({ provenance: provenance.value, dslSource: body });
+            drafts.push({ provenance: provenance.value, dslSource: body, sourceKind });
         }
     }
     // Recompute the held-out-fetch count from the frozen split rather than trust a

package/dist/spine/extract.js.map

@@ -1 +1 @@
-{"version":3,"file":"extract.js","sourceRoot":"","sources":["../../src/spine/extract.ts"],"names":[],"mappings":"AAAA,gGAAgG;AAChG,EAAE;AACF,gFAAgF;AAChF,gFAAgF;AAChF,6EAA6E;AAC7E,8EAA8E;AAC9E,iFAAiF;AACjF,oEAAoE;AACpE,kFAAkF;AAClF,mEAAmE;AACnE,EAAE;AACF,iFAAiF;AACjF,+EAA+E;AAC/E,4EAA4E;AAC5E,2EAA2E;AAC3E,YAAY;AACZ,EAAE;AACF,2CAA2C;AAC3C,4EAA4E;AAC5E,4EAA4E;AAC5E,0EAA0E;AAC1E,iFAAiF;AACjF,8EAA8E;AAC9E,4EAA4E;AAC5E,iFAAiF;AACjF,mEAAmE;AACnE,mFAAmF;AACnF,6EAA6E;AAC7E,mFAAmF;AACnF,iFAAiF;AACjF,sDAAsD;AACtD,EAAE;AACF,2EAA2E;AAC3E,gFAAgF;AAChF,0DAA0D;AAE1D,OAAO,EAAyB,sBAAsB,EAAE,MAAM,uBAAuB,CAAC;AACtF,OAAO,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAC/C,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAQ5D,OAAO,EAAE,aAAa,EAAE,MAAM,qBAAqB,CAAC;AAuIpD,gFAAgF;AAEhF;;;;;;GAMG;AACH,SAAS,iBAAiB,CAAC,OAAgC;IACzD,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,KAAK,MAAM,OAAO,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;YACtC,IAAI,OAAO,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,MAAM,GAAG,CAAC,IAAI,CAAC,aAAa,CAAC,OAAO,CAAC,MAAM,CAAC;gBAAE,KAAK,EAAE,CAAC;QAChF,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,eAAe,CAAC,OAAgC;IACvD,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC;AAC/D,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,WAAW,CAAC,SAAiB;IACpC,IAAI,SAAS,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IAChD,IAAI,CAAC;QACH,OAAO,oBAAoB,CAAC,SAAS,CAAC,KAAK,IAAI,CAAC;IAClD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,8EAA8E;QAC9E,+EAA+E;QAC/E,kEAAkE;QAClE,IAAI,GAAG,YAAY,eAAe;YAAE,OAAO,KAAK,CAAC;QACjD,MAAM,GAAG,CAAC;IACZ,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,eAAe,CACtB,EAAU,EACV,OAA4B;IAE5B,MAAM,MAAM,GAAG,sBAAsB,CAAC,SAAS,CAAC;QAC9C,QAAQ,EAAE,EAAE;QACZ,YAAY,EAAE,SAAS,EAAE,WAAW;QACpC,SAAS,EAAE,OAAO,CAAC,cAAc;KAClC,CAAC,CAAC;IACH,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;QACpB,OAAO;YACL,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;SACrE,CAAC;IACJ,CAAC;IACD,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,CAAC,IAAI,EAAE,CAAC;AAC1C,CAAC;AAED,iFAAiF;AAEjF;;;;;;;;;;;;;;;GAeG;AACH,MAAM,CAAC,KAAK,UAAU,eAAe,CACnC,KAAoB,EACpB,IAAsB;IAEtB,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IACzC,MAAM,MAAM,GAAqB,EAAE,CAAC;IACpC,MAAM,WAAW,GAAsB,EAAE,CAAC;IAC1C,MAAM,UAAU,GAA0B,EAAE,CAAC;IAE7C,MAAM,IAAI,GAAG,CAAC,QAAgB,EAAE,UAA0B,EAAE,MAAc,EAAQ,EAAE;QAClF,WAAW,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,UAAU,EAAE,MAAM,EAAE,CAAC,CAAC;IACrD,CAAC,CAAC;IAEF,6EAA6E;IAC7E,sDAAsD;IACtD,MAAM,QAAQ,GAAG,CAAC,GAAG,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;IAErD,KAAK,MAAM,EAAE,IAAI,QAAQ,EAAE,CAAC;QAC1B,4EAA4E;QAC5E,wEAAwE;QACxE,oBAAoB;QACpB,UAAU,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,CAAC,CAAC;QAE9E,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC3C,IAAI,MAAM,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;YAClC,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,MAAM,CAAC,MAAM,IAAI,2CAA2C,EAAE,EAAE,CAAC,CAAC;YAC1F,SAAS;QACX,CAAC;QACD,IAAI,MAAM,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;YAClC,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,MAAM,CAAC,MAAM,IAAI,2CAA2C,EAAE,EAAE,CAAC,CAAC;YAC1F,SAAS;QACX,CAAC;QACD,MAAM,OAAO,GAAG,MAAM,CAAC,OAAO,CAAC;QAE/B,8EAA8E;QAC9E,8EAA8E;QAC9E,iFAAiF;QACjF,IAAI,OAAO,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC;YACtB,IAAI,CACF,EAAE,EACF,uBAAuB,EACvB,uBAAuB,OAAO,CAAC,EAAE,uCAAuC,EAAE,EAAE,CAC7E,CAAC;YACF,SAAS;QACX,CAAC;QAED,2EAA2E;QAC3E,gFAAgF;QAChF,0EAA0E;QAC1E,+EAA+E;QAC/E,4DAA4D;QAC5D,MAAM,mBAAmB,GAAG,iBAAiB,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAC/D,MAAM,gBAAgB,GAAG,eAAe,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAC1D,MAAM,kBAAkB,GAAG,iBAAiB,CAAC,gBAAgB,CAAC,CAAC;QAE/D,IAAI,kBAAkB,GAAG,CAAC,EAAE,CAAC;YAC3B,IAAI,mBAAmB,IAAI,CAAC,EAAE,CAAC;gBAC7B,oEAAoE;gBACpE,uEAAuE;gBACvE,kEAAkE;gBAClE,MAAM,UAAU,GAAG,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,gBAAgB,CAAC,MAAM,CAAC;gBACpE,IAAI,CACF,EAAE,EACF,mBAAmB,EACnB,GAAG,UAAU,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,+BAA+B,kBAAkB,iCAAiC,CAC7H,CAAC;YACJ,CAAC;iBAAM,CAAC;gBACN,yEAAyE;gBACzE,yDAAyD;gBACzD,IAAI,CAAC,EAAE,EAAE,WAAW,EAAE,uDAAuD,CAAC,CAAC;YACjF,CAAC;YACD,SAAS;QACX,CAAC;QAED,0EAA0E;QAC1E,MAAM,UAAU,GAAG,eAAe,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC;QAChD,IAAI,CAAC,UAAU,CAAC,EAAE,EAAE,CAAC;YACnB,IAAI,CAAC,EAAE,EAAE,uBAAuB,EAAE,UAAU,CAAC,MAAM,CAAC,CAAC;YACrD,SAAS;QACX,CAAC;QAED,+EAA+E;QAC/E,yEAAyE;QACzE,6EAA6E;QAC7E,+EAA+E;QAC/E,+EAA+E;QAC/E,8EAA8E;QAC9E,oEAAoE;QACpE,MAAM,eAAe,GAAwB,EAAE,GAAG,OAAO,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC;QACvF,MAAM,WAAW,GAAG,MAAM,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC;QAEhE,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC7B,4EAA4E;YAC5E,6CAA6C;YAC7C,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,oDAAoD,CAAC,CAAC;YAC9E,SAAS;QACX,CAAC;QAED,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;YAC/B,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC;gBACvB,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,2DAA2D,CAAC,CAAC;gBACrF,SAAS;YACX,CAAC;YACD,MAAM,CAAC,IAAI,CAAC,EAAE,UAAU,EAAE,UAAU,CAAC,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACjE,CAAC;IACH,CAAC;IAED,+EAA+E;IAC/E,4EAA4E;IAC5E,sDAAsD;IACtD,MAAM,iBAAiB,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC;IAE7F,OAAO;QACL,MAAM;QACN,UAAU,EAAE,EAAE,OAAO,EAAE,WAAW,EAAE;QACpC,cAAc,EAAE,EAAE,OAAO,EAAE,UAAU,EAAE,iBAAiB,EAAE;QAC1D,aAAa,EAAE,EAAE,mBAAmB,EAAE,IAAI,CAAC,mBAAmB,EAAE;KACjE,CAAC;AACJ,CAAC"}
+{"version":3,"file":"extract.js","sourceRoot":"","sources":["../../src/spine/extract.ts"],"names":[],"mappings":"AAAA,gGAAgG;AAChG,EAAE;AACF,gFAAgF;AAChF,gFAAgF;AAChF,6EAA6E;AAC7E,8EAA8E;AAC9E,iFAAiF;AACjF,oEAAoE;AACpE,kFAAkF;AAClF,mEAAmE;AACnE,EAAE;AACF,iFAAiF;AACjF,+EAA+E;AAC/E,4EAA4E;AAC5E,2EAA2E;AAC3E,YAAY;AACZ,EAAE;AACF,2CAA2C;AAC3C,4EAA4E;AAC5E,4EAA4E;AAC5E,0EAA0E;AAC1E,iFAAiF;AACjF,8EAA8E;AAC9E,4EAA4E;AAC5E,iFAAiF;AACjF,mEAAmE;AACnE,mFAAmF;AACnF,6EAA6E;AAC7E,mFAAmF;AACnF,iFAAiF;AACjF,sDAAsD;AACtD,EAAE;AACF,2EAA2E;AAC3E,gFAAgF;AAChF,0DAA0D;AAE1D,OAAO,EAAE,CAAC,EAAE,MAAM,KAAK,CAAC;AAExB,OAAO,EAAyB,sBAAsB,EAAE,MAAM,uBAAuB,CAAC;AACtF,OAAO,EAAE,eAAe,EAAE,MAAM,cAAc,CAAC;AAC/C,OAAO,EAAE,oBAAoB,EAAE,MAAM,sBAAsB,CAAC;AAU5D,OAAO,EAAE,kBAAkB,EAAE,MAAM,cAAc,CAAC;AAClD,OAAO,EAAE,aAAa,EAAE,iBAAiB,EAAE,MAAM,qBAAqB,CAAC;AA+HvE;;;;;;;;;;GAUG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAG,CAAC;KAC/B,MAAM,CAAC;IACN,MAAM,EAAE,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC;IAC3B,YAAY,EAAE,kBAAkB,CAAC,QAAQ,EAAE;CAC5C,CAAC;KACD,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,MAAM,CAAC,MAAM,KAAK,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,YAAY,KAAK,SAAS,CAAC,EAAE;IACzE,OAAO,EACL,2FAA2F;IAC7F,IAAI,EAAE,CAAC,cAAc,CAAC;CACvB,CAAC,CAAC;AAkDL,gFAAgF;AAEhF;;;;;GAKG;AACH,MAAM,UAAU,kBAAkB,CAAC,MAAc;IAC/C,OAAO,iBAAiB,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC;AACrD,CAAC;AAED;;;;;;;;;;GAUG;AACH,SAAS,oBAAoB,CAAC,OAA4B;IACxD,8EAA8E;IAC9E,kFAAkF;IAClF,+EAA+E;IAC/E,6EAA6E;IAC7E,6EAA6E;IAC7E,gEAAgE;IAChE,MAAM,aAAa,GAAG,OAAO,CAAC,UAAU,KAAK,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC,cAAc,CAAC,CAAC,CAAC,OAAO,CAAC,IAAI,CAAC;IAC3F,IAAI,aAAa,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IACpD,IAAI,OAAO,CAAC,UAAU,KAAK,KAAK;QAAE,OAAO,IAAI,CAAC;IAC9C,OAAO,CAAC,aAAa,CAAC,OAAO,CAAC,MAAM,CAAC,CAAC;AACxC,CAAC;AAED;;;;GAIG;AACH,SAAS,uBAAuB,CAAC,OAAgC;IAC/D,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,KAAK,MAAM,OAAO,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;YACtC,IAAI,oBAAoB,CAAC,OAAO,CAAC;gBAAE,KAAK,EAAE,CAAC;QAC7C,CAAC;IACH,CAAC;IACD,OAAO,KAAK,CAAC;AACf,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,iBAAiB,CAAC,OAAgC;IACzD,IAAI,KAAK,GAAG,CAAC,CAAC;IACd,IAAI,GAAG,GAAG,CAAC,CAAC;IACZ,KAAK,MAAM,MAAM,IAAI,OAAO,EAAE,CAAC;QAC7B,KAAK,MAAM,OAAO,IAAI,MAAM,CAAC,QAAQ,EAAE,CAAC;YACtC,IAAI,CAAC,oBAAoB,CAAC,OAAO,CAAC;gBAAE,SAAS;YAC7C,IAAI,OAAO,CAAC,UAAU,KAAK,KAAK;gBAAE,GAAG,EAAE,CAAC;;gBACnC,KAAK,EAAE,CAAC;QACf,CAAC;IACH,CAAC;IACD,IAAI,GAAG,GAAG,CAAC,IAAI,KAAK,GAAG,CAAC;QAAE,OAAO,OAAO,CAAC;IACzC,OAAO,GAAG,GAAG,CAAC,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,OAAO,CAAC;AACnC,CAAC;AAED;;;;;;;;;;;;;GAaG;AACH,SAAS,eAAe,CAAC,OAAgC;IACvD,OAAO,OAAO,CAAC,MAAM,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC;AAC9C,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,WAAW,CAAC,SAAiB;IACpC,IAAI,SAAS,CAAC,IAAI,EAAE,CAAC,MAAM,KAAK,CAAC;QAAE,OAAO,KAAK,CAAC;IAChD,IAAI,CAAC;QACH,OAAO,oBAAoB,CAAC,SAAS,CAAC,KAAK,IAAI,CAAC;IAClD,CAAC;IAAC,OAAO,GAAG,EAAE,CAAC;QACb,8EAA8E;QAC9E,+EAA+E;QAC/E,kEAAkE;QAClE,IAAI,GAAG,YAAY,eAAe;YAAE,OAAO,KAAK,CAAC;QACjD,MAAM,GAAG,CAAC;IACZ,CAAC;AACH,CAAC;AAED;;;;;;;GAOG;AACH,SAAS,eAAe,CACtB,EAAU,EACV,OAA4B;IAE5B,MAAM,MAAM,GAAG,sBAAsB,CAAC,SAAS,CAAC;QAC9C,QAAQ,EAAE,EAAE;QACZ,YAAY,EAAE,SAAS,EAAE,WAAW;QACpC,SAAS,EAAE,OAAO,CAAC,cAAc;KAClC,CAAC,CAAC;IACH,IAAI,CAAC,MAAM,CAAC,OAAO,EAAE,CAAC;QACpB,OAAO;YACL,EAAE,EAAE,KAAK;YACT,MAAM,EAAE,MAAM,CAAC,KAAK,CAAC,MAAM,CAAC,GAAG,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC;SACrE,CAAC;IACJ,CAAC;IACD,OAAO,EAAE,EAAE,EAAE,IAAI,EAAE,KAAK,EAAE,MAAM,CAAC,IAAI,EAAE,CAAC;AAC1C,CAAC;AAED,iFAAiF;AAEjF;;;;;;;;;;;;;;;GAeG;AACH,MAAM,CAAC,KAAK,UAAU,eAAe,CACnC,KAAoB,EACpB,IAAsB;IAEtB,MAAM,QAAQ,GAAG,IAAI,GAAG,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC;IACzC,MAAM,MAAM,GAAqB,EAAE,CAAC;IACpC,MAAM,WAAW,GAAsB,EAAE,CAAC;IAC1C,MAAM,UAAU,GAA0B,EAAE,CAAC;IAE7C,MAAM,IAAI,GAAG,CACX,QAAgB,EAChB,UAA0B,EAC1B,MAAc,EACd,YAA2B,EAC3B,UAA4B,EACtB,EAAE;QACR,WAAW,CAAC,IAAI,CAAC;YACf,QAAQ;YACR,UAAU;YACV,MAAM;YACN,GAAG,CAAC,YAAY,CAAC,CAAC,CAAC,EAAE,YAAY,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;YACzC,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,EAAE,UAAU,EAAE,CAAC,CAAC,CAAC,EAAE,CAAC;SACtC,CAAC,CAAC;IACL,CAAC,CAAC;IAEF,6EAA6E;IAC7E,sDAAsD;IACtD,MAAM,QAAQ,GAAG,CAAC,GAAG,QAAQ,CAAC,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,CAAC,EAAE,EAAE,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;IAErD,KAAK,MAAM,EAAE,IAAI,QAAQ,EAAE,CAAC;QAC1B,4EAA4E;QAC5E,wEAAwE;QACxE,oBAAoB;QACpB,UAAU,CAAC,IAAI,CAAC,EAAE,QAAQ,EAAE,EAAE,EAAE,KAAK,EAAE,OAAO,EAAE,SAAS,EAAE,eAAe,EAAE,CAAC,CAAC;QAE9E,MAAM,MAAM,GAAG,MAAM,IAAI,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC3C,IAAI,MAAM,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;YAClC,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,MAAM,CAAC,MAAM,IAAI,2CAA2C,EAAE,EAAE,CAAC,CAAC;YAC1F,SAAS;QACX,CAAC;QACD,IAAI,MAAM,CAAC,IAAI,KAAK,aAAa,EAAE,CAAC;YAClC,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,MAAM,CAAC,MAAM,IAAI,2CAA2C,EAAE,EAAE,CAAC,CAAC;YAC1F,SAAS;QACX,CAAC;QACD,MAAM,OAAO,GAAG,MAAM,CAAC,OAAO,CAAC;QAE/B,8EAA8E;QAC9E,8EAA8E;QAC9E,iFAAiF;QACjF,IAAI,OAAO,CAAC,EAAE,KAAK,EAAE,EAAE,CAAC;YACtB,IAAI,CACF,EAAE,EACF,uBAAuB,EACvB,uBAAuB,OAAO,CAAC,EAAE,uCAAuC,EAAE,EAAE,CAC7E,CAAC;YACF,SAAS;QACX,CAAC;QAED,0EAA0E;QAC1E,0EAA0E;QAC1E,wEAAwE;QACxE,2EAA2E;QAC3E,2EAA2E;QAC3E,MAAM,yBAAyB,GAAG,uBAAuB,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAC3E,MAAM,gBAAgB,GAAG,eAAe,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC;QAC1D,MAAM,wBAAwB,GAAG,uBAAuB,CAAC,gBAAgB,CAAC,CAAC;QAE3E,IAAI,wBAAwB,GAAG,CAAC,EAAE,CAAC;YACjC,IAAI,yBAAyB,IAAI,CAAC,EAAE,CAAC;gBACnC,0EAA0E;gBAC1E,uEAAuE;gBACvE,gEAAgE;gBAChE,MAAM,UAAU,GAAG,OAAO,CAAC,OAAO,CAAC,MAAM,GAAG,gBAAgB,CAAC,MAAM,CAAC;gBACpE,IAAI,CACF,EAAE,EACF,mBAAmB,EACnB,GAAG,UAAU,OAAO,OAAO,CAAC,OAAO,CAAC,MAAM,sBAAsB,wBAAwB,uCAAuC,CAChI,CAAC;YACJ,CAAC;iBAAM,CAAC;gBACN,oEAAoE;gBACpE,2DAA2D;gBAC3D,IAAI,CAAC,EAAE,EAAE,WAAW,EAAE,6DAA6D,CAAC,CAAC;YACvF,CAAC;YACD,SAAS;QACX,CAAC;QAED,0EAA0E;QAC1E,MAAM,UAAU,GAAG,eAAe,CAAC,EAAE,EAAE,OAAO,CAAC,CAAC;QAChD,IAAI,CAAC,UAAU,CAAC,EAAE,EAAE,CAAC;YACnB,IAAI,CAAC,EAAE,EAAE,uBAAuB,EAAE,UAAU,CAAC,MAAM,CAAC,CAAC;YACrD,SAAS;QACX,CAAC;QAED,+EAA+E;QAC/E,yEAAyE;QACzE,6EAA6E;QAC7E,+EAA+E;QAC/E,+EAA+E;QAC/E,8EAA8E;QAC9E,oEAAoE;QACpE,MAAM,eAAe,GAAwB,EAAE,GAAG,OAAO,EAAE,OAAO,EAAE,gBAAgB,EAAE,CAAC;QACvF,gFAAgF;QAChF,0EAA0E;QAC1E,MAAM,UAAU,GAAG,iBAAiB,CAAC,gBAAgB,CAAC,CAAC;QACvD,iFAAiF;QACjF,yEAAyE;QACzE,gFAAgF;QAChF,MAAM,WAAW,GAAG,iBAAiB,CAAC,KAAK,CAAC,MAAM,IAAI,CAAC,SAAS,CAAC,KAAK,CAAC,eAAe,CAAC,CAAC,CAAC;QACzF,MAAM,WAAW,GAAG,WAAW,CAAC,MAAM,CAAC;QAEvC,IAAI,WAAW,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YAC7B,4EAA4E;YAC5E,8EAA8E;YAC9E,8EAA8E;YAC9E,4EAA4E;YAC5E,6EAA6E;YAC7E,2EAA2E;YAC3E,sEAAsE;YACtE,IAAI,CACF,EAAE,EACF,UAAU;YACV,oEAAoE;YACpE,2EAA2E;YAC3E,kEAAkE;YAClE,8DAA8D,WAAW,CAAC,YAAa,GAAG,EAC1F,WAAW,CAAC,YAAY,EACxB,UAAU,CACX,CAAC;YACF,SAAS;QACX,CAAC;QAED,KAAK,MAAM,IAAI,IAAI,WAAW,EAAE,CAAC;YAC/B,IAAI,CAAC,WAAW,CAAC,IAAI,CAAC,EAAE,CAAC;gBACvB,IAAI,CAAC,EAAE,EAAE,aAAa,EAAE,2DAA2D,CAAC,CAAC;gBACrF,SAAS;YACX,CAAC;YACD,MAAM,CAAC,IAAI,CAAC,EAAE,UAAU,EAAE,UAAU,CAAC,KAAK,EAAE,SAAS,EAAE,IAAI,EAAE,UAAU,EAAE,CAAC,CAAC;QAC7E,CAAC;IACH,CAAC;IAED,+EAA+E;IAC/E,4EAA4E;IAC5E,sDAAsD;IACtD,MAAM,iBAAiB,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,KAAK,EAAE,EAAE,CAAC,CAAC,QAAQ,CAAC,GAAG,CAAC,KAAK,CAAC,QAAQ,CAAC,CAAC,CAAC,MAAM,CAAC;IAE7F,OAAO;QACL,MAAM;QACN,UAAU,EAAE,EAAE,OAAO,EAAE,WAAW,EAAE;QACpC,cAAc,EAAE,EAAE,OAAO,EAAE,UAAU,EAAE,iBAAiB,EAAE;QAC1D,aAAa,EAAE,EAAE,mBAAmB,EAAE,IAAI,CAAC,mBAAmB,EAAE;KACjE,CAAC;AACJ,CAAC"}

package/dist/spine/extract.test.js

@@ -1,8 +1,24 @@
 import { describe, expect, it } from 'vitest';
-import { runExtractStage, } from './extract.js';
+import { classifyAuthorKind, runExtractStage, } from './extract.js';
+import { normalizeReviewChrome } from './review-normalize.js';
 import { SplitArtifactSchema } from './split.js';
 // ─── Helpers ──────────────────────────────────────────────────────────────
 const sha = (n) => String(n).padStart(40, '0');
+/**
+ * Build a slice-β-enriched comment exactly as the CLI mapping boundary does:
+ * `authorKind` via core `classifyAuthorKind`, `normalizedBody` = de-chromed for a
+ * recognized review bot, else the raw body. Keeps test comments honest w.r.t. the
+ * shipped classification rather than hand-stamping fields.
+ */
+function comment(author, body) {
+    const authorKind = classifyAuthorKind(author);
+    return {
+        author,
+        body,
+        authorKind,
+        normalizedBody: authorKind === 'bot' ? normalizeReviewChrome(body) : body,
+    };
+}
 function split(overrides) {
     return SplitArtifactSchema.parse({
         asOfCommit: sha(100),
@@ -39,7 +55,7 @@ function content(pr, overrides) {
         threads: [
             {
                 path: 'packages/core/src/x.ts',
-                comments: [{ author: 'Jane Doe', body: 'a real review note' }],
+                comments: [comment('Jane Doe', 'a real review note')],
                 isResolved: false,
                 isOutdated: false,
             },
@@ -51,7 +67,7 @@ function content(pr, overrides) {
 function thread(author, body, flags) {
     return {
         path: 'packages/core/src/x.ts',
-        comments: [{ author, body }],
+        comments: [comment(author, body)],
         isResolved: flags?.isResolved ?? false,
         isOutdated: flags?.isOutdated ?? false,
     };
@@ -75,11 +91,16 @@ function spySource(trainPrs, results) {
         },
     };
 }
-/** Fixture extractor: per-PR draft bodies from a map (default: one usable body). Async. */
+/**
+ * Fixture extractor: per-PR draft bodies from a map (default: one usable body). Async.
+ * Wraps the bare `string[]` into a `DraftResult`, tagging an empty list with a
+ * representative `all-filtered` cause so the "cause iff empty" invariant holds.
+ */
 function fixtureExtractor(byPr) {
     return {
         async draft(c) {
-            return byPr?.get(c.pr) ?? [USABLE_DSL];
+            const drafts = byPr?.get(c.pr) ?? [USABLE_DSL];
+            return drafts.length === 0 ? { drafts, noDraftCause: 'all-filtered' } : { drafts };
         },
     };
 }
@@ -135,12 +156,14 @@ describe('runExtractStage — drop reason codes', () => {
         const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: content(1, { threads: [] }) }]])), fixtureExtractor()));
         expect(dropsFor(r, 1)[0].reasonCode).toBe('truncated');
     });
-    it('truncated: a bot-only thread does not satisfy ≥1 human comment (fold 5)', async () => {
+    it('truncated: a NOISE-bot-only thread does not satisfy ≥1 substantive comment (slice β denylist)', async () => {
+        // dependabot/renovate are NOT in the review-finding allowlist → still excluded
+        // (unlike gemini/CR, which now count — see the slice-β substrate tests).
         const botThread = content(1, {
             threads: [
                 {
                     path: 'x.ts',
-                    comments: [{ author: 'coderabbitai[bot]', body: 'nit: rename this' }],
+                    comments: [comment('dependabot[bot]', 'bump lodash to 4.17.21')],
                     isResolved: false,
                     isOutdated: false,
                 },
@@ -154,7 +177,7 @@ describe('runExtractStage — drop reason codes', () => {
             threads: [
                 {
                     path: 'x.ts',
-                    comments: [{ author: 'Jane Doe', body: '   ' }],
+                    comments: [comment('Jane Doe', '   ')],
                     isResolved: false,
                     isOutdated: false,
                 },
@@ -186,20 +209,44 @@ describe('runExtractStage — drop reason codes', () => {
         expect(dropsFor(r, 1)[0].reasonCode).toBe('unparseable');
         expect(r.drafts).toEqual([]);
     });
-    it('unparseable: the extractor produced no draft from a complete thread', async () => {
+    it('no-draft: the extractor produced no draft from a complete thread — records cause + sourceKind (slice β)', async () => {
         const r = await runExtractStage(solo(), deps(spySource([1]), fixtureExtractor(new Map([[1, []]]))));
-        expect(dropsFor(r, 1)[0].reasonCode).toBe('unparseable');
+        const drop = dropsFor(r, 1)[0];
+        // Slice β: the zero-draft drop is now its own `no-draft` reason code (was the
+        // misnamed `unparseable`), with the precise sub-cause + the substrate tag.
+        expect(drop.reasonCode).toBe('no-draft');
+        // The Tenet-19 diagnostic is carried onto the drop (fixtureExtractor tags an
+        // empty list 'all-filtered') — never silently dropped as a bare [].
+        expect(drop.noDraftCause).toBe('all-filtered');
+        expect(drop.detail).toContain('all-filtered');
+        // The default `content(1)` thread is a single human comment → human substrate.
+        expect(drop.sourceKind).toBe('human');
+    });
+    it('boundary-parse fails loud on a contract-violating DraftResult (empty-without-cause)', async () => {
+        // A buggy adapter returning { drafts: [] } with no cause violates the
+        // "cause iff empty" invariant; DraftResultSchema.parse must reject it at the
+        // core boundary (Tenet 4), not silently drop-ledger a causeless empty.
+        const badExtractor = {
+            async draft() {
+                // Type-VALID (noDraftCause is optional in the static type) but RUNTIME-invalid:
+                // the schema's "cause iff empty" refine requires a cause when drafts is empty,
+                // so the core boundary parse rejects this — exactly the buggy-adapter case.
+                return { drafts: [] };
+            },
+        };
+        await expect(runExtractStage(solo(), deps(spySource([1]), badExtractor))).rejects.toThrow(/noDraftCause must be present iff drafts is empty/);
     });
     it('a contract-violating extractor throw propagates (fail-loud, not swallowed)', async () => {
-        // The port contract is: return [] on a per-PR failure (the CLI adapter catches
-        // its own IO errors). A throw VIOLATES that contract and must NOT be silently
-        // swallowed — it propagates (Tenet 4). Per-PR resilience is the adapter's job;
-        // the []-returns path is covered by "extractor produced no draft" above.
+        // The port contract is: return { drafts: [], noDraftCause } on a per-PR failure
+        // (the CLI adapter catches its own IO errors → 'invoke-error'). A throw VIOLATES
+        // that contract and must NOT be silently swallowed — it propagates (Tenet 4).
+        // Per-PR resilience is the adapter's job; the empty-result path is covered by
+        // "extractor produced no draft" above.
         const throwingExtractor = {
             async draft(c) {
                 if (c.pr === 1)
                     throw new Error('boom');
-                return [USABLE_DSL];
+                return { drafts: [USABLE_DSL] };
             },
         };
         await expect(runExtractStage(split(), deps(spySource([1, 2]), throwingExtractor))).rejects.toThrow('boom');
@@ -275,44 +322,48 @@ function recordingExtractor(byPr) {
         seen,
         async draft(c) {
             seen.push(c);
-            return byPr?.get(c.pr) ?? [USABLE_DSL];
+            const drafts = byPr?.get(c.pr) ?? [USABLE_DSL];
+            return drafts.length === 0 ? { drafts, noDraftCause: 'all-filtered' } : { drafts };
         },
     };
 }
-describe('runExtractStage — resolution-eligibility gate (slice 5a)', () => {
-    it('all-resolved-but-had-human-content drops resolved-rejected (not truncated)', async () => {
-        const allResolved = content(1, {
-            threads: [
-                thread('Jane Doe', 'a real review note', { isResolved: true }),
-                thread('John Roe', 'another note', { isOutdated: true }),
-            ],
+describe('runExtractStage — eligibility gate (slice γ: RESOLVED admitted, only OUTDATED excluded)', () => {
+    it('a RESOLVED thread is now ADMITTED (drafts; reaches the extractor input)', async () => {
+        const resolved = content(1, {
+            threads: [thread('Jane Doe', 'a real review note', { isResolved: true })],
         });
-        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: allResolved }]])), fixtureExtractor()));
-        expect(dropsFor(r, 1)[0].reasonCode).toBe('resolved-rejected');
-        expect(r.drafts).toEqual([]);
+        const extractor = recordingExtractor();
+        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: resolved }]])), extractor));
+        expect(r.drafts).toHaveLength(1);
+        expect(dropsFor(r, 1)).toEqual([]);
+        // γ: the resolved thread is no longer pre-filtered — the extractor saw it.
+        expect(extractor.seen[0].threads).toHaveLength(1);
+        expect(extractor.seen[0].threads[0].isResolved).toBe(true);
     });
-    it('the resolved-rejected drop detail carries concrete resolution evidence', async () => {
-        const allResolved = content(1, {
+    it('all-OUTDATED-but-had-substantive-content drops outdated-rejected (not truncated)', async () => {
+        const allOutdated = content(1, {
             threads: [
-                thread('Jane Doe', 'a real review note', { isResolved: true }),
+                thread('Jane Doe', 'a real review note', { isOutdated: true }),
                 thread('John Roe', 'another note', { isOutdated: true }),
             ],
         });
-        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: allResolved }]])), fixtureExtractor()));
-        const detail = dropsFor(r, 1)[0].detail ?? '';
-        expect(detail).toContain('2 of 2 threads resolved/outdated');
-        expect(detail).toContain('0 eligible human comments remain');
-    });
-    it('thin-to-begin-with (0 human comments before the gate) stays truncated, not resolved-rejected', async () => {
-        // A resolved thread that ALSO had no human comment — the resolution gate is
-        // not what emptied it; it was already thin. Keep the existing truncated path.
-        const botResolved = content(1, {
-            threads: [thread('coderabbitai[bot]', 'nit: rename this', { isResolved: true })],
+        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: allOutdated }]])), fixtureExtractor()));
+        const drop = dropsFor(r, 1)[0];
+        expect(drop.reasonCode).toBe('outdated-rejected');
+        expect(drop.detail).toContain('2 of 2 threads outdated');
+        expect(drop.detail).toContain('0 eligible substantive comments remain');
+        expect(r.drafts).toEqual([]);
+    });
+    it('thin-to-begin-with (0 substantive comments before the gate) stays truncated, not outdated-rejected', async () => {
+        // A noise-bot-only outdated thread — the gate is not what emptied it; it was
+        // already thin (dependabot is not a recognized review bot). Keep `truncated`.
+        const botOutdated = content(1, {
+            threads: [thread('dependabot[bot]', 'bump dep', { isOutdated: true })],
         });
-        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: botResolved }]])), fixtureExtractor()));
+        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: botOutdated }]])), fixtureExtractor()));
         expect(dropsFor(r, 1)[0].reasonCode).toBe('truncated');
     });
-    it('partial resolution: survivors are processed; resolved threads excluded from the draft input', async () => {
+    it('partial: OUTDATED threads excluded from the draft input; resolved + fresh survive', async () => {
         const mixed = content(1, {
             threads: [
                 thread('Jane Doe', 'eligible note A'),
@@ -322,33 +373,65 @@ describe('runExtractStage — resolution-eligibility gate (slice 5a)', () => {
         });
         const extractor = recordingExtractor();
         const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: mixed }]])), extractor));
-        // The PR is processed (a draft, no drop).
         expect(r.drafts).toHaveLength(1);
         expect(dropsFor(r, 1)).toEqual([]);
-        // The extractor only saw the ONE eligible thread — resolved/outdated excluded.
-        expect(extractor.seen).toHaveLength(1);
+        // The extractor saw the two NON-outdated threads (incl. the resolved one); only
+        // the outdated thread was excluded (γ inverts the slice-5a resolved exclusion).
         const seenThreads = extractor.seen[0].threads;
-        expect(seenThreads).toHaveLength(1);
-        expect(seenThreads[0].comments[0].body).toBe('eligible note A');
-        expect(seenThreads.every((t) => !t.isResolved && !t.isOutdated)).toBe(true);
+        expect(seenThreads).toHaveLength(2);
+        expect(seenThreads.every((t) => !t.isOutdated)).toBe(true);
+        expect(seenThreads.map((t) => t.comments[0].body).sort()).toEqual([
+            'eligible note A',
+            'resolved note B',
+        ]);
+    });
+    it('a fully-eligible thread is unaffected by the gate (no regression)', async () => {
+        const r = await runExtractStage(solo(), deps(spySource([1]), fixtureExtractor()));
+        expect(r.drafts).toHaveLength(1);
+        expect(dropsFor(r, 1)).toEqual([]);
+    });
+});
+// ─── Slice β: bot-review substrate + sourceKind diagnostic ────────────────────
+describe('runExtractStage — slice β (bot-review substrate + sourceKind)', () => {
+    it('a RECOGNIZED review-bot (coderabbitai) comment now COUNTS as substrate → drafts', async () => {
+        const crOnly = content(1, {
+            threads: [thread('coderabbitai[bot]', 'Potential issue: guard against NaN here')],
+        });
+        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: crOnly }]])), fixtureExtractor()));
+        expect(r.drafts).toHaveLength(1);
+        expect(dropsFor(r, 1)).toEqual([]);
+        expect(r.drafts[0].sourceKind).toBe('bot');
+    });
+    it('gemini-code-assist (no [bot] suffix) counts as substrate', async () => {
+        const gca = content(1, {
+            threads: [thread('gemini-code-assist', 'require is_finite() before the divide')],
+        });
+        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: gca }]])), fixtureExtractor()));
+        expect(r.drafts).toHaveLength(1);
+        expect(r.drafts[0].sourceKind).toBe('bot');
     });
-    it('partial resolution: resolved threads do not count toward the human-comment threshold', async () => {
-        // The single eligible thread is bot-only; the resolved thread is the only one
-        // with a human comment. Survivors have 0 human comments → resolved-rejected.
+    it('sourceKind is human for a human-only thread', async () => {
+        const r = await runExtractStage(solo(), deps(spySource([1]), fixtureExtractor()));
+        expect(r.drafts[0].sourceKind).toBe('human');
+    });
+    it('a badge-ONLY review-bot comment is NOT substantive — de-chromed empty → truncated, not no-draft (greptile #2242)', async () => {
+        const badgeOnly = content(1, {
+            threads: [thread('coderabbitai[bot]', '![high](https://x/high.svg)')],
+        });
+        const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: badgeOnly }]])), fixtureExtractor()));
+        // normalizedBody strips to '' → not substantive → thin from the start.
+        expect(dropsFor(r, 1)[0].reasonCode).toBe('truncated');
+        expect(r.drafts).toEqual([]);
+    });
+    it('sourceKind is mixed when human + review-bot comments both survive', async () => {
         const mixed = content(1, {
             threads: [
-                thread('coderabbitai[bot]', 'nit: rename', { isResolved: false }),
-                thread('Jane Doe', 'a real human note', { isResolved: true }),
+                thread('Jane Doe', 'the human rationale'),
+                thread('coderabbitai[bot]', 'the bot finding'),
             ],
         });
         const r = await runExtractStage(solo(), deps(spySource([1], new Map([[1, { kind: 'ok', content: mixed }]])), fixtureExtractor()));
-        expect(dropsFor(r, 1)[0].reasonCode).toBe('resolved-rejected');
-        expect(r.drafts).toEqual([]);
-    });
-    it('a fully-eligible thread is unaffected by the gate (no regression)', async () => {
-        const r = await runExtractStage(solo(), deps(spySource([1]), fixtureExtractor()));
-        expect(r.drafts).toHaveLength(1);
-        expect(dropsFor(r, 1)).toEqual([]);
+        expect(r.drafts[0].sourceKind).toBe('mixed');
     });
 });
 // ─── Determinism ──────────────────────────────────────────────────────────────
@@ -357,9 +440,9 @@ describe('runExtractStage — determinism', () => {
         const run = () => runExtractStage(split(), deps(spySource([1, 2], new Map([[2, { kind: 'unreachable' }]])), fixtureExtractor(new Map([[1, [USABLE_DSL, NO_PATTERN_DSL]]]))));
         expect(await run()).toEqual(await run());
     });
-    it('identical inputs are deterministic across the resolution gate (drafts + drops + ledgers)', async () => {
-        // PR 1: a partial-resolution mix (one eligible survivor → a draft).
-        // PR 2: all-resolved-but-had-human-content → a resolved-rejected drop.
+    it('identical inputs are deterministic across the eligibility gate (drafts + drops + ledgers)', async () => {
+        // PR 1: a partial mix (resolved + fresh both survive → a draft).
+        // PR 2: all-outdated-but-had-substantive-content → an outdated-rejected drop.
         const pr1 = content(1, {
             threads: [
                 thread('Jane Doe', 'eligible note', { isResolved: false }),
@@ -376,7 +459,7 @@ describe('runExtractStage — determinism', () => {
         const a = await run();
         const b = await run();
         expect(a).toEqual(b);
-        expect(dropsFor(a, 2)[0].reasonCode).toBe('resolved-rejected');
+        expect(dropsFor(a, 2)[0].reasonCode).toBe('outdated-rejected');
     });
 });
 //# sourceMappingURL=extract.test.js.map