vtcode_core/

primary_agent.rs

use std::collections::BTreeMap;
use std::error::Error;
use std::fmt;
use std::path::PathBuf;
use std::sync::Arc;

use vtcode_config::constants::{defaults::DEFAULT_PRIMARY_AGENT_NAME, tools};
use vtcode_config::core::permissions::AgentPermissionsConfig;
use vtcode_config::core::tools::ToolPolicy;
use vtcode_config::{
    DiscoveredSubagents, HookGroupConfig, HooksConfig, SubagentMcpServer, SubagentMemoryScope, SubagentSource,
    SubagentSpec, builtin_primary_build_agent,
};

use crate::config::{ReasoningEffortLevel, VTCodeConfig};
use crate::llm::provider::ToolDefinition;
use crate::permissions::{PermissionRequest, ResolvedPermissionDecision, evaluate_effective_permissions};
use crate::prompts::PromptContext;
use crate::subagents::{ResolvedAgentRuntimeView, inline_mcp_provider};
use tracing::warn;

#[derive(Debug, Clone, PartialEq, Eq)]
pub struct ActivePrimaryAgentSpecIdentity {
    pub name: String,
    pub source: SubagentSource,
    pub file_path: Option<PathBuf>,
}

#[derive(Debug, Clone, PartialEq)]
pub struct ActivePrimaryAgent {
    pub identity: ActivePrimaryAgentSpecIdentity,
    pub display_name: String,
    pub description: String,
    pub color: Option<String>,
    pub aliases: Vec<String>,
    pub instructions: String,
    pub tools: Option<Vec<String>>,
    pub disallowed_tools: Vec<String>,
    pub permissions: AgentPermissionsConfig,
    pub model: Option<String>,
    pub reasoning_effort: Option<ReasoningEffortLevel>,
    pub hooks: Option<HooksConfig>,
    pub skills: Vec<String>,
    pub mcp_servers: Vec<SubagentMcpServer>,
    pub memory: Option<SubagentMemoryScope>,
    pub tool_policy_overrides: BTreeMap<String, ToolPolicy>,
}

impl ActivePrimaryAgent {
    #[must_use]
    pub fn from_spec(spec: &SubagentSpec) -> Self {
        Self::from_runtime_view(&ResolvedAgentRuntimeView::from_spec(spec))
    }

    #[must_use]
    pub fn from_runtime_view(runtime: &ResolvedAgentRuntimeView) -> Self {
        Self {
            identity: ActivePrimaryAgentSpecIdentity {
                name: runtime.canonical_name.clone(),
                source: runtime.source.clone(),
                file_path: runtime.file_path.clone(),
            },
            display_name: runtime.display_name.clone(),
            description: runtime.description.clone(),
            color: runtime.color.clone(),
            aliases: runtime.aliases.clone(),
            instructions: runtime.instructions.clone(),
            tools: runtime.tools.clone(),
            disallowed_tools: runtime.disallowed_tools.clone(),
            permissions: runtime.permissions.clone(),
            model: runtime.model.clone(),
            reasoning_effort: runtime.reasoning_effort,
            hooks: runtime.hooks.clone(),
            skills: runtime.skills.clone(),
            mcp_servers: runtime.mcp_servers.clone(),
            memory: runtime.memory,
            tool_policy_overrides: runtime.tool_policy_overrides.clone(),
        }
    }

    /// Canonical primary agent name (the active session "mode").
    #[must_use]
    pub fn name(&self) -> &str {
        &self.identity.name
    }

    /// Whether this agent contributes lifecycle hooks that originate from
    /// workspace-controlled content (project agent-spec files inside the
    /// repository). Such hooks must be gated behind the workspace lifecycle
    /// hook approval just like hooks declared in a workspace `vtcode.toml`.
    #[must_use]
    pub fn contributes_workspace_controlled_hooks(&self) -> bool {
        self.hooks.is_some()
            && matches!(
                self.identity.source,
                SubagentSource::ProjectVtcode | SubagentSource::ProjectClaude | SubagentSource::ProjectCodex
            )
    }
}

#[derive(Debug, Clone, PartialEq)]
pub struct ActivePrimaryAgentState {
    active: ActivePrimaryAgent,
}

impl Default for ActivePrimaryAgentState {
    fn default() -> Self {
        Self {
            active: ActivePrimaryAgent::from_spec(&builtin_primary_build_agent()),
        }
    }
}

impl ActivePrimaryAgentState {
    #[must_use]
    pub const fn active(&self) -> &ActivePrimaryAgent {
        &self.active
    }

    #[must_use]
    pub fn from_discovery(discovered: &DiscoveredSubagents) -> Self {
        Self::from_specs(&discovered.effective)
    }

    #[must_use]
    pub fn from_specs(specs: &[SubagentSpec]) -> Self {
        Self::from_specs_with_default(specs, DEFAULT_PRIMARY_AGENT_NAME)
    }

    #[must_use]
    pub fn from_specs_with_default(specs: &[SubagentSpec], requested_default: &str) -> Self {
        let requested = if requested_default.trim().is_empty() {
            DEFAULT_PRIMARY_AGENT_NAME
        } else {
            requested_default.trim()
        };
        let active = match resolve_primary_agent(specs, requested) {
            Ok(active) => active,
            Err(_) => {
                if let Some(requested_name) = fallback_notice(requested) {
                    warn!(
                        requested = %requested_name,
                        fallback = %DEFAULT_PRIMARY_AGENT_NAME,
                        "Primary agent '{}' was not discovered; falling back to the built-in '{}' agent. \
                         Set default_primary_agent to a discovered agent or add a matching primary agent.",
                        requested_name,
                        DEFAULT_PRIMARY_AGENT_NAME,
                    );
                }
                ActivePrimaryAgent::from_spec(&builtin_primary_build_agent())
            }
        };
        Self { active }
    }

    pub fn reset_to_default_from_specs(&mut self, specs: &[SubagentSpec]) -> &ActivePrimaryAgent {
        self.active = Self::from_specs(specs).active;
        &self.active
    }

    /// Restore a previously captured active-agent snapshot. Runloop prompt
    /// boundaries use this to keep a write-capable Build handoff intact when
    /// unrelated queued UI input attempts to switch modes mid-turn.
    pub fn restore_snapshot(&mut self, snapshot: ActivePrimaryAgent) -> &ActivePrimaryAgent {
        self.active = snapshot;
        &self.active
    }

    pub fn select_from_discovery(
        &mut self,
        discovered: &DiscoveredSubagents,
        requested: &str,
    ) -> PrimaryAgentResolutionResult<&ActivePrimaryAgent> {
        self.select_from_specs(&discovered.effective, requested)
    }

    pub fn select_from_specs(
        &mut self,
        specs: &[SubagentSpec],
        requested: &str,
    ) -> PrimaryAgentResolutionResult<&ActivePrimaryAgent> {
        let active = resolve_primary_agent(specs, requested)?;
        self.active = active;
        Ok(&self.active)
    }
}

