social-autoposter 1.6.25 → 1.6.26

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "social-autoposter",
-  "version": "1.6.25",
+  "version": "1.6.26",
   "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"
@@ -21,7 +21,18 @@
     "!scripts/_seo_lane_roi.py",
     "!scripts/check_improve_runs.py",
     "!scripts/classify_all_dms.py",
-    "!scripts/migrate_engagement_styles_to_db.py",
+    "!scripts/migrate_*.py",
+    "!scripts/_gsc_roi_query.py",
+    "!scripts/batch_send_dms.py",
+    "!scripts/finalize_post_*.py",
+    "!scripts/insert_li_notifs.py",
+    "!scripts/li_notif_scan_process.py",
+    "!scripts/phase_d_edit.py",
+    "!scripts/phase_d_new_comments.py",
+    "!scripts/realign_sequences.py",
+    "!scripts/scratch_*.py",
+    "!scripts/seed_dashboard_users.py",
+    "!scripts/send_dashboard_invite.py",
     "scripts/*.sh",
     "schema-postgres.sql",
     "config.example.json",

package/scripts/score_twitter_candidates.py

@@ -30,6 +30,16 @@ from twitter_account import resolve_handle as _resolve_twitter_handle  # noqa: E
 from project_topics import topics_for_project  # noqa: E402
 
 
+# Freshness window (in hours) for the expire-stale gate that flips stale
+# pending rows to status='expired'. Sourced from the FRESHNESS_HOURS env the
+# cycle exports (run-twitter-cycle.sh) so the expiry ceiling is configured in
+# ONE place. Falls back to 18 when unset (e.g. ad-hoc / --expire-only runs) to
+# preserve the historical default. NOTE: the gate is on discovered_at
+# (discovery age), not tweet_posted_at; for logic D (≤1h discovery freshness)
+# the two are within ~1h of each other.
+EXPIRE_FRESHNESS_HOURS = int(os.environ.get("FRESHNESS_HOURS") or "18")
+
+
 # Real Twitter snowflake IDs are 18-19 digit numbers with full entropy in the
 # low bits (sequence counter + worker/datacenter ID = bottom 22 bits ≈ bottom
 # 7 decimal digits). An ID ending in 6+ zeros is statistically impossible
