nodebb-plugin-ezoic-infinite 1.7.19 → 1.7.20

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.19",
+  "version": "1.7.20",
   "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 v26
  *
  * 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,17 @@
  *      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 :
+ * 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).
- *      • 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.
+ *      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.
  */
 (function () {
   'use strict';
@@ -43,7 +43,6 @@
   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 +68,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 +95,6 @@
     burstDeadline: 0,
     burstCount:   0,
     lastBurstTs:  0,
-    scrollDir:    1,    // 1 = down, -1 = up
-    lastScrollY:  0,
   };
 
   let blockedUntil = 0;
@@ -223,49 +218,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) {
@@ -401,23 +353,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é : on attend que pruneOrphans libère des ids
 
-      // 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;
   }
@@ -743,13 +683,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 +692,6 @@
   // ── Boot ───────────────────────────────────────────────────────────────────
 
   S.pageKey = pageKey();
-  try { S.lastScrollY = window.scrollY || window.pageYOffset || 0; } catch (_) {}
   muteConsole();
   ensureTcfLocator();
   warmNetwork();