pub type PrimaryAgentResolutionResult<T> = Result<T, PrimaryAgentResolutionError>;

#[derive(Debug, Clone, PartialEq, Eq)]
pub enum PrimaryAgentResolutionError {
    UnknownAgent { requested: String },
}

impl fmt::Display for PrimaryAgentResolutionError {
    fn fmt(&self, f: &mut fmt::Formatter<'_>) -> fmt::Result {
        match self {
            Self::UnknownAgent { requested } => write!(f, "Unknown primary agent {requested}"),
        }
    }
}

impl Error for PrimaryAgentResolutionError {}

pub fn resolve_discovered_primary_agent(
    discovered: &DiscoveredSubagents,
    requested: &str,
) -> PrimaryAgentResolutionResult<ActivePrimaryAgent> {
    resolve_primary_agent(&discovered.effective, requested)
}

pub fn resolve_primary_agent(
    specs: &[SubagentSpec],
    requested: &str,
) -> PrimaryAgentResolutionResult<ActivePrimaryAgent> {
    specs
        .iter()
        .find(|spec| spec.is_primary() && spec.name.eq_ignore_ascii_case(requested))
        .or_else(|| specs.iter().find(|spec| spec.is_primary() && spec.matches_name(requested)))
        .map(ActivePrimaryAgent::from_spec)
        .ok_or_else(|| PrimaryAgentResolutionError::UnknownAgent { requested: requested.to_string() })
}

/// Returns `true` for cleanup-only child-agent tools that must remain
/// available regardless of the active primary agent's tool policy.
///
/// NOTE: after `wait_agent`/`close_agent` were folded into the unified
/// `agent` tool (action='wait'/'close'), `LIFECYCLE_CLEANUP_TOOLS` no longer
/// matches any name that is actually *emitted* to the model — the emitted
/// name is always `tools::AGENT` ("agent"). This carve-out therefore only
/// takes effect if a caller passes one of the legacy granular names
/// (`wait_agent`/`close_agent`) directly rather than the folded name.
/// Expressing true per-action granularity at runtime (allow cleanup actions
/// on the folded `agent` tool while denying `spawn`) requires
/// execution-time action gating, the same mechanism used for
/// `mcp:connect`/`mcp:disconnect` (see `action_qualified_policy_name` in
/// `tool_intent.rs`), and is deferred — see
/// `apply_primary_agent_tool_policy`'s doc comment for the specific gap.
fn is_subagent_cleanup_tool(tool_name: &str) -> bool {
    tools::LIFECYCLE_CLEANUP_TOOLS
        .iter()
        .any(|cleanup| cleanup.eq_ignore_ascii_case(tool_name))
}

/// Returns the configured agent name that should be surfaced to the operator
/// when the resolver cannot find it and falls back to the built-in `build`
/// agent, or `None` when the fallback is expected and should stay silent.
///
/// A fallback is expected (and therefore silent) when the operator asked for
/// the default `build` agent — whether explicitly or because no
/// `default_primary_agent` was configured — since the built-in `build` agent
/// *is* the canonical build. Any other requested name that misses discovery is
/// a configuration mismatch the operator should hear about.
///
/// Extracted as a pure predicate so the decision is unit-testable without a
/// tracing subscriber.
fn fallback_notice(requested_default: &str) -> Option<&str> {
    let requested = requested_default.trim();
    if requested.is_empty() || requested.eq_ignore_ascii_case(DEFAULT_PRIMARY_AGENT_NAME) {
        None
    } else {
        Some(requested)
    }
}

#[must_use]
pub fn primary_agent_allows_tool(agent: &ActivePrimaryAgent, tool_name: &str) -> bool {
    let tool_name = tool_name.trim();

    // Cleanup tools stay available so restricted agents can still join or
    // close already-running child work.
    if is_subagent_cleanup_tool(tool_name) {
        return true;
    }

    // The task tracker stays available so restricted primaries (including
    // custom `auto` agents with an explicit `tools:` list) can still report
    // progress. Planning/interview tools stay author-curated: unlike the
    // tracker they change the session mode, so an explicit omission must
    // keep working.
    if tool_name.eq_ignore_ascii_case(tools::TASK_TRACKER) {
        return true;
    }

    let allow_list_allows = agent
        .tools
        .as_ref()
        .is_none_or(|tools| tools.iter().any(|allowed| allowed.trim().eq_ignore_ascii_case(tool_name)));
    if !allow_list_allows {
        return false;
    }

    !agent
        .disallowed_tools
        .iter()
        .any(|denied| denied.trim().eq_ignore_ascii_case(tool_name))
}

/// Filters the emitted tool list down to those `primary_agent_allows_tool`
/// permits for `agent`.
///
/// KNOWN LIMITATION: filtering is by each tool's canonical
/// `function_name()`, and matching only trims and ignores ASCII case — it
/// does not alias-resolve. For multi-action tools that fold several legacy
/// actions into one emitted name (e.g. `agent` folding
/// spawn/wait/close/send_input/resume, or `mcp` folding connect/disconnect),
/// a granular allow/deny entry such as `"spawn_agent"` in
/// `disallowed_tools`/`tools` has no effect on the folded `agent` tool: the
/// whole tool is either present or absent based on whether `"agent"` itself
/// is allowed. Expressing "allow cleanup (wait/close) but deny spawn"
/// therefore cannot be done here; it requires execution-time action gating —
/// the same action-qualified mechanism already used to keep `mcp:connect`/
/// `mcp:disconnect` HITL-gated separately from bare `mcp` (see
/// `action_qualified_policy_name` in `tool_intent.rs`) — applied per-action
/// at the point the subagent tool call is dispatched. That is deferred.
#[must_use]
pub fn apply_primary_agent_tool_policy(
    tools: Option<Arc<Vec<ToolDefinition>>>,
    agent: &ActivePrimaryAgent,
) -> Option<Arc<Vec<ToolDefinition>>> {
    let tools = tools?;
    let Some(first_rejected) = tools
        .iter()
        .position(|tool| !primary_agent_allows_tool(agent, tool.function_name()))
    else {
        return (!tools.is_empty()).then_some(tools);
    };
    let filtered = tools
        .iter()
        .take(first_rejected)
        .chain(
            tools
                .iter()
                .skip(first_rejected + 1)
                .filter(|tool| primary_agent_allows_tool(agent, tool.function_name())),
        )
        .cloned()
        .collect::<Vec<_>>();

    (!filtered.is_empty()).then(|| Arc::new(filtered))
}

