@kud/gh-ink 0.32.2 → 0.34.0

package/dist/index.d.ts

@@ -140,7 +140,23 @@ type GHItem = {
      * handing back a `lastActor` that no longer claims a turn.
      */
     pinned?: boolean;
-    indent: boolean;
+    /**
+     * How deep this row hangs in the tree: 0 top level, 1 a child, 2 a
+     * grandchild. Absent means 0.
+     *
+     * Always read it through `depthOf`, never off the row — `indent` is still a
+     * legal way to spell "depth 1", and that helper is the only place that knows
+     * both spellings. A site that reads `item.indent` directly prices a `depth: 2`
+     * row as top level, which is the bug this field exists to end rather than a
+     * new one to introduce.
+     */
+    depth?: number;
+    /**
+     * @deprecated Legacy spelling of `depth: 1`. Write `depth`; this is kept so
+     * every existing producer still typechecks, and it is read in exactly one
+     * place (`depthOf`). Due to be removed at the next major.
+     */
+    indent?: boolean;
 };
 type TaskRow = {
     kind: "task";
@@ -149,7 +165,10 @@ type TaskRow = {
     url: string;
     status: string;
     age: string;
-    indent: boolean;
+    /** How deep this row hangs. See `GHItem.depth`; read it through `depthOf`. */
+    depth?: number;
+    /** @deprecated Legacy spelling of `depth: 1`. See `GHItem.indent`. */
+    indent?: boolean;
     instanceKey?: string;
     /**
      * Trailing annotation, rendered dim after the summary — a recurrence marker,
@@ -176,23 +195,35 @@ type RepoHeader = {
     kind: "repo-header";
     repo: string;
     age: string;
-    indent: boolean;
+    /** How deep this row hangs. See `GHItem.depth`; read it through `depthOf`. */
+    depth?: number;
+    /** @deprecated Legacy spelling of `depth: 1`. See `GHItem.indent`. */
+    indent?: boolean;
 };
 type ShowMore = {
     kind: "show-more";
     hidden: GHItem[];
-    indent: boolean;
+    /** How deep this row hangs. See `GHItem.depth`; read it through `depthOf`. */
+    depth?: number;
+    /** @deprecated Legacy spelling of `depth: 1`. See `GHItem.indent`. */
+    indent?: boolean;
 };
 type ShowLess = {
     kind: "show-less";
     toHide: GHItem[];
-    indent: boolean;
+    /** How deep this row hangs. See `GHItem.depth`; read it through `depthOf`. */
+    depth?: number;
+    /** @deprecated Legacy spelling of `depth: 1`. See `GHItem.indent`. */
+    indent?: boolean;
 };
 type SubgroupHeader = {
     kind: "subgroup-header";
     label: string;
     age: string;
-    indent: boolean;
+    /** How deep this row hangs. See `GHItem.depth`; read it through `depthOf`. */
+    depth?: number;
+    /** @deprecated Legacy spelling of `depth: 1`. See `GHItem.indent`. */
+    indent?: boolean;
 };
 type AnyItem = GHItem | TaskRow | RepoHeader | SubgroupHeader | ShowMore | ShowLess;
 type CiStatus = {
@@ -295,6 +326,22 @@ declare const filterBySearch: (sections: Section[], query: string) => Section[];
 declare const filterByRepos: (sections: Section[], repos: Set<string>) => Section[];
 declare const withoutItem: (sections: Section[], target: GHItem) => Section[];
 declare const reposInSections: (sections: Section[]) => string[];
+/**
+ * The next selectable row in `dir`, or `current` when there is none.
+ *
+ * A repo-header is a STOP, and a subgroup-header is not. The two look alike and
+ * are not: the fence names a thing you can act on — open the checkout, copy every
+ * URL under it — where a band label ("Your move") names an arrangement of rows
+ * and has nothing behind it to open. Landing on the second would be a keystroke
+ * spent on a row whose every key is inert.
+ *
+ * Unconditional in both directions, deliberately. Skipping the fence going down
+ * and landing on it going up would make ↓ then ↑ end somewhere other than where
+ * it started, and an arrow pair that is not its own inverse reads as the list
+ * drifting rather than as a shortcut. The cost is one press per repo group; the
+ * fence is also the "you have crossed into another repo" beat the eye takes
+ * anyway, so the press buys orientation as well as position.
+ */
 declare const moveCursor: (items: AnyItem[], current: number, dir: 1 | -1) => number;
 /**
  * What to say about the remaining budget, or nothing at all.

package/dist/index.js

@@ -780,6 +780,11 @@ var tabsOfMarks = (sections, marks) => {
   return out;
 };
 var quietly = $({ quiet: true });
+var depthOf = (item) => {
+  if (!item) return 0;
+  if (typeof item.depth === "number") return Math.max(0, Math.floor(item.depth));
+  return item.indent === true ? 1 : 0;
+};
 var relativeTime = (iso) => {
   const diff = (Date.now() - new Date(iso).getTime()) / 1e3;
   if (diff < 3600) return `${Math.floor(diff / 60)}m`;
@@ -902,7 +907,7 @@ var insertRepoHeaders = (items) => {
   const result = [];
   let lastRepo = "";
   for (const item of items) {
-    if (!item.indent && item.repo !== lastRepo) {
+    if (depthOf(item) === 0 && item.repo !== lastRepo) {
       lastRepo = item.repo;
       result.push({
         kind: "repo-header",
@@ -1057,7 +1062,7 @@ var reposInSections = (sections) => [
 ].sort();
 var moveCursor = (items, current2, dir) => {
   let next = current2 + dir;
-  while (next >= 0 && next < items.length && (items[next].kind === "repo-header" || items[next].kind === "subgroup-header"))
+  while (next >= 0 && next < items.length && items[next].kind === "subgroup-header")
     next += dir;
   if (next < 0 || next >= items.length) return current2;
   return next;
@@ -1077,14 +1082,54 @@ var budgetNotice = (budget, now = Date.now()) => {
     critical: false
   };
 };
-var isChildRow = (item) => !!item && (item.kind === "show-more" || item.kind === "show-less" || "indent" in item && item.indent === true);
+var isLastAtDepth = (items, i) => {
+  const depth = depthOf(items[i]);
+  for (let j = i + 1; j < items.length; j += 1) {
+    const d = depthOf(items[j]);
+    if (d > depth) continue;
+    return d < depth;
+  }
+  return true;
+};
+var treePrefix = (items, i) => {
+  const depth = depthOf(items[i]);
+  if (depth === 0) return "";
+  const ancestorLast = [];
+  let level = depth - 1;
+  for (let j = i - 1; j >= 0 && level >= 1; j -= 1) {
+    if (depthOf(items[j]) === level) {
+      ancestorLast[level] = isLastAtDepth(items, j);
+      level -= 1;
+    }
+  }
+  let prefix = "";
+  for (let k = 1; k < depth; k += 1) prefix += ancestorLast[k] ? "   " : "\u2502  ";
+  return prefix + (isLastAtDepth(items, i) ? "\u2514\u2500 " : "\u251C\u2500 ");
+};
+var groupUrls = (items, i) => {
+  const urls = [];
+  for (let j = i + 1; j < items.length; j += 1) {
+    const item = items[j];
+    if (item.kind === "repo-header" || item.kind === "subgroup-header") break;
+    if (item.kind === "show-more") {
+      for (const child of item.hidden) if (child.url) urls.push(child.url);
+      continue;
+    }
+    const url = item.url;
+    if (url) urls.push(url);
+  }
+  return urls;
+};
 var treeUrls = (items, i) => {
+  if (items[i]?.kind === "repo-header") return groupUrls(items, i);
   let root = i;
-  while (root > 0 && isChildRow(items[root])) root -= 1;
+  while (root > 0 && depthOf(items[root]) > 0 && items[root]?.kind !== "task")
+    root -= 1;
+  const rootDepth = depthOf(items[root]);
   const urls = [];
   for (let j = root; j < items.length; j += 1) {
     const item = items[j];
-    if (j > root && !isChildRow(item)) break;
+    if (j > root && depthOf(item) <= rootDepth) break;
     if (item.kind === "show-more") {
       for (const child of item.hidden) if (child.url) urls.push(child.url);
       continue;
@@ -1100,7 +1145,8 @@ var gapsAbove = (items, i) => {
   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]);
+  if (depthOf(item) > 0) return false;
+  return depthOf(items[i + 1]) > 0 || depthOf(items[i - 1]) > 0;
 };
 var fitCount = (items, start, budget) => {
   let lines = 0;
@@ -1345,7 +1391,7 @@ var Backdrop = ({
   }
 ) : children });
 var topLevelCount = (s) => s.items.filter(
-  (i) => i.kind !== "repo-header" && i.kind !== "subgroup-header" && !i.indent
+  (i) => i.kind !== "repo-header" && i.kind !== "subgroup-header" && depthOf(i) === 0
 ).length;
 var drillCmd = (item) => {
   if (item.kind === "task")
@@ -1593,7 +1639,7 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
       }
     });
     if (jiraBase && jiraKeyRe) {
-      const jiraKey = !item.indent ? item.title.match(jiraKeyRe)?.[0] : null;
+      const jiraKey = depthOf(item) === 0 ? item.title.match(jiraKeyRe)?.[0] : null;
       if (jiraKey) {
         actions.push({
           label: `Open ${jiraKey} in Jira`,
@@ -1761,10 +1807,21 @@ var CiStatusLine = ({
     status.age ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "  " + status.age }) : null
   ] });
 };
-var RepoHeaderRow = ({ repo, gap }) => {
+var FENCE_COLS = 46;
+var RepoHeaderRow = ({
+  repo,
+  gap,
+  active
+}) => {
   const label = `\u2500\u2500 ${repo} `;
-  const fill = Math.max(4, 46 - label.length);
-  return /* @__PURE__ */ jsx(Box, { marginTop: gap ? 1 : 0, children: /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "  " + label + "\u2500".repeat(fill) }) });
+  const fill = Math.max(4, FENCE_COLS - label.length);
+  const rule = "\u2500".repeat(fill);
+  return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
+    /* @__PURE__ */ jsx(Text2, { color: "cyan", children: active ? "\u276F " : "  " }),
+    /* @__PURE__ */ jsx(Text2, { dimColor: !active, children: "\u2500\u2500 " }),
+    /* @__PURE__ */ jsx(Text2, { dimColor: !active, bold: active, children: repo }),
+    /* @__PURE__ */ jsx(Text2, { dimColor: !active, children: " " + rule })
+  ] });
 };
 var MERGED_HOLD_MS = 5e3;
 var MERGED_FRAME_MS = 150;
