@kud/gh-ink 0.30.0 → 0.32.0

package/dist/index.d.ts

@@ -124,6 +124,22 @@ type GHItem = {
      * never about the PR. Left unset, the section id decides as before.
      */
     standing?: Standing;
+    /**
+     * A turn the viewer has claimed BY HAND, outranking everything inferred.
+     *
+     * The inference below is good and still gets things wrong in one direction it
+     * cannot see: a row can be genuinely yours while every signal says otherwise —
+     * nothing red, no unresolved thread, nobody waiting on a word from you — and
+     * no amount of reading the PR harder will discover it. That knowledge lives
+     * with the viewer, so this is the field where they put it.
+     *
+     * It is a PIN, not a correction: it applies whether or not the row was already
+     * yours, which is why it does not pair with an opposite. The other direction —
+     * "no reply is owed for this particular comment" — is a fact about one event
+     * and belongs where the events are read, not here; a host expresses it by
+     * handing back a `lastActor` that no longer claims a turn.
+     */
+    pinned?: boolean;
     indent: boolean;
 };
 type TaskRow = {
@@ -240,7 +256,7 @@ declare const sortItems: (items: GHItem[]) => GHItem[];
 declare const sortByRecency: (items: GHItem[]) => GHItem[];
 declare const insertRepoHeaders: (items: GHItem[]) => AnyItem[];
 type Standing = "authored" | "queued" | "spoken";
-declare const whoseMove: (health: Health, sectionId: string, standing?: Standing, theySpokeLast?: boolean) => "you" | "them";
+declare const whoseMove: (health: Health, sectionId: string, standing?: Standing, theySpokeLast?: boolean, pinned?: boolean) => "you" | "them";
 declare const layoutGHItems: (items: GHItem[], sectionId: string, login?: string) => AnyItem[];
 /**
  * A two-way repo split, when a host has one. `undefined` — the ordinary case —

package/dist/index.js

@@ -838,6 +838,10 @@ var checksSentence = (d) => {
   return `Checks: ${parts.join(", ")}.`;
 };
 var turnSentences = (item, login) => {
+  if (item.pinned)
+    return [
+      "You pinned this (!). It stays under Your move until you unpin it, whatever else the row says."
+    ];
   if (!item.lastActor) return ["Nothing has been said on it yet."];
   const d = item.detail;
   const when = d?.lastEventAt ? `${relativeTime(d.lastEventAt)} ago` : "earlier";
@@ -937,8 +941,9 @@ var YOURS = {
   queued: ["waiting", "pending", "threads", "approved"],
   spoken: ["threads"]
 };
-var whoseMove = (health, sectionId, standing, theySpokeLast) => {
+var whoseMove = (health, sectionId, standing, theySpokeLast, pinned) => {
   const position = standing ?? STANDING[sectionId] ?? "queued";
+  if (pinned) return "you";
   if (theySpokeLast && position === "authored") return "you";
   return YOURS[position].includes(health) ? "you" : "them";
 };
@@ -956,7 +961,8 @@ var layoutGHItems = (items, sectionId, login) => {
         i.health,
         sectionId,
         i.standing,
-        !!login && !!i.lastActor && i.lastActor !== login
+        !!login && !!i.lastActor && i.lastActor !== login,
+        i.pinned
       ) === side
     )
   }));
@@ -1778,6 +1784,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 +1793,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")
@@ -1842,7 +1851,7 @@ var ItemRow = ({
   const icon = merged ? MERGED_FRAMES[sparkFrame % MERGED_FRAMES.length] : transient === "out" ? TRANSIT_OUT_FRAMES[sparkFrame % TRANSIT_OUT_FRAMES.length] : transient === "in" ? TRANSIT_IN_FRAMES[sparkFrame % 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] = !login || !item.lastActor ? [" ", "white"] : spokeLast ? ["\u2192", "#888888"] : ["\u2190", "#FF8700"];
+  const [turnIcon, turnColor] = item.pinned ? ["!", "#FF8700"] : !login || !item.lastActor ? [" ", "white"] : spokeLast ? ["\u2192", "#888888"] : ["\u2190", "#FF8700"];
   const numStr = `#${item.number}`.padEnd(7);
   const showAuthor = !!item.author && item.author !== login;
   const unresolvedLabel = item.unresolved > 0 ? `\uF086 ${item.unresolved}` : "";
@@ -2212,6 +2221,8 @@ var BrowseScreen = ({
   skippedForBudget,
   brand,
   mergedUrls,
+  leavingUrls,
+  onLeave,
   transients,
   onTabChange
 }) => {
@@ -2281,7 +2292,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 +2324,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 +2357,7 @@ var BrowseScreen = ({
       jiraKeyRe,
       jiraTransitions,
       onRefresh,
-      removeItemFromSections,
+      dismissItem,
       openDrillView,
       { extensions, onOpenExt, onActed }
     );
@@ -2453,7 +2464,7 @@ var BrowseScreen = ({
         item: activeItem ?? void 0,
         ciJob: ciStatus?.job,
         login,
-        onRemove: (row) => removeItemFromSections(row),
+        onRemove: (row) => dismissItem(row),
         showFlash
       });
       return;
@@ -2562,7 +2573,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 +2740,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 +3055,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 +3105,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 +3155,8 @@ var App = ({
         skippedForBudget,
         hidden: overlay !== null,
         mergedUrls,
+        leavingUrls,
+        onLeave: markLeaving,
         transients,
         onTabChange: setVisibleTab,
         ciStatusState: hasCiStatus ? ciStatusState : void 0,
@@ -3166,7 +3190,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.32.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",
@@ -48,7 +48,7 @@
     "react": ">=19"
   },
   "dependencies": {
-    "@kud/gh": "0.8.0",
+    "@kud/gh": "0.9.0",
     "@kud/ink-ui": "0.14.0",
     "zx": "8.8.5"
   },