test: Upgdate CI Puppeteer to latest

packages/storycap/package.json

@@ -79,7 +79,7 @@
     "@types/yargs": "^17.0.0",
     "jest": "29.7.0",
     "minimist": "1.2.8",
-    "puppeteer": "19.11.1",
+    "puppeteer": "23.2.1",
     "ts-jest": "29.1.2",
     "typedoc": "0.25.13",
     "typescript": "5.4.5"

packages/storycap/src/node/capturing-browser.ts

@@ -372,7 +372,8 @@ export class CapturingBrowser extends StoryPreviewBrowser {
     let emittedScreenshotOptions: ScreenshotOptions | undefined;
     this.resourceWatcher.clear();
 
-    function onConsoleLog(msg: ConsoleMessage) {
+    function onConsoleLog(_msg: any) {
+      const msg = _msg as ConsoleMessage;
       const niceMessage = `From ${requestId} (${msg.type()}): ${msg.text()}`;
 
       if (forwardConsoleLogs) {
@@ -468,6 +469,8 @@ export class CapturingBrowser extends StoryPreviewBrowser {
       let buffer: Buffer | null = null;
       if (Buffer.isBuffer(rawBuffer)) {
         buffer = rawBuffer;
+      } else if (rawBuffer instanceof Uint8Array) {
+        buffer = Buffer.from(rawBuffer);
       }
 
       // We should reset elements state(e.g. focusing, hovering, clicking) for future screenshot for this story.
@@ -490,7 +493,9 @@ export class CapturingBrowser extends StoryPreviewBrowser {
 
         // Calculate the suffix and save the trace to the file.
         const suffix = variantKey.isDefault && defaultVariantSuffix ? [defaultVariantSuffix] : variantKey.keys;
-        await fileSystem.saveTrace(story.kind, story.story, suffix, traceBuffer);
+        if (traceBuffer) {
+          await fileSystem.saveTrace(story.kind, story.story, suffix, Buffer.from(traceBuffer));
+        }
       }
     }
   }

packages/storycrawler/package.json

@@ -50,7 +50,7 @@
   },
   "dependencies": {
     "chalk": "^2.4.1",
-    "puppeteer-core": "^9.0.0",
+    "puppeteer-core": "^23.2.1",
     "wait-on": "^7.0.0"
   },
   "jest": {

packages/storycrawler/src/browser/devices.ts

@@ -6,6 +6,7 @@ import type { Viewport } from 'puppeteer-core';
  *
  */
 export function getDeviceDescriptors() {
-  const dd = require('puppeteer-core').devices as Record<string, { name: string; viewport: Viewport }>;
+  const pc = require('puppeteer-core');
+  const dd = (pc.KnownDevices || pc.devices) as Record<string, { name: string; viewport: Viewport }>;
   return Object.values(dd);
 }

packages/storycrawler/src/browser/stories-browser.ts

@@ -1,3 +1,4 @@
+import { setTimeout as nodeSetTimeout } from 'node:timers/promises';
 import { BaseBrowser, BaseBrowserOptions } from './base-browser';
 import { Logger } from '../logger';
 import { NoStoriesError, StoriesTimeoutError } from '../errors';
@@ -62,12 +63,16 @@ export class StoriesBrowser extends BaseBrowser {
     this.logger.debug('Wait for stories definition.');
     await this.page.goto(this.connection.url);
     let stories: Story[] | null = null;
-    await this.page.goto(
+    // await this.page
+    //   .goto(this.connection.url + '/iframe.html?selectedKind=story-crawler-kind&selectedStory=story-crawler-story', {
+    //     timeout: 60_000,
+    //     waitUntil: 'domcontentloaded',
+    //   })
+    //   .catch(() => {
+    //     this.logger.warn('Timeout to open Storybook preview iframe.');
+    //   });
+    this.page.goto(
       this.connection.url + '/iframe.html?selectedKind=story-crawler-kind&selectedStory=story-crawler-story',
-      {
-        timeout: 60_000,
-        waitUntil: 'domcontentloaded',
-      },
     );
     await this.page.waitForFunction(
       () =>
@@ -77,7 +82,7 @@ export class StoriesBrowser extends BaseBrowser {
         timeout: 60_000,
       },
     );
-    await this.page.waitForTimeout(500);
+    await nodeSetTimeout(500);
     await this.page.evaluate(() => {
       const api = (window as ExposedWindow).__STORYBOOK_CLIENT_API__ || (window as ExposedWindow).__STORYBOOK_PREVIEW__;
       function isPreviewApi(api: API | PreviewAPI): api is PreviewAPI {