stably 4.12.11 → 4.12.12

package/dist/node_modules/playwright/lib/mcp/browser/sessionLog.js

@@ -118,40 +118,6 @@ class SessionLog {
     lines.push("");
     this._sessionFileQueue = this._sessionFileQueue.then(() => import_fs.default.promises.appendFile(this._file, lines.join("\n")));
   }
-  logUserAction(action, tab, code, isUpdate) {
-    code = code.trim();
-    // Send recorder event for user action tracking
-    const actionForLog = { ...action };
-    delete actionForLog.ariaSnapshot;
-    delete actionForLog.selector;
-    actionForLog.isUpdate = !!isUpdate;
-    sendRecorderEvent({
-      type: "user-action",
-      action: actionForLog,
-      code,
-      url: tab.page.url(),
-      timestamp: Date.now()
-    });
-    // Also log to session file
-    const lines = [""];
-    lines.push(
-      `### User action: ${action.name}`,
-      "- Args",
-      "```json",
-      JSON.stringify(actionForLog, null, 2),
-      "```"
-    );
-    if (code) {
-      lines.push(
-        "- Code",
-        "```js",
-        code,
-        "```"
-      );
-    }
-    lines.push("");
-    this._sessionFileQueue = this._sessionFileQueue.then(() => import_fs.default.promises.appendFile(this._file, lines.join("\n")));
-  }
 }
 // Annotate the CommonJS export names for ESM import in node:
 0 && (module.exports = {

package/dist/stably-browser.js

@@ -1,6 +1,6 @@
 #!/usr/bin/env node
 
-// ../../app/node_modules/.pnpm/@stablyai-internal+playwright-cli@0.4.21/node_modules/@stablyai-internal/playwright-cli/playwright-cli.js
+// ../../app/node_modules/.pnpm/@stablyai-internal+playwright-cli@0.4.22/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) {
@@ -182,25 +182,134 @@ const userConfig = userConfigModule.default || userConfigModule;`;
   const fixturesCode = `
 // Auto-generated by stably-browser run-test
 // Overrides browser/context/page fixtures to connect via CDP and preserve state.
+//
+// WHY THIS EXISTS:
+// When run-test runs a user's Playwright tests through CDP with project dependencies like:
+//   auth \u2192 app  (app depends on auth completing first)
+//
+// The auth project may run sequential tests (e.g. "authenticate US" then "authenticate EU").
+// Both are test-scoped, meaning Playwright tears down context/page fixtures between them.
+// In CDP mode, fixtures reuse the browser's default context via browser.contexts()[0].
+//
+// THE BUG (before this fix):
+// After the first auth test completes, the CDP default context can become invalid.
+// When the second auth test's context fixture runs:
+//   1. browser.contexts()[0] \u2192 undefined (context gone)
+//   2. browser.newContext()  \u2192 throws "Target page, context or browser has been closed"
+//   3. This error kills the Chrome process (CDP port becomes unreachable)
+//   4. All retries fail with ECONNREFUSED, all downstream tests fail
+//
+// The second auth test WOULD have called setup.skip() in its body, but fixtures
+// are set up BEFORE the test body runs, so the skip never executes.
+//
+// THE FIX:
+// Wrap the browser in a Proxy that catches CDP errors and reconnects.
+// Context and page fixtures also catch errors and reconnect.
+// This lets the second auth test get a valid context \u2192 run its body \u2192 call skip().
+
 const realPw = require(${JSON.stringify(realPwTestPath)});
 
+const cdpEndpoint = process.env.PLAYWRIGHT_CDP_ENDPOINT;
+if (!cdpEndpoint) throw new Error('PLAYWRIGHT_CDP_ENDPOINT not set');
+
+// Shared mutable state: holds the current CDP browser connection.
+// When a connection goes stale, reconnectBrowser() replaces it.
+const cdpState = { browser: null };
+
+async function connectBrowser(playwright) {
+  cdpState.browser = await playwright.chromium.connectOverCDP(cdpEndpoint);
+  return cdpState.browser;
+}
+
+// Replace the current CDP connection with a fresh one.
+// Closes the old connection (best-effort) to avoid leaked handles.
+async function reconnectBrowser(playwright) {
+  const previousBrowser = cdpState.browser;
+  const nextBrowser = await connectBrowser(playwright);
+  if (previousBrowser && previousBrowser !== nextBrowser) {
+    try {
+      await previousBrowser.close();
+    } catch (_) {
+      // Ignore disconnect errors from the stale CDP connection we are replacing.
+    }
+  }
+  return nextBrowser;
+}
+
+// Detect errors that indicate the CDP connection is stale but Chrome may still be alive.
+function isRecoverableCdpError(error) {
+  return !!error && typeof error.message === 'string' &&
+    (error.message.includes('Target page, context or browser has been closed') ||
+     error.message.includes('Browser has been closed') ||
+     error.message.includes('Connection closed'));
+}
+
+// Wrap the browser object in a Proxy so that calls like browser.newContext()
+// automatically retry with a fresh CDP connection on recoverable errors.
+// This is transparent to callers \u2014 they get back a working context.
+function createReconnectableBrowserProxy(playwright) {
+  return new Proxy({}, {
+    get(_target, prop) {
+      const activeBrowser = cdpState.browser;
+      const value = activeBrowser?.[prop];
+      if (prop === 'newContext' && typeof value === 'function') {
+        return async (...args) => {
+          try {
+            return await value.apply(activeBrowser, args);
+          } catch (error) {
+            if (!isRecoverableCdpError(error)) throw error;
+            const freshBrowser = await reconnectBrowser(playwright);
+            return freshBrowser.newContext(...args);
+          }
+        };
+      }
+      return typeof value === 'function' ? value.bind(activeBrowser) : value;
+    },
+  });
+}
+
 const test = realPw.test.extend({
+  // Worker-scoped: one CDP connection per Playwright worker.
+  // Wrapped in a Proxy so newContext() auto-recovers from stale connections.
   browser: [async ({ playwright }, use) => {
-    const cdpEndpoint = process.env.PLAYWRIGHT_CDP_ENDPOINT;
-    if (!cdpEndpoint) throw new Error('PLAYWRIGHT_CDP_ENDPOINT not set');
-    const browser = await playwright.chromium.connectOverCDP(cdpEndpoint);
-    await use(browser);
+    await connectBrowser(playwright);
+    await use(createReconnectableBrowserProxy(playwright));
     // No close \u2014 daemon needs the connection alive
   }, { scope: 'worker', timeout: 0 }],
 
-  context: async ({ browser }, use) => {
-    const context = browser.contexts()[0] || await browser.newContext();
+  // Test-scoped: re-acquired per test from the shared browser.
+  // After auth test #1 tears down, the default context may be gone.
+  // The try/catch catches the "Target page, context or browser has been closed" error
+  // and reconnects before the test body (which would call setup.skip()) runs.
+  context: async ({ browser, playwright }, use) => {
+    let context = browser.contexts()[0];
+    if (!context) {
+      try {
+        context = await browser.newContext();
+      } catch (error) {
+        if (!isRecoverableCdpError(error)) throw error;
+        const freshBrowser = await reconnectBrowser(playwright);
+        context = freshBrowser.contexts()[0] || await freshBrowser.newContext();
+      }
+    }
     await use(context);
     // No close \u2014 preserve state for post-test inspection
   },
 
-  page: async ({ context }, use) => {
-    const page = context.pages()[0] || await context.newPage();
+  // Test-scoped: re-acquired per test from the current context.
+  // Same recovery pattern as the context fixture.
+  page: async ({ context, browser, playwright }, use) => {
+    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();
+      }
+    }
     // Clear any stale device emulation left by a prior test's page.setViewportSize() call.
     // setViewportSize sets Emulation.setDeviceMetricsOverride which persists across run-test
     // invocations on the same daemon. Without clearing, window.innerWidth returns the emulated
@@ -274,11 +383,22 @@ module.exports = { ...realPw, test, default: test };
 // Auto-generated by stably-browser run-test
 // Intercepts require('@playwright/test') to use CDP-aware fixtures.
 const Module = require('module');
+const path = require('path');
 const originalResolve = Module._resolveFilename;
 const fixturesPath = ${JSON.stringify(fixturesPath)};
 
+// Only redirect require('@playwright/test') from USER test files to our CDP fixtures.
+// Without this guard, Playwright's own internal require('@playwright/test') calls
+// would also get redirected, breaking Playwright's internals. We check parent.filename
+// to let anything inside node_modules/playwright/ or node_modules/@playwright/ resolve
+// to the real module.
+function isPlaywrightInternal(filename) {
+  const normalizedFilename = filename.split(path.sep).join('/');
+  return normalizedFilename.includes('/node_modules/playwright/') || normalizedFilename.includes('/node_modules/@playwright/');
+}
+
 Module._resolveFilename = function(request, parent, isMain, options) {
-  if (request === '@playwright/test') {
+  if (request === '@playwright/test' && (!parent || !parent.filename || !isPlaywrightInternal(parent.filename))) {
     return fixturesPath;
   }
   return originalResolve.call(this, request, parent, isMain, options);
@@ -398,4 +518,12 @@ if (cmdIndex !== -1 && argv[cmdIndex] === "open") {
     }
   }
 }
+if (!process.env.PLAYWRIGHT_MCP_CONFIG) {
+  const os = require("os");
+  const shmFixConfigPath = path.join(os.tmpdir(), "playwright-cli-shm-fix.json");
+  fs.writeFileSync(shmFixConfigPath, JSON.stringify({
+    browser: { launchOptions: { args: ["--disable-dev-shm-usage"] } }
+  }));
+  process.env.PLAYWRIGHT_MCP_CONFIG = shmFixConfigPath;
+}
 require("playwright/lib/cli/client/program");

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "stably",
-  "version": "4.12.11",
+  "version": "4.12.12",
   "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.21",
+    "@stablyai-internal/playwright-cli": "0.4.22",
     "@stablyai-internal/pwtrace": "0.3.1",
     "@stablyai/agent-hooks": "workspace:*",
     "@stablyai/agent-schemas": "workspace:*",

package/dist/node_modules/playwright/lib/mcp/browser/config.js.rej

@@ -1,11 +0,0 @@
---- lib/mcp/browser/config.js
-+++ lib/mcp/browser/config.js
-@@ -305,7 +305,7 @@
- function outputDir(config, clientInfo) {
-   if (config.outputDir)
-     return import_path.default.resolve(config.outputDir);
-   const rootPath = (0, import_server2.firstRootPath)(clientInfo);
-   if (rootPath)
--    return import_path.default.resolve(rootPath, config.skillMode ? ".playwright-cli" : ".playwright-mcp");
-+    return import_path.default.resolve(rootPath, config.skillMode ? ".stably-browser" : ".playwright-mcp");
-   const tmpDir = process.env.PW_TMPDIR_FOR_TEST ?? import_os.default.tmpdir();