@kud/gh-ink 0.62.1 → 0.63.0

package/dist/index.d.ts

@@ -486,6 +486,12 @@ type DetailContext = {
      */
     onTyping?: (typing: boolean) => void;
 };
+type SettledResult = {
+    ok: true;
+} | {
+    ok: false;
+    reason: string;
+};
 type Action = {
     label: string;
     hint: string;
@@ -624,6 +630,19 @@ declare const buildActions: (item: AnyItem, login: string, showFlash: (msg: stri
      * the drop stops the pre-action cache outliving a quit inside that window.
      */
     onActed?: () => void;
+    /**
+     * A mutation settled (succeeded or failed). Called exactly once per
+     * invocation of the three optimistic actions: "Remove me as reviewer",
+     * "Close issue" → "Close #N", and "Close PR" → "Close #N".
+     *
+     * On success: `{ ok: true }`. On failure: `{ ok: false; reason }` where
+     * `reason` is the first line of the error message/stderr, trimmed and
+     * non-empty.
+     *
+     * When absent, behaviour is identical — the host simply does not receive
+     * the settlement signal.
+     */
+    onSettled?: (item: GHItem, result: SettledResult) => void;
 }) => Action[];
 type CiStatusState = {
     kind: "loading";
@@ -665,7 +684,7 @@ declare const ActionMenu: ({ item, actions, cursor, }: {
     cursor: number;
 }) => React.JSX.Element;
 declare const signatureOf: (sections: Section[]) => string;
-declare const App: ({ fetcher, cacheKey, title, detailFor, origin, jiraBase, jiraKeyRe, jiraSetupHint, jiraTransitions, hasCiStatus, ciJob, ciFetcher, ciPollMs, stripFetcher, stripLabel, stripPollMs, watchPath, watchDebounceMs, watchJitterMs, extensions, tabHelp, emptyHint, liveLabel, }: {
+declare const App: ({ fetcher, cacheKey, title, detailFor, origin, jiraBase, jiraKeyRe, jiraSetupHint, jiraTransitions, hasCiStatus, ciJob, ciFetcher, ciPollMs, stripFetcher, stripLabel, stripPollMs, watchPath, watchDebounceMs, watchJitterMs, extensions, tabHelp, emptyHint, liveLabel, onSettled, }: {
     fetcher: () => Promise<{
         sections: Section[];
         login: string;
@@ -739,6 +758,19 @@ declare const App: ({ fetcher, cacheKey, title, detailFor, origin, jiraBase, jir
      */
     watchJitterMs?: number;
     extensions?: InboxExtension[];
+    /**
+     * A mutation settled (succeeded or failed). Called exactly once per
+     * invocation of the three optimistic actions: "Remove me as reviewer",
+     * "Close issue" → "Close #N", and "Close PR" → "Close #N".
+     *
+     * On success: `{ ok: true }`. On failure: `{ ok: false; reason }` where
+     * `reason` is the first line of the error message/stderr, trimmed and
+     * non-empty.
+     *
+     * When absent, behaviour is identical — the host simply does not receive
+     * the settlement signal.
+     */
+    onSettled?: (item: GHItem, result: SettledResult) => void;
 }) => React.JSX.Element;
 
 type RepoFilter = {
@@ -760,4 +792,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, 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, 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

@@ -1875,6 +1875,12 @@ var explainGhAction = (e) => {
   const human = GH_ACTION_FAILURES.find(([re]) => re.test(raw))?.[1];
   return human ? `${human} (${raw})` : raw;
 };
+var firstLine = (e) => {
+  const err = e;
+  const raw = (err.stderr ?? err.message ?? err.stdout ?? String(e)).trim();
+  const line = raw.split("\n").find((l) => l.trim().length > 0);
+  return (line?.trim() ?? "failed").slice(0, 200);
+};
 var unsubscribeFrom = async (item) => {
   const route = item.kind === "pr" ? "pulls" : "issues";
   const found = await quietly`gh api repos/${item.repo}/${route}/${item.number} --jq .node_id`;
@@ -1979,13 +1985,18 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
       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((e) => {
-          showFlash(`\u2717 #${item.number}: ${explainGhAction(e)}`);
-          onRefresh?.();
-        });
+        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?.();
+            ext?.onSettled?.(item, { ok: true });
+          },
+          (e) => {
+            showFlash(`\u2717 #${item.number}: ${explainGhAction(e)}`);
+            onRefresh?.();
+            ext?.onSettled?.(item, { ok: false, reason: firstLine(e) });
+          }
+        );
       }
     });
   }
@@ -2058,10 +2069,14 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
           run: () => {
             onRemove?.(item);
             showFlash(`\u2713 Closed #${item.number}`);
-            void quietly`gh issue close ${item.number} --repo ${item.repo}`.catch(
+            void quietly`gh issue close ${item.number} --repo ${item.repo}`.then(
               () => {
+                ext?.onSettled?.(item, { ok: true });
+              },
+              (e) => {
                 showFlash(`\u2717 Close failed \u2014 restoring #${item.number}`);
                 onRefresh?.();
+                ext?.onSettled?.(item, { ok: false, reason: firstLine(e) });
               }
             );
           }
@@ -2110,10 +2125,14 @@ var buildActions = (item, login, showFlash, jiraBase, jiraKeyRe, jiraTransitions
           run: () => {
             onRemove?.(item);
             showFlash(`\u2713 Closed #${item.number}`);
-            void quietly`gh pr close ${item.number} --repo ${item.repo}`.catch(
+            void quietly`gh pr close ${item.number} --repo ${item.repo}`.then(
               () => {
+                ext?.onSettled?.(item, { ok: true });
+              },
+              (e) => {
                 showFlash(`\u2717 Close failed \u2014 restoring #${item.number}`);
                 onRefresh?.();
+                ext?.onSettled?.(item, { ok: false, reason: firstLine(e) });
               }
             );
           }
@@ -2751,6 +2770,7 @@ var BrowseScreen = ({
   jiraTransitions,
   onRefresh,
   onActed,
+  onSettled,
   refreshing,
   hasPending,
   pendingSummary,
@@ -3029,7 +3049,7 @@ var BrowseScreen = ({
       onRefresh,
       dismissItem,
       openDrillView,
-      { extensions, onOpenExt, onActed }
+      { extensions, onOpenExt, onActed, onSettled }
     );
     menu.open(actions);
   };
@@ -3310,13 +3330,18 @@ var BrowseScreen = ({
     if (input === "x" && activeItem.kind === "pr" && activeItem.standing === "queued" && login) {
       dismissItem(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((e) => {
-        showFlash(`\u2717 #${activeItem.number}: ${explainGhAction(e)}`);
-        onRefresh?.();
-      });
+      void quietly`gh pr edit ${activeItem.number} --repo ${activeItem.repo} --remove-reviewer ${login}`.then(
+        () => {
+          showFlash(`\u2713 Removed you as reviewer on #${activeItem.number}`);
+          onActed?.();
+          onSettled?.(activeItem, { ok: true });
+        },
+        (e) => {
+          showFlash(`\u2717 #${activeItem.number}: ${explainGhAction(e)}`);
+          onRefresh?.();
+          onSettled?.(activeItem, { ok: false, reason: firstLine(e) });
+        }
+      );
       return;
     }
     if (input === "b" && activeItem.kind === "pr" && activeItem.branch) {
@@ -3744,7 +3769,8 @@ var App = ({
   extensions,
   tabHelp,
   emptyHint,
-  liveLabel
+  liveLabel,
+  onSettled
 }) => {
   const { rows } = useWindowSize();
   const [state, setState] = useState({ phase: "loading" });
@@ -4155,6 +4181,7 @@ var App = ({
         jiraTransitions,
         onRefresh: applyOrRefresh,
         onActed,
+        onSettled,
         refreshing,
         hasPending: pending !== null,
         pendingSummary,

package/package.json

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