nodebb-plugin-ezoic-infinite 1.7.28 → 1.7.30

package/package.json

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

package/public/client.js

@@ -1,68 +1,58 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v33
+ * NodeBB Ezoic Infinite Ads — client.js v36
  *
  * 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.
+ * v18  Ancrage stable par data-pid / data-index au lieu d'ordinalMap fragile.
  *
- * v19  Intervalle global basé sur l'ordinal absolu (data-index) et non sur
+ * v19  Intervalle global basé sur l'ordinal absolu (data-index), pas sur
  *      la position dans le batch courant.
  *
- * v20  Table KIND : anchorAttr/ordinalAttr/baseTag explicites par kindClass.
+ * v20  Table KIND : anchorAttr / ordinalAttr / baseTag 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.
+ *      IO fixe (une instance, jamais recréée).
  *      Fix TCF locator : MutationObserver recrée l'iframe si ajaxify la retire.
  *
- * v25  Table KIND unifiée avec baseTag + ordinalAttr.
- *      Fix scroll-up / virtualisation NodeBB :
- *        – 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).
+ * v25  Fix scroll-up / virtualisation NodeBB : decluster + grace period.
  *
- * v26  Suppression définitive du recyclage d'id.
- *      KIND simplifié.
+ * v26  Suppression définitive du recyclage d'id (causait réinjection en haut).
  *
- * v27  pruneOrphans supprimé (faux-orphelins sur virtualisation NodeBB).
+ * v27  pruneOrphans supprimé (faux-orphelins sur virtualisation NodeBB posts).
  *
- * v28  decluster supprimé. pruneOrphans supprimé (v27). Wraps persistants sur session.
+ * v28  decluster supprimé. Wraps persistants pendant la session.
  *
- * v29  Fix ancrage topics : data-index → data-tid.
+ * 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.
  *
- * v32  Retour anchorAttr = data-index pour ezoic-ad-between (comme v20.1).
- *      data-tid peut être absent sur li[component="category/topic"] → clés
- *      invalides → wraps empilés sur pos0. data-index est toujours présent.
- *      pruneOrphansBetween cherche li[data-index="N"] — cohérent avec v20.1.
+ * v34  moveDistantWrap : déplace le nœud wrap existant au lieu de le supprimer
+ *      et recréer. Ezoic garde une registry interne des placeholders — recréer
+ *      un div avec le même id déclenche "already been defined" et bloque
+ *      showAds(). En déplaçant le même nœud DOM, la registry reste valide.
  *
- * v31  pruneOrphans réactivé UNIQUEMENT pour ezoic-ad-between (topics de catégorie).
- *      NodeBB NE virtualise PAS les topics dans une liste de catégorie — les ancres
- *      restent dans le DOM entre les scrolls. pruneOrphans est donc safe ici et
- *      c'est lui qui empêche l'empilement des pubs en haut après un scroll long.
- *      Pour ezoic-ad-message (posts de topic), pruneOrphans reste désactivé car
- *      NodeBB virtualise les posts hors-viewport → faux-orphelins → bug réinjection.
+ * 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,
+ *        moveDistantWrap, 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.
  *
- * v30  Fix adjacentWrap : ne compte plus les wraps orphelins (ancre hors DOM).
- *      Quand NodeBB virtualise et retire des topics du DOM, les wraps restent
- *      en place (div dans le ul). adjacentWrap(el) retournait true sur ces
- *      wraps orphelins → injection bloquée sur les topics suivants.
- *      Fix : adjacentWrap vérifie que le wrap voisin a son ancre dans le DOM.
- *      recycleOrphanId() : quand le pool est épuisé, recycle les wraps orphelins
- *      non remplis qui sont loin au-dessus du viewport.
- *      data-index = position relative dans le batch NodeBB, pas un ID stable.
- *      Lors du scroll infini, les nouveaux batches démarrent à data-index=0,
- *      ce qui créait des collisions de clés d'ancrage → mauvaise déduplication
- *      → wraps non injectés sur les nouveaux topics, puis réinjection en haut.
- *      Fix : anchorAttr = data-tid (stable et unique par topic).
- *      ordinalAttr reste data-index pour le calcul de l'intervalle.
- *      Analyse : decluster appelait dropWrap() qui faisait S.mountedIds.delete(id).
- *      Un wrap vide adjacent à un autre wrap → supprimé → id libéré → réinjecté
- *      en haut au prochain scroll. Exactement le bug observé.
- *      Les deux mécanismes de nettoyage actif (pruneOrphans + decluster) sont
- *      maintenant retirés. Seul cleanup() à la navigation peut supprimer des wraps.
+ * v35  Revue complète prod-ready :
+ *      – initPools protégé contre ré-initialisation inutile (S.poolsReady).
+ *      – muteConsole élargit à "No valid placeholders for loadMore".
+ *      – moveDistantWrap appelle ezstandalone.refresh([id]) après déplacement
+ *        pour forcer Ezoic à réactiver le placeholder sur son nouveau nœud.
+ *      – A_SHOWN initialisé à '0' dans moveDistantWrap (évite NaN dans parseInt).
+ *      – Commentaires et historique nettoyés.
  */
