@kud/gh-ink 0.11.0 → 0.12.0

package/dist/index.d.ts

@@ -225,7 +225,7 @@ declare const ActionMenu: ({ item, actions, cursor, }: {
     actions: Action[];
     cursor: number;
 }) => React.JSX.Element;
-declare const App: ({ fetcher, cacheKey, title, detailFor, isWorkRepo, initialIncludeWork, jiraBase, jiraKeyRe, jiraTransitions, workToggle, hasCiStatus, ciJob, ciFetcher, ciPollMs, extensions, tabHelp, emptyHint, }: {
+declare const App: ({ fetcher, cacheKey, title, detailFor, isWorkRepo, initialIncludeWork, jiraBase, jiraKeyRe, jiraTransitions, workToggle, hasCiStatus, ciJob, ciFetcher, ciPollMs, watchPath, watchDebounceMs, extensions, tabHelp, emptyHint, }: {
     fetcher: () => Promise<{
         sections: Section[];
         login: string;
@@ -246,6 +246,14 @@ declare const App: ({ fetcher, cacheKey, title, detailFor, isWorkRepo, initialIn
     ciJob?: string;
     ciFetcher?: () => Promise<CiStatus | null>;
     ciPollMs?: number;
+    /**
+     * A file something else touches when it has changed GitHub on your behalf —
+     * a Claude session closing an issue, a script merging a PR. Touching it makes
+     * the inbox refetch; it never repaints on its own.
+     */
+    watchPath?: string;
+    /** Bursts of writes to collapse into one refetch. */
+    watchDebounceMs?: number;
     extensions?: InboxExtension[];
 }) => React.JSX.Element;
 

package/dist/index.js

@@ -1,12 +1,12 @@
-import React2, { createContext, useState, useRef, useEffect, useContext } from 'react';
+import React2, { createContext, useState, useRef, useEffect, useMemo, useContext } from 'react';
 import { useWindowSize, useInput, Text, Box } from 'ink';
 import { colors, ScrollView, TextInput, LoadingScreen, Switch, StatusMessage, FooterHints, useListCursor, Tabs } from '@kud/ink-ui';
 import { isPassCheck, isFailCheck, resolveThread, unresolveThread, replyToThread, rerunFailedRun, mergePr, reRequestReviewer } from '@kud/gh';
 import { jsxs, Fragment, jsx } from 'react/jsx-runtime';
 import { $ } from 'zx';
 import { spawn } from 'child_process';
-import { readFileSync, mkdirSync, writeFileSync, existsSync, readdirSync, statSync, rmSync } from 'fs';
-import { join } from 'path';
+import { readFileSync, mkdirSync, writeFileSync, existsSync, readdirSync, statSync, watch, rmSync } from 'fs';
+import { join, dirname, basename } from 'path';
 import { homedir } from 'os';
 
 // src/components/comments-panel.tsx
@@ -637,6 +637,102 @@ var invalidateCache = (key) => {
   } catch {
   }
 };
+
+// src/inbox/diff.ts
+var keyOf = (item) => {
+  if (item.kind === "pr" || item.kind === "issue") return item.url;
+  if (item.kind === "jira") {
+    const row = item;
+    return row.url || `jira:${row.key}`;
+  }
+  return null;
+};
+var renderedState = (item) => {
+  if (item.kind === "pr" || item.kind === "issue") {
+    const i = item;
+    return [
+      i.title,
+      i.health,
+      i.unresolved,
+      i.conversation,
+      i.lastActor ?? "",
+      i.author ?? ""
+    ].join("|");
+  }
+  if (item.kind === "jira") {
+    const j = item;
+    return [j.summary, j.jiraStatus].join("|");
+  }
+  return "";
+};
+var indexRows = (sections) => {
+  const out = /* @__PURE__ */ new Map();
+  for (const section of sections) {
+    section.items.forEach((item, index) => {
+      const key = keyOf(item);
+      if (key) out.set(key, { item, sectionId: section.id, index });
+    });
+  }
+  return out;
+};
+var spliceBack = (next, leaving) => {
+  if (leaving.length === 0) return next;
+  const bySection = /* @__PURE__ */ new Map();
+  for (const row of leaving) {
+    const list = bySection.get(row.sectionId);
+    if (list) list.push(row);
+    else bySection.set(row.sectionId, [row]);
+  }
+  return next.map((section) => {
+    const rows = bySection.get(section.id);
+    if (!rows) return section;
+    const items = [...section.items];
+    for (const row of [...rows].sort((a, b) => a.index - b.index))
+      items.splice(Math.min(row.index, items.length), 0, row.item);
+    return { ...section, items };
+  });
+};
+var diffSections = (prev, next) => {
+  const before = indexRows(prev);
+  const after = indexRows(next);
+  const transients = /* @__PURE__ */ new Map();
+  const leaving = [];
+  for (const [key] of after) if (!before.has(key)) transients.set(key, "in");
+  for (const [key, was] of before) {
+    const now = after.get(key);
+    if (!now) {
+      transients.set(key, "out");
+      leaving.push(was);
+    } else if (renderedState(was.item) !== renderedState(now.item)) {
+      transients.set(key, "changed");
+    }
+  }
+  let added = 0;
+  let removed = 0;
+  let changed = 0;
+  for (const [, kind] of transients) {
+    if (kind === "in") added++;
+    else if (kind === "out") removed++;
+    else changed++;
+  }
+  return {
+    transients,
+    union: spliceBack(next, leaving),
+    counts: { added, removed, changed }
+  };
+};
+var summariseDiff = (counts) => {
+  const parts = [];
+  if (counts.added) parts.push(`${counts.added} new`);
+  if (counts.removed) parts.push(`${counts.removed} gone`);
+  if (counts.changed) parts.push(`${counts.changed} moved`);
+  return parts.join(" \xB7 ");
+};
+var transientOf = (transients, item) => {
+  if (!transients?.size) return void 0;
+  const key = keyOf(item);
+  return key ? transients.get(key) : void 0;
+};
 var quietly = $({ quiet: true });
 var relativeTime = (iso) => {
   const diff = (Date.now() - new Date(iso).getTime()) / 1e3;
@@ -657,15 +753,24 @@ var healthSentence = (item) => {
     case "draft":
       return `Still a draft (${glyph}), so no review is being asked for yet.`;
     case "ci-fail":
-      return `CI is failing (${glyph}) \u2014 ${plural(d?.checksFail ?? 0, "check")} red.`;
+      return `CI is failing (${glyph}) \u2014 ${plural(
+        d?.checksFail ?? 0,
+        "check"
+      )} red.`;
     case "conflict":
       return `It conflicts with the base branch (${glyph}) and cannot merge until that is resolved.`;
     case "changes-req":
       return `Changes were requested (${glyph}).`;
     case "threads":
-      return `${plural(item.unresolved, "review thread")} still open (${glyph}).`;
+      return `${plural(
+        item.unresolved,
+        "review thread"
+      )} still open (${glyph}).`;
     case "pending":
-      return `${plural(d?.checksPending ?? 0, "check")} still running (${glyph}).`;
+      return `${plural(
+        d?.checksPending ?? 0,
+        "check"
+      )} still running (${glyph}).`;
     case "approved":
       return `Approved (${glyph}) and ready to merge.`;
     case "waiting":
@@ -698,7 +803,9 @@ var turnSentences = (item, login) => {
   const lines = [`You spoke last (\u2192), ${when}. The ball is with ${them}.`];
   if (d?.lastCommitAt && d.lastEventAt && d.lastCommitAt < d.lastEventAt)
     lines.push(
-      `Nothing has been pushed since ${relativeTime(d.lastCommitAt)} ago, so it is stalled on ${them}, not on you.`
+      `Nothing has been pushed since ${relativeTime(
+        d.lastCommitAt
+      )} ago, so it is stalled on ${them}, not on you.`
     );
   return lines;
 };
@@ -1370,13 +1477,14 @@ var InboxHeader = ({
   quiet,
   refreshing,
   hasPending,
+  pendingSummary,
   fetchedAt
 }) => {
   const total = sections.reduce((n, s) => n + topLevelCount(s), 0);
   const countSeg = loading ? "      loading\u2026  " : quiet ? "  " : `  ${String(total).padStart(3)} item${total !== 1 ? "s" : ""}  \xB7  `;
   const userSeg = loading || quiet ? "" : `@${login}  `;
   const workLabel = work === void 0 ? "" : " w work \u25CF\u2500\u25CB home  ";
-  const [statusText, statusColor] = hasPending ? ["\u25CF new \xB7 r apply", "#FF8700"] : refreshing ? ["\u21BB refreshing\u2026", "cyan"] : fetchedAt ? [`updated ${agoText(fetchedAt)}`, void 0] : ["", void 0];
+  const [statusText, statusColor] = hasPending ? [`\u25CF ${pendingSummary || "new"} \xB7 r apply`, "#FF8700"] : refreshing ? ["\u21BB refreshing\u2026", "cyan"] : fetchedAt ? [`updated ${agoText(fetchedAt)}`, void 0] : ["", void 0];
   const statusSeg = statusText ? statusText + "  " : "";
   const fill = Math.max(
     4,
@@ -1452,12 +1560,27 @@ var MERGED_HOLD_MS = 3e3;
 var MERGED_FRAME_MS = 150;
 var MERGED_FRAMES = ["\u2726", "\u2727", "\u2736", "\u2727"];
 var MERGED_COLOUR = "#A371F7";
+var TRANSIT_HOLD_MS = 2500;
+var NO_TRANSIENTS = /* @__PURE__ */ new Map();
+var TRANSIT_OUT_FRAMES = ["\u25C9", "\u25CE", "\u25CB", "\xB7"];
+var TRANSIT_IN_FRAMES = ["\xB7", "\u25CB", "\u25CE", "\u25C9"];
+var TRANSIT_COLOUR = {
+  in: "#3FB950",
+  out: "#8B949E",
+  changed: "#FF8700"
+};
+var TRANSIT_LABEL = {
+  in: "NEW",
+  out: "GONE",
+  changed: "UPDATED"
+};
 var ItemRow = ({
   item,
   active,
   gap,
   login,
   merged,
+  transient,
   sparkFrame = 0
 }) => {
   if (item.kind === "repo-header")
@@ -1490,19 +1613,21 @@ var ItemRow = ({
     ] });
   }
   const { glyph: healthIcon, color: healthColor2 } = healthDisplay[item.health];
-  const icon = merged ? MERGED_FRAMES[sparkFrame % MERGED_FRAMES.length] : healthIcon;
-  const color = merged ? MERGED_COLOUR : healthColor2;
+  const icon = merged ? MERGED_FRAMES[sparkFrame % MERGED_FRAMES.length] : transient === "out" ? TRANSIT_OUT_FRAMES[sparkFrame % TRANSIT_OUT_FRAMES.length] : transient === "in" ? TRANSIT_IN_FRAMES[sparkFrame % TRANSIT_IN_FRAMES.length] : healthIcon;
+  const color = merged ? MERGED_COLOUR : transient ? TRANSIT_COLOUR[transient] : healthColor2;
   const [turnIcon, turnColor] = !login || !item.lastActor ? [" ", "white"] : item.lastActor === login ? ["\u2192", "#888888"] : ["\u2190", "#FF8700"];
   const numStr = `#${item.number}`.padEnd(7);
   const showAuthor = !!item.author && item.author !== login;
   const unresolvedLabel = item.unresolved > 0 ? `\uF086 ${item.unresolved}` : "";
   const ageLabel = item.activityAge && item.activityAge !== item.age ? `${item.activityAge} \xB7 ${item.age}` : item.age;
   const mergedLabel = merged ? "MERGED" : "";
+  const transitLabel = merged || !transient ? "" : TRANSIT_LABEL[transient];
   const suffix = [
     ageLabel || "",
     unresolvedLabel,
     showAuthor ? `by ${item.author}` : "",
-    mergedLabel
+    mergedLabel,
+    transitLabel
   ].filter(Boolean).join("  ");
   const repoLabel = item.indent ? item.repo : "";
   const fixedWidth = 2 + (item.indent ? 3 : 0) + 2 + 2 + 7 + repoLabel.length + suffix.length + 6;
@@ -1513,12 +1638,21 @@ var ItemRow = ({
     /* @__PURE__ */ jsx(Text2, { color, bold: true, children: icon + " " }),
     /* @__PURE__ */ jsx(Text2, { color: turnColor, bold: turnIcon === "\u2190", children: turnIcon + " " }),
     /* @__PURE__ */ jsx(Text2, { color: "#FF8700", children: numStr }),
-    /* @__PURE__ */ jsx(Text2, { bold: active, children: truncate(item.title, titleMax) + "  " }),
+    /* @__PURE__ */ jsx(
+      Text2,
+      {
+        bold: active || transient === "in",
+        dimColor: transient === "out",
+        strikethrough: transient === "out",
+        children: truncate(item.title, titleMax) + "  "
+      }
+    ),
     repoLabel ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: repoLabel }) : null,
     unresolvedLabel ? /* @__PURE__ */ jsx(Text2, { bold: true, color: "#FF8700", children: "  " + unresolvedLabel }) : null,
     showAuthor ? /* @__PURE__ */ jsx(Text2, { dimColor: true, italic: true, children: "  by " + item.author }) : null,
     ageLabel ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "  " + ageLabel }) : null,
-    mergedLabel ? /* @__PURE__ */ jsx(Text2, { bold: true, color: MERGED_COLOUR, children: "  " + mergedLabel }) : null
+    mergedLabel ? /* @__PURE__ */ jsx(Text2, { bold: true, color: MERGED_COLOUR, children: "  " + mergedLabel }) : null,
+    transitLabel ? /* @__PURE__ */ jsx(Text2, { bold: true, color: TRANSIT_COLOUR[transient], children: "  " + transitLabel }) : null
   ] });
 };
 var useActionMenu = () => {
@@ -1794,6 +1928,7 @@ var BrowseScreen = ({
   onActed,
   refreshing,
   hasPending,
+  pendingSummary,
   fetchedAt,
   refreshError,
   workToggle,
@@ -1808,7 +1943,8 @@ var BrowseScreen = ({
   isWorkRepo,
   initialIncludeWork,
   brand,
-  mergedUrls
+  mergedUrls,
+  transients
 }) => {
   const { rows } = useWindowSize();
   const [includeWork, setIncludeWork] = useState(
@@ -1899,7 +2035,7 @@ var BrowseScreen = ({
     setTimeout(() => setFlash(null), 2e3);
   };
   const [sparkFrame, setSparkFrame] = useState(0);
-  const sparkling = (mergedUrls?.length ?? 0) > 0;
+  const sparkling = (mergedUrls?.length ?? 0) > 0 || (transients?.size ?? 0) > 0;
   useEffect(() => {
     if (!sparkling) return;
     const id = setInterval(() => setSparkFrame((f) => f + 1), MERGED_FRAME_MS);
@@ -2231,6 +2367,7 @@ var BrowseScreen = ({
             work: workToggle ? includeWork : void 0,
             refreshing,
             hasPending,
+            pendingSummary,
             fetchedAt
           }
         ),
@@ -2250,13 +2387,7 @@ var BrowseScreen = ({
           /* @__PURE__ */ jsx(Text2, { color: "cyan", children: "  / " }),
           /* @__PURE__ */ jsx(Text2, { children: search }),
           searchInput ? /* @__PURE__ */ jsx(Text2, { color: "cyan", children: "\u258F" }) : null,
-          /* @__PURE__ */ jsx(
-            Text2,
-            {
-              dimColor: true,
-              children: `   ${matchCount} match${matchCount !== 1 ? "es" : ""}${searchInput ? "  \u21B5 accept \xB7 esc clear" : "  esc clear"}`
-            }
-          )
+          /* @__PURE__ */ jsx(Text2, { dimColor: true, children: `   ${matchCount} match${matchCount !== 1 ? "es" : ""}${searchInput ? "  \u21B5 accept \xB7 esc clear" : "  esc clear"}` })
         ] }) : repoFilter.size > 0 ? /* @__PURE__ */ jsxs(Box, { marginBottom: 1, children: [
           /* @__PURE__ */ jsx(Text2, { color: "#FF8700", children: "  \u25C9 " }),
           /* @__PURE__ */ jsx(Text2, { children: `${repoFilter.size} repo${repoFilter.size !== 1 ? "s" : ""}` }),
@@ -2271,16 +2402,24 @@ var BrowseScreen = ({
                 active: viewStart + i === cursor,
                 login,
                 merged: (item.kind === "pr" || item.kind === "issue") && !!mergedUrls?.includes(item.url),
+                transient: transientOf(transients, item),
                 sparkFrame,
-                gap: viewStart + i > 0 && (item.kind === "repo-header" || item.kind === "subgroup-header" || // A header is always followed by a blank line before its
-                // first child — in Other PRs that's "free" because the
-                // child is itself a repo-header (gap above). A jira row
-                // has no such stand-in, so it needs this explicitly. Never
-                // applies between two tickets/PRs — only right after a
-                // header.
-                item.kind === "jira" && ["repo-header", "subgroup-header"].includes(
-                  section.items[viewStart + i - 1]?.kind
-                ))
+                gap: (
+                  // Window-relative, never `viewStart + i`. fitCount prices the
+                  // window's FIRST row at one line (isFirst), so gapping it when
+                  // scrolled draws a row the budget never bought — and the frame
+                  // is sized to fill the terminal exactly, so the overflow scrolls
+                  // the whole panel up a line instead of clipping.
+                  i > 0 && (item.kind === "repo-header" || item.kind === "subgroup-header" || // A header is always followed by a blank line before its
+                  // first child — in Other PRs that's "free" because the
+                  // child is itself a repo-header (gap above). A jira row
+                  // has no such stand-in, so it needs this explicitly. Never
+                  // applies between two tickets/PRs — only right after a
+                  // header.
+                  item.kind === "jira" && ["repo-header", "subgroup-header"].includes(
+                    section.items[viewStart + i - 1]?.kind
+                  ))
+                )
               },
               `${viewStart + i}:${item.kind === "jira" ? item.instanceKey ?? item.key : item.kind === "repo-header" ? `header:${item.repo}` : item.kind === "subgroup-header" ? `subgroup:${item.label}` : item.kind === "show-more" ? `show-more:${item.hidden[0]?.repo ?? i}` : item.kind === "show-less" ? `show-less:${item.toHide[0]?.repo ?? i}` : `${item.repo}/${item.number}`}`
             )) }),
@@ -2346,6 +2485,8 @@ var App = ({
   ciJob,
   ciFetcher,
   ciPollMs = 6e4,
+  watchPath,
+  watchDebounceMs = 400,
   extensions,
   tabHelp,
   emptyHint
@@ -2363,12 +2504,47 @@ var App = ({
     return sameCiStatusState(prev, next) ? prev : next;
   });
   const displayedKey = useRef("");
+  const displayedSections = useRef([]);
+  const [transients, setTransients] = useState(NO_TRANSIENTS);
+  const transitTimer = useRef(null);
+  useEffect(
+    () => () => {
+      if (transitTimer.current) clearTimeout(transitTimer.current);
+    },
+    []
+  );
   const showData = (sections, login) => {
+    const before = displayedSections.current;
+    displayedSections.current = sections;
     displayedKey.current = signatureOf(sections);
     setPending(null);
     setFetchedAt(Date.now());
-    setState({ phase: "browse", sections, login });
+    if (before.length === 0) {
+      setTransients(NO_TRANSIENTS);
+      setState({ phase: "browse", sections, login });
+      return;
+    }
+    const { transients: marks, union } = diffSections(before, sections);
+    if (marks.size === 0) {
+      setState({ phase: "browse", sections, login });
+      return;
+    }
+    setTransients(marks);
+    setState({ phase: "browse", sections: union, login });
+    if (transitTimer.current) clearTimeout(transitTimer.current);
+    transitTimer.current = setTimeout(() => {
+      setTransients(NO_TRANSIENTS);
+      setState(
+        (prev) => prev.phase === "browse" && prev.sections === union ? { ...prev, sections } : prev
+      );
+    }, TRANSIT_HOLD_MS);
   };
+  const pendingSummary = useMemo(
+    () => pending ? summariseDiff(
+      diffSections(displayedSections.current, pending.sections).counts
+    ) : "",
+    [pending]
+  );
   const revalidate = (manual = false) => {
     if (manual) setRefreshing(true);
     fetcher().then((fresh) => {
@@ -2412,6 +2588,7 @@ var App = ({
     const painted = !!cached && cached.sections.length > 0;
     if (painted && cached) {
       displayedKey.current = signatureOf(cached.sections);
+      displayedSections.current = cached.sections;
       setFetchedAt(cached.at);
       setState({
         phase: "browse",
@@ -2438,6 +2615,29 @@ var App = ({
       clearInterval(id);
     };
   }, [hasCiStatus, ciFetcher, ciPollMs]);
+  useEffect(() => {
+    if (!watchPath) return;
+    const dir = dirname(watchPath);
+    const name = basename(watchPath);
+    if (!existsSync(dir)) return;
+    let timer = null;
+    let live = true;
+    let watcher;
+    try {
+      watcher = watch(dir, (_event, changed) => {
+        if (!live || changed && changed !== name) return;
+        if (timer) clearTimeout(timer);
+        timer = setTimeout(() => live && revalidate(), watchDebounceMs);
+      });
+    } catch {
+      return;
+    }
+    return () => {
+      live = false;
+      if (timer) clearTimeout(timer);
+      watcher.close();
+    };
+  }, [watchPath, watchDebounceMs]);
   const [mergedUrls, setMergedUrls] = useState([]);
   const mergeTimers = useRef([]);
   useEffect(() => () => mergeTimers.current.forEach(clearTimeout), []);
@@ -2514,6 +2714,7 @@ var App = ({
         onActed,
         refreshing,
         hasPending: pending !== null,
+        pendingSummary,
         fetchedAt,
         refreshError: refreshError ?? void 0,
         workToggle,
@@ -2521,6 +2722,7 @@ var App = ({
         initialIncludeWork,
         hidden: overlay !== null,
         mergedUrls,
+        transients,
         ciStatusState: hasCiStatus ? ciStatusState : void 0,
         ciJob,
         tabHelp,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kud/gh-ink",
-  "version": "0.11.0",
+  "version": "0.12.0",
   "description": "Ink components for rendering GitHub PR review comments and health — controlled, presentation-only, built on @kud/ink-ui and fed by @kud/gh.",
   "type": "module",
   "main": "dist/index.js",