nodebb-plugin-ezoic-infinite 1.7.5 → 1.7.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.5",
+  "version": "1.7.7",
   "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 v21.1
+ * NodeBB Ezoic Infinite Ads — client.js v21.3
  *
  * Historique des corrections majeures
  * ────────────────────────────────────
@@ -252,8 +252,10 @@
 
   function dropWrap(w) {
     try {
+      // Unobserve uniquement si le ph est encore connecté — un nœud détaché
+      // passé à unobserve() corrompt l'IO interne de pubads.
       const ph = w.querySelector(`[id^="${PH_PREFIX}"]`);
-      if (ph instanceof Element) S.io?.unobserve(ph);
+      if (ph instanceof Element && ph.isConnected) S.io?.unobserve(ph);
       const id = parseInt(w.getAttribute(A_WRAPID), 10);
       if (Number.isFinite(id)) S.mountedIds.delete(id);
       const key = w.getAttribute(A_ANCHOR);
@@ -385,7 +387,7 @@
 
   function observePh(id) {
     const ph = document.getElementById(`${PH_PREFIX}${id}`);
-    if (ph?.isConnected) try { getIO()?.observe(ph); } catch (_) {}
+    if (ph instanceof Element && ph.isConnected) try { getIO()?.observe(ph); } catch (_) {}
   }
 
   function enqueueShow(id) {
@@ -565,7 +567,30 @@
   function cleanup() {
     blockedUntil = ts() + 1500;
     poolsReady   = false;
-    mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
+
+    // Unobserve ciblé sur NOS placeholders avant de les retirer du DOM.
+    // On ne fait JAMAIS disconnect() — c'est un IO global partagé avec pubads ;
+    // disconnect() corrompt son état interne et provoque des erreurs sur tous
+    // les observe() suivants de Google ("parameter 1 is not of type 'Element'").
+    if (S.io) {
+      for (const id of S.mountedIds) {
+        try {
+          const ph = document.getElementById(`${PH_PREFIX}${id}`);
+          if (ph instanceof Element && ph.isConnected) S.io.unobserve(ph);
+        } catch (_) {}
+      }
+    }
+
+    mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(w => {
+      try {
+        const id = parseInt(w.getAttribute(A_WRAPID), 10);
+        if (Number.isFinite(id)) S.mountedIds.delete(id);
+        const key = w.getAttribute(A_ANCHOR);
+        if (key) wrapByKey.delete(key);
+        w.remove();
+      } catch (_) {}
+    }));
+
     wrapByKey.clear();
     S.cfg          = null;
     S.pools        = { topics: [], posts: [], categories: [] };
@@ -585,6 +610,8 @@
     if (S.domObs) return;
     const allSel = [SEL.post, SEL.topic, SEL.category];
     S.domObs = new MutationObserver(muts => {
+      // Ne rien faire pendant la navigation (cleanup posé blockedUntil)
+      // ou si c'est nous qui mutons le DOM.
       if (S.mutGuard > 0 || isBlocked()) return;
       for (const m of muts) {
         for (const n of m.addedNodes) {