swarph-cli 0.41.2__tar.gz → 0.41.3__tar.gz

{swarph_cli-0.41.2/src/swarph_cli.egg-info → swarph_cli-0.41.3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: swarph-cli
-Version: 0.41.2
+Version: 0.41.3
 Summary: The `swarph` binary — a multi-LLM CLI and mesh-gateway client + bundled server (`swarph gateway`): one-shot prompts across providers, interactive `chat`, multi-provider `spawn` (claude/codex/antigravity via a ProviderMembrane with subprocess billing-scrub), `mesh` send/inbox/register, `brain-ask` over the swarph-brain (gbrain) semantic memory, git-backed `assisted_memory`, session `import`, and a stranded-session `watchdog`.
 Author: Pierre Samson, Claude Opus
 License: MIT

{swarph_cli-0.41.2 → swarph_cli-0.41.3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "swarph-cli"
-version = "0.41.2"
+version = "0.41.3"
 description = "The `swarph` binary — a multi-LLM CLI and mesh-gateway client + bundled server (`swarph gateway`): one-shot prompts across providers, interactive `chat`, multi-provider `spawn` (claude/codex/antigravity via a ProviderMembrane with subprocess billing-scrub), `mesh` send/inbox/register, `brain-ask` over the swarph-brain (gbrain) semantic memory, git-backed `assisted_memory`, session `import`, and a stranded-session `watchdog`."
 readme = "README.md"
 license = { text = "MIT" }

{swarph_cli-0.41.2 → swarph_cli-0.41.3}/src/swarph_cli/__init__.py

@@ -16,6 +16,6 @@ The architecture splits CLI from substrate so:
 
 from __future__ import annotations
 
-__version__ = "0.41.2"
+__version__ = "0.41.3"
 
 __all__ = ["__version__"]

{swarph_cli-0.41.2 → swarph_cli-0.41.3}/src/swarph_cli/commands/board_merge_check.py

@@ -161,15 +161,41 @@ def decide(card: dict, pr_state: dict) -> Decision:
         # vacuous for exactly the backlog it must cover. ok=None => COULD NOT
         # EVALUATE => REFUSE, which is this file's existing doctrine for the
         # unreachable-GitHub case and the reason it has a tri-state at all.
+        # >>> A STAMP THAT EXISTS AND LACKS THE KEY IS AS BLIND AS NO STAMP AT ALL,
+        # AND IT IS 92% OF PRODUCTION. <<< (droplet, running decide() on the
+        # INSTALLED 0.41.2 rather than reading it.)
+        #
+        # The first version tested `_vstamp is None` — ABSENCE OF THE STAMP. The
+        # backlog's actual shape is a stamp that EXISTS (written before
+        # self_authored shipped) and lacks the KEY. That fell through to the
+        # comparison, where `.get("self_authored")` returns None, `not None` is
+        # True, and the leg reported ok=True. A textbook self-review PASSED.
+        #
+        # MEASURED: 53 link_stamps in production, 4 carrying the flag — 49 (92%)
+        # are the middle shape. So the leg was VACUOUS FOR EXACTLY THE BACKLOG IT
+        # WAS BUILT FOR: the outcome the design explicitly targets, and which I
+        # reported to the date-holder as avoided. He authorised near-total
+        # ABSTENTION on the backlog; what shipped was near-total PASS — the
+        # opposite, and MORE PERMISSIVE THAN THE PHASE-1 STATE IT REPLACES.
+        #
+        # The two blind causes are kept DISTINCT in the detail because they decay
+        # differently: a missing stamp is a producer that dropped it (an outage
+        # shape); a missing key is a verdict older than the flag (a shape that
+        # drains to zero as verdicts are rewritten). A single message would make
+        # them indistinguishable, which is the naming rule one level down.
         _vstamp = (card.get("link_stamps") or {}).get("peer_verdict")
-        if _vstamp is None:
+        if _vstamp is None or "self_authored" not in _vstamp:
+            _why = ("no link_stamp for peer_verdict at all — a producer dropped it, "
+                    "or the verdict predates #199"
+                    if _vstamp is None else
+                    f"link_stamp exists (by={_vstamp.get('by')!r}) but carries NO "
+                    f"self_authored key — the verdict predates the flag")
             add("verdict is not self-authored", None,
-                "no link_stamp for peer_verdict — cannot tell who wrote it "
-                "(pre-#199 verdict, or a producer that dropped link_stamps); "
-                "COULD NOT EVALUATE, never a pass")
+                f"{_why}; cannot tell whether the reviewee wrote it. "
+                f"COULD NOT EVALUATE, never a pass")
         else:
-            add("verdict is not self-authored", not _vstamp.get("self_authored"),
-                f"self_authored={_vstamp.get('self_authored')!r} by={_vstamp.get('by')!r} "
+            add("verdict is not self-authored", not _vstamp["self_authored"],
+                f"self_authored={_vstamp['self_authored']!r} by={_vstamp.get('by')!r} "
                 f"— the person being reviewed cannot supply the review")
         add("verdict is APPROVED", verdict["verdict"] == "APPROVED",
             f"verdict={verdict['verdict']}")

{swarph_cli-0.41.2 → swarph_cli-0.41.3/src/swarph_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: swarph-cli
-Version: 0.41.2
+Version: 0.41.3
 Summary: The `swarph` binary — a multi-LLM CLI and mesh-gateway client + bundled server (`swarph gateway`): one-shot prompts across providers, interactive `chat`, multi-provider `spawn` (claude/codex/antigravity via a ProviderMembrane with subprocess billing-scrub), `mesh` send/inbox/register, `brain-ask` over the swarph-brain (gbrain) semantic memory, git-backed `assisted_memory`, session `import`, and a stranded-session `watchdog`.
 Author: Pierre Samson, Claude Opus
 License: MIT

{swarph_cli-0.41.2 → swarph_cli-0.41.3}/tests/test_board_merge_check.py

@@ -347,3 +347,50 @@ def test_a_READY_card_with_NO_peer_verdict_REFUSES():
     d = mc.decide(_card(links={"pr": "https://github.com/o/r/pull/152"}), _pr())
     assert _fail(d, "peer verdict recorded on the card")
     assert d.verdict != "WOULD_MERGE"
+
+
+def test_a_stamp_WITHOUT_the_self_authored_KEY_abstains_like_no_stamp():
+    """>>> THE BUG THAT SHIPPED IN 0.41.2, AND IT WAS 92% OF PRODUCTION. <<<
+
+    Found by droplet RUNNING decide() on the installed package instead of reading
+    it. The first version tested `_vstamp is None` — absence of the STAMP. The
+    backlog's real shape is a stamp that EXISTS (written before self_authored
+    shipped) and lacks the KEY: it fell through to the comparison, where
+    `.get("self_authored")` returns None, `not None` is True, and A TEXTBOOK
+    SELF-REVIEW PASSED.
+
+    Measured at the time: 53 link_stamps in production, 4 carrying the flag — 49
+    (92%) were this shape. The leg was VACUOUS FOR EXACTLY THE BACKLOG IT WAS
+    BUILT FOR, which is the outcome the design explicitly targets and which I had
+    reported to the date-holder as avoided. He authorised near-total ABSTENTION;
+    what shipped was near-total PASS.
+
+    WHY THE VERIFICATION MISSED IT: the wheel audit and the end-to-end check
+    covered self_authored=True (refuses) and no-stamp (abstains). Both correct,
+    NEITHER IS THE PRODUCTION SHAPE. The untested case was the only one that
+    exists in the data.
+    """
+    stamp_no_key = {"peer_verdict": {"by": "droplet", "caller_bound": True,
+                                     "at": "2026-08-01T00:00:00Z"}}
+    d = mc.decide(_card(created_by="lab-ovh", assignee="droplet",
+                        link_stamps=stamp_no_key), _pr())
+    check = [c for c in d.checks if c.name == "verdict is not self-authored"][0]
+    assert check.ok is None, (
+        "a stamp lacking the self_authored KEY passed the self-review leg — "
+        "vacuous for 92% of production")
+    assert d.verdict != "WOULD_MERGE"
+
+
+def test_the_two_BLIND_causes_are_named_distinctly():
+    """They decay differently and must not print the same: a MISSING STAMP is a
+    producer that dropped it (outage shape); a MISSING KEY is a verdict older than
+    the flag (drains to zero as verdicts are rewritten). One message for both
+    would make a permanent outage indistinguishable from a shrinking backlog."""
+    no_stamp = mc.decide(_card(created_by="lab-ovh", link_stamps={}), _pr())
+    no_key = mc.decide(_card(created_by="lab-ovh",
+                             link_stamps={"peer_verdict": {"by": "droplet"}}), _pr())
+    d1 = [c for c in no_stamp.checks if c.name == "verdict is not self-authored"][0].detail
+    d2 = [c for c in no_key.checks if c.name == "verdict is not self-authored"][0].detail
+    assert d1 != d2, "both blind causes printed the same detail"
+    assert "at all" in d1
+    assert "predates the flag" in d2

{swarph_cli-0.41.2 → swarph_cli-0.41.3}/tests/test_brain_ask_command.py

@@ -17,7 +17,7 @@ from swarph_cli.commands import brain_ask as ba
 
 def test_version_is_0_33_0():
     import swarph_cli
-    assert swarph_cli.__version__ == "0.41.2"
+    assert swarph_cli.__version__ == "0.41.3"
 
 
 # --- request / response plumbing -------------------------------------------

{swarph_cli-0.41.2 → swarph_cli-0.41.3}/tests/test_watchdog.py

@@ -1043,4 +1043,4 @@ def test_parser_defaults_process_name_claude_liveness_cmd_none():
 
 def test_version_is_0_33_0():
     import swarph_cli
-    assert swarph_cli.__version__ == "0.41.2"
+    assert swarph_cli.__version__ == "0.41.3"