@kud/gh-ink 0.63.0 → 0.64.0

package/dist/index.d.ts

@@ -1,7 +1,7 @@
 import React, { ReactNode } from 'react';
 import { PrComments, PrHealthData, PrCheck, Health } from '@kud/gh';
 import { GHItem, AnyItem, Section, CiStatus, OriginSplit } from '@kud/gh-workflow';
-export { AnyItem, Band, CiStatus, GHDetail, GHItem, InboxConfig, Move, OriginSplit, PIN_MARK, RepoHeader, RepoProfile, Section, ShowLess, ShowMore, Standing, SubgroupHeader, TaskRow, bandOf, checkoutDirs, configureInbox, filterByOrigin, filterByRepos, filterBySearch, impliedLabels, inboxConfig, insertRepoHeaders, labelPriority, layoutGHItems, profileOf, relativeTime, repoPriority, resetInboxConfig, sortByRecency, sortItems, whoseMove } from '@kud/gh-workflow';
+export { AnyItem, Band, CiStatus, GHDetail, GHItem, InboxConfig, Move, NEEDS_YOU_SECTION, NeedsYou, OriginSplit, PIN_MARK, RepoHeader, RepoProfile, Section, ShowLess, ShowMore, Standing, SubgroupHeader, TaskRow, bandOf, checkoutDirs, configureInbox, filterByOrigin, filterByRepos, filterBySearch, impliedLabels, inboxConfig, insertRepoHeaders, labelPriority, layoutGHItems, profileOf, relativeTime, repoPriority, resetInboxConfig, sortByRecency, sortItems, whoseMove } from '@kud/gh-workflow';
 import { StyledLine } from '@kud/ink-ui';
 
 type CommentsPanelProps = {
@@ -250,6 +250,12 @@ type PrRowProps = {
     columns?: TrailingColumns;
     /** Trailing columns the section has given up — see `sectionShedOf`. */
     shed?: TrailingShed;
+    /**
+     * Width of the number cell across the section — see `numberColumnsOf`.
+     * Absent, the row measures itself, which aligns nothing but never runs the
+     * number into the title.
+     */
+    numberCols?: number;
 };
 /**
  * The trailing block, in reading order: how contested, how big, how recent.
@@ -274,7 +280,19 @@ type TrailingShed = Partial<Record<(typeof TRAILING)[number], boolean>>;
  * would not.
  */
 declare const trailingColumnsOf: (items: readonly GHItem[]) => TrailingColumns;
-type LayoutInput = Pick<PrRowProps, "item" | "login" | "cols" | "prefix" | "uniformLabels" | "announcements" | "columns" | "shed">;
+/**
+ * Width of the `#n` cell across these rows: the widest number plus a two-cell
+ * gutter, left-aligned. Measured once by the list over the whole section, the
+ * same move as `markerCols` and `trailingColumnsOf`, so the titles start on one
+ * column and that column does not move as you scroll.
+ *
+ * It replaced `padEnd(7)`, a constant that fitted the repos it was written
+ * against and nothing else: `#31805` left a one-cell gutter where `#172` left
+ * three, and a six-digit number would have run straight into its title. A
+ * section of short numbers no longer pays for a long one it does not hold.
+ */
+declare const numberColumnsOf: (items: readonly GHItem[]) => number;
+type LayoutInput = Pick<PrRowProps, "item" | "login" | "cols" | "prefix" | "uniformLabels" | "announcements" | "columns" | "shed" | "numberCols">;
 /**
  * Which trailing columns the section has to give up, so that every row gives up
  * the same ones. Each row is laid out alone and the drops are unioned: if any
@@ -300,7 +318,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, }: PrRowProps) => React.JSX.Element;
+declare const PrRow: ({ item, active, login, cols, 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
@@ -792,4 +810,4 @@ declare const matchesFilter: (repo: string, filter: RepoFilter) => boolean;
  */
 declare const parsePatterns: (value: string) => string[];
 
-export { type Action, ActionMenu, App, COLS, CiStatusLine, type CiStatusState, type Command, CommentsPanel, type CommentsPanelProps, type DetailContext, type ExplainSection, type ExtensionTarget, HealthPanel, type HealthPanelProps, type InboxBudget, type InboxExtension, type JiraTransition, type LiveLabel, MIN_SIDEBAR_COLS, PrRow, type PrRowProps, type Rails, type RepoFilter, type RowAnnouncement, type RowMotion, SIDEBAR_COLS, type SettledResult, SidePanel, type Sidebar, type SidebarRow, type StatusStrip, StatusStripLine, type StripItem, type StripState, type StripStatusState, type TrailingColumns, type TrailingShed, ageLabel, budgetNotice, buildActions, buildCheckoutCmd, clipboard, counts, drillCmd, explainGhAction, explainItem, fitCount, gapsAbove, healthColor, healthDisplay, healthGlyph, healthLegend, itermRun, jumpToRepo, jumpToRepoPane, matchesFilter, maxViewStart, moveCursor, openInTab, parsePatterns, railCapacity, railHeights, railRowsOf, railWidth, railsOf, readCache, renderMarkdown, reposInSections, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sectionShedOf, signatureOf, stripGlyph, stripLayout, stripSignature, toCiStatusState, trailingColumnsOf, truncate, useActionMenu, windowCount, withHeaders, withoutItem, workCount, writeCache };
+export { type Action, ActionMenu, App, COLS, CiStatusLine, type CiStatusState, type Command, CommentsPanel, type CommentsPanelProps, type DetailContext, type ExplainSection, type ExtensionTarget, HealthPanel, type HealthPanelProps, type InboxBudget, type InboxExtension, type JiraTransition, type LiveLabel, MIN_SIDEBAR_COLS, PrRow, type PrRowProps, type Rails, type RepoFilter, type RowAnnouncement, type RowMotion, SIDEBAR_COLS, type SettledResult, SidePanel, type Sidebar, type SidebarRow, type StatusStrip, StatusStripLine, type StripItem, type StripState, type StripStatusState, type TrailingColumns, type TrailingShed, ageLabel, budgetNotice, buildActions, buildCheckoutCmd, clipboard, counts, drillCmd, explainGhAction, explainItem, fitCount, gapsAbove, healthColor, healthDisplay, healthGlyph, healthLegend, itermRun, jumpToRepo, jumpToRepoPane, matchesFilter, maxViewStart, moveCursor, numberColumnsOf, openInTab, parsePatterns, railCapacity, railHeights, railRowsOf, railWidth, railsOf, readCache, renderMarkdown, reposInSections, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sectionShedOf, signatureOf, stripGlyph, stripLayout, stripSignature, toCiStatusState, trailingColumnsOf, truncate, useActionMenu, windowCount, withHeaders, withoutItem, workCount, writeCache };

package/dist/index.js

@@ -4,7 +4,7 @@ import { colors, ScrollView, TextInput, Pill, useAppKeys, LoadingScreen, StatusM
 import { isPassCheck, isFailCheck, isInconclusiveCheck, resolveThread, unresolveThread, replyToThread, rerunFailedRun, mergePr, reRequestReviewer } from '@kud/gh';
 import { jsxs, Fragment, jsx } from 'react/jsx-runtime';
 import { PIN_MARK, sizeOf, sizePartsOf, depthOf, impliedLabels, labelPriority, checkoutDirs, profileOf, inboxConfig, relativeTime, filterBySearch, filterByRepos, filterByOrigin } from '@kud/gh-workflow';
-export { PIN_MARK, bandOf, checkoutDirs, configureInbox, filterByOrigin, filterByRepos, filterBySearch, impliedLabels, inboxConfig, insertRepoHeaders, labelPriority, layoutGHItems, profileOf, relativeTime, repoPriority, resetInboxConfig, sortByRecency, sortItems, whoseMove } from '@kud/gh-workflow';
+export { NEEDS_YOU_SECTION, PIN_MARK, bandOf, checkoutDirs, configureInbox, filterByOrigin, filterByRepos, filterBySearch, impliedLabels, inboxConfig, insertRepoHeaders, labelPriority, layoutGHItems, profileOf, relativeTime, repoPriority, resetInboxConfig, sortByRecency, sortItems, whoseMove } from '@kud/gh-workflow';
 import { $ } from 'zx';
 import { spawn } from 'child_process';
 import { readFileSync, mkdirSync, writeFileSync, existsSync, readdirSync, statSync, watch, rmSync } from 'fs';
@@ -849,6 +849,7 @@ var trailingColumnsOf = (items) => items.reduce(
   }),
   { threads: 0, size: 0, age: 0 }
 );
