Remove rimraf dependency in favour of Node.js's rm method (#961)

.changeset/breezy-rockets-invent.md

@@ -0,0 +1,5 @@
+---
+'sku': patch
+---
+
+Remove `rimraf` dependency in favour of Node.js's `rm`

.npmrc

@@ -6,3 +6,7 @@ public-hoist-pattern[]="*prettier*"
 # This bypasses the .pnpm folder that resolveModules looks for
 # See https://github.com/seek-oss/sku/blob/275752bd3066e52885d461ef5bcd953aaac3bfff/config/webpack/resolveModules.js
 public-hoist-pattern[]="@babel/*"
+
+# `docsify-cli` depends on `docsify` which depends on `opencollective-postinstall` and calls it during `postinstall`.
+# For some reason, this call fails on CI, so we need to hoist it.
+public-hoist-pattern[]="opencollective-postinstall"

fixtures/sku-init/sku-init.test.js

@@ -26,6 +26,12 @@ describe('sku init', () => {
   it(
     'should create a sku.config.ts',
     async () => {
+      const projectName = 'new-project';
+      await fs.rm(path.join(fixtureDirectory, projectName), {
+        recursive: true,
+        force: true,
+      });
+
       const { child } = await runSkuScriptInDir('init', fixtureDirectory, [
         projectName,
       ]);

jest.config.js

@@ -3,7 +3,11 @@ module.exports = {
   transform: {
     '^.+\\.(t|j)sx?$': '@swc/jest',
   },
-  maxWorkers: process.env.CI ? 2 : undefined,
+  ...(process.env.CI
+    ? {
+        maxWorkers: 2,
+      }
+    : {}),
   preset: 'jest-puppeteer',
   setupFilesAfterEnv: ['<rootDir>/test-utils/jestSetup.ts'],
   snapshotSerializers: [

package.json

@@ -65,7 +65,6 @@
     "prettier": "^2.8.8",
     "puppeteer": "^21.6.0",
     "renovate-config-seek": "^0.4.0",
-    "rimraf": "^5.0.0",
     "typescript": "*"
   },
   "volta": {

packages/sku/lib/buildFileUtils.js

@@ -1,13 +1,15 @@
 // @ts-check
 const path = require('node:path');
 const fs = require('node:fs/promises');
-const { rimraf } = require('rimraf');
+const { fdir: Fdir } = require('fdir');
 
 const { paths } = require('../context');
 const exists = require('./exists');
 const copyDirContents = require('./copyDirContents');
 
-const cleanTargetDirectory = () => rimraf(`${paths.target}/*`, { glob: true });
+const cleanTargetDirectory = async () => {
+  fs.rm(paths.target, { recursive: true, force: true });
+};
 
 const copyPublicFiles = async () => {
   if (await exists(paths.public)) {
@@ -19,14 +21,21 @@ const ensureTargetDirectory = async () => {
   await fs.mkdir(paths.target, { recursive: true });
 };
 
-const cleanRenderJs = async () => {
-  const renderFileGlob = path.join(paths.target, '*render.js');
-  await rimraf(renderFileGlob, { glob: true });
+const cleanStaticRenderEntry = async () => {
+  const files = await new Fdir()
+    .withBasePath()
+    .filter((file) => file.endsWith('render.js'))
+    .crawl(paths.target)
+    .withPromise();
+
+  for (const file of files) {
+    await fs.rm(file);
+  }
 };
 
 module.exports = {
   cleanTargetDirectory,
   copyPublicFiles,
   ensureTargetDirectory,
-  cleanRenderJs,
+  cleanStaticRenderEntry,
 };

packages/sku/package.json

@@ -110,7 +110,6 @@
     "prettier": "^2.8.8",
     "pretty-ms": "^7.0.1",
     "react-refresh": "^0.14.0",
-    "rimraf": "^5.0.0",
     "selfsigned": "^2.1.1",
     "semver": "^7.3.4",
     "serialize-javascript": "^6.0.0",

packages/sku/scripts/build-storybook.js

@@ -1,7 +1,7 @@
 // First, ensure the build is running in production mode
 process.env.NODE_ENV = 'production';
 
-const { rimraf } = require('rimraf');
+const { rm } = require('node:fs/promises');
 const { argv, config } = require('../config/args');
 const gracefulSpawn = require('../lib/gracefulSpawn');
 const { storybookTarget } = require('../context');
@@ -11,7 +11,7 @@ const { setUpStorybookConfigDirectory } = require('../lib/storybook');
 
 (async () => {
   await runVocabCompile();
-  await rimraf(storybookTarget);
+  await rm(storybookTarget, { recursive: true, force: true });
   await setUpStorybookConfigDirectory();
 
   argv.push('build');

packages/sku/scripts/build.js

@@ -10,7 +10,7 @@ const {
   copyPublicFiles,
   cleanTargetDirectory,
   ensureTargetDirectory,
-  cleanRenderJs,
+  cleanStaticRenderEntry,
 } = require('../lib/buildFileUtils');
 const { run } = require('../lib/runWebpack');
 const createHtmlRenderPlugin = require('../config/webpack/plugins/createHtmlRenderPlugin');
@@ -31,7 +31,7 @@ const { runVocabCompile } = require('../lib/runVocab');
         }),
       ),
     );
-    await cleanRenderJs();
+    await cleanStaticRenderEntry();
     await copyPublicFiles();
 
     const timeTaken = performance.now();