-(function () {
+(function nbbEzoicInfinite() {
   'use strict';
 
   // ── Constantes ─────────────────────────────────────────────────────────────
@@ -74,13 +64,14 @@
   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 collapse wrap vide post-show
-  const MAX_INSERTS_RUN   = 6;
-  const MAX_INFLIGHT      = 4;
-  const SHOW_THROTTLE_MS  = 900;
-  const BURST_COOLDOWN_MS = 200;
+  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
 
-  // IO : marges larges fixes — une seule instance, jamais recréée
+  // 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';
 
@@ -93,13 +84,12 @@
   /**
    * Table KIND — source de vérité par kindClass.
    *
-   * 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)
+   * 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 (catégories)
    */
   const KIND = {
     'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index' },
@@ -107,21 +97,23 @@
     'ezoic-ad-categories': { sel: SEL.category, baseTag: 'li', anchorAttr: 'data-cid',   ordinalAttr: null         },
   };
 
-  // ── État ───────────────────────────────────────────────────────────────────
+  // ── É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,
-    mutGuard:     0,
-    inflight:     0,
-    pending:      [],
+    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,
@@ -130,11 +122,12 @@
   };
 
   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 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++;
@@ -162,9 +155,11 @@
   }
 
   function initPools(cfg) {
+    if (S.poolsReady) return;
     S.pools.topics     = parseIds(cfg.placeholderIds);
     S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
     S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
+    S.poolsReady = true;
   }
 
   // ── Page identity ──────────────────────────────────────────────────────────
@@ -204,15 +199,39 @@
   const getTopics     = () => Array.from(document.querySelectorAll(SEL.topic));
   const getCategories = () => Array.from(document.querySelectorAll(SEL.category));
 
+  // ── 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 : wrap déplacé (moveDistantWrap) ou registre pas encore à jour.
     const colonIdx = key.indexOf(':');
-    const klass = key.slice(0, colonIdx);
+    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}"]`)) {
+            return sib.isConnected;
+          }
+        } catch (_) {}
+      }
+    }
+    // Dernier recours : querySelector global
     try {
       const found = document.querySelector(`${cfg.sel}[${cfg.anchorAttr}="${anchorId}"]`);
       return !!(found?.isConnected);
@@ -225,6 +244,10 @@
 
   // ── 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) {
@@ -242,17 +265,19 @@
   const anchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
 
   function findWrap(key) {
-    try {
-      return document.querySelector(
-        `.${WRAP_CLASS}[${A_ANCHOR}="${key.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
-      );
-    } catch (_) { return null; }
+    const w = S.wrapByKey.get(key);
+    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;
     for (let t = 0; t < pool.length; t++) {
       const i = S.cursors[poolKey] % pool.length;
       S.cursors[poolKey] = (S.cursors[poolKey] + 1) % pool.length;
@@ -262,13 +287,19 @@
     return null;
   }
 
-  function recycleDistantWrap(klass) {
-    // Quand le pool est épuisé : récupère le wrap le plus loin au-dessus du
-    // viewport (qu'il soit orphelin ou non) pour libérer son ID.
-    // Les topics/pubs à 4+ viewports au-dessus ne sont plus visibles — on peut
-    // recycler leur slot. Non-remplis en priorité, remplis si rien d'autre.
-    const vh = window.innerHeight || 800;
-    const threshold = -vh * 4;
+  /**
+   * Pool épuisé : déplace physiquement le wrap le plus loin au-dessus du
+   * viewport vers targetEl. On NE supprime PAS le placeholder — Ezoic garde
+   * une registry interne, et recréer un div avec le même id déclenche
+   * "already been defined" puis bloque showAds(). En déplaçant le même nœud,
+   * la registry reste valide. On appelle ensuite ezstandalone.refresh([id])
+   * pour forcer Ezoic à réactiver la pub sur sa nouvelle position.
+   *
+   * Priorité : wraps non remplis (pas de pub visible perdue), puis remplis.
+   */
+  function moveDistantWrap(klass, targetEl, newKey) {
+    const vh        = window.innerHeight || 800;
+    const threshold = -vh * 4;  // 4 viewports au-dessus = hors vue assurée
 
     let bestEmpty = null, bestEmptyBottom = Infinity;
     let bestFilled = null, bestFilledBottom = Infinity;
@@ -287,20 +318,44 @@
 
     const best = bestEmpty ?? bestFilled;
     if (!best) return null;
+
     const id = parseInt(best.getAttribute(A_WRAPID), 10);
     if (!Number.isFinite(id)) return null;
-    mutate(() => dropWrap(best));
-    return id;
+
+    const oldKey = best.getAttribute(A_ANCHOR);
+    mutate(() => {
+      best.setAttribute(A_ANCHOR,  newKey);
+      best.setAttribute(A_CREATED, String(ts()));
+      best.setAttribute(A_SHOWN,   '0');
+      best.classList.remove('is-empty');
+      targetEl.insertAdjacentElement('afterend', best);
+    });
+    // Mettre à jour le registre : ancienne clé → nouvelle clé
+    if (oldKey && S.wrapByKey.get(oldKey) === best) S.wrapByKey.delete(oldKey);
+    S.wrapByKey.set(newKey, best);
+
+    // Signaler à Ezoic que ce placeholder a bougé dans le DOM
+    try {
+      const ez = window.ezstandalone;
+      if (typeof ez?.refresh === 'function') {
+        (Array.isArray(ez.cmd) ? p => ez.cmd.push(p) : p => p())(() => {
+          try { ez.refresh([id]); } catch (_) {}
+        });
+      }
+    } catch (_) {}
+
+    return { id, wrap: best };
   }
 
-  // ── Wraps DOM ──────────────────────────────────────────────────────────────
+  // ── Wraps DOM — création / suppression ────────────────────────────────────
 
   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));
     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}`;
@@ -310,13 +365,14 @@
   }
 
   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;
+    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);
+    S.wrapByKey.set(key, w);
     return w;
   }
 
@@ -326,57 +382,49 @@
       if (ph instanceof Element) S.io?.unobserve(ph);
       const id = parseInt(w.getAttribute(A_WRAPID), 10);
       if (Number.isFinite(id)) S.mountedIds.delete(id);
+      const key = w.getAttribute(A_ANCHOR);
+      if (key && S.wrapByKey.get(key) === w) S.wrapByKey.delete(key);
       w.remove();
     } catch (_) {}
   }
 
   // ── Prune (topics de catégorie uniquement) ────────────────────────────────
   //
