@kud/gh-ink 0.22.1 → 0.23.1

package/dist/index.d.ts

@@ -201,7 +201,38 @@ declare const insertRepoHeaders: (items: GHItem[]) => AnyItem[];
 type Standing = "authored" | "queued" | "spoken";
 declare const whoseMove: (health: Health, sectionId: string, standing?: Standing) => "you" | "them";
 declare const layoutGHItems: (items: GHItem[], sectionId: string) => AnyItem[];
-declare const filterByOrigin: (sections: Section[], keep: "work" | "home", isWorkRepo: (repo: string) => boolean) => Section[];
+/**
+ * A two-way repo split, when a host has one. `undefined` — the ordinary case —
+ * means no split at all: every row stands.
+ *
+ * The package deliberately does not NAME the sides. It hardcoded "work" and
+ * "home" until 2026-08-27: one reader's two lives compiled into a library
+ * anyone can install, printed in a header where no other host would recognise
+ * the word. The predicate, the side, and the word are all yours.
+ *
+ * Chosen once, by the host, before the first paint. There is no in-app toggle,
+ * because a key that flipped it could only ever put the inbox out of step with
+ * the scope the command was started in, with nothing on screen to explain the
+ * disagreement.
+ */
+type OriginSplit = {
+    /** Rows whose repo this matches are one side; everything else is the other. */
+    match: (repo: string) => boolean;
+    /** Which side to show. */
+    show: "matched" | "rest";
+    /** What to call the visible side in the header. Omitted, nothing is shown. */
+    label?: string;
+};
+/**
+ * Keep only one side of a two-way repo split, leaving non-GH rows (task)
+ * untouched. Mirrors filterByRepos' gh/other split.
+ *
+ * `keep` is positional — "the ones that matched" or "the rest" — and never a
+ * name. This took the literal strings "work" and "home" until 2026-08-27, which
+ * is one reader's life compiled into a published library: no other host has
+ * those two categories, and several have none at all.
+ */
+declare const filterByOrigin: (sections: Section[], keep: "matched" | "rest", match: (repo: string) => boolean) => Section[];
 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[];
@@ -251,6 +282,21 @@ declare const CiStatusLine: ({ state, job, }: {
     state: CiStatusState;
     job?: string;
 }) => React.JSX.Element;
