@akagilnc/pi-workflow-roles 0.1.5303 → 0.1.5307

package/dist/public-cli/main.js

@@ -30042,11 +30042,21 @@ function admitFrameSpanFromRows(rows) {
   }
   return {
     models,
-    frameSpan: { startedAt: span.startedAt, endedAt: span.endedAt },
+    frameSpan: { startedAt: span.startedAt, endedAt: span.endedAt, wallMs: endedMs - startedMs },
     partialFirstFrameAt: { status: "present", at: span.startedAt },
     partialLastFrameAt: { status: "present", at: span.endedAt }
   };
 }
+function sumAnalystRunWallMs(runs) {
+  let total = 0;
+  for (const run of runs) total += run.frameSpan.wallMs;
+  return total;
+}
+function compareAnalystLegs(a, b) {
+  if (a.book !== b.book) return a.book.localeCompare(b.book);
+  if (a.role !== b.role) return a.role.localeCompare(b.role);
+  return a.runId.localeCompare(b.runId);
+}
 async function classifyScopedRun(input) {
   const missingSources = [];
   const reasons = [];
@@ -30327,9 +30337,6 @@ var init_analyst_ledger = __esm({
 function isRecord11(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-function wallMsFromSpan(span) {
-  return Date.parse(span.endedAt) - Date.parse(span.startedAt);
-}
 function findCollectorGroups(body) {
   if (Array.isArray(body.groups)) return body.groups;
   const receipt = body.receipt;
@@ -30438,7 +30445,7 @@ function projectLegs(runs) {
       book: run.book,
       role: run.role,
       startedAt: run.frameSpan.startedAt,
-      wallMs: wallMsFromSpan(run.frameSpan),
+      wallMs: run.frameSpan.wallMs,
       terminalLabel: mapped.terminalLabel,
       accepted: mapped.accepted,
       success: mapped.success,
@@ -30448,11 +30455,7 @@ function projectLegs(runs) {
       rework: ordinal >= 2
     });
   }
-  return legs.sort((a, b) => {
-    if (a.book !== b.book) return a.book.localeCompare(b.book);
-    if (a.role !== b.role) return a.role.localeCompare(b.role);
-    return a.runId.localeCompare(b.runId);
-  });
+  return legs.sort(compareAnalystLegs);
 }
 function aggregateByRole(legs) {
   const roles = [...new Set(legs.map((leg) => leg.role))].sort((a, b) => a.localeCompare(b));
@@ -30496,12 +30499,10 @@ function aggregateByRole(legs) {
     };
   });
 }
-function reworkLens(legs) {
+function reworkLens(legs, totalWallMs) {
   let reworkWallMs = 0;
-  let totalWallMs = 0;
   let reworkLegCount = 0;
   for (const leg of legs) {
-    totalWallMs += leg.wallMs;
     if (leg.rework) {
       reworkWallMs += leg.wallMs;
       reworkLegCount += 1;
@@ -30522,13 +30523,14 @@ function buildAcceptanceSuccessReworkSection(runs) {
     kind: "analyst-acceptance-success-rework",
     legs,
     byRole: aggregateByRole(legs),
-    rework: reworkLens(legs)
+    rework: reworkLens(legs, sumAnalystRunWallMs(runs))
   };
 }
 var WORKER_ROLES, ACCEPTED_STATUS, SUCCESS_STATUS, acceptanceSuccessReworkFamily, acceptance_success_rework_default;
 var init_acceptance_success_rework = __esm({
   "src/analyst-metric-families/acceptance-success-rework.ts"() {
     "use strict";
+    init_analyst_ledger();
     init_analyst_median();
     init_packaged_role_registry();
     WORKER_ROLES = /* @__PURE__ */ new Set(["coder", "fixer"]);
@@ -30561,117 +30563,6 @@ var init_acceptance_success_rework = __esm({
   }
 });
 
-// src/analyst-model-groups.ts
-function analystModelGroupKey(models) {
-  if (models.length === 0) return void 0;
-  if (models.length === 1) return models[0];
-  return `mixed:${models.join("+")}`;
-}
-function rate(numerator, denominator) {
-  if (denominator === 0) return void 0;
-  return numerator / denominator;
-}
-function displayNameFor(rawGroupKey, combinationMapping) {
-  if (combinationMapping === void 0) return rawGroupKey;
-  const aliased = combinationMapping[rawGroupKey];
-  return aliased === void 0 ? rawGroupKey : aliased;
-}
-function sortUnreadable(unreadable) {
-  return [...unreadable].sort((a, b) => {
-    if (a.book !== b.book) return a.book.localeCompare(b.book);
-    return a.runId.localeCompare(b.runId);
-  });
-}
-function modelIdentityAbsentEntry(run) {
-  return {
-    runId: run.runId,
-    book: run.book,
-    missingSources: ["session-model"],
-    reason: "session has no usable model identity",
-    firstFrameAt: { status: "present", at: run.frameSpan.startedAt },
-    // Readable legs already admitted a full span — retain end edge for lastActivityAt.
-    lastFrameAt: { status: "present", at: run.frameSpan.endedAt }
-  };
-}
-function buildAnalystModelGroupsPage(input) {
-  const groupedRuns = [];
-  const modelAbsent = [];
-  for (const run of input.runs) {
-    const rawGroupKey = analystModelGroupKey(run.models);
-    if (rawGroupKey === void 0) {
-      modelAbsent.push(modelIdentityAbsentEntry(run));
-    } else {
-      groupedRuns.push({ run, rawGroupKey });
-    }
-  }
-  const acceptance = buildAcceptanceSuccessReworkSection(
-    groupedRuns.map(({ run }) => run)
-  );
-  const legByRunId = new Map(
-    (acceptance?.legs ?? []).map((leg) => [leg.runId, leg])
-  );
-  const byRaw = /* @__PURE__ */ new Map();
-  for (const { run, rawGroupKey } of groupedRuns) {
-    const leg = legByRunId.get(run.runId);
-    if (leg === void 0) {
-      throw new Error(
-        `analyst model-groups: missing acceptance projection for run ${run.runId}`
-      );
-    }
-    let acc = byRaw.get(rawGroupKey);
-    if (acc === void 0) {
-      acc = {
-        acceptedCount: 0,
-        successCount: 0,
-        successEligibleCount: 0,
-        noReceiptCount: 0,
-        walls: []
-      };
-      byRaw.set(rawGroupKey, acc);
-    }
-    if (leg.accepted) acc.acceptedCount += 1;
-    if (leg.success) acc.successCount += 1;
-    if (leg.successEligible) acc.successEligibleCount += 1;
-    if (leg.noReceipt) acc.noReceiptCount += 1;
-    acc.walls.push(leg.wallMs);
-  }
-  const rawKeys = [...byRaw.keys()].sort((a, b) => a.localeCompare(b));
-  const groups = rawKeys.map((rawGroupKey) => {
-    const acc = byRaw.get(rawGroupKey);
-    const legCount = acc.walls.length;
-    return {
-      rawGroupKey,
-      displayName: displayNameFor(rawGroupKey, input.combinationMapping),
-      legCount,
-      acceptedCount: acc.acceptedCount,
-      acceptanceRate: rate(acc.acceptedCount, legCount),
-      successCount: acc.successCount,
-      successEligibleCount: acc.successEligibleCount,
-      successRate: rate(acc.successCount, acc.successEligibleCount),
-      noReceiptCount: acc.noReceiptCount,
-      noReceiptRate: rate(acc.noReceiptCount, legCount),
-      wallClockMedianMs: medianNumber(acc.walls)
-    };
-  });
-  const unreadable = sortUnreadable([...input.unreadable, ...modelAbsent]);
-  return {
-    kind: "analyst-model-groups",
-    mode: "model-groups",
-    projectRoots: [...input.projectRoots].sort((a, b) => a.localeCompare(b)),
-    groups,
-    legCount: groupedRuns.length,
-    unreadableCount: unreadable.length,
-    unreadable
-  };
-}
-var init_analyst_model_groups = __esm({
-  "src/analyst-model-groups.ts"() {
-    "use strict";
-    init_acceptance_success_rework();
-    init_analyst_median();
-  }
-});
-
 // src/analyst-metric-families/b2-frame-buckets-actions.ts
 function timestampMs(iso) {
   const ms = Date.parse(iso);
@@ -30794,7 +30685,7 @@ function sortActionsDescending(actions) {
 function computeAnalystB2RunMetrics(facts) {
   const frameStartMs = timestampMs(facts.frameSpan.startedAt);
   const frameEndMs = timestampMs(facts.frameSpan.endedAt);
-  const wallMs = Math.max(0, frameEndMs - frameStartMs);
+  const wallMs = facts.frameSpan.wallMs;
   const tools = clipToolsToFrame(closedTools(facts.toolIntervals), frameStartMs, frameEndMs);
   const toolUnion = mergeUnion(tools);
   const toolBucketMs = toolUnion.reduce(
@@ -30824,15 +30715,12 @@ var init_b2_frame_buckets_actions = __esm({
   "src/analyst-metric-families/b2-frame-buckets-actions.ts"() {
     "use strict";
     init_analyst_median();
+    init_analyst_ledger();
     b2FrameBucketsActionsFamily = {
       id: "b2-frame-buckets-actions",
       contribute(input) {
         if (input.runs.length === 0) return void 0;
-        const runs = [...input.runs].map(computeAnalystB2RunMetrics).sort((a, b) => {
-          if (a.book !== b.book) return a.book.localeCompare(b.book);
-          if (a.role !== b.role) return a.role.localeCompare(b.role);
-          return a.runId.localeCompare(b.runId);
-        });
+        const runs = [...input.runs].map(computeAnalystB2RunMetrics).sort(compareAnalystLegs);
         const section = {
           kind: "analyst-b2-frame-buckets-actions",
           runs
@@ -30865,11 +30753,6 @@ function projectLeg(facts) {
     rounds
   };
 }
-function compareLegs(a, b) {
-  if (a.book !== b.book) return a.book.localeCompare(b.book);
-  if (a.role !== b.role) return a.role.localeCompare(b.role);
-  return a.runId.localeCompare(b.runId);
-}
 function emptyAccum() {
   return { rounds: 0, bounceCount: 0, passCount: 0, wallSum: 0 };
 }
@@ -30905,11 +30788,12 @@ var gateCyclesFamily, gate_cycles_default;
 var init_gate_cycles = __esm({
   "src/analyst-metric-families/gate-cycles.ts"() {
     "use strict";
+    init_analyst_ledger();
     gateCyclesFamily = {
       id: "gate-cycles",
       contribute(input) {
         if (input.runs.length === 0) return void 0;
-        const legs = input.runs.map(projectLeg).sort(compareLegs);
+        const legs = input.runs.map(projectLeg).sort(compareAnalystLegs);
         const section = {
           kind: "analyst-gate-cycles",
           legs,
@@ -30923,28 +30807,24 @@ var init_gate_cycles = __esm({
 });
 
 // src/analyst-metric-families/leg-wall-clock.ts
-function frameSpanWallMs(span) {
-  return Date.parse(span.endedAt) - Date.parse(span.startedAt);
-}
 function projectEntry(facts) {
   return {
     runId: facts.runId,
     book: facts.book,
     role: facts.role,
-    wallMs: frameSpanWallMs(facts.frameSpan)
+    wallMs: facts.frameSpan.wallMs
   };
 }
 function compareRankingDesc(a, b) {
   if (b.wallMs !== a.wallMs) return b.wallMs - a.wallMs;
-  if (a.book !== b.book) return a.book.localeCompare(b.book);
-  if (a.role !== b.role) return a.role.localeCompare(b.role);
-  return a.runId.localeCompare(b.runId);
+  return compareAnalystLegs(a, b);
 }
 var legWallClockFamily, leg_wall_clock_default;
 var init_leg_wall_clock = __esm({
   "src/analyst-metric-families/leg-wall-clock.ts"() {
     "use strict";
     init_analyst_median();
+    init_analyst_ledger();
     legWallClockFamily = {
       id: "leg-wall-clock",
       contribute(input) {
@@ -30957,13 +30837,11 @@ var init_leg_wall_clock = __esm({
         if (medianWallMs === void 0) {
           return void 0;
         }
-        let totalElapsedMs = 0;
-        for (const wallMs of walls) totalElapsedMs += wallMs;
         const section = {
           kind: "analyst-leg-wall-clock",
           ranking,
           medianWallMs,
-          totalElapsedMs
+          totalElapsedMs: sumAnalystRunWallMs(input.runs)
         };
         return { legWallClock: section };
       }
@@ -30976,9 +30854,6 @@ var init_leg_wall_clock = __esm({
 function isRecord12(value) {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
-function wallMsFromSpan2(startedAt, endedAt) {
-  return Date.parse(endedAt) - Date.parse(startedAt);
-}
 function readClassCount(body) {
   if (!isRecord12(body.outcome)) return void 0;
   if (!isRecord12(body.outcome.decisiveFacts)) return void 0;
@@ -31015,7 +30890,7 @@ function projectRunRow(facts) {
     role: facts.role,
     startedAt,
     endedAt,
-    wallMs: wallMsFromSpan2(startedAt, endedAt),
+    wallMs: facts.frameSpan.wallMs,
     terminal: projectTerminal(facts)
   };
 }
@@ -31147,13 +31022,9 @@ function analystIssuePageAddressFromPage(page) {
   };
 }
 function sortLegs(legs) {
-  return [...legs].sort((a, b) => {
-    if (a.book !== b.book) return a.book.localeCompare(b.book);
-    if (a.role !== b.role) return a.role.localeCompare(b.role);
-    return a.runId.localeCompare(b.runId);
-  });
+  return [...legs].sort(compareAnalystLegs);
 }
-function sortUnreadable2(unreadable) {
+function sortUnreadable(unreadable) {
   return [...unreadable].sort((a, b) => {
     if (a.book !== b.book) return a.book.localeCompare(b.book);
     return a.runId.localeCompare(b.runId);
@@ -31189,14 +31060,10 @@ function computeAnalystMsPerKLines(totalElapsedMs, changedLines) {
     value: totalElapsedMs / (changedLines.value / 1e3)
   };
 }
-function frameSpanWallMs2(span) {
-  return Date.parse(span.endedAt) - Date.parse(span.startedAt);
-}
 function summarizeAnalystRunEfficiency(runs, unreadable = []) {
-  let totalElapsedMs = 0;
+  const totalElapsedMs = sumAnalystRunWallMs(runs);
   let latestEndedAt;
   for (const run of runs) {
-    totalElapsedMs += frameSpanWallMs2(run.frameSpan);
     const endedAt = run.frameSpan.endedAt;
     if (latestEndedAt === void 0 || endedAt > latestEndedAt) {
       latestEndedAt = endedAt;
@@ -31221,7 +31088,7 @@ async function buildAnalystIssueMetricsPage(input) {
       role: run.role
     }))
   );
-  const unreadable = sortUnreadable2(input.unreadable);
+  const unreadable = sortUnreadable(input.unreadable);
   const scopeConflicts = sortScopeConflicts(input.scopeConflicts ?? []);
   const projectRoot = physicalPathIdentity(input.projectRoot);
   const { totalElapsedMs, lastActivityAt } = summarizeAnalystRunEfficiency(
@@ -31267,11 +31134,118 @@ var init_analyst_page = __esm({
     "use strict";
     init_atomic_write();
     init_activation_ledger_topology();
+    init_analyst_ledger();
     init_analyst_metric_families();
     init_analyst_metric_family();
   }
 });
 
+// src/analyst-model-groups.ts
+function analystModelGroupKey(models) {
+  if (models.length === 0) return void 0;
+  if (models.length === 1) return models[0];
+  return `mixed:${models.join("+")}`;
+}
+function rate(numerator, denominator) {
+  if (denominator === 0) return void 0;
+  return numerator / denominator;
+}
+function displayNameFor(rawGroupKey, combinationMapping) {
+  if (combinationMapping === void 0) return rawGroupKey;
+  const aliased = combinationMapping[rawGroupKey];
+  return aliased === void 0 ? rawGroupKey : aliased;
+}
+function modelIdentityAbsentEntry(run) {
+  return {
+    runId: run.runId,
+    book: run.book,
+    missingSources: ["session-model"],
+    reason: "session has no usable model identity",
+    firstFrameAt: { status: "present", at: run.frameSpan.startedAt },
+    // Readable legs already admitted a full span — retain end edge for lastActivityAt.
+    lastFrameAt: { status: "present", at: run.frameSpan.endedAt }
+  };
+}
+function buildAnalystModelGroupsPage(input) {
+  const groupedRuns = [];
+  const modelAbsent = [];
+  for (const run of input.runs) {
+    const rawGroupKey = analystModelGroupKey(run.models);
+    if (rawGroupKey === void 0) {
+      modelAbsent.push(modelIdentityAbsentEntry(run));
+    } else {
+      groupedRuns.push({ run, rawGroupKey });
+    }
+  }
+  const acceptance = buildAcceptanceSuccessReworkSection(
+    groupedRuns.map(({ run }) => run)
+  );
+  const legByRunId = new Map(
+    (acceptance?.legs ?? []).map((leg) => [leg.runId, leg])
+  );
+  const byRaw = /* @__PURE__ */ new Map();
+  for (const { run, rawGroupKey } of groupedRuns) {
+    const leg = legByRunId.get(run.runId);
+    if (leg === void 0) {
+      throw new Error(
+        `analyst model-groups: missing acceptance projection for run ${run.runId}`
+      );
+    }
+    let acc = byRaw.get(rawGroupKey);
+    if (acc === void 0) {
+      acc = {
+        acceptedCount: 0,
+        successCount: 0,
+        successEligibleCount: 0,
+        noReceiptCount: 0,
+        walls: []
+      };
+      byRaw.set(rawGroupKey, acc);
+    }
+    if (leg.accepted) acc.acceptedCount += 1;
+    if (leg.success) acc.successCount += 1;
+    if (leg.successEligible) acc.successEligibleCount += 1;
+    if (leg.noReceipt) acc.noReceiptCount += 1;
+    acc.walls.push(leg.wallMs);
+  }
+  const rawKeys = [...byRaw.keys()].sort((a, b) => a.localeCompare(b));
+  const groups = rawKeys.map((rawGroupKey) => {
+    const acc = byRaw.get(rawGroupKey);
+    const legCount = acc.walls.length;
+    return {
+      rawGroupKey,
+      displayName: displayNameFor(rawGroupKey, input.combinationMapping),
+      legCount,
+      acceptedCount: acc.acceptedCount,
+      acceptanceRate: rate(acc.acceptedCount, legCount),
+      successCount: acc.successCount,
+      successEligibleCount: acc.successEligibleCount,
+      successRate: rate(acc.successCount, acc.successEligibleCount),
+      noReceiptCount: acc.noReceiptCount,
+      noReceiptRate: rate(acc.noReceiptCount, legCount),
+      wallClockMedianMs: medianNumber(acc.walls)
+    };
+  });
+  const unreadable = sortUnreadable([...input.unreadable, ...modelAbsent]);
+  return {
+    kind: "analyst-model-groups",
+    mode: "model-groups",
+    projectRoots: [...input.projectRoots].sort((a, b) => a.localeCompare(b)),
+    groups,
+    legCount: groupedRuns.length,
+    unreadableCount: unreadable.length,
+    unreadable
+  };
+}
+var init_analyst_model_groups = __esm({
+  "src/analyst-model-groups.ts"() {
+    "use strict";
+    init_acceptance_success_rework();
+    init_analyst_median();
+    init_analyst_page();
+  }
+});
+
 // src/analyst-entry.ts
 import { readFile as readFile20 } from "node:fs/promises";
 function isMissingPathError7(error) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@akagilnc/pi-workflow-roles",
-  "version": "0.1.5303",
+  "version": "0.1.5307",
   "description": "Soul-bound workflow roles for Pi",
   "type": "module",
   "license": "Apache-2.0",

package/src/analyst-ledger.ts

@@ -319,7 +319,7 @@ function admitFrameSpanFromRows(rows: readonly LedgerSessionRow[]): {
   }
   return {
     models,
-    frameSpan: { startedAt: span.startedAt, endedAt: span.endedAt },
+    frameSpan: { startedAt: span.startedAt, endedAt: span.endedAt, wallMs: endedMs - startedMs },
     partialFirstFrameAt: { status: "present", at: span.startedAt },
     partialLastFrameAt: { status: "present", at: span.endedAt },
   };
@@ -329,6 +329,8 @@ function admitFrameSpanFromRows(rows: readonly LedgerSessionRow[]): {
 export type AnalystRunFrameSpan = {
   readonly startedAt: string;
   readonly endedAt: string;
+  /** Validated wall clock, computed once when the session span is admitted. */
+  readonly wallMs: number;
 };
 
 /**
@@ -371,6 +373,23 @@ export type AnalystReadableRunFacts = {
   readonly gateCycles: readonly AnalystGateCycleRound[];
 };
 
+/** Total wall clock for exactly the readable runs supplied by the caller. */
+export function sumAnalystRunWallMs(runs: readonly AnalystReadableRunFacts[]): number {
+  let total = 0;
+  for (const run of runs) total += run.frameSpan.wallMs;
+  return total;
+}
+
+/** Shared identity order for readable legs and their page/metric projections. */
+export function compareAnalystLegs(
+  a: Pick<AnalystReadableRunFacts, "book" | "role" | "runId">,
+  b: Pick<AnalystReadableRunFacts, "book" | "role" | "runId">,
+): number {
+  if (a.book !== b.book) return a.book.localeCompare(b.book);
+  if (a.role !== b.role) return a.role.localeCompare(b.role);
+  return a.runId.localeCompare(b.runId);
+}
+
 export type AnalystScopedRunScan = {
   /** Readable in-scope runs with retained typed facts (A2). */
   readonly runs: readonly AnalystReadableRunFacts[];

package/src/analyst-metric-families/acceptance-success-rework.ts

@@ -9,7 +9,7 @@
  * - success rate den = success-eligible accepted legs (no-receipt out; planned out)
  * - planned = plan-duty acceptance; never success numerator or denominator
  */
-import type { AnalystReadableRunFacts, AnalystRunTerminalFace } from "../analyst-ledger.ts";
+import { compareAnalystLegs, sumAnalystRunWallMs, type AnalystReadableRunFacts, type AnalystRunTerminalFace } from "../analyst-ledger.ts";
 import { medianNumber } from "../analyst-median.ts";
 import type { AnalystMetricFamilyModule } from "../analyst-metric-family.ts";
 import { packagedAnalystTerminal } from "../packaged-role-registry.ts";
@@ -99,10 +99,6 @@ function isRecord(value: unknown): value is Record<string, unknown> {
   return typeof value === "object" && value !== null && !Array.isArray(value);
 }
 
-function wallMsFromSpan(span: { readonly startedAt: string; readonly endedAt: string }): number {
-  return Date.parse(span.endedAt) - Date.parse(span.startedAt);
-}
-
 /** Locate collector groups array: receipt.groups or top-level groups. */
 function findCollectorGroups(body: Record<string, unknown>): unknown {
   if (Array.isArray(body.groups)) return body.groups;
@@ -235,7 +231,7 @@ function projectLegs(runs: readonly AnalystReadableRunFacts[]): AnalystAcceptanc
       book: run.book,
       role: run.role,
       startedAt: run.frameSpan.startedAt,
-      wallMs: wallMsFromSpan(run.frameSpan),
+      wallMs: run.frameSpan.wallMs,
       terminalLabel: mapped.terminalLabel,
       accepted: mapped.accepted,
       success: mapped.success,
@@ -247,11 +243,7 @@ function projectLegs(runs: readonly AnalystReadableRunFacts[]): AnalystAcceptanc
   }
 
   // Stable page order: book, role, runId (match A1 leg sort).
-  return legs.sort((a, b) => {
-    if (a.book !== b.book) return a.book.localeCompare(b.book);
-    if (a.role !== b.role) return a.role.localeCompare(b.role);
-    return a.runId.localeCompare(b.runId);
-  });
+  return legs.sort(compareAnalystLegs);
 }
 
 function aggregateByRole(legs: readonly AnalystAcceptanceLeg[]): AnalystRoleAcceptanceStats[] {
@@ -302,12 +294,10 @@ function aggregateByRole(legs: readonly AnalystAcceptanceLeg[]): AnalystRoleAcce
   });
 }
 
-function reworkLens(legs: readonly AnalystAcceptanceLeg[]): AnalystReworkLens {
+function reworkLens(legs: readonly AnalystAcceptanceLeg[], totalWallMs: number): AnalystReworkLens {
   let reworkWallMs = 0;
-  let totalWallMs = 0;
   let reworkLegCount = 0;
   for (const leg of legs) {
-    totalWallMs += leg.wallMs;
     if (leg.rework) {
       reworkWallMs += leg.wallMs;
       reworkLegCount += 1;
@@ -331,7 +321,7 @@ export function buildAcceptanceSuccessReworkSection(
     kind: "analyst-acceptance-success-rework",
     legs,
     byRole: aggregateByRole(legs),
-    rework: reworkLens(legs),
+    rework: reworkLens(legs, sumAnalystRunWallMs(runs)),
   };
 }
 

package/src/analyst-metric-families/b2-frame-buckets-actions.ts

@@ -16,7 +16,7 @@
  */
 import type { SessionToolInterval } from "../ledger-session-read.ts";
 import { medianNumber } from "../analyst-median.ts";
-import type { AnalystReadableRunFacts } from "../analyst-ledger.ts";
+import { compareAnalystLegs, type AnalystReadableRunFacts } from "../analyst-ledger.ts";
 import type { AnalystMetricFamilyModule } from "../analyst-metric-family.ts";
 
 export type AnalystB2ToolAction = {
@@ -221,7 +221,7 @@ export function computeAnalystB2RunMetrics(
 ): AnalystB2RunMetrics {
   const frameStartMs = timestampMs(facts.frameSpan.startedAt);
   const frameEndMs = timestampMs(facts.frameSpan.endedAt);
-  const wallMs = Math.max(0, frameEndMs - frameStartMs);
+  const wallMs = facts.frameSpan.wallMs;
 
   // Single frame-bounded core: clip → union once → bucket/complement/actions.
   const tools = clipToolsToFrame(closedTools(facts.toolIntervals), frameStartMs, frameEndMs);
@@ -258,11 +258,7 @@ const b2FrameBucketsActionsFamily: AnalystMetricFamilyModule = {
     if (input.runs.length === 0) return undefined;
     const runs = [...input.runs]
       .map(computeAnalystB2RunMetrics)
-      .sort((a, b) => {
-        if (a.book !== b.book) return a.book.localeCompare(b.book);
-        if (a.role !== b.role) return a.role.localeCompare(b.role);
-        return a.runId.localeCompare(b.runId);
-      });
+      .sort(compareAnalystLegs);
     const section: AnalystB2FrameBucketsActionsSection = {
       kind: "analyst-b2-frame-buckets-actions",
       runs,

package/src/analyst-metric-families/gate-cycles.ts

@@ -9,7 +9,7 @@ import type {
   AnalystGateCycleOrigin,
   AnalystGateCycleRound,
 } from "../analyst-gate-cycles-read.ts";
-import type { AnalystReadableRunFacts } from "../analyst-ledger.ts";
+import { compareAnalystLegs, type AnalystReadableRunFacts } from "../analyst-ledger.ts";
 import type { AnalystMetricFamilyModule } from "../analyst-metric-family.ts";
 
 export type AnalystGateCyclesRoundRow = {
@@ -73,12 +73,6 @@ function projectLeg(facts: AnalystReadableRunFacts): AnalystGateCyclesLeg {
   };
 }
 
-function compareLegs(a: AnalystGateCyclesLeg, b: AnalystGateCyclesLeg): number {
-  if (a.book !== b.book) return a.book.localeCompare(b.book);
-  if (a.role !== b.role) return a.role.localeCompare(b.role);
-  return a.runId.localeCompare(b.runId);
-}
-
 type OfficerAccum = {
   rounds: number;
   bounceCount: number;
@@ -132,7 +126,7 @@ const gateCyclesFamily: AnalystMetricFamilyModule = {
   id: "gate-cycles",
   contribute(input) {
     if (input.runs.length === 0) return undefined;
-    const legs = input.runs.map(projectLeg).sort(compareLegs);
+    const legs = input.runs.map(projectLeg).sort(compareAnalystLegs);
     const section: AnalystGateCyclesSection = {
       kind: "analyst-gate-cycles",
       legs,