nodebb-plugin-ezoic-infinite 1.5.31 → 1.5.32

package/package.json

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

package/public/client.js

@@ -11,11 +11,11 @@
   const MAX_INSERTS_PER_RUN = 3;
 
   // Preload before viewport (earlier load for smoother scroll)
-  const PRELOAD_MARGIN_DESKTOP = '1600px 0px 1600px 0px';
-  const PRELOAD_MARGIN_MOBILE = '900px 0px 900px 0px';
+  const PRELOAD_MARGIN_DESKTOP = '2600px 0px 2600px 0px';
+  const PRELOAD_MARGIN_MOBILE = '1500px 0px 1500px 0px';
 
-  const MAX_INFLIGHT_DESKTOP = 3;
-  const MAX_INFLIGHT_MOBILE = 2;
+  const MAX_INFLIGHT_DESKTOP = 4;
+  const MAX_INFLIGHT_MOBILE = 3;
 
   function isMobile() {
     try { return window && window.innerWidth && window.innerWidth < 768; } catch (e) { return false; }
@@ -77,28 +77,7 @@
 
   const insertingIds = new Set();
 
-  function enqueueShow(id) {
-    if (!id || isBlocked()) return;
-    const max = getMaxInflight();
-    if (state.inflight >= max) {
-      if (!state.pendingSet.has(id)) {
-        state.pending.push(id);
-        state.pendingSet.add(id);
-      }
-      return;
     }
-    startShow(id);
-  }
-
-  function drainQueue() {
-    if (isBlocked()) return;
-    const max = getMaxInflight();
-    while (state.inflight < max && state.pending.length) {
-      const id = state.pending.shift();
-      state.pendingSet.delete(id);
-      startShow(id);
-    }
-  }
 
   function markEmptyWrapper(id) {
     try {
@@ -467,29 +446,50 @@ function buildWrap(id, kindClass, afterPos) {
   }
 
   function enqueueShow(id) {
-    if (!id || isBlocked()) return;
+  if (!id || isBlocked()) return;
+
+  // Basic per-id throttle (prevents rapid re-requests when DOM churns)
+  const now = Date.now();
+  const last = state.lastShowById.get(id) || 0;
+  if (now - last < 900) return;
+
+  const max = getMaxInflight();
+  if (state.inflight >= max) {
+    if (!state.pendingSet.has(id)) {
+      state.pending.push(id);
+      state.pendingSet.add(id);
+    }
+    return;
+  }
+  startShow(id);
+}
 
-    const now = Date.now();
-    const last = state.lastShowById.get(id) || 0;
-    if (now - last < 1500) return; // basic throttle
-
-    // Defer one frame
-
-    // Budget concurrent loads
-    state.inflight++;
-    let released = false;
-    const release = () => {
-      if (released) return;
-      released = true;
-      state.inflight = Math.max(0, state.inflight - 1);
-      drainQueue();
-    };
+function drainQueue() {
+  if (isBlocked()) return;
+  const max = getMaxInflight();
+  while (state.inflight < max && state.pending.length) {
+    const id = state.pending.shift();
+    state.pendingSet.delete(id);
+    startShow(id);
+  }
+}
 
-    // Safety release in case Ezoic never fills
-    const hardTimer = setTimeout(release, 6500);
+function startShow(id) {
+  if (!id || isBlocked()) return;
 
-    // Defer one frame so the placeholder is definitely in DOM after insertion/recycle
-    requestAnimationFrame(() => {
+  state.inflight++;
+  let released = false;
+  const release = () => {
+    if (released) return;
+    released = true;
+    state.inflight = Math.max(0, state.inflight - 1);
+    drainQueue();
+  };
+
+  const hardTimer = setTimeout(release, 6500);
+
+  requestAnimationFrame(() => {
+    try {
       if (isBlocked()) return;
 
       const ph = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
@@ -497,59 +497,36 @@ function buildWrap(id, kindClass, afterPos) {
 
       const now2 = Date.now();
       const last2 = state.lastShowById.get(id) || 0;
-      if (now2 - last2 < 1200) return;
+      if (now2 - last2 < 900) return;
       state.lastShowById.set(id, now2);
 
-      try {
-        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) && typeof ez.destroyPlaceholders === 'function') {
-              // Avoid Ezoic caching state for reused placeholders
-              ez.destroyPlaceholders(id);
-            }
-          } catch (e) {}
-          try { ez.showAds(id); } catch (e) {}
-          try { state.usedOnce && state.usedOnce.add(id); } catch (e) {}
-          try { markEmptyWrapper(id); } catch (e) {}
-          // allow a short time for DOM fill; then release budget
-          setTimeout(() => { clearTimeout(hardTimer); release(); }, 700);
-        };
+      const doShow = () => {
+        try {
+          if (state.usedOnce && state.usedOnce.has(id) && typeof ez.destroyPlaceholders === 'function') {
+            try { ez.destroyPlaceholders(id); } catch (e) {}
+          }
+        } catch (e) {}
 
-        // Fast path
-        if (typeof ez.showAds === 'function') {
-          doShow();
-          return;
-        }
+        try { ez.showAds(id); } catch (e) {}
+        try { state.usedOnce && state.usedOnce.add(id); } catch (e) {}
+        try { markEmptyWrapper(id); } catch (e) {}
 
-        // Queue once for when Ezoic is ready
-        ez.cmd = ez.cmd || [];
-        if (!ph.__ezoicQueued) {
-          ph.__ezoicQueued = true;
-          ez.cmd.push(() => {
-            try {
-              if (isBlocked()) return;
-              const el = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
-              if (!el || !el.isConnected) return;
-              const ez2 = window.ezstandalone;
-              if (!ez2 || typeof ez2.showAds !== 'function') return;
-              try {
-                if (state.usedOnce && state.usedOnce.has(id) && typeof ez2.destroyPlaceholders === 'function') {
-                  ez2.destroyPlaceholders(id);
-                }
-              } catch (e) {}
-              try { ez2.showAds(id); } catch (e) {}
-              try { state.usedOnce && state.usedOnce.add(id); } catch (e) {}
-              try { markEmptyWrapper(id); } catch (e) {}
-              setTimeout(() => { clearTimeout(hardTimer); release(); }, 700);
-            } catch (e) {}
-          });
-        }
-      } catch (e) { try { clearTimeout(hardTimer); release(); } catch (e2) {} }
-    });
-  }
+        setTimeout(() => { clearTimeout(hardTimer); release(); }, 650);
+      };
+
+      if (Array.isArray(ez.cmd)) {
+        try { ez.cmd.push(doShow); } catch (e) { doShow(); }
+      } else {
+        doShow();
+      }
+    } finally {
+      // If we returned early, hardTimer will release.
+    }
+  });
+}
 
 
   // ---------- preload / above-the-fold ----------
@@ -583,7 +560,7 @@ function buildWrap(id, kindClass, afterPos) {
     // If already above fold, fire immediately
     try {
       const r = ph.getBoundingClientRect();
-      if (r.top < window.innerHeight * 1.5 && r.bottom > -200) enqueueShow(id);
+      if (r.top < window.innerHeight * 3.0 && r.bottom > -800) enqueueShow(id);
     } catch (e) {}
   }