nodebb-plugin-ezoic-infinite 0.6.4 → 0.6.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "0.6.4",
+  "version": "0.6.5",
   "description": "Ezoic ads with infinite scroll using a pool of placeholder IDs",
   "main": "library.js",
   "license": "MIT",

package/public/client.js

@@ -14,7 +14,7 @@ let rerunRequested = false;
 let pageKey = null;
 let injectedSlots = new Set();       // slot numbers already injected on this page
 let usedIds = new Set();             // IDs currently present in DOM
-let adsFIFO = [];                    // [{slot, id, selector}] in insertion order (oldest first)
+let adsFIFO = [];                    // [{type, slot, id}] oldest first
 
 function resetPageState() {
   injectedSlots = new Set();
@@ -97,6 +97,19 @@ function pickNextId(pool) {
   return null;
 }
 
+function evictOldestOne() {
+  const old = adsFIFO.shift();
+  if (!old) return false;
+
+  const sel = '[data-ezoic-slot="' + old.slot + '"][data-ezoic-id="' + old.id + '"]';
+  const $el = $(sel);
+  if ($el.length) $el.remove();
+
+  injectedSlots.delete(old.slot);
+  usedIds.delete(old.id);
+  return true;
+}
+
 function callEzoic(ids) {
   if (!ids || !ids.length) return;
 
@@ -124,21 +137,6 @@ function callEzoic(ids) {
   }, 800);
 }
 
-function fifoEvictIfNeeded(poolSize) {
-  // Keep at most poolSize ads in DOM: remove oldest when we exceed pool capacity.
-  // This makes ads "follow" the scroll (new ads appear, oldest disappear) without jumping positions.
-  while (adsFIFO.length > poolSize) {
-    const old = adsFIFO.shift();
-    if (!old) break;
-
-    const $el = $(old.selector);
-    if ($el.length) $el.remove();
-
-    injectedSlots.delete(old.slot);
-    usedIds.delete(old.id);
-  }
-}
-
 function injectBetweenIncremental($items, pool, interval, wrapperClass) {
   const total = $items.length;
   const maxSlot = Math.floor(total / interval);
@@ -153,14 +151,15 @@ function injectBetweenIncremental($items, pool, interval, wrapperClass) {
     const $target = $items.eq(index);
     if (!$target.length) continue;
 
-    const id = pickNextId(pool);
+    let id = pickNextId(pool);
     if (!id) {
-      // No free IDs right now; stop. Oldest will be evicted once new ads can be created.
-      break;
+      // Pool full: evict oldest, then reuse freed ID
+      if (!evictOldestOne()) break;
+      id = pickNextId(pool);
+      if (!id) break;
     }
 
     const placeholder = '<div id="ezoic-pub-ad-placeholder-' + id + '"></div>';
-    const selector = '.ezoic-ad-topic[data-ezoic-slot="' + slot + '"][data-ezoic-id="' + id + '"]';
     const html = makeWrapperLike(
       $target,
       wrapperClass,
@@ -172,12 +171,8 @@ function injectBetweenIncremental($items, pool, interval, wrapperClass) {
 
     injectedSlots.add(slot);
     usedIds.add(id);
-    adsFIFO.push({ slot: slot, id: id, selector: selector });
-
+    adsFIFO.push({ type: 'between', slot: slot, id: id });
     newIds.push(id);
-
-    // Enforce capacity immediately (prevents runaway and keeps ads near viewport)
-    fifoEvictIfNeeded(pool.length);
   }
 
   return newIds;
@@ -197,11 +192,14 @@ function injectMessageIncremental($posts, pool, interval) {
     const $target = $posts.eq(index);
     if (!$target.length) continue;
 
-    const id = pickNextId(pool);
-    if (!id) break;
+    let id = pickNextId(pool);
+    if (!id) {
+      if (!evictOldestOne()) break;
+      id = pickNextId(pool);
+      if (!id) break;
+    }
 
     const inner = '<div class="content"><div id="ezoic-pub-ad-placeholder-' + id + '"></div></div>';
-    const selector = '.ezoic-ad-post[data-ezoic-slot="' + slot + '"][data-ezoic-id="' + id + '"]';
     const html = makeWrapperLike(
       $target,
       'post ezoic-ad-post',
@@ -213,11 +211,8 @@ function injectMessageIncremental($posts, pool, interval) {
 
     injectedSlots.add(slot);
     usedIds.add(id);
-    adsFIFO.push({ slot: slot, id: id, selector: selector });
-
+    adsFIFO.push({ type: 'message', slot: slot, id: id });
     newIds.push(id);
-
-    fifoEvictIfNeeded(pool.length);
   }
 
   return newIds;