nodebb-plugin-ezoic-infinite 1.7.19 → 1.7.21

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.19",
+  "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,10 +1,10 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v25
+ * NodeBB Ezoic Infinite Ads — client.js v28
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
  * v18  Ancrage stable par data-pid/data-index au lieu d'ordinalMap fragile.
- *      Suppression du recyclage de wraps (moveWrapAfter). Cleanup complet navigation.
+ *      Suppression du recyclage de wraps. Cleanup complet navigation.
  *
  * v19  Intervalle global basé sur l'ordinal absolu (data-index) et non sur
  *      la position dans le batch courant.
@@ -16,17 +16,23 @@
  *      Fix unobserve(null) → corruption IO → pubads error au scroll retour.
  *      Fix TCF locator : MutationObserver recrée l'iframe si ajaxify la retire.
  *
- * v25  Base v20.1 avec :
- *      • Fix scroll-up / virtualisation NodeBB :
- *        – pruneOrphans : PRUNE_STABLE_MS = 45 s, isFilled guard en premier.
+ * v25  Table KIND unifiée avec baseTag + ordinalAttr.
+ *      Fix scroll-up / virtualisation NodeBB :
  *        – decluster : isFilled en premier, A_CREATED grace period (FILL_GRACE_MS).
- *      • Recyclage d'id (pool épuisé en infinite scroll) :
- *        – pickRecyclableWrap() : sélectionne le wrap vide le plus loin au-dessus
- *          du viewport (seuil -6 × vh), jamais pour ezoic-ad-message.
- *        – moveWrapAfter() : déplace le wrap vers sa nouvelle ancre.
- *        – scrollDir tracking pour n'autoriser le recyclage qu'en scroll down.
- *      • Table KIND unifiée avec baseTag + ordinalAttr + recyclable flag.
- *      • ordinal() : utilise KIND[klass].ordinalAttr, fallback positionnel propre.
+ *      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.
+ *      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,10 +46,7 @@
   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 RECYCLE_THRESHOLD = 6;       // nb de viewports au-dessus du seuil de recyclage
   const MAX_INSERTS_RUN   = 6;
   const MAX_INFLIGHT      = 4;
   const SHOW_THROTTLE_MS  = 900;
@@ -69,13 +72,11 @@
    *              data-pid   posts / data-index topics / data-cid catégories
    * ordinalAttr: attribut 0-based pour calcul de l'intervalle
    *              null → fallback positionnel (catégories)
-   * recyclable : autoriser le recyclage d'id quand le pool est épuisé
-   *              false pour ezoic-ad-message (sauts visuels indésirables)
    */
   const KIND = {
-    'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index', recyclable: false },
-    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-index', ordinalAttr: 'data-index', recyclable: true  },
-    'ezoic-ad-categories': { sel: SEL.category, baseTag: 'li', anchorAttr: 'data-cid',   ordinalAttr: null,         recyclable: true  },
+    'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   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         },
   };
 
   // ── État ───────────────────────────────────────────────────────────────────
@@ -98,8 +99,6 @@
     burstDeadline: 0,
     burstCount:   0,
     lastBurstTs:  0,
-    scrollDir:    1,    // 1 = down, -1 = up
-    lastScrollY:  0,
   };
 
   let blockedUntil = 0;
@@ -223,49 +222,6 @@
     return null;
   }
 
-  // ── Recyclage d'id ─────────────────────────────────────────────────────────
-
-  /**
-   * Sélectionne le wrap vide le plus éloigné au-dessus du viewport.
-   * Conditions : kindClass.recyclable = true, scroll vers le bas,
-   * wrap vide (non filled), rect.bottom < -(RECYCLE_THRESHOLD × vh).
-   */
-  function pickRecyclableWrap(klass) {
-    if (!KIND[klass]?.recyclable) return null;
-    if (S.scrollDir < 0) return null;
-
-    const vh        = Math.max(300, window.innerHeight || 800);
-    const threshold = -(vh * RECYCLE_THRESHOLD);
-    let best = null, bestBottom = Infinity;
-
-    for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
-      if (!w.isConnected || isFilled(w)) continue;
-      try {
-        const rect = w.getBoundingClientRect();
-        if (rect.bottom < threshold && rect.bottom < bestBottom) {
-          bestBottom = rect.bottom;
-          best = w;
-        }
-      } catch (_) {}
-    }
-    return best;
-  }
-
-  /**
-   * Déplace un wrap recyclé vers sa nouvelle ancre el.
-   * Réinitialise A_ANCHOR, A_CREATED, supprime A_SHOWN.
-   */
-  function moveWrapAfter(el, wrap, newKey) {
-    try {
-      if (!el || !wrap?.isConnected) return null;
-      wrap.setAttribute(A_ANCHOR,  newKey);
-      wrap.setAttribute(A_CREATED, String(ts()));
-      wrap.removeAttribute(A_SHOWN);
-      mutate(() => el.insertAdjacentElement('afterend', wrap));
-      return wrap;
-    } catch (_) { return null; }
-  }
-
   // ── Wraps DOM ──────────────────────────────────────────────────────────────
 
   function makeWrap(id, klass, key) {
@@ -303,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 ──────────────────────────────────────────────────────────────
 
@@ -401,23 +307,11 @@
       const key = anchorKey(klass, el);
       if (findWrap(key)) continue;
 
-      // 1. Tentative pool normal
       const id = pickId(poolKey);
-      if (id) {
-        const w = insertAfter(el, id, klass, key);
-        if (w) { observePh(id); inserted++; }
-        continue;
-      }
+      if (!id) continue; // pool épuisé : tous les ids sont montés, on passe au suivant
 
-      // 2. Pool épuisé → tentative de recyclage
-      const recyclable = pickRecyclableWrap(klass);
-      if (recyclable) {
-        const rid = parseInt(recyclable.getAttribute(A_WRAPID), 10);
-        const w   = moveWrapAfter(el, recyclable, key);
-        if (w && Number.isFinite(rid)) { observePh(rid); inserted++; }
-      }
-      // Pool épuisé et pas de recyclage : on continue (items suivants peuvent
-      // avoir un wrap existant via findWrap, on ne break pas)
+      const w = insertAfter(el, id, klass, key);
+      if (w) { observePh(id); inserted++; }
     }
     return inserted;
   }
@@ -558,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;
     };
 
@@ -743,13 +635,6 @@
   function bindScroll() {
     let ticking = false;
     window.addEventListener('scroll', () => {
-      // Suivi direction du scroll (nécessaire pour le recyclage conditionnel)
-      try {
-        const y = window.scrollY || window.pageYOffset || 0;
-        const d = y - S.lastScrollY;
-        if (Math.abs(d) > 4) { S.scrollDir = d > 0 ? 1 : -1; S.lastScrollY = y; }
-      } catch (_) {}
-
       if (ticking) return;
       ticking = true;
       requestAnimationFrame(() => { ticking = false; requestBurst(); });
@@ -759,7 +644,6 @@
   // ── Boot ───────────────────────────────────────────────────────────────────
 
   S.pageKey = pageKey();
-  try { S.lastScrollY = window.scrollY || window.pageYOffset || 0; } catch (_) {}
   muteConsole();
   ensureTcfLocator();
   warmNetwork();