nodebb-plugin-ezoic-infinite 1.7.3 → 1.7.4

package/package.json

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

package/public/client.js

@@ -60,19 +60,25 @@
   };
 
   /**
-   * Table centrale : kindClass → { selector DOM, attribut d'ancre stable }
+   * Table centrale par kindClass :
    *
-   * L'attribut d'ancre est l'identifiant que NodeBB pose TOUJOURS sur l'élément,
-   * quelle que soit la page ou la virtualisation :
-   *   posts    → data-pid   (id du message, unique et permanent)
-   *   topics   → data-index (position 0-based dans la liste, fourni par NodeBB)
-   *   catégories → data-cid (id de la catégorie, unique et permanent)
-   *                          ← C'était le bug v19 : on cherchait data-index ici
+   *  sel        : sélecteur CSS complet de l'élément cible
+   *  baseTag    : tag CSS à préfixer dans les querySelector de recherche
+   *               (déduit manuellement — évite le fragile split('[')[0])
+   *  anchorAttr : attribut DOM STABLE qui identifie l'élément de façon unique
+   *               et permanente, utilisé comme clé d'ancre du wrap.
+   *               → data-pid   pour posts    (id message, immuable)
+   *               → data-index pour topics   (position dans la liste)
+   *               → data-cid   pour catégories (id catégorie, immuable)
+   *  ordinalAttr: attribut utilisé pour calculer l'intervalle d'injection.
+   *               Doit être un entier 0-based fourni par NodeBB.
+   *               Pour posts ET topics : data-index (0-based, toujours présent).
+   *               Pour catégories : pas d'infinite scroll → fallback positionnel.
    */
   const KIND = {
-    'ezoic-ad-message':    { sel: SEL.post,     anchorAttr: 'data-pid'   },
-    'ezoic-ad-between':    { sel: SEL.topic,    anchorAttr: 'data-index' },
-    'ezoic-ad-categories': { sel: SEL.category, anchorAttr: 'data-cid'   },
+    'ezoic-ad-message':    { sel: SEL.post,     baseTag: '',   anchorAttr: 'data-pid',   ordinalAttr: 'data-index' },
+    'ezoic-ad-between':    { sel: SEL.topic,    baseTag: 'li', anchorAttr: 'data-index', ordinalAttr: 'data-index' },
+    'ezoic-ad-categories': { sel: SEL.category, baseTag: 'li', anchorAttr: 'data-cid',   ordinalAttr: null         },
   };
 
   // ── État ───────────────────────────────────────────────────────────────────
@@ -307,46 +313,33 @@
   // ── Prune ──────────────────────────────────────────────────────────────────
 
   /**
-   * Supprime les wraps dont l'élément-ancre n'est plus dans le DOM.
+   * Supprime les wraps VIDES dont l'élément-ancre n'est plus dans le DOM.
    *
-   * L'ancre est retrouvée via l'attribut stable défini dans KIND[kindClass].anchorAttr.
-   * Exemples :
-   *   ezoic-ad-message    → cherche [data-pid="123"]
-   *   ezoic-ad-between    → cherche [data-index="5"]
-   *   ezoic-ad-categories → cherche [data-cid="7"]   ← fix v20
-   *
-   * 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.
+   * Règles :
+   *  1. Jamais avant MIN_PRUNE_AGE_MS (DOM post-batch pas encore stabilisé).
+   *  2. Jamais un wrap rempli (player wyvern potentiellement actif).
+   *  3. L'ancre est retrouvée via KIND[klass].anchorAttr (stable par design) :
+   *       ezoic-ad-message    → [data-pid="123"]
+   *       ezoic-ad-between    → li[data-index="5"]
+   *       ezoic-ad-categories → li[data-cid="7"]
    */
   function pruneOrphans(klass) {
     const meta = KIND[klass];
     if (!meta) return;
 
-    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;
+      if (isFilled(w)) return; // jamais supprimer un wrap rempli (wyvern)
 
       const key = w.getAttribute(A_ANCHOR) ?? '';
       const sid = key.slice(klass.length + 1);
       if (!sid) { mutate(() => dropWrap(w)); return; }
 
-      const anchorEl = document.querySelector(
-        `${baseTag}[${meta.anchorAttr}="${sid.replace(/"/g, '\\"')}"]`
-      );
+      // Construire le sélecteur avec baseTag explicite depuis KIND
+      // baseTag='' pour posts → sélecteur : [data-pid="123"]  (correct, sans tag ambigu)
+      // baseTag='li' pour topics/categories → li[data-index="5"], li[data-cid="7"]
+      const sel = `${meta.baseTag}[${meta.anchorAttr}="${sid.replace(/"/g, '\\"')}"]`;
+      const anchorEl = document.querySelector(sel);
       if (!anchorEl || !anchorEl.isConnected) mutate(() => dropWrap(w));
     });
   }
@@ -388,24 +381,35 @@
   // ── Injection ──────────────────────────────────────────────────────────────
 
   /**
-   * Ordinal 0-based pour le calcul de l'intervalle.
-   * Pour posts/topics : data-index (NodeBB 3+/4+, toujours présent).
-   * Pour catégories : position dans le parent (page statique, pas d'infinite scroll).
+   * Ordinal 0-based d'un élément pour le calcul de l'intervalle d'injection.
+   *
+   * Utilise KIND[klass].ordinalAttr en priorité (data-index pour posts et topics,
+   * null pour catégories). Si absent, fallback positionnel dans le parent.
+   *
+   * Pour les posts : KIND.baseTag='' donc le fallback itère sur les enfants directs
+   * du parent en filtrant par sélecteur complet (pas juste le tag).
    */
   function ordinal(klass, el) {
-    const di = el.getAttribute('data-index');
-    if (di !== null && di !== '' && !isNaN(di)) return parseInt(di, 10);
-    // Fallback positionnel
+    const meta = KIND[klass];
+
+    // 1. Attribut ordinal explicite (data-index sur posts et topics)
+    if (meta?.ordinalAttr) {
+      const v = el.getAttribute(meta.ordinalAttr);
+      if (v !== null && v !== '' && !isNaN(v)) return parseInt(v, 10);
+    }
+
+    // 2. Fallback positionnel — compte parmi les frères qui matchent le même sélecteur
     try {
-      const tag = KIND[klass]?.sel?.split('[')?.[0] ?? '';
-      if (tag) {
-        let i = 0;
-        for (const n of el.parentElement?.querySelectorAll(`:scope > ${tag}`) ?? []) {
-          if (n === el) return i;
-          i++;
-        }
+      let i = 0;
+      const siblings = el.parentElement?.children ?? [];
+      const fullSel  = meta?.sel ?? '';
+      for (const s of siblings) {
+        if (s === el) return i;
+        // Compter uniquement les éléments du même type (pas les wraps ou autres)
+        if (!fullSel || s.matches?.(fullSel)) i++;
       }
     } catch (_) {}
+
     return 0;
   }