@m13v/s4l 1.7.4-rc.4 → 1.7.4

package/mcp/dist/version.js

@@ -94,6 +94,67 @@ export function releaseChannel() {
     }
     return "stable";
 }
+const SHARED_TTL_S = 600; // banner latency ceiling: a new release surfaces within 10 min
+const PROBE_LOCK_STALE_MS = 30_000;
+function sharedCachePath() {
+    return path.join(STATE_DIR, "latest-release.json");
+}
+function readSharedCache(channel) {
+    try {
+        const d = JSON.parse(fs.readFileSync(sharedCachePath(), "utf-8"));
+        if (!d || d.channel !== channel || typeof d.at !== "number")
+            return null;
+        return {
+            at: d.at,
+            channel,
+            version: typeof d.version === "string" ? d.version : null,
+            tag: typeof d.tag === "string" ? d.tag : null,
+            etag: typeof d.etag === "string" ? d.etag : null,
+        };
+    }
+    catch {
+        return null;
+    }
+}
+function writeSharedCache(c) {
+    try {
+        fs.mkdirSync(STATE_DIR, { recursive: true });
+        const tmp = `${sharedCachePath()}.tmp.${process.pid}`;
+        fs.writeFileSync(tmp, JSON.stringify(c));
+        fs.renameSync(tmp, sharedCachePath());
+    }
+    catch {
+        /* best effort; worst case another process re-probes */
+    }
+}
+// Single-flight guard so N concurrent processes with an expired shared cache
+// don't all probe at once. acquired=false ONLY when another probe holds a
+// FRESH lock; any other failure probes anyway rather than going blind.
+function tryProbeLock() {
+    const lock = path.join(STATE_DIR, "latest-release.lock");
+    const create = () => fs.closeSync(fs.openSync(lock, "wx"));
+    try {
+        fs.mkdirSync(STATE_DIR, { recursive: true });
+        create();
+        return { acquired: true, lock };
+    }
+    catch (e) {
+        if (e.code === "EEXIST") {
+            try {
+                if (Date.now() - fs.statSync(lock).mtimeMs > PROBE_LOCK_STALE_MS) {
+                    fs.unlinkSync(lock); // stale: holder died mid-probe
+                    create();
+                    return { acquired: true, lock };
+                }
+            }
+            catch {
+                /* raced with the holder */
+            }
+            return { acquired: false, lock: null };
+        }
+        return { acquired: true, lock: null };
+    }
+}
 function parseSemverish(v) {
     return /^\d+\.\d+\.\d+/.test(v) ? v : null;
 }
@@ -128,41 +189,45 @@ async function latestFromGithubRedirect() {
         return null;
     }
 }
