@agentium/browser 2.1.0 → 2.1.1

package/dist/index.cjs

@@ -118,29 +118,39 @@ function getStealthScript() {
       };
     }
 
-    // \u2500\u2500 Prevent iframe detection of automation \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500
-    Object.defineProperty(HTMLIFrameElement.prototype, 'contentWindow', {
-      get: function() {
-        return window;
-      },
-    });
-
-    // \u2500\u2500 Remove "cdc_" Playwright/ChromeDriver markers from DOM \u2500\u2500\u2500\u2500\u2500\u2500\u2500
-    const observer = new MutationObserver((mutations) => {
-      for (const mutation of mutations) {
-        for (const node of mutation.addedNodes) {
-          if (node.nodeType === 1) {
-            const el = node;
-            for (const attr of [...el.attributes]) {
-              if (attr.name.startsWith('cdc_') || attr.name.startsWith('__playwright')) {
-                el.removeAttribute(attr.name);
+    // \u2500\u2500 Remove "cdc_" Playwright/ChromeDriver markers from new elements \u2500
+    // Only watches childList additions, NOT every attribute mutation \u2014
+    // attributes:true subtree:true is O(every DOM write) on heavy SPAs
+    // like FreightOS and noticeably slows the page.
+    try {
+      const observer = new MutationObserver((mutations) => {
+        for (const mutation of mutations) {
+          for (const node of mutation.addedNodes) {
+            if (node && node.nodeType === 1) {
+              const el = node;
+              if (!el.attributes) continue;
+              for (const attr of [...el.attributes]) {
+                if (attr.name.startsWith('cdc_') || attr.name.startsWith('__playwright')) {
+                  el.removeAttribute(attr.name);
+                }
               }
             }
           }
         }
+      });
+      if (document.documentElement) {
+        observer.observe(document.documentElement, { childList: true, subtree: true });
       }
-    });
-    observer.observe(document.documentElement, { attributes: true, childList: true, subtree: true });
+    } catch (_e) { /* never break the page over a stealth patch */ }
+
+    // NOTE on iframe.contentWindow:
+    //   We deliberately do NOT override HTMLIFrameElement.prototype.contentWindow.
+    //   An earlier version returned the parent window, which breaks legitimate
+    //   page JS on any site that uses iframes (checkout widgets, embedded
+    //   videos, A/B test scripts, analytics). The "anti-detection" value was
+    //   minimal \u2014 modern detectors don't rely on that property anyway \u2014 and
+    //   the breakage caused pages to fail to boot, leaving font-test residue
+    //   on screen.
   `;
 }
 function buildStealthContextOpts(config, viewport) {
@@ -226,14 +236,20 @@ var BrowserProvider = class {
     }
     const stealthEnabled = !!opts?.stealth;
     const stealthCfg = typeof opts?.stealth === "object" ? opts.stealth : {};
-    const launchOpts = {
-      headless: opts?.headless ?? true
-    };
+    const headless = opts?.headless ?? true;
+    const launchOpts = { headless };
+    const args = [];
     if (stealthEnabled) {
-      const { args, proxy } = buildStealthLaunchArgs(stealthCfg);
-      launchOpts.args = args;
-      if (proxy) launchOpts.proxy = proxy;
+      const stealth = buildStealthLaunchArgs(stealthCfg);
+      args.push(...stealth.args);
+      if (stealth.proxy) launchOpts.proxy = stealth.proxy;
+    }
+    if (!headless) {
+      const CHROME_VERTICAL_CHROME_PX = 140;
+      args.push(`--window-size=${this._viewport.width},${this._viewport.height + CHROME_VERTICAL_CHROME_PX}`);
+      args.push("--window-position=0,0");
     }
+    if (args.length) launchOpts.args = args;
     this.browser = await chromium.launch(launchOpts);
     let contextOpts;
     if (stealthEnabled) {

package/dist/index.js

@@ -80,29 +80,39 @@ function getStealthScript() {
       };
     }
 
-    // \u2500\u2500 Prevent iframe detection of automation \u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500\u2500
-    Object.defineProperty(HTMLIFrameElement.prototype, 'contentWindow', {
-      get: function() {
-        return window;
-      },
-    });
-
-    // \u2500\u2500 Remove "cdc_" Playwright/ChromeDriver markers from DOM \u2500\u2500\u2500\u2500\u2500\u2500\u2500
-    const observer = new MutationObserver((mutations) => {
-      for (const mutation of mutations) {
-        for (const node of mutation.addedNodes) {
-          if (node.nodeType === 1) {
-            const el = node;
-            for (const attr of [...el.attributes]) {
-              if (attr.name.startsWith('cdc_') || attr.name.startsWith('__playwright')) {
-                el.removeAttribute(attr.name);
+    // \u2500\u2500 Remove "cdc_" Playwright/ChromeDriver markers from new elements \u2500
+    // Only watches childList additions, NOT every attribute mutation \u2014
+    // attributes:true subtree:true is O(every DOM write) on heavy SPAs
+    // like FreightOS and noticeably slows the page.
+    try {
+      const observer = new MutationObserver((mutations) => {
+        for (const mutation of mutations) {
+          for (const node of mutation.addedNodes) {
+            if (node && node.nodeType === 1) {
+              const el = node;
+              if (!el.attributes) continue;
+              for (const attr of [...el.attributes]) {
+                if (attr.name.startsWith('cdc_') || attr.name.startsWith('__playwright')) {
+                  el.removeAttribute(attr.name);
+                }
               }
             }
           }
         }
+      });
+      if (document.documentElement) {
+        observer.observe(document.documentElement, { childList: true, subtree: true });
       }
-    });
-    observer.observe(document.documentElement, { attributes: true, childList: true, subtree: true });
+    } catch (_e) { /* never break the page over a stealth patch */ }
+
+    // NOTE on iframe.contentWindow:
+    //   We deliberately do NOT override HTMLIFrameElement.prototype.contentWindow.
+    //   An earlier version returned the parent window, which breaks legitimate
+    //   page JS on any site that uses iframes (checkout widgets, embedded
+    //   videos, A/B test scripts, analytics). The "anti-detection" value was
+    //   minimal \u2014 modern detectors don't rely on that property anyway \u2014 and
+    //   the breakage caused pages to fail to boot, leaving font-test residue
+    //   on screen.
   `;
 }
 function buildStealthContextOpts(config, viewport) {
@@ -188,14 +198,20 @@ var BrowserProvider = class {
     }
     const stealthEnabled = !!opts?.stealth;
     const stealthCfg = typeof opts?.stealth === "object" ? opts.stealth : {};
-    const launchOpts = {
-      headless: opts?.headless ?? true
-    };
+    const headless = opts?.headless ?? true;
+    const launchOpts = { headless };
+    const args = [];
     if (stealthEnabled) {
-      const { args, proxy } = buildStealthLaunchArgs(stealthCfg);
-      launchOpts.args = args;
-      if (proxy) launchOpts.proxy = proxy;
+      const stealth = buildStealthLaunchArgs(stealthCfg);
+      args.push(...stealth.args);
+      if (stealth.proxy) launchOpts.proxy = stealth.proxy;
+    }
+    if (!headless) {
+      const CHROME_VERTICAL_CHROME_PX = 140;
+      args.push(`--window-size=${this._viewport.width},${this._viewport.height + CHROME_VERTICAL_CHROME_PX}`);
+      args.push("--window-position=0,0");
     }
+    if (args.length) launchOpts.args = args;
     this.browser = await chromium.launch(launchOpts);
     let contextOpts;
     if (stealthEnabled) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agentium/browser",
-  "version": "2.1.0",
+  "version": "2.1.1",
   "description": "Browser automation agent for Agentium using Playwright",
   "license": "MIT",
   "repository": {
@@ -46,7 +46,7 @@
     "typescript": "^5.6.0"
   },
   "peerDependencies": {
-    "@agentium/core": "^2.1.0",
+    "@agentium/core": "^2.1.1",
     "playwright": ">=1.40.0"
   },
   "peerDependenciesMeta": {