agentwire-dev 1.50.0__tar.gz → 1.50.1__tar.gz

{agentwire_dev-1.50.0 → agentwire_dev-1.50.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: agentwire-dev
-Version: 1.50.0
+Version: 1.50.1
 Summary: Multi-session voice web interface for AI coding agents
 Project-URL: Homepage, https://agentwire.dev
 Project-URL: Repository, https://github.com/dotdevdotdev/agentwire-dev

{agentwire_dev-1.50.0 → agentwire_dev-1.50.1}/agentwire/__init__.py

@@ -1,3 +1,3 @@
 """AgentWire - Multi-session voice web interface for AI coding agents."""
 
-__version__ = "1.50.0"
+__version__ = "1.50.1"

{agentwire_dev-1.50.0 → agentwire_dev-1.50.1}/agentwire/hooks/idle-handler.sh

@@ -370,10 +370,17 @@ complete | incomplete | error
           echo "[$(date -Iseconds)] TASK-ORPHAN: killing tmux session" >> "$dlog"
           tmux kill-session -t "$tmux_session" 2>/dev/null &
         ) &
-      elif [[ -n "$parent_session" ]]; then
-        # Orchestrator pane with parent: notify parent
-        message="${session_name} is idle"
-        $AGENTWIRE notify-parent -q --to "$parent_session" "$message" 2>/dev/null &
+      else
+        # Pane-0 session went idle: tell its parent. Resolution lives inside
+        # notify-parent (prompt_router.resolve_parent) so all three precedence
+        # sources are honored — worker pane → pane 0, creator recorded at
+        # `agentwire new` time (session metadata), then `.agentwire.yml parent:`.
+        # The old code passed only the .agentwire.yml value and skipped the call
+        # entirely when it was empty, so worktree / `agentwire new` children
+        # (whose parent lives in metadata, not config) never notified anyone.
+        # TMUX_PANE is inherited, so the CLI knows which session this is; a
+        # top-level session resolves to no parent and this is a harmless no-op.
+        $AGENTWIRE notify-parent --on-idle -q "is idle and done working" 2>/dev/null &
       fi
     fi
   fi

{agentwire_dev-1.50.0 → agentwire_dev-1.50.1}/agentwire/notify_cli.py

@@ -51,11 +51,30 @@ def cmd_notify_parent(args) -> int:
     current_session = pane_manager.get_current_session()
     current_pane = pane_manager.get_current_pane_index()
 
-    # If no explicit target, try parent from config
-    if not target_session:
-        parent = get_parent_from_config()
-        if parent:
-            target_session = parent
+    # --on-idle: the idle hook is reporting that a pane-0 session went idle.
+    # Infrastructure services (portal, tts/stt, scheduler, the idle-nag bridge,
+    # custom services) cycle active→idle constantly and are NOT delegated work,
+    # so they must never fire a "child finished" ping at their parent. A
+    # worktree / `agentwire new` child is not a service and is unaffected.
+    if getattr(args, 'on_idle', False) and current_session:
+        from agentwire import services
+
+        if services.is_service_session(current_session):
+            return _output_result(True, json_mode, "", skipped="service-session")
+
+    # If no explicit target, resolve the parent through the SAME precedence the
+    # prompt router uses (worker pane → pane 0; else creator recorded at
+    # `agentwire new` time; else `.agentwire.yml parent:`). The old path looked
+    # only at `.agentwire.yml`, so a worktree/`agentwire new` child — whose
+    # parent lives in session metadata, not config — resolved to nothing and its
+    # idle notification silently dropped (the parent never heard the child go
+    # idle). resolve_parent reads the creator metadata, closing that gap.
+    if not target_session and current_session is not None and current_pane is not None:
+        from agentwire import prompt_router
+
+        resolved = prompt_router.resolve_parent(current_session, current_pane)
+        if resolved:
+            target_session = resolved[0]
 
     # Build notification message (--raw sends verbatim — queued messages
     # already carry their own [WORKER SUMMARY ...] / [PROMPT ...] headers)
@@ -255,6 +274,9 @@ def register_notify_parser(subparsers) -> None:
     notify_cmd_parser.add_argument("-q", "--quiet", action="store_true", help="Suppress output")
     notify_cmd_parser.add_argument("--raw", action="store_true",
                                    help="Send the message verbatim (no [NOTIFY from ...] prefix)")
+    notify_cmd_parser.add_argument("--on-idle", dest="on_idle", action="store_true",
+                                   help="Idle-hook mode: suppress the notify if the current session "
+                                        "is an infrastructure service (they cycle idle constantly)")
     notify_cmd_parser.add_argument("--json", action="store_true", help="Output as JSON")
     notify_cmd_parser.set_defaults(func=cmd_notify_parent)
 

{agentwire_dev-1.50.0 → agentwire_dev-1.50.1}/agentwire/services.py

@@ -78,6 +78,42 @@ def notifications_session_name() -> str:
     )
 
 
+# Built-in infrastructure sessions — the Python mirror of the frontend's
+# SERVICE_SESSIONS allowlist (static/js/service-classification.js). These are
+# long-lived services, not delegated work, so they must never fire a
+# "child finished" idle notification at a parent (the idle-nag bridge alone
+# cycles active→idle ~1440×/day). Kept in lockstep with the JS set.
+_BUILTIN_SERVICE_SESSIONS = frozenset({
+    "agentwire-portal",
+    "agentwire-tts",
+    "agentwire-stt",
+    "agentwire-kokoro",
+    "agentwire-scheduler",
+})
+
+
+def is_service_session(name: str) -> bool:
+    """Is *name* an infrastructure service session (vs. a working session)?
+
+    True for the built-in infra sessions, the configurable idle-nag bridge, and
+    any user-defined custom service. Used to suppress idle parent-notifications
+    for services — a worktree / `agentwire new` child is NOT a service and still
+    notifies. Name-only and dependency-light so the idle hook can call it cheaply
+    on every idle event.
+    """
+    bare = (name or "").split("@")[0]
+    if not bare:
+        return False
+    if bare in _BUILTIN_SERVICE_SESSIONS or bare == notifications_session_name():
+        return True
+    try:
+        from .config import load_config
+
+        return any(s.name == bare for s in load_config().services.custom)
+    except Exception:
+        return False
+
+
 def _source_dir() -> str:
     """agentwire source dir (dev.source_dir), project for built-in services."""
     try:

{agentwire_dev-1.50.0 → agentwire_dev-1.50.1}/docs/wiki/sessions/prompt-routing.md

@@ -37,6 +37,18 @@ Depth-1 and local-machine only. Remote (`@machine`) parents are out of scope:
 each machine's own watchdog sweeps its panes; cross-machine delivery falls
 back to human-only.
 
+**Idle notifications use the same resolution.** When a pane-0 session goes idle,
+`idle-handler.sh` calls `agentwire notify-parent --on-idle`, which resolves the
+parent through the precedence above — so a worktree / `agentwire new` child whose
+parent lives in **creator metadata** (not `.agentwire.yml`) now correctly pings
+its spawner on completion. Earlier the idle hook read only the `.agentwire.yml`
+`parent:` field and silently dropped the notification when it was empty.
+`--on-idle` additionally suppresses the ping when the source is an infrastructure
+**service** (`services.is_service_session` — portal/tts/stt/kokoro/scheduler, the
+idle-nag bridge, custom services); those cycle active→idle constantly and aren't
+delegated work. Worker-pane (`notify-parent` from pane > 0) and explicit `--to`
+callers skip the service check.
+
 ## Delivery safety
 
 Every delivery goes through `safe_deliver()` (also used by