-// In-process conditional-request state: long-lived processes send If-None-Match
-// on every probe and get free 304s; each new release costs a single 200.
-const apiState = { etag: null, latest: null };
-async function latestFromGithubApi() {
+// Conditional-request state lives in the SHARED cache file (latest-release.json)
+// so the ETag survives process boundaries: short-lived MCP respawns used to pay
+// a full 200 per process; now every probe sends If-None-Match and gets a free
+// 304 between releases.
+async function curlConditional(url, etag) {
+    const args = ["-sS", "-m", "10", "-H", "Accept: application/vnd.github+json"];
+    if (etag)
+        args.push("-H", `If-None-Match: ${etag}`);
+    args.push("-w", "\n__CURL_STATUS__:%{http_code}\n__CURL_ETAG__:%header{etag}", url);
+    const res = await run("curl", args, { timeoutMs: 12000, noTee: true });
+    let status = 0;
+    let newEtag = null;
+    const body = [];
+    for (const line of (res.stdout || "").split("\n")) {
+        if (line.startsWith("__CURL_STATUS__:"))
+            status = parseInt(line.slice(16).trim(), 10) || 0;
+        else if (line.startsWith("__CURL_ETAG__:"))
+            newEtag = line.slice(14).trim() || null;
+        else
+            body.push(line);
+    }
+    return { status, etag: newEtag, body: body.join("\n") };
+}
+// Stable probe (releases/latest). On 304 serves the caller-supplied cached
+// version with the same etag; If-None-Match is only sent when there IS a
+// cached value to serve.
+async function latestFromGithubApi(etag = null, cached = null) {
     try {
-        const args = ["-sS", "-m", "10", "-H", "Accept: application/vnd.github+json"];
-        if (apiState.etag)
-            args.push("-H", `If-None-Match: ${apiState.etag}`);
-        args.push("-w", "\n__CURL_STATUS__:%{http_code}\n__CURL_ETAG__:%header{etag}", RELEASES_LATEST_API);
-        const res = await run("curl", args, { timeoutMs: 12000, noTee: true });
-        let status = 0;
-        let etag = null;
-        const body = [];
-        for (const line of (res.stdout || "").split("\n")) {
-            if (line.startsWith("__CURL_STATUS__:"))
-                status = parseInt(line.slice(16).trim(), 10) || 0;
-            else if (line.startsWith("__CURL_ETAG__:"))
-                etag = line.slice(14).trim() || null;
-            else
-                body.push(line);
-        }
-        if (status === 304)
-            return apiState.latest;
-        if (status !== 200)
-            return null;
-        const tag = JSON.parse(body.join("\n")).tag_name;
+        const r = await curlConditional(RELEASES_LATEST_API, cached ? etag : null);
+        if (r.status === 304)
+            return { version: cached, etag };
+        if (r.status !== 200)
+            return { version: null, etag: null };
+        const tag = JSON.parse(r.body).tag_name;
         const v = typeof tag === "string" ? parseSemverish(tag.replace(/^v/, "").trim()) : null;
-        if (v) {
-            apiState.etag = etag;
-            apiState.latest = v;
-        }
-        return v;
+        return v ? { version: v, etag: r.etag } : { version: null, etag: null };
     }
     catch {
-        return null;
+        return { version: null, etag: null };
     }
 }
 async function latestFromNpm() {
@@ -177,18 +242,24 @@ async function latestFromNpm() {
 }
 // Staging channel: newest release OVERALL (prereleases included) from the
 // releases LIST, since releases/latest excludes prereleases. Drafts are skipped;
-// "newest" is by the rc-aware verKey. Returns {version, tag} or null.
-async function latestFromGithubListStaging() {
+// "newest" is by the rc-aware verKey. On 304 serves the caller-supplied cached
+// {version, tag} with the same etag. Returns {version, tag, etag} or null.
+async function latestFromGithubListStaging(etag = null, cached = null) {
     try {
-        const res = await run("curl", ["-sS", "-m", "10", "-H", "Accept: application/vnd.github+json", RELEASES_LIST_API], { timeoutMs: 12000, noTee: true });
-        const rels = JSON.parse(res.stdout || "[]");
+        const haveCached = !!(cached && cached.version && cached.tag);
+        const r = await curlConditional(RELEASES_LIST_API, haveCached ? etag : null);
+        if (r.status === 304 && haveCached)
+            return { version: cached.version, tag: cached.tag, etag };
+        if (r.status !== 200)
+            return null;
+        const rels = JSON.parse(r.body || "[]");
         if (!Array.isArray(rels))
             return null;
         let best = null;
-        for (const r of rels) {
-            if (!r || typeof r !== "object" || r.draft)
+        for (const rel of rels) {
+            if (!rel || typeof rel !== "object" || rel.draft)
                 continue;
-            const tag = r.tag_name;
+            const tag = rel.tag_name;
             if (typeof tag !== "string")
                 continue;
             const v = parseSemverish(tag.replace(/^v/, "").trim());
@@ -198,7 +269,7 @@ async function latestFromGithubListStaging() {
             if (best == null || cmpKey(key, best.key) > 0)
                 best = { version: v, tag, key };
         }
-        return best ? { version: best.version, tag: best.tag } : null;
+        return best ? { version: best.version, tag: best.tag, etag: r.etag } : null;
     }
     catch {
         return null;
@@ -221,28 +292,66 @@ async function resolveLatest() {
     const now = Date.now();
     if (cache && cache.channel === channel && now - cache.at < TTL_MS)
         return { version: cache.latest, tag: cache.tag, channel };
+    // Level 2: shared cross-process cache file (see SharedCache above).
+    const shared = readSharedCache(channel);
+    if (shared && now / 1000 - shared.at < SHARED_TTL_S) {
+        cache = { at: now, latest: shared.version, tag: shared.tag, channel };
+        return { version: shared.version, tag: shared.tag, channel };
+    }
+    const { acquired, lock } = tryProbeLock();
+    if (!acquired) {
+        // Another process is probing right now; serve the stale shared value (or
+        // null) instead of doubling the request. The short in-process TTL means we
+        // pick up its fresh result within a minute.
+        const v = shared?.version ?? null;
+        const t = shared?.tag ?? null;
+        cache = { at: now, latest: v, tag: t, channel };
+        return { version: v, tag: t, channel };
+    }
     let version = null;
     let tag = null;
-    if (channel === "staging") {
-        const s = await latestFromGithubListStaging();
-        if (s) {
-            version = s.version;
-            tag = s.tag;
+    let etag = null;
+    try {
+        if (channel === "staging") {
+            const s = await latestFromGithubListStaging(shared?.etag ?? null, shared ? { version: shared.version, tag: shared.tag } : null);
+            if (s) {
+                version = s.version;
+                tag = s.tag;
+                etag = s.etag;
+            }
+            else {
+                // Degrade to the stable probes so a staging box tracks at least stable
+                // rather than going blind when the list endpoint fails. etag stays
+                // null: a releases/latest etag must never be replayed against the
+                // LIST endpoint on the next staging probe.
+                version =
+                    (await latestFromGithubApi()).version ?? (await latestFromGithubRedirect());
+                tag = version ? `v${version}` : null;
+            }
         }
         else {
-            // Degrade to the stable probes so a staging box tracks at least stable
-            // rather than going blind when the list endpoint fails.
-            version = (await latestFromGithubApi()) ?? (await latestFromGithubRedirect());
+            const r = await latestFromGithubApi(shared?.etag ?? null, shared?.version ?? null);
+            version = r.version;
+            etag = r.etag;
+            if (version == null) {
+                version = await latestFromGithubRedirect();
+                etag = null;
+            }
+            if (version == null)
+                version = await latestFromNpm();
             tag = version ? `v${version}` : null;
         }
+        writeSharedCache({ at: now / 1000, channel, version, tag, etag });
     }
-    else {
-        version = await latestFromGithubApi();
-        if (version == null)
-            version = await latestFromGithubRedirect();
-        if (version == null)
-            version = await latestFromNpm();
-        tag = version ? `v${version}` : null;
+    finally {
+        if (lock) {
+            try {
+                fs.unlinkSync(lock);
+            }
+            catch {
+                /* already gone */
+            }
+        }
     }
     cache = { at: now, latest: version, tag, channel };
     return { version, tag, channel };

package/mcp/dist/version.json

@@ -1,4 +1,4 @@
 {
-  "version": "1.7.4-rc.4",
-  "installedAt": "2026-07-11T00:05:38.650Z"
+  "version": "1.7.4",
+  "installedAt": "2026-07-14T01:43:29.220Z"
 }

package/mcp/manifest.json

@@ -2,7 +2,7 @@
   "dxt_version": "0.1",
   "name": "social-autoposter",
   "display_name": "S4L",
-  "version": "1.7.4-rc.4",
+  "version": "1.7.4",
   "description": "Draft, review, approve, and autopilot X/Twitter posts.",
   "long_description": "## **⚠️ The disclaimer above is generic Claude boilerplate.** Anthropic shows the same warning on every plugin regardless of what it does; any plugin has the same level of access as any app you download from the internet.\n\nS4L is an open source product developed by Mediar.ai Incorporated, a VC-backed San Francisco-based startup.\n\nTo get started:\n\n1\\. Copy this prompt: **Set me up on S4L plugin end to end**\n\n2\\. Quit with CMD+Q, reopen Claude, paste into a new chat.\n\nWhat happens next:\n\n* About every 5 minutes S4L scans X for posts that match your topics and drafts replies in your voice.\n* Drafts show up as review cards, usually the first within a few minutes. Nothing is posted automatically; you approve each one.\n* Posting autopilot stays off until you explicitly turn it on.",
   "author": {

package/mcp/menubar/s4l_card.py

@@ -46,10 +46,11 @@ menu bar registers at boot via `set_feedback_handler` (it ships
 decision='feedback' review events).
 
 Rejecting is a two-step flow: the Reject button swaps the card body for a
-reason picker (three one-tap categories plus Other, with an optional free-text
-note). The category feeds the feedback-digest loop that distills human
-rejections into each project's learned_preferences config block; "Skip" keeps
-the old zero-friction reject (no reason recorded). Link clicks (author profile,
+reason picker (three one-tap categories, all optional, plus an optional
+free-text note). The category feeds the feedback-digest loop that distills
+human rejections into each project's learned_preferences config block; the
+picker's red Reject button commits with no category (the note, if typed, is
+still sent). Link clicks (author profile,
 thread ↗) and per-card dwell time ride along on the decision so the digest can
 infer intent (e.g. profile-checked-then-rejected = author-quality signal) even
 when the reason is skipped.
@@ -185,13 +186,15 @@ M = 16
 NS_BEZEL_BORDER = 2  # NSBezelBorder
 
 # Reject-reason categories, in display order. Tags are 1-based button tags on
-# the reason picker; values must match the review_events.reject_category CHECK
-# constraint server-side (wrong_author | off_topic | bad_draft | other).
+# the reason picker; values must be a subset of the review_events.reject_category
+# CHECK constraint server-side (wrong_author | off_topic | bad_draft | other).
+# "other" is still valid server-side but no longer offered as a button: picking
+# a category is optional (the red Reject button commits without one) and the
+# free-text note covers everything a generic Other chip used to.
 REJECT_REASONS = (
     ("wrong_author", "Wrong author / audience"),
     ("off_topic", "Off-topic thread"),
-    ("bad_draft", "Draft doesn't sound right"),
-    ("other", "Other"),
+    ("bad_draft", "Feels like AI writing"),
 )
 
 # Client-side cap on tracked interactions per card (server clips at 50 too).
@@ -431,7 +434,21 @@ def _details_lines(d):
     d = d or {}
     lines = []
     style = (d.get("engagement_style") or "").strip()
-    if style:
+    # Two-draft cards (2026-07-11): name each slot's style so the reviewer
+    # can tell WHICH style produced the draft they are picking (Draft B is
+    # the exploration slot; its source arm renders separately below via the
+    # generic experiments lines). Single-draft cards keep the old one-liner.
+    _dual = d.get("drafts")
+    if isinstance(_dual, list) and len(_dual) == 2:
+        _slot_labels = {"a": "Draft A", "b": "Draft B"}
+        for _draft in _dual:
+            _s = (_draft.get("style") or "").strip()
+            if _s:
+                _label = _slot_labels.get(
+                    (_draft.get("variant") or "").strip().lower(), "Draft"
+                )
+                lines.append(f"{_label} style: {_s}")
+    elif style:
         lines.append(f"Style: {style}")
     desc = (d.get("style_description") or "").strip()
     if desc:
@@ -741,6 +758,12 @@ class _ReviewController(NSObject):
         # timestamp of any hover still in progress (flushed on decision).
         self._draft_hover_ms = {0: 0, 1: 0}
         self._draft_hover_open = {}
+        # Slots the caret has actually been in this card (2026-07-10 follow-up):
+        # lets the decision distinguish "clicked into B, then came BACK to A"
+        # (an explicit head-to-head choice of A, per user) from "never touched
+        # B at all". Only the UNCHOSEN slot's membership matters at decision
+        # time; the selected slot is trivially visited.
+        self._draft_visited = set()
         # Attention anchors for the unattended-review watchdog: the stack counts
         # as "touched" on present, on any tracked interaction, and on any
         # decision. No touch past the watchdog threshold = the user is not
@@ -1070,6 +1093,7 @@ class _ReviewController(NSObject):
             self._selected_draft = None
             self._draft_hover_ms = {0: 0, 1: 0}
             self._draft_hover_open = {}
+            self._draft_visited = set()
         self._reason_field = None
         content = NSView.alloc().initWithFrame_(NSMakeRect(0, 0, W, H))
 
@@ -1688,11 +1712,18 @@ class _ReviewController(NSObject):
         except Exception:
             return
         for slot, cand_tv in (self._draft_textviews or {}).items():
-            if cand_tv is tv and slot != self._selected_draft:
+            if cand_tv is not tv:
+                continue
+            # Visited even when it's already the selected slot: membership of
+            # the eventually-UNCHOSEN slot is what _record reads, and that
+            # slot only ever gets the caret via a deliberate user click (the
+            # auto-focus seat in _render targets the selected slot only).
+            self._draft_visited.add(slot)
+            if slot != self._selected_draft:
                 self._selected_draft = slot
                 self._textview = cand_tv
                 self._update_draft_borders()
-                break
+            break
 
     @objc.python_method
     def _update_draft_borders(self):
@@ -1779,6 +1810,10 @@ class _ReviewController(NSObject):
                 "unchosen_style": other.get("style") or None,
                 "hover_a_ms": int(self._draft_hover_ms.get(0, 0)),
                 "hover_b_ms": int(self._draft_hover_ms.get(1, 0)),
+                # True = the caret was in the unchosen box at some point, i.e.
+                # they tried the other draft and came back: an explicit choice
+                # even when the winner is the preselected default.
+                "visited_other": bool((1 - sel_idx) in self._draft_visited),
             }
         else:
             orig = (d.get("reply_text") or "").strip()
@@ -1958,10 +1993,11 @@ class _ReviewController(NSObject):
         content = NSView.alloc().initWithFrame_(NSMakeRect(0, 0, W, H))
 
         # Buttons at the TOP, above the title, mirroring the card's action
-        # row so the cursor doesn't have to travel between steps: "Reject,
-        # no reason" sits right-aligned exactly where Reject was (one-click
-        # double-tap keeps the old zero-friction reject), Back sits at the
-        # left where Approve was.
+        # row so the cursor doesn't have to travel between steps: "Reject"
+        # sits exactly where the card's Reject was (one-click double-tap
+        # keeps the old zero-friction reject) and commits with no category,
+        # since picking one is optional; Back sits at the left where Approve
+        # was.
         back = NSButton.alloc().initWithFrame_(NSMakeRect(M, H - 42, 90, 30))
         back.setTitle_("Back")
         back.setBezelStyle_(NSBezelStyleRounded)
@@ -1969,8 +2005,8 @@ class _ReviewController(NSObject):
         back.setAction_("rejectBack:")
         content.addSubview_(back)
 
-        skip = NSButton.alloc().initWithFrame_(NSMakeRect(W - M - 150, H - 42, 150, 30))
-        skip.setTitle_("Reject, no reason")
+        skip = NSButton.alloc().initWithFrame_(NSMakeRect(W - M - 66, H - 42, 66, 30))
+        skip.setTitle_("Reject")
         skip.setBezelStyle_(NSBezelStyleRounded)
         try:
             skip.setHasDestructiveAction_(True)  # red title on macOS 12+
@@ -1981,7 +2017,7 @@ class _ReviewController(NSObject):
         content.addSubview_(skip)
 
         content.addSubview_(
-            _label(NSMakeRect(M, H - 72, W - 2 * M, 20), "Why reject this draft?", size=13, bold=True)
+            _label(NSMakeRect(M, H - 72, W - 2 * M, 20), "Why reject this draft? (optional)", size=13, bold=True)
         )
         y = H - 106
         for i, (_, title) in enumerate(REJECT_REASONS):
@@ -2029,7 +2065,7 @@ class _ReviewController(NSObject):
         note.setEditable_(True)
         note.setFont_(NSFont.systemFontOfSize_(12))
         try:
-            note.setPlaceholderString_("Optional note (sent with whichever reason you pick)")
+            note.setPlaceholderString_("Optional note (sent with or without a reason)")
             note.cell().setWraps_(True)
         except Exception:
             pass