agentd-core 1.4.0

// SPDX-License-Identifier: AGPL-3.0-only
//! The ReAct agentic loop.
//!
//! A turn: assemble the request (system + instruction + transcript + the
//! scoped tool catalogue) → call intelligence → if the model requested tools,
//! run them via MCP and feed the results back as observations; otherwise the
//! text is the final answer. Stopping is a disjunction of cheap checks, each
//! with a distinct [`TerminalStatus`], and the loop enforces the
//! step/token/deadline budget itself rather than trusting the model to stop.
//! This loop produces neither `Stalled` nor `LoopDetected`: it has no
//! no-progress or repeated-tool detector, so a spinning agent is bounded by the
//! step and token budget instead.
//!
//! The root agent runs as a subagent process behind the control channel; the
//! loop body here is identical whether driven by the root or a nested child, so
//! there is one code path to reason about regardless of tree position.

use crate::agentloop::action::{SelfHandler, ToolClass};
use crate::agentloop::stop::{Outcome, TerminalStatus};
use crate::intel::client::IntelClient;
use crate::mcp::client::McpClient;
use crate::obs::log::Logger;
use crate::subagent::protocol::ALLOWED_TOOLS_ROLE;
use crate::supervisor::budget::Budget;
use crate::wire::intel::{Message, Request, ToolDef, Usage};
use serde_json::{Value, json};
use std::collections::HashMap;
use std::fmt;
use std::sync::Arc;
use std::sync::atomic::{AtomicBool, Ordering};
use std::time::Instant;

/// Per-response token cap (distinct from the cumulative run budget).
const PER_CALL_MAX_TOKENS: u32 = 4096;

const SYSTEM_PROMPT: &str = "You are agentd, an autonomous agent. Accomplish the user's \
instruction by calling the available tools and reasoning over their results. Call a tool when you \
need information or need to act. When the task is complete, reply with your final answer and do \
NOT call a tool. If the task cannot be done, say so plainly. Be concise and factual.";

/// A fatal infrastructure failure that aborts the run; the caller maps it to
/// exit 4 or 6. Tool-domain errors are *not* aborts — a tool that returns an
/// error is fed back to the model as an observation so it can adapt, because
/// only the infrastructure being gone makes further progress impossible.
#[derive(Debug)]
pub enum LoopAbort {
    /// The intelligence endpoint is unreachable / erroring (exit 4).
    Intel(String),
    /// A required MCP server failed (exit 6).
    Mcp(String),
}

impl fmt::Display for LoopAbort {
    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
        match self {
            LoopAbort::Intel(m) => write!(f, "intelligence: {m}"),
            LoopAbort::Mcp(m) => write!(f, "mcp: {m}"),
        }
    }
}

/// The explicit inputs the loop needs, independent of where they came from
/// (CLI `Config` for once-mode, or a `SpawnPayload` for a subagent). This is
/// the seam that lets the same loop body run in-process or in a child.
pub struct LoopInput {
    pub instruction: String,
    pub output_contract: Option<String>,
    /// Narrowed context seed as (role, content) pairs (role ∈
    /// system|user|assistant|tool).
    pub seed: Vec<(String, String)>,
    pub model: String,
    pub max_steps: u32,
    pub max_tokens: u64,
    pub deadline: Instant,
    /// A cooperative cancel flag checked at each turn boundary (set by a
    /// subagent's control thread on `ControlMsg::Cancel`). `None` for a run with
    /// no external canceller.
    pub cancel: Option<Arc<AtomicBool>>,
}

/// The durable state of an agent session: the scoped tool catalogue, the
/// resource-awareness map, and the **conversation transcript** — everything that
/// persists *across turns*. A once-mode / per-event run is a session of exactly
/// one turn ([`run_loop`]); a **warm** continue-session runs many turns over the
/// same transcript, each new event appended via [`Session::deliver`] before
/// another [`Session::run_turn`].
pub struct Session<'a> {
    servers: &'a [McpClient],
    tools: Vec<ToolDef>,
    tool_to_server: HashMap<String, usize>,
    resources: ResourceCatalogue,
    model: String,
    messages: Vec<Message>,
    /// The narrowed tool GRANTS this session runs under: a parent's
    /// `subagent.run` `tools:` list, carried on the seed under
    /// [`ALLOWED_TOOLS_ROLE`]. Each element is one grant's pattern list, and a
    /// tool must satisfy EVERY grant: a grant only ever narrows, so intersecting
    /// is the only safe way to combine two. Empty = ungranted = the full
    /// catalogue (a root / embedded run, which is nobody's subagent).
    allowed: Vec<Vec<String>>,
}

