fix: Stabilize git diff sidebar status while switching workspaces (#83)

Author: Dimillian

src/features/git/hooks/useGitDiffs.ts

@@ -22,6 +22,7 @@ export function useGitDiffs(
   const [state, setState] = useState<GitDiffState>(emptyState);
   const requestIdRef = useRef(0);
   const workspaceIdRef = useRef<string | null>(activeWorkspace?.id ?? null);
+  const cachedDiffsRef = useRef<Map<string, GitFileDiff[]>>(new Map());
 
   const fileKey = useMemo(
     () =>
@@ -53,6 +54,7 @@ export function useGitDiffs(
         return;
       }
       setState({ diffs, isLoading: false, error: null });
+      cachedDiffsRef.current.set(workspaceId, diffs);
     } catch (error) {
       console.error("Failed to load git diffs", error);
       if (
@@ -74,7 +76,16 @@ export function useGitDiffs(
     if (workspaceIdRef.current !== workspaceId) {
       workspaceIdRef.current = workspaceId;
       requestIdRef.current += 1;
-      setState(emptyState);
+      if (!workspaceId) {
+        setState(emptyState);
+        return;
+      }
+      const cached = cachedDiffsRef.current.get(workspaceId);
+      setState({
+        diffs: cached ?? [],
+        isLoading: false,
+        error: null,
+      });
     }
   }, [activeWorkspace?.id]);
 

src/features/git/hooks/useGitStatus.ts

@@ -24,6 +24,7 @@ export function useGitStatus(activeWorkspace: WorkspaceInfo | null) {
   const [status, setStatus] = useState<GitStatusState>(emptyStatus);
   const requestIdRef = useRef(0);
   const workspaceIdRef = useRef<string | null>(activeWorkspace?.id ?? null);
+  const cachedStatusRef = useRef<Map<string, GitStatusState>>(new Map());
   const workspaceId = activeWorkspace?.id ?? null;
 
   const refresh = useCallback(() => {
@@ -41,7 +42,9 @@ export function useGitStatus(activeWorkspace: WorkspaceInfo | null) {
         ) {
           return;
         }
-        setStatus({ ...data, error: null });
+        const nextStatus = { ...data, error: null };
+        setStatus(nextStatus);
+        cachedStatusRef.current.set(workspaceId, nextStatus);
       })
       .catch((err) => {
         console.error("Failed to load git status", err);
@@ -51,19 +54,26 @@ export function useGitStatus(activeWorkspace: WorkspaceInfo | null) {
         ) {
           return;
         }
-        setStatus({
+        const nextStatus = {
           ...emptyStatus,
           branchName: "unknown",
           error: err instanceof Error ? err.message : String(err),
-        });
+        };
+        setStatus(nextStatus);
+        cachedStatusRef.current.set(workspaceId, nextStatus);
       });
   }, [workspaceId]);
 
   useEffect(() => {
     if (workspaceIdRef.current !== workspaceId) {
       workspaceIdRef.current = workspaceId;
       requestIdRef.current += 1;
-      setStatus(emptyStatus);
+      if (!workspaceId) {
+        setStatus(emptyStatus);
+        return;
+      }
+      const cached = cachedStatusRef.current.get(workspaceId);
+      setStatus(cached ?? emptyStatus);
     }
   }, [workspaceId]);