Switch to direct git add execution for reliable file staging and eliminate the associated delay

Author: robertpiosik

.husky/pre-commit

@@ -1,2 +1 @@
-npm test
-npx lint-staged
+npx lint-staged --allow-empty

packages/vscode/src/utils/git-repository-utils.ts

@@ -71,11 +71,15 @@ export async function prepare_staged_changes(
     staged_changes.length === 0 &&
     repository.state.workingTreeChanges.length > 0
   ) {
-    const uris_to_stage = repository.state.workingTreeChanges.map(
+    const files_to_stage = repository.state.workingTreeChanges.map(
       (change: any) => change.uri.fsPath
     )
-    await repository.add(uris_to_stage)
-    await new Promise((resolve) => setTimeout(resolve, 500))
+    const file_args = files_to_stage
+      .map((file: string) => `"${file.replace(/"/g, '\\"')}"`)
+      .join(' ')
+    execSync(`git add -- ${file_args}`, {
+      cwd: repository.rootUri.fsPath
+    })
     await repository.status()
   }
 

packages/vscode/src/views/panel/backend/message-handlers/handle-commit-changes/handle-commit-changes.ts

@@ -142,13 +142,12 @@ export const handle_proceed_with_commit = async (
   }
 
   try {
-    // Convert relative paths to absolute URIs
-    const uris_to_stage = files_to_stage.map(
-      (file_path) => vscode.Uri.joinPath(repository.rootUri, file_path).fsPath
-    )
+    const file_args = files_to_stage
+      .map((file) => `"${file.replace(/"/g, '\\"')}"`)
+      .join(' ')
+    execSync(`git add -- ${file_args}`, { cwd: repository.rootUri.fsPath })
 
-    await repository.add(uris_to_stage)
-    await new Promise((resolve) => setTimeout(resolve, 500))
+    await repository.status()
     await proceed_with_commit_generation(panel_provider, repository, true)
   } catch (error) {
     panel_provider.send_message({ command: 'COMMIT_PROCESS_CANCELLED' })