@lvce-editor/test-with-playwright-worker 22.38.3 → 22.38.5

package/dist/coverageWorkerMain.js

@@ -509,7 +509,7 @@ const applyDirectProperties = (restoredError, error) => {
   }
 };
 const restoreMessageError = (error, _currentStack) => {
-  const restoredError = constructError(error.message, error.type, error.name);
+  const restoredError = constructError(error.message, error.type || error.data?.type, error.name);
   if (typeof error.code === 'string' || typeof error.code === 'number') {
     Object.defineProperty(restoredError, 'code', {
       configurable: true,
@@ -597,7 +597,7 @@ const getErrorProperty = (error, prettyError) => {
     data: {
       code: prettyError.code ?? error?.code,
       codeFrame: prettyError.codeFrame,
-      name: prettyError.name,
+      name: prettyError.name ?? error?.name,
       stack: getStack(prettyError),
       type: getErrorType(prettyError)
     },

package/dist/workerMain.js

@@ -3,10 +3,10 @@ import { readFile, writeFile, readdir, rm, mkdir, access, mkdtemp } from 'node:f
 import { fileURLToPath, pathToFileURL } from 'node:url';
 import net from 'node:net';
 import os, { tmpdir } from 'node:os';
-import { fork } from 'node:child_process';
 import { createRequire } from 'node:module';
+import { fork } from 'node:child_process';
 
-const require$1 = createRequire(import.meta.url);
+const require$2 = createRequire(import.meta.url);
 const before = `      _onWebSocketOpened(event) {
         const request2 = this._webSocketRequests.get(event.requestId);
         assert(request2);
@@ -61,7 +61,7 @@ const patchCoreBundleContent = content => {
   return content.split(before).join(after);
 };
 const getCoreBundlePath = () => {
-  const packageJsonPath = require$1.resolve('playwright-core/package.json');
+  const packageJsonPath = require$2.resolve('playwright-core/package.json');
   return join(dirname(packageJsonPath), 'lib', 'coreBundle.js');
 };
 const patchPlaywrightFirefoxWorkerWebSocket = async () => {
@@ -770,7 +770,7 @@ const applyDirectProperties = (restoredError, error) => {
   }
 };
 const restoreMessageError = (error, _currentStack) => {
-  const restoredError = constructError(error.message, error.type, error.name);
+  const restoredError = constructError(error.message, error.type || error.data?.type, error.name);
   if (typeof error.code === 'string' || typeof error.code === 'number') {
     Object.defineProperty(restoredError, 'code', {
       configurable: true,
@@ -858,7 +858,7 @@ const getErrorProperty = (error, prettyError) => {
     data: {
       code: prettyError.code ?? error?.code,
       codeFrame: prettyError.codeFrame,
-      name: prettyError.name,
+      name: prettyError.name ?? error?.name,
       stack: getStack(prettyError),
       type: getErrorType(prettyError)
     },
@@ -2263,6 +2263,28 @@ const getServerPath = async serverPath => {
   throw new Error(`[test-with-playwright] server path not found`);
 };
 
+const require$1 = createRequire(import.meta.url);
+const disabledFeaturesPrefix = '--disable-features=';
+const getBrowserLaunchArgs = (browser, platform = process.platform) => {
+  if (browser !== 'chromium' || platform !== 'win32') {
+    return [];
+  }
+  // SO_RANDOMIZE_PORT intermittently fails with WSAENOBUFS on Windows.
+  // Read Playwright's defaults so this override preserves its other disabled features.
+  const {
+    server
+  } = require$1('playwright-core/lib/coreBundle');
+  const playwright = server.createPlaywright({
+    sdkLanguage: 'javascript'
+  });
+  const defaultArgs = playwright.chromium._innerDefaultArgs({});
+  const disabledFeatures = defaultArgs.find(argument => argument.startsWith(disabledFeaturesPrefix));
+  if (!disabledFeatures) {
+    throw new Error('Could not find Playwright Chromium disabled features');
+  }
+  return [`${disabledFeatures},TcpPortRandomizationWin`];
+};
+
 /**
  *
  * @param {{browser:'chromium'|'firefox'|'webkit', signal:AbortSignal, headless:boolean}} options
@@ -2292,20 +2314,40 @@ const startBrowser = async ({
     await patchPlaywrightFirefoxWorkerWebSocket();
   }
   const launcher = await getLauncher(browser);
-  const browserInstance = await launcher.launch({
+  signal.throwIfAborted();
+  const options = {
+    args: getBrowserLaunchArgs(browser),
     headless
-  });
-  const page = await browserInstance.newPage();
-  // eslint-disable-next-line @typescript-eslint/no-misused-promises
-  signal.addEventListener('abort', async () => {
-    await page.close();
-    await browserInstance.close();
-  });
-  // @ts-ignore
-  return {
-    browser: browserInstance,
-    page
   };
+  // Chromium 153 crashes when restoring OPFS handles from its in-memory IndexedDB backend.
+  // An empty userDataDir gives each run a temporary disk-backed profile managed by Playwright.
+  const browserInstance = browser === 'chromium' ? await launcher.launchPersistentContext('', options) : await launcher.launch(options);
+  let disposal;
+  const dispose = () => {
+    signal.removeEventListener('abort', handleAbort);
+    disposal ||= browserInstance.close();
+    return disposal;
+  };
+  const handleAbort = () => {
+    // The caller awaits disposal during teardown, including any cleanup error.
+    void dispose().catch(() => {});
+  };
+  signal.addEventListener('abort', handleAbort, {
+    once: true
+  });
+  try {
+    signal.throwIfAborted();
+    const page = await browserInstance.newPage();
+    signal.throwIfAborted();
+    return {
+      browser: browserInstance,
+      dispose,
+      page
+    };
+  } catch (error) {
+    await dispose();
+    throw error;
+  }
 };
 
 const getServerArgs = link => {
@@ -2368,29 +2410,36 @@ const setupTests = async ({
   const port = await getPort();
   const {
     browser: browserInstance,
+    dispose,
     page
   } = await startBrowser({
     browser,
     headless,
     signal
   });
-  const resolvedServerPath = await getServerPath(serverPath);
-  const child = await startServer({
-    ...(link && link.length > 0 && {
-      link
-    }),
-    onlyExtension,
-    port,
-    serverPath: resolvedServerPath,
-    signal,
-    testPath
-  });
-  return {
-    browser: browserInstance,
-    child,
-    page,
-    port
-  };
+  try {
+    const resolvedServerPath = await getServerPath(serverPath);
+    const child = await startServer({
+      ...(link && link.length > 0 && {
+        link
+      }),
+      onlyExtension,
+      port,
+      serverPath: resolvedServerPath,
+      signal,
+      testPath
+    });
+    return {
+      browser: browserInstance,
+      child,
+      dispose,
+      page,
+      port
+    };
+  } catch (error) {
+    await dispose();
+    throw error;
+  }
 };
 
 const getProcessEnv = () => {
@@ -2636,6 +2685,7 @@ const runAllTests = async (extensionPath, testPath, cwd, browser, headless, time
   }
   const {
     child,
+    dispose,
     page,
     port
   } = await setupTests({
@@ -2704,10 +2754,14 @@ const runAllTests = async (extensionPath, testPath, cwd, browser, headless, time
       }
     });
   } finally {
-    await tearDownTests({
-      child,
-      controller
-    });
+    try {
+      await tearDownTests({
+        child,
+        controller
+      });
+    } finally {
+      await dispose();
+    }
   }
 };
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lvce-editor/test-with-playwright-worker",
-  "version": "22.38.3",
+  "version": "22.38.5",
   "description": "Worker package for test-with-playwright",
   "repository": {
     "type": "git",