chore: remove rimraf

package.json

@@ -11,7 +11,7 @@
     "build:docs": "wireit",
     "check:pinned-deps": "tsx tools/ensure-pinned-deps",
     "check": "npm run check --workspaces --if-present && run-p check:*",
-    "clean": "rimraf **/.wireit && npm run clean --workspaces --if-present",
+    "clean": "rm -rf **/.wireit && npm run clean --workspaces --if-present",
     "commitlint": "commitlint --from=HEAD~1",
     "debug": "mocha --inspect-brk",
     "docs": "run-s build:docs generate:markdown",
@@ -133,7 +133,6 @@
     "@types/pngjs": "6.0.1",
     "@types/progress": "2.0.5",
     "@types/proxy-from-env": "1.0.1",
-    "@types/rimraf": "3.0.2",
     "@types/semver": "7.3.13",
     "@types/sinon": "10.0.13",
     "@types/tar-fs": "2.0.1",

packages/browsers/package.json

@@ -5,7 +5,7 @@
   "scripts": {
     "build": "wireit",
     "build:test": "wireit",
-    "clean": "tsc --build --clean && rimraf lib",
+    "clean": "tsc --build --clean && rm -rf lib",
     "test": "wireit"
   },
   "bin": {

packages/ng-schematics/package.json

@@ -5,7 +5,7 @@
   "scripts": {
     "build:tsc": "wireit",
     "build": "wireit",
-    "clean": "tsc -b --clean && rimraf lib && rimraf test/build",
+    "clean": "tsc -b --clean && rm -rf lib && rm -rf test/build",
     "dev:test": "npm run test --watch",
     "dev": "npm run build --watch",
     "test": "wireit"

packages/puppeteer-core/package.json

@@ -31,15 +31,15 @@
     "url": "https://github.com/puppeteer/puppeteer/tree/main/packages/puppeteer-core"
   },
   "engines": {
-    "node": ">=14.1.0"
+    "node": ">=14.14.0"
   },
   "scripts": {
     "build:docs": "wireit",
     "build:tsc": "wireit",
     "build:types": "wireit",
     "build": "wireit",
     "check": "tsx tools/ensure-correct-devtools-protocol-package",
-    "clean": "tsc -b --clean && rimraf lib src/generated",
+    "clean": "tsc -b --clean && rm -rf lib src/generated",
     "generate:package-json": "wireit",
     "generate:sources": "wireit",
     "prepack": "wireit"
@@ -138,7 +138,6 @@
     "extract-zip": "2.0.1",
     "https-proxy-agent": "5.0.1",
     "proxy-from-env": "1.1.0",
-    "rimraf": "3.0.2",
     "tar-fs": "2.1.1",
     "unbzip2-stream": "1.4.3",
     "ws": "8.11.0"

packages/puppeteer-core/src/node/BrowserFetcher.ts

@@ -16,7 +16,7 @@
 
 import {exec as execChildProcess} from 'child_process';
 import {createReadStream, createWriteStream, existsSync, readdirSync} from 'fs';
-import {chmod, mkdir, readdir, unlink} from 'fs/promises';
+import {chmod, mkdir, readdir, rm, unlink} from 'fs/promises';
 import * as http from 'http';
 import * as https from 'https';
 import * as os from 'os';
@@ -31,7 +31,6 @@ import createHttpsProxyAgent, {
   HttpsProxyAgentOptions,
 } from 'https-proxy-agent';
 import {getProxyForUrl} from 'proxy-from-env';
-import removeRecursive from 'rimraf';
 import tar from 'tar-fs';
 import bzip from 'unbzip2-stream';
 
@@ -404,7 +403,7 @@ export class BrowserFetcher {
    * @remarks
    * This method is affected by the current `product`.
    * @param revision - A revision to remove for the current `product`.
-   * @returns A promise that resolves when the revision has been removes or
+   * @returns A promise that resolves when the revision has been removed or
    * throws if the revision has not been downloaded.
    */
   async remove(revision: string): Promise<void> {
@@ -413,9 +412,7 @@ export class BrowserFetcher {
       existsSync(folderPath),
       `Failed to remove: revision ${revision} is not downloaded`
     );
-    await new Promise(fulfill => {
-      return removeRecursive(folderPath, fulfill);
-    });
+    await rm(folderPath, {recursive: true, force: true});
   }
 
   /**

packages/puppeteer-core/src/node/BrowserRunner.ts

@@ -16,12 +16,11 @@
 
 import * as childProcess from 'child_process';
 import * as fs from 'fs';
+import {rm} from 'fs/promises';
 import * as path from 'path';
 import * as readline from 'readline';
 import {promisify} from 'util';
 
-import removeFolder from 'rimraf';
-
 import type {Connection as BiDiConnection} from '../common/bidi/bidi.js';
 import {Connection} from '../common/Connection.js';
 import {debug} from '../common/Debug.js';
@@ -40,7 +39,6 @@ import {isErrnoException, isErrorLike} from '../util/ErrorLike.js';
 import {LaunchOptions} from './LaunchOptions.js';
 import {PipeTransport} from './PipeTransport.js';
 
-const removeFolderAsync = promisify(removeFolder);
 const renameAsync = promisify(fs.rename);
 const unlinkAsync = promisify(fs.unlink);
 
@@ -126,7 +124,7 @@ export class BrowserRunner {
         // Cleanup as processes exit.
         if (this.#isTempUserDataDir) {
           try {
-            await removeFolderAsync(this.#userDataDir);
+            await rm(this.#userDataDir, {recursive: true, force: true});
             fulfill();
           } catch (error) {
             debugError(error);
@@ -240,7 +238,7 @@ export class BrowserRunner {
     // Attempt to remove temporary profile directory to avoid littering.
     try {
       if (this.#isTempUserDataDir) {
-        removeFolder.sync(this.#userDataDir);
+        fs.rmSync(this.#userDataDir, {recursive: true, force: true});
       }
     } catch (error) {}
 

packages/puppeteer/package.json

@@ -31,14 +31,14 @@
     "url": "https://github.com/puppeteer/puppeteer/tree/main/packages/puppeteer"
   },
   "engines": {
-    "node": ">=14.1.0"
+    "node": ">=14.14.0"
   },
   "scripts": {
     "build:docs": "wireit",
     "build:tsc": "wireit",
     "build:types": "wireit",
     "build": "wireit",
-    "clean": "tsc -b --clean && rimraf lib",
+    "clean": "tsc -b --clean && rm -rf lib",
     "generate:package-json": "wireit",
     "postinstall": "node install.js",
     "prepack": "wireit"

packages/testserver/package.json

@@ -5,7 +5,7 @@
   "main": "lib/index.js",
   "scripts": {
     "build": "wireit",
-    "clean": "tsc -b --clean && rimraf lib"
+    "clean": "tsc -b --clean && rm -rf lib"
   },
   "wireit": {
     "build": {

test/installation/package.json

@@ -6,7 +6,7 @@
   "scripts": {
     "build": "wireit",
     "build:packages": "wireit",
-    "clean": "tsc -b --clean && rimraf build",
+    "clean": "tsc -b --clean && rm -rf build",
     "test": "mocha"
   },
   "wireit": {

test/package.json

@@ -4,7 +4,7 @@
   "private": true,
   "scripts": {
     "build": "wireit",
-    "clean": "tsc -b --clean && rimraf build"
+    "clean": "tsc -b --clean && rm -rf build"
   },
   "wireit": {
     "build": {

test/src/headful.spec.ts

@@ -15,6 +15,7 @@
  */
 
 import fs from 'fs';
+import {rm} from 'fs/promises';
 import os from 'os';
 import path from 'path';
 import {promisify} from 'util';
@@ -24,11 +25,9 @@ import {
   PuppeteerLaunchOptions,
   PuppeteerNode,
 } from 'puppeteer-core/internal/node/PuppeteerNode.js';
-import rimraf from 'rimraf';
 
 import {getTestState} from './mocha-utils.js';
 
-const rmAsync = promisify(rimraf);
 const mkdtempAsync = promisify(fs.mkdtemp);
 
 const TMP_FOLDER = path.join(os.tmpdir(), 'pptr_tmp_folder-');
@@ -241,7 +240,7 @@ describe('headful tests', function () {
       });
       await headlessBrowser.close();
       // This might throw. See https://github.com/puppeteer/puppeteer/issues/2778
-      await rmAsync(userDataDir).catch(() => {});
+      await rm(userDataDir, {recursive: true, force: true}).catch(() => {});
       expect(cookie).toBe('foo=true');
     });
     // TODO: Support OOOPIF. @see https://github.com/puppeteer/puppeteer/issues/2548

test/src/launcher.spec.ts

@@ -14,6 +14,7 @@
  * limitations under the License.
  */
 import fs from 'fs';
+import {rm} from 'fs/promises';
 import os from 'os';
 import path from 'path';
 import {TLSSocket} from 'tls';
@@ -23,15 +24,16 @@ import {Protocol} from 'devtools-protocol';
 import expect from 'expect';
 import {BrowserFetcher, TimeoutError} from 'puppeteer';
 import {Page} from 'puppeteer-core/internal/api/Page.js';
-import rimraf from 'rimraf';
 import sinon from 'sinon';
 
 import {getTestState, itOnlyRegularInstall} from './mocha-utils.js';
 import utils from './utils.js';
 
+const rmAsync = (filename: string) => {
+  return rm(filename, {force: true, recursive: true});
+};
 const mkdtempAsync = promisify(fs.mkdtemp);
 const readFileAsync = promisify(fs.readFile);
-const rmAsync = promisify(rimraf);
 const statAsync = promisify(fs.stat);
 const writeFileAsync = promisify(fs.writeFile);
 

test/src/mocha-utils.ts

@@ -34,7 +34,6 @@ import {
 } from 'puppeteer-core/internal/node/PuppeteerNode.js';
 import {isErrorLike} from 'puppeteer-core/internal/util/ErrorLike.js';
 import puppeteer from 'puppeteer/lib/cjs/puppeteer/puppeteer.js';
-import rimraf from 'rimraf';
 import sinon from 'sinon';
 
 import {extendExpectWithToBeGolden} from './utils.js';
@@ -134,7 +133,7 @@ const setupGoldenAssertions = (): void => {
   const GOLDEN_DIR = path.join(__dirname, `../golden-${suffix}`);
   const OUTPUT_DIR = path.join(__dirname, `../output-${suffix}`);
   if (fs.existsSync(OUTPUT_DIR)) {
-    rimraf.sync(OUTPUT_DIR);
+    fs.rmSync(OUTPUT_DIR, {recursive: true, force: true});
   }
   extendExpectWithToBeGolden(GOLDEN_DIR, OUTPUT_DIR);
 };