npm - nodebb-plugin-ezoic-infinite - Versions diffs - 1.7.18 → 1.7.20 - Mend

nodebb-plugin-ezoic-infinite 1.7.18 → 1.7.20

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 (2) hide show

package/package.json +1 -1
package/public/client.js +164 -211

package/package.json CHANGED Viewed

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

package/public/client.js CHANGED Viewed

@@ -1,55 +1,54 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js (v20)
+ * NodeBB Ezoic Infinite Ads — client.js v26
  *
- * Correctifs critiques vs v19
- * ───────────────────────────
- * [BUG FATAL] Pubs catégories disparaissent au premier scroll sur mobile
- *   pruneOrphans cherchait l'ancre via `[data-index]`, mais les éléments
- *   `li[component="categories/category"]` ont `data-cid`, pas `data-index`.
- *   → anchorEl toujours null → suppression à chaque runCore() → disparition.
- *   Fix : table KIND_META qui mappe chaque kindClass vers son attribut d'ancre
- *   stable (data-pid pour posts, data-index pour topics, data-cid pour catégories).
+ * Historique des corrections majeures
+ * ────────────────────────────────────
+ * v18  Ancrage stable par data-pid/data-index au lieu d'ordinalMap fragile.
+ *      Suppression du recyclage de wraps. Cleanup complet navigation.
  *
- * [BUG] decluster : break trop large stoppait tout quand UN wrap était en grâce
- *   Fix : on skip uniquement le wrap courant, pas toute la boucle.
+ * v19  Intervalle global basé sur l'ordinal absolu (data-index) et non sur
+ *      la position dans le batch courant.
  *
- * [BUG] injectBetween : `break` sur pool épuisé empêchait d'observer les wraps
- *   existants sur les items suivants. Fix : `continue` au lieu de `break`.
+ * v20  Table KIND : anchorAttr/ordinalAttr/baseTag explicites par kindClass.
+ *      Fix fatal catégories : data-cid au lieu de data-index inexistant.
+ *      Fix posts : baseTag vide → ordinal fallback cassé → pubs jamais injectées.
+ *      IO fixe (une instance, jamais recréée). Burst cooldown 200 ms.
+ *      Fix unobserve(null) → corruption IO → pubads error au scroll retour.
+ *      Fix TCF locator : MutationObserver recrée l'iframe si ajaxify la retire.
  *
- * [PERF] IntersectionObserver recréé à chaque scroll boost → très coûteux mobile.
- *   Fix : marge large fixe par device, observer créé une seule fois.
+ * 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).
  *
- * [PERF] Burst cooldown 100ms trop court sur mobile → rafales en cascade.
- *   Fix : 200ms.
- *
- * Nettoyage
- * ─────────
- * - Suppression du scroll boost (complexité sans gain mesurable côté SPA statique)
- * - MAX_INFLIGHT unique desktop/mobile (inutile de différencier)
- * - getAnchorKey/getGlobalOrdinal fusionnés en helpers cohérents
- * - Commentaires internes allégés (code auto-documenté)
+ * 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';
   // ── 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 (number string)
-  const A_CREATED          = 'data-ezoic-created';  // timestamp création (ms)
-  const A_SHOWN            = 'data-ezoic-shown';    // timestamp dernier showAds (ms)
-  const MIN_PRUNE_AGE_MS   = 8_000;   // délai avant qu'un wrap puisse être purgé
-  const FILL_GRACE_MS      = 25_000;  // fenêtre post-showAds où l'on ne decluster pas
-  const EMPTY_CHECK_MS     = 20_000;  // délai avant de marquer un wrap vide
-  const MAX_INSERTS_PER_RUN = 6;
-  const MAX_INFLIGHT        = 4;
-  const SHOW_THROTTLE_MS    = 900;
-  const BURST_COOLDOWN_MS   = 200;
-  // Marges IO larges et fixes (pas de reconstruction d'observer)
+  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 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;
+  const SHOW_THROTTLE_MS  = 900;
+  const BURST_COOLDOWN_MS = 200;
+  // IO : marges larges fixes — une seule instance, jamais recréée
   const IO_MARGIN_DESKTOP = '2500px 0px 2500px 0px';
   const IO_MARGIN_MOBILE  = '3500px 0px 3500px 0px';
