@kud/gh-ink 0.29.1 → 0.30.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 = {
@@ -2807,6 +2829,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 +2884,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 +2956,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);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kud/gh-ink",
-  "version": "0.29.1",
+  "version": "0.30.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"
   },