@dev-blinq/cucumber_client 1.0.1626-dev → 1.0.1628-dev

package/bin/client/recorderv3/services.js

@@ -244,6 +244,20 @@ export class RemoteBrowserService extends EventEmitter {
         const timestamp = new Date().toISOString();
         console.log(`[${timestamp}] [RemoteBrowserService] ${message}`, data ? JSON.stringify(data, null, 2) : "");
     }
+    isTransientPageDataError(error) {
+        const message = error instanceof Error ? error.message : typeof error === "string" ? error : undefined;
+        if (!message) {
+            return false;
+        }
+        const transientIndicators = [
+            "Execution context was destroyed",
+            "Execution context is not available",
+            "Cannot find context with specified id",
+            "Target closed",
+            "Navigation interrupted",
+        ];
+        return transientIndicators.some((indicator) => message.includes(indicator));
+    }
     /**
      * Gets the CDP Target ID for a page *directly* from the page.
      * This is the only reliable method during restarts.
@@ -277,7 +291,13 @@ export class RemoteBrowserService extends EventEmitter {
         for (const page of existingPages) {
             const stableTabId = uuidv4();
             const cdpTargetId = await this.getCdpTargetId(page);
-            this.pages.set(stableTabId, { page, cdpTargetId });
+            this.pages.set(stableTabId, {
+                page,
+                cdpTargetId,
+                lastKnownTitle: undefined,
+                lastKnownUrl: page.url(),
+                lastKnownWsDebuggerUrl: cdpTargetId ? `${this.wsUrlBase}${cdpTargetId}` : undefined,
+            });
             this.log("✅ Existing page added to map", { stableTabId, cdpTargetId, url: page.url() });
             this.attachPageLifecycleListeners(page, stableTabId);
             this.log("🔍 Attached lifecycle listeners to existing page", { stableTabId });
@@ -291,7 +311,13 @@ export class RemoteBrowserService extends EventEmitter {
             this.log("🆕 New page event triggered", { stableTabId, url: page.url() });
             // We get the ID immediately, but it might be null if the page is too new
             const cdpTargetId = await this.getCdpTargetId(page);
-            this.pages.set(stableTabId, { page, cdpTargetId });
+            this.pages.set(stableTabId, {
+                page,
+                cdpTargetId,
+                lastKnownTitle: undefined,
+                lastKnownUrl: page.url(),
+                lastKnownWsDebuggerUrl: cdpTargetId ? `${this.wsUrlBase}${cdpTargetId}` : undefined,
+            });
             if (cdpTargetId) {
                 this.log("✅ Page mapped to CDP ID", { stableTabId, cdpTargetId });
             }
@@ -336,6 +362,12 @@ export class RemoteBrowserService extends EventEmitter {
         const pagesData = [];
         const pagesToDelete = []; // To clean up closed pages
         for (const [stableTabId, pageInfo] of this.pages.entries()) {
+            const pageData = {
+                id: stableTabId,
+                title: pageInfo.lastKnownTitle ?? "",
+                url: pageInfo.lastKnownUrl ?? pageInfo.page.url(),
+                wsDebuggerUrl: pageInfo.lastKnownWsDebuggerUrl ?? "",
+            };
             try {
                 if (pageInfo.page.isClosed()) {
                     this.log("🧹 Found closed page during getState, marking for deletion", { stableTabId });
@@ -349,23 +381,35 @@ export class RemoteBrowserService extends EventEmitter {
                     pageInfo.cdpTargetId = currentCdpId; // Update our internal reference
                 }
                 // Manually construct the WebSocket URL
-                const wsDebuggerUrl = currentCdpId ? `${this.wsUrlBase}${currentCdpId}` : "";
+                const wsDebuggerUrl = currentCdpId ? `${this.wsUrlBase}${currentCdpId}` : pageInfo.lastKnownWsDebuggerUrl ?? "";
                 if (!wsDebuggerUrl) {
                     this.log("⚠️ Could not get CDP ID, wsDebuggerUrl will be empty", { stableTabId });
                 }
                 const title = await pageInfo.page.title();
-                pagesData.push({
-                    id: stableTabId,
-                    title,
-                    url: pageInfo.page.url(),
-                    wsDebuggerUrl: wsDebuggerUrl, // Use the constructed URL
-                });
-                this.log(`🟥Page data: ${JSON.stringify(pagesData, null, 2)}`);
+                const currentUrl = pageInfo.page.url();
+                pageData.title = title;
+                pageData.url = currentUrl;
+                pageData.wsDebuggerUrl = wsDebuggerUrl; // Use the constructed URL
+                pageInfo.lastKnownTitle = title;
+                pageInfo.lastKnownUrl = currentUrl;
+                pageInfo.lastKnownWsDebuggerUrl = wsDebuggerUrl;
+                this.log("🟥Page data", pageData);
             }
             catch (error) {
-                this.log("❌ Error getting page data", { stableTabId, error });
-                pagesToDelete.push(stableTabId); // Mark for deletion
+                const message = error instanceof Error ? error.message : JSON.stringify(error);
+                if (this.isTransientPageDataError(error)) {
+                    this.log("⏳ Transient error getting page data, will retry", { stableTabId, message });
+                }
+                else {
+                    this.log("❌ Error getting page data", { stableTabId, message });
+                    pagesToDelete.push(stableTabId); // Mark for deletion
+                    continue;
+                }
+            }
+            if (!pageData.title) {
+                pageData.title = pageInfo.page.url() === "about:blank" ? "" : "Loading...";
             }
+            pagesData.push(pageData);
         }
         pagesToDelete.forEach((id) => this.pages.delete(id));
         if (this._selectedPageId && !this.pages.has(this._selectedPageId)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev-blinq/cucumber_client",
-  "version": "1.0.1626-dev",
+  "version": "1.0.1628-dev",
   "description": " ",
   "main": "bin/index.js",
   "types": "bin/index.d.ts",
@@ -39,7 +39,7 @@
     "@cucumber/tag-expressions": "^6.1.1",
     "@dev-blinq/cucumber-js": "1.0.208-dev",
     "@faker-js/faker": "^8.4.1",
-    "automation_model": "1.0.893-dev",
+    "automation_model": "1.0.894-dev",
     "axios": "^1.7.4",
     "chokidar": "^3.6.0",
     "create-require": "^1.1.1",
@@ -68,6 +68,7 @@
     "mocha": "^10.2.0",
     "ncp": "^2.0.0",
     "readline-sync": "^1.4.10",
+    "ts-node": "^10.9.2",
     "tsup": "^8.5.0",
     "typescript": "^5.9.2"
   }