@kud/gh-ink 0.64.0 → 0.65.0

package/dist/index.d.ts

@@ -543,18 +543,20 @@ 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.
+ * Both header kinds are STOPS. A repo-header names a thing you can act on —
+ * open the checkout, copy every URL under it — and a band header
+ * ("» Your move") does now too: `c` copies the whole band. Landing on one
+ * used to be a keystroke spent on a row whose every key was inert, which is
+ * why `moveCursor` stepped over subgroup-headers; now that the key does
+ * something there, skipping it would hide the row that says how many links
+ * the key will take.
  *
- * 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.
+ * Unconditional in both directions, deliberately. Skipping a header 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
+ * header; each is also the "you have crossed into another group" 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;
 /**

package/dist/index.js

@@ -1524,9 +1524,7 @@ var reposInSections = (sections) => [
   )
 ].sort();
 var moveCursor = (items, current, dir) => {
-  let next = current + dir;
-  while (next >= 0 && next < items.length && items[next].kind === "subgroup-header")
-    next += dir;
+  const next = current + dir;
   if (next < 0 || next >= items.length) return current;
   return next;
 };
@@ -1602,6 +1600,28 @@ var treeUrls = (items, i) => {
   }
   return urls;
 };
+var bandUrls = (items, i) => {
+  const urls = [];
+  for (let j = i + 1; j < items.length; j += 1) {
+    const item = items[j];
+    if (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 copyUrls = (items, i) => {
+  const item = items[i];
+  if (!item) return [];
+  if (item.kind === "repo-header") return groupUrls(items, i);
+  if (item.kind === "subgroup-header") return bandUrls(items, i);
+  const url = item.url;
+  return url ? [url] : [];
+};
 var gapsAbove = (items, i) => {
   const item = items[i];
   if (!item || i === 0) return false;
@@ -2381,7 +2401,8 @@ var ItemRow = ({
     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(Text3, { color: colors.accent, bold: true, children: "  \xBB " }),
+      /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
+      /* @__PURE__ */ jsx(Text3, { color: colors.accent, bold: true, children: "\xBB " }),
       /* @__PURE__ */ jsx(Text3, { bold: true, children: item.label })
     ] });
   if (item.kind === "show-more")
@@ -2879,17 +2900,7 @@ var BrowseScreen = ({
             s.items.length - 1
           );
           if (c < 0) return [s.id, 0];
-          return [
-            s.id,
-            // Only `subgroup-header`, which is the one kind the arrows refuse to
-            // stand on — `moveCursor` skips it and nothing else. A REPO header is
-            // selectable on purpose: it opens the repo and `C` copies the group,
-            // and the footer advertises both. Stepping off it here meant that
-            // standing on a repo and pressing `r` moved you onto an issue, every
-            // single time, and the rule predated repo headers being selectable
-            // at all.
-            s.items[c]?.kind === "subgroup-header" ? moveCursor(s.items, c, 1) : c
-          ];
+          return [s.id, c];
         })
       )
     );
@@ -3226,8 +3237,10 @@ var BrowseScreen = ({
         return;
       }
       if (input === "c") {
-        clipboard(repoUrl);
-        showFlash(`\u2713 Copied URL for ${repo}`);
+        const urls = copyUrls(section.items, cursor);
+        if (urls.length === 0) return;
+        clipboard(urls.join("\n"));
+        showFlash(`\u2713 Copied ${urls.length} link${urls.length === 1 ? "" : "s"}`);
         return;
       }
       if (input === "C" || input === "O") {
@@ -3246,7 +3259,16 @@ var BrowseScreen = ({
       }
       return;
     }
-    if (!activeItem || activeItem.kind === "subgroup-header") return;
+    if (activeItem?.kind === "subgroup-header") {
+      if (input === "c") {
+        const urls = copyUrls(section.items, cursor);
+        if (urls.length === 0) return;
+        clipboard(urls.join("\n"));
+        showFlash(`\u2713 Copied ${urls.length} link${urls.length === 1 ? "" : "s"}`);
+      }
+      return;
+    }
+    if (!activeItem) return;
     if (key.return && activeItem.kind === "show-more") {
       setLocalSections(
         (prev) => prev.map((s) => ({
@@ -3438,6 +3460,11 @@ var BrowseScreen = ({
       menu.open(actions);
     }
   });
+  const headerLinks = activeItem?.kind === "repo-header" || activeItem?.kind === "subgroup-header" ? copyUrls(section.items, cursor).length : 0;
+  const headerHint = [
+    "c",
+    `copy ${headerLinks} link${headerLinks === 1 ? "" : "s"}`
+  ];
   const hints = railActive ? (
     // The rail's own keymap while it holds focus, not the list's with a line
     // bolted on. A footer advertising `m actions` beside a cursor that cannot
@@ -3457,7 +3484,13 @@ var BrowseScreen = ({
     ["\u2190\u2192", "tab"],
     ["\u21B5", "open repo"],
     ["o", "browser"],
-    ["C", "copy group"],
+    headerHint,
+    ["?", "help"],
+    ["q", "quit"]
+  ] : activeItem?.kind === "subgroup-header" ? [
+    ["\u2191\u2193", "nav"],
+    ["\u2190\u2192", "tab"],
+    headerHint,
     ["?", "help"],
     ["q", "quit"]
   ] : [

package/package.json

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