nodebb-plugin-ezoic-infinite 1.5.37 → 1.5.38

package/package.json

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

package/public/client.js

@@ -8,24 +8,24 @@
   const PLACEHOLDER_PREFIX = 'ezoic-pub-ad-placeholder-';
 
   // Insert at most N ads per run to keep the UI smooth on infinite scroll
-  const MAX_INSERTS_PER_RUN = 3;
+  const MAX_INSERTS_PER_RUN = 6;
 
   // Preload before viewport (earlier load for smoother scroll)
   // Preload far enough ahead that fast scroll doesn't outrun ad loading.
   // Slightly more aggressive margins to reduce “I scrolled past it before it loaded”.
-  const PRELOAD_MARGIN_DESKTOP = '4200px 0px 4200px 0px';
-  const PRELOAD_MARGIN_MOBILE = '2400px 0px 2400px 0px';
+  const PRELOAD_MARGIN_DESKTOP = '6000px 0px 6000px 0px';
+  const PRELOAD_MARGIN_MOBILE = '3500px 0px 3500px 0px';
 
   // When the user scrolls very fast, temporarily preload more aggressively.
   // This helps ensure ads are already in-flight before the user reaches them.
-  const PRELOAD_MARGIN_DESKTOP_BOOST = '6500px 0px 6500px 0px';
-  const PRELOAD_MARGIN_MOBILE_BOOST = '4200px 0px 4200px 0px';
+  const PRELOAD_MARGIN_DESKTOP_BOOST = '8500px 0px 8500px 0px';
+  const PRELOAD_MARGIN_MOBILE_BOOST = '5500px 0px 5500px 0px';
   const BOOST_DURATION_MS = 2500;
   const BOOST_SPEED_PX_PER_MS = 2.2; // ~2200px/s
 
   // Allow a bit more parallelism; the perf profile can still dial this down on low-end devices.
-  const MAX_INFLIGHT_DESKTOP = 5;
-  const MAX_INFLIGHT_MOBILE = 4;
+  const MAX_INFLIGHT_DESKTOP = 8;
+  const MAX_INFLIGHT_MOBILE = 6;
 
   
 // Adaptive performance profile (device/network aware)
@@ -575,50 +575,53 @@ function startShow(id) {
     drainQueue();
   };
 
+  // Safety release even if the ad pipeline stalls
   const hardTimer = setTimeout(release, 6500);
 
-  requestAnimationFrame(() => {
-    try {
-      if (isBlocked()) return;
+  try {
+    if (isBlocked()) return;
 
-      const ph = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
-      if (!ph || !ph.isConnected) return;
+    const ph = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
+    if (!ph || !ph.isConnected) return;
 
-      const now2 = Date.now();
-      const last2 = state.lastShowById.get(id) || 0;
-      if (now2 - last2 < 900) return;
-      state.lastShowById.set(id, now2);
+    // Guard against rapid duplicate calls for the same id
+    const now2 = Date.now();
+    const last2 = state.lastShowById.get(id) || 0;
+    if (now2 - last2 < 250) return;
+    state.lastShowById.set(id, now2);
 
-      window.ezstandalone = window.ezstandalone || {};
-      const ez = window.ezstandalone;
+    window.ezstandalone = window.ezstandalone || {};
+    const ez = window.ezstandalone;
 
-      const doShow = () => {
-        try {
-          if (state.usedOnce && state.usedOnce.has(id)) {
-            safeDestroyById(id);
-          }
-        } catch (e) {}
+    const doShow = () => {
+      try {
+        if (state.usedOnce && state.usedOnce.has(id)) {
+          safeDestroyById(id);
+        }
+      } catch (e) {}
 
-        try { ez.showAds(id); } catch (e) {}
-        try { state.usedOnce && state.usedOnce.add(id); } catch (e) {}
+      try { ez.showAds(id); } catch (e) {}
+      try { state.usedOnce && state.usedOnce.add(id); } catch (e) {}
 
-        // Tighten any oversized reserved height once the creative is in the DOM.
-        try { scheduleTighten(id); } catch (e) {}
+      // Tighten any oversized reserved height once the creative is in the DOM.
+      try { scheduleTighten(id); } catch (e) {}
 
-        setTimeout(() => { clearTimeout(hardTimer); release(); }, 650);
-      };
+      // Release budget quickly; the creative can keep loading independently.
+      setTimeout(() => { clearTimeout(hardTimer); release(); }, 300);
+    };
 
-      if (Array.isArray(ez.cmd)) {
-        try { ez.cmd.push(doShow); } catch (e) { doShow(); }
-      } else {
-        doShow();
-      }
-    } finally {
-      // If we returned early, hardTimer will release.
+    // Use cmd queue if present, but don't delay to next frame.
+    if (Array.isArray(ez.cmd)) {
+      try { ez.cmd.push(doShow); } catch (e) { doShow(); }
+    } else {
+      doShow();
     }
-  });
+  } finally {
+    // If we returned early, hardTimer will release.
+  }
 }
 
