fix: remove rimraf dependency (#529)

We don't need this anymore. Also the cli would fail because `rimraf` was
a devDependency.

Author: styfle

package-lock.json

@@ -119,7 +119,6 @@
     "redis": "^3.1.1",
     "remark-parse": "^11.0.0",
     "remark-prism": "^1.3.6",
-    "rimraf": "^6.0.1",
     "rxjs": "^7.8.2",
     "saslprep": "^1.0.3",
     "semver": "^7.5.2",

package.json

@@ -2,8 +2,7 @@
 
 import { join, dirname, relative, isAbsolute, sep } from 'path';
 import { promises, statSync, lstatSync } from 'graceful-fs';
-const { copyFile, mkdir } = promises;
-const rimraf = require('rimraf');
+const { copyFile, mkdir, rm } = promises;
 import { nodeFileTrace } from './node-file-trace';
 import { NodeFileTraceReasons } from './types';
 
@@ -62,7 +61,7 @@ async function cli(
       }
     }
   } else if (action === 'build') {
-    rimraf.sync(join(cwd, outputDir));
+    await rm(join(cwd, outputDir), { recursive: true, force: true });
     for (const f of allFiles) {
       const src = join(cwd, f);
       const dest = join(cwd, outputDir, f);

src/cli.ts

@@ -1,16 +1,15 @@
-const { promises, mkdirSync } = require('fs');
+const { promises, mkdirSync, rmSync } = require('fs');
 const path = require('path');
 const { nodeFileTrace } = require('../out/node-file-trace');
 const os = require('os');
-const rimraf = require('rimraf');
 const { writeFile } = promises;
 
 const randomTmpId = Math.random().toString().slice(2);
 const tmpdir = path.resolve(
   os.tmpdir(),
   `node-file-trace-ecmascript${randomTmpId}`,
 );
-rimraf.sync(tmpdir);
+rmSync(tmpdir, { recursive: true, force: true });
 mkdirSync(tmpdir);
 console.log('created directory ' + tmpdir);
 

test/ecmascript.test.js

@@ -2,8 +2,7 @@ const { promises, readdirSync, mkdirSync } = require('fs');
 const path = require('path');
 const { nodeFileTrace } = require('../out/node-file-trace');
 const os = require('os');
-const rimraf = require('rimraf');
-const { readFile, writeFile, readlink, symlink, copyFile } = promises;
+const { readFile, writeFile, readlink, symlink, copyFile, rm } = promises;
 const { fork, exec: execOrig } = require('child_process');
 
 const exec = require('util').promisify(execOrig);
@@ -53,7 +52,7 @@ for (const integrationTest of filteredTestsToRun) {
         os.tmpdir(),
         `node-file-trace-${integrationTest}-${rand}`,
       );
-      rimraf.sync(tmpdir);
+      await rm(tmpdir, { recursive: true, force: true });
       mkdirSync(tmpdir);
       await copyFile(
         path.join(integrationDir, integrationTest),
@@ -87,7 +86,7 @@ for (const integrationTest of filteredTestsToRun) {
     );
     // warnings.forEach(warning => console.warn(warning));
     const tmpdir = path.resolve(os.tmpdir(), `node-file-trace-${rand}`);
-    rimraf.sync(tmpdir);
+    await rm(tmpdir, { recursive: true, force: true });
     mkdirSync(tmpdir);
 
     await Promise.all(
@@ -118,7 +117,7 @@ for (const integrationTest of filteredTestsToRun) {
     });
     const code = await new Promise((resolve) => ps.on('close', resolve));
     expect(code).toBe(fails ? 1 : 0);
-    rimraf.sync(tmpdir);
+    await rm(tmpdir, { recursive: true, force: true });
 
     // TODO: ensure analysis cache is safe for below case
     // seems this fails with cache since < 0.14.0