@bigbinary/neeto-playwright-commons 4.1.11 → 4.1.13

package/index.js

@@ -116561,6 +116561,11 @@ class ColorPickerUtils {
 }
 
 const STATIC_ASSET_PATTERN = /\.(js|css|woff2?|ttf|eot|png|svg|ico|gif|webp)(\?.*)?$/;
+const EXCLUDED_RESPONSE_HEADERS = [
+    "content-encoding",
+    "content-length",
+    "transfer-encoding",
+];
 const assetCache = new Map();
 const commands = {
     context: async ({ context }, use) => {
@@ -116571,14 +116576,26 @@ const commands = {
                 if (hit)
                     return route.fulfill(hit);
                 try {
-                    const response = await route.fetch();
-                    const body = await response.body();
-                    const entry = {
-                        body,
-                        status: response.status(),
-                        headers: response.headers(),
-                    };
-                    if (response.ok())
+                    // Use Node's native fetch instead of route.fetch() to keep these
+                    // requests out of the Playwright trace's Actions tab. Forward the
+                    // browser's request headers so cookie/session-scoped assets still
+                    // resolve; drop accept-encoding so we only get encodings Node can
+                    // decode, and HTTP/2 pseudo-headers which fetch rejects.
+                    const requestHeaders = Object.entries(await route.request().allHeaders()).filter(([name]) => name !== "accept-encoding" && !name.startsWith(":"));
+                    const response = await fetch(url, {
+                        headers: Object.fromEntries(requestHeaders),
+                    });
+                    const body = Buffer.from(await response.arrayBuffer());
+                    // Node's fetch decompresses the body but keeps the original
+                    // content-encoding/content-length headers; drop them so the
+                    // browser doesn't try to decode the already-decoded body.
+                    const headers = {};
+                    response.headers.forEach((value, key) => {
+                        if (!EXCLUDED_RESPONSE_HEADERS.includes(key))
+                            headers[key] = value;
+                    });
+                    const entry = { body, status: response.status, headers };
+                    if (response.ok)
                         assetCache.set(url, entry);
                     return route.fulfill(entry);
                 }
@@ -116722,7 +116739,8 @@ const i18nFixture = {
     i18n: [
         async ({}, use) => {
             const translation = readTranslations();
-            const page = await (await chromium.launch()).newPage();
+            const browser = await chromium.launch();
+            const page = await browser.newPage();
             const taxonomies = await mergeTaxonomies(translation, page);
             const options = {
                 debug: false,
@@ -116755,16 +116773,12 @@ const i18nFixture = {
                 cache: true,
             });
             await page.close();
+            await browser.close();
             await use(i18nInitialized);
         },
         { auto: true, scope: "worker" },
     ],
-    t: [
-        async ({ i18n }, use) => {
-            await use(i18n.t);
-        },
-        { scope: "worker" },
-    ],
+    t: [({ i18n }, use) => use(i18n.t), { scope: "worker" }],
 };
 
 class ThankYouPage {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bigbinary/neeto-playwright-commons",
-  "version": "4.1.11",
+  "version": "4.1.13",
   "description": "A package encapsulating common playwright code across neeto projects.",
   "repository": "git@github.com:bigbinary/neeto-playwright-commons.git",
   "license": "apache-2.0",