@@ -60,40 +59,37 @@
   };
   /**
-   * Table centrale : kindClass → { selector DOM, attribut d'ancre stable }
+   * Table KIND — source de vérité par kindClass.
    *
-   * L'attribut d'ancre est l'identifiant que NodeBB pose TOUJOURS sur l'élément,
-   * quelle que soit la page ou la virtualisation :
-   *   posts    → data-pid   (id du message, unique et permanent)
-   *   topics   → data-index (position 0-based dans la liste, fourni par NodeBB)
-   *   catégories → data-cid (id de la catégorie, unique et permanent)
-   *                          ← C'était le bug v19 : on cherchait data-index ici
+   * sel        : sélecteur CSS complet
+   * baseTag    : préfixe tag pour querySelector d'ancre
+   *              (vide pour posts car sélecteur commence par '[')
+   * anchorAttr : attribut DOM stable → clé unique du wrap
+   *              data-pid   posts / data-index topics / data-cid catégories
+   * ordinalAttr: attribut 0-based pour calcul de l'intervalle
+   *              null → fallback positionnel (catégories)
    */
   const KIND = {
-    'ezoic-ad-message':    { sel: SEL.post,     anchorAttr: 'data-pid'   },
-    'ezoic-ad-between':    { sel: SEL.topic,    anchorAttr: 'data-index' },
-    'ezoic-ad-categories': { sel: SEL.category, anchorAttr: 'data-cid'   },
+    '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 ───────────────────────────────────────────────────────────────────
   const S = {
-    pageKey: null,
-    cfg: null,
-    pools:      { topics: [], posts: [], categories: [] },
-    cursors:    { topics: 0,  posts: 0,  categories: 0  },
-    mountedIds: new Set(),      // IDs Ezoic actuellement dans le DOM
-    lastShow:   new Map(),      // id → timestamp dernier show
-    io:      null,
-    domObs:  null,
-    mutGuard: 0,                // compteur internalMutation
-    inflight:    0,
-    pending:     [],
-    pendingSet:  new Set(),
+    pageKey:      null,
+    cfg:          null,
+    pools:        { topics: [], posts: [], categories: [] },
+    cursors:      { topics: 0,  posts: 0,  categories: 0  },
+    mountedIds:   new Set(),
+    lastShow:     new Map(),
+    io:           null,
+    domObs:       null,
+    mutGuard:     0,
+    inflight:     0,
+    pending:      [],
+    pendingSet:   new Set(),
     runQueued:    false,
     burstActive:  false,
     burstDeadline: 0,
@@ -102,8 +98,11 @@
   };
   let blockedUntil = 0;
-  const isBlocked  = () => Date.now() < blockedUntil;
-  const ts         = () => Date.now();
+  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++;
@@ -121,12 +120,6 @@
     return S.cfg;
   }
-  function initPools(cfg) {
-    S.pools.topics     = parseIds(cfg.placeholderIds);
-    S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
-    S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
-  }
   function parseIds(raw) {
     const out = [], seen = new Set();
     for (const v of String(raw || '').split(/\r?\n/).map(s => s.trim()).filter(Boolean)) {
@@ -136,12 +129,11 @@
     return out;
   }
-  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]'));
-  const isMobile = () => { try { return window.innerWidth < 768; } catch (_) { return false; } };
+  function initPools(cfg) {
+    S.pools.topics     = parseIds(cfg.placeholderIds);
+    S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
+    S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
+  }
   // ── Page identity ──────────────────────────────────────────────────────────
@@ -165,13 +157,13 @@
     return 'other';
   }
-  // ── DOM helpers ────────────────────────────────────────────────────────────
+  // ── Items DOM ──────────────────────────────────────────────────────────────
   function getPosts() {
     return Array.from(document.querySelectorAll(SEL.post)).filter(el => {
       if (!el.isConnected) return false;
       if (!el.querySelector('[component="post/content"]')) return false;
-      const p = el.parentElement?.closest('[component="post"][data-pid]');
+      const p = el.parentElement?.closest(SEL.post);
       if (p && p !== el) return false;
       return el.getAttribute('component') !== 'post/parent';
     });
@@ -187,36 +179,28 @@
     );
   }