#[must_use]
pub fn build_primary_agent_runtime_config(parent: &VTCodeConfig, agent: &ActivePrimaryAgent) -> VTCodeConfig {
    let mut config = parent.clone();
    // "inherit" (and empty) are sentinels meaning "use the parent's model" — mirrors
    // resolve_subagent_model's handling in subagents/model.rs. Built-in primary agents
    // (e.g. "auto", "build") default their `model` field to "inherit", so without this
    // check every plain `exec` run would clobber a real, explicitly configured model
    // (via --model or [agent].default_model) with the literal string "inherit", which
    // then fails ModelId parsing entirely.
    if let Some(model) = agent.model.as_ref() {
        let trimmed = model.trim();
        if !trimmed.is_empty() && !trimmed.eq_ignore_ascii_case("inherit") {
            config.agent.default_model = model.clone();
        }
    }
    if let Some(reasoning_effort) = agent.reasoning_effort {
        config.agent.reasoning_effort = reasoning_effort;
    }
    merge_primary_mcp_servers(&mut config, agent.mcp_servers.as_slice());
    config
}

#[must_use]
pub fn build_primary_agent_hook_config(global: &HooksConfig, agent: &ActivePrimaryAgent) -> HooksConfig {
    let mut config = global.clone();
    merge_active_primary_hooks(&mut config, agent.hooks.as_ref());
    config
}

pub fn apply_primary_agent_prompt_context(context: &mut PromptContext, agent: &ActivePrimaryAgent) {
    context.replace_available_skills_with_named(agent.skills.as_slice());
}

#[must_use]
pub fn active_primary_agent_permissions(agent: &ActivePrimaryAgent) -> &AgentPermissionsConfig {
    &agent.permissions
}

#[must_use]
pub fn evaluate_active_primary_agent_permissions(
    config: &VTCodeConfig,
    agent: &ActivePrimaryAgent,
    workspace_root: &std::path::Path,
    current_dir: &std::path::Path,
    request: &PermissionRequest,
) -> ResolvedPermissionDecision {
    evaluate_effective_permissions(
        &config.permissions,
        active_primary_agent_permissions(agent),
        workspace_root,
        current_dir,
        request,
    )
}

fn merge_primary_mcp_servers(config: &mut VTCodeConfig, servers: &[SubagentMcpServer]) {
    for server in servers {
        match server {
            SubagentMcpServer::Named(_) => {}
            SubagentMcpServer::Inline(definition) => {
                for (name, value) in definition {
                    if config.mcp.providers.iter().any(|provider| provider.name == *name) {
                        continue;
                    }
                    if let Some(provider) = inline_mcp_provider(name, value) {
                        config.mcp.providers.push(provider);
                    }
                }
            }
        }
    }
}

fn merge_active_primary_hooks(config: &mut HooksConfig, hooks: Option<&HooksConfig>) {
    let Some(hooks) = hooks else {
        return;
    };

    config.lifecycle.quiet_success_output |= hooks.lifecycle.quiet_success_output;
    append_hook_groups(&mut config.lifecycle.session_start, &hooks.lifecycle.session_start);
    append_hook_groups(&mut config.lifecycle.session_end, &hooks.lifecycle.session_end);
    append_hook_groups(&mut config.lifecycle.subagent_start, &hooks.lifecycle.subagent_start);
    append_hook_groups(&mut config.lifecycle.subagent_stop, &hooks.lifecycle.subagent_stop);
    append_hook_groups(&mut config.lifecycle.user_prompt_submit, &hooks.lifecycle.user_prompt_submit);
    append_hook_groups(&mut config.lifecycle.pre_tool_use, &hooks.lifecycle.pre_tool_use);
    append_hook_groups(&mut config.lifecycle.post_tool_use, &hooks.lifecycle.post_tool_use);
    append_hook_groups(&mut config.lifecycle.permission_request, &hooks.lifecycle.permission_request);
    append_hook_groups(&mut config.lifecycle.pre_compact, &hooks.lifecycle.pre_compact);
    append_hook_groups(&mut config.lifecycle.stop, &hooks.lifecycle.stop);
    append_hook_groups(&mut config.lifecycle.notification, &hooks.lifecycle.notification);
}

fn append_hook_groups(target: &mut Vec<HookGroupConfig>, source: &[HookGroupConfig]) {
    target.extend(source.iter().cloned());
}

#[cfg(test)]
mod tests {
    use std::collections::BTreeMap;

    use serde_json::json;
    use tempfile::TempDir;
    use vtcode_config::core::permissions::{AgentPermissionsConfig, PermissionDefault};
    use vtcode_config::{
        HookCommandConfig, HooksConfig, IsolationMode, SubagentDiscoveryInput, SubagentMcpServer, SubagentMemoryScope,
        SubagentSource, builtin_plan_agent, builtin_primary_auto_agent, builtin_primary_duck_agent, builtin_subagents,
        discover_subagents,
    };

    use crate::config::constants::tools;
    use crate::permissions::{ResolvedPermissionDecision, build_permission_request};

    use super::*;

    #[test]
    fn resolves_existing_spec_by_name() {
        let spec = test_spec("planner");
        let active = resolve_primary_agent(&[spec], "planner").expect("resolved");

        assert_eq!(active.identity.name, "planner");
        assert_eq!(active.display_name, "planner");
        assert_eq!(active.description, "planner description");
        assert_eq!(active.color.as_deref(), Some("blue"));
        assert_eq!(active.instructions, "planner instructions");
        assert_eq!(active.tools, Some(vec!["code_search".to_string()]));
        assert_eq!(active.disallowed_tools, vec!["file_operation".to_string()]);
        assert_eq!(active.permissions.default, PermissionDefault::Deny);
        assert_eq!(active.model.as_deref(), Some("gpt-5.1"));
        assert_eq!(active.reasoning_effort, Some(ReasoningEffortLevel::High));
        assert!(active.hooks.is_none());
    }

    #[test]
    fn unknown_agent_error_preserves_current_active_agent() {
        let current = test_spec("current");
        let specs = vec![current.clone()];
        let mut state = ActivePrimaryAgentState::default();
        let original = state.select_from_specs(&specs, "current").expect("initial selection").clone();

        let error = state.select_from_specs(&specs, "missing").expect_err("unknown agent");

        assert_eq!(error, PrimaryAgentResolutionError::UnknownAgent { requested: "missing".to_string() });
        assert_eq!(state.active(), &original);
    }

    #[test]
    fn restore_snapshot_returns_to_write_capable_build_agent() {
        let mut state = ActivePrimaryAgentState::default();
        let build_snapshot = state.active().clone();
        state
            .select_from_specs(&[test_spec("duck")], "duck")
            .expect("select leaked agent");
        assert_eq!(state.active().identity.name, "duck");

        state.restore_snapshot(build_snapshot);

        assert_eq!(state.active().identity.name, "build");
    }

    #[test]
    fn alias_resolution_uses_existing_matches_name_semantics() {
        let mut spec = test_spec("reviewer");
        spec.aliases = vec!["critic".to_string()];

        let active = resolve_primary_agent(&[spec], "CRITIC").expect("resolved by alias");

        assert_eq!(active.identity.name, "reviewer");
        assert_eq!(active.display_name, "reviewer");
    }

    #[test]
    fn exact_name_resolution_wins_over_alias() {
        let mut build = test_spec("build");
        build.aliases = vec!["builder".to_string()];
        let builder = test_spec("builder");

        let active = resolve_primary_agent(&[build, builder], "builder").expect("resolved");

        assert_eq!(active.identity.name, "builder");
    }

