npm - nodebb-plugin-ezoic-infinite - Versions diffs - 1.7.58 → 1.7.60 - Mend

nodebb-plugin-ezoic-infinite 1.7.58 → 1.7.60

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.

Files changed (4) hide show

package/library.js CHANGED Viewed

@@ -15,11 +15,9 @@ function normalizeExcludedGroups(value) {
   // NodeBB stocke les settings multi-valeurs comme string JSON "[\"group1\",\"group2\"]"
   const s = String(value).trim();
   if (s.startsWith('[')) {
-    try {
-      const parsed = JSON.parse(s);
-      if (Array.isArray(parsed)) return parsed.map(String).filter(Boolean);
-    } catch (_) {}
+    try { const parsed = JSON.parse(s); if (Array.isArray(parsed)) return parsed.map(String).filter(Boolean); } catch (_) {}
   }
+  // Fallback : séparation par virgule
   return s.split(',').map(v => v.trim()).filter(Boolean);
 }
@@ -30,15 +28,7 @@ function parseBool(v, def = false) {
   return s === '1' || s === 'true' || s === 'on' || s === 'yes';
 }
-// ── Cache groupes (fix #7 : getAllGroups sans cache sur page ACP) ───────────
-let _groupsCache   = null;
-let _groupsCacheAt = 0;
-const GROUPS_TTL   = 60_000; // 1 minute
 async function getAllGroups() {
-  const now = Date.now();
-  if (_groupsCache && (now - _groupsCacheAt) < GROUPS_TTL) return _groupsCache;
   let names = await db.getSortedSetRange('groups:createtime', 0, -1);
   if (!names || !names.length) {
     names = await db.getSortedSetRange('groups:visible:createtime', 0, -1);
@@ -47,9 +37,7 @@ async function getAllGroups() {
   const data = await groups.getGroupsData(filtered);
   const valid = data.filter(g => g && g.name);
   valid.sort((a, b) => String(a.name).localeCompare(String(b.name), undefined, { sensitivity: 'base' }));
-  _groupsCache   = valid;
-  _groupsCacheAt = now;
-  return _groupsCache;
+  return valid;
 }
 // ── Settings cache (30s TTL) ────────────────────────────────────────────────
@@ -100,10 +88,7 @@ ezstandalone.cmd = ezstandalone.cmd || [];
 // ── Hooks ──────────────────────────────────────────────────────────────────
 plugin.onSettingsSet = function (data) {
-  if (data && data.hash === SETTINGS_KEY) {
-    _settingsCache = null;
-    _groupsCache   = null; // invalider aussi le cache groupes
-  }
+  if (data && data.hash === SETTINGS_KEY) _settingsCache = null;
 };
 plugin.addAdminNavigation = async (header) => {
@@ -116,11 +101,11 @@ plugin.addAdminNavigation = async (header) => {
  * Injecte les scripts Ezoic dans le <head> via templateData.customHTML.
  *
  * NodeBB v4 / thème Harmony : header.tpl contient {{customHTML}} dans le <head>
- * (render.js : templateValues.customHTML = meta.config.customHTML).
- * Le hook filter:middleware.renderHeader reçoit templateData = headerFooterData.
- * On préfixe customHTML pour passer AVANT le customHTML admin tout en le préservant.
- *
- * Fix #3 : erreurs loggées côté serveur plutôt qu'avalées silencieusement.
+ * (render.js ligne 232 : templateValues.customHTML = meta.config.customHTML).
+ * Le hook filter:middleware.renderHeader reçoit templateData = headerFooterData
+ * et est rendu via req.app.renderAsync('header', hookReturn.templateData).
+ * On préfixe customHTML pour que nos scripts passent AVANT le customHTML admin,
+ * tout en préservant ce dernier.
  */
 plugin.injectEzoicHead = async (data) => {
   try {
@@ -128,18 +113,17 @@ plugin.injectEzoicHead = async (data) => {
     const uid      = data.req?.uid ?? 0;
     const excluded = await isUserExcluded(uid, settings.excludedGroups);
     if (!excluded) {
+      // Préfixer : nos scripts d'abord, puis le customHTML existant de l'admin
       data.templateData.customHTML = EZOIC_SCRIPTS + (data.templateData.customHTML || '');
     }
-  } catch (err) {
-    // Log l'erreur mais ne pas planter le rendu de la page
-    console.error('[ezoic-infinite] injectEzoicHead error:', err.message || err);
-  }
+  } catch (_) {}
   return data;
 };
 plugin.init = async ({ router, middleware }) => {
   async function render(req, res) {
-    const [settings, allGroups] = await Promise.all([getSettings(), getAllGroups()]);
+    const settings  = await getSettings();
+    const allGroups = await getAllGroups();
     res.render('admin/plugins/ezoic-infinite', {
       title: 'Ezoic Infinite Ads',
       ...settings,

package/package.json CHANGED Viewed

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.58",
+  "version": "1.7.60",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",

package/public/client.js CHANGED Viewed

@@ -1,51 +1,90 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v62
+ * NodeBB Ezoic Infinite Ads — client.js v36
  *
- * Historique
- * ──────────
- * v18  Ancrage stable par data-pid / data-index.
- * v20  Table KIND. IO fixe. Fix TCF locator.
- * v25  Fix scroll-up / virtualisation NodeBB.
- * v28  Wraps persistants pendant la session.
- * v35  S.poolsReady. muteConsole élargi.
- * v36  S.wrapByKey Map O(1). MutationObserver optimisé.
- * v38  ez.refresh() supprimé. Pool épuisé → break propre.
- * v40  Recyclage destroyPlaceholders+define+displayMore (délais 300ms).
- * v43  Seuil recyclage -1vh + unobserve avant déplacement.
- * v49  Fix normalizeExcludedGroups (JSON.parse tableau NodeBB).
- * v51  fetchConfig backoff 10s. IO recrée au resize. tcfObs dans S.
- * v52  pruneOrphansBetween supprimé (NodeBB virtualise aussi les topics).
- * v53  S.recycling garde double-recyclage. pickId early-exit. cleanup complet.
- * v54  ensureTcfLocator rappelé à chaque ajaxify.end.
- * v56  scheduleEmptyCheck / is-empty supprimés (collapse prématuré).
- * v62  is-empty réintroduit : collapse 60s après insertion du wrap (pas après
- *      showAds) si isFilled est toujours false. Évite les trous permanents.
- * v61  recycleAndMove : ne pas recycler un wrap rempli depuis moins de 30s.
- *      Empêche qu'une pub qui vient de charger soit déplacée immédiatement.
- * v59  CSS : min-height 90px sur ezoic-ad-between (anti-CLS AMP ads).
- * v58  tcfObs survit aux navigations : ne plus déconnecter dans cleanup().
- *      L'iframe __tcfapiLocator doit exister en permanence pour le CMP —
- *      la fenêtre entre cleanup() et ajaxify.end causait des erreurs
- *      "Cannot read properties of null (postMessage)" et disparition des pubs.
- * v57  Nettoyage prod : A_SHOWN/A_CREATED supprimés, normBool Set O(1),
- *      muteConsole étend aux erreurs CMP getTCData, code nettoyé.
+ * Historique des corrections majeures
+ * ────────────────────────────────────
+ * v18  Ancrage stable par data-pid / data-index au lieu d'ordinalMap fragile.
+ *
+ * v19  Intervalle global basé sur l'ordinal absolu (data-index), pas sur
+ *      la position dans le batch courant.
+ *
+ * v20  Table KIND : anchorAttr / ordinalAttr / baseTag par kindClass.
+ *      Fix fatal catégories : data-cid au lieu de data-index inexistant.
+ *      IO fixe (une instance, jamais recréée).
+ *      Fix TCF locator : MutationObserver recrée l'iframe si ajaxify la retire.
+ *
+ * v25  Fix scroll-up / virtualisation NodeBB : decluster + grace period.
+ *
+ * v26  Suppression définitive du recyclage d'id (causait réinjection en haut).
+ *
+ * v27  pruneOrphans supprimé (faux-orphelins sur virtualisation NodeBB posts).
+ *
+ * v28  decluster supprimé. Wraps persistants pendant la session.
+ *
+ * v32  Retour anchorAttr = data-index pour ezoic-ad-between.
+ *      data-tid peut être absent → clés invalides → wraps empilés.
+ *      pruneOrphansBetween réactivé uniquement pour topics de catégorie :
+ *        – NodeBB NE virtualise PAS les topics dans une liste de catégorie,
+ *          les ancres (data-index) restent en DOM → prune safe et nécessaire
+ *          pour éviter l'empilement après scroll long.
+ *        – Toujours désactivé pour les posts : NodeBB virtualise les posts
+ *          hors-viewport → faux-orphelins → bug réinjection en haut.
+ *
+ * v34  moveDistantWrap — voir v38.
+ *
+ * v50  Suppression de bindLoginCheck() : NodeBB fait un rechargement complet
+ *      après login — filter:middleware.renderHeader re-évalue l'exclusion au
+ *      rechargement. Redondant depuis le fix normalizeExcludedGroups (v49).
+ *
+ * v43  Seuil de recyclage abaissé à -vh + unobserve avant recyclage.
+ *
+ * v42  Seuil -(IO_MARGIN + vh) (trop strict, peu de wraps éligibles).
+ *
+ * v41  Seuil -1vh (trop permissif sur mobile, ignorait IO_MARGIN).
+ *
+ * v40  Recyclage slots via destroyPlaceholders+define+displayMore avec délais.
+ *      Séquence : destroy → 300ms → define → 300ms → displayMore.
+ *      Testé manuellement : fonctionne. displayMore = API Ezoic infinite scroll.
+ *
+ * v38  Pool épuisé = fin de quota Ezoic par page-view. ez.refresh() interdit
+ *      sur la même page que ez.enable() — supprimé. moveDistantWrap supprimé :
+ *      déplacer un wrap "already defined" ne re-sert aucune pub. Pool épuisé →
+ *      break propre dans injectBetween. muteConsole : ajout warnings refresh.
+ *
+ * v36  Optimisations chemin critique (scroll → injectBetween) :
+ *      – S.wrapByKey Map<anchorKey,wrap> : findWrap() passe de querySelector
+ *        sur tout le doc à un lookup O(1). Mis à jour dans insertAfter,
+ *        dropWrap et cleanup.
+ *      – wrapIsLive allégé : pour les voisins immédiats on vérifie les
+ *        attributs du nœud lui-même sans querySelector global.
+ *      – MutationObserver : matches() vérifié avant querySelector() pour
+ *        court-circuiter les sous-arbres entiers ajoutés par NodeBB.
+ *
+ * v35  Revue complète prod-ready :
+ *      – initPools protégé contre ré-initialisation inutile (S.poolsReady).
+ *      – muteConsole élargit à "No valid placeholders for loadMore".
+ *      – Commentaires et historique nettoyés.
  */
 (function nbbEzoicInfinite() {
   'use strict';
   // ── Constantes ─────────────────────────────────────────────────────────────
-  const WRAP_CLASS = 'nodebb-ezoic-wrap';
-  const PH_PREFIX  = 'ezoic-pub-ad-placeholder-';
-  const A_ANCHOR   = 'data-ezoic-anchor';  // "kindClass:stableId"
-  const A_WRAPID   = 'data-ezoic-wrapid';  // id Ezoic
-  const EMPTY_CHECK_MS    = 60_000;
-  const MAX_INSERTS_RUN   = 6;    // insertions max par appel runCore
-  const MAX_INFLIGHT      = 4;    // showAds() simultanés max
-  const SHOW_THROTTLE_MS  = 900;  // anti-spam showAds() par id
-  const BURST_COOLDOWN_MS = 200;  // délai min entre deux bursts
+  const WRAP_CLASS  = 'nodebb-ezoic-wrap';
+  const PH_PREFIX   = 'ezoic-pub-ad-placeholder-';
+  const A_ANCHOR    = 'data-ezoic-anchor';   // "kindClass:stableId"
+  const A_WRAPID    = 'data-ezoic-wrapid';   // id Ezoic
+  const A_CREATED   = 'data-ezoic-created';  // timestamp création ms
+  const A_SHOWN     = 'data-ezoic-shown';    // timestamp dernier showAds ms
+  const EMPTY_CHECK_MS    = 20_000;  // délai avant collapse d'un wrap vide post-show
+  const MIN_PRUNE_AGE_MS  = 8_000;   // délai de grâce avant pruning (stabilisation DOM)
+  const MAX_INSERTS_RUN   = 6;       // max insertions par appel runCore
+  const MAX_INFLIGHT      = 4;       // max showAds() simultanés
+  const SHOW_THROTTLE_MS  = 900;     // anti-spam showAds() par id
+  const BURST_COOLDOWN_MS = 200;     // délai min entre deux déclenchements de burst
+  // Marges IO larges et fixes — observer créé une seule fois au boot
   const IO_MARGIN_DESKTOP = '2500px 0px 2500px 0px';
   const IO_MARGIN_MOBILE  = '3500px 0px 3500px 0px';
@@ -57,10 +96,13 @@
   /**
    * Table KIND — source de vérité par kindClass.
-   * sel         sélecteur CSS des éléments cibles
+   *
+   * sel         sélecteur CSS complet des éléments cibles
    * baseTag     préfixe tag pour querySelector d'ancre
+   *             (vide pour posts : le sélecteur commence par '[')
    * anchorAttr  attribut DOM stable → clé unique du wrap
-   * ordinalAttr attribut 0-based pour le calcul de l'intervalle (null = fallback positionnel)
+   * ordinalAttr attribut 0-based pour le calcul de l'intervalle
+   *             null → fallback positionnel (catégories)
    */
   const KIND = {
     'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index' },
@@ -71,39 +113,34 @@
   // ── État global ────────────────────────────────────────────────────────────
   const S = {
-    pageKey:       null,
-    cfg:           null,
-    poolsReady:    false,
-    pools:         { topics: [], posts: [], categories: [] },
-    cursors:       { topics: 0,  posts: 0,  categories: 0  },
-    mountedIds:    new Set(),
-    lastShow:      new Map(),
-    io:            null,
-    domObs:        null,
-    tcfObs:        null,
-    mutGuard:      0,
-    inflight:      0,
-    pending:       [],
-    pendingSet:    new Set(),
-    wrapByKey:     new Map(),  // anchorKey → wrap DOM node
-    recycling:     new Set(),  // ids en cours de séquence destroy→define→displayMore
-    runQueued:     false,
-    burstActive:   false,
+    pageKey:      null,
+    cfg:          null,
+    poolsReady:   false,
+    pools:        { topics: [], posts: [], categories: [] },
+    cursors:      { topics: 0,  posts: 0,  categories: 0  },
+    mountedIds:   new Set(),
+    lastShow:     new Map(),
+    io:           null,
+    domObs:       null,
+    mutGuard:     0,     // >0 : mutations DOM en cours (MutationObserver ignoré)
+    inflight:     0,     // showAds() en cours
+    pending:      [],    // ids en attente de slot inflight
+    pendingSet:   new Set(),
+    wrapByKey:    new Map(),  // anchorKey → wrap DOM node
+    runQueued:    false,
+    burstActive:  false,
     burstDeadline: 0,
-    burstCount:    0,
-    lastBurstTs:   0,
+    burstCount:   0,
+    lastBurstTs:  0,
   };
-  let blockedUntil   = 0;
-  let _cfgErrorUntil = 0;
-  let _ioMobile      = null;
+  let blockedUntil = 0;
-  const ts         = () => Date.now();
-  const isBlocked  = () => ts() < blockedUntil;
-  const isMobile   = () => { try { return window.innerWidth < 768; } catch (_) { return false; } };
-  const _BOOL_TRUE = new Set([true, 'true', 1, '1', 'on']);
-  const normBool   = v => _BOOL_TRUE.has(v);
-  const isFilled   = n => !!(n?.querySelector('iframe, ins, img, video, [data-google-container-id]'));
+  const ts        = () => Date.now();
+  const isBlocked = () => ts() < blockedUntil;
+  const isMobile  = () => { try { return window.innerWidth < 768; } catch (_) { return false; } };
+  const normBool  = v  => v === true || v === 'true' || v === 1 || v === '1' || v === 'on';
+  const isFilled  = n  => !!(n?.querySelector?.('iframe, ins, img, video, [data-google-container-id]'));
   function mutate(fn) {
     S.mutGuard++;
@@ -114,12 +151,10 @@
   async function fetchConfig() {
     if (S.cfg) return S.cfg;
-    if (Date.now() < _cfgErrorUntil) return null;
     try {
       const r = await fetch('/api/plugins/ezoic-infinite/config', { credentials: 'same-origin' });
-      if (r.ok) { S.cfg = await r.json(); }
-      else       { _cfgErrorUntil = Date.now() + 10_000; }
-    } catch (_)  { _cfgErrorUntil = Date.now() + 10_000; }
+      if (r.ok) S.cfg = await r.json();
+    } catch (_) {}
     return S.cfg;
   }
@@ -179,37 +214,53 @@
   // ── Wraps — détection ──────────────────────────────────────────────────────
+  /**
+   * Vérifie qu'un wrap a encore son ancre dans le DOM.
+   * Utilisé par adjacentWrap pour ignorer les wraps orphelins.
+   */
   function wrapIsLive(wrap) {
     if (!wrap?.classList?.contains(WRAP_CLASS)) return false;
     const key = wrap.getAttribute(A_ANCHOR);
     if (!key) return false;
+    // Lookup O(1) dans le registre — vrai si le wrap EST encore dans le registre
+    // et connecté au DOM (le registre est tenu à jour par insertAfter/dropWrap).
     if (S.wrapByKey.get(key) === wrap) return wrap.isConnected;
+    // Fallback : registre pas encore à jour ou wrap non enregistré.
     const colonIdx = key.indexOf(':');
     const klass    = key.slice(0, colonIdx);
     const anchorId = key.slice(colonIdx + 1);
     const cfg = KIND[klass];
     if (!cfg) return false;
+    // Optimisation : si l'ancre est un frère direct du wrap, pas besoin
+    // de querySelector global — on cherche parmi les voisins immédiats.
     const parent = wrap.parentElement;
     if (parent) {
       for (const sib of parent.children) {
         if (sib === wrap) continue;
         try {
-          if (sib.matches(`${cfg.baseTag || ''}[${cfg.anchorAttr}="${anchorId}"]`))
+          if (sib.matches(`${cfg.baseTag || ''}[${cfg.anchorAttr}="${anchorId}"]`)) {
             return sib.isConnected;
+          }
         } catch (_) {}
       }
     }
+    // Dernier recours : querySelector global
     try {
       const found = document.querySelector(`${cfg.sel}[${cfg.anchorAttr}="${anchorId}"]`);
       return !!(found?.isConnected);
     } catch (_) { return false; }
   }
-  const adjacentWrap = el =>
-    wrapIsLive(el.nextElementSibling) || wrapIsLive(el.previousElementSibling);
+  function adjacentWrap(el) {
+    return wrapIsLive(el.nextElementSibling) || wrapIsLive(el.previousElementSibling);
+  }
   // ── Ancres stables ─────────────────────────────────────────────────────────
+  /**
+   * Retourne la valeur de l'attribut stable pour cet élément,
+   * ou un fallback positionnel si l'attribut est absent.
+   */
   function stableId(klass, el) {
     const attr = KIND[klass]?.anchorAttr;
     if (attr) {
@@ -228,15 +279,18 @@
   function findWrap(key) {
     const w = S.wrapByKey.get(key);
-    return w?.isConnected ? w : null;
+    return (w?.isConnected) ? w : null;
   }
   // ── Pool ───────────────────────────────────────────────────────────────────
+  /**
+   * Retourne le prochain id disponible dans le pool (round-robin),
+   * ou null si tous les ids sont montés.
+   */
   function pickId(poolKey) {
     const pool = S.pools[poolKey];
     if (!pool.length) return null;
-    if (S.mountedIds.size >= pool.length && pool.every(id => S.mountedIds.has(id))) return null;
     for (let t = 0; t < pool.length; t++) {
       const i = S.cursors[poolKey] % pool.length;
       S.cursors[poolKey] = (S.cursors[poolKey] + 1) % pool.length;
@@ -248,7 +302,9 @@
   /**
    * Pool épuisé : recycle un wrap loin au-dessus du viewport.
-   * Séquence : destroy([id]) → 300ms → define([id]) → 300ms → displayMore([id])
+   * Séquence avec délais (destroyPlaceholders est asynchrone) :
+   *   destroy([id]) → 300ms → define([id]) → 300ms → displayMore([id])
+   * displayMore = API Ezoic prévue pour l'infinite scroll.
    * Priorité : wraps vides d'abord, remplis si nécessaire.
    */
   function recycleAndMove(klass, targetEl, newKey) {
@@ -257,7 +313,10 @@
         typeof ez?.define             !== 'function' ||
         typeof ez?.displayMore        !== 'function') return null;
-    const threshold = -3 * (window.innerHeight || 800);
+    const vh        = window.innerHeight || 800;
+    // Seuil : -1vh (hors viewport visible). On appelle unobserve(ph) juste
+    // après pour neutraliser l'IO — plus de showAds parasite possible.
+    const threshold = -vh;
     let bestEmpty = null, bestEmptyBottom = Infinity;
     let bestFilled = null, bestFilledBottom = Infinity;
@@ -277,13 +336,16 @@
     if (!best) return null;
     const id = parseInt(best.getAttribute(A_WRAPID), 10);
     if (!Number.isFinite(id)) return null;
-    if (S.recycling.has(id)) return null;
-    S.recycling.add(id);
     const oldKey = best.getAttribute(A_ANCHOR);
+    // Neutraliser l'IO sur ce wrap avant déplacement — évite un showAds
+    // parasite si le nœud était encore dans la zone IO_MARGIN.
     try { const ph = best.querySelector(`#${PH_PREFIX}${id}`); if (ph) S.io?.unobserve(ph); } catch (_) {}
     mutate(() => {
-      best.setAttribute(A_ANCHOR, newKey);
+      best.setAttribute(A_ANCHOR,  newKey);
+      best.setAttribute(A_CREATED, String(ts()));
+      best.setAttribute(A_SHOWN,   '0');
+      best.classList.remove('is-empty');
       const ph = best.querySelector(`#${PH_PREFIX}${id}`);
       if (ph) ph.innerHTML = '';
       targetEl.insertAdjacentElement('afterend', best);
@@ -291,14 +353,11 @@
     if (oldKey && S.wrapByKey.get(oldKey) === best) S.wrapByKey.delete(oldKey);
     S.wrapByKey.set(newKey, best);
+    // Délais requis : destroyPlaceholders est asynchrone en interne
     const doDestroy = () => { try { ez.destroyPlaceholders([id]); } catch (_) {} setTimeout(doDefine,  300); };
     const doDefine  = () => { try { ez.define([id]);              } catch (_) {} setTimeout(doDisplay, 300); };
-    const doDisplay = () => {
-      try { ez.displayMore([id]); } catch (_) {}
-      S.recycling.delete(id);
-      observePh(id);
-    };
-    try { typeof ez.cmd?.push === 'function' ? ez.cmd.push(doDestroy) : doDestroy(); } catch (_) {}
+    const doDisplay = () => { try { ez.displayMore([id]);         } catch (_) {} };
+    try { (typeof ez.cmd?.push === 'function') ? ez.cmd.push(doDestroy) : doDestroy(); } catch (_) {}
     return { id, wrap: best };
   }
@@ -308,8 +367,10 @@
   function makeWrap(id, klass, key) {
     const w = document.createElement('div');
     w.className = `${WRAP_CLASS} ${klass}`;
-    w.setAttribute(A_ANCHOR, key);
-    w.setAttribute(A_WRAPID, String(id));
+    w.setAttribute(A_ANCHOR,  key);
+    w.setAttribute(A_WRAPID,  String(id));
+    w.setAttribute(A_CREATED, String(ts()));
+    w.setAttribute(A_SHOWN,   '0');
     w.style.cssText = 'width:100%;display:block;';
     const ph = document.createElement('div');
     ph.id = `${PH_PREFIX}${id}`;
@@ -327,7 +388,6 @@
     mutate(() => el.insertAdjacentElement('afterend', w));
     S.mountedIds.add(id);
     S.wrapByKey.set(key, w);
-    scheduleEmptyCheck(id);
     return w;
   }
@@ -343,8 +403,44 @@
     } catch (_) {}
   }
+  // ── Prune (topics de catégorie uniquement) ────────────────────────────────
+  //
+  // Réactivé uniquement pour 'ezoic-ad-between' (liste de topics).
+  //
+  // Safe ici car NodeBB ne virtualise PAS les topics dans une catégorie :
+  //   les li[component="category/topic"] restent dans le DOM pendant toute
+  //   la session. Un wrap orphelin (ancre absente) signifie vraiment que le
+  //   topic a disparu. Sans ce nettoyage, les wraps s'accumulent en tête de
+  //   liste après un long scroll et bloquent les nouvelles injections.
+  //
+  // Toujours désactivé pour 'ezoic-ad-message' (posts de topic) :
+  //   NodeBB virtualise les posts hors-viewport — il les retire puis les
+  //   réinsère. pruneOrphans verrait des ancres temporairement absentes,
+  //   supprimerait les wraps, et provoquerait une réinjection en haut.
+  function pruneOrphansBetween() {
+    const klass = 'ezoic-ad-between';
+    const cfg   = KIND[klass];
+    document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
+      const created = parseInt(w.getAttribute(A_CREATED) || '0', 10);
+      if (ts() - created < MIN_PRUNE_AGE_MS) return; // grâce post-création
+      const key = w.getAttribute(A_ANCHOR) ?? '';
+      const sid = key.slice(klass.length + 1); // après "ezoic-ad-between:"
+      if (!sid) { mutate(() => dropWrap(w)); return; }
+      const anchorEl = document.querySelector(`${cfg.baseTag}[${cfg.anchorAttr}="${sid}"]`);
+      if (!anchorEl?.isConnected) mutate(() => dropWrap(w));
+    });
+  }
   // ── Injection ──────────────────────────────────────────────────────────────
+  /**
+   * Ordinal 0-based pour le calcul de l'intervalle d'injection.
+   * Utilise ordinalAttr si défini, sinon compte les frères dans le parent.
+   */
   function ordinal(klass, el) {
     const attr = KIND[klass]?.ordinalAttr;
     if (attr) {
@@ -363,14 +459,18 @@
   function injectBetween(klass, items, interval, showFirst, poolKey) {
     if (!items.length) return 0;
     let inserted = 0;
     for (const el of items) {
       if (inserted >= MAX_INSERTS_RUN) break;
       if (!el?.isConnected) continue;
       const ord = ordinal(klass, el);
       if (!(showFirst && ord === 0) && (ord + 1) % interval !== 0) continue;
       if (adjacentWrap(el)) continue;
       const key = anchorKey(klass, el);
       if (findWrap(key)) continue;
       const id = pickId(poolKey);
       if (id) {
         const w = insertAfter(el, id, klass, key);
@@ -387,10 +487,7 @@
   // ── IntersectionObserver & Show ────────────────────────────────────────────
   function getIO() {
-    const mobile = isMobile();
-    if (S.io && _ioMobile === mobile) return S.io;
-    if (S.io) { try { S.io.disconnect(); } catch (_) {} S.io = null; }
-    _ioMobile = mobile;
+    if (S.io) return S.io;
     try {
       S.io = new IntersectionObserver(entries => {
         for (const e of entries) {
@@ -399,7 +496,7 @@
           const id = parseInt(e.target.getAttribute('data-ezoic-id'), 10);
           if (Number.isFinite(id) && id > 0) enqueueShow(id);
         }
-      }, { root: null, rootMargin: mobile ? IO_MARGIN_MOBILE : IO_MARGIN_DESKTOP, threshold: 0 });
+      }, { root: null, rootMargin: isMobile() ? IO_MARGIN_MOBILE : IO_MARGIN_DESKTOP, threshold: 0 });
     } catch (_) { S.io = null; }
     return S.io;
   }
@@ -419,18 +516,6 @@
     startShow(id);
   }
-  function scheduleEmptyCheck(id) {
-    setTimeout(() => {
-      try {
-        const ph   = document.getElementById(`${PH_PREFIX}${id}`);
-        const wrap = ph?.closest?.(`.${WRAP_CLASS}`);
-        if (!wrap || !ph?.isConnected) return;
-        // Collapse uniquement si vraiment vide après 60s
-        if (!isFilled(wrap)) wrap.classList.add('is-empty');
-      } catch (_) {}
-    }, EMPTY_CHECK_MS);
-  }
   function drainQueue() {
     if (isBlocked()) return;
     while (S.inflight < MAX_INFLIGHT && S.pending.length) {
@@ -451,21 +536,24 @@
       drainQueue();
     };
     const timer = setTimeout(release, 7000);
     requestAnimationFrame(() => {
       try {
         if (isBlocked()) { clearTimeout(timer); return release(); }
         const ph = document.getElementById(`${PH_PREFIX}${id}`);
         if (!ph?.isConnected || isFilled(ph)) { clearTimeout(timer); return release(); }
         const t = ts();
         if (t - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) { clearTimeout(timer); return release(); }
         S.lastShow.set(id, t);
-        // Marquer le wrap avec le timestamp de fill pour bloquer le recyclage
-        // Si la pub charge après is-empty, retirer le collapse
-        try { document.getElementById(`${PH_PREFIX}${id}`)?.closest?.(`.${WRAP_CLASS}`)?.classList.remove('is-empty'); } catch (_) {}
+        try { ph.closest?.(`.${WRAP_CLASS}`)?.setAttribute(A_SHOWN, String(t)); } catch (_) {}
         window.ezstandalone = window.ezstandalone || {};
         const ez = window.ezstandalone;
         const doShow = () => {
           try { ez.showAds(id); } catch (_) {}
+          scheduleEmptyCheck(id, t);
           setTimeout(() => { clearTimeout(timer); release(); }, 700);
         };
         Array.isArray(ez.cmd) ? ez.cmd.push(doShow) : doShow();
@@ -473,10 +561,23 @@
     });
   }
+  function scheduleEmptyCheck(id, showTs) {
+    setTimeout(() => {
+      try {
+        const ph   = document.getElementById(`${PH_PREFIX}${id}`);
+        const wrap = ph?.closest?.(`.${WRAP_CLASS}`);
+        if (!wrap || !ph?.isConnected) return;
+        if (parseInt(wrap.getAttribute(A_SHOWN) || '0', 10) > showTs) return;
+        wrap.classList.toggle('is-empty', !isFilled(ph));
+      } catch (_) {}
+    }, EMPTY_CHECK_MS);
+  }
   // ── Patch Ezoic showAds ────────────────────────────────────────────────────
   //
-  // Intercepte ez.showAds() pour ignorer les appels pendant blockedUntil
-  // et filtrer les ids dont le placeholder n'est pas connecté au DOM.
+  // Intercepte ez.showAds() pour :
+  //   – ignorer les appels pendant blockedUntil
+  //   – filtrer les ids dont le placeholder n'est pas en DOM
   function patchShowAds() {
     const apply = () => {
@@ -512,19 +613,37 @@
   async function runCore() {
     if (isBlocked()) return 0;
     patchShowAds();
     const cfg = await fetchConfig();
     if (!cfg || cfg.excluded) return 0;
     initPools(cfg);
     const kind = getKind();
     if (kind === 'other') return 0;
     const exec = (klass, getItems, cfgEnable, cfgInterval, cfgShowFirst, poolKey) => {
       if (!normBool(cfgEnable)) return 0;
       const interval = Math.max(1, parseInt(cfgInterval, 10) || 3);
       return injectBetween(klass, getItems(), interval, normBool(cfgShowFirst), poolKey);
     };
-    if (kind === 'topic')          return exec('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');
-    return                                exec('ezoic-ad-categories', getCategories, cfg.enableCategoryAds, cfg.intervalCategories,   cfg.showFirstCategoryAd, 'categories');
+    if (kind === 'topic') return exec(
+      'ezoic-ad-message', getPosts,
+      cfg.enableMessageAds, cfg.messageIntervalPosts, cfg.showFirstMessageAd, 'posts'
+    );
+    if (kind === 'categoryTopics') {
+      pruneOrphansBetween();
+      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'
+    );
   }
   // ── Scheduler ──────────────────────────────────────────────────────────────
@@ -548,9 +667,11 @@
     S.lastBurstTs   = t;
     S.pageKey       = pageKey();
     S.burstDeadline = t + 2000;
     if (S.burstActive) return;
     S.burstActive = true;
     S.burstCount  = 0;
     const step = () => {
       if (pageKey() !== S.pageKey || isBlocked() || ts() > S.burstDeadline || S.burstCount >= 8) {
         S.burstActive = false; return;
@@ -569,27 +690,21 @@
   function cleanup() {
     blockedUntil = ts() + 1500;
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
-    S.cfg        = null;
-    _cfgErrorUntil = 0;
-    S.poolsReady = false;
-    S.pools      = { topics: [], posts: [], categories: [] };
-    S.cursors    = { topics: 0,  posts: 0,  categories: 0  };
+    S.cfg          = null;
+    S.poolsReady   = false;
+    S.pools        = { topics: [], posts: [], categories: [] };
+    S.cursors      = { topics: 0, posts: 0, categories: 0 };
     S.mountedIds.clear();
     S.lastShow.clear();
     S.wrapByKey.clear();
-    S.recycling.clear();
-    S.inflight    = 0;
-    S.pending     = [];
+    S.inflight     = 0;
+    S.pending      = [];
     S.pendingSet.clear();
-    S.burstActive = false;
-    S.runQueued   = false;
-    if (S.domObs) { S.domObs.disconnect(); S.domObs = null; }
-    // tcfObs intentionnellement préservé : l'iframe __tcfapiLocator doit
-    // exister en permanence — la déconnecter pendant la navigation cause
-    // des erreurs CMP postMessage et la disparition des pubs.
+    S.burstActive  = false;
+    S.runQueued    = false;
   }
-  // ── MutationObserver DOM ───────────────────────────────────────────────────
+  // ── MutationObserver ───────────────────────────────────────────────────────
   function ensureDomObserver() {
     if (S.domObs) return;
@@ -599,8 +714,9 @@
       for (const m of muts) {
         for (const n of m.addedNodes) {
           if (n.nodeType !== 1) continue;
-          if (allSel.some(s => { try { return n.matches(s);        } catch (_) { return false; } }) ||
-              allSel.some(s => { try { return !!n.querySelector(s); } catch (_) { return false; } })) {
+          // matches() d'abord (O(1)), querySelector() seulement si nécessaire
+          if (allSel.some(s => { try { return n.matches(s); } catch(_){return false;} }) ||
+              allSel.some(s => { try { return !!n.querySelector(s); } catch(_){return false;} })) {
             requestBurst(); return;
           }
         }
@@ -620,7 +736,6 @@
       'cannot call refresh on the same page',
       'no placeholders are currently defined in Refresh',
       'Debugger iframe already exists',
-      '[CMP] Error in custom getTCData',
       `with id ${PH_PREFIX}`,
     ];
     for (const m of ['log', 'info', 'warn', 'error']) {
@@ -643,9 +758,9 @@
         (document.body || document.documentElement).appendChild(f);
       };
       inject();
-      if (!S.tcfObs) {
-        S.tcfObs = new MutationObserver(inject);
-        S.tcfObs.observe(document.documentElement, { childList: true, subtree: true });
+      if (!window.__nbbTcfObs) {
+        window.__nbbTcfObs = new MutationObserver(inject);
+        window.__nbbTcfObs.observe(document.documentElement, { childList: true, subtree: true });
       }
     } catch (_) {}
   }
@@ -677,19 +792,22 @@
   function bindNodeBB() {
     const $ = window.jQuery;
     if (!$) return;
     $(window).off('.nbbEzoic');
     $(window).on('action:ajaxify.start.nbbEzoic', cleanup);
     $(window).on('action:ajaxify.end.nbbEzoic', () => {
       S.pageKey    = pageKey();
       blockedUntil = 0;
-      muteConsole(); ensureTcfLocator();
+      muteConsole(); ensureTcfLocator(); warmNetwork();
       patchShowAds(); getIO(); ensureDomObserver(); requestBurst();
     });
     const burstEvts = [
-      'action:ajaxify.contentLoaded', 'action:posts.loaded',    'action:topics.loaded',
-      'action:categories.loaded',     'action:category.loaded', 'action:topic.loaded',
+      'action:ajaxify.contentLoaded', 'action:posts.loaded', 'action:topics.loaded',
+      'action:categories.loaded', 'action:category.loaded', 'action:topic.loaded',
     ].map(e => `${e}.nbbEzoic`).join(' ');
     $(window).on(burstEvts, () => { if (!isBlocked()) requestBurst(); });
     try {
       require(['hooks'], hooks => {
         if (typeof hooks?.on !== 'function') return;
@@ -708,11 +826,6 @@
       ticking = true;
       requestAnimationFrame(() => { ticking = false; requestBurst(); });
     }, { passive: true });
-    let resizeTimer = 0;
-    window.addEventListener('resize', () => {
-      clearTimeout(resizeTimer);
-      resizeTimer = setTimeout(getIO, 500);
-    }, { passive: true });
   }
   // ── Boot ───────────────────────────────────────────────────────────────────

package/public/style.css CHANGED Viewed

@@ -1,5 +1,5 @@
 /*
- * NodeBB Ezoic Infinite Ads — style.css (v59)
+ * NodeBB Ezoic Infinite Ads — style.css (v20)
  */
 /* ── Wrapper ──────────────────────────────────────────────────────────────── */
@@ -56,26 +56,16 @@
   top: auto !important;
 }
-/* ── Réservation d'espace anti-CLS ───────────────────────────────────────── */
-/*
-  Réserve 90px avant que la pub charge (hauteur standard leaderboard).
-  Évite le saut de layout (CLS) quand AMP/Ezoic redimensionne l'iframe.
-*/
-.nodebb-ezoic-wrap.ezoic-ad-between {
-  min-height: 90px;
-}
 /* ── État vide ────────────────────────────────────────────────────────────── */
 /*
-  Ajouté 60s après showAds si aucun fill détecté (délai généreux pour CMP/enchères).
-  Collapse à 0 : évite la ligne de quelques pixels visible quand Ezoic
-  injecte un conteneur vide mais ne sert pas de pub.
+  Ajouté 20s après showAds si aucun fill détecté.
+  Collapse à 1px (pas 0) : reste observable par l'IO si le fill arrive tard.
 */
 .nodebb-ezoic-wrap.is-empty {
   display: block !important;
-  height: 0 !important;
-  min-height: 0 !important;
-  max-height: 0 !important;
+  height: 1px !important;
+  min-height: 1px !important;
+  max-height: 1px !important;
   margin: 0 !important;
   padding: 0 !important;
   overflow: hidden !important;