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

nodebb-plugin-ezoic-infinite 1.7.20 → 1.7.22

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 +211 -164

package/package.json CHANGED Viewed

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

package/public/client.js CHANGED Viewed

@@ -1,54 +1,55 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v26
+ * NodeBB Ezoic Infinite Ads — client.js (v20)
  *
- * 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.
+ * 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).
  *
- * v19  Intervalle global basé sur l'ordinal absolu (data-index) et non sur
- *      la position dans le batch courant.
+ * [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.
  *
- * 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.
+ * [BUG] injectBetween : `break` sur pool épuisé empêchait d'observer les wraps
+ *   existants sur les items suivants. Fix : `continue` au lieu de `break`.
  *
- * 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] IntersectionObserver recréé à chaque scroll boost → très coûteux mobile.
+ *   Fix : marge large fixe par device, observer créé une seule fois.
  *
- * 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.
+ * [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é)
  */
 (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
-  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 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 IO_MARGIN_DESKTOP = '2500px 0px 2500px 0px';
   const IO_MARGIN_MOBILE  = '3500px 0px 3500px 0px';
@@ -59,37 +60,40 @@
   };
   /**
-   * Table KIND — source de vérité par kindClass.
+   * Table centrale : kindClass → { selector DOM, attribut d'ancre stable }
    *
-   * 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)
+   * 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
    */
   const KIND = {
-    '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         },
+    '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'   },
   };
   // ── État ───────────────────────────────────────────────────────────────────
   const S = {
-    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(),
+    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(),
     runQueued:    false,
     burstActive:  false,
     burstDeadline: 0,
@@ -98,11 +102,8 @@
   };
   let blockedUntil = 0;
-  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]'));
+  const isBlocked  = () => Date.now() < blockedUntil;
+  const ts         = () => Date.now();
   function mutate(fn) {
     S.mutGuard++;
@@ -120,6 +121,12 @@
     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)) {
@@ -129,11 +136,12 @@
     return out;
   }
-  function initPools(cfg) {
-    S.pools.topics     = parseIds(cfg.placeholderIds);
-    S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
-    S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
-  }
+  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; } };
   // ── Page identity ──────────────────────────────────────────────────────────
@@ -157,13 +165,13 @@
     return 'other';
   }
-  // ── Items DOM ──────────────────────────────────────────────────────────────
+  // ── DOM helpers ────────────────────────────────────────────────────────────
   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(SEL.post);
+      const p = el.parentElement?.closest('[component="post"][data-pid]');
       if (p && p !== el) return false;
       return el.getAttribute('component') !== 'post/parent';
     });
@@ -179,28 +187,36 @@
     );
   }
-  // ── Ancres stables ─────────────────────────────────────────────────────────
+  // ── Ancres stables ────────────────────────────────────────────────────────
-  function stableId(klass, el) {
-    const attr = KIND[klass]?.anchorAttr;
+  /**
+   * 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;
     if (attr) {
       const v = el.getAttribute(attr);
       if (v !== null && v !== '') return v;
     }
-    let i = 0;
-    for (const s of el.parentElement?.children ?? []) {
-      if (s === el) return `i${i}`;
-      i++;
-    }
+    // Fallback : position dans le parent
+    try {
+      let i = 0;
+      for (const s of el.parentElement?.children ?? []) {
+        if (s === el) return `i${i}`;
+        i++;
+      }
+    } catch (_) {}
     return 'i0';
   }
-  const anchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
+  const makeAnchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
-  function findWrap(key) {
+  function findWrap(anchorKey) {
     try {
       return document.querySelector(
-        `.${WRAP_CLASS}[${A_ANCHOR}="${key.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
+        `.${WRAP_CLASS}[${A_ANCHOR}="${anchorKey.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
       );
     } catch (_) { return null; }
   }
@@ -210,7 +226,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;
@@ -221,7 +237,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));
@@ -235,10 +251,10 @@
   }
   function insertAfter(el, id, klass, key) {
-    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;
+    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;
     const w = makeWrap(id, klass, key);
     mutate(() => el.insertAdjacentElement('afterend', w));
     S.mountedIds.add(id);
@@ -247,10 +263,15 @@
   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 (_) {}
   }
@@ -258,59 +279,58 @@
   // ── Prune ──────────────────────────────────────────────────────────────────
   /**
-   * Supprime les wraps VIDES dont l'ancre a disparu du DOM.
+   * 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
    *
-   * 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.
+   * On ne prune pas les wraps < MIN_PRUNE_AGE_MS (DOM pas encore stabilisé).
    */
   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 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;
       const key = w.getAttribute(A_ANCHOR) ?? '';
-      const sid = key.slice(klass.length + 1);
-      if (!sid) { mutate(() => dropWrap(w)); continue; }
+      const sid = key.slice(klass.length + 1); // extrait la partie après "kindClass:"
+      if (!sid) { mutate(() => dropWrap(w)); return; }
-      const sel      = `${meta.baseTag}[${meta.anchorAttr}="${sid.replace(/"/g, '\\"')}"]`;
-      const anchorEl = document.querySelector(sel);
+      const anchorEl = document.querySelector(
+        `${baseTag}[${meta.anchorAttr}="${sid.replace(/"/g, '\\"')}"]`
+      );
       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
+   * 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.
    */
   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;
+      // Grace sur le wrap courant : on le saute entièrement
       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;
+        if (pShown && ts() - pShown < FILL_GRACE_MS) break; // précédent en grâce → rien
-        mutate(() => dropWrap(w));
+        if      (!isFilled(w))    mutate(() => dropWrap(w));
+        else if (!isFilled(prev)) mutate(() => dropWrap(prev));
         break;
       }
     }
