@kud/gh-ink 0.64.0 → 0.65.1

package/dist/index.d.ts

@@ -209,6 +209,13 @@ type PrRowProps = {
     item: GHItem;
     active: boolean;
     login?: string;
+    /**
+     * The selection wash for this frame, resolved by the list — see
+     * `selectionBackground` in the inbox. A prop rather than an import because
+     * the constant lives next to the inbox's `OVERLAY_BG` and importing the inbox
+     * here would cycle back through this module. Absent the row draws no wash.
+     */
+    selectionBg?: string;
     /**
      * Columns available to this row, which is NOT always the frame width: a rail
      * beside the list takes its share, and a budget that does not know the rail is
@@ -318,7 +325,7 @@ declare const sectionShedOf: (rows: readonly Pick<LayoutInput, "item" | "prefix"
  * arrival, merge and departure belongs to whoever owns the list, and reaches
  * the row as `icon`, `motion` and `announcements` after the decisions are made.
  */
-declare const PrRow: ({ item, active, login, cols, icon: iconOverride, prefix, uniformLabels, motion, announcements, columns, shed, numberCols, }: PrRowProps) => React.JSX.Element;
+declare const PrRow: ({ item, active, login, cols, selectionBg, icon: iconOverride, prefix, uniformLabels, motion, announcements, columns, shed, numberCols, }: PrRowProps) => React.JSX.Element;
 
 /**
  * One state per thing, from a closed set the strip knows how to draw. The host
@@ -543,18 +550,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

@@ -950,6 +950,7 @@ var PrRow = ({
   active,
   login,
   cols,
+  selectionBg,
   icon: iconOverride,
   prefix = "",
   uniformLabels = [],
@@ -990,68 +991,72 @@ var PrRow = ({
     shed,
     numberCols
   });
-  return /* @__PURE__ */ jsxs(Box, { width: cols, children: [
-    /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
-    /* @__PURE__ */ jsx(Text3, { dimColor: true, children: prefix }),
-    /* @__PURE__ */ jsx(Text3, { color, bold: true, children: icon + " " }),
-    /* @__PURE__ */ jsx(Text3, { color: turnColor, bold: turnIcon === "\u2190", children: turnIcon + " " }),
-    /* @__PURE__ */ jsx(Text3, { color: colors.accent, children: numStr }),
-    /* @__PURE__ */ jsx(
-      Text3,
-      {
-        bold: active || motion === "arriving",
-        dimColor: motion === "departing" || motion === "withdrawn",
-        strikethrough: motion === "withdrawn",
-        children: truncate(
-          item.needsYou?.decision ? item.needsYou.decision : item.title,
-          titleMax
-        ) + "  "
-      }
-    ),
-    labelLabel ? /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: labelLabel + "  " }) : null,
-    repoLabel && !givingUp.repo ? /* @__PURE__ */ jsx(Text3, { dimColor: true, children: repoLabel }) : null,
-    showAuthor && !givingUp.author ? /* @__PURE__ */ jsx(Text3, { dimColor: true, italic: true, children: "  by " + item.author }) : null,
-    /* @__PURE__ */ jsx(Box, { flexGrow: 1 }),
-    block.threads > 0 && !givingUp.threads ? /* @__PURE__ */ jsx(
-      Box,
-      {
-        marginLeft: 2,
-        width: block.threads,
-        justifyContent: "flex-end",
-        flexShrink: 0,
-        children: unresolvedLabel ? /* @__PURE__ */ jsx(
-          Text3,
-          {
-            bold: !spokeLast,
-            color: spokeLast ? colors.secondary : colors.accent,
-            children: unresolvedLabel
-          }
-        ) : null
-      }
-    ) : null,
-    block.size > 0 && !givingUp.size ? /* @__PURE__ */ jsx(
-      Box,
-      {
-        marginLeft: 2,
-        width: block.size,
-        justifyContent: "flex-end",
-        flexShrink: 0,
-        children: sizeParts ? /* @__PURE__ */ jsxs(Text3, { children: [
-          /* @__PURE__ */ jsx(Text3, { color: colors.success, children: sizeParts.added }),
-          " ",
-          /* @__PURE__ */ jsx(Text3, { color: colors.error, children: sizeParts.removed })
-        ] }) : null
-      }
-    ) : null,
-    block.age > 0 && !givingUp.age ? /* @__PURE__ */ jsx(Box, { marginLeft: 2, width: block.age, flexShrink: 0, children: agePair ? /* @__PURE__ */ jsxs(Text3, { children: [
-      /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: agePair.activity }),
-      /* @__PURE__ */ jsx(Text3, { dimColor: true, children: ` (${agePair.lifetime})` })
-    ] }) : ageLabel2 ? /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: ageLabel2 }) : null }) : null,
-    backdropped ? null : announcements.map((a) => /* @__PURE__ */ jsxs(React2.Fragment, { children: [
-      /* @__PURE__ */ jsx(Text3, { children: "  " }),
-      /* @__PURE__ */ jsx(Pill, { color: a.color, children: a.label })
-    ] }, a.label))
-  ] });
+  return (
+    // The wash paints every cell of the fixed-width row — an Ink Box's
+    // background covers its full measure, not just the inked cells.
+    /* @__PURE__ */ jsxs(Box, { width: cols, backgroundColor: selectionBg, children: [
+      /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
+      /* @__PURE__ */ jsx(Text3, { dimColor: true, children: prefix }),
+      /* @__PURE__ */ jsx(Text3, { color, bold: true, children: icon + " " }),
+      /* @__PURE__ */ jsx(Text3, { color: turnColor, bold: turnIcon === "\u2190", children: turnIcon + " " }),
+      /* @__PURE__ */ jsx(Text3, { color: colors.accent, children: numStr }),
+      /* @__PURE__ */ jsx(
+        Text3,
+        {
+          bold: active || motion === "arriving",
+          dimColor: motion === "departing" || motion === "withdrawn",
+          strikethrough: motion === "withdrawn",
+          children: truncate(
+            item.needsYou?.decision ? item.needsYou.decision : item.title,
+            titleMax
+          ) + "  "
+        }
+      ),
+      labelLabel ? /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: labelLabel + "  " }) : null,
+      repoLabel && !givingUp.repo ? /* @__PURE__ */ jsx(Text3, { dimColor: true, children: repoLabel }) : null,
+      showAuthor && !givingUp.author ? /* @__PURE__ */ jsx(Text3, { dimColor: true, italic: true, children: "  by " + item.author }) : null,
+      /* @__PURE__ */ jsx(Box, { flexGrow: 1 }),
+      block.threads > 0 && !givingUp.threads ? /* @__PURE__ */ jsx(
+        Box,
+        {
+          marginLeft: 2,
+          width: block.threads,
+          justifyContent: "flex-end",
+          flexShrink: 0,
+          children: unresolvedLabel ? /* @__PURE__ */ jsx(
+            Text3,
+            {
+              bold: !spokeLast,
+              color: spokeLast ? colors.secondary : colors.accent,
+              children: unresolvedLabel
+            }
+          ) : null
+        }
+      ) : null,
+      block.size > 0 && !givingUp.size ? /* @__PURE__ */ jsx(
+        Box,
+        {
+          marginLeft: 2,
+          width: block.size,
+          justifyContent: "flex-end",
+          flexShrink: 0,
+          children: sizeParts ? /* @__PURE__ */ jsxs(Text3, { children: [
+            /* @__PURE__ */ jsx(Text3, { color: colors.success, children: sizeParts.added }),
+            " ",
+            /* @__PURE__ */ jsx(Text3, { color: colors.error, children: sizeParts.removed })
+          ] }) : null
+        }
+      ) : null,
+      block.age > 0 && !givingUp.age ? /* @__PURE__ */ jsx(Box, { marginLeft: 2, width: block.age, flexShrink: 0, children: agePair ? /* @__PURE__ */ jsxs(Text3, { children: [
+        /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: agePair.activity }),
+        /* @__PURE__ */ jsx(Text3, { dimColor: true, children: ` (${agePair.lifetime})` })
+      ] }) : ageLabel2 ? /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: ageLabel2 }) : null }) : null,
+      backdropped ? null : announcements.map((a) => /* @__PURE__ */ jsxs(React2.Fragment, { children: [
+        /* @__PURE__ */ jsx(Text3, { children: "  " }),
+        /* @__PURE__ */ jsx(Pill, { color: a.color, children: a.label })
+      ] }, a.label))
+    ] })
+  );
 };
 var stripGlyph = (state) => {
   switch (state) {
@@ -1524,9 +1529,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 +1605,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;
@@ -1830,6 +1855,11 @@ var COLS = (process.stdout.columns ?? 120) - FRAME_CHROME_COLS;
 var MODAL_CHROME_COLS = 2 + 1 * 2;
 var ROW_MAX_COLS = 140;
 var OVERLAY_BG = "#22222e";
+var SELECTION_BG = "#16161f";
+var selectionBackground = (active, backdropped) => active && !backdropped ? SELECTION_BG : void 0;
+var taskKeyStyle = (active) => ({
+  bold: active
+});
 var workCount = (s) => {
   let n = 0;
   for (let i = 0; i < s.items.length; i++) {
@@ -2296,12 +2326,14 @@ var FENCE_COLS = 46;
 var RepoHeaderRow = ({
   repo,
   gap,
-  active
+  active,
+  cols,
+  selectionBg
 }) => {
   const label = `\u2500\u2500 ${repo} `;
   const fill = Math.max(4, FENCE_COLS - label.length);
   const rule = "\u2500".repeat(fill);
-  return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
+  return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, width: cols, backgroundColor: selectionBg, children: [
     /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
     /* @__PURE__ */ jsx(Text3, { dimColor: !active, children: "\u2500\u2500 " }),
     /* @__PURE__ */ jsx(
@@ -2373,26 +2405,37 @@ var ItemRow = ({
   numberCols
 }) => {
   const backdropped = useBackdropped();
+  const selBg = selectionBackground(active, backdropped);
   const transient = leaving ? "out" : refreshMark;
   const transitFrame = Math.floor(sparkFrame / TRANSIT_FRAME_TICKS);
   const rampAt = (len) => rampFrame(len, transitFrame, transitAt);
   const farewellLabel = leaving ? TRANSIT_LABEL.out : "";
   if (item.kind === "repo-header")
-    return /* @__PURE__ */ jsx(RepoHeaderRow, { repo: item.repo, gap: gap ?? false, active });
+    return /* @__PURE__ */ jsx(
+      RepoHeaderRow,
+      {
+        repo: item.repo,
+        gap: gap ?? false,
+        active,
+        cols,
+        selectionBg: selBg
+      }
+    );
   if (item.kind === "subgroup-header")
-    return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
-      /* @__PURE__ */ jsx(Text3, { color: colors.accent, bold: true, children: "  \xBB " }),
+    return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, width: cols, backgroundColor: selBg, children: [
+      /* @__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")
-    return /* @__PURE__ */ jsxs(Box, { children: [
+    return /* @__PURE__ */ jsxs(Box, { width: cols, backgroundColor: selBg, children: [
       /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: prefix }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: active ? "\u21B5 " : "  " }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: `+${item.hidden.length} more` })
     ] });
   if (item.kind === "show-less")
-    return /* @__PURE__ */ jsxs(Box, { children: [
+    return /* @__PURE__ */ jsxs(Box, { width: cols, backgroundColor: selBg, children: [
       /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: prefix }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: active ? "\u21B5 " : "  " }),
@@ -2405,13 +2448,13 @@ var ItemRow = ({
       1,
       cols - item.key.length - note.length - (item.pill ? pillWidth(item.pill) + 1 : 0) - (farewellLabel ? pillWidth(farewellLabel) + 2 : 0) - prefix.length - (markerCols > 0 ? markerCols + 1 : 0) - 12
     );
-    return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
+    return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, width: cols, backgroundColor: selBg, children: [
       /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: prefix }),
       /* @__PURE__ */ jsx(Text3, { dimColor: true, children: parent ? "\u252C " : "  " }),
       /* @__PURE__ */ jsx(Text3, { bold: true, color: transient ? TRANSIT_COLOUR[transient] : void 0, children: transitIcon + " " }),
       markerCols > 0 ? /* @__PURE__ */ jsx(Text3, { color: item.markerColor, children: (item.marker ?? "").padEnd(markerCols) + " " }) : null,
-      /* @__PURE__ */ jsx(Text3, { color: colors.accent, bold: active, children: item.key + "  " }),
+      /* @__PURE__ */ jsx(Text3, { ...taskKeyStyle(active), children: item.key + "  " }),
       /* @__PURE__ */ jsx(
         Text3,
         {
@@ -2459,6 +2502,7 @@ var ItemRow = ({
       active,
       login,
       cols,
+      selectionBg: selBg,
       prefix,
       uniformLabels,
       columns,
@@ -2879,17 +2923,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 +3260,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 +3282,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 +3483,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 +3507,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"]
   ] : [
@@ -3695,7 +3751,7 @@ var BrowseScreen = ({
                           prefix: treePrefix(section.items, viewStart + i),
                           parent: item.kind === "task" && depthOf(section.items[viewStart + i + 1]) > depthOf(item),
                           sparkFrame,
-                          cols: item.kind === "pr" || item.kind === "issue" ? rowCols : listCols,
+                          cols: item.kind === "pr" || item.kind === "issue" || item.kind === "repo-header" || item.kind === "subgroup-header" || item.kind === "task" || item.kind === "show-more" || item.kind === "show-less" ? rowCols : listCols,
                           markerCols,
                           uniformLabels,
                           columns: trailingColumns,

package/package.json

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