impl<'a> Session<'a> {
    /// Assemble a session: the tool catalogue (MCP tools + self-tools, plus
    /// `resource.read` when resources exist), the resource awareness note, and
    /// the opening transcript (system prompt + seed + the instruction as the
    /// first user turn). Resources are split deliberately: listing them makes
    /// the model aware of what exists, while reading one is an explicit tool
    /// call, so a large resource enters the context only when actually wanted.
    pub fn prepare(
        servers: &'a [McpClient],
        input: &LoopInput,
        self_handler: &mut dyn SelfHandler,
    ) -> Result<Session<'a>, LoopAbort> {
        let allowed = seed_grants(&input.seed);
        let (mut tools, mut tool_to_server) = build_catalogue(servers)?;
        // CODE-REGISTERED tools win a name collision: drop the MCP entry so the
        // catalogue offers ONE def per name, and it is the one dispatch will
        // actually run. Otherwise the model would be shown a schema it is not
        // calling, and a server could shadow a first-party tool.
        let code = crate::tools::defs();
        if !code.is_empty() {
            tools.retain(|t| !crate::tools::is_registered(&t.name));
            tools.extend(code);
        }
        tools.extend(self_handler.tools());
        let resources = collect_resources(servers);
        // Offer `resource.read` when there are MCP resources OR the handler
        // serves agentd:// self-resources (e.g. async-child completions).
        if !resources.owner.is_empty() || self_handler.serves_self_resources() {
            tools.push(resource_read_tool_def());
        }
        // The parent's narrowed grant lands LAST, over the whole assembled
        // catalogue (MCP + code + self-tools + `resource.read`) and over the
        // routing map that governs dispatch. Scope narrows monotonically, so a
        // grant of `["a"]` must mean `a` and nothing else — not "a, plus
        // everything merged in after the grant was applied".
        narrow_catalogue(&allowed, &mut tools, &mut tool_to_server);
        let mut messages = vec![Message::system(system_prompt(
            input.output_contract.as_deref(),
        ))];
        if let Some(note) = resources.catalogue_note() {
            messages.push(Message::system(note));
        }
        for (role, content) in &input.seed {
            // The grant is policy, not conversation: it never enters the
            // transcript (and so never reaches the model as a suggestion).
            if role == ALLOWED_TOOLS_ROLE {
                continue;
            }
            messages.push(seed_message(role, content));
        }
        messages.push(Message::user(&input.instruction));
        Ok(Session {
            servers,
            tools,
            tool_to_server,
            resources,
            model: input.model.clone(),
            messages,
            allowed,
        })
    }

    /// Rebuild the MCP side of the tool catalogue from the servers' CURRENT
    /// `tools/list`. Called at a turn boundary after an inbound
    /// `notifications/tools/list_changed`, so a long-lived continue-session
    /// tracks a server whose tool set changed instead of holding a stale
    /// catalogue for its whole life. Self-tools and `resource.read` are
    /// re-merged and the transcript is untouched, so a refresh costs no context.
    pub fn refresh_tools(&mut self, self_handler: &mut dyn SelfHandler) -> Result<(), LoopAbort> {
        let (mut tools, mut tool_to_server) = build_catalogue(self.servers)?;
        // Same code-tool precedence as `prepare`: first-party wins the name.
        let code = crate::tools::defs();
        if !code.is_empty() {
            tools.retain(|t| !crate::tools::is_registered(&t.name));
            tools.extend(code);
        }
        tools.extend(self_handler.tools());
        if !self.resources.owner.is_empty() || self_handler.serves_self_resources() {
            tools.push(resource_read_tool_def());
        }
        // Re-narrow: a server that ADDS a tool mid-session must not widen a
        // grant the parent already bounded. A refresh rebuilds the catalogue; it
        // is never a re-grant, so the child's scope can only stay the same or
        // shrink across one.
        narrow_catalogue(&self.allowed, &mut tools, &mut tool_to_server);
        self.tools = tools;
        self.tool_to_server = tool_to_server;
        Ok(())
    }

    /// The current catalogue size (observability for the live refresh).
    pub fn tools_len(&self) -> usize {
        self.tools.len()
    }

    /// Classify a catalogue tool by its seam: a name routed to an MCP server is
    /// [`ToolClass::Mcp`], dispatched back to that server; every other catalogue
    /// entry is agentd's own [`ToolClass::SelfControl`] surface — the self-tools
    /// plus `resource.read`. The routing map IS the MCP-tool set, and the two
    /// classes are assembled by different code paths ([`build_catalogue`] versus
    /// the [`SelfHandler`] merge), which makes this an authoritative and testable
    /// boundary between "tools from a registered server" and "agentd's own
    /// orchestration primitives".
    ///
    /// Callers must pass a name drawn from [`Session::tools`]: a name absent from
    /// the catalogue classifies as `SelfControl`, since it is by definition not a
    /// routed server tool, so classifying an arbitrary string is meaningless.
    pub fn tool_class(&self, name: &str) -> ToolClass {
        // A code-registered name classifies `Code` even when an MCP server
        // publishes the same name, matching what dispatch does: code wins, so a
        // remote server cannot steal a registered tool's calls. Registration
        // refuses self/control names, so `Code` never claims that class.
        if crate::tools::is_registered(name) {
            ToolClass::Code
        } else if self.tool_to_server.contains_key(name) {
            ToolClass::Mcp
        } else {
            ToolClass::SelfControl
        }
    }

    /// Whether this session's GRANT admits `name`. Every grant must admit it; an
    /// ungranted session — one with no parent narrowing — admits everything.
    /// The catalogue is already filtered, so this is a second gate at dispatch
    /// time, for the model that names a tool anyway: a hallucinated name, or one
    /// remembered from a transcript written when the catalogue was wider.
    pub fn tool_permitted(&self, name: &str) -> bool {
        grant_permits(&self.allowed, name)
    }

    /// Append the next event as a new user turn — the delivery point for a warm
    /// continue-session. The transcript, which is the model's memory of the
    /// session, carries forward, so the next turn continues the conversation
    /// rather than starting over.
    pub fn deliver(&mut self, content: &str) {
        self.messages.push(Message::user(content));
    }

    /// Adopt a new model for subsequent turns. The transcript is UNTOUCHED —
    /// only the model dialed for the NEXT turn changes, and a turn already in
    /// flight runs to completion on the model it started with. The `model` is
    /// what each request's `model` field carries.
    pub fn set_model(&mut self, model: &str) {
        self.model = model.to_string();
    }

    /// The current model dialed for the next turn. Used to decide whether a
    /// pending swap actually changes the model: an endpoint repoint that leaves
    /// the model alone needs no turn restart, since the output would match.
    pub fn model(&self) -> &str {
        &self.model
    }

    /// The number of transcript messages so far — a cheap pre-turn marker for the
    /// `restart-turn` policy: snapshot this before a turn, then
    /// [`truncate_transcript`](Session::truncate_transcript) back to it to discard
    /// the swapped turn's appended messages and re-run from the same pre-turn state.
    pub fn transcript_len(&self) -> usize {
        self.messages.len()
    }

    /// Truncate the transcript back to `len` for a `restart-turn`: drop every
    /// message a discarded turn appended, restoring the exact pre-turn
    /// transcript so the turn can be re-run on the new model. A no-op when `len`
    /// is already at or past the current length — this never grows the
    /// transcript, so a stale marker cannot resurrect dropped messages.
    pub fn truncate_transcript(&mut self, len: usize) {
        if len < self.messages.len() {
            self.messages.truncate(len);
        }
    }

    /// Run one turn: the ReAct loop over the persistent transcript until a
    /// terminal status, bounded by `budget`. `cancel` is polled at each turn
    /// boundary. Every assistant/tool message (including the final answer) is
    /// appended to the transcript, so a subsequent turn continues the same
    /// conversation.
    ///
    /// Returns the turn's [`Outcome`] together with the turn's token [`Usage`] —
    /// the sum of every model call in this turn. The control layer rolls this
    /// DELTA up to the supervisor as
    /// [`crate::subagent::protocol::AgentMsg::Usage`], which is what makes
    /// hierarchical token accounting (`agentd_tokens_total`) add up. The loop
    /// itself never touches the control channel: the `up` handle stays in
    /// `control.rs`, so the loop stays runnable in-process and in a child alike.
    pub fn run_turn(
        &mut self,
        intel: &IntelClient,
        self_handler: &mut dyn SelfHandler,
        log: &Logger,
        budget: &mut Budget,
        cancel: Option<&Arc<AtomicBool>>,
    ) -> Result<(Outcome, Usage), LoopAbort> {
        let mut last_text: Option<String> = None;
        // otel run trace: the `invoke_agent` span plus a `chat` child per model
        // call and an `execute_tool` child per tool call. No-op without
        // `--features otel`, so the wiring carries no `cfg`. One trace per turn.
        let run_start = crate::obs::otel::now_unix_nanos();
        let mut run_span = crate::obs::otel::run_begin(log.ctx().trace_id.as_deref(), run_start);
        let (mut tok_in, mut tok_out) = (0u64, 0u64);

        log.info(
            "loop.start",
            json!({"tools": self.tools.len(), "servers": self.servers.len(), "resources": self.resources.owner.len(), "max_steps": budget.max_steps()}),
        );

        loop {
            if cancel.is_some_and(|c| c.load(Ordering::Relaxed)) {
                log.warn(
                    "loop.final",
                    json!({"status": "cancelled", "steps": budget.steps()}),
                );
                run_span.finish(&self.model, tok_in, tok_out, false);
                return Ok((
                    Outcome {
                        status: TerminalStatus::Cancelled,
                        partial: last_text.is_some(),
                        result: json!(last_text.unwrap_or_default()),
                        scheduled: self_handler.take_scheduled(),
                        subscriptions: self_handler.take_subscriptions(),
                    },
                    Usage {
                        input_tokens: tok_in,
                        output_tokens: tok_out,
                    },
                ));
            }
            if let Some(status) = budget.exceeded() {
                log.warn("loop.final", json!({"status": status.as_str(), "steps": budget.steps(), "tokens": budget.tokens()}));
                run_span.finish(&self.model, tok_in, tok_out, false);
                return Ok((
                    Outcome {
                        status,
                        partial: last_text.is_some(),
                        result: json!(last_text.unwrap_or_default()),
                        scheduled: self_handler.take_scheduled(),
                        subscriptions: self_handler.take_subscriptions(),
                    },
                    Usage {
                        input_tokens: tok_in,
                        output_tokens: tok_out,
                    },
                ));
            }

            // Per-turn audit anchor: the running budget snapshot at the head of
            // each ReAct step, distinct from the LLM-call event below so the two
            // can be counted separately when reading a trace.
            log.debug(
                "loop.step",
                json!({"step": budget.steps(), "tokens": budget.tokens(), "messages": self.messages.len()}),
            );

            let req = Request {
                model: self.model.clone(),
                messages: self.messages.clone(),
                tools: self.tools.clone(),
                max_tokens: PER_CALL_MAX_TOKENS,
                temperature: Some(0.0),
            };

            log.debug(
                "intel.call",
                json!({"step": budget.steps(), "messages": self.messages.len()}),
            );
            let chat_start = crate::obs::otel::now_unix_nanos();
            let resp = intel
                .complete(&req)
                .map_err(|e| LoopAbort::Intel(e.to_string()))?;
            budget.record_usage(resp.usage);
            budget.record_step();
            tok_in += resp.usage.input_tokens;
            tok_out += resp.usage.output_tokens;
            run_span.record_chat(
                &self.model,
                resp.usage.input_tokens,
                resp.usage.output_tokens,
                true,
                chat_start,
            );
            log.debug(
                "intel.result",
                json!({"tool_calls": resp.tool_calls.len(), "tokens_in": resp.usage.input_tokens, "tokens_out": resp.usage.output_tokens}),
            );

            if resp.wants_tools() {
                if let Some(t) = resp.text.as_deref().filter(|t| !t.is_empty()) {
                    last_text = Some(t.to_string());
                }
                let tool_calls = resp.tool_calls.clone();
                self.messages.push(Message::Assistant {
                    text: resp.text,
                    tool_calls: tool_calls.clone(),
                });

                for tc in &tool_calls {
                    let mut call = json!({"tool": tc.name, "id": tc.id});
                    // Content capture is opt-in: by default only the tool name
                    // and body length are logged, because arguments routinely
                    // carry sensitive data. `--log-content` adds the truncated
                    // arguments and result body for debugging.
                    if log.content_capture() {
                        call["args"] = json!(truncate_for_log(&tc.arguments.to_string()));
                    }
                    log.info("tool.call", call);
                    let tool_start = crate::obs::otel::now_unix_nanos();
                    let (content, is_error) = if !self.tool_permitted(&tc.name) {
                        // Refused, never served: the grant binds the DISPATCH,
                        // not just the definitions offered. A model that names a
                        // narrowed-away tool gets an error observation it can
                        // adapt to, exactly like an unknown tool.
                        (
                            format!(
                                "error: tool '{}' is not in this subagent's allowed tools",
                                tc.name
                            ),
                            true,
                        )
                    } else if tc.name == "resource.read" {
                        // An `agentd://` URI reads agentd's own state (e.g. an
                        // async child's completion) via the self-handler; any
                        // other URI is an MCP-server resource.
                        let uri = tc
                            .arguments
                            .get("uri")
                            .and_then(Value::as_str)
                            .unwrap_or("")
                            .trim();
                        if uri.starts_with("agentd://") || uri.starts_with("agent://") {
                            self_handler.read_resource(uri).unwrap_or_else(|| {
                                (format!("unknown agentd resource: {uri}"), true)
                            })
                        } else {
                            read_resource_tool(self.servers, &self.resources.owner, &tc.arguments)
                        }
                    } else {
                        match self_handler.handle(&tc.name, &tc.arguments) {
                            Some(r) => r, // a self-tool (e.g. subagent.spawn)
                            // Code-registered tools next: first-party beats a
                            // colliding remote name.
                            None => match crate::tools::dispatch(&tc.name, &tc.arguments) {
                                Some(r) => r,
                                None => dispatch_tool(
                                    self.servers,
                                    &self.tool_to_server,
                                    &tc.name,
                                    &tc.arguments,
                                ),
                            },
                        }
                    };
                    run_span.record_tool(&tc.name, !is_error, tool_start);
                    let mut result =
                        json!({"tool": tc.name, "is_error": is_error, "bytes": content.len()});
                    if log.content_capture() {
                        result["content"] = json!(truncate_for_log(&content));
                    }
                    log.info("tool.result", result);
                    self.messages
                        .push(Message::tool_result(&tc.id, content, is_error));
                }
                continue;
            }

            // No tool calls → the model's text is the final answer for this turn.
            // Record it in the transcript so a warm session's next turn sees its
            // own prior reply (invisible to once-mode, which discards the session).
            let text = resp.text.clone().or(last_text).unwrap_or_default();
            self.messages.push(Message::Assistant {
                text: Some(text.clone()),
                tool_calls: Vec::new(),
            });
            log.info(
                "loop.final",
                json!({"status": "completed", "steps": budget.steps(), "tokens": budget.tokens()}),
            );
            run_span.finish(&self.model, tok_in, tok_out, true);
            return Ok((
                Outcome {
                    status: TerminalStatus::Completed,
                    partial: false,
                    result: json!(text),
                    scheduled: self_handler.take_scheduled(),
                    subscriptions: self_handler.take_subscriptions(),
                },
                Usage {
                    input_tokens: tok_in,
                    output_tokens: tok_out,
                },
            ));
        }
    }
}