-  // ── Ancres stables ────────────────────────────────────────────────────────
+  // ── Ancres stables ─────────────────────────────────────────────────────────
-  /**
-   * Retourne l'identifiant stable de l'élément selon son kindClass.
-   * Utilise l'attribut défini dans KIND (data-pid, data-index, data-cid).
-   * Fallback positionnel si l'attribut est absent.
-   */
-  function stableId(kindClass, el) {
-    const attr = KIND[kindClass]?.anchorAttr;
+  function stableId(klass, el) {
+    const attr = KIND[klass]?.anchorAttr;
     if (attr) {
       const v = el.getAttribute(attr);
       if (v !== null && v !== '') return v;
     }
-    // Fallback : position dans le parent
-    try {
-      let i = 0;
-      for (const s of el.parentElement?.children ?? []) {
-        if (s === el) return `i${i}`;
-        i++;
-      }
-    } catch (_) {}
+    let i = 0;
+    for (const s of el.parentElement?.children ?? []) {
+      if (s === el) return `i${i}`;
+      i++;
+    }
     return 'i0';
   }
-  const makeAnchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
+  const anchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
-  function findWrap(anchorKey) {
+  function findWrap(key) {
     try {
       return document.querySelector(
-        `.${WRAP_CLASS}[${A_ANCHOR}="${anchorKey.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
+        `.${WRAP_CLASS}[${A_ANCHOR}="${key.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
       );
     } catch (_) { return null; }
   }
@@ -226,7 +210,7 @@
   function pickId(poolKey) {
     const pool = S.pools[poolKey];
     for (let t = 0; t < pool.length; t++) {
-      const i  = S.cursors[poolKey] % pool.length;
+      const i = S.cursors[poolKey] % pool.length;
       S.cursors[poolKey] = (S.cursors[poolKey] + 1) % pool.length;
       const id = pool[i];
       if (!S.mountedIds.has(id)) return id;
@@ -237,7 +221,7 @@
   // ── Wraps DOM ──────────────────────────────────────────────────────────────
   function makeWrap(id, klass, key) {
-    const w = document.createElement('div');
+    const w  = document.createElement('div');
     w.className = `${WRAP_CLASS} ${klass}`;
     w.setAttribute(A_ANCHOR,  key);
     w.setAttribute(A_WRAPID,  String(id));
@@ -251,10 +235,10 @@
   }
   function insertAfter(el, id, klass, key) {
-    if (!el?.insertAdjacentElement) return null;
-    if (findWrap(key))            return null; // ancre déjà présente
-    if (S.mountedIds.has(id))     return null; // id déjà monté
-    if (document.getElementById(`${PH_PREFIX}${id}`)?.isConnected) return null;
+    if (!el?.insertAdjacentElement)                                   return null;
+    if (findWrap(key))                                                return null;
+    if (S.mountedIds.has(id))                                         return null;
+    if (document.getElementById(`${PH_PREFIX}${id}`)?.isConnected)   return null;
     const w = makeWrap(id, klass, key);
     mutate(() => el.insertAdjacentElement('afterend', w));
     S.mountedIds.add(id);
@@ -263,15 +247,10 @@
   function dropWrap(w) {
     try {
+      const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
+      if (ph instanceof Element) S.io?.unobserve(ph);
       const id = parseInt(w.getAttribute(A_WRAPID), 10);
       if (Number.isFinite(id)) S.mountedIds.delete(id);
-      // IMPORTANT : ne passer unobserve que si c'est un vrai Element.
-      // unobserve(null) corrompt l'état interne de l'IO (pubads lève ensuite
-      // "parameter 1 is not of type Element" sur le prochain observe).
-      try {
-        const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
-        if (ph instanceof Element) S.io?.unobserve(ph);
-      } catch (_) {}
       w.remove();
     } catch (_) {}
   }
@@ -279,58 +258,59 @@
   // ── Prune ──────────────────────────────────────────────────────────────────
   /**
-   * Supprime les wraps dont l'élément-ancre n'est plus dans le DOM.
-   *
-   * L'ancre est retrouvée via l'attribut stable défini dans KIND[kindClass].anchorAttr.
-   * Exemples :
-   *   ezoic-ad-message    → cherche [data-pid="123"]
-   *   ezoic-ad-between    → cherche [data-index="5"]
-   *   ezoic-ad-categories → cherche [data-cid="7"]   ← fix v20
+   * Supprime les wraps VIDES dont l'ancre a disparu du DOM.
    *
-   * On ne prune pas les wraps < MIN_PRUNE_AGE_MS (DOM pas encore stabilisé).
+   * 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;
-    const baseTag = meta.sel.split('[')[0]; // ex: "li", "[component=..." → "" (géré)
-    document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
-      if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < MIN_PRUNE_AGE_MS) 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); // extrait la partie après "kindClass:"
-      if (!sid) { mutate(() => dropWrap(w)); return; }
+      const sid = key.slice(klass.length + 1);
+      if (!sid) { mutate(() => dropWrap(w)); continue; }
-      const anchorEl = document.querySelector(
-        `${baseTag}[${meta.anchorAttr}="${sid.replace(/"/g, '\\"')}"]`
-      );
+      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 = situation anormale → supprimer le moins prioritaire.
-   * Priorité : filled > en grâce (fill en cours) > vide.
-   * Ne supprime jamais un wrap dont showAds() date de moins de FILL_GRACE_MS.
+   * 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}`)) {
-      // Grace sur le wrap courant : on le saute entièrement
+      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; // précédent en grâce → rien
+        if (pShown && ts() - pShown < FILL_GRACE_MS) break;
-        if      (!isFilled(w))    mutate(() => dropWrap(w));
-        else if (!isFilled(prev)) mutate(() => dropWrap(prev));
+        mutate(() => dropWrap(w));
         break;
       }
     }
@@ -340,23 +320,21 @@
   /**
    * Ordinal 0-based pour le calcul de l'intervalle.
-   * Pour posts/topics : data-index (NodeBB 3+/4+, toujours présent).
-   * Pour catégories : position dans le parent (page statique, pas d'infinite scroll).
+   * Utilise KIND[klass].ordinalAttr (data-index pour posts/topics).
+   * Catégories : ordinalAttr = null → fallback positionnel.
    */
   function ordinal(klass, el) {
-    const di = el.getAttribute('data-index');
-    if (di !== null && di !== '' && !isNaN(di)) return parseInt(di, 10);
-    // Fallback positionnel
-    try {
-      const tag = KIND[klass]?.sel?.split('[')?.[0] ?? '';
-      if (tag) {
-        let i = 0;
-        for (const n of el.parentElement?.querySelectorAll(`:scope > ${tag}`) ?? []) {
-          if (n === el) return i;
-          i++;
-        }
-      }
-    } catch (_) {}
+    const attr = KIND[klass]?.ordinalAttr;
+    if (attr) {
+      const v = el.getAttribute(attr);
+      if (v !== null && v !== '' && !isNaN(v)) return parseInt(v, 10);
+    }
+    const fullSel = KIND[klass]?.sel ?? '';
+    let i = 0;
+    for (const s of el.parentElement?.children ?? []) {
+      if (s === el) return i;
+      if (!fullSel || s.matches?.(fullSel)) i++;
+    }
     return 0;
   }
@@ -365,20 +343,18 @@
     let inserted = 0;
     for (const el of items) {
-      if (inserted >= MAX_INSERTS_PER_RUN) break;
+      if (inserted >= MAX_INSERTS_RUN) break;
       if (!el?.isConnected) continue;
-      const ord      = ordinal(klass, el);
-      const isTarget = (showFirst && ord === 0) || ((ord + 1) % interval === 0);
-      if (!isTarget) continue;
+      const ord = ordinal(klass, el);
+      if (!(showFirst && ord === 0) && (ord + 1) % interval !== 0) continue;
       if (adjacentWrap(el)) continue;
-      const key = makeAnchorKey(klass, el);
-      if (findWrap(key)) continue; // déjà là → pas de pickId inutile
+      const key = anchorKey(klass, el);
+      if (findWrap(key)) continue;
       const id = pickId(poolKey);
-      if (!id) continue; // pool momentanément épuisé, on continue (items suivants ≠ besoin d'id)
+      if (!id) continue; // pool épuisé : on attend que pruneOrphans libère des ids
       const w = insertAfter(el, id, klass, key);
       if (w) { observePh(id); inserted++; }
@@ -390,7 +366,6 @@
   function getIO() {
     if (S.io) return S.io;
-    const margin = isMobile() ? IO_MARGIN_MOBILE : IO_MARGIN_DESKTOP;
     try {
       S.io = new IntersectionObserver(entries => {
         for (const e of entries) {
@@ -399,7 +374,7 @@
           const id = parseInt(e.target.getAttribute('data-ezoic-id'), 10);
           if (Number.isFinite(id) && id > 0) enqueueShow(id);
         }
-      }, { root: null, rootMargin: margin, threshold: 0 });
+      }, { root: null, rootMargin: isMobile() ? IO_MARGIN_MOBILE : IO_MARGIN_DESKTOP, threshold: 0 });
     } catch (_) { S.io = null; }
     return S.io;
   }
@@ -450,7 +425,6 @@
         if (t - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) { clearTimeout(timer); return release(); }
         S.lastShow.set(id, t);
-        // Horodater le show sur le wrap pour grace period + emptyCheck
         try { ph.closest?.(`.${WRAP_CLASS}`)?.setAttribute(A_SHOWN, String(t)); } catch (_) {}
         window.ezstandalone = window.ezstandalone || {};
@@ -471,7 +445,6 @@
         const ph   = document.getElementById(`${PH_PREFIX}${id}`);
         const wrap = ph?.closest?.(`.${WRAP_CLASS}`);
         if (!wrap || !ph?.isConnected) return;
-        // Un show plus récent → ne pas toucher
         if (parseInt(wrap.getAttribute(A_SHOWN) || '0', 10) > showTs) return;
         wrap.classList.toggle('is-empty', !isFilled(ph));
       } catch (_) {}
@@ -490,7 +463,7 @@
         const orig = ez.showAds.bind(ez);
         ez.showAds = function (...args) {
           if (isBlocked()) return;
-          const ids = (args.length === 1 && Array.isArray(args[0])) ? args[0] : args;
+          const ids  = args.length === 1 && Array.isArray(args[0]) ? args[0] : args;
           const seen = new Set();
           for (const v of ids) {
             const id = parseInt(v, 10);
@@ -509,7 +482,7 @@
     }
   }
-  // ── Core run ───────────────────────────────────────────────────────────────
+  // ── Core ───────────────────────────────────────────────────────────────────
   async function runCore() {
     if (isBlocked()) return 0;
@@ -524,10 +497,9 @@
     const exec = (klass, getItems, cfgEnable, cfgInterval, cfgShowFirst, poolKey) => {
       if (!normBool(cfgEnable)) return 0;
-      const items    = getItems();
       const interval = Math.max(1, parseInt(cfgInterval, 10) || 3);
       pruneOrphans(klass);
-      const n = injectBetween(klass, items, interval, normBool(cfgShowFirst), poolKey);
+      const n = injectBetween(klass, getItems(), interval, normBool(cfgShowFirst), poolKey);
       if (n) decluster(klass);
       return n;
     };
@@ -540,14 +512,13 @@
       'ezoic-ad-between', getTopics,
       cfg.enableBetweenAds, cfg.intervalPosts, cfg.showFirstTopicAd, 'topics'
     );
-    if (kind === 'categories') return exec(
+    return exec(
       'ezoic-ad-categories', getCategories,
       cfg.enableCategoryAds, cfg.intervalCategories, cfg.showFirstCategoryAd, 'categories'
     );
-    return 0;
   }
-  // ── Scheduler / Burst ──────────────────────────────────────────────────────
+  // ── Scheduler ──────────────────────────────────────────────────────────────
   function scheduleRun(cb) {
     if (S.runQueued) return;
@@ -565,10 +536,8 @@
     if (isBlocked()) return;
     const t = ts();
     if (t - S.lastBurstTs < BURST_COOLDOWN_MS) return;
-    S.lastBurstTs = t;
-    const pk = pageKey();
-    S.pageKey      = pk;
+    S.lastBurstTs   = t;
+    S.pageKey       = pageKey();
     S.burstDeadline = t + 2000;
     if (S.burstActive) return;
@@ -576,7 +545,7 @@
     S.burstCount  = 0;
     const step = () => {
-      if (pageKey() !== pk || isBlocked() || ts() > S.burstDeadline || S.burstCount >= 8) {
+      if (pageKey() !== S.pageKey || isBlocked() || ts() > S.burstDeadline || S.burstCount >= 8) {
         S.burstActive = false; return;
       }
       S.burstCount++;
@@ -588,7 +557,7 @@
     step();
   }
-  // ── Cleanup (navigation ajaxify) ───────────────────────────────────────────
+  // ── Cleanup navigation ─────────────────────────────────────────────────────
   function cleanup() {
     blockedUntil = ts() + 1500;
@@ -605,19 +574,17 @@
     S.runQueued    = false;
   }
-  // ── DOM Observer ───────────────────────────────────────────────────────────
+  // ── MutationObserver ───────────────────────────────────────────────────────
   function ensureDomObserver() {
     if (S.domObs) return;
+    const allSel = [SEL.post, SEL.topic, SEL.category];
     S.domObs = new MutationObserver(muts => {
       if (S.mutGuard > 0 || isBlocked()) return;
       for (const m of muts) {
-        if (!m.addedNodes?.length) continue;
         for (const n of m.addedNodes) {
           if (n.nodeType !== 1) continue;
-          if (n.matches?.(SEL.post)     || n.querySelector?.(SEL.post)     ||
-              n.matches?.(SEL.topic)    || n.querySelector?.(SEL.topic)    ||
-              n.matches?.(SEL.category) || n.querySelector?.(SEL.category)) {
+          if (allSel.some(s => n.matches?.(s) || n.querySelector?.(s))) {
             requestBurst(); return;
           }
         }
@@ -643,29 +610,18 @@
   }
   function ensureTcfLocator() {
-    // Le CMP utilise une iframe nommée __tcfapiLocator pour router les
-    // postMessage TCF. En navigation ajaxify, NodeBB peut retirer cette
-    // iframe du DOM (vidage partiel du body), ce qui provoque :
-    //   "Cannot read properties of null (reading 'postMessage')"
-    //   "Cannot set properties of null (setting 'addtlConsent')"
-    // Solution : la recrée immédiatement si elle disparaît, via un observer.
     try {
       if (!window.__tcfapi && !window.__cmp) return;
       const inject = () => {
         if (document.getElementById('__tcfapiLocator')) return;
         const f = document.createElement('iframe');
         f.style.display = 'none'; f.id = f.name = '__tcfapiLocator';
         (document.body || document.documentElement).appendChild(f);
       };
       inject();
-      // Observer dédié — si quelqu'un retire l'iframe, on la remet.
       if (!window.__nbbTcfObs) {
-        window.__nbbTcfObs = new MutationObserver(() => inject());
-        window.__nbbTcfObs.observe(document.documentElement,
-          { childList: true, subtree: true });
+        window.__nbbTcfObs = new MutationObserver(inject);
+        window.__nbbTcfObs.observe(document.documentElement, { childList: true, subtree: true });
       }
     } catch (_) {}
   }
@@ -675,10 +631,10 @@
     const head = document.head;
     if (!head) return;
     for (const [rel, href, cors] of [
-      ['preconnect',   'https://g.ezoic.net',                    true],
-      ['preconnect',   'https://go.ezoic.net',                   true],
-      ['preconnect',   'https://securepubads.g.doubleclick.net', true],
-      ['preconnect',   'https://pagead2.googlesyndication.com',  true],
+      ['preconnect',   'https://g.ezoic.net',                    true ],
+      ['preconnect',   'https://go.ezoic.net',                   true ],
+      ['preconnect',   'https://securepubads.g.doubleclick.net', true ],
+      ['preconnect',   'https://pagead2.googlesyndication.com',  true ],
       ['dns-prefetch', 'https://g.ezoic.net',                    false],
       ['dns-prefetch', 'https://securepubads.g.doubleclick.net', false],
     ]) {
@@ -692,7 +648,7 @@
     }
   }
-  // ── Bindings NodeBB ────────────────────────────────────────────────────────
+  // ── Bindings ───────────────────────────────────────────────────────────────
   function bindNodeBB() {
     const $ = window.jQuery;
@@ -703,19 +659,16 @@
     $(window).on('action:ajaxify.end.nbbEzoic', () => {
       S.pageKey    = pageKey();
       blockedUntil = 0;
-      muteConsole(); ensureTcfLocator(); warmNetwork(); patchShowAds();
-      getIO(); ensureDomObserver(); requestBurst();
+      muteConsole(); ensureTcfLocator(); warmNetwork();
+      patchShowAds(); getIO(); ensureDomObserver(); requestBurst();
     });
-    const BURST_EVENTS = [
-      'action:ajaxify.contentLoaded',
-      'action:posts.loaded', 'action:topics.loaded',
+    const burstEvts = [
+      '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(); });
-    $(window).on(BURST_EVENTS, () => { if (!isBlocked()) requestBurst(); });
-    // Hooks AMD (NodeBB 4.x — redondant avec jQuery events mais sans risque)
     try {
       require(['hooks'], hooks => {
         if (typeof hooks?.on !== 'function') return;