wcode_protocol/

registry.rs

//! In-process address book + router — the A2A bus (§8 #3, §10.1).
//!
//! Two pieces: an **address book** (`SessionId` → `Backend`: an in-process
//! handle or a remote `Client`) and an **ownership** map (the `report_back_to`
//! **ownership** map (the `report_back_to` edge: worker → orchestrator). Routing
//! a message checks the **permitted set** — an orchestrator reaches its workers,
//! a worker its orchestrator, and nothing else (v1 star, §10.1) — then delivers
//! through the target's handle carrying the sender, so the recipient can
//! attribute it (`AgentEvent::MessageReceived { from }`).
//!
//! Addressing lives *here*, one layer above the kernel: a [`Request`] is
//! target-side and names no peer; the router turns a `to` address into a mailbox
//! given the sender's scope. The future **group** mode widens only the permitted
//! set (§10.1) — nothing else here changes.

use std::collections::{HashMap, HashSet};
use std::sync::{Arc, Mutex};

use tokio::sync::watch;
use wcode_harness::actor::SessionHandle;
use wcode_harness::protocol::{MemberState, Request, SessionId};

use crate::backend::Backend;
#[cfg(unix)]
use crate::client::Client;

/// Why a message could not be routed (§10.1 enforcement).
#[derive(Debug, Clone, PartialEq, Eq)]
pub enum RouteError {
    /// No peer is registered at that address.
    Unknown(SessionId),
    /// The sender may not address that peer — outside the permitted set.
    NotPermitted { from: SessionId, to: SessionId },
    /// The target's session has shut down (its actor ended).
    Closed,
}

impl std::fmt::Display for RouteError {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            RouteError::Unknown(id) => write!(f, "no peer at {id}"),
            RouteError::NotPermitted { from, to } => {
                write!(f, "{from} may not address {to}")
            }
            RouteError::Closed => f.write_str("the target session is closed"),
        }
    }
}

impl std::error::Error for RouteError {}

/// An in-process address book and router over session mailboxes.
///
/// Cloneable and shared: every clone refers to the same registry. There is no
/// automatic deregistration — a peer that shuts down stays listed, and a delivery
/// to it fails with [`RouteError::Closed`] (a sweep can come later; §10.1 keeps
/// this minimal).
#[derive(Clone, Default)]
pub struct Registry {
    inner: Arc<Inner>,
}

struct Inner {
    peers: Mutex<HashMap<SessionId, Backend>>,
    owners: Mutex<HashMap<SessionId, SessionId>>,
    /// Each session's effective model id (see [`Registry::set_model`]), so a
    /// socket server can name a member's model in the roster it pushes.
    models: Mutex<HashMap<SessionId, String>>,
    /// Server-side liveness per session, mirrored from the actor's `watch` cell
    /// so the roster push (and the `Sessions` reply) can name it — the twin of
    /// `models`. Cheap: one entry per served session. `Idle` for a session the
    /// registry never watched, or an older peer (a `state`-less `SessionInfo`).
    states: Mutex<HashMap<SessionId, MemberState>>,
    /// Each session's effective effort (see [`Registry::set_effort`]).
    efforts: Mutex<HashMap<SessionId, Option<String>>>,
    /// Each session's context window in tokens (see [`Registry::set_context_window`]).
    windows: Mutex<HashMap<SessionId, u64>>,
    /// The provider's advertised catalog — ONE value for the whole registry
    /// (connection), not per session (see [`Registry::set_models`]).
    catalog: Mutex<Vec<String>>,
    /// The live **local** roster (see [`Registry::locals`]), refreshed on every
    /// `register`/`register_remote` so a socket server can fan a session that
    /// appears after it started. Read via [`Registry::subscribe`].
    roster: watch::Sender<Vec<(SessionId, SessionHandle)>>,
    /// Explicit workflow-edge adjacencies: `(a, b)` means `a` and `b` may
    /// message each other (both directions stored). NOT a mesh — only
    /// edge-joined workflow nodes are inserted ([`Registry::allow_pair`],
    /// sign-off rev-2 #6).
    allows: Mutex<HashSet<(SessionId, SessionId)>>,
}

