@kud/gh-ink 0.45.0 → 0.45.2

package/dist/index.js

@@ -703,7 +703,7 @@ var healthLegend = [
 
 // src/inbox/cache.ts
 var cacheTtlMs = () => inboxConfig().cacheTtlMs;
-var CACHE_VERSION = 3;
+var CACHE_VERSION = 4;
 var cacheDir = () => join(
   process.env.XDG_CACHE_HOME || join(homedir(), ".cache"),
   inboxConfig().cacheNamespace
@@ -1786,14 +1786,20 @@ var RepoHeaderRow = ({
     /* @__PURE__ */ jsx(Text3, { dimColor: !active, children: " " + rule })
   ] });
 };
-var MERGED_HOLD_MS = 5e3;
+var MERGED_HOLD_MS = 3e3;
 var MERGED_FRAME_MS = 150;
 var MERGED_FRAMES = ["\u2726", "\u2727", "\u2736", "\u2727"];
 var MERGED_COLOUR = "#A371F7";
 var TRANSIT_HOLD_MS = 7e3;
-var TRANSIT_FRAME_TICKS = 3;
-var PULSE_SETTLE_MS = 6e4;
+var TRANSIT_FRAME_TICKS = 2;
+var TRANSIT_FRAME_MS = MERGED_FRAME_MS * TRANSIT_FRAME_TICKS;
+var rampFrame = (len, loopFrame, transitAt, now = Date.now()) => transitAt == null ? loopFrame % len : Math.max(
+  0,
+  Math.min(len - 1, Math.floor((now - transitAt) / TRANSIT_FRAME_MS))
+);
+var PULSE_SETTLE_MS = 12e3;
 var PULSE_MAX_FRAMES = Math.ceil(PULSE_SETTLE_MS / MERGED_FRAME_MS);
+var PULSE_IDLE_MS = 6e4;
 var PULSE_SETTLED_FRAME = 3;
 var NO_TRANSIENTS = /* @__PURE__ */ new Map();
 var NO_HOLDS = /* @__PURE__ */ new Map();
