channel-worker 2.5.38 → 2.5.39

package/package.json

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

package/scripts/upload_facebook.js

@@ -1556,14 +1556,30 @@ async function run({ page, payload, log }) {
           // Re-find + re-click up to 3×, and on the last try fall back to a raw
           // JS dispatch (bypasses Playwright's obscured/stable guards).
           let clickedPublish = false;
+          let attemptedClick = false;
+          // The publish button is gone → the composer closed, which on FB means
+          // the post was ACCEPTED. Critical guard against the false-failure that
+          // caused double-posts: FB registers the click and closes the composer,
+          // but Playwright's click() promise still throws a timeout as the node
+          // detaches. Treating that as a failure marked the idea unpublished →
+          // the user re-posted → duplicate reel. So: if the button vanished
+          // after we clicked, it went through.
+          const publishBtnGone = async () => !(await findByVerbs(publishVerbs, { requireBottomHalf: true }));
           for (let attempt = 0; attempt < 3 && !clickedPublish; attempt++) {
             const target = attempt === 0 ? pub : await findByVerbs(publishVerbs, { requireBottomHalf: true });
-            if (!target) { await page.waitForTimeout(1500); continue; }
+            if (!target) {
+              if (attemptedClick) { clickedPublish = true; log('info', '[fb-pw] publish button gone after click → post accepted'); break; }
+              await page.waitForTimeout(1500); continue;
+            }
+            attemptedClick = true;
             try {
               await target.hit.click({ timeout: 15000 });
               clickedPublish = true;
             } catch (ce) {
-              log('warn', `[fb-pw] publish click attempt ${attempt + 1}/3 failed: ${ce.message.slice(0, 90)}`);
+              log('warn', `[fb-pw] publish click attempt ${attempt + 1}/3 threw: ${ce.message.slice(0, 90)}`);
+              // Click may have registered before the node detached — if the
+              // button is now gone, the post went through.
+              if (await publishBtnGone()) { clickedPublish = true; log('info', '[fb-pw] click threw but publish button gone → post accepted'); break; }
               if (attempt === 2) {
                 const js = await page.evaluate(() => {
                   const el = document.querySelector("[__fbpw_target__='1']")