channel-worker 2.5.129 → 2.5.130

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "channel-worker",
-  "version": "2.5.129",
+  "version": "2.5.130",
   "description": "Channel Manager worker daemon — runs on remote machines to execute video pipeline jobs",
   "main": "lib/daemon.js",
   "bin": {

package/scripts/lib/warm-web.js

@@ -167,15 +167,22 @@ async function visitSite(page, url, log) {
 
 async function warmWeb({ page, cfg = {}, log }) {
   const out = { ran: false, sites: 0, seconds: 0, last_origin: '' };
+  // EVERY exit logs, including the ones that do nothing. A probability branch
+  // that stays silent when it skips is indistinguishable from a branch that is
+  // never reached: measured 2026-09-22, four sessions in a row rolled a skip
+  // (0.4^4 = 2.6%) and the only way to tell bad luck from a dead code path was
+  // to re-run the module by hand on each machine. One line is cheaper.
+  const say = (m) => { if (typeof log === 'function') log('info', `[warm-web] ${m}`); };
   const prob = cfg.warm_prob ?? 0.6;
-  if (!(prob > 0) || !chance(prob)) return out;
+  if (!(prob > 0)) { say(`tắt (warm_prob=${prob}) — không lướt web ngoài`); return out; }
+  if (!chance(prob)) { say(`bỏ qua lượt này (quay xác suất trượt, warm_prob=${prob})`); return out; }
 
   // A MISSING list means "the API is older than this script" → use the built-in
   // one. An EMPTY list means the caller said no sites, so browse none: a config
   // that clears the list must not quietly fall back to six hardcoded domains.
   const pool = (Array.isArray(cfg.warm_sites) ? cfg.warm_sites : DEFAULT_SITES)
     .filter((u) => typeof u === 'string' && /^https?:\/\//i.test(u));
-  if (!pool.length) return out;
+  if (!pool.length) { say('danh sách site rỗng — không lướt web ngoài'); return out; }
 
   const want = randInt(Math.max(1, cfg.warm_sites_min ?? 1), Math.max(1, cfg.warm_sites_max ?? 3));
   const budgetSec = randInt(cfg.warm_sec_min ?? 60, Math.max(cfg.warm_sec_min ?? 60, cfg.warm_sec_max ?? 150));