nodebb-plugin-ezoic-infinite 1.7.87 → 1.7.89

package/package.json

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

package/public/client.js

@@ -77,9 +77,7 @@
   const A_CREATED   = 'data-ezoic-created';  // timestamp création ms
   const A_SHOWN     = 'data-ezoic-shown';    // timestamp dernier showAds ms
 
-  // anti-race fill async (Ezoic peut remplir bien après showAds)
-  const EMPTY_CHECK_PASSES = [20_000, 25_000, 35_000];
-
+  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
@@ -122,8 +120,6 @@
     cursors:      { topics: 0,  posts: 0,  categories: 0  },
     mountedIds:   new Set(),
     lastShow:     new Map(),
-    emptyChecks:  new Map(),   // id -> [timerIds] checks is-empty multi-pass
-    fillObs:      new Map(),   // id -> MutationObserver placeholder fill tardif
     io:           null,
     domObs:       null,
     mutGuard:     0,     // >0 : mutations DOM en cours (MutationObserver ignoré)
@@ -146,48 +142,6 @@
   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 clearEmptyChecks(id) {
-    const arr = S.emptyChecks.get(id);
-    if (arr) {
-      for (const t of arr) clearTimeout(t);
-      S.emptyChecks.delete(id);
-    }
-  }
-
-  function queueEmptyCheck(id, timerId) {
-    const arr = S.emptyChecks.get(id) || [];
-    arr.push(timerId);
-    S.emptyChecks.set(id, arr);
-  }
-
-  function uncollapseIfFilled(ph) {
-    try {
-      const wrap = ph?.closest?.(`.${WRAP_CLASS}`);
-      if (!wrap) return;
-      if (isFilled(ph)) wrap.classList.remove('is-empty');
-    } catch (_) {}
-  }
-
-  function watchPlaceholderFill(id) {
-    try {
-      const ph = document.getElementById(`${PH_PREFIX}${id}`);
-      if (!ph?.isConnected) return;
-      if (S.fillObs.has(id)) return;
-      const obs = new MutationObserver(() => uncollapseIfFilled(ph));
-      obs.observe(ph, { childList: true, subtree: true, attributes: true });
-      S.fillObs.set(id, obs);
-      uncollapseIfFilled(ph);
-    } catch (_) {}
-  }
-
-  function unwatchPlaceholderFill(id) {
-    const obs = S.fillObs.get(id);
-    if (obs) {
-      try { obs.disconnect(); } catch (_) {}
-      S.fillObs.delete(id);
-    }
-  }
-
   function mutate(fn) {
     S.mutGuard++;
     try { fn(); } finally { S.mutGuard--; }
@@ -328,6 +282,28 @@
     return (w?.isConnected) ? w : null;
   }
 