@@ -1825,6 +1831,7 @@ var ItemRow = ({
   merged,
   transient: refreshMark,
   leaving,
+  transitAt,
   prefix = "",
   parent,
   sparkFrame = 0,
@@ -1833,6 +1840,7 @@ var ItemRow = ({
   const backdropped = useBackdropped();
   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 });
@@ -1857,7 +1865,7 @@ var ItemRow = ({
     ] });
   if (item.kind === "task") {
     const note = item.note ?? "";
-    const transitIcon = !transient ? " " : isDeparture(transient) ? TRANSIT_OUT_FRAMES[transitFrame % TRANSIT_OUT_FRAMES.length] : isArrival(transient) ? TRANSIT_IN_FRAMES[transitFrame % TRANSIT_IN_FRAMES.length] : "\u25C9";
+    const transitIcon = !transient ? " " : isDeparture(transient) ? TRANSIT_OUT_FRAMES[rampAt(TRANSIT_OUT_FRAMES.length)] : isArrival(transient) ? TRANSIT_IN_FRAMES[rampAt(TRANSIT_IN_FRAMES.length)] : "\u25C9";
     const titleMax2 = Math.max(
       1,
       cols - item.key.length - note.length - (item.pill ? pillWidth(item.pill) + 1 : 0) - (farewellLabel ? pillWidth(farewellLabel) + 2 : 0) - prefix.length - 12
@@ -1889,7 +1897,7 @@ var ItemRow = ({
     ] });
   }
   const { glyph: healthIcon, color: healthColor2 } = healthDisplay[item.health];
-  const icon = merged ? MERGED_FRAMES[sparkFrame % MERGED_FRAMES.length] : transient && isDeparture(transient) ? TRANSIT_OUT_FRAMES[transitFrame % TRANSIT_OUT_FRAMES.length] : transient && isArrival(transient) ? TRANSIT_IN_FRAMES[transitFrame % TRANSIT_IN_FRAMES.length] : healthIcon;
+  const icon = merged ? MERGED_FRAMES[sparkFrame % MERGED_FRAMES.length] : transient && isDeparture(transient) ? TRANSIT_OUT_FRAMES[rampAt(TRANSIT_OUT_FRAMES.length)] : transient && isArrival(transient) ? TRANSIT_IN_FRAMES[rampAt(TRANSIT_IN_FRAMES.length)] : healthIcon;
   const color = merged ? MERGED_COLOUR : transient ? TRANSIT_COLOUR[transient] : healthColor2;
   const spokeLast = !!login && !!item.lastActor && item.lastActor === login;
   const [turnIcon, turnColor] = item.pinned ? [PIN_MARK, "#FF8700"] : !login || !item.lastActor ? [" ", "white"] : spokeLast ? ["\u2192", "#888888"] : ["\u2190", "#FF8700"];
@@ -2311,6 +2319,7 @@ var BrowseScreen = ({
   brand,
   mergedUrls,
   leavingUrls,
+  transitSince,
   onLeave,
   transients,
   onTabChange,
@@ -2414,7 +2423,7 @@ var BrowseScreen = ({
         out.add(s.id);
     return out;
   }, [localSections, transients]);
-  const [railOpen, setRailOpen] = useState(true);
+  const [railOpen, setRailOpen] = useState(false);
   const showRail = railOpen && !!sidebar;
   const [railFocus, setRailFocus] = useState(false);
   const [railCursor, setRailCursor] = useState(0);
@@ -2434,20 +2443,28 @@ var BrowseScreen = ({
     mergedUrls?.length ?? 0,
     leavingUrls?.length ?? 0
   ].join("|") : "";
+  const [pulseArm, setPulseArm] = useState(0);
+  const lastKeyAt = useRef(Date.now());
+  const pulseKey = sparkKey ? `${sparkKey}#${pulseArm}` : "";
+  const [pulseSettled, setPulseSettled] = useState(false);
   useEffect(() => {
-    if (!sparkKey) return;
+    if (!pulseKey) return;
+    setPulseSettled(false);
     let frames = 0;
     const id = setInterval(() => {
       frames += 1;
       if (frames >= PULSE_MAX_FRAMES) {
         clearInterval(id);
-        setSparkFrame(PULSE_SETTLED_FRAME);
+        setPulseSettled(true);
         return;
       }
       setSparkFrame((f) => f + 1);
     }, MERGED_FRAME_MS);
     return () => clearInterval(id);
-  }, [sparkKey]);
+  }, [pulseKey]);
+  useEffect(() => {
+    if (sparkKey) setPulseArm((n) => n + 1);
+  }, [tabIdx]);
   useEffect(() => {
     if (refreshError) showFlash(`\u2717 ${refreshError.message}`);
   }, [refreshError]);
@@ -2482,6 +2499,9 @@ var BrowseScreen = ({
     menu.open(actions);
   };
   useInput((input, key) => {
+    const idleFor = Date.now() - lastKeyAt.current;
+    lastKeyAt.current = Date.now();
+    if (idleFor >= PULSE_IDLE_MS && sparkKey) setPulseArm((n) => n + 1);
     if (hidden) return;
     if (key.ctrl || key.meta) return;
     if (help) {
@@ -2945,7 +2965,11 @@ var BrowseScreen = ({
               // bar — and free to animate for exactly that reason.
               // The ticker undivided — see TAB_PULSE. The row marks' /3 buys quiet
               // for motion sitting beside text, and this is not that.
-              marker: tabMarker(markedTabs, s.id, sparkFrame),
+              marker: tabMarker(
+                markedTabs,
+                s.id,
+                pulseSettled ? PULSE_SETTLED_FRAME : sparkFrame
+              ),
               markerColor: "#FF8700"
             }))
           }
@@ -2979,6 +3003,7 @@ var BrowseScreen = ({
                       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, section.id),
+                      transitAt: "url" in item ? transitSince?.get(item.url) : void 0,
                       prefix: treePrefix(section.items, viewStart + i),
                       parent: item.kind === "task" && depthOf(section.items[viewStart + i + 1]) > depthOf(item),
                       sparkFrame,
@@ -3316,8 +3341,24 @@ var App = ({
   const [skippedForBudget, setSkippedForBudget] = useState(false);
   const [mergedUrls, setMergedUrls] = useState([]);
   const [leavingUrls, setLeavingUrls] = useState([]);
+  const leavingSince = useRef(/* @__PURE__ */ new Map());
+  const transitSince = useMemo(() => {
+    const out = new Map(leavingSince.current);
+    for (const [url, tab] of transitTabs.current) {
+      const at = heldSince.get(tab);
+      if (at != null && !out.has(url)) out.set(url, at);
+    }
+    return out;
+  }, [heldSince, transients, leavingUrls]);
   const holdTimers = useRef([]);
   useEffect(() => () => holdTimers.current.forEach(clearTimeout), []);
+  const holdFor = (fn, ms) => {
+    const id = setTimeout(() => {
+      holdTimers.current = holdTimers.current.filter((t) => t !== id);
+      fn();
+    }, ms);
+    holdTimers.current.push(id);
+  };
   if (state.phase === "loading" || state.phase === "empty" || state.phase === "failed")
     return /* @__PURE__ */ jsxs(
       Box,
@@ -3367,31 +3408,30 @@ var App = ({
   const toBrowse = () => setState({ phase: "browse", sections: state.sections, login: state.login });
   const markLeaving = (target) => {
     if (state.phase !== "browse") toBrowse();
+    if (!leavingSince.current.has(target.url))
+      leavingSince.current.set(target.url, Date.now());
     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)
-    );
+    holdFor(() => {
+      leavingSince.current.delete(target.url);
+      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]
     );
-    holdTimers.current.push(
-      setTimeout(() => {
-        setMergedUrls((prev) => prev.filter((u) => u !== target.url));
-        setState(
-          (s) => s.phase === "browse" ? { ...s, sections: withoutItem(s.sections, target) } : s
-        );
-      }, MERGED_HOLD_MS)
-    );
+    holdFor(() => {
+      setMergedUrls((prev) => prev.filter((u) => u !== target.url));
+      setState(
+        (s) => s.phase === "browse" ? { ...s, sections: withoutItem(s.sections, target) } : s
+      );
+    }, MERGED_HOLD_MS);
   };
   return /* @__PURE__ */ jsxs(Fragment, { children: [
     /* @__PURE__ */ jsx(
@@ -3417,6 +3457,7 @@ var App = ({
         hidden: overlay !== null,
         mergedUrls,
         leavingUrls,
+        transitSince,
         onLeave: markLeaving,
         transients,
         onTabChange: setVisibleTab,

package/package.json

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