impl Default for Inner {
    fn default() -> Self {
        Self {
            peers: Mutex::new(HashMap::new()),
            owners: Mutex::new(HashMap::new()),
            models: Mutex::new(HashMap::new()),
            states: Mutex::new(HashMap::new()),
            efforts: Mutex::new(HashMap::new()),
            windows: Mutex::new(HashMap::new()),
            catalog: Mutex::new(Vec::new()),
            roster: watch::channel(Vec::new()).0,
            allows: Mutex::new(HashSet::new()),
        }
    }
}

impl Registry {
    pub fn new() -> Self {
        Self::default()
    }

    /// Register a peer's mailbox at `id`; re-registering replaces it.
    pub fn register(&self, id: SessionId, handle: SessionHandle) {
        // Mirror the session's run-state into the roster: the registry keeps
        // NO back-reference into the kernel (layering) — it watches the
        // handle's `state_rx()` and re-publishes on every transition, so the
        // socket server pushes the change without polling. One task per served
        // session (a handful); it ends when the session drops.
        let mut state_rx = handle.state_rx();
        self.inner
            .peers
            .lock()
            .unwrap()
            .insert(id.clone(), Backend::Local(handle));
        let registry = self.clone();
        tokio::spawn(async move {
            while state_rx.changed().await.is_ok() {
                registry.set_state(id.clone(), *state_rx.borrow_and_update());
            }
        });
        self.refresh_roster();
    }

    /// Register a peer reachable **across a socket** — a served session this
    /// process can message, and which can reach back if it knows us (§8, S4-4).
    #[cfg(unix)]
    pub fn register_remote(&self, id: SessionId, client: Client) {
        self.inner
            .peers
            .lock()
            .unwrap()
            .insert(id, Backend::Remote(client.clone()));
        // Remote liveness rides the peer's own roster push: a served peer
        // stamps each `SessionInfo` with a `state`, so mirroring the remote
        // client's roster into `states` makes `state_of` reflect a remote
        // member with no back-channel of our own.
        let registry = self.clone();
        let mut roster = client.subscribe_roster();
        tokio::spawn(async move {
            while roster.changed().await.is_ok() {
                let sessions = roster.borrow_and_update().clone();
                for info in &sessions {
                    registry.set_state(info.id.clone(), info.state);
                }
            }
        });
        self.refresh_roster();
    }

    /// The **local** peers — the in-process handles this registry holds, sorted
    /// by address for a deterministic roster. A socket server serves exactly
    /// these ([`Self::register`]ed sessions: the root and its team); a peer
    /// reached across a socket ([`Self::register_remote`]) is not a local handle
    /// and is omitted, so it is never re-served.
    pub fn locals(&self) -> Vec<(SessionId, SessionHandle)> {
        let mut locals: Vec<(SessionId, SessionHandle)> = self
            .inner
            .peers
            .lock()
            .unwrap()
            .iter()
            .filter_map(|(id, backend)| match backend {
                Backend::Local(handle) => Some((id.clone(), handle.clone())),
                #[cfg(unix)]
                Backend::Remote(_) => None,
            })
            .collect();
        locals.sort_by(|a, b| a.0.as_str().cmp(b.0.as_str()));
        locals
    }

    /// Remove `id` from the address book and every per-session map, then
    /// re-publish the roster.
    ///
    /// Drops the `id` entry from `peers` (the address book), `owners`, `models`,
    /// `states`, `efforts`, and `windows`, then calls [`Self::refresh_roster`]
    /// so every [`Self::subscribe`]r — a socket server, the TUI — pushes the
    /// SHRUNK roster.
    ///
    /// # Notes
    /// - **The per-session state watcher** `register` spawned (`tokio::spawn` of
    ///   a `while state_rx.changed() …` loop) ends only when the session drops.
    ///   It can re-insert a `states` entry after this returns, but NEVER a
    ///   `peers` entry — so [`Self::locals`] (which reads `peers`) stays clean
    ///   and [`Self::contains`]/[`Self::resolve`] report the id gone at once.
    ///   Documented, not fixed (layering: no back-ref).
    /// - Removing an unknown `id` is a harmless no-op (`refresh_roster` still runs).
    ///
    /// Not idempotent-erroring and not socket-routed: this is the ROOT-LOCAL
    /// retire path (no `Request` ends a served peer's session).
    pub fn remove(&self, id: &SessionId) {
        self.inner.peers.lock().unwrap().remove(id);
        self.inner.owners.lock().unwrap().remove(id);
        self.inner.models.lock().unwrap().remove(id);
        self.inner.states.lock().unwrap().remove(id);
        self.inner.efforts.lock().unwrap().remove(id);
        self.inner.windows.lock().unwrap().remove(id);
        self.refresh_roster();
    }

