@testdriverai/runner 7.8.0-test.47 → 7.8.0-test.48

package/lib/ably-service.js

@@ -275,6 +275,9 @@ class AblyService extends EventEmitter {
 
       this.emit('log', `Command received: ${type} (requestId=${requestId})`);
 
+      // Stop re-publishing runner.ready once we get the first command
+      this._stopReadySignal();
+
       // Per-command timeout: use message.timeout if provided, else default 120s
       // Prevents hanging forever if screenshot capture or S3 upload stalls
       const commandTimeout = (message.timeout && message.timeout > 0)
@@ -415,8 +418,8 @@ class AblyService extends EventEmitter {
 
         // Detect discontinuity: channel re-attached but message continuity was lost.
         // Use historyBeforeSubscribe() on each subscription to recover missed messages.
-        if (current === 'attached' && stateChange.resumed === false && previous) {
-          this.emit('log', `Ably channel [session]: DISCONTINUITY (resumed=false)${reasonMsg ? ' — ' + reasonMsg : ''}`);
+        if (current === 'attached' && stateChange.resumed === false && previous === 'attached') {
+          this.emit('log', `Ably channel [session]: DISCONTINUITY (resumed=false)${reasonMsg ? ' — ' + reasonMsg : ''}`);    
 
           Sentry.withScope((scope) => {
             scope.setTag('ably.client', 'runner');
@@ -473,6 +476,39 @@ class AblyService extends EventEmitter {
     }
     await this._sessionChannel.publish('control', readyPayload);
     this.emit('log', 'Published runner.ready signal');
+
+    // Re-publish runner.ready every 3s for up to 60s.
+    // The SDK may connect after the first publish (race condition),
+    // and Ably channel history may not be enabled. Repeating ensures
+    // the SDK catches at least one live runner.ready message.
+    this._readyInterval = setInterval(async () => {
+      try {
+        readyPayload.timestamp = Date.now();
+        await this._sessionChannel.publish('control', readyPayload);
+        this.emit('log', 'Re-published runner.ready signal');
+      } catch (err) {
+        this.emit('log', `Failed to re-publish runner.ready: ${err.message}`);
+      }
+    }, 3000);
+
+    // Stop after 60s regardless
+    this._readyTimeout = setTimeout(() => {
+      this._stopReadySignal();
+    }, 60000);
+  }
+
+  /**
+   * Stop the repeated runner.ready signal (called on first command or after timeout).
+   */
+  _stopReadySignal() {
+    if (this._readyInterval) {
+      clearInterval(this._readyInterval);
+      this._readyInterval = null;
+    }
+    if (this._readyTimeout) {
+      clearTimeout(this._readyTimeout);
+      this._readyTimeout = null;
+    }
   }
 
   /**
@@ -617,6 +653,8 @@ class AblyService extends EventEmitter {
   async close() {
     this.emit('log', 'Closing Ably service...');
 
+    this._stopReadySignal();
+
     if (this._statsInterval) {
       clearInterval(this._statsInterval);
       this._statsInterval = null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@testdriverai/runner",
-  "version": "7.8.0-test.47",
+  "version": "7.8.0-test.48",
   "description": "TestDriver Runner - Ably-based remote automation agent with Node.js automation",
   "main": "index.js",
   "bin": {