    #[test]
    fn ignored_subagent_fields_do_not_enter_primary_agent_runtime() {
        let mut spec = test_spec("worker");
        spec.aliases = vec!["builder".to_string()];
        spec.skills = vec!["rust".to_string()];
        spec.mcp_servers = vec![SubagentMcpServer::Named("filesystem".to_string())];
        spec.background = true;
        spec.max_turns = Some(12);
        spec.nickname_candidates = vec!["w".to_string()];
        spec.initial_prompt = Some("start here".to_string());
        spec.memory = Some(SubagentMemoryScope::Project);
        spec.isolation = Some(IsolationMode::Full);

        let active = ActivePrimaryAgent::from_spec(&spec);

        assert_eq!(active.identity.name, "worker");
        assert_eq!(active.display_name, "worker");
        assert_eq!(active.description, "worker description");
        assert_eq!(active.color.as_deref(), Some("blue"));
        assert_eq!(active.aliases, vec!["builder".to_string()]);
        assert_eq!(active.instructions, "worker instructions");
        assert_eq!(active.tools, Some(vec!["code_search".to_string()]));
        assert_eq!(active.disallowed_tools, vec!["file_operation".to_string()]);
        assert_eq!(active.permissions.default, PermissionDefault::Deny);
        assert_eq!(active.model.as_deref(), Some("gpt-5.1"));
        assert_eq!(active.reasoning_effort, Some(ReasoningEffortLevel::High));
        assert_eq!(active.skills, vec!["rust".to_string()]);
        assert_eq!(active.mcp_servers, vec![SubagentMcpServer::Named("filesystem".to_string())]);
        assert_eq!(active.memory, Some(SubagentMemoryScope::Project));
    }

    #[test]
    fn primary_runtime_adapter_uses_shared_resolved_view_for_overlapping_fields() {
        let mut spec = test_spec("worker");
        spec.description = "Worker display metadata".to_string();
        spec.color = Some("green".to_string());
        spec.aliases = vec!["builder".to_string()];
        spec.skills = vec!["rust".to_string(), "repo".to_string()];
        spec.mcp_servers = vec![SubagentMcpServer::Named("filesystem".to_string())];
        spec.hooks = Some(HooksConfig::default());
        spec.memory = Some(SubagentMemoryScope::Project);

        let runtime = ResolvedAgentRuntimeView::from_spec(&spec);
        let active = ActivePrimaryAgent::from_runtime_view(&runtime);

        assert_eq!(runtime.canonical_name, "worker");
        assert_eq!(runtime.display_name, "worker");
        assert_eq!(runtime.description, "Worker display metadata");
        assert_eq!(runtime.color.as_deref(), Some("green"));
        assert_eq!(runtime.aliases, vec!["builder".to_string()]);
        assert_eq!(runtime.skills, vec!["rust".to_string(), "repo".to_string()]);
        assert_eq!(runtime.mcp_servers.len(), 1);
        assert!(runtime.hooks.is_some());
        assert_eq!(runtime.memory, Some(SubagentMemoryScope::Project));
        assert!(runtime.read_only);
        assert_eq!(active.identity.name, runtime.canonical_name);
        assert_eq!(active.display_name, runtime.display_name);
        assert_eq!(active.description, runtime.description);
        assert_eq!(active.color, runtime.color);
        assert_eq!(active.aliases, runtime.aliases);
        assert_eq!(active.instructions, runtime.instructions);
        assert_eq!(active.tools, runtime.tools);
        assert_eq!(active.disallowed_tools, runtime.disallowed_tools);
        assert_eq!(active.permissions, runtime.permissions);
        assert_eq!(active.model, runtime.model);
        assert_eq!(active.reasoning_effort, runtime.reasoning_effort);
        assert_eq!(active.hooks, runtime.hooks);
        assert_eq!(active.skills, runtime.skills);
        assert_eq!(active.mcp_servers, runtime.mcp_servers);
        assert_eq!(active.memory, runtime.memory);
    }

    #[test]
    fn default_state_uses_builtin_build_agent() {
        let mut state = ActivePrimaryAgentState::default();

        assert_eq!(state.active().identity.name, DEFAULT_PRIMARY_AGENT_NAME);
        assert_eq!(state.active().identity.name, "build");
        assert_eq!(state.active().identity.source, SubagentSource::Builtin);

        state.select_from_specs(&[test_spec("worker")], "worker").expect("selected");
        assert_eq!(state.active().identity.name, "worker");

        state.reset_to_default_from_specs(&[]);

        assert_eq!(state.active().identity.name, DEFAULT_PRIMARY_AGENT_NAME);
        assert_eq!(state.active().identity.name, "build");
        assert_eq!(state.active().identity.source, SubagentSource::Builtin);
    }

    #[test]
    fn from_specs_falls_back_to_builtin_build_agent() {
        let active = ActivePrimaryAgentState::from_specs(&[]);

        assert_eq!(active.active().identity.name, "build");
        assert_eq!(active.active().identity.source, SubagentSource::Builtin);
    }

    #[test]
    fn from_specs_with_default_selects_configured_primary_agent() {
        let active = ActivePrimaryAgentState::from_specs_with_default(&[test_spec("builder")], "builder");

        assert_eq!(active.active().identity.name, "builder");
    }

    #[test]
    fn from_specs_with_default_falls_back_to_build_for_missing_configured_agent() {
        let active = ActivePrimaryAgentState::from_specs_with_default(&[test_spec("builder")], "missing");

        assert_eq!(active.active().identity.name, "build");
        assert_eq!(active.active().identity.source, SubagentSource::Builtin);
    }

    #[test]
    fn fallback_notice_silent_for_build_and_empty() {
        // The built-in `build` agent is the canonical default, so falling back
        // to it when the operator asked for `build` (or nothing) is expected.
        assert_eq!(fallback_notice("build"), None);
        assert_eq!(fallback_notice("BUILD"), None);
        assert_eq!(fallback_notice("Build"), None);
        assert_eq!(fallback_notice(""), None);
        assert_eq!(fallback_notice("   "), None);
    }

    #[test]
    fn fallback_notice_surfaces_non_default_mismatches() {
        // Any other requested name that misses discovery is a configuration
        // mismatch the operator should be warned about.
        assert_eq!(fallback_notice("duck"), Some("duck"));
        assert_eq!(fallback_notice("auto"), Some("auto"));
        assert_eq!(fallback_notice("  duck  "), Some("duck"));
    }

