feat/make windows ebusy errors less fatal

package.json

@@ -102,6 +102,7 @@
     "nx": "22.5.4",
     "prettier": "3.2.5",
     "reflect-metadata": "0.2.2",
+    "rimraf": "6.1.3",
     "ts-jest": "29.4.2",
     "ts-node": "10.9.1",
     "typescript": "5.9.2",

tools/cook/infra.ts

@@ -1,18 +1,12 @@
-import { workspaceRoot } from '@nx/devkit';
 import inquirer from 'enquirer';
-import {
-  mkdirSync,
-  readdirSync,
-  readFileSync,
-  renameSync,
-  rmSync,
-  writeFileSync,
-} from 'fs';
+import { readFileSync, writeFileSync } from 'fs';
 import { execSync } from 'node:child_process';
+import { mkdirSync, readdirSync, renameSync, rmSync } from 'node:fs';
 import { basename, join, relative } from 'node:path';
 
 const { prompt } = inquirer;
 
+const TRASH_PATH = join(process.cwd(), 'tmp', 'trash');
 export class CommandRunner {
   executeCommand(
     command: string,
@@ -29,8 +23,6 @@ export class CommandRunner {
   }
 }
 
-export const TRASH_PATH = join(workspaceRoot, 'tmp', 'trash');
-
 export class FileSystemAdapter {
   readFile(path: string): string {
     return readFileSync(path, {
@@ -48,18 +40,29 @@ export class FileSystemAdapter {
     return readdirSync(path);
   }
 
+  /**
+   * Windows antiviruses can be a pain (e.g. EBUSY errors).
+   * This method will try to remove the folder multiple times,
+   * then if it fails, it will try to move the folder to tmp/trash,
+   * and finally if that fails, it will warn the user that they may need to remove it manually.
+   *
+   * Note that I've tried to use rimraf and its backoff but that wasn't enough.
+   */
   removeDir(path: string): void {
-    mkdirSync(TRASH_PATH, { recursive: true });
-    const targetPath = join(TRASH_PATH, `${basename(path)}-${Date.now()}`);
-    renameSync(path, targetPath);
     try {
-      rmSync(targetPath, { maxRetries: 5, recursive: true });
-    } catch (error: unknown) {
-      const relativeTargetPath = relative(workspaceRoot, targetPath);
-      const reason = error instanceof Error ? error.message : error;
-      console.warn(
-        `⚠️ Failed to remove ${relativeTargetPath}. You may need to remove it manually. Reason: ${reason}`,
-      );
+      rmSync(path, { maxRetries: 10, recursive: true });
+    } catch {
+      try {
+        mkdirSync(TRASH_PATH, { recursive: true });
+        const targetPath = join(TRASH_PATH, `${basename(path)}-${Date.now()}`);
+        renameSync(path, targetPath);
+      } catch (error: unknown) {
+        const relativeTargetPath = relative(process.cwd(), path);
+        const reason = error instanceof Error ? error.message : error;
+        console.warn(
+          `⚠️ Failed to remove ${relativeTargetPath}. You may need to remove it manually. Reason: ${reason}`,
+        );
+      }
     }
   }
 }

tools/vitest.config.mts

@@ -1,9 +1,6 @@
 import { defineConfig, mergeConfig } from 'vitest/config';
 import viteConfig from './vite.config.mjs';
 
-const narrowTestPatterns = ['**/!(*.wide).spec.ts'];
-const wideTestPatterns = ['**/*.wide.spec.ts'];
-
 export default mergeConfig(
   viteConfig,
   defineConfig({
@@ -15,25 +12,10 @@ export default mergeConfig(
         provider: 'v8',
       },
       environment: 'node',
+      include: ['**/*.spec.ts'],
       watch: false,
       pool: 'threads',
       isolate: false,
-      projects: [
-        {
-          extends: true,
-          test: {
-            name: 'narrow',
-            include: narrowTestPatterns,
-          },
-        },
-        {
-          extends: true,
-          test: {
-            name: 'wide',
-            include: wideTestPatterns,
-          },
-        },
-      ],
     },
   }),
 );