nodebb-plugin-ezoic-infinite 1.4.17 → 1.4.19

package/package.json

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

package/public/client.js

@@ -18,6 +18,9 @@
   // Survit aux navigations ajaxify (contrairement à state.definedIds remis à zéro dans cleanup).
   // Nécessaire pour savoir si on doit appeler destroyPlaceholders avant recyclage.
   const sessionDefinedIds = new Set();
+  
+  // IDs actuellement en cours d'insertion (lock pour éviter les doublons de race condition)
+  const insertingIds = new Set();
 
   const state = {
     pageKey: null,
@@ -221,17 +224,27 @@
   function insertAfter(target, id, kindClass, afterPos) {
     if (!target || !target.insertAdjacentElement) return null;
     if (findWrap(kindClass, afterPos)) return null;
-    // CRITICAL: Vérifier qu'aucun placeholder avec cet ID n'existe déjà dans le DOM
-    // (peut arriver si deux scheduleRun concurrents recyclent le même ID)
+    
+    // CRITICAL: Double-lock pour éviter race conditions sur les doublons
+    // 1. Vérifier qu'aucun autre thread n'est en train d'insérer cet ID
+    if (insertingIds.has(id)) return null;
+    
+    // 2. Vérifier qu'aucun placeholder avec cet ID n'existe déjà dans le DOM
     const existingPh = document.getElementById(`${PLACEHOLDER_PREFIX}${id}`);
-    if (existingPh && existingPh.isConnected) {
-      // Un placeholder avec cet ID existe déjà - ne pas créer de doublon
-      return null;
+    if (existingPh && existingPh.isConnected) return null;
+    
+    // Acquérir le lock
+    insertingIds.add(id);
+    
+    try {
+      const wrap = buildWrap(id, kindClass, afterPos);
+      target.insertAdjacentElement('afterend', wrap);
+      attachFillObserver(wrap, id);
+      return wrap;
+    } finally {
+      // Libérer le lock après 100ms (le temps que le DOM soit stable)
+      setTimeout(() => insertingIds.delete(id), 100);
     }
-    const wrap = buildWrap(id, kindClass, afterPos);
-    target.insertAdjacentElement('afterend', wrap);
-    attachFillObserver(wrap, id);
-    return wrap;
   }
 
   function destroyUsedPlaceholders() {
@@ -389,6 +402,14 @@
 
 
   function refillUnfilled() {
+    // Si tous les IDs disponibles sont blacklistés, ne pas réessayer
+    const totalPoolSize = state.poolTopics.length + state.poolPosts.length + state.poolCategories.length +
+                          state.usedTopics.size + state.usedPosts.size + state.usedCategories.size;
+    if (totalPoolSize > 0 && state.badIds.size >= totalPoolSize) {
+      // Tous les IDs disponibles sont blacklistés - arrêter les retry
+      return;
+    }
+    
     const wraps = Array.from(document.querySelectorAll(`.${WRAP_CLASS}`));
     let scheduledAny = false;
 
@@ -409,7 +430,7 @@
       }
 
       const tries = (state.retryById.get(id) || 0);
-      if (tries >= 8) { state.badIds && state.badIds.add(id); continue; }
+      if (tries >= 3) { state.badIds && state.badIds.add(id); continue; }
 
       const r = safeRect(wrap);
       if (r && (r.top > window.innerHeight + 1200 || r.bottom < -1200)) continue;
@@ -786,7 +807,7 @@
         // Réinitialiser le compteur d'attente de recyclage à chaque reprise de scroll
         state.poolWaitAttempts = 0;
         enforceNoAdjacentAds();
-        scheduleRefill(200);
+        scheduleRefill(2000);
         // Tenter aussi d'insérer de nouvelles pubs si de nouveaux items sont apparus
         scheduleRun();
       });