@kud/gh-ink 0.29.1 → 0.31.0

package/dist/index.d.ts

@@ -94,6 +94,7 @@ type GHDetail = {
     checksPass: number;
     checksFail: number;
     checksPending: number;
+    checksStale?: number;
     threadsTotal: number;
     lastCommitAt?: string;
     lastEventAt?: string;

package/dist/index.js

@@ -1,7 +1,7 @@
 import React2, { createContext, useState, useRef, useEffect, useMemo, useContext } from 'react';
 import { useWindowSize, useInput, Text, Box } from 'ink';
 import { colors, ScrollView, TextInput, LoadingScreen, StatusMessage, FooterHints, useListCursor, Tabs } from '@kud/ink-ui';
-import { isPassCheck, isFailCheck, resolveThread, unresolveThread, replyToThread, rerunFailedRun, mergePr, reRequestReviewer } from '@kud/gh';
+import { isPassCheck, isFailCheck, isInconclusiveCheck, 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';
@@ -417,7 +417,7 @@ var CommentsPanel = ({
     ) : null
   ] });
 };
-var checkIcon = (c) => isPassCheck(c) ? ["\u2713", colors.success] : isFailCheck(c) ? ["\u2717", colors.error] : ["\xB7", colors.warning];
+var checkIcon = (c) => isPassCheck(c) ? ["\u2713", colors.success] : isFailCheck(c) ? ["\u2717", colors.error] : isInconclusiveCheck(c) ? ["-", colors.warning] : ["\xB7", colors.warning];
 var checkLabel = (c) => {
   const name = c.context ?? c.name ?? "";
   return c.workflowName ? `${c.workflowName} / ${name}` : name;
@@ -491,9 +491,11 @@ var HealthPanel = ({
     }
   };
   const retrigger = () => {
-    const failing = checks.find((c) => isFailCheck(c) && runIdOf(checkUrl(c)));
-    const runId = failing && runIdOf(checkUrl(failing));
-    if (!runId) return flash("No failed Actions run to retrigger");
+    const stuck = checks.find(
+      (c) => (isFailCheck(c) || isInconclusiveCheck(c)) && runIdOf(checkUrl(c))
+    );
+    const runId = stuck && runIdOf(checkUrl(stuck));
+    if (!runId) return flash("No Actions run to retrigger");
     void act("retrigger CI", () => rerunFailedRun(repo, runId));
   };
   useInput((input, key) => {
@@ -528,7 +530,8 @@ var HealthPanel = ({
   const reviewerMap = Object.fromEntries(reviewers);
   const passed = checks.filter(isPassCheck).length;
   const failed = checks.filter(isFailCheck).length;
-  const pending = checks.length - passed - failed;
+  const stale = checks.filter(isInconclusiveCheck).length;
+  const pending = checks.length - passed - failed - stale;
   const approved = Object.values(reviewerMap).filter(
     (s) => s === "APPROVED"
   ).length;
@@ -537,6 +540,7 @@ var HealthPanel = ({
   ).length;
   const checkParts = [[`${passed} passed`, colors.success]];
   if (failed > 0) checkParts.push([`${failed} failed`, colors.error]);
+  if (stale > 0) checkParts.push([`${stale} cancelled`, colors.warning]);
   if (pending > 0) checkParts.push([`${pending} pending`, colors.warning]);
   const reviewParts = [
     approved > 0 ? [`${approved} approved`, colors.success] : ["0 approved", "white"]
@@ -823,11 +827,13 @@ var healthSentence = (item) => {
 };
 var checksSentence = (d) => {
   if (!d) return null;
-  const total = d.checksPass + d.checksFail + d.checksPending;
+  const stale = d.checksStale ?? 0;
+  const total = d.checksPass + d.checksFail + d.checksPending + stale;
   if (total === 0) return "No CI checks run on it.";
   const parts = [];
   if (d.checksPass) parts.push(`${d.checksPass} passing`);
   if (d.checksFail) parts.push(`${d.checksFail} failing`);
+  if (stale) parts.push(`${stale} cancelled`);
   if (d.checksPending) parts.push(`${d.checksPending} running`);
   return `Checks: ${parts.join(", ")}.`;
 };
@@ -920,7 +926,14 @@ var STANDING = {
   reviewed: "spoken"
 };
 var YOURS = {
-  authored: ["ci-fail", "conflict", "changes-req", "threads", "approved", "draft"],
+  authored: [
+    "ci-fail",
+    "conflict",
+    "changes-req",
+    "threads",
+    "approved",
+    "draft"
+  ],
   queued: ["waiting", "pending", "threads", "approved"],
   spoken: ["threads"]
 };
@@ -1052,7 +1065,10 @@ var budgetNotice = (budget, now = Date.now()) => {
     1,
     Math.round((new Date(budget.resetAt).getTime() - now) / 6e4)
   );
-  return fetches <= 0 ? { label: `\u26A1 API budget spent \xB7 ${mins}m`, critical: true } : { label: `\u26A1 ${fetches} fetch${fetches === 1 ? "" : "es"} left`, critical: false };
+  return fetches <= 0 ? { label: `\u26A1 API budget spent \xB7 ${mins}m`, critical: true } : {
+    label: `\u26A1 ${fetches} fetch${fetches === 1 ? "" : "es"} left`,
+    critical: false
+  };
 };
 var isChildRow = (item) => !!item && (item.kind === "show-more" || item.kind === "show-less" || "indent" in item && item.indent === true);
 var treeUrls = (items, i) => {
@@ -1074,7 +1090,8 @@ var treeUrls = (items, i) => {
 var gapsAbove = (items, i) => {
   const item = items[i];
   if (!item || i === 0) return false;
-  if (item.kind === "repo-header" || item.kind === "subgroup-header") return true;
+  if (item.kind === "repo-header" || item.kind === "subgroup-header")
+    return true;
   if (item.kind !== "task") return false;
   return isChildRow(items[i + 1]) || isChildRow(items[i - 1]);
 };
@@ -1342,9 +1359,15 @@ var UNSUBSCRIBE_MUTATION = `
   }
 `;
 var GH_ACTION_FAILURES = [
-  [/rate limit|RATE_LIMIT/i, "GitHub rate limit spent \u2014 it resets within the hour"],
+  [
+    /rate limit|RATE_LIMIT/i,
+    "GitHub rate limit spent \u2014 it resets within the hour"
+  ],
   [/\b401\b|not logged in|authentication/i, "gh is not authenticated"],
-  [/\b403\b|must have admin|not authorized|permission/i, "not permitted on this repo"],
+  [
+    /\b403\b|must have admin|not authorized|permission/i,
+    "not permitted on this repo"
+  ],
   [/\b50[0234]\b/, "GitHub's API is failing (5xx)"],
   [/ENOTFOUND|EAI_AGAIN|ETIMEDOUT|ECONNREFUSED/i, "no route to GitHub"]
 ];
@@ -1468,9 +1491,7 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
     hint: "u",
     run: () => {
       showFlash(`\u22EF Unsubscribing from #${item.number}\u2026`);
-      void unsubscribeFrom(item).then(() => showFlash(`\u2713 Unsubscribed from #${item.number}`)).catch(
-        (e) => showFlash(`\u2717 #${item.number}: ${explainGhAction(e)}`)
-      );
+      void unsubscribeFrom(item).then(() => showFlash(`\u2713 Unsubscribed from #${item.number}`)).catch((e) => showFlash(`\u2717 #${item.number}: ${explainGhAction(e)}`));
     }
   });
   if (item.kind === "pr" && item.branch) {
@@ -1744,6 +1765,7 @@ var MERGED_FRAMES = ["\u2726", "\u2727", "\u2736", "\u2727"];
 var MERGED_COLOUR = "#A371F7";
 var TRANSIT_HOLD_MS = 7e3;
 var NO_TRANSIENTS = /* @__PURE__ */ new Map();
+var NO_HOLDS = /* @__PURE__ */ new Map();
 var TRANSIT_OUT_FRAMES = ["\u25C9", "\u25CE", "\u25CB", "\xB7"];
 var TRANSIT_IN_FRAMES = ["\xB7", "\u25CB", "\u25CE", "\u25C9"];
 var TRANSIT_COLOUR = {
@@ -1756,6 +1778,7 @@ var TRANSIT_LABEL = {
   out: "GONE",
   changed: "UPDATED"
 };
+var LEAVING_HOLD_MS = 2500;
 var TAB_MARK = "\u25CF";
 var tabLabel = (label, marked, id) => marked.size === 0 ? label : `${marked.has(id) ? TAB_MARK : " "} ${label}`;
 var ItemRow = ({
@@ -1764,11 +1787,13 @@ var ItemRow = ({
   gap,
   login,
   merged,
-  transient,
+  transient: refreshMark,
+  leaving,
   lastChild,
   parent,
   sparkFrame = 0
 }) => {
+  const transient = leaving ? "out" : refreshMark;
   if (item.kind === "repo-header")
     return /* @__PURE__ */ jsx(RepoHeaderRow, { repo: item.repo, gap: gap ?? false });
   if (item.kind === "subgroup-header")
@@ -2190,6 +2215,8 @@ var BrowseScreen = ({
   skippedForBudget,
   brand,
   mergedUrls,
+  leavingUrls,
+  onLeave,
   transients,
   onTabChange
 }) => {
@@ -2259,7 +2286,7 @@ var BrowseScreen = ({
       )
     );
   }, [localSections, listHeight]);
-  const removeItemFromSections = (target) => setLocalSections((prev) => withoutItem(prev, target));
+  const dismissItem = (target) => onLeave ? onLeave(target) : setLocalSections((prev) => withoutItem(prev, target));
   const rawSection = localSections[safeTabIdx] ?? {
     id: "empty",
     label: "",
@@ -2291,7 +2318,7 @@ var BrowseScreen = ({
     return out;
   }, [localSections, transients]);
   const [sparkFrame, setSparkFrame] = useState(0);
-  const sparkling = (mergedUrls?.length ?? 0) > 0 || !!transients?.size && section.items.some((item) => transientOf(transients, item));
+  const sparkling = (mergedUrls?.length ?? 0) > 0 || (leavingUrls?.length ?? 0) > 0 || !!transients?.size && section.items.some((item) => transientOf(transients, item));
   useEffect(() => {
     if (!sparkling) return;
     const id = setInterval(() => setSparkFrame((f) => f + 1), MERGED_FRAME_MS);
@@ -2324,7 +2351,7 @@ var BrowseScreen = ({
       jiraKeyRe,
       jiraTransitions,
       onRefresh,
-      removeItemFromSections,
+      dismissItem,
       openDrillView,
       { extensions, onOpenExt, onActed }
     );
@@ -2431,7 +2458,7 @@ var BrowseScreen = ({
         item: activeItem ?? void 0,
         ciJob: ciStatus?.job,
         login,
-        onRemove: (row) => removeItemFromSections(row),
+        onRemove: (row) => dismissItem(row),
         showFlash
       });
       return;
@@ -2540,7 +2567,7 @@ var BrowseScreen = ({
       return;
     }
     if (input === "x" && activeItem.kind === "pr" && activeItem.standing === "queued" && login) {
-      removeItemFromSections(activeItem);
+      dismissItem(activeItem);
       showFlash(`\u22EF Removing you from #${activeItem.number}\u2026`);
       void quietly`gh pr edit ${activeItem.number} --repo ${activeItem.repo} --remove-reviewer ${login}`.then(() => {
         showFlash(`\u2713 Removed you as reviewer on #${activeItem.number}`);
@@ -2707,6 +2734,7 @@ var BrowseScreen = ({
                 active: viewStart + i === cursor,
                 login,
                 merged: (item.kind === "pr" || item.kind === "issue") && !!mergedUrls?.includes(item.url),
+                leaving: (item.kind === "pr" || item.kind === "issue") && !!leavingUrls?.includes(item.url),
                 transient: transientOf(transients, item),
                 lastChild: "indent" in item && item.indent ? !isChildRow(section.items[viewStart + i + 1]) : void 0,
                 parent: item.kind === "task" && isChildRow(section.items[viewStart + i + 1]),
@@ -2807,6 +2835,7 @@ var App = ({
   const displayedSections = useRef([]);
   const [transients, setTransients] = useState(NO_TRANSIENTS);
   const transitTabs = useRef(/* @__PURE__ */ new Map());
+  const [heldSince, setHeldSince] = useState(NO_HOLDS);
   const [visibleTab, setVisibleTab] = useState("");
   const showData = (sections, login) => {
     const before = displayedSections.current;
@@ -2861,15 +2890,31 @@ var App = ({
     );
   };
   useEffect(() => {
-    if (transients.size === 0) return;
-    if (state.phase !== "browse" || !visibleTab) return;
-    if (![...transitTabs.current.values()].includes(visibleTab)) return;
+    const held = new Set(transitTabs.current.values());
+    const arrived = state.phase === "browse" && !!visibleTab && held.has(visibleTab);
+    setHeldSince((prev) => {
+      const next = new Map([...prev].filter(([tab]) => held.has(tab)));
+      if (arrived && !next.has(visibleTab)) next.set(visibleTab, Date.now());
+      const same = next.size === prev.size && [...next.keys()].every((t) => prev.has(t));
+      return same ? prev : next.size === 0 ? NO_HOLDS : next;
+    });
+  }, [transients, visibleTab, state.phase]);
+  useEffect(() => {
+    if (transients.size === 0 || heldSince.size === 0) return;
+    if (state.phase !== "browse") return;
+    const held = new Set(transitTabs.current.values());
+    const running = [...heldSince].filter(([tab]) => held.has(tab));
+    if (running.length === 0) return;
+    const remaining = (at) => at + TRANSIT_HOLD_MS - Date.now();
     const timer = setTimeout(
-      () => settleTab(visibleTab, transients),
-      TRANSIT_HOLD_MS
+      () => {
+        for (const [tab, at] of running)
+          if (remaining(at) <= 0) settleTab(tab, transients);
+      },
+      Math.max(0, Math.min(...running.map(([, at]) => remaining(at))))
     );
     return () => clearTimeout(timer);
-  }, [transients, visibleTab, state.phase]);
+  }, [transients, heldSince, state.phase]);
   const pendingSummary = useMemo(
     () => pending ? summariseDiff(
       diffSections(displayedSections.current, pending.sections).counts
@@ -2917,10 +2962,7 @@ var App = ({
       }
       showData(fresh.sections, fresh.login);
     } else if (freshKey !== displayedKey.current) {
-      const { counts } = diffSections(
-        displayedSections.current,
-        fresh.sections
-      );
+      const { counts } = diffSections(displayedSections.current, fresh.sections);
       if (counts.added + counts.removed + counts.changed === 0)
         showData(fresh.sections, fresh.login);
       else setPending(fresh);
@@ -3007,8 +3049,9 @@ var App = ({
   );
   const [skippedForBudget, setSkippedForBudget] = useState(false);
   const [mergedUrls, setMergedUrls] = useState([]);
-  const mergeTimers = useRef([]);
-  useEffect(() => () => mergeTimers.current.forEach(clearTimeout), []);
+  const [leavingUrls, setLeavingUrls] = useState([]);
+  const holdTimers = useRef([]);
+  useEffect(() => () => holdTimers.current.forEach(clearTimeout), []);
   if (state.phase === "loading" || state.phase === "empty" || state.phase === "failed")
     return /* @__PURE__ */ jsxs(
       Box,
@@ -3056,17 +3099,26 @@ var App = ({
     target: state.target
   } : null;
   const toBrowse = () => setState({ phase: "browse", sections: state.sections, login: state.login });
-  const removeAndReturn = (target) => setState({
-    phase: "browse",
-    sections: withoutItem(state.sections, target),
-    login: state.login
-  });
+  const markLeaving = (target) => {
+    if (state.phase !== "browse") toBrowse();
+    setLeavingUrls(
+      (prev) => prev.includes(target.url) ? prev : [...prev, target.url]
+    );
+    holdTimers.current.push(
+      setTimeout(() => {
+        setLeavingUrls((prev) => prev.filter((u) => u !== target.url));
+        setState(
+          (s) => s.phase === "browse" ? { ...s, sections: withoutItem(s.sections, target) } : s
+        );
+      }, LEAVING_HOLD_MS)
+    );
+  };
   const markMerged = (target) => {
     toBrowse();
     setMergedUrls(
       (prev) => prev.includes(target.url) ? prev : [...prev, target.url]
     );
-    mergeTimers.current.push(
+    holdTimers.current.push(
       setTimeout(() => {
         setMergedUrls((prev) => prev.filter((u) => u !== target.url));
         setState(
@@ -3097,6 +3149,8 @@ var App = ({
         skippedForBudget,
         hidden: overlay !== null,
         mergedUrls,
+        leavingUrls,
+        onLeave: markLeaving,
         transients,
         onTabChange: setVisibleTab,
         ciStatusState: hasCiStatus ? ciStatusState : void 0,
@@ -3130,7 +3184,7 @@ var App = ({
       login: state.login,
       onBack: toBrowse,
       onRefresh: applyOrRefresh,
-      onRemove: removeAndReturn,
+      onRemove: markLeaving,
       onMerged: markMerged
     }),
     overlay?.kind === "ext" && extensions?.find((e) => e.id === overlay.extId)?.body(toBrowse, overlay.target)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kud/gh-ink",
-  "version": "0.29.1",
+  "version": "0.31.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",
@@ -48,7 +48,7 @@
     "react": ">=19"
   },
   "dependencies": {
-    "@kud/gh": "0.7.1",
+    "@kud/gh": "0.8.0",
     "@kud/ink-ui": "0.14.0",
     "zx": "8.8.5"
   },