playwright 1.58.0-alpha-2025-12-05 → 1.58.0-alpha-2025-12-06

package/lib/isomorphic/teleReceiver.js

@@ -70,10 +70,6 @@ class TeleReporterReceiver {
       this._onAttach(params.testId, params.resultId, params.attachments);
       return;
     }
-    if (method === "onTestError") {
-      this._onTestError(params.testId, params.resultId, params.error);
-      return;
-    }
     if (method === "onStepEnd") {
       this._onStepEnd(params.testId, params.resultId, params.step);
       return;
@@ -127,10 +123,8 @@ class TeleReporterReceiver {
     const result = test.results.find((r) => r._id === payload.id);
     result.duration = payload.duration;
     result.status = payload.status;
-    if (!!payload.errors) {
-      result.errors = payload.errors;
-      result.error = result.errors[0];
-    }
+    result.errors = payload.errors;
+    result.error = result.errors?.[0];
     if (!!payload.attachments)
       result.attachments = this._parseAttachments(payload.attachments);
     if (payload.annotations) {
@@ -173,12 +167,6 @@ class TeleReporterReceiver {
       body: a.base64 && globalThis.Buffer ? Buffer.from(a.base64, "base64") : void 0
     })));
   }
-  _onTestError(testId, resultId, error) {
-    const test = this._tests.get(testId);
-    const result = test.results.find((r) => r._id === resultId);
-    result.errors.push(error);
-    result.error = result.errors[0];
-  }
   _onError(error) {
     this._reporter.onError?.(error);
   }

package/lib/reporters/internalReporter.js