+/**
+ * Indexes of task rows whose row has already appeared higher up the same list.
+ *
+ * A ticket with one PR needing you and another in review lands in two bands,
+ * which is right — the band reads each PR, not the ticket. What was wrong was
+ * drawing the second occurrence identically to the first, so one ticket read as
+ * two. The repeat keeps its key and drops its summary.
+ *
+ * Identity is the URL, never `key`. `key` is a LABEL, and on a task surface that
+ * is not a tracker (`life`, whose keys are categories like "Flat") one label
+ * legitimately heads several unrelated rows — keying on it hid the summary of
+ * every row but the first. A row with no url is never a repeat, because nothing
+ * identifies it.
+ */
+declare const repeatedTaskRows: (items: readonly AnyItem[]) => Set<number>;
 declare const useActionMenu: () => {
     actions: Action[] | null;
     cursor: number;
@@ -268,7 +314,8 @@ declare const ActionMenu: ({ item, actions, cursor, }: {
     actions: Action[];
     cursor: number;
 }) => React.JSX.Element;
-declare const App: ({ fetcher, cacheKey, title, detailFor, isWorkRepo, includeWork, jiraBase, jiraKeyRe, jiraTransitions, hasCiStatus, ciJob, ciFetcher, ciPollMs, watchPath, watchDebounceMs, extensions, tabHelp, emptyHint, }: {
+declare const signatureOf: (sections: Section[]) => string;
+declare const App: ({ fetcher, cacheKey, title, detailFor, origin, jiraBase, jiraKeyRe, jiraTransitions, hasCiStatus, ciJob, ciFetcher, ciPollMs, watchPath, watchDebounceMs, extensions, tabHelp, emptyHint, }: {
     fetcher: () => Promise<{
         sections: Section[];
         login: string;
@@ -277,17 +324,8 @@ declare const App: ({ fetcher, cacheKey, title, detailFor, isWorkRepo, includeWo
     cacheKey?: string;
     title?: string;
     detailFor?: (ctx: DetailContext) => React.ReactNode;
-    isWorkRepo?: (repo: string) => boolean;
-    /**
-     * Which side of the split to show, chosen ONCE by the host at launch — there
-     * is no in-app toggle. `undefined` means no split: every row stands.
-     *
-     * The host decides from where the command was run, so the answer is settled
-     * before the first paint and cannot change under the reader. A `w` key used to
-     * flip it mid-session, which meant the inbox could disagree with the scope the
-     * command had been started in, with nothing on screen explaining why.
-     */
-    includeWork?: boolean;
+    /** A two-way repo split, and what to call the side on screen. See OriginSplit. */
+    origin?: OriginSplit;
     tabHelp?: [string, string][];
     emptyHint?: string;
     jiraBase?: string;
@@ -406,4 +444,4 @@ declare const matchesFilter: (repo: string, filter: RepoFilter) => boolean;
  */
 declare const parsePatterns: (value: string) => string[];
 
-export { type Action, ActionMenu, type AnyItem, App, COLS, type CiStatus, CiStatusLine, type CiStatusState, CommentsPanel, type CommentsPanelProps, type DetailContext, type ExplainSection, type ExtensionTarget, type GHDetail, type GHItem, HealthPanel, type HealthPanelProps, type InboxConfig, type InboxExtension, type JiraTransition, type RepoFilter, type RepoHeader, type RepoProfile, type Section, type ShowLess, type ShowMore, type Standing, type SubgroupHeader, type TaskRow, buildActions, buildCheckoutCmd, checkoutDirs, clipboard, configureInbox, drillCmd, explainItem, filterByOrigin, filterByRepos, filterBySearch, fitCount, healthColor, healthDisplay, healthGlyph, healthLegend, inboxConfig, insertRepoHeaders, itermRun, jumpToRepo, jumpToRepoPane, layoutGHItems, matchesFilter, maxViewStart, moveCursor, openInTab, parsePatterns, profileOf, readCache, relativeTime, renderMarkdown, repoPriority, reposInSections, resetInboxConfig, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sortByRecency, sortItems, toCiStatusState, topLevelCount, truncate, useActionMenu, whoseMove, windowCount, withHeaders, withoutItem, writeCache };
+export { type Action, ActionMenu, type AnyItem, App, COLS, type CiStatus, CiStatusLine, type CiStatusState, CommentsPanel, type CommentsPanelProps, type DetailContext, type ExplainSection, type ExtensionTarget, type GHDetail, type GHItem, HealthPanel, type HealthPanelProps, type InboxConfig, type InboxExtension, type JiraTransition, type OriginSplit, type RepoFilter, type RepoHeader, type RepoProfile, type Section, type ShowLess, type ShowMore, type Standing, type SubgroupHeader, type TaskRow, buildActions, buildCheckoutCmd, checkoutDirs, clipboard, configureInbox, drillCmd, explainItem, filterByOrigin, filterByRepos, filterBySearch, fitCount, healthColor, healthDisplay, healthGlyph, healthLegend, inboxConfig, insertRepoHeaders, itermRun, jumpToRepo, jumpToRepoPane, layoutGHItems, matchesFilter, maxViewStart, moveCursor, openInTab, parsePatterns, profileOf, readCache, relativeTime, renderMarkdown, repeatedTaskRows, repoPriority, reposInSections, resetInboxConfig, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, signatureOf, sortByRecency, sortItems, toCiStatusState, topLevelCount, truncate, useActionMenu, whoseMove, windowCount, withHeaders, withoutItem, writeCache };

package/dist/index.js

@@ -945,9 +945,9 @@ var layoutGHItems = (items, sectionId) => {
     ...insertRepoHeaders(rows)
   ]);
 };
-var filterByOrigin = (sections, keep, isWorkRepo) => sections.map((s) => {
+var filterByOrigin = (sections, keep, match) => sections.map((s) => {
   const kept = s.items.filter(
-    (i) => i.kind !== "repo-header" && i.kind !== "subgroup-header" && i.kind !== "show-more" && i.kind !== "show-less" && (i.kind === "pr" || i.kind === "issue" ? keep === "work" ? isWorkRepo(i.repo) : !isWorkRepo(i.repo) : true)
+    (i) => i.kind !== "repo-header" && i.kind !== "subgroup-header" && i.kind !== "show-more" && i.kind !== "show-less" && (i.kind === "pr" || i.kind === "issue" ? keep === "matched" ? match(i.repo) : !match(i.repo) : true)
   );
   const gh = kept.filter(
     (i) => i.kind === "pr" || i.kind === "issue"
@@ -1384,6 +1384,23 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
       showFlash(`\u2713 Copied ${item.repo}`);
     }
   });
+  if (item.kind === "pr" && item.standing === "queued" && login) {
+    actions.push({
+      label: "Remove me as reviewer",
+      hint: "x",
+      run: () => {
+        onRemove?.(item);
+        showFlash(`\u22EF Removing you from #${item.number}\u2026`);
+        void quietly`gh pr edit ${item.number} --repo ${item.repo} --remove-reviewer ${login}`.then(() => {
+          showFlash(`\u2713 Removed you as reviewer on #${item.number}`);
+          ext?.onActed?.();
+        }).catch(() => {
+          showFlash(`\u2717 Could not remove you from #${item.number}`);
+          onRefresh?.();
+        });
+      }
+    });
+  }
   actions.push({
     label: "Unsubscribe",
     hint: "u",
@@ -1568,7 +1585,7 @@ var InboxHeader = ({
   sections,
   login,
   brand,
-  work,
+  scopeLabel,
   loading,
   quiet,
   refreshing,
@@ -1579,7 +1596,7 @@ var InboxHeader = ({
   const total = sections.reduce((n, s) => n + topLevelCount(s), 0);
   const countSeg = loading ? "      loading\u2026  " : quiet ? "  " : `  ${String(total).padStart(3)} item${total !== 1 ? "s" : ""}  \xB7  `;
   const userSeg = loading || quiet ? "" : `@${login}  `;
-  const workLabel = work === void 0 ? "" : `  ${work ? "work" : "home"}  `;
+  const workLabel = scopeLabel ? `  ${scopeLabel}  ` : "";
   const [statusText, statusColor] = hasPending ? [`\u25CF ${pendingSummary || "new"} \xB7 r apply`, "#FF8700"] : refreshing ? ["\u21BB refreshing\u2026", "cyan"] : fetchedAt ? [`updated ${agoText(fetchedAt)}`, void 0] : ["", void 0];
   const statusSeg = statusText ? statusText + "  " : "";
   const fill = Math.max(
@@ -1590,7 +1607,7 @@ var InboxHeader = ({
     /* @__PURE__ */ jsx(Text2, { color: "#FF8700", bold: true, children: brand }),
     /* @__PURE__ */ jsx(Text2, { dimColor: true, children: countSeg }),
     userSeg ? /* @__PURE__ */ jsx(Text2, { children: userSeg }) : null,
-    work !== void 0 ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: workLabel }) : null,
+    scopeLabel ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: workLabel }) : null,
     statusText ? /* @__PURE__ */ jsx(
       Text2,
       {
@@ -1668,6 +1685,17 @@ var TRANSIT_LABEL = {
 };
 var TAB_MARK = "\u25CF";
 var tabLabel = (label, marked, id) => marked.size === 0 ? label : `${marked.has(id) ? TAB_MARK : " "} ${label}`;
+var repeatedTaskRows = (items) => {
+  const seen = /* @__PURE__ */ new Set();
+  const repeats = /* @__PURE__ */ new Set();
+  items.forEach((row, i) => {
+    if (row.kind !== "task" || !row.url) return;
+    if (seen.has(row.url)) repeats.add(i);
+    else seen.add(row.url);
+  });
+  return repeats;
+};
+var isChildRow = (item) => !!item && (item.kind === "show-more" || item.kind === "show-less" || "indent" in item && item.indent === true);
 var ItemRow = ({
   item,
   active,
@@ -1675,6 +1703,9 @@ var ItemRow = ({
   login,
   merged,
   transient,
+  lastChild,
+  parent,
+  repeat,
   sparkFrame = 0
 }) => {
   if (item.kind === "repo-header")
@@ -1709,8 +1740,9 @@ var ItemRow = ({
     return /* @__PURE__ */ jsxs(Box, { marginTop: gap ? 1 : 0, children: [
       /* @__PURE__ */ jsx(Text2, { color: "cyan", children: active ? "\u276F " : "  " }),
       /* @__PURE__ */ jsx(Text2, { bold: true, color: transient ? TRANSIT_COLOUR[transient] : void 0, children: transitIcon + " " }),
-      /* @__PURE__ */ jsx(Text2, { color: "#FF8700", bold: active, children: item.key + "  " }),
-      /* @__PURE__ */ jsx(
+      /* @__PURE__ */ jsx(Text2, { dimColor: true, children: parent ? "\u252C " : "  " }),
+      /* @__PURE__ */ jsx(Text2, { color: repeat ? void 0 : "#FF8700", dimColor: repeat, bold: active, children: item.key + "  " }),
+      repeat ? null : /* @__PURE__ */ jsx(
         Text2,
         {
           bold: active || transient === "in",
@@ -1719,7 +1751,7 @@ var ItemRow = ({
           children: truncate(item.summary, titleMax2)
         }
       ),
-      note ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: ` ${note}` }) : null,
+      note && !repeat ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: ` ${note}` }) : null,
       transitLabel2 ? /* @__PURE__ */ jsx(Text2, { bold: true, color: TRANSIT_COLOUR[transient], children: "  " + transitLabel2 }) : null
     ] });
   }
@@ -1746,7 +1778,7 @@ var ItemRow = ({
   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: "\u2514\u2500 " }) : null,
+    item.indent ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: lastChild === false ? "\u251C\u2500 " : "\u2514\u2500 " }) : null,
     /* @__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 }),
@@ -1901,6 +1933,7 @@ var HelpModal = ({
     ["j", "open repo in new tab"],
     ["p", "open repo in new pane"],
     ["u", "unsubscribe from this item"],
+    ["x", "remove yourself as reviewer"],
     ...hasJira ? [["t", "Jira: move / open ticket"]] : [],
     ["/", "search"],
     ["f", "filter by repo"],
@@ -2089,15 +2122,14 @@ var BrowseScreen = ({
   ciStatusState,
   ciJob,
   tabHelp,
-  isWorkRepo,
-  includeWork,
+  origin,
   brand,
   mergedUrls,
   transients,
   onTabChange
 }) => {
   const { rows } = useWindowSize();
-  const applyWork = (secs) => includeWork === void 0 || !isWorkRepo ? secs : filterByOrigin(secs, includeWork ? "work" : "home", isWorkRepo);
+  const applyWork = (secs) => origin ? filterByOrigin(secs, origin.show, origin.match) : secs;
   const initialSections = applyWork(sections);
   const [localSections, setLocalSections] = useState(initialSections);
   const [tabIdx, setTabIdx] = useState(0);
@@ -2177,6 +2209,10 @@ var BrowseScreen = ({
     { vimKeys: false, isActive: repoPicker }
   );
   const cursor = cursors[activeId] ?? 0;
+  const repeatedTasks = useMemo(
+    () => repeatedTaskRows(section.items),
+    [section.items]
+  );
   const viewStart = viewStarts[activeId] ?? 0;
   const visibleCount = windowCount(section.items, viewStart, listHeight);
   const visibleItems = section.items.slice(viewStart, viewStart + visibleCount);
@@ -2423,6 +2459,18 @@ var BrowseScreen = ({
       void unsubscribeFrom(activeItem).then(() => showFlash(`\u2713 Unsubscribed from #${activeItem.number}`)).catch(() => showFlash(`\u2717 Unsubscribe failed for #${activeItem.number}`));
       return;
     }
+    if (input === "x" && activeItem.kind === "pr" && activeItem.standing === "queued" && login) {
+      removeItemFromSections(activeItem);
+      showFlash(`\u22EF Removing you from #${activeItem.number}\u2026`);
+      void quietly`gh pr edit ${activeItem.number} --repo ${activeItem.repo} --remove-reviewer ${login}`.then(() => {
+        showFlash(`\u2713 Removed you as reviewer on #${activeItem.number}`);
+        onActed?.();
+      }).catch(() => {
+        showFlash(`\u2717 Could not remove you from #${activeItem.number}`);
+        onRefresh?.();
+      });
+      return;
+    }
     if (input === "b" && activeItem.kind === "pr" && activeItem.branch) {
       clipboard(activeItem.branch);
       showFlash(`\u2713 Copied ${activeItem.branch}`);
@@ -2535,7 +2583,7 @@ var BrowseScreen = ({
             brand,
             sections: localSections,
             login,
-            work: includeWork,
+            scopeLabel: origin?.label,
             refreshing,
             hasPending,
             pendingSummary,
@@ -2574,6 +2622,9 @@ var BrowseScreen = ({
                 login,
                 merged: (item.kind === "pr" || item.kind === "issue") && !!mergedUrls?.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]),
+                repeat: repeatedTasks.has(viewStart + i),
                 sparkFrame,
                 gap: (
                   // Window-relative, never `viewStart + i`. fitCount prices the
@@ -2640,14 +2691,17 @@ var NoRowsScreen = ({
     ) })
   ] });
 };
-var signatureOf = (sections) => JSON.stringify(sections, (k, v) => k === "age" ? void 0 : v);
+var RELATIVE_TIME_KEYS = /* @__PURE__ */ new Set(["age", "activityAge"]);
+var signatureOf = (sections) => JSON.stringify(
+  sections,
+  (k, v) => RELATIVE_TIME_KEYS.has(k) ? void 0 : v
+);
 var App = ({
   fetcher,
   cacheKey,
   title = "inbox",
   detailFor,
-  isWorkRepo,
-  includeWork,
+  origin,
   jiraBase,
   jiraKeyRe,
   jiraTransitions,
@@ -2762,7 +2816,13 @@ var App = ({
         }
         showData(fresh.sections, fresh.login);
       } else if (freshKey !== displayedKey.current) {
-        setPending(fresh);
+        const { counts } = diffSections(
+          displayedSections.current,
+          fresh.sections
+        );
+        if (counts.added + counts.removed + counts.changed === 0)
+          showData(fresh.sections, fresh.login);
+        else setPending(fresh);
       } else {
         setPending(null);
       }
@@ -2859,7 +2919,7 @@ var App = ({
               brand: brandOf(title),
               sections: [],
               login: "",
-              work: state.phase === "loading" ? includeWork : void 0,
+              scopeLabel: state.phase === "loading" ? origin?.label : void 0,
               loading: state.phase === "loading",
               quiet: state.phase !== "loading"
             }
@@ -2918,8 +2978,7 @@ var App = ({
         pendingSummary,
         fetchedAt,
         refreshError: refreshError ?? void 0,
-        isWorkRepo,
-        includeWork,
+        origin,
         hidden: overlay !== null,
         mergedUrls,
         transients,
@@ -2976,4 +3035,4 @@ var matchesFilter = (repo, filter) => {
 };
 var parsePatterns = (value) => value.split(",").map((p) => p.trim()).filter(Boolean);
 
-export { ActionMenu, App, COLS, CiStatusLine, CommentsPanel, HealthPanel, buildActions, buildCheckoutCmd, checkoutDirs, clipboard, configureInbox, drillCmd, explainItem, filterByOrigin, filterByRepos, filterBySearch, fitCount, healthColor, healthDisplay, healthGlyph, healthLegend, inboxConfig, insertRepoHeaders, itermRun, jumpToRepo, jumpToRepoPane, layoutGHItems, matchesFilter, maxViewStart, moveCursor, openInTab, parsePatterns, profileOf, readCache, relativeTime, renderMarkdown, repoPriority, reposInSections, resetInboxConfig, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, sortByRecency, sortItems, toCiStatusState, topLevelCount, truncate, useActionMenu, whoseMove, windowCount, withHeaders, withoutItem, writeCache };
+export { ActionMenu, App, COLS, CiStatusLine, CommentsPanel, HealthPanel, buildActions, buildCheckoutCmd, checkoutDirs, clipboard, configureInbox, drillCmd, explainItem, filterByOrigin, filterByRepos, filterBySearch, fitCount, healthColor, healthDisplay, healthGlyph, healthLegend, inboxConfig, insertRepoHeaders, itermRun, jumpToRepo, jumpToRepoPane, layoutGHItems, matchesFilter, maxViewStart, moveCursor, openInTab, parsePatterns, profileOf, readCache, relativeTime, renderMarkdown, repeatedTaskRows, repoPriority, reposInSections, resetInboxConfig, resolveRepoPath, runHere, runInPane, runInPaneHorizontal, sameCiStatusState, signatureOf, sortByRecency, sortItems, toCiStatusState, topLevelCount, truncate, useActionMenu, whoseMove, windowCount, withHeaders, withoutItem, writeCache };

package/package.json

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