🤖 fix: handle quoted Windows paths with spaces in translation

Fixed regex to properly handle paths like "C:\Users\John Doe\repo":
- Double-quoted paths: match content up to closing quote (allows spaces)
- Single-quoted paths: match content up to closing quote (allows spaces)
- Unquoted paths: match up to whitespace (no spaces allowed)

Added 3 new tests for paths with spaces.

Author: ibetitsmike

src/node/utils/main/bashPath.test.ts

@@ -173,6 +173,24 @@ describe("bashPath", () => {
         translateWindowsPathsInCommand('git -C "C:\\Users\\micha\\source\\mux" worktree list')
       ).toBe('git -C "/mnt/c/Users/micha/source/mux" worktree list');
     });
+
+    it("should handle double-quoted paths with spaces", () => {
+      expect(translateWindowsPathsInCommand('cd "C:\\Users\\John Doe\\My Documents"')).toBe(
+        'cd "/mnt/c/Users/John Doe/My Documents"'
+      );
+    });
+
+    it("should handle single-quoted paths with spaces", () => {
+      expect(translateWindowsPathsInCommand("cd 'D:\\Program Files\\My App'")).toBe(
+        "cd '/mnt/d/Program Files/My App'"
+      );
+    });
+
+    it("should handle mixed quoted paths with and without spaces", () => {
+      expect(
+        translateWindowsPathsInCommand('cp "C:\\Users\\John Doe\\file.txt" C:\\dest')
+      ).toBe('cp "/mnt/c/Users/John Doe/file.txt" /mnt/c/dest');
+    });
   });
 
   describe("getSpawnConfig with WSL path translation", () => {

src/node/utils/main/bashPath.ts

@@ -84,13 +84,34 @@ export function windowsToWslPath(windowsPath: string): string {
  * Finds patterns like C:\... or "C:\..." and converts them
  */
 export function translateWindowsPathsInCommand(command: string): string {
-  // Match Windows paths (with or without quotes)
-  // Handles: C:\path, "C:\path", 'C:\path'
+  // Match Windows paths with different quote styles:
+  // 1. Double-quoted: "C:\Users\John Doe\repo" - can contain spaces
+  // 2. Single-quoted: 'C:\Users\John Doe\repo' - can contain spaces
+  // 3. Unquoted: C:\Users\name\repo - no spaces allowed
   return command.replace(
-    /(["']?)([a-zA-Z]):[/\\]([^"'\s]*)\1/g,
-    (_match: string, quote: string, drive: string, rest: string) => {
-      const wslPath = `/mnt/${drive.toLowerCase()}/${rest.replace(/\\/g, "/")}`;
-      return quote ? `${quote}${wslPath}${quote}` : wslPath;
+    /"([a-zA-Z]):[/\\]([^"]*)"|'([a-zA-Z]):[/\\]([^']*)'|([a-zA-Z]):[/\\]([^\s]*)/g,
+    (
+      _match: string,
+      dqDrive: string | undefined,
+      dqRest: string | undefined,
+      sqDrive: string | undefined,
+      sqRest: string | undefined,
+      uqDrive: string | undefined,
+      uqRest: string | undefined
+    ) => {
+      if (dqDrive !== undefined) {
+        // Double-quoted path
+        const wslPath = `/mnt/${dqDrive.toLowerCase()}/${dqRest!.replace(/\\/g, "/")}`;
+        return `"${wslPath}"`;
+      } else if (sqDrive !== undefined) {
+        // Single-quoted path
+        const wslPath = `/mnt/${sqDrive.toLowerCase()}/${sqRest!.replace(/\\/g, "/")}`;
+        return `'${wslPath}'`;
+      } else {
+        // Unquoted path
+        const wslPath = `/mnt/${uqDrive!.toLowerCase()}/${uqRest!.replace(/\\/g, "/")}`;
+        return wslPath;
+      }
     }
   );
 }