fix: Handle removing non-empty directories and non-existing paths

packages/nx/src/tasks-runner/cache.ts

@@ -1,5 +1,5 @@
 import { workspaceRoot } from '../utils/workspace-root';
-import { mkdir, mkdirSync, pathExists, readFile, writeFile } from 'fs-extra';
+import { mkdir, mkdirSync, pathExists, readFile, writeFile, remove } from 'fs-extra';
 import { join } from 'path';
 import { performance } from 'perf_hooks';
 import { DefaultTasksRunnerOptions } from './default-tasks-runner';
@@ -90,8 +90,8 @@ export class Cache {
       const tdCommit = join(this.cachePath, `${task.hash}.commit`);
 
       // might be left overs from partially-completed cache invocations
-      await this.remove(tdCommit);
-      await this.remove(td);
+      await remove(tdCommit);
+      await remove(td);
 
       await mkdir(td);
       await writeFile(
@@ -145,7 +145,7 @@ export class Cache {
           const cached = join(cachedResult.outputsPath, f);
           if (await pathExists(cached)) {
             const src = join(this.root, f);
-            await this.remove(src);
+            await remove(src);
             await this.copy(cached, src);
           }
         })
@@ -202,18 +202,6 @@ export class Cache {
     });
   }
 
-  private async remove(path: string): Promise<void> {
-    const { remove } = require('../native');
-    return new Promise((res, rej) => {
-      try {
-        remove(path);
-        res();
-      } catch (e) {
-        rej(e);
-      }
-    });
-  }
-
   private async getFromLocalDir(task: Task) {
     const tdCommit = join(this.cachePath, `${task.hash}.commit`);
     const td = join(this.cachePath, task.hash);