@worca/app 1.3.0-rc.2 → 1.4.0-rc.1

package/ui/public/team-metrics-surfaces.mjs

@@ -15,6 +15,16 @@ function h(doc, tag, cls, text) {
 const dayLabel = (iso) => { const d = new Date(iso); return Number.isFinite(d.getTime()) ? `${MO[d.getUTCMonth()]} ${d.getUTCDate()}` : ''; };
 const dot = (doc, tone) => { const d = h(doc, 'span', `dot tm-dot ${tone}`); d.setAttribute('aria-hidden', 'true'); return d; };
 const code = (doc, text) => h(doc, 'code', 'mono', text);
+/** A project, workspace or home NAME inside prose: bold and mono, so it reads as a name, not a word. */
+const ref = (doc, name) => h(doc, 'b', 'ref mono', name);
+/** `text` with every occurrence of `name` rendered as a ref; plain text when the name is absent. */
+function withRef(doc, text, name) {
+  const frag = doc.createDocumentFragment();
+  if (!name || !String(text).includes(name)) { frag.append(String(text)); return frag; }
+  const parts = String(text).split(name);
+  parts.forEach((part, i) => { if (i) frag.append(ref(doc, name)); if (part) frag.append(part); });
+  return frag;
+}
 function btn(doc, cls, text) { const b = h(doc, 'button', `btn-ghost btn-mini ${cls}`, text); b.type = 'button'; return b; }
 
 /** Classify a project status (from /api/team-metrics/scopes → projects[]) into a cell variant. */
@@ -44,19 +54,61 @@ function recordSwitch(doc, s) {
   return label;
 }
 
+const cap = (t) => (t ? t[0].toUpperCase() + t.slice(1) : t);
+/**
+ * One project's team-metrics state in a few words: `short` is what a row chip and the project
+ * page's stat card show ("on · 3 runs", "via acme/gateway", "push failed"), `detail` the sentence
+ * behind it, `tone` the dot colour. The full cell (renderProjectTmCell) says the same at length.
+ */
+export function projectTmSummary(s) {
+  const { kind } = projectTmState(s);
+  const excluded = s.record === false;
+  const runs = s.runs == null ? '' : `${s.runs} run${s.runs === 1 ? '' : 's'}`;
+  const since = s.enabledAt ? `since ${dayLabel(s.enabledAt)}` : '';
+  switch (kind) {
+    case 'no-origin': return { kind, tone: 'muted', short: 'not available', detail: s.noGit ? 'not a git repository' : 'no origin remote' };
+    case 'off': return { kind, tone: 'grey', short: 'off', detail: 'runs stay on this machine' };
+    case 'delegate-invalid': return { kind, tone: 'red', short: 'delegate invalid', detail: `points at ${s.delegateTo} · runs are not being recorded` };
+    case 'blocked': return { kind, tone: 'amber', short: 'on · not recording', detail: s.delegateCode === 'CONFIG_UNKNOWN' ? 'branch not read yet' : (s.delegateDetail || 'the metrics branch could not be resolved') };
+    case 'rejected': return { kind, tone: 'red', short: 'push failed', detail: s.lastErrorCode === 'PUSH_REJECTED' && s.lastErrorHint ? 'branch protection' : String(s.lastError || '').trim().split('\n')[0] };
+    case 'pending': return { kind, tone: 'amber', short: `on · ${s.pending} pending push`, detail: since };
+    case 'delegated': return { kind, tone: excluded ? 'grey' : 'green', short: excluded ? `via ${s.delegateTo} · yours excluded` : `via ${s.delegateTo}`, ref: s.delegateTo, detail: [runs ? `${runs} recorded` : '', 'recorded in that project\'s branch'].filter(Boolean).join(' · ') };
+    default: return { kind, tone: excluded ? 'grey' : 'green', short: excluded ? 'on · yours excluded' : (runs ? `on · ${runs}` : 'on'), detail: since };
+  }
+}
+
+/**
+ * The Projects-row chip: a dot, the word and the short state — no controls, no hints. Everything
+ * it leaves out is on the project page's Team tab (renderProjectTmCell). Expert-level like the
+ * page tab; the row itself is the only control.
+ */
+export function renderProjectTmChip(s, { doc = globalThis.document } = {}) {
+  const chip = h(doc, 'span', 'pl-team-item pl-tm');
+  chip.dataset.key = s.key;
+  const sum = projectTmSummary(s);
+  chip.dataset.kind = sum.kind;
+  if (sum.tone !== 'muted') chip.append(dot(doc, sum.tone));
+  const state = h(doc, 'span', `pl-team-state${sum.tone === 'muted' ? ' muted' : ''}`);
+  state.append(withRef(doc, sum.short, sum.ref));
+  chip.append(h(doc, 'span', 'pl-team-name', 'Metrics'), ' ', state);
+  chip.title = `Team metrics: ${cap(sum.short)}${sum.detail ? ` · ${sum.detail}` : ''}`;
+  return chip;
+}
+
 /**
- * The Projects-list cell is a fixed two-row block: the "Team metrics" title over a
- * status line on the left, and ONE control column on the right that spans both
- * rows (see .tm-cell in style.css). The control never wraps under the status,
- * and the title makes clear what the control acts on.
+ * The project page's team-metrics block (and, before this branch, the Projects-list cell): a
+ * fixed two-row grid — the "Team metrics" title over a status line on the left, and ONE control
+ * column on the right that spans both rows (see .tm-cell in style.css). The control never wraps
+ * under the status. `heading: false` drops the title when a panel head already names it.
  */
-export function renderProjectTmCell(s, { doc = globalThis.document } = {}) {
+export function renderProjectTmCell(s, { doc = globalThis.document, heading = true } = {}) {
   const cell = h(doc, 'div', 'tm-cell');
   cell.dataset.key = s.key;
-  cell.append(h(doc, 'span', 'tm-label', 'Team metrics'));
+  if (heading) cell.append(h(doc, 'span', 'tm-label', 'Team metrics'));
   const line = h(doc, 'div', 'tm-line');
   const status = h(doc, 'span', 'tm-status');
   const { kind } = projectTmState(s);
+  cell.dataset.kind = kind;   // the Getting started guide reads it (no-origin / off / on)
   const actions = h(doc, 'div', 'tm-actions');
   const runs = s.runs == null ? '' : ` · ${s.runs} run${s.runs === 1 ? '' : 's'}`;
   const excluded = s.record === false;   // personal opt-out (the "Include my runs" switch)
@@ -74,7 +126,7 @@ export function renderProjectTmCell(s, { doc = globalThis.document } = {}) {
     case 'delegate-invalid':
       line.append(dot(doc, 'red'));
       // The resolver's code distinguishes chain / unknown from dangling; dangling keeps the mockup copy.
-      status.append(`Delegate invalid · points at `, code(doc, s.delegateTo),
+      status.append(`Delegate invalid · points at `, ref(doc, s.delegateTo),
         s.delegateCode === 'DELEGATE_CHAIN' ? ', which itself delegates (no chains)'
           : s.delegateCode === 'DELEGATE_UNKNOWN' ? ', which is not a project in Worca on this machine'
             : ', which no longer records');
@@ -102,8 +154,8 @@ export function renderProjectTmCell(s, { doc = globalThis.document } = {}) {
       line.append(dot(doc, excluded ? 'grey' : 'green'));
       // Kept short: the status shares ~220px with the switch (see .tm-status), so the
       // opted-out copy leads with what matters and drops the run count.
-      if (excluded) status.append('On for the team · in ', code(doc, s.delegateTo), ' · yours excluded');
-      else status.append('On · recorded in ', code(doc, s.delegateTo), runs);
+      if (excluded) status.append('On for the team · in ', ref(doc, s.delegateTo), ' · yours excluded');
+      else status.append('On · recorded in ', ref(doc, s.delegateTo), runs);
       actions.append(recordSwitch(doc, s));
       break;
     default:
@@ -135,7 +187,9 @@ export function renderProjectTmCell(s, { doc = globalThis.document } = {}) {
   if (s.enabled && Array.isArray(s.homeFor) && s.homeFor.length) {
     const names = s.homeFor.join(', ');
     const hint = h(doc, 'small', 'hint tm-hint tm-home-for');
-    hint.append(`Metrics home for ${names} · "Include my runs" covers its workspace runs too`);
+    hint.append('Metrics home for ');
+    s.homeFor.forEach((x, i) => { if (i) hint.append(', '); hint.append(ref(doc, x)); });
+    hint.append(' · "Include my runs" covers its workspace runs too');
     hint.title = `Workspace runs of ${names} are written to this project's branch. Turning "Include my runs" off here also stops your workspace runs from being recorded.`;
     cell.append(hint);
   }
@@ -230,7 +284,7 @@ export function renderMetricsHomePicker(members, { selectedPath, doc = globalThi
       status.append(dot(doc, 'green'), lines);
       row.append(label, status);
     } else if (m.enabled && m.delegateTo) {
-      status.append(dot(doc, 'grey'), `Delegates to ${m.delegateTo} · cannot host metrics`);
+      status.append(dot(doc, 'grey'), 'Delegates to ', ref(doc, m.delegateTo), ' · cannot host metrics');
       row.classList.add('off');
       row.append(name, status);
     } else {
@@ -276,22 +330,29 @@ function homeMark(doc) {
  * Answers what people scan for — how big, where runs go, is anything wrong — on one line.
  */
 export function renderWsSummary(ws, { doc = globalThis.document, pending = false } = {}) {
-  const frag = doc.createDocumentFragment();
+  let frag = doc.createDocumentFragment();
   const n = (ws.projectPaths || ws.members || []).length;
   frag.append(`${n} project${n === 1 ? '' : 's'}`);
+  // Everything after the count is team-metrics state: expert detail (docs/ui-levels.md), so it rides
+  // one tagged span the stylesheet can drop. `frag` is re-pointed so the appends below land in it.
+  const out = frag;
+  const tm = h(doc, 'span', 'ws-sum-tm');
+  tm.dataset.minLevel = 'expert';
+  out.append(tm);
+  frag = tm;
   // The scopes call is still out: say so instead of "no metrics home", which reads as final.
-  if (pending) { frag.append(' · ', h(doc, 'span', 'ws-sum-pending', 'checking metrics…')); return frag; }
+  if (pending) { frag.append(' · ', h(doc, 'span', 'ws-sum-pending', 'checking metrics…')); return out; }
   const home = ws.home || { state: 'unset' };
   if (home.state === 'unset') {
     frag.append(' · ', h(doc, 'span', 'muted', 'no metrics home'));
   } else {
-    frag.append(' · ', homeMark(doc), h(doc, 'span', 'mono', home.slug));
+    frag.append(' · ', homeMark(doc), ref(doc, home.slug));
     if (home.state !== 'ok') frag.append(' · ', h(doc, 'span', 'ws-sum-bad', home.detail || 'stale'));
     else if (home.runs != null) frag.append(` · ${home.runs} workspace run${home.runs === 1 ? '' : 's'}`); // project-level runs live elsewhere
   }
   const silent = (ws.members || []).filter((x) => x.state !== 'home' && !x.recordsOn).length;
   if (silent) frag.append(' · ', h(doc, 'span', 'ws-sum-warn', `${silent} not recording`));
-  return frag;
+  return out;
 }
 
 /** One table cell: what this member's metrics branch is, or that there is none. */
@@ -432,7 +493,7 @@ export function renderWsMetricsRow(ws, { doc = globalThis.document } = {}) {
     root.append(you);
   } else if (home.record === false) {
     const you = h(doc, 'small', 'hint ws-home-hint warn');
-    you.append(dot(doc, 'amber'), ` Your workspace runs are not recorded: "Include my runs" is off on ${home.slug}.`);
+    you.append(dot(doc, 'amber'), ' Your workspace runs are not recorded: "Include my runs" is off on ', ref(doc, home.slug), '.');
     root.append(you);
   }
   root.append(h(doc, 'div', 'ws-route-results'));

package/ui/public/team-metrics-view.mjs

@@ -2,7 +2,7 @@
 // Team metrics page renderers (team-metrics-design.md §4.10; mockup boards 1–3). Pure DOM:
 // every function takes an aggregate() result (src/shared/team-metrics/aggregate.mjs) and
 // returns detached elements. app.js owns fetch, mounting and delegated events.
-import { niceScale, roundedTopBar } from './stats-view.mjs';
+import { niceScale, roundedTopBar, savedMultChip } from './stats-view.mjs';
 import { safeHttpUrl } from '../../src/shared/team-metrics/aggregate.mjs';
 
 const SVG_NS = 'http://www.w3.org/2000/svg';
@@ -25,6 +25,7 @@ export const TM_FMT = {
     if (s < 3600) return `${Math.floor(s / 60)}m ${String(s % 60).padStart(2, '0')}s`;
     return `${Math.floor(s / 3600)}h ${String(Math.floor((s % 3600) / 60)).padStart(2, '0')}m`;
   },
+  hours: (h) => `${(Math.round(((Number(h) || 0) + Number.EPSILON) * 10) / 10).toLocaleString('en-US', { maximumFractionDigits: 1 })} h`,
   pct: (v) => (v == null ? '—' : `${Math.round(v * 100)}%`),
   day: (iso) => { const d = new Date(iso); return `${MO[d.getUTCMonth()]} ${d.getUTCDate()}`; },
 };
@@ -136,7 +137,8 @@ function deltaChip(doc, text, title) {
 }
 const signedPct = (v) => (v == null ? null : `${v >= 0 ? '+' : '−'}${Math.abs(Math.round(v * 100))}%`);
 
-/** The six mockup tile glyphs, path data verbatim from boards 1–2 (`svg.stat-ico`). */
+/** The six mockup tile glyphs, path data verbatim from boards 1–2 (`svg.stat-ico`); `saved` (a money bag
+ *  resting on an open hand) is not a board glyph — it is shared with the Stats view. */
 const TILE_ICONS = Object.freeze({
   spend: ['M12 3v18M17 7.5c0-1.9-2.2-3-5-3s-5 1.1-5 3 2.2 2.6 5 3 5 1.1 5 3-2.2 3-5 3-5-1.1-5-3'],
   runs: ['M5 20v-7M12 20V4M19 20v-11'],
@@ -144,6 +146,7 @@ const TILE_ICONS = Object.freeze({
   duration: ['circle:12,12,8', 'M12 8v4l3 2'],
   autonomy: ['M4 12h4l2-5 4 10 2-5h4'],
   cycles: ['M3 12a9 9 0 1 0 3-6.7', 'M3 4v4h4'],
+  saved: ['M9.6 4.6h3.8', 'M9.9 4.6c.3-1.5 2.9-1.5 3.2 0', 'M9.6 4.6C7.8 6.2 6.8 7.8 6.8 9.4c0 1.6.6 2.9 1.4 3.8h6.6c.8-.9 1.4-2.2 1.4-3.8 0-1.6-1-3.2-2.8-4.8', 'M8.2 13.2c-.5 0-1 .2-1.3.6L3 17.6', 'M7 21l1.6-1.4c.3-.4.8-.6 1.4-.6h4c1.1 0 2.1-.4 2.8-1.2l4.6-4.4a2 2 0 0 0-2.75-2.91L14.8 13.2', 'M2 16.6l6 6'],
 });
 function tileIcon(doc, key) {
   const svg = s(doc, 'svg', { class: 'stat-ico', viewBox: '0 0 24 24', fill: 'none', stroke: 'currentColor', 'stroke-width': 1.9, 'stroke-linecap': 'round', 'stroke-linejoin': 'round' });
@@ -190,6 +193,23 @@ export function renderTmKpiRow(agg, { doc = globalThis.document, now = Date.now(
       ? ['attributed to the workspace, not to single projects']
       : nowInRange ? [{ b: TM_FMT.usd(k.spendThisMonthUsd) }, ` so far in ${month}`] : [],
   }));
+
+  // Saved (money-saved design §10): Σ human hours × the effective rate − spend. Gated like the
+  // Off-policy tile: records pushed before this feature carry no `human`, and a range made only
+  // of those must not print "−$spend".
+  if (k.humanRuns > 0) {
+    const saved = Number(k.savedUsd || 0);
+    const savedTile = tile(doc, {
+      icon: 'saved', label: 'Saved', chip: deltaChip(doc, d && withTitle(signedPct(d.savedPct))),
+      value: `${saved < 0 ? '−' : ''}${TM_FMT.usd(Math.abs(saved))}`,
+      sub: ['≈ ', { b: TM_FMT.hours(k.humanHours) }, ' of human work'],
+    });
+    savedTile.querySelector('.stat-value').classList.toggle('is-neg', saved < 0);
+    savedTile.querySelector('.stat-value').classList.toggle('is-pos', saved > 0);
+    const mult = savedMultChip(doc, saved, k.spendUsd);   // "32× spend", right of the delta pill
+    if (mult) savedTile.querySelector('.stat-label').append(mult);
+    row.append(savedTile);
+  }
   row.append(tile(doc, {
     icon: 'runs', label: 'Runs', chip: deltaChip(doc, d && signedPct(d.runsPct)), value: String(k.runs),
     sub: [{ b: String(k.done) }, ` done · ${k.failed} failed · ${k.stopped} stopped · `, { b: TM_FMT.pct(k.successRate) }, ' success'],
@@ -204,16 +224,46 @@ export function renderTmKpiRow(agg, { doc = globalThis.document, now = Date.now(
   row.append(tile(doc, {
     icon: 'autonomy', label: 'Autonomy', chip: deltaChip(doc, d && d.autonomyPts != null ? `${d.autonomyPts >= 0 ? '+' : '−'}${Math.abs(d.autonomyPts)} pts` : null),
     value: TM_FMT.pct(k.autonomy), meterPct: k.autonomy == null ? 0 : k.autonomy * 100,
-    sub: ['active ÷ wall-clock · ', { b: k.interventionsPerRun == null ? '—' : String(k.interventionsPerRun) }, ' interventions per run'],
+    sub: ['active ÷ (wall-clock − paused) · ', { b: k.interventionsPerRun == null ? '—' : String(k.interventionsPerRun) }, ' interventions per run'],
   }));
   row.append(tile(doc, {
     icon: 'cycles', label: 'Review cycles', chip: deltaChip(doc, d && d.reviewCycles != null ? `${d.reviewCycles >= 0 ? '+' : '−'}${Math.abs(d.reviewCycles)}` : null),
     value: k.reviewCyclesMean == null ? '—' : String(k.reviewCyclesMean),
     sub: ['mean implement→review loops · ', { b: TM_FMT.pct(k.convergeInOneRate) }, ' converge in one'],
   }));
+  // Team policy (team-policy design §10, board 10): only once a record in range carries a policy.
+  if (k.policyRuns > 0) {
+    row.append(tile(doc, {
+      icon: 'cycles', label: 'Off-policy', value: String(k.offPolicyRuns),
+      sub: [{ b: String(k.capOverrides) }, ` cap override${k.capOverrides === 1 ? '' : 's'} · `, { b: String(k.deviations) }, ` deviation${k.deviations === 1 ? '' : 's'} · of ${k.policyRuns} run${k.policyRuns === 1 ? '' : 's'} under a policy`],
+    }));
+  }
   return row;
 }
 
+/**
+ * The pooled-budget tile (team-policy design §7, board 10): advisory, never a pause. `spentUsd`
+ * is the sum of the records in the budget's current window; the pace line projects it to the
+ * window's end. Warn from 80 %, over from 100 %.
+ */
+export function renderPooledBudgetTile({ budgetUsd, window: win = 'monthly', spentUsd = 0, windowStartMs, windowEndMs }, { doc = globalThis.document, now = Date.now() } = {}) {
+  const ratio = budgetUsd > 0 ? spentUsd / budgetUsd : 0;
+  const elapsed = windowEndMs > windowStartMs ? Math.max(0, Math.min(1, (now - windowStartMs) / (windowEndMs - windowStartMs))) : 1;
+  const pace = elapsed > 0.05 ? spentUsd / elapsed : null;
+  const end = new Date(windowEndMs);
+  const endLabel = `${MO[end.getMonth()]} ${end.getDate()}`;
+  const card = tile(doc, {
+    icon: 'spend', label: 'Pooled budget', value: TM_FMT.pct(ratio), meterPct: ratio * 100,
+    sub: [{ b: TM_FMT.usd(spentUsd) }, ' of ', { b: TM_FMT.usd(budgetUsd) }, ` this ${win === 'weekly' ? 'week' : 'month'}`,
+      ...(pace != null && ratio < 1 ? [' · on pace for ', { b: TM_FMT.usd(pace) }, ` by ${endLabel}`] : ratio >= 1 ? [' · over budget'] : [])],
+  });
+  card.classList.add('tm-pooled');
+  const fill = card.querySelector('.stat-meter-fill');
+  if (fill) fill.classList.toggle('warn', ratio >= 0.8 && ratio < 1);
+  if (fill) fill.classList.toggle('over', ratio >= 1);
+  return card;
+}
+
 const CW = 560, CH = 240, L = 44, R = 12, T = 18, B = 26;
 const PW = CW - L - R, PH = CH - T - B;
 
@@ -293,7 +343,8 @@ export function renderStackedWeekChart({ title, weeks, stacks, valueOf, yFmt, ti
 const BREAKDOWN_SPECS = {
   workflow: { title: 'By workflow', sub: 'share of spend', cols: [['label', 'Workflow'], ['runs', 'Runs'], ['usd', 'Spend'], ['perRunUsd', 'Per run'], ['successRate', 'Success'], ['share', '']] },
   source: { title: 'By ticket', sub: 'task source · top 5', limit: 5, cols: [['label', 'Source'], ['runs', 'Runs'], ['usd', 'Spend'], ['perRunUsd', 'Per run'], ['cyclesMean', 'Cycles']] },
-  actor: { title: 'By actor', sub: 'who ran it', cols: [['label', 'Actor'], ['runs', 'Runs'], ['usd', 'Spend'], ['perRunUsd', 'Per run'], ['successRate', 'Success']] },
+  // `overrides`: team caps continued past or exceeded (team policy §10, board 10); "–" without a policy.
+  actor: { title: 'By actor', sub: 'who ran it', cols: [['label', 'Actor'], ['runs', 'Runs'], ['usd', 'Spend'], ['perRunUsd', 'Per run'], ['successRate', 'Success'], ['overrides', 'Overrides']] },
   // No spend column: a run's cost is not a per-project fact. Runs touched and files changed are;
   // the bar is the share of runs in range that touched the project. Cost questions go through
   // the row filter, which narrows every panel to the runs that touched it.
@@ -306,6 +357,7 @@ function cellText(key, row) {
     case 'usd': case 'perRunUsd': return row[key] == null ? '—' : TM_FMT.usd(row[key]);
     case 'successRate': return TM_FMT.pct(row[key]);
     case 'cyclesMean': case 'filesChanged': return row[key] == null ? '—' : String(row[key]);
+    case 'overrides': return row[key] ? String(row[key]) : '–';
     default: return String(row[key] ?? '—');
   }
 }
@@ -379,7 +431,7 @@ export function renderRunsTable(runs, { doc = globalThis.document, total = runs.
   if (!runs.length) { card.append(h(doc, 'div', 'chart-empty hint', 'No runs recorded in this range')); return card; }
   const table = h(doc, 'table', 'tm-tbl tm-run-tbl');
   const thead = h(doc, 'thead'); const thr = h(doc, 'tr');
-  for (const c of ['Title', 'Workflow', 'Result', 'Cost', 'Duration', 'Cycles', 'PR', 'Actor', 'Started']) thr.append(h(doc, 'th', null, c));
+  for (const c of ['Title', 'Workflow', 'Result', 'Cost', 'Saved', 'Duration', 'Cycles', 'PR', 'Actor', 'Started']) thr.append(h(doc, 'th', null, c));
   thead.append(thr); table.append(thead);
   const tbody = h(doc, 'tbody');
   for (const r of runs.slice(0, limit)) {
@@ -388,6 +440,17 @@ export function renderRunsTable(runs, { doc = globalThis.document, total = runs.
     tr.append(h(doc, 'td', null, r.workflow ?? '—'));
     const res = h(doc, 'td'); res.append(h(doc, 'span', `badge ${r.result === 'done' ? 'green' : r.result === 'failed' ? 'red' : 'grey'}`, r.result)); tr.append(res);
     tr.append(h(doc, 'td', 'mono num', TM_FMT.usd(r.usd)));
+    // Saved per run (money-saved design §10): hours × rate − cost, coloured like the KPI tile;
+    // "—" for a record pushed without `human`.
+    const saved = h(doc, 'td', 'mono num tm-saved');
+    if (r.savedUsd == null) saved.textContent = '—';
+    else {
+      saved.textContent = `${r.savedUsd < 0 ? '−' : ''}${TM_FMT.usd(Math.abs(r.savedUsd))}`;
+      saved.classList.toggle('neg', r.savedUsd < 0);
+      saved.classList.toggle('pos', r.savedUsd > 0);
+      saved.title = `≈ ${TM_FMT.hours(r.humanHours)} of human work`;
+    }
+    tr.append(saved);
     tr.append(h(doc, 'td', 'mono num', TM_FMT.duration(r.wallMs)));
     tr.append(h(doc, 'td', 'mono num', r.reviewCycles == null ? '—' : String(r.reviewCycles)));
     const pr = h(doc, 'td', 'mono');