@@ -66,11 +66,8 @@ class InternalReporter {
   onStdErr(chunk, test, result) {
     this._reporter.onStdErr?.(chunk, test, result);
   }
-  onTestError(test, result, error) {
-    addLocationAndSnippetToError(this._config, error, test.location.file);
-    this._reporter.onTestError?.(test, result, error);
-  }
   onTestEnd(test, result) {
+    this._addSnippetToTestErrors(test, result);
     this._reporter.onTestEnd?.(test, result);
   }
   async onEnd(result) {
@@ -87,20 +84,27 @@ class InternalReporter {
     await this._reporter.onExit?.();
   }
   onError(error) {
-    addLocationAndSnippetToError(this._config, error, void 0);
+    addLocationAndSnippetToError(this._config, error);
     this._reporter.onError?.(error);
   }
   onStepBegin(test, result, step) {
     this._reporter.onStepBegin?.(test, result, step);
   }
   onStepEnd(test, result, step) {
-    if (step.error)
-      addLocationAndSnippetToError(this._config, step.error, test.location.file);
+    this._addSnippetToStepError(test, step);
     this._reporter.onStepEnd?.(test, result, step);
   }
   printsToStdio() {
     return this._reporter.printsToStdio ? this._reporter.printsToStdio() : true;
   }
+  _addSnippetToTestErrors(test, result) {
+    for (const error of result.errors)
+      addLocationAndSnippetToError(this._config, error, test.location.file);
+  }
+  _addSnippetToStepError(test, step) {
+    if (step.error)
+      addLocationAndSnippetToError(this._config, step.error, test.location.file);
+  }
 }
 function addLocationAndSnippetToError(config, error, file) {
   if (error.stack && !error.location)

package/lib/reporters/merge.js

@@ -342,7 +342,6 @@ class IdsPatcher {
       case "onProject":
         this._onProject(params.project);
         return;
-      case "onTestError":
       case "onAttach":
       case "onTestBegin":
       case "onStepBegin":
@@ -424,7 +423,7 @@ class PathSeparatorPatcher {
       test.annotations?.forEach((annotation) => this._updateAnnotationLocation(annotation));
       const testResult = jsonEvent.params.result;
       testResult.annotations?.forEach((annotation) => this._updateAnnotationLocation(annotation));
-      testResult.errors?.forEach((error) => this._updateErrorLocations(error));
+      testResult.errors.forEach((error) => this._updateErrorLocations(error));
       (testResult.attachments ?? []).forEach((attachment) => {
         if (attachment.path)
           attachment.path = this._updatePath(attachment.path);
@@ -442,10 +441,6 @@ class PathSeparatorPatcher {
       step.annotations?.forEach((annotation) => this._updateAnnotationLocation(annotation));
       return;
     }
-    if (jsonEvent.method === "onTestError") {
-      this._updateErrorLocations(jsonEvent.params.error);
-      return;
-    }
     if (jsonEvent.method === "onAttach") {
       const attach = jsonEvent.params;
       attach.attachments.forEach((attachment) => {

package/lib/reporters/multiplexer.js

@@ -48,10 +48,6 @@ class Multiplexer {
     for (const reporter of this._reporters)
       wrap(() => reporter.onStdErr?.(chunk, test, result));
   }
-  onTestError(test, result, error) {
-    for (const reporter of this._reporters)
-      wrap(() => reporter.onTestError?.(test, result, error));
-  }
   onTestEnd(test, result) {
     for (const reporter of this._reporters)
       wrap(() => reporter.onTestEnd?.(test, result));

package/lib/reporters/teleEmitter.js

@@ -66,16 +66,6 @@ class TeleReporterEmitter {
       }
     });
   }
-  onTestError(test, result, error) {
-    this._messageSink({
-      method: "onTestError",
-      params: {
-        testId: test.id,
-        resultId: result[this._idSymbol],
-        error
-      }
-    });
-  }
   onTestEnd(test, result) {
     const testEnd = {
       testId: test.id,
@@ -235,6 +225,7 @@ class TeleReporterEmitter {
       id: result[this._idSymbol],
       duration: result.duration,
       status: result.status,
+      errors: result.errors,
       annotations: result.annotations?.length ? this._relativeAnnotationLocations(result.annotations) : void 0
     };
   }

package/lib/runner/dispatcher.js

@@ -243,6 +243,7 @@ class JobDispatcher {
   _onTestEnd(params) {
     if (this._failureTracker.hasReachedMaxFailures()) {
       params.status = "interrupted";
+      params.errors = [];
     }
     const data = this._dataByTestId.get(params.testId);
     if (!data) {
@@ -252,6 +253,8 @@ class JobDispatcher {
     this._remainingByTestId.delete(params.testId);
     const { result, test } = data;
     result.duration = params.duration;
+    result.errors = params.errors;
+    result.error = result.errors[0];
     result.status = params.status;
     result.annotations = params.annotations;
     test.annotations = [...params.annotations];
@@ -338,20 +341,6 @@ class JobDispatcher {
         this._reporter.onStdErr?.("Internal error: step id not found: " + params.stepId);
     }
   }
-  _onTestErrors(params) {
-    if (this._failureTracker.hasReachedMaxFailures()) {
-      return;
-    }
-    const data = this._dataByTestId.get(params.testId);
-    if (!data)
-      return;
-    const { test, result } = data;
-    for (const error of params.errors) {
-      result.errors.push(error);
-      result.error = result.errors[0];
-      this._reporter.onTestError?.(test, result, error);
-    }
-  }
   _failTestWithErrors(test, errors) {
     const runData = this._dataByTestId.get(test.id);
     let result;
@@ -361,11 +350,8 @@ class JobDispatcher {
       result = test._appendTestResult();
       this._reporter.onTestBegin?.(test, result);
     }
-    for (const error of errors) {
-      result.errors.push(error);
-      result.error = result.errors[0];
-      this._reporter.onTestError?.(test, result, error);
-    }
+    result.errors = [...errors];
+    result.error = result.errors[0];
     result.status = errors.length ? "failed" : "skipped";
     this._reportTestEnd(test, result);
     this._failedTests.add(test);
@@ -466,32 +452,29 @@ class JobDispatcher {
       import_utils.eventsHelper.addEventListener(worker, "stepBegin", this._onStepBegin.bind(this)),
       import_utils.eventsHelper.addEventListener(worker, "stepEnd", this._onStepEnd.bind(this)),
       import_utils.eventsHelper.addEventListener(worker, "attach", this._onAttach.bind(this)),
-      import_utils.eventsHelper.addEventListener(worker, "testErrors", this._onTestErrors.bind(this)),
       import_utils.eventsHelper.addEventListener(worker, "testPaused", this._onTestPaused.bind(this, worker)),
       import_utils.eventsHelper.addEventListener(worker, "done", this._onDone.bind(this)),
       import_utils.eventsHelper.addEventListener(worker, "exit", this.onExit.bind(this))
     ];
   }
   _onTestPaused(worker, params) {
-    const data = this._dataByTestId.get(params.testId);
-    if (!data)
-      return;
-    const { test, result } = data;
     const sendMessage = async (message) => {
       try {
         if (this.jobResult.isDone())
           throw new Error("Test has already stopped");
         const response = await worker.sendCustomMessage({ testId: params.testId, request: message.request });
         if (response.error)
-          (0, import_internalReporter.addLocationAndSnippetToError)(this._config.config, response.error, test.location.file);
+          (0, import_internalReporter.addLocationAndSnippetToError)(this._config.config, response.error);
         return response;
       } catch (e) {
         const error = (0, import_util.serializeError)(e);
-        (0, import_internalReporter.addLocationAndSnippetToError)(this._config.config, error, test.location.file);
+        (0, import_internalReporter.addLocationAndSnippetToError)(this._config.config, error);
         return { response: void 0, error };
       }
     };
-    this._failureTracker.onTestPaused?.({ errors: result.errors, sendMessage });
+    for (const error of params.errors)
+      (0, import_internalReporter.addLocationAndSnippetToError)(this._config.config, error);
+    this._failureTracker.onTestPaused?.({ ...params, sendMessage });
   }
   skipWholeJob() {
     const allTestsSkipped = this.job.tests.every((test) => test.expectedStatus === "skipped");

package/lib/runner/testRunner.js

@@ -383,8 +383,7 @@ async function runAllTestsWithConfig(config) {
     (0, import_tasks.createLoadTask)("in-process", { filterOnly: true, failOnLoadErrors: true }),
     ...(0, import_tasks.createRunTestsTasks)(config)
   ];
-  const testRun = new import_tasks.TestRun(config, reporter, { pauseAtEnd: config.configCLIOverrides.debug, pauseOnError: config.configCLIOverrides.debug });
-  const status = await (0, import_tasks.runTasks)(testRun, tasks, config.config.globalTimeout);
+  const status = await (0, import_tasks.runTasks)(new import_tasks.TestRun(config, reporter), tasks, config.config.globalTimeout);
   await new Promise((resolve) => process.stdout.write("", () => resolve()));
   await new Promise((resolve) => process.stderr.write("", () => resolve()));
   return status;

package/lib/worker/testInfo.js

@@ -42,9 +42,8 @@ var import_util = require("../util");
 var import_testTracing = require("./testTracing");
 var import_util2 = require("./util");
 var import_transform = require("../transform/transform");
-var import_babelHighlightUtils = require("../transform/babelHighlightUtils");
 class TestInfoImpl {
-  constructor(configInternal, projectInternal, workerParams, test, retry, onStepBegin, onStepEnd, onAttach, onErrors, onTestPaused) {
+  constructor(configInternal, projectInternal, workerParams, test, retry, onStepBegin, onStepEnd, onAttach, onTestPaused) {
     this._snapshotNames = { lastAnonymousSnapshotIndex: 0, lastNamedSnapshotIndex: {} };
     this._ariaSnapshotNames = { lastAnonymousSnapshotIndex: 0, lastNamedSnapshotIndex: {} };
     this._interruptedPromise = new import_utils.ManualPromise();
@@ -60,12 +59,10 @@ class TestInfoImpl {
     this.status = "passed";
     this.snapshotSuffix = "";
     this.errors = [];
-    this._reportedErrorCount = 0;
     this.testId = test?.id ?? "";
     this._onStepBegin = onStepBegin;
     this._onStepEnd = onStepEnd;
     this._onAttach = onAttach;
-    this._onErrors = onErrors;
     this._onTestPaused = onTestPaused;
     this._startTime = (0, import_utils.monotonicTime)();
     this._startWallTime = Date.now();
@@ -341,32 +338,11 @@ ${(0, import_utils.stringifyStackFrames)(step.boxedStack).join("\n")}`;
   async _didFinishTestFunction() {
     const shouldPause = this._workerParams.pauseAtEnd && !this._isFailure() || this._workerParams.pauseOnError && this._isFailure();
     if (shouldPause) {
-      const location = (this._isFailure() ? this._errorLocation() : await this._testEndLocation()) ?? { file: this.file, line: this.line, column: this.column };
-      this._emitErrors();
-      this._onTestPaused({ testId: this.testId });
-      await this._runAsStep({ title: this._isFailure() ? "Paused on Error" : "Paused at End", category: "test.step", location }, async () => {
-        await this._interruptedPromise;
-      });
+      this._onTestPaused({ testId: this.testId, errors: this._isFailure() ? this.errors : [] });
+      await this._interruptedPromise;
     }
     await this._onDidFinishTestFunctionCallback?.();
   }
-  _emitErrors() {
-    const errors = this.errors.slice(this._reportedErrorCount);
-    this._reportedErrorCount = Math.max(this._reportedErrorCount, this.errors.length);
-    if (errors.length)
-      this._onErrors({ testId: this.testId, errors });
-  }
-  _errorLocation() {
-    if (this.error?.stack)
-      return (0, import_util.filteredStackTrace)(this.error.stack.split("\n"))[0];
-  }
-  async _testEndLocation() {
-    try {
-      const source = await import_fs.default.promises.readFile(this.file, "utf-8");
-      return (0, import_babelHighlightUtils.findTestEndLocation)(source, { file: this.file, line: this.line, column: this.column });
-    } catch {
-    }
-  }
   // ------------ TestInfo methods ------------
   async attach(name, options = {}) {
     const step = this._addStep({

package/lib/worker/workerMain.js

@@ -99,7 +99,6 @@ class WorkerMain extends import_process.ProcessRunner {
       }, () => {
       }, () => {
       }, () => {
-      }, () => {
       });
       const runnable = { type: "teardown" };
       await fakeTestInfo._runWithTimeout(runnable, () => this._loadIfNeeded()).catch(() => {
@@ -244,7 +243,6 @@ class WorkerMain extends import_process.ProcessRunner {
       (stepBeginPayload) => this.dispatchEvent("stepBegin", stepBeginPayload),
       (stepEndPayload) => this.dispatchEvent("stepEnd", stepEndPayload),
       (attachment) => this.dispatchEvent("attach", attachment),
-      (errors) => this.dispatchEvent("testErrors", errors),
       (testPausedPayload) => this.dispatchEvent("testPaused", testPausedPayload)
     );
     const processAnnotation = (annotation) => {
@@ -285,7 +283,6 @@ class WorkerMain extends import_process.ProcessRunner {
     });
     if (isSkipped && nextTest && !hasAfterAllToRunBeforeNextTest) {
       testInfo.status = "skipped";
-      testInfo._emitErrors();
       this.dispatchEvent("testEnd", buildTestEndPayload(testInfo));
       return;
     }
@@ -399,7 +396,6 @@ class WorkerMain extends import_process.ProcessRunner {
     testInfo.duration = testInfo._timeoutManager.defaultSlot().elapsed + afterHooksSlot.elapsed | 0;
     this._currentTest = null;
     (0, import_globals.setCurrentTestInfo)(null);
-    testInfo._emitErrors();
     this.dispatchEvent("testEnd", buildTestEndPayload(testInfo));
     const preserveOutput = this._config.config.preserveOutput === "always" || this._config.config.preserveOutput === "failures-only" && testInfo._isFailure();
     if (!preserveOutput)
@@ -502,6 +498,7 @@ function buildTestEndPayload(testInfo) {
     testId: testInfo.testId,
     duration: testInfo.duration,
     status: testInfo.status,
+    errors: testInfo.errors,
     hasNonRetriableError: testInfo._hasNonRetriableError,
     expectedStatus: testInfo.expectedStatus,
     annotations: testInfo.annotations,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "playwright",
-  "version": "1.58.0-alpha-2025-12-05",
+  "version": "1.58.0-alpha-2025-12-06",
   "description": "A high-level API to automate web browsers",
   "repository": {
     "type": "git",
@@ -64,7 +64,7 @@
   },
   "license": "Apache-2.0",
   "dependencies": {
-    "playwright-core": "1.58.0-alpha-2025-12-05"
+    "playwright-core": "1.58.0-alpha-2025-12-06"
   },
   "optionalDependencies": {
     "fsevents": "2.3.2"

package/lib/transform/babelHighlightUtils.js

@@ -1,63 +0,0 @@
-"use strict";
-var __create = Object.create;
-var __defProp = Object.defineProperty;
-var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __getProtoOf = Object.getPrototypeOf;
-var __hasOwnProp = Object.prototype.hasOwnProperty;
-var __export = (target, all) => {
-  for (var name in all)
-    __defProp(target, name, { get: all[name], enumerable: true });
-};
-var __copyProps = (to, from, except, desc) => {
-  if (from && typeof from === "object" || typeof from === "function") {
-    for (let key of __getOwnPropNames(from))
-      if (!__hasOwnProp.call(to, key) && key !== except)
-        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
-  }
-  return to;
-};
-var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
-  // If the importer is in node compatibility mode or this is not an ESM
-  // file that has been converted to a CommonJS file using a Babel-
-  // compatible transform (i.e. "__esModule" has not been set), then set
-  // "default" to the CommonJS "module.exports" for node compatibility.
-  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
-  mod
-));
-var __toCommonJS = (mod) => __copyProps(__defProp({}, "__esModule", { value: true }), mod);
-var babelHighlightUtils_exports = {};
-__export(babelHighlightUtils_exports, {
-  findTestEndLocation: () => findTestEndLocation
-});
-module.exports = __toCommonJS(babelHighlightUtils_exports);
-var import_path = __toESM(require("path"));
-var import_babelBundle = require("./babelBundle");
-function containsLocation(range, location) {
-  if (location.line < range.start.line || location.line > range.end.line)
-    return false;
-  if (location.line === range.start.line && location.column < range.start.column)
-    return false;
-  if (location.line === range.end.line && location.column > range.end.column)
-    return false;
-  return true;
-}
-function findTestEndLocation(text, testStartLocation) {
-  const ast = (0, import_babelBundle.babelParse)(text, import_path.default.basename(testStartLocation.file), false);
-  let result;
-  (0, import_babelBundle.traverse)(ast, {
-    enter(path2) {
-      if (import_babelBundle.types.isCallExpression(path2.node) && path2.node.loc && containsLocation(path2.node.loc, testStartLocation)) {
-        const callNode = path2.node;
-        const funcNode = callNode.arguments[callNode.arguments.length - 1];
-        if (callNode.arguments.length >= 2 && import_babelBundle.types.isFunction(funcNode) && funcNode.body.loc)
-          result = { file: testStartLocation.file, ...funcNode.body.loc.end };
-      }
-    }
-  });
-  return result;
-}
-// Annotate the CommonJS export names for ESM import in node:
-0 && (module.exports = {
-  findTestEndLocation
-});