channel-worker 2.5.21 → 2.5.23

package/package.json

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

package/scripts/upload_facebook.js

@@ -283,7 +283,7 @@ async function run({ page, payload, log }) {
   } = payload || {};
   if (!video_url) throw new Error('No video_url provided');
 
-  log('info', '[fb-pw] selectors version=2026.06.19b-thumb-diag');
+  log('info', '[fb-pw] selectors version=2026.06.19d-reels-link-fallback');
 
   page.on('dialog', (d) => { d.accept().catch(() => {}); });
 
@@ -454,9 +454,31 @@ async function run({ page, payload, log }) {
       .filter({ has: page.locator(":scope :text-matches('^Tạo thước phim$|^Create reel$|^Create a reel$', 'i')") })
       .first();
     if (!(await reelsDialog.isVisible({ timeout: 6000 }).catch(() => false))) {
-      await dumpInventory(page, log, 'no-reels-modal');
-      await dumpFailure(page, 'no-reels-modal', log);
-      throw new Error('FB Reels modal "Tạo thước phim" did not open after clicking Thước phim entry');
+      // FALLBACK: some accounts render a different UI from the create-post
+      // "Thước phim" entry → the composer modal never mounts. Click the Reels
+      // link (any <a href*='/reels'>) instead, then re-wait for the modal.
+      log('warn', '[fb-pw] Reels modal not open — fallback: click <a href*="/reels"> then re-wait…');
+      let xpClicked = false;
+      try {
+        const reelsLink = page.locator("xpath=//a[contains(@href, '/reels')]").first();
+        if (await reelsLink.isVisible({ timeout: 3000 }).catch(() => false)) {
+          await reelsLink.click({ timeout: 4000 });
+          xpClicked = true;
+          log('info', '[fb-pw] clicked Reels link via xpath //a[contains(@href,"/reels")]');
+        } else {
+          log('warn', '[fb-pw] fallback: no visible <a href*="/reels"> found');
+        }
+      } catch (e) { log('warn', `[fb-pw] fallback Reels-link click failed: ${e.message.slice(0, 80)}`); }
+      if (xpClicked) {
+        await page.waitForTimeout(5000);
+        await dismissBsOnboarding(page, log).catch(() => {});
+      }
+      if (!(await reelsDialog.isVisible({ timeout: 8000 }).catch(() => false))) {
+        await dumpInventory(page, log, 'no-reels-modal');
+        await dumpFailure(page, 'no-reels-modal', log);
+        throw new Error('FB Reels modal "Tạo thước phim" did not open (kể cả sau fallback xpath //a[href*="/reels"])');
+      }
+      log('info', '[fb-pw] Reels modal opened after xpath /reels fallback');
     }
     // Candidates inside the modal only:
     //   - "Tải lên" — the explicit blue upload button at the bottom (preferred)
@@ -1914,12 +1936,30 @@ async function run({ page, payload, log }) {
       }
     }
 
-    // (a.2) Reel-preview tile on profile reels tab — FB sometimes navigates
-    //       to /profile.php?sk=reels_tab after publish. The FIRST tile in
-    //       DOM order (with aria-label="Bản xem trước ô thước phim" /
-    //       "Reel tile preview") is the NEWEST reel = our just-published.
-    //       Each tile is an <a href="/reel/<id>/"> anchor — scraping that
-    //       href is far more reliable than network-id ordering.
+    // (a.2) Graph API network capture — AUTHORITATIVE. The publish-mutation
+    //       response carries the JUST-published reel ID. Only consider IDs
+    //       captured AFTER the publish-click snapshot (pre-snapshot IDs are the
+    //       page-wall feed pre-render, NOT the publish mutation). Take the FIRST
+    //       new ID (the mutation response comes back before feed-reload
+    //       responses). Prefer this over scraping the reels tab: the new reel
+    //       often isn't on the tab yet (still processing), so the tab's "first
+    //       tile" is a STALE older reel — which is the bug this ordering fixes.
+    if (!postUrl && capturedReelIds.length > capturedReelIdsSnapshotLen) {
+      const newIds = capturedReelIds.slice(capturedReelIdsSnapshotLen);
+      const preSet = new Set(capturedReelIds.slice(0, capturedReelIdsSnapshotLen));
+      const trulyNew = newIds.filter((id) => !preSet.has(id));
+      if (trulyNew.length) {
+        const firstNew = trulyNew[0];
+        postUrl = `https://www.facebook.com/reel/${firstNew}/`;
+        log('info', `[fb-pw] post URL from network (FIRST new ID after snapshot): ${postUrl} (snapshotLen=${capturedReelIdsSnapshotLen} total=${capturedReelIds.length} new=${trulyNew.length})`);
+      }
+    }
+
+    // (a.3) LAST-RESORT: first reel tile on the profile reels tab. UNRELIABLE —
+    //       the just-published reel may not be on the tab yet (still
+    //       processing), so the first tile can be a STALE older reel. Only used
+    //       when every authoritative source above (fresh-timestamp tile,
+    //       title-match, inline CTA, network capture) returned nothing.
     if (!postUrl) {
       const tileHref = await page.evaluate(() => {
         const anchors = document.querySelectorAll("a[role='link']");
@@ -1934,25 +1974,7 @@ async function run({ page, payload, log }) {
       }).catch(() => null);
       if (tileHref) {
         postUrl = tileHref.href.startsWith('http') ? tileHref.href : `https://www.facebook.com${tileHref.href}`;
-        log('info', `[fb-pw] post URL from FIRST reel tile on reels_tab: ${postUrl}`);
-      }
-    }
-
-    // (b) Graph API network capture — only consider IDs captured AFTER the
-    //     publish click snapshot (pre-snapshot IDs are from the page-wall
-    //     feed pre-render, NOT the publish mutation). Take the FIRST new ID
-    //     since FB's publish-mutation response comes back before any
-    //     feed-reload responses.
-    if (!postUrl && capturedReelIds.length > capturedReelIdsSnapshotLen) {
-      const newIds = capturedReelIds.slice(capturedReelIdsSnapshotLen);
-      // Filter out IDs that appeared in the pre-snapshot to be safe (Set
-      // diff). Then take the FIRST new one.
-      const preSet = new Set(capturedReelIds.slice(0, capturedReelIdsSnapshotLen));
-      const trulyNew = newIds.filter((id) => !preSet.has(id));
-      if (trulyNew.length) {
-        const firstNew = trulyNew[0];
-        postUrl = `https://www.facebook.com/reel/${firstNew}/`;
-        log('info', `[fb-pw] post URL from network (FIRST new ID after snapshot): ${postUrl} (snapshotLen=${capturedReelIdsSnapshotLen} total=${capturedReelIds.length} new=${trulyNew.length})`);
+        log('warn', `[fb-pw] post URL from FIRST reel tile on reels_tab (LAST-RESORT, may be STALE): ${postUrl}`);
       }
     }