@@ -1796,13 +1853,13 @@ var ItemRow = ({
   merged,
   transient: refreshMark,
   leaving,
-  lastChild,
+  prefix = "",
   parent,
   sparkFrame = 0
 }) => {
   const transient = leaving ? "out" : refreshMark;
   if (item.kind === "repo-header")
-    return /* @__PURE__ */ jsx(RepoHeaderRow, { repo: item.repo, gap: gap ?? false });
+    return /* @__PURE__ */ jsx(RepoHeaderRow, { repo: item.repo, gap: gap ?? false, active });
   if (item.kind === "subgroup-header")
     return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
       /* @__PURE__ */ jsx(Text2, { color: "#FF8700", bold: true, children: "  \xBB " }),
@@ -1811,14 +1868,14 @@ var ItemRow = ({
   if (item.kind === "show-more")
     return /* @__PURE__ */ jsxs(Box, { children: [
       /* @__PURE__ */ jsx(Text2, { color: "cyan", children: active ? "\u276F " : "  " }),
-      /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "\u2514\u2500 " }),
+      /* @__PURE__ */ jsx(Text2, { dimColor: true, children: prefix }),
       /* @__PURE__ */ jsx(Text2, { dimColor: true, children: active ? "\u21B5 " : "  " }),
       /* @__PURE__ */ jsx(Text2, { dimColor: true, children: `+${item.hidden.length} more` })
     ] });
   if (item.kind === "show-less")
     return /* @__PURE__ */ jsxs(Box, { children: [
       /* @__PURE__ */ jsx(Text2, { color: "cyan", children: active ? "\u276F " : "  " }),
-      /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "\u2514\u2500 " }),
+      /* @__PURE__ */ jsx(Text2, { dimColor: true, children: prefix }),
       /* @__PURE__ */ jsx(Text2, { dimColor: true, children: active ? "\u21B5 " : "  " }),
       /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "show less" })
     ] });
