nodebb-plugin-ezoic-infinite 1.7.2 → 1.7.3

package/package.json

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

package/public/client.js

@@ -261,17 +261,45 @@
     return w;
   }
 
+  /**
+   * Retire proprement un wrap du DOM.
+   *
+   * Sans précaution, retirer un wrap contenant un player vidéo Ezoic (wyvern.js)
+   * déclenche des erreurs async sur des nœuds détachés :
+   *   "Cannot read properties of null (reading 'paused')"
+   *   "Cannot read properties of null (reading 'offsetWidth')"
+   *   "Invalid target for null#trigger / null#on"
+   *
+   * On pause les media et on tente de notifier l'API wyvern avant remove().
+   */
   function dropWrap(w) {
     try {
-      const id = parseInt(w.getAttribute(A_WRAPID), 10);
-      if (Number.isFinite(id)) S.mountedIds.delete(id);
-      // IMPORTANT : ne passer unobserve que si c'est un vrai Element.
-      // unobserve(null) corrompt l'état interne de l'IO (pubads lève ensuite
-      // "parameter 1 is not of type Element" sur le prochain observe).
+      // 1. Pause tous les media actifs avant détachement
+      try {
+        w.querySelectorAll('video, audio').forEach(m => {
+          try { if (!m.paused) m.pause(); } catch (_) {}
+        });
+      } catch (_) {}
+
+      // 2. Notifier l'API wyvern si disponible
+      try {
+        if (window.wyvern && typeof window.wyvern === 'object') {
+          w.querySelectorAll('[id^="ezoic-"],[class*="wyvern"],[class*="ezoic-video"]')
+            .forEach(n => { try { window.wyvern.destroy?.(n); } catch (_) {} });
+        }
+      } catch (_) {}
+
+      // 3. Unobserve (guard instanceof Element — unobserve(null) corrompt l'IO)
       try {
         const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
         if (ph instanceof Element) S.io?.unobserve(ph);
       } catch (_) {}
+
+      // 4. Libérer l'id du pool
+      const id = parseInt(w.getAttribute(A_WRAPID), 10);
+      if (Number.isFinite(id)) S.mountedIds.delete(id);
+
+      // 5. Retrait DOM
       w.remove();
     } catch (_) {}
   }
@@ -289,17 +317,31 @@
    *
    * On ne prune pas les wraps < MIN_PRUNE_AGE_MS (DOM pas encore stabilisé).
    */
+  /**
+   * Supprime les wraps dont l'élément-ancre n'est plus dans le DOM.
+   *
+   * Règle stricte : on ne supprime JAMAIS un wrap rempli (filled).
+   *   - Il peut contenir un player wyvern actif → .remove() déclenche des
+   *     erreurs async ("Cannot read 'paused'", "offsetWidth", "getChild"…).
+   *   - Le post-ancre peut être temporairement virtualisé par NodeBB puis
+   *     revenir — dans ce cas le wrap filled doit rester en place.
+   *
+   * Seuls les wraps VIDES dont l'ancre a disparu sont supprimés.
+   */
   function pruneOrphans(klass) {
     const meta = KIND[klass];
     if (!meta) return;
 
-    const baseTag = meta.sel.split('[')[0]; // ex: "li", "[component=..." → "" (géré)
+    const baseTag = meta.sel.split('[')[0];
 
     document.querySelectorAll(`.${WRAP_CLASS}.${klass}`).forEach(w => {
       if (ts() - parseInt(w.getAttribute(A_CREATED) || '0', 10) < MIN_PRUNE_AGE_MS) return;
 
+      // Ne jamais retirer un wrap qui contient du contenu (player potentiellement actif)
+      if (isFilled(w)) return;
+
       const key = w.getAttribute(A_ANCHOR) ?? '';
-      const sid = key.slice(klass.length + 1); // extrait la partie après "kindClass:"
+      const sid = key.slice(klass.length + 1);
       if (!sid) { mutate(() => dropWrap(w)); return; }
 
       const anchorEl = document.querySelector(
@@ -314,11 +356,12 @@
   /**
    * 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ègle de sécurité wyvern : on ne supprime JAMAIS un wrap rempli.
+   * Si les deux wraps adjacents sont remplis, on laisse en place (edge case rare).
    */
   function decluster(klass) {
     for (const w of document.querySelectorAll(`.${WRAP_CLASS}.${klass}`)) {
-      // Grace sur le wrap courant : on le saute entièrement
       const wShown = parseInt(w.getAttribute(A_SHOWN) || '0', 10);
       if (wShown && ts() - wShown < FILL_GRACE_MS) continue;
 
@@ -327,10 +370,16 @@
         if (!prev.classList?.contains(WRAP_CLASS)) { prev = prev.previousElementSibling; continue; }
 
         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;
+
+        const wFilled = isFilled(w);
+        const pFilled = isFilled(prev);
 
-        if      (!isFilled(w))    mutate(() => dropWrap(w));
-        else if (!isFilled(prev)) mutate(() => dropWrap(prev));
+        // Ne jamais retirer un wrap rempli (player actif potentiel)
+        if      (!wFilled && !pFilled) mutate(() => dropWrap(w));    // les deux vides → retirer le courant
+        else if (!wFilled &&  pFilled) mutate(() => dropWrap(w));    // courant vide, précédent rempli → retirer le courant
+        else if ( wFilled && !pFilled) mutate(() => dropWrap(prev)); // courant rempli, précédent vide → retirer le précédent
+        // les deux remplis → rien (on ne touche pas)
         break;
       }
     }
@@ -592,6 +641,16 @@
 
   function cleanup() {
     blockedUntil = ts() + 1500;
+
+    // Pause tous les media dans nos wraps AVANT de les retirer du DOM.
+    // Empêche wyvern.js de continuer d'exécuter ses callbacks async sur des
+    // nœuds détachés (erreurs "Cannot read 'paused'", "offsetWidth"…).
+    try {
+      document.querySelectorAll(`.${WRAP_CLASS} video, .${WRAP_CLASS} audio`).forEach(m => {
+        try { if (!m.paused) m.pause(); } catch (_) {}
+      });
+    } catch (_) {}
+
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };