channel-worker 2.5.51 → 2.5.52

package/package.json

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

package/scripts/nurture_facebook.js

@@ -282,6 +282,11 @@ async function openPostAndRead(page, log) {
     for (const a of best.querySelectorAll("a[href*='/posts/'], a[href*='/permalink/'], a[href*='story_fbid'], a[href*='/photo']")) {
       const r = a.getBoundingClientRect();
       if (r.width < 8 || r.height < 8 || a.offsetParent === null) continue;
+      // Facebook ships most permalinks with target="_blank": clicked as-is the
+      // post lands in a NEW tab while the script keeps driving the old one, so
+      // every opened post leaves a dead tab behind. Force same-tab — goBack()
+      // below is what returns us to the feed.
+      a.setAttribute('target', '_self');
       a.setAttribute('__nur_open__', '1');
       return true;
     }
@@ -377,6 +382,43 @@ async function detectProfileMode(page) {
   }).catch(() => ({ isPage: false, why: 'detect failed', navSample: '' }));
 }
 
+// ONE tab, always.
+//
+// Two things pile tabs up behind a session, and neither is ours: NSTBrowser
+// restores whatever the profile had open the last time it was used (publishing
+// leaves Studio/Business tabs), and Facebook opens permalinks, photos and
+// notification links with target="_blank". The script drives pages()[0] and
+// never looks at the rest, so they just sit there — by the third session the
+// window is a wall of tabs.
+//
+// Closing them is safe: nurturing needs exactly one tab (the feed), and the
+// permalink click is forced to same-tab navigation in openPostAndRead.
+async function closeOtherTabs(context, keep, log) {
+  const before = context.pages().length;
+  let closed = 0;
+  for (const p of context.pages()) {
+    if (p === keep || p.isClosed()) continue;
+    try { await p.close({ runBeforeUnload: false }); closed++; } catch { /* already gone */ }
+  }
+  // Log even when nothing was closed: "no line" would otherwise read the same
+  // as "the fix never loaded", and that is exactly what we need to tell apart.
+  if (log) log('info', `[nurture-fb] tab: ${before} lúc vào → đóng ${closed} → còn ${context.pages().length}`);
+  return closed;
+}
+
+// Anything Facebook manages to pop open mid-session dies on arrival. Registered
+// once per session; the listener is scoped to this context, which the runner
+// detaches from when the script returns.
+function guardSingleTab(context, keep, log) {
+  context.on('page', async (p) => {
+    if (p === keep || p.isClosed()) return;
+    try {
+      await p.close({ runBeforeUnload: false });
+      log('info', '[nurture-fb] tab mới bị Facebook bung ra — đã đóng ngay');
+    } catch { /* raced with its own close */ }
+  });
+}
+
 // debug:true only — screenshot the feed and hand back a viewable URL. Cheaper
 // than another round of guessing at selectors from log lines.
 async function dumpFeedShot(page, log, tag) {
@@ -398,7 +440,7 @@ async function dumpFeedShot(page, log, tag) {
 }
 
 // ─── main ───────────────────────────────────────────────────────────────────
-async function run({ page, payload, log }) {
+async function run({ page, context, payload, log }) {
   const t0 = Date.now();
   const cfg = payload.config || {};
   const phase = Math.max(1, Math.min(3, parseInt(payload.phase, 10) || 1));
@@ -417,6 +459,12 @@ async function run({ page, payload, log }) {
 
   page.on('dialog', (d) => { d.accept().catch(() => {}); });
 
+  // Clear the profile's leftovers BEFORE loading the feed, then keep it clear.
+  if (context) {
+    await closeOtherTabs(context, page, log);
+    guardSingleTab(context, page, log);
+  }
+
   await page.goto('https://www.facebook.com/', { waitUntil: 'domcontentloaded', timeout: 60000 });
   await page.waitForTimeout(randInt(3000, 5000));
   // Guard FIRST — everything below assumes a live logged-in session.
@@ -522,6 +570,9 @@ async function run({ page, payload, log }) {
     // moment activity trips a threshold — better to stop than to keep poking.
     if (ticks % 12 === 0) {
       await assertAccountUsable(page, log);
+      // Belt to the context listener: a tab opened as a separate window, or one
+      // that slipped in while we were navigating, still gets swept.
+      if (context && context.pages().length > 1) await closeOtherTabs(context, page, log);
       log('info', `[nurture-fb] …${Math.round((Date.now() - t0) / 1000)}s: posts=${postsSeen} likes=${likes}/${likesTarget} videos=${videos} scrollY=${lastScan.scrollY} units=${lastScan.articlesInDom} sel=${lastScan.selector}`);
     }