@@ -320,21 +340,23 @@
   /**
    * Ordinal 0-based pour le calcul de l'intervalle.
-   * Utilise KIND[klass].ordinalAttr (data-index pour posts/topics).
-   * Catégories : ordinalAttr = null → fallback positionnel.
+   * Pour posts/topics : data-index (NodeBB 3+/4+, toujours présent).
+   * Pour catégories : position dans le parent (page statique, pas d'infinite scroll).
    */
   function ordinal(klass, el) {
-    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++;
-    }
+    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 (_) {}
     return 0;
   }
@@ -343,18 +365,20 @@
     let inserted = 0;
     for (const el of items) {
-      if (inserted >= MAX_INSERTS_RUN) break;
+      if (inserted >= MAX_INSERTS_PER_RUN) break;
       if (!el?.isConnected) continue;
-      const ord = ordinal(klass, el);
-      if (!(showFirst && ord === 0) && (ord + 1) % interval !== 0) continue;
+      const ord      = ordinal(klass, el);
+      const isTarget = (showFirst && ord === 0) || ((ord + 1) % interval === 0);
+      if (!isTarget) continue;
       if (adjacentWrap(el)) continue;
-      const key = anchorKey(klass, el);
-      if (findWrap(key)) continue;
+      const key = makeAnchorKey(klass, el);
+      if (findWrap(key)) continue; // déjà là → pas de pickId inutile
       const id = pickId(poolKey);
-      if (!id) continue; // pool épuisé : on attend que pruneOrphans libère des ids
+      if (!id) continue; // pool momentanément épuisé, on continue (items suivants ≠ besoin d'id)
       const w = insertAfter(el, id, klass, key);
       if (w) { observePh(id); inserted++; }
@@ -366,6 +390,7 @@
   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) {
@@ -374,7 +399,7 @@
           const id = parseInt(e.target.getAttribute('data-ezoic-id'), 10);
           if (Number.isFinite(id) && id > 0) enqueueShow(id);
         }
-      }, { root: null, rootMargin: isMobile() ? IO_MARGIN_MOBILE : IO_MARGIN_DESKTOP, threshold: 0 });
+      }, { root: null, rootMargin: margin, threshold: 0 });
     } catch (_) { S.io = null; }
     return S.io;
   }
@@ -425,6 +450,7 @@
         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 || {};
@@ -445,6 +471,7 @@
         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 (_) {}
@@ -463,7 +490,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);
@@ -482,7 +509,7 @@
     }
   }
-  // ── Core ───────────────────────────────────────────────────────────────────
+  // ── Core run ───────────────────────────────────────────────────────────────
   async function runCore() {
     if (isBlocked()) return 0;
@@ -497,9 +524,10 @@
     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, getItems(), interval, normBool(cfgShowFirst), poolKey);
+      const n = injectBetween(klass, items, interval, normBool(cfgShowFirst), poolKey);
       if (n) decluster(klass);
       return n;
     };
@@ -512,13 +540,14 @@
       'ezoic-ad-between', getTopics,
       cfg.enableBetweenAds, cfg.intervalPosts, cfg.showFirstTopicAd, 'topics'
     );
-    return exec(
+    if (kind === 'categories') return exec(
       'ezoic-ad-categories', getCategories,
       cfg.enableCategoryAds, cfg.intervalCategories, cfg.showFirstCategoryAd, 'categories'
     );
+    return 0;
   }
-  // ── Scheduler ──────────────────────────────────────────────────────────────
+  // ── Scheduler / Burst ──────────────────────────────────────────────────────
   function scheduleRun(cb) {
     if (S.runQueued) return;
@@ -536,8 +565,10 @@
     if (isBlocked()) return;
     const t = ts();
     if (t - S.lastBurstTs < BURST_COOLDOWN_MS) return;
-    S.lastBurstTs   = t;
-    S.pageKey       = pageKey();
+    S.lastBurstTs = t;
+    const pk = pageKey();
+    S.pageKey      = pk;
     S.burstDeadline = t + 2000;
     if (S.burstActive) return;
@@ -545,7 +576,7 @@
     S.burstCount  = 0;
     const step = () => {
-      if (pageKey() !== S.pageKey || isBlocked() || ts() > S.burstDeadline || S.burstCount >= 8) {
+      if (pageKey() !== pk || isBlocked() || ts() > S.burstDeadline || S.burstCount >= 8) {
         S.burstActive = false; return;
       }
       S.burstCount++;
@@ -557,7 +588,7 @@
     step();
   }
-  // ── Cleanup navigation ─────────────────────────────────────────────────────
+  // ── Cleanup (navigation ajaxify) ───────────────────────────────────────────
   function cleanup() {
     blockedUntil = ts() + 1500;
@@ -574,17 +605,19 @@
     S.runQueued    = false;
   }
-  // ── MutationObserver ───────────────────────────────────────────────────────
+  // ── DOM Observer ───────────────────────────────────────────────────────────
   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 (allSel.some(s => n.matches?.(s) || n.querySelector?.(s))) {
+          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)) {
             requestBurst(); return;
           }
         }
@@ -610,18 +643,29 @@
   }
   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 (_) {}
   }
@@ -631,10 +675,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],
     ]) {
@@ -648,7 +692,7 @@
     }
   }
-  // ── Bindings ───────────────────────────────────────────────────────────────
+  // ── Bindings NodeBB ────────────────────────────────────────────────────────
   function bindNodeBB() {
     const $ = window.jQuery;
@@ -659,16 +703,19 @@
     $(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 burstEvts = [
-      'action:ajaxify.contentLoaded', 'action:posts.loaded', 'action:topics.loaded',
+    const BURST_EVENTS = [
+      '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;