Adds better support for CommonJS

README.md

@@ -1,7 +1,7 @@
 # Workers version of Puppeteer Core
 
 This repo is a fork of main puppeteer project. It creates a version of
-puppeteer core specialized for use in Cloudflare workers.
+puppeteer core specialized for use in Cloudflare workers [Browser Rendering](https://developers.cloudflare.com/browser-rendering) API.
 
 The goals of the fork are:
 
@@ -14,6 +14,13 @@ Note that the main branch in this repo is branched off of version 17.0.0 of
 the library, to match the currently deployed version of Chromium on the
 edge.
 
+### Requirements
+
+- You need [Wrangler](https://github.com/cloudflare/workers-sdk) 3.0.0 or higher.
+- Puppeteer requires a few Node.js APIs to work. Add `compatibility_flags = [ "nodejs_compat" ]` to your [wrangler.toml](https://developers.cloudflare.com/workers/wrangler/configuration/) configuration.
+
+Read our [Developer Documentation](https://developers.cloudflare.com/browser-rendering/platform/puppeteer/) for more information.
+
 Original README follows...
 
 # Puppeteer

compat/cjs/compat.ts

@@ -1,4 +1,4 @@
-import {dirname} from 'path';
+import {dirname} from 'node:path';
 
 /**
  * @internal

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@cloudflare/puppeteer",
-  "version": "0.0.4",
+  "version": "0.0.5",
   "description": "A high-level API to control headless Chrome over the DevTools Protocol for use in Workers",
   "keywords": [
     "puppeteer",

src/common/ElementHandle.ts

@@ -764,9 +764,9 @@ export class ElementHandle<
     );
 
     // Locate all files and confirm that they exist.
-    let path: typeof import('path');
+    let path: typeof import('node:path');
     try {
-      path = await import('path');
+      path = await import('node:path');
     } catch (error) {
       if (error instanceof TypeError) {
         throw new Error(

src/common/IsolatedWorld.ts

@@ -366,21 +366,9 @@ export class IsolatedWorld {
     }
 
     if (path !== null) {
-      let fs;
-      try {
-        fs = (await import('fs')).promises;
-      } catch (error) {
-        if (error instanceof TypeError) {
-          throw new Error(
-            'Can only pass a filepath to addScriptTag in a Node-like environment.'
-          );
-        }
-        throw error;
-      }
-      let contents = await fs.readFile(path, 'utf8');
-      contents += '//# sourceURL=' + path.replace(/\n/g, '');
-      const context = await this.executionContext();
-      return await context.evaluateHandle(addScriptContent, contents, id, type);
+      throw new Error(
+        'Can only pass a filepath to addScriptTag in a Node-like environment.'
+      );
     }
 
     if (content !== null) {

src/constants.ts

@@ -1,4 +1,4 @@
-import {dirname} from 'path';
+import {dirname} from 'node:path';
 import {puppeteerDirname} from './compat.js';
 
 /**

src/util/getPackageDirectory.ts

@@ -1,15 +1,5 @@
-import {existsSync} from 'fs';
-import {dirname, join, parse} from 'path';
-
 export const getPackageDirectory = (from: string): string => {
-  let found = existsSync(join(from, 'package.json'));
-  const root = parse(from).root;
-  while (!found) {
-    if (from === root) {
-      throw new Error('Cannot find package directory');
-    }
-    from = dirname(from);
-    found = existsSync(join(from, 'package.json'));
-  }
-  return from;
+  throw new Error(
+    `Can only use the filesystem in a Node-like environment (${from}).`
+  );
 };