/// The agentic loop over explicit inputs — one session, one turn. Used by
/// once-mode (`run_root`) and a per-event subagent run (`subagent::control`).
/// `self_handler` supplies agentd's in-process self-tools (e.g. `subagent.spawn`);
/// the loop tries it before MCP. A warm continue-session instead drives
/// [`Session`] directly across many turns.
///
/// Returns the run's [`Outcome`] together with the run's total token [`Usage`].
/// A one-shot run is exactly one turn, so the run total IS that turn's usage,
/// and the control layer emits it once per run as a single
/// [`crate::subagent::protocol::AgentMsg::Usage`]. Emitting both a cumulative
/// and a per-turn figure would double-count against the tree's token ceiling.
pub fn run_loop(
    intel: &IntelClient,
    servers: &[McpClient],
    input: &LoopInput,
    self_handler: &mut dyn SelfHandler,
    log: &Logger,
) -> Result<(Outcome, Usage), LoopAbort> {
    let mut session = Session::prepare(servers, input, self_handler)?;
    let mut budget = Budget::new(input.max_steps, input.max_tokens, input.deadline);
    session.run_turn(intel, self_handler, log, &mut budget, input.cancel.as_ref())
}

/// Max characters of tool content recorded under `--log-content`. Bounds a log
/// line so a large tool body can't bloat the telemetry stream; the full body
/// still flows to the model as the observation.
const CONTENT_LOG_CAP: usize = 4096;

