nodebb-plugin-ezoic-infinite 1.4.17 → 1.4.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.4.17",
+  "version": "1.4.18",
   "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() {