nodebb-plugin-ezoic-infinite 1.7.7 → 1.7.9

package/package.json

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

package/public/client.js

@@ -1,5 +1,5 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v21.3
+ * NodeBB Ezoic Infinite Ads — client.js v21
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -96,7 +96,6 @@
   };
 
   let blockedUntil = 0;
-  let poolsReady   = false;  // true dès que les pools sont initialisés pour la page courante
   const ts        = () => Date.now();
   const isBlocked = () => ts() < blockedUntil;
   const isMobile  = () => { try { return window.innerWidth < 768; } catch (_) { return false; } };
@@ -129,11 +128,9 @@
   }
 
   function initPools(cfg) {
-    if (poolsReady) return;
     S.pools.topics     = parseIds(cfg.placeholderIds);
     S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
     S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
-    poolsReady = true;
   }
 
   // ── Page identity ──────────────────────────────────────────────────────────
@@ -182,9 +179,6 @@
 
   // ── Ancres stables ─────────────────────────────────────────────────────────
 
-  // Map anchorKey → wrap Element — évite un querySelector full-DOM à chaque injection
-  const wrapByKey = new Map();
-
   function stableId(klass, el) {
     const attr = KIND[klass]?.anchorAttr;
     if (attr) {
@@ -201,13 +195,13 @@
 
   const anchorKey = (klass, el) => `${klass}:${stableId(klass, el)}`;
 
-  const findWrap  = (key) => {
-    const w = wrapByKey.get(key);
-    // Vérifier que le wrap est toujours dans le DOM (il peut avoir été dropWrap'd)
-    if (w && w.isConnected) return w;
-    if (w) wrapByKey.delete(key); // nettoyage lazy
-    return null;
-  };
+  function findWrap(key) {
+    try {
+      return document.querySelector(
+        `.${WRAP_CLASS}[${A_ANCHOR}="${key.replace(/\\/g, '\\\\').replace(/"/g, '\\"')}"]`
+      );
+    } catch (_) { return null; }
+  }
 
   // ── Pool ───────────────────────────────────────────────────────────────────
 
@@ -246,20 +240,17 @@
     const w = makeWrap(id, klass, key);
     mutate(() => el.insertAdjacentElement('afterend', w));
     S.mountedIds.add(id);
-    wrapByKey.set(key, w);
     return w;
   }
 
   function dropWrap(w) {
     try {
-      // Unobserve uniquement si le ph est encore connecté — un nœud détaché
-      // passé à unobserve() corrompt l'IO interne de pubads.
+      // 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 && ph.isConnected) S.io?.unobserve(ph);
+      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) wrapByKey.delete(key);
       w.remove();
     } catch (_) {}
   }
@@ -387,7 +378,7 @@
 
   function observePh(id) {
     const ph = document.getElementById(`${PH_PREFIX}${id}`);
-    if (ph instanceof Element && ph.isConnected) try { getIO()?.observe(ph); } catch (_) {}
+    if (ph?.isConnected) try { getIO()?.observe(ph); } catch (_) {}
   }
 
   function enqueueShow(id) {
@@ -492,6 +483,7 @@
 
   async function runCore() {
     if (isBlocked()) return 0;
+    patchShowAds();
 
     const cfg = await fetchConfig();
     if (!cfg || cfg.excluded) return 0;
@@ -566,32 +558,7 @@
 
   function cleanup() {
     blockedUntil = ts() + 1500;
-    poolsReady   = false;
-
-    // Unobserve ciblé sur NOS placeholders avant de les retirer du DOM.
-    // On ne fait JAMAIS disconnect() — c'est un IO global partagé avec pubads ;
-    // disconnect() corrompt son état interne et provoque des erreurs sur tous
-    // les observe() suivants de Google ("parameter 1 is not of type 'Element'").
-    if (S.io) {
-      for (const id of S.mountedIds) {
-        try {
-          const ph = document.getElementById(`${PH_PREFIX}${id}`);
-          if (ph instanceof Element && ph.isConnected) S.io.unobserve(ph);
-        } catch (_) {}
-      }
-    }
-
-    mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(w => {
-      try {
-        const id = parseInt(w.getAttribute(A_WRAPID), 10);
-        if (Number.isFinite(id)) S.mountedIds.delete(id);
-        const key = w.getAttribute(A_ANCHOR);
-        if (key) wrapByKey.delete(key);
-        w.remove();
-      } catch (_) {}
-    }));
-
-    wrapByKey.clear();
+    mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };
     S.cursors      = { topics: 0, posts: 0, categories: 0 };
@@ -610,8 +577,6 @@
     if (S.domObs) return;
     const allSel = [SEL.post, SEL.topic, SEL.category];
     S.domObs = new MutationObserver(muts => {
-      // Ne rien faire pendant la navigation (cleanup posé blockedUntil)
-      // ou si c'est nous qui mutons le DOM.
       if (S.mutGuard > 0 || isBlocked()) return;
       for (const m of muts) {
         for (const n of m.addedNodes) {