+
 // ---------- height normalization (reduce empty space) ----------
 // Some Ezoic / GPT wrappers reserve a larger min-height (e.g., 400px) than the rendered creative (e.g., 250px),
 // leaving visible empty space. We “tighten” the reserved min-height to the actual iframe/container height after load.
@@ -676,16 +679,43 @@ function ensureHeightObserver(id) {
       tightenEzoicHeightFor(id);
     }) : null;
 
-    if (ro) {
-      const ad = ph.querySelector('span.ezoic-ad');
-      const container = ad && ad.querySelector && ad.querySelector('div[id$="__container__"]');
-      const iframe = ad && ad.querySelector && ad.querySelector('iframe');
-      try { if (container) ro.observe(container); } catch (e) {}
-      try { if (iframe) ro.observe(iframe); } catch (e) {}
-      try { if (ad) ro.observe(ad); } catch (e) {}
+    // Also watch for Ezoic re-applying an oversized inline min-height (style attribute changes).
+    const mo = (typeof MutationObserver === 'function') ? new MutationObserver(() => {
+      tightenEzoicHeightFor(id);
+    }) : null;
+
+    const attach = () => {
+      const ad = ph.querySelector('span.ezoic-ad[data-ez-name], span.ezoic-ad');
+      if (!ad) return false;
+
+      if (ro) {
+        const container = ad.querySelector && ad.querySelector('div[id$="__container__"]');
+        const iframe = ad.querySelector && ad.querySelector('iframe');
+        try { if (container) ro.observe(container); } catch (e) {}
+        try { if (iframe) ro.observe(iframe); } catch (e) {}
+        try { ro.observe(ad); } catch (e) {}
+      }
+      if (mo) {
+        try { mo.observe(ad, { attributes: true, attributeFilter: ['style', 'class'] }); } catch (e) {}
+      }
+      return true;
+    };
+
+    // If the ad wrapper is injected slightly later, observe the placeholder briefly for children.
+    if (!attach() && typeof MutationObserver === 'function') {
+      try {
+        const tmp = new MutationObserver(() => {
+          if (attach()) {
+            try { tmp.disconnect(); } catch (e) {}
+            tightenEzoicHeightFor(id);
+          }
+        });
+        tmp.observe(ph, { childList: true, subtree: true });
+        setTimeout(() => { try { tmp.disconnect(); } catch (e) {} }, 3000);
+      } catch (e) {}
     }
 
-    _heightObsByPlaceholder.set(id, ro || true);
+    _heightObsByPlaceholder.set(id, { ro: ro || null, mo: mo || null });
   } catch (e) {}
 }
 
@@ -777,7 +807,7 @@ function scheduleTighten(id) {
     const targets = computeTargets(items.length, interval, showFirst);
     let inserted = 0;
     const perf = getPerfProfile();
-    const maxInserts = perf.maxInsertsPerRun + (isBoosted() ? 1 : 0);
+    const maxInserts = perf.maxInsertsPerRun + (isBoosted() ? 2 : 0);
 
     for (const afterPos of targets) {
       if (inserted >= maxInserts) break;
@@ -959,7 +989,9 @@ function scheduleTighten(id) {
     // disconnect any ResizeObservers used for height tightening
     try {
       for (const v of _heightObsByPlaceholder.values()) {
-        try { v && v.disconnect && v.disconnect(); } catch (e) {}
+        try { if (v && v.ro && v.ro.disconnect) v.ro.disconnect(); } catch (e) {}
+        try { if (v && v.mo && v.mo.disconnect) v.mo.disconnect(); } catch (e) {}
+        try { if (v && v.disconnect) v.disconnect(); } catch (e) {} // backward compat
       }
       _heightObsByPlaceholder.clear();
     } catch (e) {}