@dev-blinq/cucumber_client 1.0.1586-dev → 1.0.1588-dev

package/bin/client/recorderv3/services.js

@@ -303,15 +303,16 @@ export class RemoteBrowserService extends EventEmitter {
         try {
             const pageUrl = page.url();
             this.log("🔍 Getting page ID", { pageUrl });
+            // Fetch debug data from /json endpoint (more reliable for matching)
             const debugData = await this.getDebugURLs();
             this.log("📊 CDP debug data received", {
                 totalPages: debugData.length,
                 pages: debugData.map((p) => ({ id: p.id, type: p.type, url: p.url })),
             });
-            // Exact match
+            // Exact URL match
             for (const pageData of debugData) {
                 if (pageData.type === "page" && pageData.url === pageUrl) {
-                    this.log("✅ Found exact URL match", {
+                    this.log("✅ Found exact URL match in /json", {
                         id: pageData.id,
                         url: pageUrl,
                     });
@@ -319,16 +320,14 @@ export class RemoteBrowserService extends EventEmitter {
                 }
             }
             this.log("⚠️ No exact match found, trying normalized URLs", { pageUrl });
-            // Normalized match
+            // Normalized URL match
             const normalizeUrl = (url) => {
                 try {
                     const u = new URL(url);
                     const normalized = u.hostname.replace(/^www\./, "") + u.pathname + u.search;
-                    this.log("🔧 Normalized URL", { original: url, normalized });
                     return normalized;
                 }
                 catch {
-                    this.log("❌ Failed to normalize URL", { url });
                     return url;
                 }
             };
@@ -337,7 +336,7 @@ export class RemoteBrowserService extends EventEmitter {
                 if (pageData.type === "page") {
                     const normalizedDebugUrl = normalizeUrl(pageData.url);
                     if (normalizedDebugUrl === normalizedPageUrl) {
-                        this.log("✅ Found normalized URL match", {
+                        this.log("✅ Found normalized URL match in /json", {
                             id: pageData.id,
                             pageUrl: normalizedPageUrl,
                             debugUrl: normalizedDebugUrl,
@@ -346,6 +345,35 @@ export class RemoteBrowserService extends EventEmitter {
                     }
                 }
             }
+            // If still not found, try CDP session as fallback
+            this.log("⚠️ Not found in /json, trying CDP Target.getTargetInfo", { pageUrl });
+            try {
+                const cdpSession = await page.context().newCDPSession(page);
+                const { targetInfo } = await cdpSession.send("Target.getTargetInfo");
+                await cdpSession.detach();
+                if (targetInfo && targetInfo.targetId) {
+                    // Verify this target ID exists in debug data
+                    const targetExists = debugData.some((d) => d.id === targetInfo.targetId);
+                    if (targetExists) {
+                        this.log("✅ Got target ID from CDP session and verified in /json", {
+                            targetId: targetInfo.targetId,
+                            url: pageUrl,
+                        });
+                        return targetInfo.targetId;
+                    }
+                    else {
+                        this.log("⚠️ Target ID from CDP session not found in /json (ID mismatch)", {
+                            cdpTargetId: targetInfo.targetId,
+                            availableIds: debugData.map((d) => d.id),
+                        });
+                    }
+                }
+            }
+            catch (cdpError) {
+                this.log("⚠️ Failed to get target ID from CDP session", {
+                    error: cdpError instanceof Error ? cdpError.message : String(cdpError),
+                });
+            }
             this.log("❌ No match found for page", {
                 pageUrl,
                 normalizedPageUrl,
@@ -417,44 +445,88 @@ export class RemoteBrowserService extends EventEmitter {
             debugPages: debugData.map((p) => ({ id: p.id, type: p.type, url: p.url })),
         });
         const pagesData = [];
-        for (const [id, page] of this.pages.entries()) {
+        const updatedPages = new Map();
+        const matchedDebugIds = new Set(); // Track which CDP IDs we've already matched
+        let updatedSelectedPageId = this._selectedPageId;
+        for (const [oldId, page] of this.pages.entries()) {
             this.log("🔍 Processing page from internal map", {
-                id,
+                oldId,
                 url: page.url(),
             });
-            const debugInfo = debugData.find((d) => d.id === id);
-            if (debugInfo) {
-                this.log("✅ Found matching debug info", {
-                    id,
+            // Try to find by old ID first (most common case - ID hasn't changed)
+            let debugInfo = debugData.find((d) => d.id === oldId && !matchedDebugIds.has(d.id));
+            // Fallback: Try to find by URL (page ID may have changed)
+            if (!debugInfo) {
+                this.log("⚠️ Page ID not found in CDP, attempting to match by URL", {
+                    oldId,
+                    pageUrl: page.url(),
+                });
+                debugInfo = debugData.find((d) => d.type === "page" && d.url === page.url() && !matchedDebugIds.has(d.id));
+                if (debugInfo) {
+                    this.log("✅ Found page by URL match, updating ID", {
+                        oldId,
+                        newId: debugInfo.id,
+                        url: page.url(),
+                    });
+                    // Update selected page ID if this was the selected page and ID changed
+                    if (oldId === this._selectedPageId) {
+                        updatedSelectedPageId = debugInfo.id;
+                        this.log("🔄 Updated selected page ID", {
+                            oldId,
+                            newId: debugInfo.id,
+                        });
+                    }
+                }
+            }
+            else {
+                this.log("✅ Found matching debug info by ID", {
+                    id: debugInfo.id,
                     debugUrl: debugInfo.url,
                     pageUrl: page.url(),
                 });
+            }
+            if (debugInfo) {
+                // Mark this CDP ID as matched to avoid duplicate matches
+                matchedDebugIds.add(debugInfo.id);
                 try {
                     const pageData = {
-                        id,
+                        id: debugInfo.id,
                         title: await page.title(),
                         url: page.url(),
                         wsDebuggerUrl: debugInfo.webSocketDebuggerUrl || "",
                     };
                     pagesData.push(pageData);
+                    updatedPages.set(debugInfo.id, page);
                     this.log("✅ Page added to state", pageData);
                 }
                 catch (error) {
                     this.log("❌ Error getting page data", {
-                        id,
+                        id: oldId,
                         error: error instanceof Error ? error.message : String(error),
                     });
                 }
             }
             else {
-                this.log("⚠️ No matching debug info found", {
-                    id,
+                this.log("⚠️ No matching debug info found by ID or URL", {
+                    oldId,
                     pageUrl: page.url(),
                     availableDebugIds: debugData.map((d) => d.id),
                     availableDebugUrls: debugData.map((d) => d.url),
+                    alreadyMatched: Array.from(matchedDebugIds),
                 });
             }
         }
+        // Update the internal pages map with current CDP IDs
+        if (updatedPages.size !== this.pages.size || updatedSelectedPageId !== this._selectedPageId) {
+            this.log("🔄 Updating internal state with new CDP IDs", {
+                oldPagesCount: this.pages.size,
+                newPagesCount: updatedPages.size,
+                oldSelectedId: this._selectedPageId,
+                newSelectedId: updatedSelectedPageId,
+            });
+            this.pages = updatedPages;
+            this._selectedPageId = updatedSelectedPageId;
+        }
         const state = {
             pages: pagesData,
             selectedPageId: this._selectedPageId,
@@ -472,11 +544,27 @@ export class RemoteBrowserService extends EventEmitter {
                 await page.goto(url, { waitUntil: "domcontentloaded" });
                 this.log("✅ Navigation complete", { finalUrl: page.url() });
             }
-            // Wait for CDP to register the page
-            this.log("⏳ Waiting for CDP registration...");
-            await new Promise((resolve) => setTimeout(resolve, 500));
-            const id = await this.getPageId(page);
-            this.log("🔍 Retrieved page ID after wait", { id, url: page.url() });
+            // Wait longer for CDP to register and stabilize the page
+            this.log("⏳ Waiting for CDP registration and stabilization...");
+            await new Promise((resolve) => setTimeout(resolve, 1000)); // Increased to 1 second
+            // Try multiple times to get a stable page ID
+            let id = null;
+            let attempts = 0;
+            const maxAttempts = 5;
+            while (!id && attempts < maxAttempts) {
+                attempts++;
+                this.log(`🔍 Attempt ${attempts}/${maxAttempts} to get page ID`);
+                id = await this.getPageId(page);
+                if (!id) {
+                    this.log(`⏳ Page ID not found, waiting 500ms before retry...`);
+                    await new Promise((resolve) => setTimeout(resolve, 500));
+                }
+            }
+            this.log("🔍 Retrieved page ID after retries", {
+                id,
+                url: page.url(),
+                attempts,
+            });
             if (id) {
                 this.pages.set(id, page);
                 this._selectedPageId = id;
@@ -489,7 +577,10 @@ export class RemoteBrowserService extends EventEmitter {
                 });
             }
             else {
-                this.log("❌ Failed to get page ID for new tab", { url: page.url() });
+                this.log("❌ Failed to get page ID for new tab after all retries", {
+                    url: page.url(),
+                    attempts,
+                });
                 // Try to get debug data to see what's available
                 const debugData = await this.getDebugURLs();
                 this.log("🔍 Current CDP state after failed ID retrieval", {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@dev-blinq/cucumber_client",
-  "version": "1.0.1586-dev",
+  "version": "1.0.1588-dev",
   "description": " ",
   "main": "bin/index.js",
   "types": "bin/index.d.ts",