-  // pruneOrphans est réactivé UNIQUEMENT pour 'ezoic-ad-between'.
-  //
-  // Pourquoi safe pour les topics ?
-  //   NodeBB ne virtualise PAS la liste des topics dans une catégorie.
-  //   Les <li component="category/topic"> restent dans le DOM pendant toute
-  //   la session. Leurs ancres (data-tid) sont donc stables — un wrap orphelin
-  //   signifie vraiment que le topic a été retiré (navigation, filtre, etc.).
+  // Réactivé uniquement pour 'ezoic-ad-between' (liste de topics).
   //
-  // Pourquoi désactivé pour les posts ?
-  //   NodeBB virtualise les posts hors-viewport : il retire les <li> du DOM
-  //   puis les réinsère au scroll retour. pruneOrphans verait des ancres
-  //   absentes → supprimerait les wraps → réinjection en haut au scroll retour.
+  // 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.
   //
-  // MIN_PRUNE_AGE_MS : délai de grâce après création (stabilisation du DOM).
-
-  const MIN_PRUNE_AGE_MS = 8_000;
+  // 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 => {
-      // Délai de grâce : ne pas pruner un wrap trop récent
       const created = parseInt(w.getAttribute(A_CREATED) || '0', 10);
-      if (ts() - created < MIN_PRUNE_AGE_MS) return;
+      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); // partie après "ezoic-ad-between:"
+      const sid = key.slice(klass.length + 1); // après "ezoic-ad-between:"
       if (!sid) { mutate(() => dropWrap(w)); return; }
 
