@rstest/browser 0.10.6 → 0.11.1

package/dist/protocol.d.ts

@@ -65,9 +65,15 @@ export type BrowserHostConfig = {
    * Used by browser RPC calls to prevent stale requests from previous reruns.
    */ runId?: string;
     /**
-   * Base URL for runner (iframe) pages.
+   * Base URL for runner (iframe) pages. Container origin; used as a fallback
+   * when a project has no entry in `projectRunnerUrls`.
    */ runnerUrl?: string;
     /**
+   * Per-project runner origin base URLs, keyed by project name. Each browser
+   * project runs on its own dev server, so the container must load each test
+   * file's iframe from that project's own origin.
+   */ projectRunnerUrls?: Record<string, string>;
+    /**
    * WebSocket port for container RPC.
    */ wsPort?: number;
     /**

package/dist/providers/index.d.ts

@@ -41,6 +41,9 @@ import type { BrowserRpcRequest } from '../rpcProtocol.js';
     on: {
         (event: 'popup', listener: (page: BrowserProviderPage) => void) : void;
         (event: 'console', listener: (message: BrowserConsoleMessage) => void) : void;
+        // Page died: renderer crash or unexpected close. Headless scheduling wires
+        // these to fail the file immediately. Listeners take no payload.
+        (event: 'crash' | 'close', listener: () => void) : void;
     };
     close: () => Promise<void>;
     [Symbol.asyncDispose]: () => Promise<void>;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@rstest/browser",
-  "version": "0.10.6",
+  "version": "0.11.1",
   "description": "Browser mode support for Rstest testing framework.",
   "keywords": [
     "rstest",
@@ -48,21 +48,21 @@
     "ws": "^8.21.0"
   },
   "devDependencies": {
-    "@rslib/core": "0.23.0",
+    "@rslib/core": "0.23.2",
     "@types/convert-source-map": "^2.0.3",
     "@types/picomatch": "^4.0.3",
     "@types/ws": "^8.18.1",
     "@vitest/snapshot": "^3.2.6",
     "birpc": "^4.0.0",
-    "picomatch": "^4.0.4",
-    "playwright": "^1.61.0",
-    "@rstest/browser-ui": "0.0.0",
+    "picomatch": "^4.0.5",
+    "playwright": "^1.61.1",
+    "@rstest/core": "0.11.1",
     "@rstest/tsconfig": "0.0.1",
-    "@rstest/core": "0.10.6"
+    "@rstest/browser-ui": "0.0.0"
   },
   "peerDependencies": {
     "playwright": "^1.49.1",
-    "@rstest/core": "^0.10.6"
+    "@rstest/core": "^0.11.1"
   },
   "peerDependenciesMeta": {
     "playwright": {

package/src/client/entry.ts

@@ -32,6 +32,7 @@ import {
   createRunnerLifecycleRequest,
   sendRunnerLifecycle,
 } from './dispatchTransport';
+import { formatConsoleArgs } from './formatConsole';
 import { BrowserSnapshotEnvironment } from './snapshot';
 import {
   findNewScriptUrl,
@@ -109,24 +110,6 @@ const ensureRuntimeEnv = (env: RuntimeConfig['env'] | undefined): void => {
   }
 };
 
-/**
- * Format an argument for console output.
- */
-const formatArg = (arg: unknown): string => {
-  if (arg === null) return 'null';
-  if (arg === undefined) return 'undefined';
-  if (typeof arg === 'string') return arg;
-  if (typeof arg === 'number' || typeof arg === 'boolean') return String(arg);
-  if (arg instanceof Error) {
-    return arg.stack || `${arg.name}: ${arg.message}`;
-  }
-  try {
-    return JSON.stringify(arg, null, 2);
-  } catch {
-    return String(arg);
-  }
-};
-
 const getFileTaskId = (testPath: string): string => {
   return `file:${testPath}`;
 };
@@ -161,8 +144,7 @@ const interceptConsole = (
       // Call original for browser DevTools
       originalConsole[level](...args);
 
-      // Format message
-      const content = args.map(formatArg).join(' ');
+      const content = formatConsoleArgs(args);
       const currentTask = getCurrentTask();
 
       // Send to host
@@ -284,10 +266,19 @@ const toContextKey = (absolutePath: string, projectRoot: string): string => {
   const normalizedAbsolute = normalize(absolutePath);
   const normalizedRoot = normalize(projectRoot);
 
-  let relative = normalizedAbsolute;
-  if (normalizedAbsolute.startsWith(normalizedRoot)) {
-    relative = normalizedAbsolute.slice(normalizedRoot.length);
+  // Only strip the root at a path boundary: a bare `startsWith` would mangle a
+  // sibling like `/repo/pkg-extra/a.test.ts` under root `/repo/pkg`. Must stay
+  // in sync with the host `toContextKey` (hostController.ts) so the non-watch
+  // import-map keys resolve.
+  const withinRoot =
+    normalizedAbsolute === normalizedRoot ||
+    normalizedAbsolute.startsWith(`${normalizedRoot}/`);
+  if (!withinRoot) {
+    // Test file outside the project root: keep the absolute path as the key so
+    // `toAbsolutePath` round-trips it instead of re-rooting under projectRoot.
+    return normalizedAbsolute;
   }
+  const relative = normalizedAbsolute.slice(normalizedRoot.length);
   return relative.startsWith('/') ? `.${relative}` : `./${relative}`;
 };
 
@@ -296,6 +287,11 @@ const toContextKey = (absolutePath: string, projectRoot: string): string => {
  * e.g., './src/foo.test.ts' -> '/project/src/foo.test.ts'
  */
 const toAbsolutePath = (key: string, projectRoot: string): string => {
+  // An absolute key (test file outside the project root, see `toContextKey`)
+  // round-trips as-is; only `./`-prefixed relative keys are re-rooted.
+  if (!key.startsWith('.')) {
+    return key;
+  }
   // key format: ./src/foo.test.ts
   // Ensure no double slashes by removing trailing slash from projectRoot
   const normalizedRoot = normalize(projectRoot).replace(/\/$/, '');
@@ -482,6 +478,10 @@ const run = async () => {
         rootPath: options.rootPath,
         runtimeConfig,
         taskId: 0,
+        // The kept-module-cache flush keyed on `buildId` is node worker-pool
+        // only (#1373); browser runners never reuse a node worker, so a constant
+        // inert id is correct here.
+        buildId: 0,
         outputModule: false,
         environment: 'browser',
         testPath,
@@ -578,6 +578,8 @@ const run = async () => {
       rootPath: options.rootPath,
       runtimeConfig,
       taskId: 0,
+      // See the `buildId` note above: inert in browser mode.
+      buildId: 0,
       outputModule: false,
       environment: 'browser',
       currentTask: taskStack[0],

package/src/client/formatConsole.ts

@@ -0,0 +1,77 @@
+/** Stringify a single console argument for terminal forwarding. */
+export const formatArg = (arg: unknown): string => {
+  if (arg === null) return 'null';
+  if (arg === undefined) return 'undefined';
+  if (typeof arg === 'string') return arg;
+  if (typeof arg === 'number' || typeof arg === 'boolean') return String(arg);
+  // `JSON.stringify(symbol)` is `undefined`, so a symbol must be handled before
+  // the JSON path or it would forward as an empty/`undefined` log.
+  if (typeof arg === 'symbol') return arg.toString();
+  if (arg instanceof Error) {
+    return arg.stack || `${arg.name}: ${arg.message}`;
+  }
+  try {
+    return JSON.stringify(arg, null, 2);
+  } catch {
+    return String(arg);
+  }
+};
+
+// Format specifiers understood by the browser console. `%c` applies CSS in a
+// real console; forwarded to a terminal there is no styling, so it is consumed
+// and dropped (matching how Node's `util.format` handles it). `%%` is a literal
+// percent sign. Kept aligned with the browser console spec (no Node-only `%j`,
+// since these logs originate in the browser).
+// cspell:ignore sdifo WHATWG
+const HAS_SPECIFIER = /%[sdifoOc%]/;
+const SPECIFIER = /%[sdifoOc%]/g;
+
+/**
+ * Join console arguments the way a console does. When the first argument is a
+ * string carrying `printf`-style specifiers, substitute the following arguments
+ * into it (consuming `%c` styles without emitting them), then append any
+ * leftover arguments. Otherwise fall back to a plain space-join.
+ *
+ * Without this, `console.info('%cText', 'font-weight:bold')` (e.g. React's
+ * DevTools notice) leaks the raw `%c` directive and its CSS argument into the
+ * forwarded terminal output.
+ */
+export const formatConsoleArgs = (args: unknown[]): string => {
+  const first = args[0];
+  if (typeof first !== 'string' || !HAS_SPECIFIER.test(first)) {
+    return args.map(formatArg).join(' ');
+  }
+
+  let next = 1;
+  const substituted = first.replace(SPECIFIER, (spec) => {
+    if (spec === '%%') return '%';
+    if (next >= args.length) return spec;
+    const arg = args[next++];
+    switch (spec) {
+      case '%c':
+        return ''; // CSS directive: swallow the style argument
+      case '%s':
+        return formatArg(arg);
+      case '%d':
+      case '%i': {
+        // Match the browser console formatter (WHATWG): parseInt on the value,
+        // so unit-suffixed strings like '42px' format as 42 (not NaN).
+        // `String()` first keeps Symbol safe — a bare `parseInt(symbol)` throws
+        // via ToString, whereas the spec maps a Symbol to NaN.
+        const num = Number.parseInt(String(arg), 10);
+        return Number.isNaN(num) ? 'NaN' : String(num);
+      }
+      case '%f': {
+        const num = Number.parseFloat(String(arg));
+        return Number.isNaN(num) ? 'NaN' : String(num);
+      }
+      default: // %o, %O
+        return formatArg(arg);
+    }
+  });
+
+  if (next >= args.length) {
+    return substituted;
+  }
+  return [substituted, ...args.slice(next).map(formatArg)].join(' ');
+};

package/src/client/locator.ts

@@ -29,11 +29,7 @@ export type LocatorTextOptions = {
 };
 
 export type LocatorKeyboardModifier =
-  | 'Alt'
-  | 'Control'
-  | 'ControlOrMeta'
-  | 'Meta'
-  | 'Shift';
+  'Alt' | 'Control' | 'ControlOrMeta' | 'Meta' | 'Shift';
 
 export type LocatorMouseButton = 'left' | 'right' | 'middle';