/// Truncate a body for content-capture logging, appending a byte-count marker
/// when clipped. Char-based so a multi-byte boundary is never split.
fn truncate_for_log(s: &str) -> String {
    if s.chars().count() <= CONTENT_LOG_CAP {
        return s.to_string();
    }
    let mut t: String = s.chars().take(CONTENT_LOG_CAP).collect();
    t.push_str(&format!(
        "…(+{} more bytes)",
        s.len().saturating_sub(t.len())
    ));
    t
}

/// Build the model's tool catalogue from every connected server, plus a
/// name→server-index routing map. On a name collision the FIRST server in
/// configuration order wins, so routing is deterministic and a later server
/// cannot capture a name an earlier one already publishes. A call to a name no
/// server publishes is reported as unknown at dispatch time.
fn build_catalogue(
    servers: &[McpClient],
) -> Result<(Vec<ToolDef>, HashMap<String, usize>), LoopAbort> {
    let mut tools = Vec::new();
    let mut routing = HashMap::new();
    for (i, server) in servers.iter().enumerate() {
        let listed = server
            .list_tools()
            .map_err(|e| LoopAbort::Mcp(e.to_string()))?;
        for t in listed {
            routing.entry(t.name.clone()).or_insert(i);
            tools.push(ToolDef {
                name: t.name,
                description: t.description.unwrap_or_default(),
                input_schema: t.input_schema,
            });
        }
    }
    Ok((tools, routing))
}

