nodebb-plugin-ezoic-infinite 1.6.65 → 1.6.66

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.6.65",
+  "version": "1.6.66",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -79,17 +79,12 @@
     } catch (e) {}
   }
 
-  // ─── CMP/TCF stability ──────────────────────────────────────────────────────
-  function ensureTcfApiLocator() {
-    try {
-      if (typeof window.__tcfapi !== 'function' && typeof window.__cmp !== 'function') return;
-      if (document.getElementById('__tcfapiLocator')) return;
-      const f = document.createElement('iframe');
-      f.style.display = 'none';
-      f.id = f.name = '__tcfapiLocator';
-      (document.body || document.documentElement).appendChild(f);
-    } catch (e) {}
-  }
+  // ensureTcfApiLocator intentionally removed:
+  // Creating our own __tcfapiLocator iframe conflicts with the CMP's own locator
+  // management and causes "Cannot read properties of null (reading 'postMessage')"
+  // errors when the CMP tries to postMessage to an iframe we created then navigated away.
+  // The CMP (Ezoic/IAB TCF) handles its own locator iframe lifecycle.
+  function ensureTcfApiLocator() { /* no-op */ }
 
   // ─── Ezoic min-height tightener ─────────────────────────────────────────────
   // Ezoic injects `min-height:400px !important` on nested wrappers via inline
@@ -909,23 +904,18 @@
     const maxInserts = MAX_INSERTS_PER_RUN + (isBoosted() ? 1 : 0);
     let inserted = 0;
 
-    // Viewport top — we never inject after an element that is fully above this.
-    // This is the definitive guard against NodeBB loading content above the fold
-    // and our scanner immediately injecting ads on those newly-loaded top items.
-    // We use a generous negative margin so items just barely scrolled above still
-    // get ads injected (they'll be visible when the user scrolls back a little).
-    const viewportSafeTop = -(window.innerHeight || 800) * 0.5;
-
     for (const afterPos of targets) {
       if (inserted >= maxInserts) break;
       const el = ordinalMap.get(afterPos);
       if (!el || !el.isConnected) continue;
 
-      // Skip elements that are well above the viewport.
-      // getBoundingClientRect is cheap on modern engines (no forced layout).
+      // Never inject after an element that is fully above the viewport top.
+      // This is the definitive guard: NodeBB loads items above the fold when the
+      // user scrolls up. Those items have rect.bottom < 0 at the moment of load.
+      // Any negative margin ("a little above is ok") still causes the pile-up
+      // because NodeBB loads a whole batch just above — we must use exactly 0.
       try {
-        const rect = el.getBoundingClientRect();
-        if (rect.bottom < viewportSafeTop) continue;
+        if (el.getBoundingClientRect().bottom < 0) continue;
       } catch (e) {}
 
       if (isAdjacentAd(el)) continue;