@kud/gh-ink 0.30.0 → 0.31.0

package/dist/index.js

@@ -1778,6 +1778,7 @@ var TRANSIT_LABEL = {
   out: "GONE",
   changed: "UPDATED"
 };
+var LEAVING_HOLD_MS = 2500;
 var TAB_MARK = "\u25CF";
 var tabLabel = (label, marked, id) => marked.size === 0 ? label : `${marked.has(id) ? TAB_MARK : " "} ${label}`;
 var ItemRow = ({
@@ -1786,11 +1787,13 @@ var ItemRow = ({
   gap,
   login,
   merged,
-  transient,
+  transient: refreshMark,
+  leaving,
   lastChild,
   parent,
   sparkFrame = 0
 }) => {
+  const transient = leaving ? "out" : refreshMark;
   if (item.kind === "repo-header")
     return /* @__PURE__ */ jsx(RepoHeaderRow, { repo: item.repo, gap: gap ?? false });
   if (item.kind === "subgroup-header")
@@ -2212,6 +2215,8 @@ var BrowseScreen = ({
   skippedForBudget,
   brand,
   mergedUrls,
+  leavingUrls,
+  onLeave,
   transients,
   onTabChange
 }) => {
@@ -2281,7 +2286,7 @@ var BrowseScreen = ({
       )
     );
   }, [localSections, listHeight]);
-  const removeItemFromSections = (target) => setLocalSections((prev) => withoutItem(prev, target));
+  const dismissItem = (target) => onLeave ? onLeave(target) : setLocalSections((prev) => withoutItem(prev, target));
   const rawSection = localSections[safeTabIdx] ?? {
     id: "empty",
     label: "",
@@ -2313,7 +2318,7 @@ var BrowseScreen = ({
     return out;
   }, [localSections, transients]);
   const [sparkFrame, setSparkFrame] = useState(0);
-  const sparkling = (mergedUrls?.length ?? 0) > 0 || !!transients?.size && section.items.some((item) => transientOf(transients, item));
+  const sparkling = (mergedUrls?.length ?? 0) > 0 || (leavingUrls?.length ?? 0) > 0 || !!transients?.size && section.items.some((item) => transientOf(transients, item));
   useEffect(() => {
     if (!sparkling) return;
     const id = setInterval(() => setSparkFrame((f) => f + 1), MERGED_FRAME_MS);
@@ -2346,7 +2351,7 @@ var BrowseScreen = ({
       jiraKeyRe,
       jiraTransitions,
       onRefresh,
-      removeItemFromSections,
+      dismissItem,
       openDrillView,
       { extensions, onOpenExt, onActed }
     );
@@ -2453,7 +2458,7 @@ var BrowseScreen = ({
         item: activeItem ?? void 0,
         ciJob: ciStatus?.job,
         login,
-        onRemove: (row) => removeItemFromSections(row),
+        onRemove: (row) => dismissItem(row),
         showFlash
       });
       return;
@@ -2562,7 +2567,7 @@ var BrowseScreen = ({
       return;
     }
     if (input === "x" && activeItem.kind === "pr" && activeItem.standing === "queued" && login) {
-      removeItemFromSections(activeItem);
+      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}`);
@@ -2729,6 +2734,7 @@ var BrowseScreen = ({
                 active: viewStart + i === cursor,
                 login,
                 merged: (item.kind === "pr" || item.kind === "issue") && !!mergedUrls?.includes(item.url),
+                leaving: (item.kind === "pr" || item.kind === "issue") && !!leavingUrls?.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]),
@@ -3043,8 +3049,9 @@ var App = ({
   );
   const [skippedForBudget, setSkippedForBudget] = useState(false);
   const [mergedUrls, setMergedUrls] = useState([]);
-  const mergeTimers = useRef([]);
-  useEffect(() => () => mergeTimers.current.forEach(clearTimeout), []);
+  const [leavingUrls, setLeavingUrls] = useState([]);
+  const holdTimers = useRef([]);
+  useEffect(() => () => holdTimers.current.forEach(clearTimeout), []);
   if (state.phase === "loading" || state.phase === "empty" || state.phase === "failed")
     return /* @__PURE__ */ jsxs(
       Box,
@@ -3092,17 +3099,26 @@ var App = ({
     target: state.target
   } : null;
   const toBrowse = () => setState({ phase: "browse", sections: state.sections, login: state.login });
-  const removeAndReturn = (target) => setState({
-    phase: "browse",
-    sections: withoutItem(state.sections, target),
-    login: state.login
-  });
+  const markLeaving = (target) => {
+    if (state.phase !== "browse") toBrowse();
+    setLeavingUrls(
+      (prev) => prev.includes(target.url) ? prev : [...prev, target.url]
+    );
+    holdTimers.current.push(
+      setTimeout(() => {
+        setLeavingUrls((prev) => prev.filter((u) => u !== target.url));
+        setState(
+          (s) => s.phase === "browse" ? { ...s, sections: withoutItem(s.sections, target) } : s
+        );
+      }, LEAVING_HOLD_MS)
+    );
+  };
   const markMerged = (target) => {
     toBrowse();
     setMergedUrls(
       (prev) => prev.includes(target.url) ? prev : [...prev, target.url]
     );
-    mergeTimers.current.push(
+    holdTimers.current.push(
       setTimeout(() => {
         setMergedUrls((prev) => prev.filter((u) => u !== target.url));
         setState(
@@ -3133,6 +3149,8 @@ var App = ({
         skippedForBudget,
         hidden: overlay !== null,
         mergedUrls,
+        leavingUrls,
+        onLeave: markLeaving,
         transients,
         onTabChange: setVisibleTab,
         ciStatusState: hasCiStatus ? ciStatusState : void 0,
@@ -3166,7 +3184,7 @@ var App = ({
       login: state.login,
       onBack: toBrowse,
       onRefresh: applyOrRefresh,
-      onRemove: removeAndReturn,
+      onRemove: markLeaving,
       onMerged: markMerged
     }),
     overlay?.kind === "ext" && extensions?.find((e) => e.id === overlay.extId)?.body(toBrowse, overlay.target)

package/package.json

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