fix(core): git hasher should handle unstaged files with spaces

Author: vsavkin

Diff for: packages/workspace/src/core/hasher/git-hasher.spec.ts

@@ -78,7 +78,28 @@ describe('git-hasher', () => {
     run(`echo AAA > "a b".txt`);
     run(`git add .`);
     run(`git commit -am init`);
-    expect([...getFileHashes(dir).keys()]).toEqual([`${dir}/a b.txt`]);
+    run(`touch "x y z.txt"`); // unstaged
+    expect([...getFileHashes(dir).keys()]).toEqual([
+      `${dir}/a b.txt`,
+      `${dir}/x y z.txt`,
+    ]);
+    run(`git add .`);
+    expect([...getFileHashes(dir).keys()]).toEqual([
+      `${dir}/a b.txt`,
+      `${dir}/x y z.txt`,
+    ]);
+    run(`mv "a b.txt" "a b moved.txt"`);
+    expect([...getFileHashes(dir).keys()]).toEqual([
+      `${dir}/x y z.txt`,
+      `${dir}/a b moved.txt`,
+    ]);
+    run(`git add .`);
+    expect([...getFileHashes(dir).keys()]).toEqual([
+      `${dir}/a b moved.txt`,
+      `${dir}/x y z.txt`,
+    ]);
+    run(`rm "x y z.txt"`);
+    expect([...getFileHashes(dir).keys()]).toEqual([`${dir}/a b moved.txt`]);
   });
 
   it('should handle renames and modifications', () => {

Diff for: packages/workspace/src/core/hasher/git-hasher.ts

@@ -36,15 +36,20 @@ function parseGitStatus(output: string): Map<string, string> {
         .match(/(?:[^\s"]+|"[^"]*")+/g)
         .map((r) => (r.startsWith('"') ? r.substring(1, r.length - 1) : r))
         .filter((r) => !!r);
+
       if (changeType && filenames && filenames.length > 0) {
         // the before filename we mark as deleted, so we remove it from the map
         // changeType can be A/D/R/RM etc
         // if it R and RM, we need to split the output into before and after
         // the before part gets marked as deleted
         if (changeType[0] === 'R') {
           changes.set(filenames[0], 'D');
+          changes.set(filenames[filenames.length - 1], changeType);
+        } else if (changeType === '??') {
+          changes.set(filenames.join(' '), changeType);
+        } else {
+          changes.set(filenames[filenames.length - 1], changeType);
         }
-        changes.set(filenames[filenames.length - 1], changeType);
       }
     });
   return changes;