nodebb-plugin-ezoic-infinite 1.7.63 → 1.7.65

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.7.63",
+  "version": "1.7.65",
   "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 v62
+ * NodeBB Ezoic Infinite Ads — client.js v69
  *
  * Historique
  * ──────────
@@ -7,28 +7,19 @@
  * v20  Table KIND. IO fixe. Fix TCF locator.
  * v25  Fix scroll-up / virtualisation NodeBB.
  * v28  Wraps persistants pendant la session.
- * v35  S.poolsReady. muteConsole élargi.
  * v36  S.wrapByKey Map O(1). MutationObserver optimisé.
- * v38  ez.refresh() supprimé. Pool épuisé → break propre.
- * v40  Recyclage destroyPlaceholders+define+displayMore (délais 300ms).
- * v43  Seuil recyclage -1vh + unobserve avant déplacement.
+ * v38  Pool épuisé → break propre (ez.refresh supprimé).
  * v49  Fix normalizeExcludedGroups (JSON.parse tableau NodeBB).
- * v51  fetchConfig backoff 10s. IO recrée au resize. tcfObs dans S.
- * v52  pruneOrphansBetween supprimé (NodeBB virtualise aussi les topics).
- * v53  S.recycling garde double-recyclage. pickId early-exit. cleanup complet.
- * v54  ensureTcfLocator rappelé à chaque ajaxify.end.
- * v56  scheduleEmptyCheck / is-empty supprimés (collapse prématuré).
- * v62  is-empty réintroduit : collapse 60s après insertion du wrap (pas après
- *      showAds) si isFilled est toujours false. Évite les trous permanents.
- * v61  recycleAndMove : ne pas recycler un wrap rempli depuis moins de 30s.
- *      Empêche qu'une pub qui vient de charger soit déplacée immédiatement.
- * v59  CSS : min-height 90px sur ezoic-ad-between (anti-CLS AMP ads).
- * v58  tcfObs survit aux navigations : ne plus déconnecter dans cleanup().
- *      L'iframe __tcfapiLocator doit exister en permanence pour le CMP —
- *      la fenêtre entre cleanup() et ajaxify.end causait des erreurs
- *      "Cannot read properties of null (postMessage)" et disparition des pubs.
- * v57  Nettoyage prod : A_SHOWN/A_CREATED supprimés, normBool Set O(1),
- *      muteConsole étend aux erreurs CMP getTCData, code nettoyé.
+ * v51  fetchConfig backoff 10s. IO recrée au resize.
+ * v52  pruneOrphansBetween supprimé (NodeBB virtualise les topics).
+ * v56  scheduleEmptyCheck supprimé (collapse prématuré).
+ * v58  tcfObs survit aux navigations (iframe CMP permanente).
+ * v62  is-empty réintroduit, déclenché à l'insertion du wrap.
+ * v64  recycleAndMove supprimé (slots restaient en 'unused' après destroy).
+ * v67  define(allIds) au boot — Ezoic enregistre tous les slots en interne.
+ * v68  setIsSinglePageApplication(true) + newPage() à chaque navigation.
+ * v69  Nettoyage prod final : S.recycling orphelin supprimé, helpers Ezoic
+ *      SPA extraits en fonctions dédiées, commentaires legacy retirés.
  */
 (function nbbEzoicInfinite() {
   'use strict';
@@ -40,11 +31,11 @@
   const A_ANCHOR   = 'data-ezoic-anchor';  // "kindClass:stableId"
   const A_WRAPID   = 'data-ezoic-wrapid';  // id Ezoic
 
-  const EMPTY_CHECK_MS    = 5_000;  // 5s après showAds — séquence define+display+show prend ~1.4s
-  const MAX_INSERTS_RUN   = 6;    // insertions max par appel runCore
-  const MAX_INFLIGHT      = 4;    // showAds() simultanés max
-  const SHOW_THROTTLE_MS  = 900;  // anti-spam showAds() par id
-  const BURST_COOLDOWN_MS = 200;  // délai min entre deux bursts
+  const EMPTY_CHECK_MS    = 5_000;  // collapse wrap vide 5s après insertion
+  const MAX_INSERTS_RUN   = 6;      // insertions max par appel runCore
+  const MAX_INFLIGHT      = 4;      // showAds() simultanés max
+  const SHOW_THROTTLE_MS  = 900;    // anti-spam showAds() par id
+  const BURST_COOLDOWN_MS = 200;    // délai min entre deux bursts
 
   const IO_MARGIN_DESKTOP = '2500px 0px 2500px 0px';
   const IO_MARGIN_MOBILE  = '3500px 0px 3500px 0px';
@@ -60,7 +51,7 @@
    * sel         sélecteur CSS des éléments cibles
    * baseTag     préfixe tag pour querySelector d'ancre
    * anchorAttr  attribut DOM stable → clé unique du wrap
-   * ordinalAttr attribut 0-based pour le calcul de l'intervalle (null = fallback positionnel)
+   * ordinalAttr attribut 0-based pour calcul de l'intervalle (null = fallback positionnel)
    */
   const KIND = {
     'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index' },
@@ -80,13 +71,12 @@
     lastShow:      new Map(),
     io:            null,
     domObs:        null,
-    tcfObs:        null,
+    tcfObs:        null,      // survit aux navigations — ne jamais déconnecter
     mutGuard:      0,
     inflight:      0,
     pending:       [],
     pendingSet:    new Set(),
     wrapByKey:     new Map(),  // anchorKey → wrap DOM node
-    recycling:     new Set(),  // ids en cours de séquence destroy→define→displayMore
     runQueued:     false,
     burstActive:   false,
     burstDeadline: 0,
@@ -138,16 +128,26 @@
     S.pools.posts      = parseIds(cfg.messagePlaceholderIds);
     S.pools.categories = parseIds(cfg.categoryPlaceholderIds);
     S.poolsReady = true;
-    // Déclarer tous les ids du pool une seule fois — Ezoic les enregistre en interne
+    // Déclarer tous les ids en une seule fois — requis pour que showAds()
+    // fonctionne sur des slots insérés dynamiquement (infinite scroll).
     const allIds = [...S.pools.topics, ...S.pools.posts, ...S.pools.categories];
-    if (allIds.length) {
+    if (allIds.length) ezCmd(ez => ez.define(allIds));
+  }
+
+  // ── Helpers Ezoic ──────────────────────────────────────────────────────────
+
+  function ezCmd(fn) {
+    try {
       window.ezstandalone = window.ezstandalone || {};
       const ez = window.ezstandalone;
-      const doDef = () => { try { ez.define(allIds); } catch (_) {} };
-      typeof ez.cmd?.push === 'function' ? ez.cmd.push(doDef) : doDef();
-    }
+      const exec = () => { try { fn(ez); } catch (_) {} };
+      typeof ez.cmd?.push === 'function' ? ez.cmd.push(exec) : exec();
+    } catch (_) {}
   }
 
+  function notifyEzoicSpa()     { ezCmd(ez => ez.setIsSinglePageApplication?.(true)); }
+  function notifyEzoicNewPage() { ezCmd(ez => ez.newPage?.()); }
+
   // ── Page identity ──────────────────────────────────────────────────────────
 
   function pageKey() {
@@ -355,28 +355,27 @@
     if (ph?.isConnected) try { getIO()?.observe(ph); } catch (_) {}
   }
 
-  function enqueueShow(id) {
-    if (!id || isBlocked()) return;
-    if (ts() - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) return;
-    if (S.inflight >= MAX_INFLIGHT) {
-      if (!S.pendingSet.has(id)) { S.pending.push(id); S.pendingSet.add(id); }
-      return;
-    }
-    startShow(id);
-  }
-
   function scheduleEmptyCheck(id) {
     setTimeout(() => {
       try {
         const ph   = document.getElementById(`${PH_PREFIX}${id}`);
         const wrap = ph?.closest?.(`.${WRAP_CLASS}`);
         if (!wrap || !ph?.isConnected) return;
-        // Collapse uniquement si vraiment vide après 60s
         if (!isFilled(wrap)) wrap.classList.add('is-empty');
       } catch (_) {}
     }, EMPTY_CHECK_MS);
   }
 
+  function enqueueShow(id) {
+    if (!id || isBlocked()) return;
+    if (ts() - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) return;
+    if (S.inflight >= MAX_INFLIGHT) {
+      if (!S.pendingSet.has(id)) { S.pending.push(id); S.pendingSet.add(id); }
+      return;
+    }
+    startShow(id);
+  }
+
   function drainQueue() {
     if (isBlocked()) return;
     while (S.inflight < MAX_INFLIGHT && S.pending.length) {
@@ -405,16 +404,11 @@
         const t = ts();
         if (t - (S.lastShow.get(id) ?? 0) < SHOW_THROTTLE_MS) { clearTimeout(timer); return release(); }
         S.lastShow.set(id, t);
-        // Marquer le wrap avec le timestamp de fill pour bloquer le recyclage
-        // Si la pub charge après is-empty, retirer le collapse
-        try { document.getElementById(`${PH_PREFIX}${id}`)?.closest?.(`.${WRAP_CLASS}`)?.classList.remove('is-empty'); } catch (_) {}
-        window.ezstandalone = window.ezstandalone || {};
-        const ez = window.ezstandalone;
-        const doShow = () => {
+        try { ph.closest?.(`.${WRAP_CLASS}`)?.classList.remove('is-empty'); } catch (_) {}
+        ezCmd(ez => {
           try { ez.showAds(id); } catch (_) {}
           setTimeout(() => { clearTimeout(timer); release(); }, 700);
-        };
-        Array.isArray(ez.cmd) ? ez.cmd.push(doShow) : doShow();
+        });
       } catch (_) { clearTimeout(timer); release(); }
     });
   }
