@analitikisler/ai-tag-debugger 0.7.3 → 0.7.5

package/CHANGELOG.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 0.7.5 (2026-10-14)
+
+- A saved click keeps the element's text as well as its path through the page. When a replay finds nothing visible at that path (a banner or overlay shifted the layout), it clicks the element with that text instead. Sites saved before 0.7.5 have no text to fall back on; save them again to use it.
+
+## 0.7.4 (2026-10-14)
+
+- Saving a site no longer fails on pages that keep a request open after they are left (a GA4 hit to `www.google.com/g/collect`, for example). The recording waited for it until the 30-second limit and was then skipped, so the site was not saved. The browser's pages are now closed before the recording is written.
+- Fewer test hits from popups reach the real account while a recording is written.
+
 ## 0.7.3 (2026-10-13)
 
 - `replay` no longer crashes with `NavigationAbortedError: net::ERR_BLOCKED_BY_CLIENT` when a saved page redirects to another site. The redirect is refused again, so the journey stays unfinished as it was saved, and the other sites are still replayed.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@analitikisler/ai-tag-debugger",
-  "version": "0.7.3",
+  "version": "0.7.5",
   "description": "Open-source AI agent that QA-tests your website's analytics, ad and consent tracking (GA4, GTM, Google Ads, Meta, TikTok). Bring your own model (Claude, OpenAI, Gemini) or use it from your chat app over MCP.",
   "type": "module",
   "bin": {

package/src/browser.js

@@ -202,7 +202,7 @@ function trackingBody(request) {
  * else a CSS path to the element, else its visible text. Refs only mean something in one run.
  */
 async function replayTarget(a, locator) {
-  if (a.selector) return { selector: a.selector };
+  if (a.selector) return { selector: a.selector, ...(a.text ? { text: a.text } : {}) };
   const found = await locator
     .evaluate((el) => {
       const text = (el.innerText || el.value || el.getAttribute("aria-label") || "").replace(/\s+/g, " ").trim().slice(0, 80);
@@ -220,8 +220,10 @@ async function replayTarget(a, locator) {
       return { selector: parts.join(" > "), text };
     }, null, { timeout: 5000 })
     .catch(() => null);
-  if (found?.selector) return { selector: found.selector };
-  return found?.text ? { text: found.text } : a.text ? { text: a.text } : {};
+  const text = found?.text || a.text;
+  // The text too, since a path through the page's layout breaks when a banner or overlay shifts it.
+  if (found?.selector) return { selector: found.selector, ...(text ? { text } : {}) };
+  return text ? { text } : {};
 }
 
 // Images, fonts and video don't change what tracking sends, and are most of a recording's size.
@@ -739,7 +741,15 @@ export class AuditSession {
         case "fill":
         case "select": {
           let target;
-          if (a.selector || Number.isInteger(a.ref)) target = this.locatorFor(a);
+          if (a.selector && typeof a.text === "string" && a.text.trim()) {
+            // A saved step: its path first, then its text if the path finds nothing to click.
+            target = this.locatorFor(a);
+            const shown = await target.waitFor({ state: "visible", timeout: 5000 }).then(() => true, () => false);
+            if (!shown) {
+              const found = await this.locateByText(a.text.trim());
+              if (!found.error) target = found.locator;
+            }
+          } else if (a.selector || Number.isInteger(a.ref)) target = this.locatorFor(a);
           else if (typeof a.text === "string" && a.text.trim()) {
             const found = await this.locateByText(a.text.trim());
             if (found.error) return found.error;
@@ -791,7 +801,20 @@ export class AuditSession {
   async close() {
     // Leaving the page first cancels what it is still loading (the recording would otherwise wait for it),
     // while tracking requests sent on the way out are still answered locally.
-    if (this.opts.recordHar) await this.page?.goto("about:blank", { timeout: 5000 }).catch(() => {});
+    if (this.opts.recordHar && this.context) {
+      await this.page?.goto("about:blank", { timeout: 5000 }).catch(() => {});
+      // The recording also waits for requests the browser holds for an open page, such as a tracking hit
+      // sent as the page was left, which can stay open for good. Once their page is closed it stops waiting.
+      // A moment first lets hits already sent be answered locally: once their page is closed,
+      // the browser would send them on its own.
+      await new Promise((resolve) => setTimeout(resolve, 1000));
+      let pagesTimer;
+      await Promise.race([
+        Promise.all(this.context.pages().map((p) => p.close().catch(() => {}))),
+        new Promise((resolve) => (pagesTimer = setTimeout(resolve, 5000))),
+      ]);
+      clearTimeout(pagesTimer);
+    }
     if (this.opts.replayHar) {
       // The recording's own route callbacks would otherwise throw for requests still in flight once the
       // browser is gone. Without routes, those go nowhere (see start). Elsewhere the routes stay, so a