nodebb-plugin-ezoic-infinite 1.5.71 → 1.5.72

package/package.json

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

package/public/client.js

@@ -12,6 +12,10 @@
   // Smoothness caps
   const MAX_INSERTS_PER_RUN = 3;
 
+  // Keep empty (unfilled) wraps alive for a while. Topics/messages can fill late (auction/CMP).
+  // Pruning too early makes ads look like they "disappear" while scrolling.
+  const KEEP_EMPTY_WRAP_MS = 30000;
+
   // Preload margins
   const PRELOAD_MARGIN_DESKTOP = '2600px 0px 2600px 0px';
   const PRELOAD_MARGIN_MOBILE = '1500px 0px 1500px 0px';
@@ -207,6 +211,22 @@
     return el;
   }
 
+  function primePlaceholderPool(allIds) {
+    try {
+      if (!Array.isArray(allIds) || !allIds.length) return;
+      const pool = getPoolEl();
+      for (const id of allIds) {
+        if (!id) continue;
+        const domId = `${PLACEHOLDER_PREFIX}${id}`;
+        if (document.getElementById(domId)) continue;
+        const ph = document.createElement('div');
+        ph.id = domId;
+        ph.setAttribute('data-ezoic-id', String(id));
+        pool.appendChild(ph);
+      }
+    } catch (e) {}
+  }
+
   function isInPool(ph) {
     try { return !!(ph && ph.closest && ph.closest('#' + POOL_ID)); } catch (e) { return false; }
   }
@@ -319,6 +339,13 @@
     if (!state.allTopics.length) state.allTopics = parsePool(cfg.placeholderIds);
     if (!state.allPosts.length) state.allPosts = parsePool(cfg.messagePlaceholderIds);
     if (!state.allCategories.length) state.allCategories = parsePool(cfg.categoryPlaceholderIds);
+
+    // Create placeholders up-front in an offscreen pool.
+    // Ezoic may attempt to define/show ids during load; if they don't exist yet,
+    // it can spam errors and sometimes short-circuit. Pooling keeps ids existing without layout.
+    primePlaceholderPool(state.allTopics);
+    primePlaceholderPool(state.allPosts);
+    primePlaceholderPool(state.allCategories);
   }
 
   // ---------------- insertion primitives ----------------
@@ -328,6 +355,7 @@
     wrap.className = `${WRAP_CLASS} ${kindClass}`;
     wrap.setAttribute('data-ezoic-after', String(afterPos));
     wrap.setAttribute('data-ezoic-wrapid', String(id));
+    wrap.setAttribute('data-created', String(now()));
     wrap.style.width = '100%';
 
     if (createPlaceholder) {
@@ -398,12 +426,12 @@
     const hasNearbyItem = (wrap) => {
       // NodeBB/skins can inject separators/spacers; be tolerant.
       let prev = wrap.previousElementSibling;
-      for (let i = 0; i < 8 && prev; i++) {
+      for (let i = 0; i < 14 && prev; i++) {
         if (itemSet.has(prev)) return true;
         prev = prev.previousElementSibling;
       }
       let next = wrap.nextElementSibling;
-      for (let i = 0; i < 8 && next; i++) {
+      for (let i = 0; i < 14 && next; i++) {
         if (itemSet.has(next)) return true;
         next = next.nextElementSibling;
       }
@@ -412,6 +440,13 @@
 
     wraps.forEach((wrap) => {
       if (isFilled(wrap)) return; // never prune filled ads
+
+      // Never prune a fresh wrap: it may fill late.
+      try {
+        const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
+        if (created && (now() - created) < KEEP_EMPTY_WRAP_MS) return;
+      } catch (e) {}
+
       if (hasNearbyItem(wrap)) return;
 
       withInternalDomChange(() => releaseWrapNode(wrap));
@@ -428,13 +463,32 @@
 
     const isWrap = (el) => !!(el && el.classList && el.classList.contains(WRAP_CLASS));
 
+    const isFilled = (wrap) => {
+      return !!(wrap && wrap.querySelector && wrap.querySelector('iframe, ins, img, video, [data-google-container-id]'));
+    };
+
+    const isFresh = (wrap) => {
+      try {
+        const created = parseInt(wrap.getAttribute('data-created') || '0', 10);
+        return created && (now() - created) < KEEP_EMPTY_WRAP_MS;
+      } catch (e) {
+        return false;
+      }
+    };
+
     let removed = 0;
     for (const w of wraps) {
       let prev = w.previousElementSibling;
       for (let i = 0; i < 3 && prev; i++) {
         if (isWrap(prev)) {
-          withInternalDomChange(() => releaseWrapNode(w));
-          removed++;
+          // Don't decluster two "fresh" empty wraps — it can reduce fill on slow auctions.
+          // Only decluster when at least one is filled, or when the newer one is stale.
+          const prevFilled = isFilled(prev);
+          const curFilled = isFilled(w);
+          if (prevFilled || curFilled || !isFresh(w)) {
+            withInternalDomChange(() => releaseWrapNode(w));
+            removed++;
+          }
           break;
         }
         prev = prev.previousElementSibling;
@@ -545,10 +599,17 @@
           if (!ph2 || !ph2.isConnected) return;
           const w2 = ph2.closest ? ph2.closest(`.${WRAP_CLASS}`) : null;
           if (!w2) return;
+
+          // Don't collapse "fresh" placements; slow auctions/CMP can fill late.
+          try {
+            const created = parseInt(w2.getAttribute('data-created') || '0', 10);
+            if (created && (now() - created) < KEEP_EMPTY_WRAP_MS) return;
+          } catch (e) {}
+
           const hasAd = !!(ph2.querySelector && ph2.querySelector('iframe, ins, img, .ez-ad, .ezoic-ad'));
           if (!hasAd) w2.classList.add('is-empty');
         } catch (e) {}
-      }, 3500);
+      }, 15000);
     } catch (e) {}
   }