Replace rimraf with native rm

.github/dependabot.yml

@@ -54,9 +54,6 @@ updates:
       - dependency-name: 'glob'
         update-types:
           - version-update:semver-major
-      - dependency-name: 'rimraf'
-        update-types:
-          - version-update:semver-major
       - dependency-name: 'which'
         update-types:
           - version-update:semver-major

lib/autofix.js

@@ -109,7 +109,9 @@ function askConfirmationToFixWithOptions(options, app, elmVersion) {
             modifiedElmJson = file;
           }
 
-          return await FS.writeFile(filePath, file.source).then(() => file);
+          await FS.writeFile(filePath, file.source);
+
+          return file;
         })
       );
 

lib/elm-app-worker.js

@@ -31,14 +31,12 @@ if (parentPort) {
  * @returns {void}
  */
 function subscribe(parentPort) {
-  parentPort.on('message', ([port, data]) => {
+  parentPort.on('message', async ([port, data]) => {
     if (port === 'startReview' || port === 'startGeneratingSuppressions') {
-      loadCachePromise.then(() => {
-        app.ports[port].send(data);
-      });
-    } else {
-      app.ports[port].send(data);
+      await loadCachePromise;
     }
+
+    app.ports[port].send(data);
   });
 
   app.ports.requestReadingFiles.subscribe((data) => {

lib/fs-wrapper.js

@@ -4,7 +4,6 @@
  */
 const fs = require('graceful-fs');
 const fsp = fs.promises;
-const {rimraf} = require('rimraf');
 
 /**
  * Read a JSON file.
@@ -77,6 +76,17 @@ function mkdirpSync(dir) {
   fs.mkdirSync(dir, {recursive: true});
 }
 
+/**
+ * @param {fs.PathLike} file
+ */
+async function remove(file) {
+  await fsp.rm(
+    file,
+    // Rimraf compatibility:
+    {recursive: true, force: true, maxRetries: 10}
+  );
+}
+
 module.exports = {
   readFile,
   readJsonFile,
@@ -88,9 +98,9 @@ module.exports = {
   mkdirp,
   mkdirpSync,
 
-  readdir: fsp.readdir,
+  remove,
 
-  remove: rimraf,
+  readdir: fsp.readdir,
 
   stat: fsp.stat
 };

lib/suppressed-errors.js

@@ -73,7 +73,7 @@ Please check that ${chalk.greenBright('elm-review')} has write permissions to th
       return null;
     }
 
-    return await FS.remove(filePath);
+    await FS.remove(filePath);
   });
   await Promise.all(writePromises);
 }

lib/types/state.ts

@@ -1,4 +1,4 @@
-import type {ElmFile, Readme, Source} from './content.js';
+import type {ElmFile, File, Readme} from './content.js';
 import type {Path} from './path.js';
 
 export type Model = {
@@ -23,7 +23,7 @@ export type ExitRequest = {
 
 export type AppUnsubscribeFunction = () => void;
 
-export type FilesProposedByCurrentFix = {path: Path; source: Source}[];
+export type FilesProposedByCurrentFix = File[];
 
 export type FileId = string;
 

package.json

@@ -75,7 +75,6 @@
     "ora": "^5.4.0",
     "path-key": "^3.1.1",
     "prompts": "^2.2.1",
-    "rimraf": "^5.0.0",
     "strip-ansi": "^6.0.0",
     "terminal-link": "^2.1.1",
     "tinyglobby": "^0.2.10",