@kud/gh-ink 0.14.0 → 0.16.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;
@@ -1575,6 +1584,8 @@ var TRANSIT_LABEL = {
   out: "GONE",
   changed: "UPDATED"
 };
+var TAB_MARK = "\u25CF";
+var tabLabel = (label, marked, id) => marked.size === 0 ? label : `${marked.has(id) ? TAB_MARK : " "} ${label}`;
 var ItemRow = ({
   item,
   active,
@@ -1607,12 +1618,27 @@ var ItemRow = ({
     ] });
   if (item.kind === "task") {
     const note = item.note ?? "";
-    const titleMax2 = Math.max(20, COLS - item.key.length - note.length - 10);
+    const transitIcon = !transient ? " " : transient === "out" ? TRANSIT_OUT_FRAMES[sparkFrame % TRANSIT_OUT_FRAMES.length] : transient === "in" ? TRANSIT_IN_FRAMES[sparkFrame % TRANSIT_IN_FRAMES.length] : "\u25C9";
+    const transitLabel2 = transient ? TRANSIT_LABEL[transient] : "";
+    const titleMax2 = Math.max(
+      20,
+      COLS - item.key.length - note.length - transitLabel2.length - 12
+    );
     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(Text2, { bold: active, children: truncate(item.summary, titleMax2) }),
-      note ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: ` ${note}` }) : null
+      /* @__PURE__ */ jsx(
+        Text2,
+        {
+          bold: active || transient === "in",
+          dimColor: transient === "out",
+          strikethrough: transient === "out",
+          children: truncate(item.summary, titleMax2)
+        }
+      ),
+      note ? /* @__PURE__ */ jsx(Text2, { dimColor: true, children: ` ${note}` }) : null,
+      transitLabel2 ? /* @__PURE__ */ jsx(Text2, { bold: true, color: TRANSIT_COLOUR[transient], children: "  " + transitLabel2 }) : null
     ] });
   }
   const { glyph: healthIcon, color: healthColor2 } = healthDisplay[item.health];
@@ -1947,7 +1973,8 @@ var BrowseScreen = ({
   initialIncludeWork,
   brand,
   mergedUrls,
-  transients
+  transients,
+  onTabChange
 }) => {
   const { rows } = useWindowSize();
   const [includeWork, setIncludeWork] = useState(
@@ -1963,6 +1990,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);
@@ -2037,8 +2067,15 @@ var BrowseScreen = ({
     setFlash(msg);
     setTimeout(() => setFlash(null), 2e3);
   };
+  const markedTabs = useMemo(() => {
+    const out = /* @__PURE__ */ new Set();
+    if (!transients?.size) return out;
+    for (const s of localSections)
+      if (s.items.some((item) => transientOf(transients, item))) out.add(s.id);
+    return out;
+  }, [localSections, transients]);
   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);
@@ -2387,7 +2424,7 @@ var BrowseScreen = ({
             active: section.id,
             items: localSections.map((s) => ({
               value: s.id,
-              label: s.label,
+              label: tabLabel(s.label, markedTabs, s.id),
               count: topLevelCount(s)
             }))
           }
@@ -2515,13 +2552,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 +2561,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 +2800,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.16.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",