channel-worker 2.5.27 → 2.5.28

package/package.json

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

package/scripts/warmup_youtube.js

@@ -114,28 +114,24 @@ async function collectResultVideoUrls(page) {
 // skippable ad; non-skippable ads have no button (we wait them out). Multiple
 // pre-rolls can chain, so this is called every tick of the watch loop.
 async function trySkipAd(page) {
-  return page.evaluate(() => {
-    // ONLY click inside the player's real skip-ad button (stable YT classes).
-    // A loose text match like /skip/i clicked random buttons every tick on
-    // videos with no ad — keep it strict to the ad-skip control.
-    const sels = [
-      '.ytp-ad-skip-button-modern',
-      '.ytp-ad-skip-button',
-      '.ytp-skip-ad-button',
-      '.ytp-ad-skip-button-container button',
-    ];
-    for (const s of sels) {
-      const btn = document.querySelector(s);
-      if (btn && btn.offsetParent !== null) { btn.click(); return true; }
-    }
-    // Strict text fallback — require an ad-specific phrase, NOT bare "skip"/"bỏ qua".
-    const phrases = /bỏ qua quảng cáo|skip ad\b|skip ads/i;
-    for (const b of document.querySelectorAll('.ytp-ad-skip-button-text, .ytp-ad-skip-button button, button')) {
-      const t = (b.innerText || b.textContent || '').trim();
-      if (t && phrases.test(t) && b.offsetParent !== null) { b.click(); return true; }
+  // Use Playwright's real .click() (trusted pointer event) — a synthetic
+  // el.click() from page.evaluate is ignored by YouTube's player, so the ad
+  // never actually skipped (saw 52 "skips" with the ad still playing). The
+  // skip button only becomes clickable ~5s into the ad; an early click times
+  // out → return false → retried next tick.
+  const sels = [
+    '.ytp-ad-skip-button-modern',
+    '.ytp-ad-skip-button',
+    '.ytp-skip-ad-button',
+    '.ytp-ad-skip-button-container button',
+  ];
+  for (const s of sels) {
+    const loc = page.locator(s).first();
+    if (await loc.isVisible().catch(() => false)) {
+      try { await loc.click({ timeout: 1500 }); return true; } catch { /* not yet clickable */ }
     }
-    return false;
-  }).catch(() => false);
+  }
+  return false;
 }
 
 // True while an ad is ACTUALLY playing (pre/mid-roll). Relies on the player's
@@ -187,7 +183,8 @@ async function watchVideo(page, url, minSec, maxSec, log) {
       await page.waitForTimeout(TICK);
     }
     if (skips) log('info', `[warmup] ${url.slice(-11)} — skipped ${skips} ad tick(s), real watch ${Math.round(realWatched/1000)}s`);
-    return true;
+    // Only count as a watched video if real (non-ad) content actually played.
+    return realWatched > 0;
   } catch (e) {
     log('warn', `[warmup] watch failed (${url.slice(-11)}): ${String(e.message || e).slice(0, 100)}`);
     return false;