nodebb-plugin-ezoic-infinite 1.7.24 → 1.7.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.24",
+  "version": "1.7.26",
   "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 v30
+ * NodeBB Ezoic Infinite Ads — client.js v32
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -31,6 +31,18 @@
  *
  * v29  Fix ancrage topics : data-index → data-tid.
  *
+ * v32  Retour anchorAttr = data-index pour ezoic-ad-between (comme v20.1).
+ *      data-tid peut être absent sur li[component="category/topic"] → clés
+ *      invalides → wraps empilés sur pos0. data-index est toujours présent.
+ *      pruneOrphansBetween cherche li[data-index="N"] — cohérent avec v20.1.
+ *
+ * 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
@@ -91,7 +103,7 @@
    */
   const KIND = {
     'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index' },
-    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-tid',   ordinalAttr: 'data-index' },
+    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-index', ordinalAttr: 'data-index' },
     'ezoic-ad-categories': { sel: SEL.category, baseTag: 'li', anchorAttr: 'data-cid',   ordinalAttr: null         },
   };
 
@@ -319,15 +331,45 @@
     } catch (_) {}
   }
 
-  // ── Prune : désactivé ─────────────────────────────────────────────────────
+  // ── Prune (topics de catégorie uniquement) ────────────────────────────────
+  //
+  // pruneOrphans est réactivé UNIQUEMENT pour 'ezoic-ad-between'.
+  //
+  // 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.).
   //
-  // 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 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.
   //
-  // Les wraps ne sont supprimés que par cleanup() à chaque navigation ajaxify.
-  // decluster() et pruneOrphans() sont désactivés — voir v28.
+  // 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 ──────────────────────────────────────────────────────────────
@@ -520,10 +562,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'