nodebb-plugin-ezoic-infinite 1.7.17 → 1.7.19

package/library.js

@@ -9,10 +9,8 @@ const plugin = {};
 
 function normalizeExcludedGroups(value) {
   if (!value) return [];
-  const arr = Array.isArray(value) ? value : String(value).split(',');
-  return arr
-    .map(s => String(s).trim())
-    .filter(Boolean);
+  if (Array.isArray(value)) return value;
+  return String(value).split(',').map(s => s.trim()).filter(Boolean);
 }
 
 function parseBool(v, def = false) {
@@ -22,30 +20,18 @@ function parseBool(v, def = false) {
   return s === '1' || s === 'true' || s === 'on' || s === 'yes';
 }
 
-let _groupsCache = null;
-let _groupsCacheAt = 0;
-const GROUPS_TTL = 60000; // 60s
-
 async function getAllGroups() {
-  const now = Date.now();
-  if (_groupsCache && (now - _groupsCacheAt) < GROUPS_TTL) return _groupsCache;
-
   let names = await db.getSortedSetRange('groups:createtime', 0, -1);
   if (!names || !names.length) {
     names = await db.getSortedSetRange('groups:visible:createtime', 0, -1);
   }
   const filtered = names.filter(name => !groups.isPrivilegeGroup(name));
   const data = await groups.getGroupsData(filtered);
-
   // Filter out nulls (groups deleted between the sorted-set read and getGroupsData)
-  const valid = (data || []).filter(g => g && g.name);
+  const valid = data.filter(g => g && g.name);
   valid.sort((a, b) => String(a.name).localeCompare(String(b.name), undefined, { sensitivity: 'base' }));
-
-  _groupsCache = valid;
-  _groupsCacheAt = now;
-  return _groupsCache;
+  return valid;
 }
-
 let _settingsCache = null;
 let _settingsCacheAt = 0;
 const SETTINGS_TTL = 30000; // 30s
@@ -53,13 +39,8 @@ const SETTINGS_TTL = 30000; // 30s
 async function getSettings() {
   const now = Date.now();
   if (_settingsCache && (now - _settingsCacheAt) < SETTINGS_TTL) return _settingsCache;
-  let s = {};
-  try {
-    s = await meta.settings.get(SETTINGS_KEY);
-  } catch (err) {
-    s = {};
-  }
-  _settingsCacheAt = now;
+  const s = await meta.settings.get(SETTINGS_KEY);
+  _settingsCacheAt = Date.now();
   _settingsCache = {
     // Between-post ads (simple blocks) in category topic list
     enableBetweenAds: parseBool(s.enableBetweenAds, true),
@@ -79,9 +60,6 @@ async function getSettings() {
     messagePlaceholderIds: (s.messagePlaceholderIds || '').trim(),
     messageIntervalPosts: Math.max(1, parseInt(s.messageIntervalPosts, 10) || 3),
 
-    // Avoid globally muting console unless explicitly enabled
-    muteEzoicConsole: parseBool(s.muteEzoicConsole, false),
-
     excludedGroups: normalizeExcludedGroups(s.excludedGroups),
   };
   return _settingsCache;
@@ -90,8 +68,7 @@ async function getSettings() {
 async function isUserExcluded(uid, excludedGroups) {
   if (!uid || !excludedGroups.length) return false;
   const userGroups = await groups.getUserGroups([uid]);
-  const excluded = new Set(excludedGroups.map(n => String(n).toLowerCase().trim()).filter(Boolean));
-  return (userGroups[0] || []).some(g => excluded.has(String(g.name).toLowerCase().trim()));
+  return (userGroups[0] || []).some(g => excludedGroups.includes(g.name));
 }
 
 plugin.onSettingsSet = function (data) {
@@ -116,53 +93,21 @@ plugin.init = async ({ router, middleware }) => {
     const settings = await getSettings();
     const allGroups = await getAllGroups();
 
-    const excludedSet = new Set((settings.excludedGroups || []).map(n => String(n).toLowerCase().trim()).filter(Boolean));
-    const allGroupsWithSelected = (allGroups || []).map(g => ({
-      ...g,
-      selected: excludedSet.has(String(g.name).toLowerCase().trim()) ? 'selected' : '',
-    }));
-
     res.render('admin/plugins/ezoic-infinite', {
       title: 'Ezoic Infinite Ads',
       ...settings,
-
-      // SSR-friendly checkbox states
       enableBetweenAds_checked: settings.enableBetweenAds ? 'checked' : '',
-      showFirstTopicAd_checked: settings.showFirstTopicAd ? 'checked' : '',
-
-      enableCategoryAds_checked: settings.enableCategoryAds ? 'checked' : '',
-      showFirstCategoryAd_checked: settings.showFirstCategoryAd ? 'checked' : '',
-
       enableMessageAds_checked: settings.enableMessageAds ? 'checked' : '',
-      showFirstMessageAd_checked: settings.showFirstMessageAd ? 'checked' : '',
-
-      muteEzoicConsole_checked: settings.muteEzoicConsole ? 'checked' : '',
-
-      allGroups: allGroupsWithSelected,
+      allGroups,
     });
   }
 
-  router.get('/admin/plugins/ezoic-infinite', middleware.ensureLoggedIn, middleware.admin.checkPrivileges, middleware.admin.buildHeader, render);
-  router.get('/api/admin/plugins/ezoic-infinite', middleware.ensureLoggedIn, middleware.admin.checkPrivileges, render);
+  router.get('/admin/plugins/ezoic-infinite', middleware.admin.buildHeader, render);
+  router.get('/api/admin/plugins/ezoic-infinite', render);
 
   router.get('/api/plugins/ezoic-infinite/config', async (req, res) => {
-    let settings;
-    try {
-      settings = await getSettings();
-    } catch (err) {
-      settings = null;
-    }
-
-    if (!settings) {
-      return res.json({ excluded: false });
-    }
-
-    let excluded = false;
-    try {
-      excluded = await isUserExcluded(req.uid, settings.excludedGroups);
-    } catch (err) {
-      excluded = false;
-    }
+    const settings = await getSettings();
+    const excluded = await isUserExcluded(req.uid, settings.excludedGroups);
 
     res.json({
       excluded,
@@ -178,7 +123,6 @@ plugin.init = async ({ router, middleware }) => {
       showFirstMessageAd: settings.showFirstMessageAd,
       messagePlaceholderIds: settings.messagePlaceholderIds,
       messageIntervalPosts: settings.messageIntervalPosts,
-      muteEzoicConsole: settings.muteEzoicConsole,
     });
   });
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.17",
+  "version": "1.7.19",
   "description": "Production-ready Ezoic infinite ads integration for NodeBB 4.x",
   "main": "library.js",
   "license": "MIT",
@@ -12,19 +12,10 @@
     "infinite-scroll"
   ],
   "engines": {
-    "nodebb": ">=4.0.0",
-    "node": ">=18"
+    "nodebb": ">=4.0.0"
   },
   "nbbpm": {
     "compatibility": "^4.0.0"
   },
-  "private": false,
-  "files": [
-    "library.js",
-    "plugin.json",
-    "public/"
-  ],
-  "scripts": {
-    "lint": "node -c library.js && node -c public/admin.js && node -c public/client.js"
-  }
-}
+  "private": false
+}

package/public/client.js

@@ -1,55 +1,55 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js (v20)
+ * NodeBB Ezoic Infinite Ads — client.js v25
  *
- * 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 (moveWrapAfter). 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.
- *
- * [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é)
+ * v25  Base v20.1 avec :
+ *      • 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).
+ *      • Recyclage d'id (pool épuisé en infinite scroll) :
+ *        – pickRecyclableWrap() : sélectionne le wrap vide le plus loin au-dessus
+ *          du viewport (seuil -6 × vh), jamais pour ezoic-ad-message.
+ *        – moveWrapAfter() : déplace le wrap vers sa nouvelle ancre.
+ *        – scrollDir tracking pour n'autoriser le recyclage qu'en scroll down.
+ *      • Table KIND unifiée avec baseTag + ordinalAttr + recyclable flag.
+ *      • ordinal() : utilise KIND[klass].ordinalAttr, fallback positionnel propre.
  */
 (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 RECYCLE_THRESHOLD = 6;       // nb de viewports au-dessus du seuil de recyclage
+  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,51 +60,54 @@
   };
 
   /**
-   * 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)
+   * recyclable : autoriser le recyclage d'id quand le pool est épuisé
+   *              false pour ezoic-ad-message (sauts visuels indésirables)
    */
   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', recyclable: false },
+    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-index', ordinalAttr: 'data-index', recyclable: true  },
+    'ezoic-ad-categories': { sel: SEL.category, baseTag: 'li', anchorAttr: 'data-cid',   ordinalAttr: null,         recyclable: true  },
   };
 
   // ── É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
-    reuseSeq:   new Map(),      // id → compteur pour générer des IDs DOM uniques
-    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,
     burstCount:   0,
     lastBurstTs:  0,
+    scrollDir:    1,    // 1 = down, -1 = up
+    lastScrollY:  0,
   };
 
   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++;