@@ -1828,10 +1885,11 @@ var ItemRow = ({
     const transitLabel2 = transient ? TRANSIT_LABEL[transient] : "";
     const titleMax2 = Math.max(
       20,
-      COLS - item.key.length - note.length - transitLabel2.length - 12
+      COLS - item.key.length - note.length - transitLabel2.length - prefix.length - 12
     );
     return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
       /* @__PURE__ */ jsx(Text2, { color: "cyan", children: active ? "\u276F " : "  " }),
+      /* @__PURE__ */ jsx(Text2, { dimColor: true, children: prefix }),
       /* @__PURE__ */ jsx(Text2, { dimColor: true, children: parent ? "\u252C " : "  " }),
       /* @__PURE__ */ jsx(Text2, { bold: true, color: transient ? TRANSIT_COLOUR[transient] : void 0, children: transitIcon + " " }),
       /* @__PURE__ */ jsx(Text2, { color: "#FF8700", bold: active, children: item.key + "  " }),
@@ -1866,12 +1924,12 @@ var ItemRow = ({
     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;
+  const repoLabel = depthOf(item) > 0 ? item.repo : "";
+  const fixedWidth = 2 + prefix.length + 2 + 2 + 7 + repoLabel.length + suffix.length + 6;
   const titleMax = Math.max(20, COLS - fixedWidth);
   return /* @__PURE__ */ jsxs(Box, { children: [
     /* @__PURE__ */ jsx(Text2, { color: "cyan", children: active ? "\u276F " : "  " }),
-    item.indent ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: lastChild === false ? "\u251C\u2500 " : "\u2514\u2500 " }) : null,
+    /* @__PURE__ */ jsx(Text2, { dimColor: true, children: prefix }),
     /* @__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 }),
@@ -2016,13 +2074,13 @@ var HelpModal = ({
   const keys = [
     ["\u2191 \u2193", "navigate"],
     ["\u2190 \u2192 \xB7 tab", "switch tab"],
-    ["\u21B5 \xB7 d", "open / drill in"],
+    ["\u21B5 \xB7 d", "open / drill in \xB7 repo: checkout"],
     ["m", "actions \xB7 close"],
     ["e", "explain this row"],
     ["o", "open in browser"],
-    ["O", "open ticket + its PRs"],
+    ["O", "open tree \xB7 whole repo group"],
     ["c", "copy URL"],
-    ["C", "copy ticket + its PRs"],
+    ["C", "copy tree \xB7 whole repo group"],
     ["b", "copy branch"],
     ["s", "switch to branch here"],
     ["j", "open repo in new tab"],
@@ -2462,7 +2520,11 @@ var BrowseScreen = ({
     const ext = extensionFor(input, extensions);
     if (ext) {
       onOpenExt?.(ext.id, {
-        item: activeItem ?? void 0,
+        // ExtensionTarget.item has always been documented as absent on a header
+        // row, and until the fence became selectable that was true for free.
+        // Now it has to be said: every extension body narrows on `kind` for a
+        // pr / issue / task and none of them expect a repo-header.
+        item: activeItem && !isHeader(activeItem) ? activeItem : void 0,
         ciJob: ciStatus?.job,
         login,
         onRemove: (row) => dismissItem(row),
@@ -2470,8 +2532,42 @@ var BrowseScreen = ({
       });
       return;
     }
-    if (!activeItem || activeItem.kind === "repo-header" || activeItem.kind === "subgroup-header")
+    if (activeItem?.kind === "repo-header") {
+      const { repo } = activeItem;
+      const repoUrl = `https://github.com/${repo}`;
+      if (key.return || input === "d" || input === "j") {
+        showFlash(`\u22EF Opening ${repo}\u2026`);
+        void jumpToRepo(repo, "", login).then(() => showFlash(`\u2197 Opened ${repo} in new tab`)).catch(() => showFlash("\u2717 Jump failed"));
+        return;
+      }
+      if (input === "o") {
+        quietly`open ${repoUrl}`.catch(() => {
+        });
+        showFlash(`\u2197 Opened ${repo}`);
+        return;
+      }
+      if (input === "c") {
+        clipboard(repoUrl);
+        showFlash(`\u2713 Copied URL for ${repo}`);
+        return;
+      }
+      if (input === "C" || input === "O") {
+        const urls = treeUrls(section.items, cursor);
+        if (urls.length === 0) return;
+        const n = `${urls.length} URL${urls.length === 1 ? "" : "s"}`;
+        if (input === "C") {
+          clipboard(urls.join("\n"));
+          showFlash(`\u2713 Copied ${n} from ${repo}`);
+        } else {
+          quietly`open ${urls}`.catch(() => {
+          });
+          showFlash(`\u2197 Opened ${n} from ${repo}`);
+        }
+        return;
+      }
       return;
+    }
+    if (!activeItem || activeItem.kind === "subgroup-header") return;
     if (key.return && activeItem.kind === "show-more") {
       setLocalSections(
         (prev) => prev.map((s) => ({
@@ -2482,7 +2578,12 @@ var BrowseScreen = ({
               {
                 kind: "show-less",
                 toHide: activeItem.hidden,
-                indent: true
+                // The depth of the row it replaces, never a constant. A
+                // collapsed row sitting under a story is at depth 2, and
+                // pinning it to 1 makes the walk in `treeUrls` stop one
+                // level early and drop every row it holds — silently, since
+                // nothing on screen reports what the clipboard missed.
+                depth: depthOf(activeItem)
               }
             ] : [i]
           )
@@ -2499,7 +2600,9 @@ var BrowseScreen = ({
               {
                 kind: "show-more",
                 hidden: activeItem.toHide,
-                indent: true
+                // Same reasoning as the show-less above: carry the depth
+                // of the row being replaced, not a constant.
+                depth: depthOf(activeItem)
               }
             ] : [i]
           )
@@ -2651,7 +2754,15 @@ var BrowseScreen = ({
       menu.open(actions);
     }
   });
-  const hints = [
+  const hints = activeItem?.kind === "repo-header" ? [
+    ["\u2191\u2193", "nav"],
+    ["\u2190\u2192", "tab"],
+    ["\u21B5", "open repo"],
+    ["o", "browser"],
+    ["C", "copy group"],
+    ["?", "help"],
+    ["q", "quit"]
+  ] : [
     ["\u2191\u2193", "nav"],
     ["\u2190\u2192", "tab"],
     ["\u21B5/d", "open"],
@@ -2743,8 +2854,8 @@ var BrowseScreen = ({
                 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]),
+                prefix: treePrefix(section.items, viewStart + i),
+                parent: item.kind === "task" && depthOf(section.items[viewStart + i + 1]) > depthOf(item),
                 sparkFrame,
                 gap: i > 0 && gapsAbove(section.items, viewStart + i)
               },
@@ -2952,7 +3063,7 @@ var App = ({
       receive(fresh);
     }).catch((err) => {
       setRefreshing(false);
-      const message = err.message;
+      const message = explainGhAction(err);
       if (!displayedKey.current) {
         setState({ phase: "failed", message });
         return;

package/package.json

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