    /// A live view of [`Self::locals`]: a receiver **seeded** with the current
    /// local roster and updated whenever it changes (a session is
    /// `register`ed). A socket server serves it, so a session that appears
    /// *after* the server started is still reachable — see [`crate::serve`].
    pub fn subscribe(&self) -> watch::Receiver<Vec<(SessionId, SessionHandle)>> {
        self.inner.roster.subscribe()
    }

    /// Recompute the local roster and publish it to every [`Self::subscribe`]r.
    /// `send_replace` stores the value even when nobody is watching yet, so a
    /// session registered *before* the first subscriber — a `[team]` member that
    /// starts ahead of the socket server — still seeds a later [`Self::subscribe`].
    /// (`watch::Sender::send` would drop it: it is a no-op with no receivers.)
    fn refresh_roster(&self) {
        let _ = self.inner.roster.send_replace(self.locals());
    }

    /// Whether a peer is already registered at `id`. A plain existence probe —
    /// unlike [`Self::resolve`], it needs no `from` and enforces no permitted set.
    pub fn contains(&self, id: &SessionId) -> bool {
        self.inner.peers.lock().unwrap().contains_key(id)
    }

    /// Record the `report_back_to` edge: `worker` reports to `owner` (its
    /// orchestrator). This is the whole ownership tree — a star in v1.
    pub fn set_owner(&self, worker: SessionId, owner: SessionId) {
        self.inner.owners.lock().unwrap().insert(worker, owner);
    }

    /// Allow `from` ↔ `to` to address each other — a workflow edge's adjacency
    /// (A1). Inserts BOTH directions, so either node may open the conversation.
    /// Idempotent. Explicit, not a mesh: only edge-joined workflow nodes are
    /// ever registered here; a plain `[team]` registers none, so its permitted
    /// set is today's exact star (non-scope: no mesh, no `[team]` widening).
    ///
    /// Does NOT re-publish the roster (like [`Registry::set_owner`]) — adjacency
    /// names no session metadata.
    pub fn allow_pair(&self, from: &SessionId, to: &SessionId) {
        let mut allows = self.inner.allows.lock().unwrap();
        allows.insert((from.clone(), to.clone()));
        allows.insert((to.clone(), from.clone()));
    }

    /// Record a session's effective model id, so the roster this registry feeds
    /// (a socket server's `Sessions` push) can name it. Mirrors [`Self::set_owner`]
    /// — a plain metadata write, no effect on routing; it RE-PUBLISHES the roster.
    pub fn set_model(&self, id: SessionId, model: impl Into<String>) {
        self.inner.models.lock().unwrap().insert(id, model.into());
        self.refresh_roster();
    }

    /// Mirror a session's run-state. Called by the per-session watcher
    /// `register` spawns — NOT by the actor, so the registry keeps NO
    /// back-reference into the kernel (layering). Re-publishes the roster, so
    /// the socket server pushes the change without polling.
    pub fn set_state(&self, id: SessionId, state: MemberState) {
        self.inner.states.lock().unwrap().insert(id, state);
        self.refresh_roster();
    }

    /// The state last mirrored for `id`; `Idle` when unknown (a remote peer the
    /// registry never watched, or an older server).
    pub fn state_of(&self, id: &SessionId) -> MemberState {
        *self
            .inner
            .states
            .lock()
            .unwrap()
            .get(id)
            .unwrap_or(&MemberState::Idle)
    }
    /// The model recorded for `id`, if any ([`Self::set_model`]). `None` for a
    /// peer registered without one (e.g. a remote reached across a socket).
    pub fn model_of(&self, id: &SessionId) -> Option<String> {
        self.inner.models.lock().unwrap().get(id).cloned()
    }