@@ -122,30 +125,20 @@
     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(/[\s,]+/).map(s => s.trim()).filter(Boolean)) {
+    for (const v of String(raw || '').split(/\r?\n/).map(s => s.trim()).filter(Boolean)) {
       const n = parseInt(v, 10);
       if (n > 0 && Number.isFinite(n) && !seen.has(n)) { seen.add(n); out.push(n); }
     }
     return out;
   }
 
-  // La réutilisation des IDs du pool est toujours autorisée pour éviter les
-  // situations "pool épuisé". Les IDs HTML des placeholders sont rendus uniques
-  // (suffixés) afin de ne pas dupliquer d'ID dans le DOM.
-  const allowReuse = () => true;
-
-  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 ──────────────────────────────────────────────────────────
 
@@ -169,13 +162,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';
     });
@@ -191,36 +184,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; }
   }
@@ -230,39 +215,78 @@
   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];
-      return id;
+      if (!S.mountedIds.has(id)) return id;
     }
     return null;
   }
 
-  // ── Wraps DOM ──────────────────────────────────────────────────────────────
+  // ── Recyclage d'id ─────────────────────────────────────────────────────────
+
+  /**
+   * Sélectionne le wrap vide le plus éloigné au-dessus du viewport.
+   * Conditions : kindClass.recyclable = true, scroll vers le bas,
+   * wrap vide (non filled), rect.bottom < -(RECYCLE_THRESHOLD × vh).
+   */
+  function pickRecyclableWrap(klass) {
+    if (!KIND[klass]?.recyclable) return null;
+    if (S.scrollDir < 0) return null;
+
+    const vh        = Math.max(300, window.innerHeight || 800);
+    const threshold = -(vh * RECYCLE_THRESHOLD);
+    let best = null, bestBottom = Infinity;
+
+    for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
+      if (!w.isConnected || isFilled(w)) continue;
+      try {
+        const rect = w.getBoundingClientRect();
+        if (rect.bottom < threshold && rect.bottom < bestBottom) {
+          bestBottom = rect.bottom;
+          best = w;
+        }
+      } catch (_) {}
+    }
+    return best;
+  }
 