    #[test]
    fn discovery_precedence_overrides_builtin_build_agent() {
        let temp = TempDir::new().expect("tempdir");
        let discovered = discover_subagents(&SubagentDiscoveryInput {
            workspace_root: temp.path().to_path_buf(),
            cli_agents: Some(json!({
                "build": {
                    "description": "CLI build",
                    "prompt": "cli build instructions",
                    "model": "gpt-cli",
                    "mode": "primary",
                    "permissions": { "default": "ask" }
                }
            })),
            plugin_agent_files: Vec::new(),
            include_user_agents: false,
        })
        .expect("discovered subagents");

        let active = ActivePrimaryAgentState::from_discovery(&discovered);

        assert_eq!(active.active().identity.name, "build");
        assert_eq!(active.active().identity.source, SubagentSource::Cli);
        assert_eq!(active.active().instructions, "cli build instructions");
        assert_eq!(active.active().model.as_deref(), Some("gpt-cli"));
    }

    #[test]
    fn default_build_agent_allows_baseline_read_and_exec_tools() {
        let active = ActivePrimaryAgentState::default();

        assert!(primary_agent_allows_tool(active.active(), tools::CODE_SEARCH));
        assert!(primary_agent_allows_tool(active.active(), tools::EXEC_COMMAND));
        assert!(primary_agent_allows_tool(active.active(), tools::APPLY_PATCH));
        assert!(primary_agent_allows_tool(active.active(), tools::RUN_PTY_CMD));
        assert_eq!(active.active().permissions.default, PermissionDefault::Ask);
    }

    #[test]
    fn build_and_auto_expose_the_same_tools_and_policy_decisions() {
        let build = ActivePrimaryAgent::from_spec(&builtin_primary_build_agent());
        let auto = ActivePrimaryAgent::from_spec(&builtin_primary_auto_agent());
        let tools_to_compare = [
            tools::CODE_SEARCH,
            tools::EXEC_COMMAND,
            tools::APPLY_PATCH,
            tools::RUN_PTY_CMD,
            tools::WRITE_FILE,
            tools::REQUEST_USER_INPUT,
        ];

        for tool in tools_to_compare {
            assert_eq!(
                primary_agent_allows_tool(&build, tool),
                primary_agent_allows_tool(&auto, tool),
                "Build and Auto must expose the same capability for {tool}"
            );
        }
    }

    #[test]
    fn readonly_primaries_expose_public_read_tools_and_block_apply_patch() {
        for spec in [builtin_primary_duck_agent(), builtin_plan_agent()] {
            let name = spec.name.clone();
            let active = ActivePrimaryAgent::from_spec(&spec);

            assert!(primary_agent_allows_tool(&active, tools::CODE_SEARCH), "{name} should expose code_search");
            assert!(primary_agent_allows_tool(&active, tools::EXEC_COMMAND), "{name} should expose exec_command");
            assert!(primary_agent_allows_tool(&active, tools::REQUEST_USER_INPUT));
            assert!(primary_agent_allows_tool(&active, tools::RECORD_DECISION));

            assert!(!primary_agent_allows_tool(&active, tools::APPLY_PATCH), "{name} must not expose apply_patch");
            assert!(!primary_agent_allows_tool(&active, tools::RUN_PTY_CMD), "{name} must not expose run_pty_cmd");
            assert!(!primary_agent_allows_tool(&active, tools::WRITE_FILE), "{name} must not expose write_file");
        }
    }

