Refactor file selection logic to prevent event propagation and ensure async handling (#444)

Author: stijnpotters1

src/main/frontend/app/components/git/git-changes.tsx

@@ -112,12 +112,14 @@ function FileSection({
                 )}
                 onClick={() => onSelectFile(file)}
               >
-                <IndeterminateCheckbox
-                  checked={checkboxChecked}
-                  indeterminate={checkboxIndeterminate}
-                  onChange={() => onToggleFile(file)}
-                  title={checkboxChecked ? 'Deselect all chunks' : 'Select all chunks'}
-                />
+                <div onClick={(e) => e.stopPropagation()}>
+                  <IndeterminateCheckbox
+                    checked={checkboxChecked}
+                    indeterminate={checkboxIndeterminate}
+                    onChange={() => onToggleFile(file)}
+                    title={checkboxChecked ? 'Deselect all chunks' : 'Select all chunks'}
+                  />
+                </div>
                 <span
                   className={clsx(
                     'flex h-4 w-4 flex-shrink-0 items-center justify-center rounded text-[10px] font-bold',

src/main/frontend/app/components/git/git-panel.tsx

@@ -84,13 +84,14 @@ export default function GitPanel({ projectName, hasStoredToken }: GitPanelProps)
   )
 
   const handleToggleFile = useCallback(
-    (filePath: string) => {
+    async (filePath: string) => {
       const hunkState = useGitStore.getState().fileHunkStates[filePath]
       if (!hunkState || hunkState.totalHunks === 0) {
-        handleSelectFile(filePath)
-        return
+        await handleSelectFile(filePath)
       }
-      if (hunkState.selectedHunks.size === hunkState.totalHunks) {
+
+      const updatedState = useGitStore.getState().fileHunkStates[filePath]
+      if (updatedState.selectedHunks.size === updatedState.totalHunks) {
         clearFileHunks(filePath)
       } else {
         selectAllFileHunks(filePath)