nodebb-plugin-ezoic-infinite 1.8.92 → 1.8.94

package/library.js

@@ -93,17 +93,15 @@ async function isUserExcluded(uid, excludedGroups) {
 
   _excludeCache.set(key, { value, at: Date.now() });
   if (_excludeCache.size > 1000) {
-    const now = Date.now();
     let toDel = 100;
-    // Supprimer en priorité les entrées expirées, puis les plus anciennes
+    const t = Date.now();
     for (const [k, v] of _excludeCache) {
-      if (now - v.at >= EXCLUDE_TTL) { _excludeCache.delete(k); if (--toDel <= 0) break; }
+      if (!toDel) break;
+      if (t - v.at >= EXCLUDE_TTL) { _excludeCache.delete(k); toDel--; }
     }
-    if (toDel > 0) {
-      for (const k of _excludeCache.keys()) {
-        _excludeCache.delete(k);
-        if (--toDel <= 0) break;
-      }
+    for (const k of _excludeCache.keys()) {
+      if (!toDel) break;
+      _excludeCache.delete(k); toDel--;
     }
   }
 
@@ -119,21 +117,8 @@ function clearCaches() {
 // ── Client config ────────────────────────────────────────────────────────────
 
 function buildClientConfig(settings, excluded) {
-  return {
-    excluded,
-    enableBetweenAds:       settings.enableBetweenAds,
-    showFirstTopicAd:       settings.showFirstTopicAd,
-    placeholderIds:         settings.placeholderIds,
-    intervalPosts:          settings.intervalPosts,
-    enableCategoryAds:      settings.enableCategoryAds,
-    showFirstCategoryAd:    settings.showFirstCategoryAd,
-    categoryPlaceholderIds: settings.categoryPlaceholderIds,
-    intervalCategories:     settings.intervalCategories,
-    enableMessageAds:       settings.enableMessageAds,
-    showFirstMessageAd:     settings.showFirstMessageAd,
-    messagePlaceholderIds:  settings.messagePlaceholderIds,
-    messageIntervalPosts:   settings.messageIntervalPosts,
-  };
+  const { excludedGroups, ...rest } = settings;
+  return { excluded, ...rest };
 }
 
 function serializeInlineConfig(cfg) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodebb-plugin-ezoic-infinite",
-  "version": "1.8.92",
+  "version": "1.8.94",
   "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 v2.5.0
+ * NodeBB Ezoic Infinite Ads — client.js v2.5.1
  *
  * Architecture: proven v50 core + targeted improvements.
  * Ezoic API: showAds() + destroyPlaceholders() per official docs.
@@ -177,8 +177,8 @@
     return out;
   }
 
-  const getTopics     = () => Array.from(document.querySelectorAll(SEL.topic));
-  const getCategories = () => Array.from(document.querySelectorAll(SEL.category));
+  const getTopics     = () => document.querySelectorAll(SEL.topic);
+  const getCategories = () => document.querySelectorAll(SEL.category);
 
   // ── Anchor keys & wrap registry ────────────────────────────────────────────
 
@@ -268,9 +268,8 @@
   }
 
   function addWrapTimer(id, ms, fn) {
-    const timers = S.wrapTimers.get(id) || [];
-    timers.push(setTimeout(() => { try { fn(); } catch (_) {} }, ms));
-    S.wrapTimers.set(id, timers);
+    if (!S.wrapTimers.has(id)) S.wrapTimers.set(id, []);
+    S.wrapTimers.get(id).push(setTimeout(() => { try { fn(); } catch (_) {} }, ms));
   }
 
   function scheduleUncollapseChecks(wrap, id) {
@@ -434,12 +433,13 @@
 
   // ── Injection ──────────────────────────────────────────────────────────────
 
-  function ordinal(klass, el) {
+  function ordinal(klass, el, hint) {
     const attr = KIND[klass]?.ordinalAttr;
     if (attr) {
       const v = el.getAttribute(attr);
       if (v != null && v !== '' && !isNaN(v)) return parseInt(v, 10);
     }
+    if (hint !== undefined) return hint;
     const fullSel = KIND[klass]?.sel ?? '';
     let i = 0;
     for (const s of el.parentElement?.children ?? []) {
@@ -452,10 +452,11 @@
   function injectBetween(klass, items, interval, showFirst, poolKey) {
     if (!items.length) return 0;
     let inserted = 0;
+    let pos = 0;
     for (const el of items) {
       if (inserted >= MAX_INSERTS_RUN) break;
       if (!el?.isConnected) continue;
-      const ord = ordinal(klass, el);
+      const ord = ordinal(klass, el, pos++);
       if (!(showFirst && ord === 0) && (ord + 1) % interval !== 0) continue;
       if (adjacentWrap(el)) continue;
       const key = anchorKey(klass, el);
@@ -608,7 +609,6 @@
 
   async function runCore() {
     if (isBlocked()) return 0;
-    patchShowAds();
 
     const t = now();
     if (t - _lastGc > 30_000) { _lastGc = t; gcDisconnectedWraps(); }
@@ -886,38 +886,38 @@
   //    in ez-standalone.js → onStandaloneLoadEvent crash). After ~6s, remove and reload
   //    sa.min.js; Ezoic's partial first-run state may let the second run succeed.
   //    Once recovered, re-enqueue all mounted placeholders so showAds() fires.
-  let _retries = 0;
-  let _scriptReloaded = false;
-  let _postReloadShown = false;
+  const RETRY = { count: 0, scriptReloaded: false, postReloadShown: false };
   function retryBoot() {
-    if (_retries >= 12) return;
-    _retries++;
+    if (RETRY.count >= 12) return;
+    RETRY.count++;
     patchShowAds();
 
     const ez = window.ezstandalone;
     const status = ez?.loadingStatus;
 
     // After reload: once Ezoic reaches 'complete', re-call showAds for all mounted placeholders
-    if (_scriptReloaded && !_postReloadShown && status === 'complete') {
-      _postReloadShown = true;
+    if (RETRY.scriptReloaded && !RETRY.postReloadShown && status === 'complete') {
+      RETRY.postReloadShown = true;
       for (const id of S.mountedIds) { try { enqueueShow(id); } catch (_) {} }
       return;
     }
 
-    // Normal exit: placeholders mounted, no reload triggered (healthy load)
-    if (S.mountedIds.size > 0 && !_scriptReloaded) return;
+    // Normal exit: placeholders mounted, no reload triggered (healthy load).
+    // Exception: if Ezoic status is stuck (not absent and not 'complete'), continue
+    // so crash detection below can trigger the sa.min.js reload.
+    if (S.mountedIds.size > 0 && !RETRY.scriptReloaded && (!status || status === 'complete')) return;
     // Exit once reload is done and re-show has been triggered
-    if (_scriptReloaded && _postReloadShown) return;
+    if (RETRY.scriptReloaded && RETRY.postReloadShown) return;
 
-    if (!_scriptReloaded) {
+    if (!RETRY.scriptReloaded) {
       // Case 1: sa.min.js never injected (Rocket Loader stripped it, etc.)
-      const neverInjected = !status && !document.querySelector('script[src*="sa.min.js"]') && _retries <= 3;
+      const neverInjected = !status && !document.querySelector('script[src*="sa.min.js"]') && RETRY.count <= 3;
       // Case 2: cold-start crash — loadingStatus stuck before 'complete' after enough wait.
       // Do NOT reload when status==='complete' (post-defineScript-failure would re-trigger).
       const anyFilled = !!document.querySelector(`.${WRAP_CLASS} ${FILL_SEL}`);
-      const crashed = _retries >= 8 && !anyFilled && status && status !== 'complete';
+      const crashed = RETRY.count >= 8 && !anyFilled && status && status !== 'complete';
       if (neverInjected || crashed) {
-        _scriptReloaded = true;
+        RETRY.scriptReloaded = true;
         try {
           if (crashed) {
             // Remove the broken script before re-adding to avoid a duplicate
@@ -940,7 +940,7 @@
       S.lastBurstTs = now() - TIMING.BURST_COOLDOWN_MS;
       requestBurst();
     }
-    setTimeout(retryBoot, _retries <= 4 ? 300 : 1000);
+    setTimeout(retryBoot, RETRY.count <= 4 ? 300 : 1000);
   }
   setTimeout(retryBoot, 250);