nodebb-plugin-ezoic-infinite 1.7.23 → 1.7.24

package/package.json

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

package/public/client.js

@@ -1,5 +1,5 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v29
+ * NodeBB Ezoic Infinite Ads — client.js v30
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -30,6 +30,14 @@
  * v28  decluster supprimé. pruneOrphans supprimé (v27). Wraps persistants sur session.
  *
  * v29  Fix ancrage topics : data-index → data-tid.
+ *
+ * v30  Fix adjacentWrap : ne compte plus les wraps orphelins (ancre hors DOM).
+ *      Quand NodeBB virtualise et retire des topics du DOM, les wraps restent
+ *      en place (div dans le ul). adjacentWrap(el) retournait true sur ces
+ *      wraps orphelins → injection bloquée sur les topics suivants.
+ *      Fix : adjacentWrap vérifie que le wrap voisin a son ancre dans le DOM.
+ *      recycleOrphanId() : quand le pool est épuisé, recycle les wraps orphelins
+ *      non remplis qui sont loin au-dessus du viewport.
  *      data-index = position relative dans le batch NodeBB, pas un ID stable.
  *      Lors du scroll infini, les nouveaux batches démarrent à data-index=0,
  *      ce qui créait des collisions de clés d'ancrage → mauvaise déduplication
@@ -184,11 +192,23 @@
   const getTopics     = () => Array.from(document.querySelectorAll(SEL.topic));
   const getCategories = () => Array.from(document.querySelectorAll(SEL.category));
 
+  function wrapIsLive(wrap) {
+    if (!wrap?.classList?.contains(WRAP_CLASS)) return false;
+    const key = wrap.getAttribute(A_ANCHOR);
+    if (!key) return false;
+    const colonIdx = key.indexOf(':');
+    const klass = key.slice(0, colonIdx);
+    const anchorId = key.slice(colonIdx + 1);
+    const cfg = KIND[klass];
+    if (!cfg) return false;
+    try {
+      const found = document.querySelector(`${cfg.sel}[${cfg.anchorAttr}="${anchorId}"]`);
+      return !!(found?.isConnected);
+    } catch (_) { return false; }
+  }
+
   function adjacentWrap(el) {
-    return !!(
-      el.nextElementSibling?.classList?.contains(WRAP_CLASS) ||
-      el.previousElementSibling?.classList?.contains(WRAP_CLASS)
-    );
+    return wrapIsLive(el.nextElementSibling) || wrapIsLive(el.previousElementSibling);
   }
 
   // ── Ancres stables ─────────────────────────────────────────────────────────
@@ -230,6 +250,38 @@
     return null;
   }
 
+  function recycleOrphanId(klass) {
+    // Quand le pool est épuisé : cherche un wrap orphelin (ancre hors DOM, non rempli)
+    // loin au-dessus du viewport et libère son ID.
+    const vh = window.innerHeight || 800;
+    const threshold = -vh * 3;
+    let best = null, bestBottom = Infinity;
+    document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(wrap => {
+      if (wrap.getAttribute(A_CREATED) === null) return;
+      if (isFilled(wrap)) return;
+      const key = wrap.getAttribute(A_ANCHOR);
+      if (!key) return;
+      const colonIdx = key.indexOf(':');
+      const anchorId = key.slice(colonIdx + 1);
+      const cfg = KIND[klass];
+      if (!cfg) return;
+      try {
+        const found = document.querySelector(`${cfg.sel}[${cfg.anchorAttr}="${anchorId}"]`);
+        if (found?.isConnected) return; // ancre encore dans le DOM, pas orphelin
+      } catch (_) { return; }
+      try {
+        const rect = wrap.getBoundingClientRect();
+        if (rect.bottom > threshold) return;
+        if (rect.bottom < bestBottom) { bestBottom = rect.bottom; best = wrap; }
+      } catch (_) {}
+    });
+    if (!best) return null;
+    const id = parseInt(best.getAttribute(A_WRAPID), 10);
+    if (!Number.isFinite(id)) return null;
+    mutate(() => dropWrap(best));
+    return id;
+  }
+
   // ── Wraps DOM ──────────────────────────────────────────────────────────────
 
   function makeWrap(id, klass, key) {
@@ -315,8 +367,8 @@
       const key = anchorKey(klass, el);
       if (findWrap(key)) continue;
 
-      const id = pickId(poolKey);
-      if (!id) continue; // pool épuisé : tous les ids sont montés, on passe au suivant
+      let id = pickId(poolKey);
+      if (!id) { id = recycleOrphanId(klass); if (!id) continue; }
 
       const w = insertAfter(el, id, klass, key);
       if (w) { observePh(id); inserted++; }