@d3lm/pr-stats 0.2.6 → 0.2.7

package/README.md

@@ -25,7 +25,15 @@ You can also run it without installing through `npx @d3lm/pr-stats`. The bunx eq
 pr-stats
 ```
 
-Without flags, it looks at PRs from the last 90 days across all repositories you can access. It opens on a tab with two lists, the open PRs awaiting your review with how long each has been waiting, and below them the open PRs you already reviewed or commented on with how long ago that was, so a PR stays visible until it merges or closes. A fresh review request moves a PR from the reviewing list back into the awaiting one. The Your PRs tab next to it has two sub-tabs, which the `t` key switches. The first lists your own authored PRs that are still open with their age and size. The second charts how your authored PRs got created, merged, and closed, telling a merge apart from a close without one, with time-to-merge percentiles, a histogram and trend, a merge-time heatmap, weekly created and merged volumes, an outcome gauge, and the most recently merged and closed PRs. The remaining tabs hold the time-to-review report, the PR size report, and the comments report with a histogram of comments per PR, a scatter of comments against PR size, and the most commented PRs. When the data spans multiple repos, every tab opens on a repo picker that drills into one repo or the aggregate across all of them, and on the two queue lists the `g` key groups the aggregate list by repo.
+Without flags, it looks at PRs from the last 90 days across all repositories you can access. The tabs hold these views.
+
+- The queue tab, which it opens on, shows two lists. The open PRs awaiting your review come first with how long each has been waiting, and below them sit the open PRs you already reviewed or commented on with how long ago that was, so a PR stays visible until it merges or closes. A fresh review request moves a PR from the reviewing list back into the awaiting one.
+- The Your PRs tab has two sub-tabs, which the `t` key switches. The first lists your own authored PRs that are still open with their age and size. The second reports how your authored PRs got created, merged, and closed, telling a merge apart from a close without one. It charts time-to-merge percentiles, a histogram and trend, a merge-time heatmap, and a scatter of merge time against PR size. It also plots a merge-rate trend over the concluded PRs, cumulative created and merged lines whose gap shows the backlog, weekly created and merged volumes, an outcome gauge, and the most recently merged and closed PRs.
+- The time-to-review report pairs its histogram, trend, heatmap, and weekly volume with the completed review cycles per PR and a verdict gauge splitting approvals from change requests. It also shows the age of the requests still waiting on you, how old PRs already were when the request reached you, and an off-hours gauge that splits weekdays into work hours and after hours once `--work-hours` is set. On the aggregate view it additionally compares median review times by repo.
+- The PR size report carries the same histogram, trend, heatmap, and weekly volume for PR sizes and adds a net-lines trend that sums additions minus deletions per week.
+- The comments report holds a histogram of comments per PR, a scatter of comments against PR size, and the most commented PRs.
+
+When the data spans multiple repos, every tab opens on a repo picker that drills into one repo or the aggregate across all of them, and on the two queue lists the `g` key groups the aggregate list by repo.
 
 Node gates the FFI that OpenTUI renders through behind the `--experimental-ffi` flag, and the launcher re-executes itself with that flag when it is missing, so a plain `pr-stats` works without extra flags on both runtimes.
 

package/dist/tui-app.mjs

@@ -726,7 +726,19 @@ function wallParts(instantMs) {
 function zonedStamp(date) {
   const parts = wallParts(date.getTime());
   const dayUtcMs = Date.UTC(parts.year, parts.month - 1, parts.day);
-  return { dayUtcMs, weekday: new Date(dayUtcMs).getUTCDay(), hour: parts.hour };
+  return { dayUtcMs, weekday: new Date(dayUtcMs).getUTCDay(), hour: parts.hour, minute: parts.minute };
+}
+function hasWorkWindows() {
+  const covered = timeMode.workWindows.reduce((sum, window) => sum + (window.endMin - window.startMin), 0);
+  return covered < 24 * 60;
+}
+function classifyInstant(date) {
+  const { weekday, hour, minute } = zonedStamp(date);
+  if (weekday === 0 || weekday === 6) {
+    return "weekend";
+  }
+  const minuteOfDay = hour * 60 + minute;
+  return timeMode.workWindows.some((window) => minuteOfDay >= window.startMin && minuteOfDay < window.endMin) ? "work" : "after";
 }
 function utcFromWall(wallTargetMs) {
   let guess = wallTargetMs;
@@ -780,7 +792,8 @@ function computeReviewStats(results, { targetHours, now = /* @__PURE__ */ new Da
         pr: result.pr,
         requestedAt: result.requestedAt,
         reviewedAt: result.reviewedAt,
-        hours: durationHours(result.requestedAt, result.reviewedAt)
+        hours: durationHours(result.requestedAt, result.reviewedAt),
+        verdict: result.verdict
       });
     } else if (result.kind === "pending" && result.pr.state === "open") {
       pending.push({ pr: result.pr, requestedAt: result.requestedAt, hours: durationHours(result.requestedAt, now) });
@@ -816,7 +829,22 @@ function computeReviewStats(results, { targetHours, now = /* @__PURE__ */ new Da
   }
   const byRepo = [...hoursByRepo.entries()].toSorted((a, b) => b[1].length - a[1].length);
   const misses = targetHours === void 0 ? [] : reviewed.filter((result) => result.hours > targetHours).toSorted((a, b) => b.hours - a.hours);
-  return { reviewed, pending, reviewing, expired, unrequested, allHours, byRepo, misses };
+  const cyclesByPr = /* @__PURE__ */ new Map();
+  for (const result of reviewed) {
+    const key = `${result.pr.repo}#${result.pr.number}`;
+    cyclesByPr.set(key, (cyclesByPr.get(key) ?? 0) + 1);
+  }
+  return {
+    reviewed,
+    pending,
+    reviewing,
+    expired,
+    unrequested,
+    allHours,
+    byRepo,
+    misses,
+    cycles: [...cyclesByPr.values()]
+  };
 }
 function computeSizeStats(sizes, { sizeTarget } = {}) {
   const metrics = [
@@ -918,6 +946,13 @@ var FILE_BUCKETS = [
   { label: "21-50", max: 51 },
   { label: "> 50", max: Infinity }
 ];
+var CYCLE_BUCKETS = [
+  { label: "1", max: 2 },
+  { label: "2", max: 3 },
+  { label: "3", max: 4 },
+  { label: "4-5", max: 6 },
+  { label: "> 5", max: Infinity }
+];
 var COMMENT_BUCKETS = [
   { label: "0", max: 1 },
   { label: "1-2", max: 3 },
@@ -1094,18 +1129,18 @@ function ChartsPanel({
   if (view.empty !== null) {
     return /* @__PURE__ */ jsx4("box", { flexGrow: 1, alignItems: "center", justifyContent: "center", children: /* @__PURE__ */ jsx4("text", { fg: theme.muted, children: view.empty }) });
   }
-  const leftWidth = Math.max(0, ...view.left.map(cardWidth));
-  const rightWidth = Math.max(0, ...view.right.map(cardWidth));
-  const twoColumns = view.left.length > 0 && view.right.length > 0 && width - 4 >= leftWidth + COLUMN_GAP + rightWidth;
-  const rows = Array.from({ length: Math.max(view.left.length, view.right.length) }, (_, i) => {
-    const left = view.left.at(i);
-    const right = view.right.at(i);
-    return { key: left?.title ?? right?.title ?? "", left, right };
+  const left = view.cards.filter((_, i) => i % 2 === 0);
+  const right = view.cards.filter((_, i) => i % 2 === 1);
+  const leftWidth = Math.max(0, ...left.map((card) => cardWidth(card)));
+  const rightWidth = Math.max(0, ...right.map((card) => cardWidth(card)));
+  const twoColumns = right.length > 0 && width - 4 >= leftWidth + COLUMN_GAP + rightWidth;
+  const rows = left.map((card, i) => {
+    return { key: card.title, left: card, right: right.at(i) };
   });
   const cards = twoColumns ? /* @__PURE__ */ jsx4("box", { flexDirection: "column", rowGap: 1, children: rows.map((row) => /* @__PURE__ */ jsxs3("box", { flexDirection: "row", alignItems: "flex-start", columnGap: COLUMN_GAP, children: [
-    /* @__PURE__ */ jsx4("box", { flexDirection: "column", width: leftWidth, flexShrink: 0, children: row.left !== void 0 && /* @__PURE__ */ jsx4(ChartCard, { card: row.left }) }),
+    /* @__PURE__ */ jsx4("box", { flexDirection: "column", width: leftWidth, flexShrink: 0, children: /* @__PURE__ */ jsx4(ChartCard, { card: row.left }) }),
     /* @__PURE__ */ jsx4("box", { flexDirection: "column", width: rightWidth, flexShrink: 0, children: row.right !== void 0 && /* @__PURE__ */ jsx4(ChartCard, { card: row.right }) })
-  ] }, row.key)) }) : /* @__PURE__ */ jsx4("box", { flexDirection: "column", rowGap: 1, children: [...view.left, ...view.right].map((card) => /* @__PURE__ */ jsx4(ChartCard, { card }, card.title)) });
+  ] }, row.key)) }) : /* @__PURE__ */ jsx4("box", { flexDirection: "column", rowGap: 1, children: view.cards.map((card) => /* @__PURE__ */ jsx4(ChartCard, { card }, card.title)) });
   return /* @__PURE__ */ jsxs3("box", { flexGrow: 1, flexDirection: "column", children: [
     /* @__PURE__ */ jsx4("box", { height: 1, children: /* @__PURE__ */ jsx4("text", { wrapMode: "none", fg: theme.border, children: "\u2500".repeat(width) }) }),
     /* @__PURE__ */ jsx4("box", { height: 1, paddingLeft: 1, paddingRight: 2, flexDirection: "row", justifyContent: "space-between", children: keyed(view.strip, lineKey).map(({ key, item }) => /* @__PURE__ */ jsx4(ChartLine, { line: item }, key)) }),
@@ -1146,7 +1181,7 @@ function ChartsPanel({
             ] }),
             /* @__PURE__ */ jsx4("box", { flexDirection: "column", marginTop: 1, children: keyed(view.distribution.lines, lineKey).map(({ key, item }) => /* @__PURE__ */ jsx4(ChartLine, { line: item }, key)) })
           ] }),
