@skyramp/skyramp 1.3.21 → 1.3.23

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@skyramp/skyramp",
-  "version": "1.3.21",
+  "version": "1.3.23",
   "description": "module for leveraging skyramp cli functionality",
   "scripts": {
     "lint": "eslint 'src/**/*.js' 'src/**/*.ts' --fix",

package/src/classes/SmartPlaywright.js

@@ -853,6 +853,21 @@ class SkyrampPlaywrightPage {
             return Promise.resolve();
         });
 
+        // Enable filechooser interception upfront. Without this, generated tests
+        // that use page.waitForEvent('filechooser') register the listener inline
+        // right before the triggering click, which races with native dialog
+        // rendering on macOS headed runs and leaves the OS file picker visible
+        // until dismissed manually. Registering an empty listener here means
+        // the browser is told "intercept filechoosers" before any click fires,
+        // so the native dialog never paints. The test's own waitForEvent and
+        // setFiles path still runs unchanged; this listener does not consume
+        // the FileChooser object. Guarded with a per-page flag so re-wrapping
+        // the same page does not register duplicates.
+        if (!this._page.__skyrampFileChooserInstalled) {
+            this._page.__skyrampFileChooserInstalled = true;
+            this._page.on('filechooser', () => {});
+        }
+
         return new Proxy(this, {
             // The `get` trap is the key to forwarding.
             // This will foraward any methods not implemented in this struct