channel-worker 2.5.70 → 2.5.72

package/lib/daemon.js

@@ -14,6 +14,12 @@ class Daemon {
     // Nstbrowser agent of THIS Windows session. On a multi-user box the second
     // session's agent binds 8849+ because 8848 is taken machine-wide.
     if (config.nst_api_address) process.env.NST_API_ADDRESS = config.nst_api_address;
+    // Tell every NstManager we are pinned to one Windows user's agent, so an
+    // unresolved address REFUSES commands instead of defaulting to 8848
+    // (= the other session's agent on a multi-user box).
+    if (config.nst_agent_user && !config.nst_api_address) {
+      process.env.NST_AGENT_PINNED_USER = config.nst_agent_user;
+    }
     this.api = new ApiClient(config.api_url, config.worker_token);
     this.heartbeat = new Heartbeat(this.api, config.worker_id, 30000, config);
     this.poller = new JobPoller(this.api, config);
@@ -46,9 +52,16 @@ class Daemon {
       return;
     }
     if (!found) {
-      if (!quiet) {
+      // The pinned user's agent is GONE. Drop a previously resolved address —
+      // after a relog the same port can belong to the other session's agent,
+      // and driving it opens blank profiles. Fail-closed until it re-appears.
+      if (process.env.NST_API_ADDRESS) {
+        console.warn(`[daemon] NST agent for "${user}" disappeared `
+          + `(was ${process.env.NST_API_ADDRESS}) — refusing NST commands until it is back.`);
+        delete process.env.NST_API_ADDRESS;
+      } else if (!quiet) {
         console.warn(`[daemon] No Nstbrowser agent for Windows user "${user}" — `
-          + 'NST commands will fail until that user is logged in.');
+          + 'NST commands are refused (fail-closed) until that user is logged in.');
       }
       return;
     }
@@ -73,7 +86,10 @@ class Daemon {
 ╚══════════════════════════════════════╝
 `);
     console.log(`[daemon] cache server port: ${this.config.cache_port || 8849}`);
-    console.log(`[daemon] NST agent: ${process.env.NST_API_ADDRESS || 'http://localhost:8848/api/v2 (default)'}`);
+    console.log(`[daemon] NST agent: ${process.env.NST_API_ADDRESS
+      || (this.config.nst_agent_user
+        ? `UNRESOLVED — refusing NST commands until agent of "${this.config.nst_agent_user}" is found`
+        : 'http://localhost:8848/api/v2 (default)')}`);
 
     // Start local cache server (extension shares this across all profiles)
     try {

package/lib/nst-manager.js

@@ -24,10 +24,31 @@ function nstLaunchError(response, { profileId = '', runningCount = 0 } = {}) {
 class NstManager {
   constructor(apiKey, options = {}) {
     this.apiKey = apiKey;
-    // Multi-user Windows: each logged-in session runs its OWN Nstbrowser agent,
-    // and the second one lands on 8849+ because 8848 is already taken machine-wide.
-    // NST_API_ADDRESS lets a daemon target its own session's agent.
-    this.baseUrl = options.apiAddress || process.env.NST_API_ADDRESS || 'http://localhost:8848/api/v2';
+    this._apiAddress = options.apiAddress || null;
+  }
+
+  // Multi-user Windows: each logged-in session runs its OWN Nstbrowser agent,
+  // and the second one lands on 8849+ because 8848 is already taken machine-wide.
+  // Resolved at CALL time, not construct time: the daemon's agent watcher
+  // rewrites NST_API_ADDRESS whenever ports move, and an instance built before
+  // the first resolution must not keep a stale target forever.
+  //
+  // Fail-closed: when the daemon is pinned to a Windows user
+  // (NST_AGENT_PINNED_USER) and that user's agent hasn't been located, REFUSE
+  // instead of falling back to 8848 — on a two-session box 8848 is the OTHER
+  // user's agent, and opening a profile there starts a blank Chrome: FB shows
+  // a login form and fb-guard wrongly flags the account as logged out
+  // (35 channels hit 2026-08-31…09-02).
+  get baseUrl() {
+    if (this._apiAddress) return this._apiAddress;
+    if (process.env.NST_API_ADDRESS) return process.env.NST_API_ADDRESS;
+    const pinned = process.env.NST_AGENT_PINNED_USER;
+    if (pinned) {
+      throw new Error(`NST_AGENT_UNRESOLVED: chưa thấy Nstbrowser agent của Windows user "${pinned}" `
+        + '— không fallback :8848 (agent của session khác → profile trắng → báo logout oan). '
+        + 'Lệnh CHƯA chạm nền tảng; sẽ tự chạy lại khi agent của user đó lên.');
+    }
+    return 'http://localhost:8848/api/v2';
   }
 
   async api(path, options = {}) {

package/package.json

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

package/scripts/upload_facebook.js

@@ -574,6 +574,30 @@ async function isThumbnailEditorVisible(page) {
   }, THUMBNAIL_EDITOR_TEXT_MARKERS).catch(() => false);
 }
 
+// Bằng chứng "thumb tuỳ chỉnh ĐANG được áp" không thể nhầm: nút "Gỡ hình thu
+// nhỏ tùy chỉnh khỏi thước phim" chỉ tồn tại khi đã có thumb tuỳ chỉnh. Cần
+// cho biến thể INLINE (bắt 03/09, idea 6a965a32 fail 5 lượt liên tiếp):
+// editor "nested" nằm HẲN trong bước "Cài đặt thước phim" và không bao giờ
+// "đóng" — marker "Chỉnh sửa hình thu nhỏ" thường trực trong innerText của
+// dialog nên isThumbnailEditorVisible kẹt true, vòng chờ đóng cạn 60s rồi
+// ESC+Hủy phá lượt dù thumb đã dán xong (diag 158386: Gỡ/Lưu/Hủy + Đăng
+// cùng hiện trên một bước).
+async function isCustomThumbApplied(page) {
+  return page.evaluate(() => {
+    const vis = (el) => {
+      const r = el.getBoundingClientRect();
+      if (r.width < 8 || r.height < 8) return false;
+      const cs = getComputedStyle(el);
+      return cs.visibility !== 'hidden' && cs.display !== 'none' && cs.opacity !== '0';
+    };
+    for (const el of document.querySelectorAll('[aria-label]')) {
+      const al = el.getAttribute('aria-label') || '';
+      if (/Gỡ hình thu nhỏ tùy chỉnh|Remove( your)? custom thumbnail/i.test(al) && vis(el)) return true;
+    }
+    return false;
+  }).catch(() => false);
+}
+
 const SAFE_RETRY_COMPOSER_CLOSED = 'FB_SAFE_RETRY_COMPOSER_CLOSED';
 
 function safeComposerRetryError(message) {
@@ -1654,6 +1678,7 @@ async function runOnce({ page, payload, log }) {
                 const closeDeadline = Date.now() + 60_000;
                 let modalClosed = false;
                 let closedPolls = 0;
+                let appliedPolls = 0;
                 let retryCount = 0;
                 const RETRY_AFTER_MS = 12_000;
                 let lastRetryAt = Date.now();
@@ -1663,6 +1688,23 @@ async function runOnce({ page, payload, log }) {
                   // nested form (no own dialog — its upload button, aria
                   // "Tải hình thu nhỏ tùy chỉnh lên…", only exists while the
                   // editor is showing).
+                  // Biến thể inline: panel không bao giờ đóng nhưng nút "Gỡ hình
+                  // thu nhỏ tùy chỉnh" đã hiện = thumb ĐÃ áp. Chỉ tin ở nhánh
+                  // nested (modal thật vẫn phải đóng sau Lưu như cũ) và đòi 2
+                  // mẫu liên tiếp như luật closedPolls; bấm Lưu chốt một nhịp
+                  // cuối phòng cohort nào đó vẫn cần Lưu để giữ trạng thái.
+                  if (thumbEditorVia === 'nested' && await isCustomThumbApplied(page)) {
+                    if (++appliedPolls >= 2) {
+                      for (const sel of saveCandidates) {
+                        const btn = await firstVisible(page.locator(sel), 1);
+                        if (btn) { await btn.click({ timeout: 1500 }).catch(() => {}); break; }
+                      }
+                      log('info', '[fb-pw] page-wall thumb — panel inline không đóng nhưng nút "Gỡ hình thu nhỏ" đang hiện → coi như dán xong');
+                      modalClosed = true; break;
+                    }
+                  } else {
+                    appliedPolls = 0;
+                  }
                   const still = await isThumbnailEditorVisible(page);
                   if (!still) {
                     // Require two consecutive closed samples. Facebook briefly
@@ -1702,6 +1744,11 @@ async function runOnce({ page, payload, log }) {
                   }
                   thumbApplied = true;
                   log('info', `[fb-pw] page-wall thumb — modal closed after save (retries=${retryCount})`);
+                } else if (thumbEditorVia === 'nested' && await isCustomThumbApplied(page)) {
+                  // Hết giờ chờ đóng nhưng bằng chứng áp thumb còn đó — tuyệt
+                  // đối không ESC/Hủy (Hủy trên panel inline là GỠ thumb).
+                  log('info', '[fb-pw] page-wall thumb — hết giờ chờ đóng nhưng nút "Gỡ hình thu nhỏ" đang hiện → giữ panel, coi như dán xong');
+                  thumbApplied = true;
                 } else {
                   log('warn', '[fb-pw] page-wall thumb — modal still open 60s after Lưu click; trying ESC + click Hủy fallback');
                   await page.keyboard.press('Escape').catch(() => {});
@@ -2077,13 +2124,22 @@ async function runOnce({ page, payload, log }) {
         // output. (Observed on reel 1506614811005729: step 1 → click Tiếp
         // → publish, the thumb-edit pill never appeared.)
         if (thumbPath && !customThumbDone) {
-          // Defensive: we normally fail at the caption step (before any Tiếp)
-          // when a required custom thumb can't be applied. If we somehow reach
-          // here without it, refuse to publish — but BEFORE clicking Đăng, so
-          // nothing ships and a retry can't double-post. (The v2.5.14 double-post
-          // came from failing AFTER publish; this throw is pre-publish.)
-          await dumpFailure(page, 'thumb-required-at-publish', log);
-          throw new Error('Custom thumbnail required but not applied — refusing to publish with FB auto-thumb');
+          // Cứu theo BẰNG CHỨNG trước khi chặn: biến thể inline có thể làm cờ
+          // customThumbDone lỡ false trong khi nút "Gỡ hình thu nhỏ tùy chỉnh"
+          // ngay trên bước này chứng minh thumb ĐANG được áp — cả 5 lượt fail
+          // 02-03/09 của idea 6a965a32 đều ở trạng thái này (diag 158386).
+          if (await isCustomThumbApplied(page)) {
+            log('info', '[fb-pw] pre-publish: nút "Gỡ hình thu nhỏ tùy chỉnh" đang hiện — thumb đã áp dù cờ chưa bật, tiếp tục đăng');
+            customThumbDone = true;
+          } else {
+            // Defensive: we normally fail at the caption step (before any Tiếp)
+            // when a required custom thumb can't be applied. If we somehow reach
+            // here without it, refuse to publish — but BEFORE clicking Đăng, so
+            // nothing ships and a retry can't double-post. (The v2.5.14 double-post
+            // came from failing AFTER publish; this throw is pre-publish.)
+            await dumpFailure(page, 'thumb-required-at-publish', log);
+            throw new Error('Custom thumbnail required but not applied — refusing to publish with FB auto-thumb');
+          }
         }
         log('info', `[fb-pw] click publish "${pub.verb}" via "${pub.sel}" (step ${step + 1})`);
         // Snapshot the captured-IDs list RIGHT BEFORE the publish click. The
@@ -2850,6 +2906,7 @@ module.exports.__testables = {
   PUBLISH_ENABLE_TIMEOUT_MS,
   PUBLISH_CTA_ABSENT_GRACE_MS,
   THUMBNAIL_EDITOR_TEXT_MARKERS,
+  isCustomThumbApplied,
   REEL_CREATE_SELECTORS,
   REEL_ENTRY_SELECTORS,
   ADD_VIDEO_SELECTORS,