nodebb-plugin-ezoic-infinite 1.7.23 → 1.7.25

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.23",
+  "version": "1.7.25",
   "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 v31
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -30,6 +30,21 @@
  * v28  decluster supprimé. pruneOrphans supprimé (v27). Wraps persistants sur session.
  *
  * v29  Fix ancrage topics : data-index → data-tid.
+ *
+ * v31  pruneOrphans réactivé UNIQUEMENT pour ezoic-ad-between (topics de catégorie).
+ *      NodeBB NE virtualise PAS les topics dans une liste de catégorie — les ancres
+ *      restent dans le DOM entre les scrolls. pruneOrphans est donc safe ici et
+ *      c'est lui qui empêche l'empilement des pubs en haut après un scroll long.
+ *      Pour ezoic-ad-message (posts de topic), pruneOrphans reste désactivé car
+ *      NodeBB virtualise les posts hors-viewport → faux-orphelins → bug réinjection.
+ *
+ * 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 +199,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 +257,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) {
@@ -267,15 +326,45 @@
     } catch (_) {}
   }
 
-  // ── Prune : désactivé ─────────────────────────────────────────────────────
+  // ── Prune (topics de catégorie uniquement) ────────────────────────────────
+  //
+  // pruneOrphans est réactivé UNIQUEMENT pour 'ezoic-ad-between'.
   //
-  // pruneOrphans() a été supprimé car il causait le bug "pubs en haut".
-  // NodeBB virtualise les posts hors viewport → les ancres disparaissent du DOM
-  // temporairement → pruneOrphans supprimait les wraps → scroll retour → les
-  // ancres revenaient → injectBetween réinjectait tout en haut.
+  // Pourquoi safe pour les topics ?
+  //   NodeBB ne virtualise PAS la liste des topics dans une catégorie.
+  //   Les <li component="category/topic"> restent dans le DOM pendant toute
+  //   la session. Leurs ancres (data-tid) sont donc stables — un wrap orphelin
+  //   signifie vraiment que le topic a été retiré (navigation, filtre, etc.).
   //
-  // Les wraps ne sont supprimés que par cleanup() à chaque navigation ajaxify.
-  // decluster() et pruneOrphans() sont désactivés — voir v28.
+  // Pourquoi désactivé pour les posts ?
+  //   NodeBB virtualise les posts hors-viewport : il retire les <li> du DOM
+  //   puis les réinsère au scroll retour. pruneOrphans verait des ancres
+  //   absentes → supprimerait les wraps → réinjection en haut au scroll retour.
+  //
+  // MIN_PRUNE_AGE_MS : délai de grâce après création (stabilisation du DOM).
+
+  const MIN_PRUNE_AGE_MS = 8_000;
+
+  function pruneOrphansBetween() {
+    const klass = 'ezoic-ad-between';
+    const cfg   = KIND[klass];
+
+    document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
+      // Délai de grâce : ne pas pruner un wrap trop récent
+      const created = parseInt(w.getAttribute(A_CREATED) || '0', 10);
+      if (ts() - created < MIN_PRUNE_AGE_MS) return;
+
+      const key = w.getAttribute(A_ANCHOR) ?? '';
+      const sid = key.slice(klass.length + 1); // partie après "ezoic-ad-between:"
+      if (!sid) { mutate(() => dropWrap(w)); return; }
+
+      // Chercher l'ancre par data-tid
+      const anchorEl = document.querySelector(`${cfg.baseTag}[${cfg.anchorAttr}="${sid}"]`);
+      if (!anchorEl || !anchorEl.isConnected) {
+        mutate(() => dropWrap(w));
+      }
+    });
+  }
 
 
   // ── Injection ──────────────────────────────────────────────────────────────
@@ -315,8 +404,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++; }
@@ -468,10 +557,13 @@
       'ezoic-ad-message', getPosts,
       cfg.enableMessageAds, cfg.messageIntervalPosts, cfg.showFirstMessageAd, 'posts'
     );
-    if (kind === 'categoryTopics') return exec(
-      'ezoic-ad-between', getTopics,
-      cfg.enableBetweenAds, cfg.intervalPosts, cfg.showFirstTopicAd, 'topics'
-    );
+    if (kind === 'categoryTopics') {
+      pruneOrphansBetween(); // nettoie les wraps dont le topic a disparu du DOM
+      return exec(
+        'ezoic-ad-between', getTopics,
+        cfg.enableBetweenAds, cfg.intervalPosts, cfg.showFirstTopicAd, 'topics'
+      );
+    }
     return exec(
       'ezoic-ad-categories', getCategories,
       cfg.enableCategoryAds, cfg.intervalCategories, cfg.showFirstCategoryAd, 'categories'