@kud/gh-ink 0.14.0 → 0.15.0

package/dist/index.js

@@ -607,7 +607,7 @@ var healthLegend = [
   ["closed", "Closed"]
 ];
 var CACHE_TTL_MS = 12e4;
-var CACHE_VERSION = 1;
+var CACHE_VERSION = 2;
 var cacheDir = () => join(process.env.XDG_CACHE_HOME || join(homedir(), ".cache"), "ambre");
 var cacheFile = (key) => join(cacheDir(), `${key.replace(/[^a-z0-9._-]/gi, "-")}.json`);
 var readCache = (key) => {
@@ -733,6 +733,15 @@ var transientOf = (transients, item) => {
   const key = keyOf(item);
   return key ? transients.get(key) : void 0;
 };
+var tabsOfMarks = (sections, marks) => {
+  const out = /* @__PURE__ */ new Map();
+  for (const section of sections)
+    for (const item of section.items) {
+      const key = keyOf(item);
+      if (key && marks.has(key)) out.set(key, section.id);
+    }
+  return out;
+};
 var quietly = $({ quiet: true });
 var relativeTime = (iso) => {
   const diff = (Date.now() - new Date(iso).getTime()) / 1e3;
@@ -1947,7 +1956,8 @@ var BrowseScreen = ({
   initialIncludeWork,
   brand,
   mergedUrls,
-  transients
+  transients,
+  onTabChange
 }) => {
   const { rows } = useWindowSize();
   const [includeWork, setIncludeWork] = useState(
@@ -1963,6 +1973,9 @@ var BrowseScreen = ({
   const [viewStarts, setViewStarts] = useState({});
   const safeTabIdx = Math.min(tabIdx, Math.max(0, localSections.length - 1));
   const activeId = localSections[safeTabIdx]?.id ?? "";
+  useEffect(() => {
+    onTabChange?.(activeId);
+  }, [activeId]);
   const [flash, setFlash] = useState(null);
   const menu = useActionMenu();
   const [search, setSearch] = useState(null);
@@ -2038,7 +2051,7 @@ var BrowseScreen = ({
     setTimeout(() => setFlash(null), 2e3);
   };
   const [sparkFrame, setSparkFrame] = useState(0);
-  const sparkling = (mergedUrls?.length ?? 0) > 0 || (transients?.size ?? 0) > 0;
+  const sparkling = (mergedUrls?.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);
@@ -2515,13 +2528,8 @@ var App = ({
   const displayedKey = useRef("");
   const displayedSections = useRef([]);
   const [transients, setTransients] = useState(NO_TRANSIENTS);
-  const transitTimer = useRef(null);
-  useEffect(
-    () => () => {
-      if (transitTimer.current) clearTimeout(transitTimer.current);
-    },
-    []
-  );
+  const transitTabs = useRef(/* @__PURE__ */ new Map());
+  const [visibleTab, setVisibleTab] = useState("");
   const showData = (sections, login) => {
     const before = displayedSections.current;
     displayedSections.current = sections;
@@ -2529,25 +2537,61 @@ var App = ({
     setPending(null);
     setFetchedAt(Date.now());
     if (before.length === 0) {
+      transitTabs.current = /* @__PURE__ */ new Map();
       setTransients(NO_TRANSIENTS);
       setState({ phase: "browse", sections, login });
       return;
     }
-    const { transients: marks, union } = diffSections(before, sections);
+    const { transients: fresh, union } = diffSections(before, sections);
+    const marks = new Map(fresh);
+    for (const [key, mark] of transients)
+      if (!marks.has(key)) marks.set(key, mark);
+    const tabs = tabsOfMarks(union, marks);
+    for (const key of [...marks.keys()]) if (!tabs.has(key)) marks.delete(key);
     if (marks.size === 0) {
+      transitTabs.current = /* @__PURE__ */ new Map();
+      setTransients(NO_TRANSIENTS);
       setState({ phase: "browse", sections, login });
       return;
     }
+    transitTabs.current = tabs;
     setTransients(marks);
     setState({ phase: "browse", sections: union, login });
-    if (transitTimer.current) clearTimeout(transitTimer.current);
-    transitTimer.current = setTimeout(() => {
-      setTransients(NO_TRANSIENTS);
-      setState(
-        (prev) => prev.phase === "browse" && prev.sections === union ? { ...prev, sections } : prev
-      );
-    }, TRANSIT_HOLD_MS);
   };
+  const settleTab = (tabId, marks) => {
+    const keys = [...transitTabs.current].filter(([, tab]) => tab === tabId).map(([key]) => key);
+    for (const key of keys) transitTabs.current.delete(key);
+    const gone = new Set(keys.filter((key) => marks.get(key) === "out"));
+    setTransients((prev) => {
+      const next = new Map(prev);
+      for (const key of keys) next.delete(key);
+      return next.size === 0 ? NO_TRANSIENTS : next;
+    });
+    setState(
+      (prev) => prev.phase !== "browse" ? prev : {
+        ...prev,
+        sections: prev.sections.map(
+          (section) => section.id !== tabId ? section : {
+            ...section,
+            items: section.items.filter((item) => {
+              const key = keyOf(item);
+              return !(key && gone.has(key));
+            })
+          }
+        )
+      }
+    );
+  };
+  useEffect(() => {
+    if (transients.size === 0) return;
+    if (state.phase !== "browse" || !visibleTab) return;
+    if (![...transitTabs.current.values()].includes(visibleTab)) return;
+    const timer = setTimeout(
+      () => settleTab(visibleTab, transients),
+      TRANSIT_HOLD_MS
+    );
+    return () => clearTimeout(timer);
+  }, [transients, visibleTab, state.phase]);
   const pendingSummary = useMemo(
     () => pending ? summariseDiff(
       diffSections(displayedSections.current, pending.sections).counts
@@ -2732,6 +2776,7 @@ var App = ({
         hidden: overlay !== null,
         mergedUrls,
         transients,
+        onTabChange: setVisibleTab,
         ciStatusState: hasCiStatus ? ciStatusState : void 0,
         ciJob,
         tabHelp,

package/package.json

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