@analitikisler/ai-tag-debugger 0.7.4 → 0.7.6

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## 0.7.6 (2026-10-14)
+
+- `replay` says when a saved click was found by its text rather than its saved place (`~` lines), since that may not be the element that was saved.
+- Only clicks fall back to their text. Typing into a field or choosing an option uses its saved place only.
+- A click's text is not saved when it looks like an email address or phone number.
+
+## 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.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@analitikisler/ai-tag-debugger",
-  "version": "0.7.4",
+  "version": "0.7.6",
   "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

@@ -201,8 +201,13 @@ function trackingBody(request) {
  * How a scripted run can find the element an action used again: the selector it was given,
  * else a CSS path to the element, else its visible text. Refs only mean something in one run.
  */
+// Text that looks like an email address or phone number is not saved with a click.
+const savableText = (text) => (typeof text === "string" && text && !/@|\d[\d\s().+-]{6,}\d/.test(text) ? text : undefined);
+
 async function replayTarget(a, locator) {
-  if (a.selector) return { selector: a.selector };
+  // Only a click keeps its text: a field's text is what was typed in it, not what identifies it.
+  const keepText = a.action === "click";
+  if (a.selector) return { selector: a.selector, ...(keepText && savableText(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 +225,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, ...(keepText && savableText(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 +746,18 @@ export class AuditSession {
         case "fill":
         case "select": {
           let target;
-          if (a.selector || Number.isInteger(a.ref)) target = this.locatorFor(a);
+          if (a.action === "click" && a.selector && typeof a.text === "string" && a.text.trim()) {
+            // A saved click: 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;
+                step.foundBy = "text"; // the replay summary says so, in case it is not the element saved
+              }
+            }
+          } 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;

package/src/cli.js

@@ -125,6 +125,7 @@ async function main() {
       for (const k of r.extra) console.log(`    + ${k}`);
       for (const k of r.missing) console.log(`    - ${k}`);
       for (const note of r.incomplete) console.log(`    ! ${note}`);
+      for (const note of r.foundByText) console.log(`    ~ ${note}`);
     }
     if (changed && !values.update) process.exitCode = 1;
     return;

package/src/replay.js

@@ -65,7 +65,7 @@ export async function writeSavedSite(dir, { config, plan, runs, findings }) {
 /**
  * Audits a saved site again, offline and without AI, and compares the findings with the expected ones.
  * update: make this replay's findings the expected ones.
- * @returns {Promise<{ site: string, missing: string[], extra: string[], reportPaths: string[], incomplete: string[] }>}
+ * @returns {Promise<{ site: string, missing: string[], extra: string[], reportPaths: string[], incomplete: string[], foundByText: string[] }>}
  */
 export async function replaySavedSite(dir, { update = false, outDir, log = () => {} } = {}) {
   const file = path.join(dir, SAVED_SITE_FILE);
@@ -79,6 +79,7 @@ export async function replaySavedSite(dir, { update = false, outDir, log = () =>
   const findings = [];
   const reportPaths = [];
   const incomplete = [];
+  const foundByText = [];
   for (const viewport of viewports) {
     const journeys = saved.journeys.filter((j) => j.viewport === viewport).map(({ viewport: _v, ...j }) => j);
     const out = path.resolve(outDir ?? path.join(dir, "last-replay"), viewports.length > 1 ? viewport : "");
@@ -98,11 +99,14 @@ export async function replaySavedSite(dir, { update = false, outDir, log = () =>
     // Journeys that were already unfinished when saved are compared through their findings only.
     const wasIncomplete = (r) => journeys.find((j) => j.id === r.journeyId)?.incomplete === r.note;
     incomplete.push(...result.runs.filter((r) => !r.completed && !wasIncomplete(r)).map((r) => `${r.journeyId} (${r.viewport}): ${r.note}`));
+    for (const r of result.runs) {
+      for (const s of r.steps.filter((s) => s.foundBy === "text")) foundByText.push(`${r.journeyId} (${r.viewport}): "${s.label}" was clicked by its text, since nothing was at its saved place.`);
+    }
   }
   const actual = findingKeys(findings);
   const { missing, extra } = compareKeys(saved.expected ?? [], actual);
   if (update && (missing.length || extra.length)) {
     await writeFile(file, JSON.stringify({ ...saved, expected: actual }, null, 2));
   }
-  return { site: saved.config.site, missing, extra, reportPaths, incomplete };
+  return { site: saved.config.site, missing, extra, reportPaths, incomplete, foundByText };
 }