    #[test]
    fn tool_policy_intersects_allow_list_then_applies_deny_list() {
        let mut spec = test_spec("worker");
        spec.tools = Some(vec![" \tCoDe_SeArCh\n".to_string(), "\u{2003}ExEc_CoMmAnD\u{a0}".to_string()]);
        spec.disallowed_tools = vec!["\u{a0}CODE_search\u{2003}".to_string()];
        let active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, "\u{a0}EXEC_command\u{2003}"));
        assert!(!primary_agent_allows_tool(&active, " \tcode_SEARCH\n"));
        assert!(!primary_agent_allows_tool(&active, tools::APPLY_PATCH));
        assert!(!primary_agent_allows_tool(&active, "exec_command_extra"));
        assert!(!primary_agent_allows_tool(&active, tools::RUN_PTY_CMD));
    }

    #[test]
    fn absent_tool_allow_list_allows_names_except_deny_matches() {
        let mut spec = test_spec("worker");
        spec.tools = None;
        spec.disallowed_tools = vec![" \tCoDe_SeArCh\n".to_string()];
        let active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, " ExEc_CoMmAnD "));
        assert!(primary_agent_allows_tool(&active, "custom_tool"));
        assert!(!primary_agent_allows_tool(&active, " CODE_search "));
    }

    #[test]
    fn tool_policy_preserves_empty_name_matching() {
        let mut spec = test_spec("worker");
        spec.tools = None;
        spec.disallowed_tools.clear();
        let mut active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, ""));
        assert!(primary_agent_allows_tool(&active, "\u{2003}\t\u{a0}"));
        active.tools = Some(Vec::new());
        assert!(!primary_agent_allows_tool(&active, ""));
        assert!(!primary_agent_allows_tool(&active, "\u{2003}\t\u{a0}"));
        active.tools = Some(vec!["\u{a0} \t".to_string()]);
        assert!(primary_agent_allows_tool(&active, ""));
        assert!(primary_agent_allows_tool(&active, "\u{2003}"));
        assert!(!primary_agent_allows_tool(&active, "custom_tool"));
        active.disallowed_tools = vec![String::new()];
        assert!(!primary_agent_allows_tool(&active, ""));
        assert!(!primary_agent_allows_tool(&active, "\u{2003}"));
    }

    #[test]
    fn tool_policy_keeps_non_ascii_case_and_non_whitespace_distinct() {
        let mut spec = test_spec("worker");
        spec.tools = Some(vec!["\u{2003}CAFÉ\u{a0}".to_string()]);
        spec.disallowed_tools = vec![" café ".to_string()];
        let mut active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, " caFÉ "));
        assert!(!primary_agent_allows_tool(&active, "café"));
        assert!(!primary_agent_allows_tool(&active, "CAFE"));
        assert!(!primary_agent_allows_tool(&active, "\u{200b}CAFÉ\u{200b}"));

        active.tools = None;
        assert!(primary_agent_allows_tool(&active, "CAFÉ"));
        assert!(!primary_agent_allows_tool(&active, "CaFé"));
    }

    #[test]
    fn mixed_case_cleanup_and_tracker_names_bypass_empty_allow_list_and_explicit_denials() {
        let mut spec = test_spec("restricted");
        spec.tools = Some(Vec::new());
        spec.disallowed_tools = vec![
            " Wait_AGENT ".to_string(),
            "\u{2003}Close_Agent\u{a0}".to_string(),
            "\tTask_TRACKER\n".to_string(),
        ];
        let active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, "\u{a0}WAIT_agent\u{2003}"));
        assert!(primary_agent_allows_tool(&active, " Close_AGENT "));
        assert!(primary_agent_allows_tool(&active, " TaSk_TrAcKeR "));
        assert!(!primary_agent_allows_tool(&active, " Agent "));
        assert!(!primary_agent_allows_tool(&active, " Spawn_AGENT "));
        assert!(!primary_agent_allows_tool(&active, "task_tracker_extra"));
        assert!(!primary_agent_allows_tool(&active, "wait_agent_extra"));
    }

    #[test]
    fn filtered_tool_catalog_preserves_surviving_names_and_order() {
        let mut spec = test_spec("worker");
        spec.tools = Some(vec![
            " APPLY_patch ".to_string(),
            " EXEC_command ".to_string(),
            " CODE_search ".to_string(),
        ]);
        spec.disallowed_tools = vec![" code_SEARCH ".to_string()];
        let active = ActivePrimaryAgent::from_spec(&spec);
        let catalog = Arc::new(
            [
                "Exec_Command",
                "code_search",
                "WaIt_AgEnT",
                "apply_patch",
                "TASK_TRACKER",
                "run_pty_cmd",
                "agent",
            ]
            .into_iter()
            .map(|name| ToolDefinition::function(name.to_string(), String::new(), json!({"type": "object"})))
            .collect(),
        );

        let filtered = apply_primary_agent_tool_policy(Some(Arc::clone(&catalog)), &active).expect("surviving tools");

        assert!(!Arc::ptr_eq(&filtered, &catalog));
        assert_eq!(
            filtered.iter().map(ToolDefinition::function_name).collect::<Vec<_>>(),
            ["Exec_Command", "WaIt_AgEnT", "apply_patch", "TASK_TRACKER"]
        );
        assert_eq!(catalog.len(), 7);
        assert_eq!(catalog[1].function_name(), "code_search");
    }

    #[test]
    fn unchanged_tool_catalog_reuses_shared_definitions() {
        let mut spec = test_spec("worker");
        spec.tools = None;
        spec.disallowed_tools.clear();
        let active = ActivePrimaryAgent::from_spec(&spec);
        let catalog = Arc::new(vec![ToolDefinition::function(
            "exec_command".to_string(),
            "Run a command".to_string(),
            json!({"type": "object", "properties": {"cmd": {"type": "string"}}}),
        )]);

        let filtered = apply_primary_agent_tool_policy(Some(Arc::clone(&catalog)), &active).expect("unchanged catalog");

        assert!(Arc::ptr_eq(&filtered, &catalog));
        assert_eq!(filtered[0].function_name(), "exec_command");
    }

    #[test]
    fn tool_policy_returns_none_for_absent_empty_or_fully_denied_catalogs() {
        let active = ActivePrimaryAgent::from_spec(&test_spec("worker"));
        assert!(apply_primary_agent_tool_policy(None, &active).is_none());
        assert!(apply_primary_agent_tool_policy(Some(Arc::new(Vec::new())), &active).is_none());
        let denied = Arc::new(vec![ToolDefinition::function(
            "apply_patch".to_string(),
            String::new(),
            json!({"type": "object"}),
        )]);
        assert!(apply_primary_agent_tool_policy(Some(denied), &active).is_none());
    }

    #[test]
    fn empty_present_tool_allow_list_exposes_no_tools() {
        let mut spec = test_spec("worker");
        spec.tools = Some(Vec::new());
        spec.disallowed_tools = Vec::new();
        let active = ActivePrimaryAgent::from_spec(&spec);

        assert!(!primary_agent_allows_tool(&active, tools::CODE_SEARCH));
    }

    // NOTE: `spawn_agent`/`wait_agent`/`close_agent`/etc. are legacy granular
    // subagent-lifecycle tool names. Since wait/close were folded into the
    // unified `agent` tool (action='wait'/'close'), these granular names are
    // never actually emitted to the model — the only subagent tool name that
    // ever reaches `apply_primary_agent_tool_policy`'s runtime filtering is
    // `tools::AGENT` ("agent"). The assertions below against the granular
    // names are legacy-taxonomy coverage of the `primary_agent_allows_tool`/
    // `is_subagent_cleanup_tool` predicates directly (useful if a caller ever
    // passes a legacy name explicitly), NOT a test of the runtime filtering
    // path exercised by `apply_primary_agent_tool_policy`. The final
    // assertion covers that real runtime path: with an allow-list that
    // excludes "agent", the whole folded tool is filtered out, and there is
    // currently no way to allow cleanup (wait/close) while denying spawn for
    // the emitted "agent" tool (see `apply_primary_agent_tool_policy`'s doc
    // comment for the deferred execution-time action-gating fix).
    #[test]
    fn subagent_cleanup_tools_bypass_policy_but_new_work_obeys_policy() {
        let mut spec = test_spec("restricted");
        spec.tools = Some(vec!["code_search".to_string()]);
        spec.disallowed_tools = vec![
            tools::SPAWN_AGENT.to_string(),
            tools::WAIT_AGENT.to_string(),
            tools::CLOSE_AGENT.to_string(),
            tools::SEND_INPUT.to_string(),
            tools::RESUME_AGENT.to_string(),
            tools::SPAWN_BACKGROUND_SUBPROCESS.to_string(),
        ];
        let active = ActivePrimaryAgent::from_spec(&spec);

        // Non-cleanup tools respect the policy.
        assert!(!primary_agent_allows_tool(&active, "command_session"));
        assert!(!primary_agent_allows_tool(&active, "file_operation"));
        assert!(!primary_agent_allows_tool(&active, tools::SPAWN_AGENT));
        assert!(!primary_agent_allows_tool(&active, tools::SPAWN_BACKGROUND_SUBPROCESS));
        assert!(!primary_agent_allows_tool(&active, tools::SEND_INPUT));
        assert!(!primary_agent_allows_tool(&active, tools::RESUME_AGENT));

        // Cleanup tools remain available even when listed in disallowed_tools
        // and absent from the allow list.
        assert!(primary_agent_allows_tool(&active, tools::WAIT_AGENT));
        assert!(primary_agent_allows_tool(&active, tools::CLOSE_AGENT));

        // Runtime reality: the allow list excludes "agent" (the only name
        // ever emitted for subagent tools post-fold), so the whole tool is
        // filtered out regardless of the cleanup carve-out above.
        assert!(!primary_agent_allows_tool(&active, tools::AGENT));
    }

    #[test]
    fn task_tracker_bypasses_restricted_tool_allow_list_but_mode_tools_obey_it() {
        // A custom primary with an explicit `tools:` list omitting the
        // tracker must still expose it, while planning/interview tools stay
        // author-curated and other tools stay gated.
        let mut spec = test_spec("restricted");
        spec.tools = Some(vec![tools::CODE_SEARCH.to_string()]);
        let active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, tools::TASK_TRACKER));
        assert!(!primary_agent_allows_tool(&active, tools::START_PLANNING));
        assert!(!primary_agent_allows_tool(&active, tools::APPLY_PATCH));
    }

    #[test]
    fn task_tracker_bypass_survives_explicit_disallow_entry() {
        // Mirrors the cleanup-tool guarantee: even a `disallowed_tools`
        // entry must not silence progress reporting.
        let mut spec = test_spec("restricted");
        spec.tools = Some(vec![tools::CODE_SEARCH.to_string()]);
        spec.disallowed_tools = vec![tools::TASK_TRACKER.to_string()];
        let active = ActivePrimaryAgent::from_spec(&spec);

        assert!(primary_agent_allows_tool(&active, tools::TASK_TRACKER));
        assert!(!primary_agent_allows_tool(&active, tools::APPLY_PATCH));
    }

    #[test]
    fn build_primary_agent_runtime_config_preserves_baseline_fields_and_merges_mcp() {
        let mut parent = VTCodeConfig::default();
        parent.agent.default_model = "parent-model".to_string();
        parent.mcp.providers.push(
            serde_json::from_value(json!({
                "name": "global",
                "command": "global-mcp",
                "args": []
            }))
            .expect("global provider"),
        );

        let mut spec = test_spec("worker");
        spec.permissions = AgentPermissionsConfig::new(PermissionDefault::Auto);
        spec.model = Some("agent-model".to_string());
        spec.reasoning_effort = Some(ReasoningEffortLevel::Low);
        spec.mcp_servers = vec![SubagentMcpServer::Inline(BTreeMap::from([
            (
                "global".to_string(),
                json!({
                    "type": "stdio",
                    "command": "duplicate-mcp"
                }),
            ),
            (
                "local".to_string(),
                json!({
                    "type": "stdio",
                    "command": "local-mcp"
                }),
            ),
        ]))];
        let active = ActivePrimaryAgent::from_spec(&spec);

        let runtime = build_primary_agent_runtime_config(&parent, &active);

        assert_eq!(runtime.agent.default_model, "agent-model");
        assert_eq!(runtime.agent.reasoning_effort, ReasoningEffortLevel::Low);
        assert_eq!(runtime.mcp.providers.len(), 2);
        assert_eq!(runtime.mcp.providers[0].name, "global");
        assert_eq!(runtime.mcp.providers[1].name, "local");
        let providers = serde_json::to_value(&runtime.mcp.providers).expect("serialize providers");
        assert_eq!(providers[0]["command"], "global-mcp");
        assert_eq!(providers[1]["command"], "local-mcp");
        assert_eq!(providers[1]["args"], json!([]));
    }

    #[test]
    fn build_primary_agent_runtime_config_inherits_parent_model_when_spec_says_inherit() {
        let mut parent = VTCodeConfig::default();
        parent.agent.default_model = "parent-model".to_string();

        let mut spec = test_spec("auto");
        spec.model = Some("inherit".to_string());
        let active = ActivePrimaryAgent::from_spec(&spec);

        let runtime = build_primary_agent_runtime_config(&parent, &active);

        assert_eq!(runtime.agent.default_model, "parent-model");
    }

    #[test]
    fn build_primary_agent_runtime_config_inherits_parent_model_when_spec_model_empty() {
        let mut parent = VTCodeConfig::default();
        parent.agent.default_model = "parent-model".to_string();

        let mut spec = test_spec("auto");
        spec.model = Some(String::new());
        let active = ActivePrimaryAgent::from_spec(&spec);

        let runtime = build_primary_agent_runtime_config(&parent, &active);

        assert_eq!(runtime.agent.default_model, "parent-model");
    }

    #[test]
    fn built_in_primary_agents_resolve_required_permission_policy() {
        let builtins = builtin_subagents();

        for name in ["duck", "plan", "build", "auto"] {
            let active = resolve_primary_agent(&builtins, name)
                .unwrap_or_else(|_| panic!("missing built-in primary agent {name}"));
            assert_eq!(active.identity.name, name);
            let expected_default = match name {
                "build" => PermissionDefault::Ask,
                "auto" => PermissionDefault::Auto,
                "plan" | "duck" => PermissionDefault::Deny,
                _ => unreachable!("unexpected built-in primary agent"),
            };
            assert_eq!(active.permissions.default, expected_default);
        }
    }

    #[test]
    fn active_primary_permissions_overlay_runtime_decisions() {
        let builtins = builtin_subagents();
        let mut state = ActivePrimaryAgentState::from_specs(&builtins);
        let config = VTCodeConfig::default();
        let workspace = TempDir::new().expect("workspace");
        let current_dir = workspace.path();

        state.select_from_specs(&builtins, "auto").expect("auto primary");
        let exec = build_permission_request(
            workspace.path(),
            current_dir,
            tools::UNIFIED_EXEC,
            Some(&json!({"command": "cargo test"})),
        );
        assert_eq!(
            evaluate_active_primary_agent_permissions(&config, state.active(), workspace.path(), current_dir, &exec,),
            ResolvedPermissionDecision::Auto
        );

        state.select_from_specs(&builtins, "plan").expect("plan primary");
        let edit = build_permission_request(
            workspace.path(),
            current_dir,
            tools::UNIFIED_FILE,
            Some(&json!({"action": "edit", "path": "src/lib.rs"})),
        );
        assert_eq!(
            evaluate_active_primary_agent_permissions(&config, state.active(), workspace.path(), current_dir, &edit,),
            ResolvedPermissionDecision::Deny
        );
    }

    #[test]
    fn primary_agent_switching_changes_permission_policy_without_mutating_parent_config() {
        let builtins = builtin_subagents();
        let mut state = ActivePrimaryAgentState::from_specs(&builtins);
        let initial_model = state.active().model.clone();
        let initial_tools = state.active().tools.clone();

        state.select_from_specs(&builtins, "auto").expect("auto primary");
        assert_eq!(state.active().identity.name, "auto");
        assert_eq!(state.active().permissions.default, PermissionDefault::Auto);
        assert_eq!(state.active().model, initial_model);
        assert_eq!(state.active().tools, initial_tools);

        state.select_from_specs(&builtins, "duck").expect("duck primary");
        assert_eq!(state.active().identity.name, "duck");
        assert_eq!(state.active().permissions.default, PermissionDefault::Deny);
        assert_eq!(state.active().model, initial_model);
        assert_ne!(state.active().tools, initial_tools);
    }

    #[test]
    fn primary_hook_config_merges_supported_main_session_events_after_global_hooks() {
        let mut global = HooksConfig::default();
        global.lifecycle.user_prompt_submit = vec![hook_group("global-user")];
        global.lifecycle.pre_tool_use = vec![hook_group("global-pre")];
        global.lifecycle.post_tool_use = vec![hook_group("global-post")];
        global.lifecycle.permission_request = vec![hook_group("global-permission")];
        global.lifecycle.pre_compact = vec![hook_group("global-compact")];
        global.lifecycle.stop = vec![hook_group("global-stop")];
        global.lifecycle.notification = vec![hook_group("global-notification")];

        let mut primary_hooks = HooksConfig::default();
        primary_hooks.lifecycle.user_prompt_submit = vec![hook_group("primary-user")];
        primary_hooks.lifecycle.pre_tool_use = vec![hook_group("primary-pre")];
        primary_hooks.lifecycle.post_tool_use = vec![hook_group("primary-post")];
        primary_hooks.lifecycle.permission_request = vec![hook_group("primary-permission")];
        primary_hooks.lifecycle.pre_compact = vec![hook_group("primary-compact")];
        primary_hooks.lifecycle.stop = vec![hook_group("primary-stop")];
        primary_hooks.lifecycle.notification = vec![hook_group("primary-notification")];

        let mut spec = test_spec("worker");
        spec.hooks = Some(primary_hooks);
        let active = ActivePrimaryAgent::from_spec(&spec);

        let merged = build_primary_agent_hook_config(&global, &active);

        assert_hook_commands(&merged.lifecycle.user_prompt_submit, &["global-user", "primary-user"]);
        assert_hook_commands(&merged.lifecycle.pre_tool_use, &["global-pre", "primary-pre"]);
        assert_hook_commands(&merged.lifecycle.post_tool_use, &["global-post", "primary-post"]);
        assert_hook_commands(&merged.lifecycle.permission_request, &["global-permission", "primary-permission"]);
        assert_hook_commands(&merged.lifecycle.pre_compact, &["global-compact", "primary-compact"]);
        assert_hook_commands(&merged.lifecycle.stop, &["global-stop", "primary-stop"]);
        assert_hook_commands(&merged.lifecycle.notification, &["global-notification", "primary-notification"]);
    }

    #[test]
    fn primary_hook_config_merges_session_and_subagent_lifecycle_events() {
        let global = HooksConfig::default();
        let mut primary_hooks = HooksConfig::default();
        primary_hooks.lifecycle.session_start = vec![hook_group("primary-session-start")];
        primary_hooks.lifecycle.session_end = vec![hook_group("primary-session-end")];
        primary_hooks.lifecycle.subagent_start = vec![hook_group("primary-subagent-start")];
        primary_hooks.lifecycle.subagent_stop = vec![hook_group("primary-subagent-stop")];
        primary_hooks.lifecycle.task_completion = vec![hook_group("primary-task-completion")];
        primary_hooks.lifecycle.task_completed = vec![hook_group("primary-task-completed")];

        let mut spec = test_spec("worker");
        spec.hooks = Some(primary_hooks);
        let active = ActivePrimaryAgent::from_spec(&spec);

        let merged = build_primary_agent_hook_config(&global, &active);

        assert_hook_commands(&merged.lifecycle.session_start, &["primary-session-start"]);
        assert_hook_commands(&merged.lifecycle.session_end, &["primary-session-end"]);
        assert_hook_commands(&merged.lifecycle.subagent_start, &["primary-subagent-start"]);
        assert_hook_commands(&merged.lifecycle.subagent_stop, &["primary-subagent-stop"]);
        assert!(merged.lifecycle.task_completion.is_empty());
        assert!(merged.lifecycle.task_completed.is_empty());
        assert!(merged.lifecycle.stop.is_empty());
    }

    #[test]
    fn primary_hook_config_recomputes_without_previous_primary_leakage() {
        let global = HooksConfig::default();
        let mut first_hooks = HooksConfig::default();
        first_hooks.lifecycle.pre_tool_use = vec![hook_group("first-pre")];
        let mut second_hooks = HooksConfig::default();
        second_hooks.lifecycle.pre_tool_use = vec![hook_group("second-pre")];

        let mut first = test_spec("first");
        first.hooks = Some(first_hooks);
        let mut second = test_spec("second");
        second.hooks = Some(second_hooks);
        let specs = vec![first, second];
        let mut state = ActivePrimaryAgentState::default();

        state.select_from_specs(&specs, "first").expect("selected first");
        let first_config = build_primary_agent_hook_config(&global, state.active());
        assert_hook_commands(&first_config.lifecycle.pre_tool_use, &["first-pre"]);

        state.select_from_specs(&specs, "second").expect("selected second");
        let second_config = build_primary_agent_hook_config(&global, state.active());
        assert_hook_commands(&second_config.lifecycle.pre_tool_use, &["second-pre"]);
    }

    #[test]
    fn active_primary_state_recomputes_skills_mcp_and_metadata_on_switch() {
        let mut first = test_spec("first");
        first.description = "First metadata".to_string();
        first.color = Some("red".to_string());
        first.aliases = vec!["one".to_string()];
        first.skills = vec!["rust".to_string()];
        first.mcp_servers = vec![SubagentMcpServer::Inline(BTreeMap::from([(
            "first-mcp".to_string(),
            json!({
                "type": "stdio",
                "command": "first-mcp"
            }),
        )]))];
        let second = test_spec("second");
        let specs = vec![first, second];
        let mut state = ActivePrimaryAgentState::default();

        state.select_from_specs(&specs, "one").expect("selected first by alias");
        assert_eq!(state.active().identity.name, "first");
        assert_eq!(state.active().description, "First metadata");
        assert_eq!(state.active().color.as_deref(), Some("red"));
        assert_eq!(state.active().aliases, vec!["one".to_string()]);
        assert_eq!(state.active().skills, vec!["rust".to_string()]);
        assert_eq!(state.active().mcp_servers.len(), 1);

        state.select_from_specs(&specs, "second").expect("selected second");
        assert_eq!(state.active().identity.name, "second");
        assert_eq!(state.active().description, "second description");
        assert_eq!(state.active().color.as_deref(), Some("blue"));
        assert!(state.active().aliases.is_empty());
        assert!(state.active().skills.is_empty());
        assert!(state.active().mcp_servers.is_empty());
    }

    fn test_spec(name: &str) -> SubagentSpec {
        SubagentSpec {
            name: name.to_string(),
            description: format!("{name} description"),
            prompt: format!("{name} instructions"),
            tools: Some(vec!["code_search".to_string()]),
            disallowed_tools: vec!["file_operation".to_string()],
            model: Some("gpt-5.1".to_string()),
            color: Some("blue".to_string()),
            reasoning_effort: Some(ReasoningEffortLevel::High),
            permissions: AgentPermissionsConfig::new(PermissionDefault::Deny),
            skills: Vec::new(),
            mcp_servers: Vec::new(),
            hooks: None,
            background: false,
            mode: vtcode_config::AgentMode::Primary,
            max_turns: None,
            nickname_candidates: Vec::new(),
            initial_prompt: None,
            memory: None,
            isolation: None,
            aliases: Vec::new(),
            source: SubagentSource::ProjectVtcode,
            file_path: None,
            warnings: Vec::new(),
            tool_policy_overrides: std::collections::BTreeMap::new(),
        }
    }

    fn hook_group(command: &str) -> HookGroupConfig {
        HookGroupConfig {
            matcher: None,
            hooks: vec![HookCommandConfig {
                command: command.to_string(),
                ..HookCommandConfig::default()
            }],
        }
    }

    fn assert_hook_commands(groups: &[HookGroupConfig], expected: &[&str]) {
        let commands = groups
            .iter()
            .flat_map(|group| group.hooks.iter())
            .map(|hook| hook.command.as_str())
            .collect::<Vec<_>>();
        assert_eq!(commands, expected);
    }
}