@checkly/playwright-core 1.41.26 → 1.42.11

package/lib/utils/processLauncher.js

@@ -95,6 +95,11 @@ function addProcessHandlerIfNeeded(name) {
     process.on(name, processHandlers[name]);
   }
 }
+function removeProcessHandlersIfNeeded() {
+  if (killSet.size) return;
+  for (const handler of installedHandlers) process.off(handler, processHandlers[handler]);
+  installedHandlers.clear();
+}
 async function launchProcess(options) {
   const stdio = options.stdio === 'pipe' ? ['ignore', 'pipe', 'pipe', 'pipe', 'pipe'] : ['pipe', 'pipe', 'pipe'];
   options.log(`<launching> ${options.command} ${options.args ? options.args.join(' ') : ''}`);
@@ -149,6 +154,7 @@ async function launchProcess(options) {
     processClosed = true;
     gracefullyCloseSet.delete(gracefullyClose);
     killSet.delete(killProcessAndCleanup);
+    removeProcessHandlersIfNeeded();
     options.onExit(exitCode, signal);
     // Cleanup as process exits.
     cleanup().then(fulfillCleanup);
@@ -182,6 +188,7 @@ async function launchProcess(options) {
   function killProcess() {
     gracefullyCloseSet.delete(gracefullyClose);
     killSet.delete(killProcessAndCleanup);
+    removeProcessHandlersIfNeeded();
     options.log(`[pid=${spawnedProcess.pid}] <kill>`);
     if (spawnedProcess.pid && !spawnedProcess.killed && !processClosed) {
       options.log(`[pid=${spawnedProcess.pid}] <will force kill>`);

package/lib/utils/rtti.js

@@ -7,7 +7,13 @@ exports.isError = isError;
 exports.isLikelyNpxGlobal = void 0;
 exports.isObject = isObject;
 exports.isRegExp = isRegExp;
-exports.isString = isString;
+Object.defineProperty(exports, "isString", {
+  enumerable: true,
+  get: function () {
+    return _stringUtils.isString;
+  }
+});
+var _stringUtils = require("./isomorphic/stringUtils");
 /**
  * Copyright (c) Microsoft Corporation.
  *
@@ -24,9 +30,6 @@ exports.isString = isString;
  * limitations under the License.
  */
 
-function isString(obj) {
-  return typeof obj === 'string' || obj instanceof String;
-}
 function isRegExp(obj) {
   return obj instanceof RegExp || Object.prototype.toString.call(obj) === '[object RegExp]';
 }

package/lib/utils/wsServer.js

@@ -71,16 +71,16 @@ class WSServer {
       noServer: true,
       perMessageDeflate
     });
-    this._wsServer.on('headers', headers => this._delegate.onHeaders(headers));
+    if (this._delegate.onHeaders) this._wsServer.on('headers', headers => this._delegate.onHeaders(headers));
     server.on('upgrade', (request, socket, head) => {
-      var _this$_wsServer;
+      var _this$_delegate$onUpg, _this$_delegate, _this$_wsServer;
       const pathname = new URL('http://localhost' + request.url).pathname;
       if (pathname !== path) {
         socket.write(`HTTP/${request.httpVersion} 400 Bad Request\r\n\r\n`);
         socket.destroy();
         return;
       }
-      const upgradeResult = this._delegate.onUpgrade(request, socket);
+      const upgradeResult = (_this$_delegate$onUpg = (_this$_delegate = this._delegate).onUpgrade) === null || _this$_delegate$onUpg === void 0 ? void 0 : _this$_delegate$onUpg.call(_this$_delegate, request, socket);
       if (upgradeResult) {
         socket.write(upgradeResult.error);
         socket.destroy();
@@ -102,6 +102,7 @@ class WSServer {
     return wsEndpoint;
   }
   async close() {
+    var _this$_delegate$onClo, _this$_delegate2;
     const server = this._wsServer;
     if (!server) return;
     _debugLogger.debugLogger.log('server', 'closing websocket server');
@@ -120,6 +121,7 @@ class WSServer {
     this._wsServer = undefined;
     this.server = undefined;
     _debugLogger.debugLogger.log('server', 'closed server');
+    await ((_this$_delegate$onClo = (_this$_delegate2 = this._delegate).onClose) === null || _this$_delegate$onClo === void 0 ? void 0 : _this$_delegate$onClo.call(_this$_delegate2));
   }
 }
 exports.WSServer = WSServer;