-  function nextDomPlaceholderId(id) {
-    const cur = (S.reuseSeq.get(id) || 0) + 1;
-    S.reuseSeq.set(id, cur);
-    return `${PH_PREFIX}${id}-${cur}`;
+  /**
+   * Déplace un wrap recyclé vers sa nouvelle ancre el.
+   * Réinitialise A_ANCHOR, A_CREATED, supprime A_SHOWN.
+   */
+  function moveWrapAfter(el, wrap, newKey) {
+    try {
+      if (!el || !wrap?.isConnected) return null;
+      wrap.setAttribute(A_ANCHOR,  newKey);
+      wrap.setAttribute(A_CREATED, String(ts()));
+      wrap.removeAttribute(A_SHOWN);
+      mutate(() => el.insertAdjacentElement('afterend', wrap));
+      return wrap;
+    } catch (_) { return null; }
   }
 
+  // ── 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));
     w.setAttribute(A_CREATED, String(ts()));
     w.style.cssText = 'width:100%;display:block;';
     const ph = document.createElement('div');
-    ph.id = nextDomPlaceholderId(id);
+    ph.id = `${PH_PREFIX}${id}`;
     ph.setAttribute('data-ezoic-id', String(id));
     w.appendChild(ph);
     return w;
   }
 
   function insertAfter(el, id, klass, key) {
-    if (!el?.insertAdjacentElement) return null;
-    if (findWrap(key))            return null; // ancre déjà présente
+    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);
@@ -271,15 +295,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('[data-ezoic-id]') || w.querySelector(`[id^="${PH_PREFIX}"]`);
-        if (ph instanceof Element) S.io?.unobserve(ph);
-      } catch (_) {}
       w.remove();
     } catch (_) {}
   }
