gh-manager-cli 1.10.3 → 1.10.4

package/CHANGELOG.md

@@ -1,3 +1,10 @@
+## [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)
 
 

package/dist/index.js

@@ -23,7 +23,7 @@ var require_package = __commonJS({
   "package.json"(exports, module) {
     module.exports = {
       name: "gh-manager-cli",
-      version: "1.10.3",
+      version: "1.10.4",
       private: false,
       description: "Interactive CLI to manage your GitHub repos (personal) with Ink",
       license: "MIT",
@@ -486,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);
@@ -504,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);
@@ -860,53 +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 = {
-              ...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.");
-          }
-        })();
         return;
       }
-      return;
+      if (!key.return) {
+        return;
+      }
     }
     if (logoutMode) {
       if (key.escape || input && input.toUpperCase() === "C") {
@@ -1503,44 +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 = {
-                    ...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.");
-                }
-              })();
+              executeSync();
             } else {
               closeSyncModal();
             }

package/package.json

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