nodebb-plugin-ezoic-infinite 1.7.94 → 1.7.96

package/package.json

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

package/public/client.js

@@ -77,17 +77,17 @@
   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 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      = 2;       // max showAds() simultanés (réduit le churn/unused)
-  const SHOW_THROTTLE_MS  = 1200;    // anti-spam showAds() par id
-  const UNUSED_COOLDOWN_MS = 15_000; // cooldown après tentative sans fill (unused probable)
-  const PENDING_TIMEOUT_MS = 12_000; // au-delà, tentative considérée obsolète
-  const BURST_COOLDOWN_MS = 200;     // délai min entre deux déclenchements de burst
+  const MAX_INSERTS_RUN   = 10;      // max insertions par appel runCore (plus réactif)
+  const MAX_INFLIGHT      = 4;       // max showAds() simultanés
+  const SHOW_THROTTLE_MS  = 600;     // anti-spam showAds() par id (plus réactif)
+  const BURST_COOLDOWN_MS = 120;     // délai min entre deux déclenchements de burst
 
   // 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';
+  const FAST_SHOW_MARGIN_PX = 900; // show immédiat si slot déjà proche viewport
 
   const SEL = {
     post:     '[component="post"][data-pid]',
@@ -125,19 +125,16 @@
     domObs:       null,
     mutGuard:     0,     // >0 : mutations DOM en cours (MutationObserver ignoré)
     inflight:     0,     // showAds() en cours
-    pending:      [],    // ids en attente de slot inflight (triés par priorité)
+    pending:      [],    // ids en attente de slot inflight
     pendingSet:   new Set(),
-    pendingShows: new Set(),  // ids avec tentative show en cours
-    cooldownUntil:new Map(),  // id -> ts de fin de cooldown (unused probable)
-    fillObsById:  new Map(),  // id -> MutationObserver placeholder fill
     wrapByKey:    new Map(),  // anchorKey → wrap DOM node
     runQueued:    false,
     burstActive:  false,
     burstDeadline: 0,
     burstCount:   0,
     lastBurstTs:  0,
-    lastScrollY:  0,
     scrollDir:    1,
+    lastScrollY:  0,
   };
 
   let blockedUntil = 0;
@@ -147,69 +144,6 @@
   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 isCooling = id => (S.cooldownUntil.get(id) || 0) > ts();
-
-  function viewportDistance(el) {
-    try {
-      const r = el?.getBoundingClientRect?.();
-      if (!r) return Infinity;
-      const vh = window.innerHeight || 800;
-      if (r.bottom >= 0 && r.top <= vh) return 0;
-      if (r.top > vh) return Math.max(0, r.top - vh);
-      return Math.max(0, -r.bottom);
-    } catch (_) { return Infinity; }
-  }
-
-  function pendingEnqueue(id) {
-    if (S.pendingSet.has(id)) return;
-    const ph = document.getElementById(`${PH_PREFIX}${id}`);
-    const prio = viewportDistance(ph?.closest?.(`.${WRAP_CLASS}`) || ph);
-    const item = { id, prio, at: ts() };
-    let i = S.pending.length;
-    while (i > 0 && S.pending[i - 1].prio > prio) i--;
-    S.pending.splice(i, 0, item);
-    S.pendingSet.add(id);
-  }
-
-  function markUnusedProbable(id) {
-    if (!id) return;
-    S.pendingShows.delete(id);
-    S.cooldownUntil.set(id, ts() + UNUSED_COOLDOWN_MS);
-  }
-
-  function clearCooldownIfFilled(id) {
-    if (!id) return;
-    S.cooldownUntil.delete(id);
-    S.pendingShows.delete(id);
-  }
-
-  function unobserveFill(id) {
-    const obs = S.fillObsById.get(id);
-    if (obs) { try { obs.disconnect(); } catch (_) {} }
-    S.fillObsById.delete(id);
-  }
-
-  function watchPlaceholderFill(id) {
-    const ph = document.getElementById(`${PH_PREFIX}${id}`);
-    if (!ph?.isConnected) return;
-    unobserveFill(id);
-    const onMut = () => {
-      try {
-        if (!ph.isConnected) return unobserveFill(id);
-        if (!isFilled(ph)) return;
-        clearCooldownIfFilled(id);
-        const wrap = ph.closest?.(`.${WRAP_CLASS}`);
-        if (wrap) wrap.classList.remove('is-empty');
-      } catch (_) {}
-    };
-    try {
-      const obs = new MutationObserver(onMut);
-      obs.observe(ph, { childList: true, subtree: true, attributes: true });
-      S.fillObsById.set(id, obs);
-    } catch (_) {}
-    onMut();
-  }
-
 
   function mutate(fn) {
     S.mutGuard++;
@@ -351,21 +285,6 @@
     return (w?.isConnected) ? w : null;
   }
 