@@ -287,58 +306,59 @@
   // ── Prune ──────────────────────────────────────────────────────────────────
 
   /**
-   * Supprime les wraps dont l'élément-ancre n'est plus dans le DOM.
+   * Supprime les wraps VIDES dont l'ancre a disparu du 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
-   *
-   * 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;
       }
     }
@@ -348,23 +368,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;
   }
 
@@ -373,23 +391,33 @@
     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;
 
+      // 1. Tentative pool normal
       const id = pickId(poolKey);
-      if (!id) continue; // pool momentanément épuisé, on continue (items suivants ≠ besoin d'id)
+      if (id) {
+        const w = insertAfter(el, id, klass, key);
+        if (w) { observePh(id); inserted++; }
+        continue;
+      }
 
-      const w = insertAfter(el, id, klass, key);
-      if (w) { observePh(id); inserted++; }
+      // 2. Pool épuisé → tentative de recyclage
+      const recyclable = pickRecyclableWrap(klass);
+      if (recyclable) {
+        const rid = parseInt(recyclable.getAttribute(A_WRAPID), 10);
+        const w   = moveWrapAfter(el, recyclable, key);
+        if (w && Number.isFinite(rid)) { observePh(rid); inserted++; }
+      }
+      // Pool épuisé et pas de recyclage : on continue (items suivants peuvent
+      // avoir un wrap existant via findWrap, on ne break pas)
     }
     return inserted;
   }
@@ -398,7 +426,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) {
@@ -407,7 +434,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;
   }
@@ -458,7 +485,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 || {};
@@ -479,7 +505,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 (_) {}
@@ -498,7 +523,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);
@@ -517,7 +542,7 @@
     }
   }
 
-  // ── Core run ───────────────────────────────────────────────────────────────
+  // ── Core ───────────────────────────────────────────────────────────────────
 
   async function runCore() {
     if (isBlocked()) return 0;
@@ -532,10 +557,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;
     };
@@ -548,14 +572,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;
@@ -573,10 +596,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;
@@ -584,7 +605,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++;
@@ -596,14 +617,10 @@
     step();
   }
 
-  // ── Cleanup (navigation ajaxify) ───────────────────────────────────────────
+  // ── Cleanup navigation ─────────────────────────────────────────────────────
 
   function cleanup() {
     blockedUntil = ts() + 1500;
-
-    // Disconnect observers to avoid doing work while ajaxify swaps content
-    try { if (S.domObs) { S.domObs.disconnect(); S.domObs = null; } } catch (_) {}
-    try { if (window.__nbbTcfObs) { window.__nbbTcfObs.disconnect(); window.__nbbTcfObs = null; } } catch (_) {}
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };
@@ -617,19 +634,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;
           }
         }
@@ -641,7 +656,6 @@
   // ── Utilitaires ────────────────────────────────────────────────────────────
 
   function muteConsole() {
-    if (!S.cfg || !S.cfg.muteEzoicConsole) return;
     if (window.__nbbEzMuted) return;
     window.__nbbEzMuted = true;
     const MUTED = ['[EzoicAds JS]: Placeholder Id', 'Debugger iframe already exists', `with id ${PH_PREFIX}`];
@@ -656,29 +670,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 (_) {}
   }
@@ -688,10 +691,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],
     ]) {
@@ -705,7 +708,7 @@
     }
   }
 
-  // ── Bindings NodeBB ────────────────────────────────────────────────────────
+  // ── Bindings ───────────────────────────────────────────────────────────────
 
   function bindNodeBB() {
     const $ = window.jQuery;
@@ -716,19 +719,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;
@@ -743,12 +743,14 @@
   function bindScroll() {
     let ticking = false;
     window.addEventListener('scroll', () => {
-      if (ticking) return;
-      // Scroll bursts only matter on pages where content streams in
-      const kind = getKind();
-      if (kind !== 'topic' && kind !== 'categoryTopics' && kind !== 'categories') return;
-      if (isBlocked()) return;
+      // Suivi direction du scroll (nécessaire pour le recyclage conditionnel)
+      try {
+        const y = window.scrollY || window.pageYOffset || 0;
+        const d = y - S.lastScrollY;
+        if (Math.abs(d) > 4) { S.scrollDir = d > 0 ? 1 : -1; S.lastScrollY = y; }
+      } catch (_) {}
 
+      if (ticking) return;
       ticking = true;
       requestAnimationFrame(() => { ticking = false; requestBurst(); });
     }, { passive: true });
@@ -757,6 +759,7 @@
   // ── Boot ───────────────────────────────────────────────────────────────────
 
   S.pageKey = pageKey();
+  try { S.lastScrollY = window.scrollY || window.pageYOffset || 0; } catch (_) {}
   muteConsole();
   ensureTcfLocator();
   warmNetwork();

package/public/style.css

@@ -71,8 +71,8 @@
   overflow: hidden !important;
 }
 
-/* ── Note ───────────────────────────────────────────────────────────── */
-/*
-  On évite volontairement de cibler .ezoic-ad globalement,
-  pour ne pas impacter d'autres emplacements/optimisations Ezoic hors plugin.
-*/
+/* ── Ezoic global (hors de nos wraps) ────────────────────────────────────── */
+.ezoic-ad {
+  margin: 0 !important;
+  padding: 0 !important;
+}

