@analitikisler/ai-tag-debugger 0.7.3 → 0.7.4

package/CHANGELOG.md

@@ -1,5 +1,10 @@
 # Changelog
 
+## 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.4",
   "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

@@ -791,7 +791,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