gh-manager-cli 1.10.2 → 1.10.4

package/CHANGELOG.md

@@ -1,3 +1,17 @@
+## [1.10.4](https://github.com/wiiiimm/gh-manager-cli/compare/v1.10.3...v1.10.4) (2025-09-01)
+
+
+### Bug Fixes
+
+* eliminate duplicate sync handler to prevent double execution ([bf838b6](https://github.com/wiiiimm/gh-manager-cli/commit/bf838b66f5c8b7def294d3835278cd32254bce1f))
+
+## [1.10.3](https://github.com/wiiiimm/gh-manager-cli/compare/v1.10.2...v1.10.3) (2025-09-01)
+
+
+### Performance Improvements
+
+* optimize sync operation to update locally without API call ([b851c85](https://github.com/wiiiimm/gh-manager-cli/commit/b851c854a277b53f908e92d330622c4513642600))
+
 ## [1.10.2](https://github.com/wiiiimm/gh-manager-cli/compare/v1.10.1...v1.10.2) (2025-09-01)
 
 

package/dist/index.js

@@ -3,7 +3,6 @@ import {
   __commonJS,
   archiveRepositoryById,
   deleteRepositoryRest,
-  fetchRepositoryById,
   fetchViewerOrganizations,
   fetchViewerReposPageUnified,
   getRepositoryFromCache,
@@ -24,7 +23,7 @@ var require_package = __commonJS({
   "package.json"(exports, module) {
     module.exports = {
       name: "gh-manager-cli",
-      version: "1.10.2",
+      version: "1.10.4",
       private: false,
       description: "Interactive CLI to manage your GitHub repos (personal) with Ink",
       license: "MIT",
@@ -487,6 +486,7 @@ function RepoList({ token, maxVisibleRows, onLogout, viewerLogin, onOrgContextCh
   const [syncing, setSyncing] = useState3(false);
   const [syncError, setSyncError] = useState3(null);
   const [syncFocus, setSyncFocus] = useState3("confirm");
+  const [syncTrigger, setSyncTrigger] = useState3(false);
   const [infoMode, setInfoMode] = useState3(false);
   const [infoRepo, setInfoRepo] = useState3(null);
   const [logoutMode, setLogoutMode] = useState3(false);
@@ -505,6 +505,46 @@ function RepoList({ token, maxVisibleRows, onLogout, viewerLogin, onOrgContextCh
     setSyncing(false);
     setSyncError(null);
     setSyncFocus("confirm");
+    setSyncTrigger(false);
+  }
+  async function executeSync() {
+    if (!syncTarget || syncing) return;
+    try {
+      setSyncing(true);
+      const [owner, repo] = syncTarget.nameWithOwner.split("/");
+      const branchName = syncTarget.defaultBranchRef?.name || "main";
+      const result = await syncForkWithUpstream(token, owner, repo, branchName);
+      const updatedRepo = {
+        ...syncTarget,
+        updatedAt: (/* @__PURE__ */ new Date()).toISOString(),
+        // If we're tracking fork commits and this is a fork with parent data, set commits to be in sync
+        ...forkTracking && syncTarget.isFork && syncTarget.parent && syncTarget.defaultBranchRef?.target?.history && syncTarget.parent.defaultBranchRef?.target?.history ? {
+          defaultBranchRef: {
+            ...syncTarget.defaultBranchRef,
+            target: {
+              ...syncTarget.defaultBranchRef.target,
+              history: {
+                // Set fork's commit count equal to parent's (0 commits behind)
+                totalCount: syncTarget.parent.defaultBranchRef.target.history.totalCount
+              }
+            }
+          }
+        } : {}
+      };
+      await updateCacheWithRepository(token, updatedRepo);
+      const updateSyncedRepo = (r) => {
+        if (r.id === syncTarget.id) {
+          return updatedRepo;
+        }
+        return r;
+      };
+      setItems((prev) => prev.map(updateSyncedRepo));
+      setSearchItems((prev) => prev.map(updateSyncedRepo));
+      closeSyncModal();
+    } catch (e) {
+      setSyncing(false);
+      setSyncError(e.message || "Failed to sync fork. Check permissions and network.");
+    }
   }
   function handleOrgContextChange(newContext) {
     setOwnerContext(newContext);
@@ -861,59 +901,17 @@ function RepoList({ token, maxVisibleRows, onLogout, viewerLogin, onOrgContextCh
         setSyncFocus("cancel");
         return;
       }
-      if (key.return || input && input.toUpperCase() === "Y") {
+      if (input && input.toUpperCase() === "Y") {
         if (syncFocus === "cancel") {
           closeSyncModal();
-          return;
+        } else {
+          executeSync();
         }
-        if (!syncTarget) return;
-        (async () => {
-          try {
-            setSyncing(true);
-            const [owner, repo] = syncTarget.nameWithOwner.split("/");
-            const branchName = syncTarget.defaultBranchRef?.name || "main";
-            const result = await syncForkWithUpstream(token, owner, repo, branchName);
-            const updatedRepo = await fetchRepositoryById(client, syncTarget.id, forkTracking);
-            if (updatedRepo) {
-              await updateCacheWithRepository(token, updatedRepo);
-              const updateSyncedRepo = (r) => {
-                if (r.id === syncTarget.id) {
-                  return updatedRepo;
-                }
-                return r;
-              };
-              setItems((prev) => prev.map(updateSyncedRepo));
-              setSearchItems((prev) => prev.map(updateSyncedRepo));
-            } else {
-              const updateSyncedRepo = (r) => {
-                if (r.id === syncTarget.id && r.parent && r.defaultBranchRef?.target?.history && r.parent.defaultBranchRef?.target?.history) {
-                  return {
-                    ...r,
-                    defaultBranchRef: {
-                      ...r.defaultBranchRef,
-                      target: {
-                        ...r.defaultBranchRef.target,
-                        history: {
-                          totalCount: r.parent.defaultBranchRef.target.history.totalCount
-                        }
-                      }
-                    }
-                  };
-                }
-                return r;
-              };
-              setItems((prev) => prev.map(updateSyncedRepo));
-              setSearchItems((prev) => prev.map(updateSyncedRepo));
-            }
-            closeSyncModal();
-          } catch (e) {
-            setSyncing(false);
-            setSyncError(e.message || "Failed to sync fork. Check permissions and network.");
-          }
-        })();
         return;
       }
-      return;
+      if (!key.return) {
+        return;
+      }
     }
     if (logoutMode) {
       if (key.escape || input && input.toUpperCase() === "C") {
@@ -1510,50 +1508,7 @@ function RepoList({ token, maxVisibleRows, onLogout, viewerLogin, onOrgContextCh
           },
           onSubmit: () => {
             if (syncFocus === "confirm") {
-              (async () => {
-                try {
-                  setSyncing(true);
-                  const [owner, repo] = syncTarget.nameWithOwner.split("/");
-                  const branchName = syncTarget.defaultBranchRef?.name || "main";
-                  const result = await syncForkWithUpstream(token, owner, repo, branchName);
-                  const updatedRepo = await fetchRepositoryById(client, syncTarget.id, forkTracking);
-                  if (updatedRepo) {
-                    await updateCacheWithRepository(token, updatedRepo);
-                    const updateSyncedRepo = (r) => {
-                      if (r.id === syncTarget.id) {
-                        return updatedRepo;
-                      }
-                      return r;
-                    };
-                    setItems((prev) => prev.map(updateSyncedRepo));
-                    setSearchItems((prev) => prev.map(updateSyncedRepo));
-                  } else {
-                    const updateSyncedRepo = (r) => {
-                      if (r.id === syncTarget.id && r.parent && r.defaultBranchRef?.target?.history && r.parent.defaultBranchRef?.target?.history) {
-                        return {
-                          ...r,
-                          defaultBranchRef: {
-                            ...r.defaultBranchRef,
-                            target: {
-                              ...r.defaultBranchRef.target,
-                              history: {
-                                totalCount: r.parent.defaultBranchRef.target.history.totalCount
-                              }
-                            }
-                          }
-                        };
-                      }
-                      return r;
-                    };
-                    setItems((prev) => prev.map(updateSyncedRepo));
-                    setSearchItems((prev) => prev.map(updateSyncedRepo));
-                  }
-                  closeSyncModal();
-                } catch (e) {
-                  setSyncing(false);
-                  setSyncError(e.message || "Failed to sync fork. Check permissions and network.");
-                }
-              })();
+              executeSync();
             } else {
               closeSyncModal();
             }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gh-manager-cli",
-  "version": "1.10.2",
+  "version": "1.10.4",
   "private": false,
   "description": "Interactive CLI to manage your GitHub repos (personal) with Ink",
   "license": "MIT",