nodebb-plugin-ezoic-infinite 1.4.16 → 1.4.18

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.4.16",
+  "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,10 +224,27 @@
   function insertAfter(target, id, kindClass, afterPos) {
     if (!target || !target.insertAdjacentElement) return null;
     if (findWrap(kindClass, afterPos)) return null;
-    const wrap = buildWrap(id, kindClass, afterPos);
-    target.insertAdjacentElement('afterend', wrap);
-    attachFillObserver(wrap, id);
-    return wrap;
+    
+    // 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) 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);
+    }
   }
 
   function destroyUsedPlaceholders() {
@@ -339,6 +359,10 @@
     if (!id) return;
     if (state.badIds && state.badIds.has(id)) return;
     if (state.retryQueueSet.has(id)) return;
+    // Ne pas enqueue si showAds a été appelé récemment (< 2s) pour éviter "already been defined"
+    const now = Date.now();
+    const last = state.lastShowById.get(id) || 0;
+    if (now - last < 2000) return;
     state.retryQueueSet.add(id);
     state.retryQueue.push(id);
     processRetryQueue();