@@ -505,9 +515,10 @@ def upsert_candidates(tweets, config, batch_id=None, attempts_map=None, scored_s
             print(f"  Error inserting {url}: {e}", file=sys.stderr)
             continue
 
-    # Expire old pending candidates (> 18h). This is a freshness GATE
-    # (status flip), not a delete — we keep the row forever for analytics.
-    api_post("/api/v1/twitter-candidates/expire-stale", {"freshness_hours": 18})
+    # Expire old pending candidates past the freshness window. This is a
+    # freshness GATE (status flip), not a delete — we keep the row forever
+    # for analytics.
+    api_post("/api/v1/twitter-candidates/expire-stale", {"freshness_hours": EXPIRE_FRESHNESS_HOURS})
 
     # NO PRUNING. We keep every twitter_candidates row forever (chosen, skipped,
     # expired) so we can audit project routing, skip reasons, growth dynamics,
@@ -570,7 +581,7 @@ def main():
         # it off and prints the count.
         resp = api_post(
             "/api/v1/twitter-candidates/expire-stale",
-            {"freshness_hours": 18},
+            {"freshness_hours": EXPIRE_FRESHNESS_HOURS},
         )
         expired = (resp.get("data") or {}).get("expired_count", 0)
         print(f"Expired {expired} old pending candidates (no row deletion)")

package/skill/lib/twitter-backend.sh

@@ -227,3 +227,65 @@ defer_if_foreign_for_backend() {
     # the twitter-agent profile. Always return 1 (do not defer).
     return 1
 }
+
+# --- browser-harness `-c` capability self-heal (added 2026-06-02) -----------
+# A stale ~/Developer/browser-harness checkout that PREDATES the `-c` interface
+# makes `browser-harness -c "<script>"` print its usage string instead of
+# running the script. The Phase 1 scan loop in run-twitter-cycle.sh then yields
+# zero tweets with no obvious cause. cli.js documents the same failure for the
+# bh_run MCP path. When this bit the testing machine, the debugging agent saw
+# the `-c` flag, WRONGLY assumed it was unsupported, and proposed rewriting the
+# call to a nonexistent "stdin form" (browser-harness has no stdin mode — `-c`
+# is the only interface; see run.py). This runs at source-time, before any
+# `-c` call, so all twitter harness scripts (cycle/threads/engage/dm/followups)
+# get auto-repair. Static probe is one grep when fresh (zero steady-state cost);
+# the git+uv refresh only fires when the checkout is actually stale.
+_sa_harness_log() {
+    # Use the caller's log() FUNCTION when present; `declare -F` matches only a
+    # shell function, never the macOS /usr/bin/log binary (command -v would).
+    if declare -F log >/dev/null 2>&1; then log "$*"; else echo "[$(date +%H:%M:%S)] $*" >&2; fi
+}
+_sa_resolve_uv() {
+    local c
+    c="$(command -v uv 2>/dev/null)" && { echo "$c"; return 0; }
+    for c in "$HOME/.local/bin/uv" /opt/homebrew/bin/uv /usr/local/bin/uv; do
+        [ -x "$c" ] && { echo "$c"; return 0; }
+    done
+    return 1
+}
+ensure_harness_c_support() {
+    # Run once per process even if several libs source this file.
+    [ -n "${_SA_HARNESS_C_CHECKED:-}" ] && return 0
+    export _SA_HARNESS_C_CHECKED=1
+
+    local src="$HOME/Developer/browser-harness"
+    local run_py="$src/src/browser_harness/run.py"
+    local bh="$HOME/.local/bin/browser-harness"
+
+    # Static capability probe — no daemon/Chrome needed. The CLI is an editable
+    # (`uv tool install -e`) install of $src, so its run.py is the source of
+    # truth for whether `-c` is recognized.
+    if [ -f "$run_py" ] && grep -q '"-c"' "$run_py"; then
+        return 0
+    fi
+
+    _sa_harness_log "[harness] browser-harness checkout missing/stale (no -c support) -> self-healing via git + uv..."
+    if [ -d "$src/.git" ]; then
+        git -C "$src" fetch --depth 1 origin HEAD >/dev/null 2>&1 \
+            && git -C "$src" reset --hard FETCH_HEAD >/dev/null 2>&1
+    fi
+    local uv; uv="$(_sa_resolve_uv || true)"
+    if [ -n "$uv" ] && [ -d "$src" ]; then
+        "$uv" tool install --force -e "$src" >/dev/null 2>&1 || true
+    fi
+    [ -x "$bh" ] && "$bh" --reload >/dev/null 2>&1 || true
+
+    if [ -f "$run_py" ] && grep -q '"-c"' "$run_py"; then
+        _sa_harness_log "[harness] self-heal OK -> browser-harness -c is supported now"
+        return 0
+    fi
+    _sa_harness_log "[harness] ERROR: browser-harness still lacks -c after self-heal."
+    _sa_harness_log "[harness] FIX: run 'social-autoposter update' (re-clones/refreshes $src + reinstalls the CLI). The -c flag is CORRECT; browser-harness has NO stdin mode, so do NOT rewrite the cycle to a stdin form."
+    return 1
+}
+ensure_harness_c_support || true

package/skill/run-twitter-cycle.sh

@@ -147,7 +147,9 @@ FRESHNESS_HOURS=2
 # since-rewrite hook) stays tightened to 1h, the winning D setting.
 TWITTER_CYCLE_VARIANT=D
 FRESHNESS_HOURS_DISCOVER=1
-export TWITTER_CYCLE_VARIANT FRESHNESS_HOURS_DISCOVER
+# Export FRESHNESS_HOURS too so score_twitter_candidates.py inherits it and
+# drives the expire-stale gate from the same knob (was hardcoded 18h there).
+export TWITTER_CYCLE_VARIANT FRESHNESS_HOURS_DISCOVER FRESHNESS_HOURS
 # Hook env: ~/.claude/hooks/twitter-search-since-rewrite.py reads this and
 # uses it in place of its hardcoded 6h default when present.
 export FRESHNESS_HOURS_OVERRIDE=$FRESHNESS_HOURS_DISCOVER

package/scripts/_gsc_roi_query.py

@@ -1,231 +0,0 @@
-#!/usr/bin/env python3
-"""One-off: GSC pipeline ROI per project (last 30d).
-
-For each project in config.json with GSC-pipeline pages in the last 30 days,
-queries PostHog (HogQL) for total unique-distinct_id visitors to those exact
-$pathname values, and the top-3 pages by visitors. Joins with Postgres for
-GSC clicks and spend.
-
-Run with:
-    /opt/homebrew/bin/python3.11 scripts/_gsc_roi_query.py
-"""
-import json
-import os
-import subprocess
-import sys
-import urllib.parse
-import urllib.request
-from collections import defaultdict
-
-ROOT = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
-
-
-def load_env():
-    with open(os.path.join(ROOT, '.env')) as f:
-        for line in f:
-            line = line.strip()
-            if line and not line.startswith('#') and '=' in line:
-                k, v = line.split('=', 1)
-                os.environ.setdefault(k.strip(), v.strip())
-
-
-def ph_key():
-    return subprocess.check_output(
-        ['security', 'find-generic-password', '-s', 'PostHog-Personal-API-Key-m13v', '-w'],
-        text=True,
-    ).strip()
-
-
-def hogql(project_id: str, sql: str, key: str):
-    body = json.dumps({"query": {"kind": "HogQLQuery", "query": sql}}).encode()
-    req = urllib.request.Request(
-        f"https://us.posthog.com/api/projects/{project_id}/query/",
-        data=body,
-        headers={
-            "Authorization": f"Bearer {key}",
-            "Content-Type": "application/json",
-        },
-        method="POST",
-    )
-    with urllib.request.urlopen(req, timeout=60) as r:
-        return json.loads(r.read().decode())
-
-
-def main():
-    load_env()
-    import psycopg2
-
-    # Build project -> posthog project_id map from config.json
-    with open(os.path.join(ROOT, 'config.json')) as f:
-        cfg = json.load(f)
-    posthog_map = {}
-    site_map = {}
-    for p in cfg['projects']:
-        name = p['name']
-        ph = (p.get('posthog') or {}).get('project_id')
-        if ph:
-            posthog_map[name.lower()] = str(ph)
-        site_map[name.lower()] = (p.get('landing_pages') or {}).get('base_url') or p.get('website') or ''
-
-    # Pull GSC pages + spend per project (last 30d)
-    conn = psycopg2.connect(os.environ['DATABASE_URL'])
-    cur = conn.cursor()
-    cur.execute("""
-        SELECT gq.product, gq.page_url, gq.clicks, gq.impressions,
-               COALESCE(cs.total_cost_usd, 0)::float AS cost
-        FROM gsc_queries gq
-        LEFT JOIN claude_sessions cs ON cs.session_id = gq.claude_session_id
-        WHERE gq.status='done'
-          AND gq.completed_at >= NOW() - INTERVAL '30 days'
-          AND gq.page_url IS NOT NULL AND gq.page_url <> ''
-    """)
-    rows = cur.fetchall()
-    cur.close(); conn.close()
-
-    # Group by product
-    per_project = defaultdict(list)
-    for product, url, clicks, impr, cost in rows:
-        # Normalize: if URL is relative, leave path; if absolute, parse path
-        if url.startswith('http'):
-            u = urllib.parse.urlparse(url)
-            host = u.netloc
-            path = u.path or '/'
-        else:
-            host = ''
-            path = url
-        per_project[product].append({
-            "host": host, "path": path, "clicks": int(clicks or 0),
-            "impressions": int(impr or 0), "cost": float(cost or 0),
-        })
-
-    key = ph_key()
-    summary = []
-
-    for product, pages in per_project.items():
-        ph_proj = posthog_map.get(product.lower())
-        n_pages = len(pages)
-        spend = sum(p['cost'] for p in pages)
-        gsc_clicks = sum(p['clicks'] for p in pages)
-        gsc_impr = sum(p['impressions'] for p in pages)
-
-        if not ph_proj:
-            summary.append({
-                "product": product, "n_pages": n_pages, "spend": spend,
-                "visitors": None, "gsc_clicks": gsc_clicks, "gsc_impr": gsc_impr,
-                "top": [], "note": "no posthog project_id in config.json",
-            })
-            continue
-
-        # PostHog HogQL: count unique distinct_id per $pathname for the matching paths.
-        # Filter by path IN (...) — host doesn't matter inside a single PostHog project.
-        paths = sorted({p['path'] for p in pages if p['path']})
-        if not paths:
-            summary.append({
-                "product": product, "n_pages": n_pages, "spend": spend,
-                "visitors": 0, "gsc_clicks": gsc_clicks, "gsc_impr": gsc_impr,
-                "top": [], "note": "no paths to query",
-            })
-            continue
-        path_list = ", ".join(f"'{p.replace(chr(39), chr(39)+chr(39))}'" for p in paths)
-
-        sql = (
-            "SELECT properties.$pathname AS path, count(DISTINCT distinct_id) AS visitors "
-            "FROM events "
-            "WHERE event = '$pageview' "
-            f"AND timestamp > now() - INTERVAL 30 DAY "
-            f"AND properties.$pathname IN ({path_list}) "
-            "GROUP BY properties.$pathname "
-            "ORDER BY visitors DESC "
-            "LIMIT 500"
-        )
-        try:
-            res = hogql(ph_proj, sql, key)
-            results = res.get('results') or []
-            total_visitors = sum(r[1] for r in results)
-            top = [(r[0], r[1]) for r in results[:5]]
-        except Exception as e:
-            total_visitors = None
-            top = []
-            err = str(e)[:120]
-            summary.append({
-                "product": product, "n_pages": n_pages, "spend": spend,
-                "visitors": None, "gsc_clicks": gsc_clicks, "gsc_impr": gsc_impr,
-                "top": [], "note": f"PostHog error: {err}",
-            })
-            continue
-
-        summary.append({
-            "product": product, "n_pages": n_pages, "spend": spend,
-            "visitors": total_visitors, "gsc_clicks": gsc_clicks, "gsc_impr": gsc_impr,
-            "top": top, "ph_proj": ph_proj, "note": "",
-        })
-
-    # Sort by visitors desc (None at bottom)
-    summary.sort(key=lambda s: (s['visitors'] is None, -(s['visitors'] or 0)))
-
-    print()
-    print(f"{'Project':<14} {'Pages':>5} {'Spend':>9} {'Visitors':>9} {'GSCclicks':>9} {'GSCimpr':>8} {'$/visitor':>10}  Top page (visitors)")
-    print("-" * 130)
-    for s in summary:
-        v = s['visitors']
-        vs = "n/a" if v is None else f"{v:,}"
-        cpv = "n/a" if not v else f"${s['spend']/v:.2f}"
-        top1 = s['top'][0] if s['top'] else ("", 0)
-        top1_str = f"{top1[0][:60]} ({top1[1]})" if top1[0] else "-"
-        print(f"{s['product']:<14} {s['n_pages']:>5} ${s['spend']:>7.0f} {vs:>9} {s['gsc_clicks']:>9} {s['gsc_impr']:>8} {cpv:>10}  {top1_str}")
-        if s['note']:
-            print(f"   note: {s['note']}")
-
-    print("\n\nTOP-3 GSC PAGES PER PROJECT BY POSTHOG VISITORS:")
-    print("=" * 100)
-    for s in summary:
-        if not s['top']:
-            continue
-        print(f"\n{s['product']} (ph={s.get('ph_proj','?')}):")
-        for path, v in s['top'][:3]:
-            print(f"  {v:>6,}  {path}")
-
-    # Now for the top-N pages overall across all projects, break down by $referrer / utm_source
-    print("\n\nTRAFFIC SOURCE BREAKDOWN (Top 5 GSC pages by visitors, per project):")
-    print("=" * 100)
-    for s in summary:
-        if not s['top']:
-            continue
-        ph_proj = s.get('ph_proj')
-        if not ph_proj:
-            continue
-        top_paths = [p for p, _ in s['top'][:5]]
-        path_list = ", ".join(f"'{p.replace(chr(39), chr(39)+chr(39))}'" for p in top_paths)
-        sql = (
-            "SELECT properties.$pathname AS path, "
-            "coalesce(properties.$referring_domain, 'direct/none') AS ref, "
-            "count(DISTINCT distinct_id) AS visitors "
-            "FROM events "
-            "WHERE event = '$pageview' "
-            "AND timestamp > now() - INTERVAL 30 DAY "
-            f"AND properties.$pathname IN ({path_list}) "
-            "GROUP BY path, ref "
-            "ORDER BY path, visitors DESC "
-            "LIMIT 200"
-        )
-        try:
-            r = hogql(ph_proj, sql, key)
-            res = r.get('results') or []
-        except Exception as e:
-            print(f"\n{s['product']}: PostHog source-breakdown error: {str(e)[:80]}")
-            continue
-        by_path = defaultdict(list)
-        for path, ref, v in res:
-            by_path[path].append((ref, v))
-        print(f"\n{s['product']}:")
-        for path in top_paths:
-            srcs = by_path.get(path) or []
-            srcs.sort(key=lambda x: -x[1])
-            total = sum(v for _, v in srcs)
-            head = ", ".join(f"{ref}={v}" for ref, v in srcs[:5])
-            print(f"  [{total:>5,}] {path}")
-            print(f"          {head}")
-
-
-if __name__ == "__main__":
-    main()

package/scripts/batch_send_dms.py

@@ -1,148 +0,0 @@
-#!/usr/bin/env python3
-"""Batch send Reddit DMs with retries and DB updates."""
-
-import json
-import os
-import subprocess
-import sys
-import time
-
-REPO_DIR = os.path.dirname(os.path.dirname(os.path.abspath(__file__)))
-sys.path.insert(0, os.path.join(REPO_DIR, "scripts"))
-from db import load_env
-
-load_env()
-DB_URL = os.environ["DATABASE_URL"]
-
-DMS = [
-    (675, "RobotEnthusiast1980", "glass break sensor testing", "did you get around to testing the placement yet? curious how the 7m spec holds up in your actual space"),
-    (676, "EverySecondCountss", "zed + mcp daily driver", "did you end up getting a stable daily driver setup with zed or still tweaking configs? genuinely want to try it but don't want to spend a week debugging glue"),
-    (677, "Worf-", "NVR software choice", "solid plan on the axis cameras. what NVR software are you leaning toward? if you go with something like frigate the used ones should just work over onvif"),
-    (678, "raja-rancho", "good automation channels", "what videos or channels were most useful when you were starting out with automations? always looking for good recs to share"),
-    (679, "ShermanHoax", "sysco alternatives", "once that pricing creep sets in its brutal. have you found any regional distributors worth working with as a backup?"),
-    (680, "Anonymously_Joe", "cold ferment specs", "cold ferment is the move for volume. how long are you fermenting and what hydration? always curious about other shops setups"),
-    (681, "Lost_Drunken_Sailor", "cutting phone orders", "have you actually seen places pull off online only without losing the older regulars? curious if the drop off is as bad as i imagine"),
-    (682, "rocketstart1", "tool integration gaps", "the human glue thing is exactly the problem. which tools are you using day to day? curious if anything covers the custom integrations zapier cant handle"),
-    (683, "ghl92479", "AST based test gen", "the AST based test generation is clever. what parser are you using for the tree, or just feeding file structure to the agent directly?"),
-    (684, "Other_Grocery_9829", "alternatives to stealth", "since you mentioned puppeteer stealth, what are you actually automating? curious if theres a native API approach that could skip the detection game"),
-    (685, "Hot-Confection-3459", "liquid ai in production", "lol the trade secrets slip was great. seriously though are you using liquid models in production? curious how they compare for coding tasks"),
-    (686, "Triple-F-Jeff", "lorex web interface fix", "did you get the web interface working? accessing cameras directly by IP usually bypasses the NVR firmware limitations"),
-    (687, "socialmatehq", "supabase affiliate tracking", "your supabase CRM setup sounds clean. curious about the affiliate tracking, did you build custom tables or find an existing schema?"),
-    (688, "Zatetics", "corrections file approach", "the skills and memory approach you described is exactly my workflow too. what does your corrections file look like, do you append or prune old entries?"),
-    (689, "partstable", "corrections database ROI", "800 tokens preventing 90% of mistakes is incredible ROI. how do you decide when a correction gets promoted vs retired?"),
-    (690, "stumptowndoug", "multi agent file conflicts", "nice work shipping the fs watcher. how does it handle when multiple agents write the same file? thats where my polling setup kept breaking"),
-    (691, "Virtual_Armadillo126", "perception vs permissions", "your point about what agents can see vs do is underrated. have you seen any framework that cleanly separates perception from permission boundaries?"),
-    (692, "Limp_Cauliflower5192", "wildest user edge case", "first week after shipping is always humbling. what was the wildest edge case a real user found in your agent?"),
-    (693, "Intelligent-School64", "batch processing technique", "going from 90s to 15s latency is massive. would love to hear more about the batch processing approach you landed on"),
-    (694, "chocboyfish", "lighting upgrade idea", "did the lighting swap idea resonate? even going from overhead fluorescent to warmer pendants can change the whole vibe for cheap"),
-    (695, "Individual-Love-9342", "per agent cost tracking", "the postgres logging approach has been solid for us. what granularity are you tracking, per call or per task?"),
-    (696, "symmetry_seeking", "doc drift detection", "hierarchy docs drifting from code is the exact problem. have you tried any automated checks to flag when docs and code diverge?"),
-    (697, "commanderjack_EDH", "camera ONVIF check", "did you check the model number yet? if they do ONVIF you have tons of options for third party NVR software even without the app"),
-    (698, "Silver_Ad4449", "tiktok content approach", "smart move going deep on one platform. what kind of content is working for you on tiktok? the algorithm rewards very different stuff"),
-    (699, "upvotes2doge", "ARIA tree on different sites", "curious if youve tried inspector jake on SPAs specifically, some seem to have pretty sparse accessibility trees which limits what the agent can actually target"),
-    (700, "Abject_Elevator5461", "online ordering shift rate", "the ATP difference between phone and online is legit. curious what percentage of orders you see shift when places push for online"),
-    (701, "JimiJohhnySRV", "hallway camera setup", "the activity zone trick for high traffic areas is a game changer. what camera did you end up using for the hallway?"),
-    (702, "handsnerfin", "outline before AI approach", "the outline before asking approach really does change retention. curious if youve found specific problem types where it helps more"),
-    (703, "Objective_River_5218", "global skill noise", "interesting that global skills havent gotten noisy for you yet. how many are in your knowledge base right now?"),
-    (704, "bobo-the-merciful", "nelson quarterdeck rhythm", "the admiral pattern is well thought out. does the quarterdeck rhythm catch things the escalation path misses in practice?"),
-    (705, "MrBrightside_119", "stuck session detection", "the cron based stuck detection is clever. whats your time threshold and how often does it false positive?"),
-    (706, "Pritom14", "AX tree token savings", "landed on the same AX tree plus screenshot combo. have you benchmarked token savings with tree data vs pure screenshots through your proxy?"),
-    (707, "EbbCommon9300", "autonomy session risk", "the near miss data angle is fascinating. are you scoping risk per agent session or across delegated action chains?"),
-    (708, "Beautiful_Ad789", "workflow format", "totally agree vibe workflow is its own thing. what format did you end up using instead of json for workflow definitions?"),
-    (709, "eazyigz123", "hard block rollback rate", "the hard block vs soft memory distinction is interesting. how often have you had to roll blocks back when context changed?"),
-    (710, "entheosoul", "MCP structured output case", "fair pushback on MCP vs bash. the structured output case is where MCP shines but agreed its not always worth the overhead"),
-    (711, "dawsonvpowell", "Jobber vs Housecall gaps", "everything in one system makes sense. which of those tools did you go with and where were the gaps?"),
-    (712, "KiteWhisperer", "clinic manager tipping point", "thats the goal for year two. what patient volume did you find makes a clinic manager pay for themselves?"),
-    (713, "Ambitious_Bridge_180", "other OD communities", "just checked out ODs on Finance, great group. any other communities youd recommend for the business ops side?"),
-    (714, "EthanWalker483", "lead building fix", "monday morning reminder emails are exactly it. did switching to a database fix the lead building or just shift the bottleneck?"),
-]
-
-def send_dm(dm_id, author, subject, body):
-    max_retries = 3
-    for attempt in range(1, max_retries + 1):
-        print(f"[{dm_id}] Attempt {attempt}: Sending to {author}...")
-        try:
-            result = subprocess.run(
-                ["python3", os.path.join(REPO_DIR, "scripts", "reddit_browser.py"),
-                 "compose-dm", author, subject, body],
-                capture_output=True, text=True, timeout=60
-            )
-            data = json.loads(result.stdout.strip().split('\n')[-1] if result.stdout.strip() else '{}')
-        except Exception as e:
-            data = {"ok": False, "error": str(e)}
-
-        if data.get("ok"):
-            print(f"[{dm_id}] SUCCESS: Sent to {author}")
-            # Update DB
-            try:
-                subprocess.run(
-                    ["psql", DB_URL, "-c",
-                     f"UPDATE dms SET status='sent', our_dm_content=$${body}$$, sent_at=NOW() WHERE id={dm_id};"],
-                    capture_output=True, timeout=15
-                )
-            except Exception:
-                pass
-            # Log outbound
-            try:
-                subprocess.run(
-                    ["python3", os.path.join(REPO_DIR, "scripts", "dm_conversation.py"),
-                     "log-outbound", "--dm-id", str(dm_id), "--content", body],
-                    capture_output=True, timeout=15
-                )
-            except Exception:
-                pass
-            # Stamp chat_url from the sender's returned thread_url so the dashboard
-            # "open chat" button points at the actual room. set-url refuses anything
-            # that isn't a real DM-thread URL, so bad data can't leak in.
-            thread_url = data.get("thread_url") or ""
-            if thread_url:
-                try:
-                    subprocess.run(
-                        ["python3", os.path.join(REPO_DIR, "scripts", "dm_conversation.py"),
-                         "set-url", "--dm-id", str(dm_id), "--url", thread_url],
-                        capture_output=True, timeout=15
-                    )
-                except Exception:
-                    pass
-            return True
-
-        error = data.get("error", "unknown")
-        print(f"[{dm_id}] FAILED (attempt {attempt}): {error}")
-        if attempt < max_retries:
-            print(f"[{dm_id}] Waiting 30s before retry...")
-            time.sleep(30)
-
-    # All retries exhausted
-    print(f"[{dm_id}] ERROR: All retries exhausted for {author}")
-    try:
-        subprocess.run(
-            ["psql", DB_URL, "-c",
-             f"UPDATE dms SET status='error', skip_reason=$$Failed after 3 attempts: {error}$$ WHERE id={dm_id};"],
-            capture_output=True, timeout=15
-        )
-    except Exception:
-        pass
-    return False
-
-
-if __name__ == "__main__":
-    # Force unbuffered output
-    sys.stdout = os.fdopen(sys.stdout.fileno(), 'w', buffering=1)
-    sys.stderr = os.fdopen(sys.stderr.fileno(), 'w', buffering=1)
-
-    succeeded = 0
-    failed = 0
-    for dm_id, author, subject, body in DMS:
-        if send_dm(dm_id, author, subject, body):
-            succeeded += 1
-        else:
-            failed += 1
-        sys.stdout.flush()
-        # Small delay between DMs
-        time.sleep(2)
-
-    print(f"\n=== SUMMARY ===")
-    print(f"Succeeded: {succeeded}")
-    print(f"Failed: {failed}")
-    print(f"Total: {succeeded + failed}")
-    sys.stdout.flush()