@@ -515,11 +509,11 @@
   function cleanup() {
     blockedUntil = ts() + 1500;
     mutate(() => document.querySelectorAll(`.${WRAP_CLASS}`).forEach(dropWrap));
-    S.cfg        = null;
+    S.cfg          = null;
     _cfgErrorUntil = 0;
-    S.poolsReady = false;
-    S.pools      = { topics: [], posts: [], categories: [] };
-    S.cursors    = { topics: 0,  posts: 0,  categories: 0  };
+    S.poolsReady   = false;
+    S.pools        = { topics: [], posts: [], categories: [] };
+    S.cursors      = { topics: 0,  posts: 0,  categories: 0  };
     S.mountedIds.clear();
     S.lastShow.clear();
     S.wrapByKey.clear();
@@ -530,8 +524,8 @@
     S.runQueued   = false;
     if (S.domObs) { S.domObs.disconnect(); S.domObs = null; }
     // tcfObs intentionnellement préservé : l'iframe __tcfapiLocator doit
-    // exister en permanence — la déconnecter pendant la navigation cause
-    // des erreurs CMP postMessage et la disparition des pubs.
+    // rester en vie pendant toute la session — la déconnecter entre deux
+    // navigations cause des erreurs CMP postMessage.
   }
 
   // ── MutationObserver DOM ───────────────────────────────────────────────────
@@ -627,7 +621,9 @@
     $(window).on('action:ajaxify.end.nbbEzoic', () => {
       S.pageKey    = pageKey();
       blockedUntil = 0;
-      muteConsole(); ensureTcfLocator();
+      muteConsole();
+      ensureTcfLocator();
+      notifyEzoicNewPage();
       patchShowAds(); getIO(); ensureDomObserver(); requestBurst();
     });
     const burstEvts = [
@@ -666,6 +662,7 @@
   muteConsole();
   ensureTcfLocator();
   warmNetwork();
+  notifyEzoicSpa();   // NodeBB est une SPA — Ezoic ajuste son cycle interne
   patchShowAds();
   getIO();
   ensureDomObserver();