@d3lm/pr-stats 0.2.7 → 0.2.8

package/README.md

@@ -28,12 +28,12 @@ pr-stats
 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 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. A reviewer leaderboard ranks who reviews your PRs by distinct PRs reviewed, and a review-coverage gauge counts the merged PRs that never received a review. Your own replies to review threads never count as a review for either of them.
 - 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.
+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. Comparison cards like the reviewer leaderboard cap themselves at eight rows and fold the rest into an overflow line, and the `x` key lifts the cap on the open stats tab and restores it. The footer names the key whenever the tab has a capped card.
 
 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

@@ -146,7 +146,7 @@ import { join as join2 } from "node:path";
 import { mkdirSync, readFileSync, renameSync, rmSync, writeFileSync } from "node:fs";
 import { homedir } from "node:os";
 import { dirname, join } from "node:path";
-var VERSION = 2;
+var VERSION = 3;
 var enabled = false;
 function configureCache(on) {
   enabled = on;
@@ -621,10 +621,12 @@ function hintsFor(modal, editing, tab, authoredTab, views, copyLinks2) {
   if (scope?.view === "list") {
     return `\u2191/\u2193 select \xB7 enter open \xB7 ${toggle}\u2190/\u2192 tabs \xB7 o options \xB7 s settings \xB7 r reload \xB7 R refetch \xB7 q quit`;
   }
+  const view = views === null ? null : tab === 1 ? views.merged : tab === 2 ? views.review : tab === 3 ? views.size : views.comments;
+  const expand = view?.expandable ? view.expanded ? "x collapse \xB7 " : "x expand \xB7 " : "";
   if (scope !== null && repos.length > 0) {
-    return `${toggle}esc back \xB7 j/k scroll \xB7 1-5 tabs \xB7 o options \xB7 s settings \xB7 r reload \xB7 R refetch \xB7 q quit`;
+    return `${toggle}${expand}esc back \xB7 j/k scroll \xB7 1-5 tabs \xB7 o options \xB7 s settings \xB7 r reload \xB7 R refetch \xB7 q quit`;
   }
-  return `${toggle}1-5 tabs \xB7 j/k scroll \xB7 o options \xB7 s settings \xB7 r reload \xB7 R refetch \xB7 q quit`;
+  return `${toggle}${expand}1-5 tabs \xB7 j/k scroll \xB7 o options \xB7 s settings \xB7 r reload \xB7 R refetch \xB7 q quit`;
 }
 
 // src/tui/components/Spinner.tsx
@@ -883,6 +885,37 @@ function computeMergeStats(sizes) {
   closed.sort((a, b) => b.closedAt.getTime() - a.closedAt.getTime());
   return { merged, closed, open, allHours: merged.map((result) => result.hours) };
 }
+function computeReviewerStats(sizes, author) {
+  const byLogin = /* @__PURE__ */ new Map();
+  let mergedReviewed = 0;
+  let mergedUnreviewed = 0;
+  for (const entry of sizes) {
+    const others = entry.reviewers.filter((login) => login !== author);
+    for (const login of others) {
+      const counts = byLogin.get(login) ?? { prs: 0, reviews: 0 };
+      counts.reviews += 1;
+      byLogin.set(login, counts);
+    }
+    const distinct = new Set(others);
+    for (const login of distinct) {
+      const counts = byLogin.get(login);
+      if (counts !== void 0) {
+        counts.prs += 1;
+      }
+    }
+    if (entry.mergedAt !== null) {
+      if (others.length > 0) {
+        mergedReviewed += 1;
+      } else {
+        mergedUnreviewed += 1;
+      }
+    }
+  }
+  const leaderboard = [...byLogin.entries()].map(([login, counts]) => {
+    return { login, ...counts };
+  }).toSorted((a, b) => b.prs - a.prs || b.reviews - a.reviews || a.login.localeCompare(b.login));
+  return { leaderboard, mergedReviewed, mergedUnreviewed };
+}
 function computeCommentStats(sizes) {
   const metrics = [
     { label: "discussion comments", values: sizes.map((size) => size.comments.discussion) },
@@ -1086,9 +1119,24 @@ function useScrollbarSettle(scrollRef, mounted = true) {
       renderer2.off(CliRenderEvents.FRAME, release);
       scrollRef.current?.verticalScrollBar.resetVisibilityControl();
     };
+    const resyncThumb = () => {
+      const bar = scrollRef.current?.verticalScrollBar;
+      if (!bar) {
+        return;
+      }
+      const height = bar.viewportSize;
+      const wanted = Math.max(1, height);
+      if (bar.scrollSize < wanted || bar.slider.viewPortSize === wanted) {
+        return;
+      }
+      bar.viewportSize = 0;
+      bar.viewportSize = height;
+    };
     renderer2.on(CliRenderEvents.FRAME, release);
+    renderer2.on(CliRenderEvents.FRAME, resyncThumb);
     return () => {
       renderer2.off(CliRenderEvents.FRAME, release);
+      renderer2.off(CliRenderEvents.FRAME, resyncThumb);
     };
   }, [scrollRef, renderer2, mounted]);
 }