package/public/templates/admin/plugins/ezoic-infinite.tpl

@@ -8,7 +8,7 @@
       <input class="form-check-input" type="checkbox" id="enableBetweenAds" name="enableBetweenAds" {enableBetweenAds_checked}>
       <label class="form-check-label" for="enableBetweenAds">Activer les pubs entre les posts</label>
 				<div class="form-check mt-2">
-					<input class="form-check-input" type="checkbox" name="showFirstTopicAd" {showFirstTopicAd_checked} />
+					<input class="form-check-input" type="checkbox" name="showFirstTopicAd" />
 					<label class="form-check-label">Afficher une pub après le 1er sujet</label>
 				</div>
     </div>
@@ -16,7 +16,7 @@
     <div class="mb-3">
       <label class="form-label" for="placeholderIds">Pool d’IDs Ezoic (entre posts)</label>
       <textarea id="placeholderIds" name="placeholderIds" class="form-control" rows="4">{placeholderIds}</textarea>
-      <p class="form-text">Un ID par ligne (ou séparé par virgules/espaces).</p>
+      <p class="form-text">Un ID par ligne (ou séparé par virgules/espaces). Le nombre d’IDs = nombre max de pubs simultanées.</p>
     </div>
 
     <div class="mb-3">
@@ -33,10 +33,10 @@
     <p class="form-text">Insère des pubs entre les catégories sur la page d’accueil (liste des catégories).</p>
 
     <div class="form-check mb-3">