-      // Chercher l'ancre par data-tid
       const anchorEl = document.querySelector(`${cfg.baseTag}[${cfg.anchorAttr}="${sid}"]`);
-      if (!anchorEl || !anchorEl.isConnected) {
-        mutate(() => dropWrap(w));
-      }
+      if (!anchorEl?.isConnected) mutate(() => dropWrap(w));
     });
   }
 
-
   // ── Injection ──────────────────────────────────────────────────────────────
 
   /**
-   * Ordinal 0-based pour le calcul de l'intervalle.
-   * Utilise KIND[klass].ordinalAttr (data-index pour posts/topics).
-   * Catégories : ordinalAttr = null → fallback positionnel.
+   * 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;
@@ -408,11 +456,18 @@
       const key = anchorKey(klass, el);
       if (findWrap(key)) continue;
 
-      let id = pickId(poolKey);
-      if (!id) { id = recycleDistantWrap(klass); if (!id) continue; }
-
-      const w = insertAfter(el, id, klass, key);
-      if (w) { observePh(id); inserted++; }
+      const id = pickId(poolKey);
+      if (id) {
+        // Pool disponible : créer un nouveau wrap
+        const w = insertAfter(el, id, klass, key);
+        if (w) { observePh(id); inserted++; }
+      } else {
+        // Pool épuisé : déplacer un wrap distant (préserve le nœud placeholder)
+        const moved = moveDistantWrap(klass, el, key);
+        if (!moved) continue;
+        observePh(moved.id);
+        inserted++;
+      }
     }
     return inserted;
   }
@@ -507,6 +562,10 @@
   }
 
   // ── Patch Ezoic showAds ────────────────────────────────────────────────────
+  //
+  // Intercepte ez.showAds() pour :
+  //   – ignorer les appels pendant blockedUntil
+  //   – filtrer les ids dont le placeholder n'est pas en DOM
 
   function patchShowAds() {
     const apply = () => {
@@ -553,21 +612,22 @@
     const exec = (klass, getItems, cfgEnable, cfgInterval, cfgShowFirst, poolKey) => {
       if (!normBool(cfgEnable)) return 0;
       const interval = Math.max(1, parseInt(cfgInterval, 10) || 3);
-      const n = injectBetween(klass, getItems(), interval, normBool(cfgShowFirst), poolKey);
-      return n;
+      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') {
-      pruneOrphansBetween(); // nettoie les wraps dont le topic a disparu du DOM
+      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'
@@ -619,10 +679,12 @@
     blockedUntil = ts() + 1500;
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     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.inflight     = 0;
     S.pending      = [];
     S.pendingSet.clear();
@@ -640,7 +702,9 @@
       for (const m of muts) {
         for (const n of m.addedNodes) {
           if (n.nodeType !== 1) continue;
-          if (allSel.some(s => n.matches?.(s) || n.querySelector?.(s))) {
+          // 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;
           }
         }
@@ -654,7 +718,12 @@
   function muteConsole() {
     if (window.__nbbEzMuted) return;
     window.__nbbEzMuted = true;
-    const MUTED = ['[EzoicAds JS]: Placeholder Id', 'Debugger iframe already exists', `with id ${PH_PREFIX}`];
+    const MUTED = [
+      '[EzoicAds JS]: Placeholder Id',
+      'No valid placeholders for loadMore',
+      'Debugger iframe already exists',
+      `with id ${PH_PREFIX}`,
+    ];
     for (const m of ['log', 'info', 'warn', 'error']) {
       const orig = console[m];
       if (typeof orig !== 'function') continue;