    /// Record a session's effective effort. A metadata write, no routing effect —
    /// but it RE-PUBLISHES the roster, so a socket push reflects it.
    pub fn set_effort(&self, id: SessionId, effort: Option<String>) {
        self.inner.efforts.lock().unwrap().insert(id, effort);
        self.refresh_roster();
    }

    /// The effort last recorded for `id`; `None` when unset OR cleared.
    pub fn effort_of(&self, id: &SessionId) -> Option<String> {
        self.inner.efforts.lock().unwrap().get(id).cloned().flatten()
    }

    /// Record a session's context window in tokens. RE-PUBLISHES the roster.
    pub fn set_context_window(&self, id: SessionId, window: u64) {
        self.inner.windows.lock().unwrap().insert(id, window);
        self.refresh_roster();
    }

    /// The window last recorded for `id`; `None` when unknown.
    pub fn window_of(&self, id: &SessionId) -> Option<u64> {
        self.inner.windows.lock().unwrap().get(id).copied()
    }

    /// Record the connection's model catalog. Does NOT re-publish — the catalog
    /// names no session, so it is not roster data.
    pub fn set_models(&self, models: Vec<String>) {
        *self.inner.catalog.lock().unwrap() = models;
    }

    /// The recorded catalog (empty when unset — `ListModels` then answers `[]`).
    pub fn models(&self) -> Vec<String> {
        self.inner.catalog.lock().unwrap().clone()
    }

    /// Whether `from` may address `to` (§10.1): the pair must be joined by an
    /// ownership edge in either direction — an orchestrator and one of its
    /// workers — OR by an explicit [`Registry::allow_pair`] adjacency: the
    /// workflow-scoped lateral exception (W025 / D030), which amends
    /// `task-dag-plan.md` §11 #1. A plain `[team]` registers no pair, so its
    /// permitted set stays today's exact star.
    pub fn permitted(&self, from: &SessionId, to: &SessionId) -> bool {
        // A7: take-and-drop the `owners` lock FIRST, then lock `allows` — never
        // hold both at once (both are leaf mutexes, but this keeps the order
        // trivially deadlock-free).
        let owned = {
            let owners = self.inner.owners.lock().unwrap();
            owners.get(from) == Some(to) || owners.get(to) == Some(from)
        };
        owned
            || self
                .inner
                .allows
                .lock()
                .unwrap()
                .contains(&(from.clone(), to.clone()))
    }

    /// Resolve `to` for `from`, enforcing the permitted set. Fails
    /// [`RouteError::Unknown`] if no peer is registered, else
    /// [`RouteError::NotPermitted`] if the sender is out of scope.
    pub fn resolve(&self, from: &SessionId, to: &SessionId) -> Result<Backend, RouteError> {
        let endpoint = self
            .inner
            .peers
            .lock()
            .unwrap()
            .get(to)
            .cloned()
            .ok_or_else(|| RouteError::Unknown(to.clone()))?;
        if !self.permitted(from, to) {
            return Err(RouteError::NotPermitted {
                from: from.clone(),
                to: to.clone(),
            });
        }
        Ok(endpoint)
    }

