testdriverai 7.3.35 → 7.3.36

package/CHANGELOG.md

@@ -1,3 +1,12 @@
+## [7.3.36](https://github.com/testdriverai/testdriverai/compare/v7.3.35...v7.3.36) (2026-02-24)
+
+
+### Reverts
+
+* Revert "Fix hanging node processes on Ctrl+C (#654)" (#683) ([5e68748](https://github.com/testdriverai/testdriverai/commit/5e6874825c6718e006bbf84e2ba5edae57d173ac)), closes [#654](https://github.com/testdriverai/testdriverai/issues/654) [#683](https://github.com/testdriverai/testdriverai/issues/683)
+
+
+
 ## [7.3.35](https://github.com/testdriverai/testdriverai/compare/v7.3.34...v7.3.35) (2026-02-24)
 
 

package/agent/lib/sandbox.js

@@ -128,11 +128,6 @@ const createSandbox = (emitter, analytics, sessionInstance) => {
             );
           }
         }, timeout);
-        // Don't let pending message timeouts prevent Node process from exiting
-        // (unref is not available in browser/non-Node environments)
-        if (timeoutId.unref) {
-          timeoutId.unref();
-        }
 
         // Track timeout so close() can clear it
         this.pendingTimeouts.set(requestId, timeoutId);
@@ -365,11 +360,6 @@ const createSandbox = (emitter, analytics, sessionInstance) => {
           this.reconnecting = false;
         }
       }, delay);
-      // Don't let the reconnect timer prevent Node process from exiting
-      // (unref is not available in browser/non-Node environments)
-      if (this.reconnectTimer.unref) {
-        this.reconnectTimer.unref();
-      }
     }
 
     /**

package/lib/vitest/hooks.mjs

@@ -214,33 +214,6 @@ function cleanupConsoleSpy(client) {
 const testDriverInstances = new WeakMap();
 const lifecycleHandlers = new WeakMap();
 
-// Set to track all active TestDriver instances for signal-based cleanup
-const activeInstances = new Set();
-
-// Register signal handlers once to clean up all active instances on forced exit
-let signalHandlersRegistered = false;
-function registerSignalHandlers() {
-  if (signalHandlersRegistered) return;
-  signalHandlersRegistered = true;
-
-  const cleanup = async () => {
-    const instances = Array.from(activeInstances);
-    activeInstances.clear();
-    await Promise.race([
-      Promise.all(instances.map((inst) => inst.disconnect().catch(() => {}))),
-      new Promise((resolve) => setTimeout(resolve, 5000)), // 5s max for cleanup
-    ]);
-  };
-
-  process.on("SIGINT", () => {
-    cleanup().finally(() => process.exit(130));
-  });
-
-  process.on("SIGTERM", () => {
-    cleanup().finally(() => process.exit(143));
-  });
-}
-
 /**
  * Create a TestDriver client in a Vitest test with automatic lifecycle management
  *
@@ -318,8 +291,6 @@ export function TestDriver(context, options = {}) {
   testdriver.__vitestContext = context.task;
   testdriver._debugOnFailure = mergedOptions.debugOnFailure || false;
   testDriverInstances.set(context.task, testdriver);
-  activeInstances.add(testdriver);
-  registerSignalHandlers();
 
   // Set platform metadata early so the reporter can show the correct OS from the start
   if (!context.task.meta) {
@@ -455,9 +426,6 @@ export function TestDriver(context, options = {}) {
       // Clean up console spies
       cleanupConsoleSpy(currentInstance);
 
-      // Remove from active instances tracking (even in debug mode we clean up tracking)
-      activeInstances.delete(currentInstance);
-
       // DO NOT disconnect or terminate - keep sandbox alive for debugging
       return;
     }
@@ -584,8 +552,6 @@ export function TestDriver(context, options = {}) {
     } catch (error) {
       console.error("Error disconnecting client:", error);
     } finally {
-      // Remove from active instances tracking
-      activeInstances.delete(currentInstance);
       // Terminate AWS instance if one was spawned for this test
       // This must happen AFTER dashcam.stop() to ensure recording is saved
       // AND it must happen even if disconnect() fails

package/lib/vitest/setup-aws.mjs

@@ -115,16 +115,16 @@ function cleanupAllInstances() {
 process.on("exit", cleanupAllInstances);
 process.on("SIGINT", () => {
   cleanupAllInstances();
-  process.exit(130); // Restore default SIGINT exit behavior (128 + signal 2)
+  // Don't call process.exit here - let the signal handler do its job
 });
 process.on("SIGTERM", () => {
   cleanupAllInstances();
-  process.exit(143); // Restore default SIGTERM exit behavior (128 + signal 15)
+  // Don't call process.exit here - let the signal handler do its job
 });
 process.on("uncaughtException", (error) => {
   console.error("[TestDriver] Uncaught exception:", error);
   cleanupAllInstances();
-  process.exit(1); // Exit after uncaught exception cleanup
+  // Don't call process.exit here - let Node.js handle the exception
 });
 
 beforeEach(async (context) => {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "testdriverai",
-  "version": "7.3.35",
+  "version": "7.3.36",
   "description": "Next generation autonomous AI agent for end-to-end testing of web & desktop",
   "main": "sdk.js",
   "types": "sdk.d.ts",