swarph-cli 0.72.43__tar.gz → 0.72.44__tar.gz

{swarph_cli-0.72.43/src/swarph_cli.egg-info → swarph_cli-0.72.44}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: swarph-cli
-Version: 0.72.43
+Version: 0.72.44
 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`, per-invocation `gh` identity routing, a silent DM-wake hook, session `import`, and a stranded-session `watchdog`.
 Author: Pierre Samson, Claude Opus
 License: MIT

{swarph_cli-0.72.43 → swarph_cli-0.72.44}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "swarph-cli"
-version = "0.72.43"
+version = "0.72.44"
 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`, per-invocation `gh` identity routing, a silent DM-wake hook, session `import`, and a stranded-session `watchdog`."
 readme = "README.md"
 license = { text = "MIT" }

{swarph_cli-0.72.43 → swarph_cli-0.72.44}/src/swarph_cli/__init__.py

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

{swarph_cli-0.72.43 → swarph_cli-0.72.44}/src/swarph_cli/commands/triage.py

@@ -62,8 +62,9 @@ def _stuck_records(rows, by_id, peers, http_get, gateway, token, now,
         if waiter.get("status") != "open" or not waiter.get("taken_at"):
             continue
         step = waiter.get("step")
-        if not step:
-            continue
+        # A waiter with no step still matches a graph entry whose step is
+        # the same empty value. The DM prints a dash, not None (#1478).
+        waiter_step = step if step else "-"
         card_id = waiter.get("card_id")
         if not isinstance(card_id, int) or isinstance(card_id, bool):
             continue
@@ -91,6 +92,8 @@ def _stuck_records(rows, by_id, peers, http_get, gateway, token, now,
             base = {
                 "source_organ": _sensor.ORGAN_STUCK,
                 "waiter_id": waiter.get("id"),
+                "waiter": waiter.get("holder"),
+                "waiter_step": waiter_step,
                 "card_id": card_id,
                 "need": need.get("step"),
                 "need_id": rid,
@@ -109,6 +112,7 @@ def _stuck_records(rows, by_id, peers, http_get, gateway, token, now,
             holder = dep.get("holder")
             blocker = (holder if holder and dep.get("holder_known", True)
                        else None)
+            age_days = None
             if dep.get("status") == "closed":
                 classified = _sensor.map_stuck_finding(
                     {"blocker": blocker, "id": waiter.get("id"),
@@ -118,6 +122,7 @@ def _stuck_records(rows, by_id, peers, http_get, gateway, token, now,
                 changed = _sweep._row_last_change(dep)
                 if changed is None or now - changed < _sweep._STUCK_H * 3600:
                     continue
+                age_days = _sweep._age_days(now, changed)
                 need_card_id = dep.get("card_id")
                 due_at = (_need_card_due_at(http_get, gateway, token,
                                             need_card_id, card_cache)
@@ -135,7 +140,7 @@ def _stuck_records(rows, by_id, peers, http_get, gateway, token, now,
                 evidence=_sensor.stuck_evidence(waiter.get("id"), rid),
                 item_id=(f"stuck:{waiter.get('id')}:{rid}:"
                          f"{need.get('step')}"),
-                security=False)
+                security=False, age_days=age_days)
             records.append(record)
     return records
 
@@ -175,16 +180,17 @@ def run_triage(*, gateway, token, sender, owners_map, state_file,
     sent = _load_state(state_file)
 
     def dm_sender(record, owner):
+        raw_step = record.get("waiter_step")
         finding = {
             "kind": "row",
             "waiter_id": record.get("waiter_id"),
-            "waiter": None,
-            "waiter_step": None,
+            "waiter": record.get("waiter"),
+            "waiter_step": raw_step if raw_step else "-",
             "card_id": record.get("card_id"),
             "need": record.get("need"),
             "need_id": record.get("need_id"),
             "blocker": record.get("owner_hint"),
-            "age_days": None,
+            "age_days": record.get("age_days"),
         }
         key = _sweep._finding_key(finding)
         try:

{swarph_cli-0.72.43 → swarph_cli-0.72.44/src/swarph_cli.egg-info}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: swarph-cli
-Version: 0.72.43
+Version: 0.72.44
 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`, per-invocation `gh` identity routing, a silent DM-wake hook, session `import`, and a stranded-session `watchdog`.
 Author: Pierre Samson, Claude Opus
 License: MIT

{swarph_cli-0.72.43 → swarph_cli-0.72.44}/src/swarph_cli.egg-info/SOURCES.txt

@@ -285,6 +285,7 @@ tests/test_1452_rework2_engine.py
 tests/test_1452_rework3_engine.py
 tests/test_1452_standing_wake_expiry.py
 tests/test_1470_gh_router_fail_closed.py
+tests/test_1478_triage_dm.py
 tests/test_183_card_anchor.py
 tests/test_184_notify_honesty.py
 tests/test_201_win32_bash_refusal.py

swarph_cli-0.72.44/tests/test_1478_triage_dm.py

@@ -0,0 +1,99 @@
+"""card #1478 — stuck-row DMs name the waiter, the step, and the age.
+
+run_triage.dm_sender used to hand sweep._dm_content a finding whose
+waiter, waiter_step, and age_days were None, so a blocker read
+"row #N (None, None, card #C) ... unchanged Noned". Rows in this file
+are built in the test, not loaded from a fixture.
+"""
+from __future__ import annotations
+
+from datetime import datetime, timezone
+
+from swarph_cli.commands import triage as triage_cmd
+from swarph_cli.triage import backend as backend
+from swarph_cli.triage import router as router
+
+NOW = 1_790_000_000.0
+HOLDER = "holder-rowan"
+BLOCKER = "blocker-peer"
+STEP = "validate"
+
+
+def _iso(age_s):
+    return datetime.fromtimestamp(NOW - age_s, timezone.utc).isoformat()
+
+
+def _http(graph_steps):
+    posts = []
+
+    def http_get(url, token):
+        if url.endswith("/board/cards/7/graph"):
+            return 200, {"steps": graph_steps}
+        if url.endswith("/board/cards/8"):
+            return 200, {"id": 8, "due_at": None}
+        raise AssertionError(f"unexpected GET {url}")
+
+    def http_post(url, body, token):
+        posts.append((url, body))
+        return 200, {"id": 1, **body}
+
+    return http_get, http_post, posts
+
+
+def _run(tmp_path, waiter, dep, graph_steps):
+    http_get, http_post, posts = _http(graph_steps)
+    rows = [waiter, dep]
+    rc, summary = triage_cmd.run_triage(
+        gateway="http://gw:9", token="tok", sender="triage",
+        owners_map=router.load_owners(triage_cmd._OWNERS_DEFAULT),
+        state_file=str(tmp_path / "state.json"),
+        store_backend=backend.MemoryBackend(),
+        peers_fetcher=None,
+        board_fetcher=lambda: (rows, {r["id"]: r for r in rows}, {BLOCKER}),
+        http_get=http_get, http_post=http_post, now=NOW)
+    assert rc == 0 and summary["dm_sent"] == 1
+    assert len(posts) == 1 and posts[0][0].endswith("/messages")
+    return posts[0][1]["content"]
+
+
+def _waiter(step):
+    row = {
+        "id": 8, "status": "open", "holder": HOLDER, "card_id": 7,
+        "taken_at": _iso(86400), "created_at": _iso(10 * 86400),
+        "holder_known": True,
+    }
+    if step is not None:
+        row["step"] = step
+    return row
+
+
+def _dep():
+    """Open need whose last change (_row_last_change) is taken_at, 3 days ago."""
+    return {
+        "id": 9, "status": "open", "holder": BLOCKER, "card_id": 8,
+        "holder_known": True,
+        "taken_at": _iso(3 * 86400),
+        "created_at": _iso(10 * 86400),
+    }
+
+
+def test_stuck_dm_names_holder_step_and_age(tmp_path):
+    """Known waiter, dependency last changed 3 days ago: holder, step, 3d, no None."""
+    content = _run(
+        tmp_path, _waiter(STEP), _dep(),
+        [{"step": STEP, "needs": [
+            {"step": "build", "row_id": 9, "satisfied": False}]}])
+    assert HOLDER in content
+    assert STEP in content
+    assert "3d" in content
+    assert "None" not in content
+
+
+def test_waiter_with_no_step_prints_dash_not_none(tmp_path):
+    """A waiter with no step prints a dash in the DM, not the string None."""
+    content = _run(
+        tmp_path, _waiter(None), _dep(),
+        [{"step": None, "needs": [
+            {"step": "build", "row_id": 9, "satisfied": False}]}])
+    assert f"({HOLDER}, -, card #7)" in content
+    assert "None" not in content