social-autoposter 1.6.98 → 1.6.99

package/mcp/dist/index.js

@@ -524,7 +524,7 @@ function renderDraftsTable(plan) {
         // Number by FULL-array index (matches post_drafts + the menu bar), then drop
         // already-finished entries so the cards only show what's still pending.
         .map((c, i) => ({ c, n: i + 1 }))
-        .filter((e) => e.c.posted !== true && e.c.terminal !== true)
+        .filter((e) => e.c.posted !== true && e.c.terminal !== true && e.c.approved !== true)
         // The queue is append-only; newest drafts have the highest stable index.
         // Show those first so review starts with likely-live tweets instead of stale
         // low-number drafts that have been sitting around for hours.
@@ -637,7 +637,13 @@ async function ensurePostingHandle() {
     }
 }
 async function postApproved(batchId, plan) {
-    const approved = (plan.candidates || []).filter((c) => c.approved === true);
+    // Post every card the user APPROVED that hasn't already landed or been ruled out.
+    // `approved` is now a DURABLE decision (sticky, never cleared by a later call), so
+    // filtering out posted/terminal here makes this idempotent: re-running it only
+    // drains the not-yet-posted approved backlog (e.g. a card a restart interrupted),
+    // never re-posts a done one. This is what lets the startup backlog-drain and the
+    // per-card menu-bar calls share one code path safely.
+    const approved = (plan.candidates || []).filter((c) => c.approved === true && c.posted !== true && c.terminal !== true);
     if (approved.length === 0)
         return { attempted: 0, exit_code: 0, summary: "nothing approved" };
     // PREFLIGHT: posting needs a configured @handle, or twitter_browser.py refuses
@@ -1402,8 +1408,13 @@ tool("post_drafts", {
             .optional()
             .describe("Rewrites: each {n, text} replaces draft n's wording, then posts it."),
         post_all: z.boolean().optional().describe("Post every draft in the batch."),
+        reject: z
+            .array(z.number().int().positive())
+            .optional()
+            .describe("1-based draft numbers the user REJECTED. They are marked done and never " +
+            "shown for review again, and are not posted."),
     },
-}, async ({ batch_id, post, edits, post_all }) => {
+}, async ({ batch_id, post, edits, post_all, reject }) => {
     const plan = readPlan(batch_id);
     if (!plan || !(plan.candidates && plan.candidates.length)) {
         return textContent(`No drafts found for batch ${batch_id}. Run scan_candidates then submit_drafts again to produce a fresh batch.`);
@@ -1412,6 +1423,25 @@ tool("post_drafts", {
     const total = candidates.length;
     const warnings = [];
     const inRange = (n) => n >= 1 && n <= total;
+    // ---- Rejections: durable + final --------------------------------------
+    // A rejected draft is marked terminal so it NEVER re-appears for review and is
+    // never posted. A reject overrides any earlier approve on the same card.
+    const rejected = [];
+    (reject || []).forEach((n) => {
+        if (!inRange(n)) {
+            warnings.push(`ignored reject #${n}: out of range (1-${total})`);
+            return;
+        }
+        const c = candidates[n - 1];
+        if (c.posted === true) {
+            warnings.push(`#${n} already posted; not rejecting`);
+            return;
+        }
+        c.terminal = true;
+        c.terminal_reason = "rejected";
+        c.approved = false;
+        rejected.push(n);
+    });
     // Apply edits first; an edited draft is always posted.
     const approve = new Set();
     let editedCount = 0;
@@ -1440,27 +1470,39 @@ tool("post_drafts", {
             warnings.push(`ignored #${n}: out of range (1-${total})`);
     });
     // Cross-surface de-dup: chat and the menu-bar pop-ups can both approve, so
-    // never re-post a candidate the other surface already posted.
-    const alreadyPosted = [];
+    // never re-post a candidate the other surface already posted OR ruled out.
+    const alreadyDone = [];
     for (const n of Array.from(approve)) {
-        if (candidates[n - 1]?.posted === true) {
+        if (candidates[n - 1]?.posted === true || candidates[n - 1]?.terminal === true) {
             approve.delete(n);
-            alreadyPosted.push(n);
+            alreadyDone.push(n);
         }
     }
-    if (alreadyPosted.length) {
-        warnings.push(`already posted (skipped): ${alreadyPosted.sort((a, b) => a - b).join(", ")}`);
-    }
-    candidates.forEach((c, i) => (c.approved = approve.has(i + 1)));
+    if (alreadyDone.length) {
+        warnings.push(`already posted/decided (skipped): ${alreadyDone.sort((a, b) => a - b).join(", ")}`);
+    }
+    // STICKY approve: record the approval DURABLY and never clear another card's
+    // prior approval. The old `c.approved = approve.has(i+1)` reset every card on
+    // each call, so a later post_drafts for a different card dropped a
+    // restart-interrupted approved card back into "pending". postApproved filters
+    // posted/terminal, so the approved set only ever drains what's genuinely left.
+    approve.forEach((n) => {
+        const c = candidates[n - 1];
+        if (c)
+            c.approved = true;
+    });
     writePlan(batch_id, plan);
     if (approve.size === 0) {
         return jsonContent({
             batch_id,
             drafted: total,
             posted: 0,
+            rejected: rejected.length,
             skipped: total,
             edited: editedCount,
-            note: "No drafts selected to post. Nothing was posted.",
+            note: rejected.length
+                ? `Rejected ${rejected.length} draft(s); they won't be shown for review again. Nothing was posted.`
+                : "No drafts selected to post. Nothing was posted.",
             warnings,
         });
     }
@@ -1478,6 +1520,7 @@ tool("post_drafts", {
         drafted: total,
         posted: actuallyPosted,
         approved: approve.size,
+        rejected: rejected.length,
         skipped: total - actuallyPosted,
         edited: editedCount,
         result,
@@ -3056,7 +3099,10 @@ tool("submit_drafts", {
         added++;
     }
     writePlan(REVIEW_QUEUE_ID, { candidates: queue });
-    const pending = queue.filter((c) => c.posted !== true && c.terminal !== true);
+    // Pending = NOT YET DECIDED. A card that's posted, terminal (rejected/dead), OR
+    // already approved is a settled decision and must never be re-presented for
+    // review — approved ones just proceed to post (see drainApprovedBacklog).
+    const pending = queue.filter((c) => c.posted !== true && c.terminal !== true && c.approved !== true);
     // Drafts queued = the pipeline verified end-to-end without posting. This is the
     // onboarding "draft_verified" terminal goal (formerly completed by draft_cycle).
     if (added > 0)
@@ -3266,6 +3312,24 @@ registerAppResource(server, "S4L product link", PRODUCT_LINK_URI, { mimeType: RE
         },
     ],
 }));
+// Post any cards the user APPROVED that never landed — e.g. a restart killed the
+// batch mid-way. "Proceed to post the already-approved items." postApproved is
+// idempotent (it filters posted/terminal), so this only drains the genuine
+// backlog and never double-posts. Best-effort; never throws.
+async function drainApprovedBacklog() {
+    try {
+        const plan = readPlan(REVIEW_QUEUE_ID);
+        const cands = plan?.candidates || [];
+        const backlog = cands.filter((c) => c.approved === true && c.posted !== true && c.terminal !== true);
+        if (!backlog.length)
+            return;
+        console.error(`[post] draining ${backlog.length} approved-but-unposted card(s) left from before`);
+        await postApproved(REVIEW_QUEUE_ID, plan);
+    }
+    catch (e) {
+        console.error("[post] drainApprovedBacklog error:", e?.message || e);
+    }
+}
 async function main() {
     initSentry();
     // Tee the verbatim stdout/stderr of every pipeline subprocess to the s4l
@@ -3330,6 +3394,13 @@ async function main() {
     void startLocalPanel()
         .then((url) => console.error(`[social-autoposter-mcp] panel loopback ready at ${url}`))
         .catch((e) => console.error("[social-autoposter-mcp] panel loopback start failed:", e?.message || e));
+    // Resume posting any approved-but-unposted cards a prior run/restart left behind.
+    // Delayed so the runtime + harness Chrome have settled; never blocks boot.
+    {
+        const t = setTimeout(() => void drainApprovedBacklog(), 30_000);
+        if (typeof t.unref === "function")
+            t.unref();
+    }
     // Ensure the macOS menu bar mini-dashboard is installed + running. Idempotent
     // and cheap when already present, so existing installs pick it up on the next
     // Claude restart without re-provisioning. Best-effort: never blocks boot.

package/mcp/dist/version.json

@@ -1,4 +1,4 @@
 {
-  "version": "1.6.98",
-  "installedAt": "2026-06-24T02:52:38.060Z"
+  "version": "1.6.99",
+  "installedAt": "2026-06-24T03:00:44.774Z"
 }

package/mcp/manifest.json

@@ -2,7 +2,7 @@
   "dxt_version": "0.1",
   "name": "social-autoposter",
   "display_name": "S4L",
-  "version": "1.6.98",
+  "version": "1.6.99",
   "description": "Draft, review, approve, and autopilot X/Twitter posts. Thin desktop client over the S4L pipeline.",
   "long_description": "A guided assistant that drafts, reviews, and autopilots X/Twitter posts.\nTo get started:\n1. Click **Configure** and set every tool permission to **Always Allow**.\n2. Copy this prompt: **Set me up on S4L end to end**.\n3. Quit fully with CMD+Q, restart Claude, and paste the prompt into a new chat.",
   "author": {

package/mcp/menubar/s4l_menubar.py

@@ -544,10 +544,20 @@ class S4LMenuBar(rumps.App):
 
     def _on_card_decision(self, batch, decision):
         # Runs on the main thread the INSTANT a card is approved/rejected. An
-        # approved card is enqueued for immediate posting; a rejected card does
-        # nothing. We never post inline here — posting can take minutes and would
-        # freeze the card UI while the user reviews the rest of the stack.
+        # approved card is enqueued for immediate posting; a REJECTED card is
+        # persisted (marked done so it's never re-shown for review) on a quick
+        # background thread. We never block inline here — posting can take minutes
+        # and would freeze the card UI while the user reviews the rest of the stack.
         if not decision.get("approved"):
+            n = decision.get("n")
+
+            def _persist_reject():
+                try:
+                    st.post_drafts(batch, reject=[n], timeout=30)
+                except Exception:
+                    pass
+
+            threading.Thread(target=_persist_reject, daemon=True).start()
             return
         with self._review_lock:
             self._posts_outstanding += 1

package/mcp/menubar/s4l_state.py

@@ -350,10 +350,12 @@ def read_plan(plan_path):
 
 
 def review_drafts(plan):
-    """Flatten a plan into the card model: only unfinished candidates."""
+    """Flatten a plan into the card model: only UNDECIDED candidates. A card that's
+    posted, terminal (rejected/dead), or already approved is a settled decision and
+    must never be re-presented for review (approved ones proceed to post)."""
     out = []
     for i, c in enumerate(((plan or {}).get("candidates") or [])):
-        if c.get("posted") is True or c.get("terminal") is True:
+        if c.get("posted") is True or c.get("terminal") is True or c.get("approved") is True:
             continue
         out.append(
             {
@@ -370,11 +372,12 @@ def review_drafts(plan):
     return out
 
 
-def post_drafts(batch_id, post=None, edits=None, timeout=900, activity_label=None):
-    """Post approved drafts via the loopback tool. `post` = 1-based numbers to
-    post as-is; `edits` = [{n, text}] to rewrite then post. Returns the parsed
+def post_drafts(batch_id, post=None, edits=None, reject=None, timeout=900, activity_label=None):
+    """Post / reject drafts via the loopback tool. `post` = 1-based numbers to post
+    as-is; `edits` = [{n, text}] to rewrite then post; `reject` = numbers to mark
+    DONE so they're never shown for review again (not posted). Returns the parsed
     result, or None if the loopback is unreachable (Claude Desktop closed)."""
-    args = {"batch_id": batch_id, "post": post or [], "edits": edits or []}
+    args = {"batch_id": batch_id, "post": post or [], "edits": edits or [], "reject": reject or []}
     if activity_label:
         args["__saps_activity_label"] = activity_label
     return loopback_tool("post_drafts", args, timeout=timeout)

package/mcp/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@m13v/social-autoposter-mcp",
-  "version": "1.6.98",
+  "version": "1.6.99",
   "private": true,
   "description": "Desktop MCP client for social-autoposter (X/Twitter rail): manual draft/review/approve loop, autopilot control, and stats. Thin wrapper over the existing pipeline scripts.",
   "license": "MIT",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "social-autoposter",
-  "version": "1.6.98",
+  "version": "1.6.99",
   "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"