nodebb-plugin-ezoic-infinite 1.7.11 → 1.7.13

package/package.json

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

package/public/client.js

@@ -1,5 +1,5 @@
 /**
- * NodeBB Ezoic Infinite Ads — client.js v22
+ * NodeBB Ezoic Infinite Ads — client.js v23
  *
  * Correctifs critiques vs v19
  * ───────────────────────────
@@ -317,24 +317,32 @@
 
   /**
    * Deux wraps adjacents = situation anormale → supprimer le moins prioritaire.
-   * Priorité : filled > en grâce (fill en cours) > vide.
-   * Ne supprime jamais un wrap dont showAds() date de moins de FILL_GRACE_MS.
+   * Règles absolues :
+   *  - Jamais supprimer un wrap filled (pub affichée)
+   *  - Jamais supprimer un wrap < FILL_GRACE_MS depuis création OU depuis showAds
+   *    (le fill Ezoic est async : l'enchère SSP peut prendre plusieurs secondes
+   *     après showAds, et showAds lui-même peut arriver bien après l'injection)
+   *  - Si les deux sont filled → on ne touche rien
    */
   function decluster(klass) {
     for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
-      // Grace sur le wrap courant : on le saute entièrement
+      if (isFilled(w)) continue; // filled = intouchable
+      const wCreated = parseInt(w.getAttribute(A_CREATED) || '0', 10);
+      if (ts() - wCreated < FILL_GRACE_MS) continue; // trop récent
       const wShown = parseInt(w.getAttribute(A_SHOWN) || '0', 10);
-      if (wShown && ts() - wShown < FILL_GRACE_MS) continue;
+      if (wShown && ts() - wShown < FILL_GRACE_MS) continue; // showAds récent
 
       let prev = w.previousElementSibling, steps = 0;
       while (prev && steps++ < 3) {
         if (!prev.classList?.contains(WRAP_CLASS)) { prev = prev.previousElementSibling; continue; }
-
+        if (isFilled(prev)) break; // précédent filled = intouchable
+        const pCreated = parseInt(prev.getAttribute(A_CREATED) || '0', 10);
+        if (ts() - pCreated < FILL_GRACE_MS) break;
         const pShown = parseInt(prev.getAttribute(A_SHOWN) || '0', 10);
-        if (pShown && ts() - pShown < FILL_GRACE_MS) break; // précédent en grâce → rien
+        if (pShown && ts() - pShown < FILL_GRACE_MS) break;
 
-        if      (!isFilled(w))    mutate(() => dropWrap(w));
-        else if (!isFilled(prev)) mutate(() => dropWrap(prev));
+        // Les deux vides et hors grâce → supprimer le courant
+        mutate(() => dropWrap(w));
         break;
       }
     }