nodebb-plugin-ezoic-infinite 1.7.46 → 1.7.47

package/package.json

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

package/public/client.js

@@ -39,7 +39,6 @@
   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      = 4;       // max showAds() simultanés
   const SHOW_THROTTLE_MS  = 900;     // anti-spam showAds() par id
@@ -88,6 +87,7 @@
     pending:      [],    // ids en attente de slot inflight
     pendingSet:   new Set(),
     wrapByKey:    new Map(),  // anchorKey → wrap DOM node
+    recycling:    new Set(),   // ids en cours de séquence destroy→define→displayMore
     tcfObs:       null,        // MutationObserver TCF locator
     runQueued:    false,
     burstActive:  false,
@@ -112,7 +112,7 @@
   // ── Config ─────────────────────────────────────────────────────────────────
 
   // Fix #2 : backoff 10s sur échec pour éviter de spammer l'API
-  // si le réseau est lent ou la route indisponible.
+  // Fix #3 : _cfgErrorUntil au scope IIFE pour être réinitialisé dans cleanup()
   let _cfgErrorUntil = 0;
   async function fetchConfig() {
     if (S.cfg) return S.cfg;
@@ -258,6 +258,9 @@
   function pickId(poolKey) {
     const pool = S.pools[poolKey];
     if (!pool.length) return null;
+    // Fix #2 : early-exit si tous les ids sont déjà montés — évite O(n) inutile
+    if (S.mountedIds.size >= pool.length &&
+        pool.every(id => S.mountedIds.has(id))) 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;
@@ -303,6 +306,9 @@
     if (!best) return null;
     const id = parseInt(best.getAttribute(A_WRAPID), 10);
     if (!Number.isFinite(id)) return null;
+    // Fix #1 : éviter de recycler un slot dont la séquence Ezoic est en cours
+    if (S.recycling.has(id)) return null;
+    S.recycling.add(id);
 
     const oldKey = best.getAttribute(A_ANCHOR);
     // Neutraliser l'IO sur ce wrap avant déplacement — évite un showAds
@@ -327,6 +333,7 @@
     // si la pub ne charge pas (détection wrap vide).
     const doDisplay = () => {
       try { ez.displayMore([id]); } catch (_) {}
+      S.recycling.delete(id); // Fix #1 : séquence terminée
       observePh(id);
       const t = ts();
       try { best.setAttribute(A_SHOWN, String(t)); } catch (_) {}
@@ -641,6 +648,7 @@
     blockedUntil = ts() + 1500;
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     S.cfg          = null;
+    _cfgErrorUntil = 0;   // Fix #3 : réinitialiser le backoff entre les pages
     S.poolsReady   = false;
     S.pools        = { topics: [], posts: [], categories: [] };
     S.cursors      = { topics: 0, posts: 0, categories: 0 };
@@ -652,6 +660,7 @@
     S.pendingSet.clear();
     S.burstActive  = false;
     S.runQueued    = false;
+    S.recycling.clear();
     if (S.domObs) { S.domObs.disconnect(); S.domObs = null; }
     if (S.tcfObs) { S.tcfObs.disconnect(); S.tcfObs = null; }
   }
@@ -751,7 +760,9 @@
     $(window).on('action:ajaxify.end.nbbEzoic', () => {
       S.pageKey    = pageKey();
       blockedUntil = 0;
-      muteConsole(); ensureTcfLocator(); warmNetwork();
+      // Fix #4 : muteConsole/ensureTcfLocator/warmNetwork déjà appelés au boot
+      // et sont idempotents — pas besoin de les rappeler à chaque navigation.
+      muteConsole();
       patchShowAds(); getIO(); ensureDomObserver(); requestBurst();
     });