-  function sweepDeadWraps() {
-    for (const [key, w] of Array.from(S.wrapByKey.entries())) {
-      if (w?.isConnected) continue;
-      S.wrapByKey.delete(key);
-      const id = parseInt(w?.getAttribute?.(A_WRAPID), 10);
-      if (Number.isFinite(id)) {
-        S.mountedIds.delete(id);
-        S.pendingSet.delete(id);
-        S.pending = S.pending.filter(it => (it?.id ?? it) !== id);
-        S.pendingShows.delete(id);
-        unobserveFill(id);
-      }
-    }
-  }
-
   // ── Pool ───────────────────────────────────────────────────────────────────
 
   /**
@@ -397,40 +316,59 @@
         typeof ez?.define             !== 'function' ||
         typeof ez?.displayMore        !== 'function') return null;
 
-    const vh = window.innerHeight || 800;
-    const targetRect = targetEl?.getBoundingClientRect?.();
-    const dir = S.scrollDir >= 0 ? 1 : -1;
+    const vh   = window.innerHeight || 800;
+    const dir  = S.scrollDir >= 0 ? 1 : -1;
+    const farTopThreshold    = -vh;     // hors écran au-dessus
+    const farBottomThreshold = vh * 2;  // loin sous le viewport (scroll-up)
 
-    let best = null, bestScore = Infinity;
-    let fallback = null, fallbackScore = Infinity;
+    let prefEmpty = null, prefEmptyScore = -Infinity;
+    let prefFilled = null, prefFilledScore = -Infinity;
+    let altEmpty = null, altEmptyScore = -Infinity;
+    let altFilled = null, altFilledScore = -Infinity;
 
     for (const wrap of S.wrapByKey.values()) {
+      if (!wrap?.isConnected) continue;
+      if (!wrap.classList?.contains(WRAP_CLASS) || !wrap.classList.contains(klass)) continue;
       try {
-        if (!wrap?.isConnected || !wrap.classList?.contains(WRAP_CLASS) || !wrap.classList.contains(klass)) continue;
         const rect = wrap.getBoundingClientRect();
-        const farAbove = rect.bottom < -vh;
-        const farBelow = rect.top > (window.innerHeight || 800) + vh;
-        if (!farAbove && !farBelow) continue;
-
-        const preferredSide = dir > 0 ? farAbove : farBelow;
-        const dist = targetRect ? Math.abs(rect.top - targetRect.top) : Math.abs(rect.top);
-        const score = (isFilled(wrap) ? 10000000 : 0) + dist;
-
-        if (preferredSide) {
-          if (score < bestScore) { best = wrap; bestScore = score; }
-        } else if (score < fallbackScore) {
-          fallback = wrap; fallbackScore = score;
+        const farAbove = rect.bottom <= farTopThreshold;
+        const farBelow = rect.top >= farBottomThreshold;
+        let preferred = false;
+        let score = -Infinity;
+
+        if (dir >= 0) {
+          if (farAbove) { preferred = true; score = Math.abs(rect.bottom); }
+          else if (farBelow) { score = Math.abs(rect.top); }
+        } else {
+          if (farBelow) { preferred = true; score = Math.abs(rect.top); }
+          else if (farAbove) { score = Math.abs(rect.bottom); }
+        }
+        if (score === -Infinity) continue;
+
+        const filled = isFilled(wrap);
+        if (preferred) {
+          if (!filled) {
+            if (score > prefEmptyScore) { prefEmptyScore = score; prefEmpty = wrap; }
+          } else {
+            if (score > prefFilledScore) { prefFilledScore = score; prefFilled = wrap; }
+          }
+        } else {
+          if (!filled) {
+            if (score > altEmptyScore) { altEmptyScore = score; altEmpty = wrap; }
+          } else {
+            if (score > altFilledScore) { altFilledScore = score; altFilled = wrap; }
+          }
         }
       } catch (_) {}
     }
 
-    best = best || fallback;
+    const best = prefEmpty ?? prefFilled ?? altEmpty ?? altFilled;
     if (!best) return null;
     const id = parseInt(best.getAttribute(A_WRAPID), 10);
     if (!Number.isFinite(id)) return null;
 
     const oldKey = best.getAttribute(A_ANCHOR);
-    try { const ph = best.querySelector(`#${PH_PREFIX}${id}`); if (ph) { S.io?.unobserve(ph); unobserveFill(id); } } catch (_) {}
+    try { const ph = best.querySelector(`#${PH_PREFIX}${id}`); if (ph) S.io?.unobserve(ph); } catch (_) {}
     mutate(() => {
       best.setAttribute(A_ANCHOR,  newKey);
       best.setAttribute(A_CREATED, String(ts()));
@@ -443,9 +381,9 @@
     if (oldKey && S.wrapByKey.get(oldKey) === best) S.wrapByKey.delete(oldKey);
     S.wrapByKey.set(newKey, best);
 
-    const doDestroy = () => { try { ez.destroyPlaceholders([id]); } catch (_) {} setTimeout(doDefine, 300); };
-    const doDefine  = () => { try { ez.define([id]); } catch (_) {} setTimeout(doDisplay, 300); };
-    const doDisplay = () => { try { ez.displayMore([id]); } catch (_) {} };
+    const doDestroy = () => { try { ez.destroyPlaceholders([id]); } catch (_) {} setTimeout(doDefine, 220); };
+    const doDefine  = () => { try { ez.define([id]);              } catch (_) {} setTimeout(doDisplay,220); };
+    const doDisplay = () => { try { ez.displayMore([id]);         } catch (_) {} };
     try { (typeof ez.cmd?.push === 'function') ? ez.cmd.push(doDestroy) : doDestroy(); } catch (_) {}
 
     return { id, wrap: best };
@@ -485,7 +423,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)) unobserveFill(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);
@@ -549,7 +486,6 @@
   function injectBetween(klass, items, interval, showFirst, poolKey) {
     if (!items.length) return 0;
     let inserted = 0;
-    sweepDeadWraps();
 
     for (const el of items) {
       if (inserted >= MAX_INSERTS_RUN) break;
@@ -562,15 +498,13 @@
       const key = anchorKey(klass, el);
       if (findWrap(key)) continue;
 
-      let id = pickId(poolKey);
-      if (!id) { sweepDeadWraps(); id = pickId(poolKey); }
+      const id = pickId(poolKey);
       if (id) {
         const w = insertAfter(el, id, klass, key);
         if (w) { observePh(id); inserted++; }
       } else {
         const recycled = recycleAndMove(klass, el, key);
         if (!recycled) break;
-        observePh(recycled.id);
         inserted++;
       }
     }
@@ -596,54 +530,54 @@
 
   function observePh(id) {
     const ph = document.getElementById(`${PH_PREFIX}${id}`);
-    if (ph?.isConnected) {
-      try { getIO()?.observe(ph); } catch (_) {}
-      watchPlaceholderFill(id);
-    }
+    if (!ph?.isConnected) return;
+    try { getIO()?.observe(ph); } catch (_) {}
+    try {
+      const r = ph.getBoundingClientRect();
+      const vh = window.innerHeight || 800;
+      if (r.bottom >= -FAST_SHOW_MARGIN_PX && r.top <= vh + FAST_SHOW_MARGIN_PX) {
+        const pid = parseInt(ph.getAttribute('data-ezoic-id'), 10);
+        if (Number.isFinite(pid) && pid > 0) enqueueShow(pid);
+      }
+    } catch (_) {}
   }
 
   function enqueueShow(id) {
     if (!id || isBlocked()) return;
-    if (isCooling(id) || S.pendingShows.has(id)) return;
     if (ts() - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) return;
-    if (S.inflight >= MAX_INFLIGHT) { pendingEnqueue(id); return; }
+    if (S.inflight >= MAX_INFLIGHT) {
+      if (!S.pendingSet.has(id)) { S.pending.push(id); S.pendingSet.add(id); }
+      return;
+    }
     startShow(id);
   }
 
   function drainQueue() {
     if (isBlocked()) return;
     while (S.inflight < MAX_INFLIGHT && S.pending.length) {
-      const item = S.pending.shift();
-      const id = item?.id ?? item;
+      const id = S.pending.shift();
       S.pendingSet.delete(id);
-      if (!id || isCooling(id) || S.pendingShows.has(id)) continue;
-      if (!document.getElementById(`${PH_PREFIX}${id}`)?.isConnected) continue;
-      if (ts() - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) continue;
       startShow(id);
     }
   }
 
   function startShow(id) {
     if (!id || isBlocked()) return;
-    if (S.pendingShows.has(id) || isCooling(id)) return;
-    S.pendingShows.add(id);
     S.inflight++;
     let done = false;
     const release = () => {
       if (done) return;
       done = true;
-      S.pendingShows.delete(id);
       S.inflight = Math.max(0, S.inflight - 1);
       drainQueue();
     };
-    const timer = setTimeout(() => { markUnusedProbable(id); release(); }, PENDING_TIMEOUT_MS);
+    const timer = setTimeout(release, 7000);
 
     requestAnimationFrame(() => {
       try {
         if (isBlocked()) { clearTimeout(timer); return release(); }
         const ph = document.getElementById(`${PH_PREFIX}${id}`);
-        if (!ph?.isConnected) { clearTimeout(timer); markUnusedProbable(id); return release(); }
-        if (isFilled(ph)) { clearTimeout(timer); clearCooldownIfFilled(id); return release(); }
+        if (!ph?.isConnected || isFilled(ph)) { clearTimeout(timer); return release(); }
 
         const t = ts();
         if (t - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) { clearTimeout(timer); return release(); }
@@ -655,13 +589,25 @@
         const ez = window.ezstandalone;
         const doShow = () => {
           try { ez.showAds(id); } catch (_) {}
-          setTimeout(() => { clearTimeout(timer); release(); }, 700);
+          scheduleEmptyCheck(id, t);
+          setTimeout(() => { clearTimeout(timer); release(); }, 350);
         };
         Array.isArray(ez.cmd) ? ez.cmd.push(doShow) : doShow();
       } catch (_) { clearTimeout(timer); release(); }
     });
   }
 
+  function scheduleEmptyCheck(id, showTs) {
+    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;
+        wrap.classList.toggle('is-empty', !isFilled(ph));
+      } catch (_) {}
+    }, EMPTY_CHECK_MS);
+  }
 
   // ── Patch Ezoic showAds ────────────────────────────────────────────────────
   //
@@ -703,7 +649,6 @@
   async function runCore() {
     if (isBlocked()) return 0;
     patchShowAds();
-    sweepDeadWraps();
 
     const cfg = await fetchConfig();
     if (!cfg || cfg.excluded) return 0;
@@ -770,7 +715,7 @@
       S.burstCount++;
       scheduleRun(n => {
         if (!n && !S.pending.length) { S.burstActive = false; return; }
-        setTimeout(step, n > 0 ? 150 : 300);
+        setTimeout(step, n > 0 ? 80 : 180);
       });
     };
     step();
@@ -791,9 +736,6 @@
     S.inflight     = 0;
     S.pending      = [];
     S.pendingSet.clear();
-    S.pendingShows.clear();
-    S.cooldownUntil.clear();
-    for (const id of Array.from(S.fillObsById.keys())) unobserveFill(id);
     S.burstActive  = false;
     S.runQueued    = false;
   }
@@ -805,12 +747,7 @@
     const allSel = [SEL.post, SEL.topic, SEL.category];
     S.domObs = new MutationObserver(muts => {
       if (S.mutGuard > 0 || isBlocked()) return;
-      let sawWrapRemoval = false;
       for (const m of muts) {
-        for (const n of (m.removedNodes || [])) {
-          if (n?.nodeType !== 1) continue;
-          try { if ((n.classList && n.classList.contains(WRAP_CLASS)) || n.querySelector?.(`.${WRAP_CLASS}`)) sawWrapRemoval = true; } catch (_) {}
-        }
         for (const n of m.addedNodes) {
           if (n.nodeType !== 1) continue;
           // matches() d'abord (O(1)), querySelector() seulement si nécessaire
@@ -820,7 +757,6 @@
           }
         }
       }
-      if (sawWrapRemoval) sweepDeadWraps();
     });
     try { S.domObs.observe(document.body, { childList: true, subtree: true }); } catch (_) {}
   }
@@ -924,16 +860,22 @@
     window.addEventListener('scroll', () => {
       if (ticking) return;
       ticking = true;
-      const y = window.scrollY || window.pageYOffset || 0;
-      S.scrollDir = y >= (S.lastScrollY || 0) ? 1 : -1;
-      S.lastScrollY = y;
-      requestAnimationFrame(() => { ticking = false; requestBurst(); });
+      requestAnimationFrame(() => {
+        ticking = false;
+        try {
+          const y = window.scrollY || window.pageYOffset || 0;
+          S.scrollDir = (y < (S.lastScrollY || 0)) ? -1 : 1;
+          S.lastScrollY = y;
+        } catch (_) {}
+        requestBurst();
+      });
     }, { passive: true });
   }
 
   // ── Boot ───────────────────────────────────────────────────────────────────
 
   S.pageKey = pageKey();
+  try { S.lastScrollY = window.scrollY || window.pageYOffset || 0; } catch (_) {}
   muteConsole();
   ensureTcfLocator();
   warmNetwork();