@ondrej-svec/hog 1.7.0 → 1.7.2

package/dist/cli.js

@@ -1317,6 +1317,36 @@ async function triggerCompletionActionAsync(action, repoName, issueNumber) {
       break;
   }
 }
+function applyBulkOptimisticStatusUpdates(ids, optionId, repos, config2, mutateData, registerPendingMutation) {
+  for (const id of ids) {
+    const ctx = findIssueContext(repos, id, config2);
+    if (!(ctx.issue && ctx.repoName)) continue;
+    const { issue: ctxIssue, repoName: ctxRepo, statusOptions: ctxOpts } = ctx;
+    mutateData((data) => optimisticSetStatus(data, ctxRepo, ctxIssue.number, ctxOpts, optionId));
+    const ctxStatusName = ctxOpts.find((o) => o.id === optionId)?.name;
+    if (ctxStatusName) {
+      registerPendingMutation?.(ctxRepo, ctxIssue.number, { projectStatus: ctxStatusName });
+    }
+  }
+}
+function resolveOptionName(repos, ids, config2, optionId) {
+  for (const id of ids) {
+    const name = findIssueContext(repos, id, config2).statusOptions.find(
+      (o) => o.id === optionId
+    )?.name;
+    if (name) return name;
+  }
+  return optionId;
+}
+function clearFailedMutations(failedIds, clearFn) {
+  if (!clearFn) return;
+  for (const failedId of failedIds) {
+    const lastColon = failedId.lastIndexOf(":");
+    const failedRepo = failedId.slice(3, lastColon);
+    const failedIssueNumber = parseInt(failedId.slice(lastColon + 1), 10);
+    clearFn(failedRepo, failedIssueNumber);
+  }
+}
 function optimisticSetStatus(data, repoName, issueNumber, statusOptions, optionId) {
   const statusName = statusOptions.find((o) => o.id === optionId)?.name;
   if (!statusName) return data;
@@ -1341,16 +1371,22 @@ function useActions({
   refresh,
   mutateData,
   onOverlayDone,
-  pushEntry
+  pushEntry,
+  registerPendingMutation,
+  clearPendingMutation
 }) {
   const configRef = useRef2(config2);
   const reposRef = useRef2(repos);
   const selectedIdRef = useRef2(selectedId);
   const pushEntryRef = useRef2(pushEntry);
+  const registerPendingMutationRef = useRef2(registerPendingMutation);
+  const clearPendingMutationRef = useRef2(clearPendingMutation);
   configRef.current = config2;
   reposRef.current = repos;
   selectedIdRef.current = selectedId;
   pushEntryRef.current = pushEntry;
+  registerPendingMutationRef.current = registerPendingMutation;
+  clearPendingMutationRef.current = clearPendingMutation;
   const handlePick = useCallback2(() => {
     const ctx = findIssueContext(reposRef.current, selectedIdRef.current, configRef.current);
     if (!(ctx.issue && ctx.repoConfig)) return;
@@ -1433,6 +1469,12 @@ function useActions({
       mutateData(
         (data) => optimisticSetStatus(data, repoName, issue.number, statusOptions, optionId)
       );
+      const statusName = statusOptions.find((o) => o.id === optionId)?.name;
+      if (statusName) {
+        registerPendingMutationRef.current?.(repoName, issue.number, {
+          projectStatus: statusName
+        });
+      }
       const t = toast.loading("Moving...");
       const projectConfig = {
         projectNumber: repoConfig.projectNumber,
@@ -1472,6 +1514,7 @@ function useActions({
           status: "error",
           ago: Date.now()
         });
+        clearPendingMutationRef.current?.(repoName, issue.number);
         refresh();
       }).finally(() => {
         onOverlayDone();
@@ -1694,15 +1737,14 @@ ${dueLine}` : dueLine;
   );
   const handleBulkStatusChange = useCallback2(
     async (ids, optionId) => {
-      for (const id of ids) {
-        const ctx = findIssueContext(reposRef.current, id, configRef.current);
-        if (ctx.issue && ctx.repoName) {
-          const { issue: ctxIssue, repoName: ctxRepo, statusOptions: ctxOpts } = ctx;
-          mutateData(
-            (data) => optimisticSetStatus(data, ctxRepo, ctxIssue.number, ctxOpts, optionId)
-          );
-        }
-      }
+      applyBulkOptimisticStatusUpdates(
+        ids,
+        optionId,
+        reposRef.current,
+        configRef.current,
+        mutateData,
+        registerPendingMutationRef.current
+      );
       const t = toast.loading(`Moving ${ids.size} issue${ids.size > 1 ? "s" : ""}...`);
       const failed = [];
       for (const id of ids) {
@@ -1724,18 +1766,12 @@ ${dueLine}` : dueLine;
       }
       const total = ids.size;
       const ok = total - failed.length;
-      const optionName = (() => {
-        for (const id of ids) {
-          const ctx = findIssueContext(reposRef.current, id, configRef.current);
-          const name = ctx.statusOptions.find((o) => o.id === optionId)?.name;
-          if (name) return name;
-        }
-        return optionId;
-      })();
+      const optionName = resolveOptionName(reposRef.current, ids, configRef.current, optionId);
       if (failed.length === 0) {
         t.resolve(`Moved ${total} issue${total > 1 ? "s" : ""} to ${optionName}`);
       } else {
         t.reject(`${ok} moved to ${optionName}, ${failed.length} failed`);
+        clearFailedMutations(failed, clearPendingMutationRef.current);
         refresh();
       }
       return failed;
@@ -1770,6 +1806,24 @@ var init_use_actions = __esm({
 // src/board/hooks/use-data.ts
 import { Worker } from "worker_threads";
 import { useCallback as useCallback3, useEffect, useRef as useRef3, useState as useState2 } from "react";
+function applyPendingMutations(data, pending) {
+  const now = Date.now();
+  for (const [key, m] of pending) {
+    if (m.expiresAt <= now) pending.delete(key);
+  }
+  if (pending.size === 0) return data;
+  return {
+    ...data,
+    repos: data.repos.map((rd) => ({
+      ...rd,
+      issues: rd.issues.map((issue) => {
+        const mutation = pending.get(`${rd.repo.name}:${issue.number}`);
+        if (!mutation || mutation.expiresAt <= now) return issue;
+        return mutation.projectStatus !== void 0 ? { ...issue, projectStatus: mutation.projectStatus } : issue;
+      })
+    }))
+  };
+}
 function refreshAgeColor(lastRefresh) {
   if (!lastRefresh) return "gray";
   const age = Date.now() - lastRefresh.getTime();
@@ -1782,18 +1836,21 @@ function useData(config2, options, refreshIntervalMs) {
   const activeRequestRef = useRef3(null);
   const workerRef = useRef3(null);
   const intervalRef = useRef3(null);
+  const pendingMutationsRef = useRef3(/* @__PURE__ */ new Map());
   const configRef = useRef3(config2);
   const optionsRef = useRef3(options);
   configRef.current = config2;
   optionsRef.current = options;
-  const refresh = useCallback3(() => {
+  const refresh = useCallback3((silent = false) => {
     if (activeRequestRef.current) {
       activeRequestRef.current.canceled = true;
     }
     workerRef.current?.terminate();
     const token = { canceled: false };
     activeRequestRef.current = token;
-    setState((prev) => ({ ...prev, isRefreshing: true }));
+    if (!silent) {
+      setState((prev) => ({ ...prev, isRefreshing: true }));
+    }
     const worker = new Worker(
       new URL(
         import.meta.url.endsWith(".ts") ? "../fetch-worker.ts" : "./fetch-worker.js",
@@ -1809,11 +1866,12 @@ function useData(config2, options, refreshIntervalMs) {
         return;
       }
       if (msg.type === "success" && msg.data) {
-        const data = msg.data;
-        data.fetchedAt = new Date(data.fetchedAt);
-        for (const ev of data.activity) {
+        const raw = msg.data;
+        raw.fetchedAt = new Date(raw.fetchedAt);
+        for (const ev of raw.activity) {
           ev.timestamp = new Date(ev.timestamp);
         }
+        const data = applyPendingMutations(raw, pendingMutationsRef.current);
         setState({
           status: "success",
           data,
@@ -1862,7 +1920,7 @@ function useData(config2, options, refreshIntervalMs) {
     if (refreshIntervalMs <= 0) return;
     intervalRef.current = setInterval(() => {
       if (!stateRef.current.autoRefreshPaused) {
-        refresh();
+        refresh(true);
       }
     }, refreshIntervalMs);
     return () => {
@@ -1891,7 +1949,27 @@ function useData(config2, options, refreshIntervalMs) {
   const resumeAutoRefresh = useCallback3(() => {
     setState((prev) => ({ ...prev, autoRefreshPaused: false }));
   }, []);
-  return { ...state, refresh, mutateData, pauseAutoRefresh, resumeAutoRefresh };
+  const registerPendingMutation = useCallback3(
+    (repoName, issueNumber, fields, ttlMs = 9e4) => {
+      pendingMutationsRef.current.set(`${repoName}:${issueNumber}`, {
+        ...fields,
+        expiresAt: Date.now() + ttlMs
+      });
+    },
+    []
+  );
+  const clearPendingMutation = useCallback3((repoName, issueNumber) => {
+    pendingMutationsRef.current.delete(`${repoName}:${issueNumber}`);
+  }, []);
+  return {
+    ...state,
+    refresh,
+    mutateData,
+    pauseAutoRefresh,
+    resumeAutoRefresh,
+    registerPendingMutation,
+    clearPendingMutation
+  };
 }
 var INITIAL_STATE, STALE_THRESHOLDS, MAX_REFRESH_FAILURES;
 var init_use_data = __esm({
@@ -2271,7 +2349,7 @@ function navReducer(state, action) {
       const collapsedSections = isFirstLoad ? new Set(sections.filter((s) => s === "activity")) : state.collapsedSections;
       const selectionValid = state.selectedId != null && action.items.some((i) => i.id === state.selectedId);
       if (!isFirstLoad && selectionValid && arraysEqual(sections, state.sections)) {
-        return state;
+        return state.allItems === action.items ? state : { ...state, allItems: action.items };
       }
       if (selectionValid) {
         const selected = action.items.find((i) => i.id === state.selectedId);
@@ -5197,7 +5275,9 @@ function Dashboard({ config: config2, options, activeProfile }) {
     refresh,
     mutateData,
     pauseAutoRefresh,
-    resumeAutoRefresh
+    resumeAutoRefresh,
+    registerPendingMutation,
+    clearPendingMutation
   } = useData(config2, options, refreshMs);
   const allRepos = useMemo3(() => data?.repos ?? [], [data?.repos]);
   const allTasks = useMemo3(
@@ -5268,7 +5348,9 @@ function Dashboard({ config: config2, options, activeProfile }) {
     refresh,
     mutateData,
     onOverlayDone: ui.exitOverlay,
-    pushEntry
+    pushEntry,
+    registerPendingMutation,
+    clearPendingMutation
   });
   const pendingPickRef = useRef13(null);
   const labelCacheRef = useRef13({});
@@ -6916,7 +6998,7 @@ function resolveProjectId(projectId) {
   process.exit(1);
 }
 var program = new Command();
-program.name("hog").description("Personal command deck \u2014 unified task dashboard for GitHub Projects + TickTick").version("1.7.0").option("--json", "Force JSON output").option("--human", "Force human-readable output").hook("preAction", (thisCommand) => {
+program.name("hog").description("Personal command deck \u2014 unified task dashboard for GitHub Projects + TickTick").version("1.7.2").option("--json", "Force JSON output").option("--human", "Force human-readable output").hook("preAction", (thisCommand) => {
   const opts = thisCommand.opts();
   if (opts.json) setFormat("json");
   if (opts.human) setFormat("human");