fix(vcs): handle case when file is removed while listing VCS files

garden-service/src/vcs/git.ts

@@ -97,13 +97,23 @@ export class GitHandler extends VcsHandler {
 
     return Bluebird.map(filtered, async (f) => {
       const resolvedPath = resolve(path, f.path)
+
       if (!f.hash || modified.has(f.path)) {
-        const hash = (await git("hash-object", resolvedPath))[0]
+        // If we can't compute the hash, i.e. the file is gone, we filter it out below
+        let hash = ""
+        try {
+          hash = (await git("hash-object", resolvedPath))[0]
+        } catch (err) {
+          // 128 = File no longer exists
+          if (err.code !== 128) {
+            throw err
+          }
+        }
         return { path: resolvedPath, hash }
       } else {
         return { path: resolvedPath, hash: f.hash }
       }
-    })
+    }).filter(f => f.hash !== "")
   }
 
   // TODO Better auth handling

garden-service/src/watch.ts

@@ -93,9 +93,9 @@ export class Watcher {
     const _this = this
 
     return (path: string) => {
+      // Make sure Promise errors are handled appropriately.
       listener(path)
         .catch(err => {
-          console.log("catch", err)
           _this.watcher.emit("error", err)
         })
     }