agentwrangler 0.1.1-next.bca85ea → 0.1.1-next.fbbbe7f

package/dist/detector/detectors/d2_session_long_full_context.js

@@ -81,12 +81,12 @@ export const d2Detector = {
             scopeKey: `D2|global|${formula.model}`,
             category: "CONTEXT",
             scope_workspace_id: null,
-            lever: "Use /clear between unrelated tasks, split long work into separate sessions, and avoid automatic compaction while you are working.",
+            lever: "/clear between unrelated tasks; split long work; avoid mid-task /compact.",
             target_metric: "avg_context_per_turn",
             modeled_savings_u_per_wk: savingsU,
             modeled_formula: formula,
             evidence: {
-                title: `Split ${n} long session${n === 1 ? "" : "s"} this week`,
+                title: `Shorten sessions: ${n} long-context run${n === 1 ? "" : "s"} this week`,
                 qualifying_session_count: qualifying.length,
                 qualifying_turn_count: qualifyingTurnCount,
                 session_ids: sessionIds,

package/dist/detector/detectors/d4_model_mismatch.js

@@ -192,7 +192,7 @@ export const d4Detector = {
                 ? {
                     withheld: true,
                     withheld_reason: `Sonnet weekly cap is the binding constraint (Sonnet util ${bindingSonnet.utilization} >= all-models ${perModelSnapshot.seven_day_util}) — routing Opus->Sonnet would worsen it`,
-                    title: `Model change is not recommended: ${mismatchPct}% of Opus turns have large context and little output`,
+                    title: `[withheld] Route Opus→Sonnet: ${mismatchPct}% of turns are high-context low-output`,
                 }
                 : perModelSnapshot && sonnetEntries && sonnetEntries.length > 0
                     ? { cap_attribution: "all_models_or_opus_binds" }
@@ -202,7 +202,7 @@ export const d4Detector = {
                 category: "MODEL",
                 scope_workspace_id: workspace_id,
                 // Advisory gate (W0.3): which cap binds is NOT inferable from JSONL. Conditional lever.
-                lever: "Check /usage first. If your overall, Opus, or 5-hour limit is filling, these large-context Opus turns with little output may be suitable for Sonnet. Do not switch if your Sonnet weekly limit is the one filling.",
+                lever: "If your all-models / Opus / 5h cap is the one binding — check /usage — these high-context low-output Opus turns are Sonnet-movable. This does NOT help, and can hurt, if your Sonnet-specific weekly cap is the binding constraint.",
                 target_metric: "model_mix_opus_fraction",
                 // Advisory gate: suppress the crisp $/wk headline until live /usage cap-attribution exists.
                 modeled_savings_u_per_wk: null,
@@ -210,7 +210,7 @@ export const d4Detector = {
                 // Destructure out result_usd_per_wk so the advisory formula carries no crisp $/wk figure.
                 modeled_formula: (({ result_usd_per_wk: _, ...rest }) => ({ ...rest, kind: "ADVISORY" }))(formula),
                 evidence: {
-                    title: `Review model choice: ${mismatchPct}% of Opus turns have large context and little output`,
+                    title: `Route Opus→Sonnet: ${mismatchPct}% of turns are high-context low-output`,
                     workspace_id,
                     total_opus_turns_per_week: totalOpus,
                     mismatch_turns_per_week: mismatchCount,

package/dist/detector/detectors/d6_tool_result_bloat.js

@@ -262,7 +262,7 @@ export const d6Detector = {
                 modeled_savings_u_per_wk: modeledSavingsU,
                 modeled_formula: formula,
                 evidence: {
-                    title: `Reduce large tool results: ${Math.round(bloatShare * 100)}% of this session's context`,
+                    title: `Trim tool output: ${Math.round(bloatShare * 100)}% bloat share in session`,
                     session_id: row.session_id,
                     workspace_id: row.workspace_id,
                     tool_result_bytes: row.tool_result_bytes,

package/dist/detector/detectors/d7_loop_retry_waste.js

@@ -299,7 +299,7 @@ export const d7Detector = {
                     expression: "cap-weighted exposure of turns owning repeat-excess events; not an avoidable-token or USD savings estimate",
                 },
                 evidence: {
-                    title: `Stop repeated attempts: ${flaggedTurns.size} affected turn${flaggedTurns.size === 1 ? "" : "s"} in this session`,
+                    title: `Break retry loops: ${flaggedTurns.size} flagged turn${flaggedTurns.size === 1 ? "" : "s"} in session`,
                     session_id: sessionId,
                     workspace_id: session.workspaceId,
                     loop_flagged_event_count: flaggedEventIds.size,

package/dist/detector/detectors/d8_cache_write_churn.js

@@ -125,7 +125,7 @@ export const d8Detector = {
             // TTL-regime facet: is this session dominated by 5m-tier creation?
             const fiveMinCreation = events.reduce((s, e) => s + e.cache_write_5m, 0);
             const regime5m = totalCreation > 0 && fiveMinCreation / totalCreation >= D8_REGIME_5M_SHARE;
-            const baseLever = "Before a long break, use /clear or resume from a summary. Make instruction-file changes between sessions so they do not rebuild the cache while you work.";
+            const baseLever = "Use /clear (or resume-from-summary) before idling past the cache TTL, and batch prefix/CLAUDE.md edits to a session boundary so they don't invalidate the warm cache mid-session.";
             const lever = regime5m
                 ? `${baseLever} This session's creation is mostly 5m-tier — enable the 1h cache regime (ENABLE_PROMPT_CACHING_1H) where long pauses are unavoidable.`
                 : baseLever;
@@ -138,7 +138,7 @@ export const d8Detector = {
                 modeled_savings_u_per_wk: savingsU,
                 modeled_formula: formula,
                 evidence: {
-                    title: `Avoid repeated cache rebuilds: ${events.length} after a long pause`,
+                    title: `Reduce cache-write churn: ${events.length} re-write${events.length === 1 ? "" : "s"} in session`,
                     session_id: sessionId,
                     workspace_id,
                     churn_event_count: events.length,

package/dist/detector/detectors/d9_idle_background_session.js

@@ -62,7 +62,7 @@ export const d9Detector = {
                 modeled_savings_u_per_wk: null,
                 modeled_formula: d9Formula(sidechainCap, D9_UNPRODUCTIVE_FRACTION),
                 evidence: {
-                    title: `Review background-agent work: ${Math.round(share * 100)}% of estimated limit use`,
+                    title: `Review background fan-out: ${Math.round(share * 100)}% cap-weighted sidechain`,
                     workspace_id: row.workspace_id,
                     sidechain_cap_weighted_tokens: sidechainCap,
                     total_cap_weighted_tokens: totalCap,

package/dist/detector/measurement.js

@@ -20,8 +20,6 @@
  *
  * Injected `now: Date` everywhere — never call new Date() in this module.
  */
-import { ESF_EFFECT_WRITER_ENABLED } from "../effects/gate.js";
-import { PREMIUM_MODEL_SQL } from "../ingest/pricing.js";
 import { GLOBAL_WORKSPACE_ID } from "./context-probe.js";
 import { isD1SourceBackedRecommendation, parseD1SourceIdentity } from "./d1-source-identity.js";
 /** Wait ≥1 day after adoption before entering MEASURING. */
@@ -84,14 +82,14 @@ function avgFloorContext(db, scopeWorkspaceId, fromIso, toIso) {
 /**
  * D4 signal: routing-adherence score = ROUND(100 × (1 − premium_share)) over
  * non-sidechain reconciled turns in [fromIso, toIso), where premium = model
- * matches PREMIUM_MODEL_SQL (Opus/Fable/Mythos). The data-model's "mechanical turn" classifier is not yet
+ * contains 'opus'. The data-model's "mechanical turn" classifier is not yet
  * shipped, so all reconciled turns are the mechanical set (conservative proxy —
  * documented in the W4 build digest).
  */
 function routingAdherenceScore(db, scopeWorkspaceId, fromIso, toIso) {
     const scoped = scopeWorkspaceId !== null;
     const sql = `SELECT COUNT(*) AS n,
-                      AVG(CASE WHEN ${PREMIUM_MODEL_SQL} THEN 1.0 ELSE 0.0 END) AS premium_share
+                      AVG(CASE WHEN model LIKE '%opus%' THEN 1.0 ELSE 0.0 END) AS premium_share
                  FROM turns
                 WHERE is_sidechain = 0 AND provisional = 0 AND ts >= ? AND ts < ? ${scoped ? "AND workspace_id = ?" : ""}`;
     const params = scoped ? [fromIso, toIso, scopeWorkspaceId] : [fromIso, toIso];
@@ -136,8 +134,7 @@ function handlerFor(rec) {
 }
 /** Warning-class recs (D5 / LIMIT / explicit NONE) skip the lifecycle entirely (design §2d). */
 export function isWarningClass(rec) {
-    return (rec.detector_id === "D5" ||
-        rec.category === "LIMIT" ||
+    return (rec.category === "LIMIT" ||
         rec.target_metric === "NONE" ||
         rec.target_metric === "forecast_margin");
 }
@@ -366,9 +363,6 @@ function closeMeasurement(db, rec, now, result) {
  */
 export function runMeasurementPass(db, now, opts) {
     const result = { to_measuring: 0, verdicts: 0, skipped: 0 };
-    // The versioned writer leaves every historical W4 row unchanged, including open rows.
-    if (ESF_EFFECT_WRITER_ENABLED)
-        return result;
     if (!tableExists(db, "context_inventory_history")) {
         console.warn("W4: context_inventory_history not available; measurement pass skipped");
         return result;
@@ -386,16 +380,11 @@ export function runMeasurementPass(db, now, opts) {
         }
     }
     try {
-        // A code-gate rollback must not let W4 take over an already-versioned recommendation.
-        const versionedExclusion = tableExists(db, "effect_cycles")
-            ? "AND NOT EXISTS (SELECT 1 FROM effect_cycles c WHERE c.rec_id=recommendations.rec_id)"
-            : "";
         const rows = db
             .prepare(`SELECT rec_id, detector_id, category, scope_workspace_id, evidence_json,
                 target_metric, state, adopted_at
            FROM recommendations
           WHERE state IN ('ADOPTED', 'MEASURING') AND adopted_at IS NOT NULL
-          ${versionedExclusion}
           ORDER BY adopted_at ASC, rec_id ASC`)
             .all();
         for (const rec of rows) {

package/dist/hook/precompact-checkpoint-hook.mjs

@@ -11,7 +11,6 @@
  * committed). The hook writes no transcript content to stdout/stderr.
  */
 
-import { randomBytes } from "node:crypto";
 import * as fs from "node:fs";
 import * as os from "node:os";
 import * as path from "node:path";
@@ -20,11 +19,9 @@ import { fileURLToPath } from "node:url";
 
 const MAX_COUNT = 20;
 const MAX_BYTES = 500 * 1024 * 1024; // 500 MiB across all snapshots
-const MAX_COLLISION_ATTEMPTS = 16;
 
-// A per-process counter preserves local write ordering.  The random token prevents
-// separately spawned hooks with the same session and millisecond from choosing the
-// same name; exclusive creation below is the final no-overwrite guard.
+// Monotonic counter so two firings in the same millisecond get distinct filenames
+// that still sort in write order (the timestamp dominates; this breaks same-ms ties).
 let snapshotCounter = 0;
 
 /** The directory snapshots are written to (overridable for tests). */
@@ -36,8 +33,7 @@ export function checkpointDir() {
 export function snapshotName(sessionId, now) {
   snapshotCounter = (snapshotCounter + 1) % 1_000_000;
   const stamp = now.toISOString().replace(/[:.]/g, "-");
-  const counter = String(snapshotCounter).padStart(6, "0");
-  return `${encodeURIComponent(sessionId)}-${stamp}-${counter}-${randomBytes(6).toString("hex")}.jsonl`;
+  return `${encodeURIComponent(sessionId)}-${stamp}-${String(snapshotCounter).padStart(6, "0")}.jsonl`;
 }
 
 /**
@@ -47,22 +43,14 @@ export function snapshotName(sessionId, now) {
 export function writeCheckpoint(transcriptPath, sessionId, dir, now = new Date()) {
   if (typeof transcriptPath !== "string" || !fs.existsSync(transcriptPath)) return null;
   fs.mkdirSync(dir, { recursive: true });
-  for (let attempt = 0; attempt < MAX_COLLISION_ATTEMPTS; attempt += 1) {
-    const dest = path.join(dir, snapshotName(sessionId, now));
-    try {
-      fs.copyFileSync(transcriptPath, dest, fs.constants.COPYFILE_EXCL);
-    } catch (error) {
-      if (error?.code === "EEXIST") continue;
-      throw error;
-    }
-    try {
-      fs.chmodSync(dest, 0o600);
-    } catch {
-      // chmod is a no-op boundary on Windows; the user-profile ACL is the equivalent guard.
-    }
-    return dest;
+  const dest = path.join(dir, snapshotName(sessionId, now));
+  fs.copyFileSync(transcriptPath, dest);
+  try {
+    fs.chmodSync(dest, 0o600);
+  } catch {
+    // chmod is a no-op boundary on Windows; the user-profile ACL is the equivalent guard.
   }
-  return null;
+  return dest;
 }
 
 /**
@@ -70,10 +58,9 @@ export function writeCheckpoint(transcriptPath, sessionId, dir, now = new Date()
  * not create. Malformed .jsonl files are deliberately left alone by retention.
  */
 function snapshotTimestamp(name) {
-  const match =
-    /-(\d{4})-(\d{2})-(\d{2})T(\d{2})-(\d{2})-(\d{2})-(\d{3})Z-\d{6}(?:-[a-f0-9]{12})?\.jsonl$/.exec(
-      name,
-    );
+  const match = /-(\d{4})-(\d{2})-(\d{2})T(\d{2})-(\d{2})-(\d{2})-(\d{3})Z-\d{6}\.jsonl$/.exec(
+    name,
+  );
   if (!match) return null;
   const [, year, month, day, hour, minute, second, millisecond] = match;
   const timestamp = Date.parse(

package/dist/ingest/pricing.js

@@ -62,21 +62,6 @@ export function modelTier(model) {
     }
     return null;
 }
-/**
- * SQL predicate matching premium-priced models on a `model` column: Opus plus
- * the Fable/Mythos families (Fable's $10/$50 base is 2× Opus). Shared by every
- * premium-share aggregate so the family list can't drift between queries.
- */
-export const PREMIUM_MODEL_SQL = "(model LIKE '%opus%' OR model LIKE '%fable%' OR model LIKE '%mythos%')";
-/**
- * Input list price ($/MTok) for a model, for ordering model lists
- * most-expensive first. Derived from LIST_PRICES via modelTier so a new tier
- * automatically sorts by its price; unpriceable models return 0 (sort last).
- */
-export function modelInputListPrice(model) {
-    const tier = modelTier(model);
-    return tier === null ? 0 : (LIST_PRICES[tier]?.[0] ?? 0);
-}
 function addDaysIso(iso, days) {
     const d = new Date(iso);
     d.setUTCDate(d.getUTCDate() + days);

package/dist/query/api/cost-per-success.js

@@ -21,7 +21,6 @@ export function getCostPerSuccess(db, workspaceId, from, to) {
             to,
             workspaceId,
             workspaceId,
-            workspaceId,
             from,
             to,
             workspaceId,
@@ -47,14 +46,6 @@ export function getCostPerSuccess(db, workspaceId, from, to) {
            JOIN merged_work_items mwi ON mwi.work_item_id = swl.work_item_id
           WHERE 1 = 1${sessionFilter}
        ),
-       merged_session_links AS (
-         SELECT swl.session_id, COUNT(DISTINCT swl.work_item_id) AS merged_pr_links
-           FROM session_work_links swl
-           JOIN merged_work_items mwi ON mwi.work_item_id = swl.work_item_id
-           JOIN sessions s ON s.session_id = swl.session_id
-          WHERE 1 = 1${sessionFilter}
-          GROUP BY swl.session_id
-       ),
        in_window_sessions AS (
          SELECT s.session_id, s.cost_equiv_u
            FROM sessions s
@@ -73,8 +64,6 @@ export function getCostPerSuccess(db, workspaceId, from, to) {
        SELECT
          (SELECT COUNT(*) FROM merged_work_items) AS merged_pr_count,
          (SELECT COUNT(*) FROM closed_unmerged_work_items) AS closed_unmerged_count,
-         (SELECT COUNT(*) FROM merged_session_links) AS unique_linked_session_count,
-         (SELECT COUNT(*) FROM merged_session_links WHERE merged_pr_links > 1) AS shared_linked_session_count,
          CASE WHEN (SELECT COUNT(*) FROM merged_work_items) = 0 THEN NULL
               ELSE (SELECT COALESCE(SUM(cost_equiv_u), 0) FROM merged_session_costs) * 1.0
                    / (SELECT COUNT(*) FROM merged_work_items)
@@ -102,14 +91,13 @@ export function getCostPerSuccess(db, workspaceId, from, to) {
     };
     return buildResponse(data, {
         claim_kind: "OBS_PROXY",
-        metric_definition_version: "esf-1",
         n: data.merged_pr_count,
         window: { from, to },
         qualification: {
             provisional_excluded: false,
             unpriced_turns: 0,
             claim_kinds_count: 1,
-            note: "Directional (OBS_PROXY): Unique linked-session cost / merged PRs. The terminal-date PR cohort counts each linked session once, even when it links to multiple merged PRs; shared sessions are coverage, not per-PR allocation. Full linked-session lifecycle cost is used, while linkage_coverage_pct is a separate session-start coverage observation. Unknown pricing is not free. Survivorship bias and reviewer dependence remain.",
+            note: "Directional (OBS_PROXY): survivorship bias (heavy-spend sessions that never open a PR are invisible); reviewer-dependence (merge is a human decision, not a quality guarantee); linkage-coverage cap (only linkage_coverage_pct% of in-window sessions are linked to a PR, so unlinked spend is excluded). cost_per_merged_pr_u uses lifecycle attribution: each merged PR carries the full cost of every linked session whenever it ran, so narrowing the window changes the PR population, not the per-PR cost.",
         },
         ...(workspaceId === null ? {} : { drilldown_ids: { workspace_id: workspaceId } }),
     });

package/dist/query/api/delivery.js

@@ -44,20 +44,8 @@ export function getDeliveryMetrics(db, opts) {
                     FROM tool_events te
                    WHERE te.session_id = iws.session_id
                      AND te.commit_sha IS NOT NULL
-                ) AND s.state = 'RECONCILED' AS is_abandoned_session,
-                s.state = 'LIVE'
-                  AND EXISTS (
-                    SELECT 1 FROM tool_events te
-                     WHERE te.session_id = iws.session_id
-                       AND te.tool_name IN ('Bash', 'Edit', 'Write', 'NotebookEdit')
-                  )
-                  AND NOT EXISTS (
-                    SELECT 1 FROM tool_events te
-                     WHERE te.session_id = iws.session_id
-                       AND te.commit_sha IS NOT NULL
-                  ) AS is_live_session
+                ) AS is_abandoned_session
            FROM in_window_sessions iws
-           JOIN sessions s USING (session_id)
        ),
        session_spend AS (
          SELECT session_id,
@@ -79,16 +67,7 @@ export function getDeliveryMetrics(db, opts) {
               CASE WHEN COALESCE(SUM(ss.spend_u), 0) = 0 THEN NULL
                    ELSE SUM(CASE WHEN sf.is_abandoned_session THEN ss.spend_u ELSE 0 END) * 1.0
                         / SUM(ss.spend_u)
-              END AS abandoned_spend_share,
-              COALESCE(SUM(sf.is_abandoned_session), 0) AS no_commit_activity_session_count,
-              COALESCE(SUM(CASE WHEN sf.is_abandoned_session THEN ss.spend_u ELSE 0 END), 0)
-                AS no_commit_activity_spend_u,
-              CASE WHEN COALESCE(SUM(ss.spend_u), 0) = 0 THEN NULL
-                   ELSE SUM(CASE WHEN sf.is_abandoned_session THEN ss.spend_u ELSE 0 END) * 1.0
-                        / SUM(ss.spend_u)
-              END AS no_commit_activity_spend_share,
-              COALESCE(SUM(CASE WHEN sf.is_live_session THEN 1 ELSE 0 END), 0)
-                AS live_session_excluded_from_no_commit_activity_count
+              END AS abandoned_spend_share
          FROM session_flags sf
          JOIN session_spend ss USING (session_id)`)
         .get(...params);
@@ -100,14 +79,13 @@ export function getDeliveryMetrics(db, opts) {
     };
     return buildResponse(data, {
         claim_kind: "OBS_PROXY",
-        metric_definition_version: "esf-1",
         n: data.commit_session_count,
         window: { from: opts.from, to: opts.to },
         qualification: {
             provisional_excluded: true,
             unpriced_turns: 0,
             claim_kinds_count: 1,
-            note: "Commit SHA presence is an observed delivery proxy. No-commit activity means RECONCILED Bash/Edit/Write/NotebookEdit activity without an observed commit; LIVE sessions remain in total spend but are excluded from that activity cohort. Legacy abandoned_* fields are compatibility aliases, not an abandonment finding.",
+            note: "Commit SHA presence is an observed delivery proxy.",
         },
         ...(opts.workspaceId === null ? {} : { drilldown_ids: { workspace_id: opts.workspaceId } }),
     });

package/dist/query/api/effectiveness.js

@@ -34,17 +34,11 @@ export function computeSessionDelivery(db, sessionId) {
         turns_to_first_commit = countRow.n;
     }
     const sessRow = db
-        .prepare(`SELECT user_turn_count, state,
-              EXISTS (SELECT 1 FROM tool_events te WHERE te.session_id = sessions.session_id
-                      AND te.tool_name IN ('Bash','Write','Edit','NotebookEdit')) AS has_activity
-         FROM sessions WHERE session_id = ?`)
+        .prepare("SELECT user_turn_count, state FROM sessions WHERE session_id = ?")
         .get(sessionId);
     const user_turn_count = sessRow?.user_turn_count ?? 0;
     const state = sessRow?.state ?? "";
-    const deep_abandoned = user_turn_count >= 10 &&
-        firstCommitTs === null &&
-        state === "RECONCILED" &&
-        sessRow?.has_activity === 1;
+    const deep_abandoned = user_turn_count >= 10 && firstCommitTs === null && state === "RECONCILED";
     return { turns_to_first_commit, deep_abandoned };
 }
 // ---------------------------------------------------------------------------
@@ -89,9 +83,8 @@ export function getAbandonedSpendSplit(db, opts) {
                     FROM tool_events te
                    WHERE te.session_id = iws.session_id
                      AND te.commit_sha IS NOT NULL
-                ) AND s.state = 'RECONCILED' AS is_abandoned_session
+                ) AS is_abandoned_session
            FROM in_window_sessions iws
-           JOIN sessions s USING (session_id)
        ),
        session_spend AS (
          SELECT session_id,
@@ -152,24 +145,15 @@ export function getClosureProxy(db, opts) {
         unresolved_count: 0,
         pending_count: 0,
         resolved_share: null,
-        no_later_workspace_session_count: 0,
-        later_workspace_session_count: 0,
-        no_later_workspace_session_share: null,
         window_hours: windowHours,
         workspace_id: opts.workspaceId,
     });
     if (candidates.length === 0) {
         return buildResponse(emptyProxy(), {
             claim_kind: "EXPERIMENTAL",
-            metric_definition_version: "esf-1",
             n: 0,
             window: { from: opts.now, to: opts.now },
-            qualification: {
-                provisional_excluded: false,
-                unpriced_turns: 0,
-                claim_kinds_count: 1,
-                note: `${note} No later workspace session is an observation, not task resolution.`,
-            },
+            qualification: { provisional_excluded: false, unpriced_turns: 0, claim_kinds_count: 1, note },
             ...(opts.workspaceId === null ? {} : { drilldown_ids: { workspace_id: opts.workspaceId } }),
         });
     }
@@ -220,23 +204,14 @@ export function getClosureProxy(db, opts) {
         unresolved_count,
         pending_count,
         resolved_share,
-        no_later_workspace_session_count: resolved_count,
-        later_workspace_session_count: unresolved_count,
-        no_later_workspace_session_share: resolved_share,
         window_hours: windowHours,
         workspace_id: opts.workspaceId,
     };
     return buildResponse(proxy, {
         claim_kind: "EXPERIMENTAL",
-        metric_definition_version: "esf-1",
         n: no_commit_session_count,
         window: { from: opts.now, to: opts.now },
-        qualification: {
-            provisional_excluded: false,
-            unpriced_turns: 0,
-            claim_kinds_count: 1,
-            note: `${note} "Resolved" and "unresolved" are legacy aliases for no later workspace session and a later workspace session; neither establishes task resolution.`,
-        },
+        qualification: { provisional_excluded: false, unpriced_turns: 0, claim_kinds_count: 1, note },
         ...(opts.workspaceId === null ? {} : { drilldown_ids: { workspace_id: opts.workspaceId } }),
     });
 }

package/dist/query/api/efficiency-headroom.js

@@ -37,19 +37,22 @@ export function getEfficiencyHeadroom(db, opts) {
          FROM turns
         WHERE ts >= ? AND ts < ? AND provisional = 0`)
         .get(from, to);
-    const opportunities = db
-        .prepare(`SELECT rec_id, modeled_savings_u_per_wk
-         FROM recommendations
-        WHERE state IN ('PROPOSED', 'ADOPTED') AND modeled_savings_u_per_wk IS NOT NULL
-        ORDER BY rec_id`)
-        .all();
     const headroom_u_per_wk = recs.sum_u ?? 0;
     const actual_u_per_wk = spend.actual_u;
-    // Individual recommendation models overlap and their weekly horizon differs from
-    // the trailing spend window, so a combined percentage would be misleading.
-    const headroom_pct = null;
+    const openRecsExist = recs.open_total > 0;
+    // Null when: no spend (avoid /0 → ∞/NaN), or open recs exist but none carries a
+    // savings figure (an all-null modeled set is not a defensible ceiling).
+    let headroom_pct;
+    if (actual_u_per_wk === 0) {
+        headroom_pct = null;
+    }
+    else if (openRecsExist && recs.with_savings === 0) {
+        headroom_pct = null;
+    }
+    else {
+        headroom_pct = headroom_u_per_wk / actual_u_per_wk;
+    }
     const data = {
-        opportunities,
         headroom_u_per_wk,
         actual_u_per_wk,
         headroom_pct,
@@ -59,14 +62,13 @@ export function getEfficiencyHeadroom(db, opts) {
     };
     return buildResponse(data, {
         claim_kind: "EXPERIMENTAL",
-        metric_definition_version: "esf-1",
         n: recs.with_savings,
         window: { from, to },
         qualification: {
             provisional_excluded: true,
             unpriced_turns: 0,
             claim_kinds_count: 1,
-            note: "Individual modeled weekly opportunities are not summed: recommendations can overlap and their weekly horizon differs from the trailing spend window.",
+            note: "Modeled headroom — if every open recommendation were applied; built from unvalidated per-detector fractions.",
         },
     });
 }

package/dist/query/api/outcomes.js

@@ -11,7 +11,6 @@
  *
  * SQL references: Data Model v2 §2 lines 261-275 / 303-314 / 329-334.
  */
-import { PREMIUM_MODEL_SQL } from "../../ingest/pricing.js";
 import { getQueryDb } from "../db-context.js";
 import { buildResponse } from "../envelope.js";
 // ---------------------------------------------------------------------------
@@ -137,7 +136,7 @@ export function listWorkspaceOutcomes() {
     const lrByWs = new Map(lrRows.map((r) => [r.workspace_id, r.linkage_rate]));
     const adherenceRows = db
         .prepare(`SELECT workspace_id, COUNT(*) AS n,
-              AVG(CASE WHEN ${PREMIUM_MODEL_SQL} THEN 1.0 ELSE 0.0 END) AS premium_share
+              AVG(CASE WHEN model LIKE '%opus%' THEN 1.0 ELSE 0.0 END) AS premium_share
          FROM turns
         WHERE is_sidechain = 0 AND provisional = 0
         GROUP BY workspace_id`)