/// The narrowed tool grants a spawn payload carried on its context seed: one
/// pattern list per [`ALLOWED_TOOLS_ROLE`] entry. Normally zero, meaning no
/// narrowing, or one, since the supervisor mints exactly one grant per child.
fn seed_grants(seed: &[(String, String)]) -> Vec<Vec<String>> {
    seed.iter()
        .filter(|(role, _)| role == ALLOWED_TOOLS_ROLE)
        .map(|(_, content)| crate::subagent::protocol::parse_allowed_tools(content))
        .collect()
}

/// Whether every grant admits `name` — patterns are the registry's (`*`, an
/// exact name, `prefix*`), so a `tools:` list reads the same here as it does in
/// a workflow `agent` step. No grants ⇒ admitted.
fn grant_permits(grants: &[Vec<String>], name: &str) -> bool {
    grants
        .iter()
        .all(|g| g.iter().any(|p| crate::registry::pattern_matches(p, name)))
}

/// Drop everything the grants exclude from an assembled catalogue AND from the
/// routing map — the map is what `dispatch_tool` consults, so filtering both is
/// what makes an excluded MCP tool unreachable rather than merely unadvertised.
fn narrow_catalogue(
    grants: &[Vec<String>],
    tools: &mut Vec<ToolDef>,
    routing: &mut HashMap<String, usize>,
) {
    if grants.is_empty() {
        return;
    }
    tools.retain(|t| grant_permits(grants, &t.name));
    routing.retain(|name, _| grant_permits(grants, name));
}

/// Route one tool call to its owning server. A transport error comes back as an
/// error *observation* (`is_error = true`) rather than an abort, so the model
/// can adapt or try another route; a server that stays wedged is ultimately
/// bounded by the step and deadline budget rather than by this call.
fn dispatch_tool(
    servers: &[McpClient],
    routing: &HashMap<String, usize>,
    name: &str,
    arguments: &Value,
) -> (String, bool) {
    match routing.get(name) {
        Some(&i) => {
            // A flat subagent paces its own calls against the per-server rate
            // limit too, not just the reactor: its pacing registry was seeded
            // when it dialed its granted servers, so a fan-out of children
            // cannot collectively outrun a server's declared rate.
            if let Err(e) = crate::mcp::pace::take(servers[i].name()) {
                return (e, true);
            }
            match servers[i].call_tool(name, Some(arguments.clone())) {
                Ok(res) => (res.text(), res.is_error()),
                Err(e) => (format!("tool transport error: {e}"), true),
            }
        }
        None => (format!("error: no such tool '{name}'"), true),
    }
}

/// The system prompt, with the delegation output contract appended when the
/// spawn payload carried one, so a child sees the shape its result must take.
fn system_prompt(contract: Option<&str>) -> String {
    match contract {
        Some(c) if !c.is_empty() => format!("{SYSTEM_PROMPT}\n\nOutput contract:\n{c}"),
        _ => SYSTEM_PROMPT.to_string(),
    }
}

/// Map a seed (role, content) pair to a loop message. A `tool` seed has no
/// tool-call id to replay against, so it degrades to a user note.
fn seed_message(role: &str, content: &str) -> Message {
    match role {
        "system" => Message::system(content),
        "assistant" => Message::Assistant {
            text: Some(content.to_string()),
            tool_calls: Vec::new(),
        },
        _ => Message::user(content),
    }
}

/// Cap on the injected resource catalogue (URIs only; bodies are pulled on
/// demand). A server exposing thousands is truncated with a note.
const RESOURCE_CAP: usize = 50;

/// The compact resource awareness catalogue plus a uri→owning-server map for
/// `resource.read`. Listing makes the model aware a resource exists; reading its
/// body is a separate tool call, so nothing large enters the context unasked.
struct ResourceCatalogue {
    owner: HashMap<String, usize>,
    entries: Vec<(String, String)>, // (uri, label)
    truncated: bool,
}

impl ResourceCatalogue {
    /// The system note listing readable resources (never their bodies).
    fn catalogue_note(&self) -> Option<String> {
        if self.entries.is_empty() {
            return None;
        }
        let mut s = String::from(
            "Available MCP resources — read the current content of any with the resource.read tool:\n",
        );
        for (uri, label) in &self.entries {
            if label.is_empty() {
                s.push_str(&format!("- {uri}\n"));
            } else {
                s.push_str(&format!("- {uri} — {label}\n"));
            }
        }
        if self.truncated {
            s.push_str(&format!(
                "(… more than {RESOURCE_CAP} resources; list truncated)\n"
            ));
        }
        Some(s)
    }
}

/// List resources from every server (first owner wins for a duplicate URI),
/// capped. `resources/list` is capability-gated in the client (empty if unsupported).
fn collect_resources(servers: &[McpClient]) -> ResourceCatalogue {
    let mut owner = HashMap::new();
    let mut entries = Vec::new();
    let mut truncated = false;
    'outer: for (i, s) in servers.iter().enumerate() {
        let Ok(list) = s.list_resources() else {
            continue;
        };
        for r in list {
            if entries.len() >= RESOURCE_CAP {
                truncated = true;
                break 'outer;
            }
            if !owner.contains_key(&r.uri) {
                let label = r.title.or(r.name).or(r.description).unwrap_or_default();
                owner.insert(r.uri.clone(), i);
                entries.push((r.uri, label));
            }
        }
    }
    ResourceCatalogue {
        owner,
        entries,
        truncated,
    }
}

fn resource_read_tool_def() -> ToolDef {
    ToolDef {
        name: "resource.read".into(),
        description: "Read the current content of an available MCP resource by its uri (see the \
            resource catalogue). Use this to pull a resource's body when you need it."
            .into(),
        input_schema: json!({
            "type": "object",
            "properties": {"uri": {"type": "string", "description": "the resource uri to read"}},
            "required": ["uri"]
        }),
    }
}

