channel-worker 2.5.20 → 2.5.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "channel-worker",
-  "version": "2.5.20",
+  "version": "2.5.22",
   "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.19a-thumb-advance');
+  log('info', '[fb-pw] selectors version=2026.06.19c-posturl-network-first');
 
   page.on('dialog', (d) => { d.accept().catch(() => {}); });
 
@@ -968,7 +968,9 @@ async function run({ page, payload, log }) {
           const dlgs = [];
           for (const d of allDlgs) {
             const txt = (d.innerText || '').slice(0, 400);
-            if (/Tạo thước phim|Create reel|Create a reel/i.test(txt)) dlgs.push(d);
+            // Match ALL reel-composer steps: "Tạo thước phim" (video step),
+            // the caption/cover step, and "Cài đặt thước phim" (settings step).
+            if (/thước phim|create reel|create a reel|reel settings/i.test(txt)) dlgs.push(d);
           }
           if (dlgs.length === 0) return null;
           for (const dlg of dlgs) {
@@ -1165,6 +1167,27 @@ async function run({ page, payload, log }) {
         // applied across ANY step, so nothing ships with FB's auto-thumb and a
         // retry can't double-post.
         if (!customThumbDone) {
+          // DIAG: dump the reel-dialog buttons on this step so we can locate the
+          // cover/thumbnail control when detection misses it (different UI variant).
+          try {
+            const diag = await page.evaluate(() => {
+              const dlgs = [...document.querySelectorAll("[role='dialog']")]
+                .filter(d => /thước phim|create reel|reel settings/i.test((d.innerText || '').slice(0, 400)));
+              const out = [];
+              for (const d of dlgs) {
+                const hdr = (d.innerText || '').split('\n')[0].slice(0, 40);
+                for (const el of d.querySelectorAll("[role='button'],button,[aria-label],img")) {
+                  const t = (el.innerText || '').trim().slice(0, 28);
+                  const al = (el.getAttribute('aria-label') || '').trim().slice(0, 40);
+                  const r = el.getBoundingClientRect();
+                  if (r.width < 10 || r.height < 10) continue;
+                  if (t || al) out.push(`[${hdr}] <${el.tagName.toLowerCase()}> t="${t}" al="${al}" ${Math.round(r.width)}x${Math.round(r.height)}`);
+                }
+              }
+              return out.slice(0, 45);
+            }).catch(() => []);
+            log('info', `[fb-pw] thumb-diag step ${step + 1}: ${JSON.stringify(diag)}`);
+          } catch { /* diag best-effort */ }
           log('info', '[fb-pw] thumb-edit pill not on this step yet — advancing via "Tiếp" to retry on the next step');
         }
       }
@@ -1891,12 +1914,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']");
@@ -1911,25 +1952,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}`);
       }
     }