npm - nodebb-plugin-ezoic-infinite - Versions diffs - 1.7.9 → 1.7.11 - Mend

nodebb-plugin-ezoic-infinite 1.7.9 → 1.7.11

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 +204 -156

package/package.json CHANGED Viewed

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

package/public/client.js CHANGED Viewed

@@ -1,49 +1,55 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v21
+ * NodeBB Ezoic Infinite Ads — client.js v22
  *
- * Historique des corrections majeures
- * ────────────────────────────────────
- * v18  Ancrage stable par data-pid/data-index au lieu d'ordinalMap fragile.
- *      Suppression du recyclage de wraps (moveWrapAfter). Cleanup complet navigation.
+ * 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`.
  *
- * v21  Suppression de toute la logique wyvern.js (pause/destroy avant remove) :
- *      les erreurs wyvern viennent du SDK Ezoic lui-même lors de ses propres
- *      refreshes internes, pas de nos suppressions. Nos wraps filled ne sont
- *      de toute façon jamais supprimés (règle pruneOrphans/decluster).
- *      Refactorisation finale prod-ready : code unifié, zéro duplication,
- *      commentaires essentiels uniquement.
+ * [PERF] IntersectionObserver recréé à chaque scroll boost → très coûteux mobile.
+ *   Fix : marge large fixe par device, observer créé une seule fois.
+ *
+ * [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 MIN_PRUNE_AGE_MS  = 8_000;  // garde-fou post-batch NodeBB
-  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';
@@ -54,40 +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 les querySelector de recherche d'ancre
-   *              (vide pour posts car leur sélecteur commence par '[')
-   * anchorAttr : attribut DOM STABLE → clé unique du wrap, permanent
-   *              data-pid   posts      (id message, immuable)
-   *              data-index topics     (index dans la liste)
-   *              data-cid   catégories (id catégorie, immuable)
-   * ordinalAttr: attribut 0-based pour le calcul de l'intervalle
-   *              data-index posts + topics (fourni par NodeBB)
-   *              null       catégories (page statique → fallback positionnel)
+   * 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(),   // IDs Ezoic montés dans le DOM
-    lastShow:     new Map(),   // id → timestamp dernier show
-    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,
@@ -96,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++;
@@ -118,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)) {
@@ -127,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 ──────────────────────────────────────────────────────────
@@ -155,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';
     });
@@ -177,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; }
   }
@@ -208,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;
@@ -219,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));
@@ -233,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);
@@ -245,12 +263,15 @@
   function dropWrap(w) {
     try {
-      // Unobserve avant remove — guard instanceof évite unobserve(null)
-      // qui corrompt l'état interne de l'IO (pubads error au scroll suivant)
-      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,42 +279,50 @@
   // ── 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.
    *
-   * On ne supprime JAMAIS un wrap rempli (filled) :
-   *   - Les wraps remplis peuvent être temporairement orphelins lors d'une
-   *     virtualisation NodeBB — l'ancre reviendra.
-   *   - Le SDK Ezoic (wyvern, GAM) exécute des callbacks async sur le contenu ;
-   *     retirer le nœud sous ses pieds génère des erreurs non critiques mais
-   *     inutiles. Le cleanup de navigation gère la suppression définitive.
+   * 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
+   *
+   * 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 (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < MIN_PRUNE_AGE_MS) continue;
-      if (isFilled(w)) continue;
+    // baseTag déduit du sélecteur : 'li' pour topics/catégories, '' pour posts.
+    // Pour les posts (baseTag=''), on cherche juste [data-pid="X"] sans préfixe
+    // — c'est correct car data-pid est unique dans le DOM.
+    const baseTag = meta.sel.split('[')[0];
+    document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
+      if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < MIN_PRUNE_AGE_MS) return;
+      if (isFilled(w)) return; // jamais supprimer un wrap rempli
       const key = w.getAttribute(A_ANCHOR) ?? '';
       const sid = key.slice(klass.length + 1);
-      if (!sid) { mutate(() => dropWrap(w)); continue; }
+      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 moins prioritaire.
-   * Priorité : filled > en grâce de fill > vide.
-   * Jamais de suppression d'un wrap rempli (même raison que pruneOrphans).
+   * 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}`)) {
+      // 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;
@@ -302,11 +331,10 @@
         if (!prev.classList?.contains(WRAP_CLASS)) { prev = prev.previousElementSibling; continue; }
         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
         if      (!isFilled(w))    mutate(() => dropWrap(w));
         else if (!isFilled(prev)) mutate(() => dropWrap(prev));
-        // les deux remplis → on ne touche pas
         break;
       }
     }
@@ -316,16 +344,14 @@
   /**
    * Ordinal 0-based pour le calcul de l'intervalle.
-   * Posts et topics : data-index (NodeBB 4.x, stable entre les batches).
-   * Catégories      : fallback positionnel (page statique, pas d'infinite scroll).
+   * 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);
-    }
-    // Fallback positionnel — compte uniquement les éléments du même type
+    const di = el.getAttribute('data-index');
+    if (di !== null && di !== '' && !isNaN(di)) return parseInt(di, 10);
+    // Fallback positionnel — filtre par sélecteur complet pour éviter le bug
+    // baseTag='' (posts) où `:scope > ` sans tag ne fonctionne pas.
     const fullSel = KIND[klass]?.sel ?? '';
     let i = 0;
     for (const s of el.parentElement?.children ?? []) {
@@ -340,18 +366,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;
+      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++; }
@@ -363,6 +391,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) {
@@ -371,7 +400,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;
   }
@@ -422,6 +451,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 || {};
@@ -442,6 +472,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 (_) {}
@@ -460,7 +491,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);
@@ -479,7 +510,7 @@
     }
   }
-  // ── Core ───────────────────────────────────────────────────────────────────
+  // ── Core run ───────────────────────────────────────────────────────────────
   async function runCore() {
     if (isBlocked()) return 0;
@@ -494,9 +525,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;
     };
@@ -509,13 +541,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;
@@ -525,7 +558,7 @@
       if (S.pageKey && pageKey() !== S.pageKey) return;
       let n = 0;
       try { n = await runCore(); } catch (_) {}
-      cb?.(n);
+      try { cb?.(n); } catch (_) {}
     });
   }
@@ -533,8 +566,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;
@@ -542,7 +577,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++;
@@ -554,7 +589,7 @@
     step();
   }
-  // ── Cleanup navigation ─────────────────────────────────────────────────────
+  // ── Cleanup (navigation ajaxify) ───────────────────────────────────────────
   function cleanup() {
     blockedUntil = ts() + 1500;
@@ -571,17 +606,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;
           }
         }
@@ -607,21 +644,29 @@
   }
   function ensureTcfLocator() {
-    // L'iframe __tcfapiLocator route les appels postMessage du CMP.
-    // En navigation ajaxify, NodeBB peut la retirer du DOM → erreurs CMP.
-    // Un MutationObserver la recrée dès qu'elle disparaît.
+    // 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 +676,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 +693,7 @@
     }
   }
-  // ── Bindings ───────────────────────────────────────────────────────────────
+  // ── Bindings NodeBB ────────────────────────────────────────────────────────
   function bindNodeBB() {
     const $ = window.jQuery;
@@ -659,16 +704,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;