social-autoposter 1.6.28 → 1.6.29

package/bin/cli.js

@@ -549,23 +549,28 @@ function installBrowserHarness() {
       spawnSync(harnessBin, ['--reload'], { stdio: 'inherit' });
     }
 
-    // Contract check: server.py invokes `browser-harness -c <script>`. If an
-    // offline/rate-limited fetch left a stale checkout that predates the `-c`
-    // interface, the CLI still "installs" fine but every bh_run returns the
-    // usage banner and CDP looks "not connected". Verify the installed binary
-    // actually speaks `-c` (its no-arg usage string mentions it) and fail
-    // LOUDLY here instead of shipping a silently-broken twitter-harness.
+    // Contract check: server.py pipes the script to browser-harness via stdin.
+    // Upstream supports two banner shapes — older builds advertise `-c <script>`
+    // and newer builds advertise the `<<'PY' ... PY` heredoc form. Either is
+    // fine for our use case (we pass the script via stdin, which both accept).
+    // Fail loudly if the installed binary advertises NEITHER, which usually
+    // means an offline/partial clone left a broken CLI that will silently make
+    // every bh_run look like "CDP not connected".
     if (fs.existsSync(harnessBin)) {
       const probe = spawnSync(harnessBin, [], { stdio: 'pipe', encoding: 'utf8', timeout: 15000 });
       const usage = `${probe.stdout || ''}${probe.stderr || ''}`;
-      if (!/\b-c\b/.test(usage)) {
-        console.error('    ERROR: installed browser-harness CLI does not accept `-c` (stale clone).');
-        console.error('    The twitter-harness MCP will return a usage banner / "CDP not connected".');
+      const supportsDashC = /\b-c\b/.test(usage);
+      const supportsStdin = /<<'PY'|<<"PY"|<<PY\b/.test(usage);
+      if (!supportsDashC && !supportsStdin) {
+        console.error('    ERROR: installed browser-harness CLI advertises neither `-c` nor a stdin heredoc.');
+        console.error('    This usually means a partial/corrupted install. The twitter-harness MCP will');
+        console.error('    return a usage banner / "CDP not connected" on every call.');
         console.error(`    Fix: rm -rf ${harnessDir} && re-run \`social-autoposter init\` while online,`);
         console.error('    or manually: git clone https://github.com/browser-use/browser-harness ' + harnessDir +
           ' && ' + uvBin + ' tool install --force -e ' + harnessDir);
       } else {
-        console.log('    browser-harness CLI verified (accepts -c).');
+        const shape = supportsStdin ? 'stdin heredoc' : '-c flag';
+        console.log(`    browser-harness CLI verified (${shape}).`);
       }
     }
   }

package/mcp-servers/browser-harness/server.py

@@ -407,9 +407,14 @@ def _run_harness(script: str, timeout: int = EXEC_TIMEOUT_SEC) -> dict:
     # Make sure ~/.local/bin is on PATH (uv tools live there).
     env["PATH"] = f"{Path.home()}/.local/bin:" + env.get("PATH", "")
 
+    # Upstream browser-harness dropped the `-c <script>` flag and now reads the
+    # script from stdin only (heredoc style). Pass via stdin so we work against
+    # current upstream; the old `-c` form returns the usage banner and exits 1,
+    # which used to surface as "CDP not connected" on every fresh install.
     try:
         proc = subprocess.run(
-            [BROWSER_HARNESS_BIN, "-c", script],
+            [BROWSER_HARNESS_BIN],
+            input=script,
             env=env,
             capture_output=True,
             text=True,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "social-autoposter",
-  "version": "1.6.28",
+  "version": "1.6.29",
   "description": "Automated social posting pipeline for Reddit, X/Twitter, LinkedIn, and Moltbook. Install as a Claude Code agent skill.",
   "bin": {
     "social-autoposter": "bin/cli.js"