agentic-comms 0.9.0__tar.gz → 0.9.2__tar.gz

{agentic_comms-0.9.0 → agentic_comms-0.9.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-comms
-Version: 0.9.0
+Version: 0.9.2
 Summary: CLI message board for AI agents — coordinate between sessions, projects, and machines
 Author: jazcogames
 License: MIT

{agentic_comms-0.9.0 → agentic_comms-0.9.2}/agent_comms/hook.py

@@ -677,9 +677,15 @@ def _build_wrong_handle_warning(armed_handle: str, armed_pid: int, current_handl
     )
 
 
-def _check_wrong_handle_monitors(current_handle: str) -> list[tuple[str, int]]:
+def _check_wrong_handle_monitors(current_handle: str, claude_pid: int | None) -> list[tuple[str, int]]:
     """Scan ~/.cache/agent-comms/monitor-armed-* for any live Monitor (last_beat
-    within 60s) whose handle != current_handle. Returns list of (handle, pid).
+    within 60s) whose handle != current_handle AND whose process is a child of
+    THIS Claude session.
+
+    The claude_pid ancestry filter eliminates the false-positive where two
+    Claude sessions share a host: each session has its own legitimate Monitor
+    with its own (correct-for-that-session) handle. Without the filter we'd
+    flag the OTHER session's perfectly-valid Monitor as wrong.
 
     A file present but stale (>60s since last_beat) means the Monitor crashed
     without atexit cleanup — ignored here, the regular `monitor_not_armed`
@@ -695,12 +701,22 @@ def _check_wrong_handle_monitors(current_handle: str) -> list[tuple[str, int]]:
         except Exception:
             continue
         last_beat = float(data.get("last_beat") or 0)
-        if now - last_beat > 60:  # stale; covered by monitor_not_armed path
+        if now - last_beat > 60:
             continue
         armed_handle = data.get("handle") or ""
         armed_pid = int(data.get("pid") or 0)
-        if armed_handle and armed_handle != current_handle:
-            wrong.append((armed_handle, armed_pid))
+        if not armed_handle or armed_handle == current_handle:
+            continue
+        # Ancestry filter: walk armed_pid's PPID chain. Only flag as "wrong"
+        # if our claude_pid appears in that ancestry — meaning this Monitor
+        # was spawned by THIS Claude session but somehow got the wrong handle.
+        # If claude_pid isn't found, the Monitor belongs to a sibling Claude
+        # session (legitimate, leave it alone).
+        if claude_pid and armed_pid:
+            chain = config._ppid_chain(armed_pid)
+            if claude_pid not in chain:
+                continue
+        wrong.append((armed_handle, armed_pid))
     return wrong
 
 
@@ -861,7 +877,7 @@ def main() -> int:
         # then loop forever as the operator re-armed-with-stale-handle and
         # re-armed-with-stale-handle.
         try:
-            wrong_monitors = _check_wrong_handle_monitors(handle)
+            wrong_monitors = _check_wrong_handle_monitors(handle, claude_pid)
             for armed_handle, armed_pid in wrong_monitors:
                 early_pieces.append(_build_wrong_handle_warning(armed_handle, armed_pid, handle))
         except Exception:

{agentic_comms-0.9.0 → agentic_comms-0.9.2}/agent_comms/monitor.py

@@ -187,6 +187,15 @@ def main() -> int:
         if not args.quiet:
             sys.stderr.write(f"agent_comms.monitor: startup pending failed: {e}\n")
 
+    # Dedicated monitor_heartbeat POST cadence (every ~10s). Separate from the
+    # inbox poll so we don't spam the server with metadata at 1Hz. The endpoint
+    # stores {pid, version, handle} in identities.meta.monitor server-side —
+    # used for wrong-handle detection across machines and version-drift
+    # diagnostics. Wrapped in try/except so a server that doesn't have the
+    # endpoint (pre-0.9 deployments) doesn't break the poll loop.
+    last_meta_beat = 0.0
+    META_BEAT_INTERVAL = 10.0
+
     # Main loop. Refresh self-handle file every iteration so stale files (Monitor
     # crashed mid-loop) are detectable by mtime / last_beat.
     while True:
@@ -202,6 +211,21 @@ def main() -> int:
         except Exception as e:
             if not args.quiet:
                 sys.stderr.write(f"agent_comms.monitor: poll failed (will retry): {e}\n")
+        # Send dedicated metadata heartbeat every ~10s (Phil's review note:
+        # version=unknown showed up server-side because we were only piggybacking
+        # on ?monitor=true inbox poll and never POSTing the new endpoint).
+        now = time.time()
+        if now - last_meta_beat >= META_BEAT_INTERVAL:
+            try:
+                client.monitor_heartbeat(args.handle, pid=os.getpid())
+                last_meta_beat = now
+            except ClientTooOldError:
+                sys.stderr.write(_too_old_message() + "\n")
+                return 2
+            except Exception:
+                # Server may be pre-0.9 (no endpoint) — fall back silently;
+                # the inbox poll's monitor=true bump still keeps online=True.
+                last_meta_beat = now  # back off so we don't spam-fail
         time.sleep(max(0.1, args.interval))
 
 

{agentic_comms-0.9.0 → agentic_comms-0.9.2}/agentic_comms.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentic-comms
-Version: 0.9.0
+Version: 0.9.2
 Summary: CLI message board for AI agents — coordinate between sessions, projects, and machines
 Author: jazcogames
 License: MIT

{agentic_comms-0.9.0 → agentic_comms-0.9.2}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "agentic-comms"
-version = "0.9.0"
+version = "0.9.2"
 description = "CLI message board for AI agents — coordinate between sessions, projects, and machines"
 readme = "README.md"
 requires-python = ">=3.10"