feat: export public types only

.eslintignore

@@ -5,4 +5,4 @@ doclint/
 lib/
 test-ts-types/
 tsconfig.tsbuildinfo
-vendor/
+vendor/

.gitignore

@@ -13,6 +13,7 @@ docs/api.html
 lib/
 node_modules/
 package-lock.json
+puppeteer.api.json
 puppeteer*.tgz
 test-ts-types/**/dist/
 test-ts-types/**/node_modules

api-extractor.json

@@ -1,6 +1,6 @@
 {
   "$schema": "https://developer.microsoft.com/json-schemas/api-extractor/v7/api-extractor.schema.json",
-  "mainEntryPointFilePath": "<projectFolder>/lib/cjs/puppeteer/api-docs-entry.d.ts",
+  "mainEntryPointFilePath": "<projectFolder>/lib/esm/puppeteer/types.d.ts",
   "bundledPackages": [],
 
   "apiReport": {
@@ -9,12 +9,13 @@
 
   "docModel": {
     "enabled": true,
-    "apiJsonFilePath": "<projectFolder>/docs-api-json/<unscopedPackageName>.api.json"
+    "apiJsonFilePath": "<projectFolder>/docs/<unscopedPackageName>.api.json"
   },
 
   "dtsRollup": {
     "enabled": true,
-    "untrimmedFilePath": "lib/types.d.ts"
+    "untrimmedFilePath": "",
+    "publicTrimmedFilePath": "lib/types.d.ts"
   },
 
   "tsdocMetadata": {

compat/cjs/compat.ts

@@ -1,5 +1,8 @@
 import {dirname} from 'path';
 
+/**
+ * @internal
+ */
 let puppeteerDirname: string;
 
 try {

compat/esm/compat.ts

@@ -4,6 +4,9 @@ import {fileURLToPath} from 'url';
 
 const require = createRequire(import.meta.url);
 
+/**
+ * @internal
+ */
 let puppeteerDirname: string;
 
 try {

src/common/BrowserWebSocketTransport.ts

@@ -15,6 +15,9 @@
  */
 import {ConnectionTransport} from './ConnectionTransport.js';
 
+/**
+ * @internal
+ */
 export class BrowserWebSocketTransport implements ConnectionTransport {
   static create(url: string): Promise<BrowserWebSocketTransport> {
     return new Promise((resolve, reject) => {

src/common/DOMWorld.ts

@@ -24,7 +24,7 @@ import {Frame, FrameManager} from './FrameManager.js';
 import {MouseButton} from './Input.js';
 import {JSHandle} from './JSHandle.js';
 import {LifecycleWatcher, PuppeteerLifeCycleEvent} from './LifecycleWatcher.js';
-import {_getQueryHandlerAndSelector} from './QueryHandler.js';
+import {getQueryHandlerAndSelector} from './QueryHandler.js';
 import {TimeoutSettings} from './TimeoutSettings.js';
 import {EvaluateFunc, HandleFor} from './types.js';
 import {
@@ -653,7 +653,7 @@ export class DOMWorld {
     options: WaitForSelectorOptions
   ): Promise<ElementHandle | null> {
     const {updatedSelector, queryHandler} =
-      _getQueryHandlerAndSelector(selector);
+      getQueryHandlerAndSelector(selector);
     assert(queryHandler.waitFor, 'Query handler does not support waiting');
     return queryHandler.waitFor(this, updatedSelector, options);
   }

src/common/Debug.ts

@@ -25,14 +25,10 @@ declare global {
  * A debug function that can be used in any environment.
  *
  * @remarks
- *
  * If used in Node, it falls back to the
  * {@link https://www.npmjs.com/package/debug | debug module}. In the browser it
  * uses `console.log`.
  *
- * @param prefix - this will be prefixed to each log.
- * @returns a function that can be called to log to that debug channel.
- *
  * In Node, use the `DEBUG` environment variable to control logging:
  *
  * ```
@@ -56,6 +52,11 @@ declare global {
  * log('new page created')
  * // logs "Page: new page created"
  * ```
+ *
+ * @param prefix - this will be prefixed to each log.
+ * @returns a function that can be called to log to that debug channel.
+ *
+ * @internal
  */
 export const debug = (prefix: string): ((...args: unknown[]) => void) => {
   if (isNode) {

src/common/DeviceDescriptors.ts

@@ -30,7 +30,7 @@ export interface Device {
   };
 }
 
-const devices: Device[] = [
+const deviceArray: Device[] = [
   {
     name: 'Blackberry PlayBook',
     userAgent:
@@ -1536,10 +1536,30 @@ export type DevicesMap = {
 };
 
 /**
- * @internal
+ * A list of devices to be used with `page.emulate(options)`. Actual list of devices can be found in {@link https://github.com/puppeteer/puppeteer/blob/main/src/common/DeviceDescriptors.ts | src/common/DeviceDescriptors.ts}.
+ *
+ * @example
+ *
+ * ```js
+ * const puppeteer = require('puppeteer');
+ * const iPhone = puppeteer.devices['iPhone 6'];
+ *
+ * (async () => {
+ *   const browser = await puppeteer.launch();
+ *   const page = await browser.newPage();
+ *   await page.emulate(iPhone);
+ *   await page.goto('https://www.google.com');
+ *   // other actions...
+ *   await browser.close();
+ * })();
+ * ```
+ *
+ * @public
  */
-export const _devicesMap: DevicesMap = {};
+const devices: DevicesMap = {};
 
-for (const device of devices) {
-  _devicesMap[device.name] = device;
+for (const device of deviceArray) {
+  devices[device.name] = device;
 }
+
+export {devices};

src/common/ElementHandle.ts

@@ -8,14 +8,13 @@ import {
   BoundingBox,
   BoxModel,
   ClickOptions,
-  computeQuadArea,
   JSHandle,
   Offset,
   Point,
   PressOptions,
 } from './JSHandle.js';
 import {Page, ScreenshotOptions} from './Page.js';
-import {_getQueryHandlerAndSelector} from './QueryHandler.js';
+import {getQueryHandlerAndSelector} from './QueryHandler.js';
 import {EvaluateFunc} from './types.js';
 import {KeyInput} from './USKeyboardLayout.js';
 import {debugError, isString} from './util.js';
@@ -841,7 +840,7 @@ export class ElementHandle<
   async $(selector: string): Promise<ElementHandle | null>;
   async $(selector: string): Promise<ElementHandle | null> {
     const {updatedSelector, queryHandler} =
-      _getQueryHandlerAndSelector(selector);
+      getQueryHandlerAndSelector(selector);
     assert(
       queryHandler.queryOne,
       'Cannot handle queries for a single element with the given selector'
@@ -866,7 +865,7 @@ export class ElementHandle<
   async $$(selector: string): Promise<ElementHandle[]>;
   async $$(selector: string): Promise<ElementHandle[]> {
     const {updatedSelector, queryHandler} =
-      _getQueryHandlerAndSelector(selector);
+      getQueryHandlerAndSelector(selector);
     assert(
       queryHandler.queryAll,
       'Cannot handle queries for a multiple element with the given selector'
@@ -986,7 +985,7 @@ export class ElementHandle<
     ...args: Params
   ): Promise<Awaited<ReturnType<Func>>> {
     const {updatedSelector, queryHandler} =
-      _getQueryHandlerAndSelector(selector);
+      getQueryHandlerAndSelector(selector);
     assert(queryHandler.queryAllArray);
     const arrayHandle = await queryHandler.queryAllArray(this, updatedSelector);
     const result = await arrayHandle.evaluate(pageFunction, ...args);
@@ -1046,3 +1045,16 @@ export class ElementHandle<
     }, threshold);
   }
 }
+
+function computeQuadArea(quad: Point[]): number {
+  /* Compute sum of all directed areas of adjacent triangles
+     https://en.wikipedia.org/wiki/Polygon#Simple_polygons
+   */
+  let area = 0;
+  for (let i = 0; i < quad.length; ++i) {
+    const p1 = quad[i]!;
+    const p2 = quad[(i + 1) % quad.length]!;
+    area += (p1.x * p2.y - p2.x * p1.y) / 2;
+  }
+  return Math.abs(area);
+}

src/common/EmulationManager.ts

@@ -17,6 +17,9 @@ import {CDPSession} from './Connection.js';
 import {Viewport} from './PuppeteerViewport.js';
 import {Protocol} from 'devtools-protocol';
 
+/**
+ * @internal
+ */
 export class EmulationManager {
   #client: CDPSession;
   #emulatingMobile = false;