-          view.left.length === 0 ? /* @__PURE__ */ jsx4("text", { wrapMode: "none", fg: theme.muted, marginTop: 1, children: view.noCharts }) : cards
+          view.cards.length === 0 ? /* @__PURE__ */ jsx4("text", { wrapMode: "none", fg: theme.muted, marginTop: 1, children: view.noCharts }) : cards
         ]
       }
     )
@@ -3468,20 +3503,22 @@ function classifyPr(pr, details, user) {
     (node) => node?.requestedReviewer?.login === user ? [new Date(node.createdAt)] : []
   );
   const reviews = details.reviews.nodes.flatMap(
-    (node) => node?.author?.login === user && node.submittedAt ? [new Date(node.submittedAt)] : []
+    (node) => node?.author?.login === user && node.submittedAt ? [{ at: new Date(node.submittedAt), state: node.state }] : []
   );
   if (requests.length === 0) {
     if (reviews.length === 0) {
       return [{ kind: "inaccessible", pr }];
     }
-    return [{ kind: "unrequested", pr, reviewedAt: new Date(Math.max(...reviews.map((review) => review.getTime()))) }];
+    return [
+      { kind: "unrequested", pr, reviewedAt: new Date(Math.max(...reviews.map((review) => review.at.getTime()))) }
+    ];
   }
   const events = [
     ...requests.map((at) => {
-      return { at, isRequest: true };
+      return { at, isRequest: true, state: "" };
     }),
-    ...reviews.map((at) => {
-      return { at, isRequest: false };
+    ...reviews.map(({ at, state }) => {
+      return { at, isRequest: false, state };
     })
   ].toSorted((a, b) => a.at.getTime() - b.at.getTime() || Number(b.isRequest) - Number(a.isRequest));
   const results = [];
@@ -3490,7 +3527,7 @@ function classifyPr(pr, details, user) {
     if (event.isRequest) {
       openedAt ??= event.at;
     } else if (openedAt !== null) {
-      results.push({ kind: "reviewed", pr, requestedAt: openedAt, reviewedAt: event.at });
+      results.push({ kind: "reviewed", pr, requestedAt: openedAt, reviewedAt: event.at, verdict: event.state });
       openedAt = null;
     }
   }
@@ -3619,6 +3656,12 @@ function loadSnapshot(options) {
   if (data.reviewResults.some((result) => result.kind === "unrequested" && Number.isNaN(result.reviewedAt.getTime()))) {
     return null;
   }
+  const hasMissingVerdict = data.reviewResults.some((result) => {
+    return result.kind === "reviewed" && result.verdict === void 0;
+  });
+  if (hasMissingVerdict) {
+    return null;
+  }
   if (sinceIso === data.sinceIso) {
     return data;
   }
@@ -4002,6 +4045,137 @@ function hbar(fraction, width, color) {
   return line;
 }
 
+// src/tui/views/charts/bars.ts
+var BAR_WIDTH2 = 24;
+var MAX_BARS = 8;
+function buildBarsCard({ title, subtitle, rows, format }) {
+  const shown = rows.slice(0, MAX_BARS);
+  const max = Math.max(...shown.map((row) => row.value), 0);
+  const labelWidth = Math.max(...shown.map((row) => row.label.length));
+  const valueWidth = Math.max(...shown.map((row) => format(row.value).length));
+  const lines = shown.map((row) => {
+    const line = [{ text: `${row.label.padEnd(labelWidth)} `, fg: theme.muted }];
+    if (row.value <= 0 || max <= 0) {
+      line.push({ text: " ".repeat(BAR_WIDTH2) });
+    } else {
+      line.push(...hbar(row.value / max, BAR_WIDTH2, row.value === max ? theme.accent : theme.chartBar));
+    }
+    line.push(
+      { text: ` ${format(row.value).padStart(valueWidth)}`, fg: theme.text },
+      { text: `  ${row.detail}`, fg: theme.dim }
+    );
+    return line;
+  });
+  if (rows.length > shown.length) {
+    lines.push([{ text: `+ ${rows.length - shown.length} more`, fg: theme.dim }]);
+  }
+  return { title, subtitle, lines };
+}
+
+// src/tui/views/charts/weeks.ts
+var DAY_MS = 864e5;
+var WEEK_MS = 7 * DAY_MS;
+function mondayOf(dayUtcMs) {
+  return dayUtcMs - (new Date(dayUtcMs).getUTCDay() + 6) % 7 * DAY_MS;
+}
+function dateLabel(ms) {
+  return new Date(ms).toLocaleDateString("en-US", { month: "short", day: "numeric", timeZone: "UTC" });
+}
+function weekAxisRow(width, prefix, points, columnOf, mondayOfPoint) {
+  const cells = blankCells(width);
+  const every = Math.ceil(points / 4);
+  let lastEnd = -2;
+  for (let point = 0; point < points; point += every) {
+    const label = dateLabel(mondayOfPoint(point));
+    const at = prefix + columnOf(point);
+    if (at >= lastEnd + 2 && at + label.length <= width) {
+      placeText(cells, at, label, theme.dim);
+      lastEnd = at + label.length;
+    }
+  }
+  return mergeCells(cells);
+}
+
+// src/tui/views/charts/cumulative.ts
+var CUM_WIDTH = 36;
+var CUM_HEIGHT = 8;
+function weekOf(date) {
+  return mondayOf(zonedStamp(date).dayUtcMs);
+}
+function buildCumulativeCard({ title, series, legend }) {
+  const subtitle = series.flatMap((entry, i) => [
+    { text: `${i === 0 ? "" : "  "}\u2500\u2500`, fg: entry.color },
+    { text: ` ${entry.label}`, fg: theme.muted }
+  ]);
+  subtitle.push({ text: `, ${legend}`, fg: theme.muted });
+  const mondays = series.flatMap((entry) => entry.dates.map((date) => weekOf(date)));
+  const first = Math.min(...mondays);
+  const weekCount = (Math.max(...mondays) - first) / WEEK_MS + 1;
+  if (weekCount < 2) {
+    return { title, subtitle, lines: [[{ text: "not enough weeks to draw a trend", fg: theme.muted }]] };
+  }
+  const totals = series.map((entry) => {
+    const weekly = Array.from({ length: weekCount }, () => 0);
+    for (const date of entry.dates) {
+      weekly[(weekOf(date) - first) / WEEK_MS] += 1;
+    }
+    let running = 0;
+    return weekly.map((count2) => running += count2);
+  });
+  const maxY = Math.max(...totals.map((cumulative) => cumulative.at(-1) ?? 0), 1);
+  const grid = Array.from(
+    { length: CUM_HEIGHT },
+    () => Array.from({ length: CUM_WIDTH }, () => null)
+  );
+  for (const [i, cumulative] of totals.entries()) {
+    const rows = Array.from({ length: CUM_WIDTH }, (_, x) => {
+      const weekPos = x / (CUM_WIDTH - 1) * (weekCount - 1);
+      const week = Math.floor(weekPos);
+      const nextWeek = Math.min(week + 1, weekCount - 1);
+      const value2 = cumulative[week] + (cumulative[nextWeek] - cumulative[week]) * (weekPos - week);
+      return CUM_HEIGHT - 1 - Math.round(value2 / maxY * (CUM_HEIGHT - 1));
+    });
+    const color = series[i].color;
+    for (let x = 0; x < CUM_WIDTH; x++) {
+      const here = rows[x];
+      const next = rows[Math.min(x + 1, CUM_WIDTH - 1)];
+      if (here === next) {
+        grid[here][x] = { ch: "\u2500", fg: color };
+        continue;
+      }
+      grid[next][x] = { ch: next < here ? "\u256D" : "\u2570", fg: color };
+      grid[here][x] = { ch: next < here ? "\u256F" : "\u256E", fg: color };
+      for (let row = Math.min(here, next) + 1; row < Math.max(here, next); row++) {
+        grid[row][x] = { ch: "\u2502", fg: color };
+      }
+    }
+  }
+  const topLabel = String(maxY);
+  const yWidth = topLabel.length;
+  const lines = grid.map((row, i) => {
+    const label = i === 0 ? topLabel : i === CUM_HEIGHT - 1 ? "0" : "";
+    const line = [
+      { text: `${label.padStart(yWidth)} `, fg: theme.muted },
+      { text: label === "" ? "\u2502" : "\u2524", fg: theme.dim }
+    ];
+    for (const cell of row) {
+      line.push(cell === null ? { text: " " } : { text: cell.ch, fg: cell.fg });
+    }
+    return line;
+  });
+  lines.push(
+    [{ text: `${" ".repeat(yWidth)} \u2514${"\u2500".repeat(CUM_WIDTH)}`, fg: theme.dim }],
+    weekAxisRow(
+      yWidth + 2 + CUM_WIDTH,
+      yWidth + 2,
+      weekCount,
+      (week) => Math.round(week / (weekCount - 1) * (CUM_WIDTH - 1)),
+      (week) => first + week * WEEK_MS
+    )
+  );
+  return { title, subtitle, lines };
+}
+
 // src/tui/views/charts/distribution.ts
 var DURATION_TICKS = [5 / 60, 0.25, 0.5, 1, 2, 4, 8, 24, 48, 96, 168, 336, 720, 2160];
 var COUNT_TICKS = [1, 2, 5, 10, 25, 50, 100, 250, 500, 1e3, 2500, 5e3, 1e4, 25e3, 5e4, 1e5];
@@ -4322,39 +4496,20 @@ function buildSpreadCard(title, metrics, format) {
 
 // src/tui/views/charts/trend.ts
 var asciichart = __toESM(require_asciichart(), 1);
-
-// src/tui/views/charts/weeks.ts
-var DAY_MS = 864e5;
-var WEEK_MS = 7 * DAY_MS;
-function mondayOf(dayUtcMs) {
-  return dayUtcMs - (new Date(dayUtcMs).getUTCDay() + 6) % 7 * DAY_MS;
-}
-function dateLabel(ms) {
-  return new Date(ms).toLocaleDateString("en-US", { month: "short", day: "numeric", timeZone: "UTC" });
-}
-function weekAxisRow(width, prefix, points, columnOf, mondayOfPoint) {
-  const cells = blankCells(width);
-  const every = Math.ceil(points / 4);
-  let lastEnd = -2;
-  for (let point = 0; point < points; point += every) {
-    const label = dateLabel(mondayOfPoint(point));
-    const at = prefix + columnOf(point);
-    if (at >= lastEnd + 2 && at + label.length <= width) {
-      placeText(cells, at, label, theme.dim);
-      lastEnd = at + label.length;
-    }
-  }
-  return mergeCells(cells);
-}
-
-// src/tui/views/charts/trend.ts
 var TREND_POINTS = 40;
 var TREND_HEIGHT = 6;
 function axisSplit(line) {
   const positions = [line.indexOf("\u2524"), line.indexOf("\u253C")].filter((at) => at >= 0);
   return positions.length === 0 ? 0 : Math.min(...positions) + 1;
 }
-function buildTrendCard({ title, entries, format, floor }) {
+function buildTrendCard({
+  title,
+  entries,
+  format,
+  floor = 1,
+  scale = "log",
+  valueLabel = "median"
+}) {
   const byWeek = /* @__PURE__ */ new Map();
   for (const entry of entries) {
     const monday = mondayOf(zonedStamp(entry.date).dayUtcMs);
@@ -4365,10 +4520,11 @@ function buildTrendCard({ title, entries, format, floor }) {
   const mondays = [...byWeek.keys()];
   const first = Math.min(...mondays);
   const weekCount = (Math.max(...mondays) - first) / WEEK_MS + 1;
+  const scaleSuffix = scale === "log" ? ", log scale" : "";
   if (weekCount < 2) {
     return {
       title,
-      subtitle: "weekly median, log scale",
+      subtitle: `weekly ${valueLabel}${scaleSuffix}`,
       lines: [[{ text: "not enough weeks to draw a trend", fg: theme.muted }]]
     };
   }
@@ -4386,9 +4542,11 @@ function buildTrendCard({ title, entries, format, floor }) {
       medians.push(medians.at(-1) ?? 0);
     }
   }
-  const firstKnown = medians.find((value2) => value2 > 0) ?? 0;
-  for (let i = 0; i < medians.length && medians[i] === 0; i++) {
-    medians[i] = firstKnown;
+  if (scale === "log") {
+    const firstKnown = medians.find((value2) => value2 > 0) ?? 0;
+    for (let i = 0; i < medians.length && medians[i] === 0; i++) {
+      medians[i] = firstKnown;
+    }
   }
   const chunk = Math.ceil(weekCount / TREND_POINTS);
   const points = [];
@@ -4398,23 +4556,23 @@ function buildTrendCard({ title, entries, format, floor }) {
   }
   const stretch = Math.max(1, Math.floor(TREND_POINTS / points.length));
   const series = points.flatMap((value2) => Array.from({ length: stretch }, () => value2));
-  const logs = series.map((value2) => Math.log2(Math.max(value2, floor)));
-  let lo = Math.min(...logs);
-  let hi = Math.max(...logs);
+  const plotted = scale === "log" ? series.map((value2) => Math.log2(Math.max(value2, floor))) : series;
+  let lo = Math.min(...plotted);
+  let hi = Math.max(...plotted);
   if (hi - lo < 1e-9) {
     lo -= 1;
     hi += 1;
   }
-  const chart = asciichart.plot(logs, {
+  const chart = asciichart.plot(plotted, {
     height: TREND_HEIGHT,
     min: lo,
     max: hi,
-    format: (x) => format(2 ** x).padStart(6)
+    format: (x) => format(scale === "log" ? 2 ** x : x).padStart(6)
   });
   const ratio = TREND_HEIGHT / (hi - lo);
   const min2 = Math.round(lo * ratio);
   const rows = Math.abs(Math.round(hi * ratio) - min2);
-  const lastRow = Math.min(rows, Math.max(0, rows - (Math.round((logs.at(-1) ?? 0) * ratio) - min2)));
+  const lastRow = Math.min(rows, Math.max(0, rows - (Math.round((plotted.at(-1) ?? 0) * ratio) - min2)));
   const raw = chart.split("\n");
   const lines = raw.map((line, i) => {
     const split = axisSplit(line);
@@ -4437,7 +4595,7 @@ function buildTrendCard({ title, entries, format, floor }) {
       (point) => first + point * chunk * WEEK_MS
     )
   );
-  const subtitle = chunk === 1 ? "weekly median, log scale" : `median per ${chunk} weeks, log scale`;
+  const subtitle = chunk === 1 ? `weekly ${valueLabel}${scaleSuffix}` : `${valueLabel} per ${chunk} weeks${scaleSuffix}`;
   return { title, subtitle, lines };
 }
 
@@ -4505,6 +4663,79 @@ function countCell(count2, label, dimWhenZero = false) {
     { text: ` ${label}`, fg: dim2 ? theme.dim : theme.muted }
   ];
 }
+function mondayNoon(monday) {
+  return new Date(monday + 12 * 36e5);
+}
+function weeklySums(entries) {
+  const byWeek = /* @__PURE__ */ new Map();
+  for (const entry of entries) {
+    const monday = mondayOf(zonedStamp(entry.date).dayUtcMs);
+    byWeek.set(monday, (byWeek.get(monday) ?? 0) + entry.value);
+  }
+  const mondays = [...byWeek.keys()];
+  const first = Math.min(...mondays);
+  const last = Math.max(...mondays);
+  const result = [];
+  for (let monday = first; monday <= last; monday += WEEK_MS) {
+    result.push({ date: mondayNoon(monday), value: byWeek.get(monday) ?? 0 });
+  }
+  return result;
+}
+function mergeRateEntries(stats) {
+  const byWeek = /* @__PURE__ */ new Map();
+  const add = (createdAt, merged) => {
+    const monday = mondayOf(zonedStamp(createdAt).dayUtcMs);
+    const counts = byWeek.get(monday) ?? { merged: 0, concluded: 0 };
+    counts.concluded += 1;
+    counts.merged += merged ? 1 : 0;
+    byWeek.set(monday, counts);
+  };
+  for (const result of stats.merged) {
+    add(result.entry.pr.createdAt, true);
+  }
+  for (const result of stats.closed) {
+    add(result.entry.pr.createdAt, false);
+  }
+  return [...byWeek.entries()].map(([monday, counts]) => {
+    return { date: mondayNoon(monday), value: counts.merged / counts.concluded * 100 };
+  });
+}
+function buildOffHoursCard(subtitle, dates) {
+  const counts = { work: 0, after: 0, weekend: 0 };
+  for (const date of dates) {
+    counts[classifyInstant(date)] += 1;
+  }
+  const rows = hasWorkWindows() ? [
+    { label: "work hours", count: counts.work, color: theme.accent },
+    { label: "after hours", count: counts.after, color: theme.warn },
+    { label: "weekend", count: counts.weekend, color: theme.chartDim }
+  ] : [
+    { label: "weekday", count: counts.work + counts.after, color: theme.accent },
+    { label: "weekend", count: counts.weekend, color: theme.chartDim }
+  ];
+  return buildGaugeCard({ title: "Off-hours share", subtitle, rows });
+}
+function buildVerdictCard(reviewed) {
+  const countOf = (state) => reviewed.filter((entry) => entry.verdict === state).length;
+  const approved = countOf("APPROVED");
+  const changes = countOf("CHANGES_REQUESTED");
+  const commented = countOf("COMMENTED");
+  const other = reviewed.length - approved - changes - commented;
+  const counts = [
+    { label: "approved", count: approved },
+    { label: "changes requested", count: changes },
+    { label: "commented", count: commented },
+    ...other > 0 ? [{ label: "other", count: other }] : []
+  ];
+  const max = Math.max(...counts.map((row) => row.count));
+  return buildGaugeCard({
+    title: "Review verdicts",
+    subtitle: "how your requested reviews concluded",
+    rows: counts.map((row) => {
+      return { ...row, color: row.count === max ? theme.accent : theme.chartBar };
+    })
+  });
+}
 function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100) {
   const results = repo === null ? raw.reviewResults : raw.reviewResults.filter((result) => result.pr.repo === repo);
   const stats = computeReviewStats(results, { targetHours, now: raw.fetchedAt });
@@ -4519,8 +4750,7 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
     headline: null,
     distributionTitle: "Review time distribution",
     noCharts: "No completed reviews to chart.",
-    left: [],
-    right: [],
+    cards: [],
     distribution: null
   };
   if (results.length === 0) {
@@ -4533,8 +4763,15 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
       rows: toPrRows(stats.misses, stats.misses.map(durationLead))
     });
   }
+  const pendingCard = stats.pending.length === 0 ? null : buildHistogramCard({
+    title: "Pending request age",
+    subtitle: "how long open requests have waited",
+    values: stats.pending.map((entry) => entry.hours),
+    buckets: currentBuckets(),
+    format: formatDuration
+  });
   if (stats.reviewed.length === 0) {
-    return { empty: null, ...base, lists };
+    return { empty: null, ...base, cards: pendingCard === null ? [] : [pendingCard], lists };
   }
   const sorted = [...stats.allHours].toSorted((a, b) => a - b);
   const total = raw.reviewResults.filter((result) => result.kind === "reviewed").length;
@@ -4547,7 +4784,39 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
   ];
   const requestDates = [...stats.reviewed, ...stats.pending].map((entry) => entry.requestedAt);
   const reviewDates = stats.reviewed.map((entry) => entry.reviewedAt);
-  const left = [
+  const requestAges = [...stats.reviewed, ...stats.pending].map(
+    (entry) => durationHours(entry.pr.createdAt, entry.requestedAt)
+  );
+  let serviceCard = null;
+  if (targetHours !== void 0 && targetLabel !== void 0) {
+    const inside = stats.allHours.filter((value2) => value2 <= targetHours).length;
+    const overdue = stats.pending.filter((entry) => entry.hours > targetHours).length;
+    serviceCard = buildGaugeCard({
+      title: "Service level",
+      subtitle: `reviewed within ${targetLabel}`,
+      rows: [
+        { label: `inside ${targetLabel}`, count: inside, color: theme.accent },
+        { label: `over ${targetLabel}`, count: stats.allHours.length - inside, color: theme.chartDim },
+        ...overdue > 0 ? [{ label: "awaiting and already over", count: overdue, color: theme.warn }] : []
+      ]
+    });
+  }
+  const byRepoCard = repo === null && stats.byRepo.length > 1 ? buildBarsCard({
+    title: "Review time by repo",
+    subtitle: "median review time, slowest first",
+    rows: stats.byRepo.map(([name, hours]) => {
+      return {
+        label: name,
+        value: percentile(
+          hours.toSorted((a, b) => a - b),
+          50
+        ),
+        detail: `n=${hours.length}`
+      };
+    }).toSorted((a, b) => b.value - a.value),
+    format: formatDuration
+  }) : null;
+  const cards = [
     buildHistogramCard({
       title: "Time to review",
       subtitle: "elapsed time, request \u2192 review",
@@ -4555,6 +4824,14 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
       buckets: currentBuckets(),
       format: formatDuration
     }),
+    buildTrendCard({
+      title: "Review time trend",
+      entries: stats.reviewed.map((entry) => {
+        return { date: entry.reviewedAt, value: entry.hours };
+      }),
+      format: formatDuration,
+      floor: 1 / 60
+    }),
     buildHeatmapCard({
       title: "When you review",
       subtitle: "reviews submitted, weekday \xD7 hour, local time",
@@ -4564,34 +4841,28 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
         { label: "req", dates: requestDates, muted: true }
       ],
       legend: "reviews in that hour"
-    })
-  ];
-  const right = [
-    buildTrendCard({
-      title: "Review time trend",
-      entries: stats.reviewed.map((entry) => {
-        return { date: entry.reviewedAt, value: entry.hours };
-      }),
-      format: formatDuration,
-      floor: 1 / 60
     }),
-    buildVolumeCard("Reviews completed per week", reviewDates)
+    buildVolumeCard("Reviews completed per week", reviewDates),
+    buildHistogramCard({
+      title: "Review cycles per PR",
+      subtitle: "completed request \u2192 review rounds per PR",
+      values: stats.cycles,
+      buckets: CYCLE_BUCKETS,
+      format: count
+    }),
+    buildHistogramCard({
+      title: "PR age at request",
+      subtitle: "elapsed time, PR created \u2192 review requested",
+      values: requestAges,
+      buckets: currentBuckets(),
+      format: formatDuration
+    }),
+    buildVerdictCard(stats.reviewed),
+    ...pendingCard === null ? [] : [pendingCard],
+    buildOffHoursCard("reviews submitted, local time", reviewDates),
+    ...serviceCard === null ? [] : [serviceCard],
+    ...byRepoCard === null ? [] : [byRepoCard]
   ];
-  if (targetHours !== void 0 && targetLabel !== void 0) {
-    const inside = stats.allHours.filter((value2) => value2 <= targetHours).length;
-    const overdue = stats.pending.filter((entry) => entry.hours > targetHours).length;
-    left.push(
-      buildGaugeCard({
-        title: "Service level",
-        subtitle: `reviewed within ${targetLabel}`,
-        rows: [
-          { label: `inside ${targetLabel}`, count: inside, color: theme.accent },
-          { label: `over ${targetLabel}`, count: stats.allHours.length - inside, color: theme.chartDim },
-          ...overdue > 0 ? [{ label: "awaiting and already over", count: overdue, color: theme.warn }] : []
-        ]
-      })
-    );
-  }
   const distribution = buildDistribution({
     values: stats.allHours,
     width: Math.max(width - 3, 40),
@@ -4599,7 +4870,7 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
     ticks: DURATION_TICKS,
     flat: (count2, value2) => `all ${count2} ${count2 === 1 ? "review" : "reviews"} took ${value2}`
   });
-  return { empty: null, ...base, headline, left, right, distribution, lists };
+  return { empty: null, ...base, headline, cards, distribution, lists };
 }
 function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
   const base = {
@@ -4607,8 +4878,7 @@ function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
     headline: null,
     distributionTitle: "PR size distribution",
     noCharts: "No authored PRs to chart.",
-    left: [],
-    right: [],
+    cards: [],
     distribution: null,
     lists: []
   };
@@ -4639,7 +4909,15 @@ function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
     { text: `${count(percentile(sorted, 90))} lines`, fg: theme.accent },
     { text: `   ${sizes.length} of ${raw.sizes.length} PRs`, fg: theme.muted }
   ];
-  const left = [
+  const targetCard = stats.met !== void 0 && stats.targetLabel !== void 0 ? buildGaugeCard({
+    title: "Size target",
+    subtitle: `authored within ${stats.targetLabel}`,
+    rows: [
+      { label: "inside target", count: stats.met, color: theme.accent },
+      { label: "over target", count: sizes.length - stats.met, color: theme.chartDim }
+    ]
+  }) : null;
+  const cards = [
     buildHistogramCard({
       title: "PR size",
       subtitle: "total lines changed per authored PR",
@@ -4647,6 +4925,14 @@ function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
       buckets: LINE_BUCKETS,
       format: count
     }),
+    buildTrendCard({
+      title: "PR size trend",
+      entries: sizes.map((size) => {
+        return { date: size.pr.createdAt, value: size.total };
+      }),
+      format: count,
+      floor: 1
+    }),
     buildHistogramCard({
       title: "Files touched",
       subtitle: "files changed per authored PR",
@@ -4654,38 +4940,28 @@ function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
       buckets: FILE_BUCKETS,
       format: count
     }),
+    buildTrendCard({
+      title: "Net lines trend",
+      entries: weeklySums(
+        sizes.map((size) => {
+          return { date: size.pr.createdAt, value: size.additions - size.deletions };
+        })
+      ),
+      format: netCount,
+      scale: "linear",
+      valueLabel: "net lines"
+    }),
     buildHeatmapCard({
       title: "When you open PRs",
       subtitle: "PRs opened, weekday \xD7 hour, local time",
       grid: created,
       columns: [{ label: "opened", dates: created }],
       legend: "PRs opened in that hour"
-    })
-  ];
-  const right = [
-    buildTrendCard({
-      title: "PR size trend",
-      entries: sizes.map((size) => {
-        return { date: size.pr.createdAt, value: size.total };
-      }),
-      format: count,
-      floor: 1
     }),
     buildVolumeCard("PRs opened per week", created),
+    ...targetCard === null ? [] : [targetCard],
     buildSpreadCard("Size spread", stats.metrics, count)
   ];
-  if (stats.met !== void 0 && stats.targetLabel !== void 0) {
-    left.push(
-      buildGaugeCard({
-        title: "Size target",
-        subtitle: `authored within ${stats.targetLabel}`,
-        rows: [
-          { label: "inside target", count: stats.met, color: theme.accent },
-          { label: "over target", count: sizes.length - stats.met, color: theme.chartDim }
-        ]
-      })
-    );
-  }
   const distribution = buildDistribution({
     values: totals,
     width: Math.max(width - 3, 40),
@@ -4703,7 +4979,7 @@ function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
       )
     });
   }
-  return { empty: null, ...base, strip, headline, left, right, distribution, lists };
+  return { empty: null, ...base, strip, headline, cards, distribution, lists };
 }
 function buildCommentView(raw, repo = null, width = 100) {
   const base = {
@@ -4711,8 +4987,7 @@ function buildCommentView(raw, repo = null, width = 100) {
     headline: null,
     distributionTitle: "Comments per PR distribution",
     noCharts: "No authored PRs to chart.",
-    left: [],
-    right: [],
+    cards: [],
     distribution: null,
     lists: []
   };
@@ -4742,7 +5017,7 @@ function buildCommentView(raw, repo = null, width = 100) {
     { text: `${count(percentile(sorted, 90))} comments`, fg: theme.accent },
     { text: `   ${sizes.length} of ${raw.sizes.length} PRs`, fg: theme.muted }
   ];
-  const left = [
+  const cards = [
     buildHistogramCard({
       title: "Comments per PR",
       subtitle: "discussion plus review comments per authored PR",
@@ -4750,6 +5025,14 @@ function buildCommentView(raw, repo = null, width = 100) {
       buckets: COMMENT_BUCKETS,
       format: count
     }),
+    buildTrendCard({
+      title: "Comment trend",
+      entries: sizes.map((size) => {
+        return { date: size.pr.createdAt, value: size.comments.total };
+      }),
+      format: count,
+      floor: 1
+    }),
     buildScatterCard({
       title: "Comments vs size",
       subtitle: "comments against lines changed, log scale",
@@ -4759,18 +5042,8 @@ function buildCommentView(raw, repo = null, width = 100) {
       formatX: count,
       formatY: count
     }),
-    buildSpreadCard("Comment spread", stats.metrics, count)
-  ];
-  const right = [
-    buildTrendCard({
-      title: "Comment trend",
-      entries: sizes.map((size) => {
-        return { date: size.pr.createdAt, value: size.comments.total };
-      }),
-      format: count,
-      floor: 1
-    }),
     buildVolumeCard("Comments received per week", created, stats.totals),
+    buildSpreadCard("Comment spread", stats.metrics, count),
     buildGaugeCard({
       title: "Feedback rate",
       subtitle: "authored PRs that received comments",
@@ -4800,7 +5073,7 @@ function buildCommentView(raw, repo = null, width = 100) {
       )
     });
   }
-  return { empty: null, ...base, strip, headline, left, right, distribution, lists };
+  return { empty: null, ...base, strip, headline, cards, distribution, lists };
 }
 function buildMergedView(raw, repo = null, width = 100) {
   const base = {
@@ -4808,8 +5081,7 @@ function buildMergedView(raw, repo = null, width = 100) {
     headline: null,
     distributionTitle: "Time to merge distribution",
     noCharts: "No merged PRs to chart.",
-    left: [],
-    right: [],
+    cards: [],
     distribution: null,
     lists: []
   };
@@ -4864,7 +5136,7 @@ function buildMergedView(raw, repo = null, width = 100) {
   ];
   const mergeDates = stats.merged.map((result) => result.mergedAt);
   const created = sizes.map((size) => size.pr.createdAt);
-  const left = [
+  const cards = [
     buildHistogramCard({
       title: "Time to merge",
       subtitle: "elapsed time, created \u2192 merged",
@@ -4872,6 +5144,14 @@ function buildMergedView(raw, repo = null, width = 100) {
       buckets: currentBuckets(),
       format: formatDuration
     }),
+    buildTrendCard({
+      title: "Time to merge trend",
+      entries: stats.merged.map((result) => {
+        return { date: result.mergedAt, value: result.hours };
+      }),
+      format: formatDuration,
+      floor: 1 / 60
+    }),
     buildHeatmapCard({
       title: "When your PRs merge",
       subtitle: "PRs merged, weekday \xD7 hour, local time",
@@ -4882,6 +5162,38 @@ function buildMergedView(raw, repo = null, width = 100) {
       ],
       legend: "PRs merged in that hour"
     }),
+    buildTrendCard({
+      title: "Merge rate trend",
+      entries: mergeRateEntries(stats),
+      format: (value2) => `${Math.round(value2)}%`,
+      scale: "linear",
+      valueLabel: "merge rate"
+    }),
+    buildScatterCard({
+      title: "Merge time vs size",
+      subtitle: "time to merge against lines changed, log scale",
+      points: stats.merged.map((result) => {
+        return { x: result.entry.total, y: result.hours };
+      }),
+      formatX: count,
+      formatY: formatDuration
+    }),
+    buildCumulativeCard({
+      title: "Created vs merged",
+      series: [
+        /**
+         * The created line stays a neutral gray because every theme preset
+         * keeps chartLine and accent in one hue family, which made the two
+         * lines indistinguishable. The gray also survives theme changes,
+         * since the presets only rotate the hue-carrying colors.
+         */
+        { label: "created", dates: created, color: theme.muted },
+        { label: "merged", dates: mergeDates, color: theme.accent }
+      ],
+      legend: "cumulative PRs by week"
+    }),
+    buildVolumeCard("PRs created per week", created),
+    buildVolumeCard("PRs merged per week", mergeDates),
     buildGaugeCard({
       title: "Outcomes",
       subtitle: "where your authored PRs ended up",
@@ -4892,18 +5204,6 @@ function buildMergedView(raw, repo = null, width = 100) {
       ]
     })
   ];
-  const right = [
-    buildTrendCard({
-      title: "Time to merge trend",
-      entries: stats.merged.map((result) => {
-        return { date: result.mergedAt, value: result.hours };
-      }),
-      format: formatDuration,
-      floor: 1 / 60
-    }),
-    buildVolumeCard("PRs merged per week", mergeDates),
-    buildVolumeCard("PRs created per week", created)
-  ];
   const distribution = buildDistribution({
     values: stats.allHours,
     width: Math.max(width - 3, 40),
@@ -4911,11 +5211,14 @@ function buildMergedView(raw, repo = null, width = 100) {
     ticks: DURATION_TICKS,
     flat: (n, value2) => `all ${n} merged ${n === 1 ? "PR" : "PRs"} took ${value2}`
   });
-  return { empty: null, ...base, strip, headline, left, right, distribution, lists };
+  return { empty: null, ...base, strip, headline, cards, distribution, lists };
 }
 function count(value2) {
   return formatCount(Math.round(value2));
 }
+function netCount(value2) {
+  return value2 < 0 ? `-${count(-value2)}` : `+${count(value2)}`;
+}
 
 // src/tui/hooks/useViewModel.ts
 function resolveScope(scope, repos) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@d3lm/pr-stats",
-  "version": "0.2.6",
+  "version": "0.2.7",
   "description": "GitHub PR stats in an interactive terminal UI, via the gh CLI or an access token",
   "type": "module",
   "author": "Dominic Elm",