nodebb-plugin-ezoic-infinite 1.7.20 → 1.7.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.20",
+  "version": "1.7.21",
   "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 v26
+ * NodeBB Ezoic Infinite Ads — client.js v28
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -18,15 +18,21 @@
  *
  * v25  Table KIND unifiée avec baseTag + ordinalAttr.
  *      Fix scroll-up / virtualisation NodeBB :
- *        – pruneOrphans : PRUNE_STABLE_MS = 45 s, isFilled guard en premier.
  *        – decluster : isFilled en premier, A_CREATED grace period (FILL_GRACE_MS).
  *      Tentative recyclage d'id (v25) → cause exactement le même bug (wraps
  *      déplacés laissent les positions originales libres → réinjection en haut).
  *
  * v26  Suppression définitive du recyclage d'id.
- *      Pool épuisé = on attend que pruneOrphans libère des ids (> 45 s hors DOM).
- *      Suppression de scrollDir, pickRecyclableWrap, moveWrapAfter.
- *      KIND simplifié : retrait du flag recyclable inutile.
+ *      KIND simplifié.
+ *
+ * v27  pruneOrphans supprimé (faux-orphelins sur virtualisation NodeBB).
+ *
+ * v28  decluster supprimé.
+ *      Analyse : decluster appelait dropWrap() qui faisait S.mountedIds.delete(id).
+ *      Un wrap vide adjacent à un autre wrap → supprimé → id libéré → réinjecté
+ *      en haut au prochain scroll. Exactement le bug observé.
+ *      Les deux mécanismes de nettoyage actif (pruneOrphans + decluster) sont
+ *      maintenant retirés. Seul cleanup() à la navigation peut supprimer des wraps.
  */
 (function () {
   'use strict';
@@ -40,8 +46,6 @@
   const A_CREATED   = 'data-ezoic-created';  // timestamp création ms
   const A_SHOWN     = 'data-ezoic-shown';    // timestamp dernier showAds ms
 
-  const PRUNE_STABLE_MS   = 45_000;  // délai avant pruning (évite faux-orphelins scroll-up)
-  const FILL_GRACE_MS     = 25_000;  // fenêtre fill async Ezoic (SSP auction)
   const EMPTY_CHECK_MS    = 20_000;  // délai collapse wrap vide post-show
   const MAX_INSERTS_RUN   = 6;
   const MAX_INFLIGHT      = 4;
@@ -255,66 +259,16 @@
     } catch (_) {}
   }
 
-  // ── Prune ──────────────────────────────────────────────────────────────────
+  // ── Prune : désactivé ─────────────────────────────────────────────────────
+  //
+  // 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.
+  //
+  // Les wraps ne sont supprimés que par cleanup() à chaque navigation ajaxify.
+  // decluster() et pruneOrphans() sont désactivés — voir v28.
 
-  /**
-   * Supprime les wraps VIDES dont l'ancre a disparu du DOM.
-   *
-   * isFilled en premier : un wrap rempli n'est JAMAIS supprimé.
-   * PRUNE_STABLE_MS (45 s) : pendant cette fenêtre une ancre absente est
-   * considérée comme virtualisée par NodeBB (scroll up), pas comme orphelin réel.
-   */
-  function pruneOrphans(klass) {
-    const meta = KIND[klass];
-    if (!meta) return;
-
-    for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
-      if (isFilled(w)) continue;
-      if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < PRUNE_STABLE_MS) continue;
-
-      const key = w.getAttribute(A_ANCHOR) ?? '';
-      const sid = key.slice(klass.length + 1);
-      if (!sid) { mutate(() => dropWrap(w)); continue; }
-
-      const sel      = `${meta.baseTag}[${meta.anchorAttr}="${sid.replace(/"/g, '\\"')}"]`;
-      const anchorEl = document.querySelector(sel);
-      if (!anchorEl || !anchorEl.isConnected) mutate(() => dropWrap(w));
-    }
-  }
-
-  // ── Decluster ──────────────────────────────────────────────────────────────
-
-  /**
-   * Deux wraps adjacents → supprimer le courant s'il est vide et hors grâce.
-   * Guards dans l'ordre :
-   *   1. isFilled(w)           → jamais toucher un wrap rempli
-   *   2. A_CREATED < FILL_GRACE → wrap trop récent (pas encore showAds'd)
-   *   3. A_SHOWN grace          → fill en cours
-   *   4. isFilled(prev)         → voisin rempli, intouchable → break
-   *   5. A_CREATED prev grace   → voisin trop récent → break
-   *   6. A_SHOWN prev grace     → break
-   *   → les deux vides et hors grâce : supprimer le courant
-   */
-  function decluster(klass) {
-    for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
-      if (isFilled(w)) continue;
-      if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < FILL_GRACE_MS) continue;
-      const wShown = parseInt(w.getAttribute(A_SHOWN) || '0', 10);
-      if (wShown && ts() - wShown < FILL_GRACE_MS) continue;
-
-      let prev = w.previousElementSibling, steps = 0;
-      while (prev && steps++ < 3) {
-        if (!prev.classList?.contains(WRAP_CLASS)) { prev = prev.previousElementSibling; continue; }
-        if (isFilled(prev)) break;
-        if (ts() - parseInt(prev.getAttribute(A_CREATED) || '0', 10) < FILL_GRACE_MS) break;
-        const pShown = parseInt(prev.getAttribute(A_SHOWN) || '0', 10);
-        if (pShown && ts() - pShown < FILL_GRACE_MS) break;
-
-        mutate(() => dropWrap(w));
-        break;
-      }
-    }
-  }
 
   // ── Injection ──────────────────────────────────────────────────────────────
 
@@ -354,7 +308,7 @@
       if (findWrap(key)) continue;
 
       const id = pickId(poolKey);
-      if (!id) continue; // pool épuisé : on attend que pruneOrphans libère des ids
+      if (!id) continue; // pool épuisé : tous les ids sont montés, on passe au suivant
 
       const w = insertAfter(el, id, klass, key);
       if (w) { observePh(id); inserted++; }
@@ -498,9 +452,7 @@
     const exec = (klass, getItems, cfgEnable, cfgInterval, cfgShowFirst, poolKey) => {
       if (!normBool(cfgEnable)) return 0;
       const interval = Math.max(1, parseInt(cfgInterval, 10) || 3);
-      pruneOrphans(klass);
       const n = injectBetween(klass, getItems(), interval, normBool(cfgShowFirst), poolKey);
-      if (n) decluster(klass);
       return n;
     };