+var numberColumnsOf = (items) => items.reduce((w, item) => Math.max(w, cellsOf(`#${item.number}`)), 0) + 2;
 var sectionShedOf = (rows, context) => rows.reduce((shed, row) => {
   const { givingUp } = layoutOf({ ...context, ...row });
   return {
@@ -865,9 +866,11 @@ var layoutOf = ({
   uniformLabels = [],
   announcements = [],
   columns,
-  shed = {}
+  shed = {},
+  numberCols
 }) => {
-  const numStr = `#${item.number}`.padEnd(7);
+  const numCols = numberCols ?? numberColumnsOf([item]);
+  const numStr = `#${item.number}`.padEnd(numCols);
   const showAuthor = !!item.author && item.author !== login;
   const sizeParts = sizePartsOf(item);
   const unresolvedLabel = threadsLabelOf(item);
@@ -911,7 +914,7 @@ var layoutOf = ({
     const author = showAuthor && !givingUp.author ? `  by ${item.author}`.length : 0;
     const pills = announcements.reduce((w, a) => w + 2 + a.label.length, 0);
     const cell = labelCell();
-    return 2 + prefix.length + 2 + 2 + 7 + (cell ? 2 + cell.length + 1 : 0) + (givingUp.repo ? 0 : repoLabel.length) + author + block + pills + pillCaps + 4;
+    return 2 + prefix.length + 2 + 2 + numCols + (cell ? 2 + cell.length + 1 : 0) + (givingUp.repo ? 0 : repoLabel.length) + author + block + pills + pillCaps + 4;
   };
   const MIN_TITLE = 24;
   for (const give of [
@@ -953,7 +956,8 @@ var PrRow = ({
   motion,
   announcements = [],
   columns,
-  shed
+  shed,
+  numberCols
 }) => {
   const backdropped = useBackdropped();
   const { glyph: healthIcon, color: healthColor2 } = displayFor(item.health);
@@ -983,7 +987,8 @@ var PrRow = ({
     uniformLabels,
     announcements,
     columns,
-    shed
+    shed,
+    numberCols
   });
   return /* @__PURE__ */ jsxs(Box, { width: cols, children: [
     /* @__PURE__ */ jsx(Text3, { color: colors.info, children: active ? "\u276F " : "  " }),
@@ -997,7 +1002,10 @@ var PrRow = ({
         bold: active || motion === "arriving",
         dimColor: motion === "departing" || motion === "withdrawn",
         strikethrough: motion === "withdrawn",
-        children: truncate(item.title, titleMax) + "  "
+        children: truncate(
+          item.needsYou?.decision ? item.needsYou.decision : item.title,
+          titleMax
+        ) + "  "
       }
     ),
     labelLabel ? /* @__PURE__ */ jsx(Text3, { color: colors.secondary, children: labelLabel + "  " }) : null,
@@ -1820,6 +1828,7 @@ var FRAME_PAD_X = 1;
 var FRAME_CHROME_COLS = 2 + FRAME_PAD_X * 2;
 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 workCount = (s) => {
   let n = 0;
@@ -1994,7 +2003,10 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
           (e) => {
             showFlash(`\u2717 #${item.number}: ${explainGhAction(e)}`);
             onRefresh?.();
-            ext?.onSettled?.(item, { ok: false, reason: firstLine(e) });
+            ext?.onSettled?.(item, {
+              ok: false,
+              reason: firstLine(e)
+            });
           }
         );
       }
@@ -2076,7 +2088,10 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
               (e) => {
                 showFlash(`\u2717 Close failed \u2014 restoring #${item.number}`);
                 onRefresh?.();
-                ext?.onSettled?.(item, { ok: false, reason: firstLine(e) });
+                ext?.onSettled?.(item, {
+                  ok: false,
+                  reason: firstLine(e)
+                });
               }
             );
           }
@@ -2132,7 +2147,10 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
               (e) => {
                 showFlash(`\u2717 Close failed \u2014 restoring #${item.number}`);
                 onRefresh?.();
-                ext?.onSettled?.(item, { ok: false, reason: firstLine(e) });
+                ext?.onSettled?.(item, {
+                  ok: false,
+                  reason: firstLine(e)
+                });
               }
             );
           }
@@ -2351,7 +2369,8 @@ var ItemRow = ({
   markerCols = 0,
   uniformLabels = [],
   columns,
-  shed
+  shed,
+  numberCols
 }) => {
   const backdropped = useBackdropped();
   const transient = leaving ? "out" : refreshMark;
@@ -2444,6 +2463,7 @@ var ItemRow = ({
       uniformLabels,
       columns,
       shed,
+      numberCols,
       icon,
       motion,
       announcements
@@ -2967,11 +2987,19 @@ var BrowseScreen = ({
   const railCols = railWidth(COLS);
   const listCols = showRail ? COLS - railCols : COLS;
   const trailingColumns = trailingColumnsOf(labelBearing);
+  const numberCols = numberColumnsOf(labelBearing);
+  const rowCols = Math.min(listCols, ROW_MAX_COLS);
   const trailingShed = sectionShedOf(
     section.items.flatMap(
       (item, i) => item.kind === "pr" || item.kind === "issue" ? [{ item, prefix: treePrefix(section.items, i) }] : []
     ),
-    { login, cols: listCols, uniformLabels, columns: trailingColumns }
+    {
+      login,
+      cols: rowCols,
+      uniformLabels,
+      columns: trailingColumns,
+      numberCols
+    }
   );
   const [sparkFrame, setSparkFrame] = useState(0);
   const sparkling = (mergedUrls?.length ?? 0) > 0 || (leavingUrls?.length ?? 0) > 0 || // Any MARKED TAB, not just rows on screen. The scope used to be this tab's
@@ -3667,11 +3695,12 @@ var BrowseScreen = ({
                           prefix: treePrefix(section.items, viewStart + i),
                           parent: item.kind === "task" && depthOf(section.items[viewStart + i + 1]) > depthOf(item),
                           sparkFrame,
-                          cols: listCols,
+                          cols: item.kind === "pr" || item.kind === "issue" ? rowCols : listCols,
                           markerCols,
                           uniformLabels,
                           columns: trailingColumns,
                           shed: trailingShed,
+                          numberCols,
                           gap: i > 0 && gapsAbove(section.items, viewStart + i)
                         },
                         `${viewStart + i}:${rowKey(item)}`
@@ -4260,4 +4289,4 @@ var matchesFilter = (repo, filter) => {
 };
 var parsePatterns = (value) => value.split(",").map((p) => p.trim()).filter(Boolean);
 
-export { ActionMenu, App, COLS, CiStatusLine, CommentsPanel, HealthPanel, MIN_SIDEBAR_COLS, PrRow, SIDEBAR_COLS, SidePanel, StatusStripLine, ageLabel, budgetNotice, buildActions, buildCheckoutCmd, clipboard, counts, drillCmd, explainGhAction, explainItem, fitCount, gapsAbove, healthColor, healthDisplay, healthGlyph, healthLegend, itermRun, jumpToRepo, jumpToRepoPane, matchesFilter, maxViewStart, moveCursor, openInTab, parsePatterns, railCapacity, railHeights, railRowsOf, railWidth, railsOf, readCache, renderMarkdown, reposInSections, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sectionShedOf, signatureOf, stripGlyph, stripLayout, stripSignature, toCiStatusState, trailingColumnsOf, truncate, useActionMenu, windowCount, withHeaders, withoutItem, workCount, writeCache };
+export { ActionMenu, App, COLS, CiStatusLine, CommentsPanel, HealthPanel, MIN_SIDEBAR_COLS, PrRow, SIDEBAR_COLS, SidePanel, StatusStripLine, ageLabel, budgetNotice, buildActions, buildCheckoutCmd, clipboard, counts, drillCmd, explainGhAction, explainItem, fitCount, gapsAbove, healthColor, healthDisplay, healthGlyph, healthLegend, itermRun, jumpToRepo, jumpToRepoPane, matchesFilter, maxViewStart, moveCursor, numberColumnsOf, openInTab, parsePatterns, railCapacity, railHeights, railRowsOf, railWidth, railsOf, readCache, renderMarkdown, reposInSections, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sectionShedOf, signatureOf, stripGlyph, stripLayout, stripSignature, toCiStatusState, trailingColumnsOf, truncate, useActionMenu, windowCount, withHeaders, withoutItem, workCount, writeCache };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@kud/gh-ink",
-  "version": "0.63.0",
+  "version": "0.64.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,8 +48,8 @@
     "react": ">=19"
   },
   "dependencies": {
-    "@kud/gh": "0.18.0",
-    "@kud/gh-workflow": "0.13.1",
+    "@kud/gh": "0.19.0",
+    "@kud/gh-workflow": "0.14.0",
     "@kud/ink-ui": "0.31.0",
     "zx": "8.8.5"
   },