-      <input class="form-check-input" type="checkbox" id="enableCategoryAds" name="enableCategoryAds" {enableCategoryAds_checked}>
+      <input class="form-check-input" type="checkbox" id="enableCategoryAds" name="enableCategoryAds">
       <label class="form-check-label" for="enableCategoryAds">Activer les pubs entre les catégories</label>
       <div class="form-check mt-2">
-        <input class="form-check-input" type="checkbox" name="showFirstCategoryAd" {showFirstCategoryAd_checked} />
+        <input class="form-check-input" type="checkbox" name="showFirstCategoryAd" />
         <label class="form-check-label">Afficher une pub après la 1ère catégorie</label>
       </div>
     </div>
@@ -44,7 +44,7 @@
     <div class="mb-3">
       <label class="form-label" for="categoryPlaceholderIds">Pool d’IDs Ezoic (catégories)</label>
       <textarea id="categoryPlaceholderIds" name="categoryPlaceholderIds" class="form-control" rows="4">{categoryPlaceholderIds}</textarea>
-      <p class="form-text">IDs numériques, un par ligne (ou séparés par virgules/espaces). Utilise un pool dédié (différent des pools topics/messages).</p>
+      <p class="form-text">IDs numériques, un par ligne. Utilise un pool dédié (différent des pools topics/messages).</p>
     </div>
 
     <div class="mb-3">
@@ -59,7 +59,7 @@
       <input class="form-check-input" type="checkbox" id="enableMessageAds" name="enableMessageAds" {enableMessageAds_checked}>
       <label class="form-check-label" for="enableMessageAds">Activer les pubs “message”</label>
 				<div class="form-check mt-2">
-					<input class="form-check-input" type="checkbox" name="showFirstMessageAd" {showFirstMessageAd_checked} />
+					<input class="form-check-input" type="checkbox" name="showFirstMessageAd" />
 					<label class="form-check-label">Afficher une pub après le 1er message</label>
 				</div>
     </div>
@@ -67,7 +67,7 @@
     <div class="mb-3">
       <label class="form-label" for="messagePlaceholderIds">Pool d’IDs Ezoic (message)</label>
       <textarea id="messagePlaceholderIds" name="messagePlaceholderIds" class="form-control" rows="4">{messagePlaceholderIds}</textarea>
-      <p class="form-text">Pool séparé recommandé pour limiter la réutilisation d’IDs entre emplacements.</p>
+      <p class="form-text">Pool séparé recommandé pour éviter la réutilisation d’IDs. IMPORTANT : ne réutilise pas les mêmes IDs dans les deux pools.</p>
     </div>
 
     <div class="mb-3">
@@ -82,22 +82,12 @@
       <label class="form-label" for="excludedGroups">Groupes exclus</label>
       <select id="excludedGroups" name="excludedGroups" class="form-select" multiple>
         <!-- BEGIN allGroups -->
-        <option value="{allGroups.name}" {allGroups.selected}>{allGroups.name}</option>
+        <option value="{allGroups.name}">{allGroups.name}</option>
         <!-- END allGroups -->
       </select>
       <p class="form-text">Si l’utilisateur appartient à un de ces groupes, aucune pub n’est injectée.</p>
     </div>
 
-    
-    <hr/>
-
-    <h4 class="mt-3">Options avancées</h4>
-
-    <div class="form-check mb-3">
-      <input class="form-check-input" type="checkbox" id="muteEzoicConsole" name="muteEzoicConsole" {muteEzoicConsole_checked}>
-      <label class="form-check-label" for="muteEzoicConsole">Filtrer certains logs Ezoic dans la console (évite le bruit)</label>
-      <p class="form-text">Désactivé par défaut pour ne pas impacter les logs d’autres plugins.</p>
-    </div>
     <button id="save" class="btn btn-primary">Enregistrer</button>
   </form>
 </div>