Feat zoning 010526

[jxiwang]

Summary

Checklist

• Does your PR title have the correct title format?
• Does your PR have a breaking change?:

[macroscopeapp]

Add zoning feature by upgrading snippets to analytics-browser 2.33.2-feat-zoning-010526.0 and implement 150ms exposure tracking with plugin-autocapture-browser viewport content updates

Upgrade browser and GTM snippets to 2.33.2-feat-zoning-010526.0 with new SRI, add scroll and exposure observables, track exposures after 150ms, batch and emit [Amplitude] Viewport Content Updated, and include [Amplitude] Page View ID in page view events with sessionStorage persistence.

📍Where to Start

Start with autocapturePlugin in autocapture-plugin.ts, then review trackExposure in track-exposure.ts and fireViewportContentUpdated in track-viewport-content-updated.ts.

---

📊 Macroscope summarized 3a8470e. 32 files reviewed, 14 issues evaluated, 0 issues filtered, 3 comments posted. View details

[macroscopeapp]

🟡 Medium

packages/plugin-autocapture-browser/src/autocapture-plugin.ts:346

The lastScroll parameter is reassigned locally (lastScroll = { ... }), so the caller's object never updates. Consider mutating the object's properties directly (lastScroll.maxX = ...; lastScroll.maxY = ...) instead of reassigning.

🚀 Want me to fix this? Reply ex: "fix it for me".

[macroscopeapp]

🟡 Medium

packages/plugin-page-view-tracking-browser/src/page-view-tracking.ts:114

Accessing globalScope.sessionStorage can throw a SecurityError in restricted environments (e.g., cross-origin iframes). Consider wrapping the BrowserStorage initialization in a try/catch to gracefully degrade.

-        // init session storage
-        sessionStorage = new BrowserStorage<PageViewSessionStorage>(globalScope.sessionStorage);
+        // init session storage
+        try {
+          sessionStorage = new BrowserStorage<PageViewSessionStorage>(globalScope.sessionStorage);
+        } catch {
+          // Storage may be restricted in some environments
+        }

🚀 Want me to fix this? Reply ex: "fix it for me".

[macroscopeapp]

🟡 Medium

packages/plugin-autocapture-browser/src/autocapture/track-viewport-content-updated.ts:66

Reassigning the local lastScroll parameter (line 66) doesn't update the caller's state, so deduplication will fail after the first scroll. Consider mutating the object properties directly (lastScroll.maxX = ...; lastScroll.maxY = ...) instead of reassigning.

Suggested change

	lastScroll = { maxX: pageScrollMaxState.maxX, maxY: pageScrollMaxState.maxY };
	lastScroll.maxX = pageScrollMaxState.maxX;
	lastScroll.maxY = pageScrollMaxState.maxY;

🚀 Want me to fix this? Reply ex: "fix it for me".