+  /**
+   * Libère les ids de wraps supprimés du DOM (virtualisation NodeBB / rerender).
+   * Sans ce sweep, mountedIds peut conserver des ids fantômes → pool épuisé
+   * après long scroll alors qu'aucun wrap recyclable n'existe encore en DOM.
+   */
+  function sweepDeadWraps() {
+    for (const [key, w] of S.wrapByKey.entries()) {
+      if (w?.isConnected) continue;
+      const id = parseInt(w?.getAttribute?.(A_WRAPID), 10);
+      if (Number.isFinite(id)) {
+        S.mountedIds.delete(id);
+        S.lastShow.delete(id);
+        S.pendingSet.delete(id);
+      }
+      S.wrapByKey.delete(key);
+    }
+    if (S.pending.length) {
+      S.pending = S.pending.filter(id => !S.pendingSet.has(id) || document.getElementById(`${PH_PREFIX}${id}`)?.isConnected);
+      S.pendingSet = new Set(S.pending);
+    }
+  }
+
   // ── Pool ───────────────────────────────────────────────────────────────────
 
   /**
@@ -387,7 +363,6 @@
     // Neutraliser l'IO sur ce wrap avant déplacement — évite un showAds
     // parasite si le nœud était encore dans la zone IO_MARGIN.
     try { const ph = best.querySelector(`#${PH_PREFIX}${id}`); if (ph) S.io?.unobserve(ph); } catch (_) {}
-    if (Number.isFinite(id)) clearEmptyChecks(id);
     mutate(() => {
       best.setAttribute(A_ANCHOR,  newKey);
       best.setAttribute(A_CREATED, String(ts()));
@@ -399,7 +374,6 @@
     });
     if (oldKey && S.wrapByKey.get(oldKey) === best) S.wrapByKey.delete(oldKey);
     S.wrapByKey.set(newKey, best);
-    observePh(id);
 
     // Délais requis : destroyPlaceholders est asynchrone en interne
     const doDestroy = () => { try { ez.destroyPlaceholders([id]); } catch (_) {} setTimeout(doDefine,  300); };
@@ -444,7 +418,6 @@
       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)) { clearEmptyChecks(id); unwatchPlaceholderFill(id); }
       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);
@@ -520,7 +493,13 @@
       const key = anchorKey(klass, el);
       if (findWrap(key)) continue;
 
-      const id = pickId(poolKey);
+      let id = pickId(poolKey);
+      if (!id) {
+        // Réessaie après sweep : des wraps ont pu être retirés du DOM (virtualisation)
+        // sans passer par dropWrap, laissant des ids fantômes dans mountedIds.
+        sweepDeadWraps();
+        id = pickId(poolKey);
+      }
       if (id) {
         const w = insertAfter(el, id, klass, key);
         if (w) { observePh(id); inserted++; }
@@ -552,10 +531,7 @@
 
   function observePh(id) {
     const ph = document.getElementById(`${PH_PREFIX}${id}`);
-    if (ph?.isConnected) {
-      try { getIO()?.observe(ph); } catch (_) {}
-      watchPlaceholderFill(id);
-    }
+    if (ph?.isConnected) try { getIO()?.observe(ph); } catch (_) {}
   }
 
   function enqueueShow(id) {
@@ -595,9 +571,6 @@
         const ph = document.getElementById(`${PH_PREFIX}${id}`);
         if (!ph?.isConnected || isFilled(ph)) { clearTimeout(timer); return release(); }
 
-        clearEmptyChecks(id);
-        try { ph.closest?.(`.${WRAP_CLASS}`)?.classList.remove('is-empty'); } catch (_) {}
-
         const t = ts();
         if (t - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) { clearTimeout(timer); return release(); }
         S.lastShow.set(id, t);
@@ -617,24 +590,15 @@
   }
 
   function scheduleEmptyCheck(id, showTs) {
-    clearEmptyChecks(id);
-
-    const runCheck = () => {
+    setTimeout(() => {
       try {
         const ph   = document.getElementById(`${PH_PREFIX}${id}`);
         const wrap = ph?.closest?.(`.${WRAP_CLASS}`);
         if (!wrap || !ph?.isConnected) return;
         if (parseInt(wrap.getAttribute(A_SHOWN) || '0', 10) > showTs) return;
-        const filled = isFilled(ph);
-        if (filled) wrap.classList.remove('is-empty');
-        else wrap.classList.add('is-empty');
+        wrap.classList.toggle('is-empty', !isFilled(ph));
       } catch (_) {}
-    };
-
-    for (const delay of EMPTY_CHECK_PASSES) {
-      const tid = setTimeout(runCheck, delay);
-      queueEmptyCheck(id, tid);
-    }
+    }, EMPTY_CHECK_MS);
   }
 
   // ── Patch Ezoic showAds ────────────────────────────────────────────────────
@@ -677,6 +641,7 @@
   async function runCore() {
     if (isBlocked()) return 0;
     patchShowAds();
+    sweepDeadWraps();
 
     const cfg = await fetchConfig();
     if (!cfg || cfg.excluded) return 0;
@@ -764,10 +729,6 @@
     S.inflight     = 0;
     S.pending      = [];
     S.pendingSet.clear();
-    S.emptyChecks.forEach(arr => { try { arr.forEach(clearTimeout); } catch (_) {} });
-    S.emptyChecks.clear();
-    S.fillObs.forEach(obs => { try { obs.disconnect(); } catch (_) {} });
-    S.fillObs.clear();
     S.burstActive  = false;
     S.runQueued    = false;
   }
@@ -779,16 +740,26 @@
     const allSel = [SEL.post, SEL.topic, SEL.category];
     S.domObs = new MutationObserver(muts => {
       if (S.mutGuard > 0 || isBlocked()) return;
+      let sawRelevantAdd = false;
+      let sawWrapRemoval  = false;
       for (const m of muts) {
+        for (const n of m.removedNodes) {
+          if (n.nodeType !== 1) continue;
+          if ((n.matches && n.matches(`.${WRAP_CLASS}`)) || n.querySelector?.(`.${WRAP_CLASS}`)) {
+            sawWrapRemoval = true;
+          }
+        }
         for (const n of m.addedNodes) {
           if (n.nodeType !== 1) continue;
           // 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;
+            sawRelevantAdd = true;
           }
         }
       }
+      if (sawWrapRemoval) sweepDeadWraps();
+      if (sawRelevantAdd) requestBurst();
     });
     try { S.domObs.observe(document.body, { childList: true, subtree: true }); } catch (_) {}
   }

package/public/style.css

@@ -71,14 +71,6 @@
   overflow: hidden !important;
 }
 
-/* Filet de sécurité : si un fill est présent malgré is-empty, on ne collapse pas */
-.nodebb-ezoic-wrap.is-empty:has(iframe, ins, img, video, [data-google-container-id]) {
-  height: auto !important;
-  min-height: 1px !important;
-  max-height: none !important;
-  overflow: visible !important;
-}
-
 /* ── Ezoic global (hors de nos wraps) ────────────────────────────────────── */
 .ezoic-ad {
   margin: 0 !important;