/// Handle a `resource.read` call against the connected servers: read from the
/// owning server (or try each), returning the text as the observation.
fn read_resource_tool(
    servers: &[McpClient],
    owner: &HashMap<String, usize>,
    args: &Value,
) -> (String, bool) {
    let uri = args.get("uri").and_then(Value::as_str).unwrap_or("").trim();
    if uri.is_empty() {
        return ("error: resource.read requires a 'uri'".into(), true);
    }
    let candidates: Vec<usize> = match owner.get(uri) {
        Some(i) => vec![*i],
        None => (0..servers.len()).collect(), // a templated/unlisted uri — try all
    };
    for i in candidates {
        if let Ok(r) = servers[i].read_resource(uri) {
            return (r.text(), false);
        }
    }
    (format!("resource.read: no server could read '{uri}'"), true)
}

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

    #[test]
    fn resource_catalogue_note_lists_uris() {
        let c = ResourceCatalogue {
            owner: HashMap::new(),
            entries: vec![
                ("file:///a.json".into(), "inbox".into()),
                ("db://orders".into(), String::new()),
            ],
            truncated: false,
        };
        let note = c.catalogue_note().unwrap();
        assert!(note.contains("resource.read"));
        assert!(note.contains("file:///a.json — inbox"));
        assert!(note.contains("- db://orders\n"));
    }

    #[test]
    fn empty_catalogue_is_no_note() {
        let c = ResourceCatalogue {
            owner: HashMap::new(),
            entries: vec![],
            truncated: false,
        };
        assert!(c.catalogue_note().is_none());
    }

    #[test]
    fn resource_read_rejects_missing_uri() {
        let (msg, err) = read_resource_tool(&[], &HashMap::new(), &json!({}));
        assert!(err);
        assert!(msg.contains("uri"));
    }

    #[test]
    fn resource_read_no_server_is_an_error_observation() {
        let (msg, err) = read_resource_tool(&[], &HashMap::new(), &json!({"uri": "file:///x"}));
        assert!(err);
        assert!(msg.contains("file:///x"));
    }

    #[test]
    fn system_prompt_appends_contract() {
        let p = system_prompt(Some("Return JSON."));
        assert!(p.contains("Output contract:"));
        assert!(p.contains("Return JSON."));
        assert_eq!(system_prompt(None), SYSTEM_PROMPT);
    }

    #[test]
    fn a_code_registered_tool_classifies_code_and_wins_a_name_collision() {
        // A first-party (code-registered) tool beats a remote MCP tool of the
        // same name, in classification and therefore in dispatch. Tool names
        // here must be unique: the registry is process-global and tests share a
        // process.
        let _guard = crate::tools::test_registry_guard();
        crate::tools::register(crate::tools::CodeTool::new(
            "runner.code_tool",
            "a native tool",
            json!({"type": "object"}),
            |_| Ok(json!("native")),
        ))
        .expect("register");
        let mut tool_to_server = HashMap::new();
        // The MCP side ALSO publishes the colliding name (a rogue/coincidental server).
        tool_to_server.insert("runner.code_tool".to_string(), 0usize);
        let sess = Session {
            servers: &[],
            tools: vec![],
            tool_to_server,
            resources: ResourceCatalogue {
                owner: HashMap::new(),
                entries: vec![],
                truncated: false,
            },
            model: "m".into(),
            messages: vec![],
            allowed: Vec::new(),
        };
        assert_eq!(
            sess.tool_class("runner.code_tool"),
            ToolClass::Code,
            "code wins the collision — a server cannot steal a registered tool's calls"
        );
        // And the dispatch agrees with the classification.
        let (content, is_err) =
            crate::tools::dispatch("runner.code_tool", &json!({})).expect("code tool dispatches");
        assert!(!is_err);
        assert_eq!(content, "native");
        assert!(crate::tools::unregister("runner.code_tool"));
    }

    #[test]
    fn catalogue_partitions_into_mcp_and_self_control_classes() {
        use crate::agentloop::action::SELF_CONTROL_TOOLS;
        // A catalogue: two MCP-server tools (routed) plus agentd's full
        // self/control surface and resource.read. Every entry must classify into
        // exactly one class — the MCP side is precisely the routed set, and the
        // rest is agentd's own control surface — and no self/control tool is a
        // local-execution primitive.
        let mcp = ["db.query", "http.get"];
        let mut tool_to_server = HashMap::new();
        let mut tools: Vec<ToolDef> = Vec::new();
        for n in mcp {
            tool_to_server.insert(n.to_string(), 0usize);
            tools.push(ToolDef {
                name: n.into(),
                description: String::new(),
                input_schema: json!({}),
            });
        }
        // The full self/control surface a root handler with peers advertises, plus
        // the runner-added resource.read — i.e. the whole named class.
        for n in SELF_CONTROL_TOOLS {
            tools.push(ToolDef {
                name: (*n).into(),
                description: String::new(),
                input_schema: json!({}),
            });
        }
        let sess = Session {
            servers: &[],
            tools,
            tool_to_server,
            resources: ResourceCatalogue {
                owner: HashMap::new(),
                entries: vec![],
                truncated: false,
            },
            model: "m".into(),
            messages: vec![],
            allowed: Vec::new(),
        };
        // Routed names → Mcp; every self/control name → SelfControl.
        for n in mcp {
            assert_eq!(sess.tool_class(n), ToolClass::Mcp, "{n} is an MCP tool");
        }
        for n in SELF_CONTROL_TOOLS {
            assert_eq!(
                sess.tool_class(n),
                ToolClass::SelfControl,
                "{n} is self/control"
            );
        }
        // The classes EXACTLY cover the catalogue (no unclassified tool; no
        // code tools are registered in this test, so `Code` counts zero).
        let (mut n_mcp, mut n_self, mut n_code) = (0usize, 0usize, 0usize);
        for t in &sess.tools {
            match sess.tool_class(&t.name) {
                ToolClass::Mcp => n_mcp += 1,
                ToolClass::SelfControl => n_self += 1,
                ToolClass::Code => n_code += 1,
            }
        }
        assert_eq!(n_code, 0, "no code tools registered here");
        assert_eq!(n_mcp, mcp.len(), "every MCP tool classified");
        assert_eq!(
            n_self,
            SELF_CONTROL_TOOLS.len(),
            "every self tool classified"
        );
        // The self/control class holds NO local-execution primitive.
        for bad in [
            "exec", "shell", "bash", "sh", "command", "system", "eval", "run",
        ] {
            assert!(
                !SELF_CONTROL_TOOLS.contains(&bad),
                "no local-exec self-tool: {bad}"
            );
        }
    }

    #[test]
    fn dispatch_unknown_tool_is_error_observation() {
        let routing = HashMap::new();
        let (content, is_error) = dispatch_tool(&[], &routing, "ghost", &Value::Null);
        assert!(is_error);
        assert!(content.contains("ghost"));
    }

    #[test]
    fn loop_abort_display() {
        assert!(LoopAbort::Intel("down".into()).to_string().contains("down"));
    }

    #[test]
    fn truncate_for_log_caps_and_marks() {
        let short = "{\"a\":1}";
        assert_eq!(truncate_for_log(short), short); // under the cap: verbatim
        let big = "x".repeat(CONTENT_LOG_CAP + 500);
        let out = truncate_for_log(&big);
        assert!(out.len() < big.len());
        assert!(
            out.contains("more bytes"),
            "truncation is marked: {}",
            &out[out.len() - 32..]
        );
        // multi-byte safety: never panics on a char boundary
        let multi = "é".repeat(CONTENT_LOG_CAP + 10);
        let _ = truncate_for_log(&multi);
    }

    #[test]
    fn refresh_tools_picks_up_a_changed_handler_catalogue() {
        // A handler whose advertised tool set CHANGES between turns: refresh
        // rebuilds the catalogue in place and leaves the transcript untouched.
        // Hold the registry guard: `tools_len()` reads the process-global
        // code-tool registry, so a concurrent register/unregister in another
        // test must not perturb the exact +1 delta asserted below.
        let _guard = crate::tools::test_registry_guard();
        struct GrowingHandler {
            grown: bool,
        }
        impl SelfHandler for GrowingHandler {
            fn tools(&self) -> Vec<ToolDef> {
                let mut t = vec![ToolDef {
                    name: "alpha".into(),
                    description: String::new(),
                    input_schema: Value::Null,
                }];
                if self.grown {
                    t.push(ToolDef {
                        name: "beta".into(),
                        description: String::new(),
                        input_schema: Value::Null,
                    });
                }
                t
            }
            fn handle(&mut self, _name: &str, _args: &Value) -> Option<(String, bool)> {
                None
            }
        }
        let input = LoopInput {
            instruction: "x".into(),
            output_contract: None,
            seed: Vec::new(),
            model: "m".into(),
            max_steps: 5,
            max_tokens: 1000,
            deadline: std::time::Instant::now() + std::time::Duration::from_secs(5),
            cancel: None,
        };
        let mut handler = GrowingHandler { grown: false };
        let mut session = Session::prepare(&[], &input, &mut handler).unwrap();
        let before = session.tools_len();
        let transcript = session.transcript_len();
        handler.grown = true;
        session.refresh_tools(&mut handler).unwrap();
        assert_eq!(session.tools_len(), before + 1, "the new tool is live");
        assert_eq!(session.transcript_len(), transcript, "transcript untouched");
        // And the class boundary still holds: a self-tool is SelfControl.
        assert_eq!(session.tool_class("beta"), ToolClass::SelfControl);
    }

    #[test]
    fn a_seed_grant_narrows_the_catalogue_the_dispatch_and_nothing_else() {
        // With the `subagent.run` `tools:` grant, a child granted ["alpha"] sees
        // ONLY alpha: the grant filters the assembled catalogue, and dispatch
        // refuses a name the model produces anyway. The grant itself is policy,
        // so it never lands in the transcript.
        let _guard = crate::tools::test_registry_guard();
        struct TwoTools;
        impl SelfHandler for TwoTools {
            fn tools(&self) -> Vec<ToolDef> {
                ["alpha", "beta"]
                    .into_iter()
                    .map(|n| ToolDef {
                        name: n.into(),
                        description: String::new(),
                        input_schema: Value::Null,
                    })
                    .collect()
            }
            fn handle(&mut self, _name: &str, _args: &Value) -> Option<(String, bool)> {
                Some(("served".into(), false))
            }
        }
        let grant = LoopInput {
            instruction: "x".into(),
            output_contract: None,
            seed: vec![
                (
                    crate::subagent::protocol::ALLOWED_TOOLS_ROLE.to_string(),
                    "[\"alpha\"]".to_string(),
                ),
                ("user".to_string(), "a real seed message".to_string()),
            ],
            model: "m".into(),
            max_steps: 5,
            max_tokens: 1000,
            deadline: std::time::Instant::now() + std::time::Duration::from_secs(5),
            cancel: None,
        };
        let mut handler = TwoTools;
        let narrowed = Session::prepare(&[], &grant, &mut handler).unwrap();
        assert_eq!(narrowed.tools_len(), 1, "only the granted tool is offered");
        assert!(narrowed.tool_permitted("alpha"));
        assert!(
            !narrowed.tool_permitted("beta"),
            "a filtered-out tool is refused at dispatch, not served"
        );
        // The grant is not conversation: system prompt + the real seed + the
        // instruction — the marker is gone.
        assert_eq!(narrowed.transcript_len(), 3);

        // The same payload WITHOUT the grant is the unnarrowed baseline.
        let mut plain = grant;
        plain.seed.remove(0);
        let wide = Session::prepare(&[], &plain, &mut handler).unwrap();
        assert_eq!(wide.tools_len(), 2);
        assert!(wide.tool_permitted("beta"));
    }

    // ---- the run_turn / run_loop token-usage producer ----
    //
    // `run_turn` and `run_loop` return the turn's / run's `Usage` so `control.rs`
    // can roll it up to the supervisor as `AgentMsg::Usage`: they are the
    // producer end of the producer → consumer → `agentd_tokens_total` chain, and
    // a zero here silently zeroes the whole chain. These tests drive the *real*
    // loop against the built-in mock LLM and assert the returned `Usage` carries
    // the model's reported tokens. The consumer half is covered by the
    // `obs::metrics` `record_tokens` tests, and end to end by the reactive
    // `/metrics` scrape in `reactive_e2e`.
    #[cfg(unix)]
    mod usage_producer {
        use super::*;
        use crate::intel::client::IntelClient;
        use crate::obs::log::{Comp, Level, LogCtx, Logger};
        use std::time::{Duration, Instant};

        /// A SelfHandler that advertises no self-tools and handles nothing — the
        /// loop falls through to MCP (here: no servers), so a `final` script's
        /// answer ends the turn at once.
        struct NoopHandler;
        impl SelfHandler for NoopHandler {
            fn tools(&self) -> Vec<ToolDef> {
                Vec::new()
            }
            fn handle(&mut self, _name: &str, _args: &Value) -> Option<(String, bool)> {
                None
            }
        }

        fn test_log() -> Logger {
            Logger::new(
                LogCtx {
                    run_id: "r".into(),
                    agent_id: "0".into(),
                    agent_path: "0".into(),
                    comp: Comp::Agent,
                    pid: 0,
                    trace_id: None,
                },
                Level::Error, // keep the test quiet
            )
        }

        /// Run the built-in mock LLM with `script` on a background thread, and
        /// return its `http://<addr>` intelligence URL. Blocks until the server
        /// has announced its address through `addr_file`, so the first
        /// `complete()` connects instead of racing the bind.
        fn start_mock_llm(addr_file: &std::path::Path, script: &'static str) -> String {
            let s = addr_file.to_str().unwrap().to_string();
            std::thread::spawn(move || {
                crate::intel::mock::run(&s, script);
            });
            let deadline = Instant::now() + Duration::from_secs(3);
            while !addr_file.exists() {
                assert!(Instant::now() < deadline, "mock-llm never announced");
                std::thread::sleep(Duration::from_millis(10));
            }
            let addr = std::fs::read_to_string(addr_file).expect("read mock-llm addr-file");
            format!("http://{}", addr.trim())
        }

        fn input(instruction: &str) -> LoopInput {
            LoopInput {
                instruction: instruction.into(),
                output_contract: None,
                seed: Vec::new(),
                model: "mock".into(),
                max_steps: 8,
                max_tokens: 100_000,
                deadline: Instant::now() + Duration::from_secs(10),
                cancel: None,
            }
        }

        #[test]
        fn run_turn_returns_the_turns_token_usage() {
            // The `final` script answers in one model call reporting
            // usage{prompt_tokens: 11, completion_tokens: 5} (intel::mock). The
            // turn must surface exactly that split, non-zero, since it is the
            // value control.rs emits upward.
            let dir = tempfile::tempdir().unwrap();
            let sock = dir.path().join("llm.addr");
            let url = start_mock_llm(&sock, "final");

            let intel = IntelClient::from_parts(&url, None).unwrap();
            let inp = input("do the thing");
            let mut handler = NoopHandler;
            let mut session = Session::prepare(&[], &inp, &mut handler).unwrap();
            let mut budget = Budget::new(inp.max_steps, inp.max_tokens, inp.deadline);

            let (outcome, usage) = session
                .run_turn(&intel, &mut handler, &test_log(), &mut budget, None)
                .expect("turn runs against the mock LLM");

            assert_eq!(outcome.status, TerminalStatus::Completed);
            // The producer half: the turn's reported tokens, non-zero, so the
            // AgentMsg::Usage control.rs sends carries real tokens.
            assert_eq!(
                usage.input_tokens, 11,
                "input tokens surfaced from the model"
            );
            assert_eq!(
                usage.output_tokens, 5,
                "output tokens surfaced from the model"
            );
            assert!(usage.total() > 0, "the rolled-up Usage is non-zero");
        }

        #[test]
        fn run_loop_returns_the_runs_total_token_usage() {
            // The one-shot path: run_loop is a single turn, so its returned Usage IS
            // that turn's usage — one Usage per run (no double-count). The `read`
            // script makes a tool call then answers: two model calls, so the run
            // total SUMS both turns' tokens (each reports 11 in; 7 then 5 out).
            let dir = tempfile::tempdir().unwrap();
            let sock = dir.path().join("llm.addr");
            let url = start_mock_llm(&sock, "read");

            let intel = IntelClient::from_parts(&url, None).unwrap();
            let inp = input("read the resource");
            let mut handler = NoopHandler;

            let (outcome, usage) =
                run_loop(&intel, &[], &inp, &mut handler, &test_log()).expect("one-shot run");

            assert_eq!(outcome.status, TerminalStatus::Completed);
            // Two model calls in the run (tool call then final answer) — the run
            // total accumulates both, proving run_loop sums across its turns' calls.
            assert_eq!(usage.input_tokens, 22, "summed input over both model calls");
            assert_eq!(
                usage.output_tokens, 12,
                "summed output over both model calls"
            );
        }
    }
}