replace rm -rf by fsp.rm

Author: millotp

scripts/generate.ts

@@ -6,8 +6,8 @@ import { createSpinner } from './oraLog';
 import { generateOpenapitools, removeExistingCodegen } from './pre-gen';
 import type { Generator } from './types';
 
-async function preGen(gen: Generator, verbose?: boolean): Promise<void> {
-  await removeExistingCodegen(gen, verbose);
+async function preGen(gen: Generator): Promise<void> {
+  await removeExistingCodegen(gen);
 }
 
 async function generateClient(
@@ -48,7 +48,7 @@ export async function generate(
 
   for (const gen of generators) {
     const spinner = createSpinner(`pre-gen ${gen.key}`, verbose).start();
-    await preGen(gen, verbose);
+    await preGen(gen);
 
     spinner.text = `generating ${gen.key}`;
     await generateClient(gen, verbose);

scripts/pre-gen/removeExistingCodegen.ts

@@ -1,16 +1,18 @@
+import fsp from 'fs/promises';
 import path from 'path';
 
-import { createClientName, run, toAbsolutePath } from '../common';
+import { createClientName, toAbsolutePath } from '../common';
 import { getLanguageApiFolder, getLanguageModelFolder } from '../config';
 import type { Generator } from '../types';
 
 /**
  * Remove `model` folder for the current language and client.
  */
-export async function removeExistingCodegen(
-  { language, client, output }: Generator,
-  verbose?: boolean
-): Promise<void> {
+export async function removeExistingCodegen({
+  language,
+  client,
+  output,
+}: Generator): Promise<void> {
   const baseModelFolder = getLanguageModelFolder(language);
   const baseApiFolder = getLanguageApiFolder(language);
   const clientName = createClientName(client, language);
@@ -23,13 +25,11 @@ export async function removeExistingCodegen(
       if (client === 'query-suggestions') {
         // eslint-disable-next-line no-warning-comments
         // TODO: temporary solution, remove in next PR
-        await run(
-          `rm -rf ${toAbsolutePath(
+        await fsp.rm(
+          toAbsolutePath(
             path.resolve('..', output, baseModelFolder, 'querySuggestions')
-          )}`,
-          {
-            verbose,
-          }
+          ),
+          { force: true, recursive: true }
         );
       }
       clientModel = client.replace('-', '');
@@ -42,44 +42,31 @@ export async function removeExistingCodegen(
     case 'javascript':
       // We want to also delete the nested `lite` client or folders that only exists in JS
       if (clientName === 'algoliasearch') {
-        await run(
-          `rm -rf ${toAbsolutePath(path.resolve('..', output, 'lite'))}`,
-          {
-            verbose,
-          }
-        );
+        await fsp.rm(toAbsolutePath(path.resolve('..', output, 'lite')), {
+          force: true,
+          recursive: true,
+        });
       }
 
       // Delete `builds` folder
-      await run(
-        `rm -rf ${toAbsolutePath(path.resolve('..', output, 'builds'))}`,
-        {
-          verbose,
-        }
-      );
-
+      await fsp.rm(toAbsolutePath(path.resolve('..', output, 'builds')), {
+        force: true,
+        recursive: true,
+      });
       break;
     default:
       break;
   }
 
   // Delete client model folder/file
-  await run(
-    `rm -rf ${toAbsolutePath(
-      path.resolve('..', output, baseModelFolder, clientModel)
-    )}`,
-    {
-      verbose,
-    }
+  await fsp.rm(
+    toAbsolutePath(path.resolve('..', output, baseModelFolder, clientModel)),
+    { force: true, recursive: true }
   );
 
   // Delete client api folder/file
-  await run(
-    `rm -rf ${toAbsolutePath(
-      path.resolve('..', output, baseApiFolder, clientApi)
-    )}`,
-    {
-      verbose,
-    }
+  await fsp.rm(
+    toAbsolutePath(path.resolve('..', output, baseApiFolder, clientApi)),
+    { force: true, recursive: true }
   );
 }

scripts/release/common.ts

@@ -37,7 +37,7 @@ export async function cloneRepository({
 
   const gitHubUrl = getGitHubUrl(lang, { token: githubToken });
   const tempGitDir = path.resolve(tempDir, lang);
-  await run(`rm -rf ${tempGitDir}`);
+  await fsp.rm(tempGitDir, { force: true, recursive: true });
   await run(
     `git clone --depth 1 --branch ${targetBranch} ${gitHubUrl} ${tempGitDir}`
   );