testdriverai 7.2.85 → 7.2.87

package/agent/lib/redraw.js

@@ -21,7 +21,8 @@ const createRedraw = (
   // Merge default options with provided defaults
   const baseOptions = { ...DEFAULT_REDRAW_OPTIONS, ...defaultOptions };
   
-  const networkUpdateInterval = 15000;
+  // Network check interval (ms) - used for speed calculation display
+  const networkCheckInterval = 250;
 
   let lastTxBytes = null;
   let lastRxBytes = null;
@@ -125,15 +126,32 @@ const createRedraw = (
     }
   };
 
+  // Track if a network request is in flight to prevent overlapping requests
+  let networkRequestInFlight = false;
+
   async function updateNetwork() {
+    // Prevent overlapping requests - if one is already in flight, skip this cycle
+    if (networkRequestInFlight) {
+      emitter.emit(events.log.debug, '[redraw] updateNetwork() - skipping, request already in flight');
+      return;
+    }
+
     if (sandbox && sandbox.instanceSocketConnected) {
-      let network = await sandbox.send({
-        type: "system.network",
-      });
-      parseNetworkStats(
-        network.out.totalBytesReceived,
-        network.out.totalBytesSent,
-      );
+      networkRequestInFlight = true;
+      try {
+        let network = await sandbox.send({
+          type: "system.network",
+        }, 10000); // Use a shorter 10 second timeout for network stats
+        parseNetworkStats(
+          network.out.totalBytesReceived,
+          network.out.totalBytesSent,
+        );
+      } catch (error) {
+        // Log the error but don't throw - network monitoring is non-critical
+        emitter.emit(events.log.debug, `[redraw] updateNetwork() failed: ${error.message}`);
+      } finally {
+        networkRequestInFlight = false;
+      }
     }
   }
 
@@ -181,14 +199,7 @@ const createRedraw = (
     }
   }
 
-  // Start network monitoring only when needed
-  function startNetworkMonitoring() {
-    if (!networkInterval) {
-      networkInterval = setInterval(updateNetwork, networkUpdateInterval);
-    }
-  }
-
-  // Stop network monitoring
+  // Stop network monitoring (cleanup any residual interval)
   function stopNetworkMonitoring() {
     if (networkInterval) {
       clearInterval(networkInterval);
@@ -220,11 +231,6 @@ const createRedraw = (
     
     resetState();
     
-    // Only start network monitoring if enabled
-    if (currentOptions.networkMonitor) {
-      startNetworkMonitoring();
-    }
-    
     // Capture initial image for screen stability monitoring
     if (currentOptions.screenRedraw) {
       initialScreenImage = await system.captureScreenPNG(0.25, true);
@@ -244,6 +250,11 @@ const createRedraw = (
       return;
     }
     
+    // Update network stats on each check (with guard against overlapping requests)
+    if (networkMonitor) {
+      await updateNetwork();
+    }
+    
     let nowImage = screenRedraw ? await system.captureScreenPNG(0.25, true) : null;
     let timeElapsed = Date.now() - startTime;
     let diffFromInitial = 0;
@@ -305,7 +316,7 @@ const createRedraw = (
       : effectiveNetworkSettled
         ? theme.green(`y`)
         : theme.dim(
-            `${Math.trunc((diffRxBytes + diffTxBytes) / networkUpdateInterval)}b/s`,
+            `${Math.trunc((diffRxBytes + diffTxBytes) / (networkCheckInterval / 1000))}b/s`,
           );
     let timeoutText = isTimeout
       ? theme.green(`y`)
@@ -370,10 +381,6 @@ const createRedraw = (
     
     return new Promise((resolve) => {
       const startTime = Date.now();
-      // Start network monitoring if not already started and enabled
-      if (waitOptions.networkMonitor) {
-        startNetworkMonitoring();
-      }
       checkCondition(resolve, startTime, timeoutMs, waitOptions);
     });
   }

package/package.json

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