@kud/gh-ink 0.32.2 → 0.33.0

package/dist/index.d.ts

@@ -295,6 +295,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

@@ -1057,7 +1057,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;
@@ -1078,7 +1078,22 @@ var budgetNotice = (budget, now = Date.now()) => {
   };
 };
 var isChildRow = (item) => !!item && (item.kind === "show-more" || item.kind === "show-less" || "indent" in item && item.indent === true);
+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;
   const urls = [];
@@ -1761,10 +1776,20 @@ var CiStatusLine = ({
     status.age ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: "  " + status.age }) : null
   ] });
 };
-var RepoHeaderRow = ({ repo, gap }) => {
+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 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;
@@ -1802,7 +1827,7 @@ var ItemRow = ({
 }) => {
   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 " }),
@@ -2016,13 +2041,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 +2487,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 +2499,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) => ({
@@ -2651,7 +2714,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"],

package/package.json

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