@@ -1428,7 +1476,8 @@ var initialBrowseState = {
   },
   repoCursors: { pending: 0, open: 0, review: 0, size: 0, comment: 0, merged: 0 },
   rowCursors: { pending: 0, open: 0 },
-  grouped: { pending: false, open: false }
+  grouped: { pending: false, open: false },
+  expanded: { review: false, size: false, comment: false, merged: false }
 };
 function dropVanishedRepo(scope, repos) {
   if (scope.view === "detail" && scope.repo !== null && !repos.some((option) => option.repo === scope.repo)) {
@@ -1477,6 +1526,9 @@ function browseReducer(state, action) {
     case "groupingToggled": {
       return { ...state, grouped: { ...state.grouped, [action.tab]: !state.grouped[action.tab] } };
     }
+    case "expandToggled": {
+      return { ...state, expanded: { ...state.expanded, [action.tab]: !state.expanded[action.tab] } };
+    }
     case "dataLoaded": {
       return {
         ...state,
@@ -3402,6 +3454,7 @@ async function fetchPrSizes(prs) {
           }
           reviews(first: 100) {
             nodes {
+              author { login }
               comments {
                 totalCount
               }
@@ -3596,6 +3649,7 @@ async function fetchSizeRaw(prs, onProgress, options = {}) {
     if (details) {
       const discussion = details.comments.totalCount;
       const review = details.reviews.nodes.reduce((sum, node) => sum + (node?.comments.totalCount ?? 0), 0);
+      const reviewers = details.reviews.nodes.flatMap((node) => node?.author == null ? [] : [node.author.login]);
       sizes.push({
         pr,
         files: details.changedFiles,
@@ -3604,7 +3658,8 @@ async function fetchSizeRaw(prs, onProgress, options = {}) {
         total: details.additions + details.deletions,
         mergedAt: details.mergedAt === null ? null : new Date(details.mergedAt),
         closedAt: details.closedAt === null ? null : new Date(details.closedAt),
-        comments: { discussion, review, total: discussion + review }
+        comments: { discussion, review, total: discussion + review },
+        reviewers
       });
     }
   }
@@ -4048,8 +4103,8 @@ function hbar(fraction, width, color) {
 // 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);
+function buildBarsCard({ title, subtitle, rows, format, expanded = false }) {
+  const shown = expanded ? rows : 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));
@@ -4067,7 +4122,7 @@ function buildBarsCard({ title, subtitle, rows, format }) {
     return line;
   });
   if (rows.length > shown.length) {
-    lines.push([{ text: `+ ${rows.length - shown.length} more`, fg: theme.dim }]);
+    lines.push([{ text: `+ ${rows.length - shown.length} more \xB7 x expands`, fg: theme.dim }]);
   }
   return { title, subtitle, lines };
 }
@@ -4736,7 +4791,7 @@ function buildVerdictCard(reviewed) {
     })
   });
 }
-function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100) {
+function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100, expanded = false) {
   const results = repo === null ? raw.reviewResults : raw.reviewResults.filter((result) => result.pr.repo === repo);
   const stats = computeReviewStats(results, { targetHours, now: raw.fetchedAt });
   const strip = [
@@ -4751,7 +4806,9 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
     distributionTitle: "Review time distribution",
     noCharts: "No completed reviews to chart.",
     cards: [],
-    distribution: null
+    distribution: null,
+    expandable: false,
+    expanded
   };
   if (results.length === 0) {
     return { empty: "No reviewed or review-requested PRs found.", ...base, lists: [] };
@@ -4814,7 +4871,8 @@ function buildReviewView(raw, targetHours, targetLabel, repo = null, width = 100
         detail: `n=${hours.length}`
       };
     }).toSorted((a, b) => b.value - a.value),
-    format: formatDuration
+    format: formatDuration,
+    expanded
   }) : null;
   const cards = [
     buildHistogramCard({
@@ -4870,7 +4928,15 @@ 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, cards, distribution, lists };
+  return {
+    empty: null,
+    ...base,
+    headline,
+    cards,
+    distribution,
+    lists,
+    expandable: byRepoCard !== null && stats.byRepo.length > MAX_BARS
+  };
 }
 function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
   const base = {
@@ -4880,7 +4946,9 @@ function buildSizeView(raw, sizeTarget, repo = null, width = 100) {
     noCharts: "No authored PRs to chart.",
     cards: [],
     distribution: null,
-    lists: []
+    lists: [],
+    expandable: false,
+    expanded: false
   };
   if (raw.authoredTotal === 0) {
     return { empty: "No authored PRs found.", ...base };
@@ -4989,7 +5057,9 @@ function buildCommentView(raw, repo = null, width = 100) {
     noCharts: "No authored PRs to chart.",
     cards: [],
     distribution: null,
-    lists: []
+    lists: [],
+    expandable: false,
+    expanded: false
   };
   if (raw.authoredTotal === 0) {
     return { empty: "No authored PRs found.", ...base };
@@ -5075,7 +5145,7 @@ function buildCommentView(raw, repo = null, width = 100) {
   }
   return { empty: null, ...base, strip, headline, cards, distribution, lists };
 }
-function buildMergedView(raw, repo = null, width = 100) {
+function buildMergedView(raw, repo = null, width = 100, expanded = false) {
   const base = {
     strip: [],
     headline: null,
@@ -5083,7 +5153,9 @@ function buildMergedView(raw, repo = null, width = 100) {
     noCharts: "No merged PRs to chart.",
     cards: [],
     distribution: null,
-    lists: []
+    lists: [],
+    expandable: false,
+    expanded
   };
   if (raw.authoredTotal === 0) {
     return { empty: "No authored PRs found.", ...base };
@@ -5093,6 +5165,7 @@ function buildMergedView(raw, repo = null, width = 100) {
     return { empty: "No accessible authored PRs to analyze.", ...base };
   }
   const stats = computeMergeStats(sizes);
+  const reviewers = computeReviewerStats(sizes, raw.user);
   const strip = [
     countCell(sizes.length, "PRs created"),
     countCell(stats.merged.length, "merged"),
@@ -5123,8 +5196,22 @@ function buildMergedView(raw, repo = null, width = 100) {
       )
     });
   }
+  const reviewerCard = reviewers.leaderboard.length === 0 ? null : buildBarsCard({
+    title: "Who reviews your PRs",
+    subtitle: "distinct PRs reviewed per person",
+    rows: reviewers.leaderboard.map((row) => {
+      return {
+        label: row.login,
+        value: row.prs,
+        detail: row.reviews === 1 ? "1 review" : `${row.reviews} reviews`
+      };
+    }),
+    format: count,
+    expanded
+  });
+  const expandable = reviewers.leaderboard.length > MAX_BARS;
   if (stats.merged.length === 0) {
-    return { empty: null, ...base, strip, lists };
+    return { empty: null, ...base, strip, cards: reviewerCard === null ? [] : [reviewerCard], lists, expandable };
   }
   const sorted = [...stats.allHours].toSorted((a, b) => a - b);
   const headline = [
@@ -5202,7 +5289,16 @@ function buildMergedView(raw, repo = null, width = 100) {
         { label: "closed unmerged", count: stats.closed.length, color: theme.warn },
         ...stats.open.length > 0 ? [{ label: "still open", count: stats.open.length, color: theme.chartDim }] : []
       ]
-    })
+    }),
+    buildGaugeCard({
+      title: "Review coverage",
+      subtitle: "merged PRs that received a review",
+      rows: [
+        { label: "reviewed", count: reviewers.mergedReviewed, color: theme.accent },
+        { label: "merged unreviewed", count: reviewers.mergedUnreviewed, color: theme.warn }
+      ]
+    }),
+    ...reviewerCard === null ? [] : [reviewerCard]
   ];
   const distribution = buildDistribution({
     values: stats.allHours,
@@ -5211,7 +5307,7 @@ 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, cards, distribution, lists };
+  return { empty: null, ...base, strip, headline, cards, distribution, lists, expandable };
 }
 function count(value2) {
   return formatCount(Math.round(value2));
@@ -5227,7 +5323,7 @@ function resolveScope(scope, repos) {
   }
   return dropVanishedRepo(scope, repos);
 }
-function useViewModel(raw, options, width, scopes, grouping, themeEpoch) {
+function useViewModel(raw, options, width, scopes, grouping, expanded, themeEpoch) {
   return useMemo(() => {
     void themeEpoch;
     configureTimeMode({
@@ -5253,7 +5349,7 @@ function useViewModel(raw, options, width, scopes, grouping, themeEpoch) {
     const reviewScope = resolveScope(scopes.review, reviewRepos);
     const sizeScope = resolveScope(scopes.size, sizeRepos);
     const commentScope = resolveScope(scopes.comment, commentRepos);
-    const review = reviewScope.view === "detail" ? buildReviewView(raw, targetHours, targetLabelOf(options.target), reviewScope.repo, width) : null;
+    const review = reviewScope.view === "detail" ? buildReviewView(raw, targetHours, targetLabelOf(options.target), reviewScope.repo, width, expanded.review) : null;
     return {
       pendingRepos,
       openRepos,
@@ -5269,7 +5365,7 @@ function useViewModel(raw, options, width, scopes, grouping, themeEpoch) {
       commentScope,
       pending: pendingScope.view === "detail" ? buildPendingReviewView(raw, pendingScope.repo, grouping.pending) : null,
       open: openScope.view === "detail" ? buildOpenAuthoredView(raw, openScope.repo, grouping.open) : null,
-      merged: mergedScope.view === "detail" ? buildMergedView(raw, mergedScope.repo, width) : null,
+      merged: mergedScope.view === "detail" ? buildMergedView(raw, mergedScope.repo, width, expanded.merged) : null,
       review,
       size: sizeScope.view === "detail" ? buildSizeView(raw, sizeTarget, sizeScope.repo, width) : null,
       comments: commentScope.view === "detail" ? buildCommentView(raw, commentScope.repo, width) : null
@@ -5290,6 +5386,8 @@ function useViewModel(raw, options, width, scopes, grouping, themeEpoch) {
     scopes.comment,
     grouping.pending,
     grouping.open,
+    expanded.review,
+    expanded.merged,
     themeEpoch
   ]);
 }
@@ -5519,31 +5617,35 @@ function statsTabOf(context) {
     return {
       key: "merged",
       repos: views?.mergedRepos ?? [],
-      scope: views?.mergedScope ?? null
+      scope: views?.mergedScope ?? null,
+      view: views?.merged ?? null
     };
   }
   if (context.browse.tab === 2) {
     return {
       key: "review",
       repos: views?.reviewRepos ?? [],
-      scope: views?.reviewScope ?? null
+      scope: views?.reviewScope ?? null,
+      view: views?.review ?? null
     };
   }
   if (context.browse.tab === 3) {
     return {
       key: "size",
       repos: views?.sizeRepos ?? [],
-      scope: views?.sizeScope ?? null
+      scope: views?.sizeScope ?? null,
+      view: views?.size ?? null
     };
   }
   return {
     key: "comment",
     repos: views?.commentRepos ?? [],
-    scope: views?.commentScope ?? null
+    scope: views?.commentScope ?? null,
+    view: views?.comments ?? null
   };
 }
 function handleStatsKey(key, context) {
-  const { key: tab, repos, scope } = statsTabOf(context);
+  const { key: tab, repos, scope, view } = statsTabOf(context);
   if (scope !== null && scope.view === "list") {
     switch (key.name) {
       case "up":
@@ -5564,6 +5666,8 @@ function handleStatsKey(key, context) {
     }
   } else if ((key.name === "escape" || key.name === "backspace") && repos.length > 0) {
     context.dispatchBrowse({ type: "pickerReturned", tab });
+  } else if (key.name === "x" && view?.expandable === true) {
+    context.dispatchBrowse({ type: "expandToggled", tab });
   } else if (key.name === "j") {
     context.scrollBy(context.browse.tab, 2);
   } else if (key.name === "k") {
@@ -5829,7 +5933,7 @@ function App({
       }
     });
   });
-  const views = useViewModel(raw, options, width, browse.scopes, browse.grouped, themeState);
+  const views = useViewModel(raw, options, width, browse.scopes, browse.grouped, browse.expanded, themeState);
   const showLoad = useDeferredLoading(loading, isSnapshot ? { showDelay: 0 } : void 0);
   const draftRef = useRef5("");
   const commitField = () => {

package/package.json

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