    /// Deliver a target-side `request` to `to`, attributed to `from`. The
    /// recipient sees it as if `from` had sent it directly
    /// ([`SessionHandle::send_from`]).
    pub fn deliver(
        &self,
        from: &SessionId,
        to: &SessionId,
        request: Request,
    ) -> Result<(), RouteError> {
        let endpoint = self.resolve(from, to)?;
        endpoint
            .send_from(from.clone(), request)
            .map_err(|_| RouteError::Closed)
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use std::time::Duration;

    use wcode_harness::actor::SessionActor;
    use wcode_harness::agent::{Agent, AgentConfig};
    use wcode_harness::compaction::CompactionPolicy;
    use wcode_harness::event::{AgentEvent, LlmStreamEvent};
    use wcode_harness::hooks::HooksSet;
    use wcode_harness::loop_::DEFAULT_MAX_TURNS;
    use wcode_harness::streamfn::{LlmOpts, LlmStream, StreamFn};

    use futures::StreamExt as _;

    /// A live session whose model never streams — enough for routing tests (the
    /// messages we send are `Notify`, which run no turn).
    fn session() -> SessionHandle {
        session_with(Arc::new(|_ctx, _sys, _tools, _opts| {
            Box::pin(futures::stream::empty()) as LlmStream
        }))
    }

    /// A session over a caller-supplied stream — for tests that drive a real
    /// turn (a hang, a stream error), not just routing.
    fn session_with(stream_fn: StreamFn) -> SessionHandle {
        SessionActor::spawn(Agent::new(AgentConfig {
            system: "sys".into(),
            tools: Vec::new(),
            llm: LlmOpts {
                model: "m".into(),
                ..LlmOpts::default()
            },
            stream_fn,
            hooks: HooksSet::default(),
            session: None,
            context: Vec::new(),
            working_dir: std::path::PathBuf::new(),
            max_turns: DEFAULT_MAX_TURNS,
            parallel_tools: true,
            compaction: CompactionPolicy::default(),
            plan_mode: wcode_harness::hooks::PlanModeHandle::new(),
        }))
    }

    async fn next_message(
        rx: &mut tokio::sync::broadcast::Receiver<AgentEvent>,
    ) -> AgentEvent {
        tokio::time::timeout(Duration::from_secs(2), rx.recv())
            .await
            .expect("an event")
            .expect("open")
    }

    #[tokio::test]
    async fn orchestrator_and_worker_reach_each_other() {
        let reg = Registry::new();
        let orch = SessionId::agent("orch");
        let worker = SessionId::agent("w1");
        let h_orch = session();
        let h_worker = session();
        reg.register(orch.clone(), h_orch.clone());
        reg.register(worker.clone(), h_worker.clone());
        reg.set_owner(worker.clone(), orch.clone());

        // Orchestrator → worker: the worker receives it, attributed to the orch.
        let mut rx = h_worker.subscribe();
        reg.deliver(
            &orch,
            &worker,
            Request::Notify {
                content: "do the thing".into(),
            },
        )
        .unwrap();
        match next_message(&mut rx).await {
            AgentEvent::MessageReceived { from, content } => {
                assert_eq!(from, orch);
                assert_eq!(content, "do the thing");
            }
            other => panic!("expected MessageReceived, got {other:?}"),
        }

        // Worker → orchestrator (report back): permitted.
        let mut rx = h_orch.subscribe();
        reg.deliver(
            &worker,
            &orch,
            Request::Notify {
                content: "done".into(),
            },
        )
        .unwrap();
        match next_message(&mut rx).await {
            AgentEvent::MessageReceived { from, content } => {
                assert_eq!(from, worker);
                assert_eq!(content, "done");
            }
            other => panic!("expected MessageReceived, got {other:?}"),
        }
    }

    /// `allow_pair(w1, w2)` → `w1` may `deliver` to `w2`, and the reverse — the
    /// workflow-scoped adjacency (A1 / D030).
    #[tokio::test]
    async fn an_adjacent_pair_may_message() {
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        let w2 = SessionId::agent("w2");
        for id in [&w1, &w2] {
            reg.register(id.clone(), session());
        }
        reg.allow_pair(&w1, &w2);
        assert!(reg.permitted(&w1, &w2));
        assert!(reg.permitted(&w2, &w1), "both directions stored");
        reg.deliver(
            &w1,
            &w2,
            Request::Notify {
                content: "psst".into(),
            },
        )
        .unwrap();
    }

    /// Without `allow_pair`, the star bound survives (this REPLACES
    /// `lateral_message_between_workers_is_refused`): w1/w2 both owned by `orch`,
    /// neither paired → `NotPermitted`.
    #[tokio::test]
    async fn a_non_adjacent_pair_is_still_refused() {
        let reg = Registry::new();
        let orch = SessionId::agent("orch");
        let w1 = SessionId::agent("w1");
        let w2 = SessionId::agent("w2");
        for id in [&orch, &w1, &w2] {
            reg.register(id.clone(), session());
        }
        reg.set_owner(w1.clone(), orch.clone());
        reg.set_owner(w2.clone(), orch.clone());

        assert_eq!(
            reg.deliver(
                &w1,
                &w2,
                Request::Notify {
                    content: "psst".into()
                }
            ),
            Err(RouteError::NotPermitted {
                from: w1.clone(),
                to: w2.clone()
            })
        );
        assert!(!reg.permitted(&w1, &w2));
    }

    #[tokio::test]
    async fn unknown_target_is_reported() {
        let reg = Registry::new();
        let orch = SessionId::agent("orch");
        reg.register(orch.clone(), session());

        let ghost = SessionId::agent("ghost");
        assert_eq!(
            reg.deliver(&orch, &ghost, Request::Notify { content: "?".into() }),
            Err(RouteError::Unknown(ghost))
        );
    }

    #[tokio::test]
    async fn locals_lists_only_local_peers_sorted_by_id() {
        let reg = Registry::new();
        let b = SessionId::agent("b");
        let a = SessionId::agent("a");
        reg.register(b.clone(), session());
        reg.register(a.clone(), session());
        #[cfg(unix)]
        reg.register_remote(
            SessionId::agent("remote"),
            Client::lazy(std::path::Path::new("/nonexistent-t2.sock")),
        );
        let ids: Vec<SessionId> = reg.locals().into_iter().map(|(id, _)| id).collect();
        assert_eq!(ids, vec![a, b]);
    }

    #[tokio::test]
    async fn set_model_is_read_back_per_id() {
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        let ghost = SessionId::agent("ghost");
        assert_eq!(reg.model_of(&w1), None, "unset id has no model");
        reg.set_model(w1.clone(), "m1");
        assert_eq!(reg.model_of(&w1).as_deref(), Some("m1"));
        assert_eq!(reg.model_of(&ghost), None);
    }

    /// Await the next roster change, failing the test if it never comes.
    async fn next_change(rx: &mut watch::Receiver<Vec<(SessionId, SessionHandle)>>) {
        tokio::time::timeout(Duration::from_secs(2), rx.changed())
            .await
            .expect("a roster change")
            .expect("open");
    }

    #[tokio::test]
    async fn set_effort_window_catalog_are_read_back() {
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        assert_eq!(reg.effort_of(&w1), None);
        reg.set_effort(w1.clone(), Some("high".into()));
        assert_eq!(reg.effort_of(&w1).as_deref(), Some("high"));
        reg.set_effort(w1.clone(), None); // cleared == None
        assert_eq!(reg.effort_of(&w1), None);

        assert_eq!(reg.window_of(&w1), None);
        reg.set_context_window(w1.clone(), 200_000);
        assert_eq!(reg.window_of(&w1), Some(200_000));

        assert_eq!(reg.models(), Vec::<String>::new());
        reg.set_models(vec!["m1".into(), "m2".into()]);
        assert_eq!(reg.models(), vec!["m1".to_string(), "m2".to_string()]);
    }

    #[tokio::test]
    async fn set_model_and_set_effort_republish_the_roster() {
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        let mut rx = reg.subscribe();
        reg.set_model(w1.clone(), "m1"); // NEW: re-publishes
        next_change(&mut rx).await;
        reg.set_effort(w1.clone(), Some("low".into()));
        next_change(&mut rx).await;
        reg.set_context_window(w1.clone(), 1_000);
        next_change(&mut rx).await;
    }

    #[tokio::test]
    async fn set_state_is_read_back_and_republishes() {
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        assert_eq!(reg.state_of(&w1), MemberState::Idle, "unset id reads Idle");

        let mut rx = reg.subscribe();
        reg.set_state(w1.clone(), MemberState::Running);
        assert_eq!(reg.state_of(&w1), MemberState::Running);
        // `set_state` re-publishes, so a subscriber (the server's push) wakes.
        tokio::time::timeout(Duration::from_secs(2), rx.changed())
            .await
            .expect("a roster change")
            .expect("open");
    }

    #[tokio::test]
    async fn register_mirrors_the_actor_state_into_the_roster() {
        // A stream that emits one delta then hangs: Running is observable until
        // a cancel ends the run.
        let stream_fn: StreamFn = Arc::new(|_ctx, _sys, _tools, _opts| {
            let head =
                futures::stream::iter(vec![LlmStreamEvent::TextDelta("part".into())]);
            Box::pin(head.chain(futures::stream::pending())) as LlmStream
        });
        let handle = session_with(stream_fn);
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        reg.register(w1.clone(), handle.clone());
        assert_eq!(reg.state_of(&w1), MemberState::Idle, "a virgin session is idle");

        handle.send(Request::Submit { text: "hi".into() }).unwrap();
        let mut running = false;
        for _ in 0..500 {
            if reg.state_of(&w1) == MemberState::Running {
                running = true;
                break;
            }
            tokio::time::sleep(Duration::from_millis(10)).await;
        }
        assert!(running, "the registry mirrors the actor's Running");

        handle.send(Request::Cancel).unwrap();
        let mut done = false;
        for _ in 0..500 {
            if reg.state_of(&w1) == MemberState::Done {
                done = true;
                break;
            }
            tokio::time::sleep(Duration::from_millis(10)).await;
        }
        assert!(done, "the registry mirrors the actor's Done");
    }

    #[tokio::test]
    async fn subscribe_yields_the_new_roster_after_register() {
        let reg = Registry::new();
        let mut rx = reg.subscribe();
        let w1 = SessionId::agent("w1");
        reg.register(w1.clone(), session());
        rx.changed().await.unwrap();
        let ids: Vec<SessionId> = rx.borrow().iter().map(|(id, _)| id.clone()).collect();
        assert_eq!(ids, vec![w1]);
    }

    #[tokio::test]
    async fn subscribe_seeds_a_session_registered_before_it() {
        // A session registered with **no** subscriber in existence must still be
        // visible to a `subscribe()` that starts later — the `[team]` startup
        // members, registered before the socket server's first `subscribe()`.
        // (`watch::Sender::send` drops a value with no receivers; `send_replace`
        // keeps it, which is what `refresh_roster` relies on.)
        let reg = Registry::new();
        let w1 = SessionId::agent("w1");
        reg.register(w1.clone(), session());
        let rx = reg.subscribe();
        let ids: Vec<SessionId> = rx.borrow().iter().map(|(id, _)| id.clone()).collect();
        assert_eq!(ids, vec![w1]);
    }

    #[tokio::test]
    async fn an_unowned_pair_is_not_permitted() {
        let reg = Registry::new();
        let a = SessionId::agent("a");
        let b = SessionId::agent("b");
        reg.register(a.clone(), session());
        reg.register(b.clone(), session());
        // No ownership edge recorded → they may not address each other.
        assert_eq!(
            reg.deliver(&a, &b, Request::Notify { content: "?".into() }),
            Err(RouteError::NotPermitted {
                from: a.clone(),
                to: b.clone()
            })
        );
    }
    #[tokio::test]
    async fn remove_drops_peers_states_models_and_republishes() {
        let reg = Registry::new();
        let orch = SessionId::agent("orch");
        let w1 = SessionId::agent("w1");
        reg.register(w1.clone(), session());
        reg.set_owner(w1.clone(), orch.clone());
        reg.set_model(w1.clone(), "m1");
        reg.set_effort(w1.clone(), Some("high".into()));
        reg.set_context_window(w1.clone(), 1_000);
        reg.set_state(w1.clone(), MemberState::Running);
        assert!(reg.contains(&w1));

        let mut rx = reg.subscribe();
        reg.remove(&w1);

        // Every per-session map is cleared...
        assert!(!reg.contains(&w1), "peers entry dropped");
        assert_eq!(reg.state_of(&w1), MemberState::Idle, "state cleared");
        assert_eq!(reg.model_of(&w1), None, "model cleared");
        assert_eq!(reg.effort_of(&w1), None, "effort cleared");
        assert_eq!(reg.window_of(&w1), None, "window cleared");
        // ...and routing reports it gone (unknown, not merely unpermitted).
        assert!(
            matches!(reg.resolve(&orch, &w1), Err(RouteError::Unknown(id)) if id == w1),
            "an unknown route after remove"
        );
        assert!(reg.locals().is_empty(), "the local roster excludes it");
        // A subscriber (the socket server, the TUI) WOKE with the smaller list.
        next_change(&mut rx).await;
        assert!(rx.borrow().is_empty(), "the shrunk roster was published");
    }

    #[tokio::test]
    async fn remove_of_an_unknown_id_is_a_no_op() {
        let reg = Registry::new();
        let ghost = SessionId::agent("ghost");
        // No panic and no error; it simply re-publishes the (unchanged) roster.
        reg.remove(&ghost);
        assert!(!reg.contains(&ghost));
    }
}