Merge pull request #602 from desktop/rm-rf-rimraf

Remove rimraf dependency in favor of Node native rm

package.json

@@ -5,7 +5,7 @@
   "main": "./build/lib/index.js",
   "typings": "./build/lib/index.d.ts",
   "scripts": {
-    "clean": "rimraf build",
+    "clean": "node script/clean.js",
     "build": "yarn clean && tsc -p ./tsconfig.json && tsc -p ./examples/tsconfig.json",
     "prepack": "yarn build && yarn test",
     "postpublish": "git push --follow-tags",
@@ -36,11 +36,9 @@
   "devDependencies": {
     "@types/node": "20",
     "@types/progress": "^2.0.1",
-    "@types/rimraf": "2.0.2",
     "@types/temp": "^0.9.4",
     "node-test-github-reporter": "^1.2.0",
     "prettier": "^3.3.1",
-    "rimraf": "^5.0.7",
     "temp": "^0.9.4",
     "tsx": "^4.10.5",
     "typescript": "^5.4.5"

script/clean.js

@@ -0,0 +1,6 @@
+const { rm } = require('fs/promises')
+const { join } = require('path')
+
+rm(join(__dirname, '..', 'build'), { recursive: true, force: true }).catch(
+  console.error
+)

script/test.mjs

@@ -1,43 +1,23 @@
 import { spawn } from 'child_process'
-import { glob } from 'glob'
-import { dirname, resolve } from 'path'
-import { fileURLToPath } from 'url'
+import { join } from 'path'
+import { readdir } from 'fs/promises'
 
-if (process.argv.some(arg => ['-h', '--help'].includes(arg))) {
-  console.log(`Usage: ${process.argv0} [kind]`)
-  console.log(
-    '  kind: The kind of tests to run (e.g. "fast", "slow", "external", "all")'
-  )
-  process.exit(0)
+function reporter(r) {
+  return ['--test-reporter', r, '--test-reporter-destination', 'stdout']
 }
 
-;(async function (kind) {
-  const wildcard = kind && kind !== 'all' ? `${kind}/**` : '**'
-  const files = await glob(`test/${wildcard}/*-test.ts`)
-  const reporterDestinationArgs = ['--test-reporter-destination', 'stdout']
-  const specTestReporterArgs = [
-    '--test-reporter',
-    'spec',
-    ...reporterDestinationArgs,
-  ]
+const files = await readdir('test', { recursive: true }).then(x =>
+  x.filter(f => f.endsWith('-test.ts')).map(f => join('test', f))
+)
 
-  const testReporterArgs = process.env.GITHUB_ACTIONS
-    ? [
-        '--test-reporter',
-        'node-test-github-reporter',
-        ...reporterDestinationArgs,
-        ...specTestReporterArgs,
-      ]
-    : specTestReporterArgs
+process.env.LOCAL_GIT_DIRECTORY = 'git'
 
-  spawn('node', ['--import', 'tsx', ...testReporterArgs, '--test', ...files], {
-    stdio: 'inherit',
-    env: {
-      ...process.env,
-      LOCAL_GIT_DIRECTORY: resolve(
-        dirname(fileURLToPath(import.meta.url)),
-        '../git/'
-      ),
-    },
-  }).on('exit', process.exit)
-})(process.argv[2])
+const args = [
+  ...['--import', 'tsx'],
+  '--test',
+  ...reporter('spec'),
+  ...(process.env.GITHUB_ACTIONS ? reporter('node-test-github-reporter') : []),
+  ...files,
+]
+
+spawn('node', args, { stdio: 'inherit' }).on('exit', process.exit)