chore: remove fs-extra dependency

Manual rebase of #147.

Co-authored-by: Jeroen Engels <[email protected]>

lib/fs-wrapper.js

@@ -2,6 +2,7 @@
  * @import {Replacer, Reviver} from './types/json';
  * @import {Path} from './types/path';
  */
+const path = require('node:path');
 const fs = require('graceful-fs');
 const fsp = fs.promises;
 
@@ -95,6 +96,21 @@ async function remove(file) {
   );
 }
 
+/**
+ * Copy files from srcFolder to destFolder.
+ * @param {Path} srcFolder
+ * @param {Path} destFolder
+ * @param {Path[]} files
+ * @returns {Promise<void>}
+ */
+async function copyFiles(srcFolder, destFolder, files) {
+  const promises = files.map(async (file) => {
+    await fsp.copyFile(path.join(srcFolder, file), path.join(destFolder, file));
+  });
+
+  await Promise.all(promises);
+}
+
 module.exports = {
   readFile,
   readJsonFile,
@@ -106,6 +122,9 @@ module.exports = {
   mkdirp,
   mkdirpSync,
 
+  // TODO(@lishaduck) [engine:node@>=16.7.0]: Use native `cp`.
+  copyFiles,
+
   remove,
 
   readdir: fsp.readdir,

lib/init.js

@@ -6,7 +6,6 @@
 const path = require('node:path');
 const fs = require('graceful-fs');
 // TODO(@lishaduck) [engine:node@>=16.7.0]: Use native `cp`.
-const fsExtra = require('fs-extra');
 const chalk = require('chalk');
 const prompts = require('prompts');
 const FS = require('./fs-wrapper');
@@ -197,7 +196,7 @@ async function createElmJson(options, directory) {
  * @returns {void}
  */
 function createReviewConfig(directory, template) {
-  fsExtra.copyFileSync(
+  fs.copyFileSync(
     path.join(__dirname, '../init-templates/', template),
     path.join(directory, 'ReviewConfig.elm')
   );

lib/new-package.js

@@ -8,8 +8,6 @@
 const childProcess = require('node:child_process');
 const path = require('node:path');
 const chalk = require('chalk');
-// TODO(@lishaduck) [engine:node@>=16.7.0]: Use native `cp`.
-const fsExtra = require('fs-extra');
 const prompts = require('prompts');
 const ErrorMessage = require('./error-message');
 const FS = require('./fs-wrapper');
@@ -293,10 +291,15 @@ ElmjutsuDumMyM0DuL3.elm
   );
 
   Spinner.succeedAndNowDo('Adding GitHub Actions');
-  await fsExtra.copy(
+  const githubDestFiles = path.join(dir, '.github/');
+  await Promise.all([
+    FS.mkdirp(path.join(githubDestFiles, 'ISSUE_TEMPLATE')),
+    FS.mkdirp(path.join(githubDestFiles, 'workflows'))
+  ]);
+  await FS.copyFiles(
     path.join(__dirname, '../new-package/github/'),
-    path.join(dir, '.github/'),
-    {overwrite: true}
+    githubDestFiles,
+    ['ISSUE_TEMPLATE/new-rule-idea.md', 'workflows/test.yml']
   );
 
   Spinner.succeedAndNowDo(
@@ -334,26 +337,27 @@ ElmjutsuDumMyM0DuL3.elm
   Spinner.succeedAndNowDo('Adding maintenance scripts');
   const maintenancePath = path.join(process.cwd(), packageName, 'maintenance');
   await FS.mkdirp(maintenancePath);
-  await fsExtra.copy(
+  await FS.copyFiles(
     path.join(__dirname, '../new-package/maintenance/'),
     maintenancePath,
-    {
-      overwrite: true
-    }
+    ['MAINTENANCE.md', 'update-examples-from-preview.js']
   );
 
   const packageTests = path.join(
     process.cwd(),
     packageName,
     'elm-review-package-tests'
   );
-  await FS.mkdirp(packageTests);
-  await fsExtra.copy(
-    path.join(__dirname, '../new-package/elm-review-package-tests/'),
+  FS.mkdirpSync(path.join(packageTests, 'helpers'));
+  await FS.copyFiles(
+    path.join(__dirname, '../new-package/elm-review-package-tests'),
     packageTests,
-    {
-      overwrite: true
-    }
+    [
+      'check-previews-compile.js',
+      'check-examples-were-updated.js',
+      'helpers/ansi.js',
+      'helpers/find-configurations.js'
+    ]
   );
 
   const checkPreviewsCompile = path.join(

new-package/maintenance/update-examples-from-preview.js

@@ -47,7 +47,12 @@ function copyPreviewToExample(pathToPreviewFolder) {
     /preview/g,
     'example'
   );
-  fs.removeSync(pathToExampleFolder);
+
+  fs.rmSync(pathToExampleFolder, {
+    recursive: true,
+    force: true,
+    maxRetries: 10
+  });
   fs.copySync(pathToPreviewFolder, pathToExampleFolder, {overwrite: true});
 
   const pathToElmJson = path.resolve(pathToExampleFolder, 'elm.json');

package.json

@@ -67,7 +67,6 @@
     "fastest-levenshtein": "^1.0.16",
     "find-up": "^4.1.0 || ^5.0.0",
     "folder-hash": "^3.3.0",
-    "fs-extra": "^9.0.0",
     "glob": "^10.2.6",
     "got": "^11.8.5",
     "graceful-fs": "^4.2.11",

test/run-snapshots/elm-review-something-for-new-rule/maintenance/update-examples-from-preview.js

@@ -47,7 +47,12 @@ function copyPreviewToExample(pathToPreviewFolder) {
     /preview/g,
     'example'
   );
-  fs.removeSync(pathToExampleFolder);
+
+  fs.rmSync(pathToExampleFolder, {
+    recursive: true,
+    force: true,
+    maxRetries: 10
+  });
   fs.copySync(pathToPreviewFolder, pathToExampleFolder, {overwrite: true});
 
   const pathToElmJson = path.resolve(pathToExampleFolder, 'elm.json');

test/run-snapshots/elm-review-something/maintenance/update-examples-from-preview.js

@@ -47,7 +47,12 @@ function copyPreviewToExample(pathToPreviewFolder) {
     /preview/g,
     'example'
   );
-  fs.removeSync(pathToExampleFolder);
+
+  fs.rmSync(pathToExampleFolder, {
+    recursive: true,
+    force: true,
+    maxRetries: 10
+  });
   fs.copySync(pathToPreviewFolder, pathToExampleFolder, {overwrite: true});
 
   const pathToElmJson = path.resolve(pathToExampleFolder, 'elm.json');