stably 4.12.16 → 4.12.17

package/dist/stably-browser.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-// ../../app/node_modules/.pnpm/@stablyai-internal+playwright-cli@0.4.30/node_modules/@stablyai-internal/playwright-cli/playwright-cli.js
+// ../../app/node_modules/.pnpm/@stablyai-internal+playwright-cli@0.4.32/node_modules/@stablyai-internal/playwright-cli/playwright-cli.js
 var fs = require("fs");
 var path = require("path");
 if (process.platform === "darwin" && !process.env.PLAYWRIGHT_DAEMON_SOCKETS_DIR) {
@@ -261,7 +261,10 @@ function createReconnectableBrowserProxy(playwright) {
           } catch (error) {
             if (!isRecoverableCdpError(error)) throw error;
             const freshBrowser = await reconnectBrowser(playwright);
-            return freshBrowser.newContext(...args);
+            // Prefer the default context (from connectOverCDP) over creating a new one.
+            // Creating a new context isolates cookies/localStorage from the default
+            // context where the auth login session lives, causing auth failures.
+            return freshBrowser.contexts()[0] || freshBrowser.newContext(...args);
           }
         };
       }
@@ -369,7 +372,12 @@ const test = realPw.test.extend({
     }
 
     // Apply storageState from the fixture chain (custom fixture or project config).
-    // This covers both: use: { storageState: 'path' } AND custom fixture overrides.
+    // Store the path so the page fixture can re-apply localStorage on new pages.
+    if (storageState && typeof storageState === 'string') {
+      const path = require('path');
+      process.env.__STABLY_STORAGE_STATE_PATH__ = path.isAbsolute(storageState)
+        ? storageState : path.resolve(process.cwd(), storageState);
+    }
     if (storageState) {
       try {
         // Try Playwright's native setStorageState first (newer versions)
@@ -405,39 +413,31 @@ const test = realPw.test.extend({
     // No close \u2014 preserve state for post-test inspection
   },
 
-  // Test-scoped: create a fresh page for each test to avoid stale CDP frame references.
-  // The previous test's page is closed to free resources. The new page gets a fresh
-  // CDP frame, avoiding the "second test always fails" issue where the reused page's
-  // internal frame reference becomes invalid between fixture teardown/setup cycles.
+  // Test-scoped: reuse the existing page for each test.
+  // Between tests, Playwright's frame tree can go stale \u2014 execution contexts are
+  // destroyed but not recreated. Navigating to about:blank triggers
+  // Runtime.executionContextDestroyed + Created, giving fresh contexts.
+  // Reusing the page (instead of creating new ones) ensures the daemon's MCP
+  // _currentTab always points to the actual test page with full state after run-test.
   page: async ({ context, browser, playwright, baseURL, extraHTTPHeaders, httpCredentials, geolocation, permissions, offline }, use, testInfo) => {
-    // Get or create a page. We reuse the existing page (don't close it \u2014 closing
-    // the last page on a CDP default context can terminate the browser process).
-    // Instead, navigate to about:blank to reset frame state.
-    let page = context.pages()[0];
-    if (!page) {
-      try {
-        page = await context.newPage();
-      } catch (error) {
-        if (!isRecoverableCdpError(error)) throw error;
-        const freshBrowser = await reconnectBrowser(playwright);
-        const ctx = freshBrowser.contexts()[0] || await freshBrowser.newContext();
-        page = ctx.pages()[0] || await ctx.newPage();
-      }
-    }
-    // Navigate to about:blank to get a fresh frame/document, avoiding stale
-    // frame references from the previous test. This is cheaper than closing
-    // and recreating the page (which can kill the CDP browser).
+    // Reuse the existing page instead of creating new pages.
+    // This ensures the daemon's MCP _currentTab always points to the actual
+    // test page with full state (DOM, cookies, localStorage, etc.).
+    // Navigate to about:blank between tests to refresh Playwright's frame tree
+    // (triggers Runtime.executionContextDestroyed + Created, fixing stale frames).
+    let page;
     try {
-      await page.goto('about:blank', { timeout: 5000 });
-    } catch {
-      // If navigation fails, the page might be stale \u2014 create a new one
-      try {
-        page = await context.newPage();
-      } catch (error) {
-        if (!isRecoverableCdpError(error)) throw error;
+      page = context.pages()[0];
+      if (!page) page = await context.newPage();
+      await page.goto('about:blank');
+    } catch (error) {
+      if (isRecoverableCdpError(error)) {
         const freshBrowser = await reconnectBrowser(playwright);
         const ctx = freshBrowser.contexts()[0] || await freshBrowser.newContext();
         page = ctx.pages()[0] || await ctx.newPage();
+        await page.goto('about:blank').catch(() => {});
+      } else {
+        throw error;
       }
     }
 
@@ -502,7 +502,7 @@ const test = realPw.test.extend({
       }
     }
     await use(page);
-    // No close \u2014 page must survive for snapshot
+    // No close \u2014 page must survive for daemon interaction after run-test.
   },
 });
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stably",
-  "version": "4.12.16",
+  "version": "4.12.17",
   "packageManager": "pnpm@10.24.0",
   "description": "AI-powered E2E Playwright testing CLI. Stably can understand your codebase, edit/run tests, and handle complex test scenarios for you.",
   "main": "dist/index.mjs",
@@ -87,7 +87,7 @@
     "playwright": "1.59.0-alpha-1771104257000",
     "@stablyai/codegen-agent-constants": "workspace:*",
     "@stablyai-internal/api-client": "workspace:*",
-    "@stablyai-internal/playwright-cli": "0.4.30",
+    "@stablyai-internal/playwright-cli": "0.4.32",
     "@stablyai-internal/pwtrace": "0.3.1",
     "@stablyai/agent-hooks": "workspace:*",
     "@stablyai/agent-schemas": "workspace:*",