channel-worker 2.5.21 → 2.5.22

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "channel-worker",
-  "version": "2.5.21",
+  "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.19b-thumb-diag');
+  log('info', '[fb-pw] selectors version=2026.06.19c-posturl-network-first');
 
   page.on('dialog', (d) => { d.accept().catch(() => {}); });
 
@@ -1914,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']");
@@ -1934,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}`);
       }
     }