agent_engine/extensions/

manager.rs

//! Extension manager — discovers, starts, and manages extension lifecycles.

use std::collections::HashMap;
use std::path::PathBuf;
use std::sync::Arc;

use super::capability::{
    ExtensionCapabilitySnapshot, FutureCapabilityEntry, HookCapabilityEntry, ToolCapabilityEntry,
};
use super::config::{diagnose_extension_config, ExtensionConfigDiagnostics};
use super::context_provider::{
    dormant_context_provider_descriptors, resolve_context_provider, ContextProviderId,
    RegisteredContextProviderDescriptor, SharedContextProviderCatalog,
};
use super::hooks::HookBus;
use super::info::PluginInfo;
use super::manifest::{ExtensionConfigEntry, ExtensionManifest};
use super::providers::{ProviderRegistry, RegisteredProvider, RegisteredProviderSummary};
use super::runtime::process::ProcessExtension;
use super::runtime::{ExtensionHandler, ExtensionHealth};
use serde_json::{Map, Value};

fn project_plugins_disabled() -> bool {
    std::env::var("SYNAPS_DISABLE_PROJECT_PLUGINS")
        .map(|value| {
            let normalized = value.trim().to_ascii_lowercase();
            matches!(normalized.as_str(), "1" | "true" | "yes" | "on")
        })
        .unwrap_or(false)
}

/// Whether a `plugins/` directory entry name denotes an INERT copy of a
/// plugin — a backup, disabled, or temporary directory — rather than an
/// installed plugin. Inert directories are excluded from discovery so a
/// stale copy can never register tools or execute alongside the real one
/// (user incident: `axel-memory-manager.backup.<TS>` dirs were discovered
/// as live plugins and their binaries spawned next to the real v0.3).
///
/// Name-based and conservative: only well-known suffix conventions (plus
/// the timestamped `.backup.<...>` infix) are excluded, so legitimate
/// dotted/hyphenated plugin names (`com.example.notes`, `backup-manager`)
/// stay discoverable. Exact plugin IDs of surviving plugins are untouched.
fn is_inert_plugin_dir_name(name: &str) -> bool {
    let lower = name.to_ascii_lowercase();
    // Timestamped backups: `<plugin>.backup.20260720101416`.
    if lower.contains(".backup.") {
        return true;
    }
    const INERT_SUFFIXES: &[&str] = &[
        ".backup",
        ".update-backup", // written by the plugin updater
        ".bak",
        ".disabled",
        ".old",
        ".orig",
        ".tmp",
        ".temp",
    ];
    INERT_SUFFIXES.iter().any(|s| lower.ends_with(s))
}

fn installed_plugin_setup_failure_in(
    state: &crate::skills::state::PluginsState,
    plugin_name: &str,
) -> Option<String> {
    let plugin = state.installed.iter().find(|p| p.name == plugin_name)?;
    match &plugin.setup_status {
        crate::skills::state::SetupStatus::Failed { message, .. } => Some(message.clone()),
        _ => None,
    }
}

fn sanitize_hint_fragment(input: &str) -> String {
    input
        .chars()
        .map(|ch| if ch.is_control() { '?' } else { ch })
        .collect::<String>()
}

/// Actionable discovery/load failure for an installed plugin extension.
#[derive(Debug, Clone, PartialEq, Eq)]
pub struct ExtensionLoadFailure {
    pub plugin: String,
    pub manifest_path: Option<PathBuf>,
    pub reason: String,
    pub hint: String,
}

impl ExtensionLoadFailure {
    fn new(
        plugin: impl Into<String>,
        manifest_path: Option<PathBuf>,
        reason: impl Into<String>,
        hint: impl Into<String>,
    ) -> Self {
        Self {
            plugin: plugin.into(),
            manifest_path,
            reason: reason.into(),
            hint: hint.into(),
        }
    }

    pub fn concise_message(&self) -> String {
        match &self.manifest_path {
            Some(path) => format!(
                "{} (manifest: {}; hint: {})",
                self.reason,
                path.display(),
                self.hint
            ),
            None => format!("{} (hint: {})", self.reason, self.hint),
        }
    }
}

/// Snapshot of a loaded extension's runtime status.
#[derive(Debug, Clone, PartialEq, Eq)]
pub struct ExtensionStatus {
    pub id: String,
    pub health: ExtensionHealth,
    pub restart_count: usize,
}

/// Compute the hint for an extension load failure.
///
/// Two cases:
/// 1. **Missing extension binary AND plugin declares
///    `provides.sidecar.setup`** — the plugin ships source only (the
///    binary is typically gitignored) and the setup script needs to
///    be run. The hint points the user at the exact command. This is
///    the common case for fresh marketplace installs of plugins that
///    build their extension binary from source.
/// 2. **Anything else** — the generic "run plugin validate" hint.
///
/// Pure function for unit-testability. Lives here (not in
/// `ExtensionLoadFailure`) because the sidecar/setup convention is a
/// plugin-layer concern.
pub fn compute_extension_load_hint(
    error: &str,
    plugin_dir: &std::path::Path,
    declared_setup: Option<&str>,
) -> String {
    let missing_binary =
        error.contains("No such file or directory") || error.contains("os error 2");
    match (missing_binary, declared_setup) {
        (true, Some(setup)) => format!(
            "Extension binary missing — this plugin ships source only. Run the setup script from the plugin directory, then reload. plugin_dir={}, setup={}",
            sanitize_hint_fragment(&plugin_dir.display().to_string()),
            sanitize_hint_fragment(setup),
        ),
        _ => "Run `plugin validate <plugin-dir>` and confirm the extension command is installed"
            .to_string(),
    }
}

/// Manages the lifecycle of all loaded extensions.
pub struct ExtensionManager {
    /// The shared hook bus.
    hook_bus: Arc<HookBus>,
    /// Optional shared tool registry for extension-provided tools.
    tools: Option<Arc<tokio::sync::RwLock<crate::ToolRegistry>>>,
    /// Provider metadata registered by loaded extensions. Routing is not wired yet.
    providers: ProviderRegistry,
    /// Running extensions keyed by ID.
    extensions: HashMap<String, Arc<dyn ExtensionHandler>>,
    /// Declared manifest config entries per loaded extension, kept so we can
    /// produce diagnostics without re-reading the manifest.
    manifest_configs: HashMap<String, Vec<ExtensionConfigEntry>>,
    /// Capability declarations per loaded extension. Each plugin may
    /// declare zero or more capabilities (kind is plugin-defined; core
    /// does not enumerate). Populated on load.
    capabilities: HashMap<String, Vec<crate::extensions::runtime::process::CapabilityDeclaration>>,
    /// Optional plugin-reported info from the `info.get` RPC.
    plugin_info: HashMap<String, PluginInfo>,
    /// Theme tokens declared in each loaded extension's manifest (P19.2),
    /// kept so the TUI can merge them into the theme under
    /// `ext.<plugin-id>.<token>` without re-reading manifests. Extensions
    /// that declare none simply have no entry.
    manifest_theme_tokens: HashMap<String, std::collections::BTreeMap<String, String>>,
    /// Task 20: when true (progressive disclosure), tool-only manifests
    /// with validated passive declarations are NOT spawned at load — their
    /// dormant deferred tools are cataloged instead and the launch record
    /// is retained for exact-activation lease acquisition.
    progressive_deferral: bool,
    /// Launch records for deferred tool-only extensions (manifest + cwd +
    /// resolved config), keyed by plugin id, behind a SHARED handle: the
    /// [`crate::extensions::lease::ExtensionRuntimeManager`] reads the
    /// CURRENT record at every lease acquisition (sync lock, map ops only,
    /// never held across I/O). Consumed by the lease lifecycle only.
    deferred_tool_only: SharedDeferredRecords,
    /// Lazily-created shared extension runtime lease manager (Task 20
    /// Commit B). One instance shared by this manager (unload revocation)
    /// and the `Runtime` (per-session capabilities + session-end scope).
    extension_runtime: std::sync::OnceLock<Arc<crate::extensions::lease::ExtensionRuntimeManager>>,
    /// Task A6: DORMANT context-provider descriptors declared by loaded
    /// extensions (`deferred.context_providers`, task A3) — the OWNED
    /// catalog behind [`Self::context_providers`]/[`Self::context_provider`],
    /// mirroring how `providers` backs the model-provider catalog reads.
    /// Pure metadata: cataloging a descriptor spawns nothing (discovery/
    /// status spawn zero processes); the only activation path is a future
    /// exact memory-context lease routed through the lease manager.
    context_providers: Vec<RegisteredContextProviderDescriptor>,
    /// Published snapshot of `context_providers`, shared with the lease
    /// manager exactly like `deferred_tool_only`: every catalog mutation
    /// republishes via [`Self::publish_context_providers`], and the
    /// `Runtime` memory-context enable path reads the CURRENT snapshot
    /// through `ExtensionRuntimeManager::resolve_context_provider` (sync
    /// lock, slice ops only, never held across I/O, never spawning).
    context_provider_catalog: SharedContextProviderCatalog,
    /// HOST-OWNED trusted project root, recorded ONCE per manager on first
    /// use (canonical `ProjectScope::discover` walk from the host's own
    /// working directory). Injected into resolved config for manifest
    /// entries declaring `host_context: "project_root"`; never sourced
    /// from env overrides, user config, or model input, and never
    /// forwarded as an environment variable.
    host_project_root: std::sync::OnceLock<Option<String>>,
}

/// Shared handle to the retained deferred launch records. Crate-private:
/// records hold RESOLVED (potentially secret) config.
pub(crate) type SharedDeferredRecords =
    Arc<std::sync::Mutex<HashMap<String, DeferredExtensionRecord>>>;

/// Retained launch expectation for a spawn-deferred tool-only extension.
///
/// INTERNAL AND OPAQUE (review fix A3): `config` holds RESOLVED values —
/// including `secret_env` material — so this record is crate-private,
/// exposes no public accessor, and deliberately implements neither
/// `Debug` nor `Serialize`. Public diagnostics may reveal only booleans/
/// counts (see [`ExtensionManager::is_deferred_tool_only`]); runtime
/// acquisition stays inside the manager/lease lifecycle API.
#[derive(Clone)]
pub(crate) struct DeferredExtensionRecord {
    pub(crate) manifest: ExtensionManifest,
    pub(crate) cwd: Option<std::path::PathBuf>,
    pub(crate) config: Value,
}

impl ExtensionManager {
    /// Create a new manager with a shared hook bus.
    pub fn new(hook_bus: Arc<HookBus>) -> Self {
        Self {
            hook_bus,
            tools: None,
            providers: ProviderRegistry::new(),
            extensions: HashMap::new(),
            manifest_configs: HashMap::new(),
            capabilities: HashMap::new(),
            plugin_info: HashMap::new(),
            manifest_theme_tokens: HashMap::new(),
            progressive_deferral: false,
            deferred_tool_only: Arc::new(std::sync::Mutex::new(HashMap::new())),
            extension_runtime: std::sync::OnceLock::new(),
            context_providers: Vec::new(),
            context_provider_catalog: Arc::new(std::sync::Mutex::new(Vec::new())),
            host_project_root: std::sync::OnceLock::new(),
        }
    }

    /// Create a new manager with shared hook bus and tool registry.
    pub fn new_with_tools(
        hook_bus: Arc<HookBus>,
        tools: Arc<tokio::sync::RwLock<crate::ToolRegistry>>,
    ) -> Self {
        Self {
            hook_bus,
            tools: Some(tools),
            providers: ProviderRegistry::new(),
            extensions: HashMap::new(),
            manifest_configs: HashMap::new(),
            capabilities: HashMap::new(),
            plugin_info: HashMap::new(),
            manifest_theme_tokens: HashMap::new(),
            progressive_deferral: false,
            deferred_tool_only: Arc::new(std::sync::Mutex::new(HashMap::new())),
            extension_runtime: std::sync::OnceLock::new(),
            context_providers: Vec::new(),
            context_provider_catalog: Arc::new(std::sync::Mutex::new(Vec::new())),
            host_project_root: std::sync::OnceLock::new(),
        }
    }

    /// Load and start an extension from its manifest.
    pub async fn load(&mut self, id: &str, manifest: &ExtensionManifest) -> Result<(), String> {
        self.load_with_cwd(id, manifest, None).await
    }

    /// Load and start an extension from its manifest with a process cwd.
    pub async fn load_with_cwd(
        &mut self,
        id: &str,
        manifest: &ExtensionManifest,
        cwd: Option<std::path::PathBuf>,
    ) -> Result<(), String> {
        let config = Self::resolve_config(id, &manifest.config, self.host_project_root())?;
        self.load_with_cwd_and_config(id, manifest, cwd, config)
            .await
    }

    /// Enable Task 20 progressive spawn deferral (set at engine boot from
    /// the progressive-disclosure flag; default false keeps every legacy
    /// eager path byte-for-byte).
    pub fn set_progressive_deferral(&mut self, enabled: bool) {
        self.progressive_deferral = enabled;
    }

    /// The HOST-OWNED trusted project root, discovered ONCE per manager
    /// from the host's own working directory via the canonical
    /// `agent_core` project-scope walk (nearest `.git`/`.synaps-project`
    /// marker, canonicalized). Recorded here so every extension load in
    /// this manager's lifetime sees one stable value. `None` when the
    /// working directory is unavailable or cannot be canonicalized.
    pub fn host_project_root(&self) -> Option<&str> {
        self.host_project_root
            .get_or_init(|| {
                let cwd = std::env::current_dir().ok()?;
                agent_core::memory::store::ProjectScope::discover(&cwd)
                    .ok()
                    .map(|scope| scope.root().display().to_string())
            })
            .as_deref()
    }

    /// Whether one plugin id is currently deferred (any Task 20 class:
    /// tool-only, provider, hook/lifecycle, UI/sidecar, mixed — zero
    /// spawn). Boolean-only diagnostic: the underlying launch record
    /// (which holds resolved — potentially secret — config) is never
    /// exposed.
    pub fn is_deferred(&self, id: &str) -> bool {
        self.deferred_records()
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner)
            .contains_key(id)
    }

    /// Compatibility alias for [`Self::is_deferred`] (Commit B name).
    pub fn is_deferred_tool_only(&self, id: &str) -> bool {
        self.is_deferred(id)
    }

    /// Number of retained deferred tool-only launch records (count-only
    /// diagnostic; never the records).
    pub fn deferred_tool_only_count(&self) -> usize {
        self.deferred_records()
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner)
            .len()
    }

    fn deferred_records(&self) -> &SharedDeferredRecords {
        &self.deferred_tool_only
    }

    /// Republish the owned context-provider catalog into the shared
    /// snapshot the lease manager reads (task A6). Called after EVERY
    /// catalog mutation so `resolve_context_provider` always observes the
    /// current loaded set — a removed extension's providers can never be
    /// resolved from a stale snapshot.
    fn publish_context_providers(&self) {
        let mut published = self
            .context_provider_catalog
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner);
        *published = self.context_providers.clone();
    }

    /// Catalog one loaded extension's DORMANT context-provider
    /// declarations (task A6) via the task-A3 validated builder. Pure
    /// metadata registration — spawns nothing; invalid or ungated
    /// declaration sets yield no descriptors (fail closed inside
    /// [`dormant_context_provider_descriptors`]).
    fn catalog_context_providers(&mut self, id: &str, manifest: &ExtensionManifest) {
        let descriptors = dormant_context_provider_descriptors(id, manifest);
        if descriptors.is_empty() {
            return;
        }
        self.context_providers.extend(descriptors);
        self.publish_context_providers();
    }

    /// Drop every cataloged context-provider descriptor owned by one
    /// extension (unload/teardown path, task A6) and republish.
    fn uncatalog_context_providers(&mut self, id: &str) {
        let before = self.context_providers.len();
        self.context_providers
            .retain(|descriptor| descriptor.extension_id() != id);
        if self.context_providers.len() != before {
            self.publish_context_providers();
        }
    }

    /// The SHARED extension runtime lease manager (Task 20 Commit B),
    /// created on first use against this manager's retained launch
    /// records. The engine installs the same instance on the `Runtime`
    /// (session capabilities + durable session-end scope); this manager
    /// uses it for unload/reload lease revocation. Runtime acquisition —
    /// spawn, re-validation, exact declaration matching — happens ONLY
    /// inside that manager's lease lifecycle.
    pub fn extension_runtime(&self) -> Arc<crate::extensions::lease::ExtensionRuntimeManager> {
        self.extension_runtime_with_idle(crate::extensions::lease::DEFAULT_IDLE_MAX)
    }

    /// As [`Self::extension_runtime`], but the FIRST caller fixes the idle
    /// reap bound (tests use short bounds; production uses the default).
    pub fn extension_runtime_with_idle(
        &self,
        idle_max: std::time::Duration,
    ) -> Arc<crate::extensions::lease::ExtensionRuntimeManager> {
        Arc::clone(self.extension_runtime.get_or_init(|| {
            Arc::new(crate::extensions::lease::ExtensionRuntimeManager::new(
                Arc::clone(&self.deferred_tool_only),
                Arc::clone(&self.context_provider_catalog),
                idle_max,
            ))
        }))
    }

    /// Test-only internal introspection: clone one retained launch record
    /// (the lease manager reads the shared records handle directly).
    #[cfg(test)]
    pub(crate) fn deferred_launch_record(&self, id: &str) -> Option<DeferredExtensionRecord> {
        self.deferred_records()
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner)
            .get(id)
            .cloned()
    }

    /// Test seam: tamper the retained manifest permissions of one deferred
    /// record IN PLACE (simulates internal-state drift the public API can
    /// never produce, to prove acquisition re-validation fails closed).
    /// Reveals nothing.
    #[doc(hidden)]
    pub fn tamper_deferred_manifest_permissions_for_tests(
        &self,
        id: &str,
        permissions: Vec<String>,
    ) -> bool {
        let mut map = self
            .deferred_tool_only
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner);
        match map.get_mut(id) {
            Some(record) => {
                record.manifest.permissions = permissions;
                true
            }
            None => false,
        }
    }

    /// Test seam: tamper the retained RESOLVED config of one deferred
    /// record in place (simulates launch-record/config drift under a live
    /// lease). Reveals nothing.
    #[doc(hidden)]
    pub fn tamper_deferred_config_for_tests(&self, id: &str, config: Value) -> bool {
        let mut map = self
            .deferred_tool_only
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner);
        match map.get_mut(id) {
            Some(record) => {
                record.config = config;
                true
            }
            None => false,
        }
    }

    async fn load_with_cwd_and_config(
        &mut self,
        id: &str,
        manifest: &ExtensionManifest,
        cwd: Option<std::path::PathBuf>,
        config: Value,
    ) -> Result<(), String> {
        // Don't load duplicates (live OR deferred).
        if self.extensions.contains_key(id) || self.is_deferred_tool_only(id) {
            return Err(format!("Extension '{}' is already loaded", id));
        }

        // Validate permissions and hook subscriptions before spawning the
        // extension process. This keeps malformed manifests from leaking child
        // processes when a later subscription step fails.
        let validated = manifest.validate(id)?;
        let permissions = validated.permissions;
        let subscriptions = validated.subscriptions;

        // Task 20: under progressive disclosure, EVERY manifest with a
        // validated `deferred` block defers its spawn entirely (spec §7.5):
        //  - tool-only: dormant descriptor tools cataloged; exact
        //    activation acquires the lease;
        //  - provider: declared provider METADATA registered with a lazy
        //    handler; first selected complete/stream acquires;
        //  - hook/lifecycle: permission-validated subscriptions registered
        //    with a lazy handler; the first AUTHORIZED matching event
        //    acquires;
        //  - UI/sidecar: record retained only; explicit user APIs acquire;
        //  - mixed: all of the above against ONE shared per-plugin lease
        //    (earliest legitimate capability starts it; tool SEARCH alone
        //    never does).
        // Legacy manifests (no `deferred` block) and flag-off stay on the
        // documented eager path byte-for-byte.
        let class = crate::extensions::lifecycle::classify(manifest);
        if self.progressive_deferral
            && class != crate::extensions::lifecycle::ExtensionClass::LegacyEager
        {
            let declared_tool_names: Vec<String> = manifest
                .deferred
                .as_ref()
                .map(|d| d.tools.iter().map(|t| format!("{id}:{}", t.name)).collect())
                .unwrap_or_default();
            let dormant = crate::extensions::lifecycle::dormant_extension_tools(id, manifest);
            let tool_count = dormant.len();
            if !dormant.is_empty() {
                let Some(tools) = &self.tools else {
                    return Err(format!(
                        "Extension '{}' declares deferred tools but no tool registry is available",
                        id
                    ));
                };
                tools
                    .write()
                    .await
                    .try_register_batch(dormant)
                    .map_err(|e| {
                        format!("Extension '{}' deferred tool catalog rejected: {}", id, e)
                    })?;
            }
            // Retain the launch record BEFORE registering lazy capability
            // surfaces so a capability firing immediately can acquire.
            self.deferred_tool_only
                .lock()
                .unwrap_or_else(std::sync::PoisonError::into_inner)
                .insert(
                    id.to_string(),
                    DeferredExtensionRecord {
                        manifest: manifest.clone(),
                        cwd,
                        config: config.clone(),
                    },
                );
            // Bounded unwind shared by the failure paths below: remove the
            // record, the dormant batch, and any provider metadata.
            macro_rules! unwind_deferred {
                () => {{
                    self.deferred_tool_only
                        .lock()
                        .unwrap_or_else(std::sync::PoisonError::into_inner)
                        .remove(id);
                    self.providers.unregister_plugin(id);
                    if let Some(tools) = &self.tools {
                        let _ = tools.write().await.try_disable(&declared_tool_names);
                    }
                    self.hook_bus.unsubscribe_all(id).await;
                }};
            }
            let lazy: Option<Arc<dyn ExtensionHandler>> = {
                // Created only when a lazy capability surface exists, so a
                // tool-only load never pre-initializes the runtime manager
                // (its idle policy stays first-caller-configurable).
                let needs_lazy = manifest
                    .deferred
                    .as_ref()
                    .map(|d| !d.providers.is_empty())
                    .unwrap_or(false)
                    || !manifest.hooks.is_empty();
                needs_lazy.then(|| {
                    Arc::new(crate::extensions::lease::LazyExtensionHandler::new(
                        id,
                        self.extension_runtime(),
                    )) as Arc<dyn ExtensionHandler>
                })
            };
            // Declared provider METADATA (validated shape) registered with
            // the lazy handler; routing re-validates the live runtime's
            // declarations against the manifest before any call.
            let declared_providers: Vec<_> = manifest
                .deferred
                .as_ref()
                .map(|d| d.providers.clone())
                .unwrap_or_default();
            let mut registered_provider_ids = Vec::new();
            for declared in &declared_providers {
                let spec = declared.to_registered_spec();
                if let Err(error) = Self::validate_provider_config_requirements(id, &spec, &config)
                {
                    unwind_deferred!();
                    return Err(error);
                }
                let handler = lazy.clone().expect("providers imply a lazy handler");
                match self
                    .providers
                    .register_with_handler(id, spec, Some(handler))
                {
                    Ok(runtime_id) => registered_provider_ids.push(runtime_id),
                    Err(error) => {
                        unwind_deferred!();
                        return Err(error);
                    }
                }
            }
            // Tool-use audit parity with the eager path.
            for runtime_id in &registered_provider_ids {
                if let Some(provider) = self.providers.get(runtime_id) {
                    let tool_use = provider.spec.models.iter().any(|m| {
                        m.capabilities
                            .get("tool_use")
                            .and_then(|v| v.as_bool())
                            .unwrap_or(false)
                    });
                    if tool_use {
                        tracing::warn!(
                            "Provider '{}' is tool-use capable: it can request Synaps tools through provider mediation. Use `/extensions trust disable {}` to block routing.",
                            runtime_id,
                            runtime_id,
                        );
                    }
                }
            }
            // Permission-validated hook subscriptions with the lazy
            // handler: the first AUTHORIZED matching event acquires.
            for (kind, tool_filter, matcher) in subscriptions {
                let handler = lazy.clone().expect("subscriptions imply a lazy handler");
                if let Err(error) = self
                    .hook_bus
                    .subscribe(kind, handler, tool_filter, matcher, permissions.clone())
                    .await
                {
                    unwind_deferred!();
                    return Err(error);
                }
            }
            self.manifest_configs
                .insert(id.to_string(), manifest.config.clone());
            if !manifest.theme_tokens.is_empty() {
                self.manifest_theme_tokens
                    .insert(id.to_string(), manifest.theme_tokens.clone());
            }
            // Task A6: catalog DORMANT context-provider declarations LAST —
            // every fallible registration step above has succeeded, so the
            // unwind path never needs to reason about a half-published
            // catalog. Metadata only; nothing spawns.
            self.catalog_context_providers(id, manifest);
            tracing::info!(extension = %id, class = ?class, tools = tool_count,
                providers = registered_provider_ids.len(), hooks = manifest.hooks.len(),
                context_providers = self.context_providers.iter().filter(|d| d.extension_id() == id).count(),
                "Extension deferred: passive surfaces registered, no process started");
            return Ok(());
        }

        // Spawn the extension process only after the manifest is known-good.
        let process =
            ProcessExtension::spawn_with_cwd(id, &manifest.command, &manifest.args, cwd.clone())
                .await?;
        // Publish permissions to the inbound-request dispatcher so memory.*
        // calls during initialize can be authorized correctly.
        process.set_permissions(permissions.clone()).await;
        let capabilities = match process.initialize(cwd.clone(), config.clone()).await {
            Ok(capabilities) => capabilities,
            Err(error) => {
                process.shutdown().await;
                return Err(error);
            }
        };
        let registered_tools = capabilities.tools;
        let registered_providers = capabilities.providers;
        let capability_declarations = capabilities.capabilities;
        let should_probe_info = !registered_tools.is_empty()
            || !registered_providers.is_empty()
            || !capability_declarations.is_empty();
        let handler: Arc<dyn ExtensionHandler> = Arc::new(process);
        if !registered_tools.is_empty()
            && !permissions.has(crate::extensions::permissions::Permission::ToolsRegister)
        {
            handler.shutdown().await;
            return Err(format!(
                "Extension '{}' registered tools but lacks permission 'tools.register'",
                id
            ));
        }
        if !registered_providers.is_empty()
            && !permissions.has(crate::extensions::permissions::Permission::ProvidersRegister)
        {
            handler.shutdown().await;
            return Err(format!(
                "Extension '{}' registered providers but lacks permission 'providers.register'",
                id
            ));
        }
        for decl in &capability_declarations {
            if let Err(err) =
                crate::extensions::runtime::process::validate_capability(decl, &permissions)
            {
                handler.shutdown().await;
                return Err(format!(
                    "Extension '{}' capability '{}' invalid: {}",
                    id, decl.kind, err
                ));
            }
        }
        if !registered_providers.is_empty() {
            let mut registered_ids = Vec::new();
            for provider in registered_providers {
                if let Err(error) =
                    Self::validate_provider_config_requirements(id, &provider, &config)
                {
                    self.providers.unregister_plugin(id);
                    handler.shutdown().await;
                    return Err(error);
                }
                match self
                    .providers
                    .register_with_handler(id, provider, Some(handler.clone()))
                {
                    Ok(runtime_id) => registered_ids.push(runtime_id),
                    Err(error) => {
                        self.providers.unregister_plugin(id);
                        handler.shutdown().await;
                        return Err(error);
                    }
                }
            }
            tracing::info!(extension = %id, providers = ?registered_ids, "Extension provider metadata registered");
            // Warn for tool-use-capable providers so authors and users can audit them.
            for runtime_id in &registered_ids {
                if let Some(provider) = self.providers.get(runtime_id) {
                    let tool_use = provider.spec.models.iter().any(|m| {
                        m.capabilities
                            .get("tool_use")
                            .and_then(|v| v.as_bool())
                            .unwrap_or(false)
                    });
                    if tool_use {
                        tracing::warn!(
                            "Provider '{}' is tool-use capable: it can request Synaps tools through provider mediation. Use `/extensions trust disable {}` to block routing.",
                            runtime_id,
                            runtime_id,
                        );
                    }
                }
            }
        }
        if !registered_tools.is_empty() {
            let Some(tools) = &self.tools else {
                handler.shutdown().await;
                return Err(format!(
                    "Extension '{}' registered tools but no tool registry is available",
                    id
                ));
            };
            let mut registry = tools.write().await;
            for spec in registered_tools {
                let tool_name = spec.name.clone();
                if let Err(e) = registry.try_register(Arc::new(crate::tools::ExtensionTool::new(
                    id,
                    spec,
                    handler.clone(),
                ))) {
                    tracing::warn!(
                        extension = %id,
                        tool = %tool_name,
                        error = %e,
                        "Refusing to expose extension tool the capability catalog could not record"
                    );
                }
            }
        }

        // Do not probe optional info.get for legacy hook-only extensions. The
        // best-effort call can race with simple fixtures that exit after
        // shutdown/EOF and is only needed for richer extension-capability
        // surfaces (providers/tools/plugin-defined capabilities).
        let info = if should_probe_info {
            match handler.get_info().await {
                Ok(info) => Some(info),
                Err(error) => {
                    if error.contains("method not found") || error.contains("unknown method") {
                        tracing::debug!(
                            extension = %id,
                            error = %error,
                            "Extension did not provide optional info.get metadata",
                        );
                        None
                    } else {
                        tracing::warn!(
                            extension = %id,
                            error = %error,
                            "Ignoring invalid optional info.get metadata",
                        );
                        None
                    }
                }
            }
        } else {
            None
        };

        // Register hook subscriptions
        for (kind, tool_filter, matcher) in subscriptions {
            self.hook_bus
                .subscribe(
                    kind,
                    handler.clone(),
                    tool_filter,
                    matcher,
                    permissions.clone(),
                )
                .await?;
        }

        self.extensions.insert(id.to_string(), handler);
        self.manifest_configs
            .insert(id.to_string(), manifest.config.clone());
        if !manifest.theme_tokens.is_empty() {
            self.manifest_theme_tokens
                .insert(id.to_string(), manifest.theme_tokens.clone());
        }
        if !capability_declarations.is_empty() {
            self.capabilities
                .insert(id.to_string(), capability_declarations);
        }
        if let Some(info) = info {
            self.plugin_info.insert(id.to_string(), info);
        }
        // Task A6: catalog validated DORMANT `deferred.context_providers`
        // declarations for eagerly-loaded extensions too, so the context-
        // provider catalog reflects EVERY loaded extension regardless of
        // lifecycle class. Metadata only — the process above was spawned by
        // the legacy eager path, never by this catalog write.
        self.catalog_context_providers(id, manifest);
        tracing::info!(extension = %id, hooks = manifest.hooks.len(), "Extension loaded");
        Ok(())
    }

    fn validate_provider_config_requirements(
        id: &str,
        provider: &crate::extensions::runtime::process::RegisteredProviderSpec,
        config: &Value,
    ) -> Result<(), String> {
        let Some(required) = provider
            .config_schema
            .as_ref()
            .and_then(|schema| schema.get("required"))
            .and_then(Value::as_array)
        else {
            return Ok(());
        };
        for key in required {
            let Some(key) = key.as_str() else {
                return Err(format!(
                    "Extension '{}' provider '{}' config_schema.required must contain only strings",
                    id, provider.id,
                ));
            };
            let present = config
                .as_object()
                .map(|map| map.contains_key(key))
                .unwrap_or(false);
            if !present {
                return Err(format!(
                    "Extension '{}' provider '{}' missing required provider config '{}'",
                    id, provider.id, key,
                ));
            }
        }
        Ok(())
    }

    /// Env vars an extension may never pull in through `secret_env`.
    ///
    /// Security (#207 MED-2): `secret_env` names an arbitrary process
    /// environment variable, and [`Self::resolve_config`] reads it verbatim
    /// into the config handed to the extension at `initialize`. Nothing
    /// constrained that name, so a manifest could declare the HOST's own
    /// provider credential — `secret_env: "ANTHROPIC_API_KEY"` — and a plugin
    /// installed for something unrelated would be handed the user's API key.
    ///
    /// Default-deny the credentials the host itself holds, plus the whole
    /// `SYNAPS_*` host namespace except this extension's own scoped override.
    /// Third-party secrets an extension legitimately needs (`GITHUB_TOKEN`,
    /// and friends) are unaffected.
    fn denied_secret_env(var: &str, id: &str) -> Option<&'static str> {
        const HOST_CREDENTIALS: &[&str] = &[
            "ANTHROPIC_API_KEY",
            "OPENAI_API_KEY",
            "GEMINI_API_KEY",
            "GOOGLE_API_KEY",
            "XAI_API_KEY",
            "GROQ_API_KEY",
            "OPENROUTER_API_KEY",
            "DEEPSEEK_API_KEY",
            "MISTRAL_API_KEY",
            "AWS_SECRET_ACCESS_KEY",
            "AWS_SESSION_TOKEN",
        ];
        let up = var.trim().to_ascii_uppercase();
        if up.is_empty() {
            return Some("secret_env must not be empty");
        }
        if HOST_CREDENTIALS.contains(&up.as_str()) {
            return Some(
                "that variable holds a host credential; extensions are never handed the host's own provider keys",
            );
        }
        let own_prefix = format!(
            "SYNAPS_EXTENSION_{}_",
            id.replace('-', "_").to_ascii_uppercase()
        );
        if up.starts_with("SYNAPS_") && !up.starts_with(&own_prefix) {
            return Some(
                "SYNAPS_* is the host namespace; declare your own SYNAPS_EXTENSION_<ID>_<KEY> variable instead",
            );
        }
        None
    }

    fn resolve_config(
        id: &str,
        entries: &[ExtensionConfigEntry],
        host_project_root: Option<&str>,
    ) -> Result<Value, String> {
        let mut out = Map::new();
        for entry in entries {
            let key = entry.key.trim();
            if key.is_empty() {
                return Err(format!("Extension '{}' declares config with empty key", id));
            }
            if key.contains('.') || key.contains('/') || key.contains(' ') {
                return Err(format!(
                    "Extension '{}' config key '{}' must not contain dots, slashes, or spaces",
                    id, key,
                ));
            }
            // RESERVED host-context entries: the value comes EXCLUSIVELY
            // from trusted host state recorded once by this manager. Env
            // overrides, secret_env, plugin config, and legacy config keys
            // are all deliberately skipped — user or model input can never
            // supply or widen a host-context value.
            if let Some(host_key) = entry.host_context {
                if entry.secret_env.is_some() {
                    return Err(format!(
                        "Extension '{}' config key '{}' declares both host_context and secret_env; host-context values never carry secrets",
                        id, key,
                    ));
                }
                match host_key {
                    crate::extensions::manifest::HostContextKey::ProjectRoot => {
                        if let Some(root) = host_project_root {
                            out.insert(key.to_string(), Value::String(root.to_string()));
                        } else if let Some(default) = &entry.default {
                            out.insert(key.to_string(), default.clone());
                        } else if entry.required {
                            return Err(format!(
                                "Extension '{}' requires host context 'project_root' but the host could not resolve a trusted project root",
                                id,
                            ));
                        }
                    }
                }
                continue;
            }
            let config_key = format!("extension.{}.{}", id, key);
            // Security (#207 MED-2): validate the declared env var name BEFORE
            // any resolution path can read it. Fails at load, loudly, rather
            // than silently handing a host credential to the extension.
            if let Some(secret_env) = &entry.secret_env {
                if let Some(reason) = Self::denied_secret_env(secret_env, id) {
                    return Err(format!(
                        "Extension '{}' config key '{}' declares secret_env '{}': {}",
                        id, key, secret_env, reason
                    ));
                }
            }
            if let Ok(value) = std::env::var(format!(
                "SYNAPS_EXTENSION_{}_{}",
                id.replace('-', "_").to_ascii_uppercase(),
                key.replace('-', "_").to_ascii_uppercase()
            )) {
                out.insert(key.to_string(), Value::String(value));
                continue;
            }
            if let Some(secret_env) = &entry.secret_env {
                if let Ok(value) = std::env::var(secret_env) {
                    out.insert(key.to_string(), Value::String(value));
                    continue;
                }
            }
            if let Some(value) = crate::extensions::config_store::read_plugin_config(id, key) {
                out.insert(key.to_string(), Value::String(value));
                continue;
            }
            if let Some(value) = crate::config::read_config_value(&config_key) {
                out.insert(key.to_string(), Value::String(value));
                continue;
            }
            if let Some(default) = &entry.default {
                out.insert(key.to_string(), default.clone());
                continue;
            }
            if entry.required {
                let hint = if let Some(secret_env) = &entry.secret_env {
                    format!(
                        "set environment variable '{}' or config key '{}'",
                        secret_env, config_key
                    )
                } else {
                    format!("set config key '{}'", config_key)
                };
                return Err(format!(
                    "Extension '{}' missing required config '{}': {}",
                    id, key, hint
                ));
            }
        }
        Ok(Value::Object(out))
    }

    /// Test-only seeder: synthetically insert capability declarations
    /// for an extension id. Used to exercise capability snapshot
    /// rendering without spinning up a real plugin process.
    #[cfg(test)]
    pub(crate) fn test_seed_capabilities(
        &mut self,
        id: &str,
        decls: Vec<crate::extensions::runtime::process::CapabilityDeclaration>,
    ) {
        self.capabilities.insert(id.to_string(), decls);
    }

    /// Unload an extension — unsubscribe hooks and shut down the process.
    ///
    /// A DEFERRED tool-only extension (Task 20) has no live handler:
    /// unloading it removes the retained launch record, deregisters its
    /// dormant catalog batch (advancing the catalog generation so stale
    /// grants cannot survive), and terminates every session's runtime
    /// lease for the plugin.
    pub async fn unload(&mut self, id: &str) -> Result<(), String> {
        let deferred = self
            .deferred_tool_only
            .lock()
            .unwrap_or_else(std::sync::PoisonError::into_inner)
            .remove(id);
        if let Some(record) = deferred {
            // Record removed FIRST: any concurrent lease acquisition now
            // fails closed (NotDeferred) regardless of the steps below.
            // Context providers go with it (task A6): once unloading has
            // begun, a memory-context enable can no longer resolve this
            // extension's declared providers.
            self.uncatalog_context_providers(id);
            if let Some(runtime) = self.extension_runtime.get() {
                let revoked = runtime.revoke_plugin_all_sessions(id);
                if revoked > 0 {
                    tracing::info!(extension = %id, leases = revoked,
                        "Terminated live runtime leases of unloaded deferred extension");
                }
            }
            self.hook_bus.unsubscribe_all(id).await;
            self.providers.unregister_plugin(id);
            if let Some(tools) = &self.tools {
                let names: Vec<String> = record
                    .manifest
                    .deferred
                    .as_ref()
                    .map(|d| {
                        d.tools
                            .iter()
                            .map(|t| format!("{}:{}", id, t.name))
                            .collect()
                    })
                    .unwrap_or_default();
                if let Err(error) = tools.write().await.try_disable(&names) {
                    // Fail-closed either way: with the record gone, dormant
                    // descriptors can never execute; surface the partial
                    // cleanup instead of hiding it.
                    return Err(format!(
                        "Extension '{}' unloaded (record removed, leases revoked) but its dormant catalog batch could not be deregistered: {}",
                        id, error
                    ));
                }
            }
            self.manifest_configs.remove(id);
            self.manifest_theme_tokens.remove(id);
            tracing::info!(extension = %id, "Deferred extension unloaded");
            return Ok(());
        }
        let handler = self
            .extensions
            .remove(id)
            .ok_or_else(|| format!("Extension '{}' not found", id))?;
        // Task A6: cataloged context providers of a live extension are
        // removed with it.
        self.uncatalog_context_providers(id);

        self.hook_bus.unsubscribe_all(id).await;
        self.providers.unregister_plugin(id);
        self.manifest_configs.remove(id);
        self.capabilities.remove(id);
        self.plugin_info.remove(id);
        self.manifest_theme_tokens.remove(id);
        handler.shutdown().await;

        tracing::info!(extension = %id, "Extension unloaded");
        Ok(())
    }

    /// Reload one extension by unloading any existing instance first, then loading
    /// the supplied manifest. If the new load fails, the previous instance remains
    /// unloaded so duplicate handlers cannot survive a broken reload.
    pub async fn reload(
        &mut self,
        id: &str,
        manifest: &ExtensionManifest,
        cwd: Option<std::path::PathBuf>,
    ) -> Result<(), String> {
        if self.extensions.contains_key(id) || self.is_deferred_tool_only(id) {
            self.unload(id).await?;
        }
        self.load_with_cwd(id, manifest, cwd).await
    }

    /// Shut down all extensions gracefully.
    pub async fn shutdown_all(&mut self) {
        let ids: Vec<String> = self.extensions.keys().cloned().collect();
        for id in ids {
            let _ = self.unload(&id).await;
        }
        // Deferred extensions (Task 20): unload each retained record so
        // dormant catalog batches and any live runtime leases go with it.
        let deferred_ids: Vec<String> = {
            let map = self
                .deferred_tool_only
                .lock()
                .unwrap_or_else(std::sync::PoisonError::into_inner);
            map.keys().cloned().collect()
        };
        for id in deferred_ids {
            let _ = self.unload(&id).await;
        }
        if let Some(runtime) = self.extension_runtime.get() {
            runtime.terminate_all();
        }
    }

    /// Start shutting down all extensions in the background.
    ///
    /// This is intended for process exit: the UI should not hang waiting for
    /// extension child processes to acknowledge shutdown. Dropping the join handle
    /// lets Tokio abort remaining work when the runtime exits.
    pub fn shutdown_all_detached(
        manager: Arc<tokio::sync::RwLock<Self>>,
    ) -> tokio::task::JoinHandle<()> {
        tokio::spawn(async move {
            manager.write().await.shutdown_all().await;
        })
    }

    /// List running extension IDs.
    pub fn list(&self) -> Vec<&str> {
        self.extensions.keys().map(|s| s.as_str()).collect()
    }

    /// Return Arc references to all running extension handlers, sorted by ID.
    /// Intended for background notification watchers that need to hold onto
    /// handlers beyond the lifetime of a manager lock.
    pub fn handlers(&self) -> Vec<(String, Arc<dyn super::runtime::ExtensionHandler>)> {
        let mut out: Vec<_> = self
            .extensions
            .iter()
            .map(|(id, h)| (id.clone(), Arc::clone(h)))
            .collect();
        out.sort_by(|a, b| a.0.cmp(&b.0));
        out
    }

    /// Number of running extensions.
    pub fn count(&self) -> usize {
        self.extensions.len()
    }

    /// Return health snapshots for all loaded extensions, sorted by ID.
    pub async fn statuses(&self) -> Vec<ExtensionStatus> {
        let mut handlers: Vec<(String, Arc<dyn ExtensionHandler>)> = self
            .extensions
            .iter()
            .map(|(id, handler)| (id.clone(), handler.clone()))
            .collect();
        handlers.sort_by(|a, b| a.0.cmp(&b.0));

        let mut statuses = Vec::with_capacity(handlers.len());
        for (id, handler) in handlers {
            statuses.push(ExtensionStatus {
                id,
                health: handler.health().await,
                restart_count: handler.restart_count().await,
            });
        }
        statuses
    }

    /// Return registered provider metadata sorted by runtime id.
    pub fn providers(&self) -> Vec<&RegisteredProvider> {
        self.providers.list()
    }

    /// Return registered provider metadata by runtime id.
    pub fn provider(&self, runtime_id: &str) -> Option<&RegisteredProvider> {
        self.providers.get(runtime_id)
    }

    /// Return DORMANT context-provider descriptors declared by loaded
    /// extensions, sorted by runtime address (task A6; mirrors
    /// [`Self::providers`] for the UNRELATED model-provider catalog).
    /// Pure catalog read — discovery/status spawn zero processes, and
    /// holding a descriptor grants nothing.
    pub fn context_providers(&self) -> Vec<&RegisteredContextProviderDescriptor> {
        let mut out: Vec<&RegisteredContextProviderDescriptor> =
            self.context_providers.iter().collect();
        out.sort_by_key(|descriptor| descriptor.runtime_address());
        out
    }

    /// Exact context-provider lookup by declared id (task A6; mirrors
    /// [`Self::provider`]). `Some` iff EXACTLY ONE loaded extension
    /// declares `provider_id` — zero matches and overlapping declarations
    /// (the same id declared by multiple installed extensions) both
    /// resolve to `None`, fail closed, no partial matches. Pure catalog
    /// read — never spawns.
    pub fn context_provider(
        &self,
        provider_id: &ContextProviderId,
    ) -> Option<&RegisteredContextProviderDescriptor> {
        resolve_context_provider(&self.context_providers, Some(provider_id)).ok()
    }

    /// Return optional cached plugin info reported by `info.get`.
    pub fn plugin_info(&self, id: &str) -> Option<&PluginInfo> {
        self.plugin_info.get(id)
    }

    /// Manifest-declared theme tokens for every loaded extension, sorted by
    /// extension ID (P19.2). Each entry is `(plugin-id, token -> hex color)`;
    /// the TUI namespaces these as `ext.<plugin-id>.<token>` when merging
    /// into the active theme. Extensions without `theme_tokens` don't appear.
    pub fn theme_tokens(&self) -> Vec<(String, std::collections::BTreeMap<String, String>)> {
        let mut out: Vec<_> = self
            .manifest_theme_tokens
            .iter()
            .map(|(id, tokens)| (id.clone(), tokens.clone()))
            .collect();
        out.sort_by(|a, b| a.0.cmp(&b.0));
        out
    }

    /// Ask a plugin for its sidecar spawn arguments. Best-effort —
    /// plugins that don't host a sidecar (or pre-Phase-7 plugins that
    /// haven't implemented the RPC yet) return `Err`. Callers are
    /// expected to treat that as "no overrides; use manifest defaults".
    /// Handler for explicit USER-triggered plugin APIs (sidecar spawn,
    /// interactive commands, settings editors). Live extensions use their
    /// running handler; a DEFERRED extension (Task 20) gets a lazy handler
    /// whose first use acquires the shared per-plugin runtime lease — a
    /// user action is a legitimate activation trigger. Discovery/search/
    /// diagnostics never call this.
    fn user_action_handler(
        &self,
        id: &str,
    ) -> Result<Arc<dyn super::runtime::ExtensionHandler>, String> {
        if let Some(handler) = self.extensions.get(id) {
            return Ok(handler.clone());
        }
        if self.is_deferred(id) {
            return Ok(Arc::new(
                crate::extensions::lease::LazyExtensionHandler::new(id, self.extension_runtime()),
            ));
        }
        Err(format!("unknown extension '{}'", id))
    }

    pub async fn sidecar_spawn_args(
        &self,
        id: &str,
    ) -> Result<crate::sidecar::spawn::SidecarSpawnArgs, String> {
        let handler = self.user_action_handler(id)?;
        handler.sidecar_spawn_args().await
    }

    /// Invoke an interactive plugin command on extension `id`. Streams
    /// `command.output` (matching `request_id`) and `task.*` notifications
    /// to `sink`. Returns the final JSON-RPC response value.
    ///
    /// CP-11 fix-3: `sink` is the bounded, metered producer half of
    /// [`crate::extensions::invoke_output::invoke_event_channel`]; the
    /// caller MUST drive the matching collector concurrently with this
    /// call (an awaited bounded send with no consumer would stall the
    /// invocation loop until the 120 s timeout). Prefer
    /// [`Self::invoke_command_collected`], which wires that correctly.
    pub async fn invoke_command(
        &self,
        id: &str,
        command: &str,
        args: Vec<String>,
        request_id: &str,
        sink: crate::extensions::invoke_output::InvokeEventSink,
    ) -> Result<serde_json::Value, String> {
        let handler = self.user_action_handler(id)?;
        handler
            .invoke_command(command, args, request_id, sink)
            .await
    }

    /// Invoke an interactive plugin command and collect its budgeted
    /// output. This is THE production entry point for user-facing
    /// surfaces: it joins the invocation with an eagerly concurrent
    /// collector so hostile `command.output` floods are paced and capped
    /// at production time (never parked unbounded), then returns the
    /// final response alongside the retained events and exact
    /// produced/consumed/retained/truncated/dropped accounting.
    ///
    /// Dropping the returned future cancels the invocation, drops the
    /// sink, and releases the collector plus any blocked producers — no
    /// detached tasks are spawned.
    pub async fn invoke_command_collected(
        &self,
        id: &str,
        command: &str,
        args: Vec<String>,
        request_id: &str,
    ) -> (
        Result<serde_json::Value, String>,
        crate::extensions::invoke_output::InvokeOutputReport,
    ) {
        let (sink, collector) = crate::extensions::invoke_output::invoke_event_channel(
            crate::extensions::invoke_output::InvokeOutputBudget::default(),
        );
        let handler = match self.user_action_handler(id) {
            Ok(handler) => handler,
            Err(error) => {
                drop(sink);
                return (Err(error), collector.collect().await);
            }
        };
        let invoke = handler.invoke_command(command, args, request_id, sink);
        let (result, report) = tokio::join!(invoke, collector.collect());
        (result, report)
    }

    pub async fn settings_editor_open(
        &self,
        id: &str,
        category: &str,
        field: &str,
    ) -> Result<serde_json::Value, String> {
        let handler = self.user_action_handler(id)?;
        handler.settings_editor_open(category, field).await
    }

    pub async fn settings_editor_key(
        &self,
        id: &str,
        category: &str,
        field: &str,
        key: &str,
    ) -> Result<serde_json::Value, String> {
        let handler = self.user_action_handler(id)?;
        handler.settings_editor_key(category, field, key).await
    }

    pub async fn settings_editor_commit(
        &self,
        id: &str,
        category: &str,
        field: &str,
        value: serde_json::Value,
    ) -> Result<serde_json::Value, String> {
        let handler = self.user_action_handler(id)?;
        handler.settings_editor_commit(category, field, value).await
    }

    /// Return all cached plugin info sorted by extension id.
    pub fn plugin_infos(&self) -> Vec<(&str, &PluginInfo)> {
        let mut entries: Vec<_> = self
            .plugin_info
            .iter()
            .map(|(id, info)| (id.as_str(), info))
            .collect();
        entries.sort_by(|a, b| a.0.cmp(b.0));
        entries
    }

    /// Return provider status summaries sorted by provider runtime id.
    pub fn provider_summaries(&self) -> Vec<RegisteredProviderSummary> {
        self.providers.summaries()
    }

    /// Unified capability snapshot per loaded extension, sorted by id.
    ///
    /// Aggregates hook subscriptions, extension-provided tools, and registered
    /// providers. `future` carries plugin-defined capability kinds and
    /// capabilities land.
    pub async fn capability_snapshots(&self) -> Vec<ExtensionCapabilitySnapshot> {
        let mut handlers: Vec<(String, Arc<dyn ExtensionHandler>)> = self
            .extensions
            .iter()
            .map(|(id, handler)| (id.clone(), handler.clone()))
            .collect();
        handlers.sort_by(|a, b| a.0.cmp(&b.0));

        let provider_summaries = self.providers.summaries();
        let plugin_id_lookup: std::collections::HashMap<String, String> = self
            .providers
            .list()
            .into_iter()
            .map(|p| (p.runtime_id.clone(), p.plugin_id.clone()))
            .collect();

        let mut out = Vec::with_capacity(handlers.len());
        for (id, handler) in handlers {
            let health = handler.health().await;
            let restart_count = handler.restart_count().await;

            let hook_pairs = self.hook_bus.subscriptions_for(&id).await;
            let hooks: Vec<HookCapabilityEntry> = hook_pairs
                .into_iter()
                .map(|(kind, tool_filter)| HookCapabilityEntry {
                    kind: kind.as_str().to_string(),
                    tool_filter,
                })
                .collect();

            let tools: Vec<ToolCapabilityEntry> = if let Some(tools) = &self.tools {
                let registry = tools.read().await;
                registry
                    .tool_names_for_extension(&id)
                    .into_iter()
                    .map(|name| ToolCapabilityEntry { name })
                    .collect()
            } else {
                Vec::new()
            };

            let providers: Vec<RegisteredProviderSummary> = provider_summaries
                .iter()
                .filter(|summary| {
                    plugin_id_lookup
                        .get(&summary.runtime_id)
                        .map(|p| p == &id)
                        .unwrap_or(false)
                })
                .cloned()
                .collect();

            let future: Vec<FutureCapabilityEntry> = self
                .capabilities
                .get(&id)
                .map(|decls| {
                    decls
                        .iter()
                        .map(|d| FutureCapabilityEntry {
                            kind: d.kind.clone(),
                            name: d.name.clone(),
                        })
                        .collect()
                })
                .unwrap_or_default();

            out.push(ExtensionCapabilitySnapshot {
                id,
                health,
                restart_count,
                hooks,
                tools,
                providers,
                future,
            });
        }
        out
    }

    /// Return runtime ids of registered providers that declare at least one
    /// tool-use-capable model. Sorted by runtime id.
    pub fn provider_tool_use_runtime_ids(&self) -> Vec<String> {
        let mut ids: Vec<String> = self
            .providers
            .list()
            .into_iter()
            .filter(|p| {
                p.spec.models.iter().any(|m| {
                    m.capabilities
                        .get("tool_use")
                        .and_then(|v| v.as_bool())
                        .unwrap_or(false)
                })
            })
            .map(|p| p.runtime_id.clone())
            .collect();
        ids.sort();
        ids
    }

    /// Return a `runtime_id -> enabled` map for every registered provider, computed
    /// from the persisted trust state. Providers without an entry default to
    /// enabled. If the trust state file is missing, all providers are reported
    /// as enabled (default). If the file is corrupt, all providers are reported
    /// as **disabled** (fail-closed) and a warning is logged.
    pub fn provider_trust_view(&self) -> std::collections::BTreeMap<String, bool> {
        let trust = match crate::extensions::trust::load_trust_state() {
            Ok(t) => t,
            Err(e) => {
                tracing::warn!("trust.json corrupt or unreadable, failing closed (all providers disabled): {e}");
                // Return all providers as disabled
                return self
                    .providers
                    .list()
                    .into_iter()
                    .map(|p| (p.runtime_id.clone(), false))
                    .collect();
            }
        };
        self.providers
            .list()
            .into_iter()
            .map(|p| {
                let enabled = crate::extensions::trust::is_provider_enabled(&trust, &p.runtime_id);
                (p.runtime_id.clone(), enabled)
            })
            .collect()
    }

    /// Compute config diagnostics for a loaded extension by id.
    /// Returns `None` if the extension is not loaded.
    pub fn config_diagnostics(&self, id: &str) -> Option<ExtensionConfigDiagnostics> {
        let manifest_config = self.manifest_configs.get(id)?;

        // Collect provider required keys from registered providers' config_schema.
        let mut provider_required: Vec<(String, Vec<String>)> = Vec::new();
        for provider in self.providers.list() {
            if provider.plugin_id != id {
                continue;
            }
            let required: Vec<String> = provider
                .spec
                .config_schema
                .as_ref()
                .and_then(|schema| schema.get("required"))
                .and_then(Value::as_array)
                .map(|arr| {
                    arr.iter()
                        .filter_map(|v| v.as_str().map(|s| s.to_string()))
                        .collect()
                })
                .unwrap_or_default();
            provider_required.push((provider.provider_id.clone(), required));
        }
        provider_required.sort_by(|a, b| a.0.cmp(&b.0));

        let env_lookup = |name: &str| std::env::var(name).ok();
        let plugin_config_lookup =
            |key: &str| crate::extensions::config_store::read_plugin_config(id, key);
        let legacy_config_lookup = |key: &str| crate::config::read_config_value(key);

        Some(diagnose_extension_config(
            id,
            manifest_config,
            &provider_required,
            &env_lookup,
            &plugin_config_lookup,
            &legacy_config_lookup,
        ))
    }

    /// Diagnostics for all loaded extensions, sorted alphabetically by id.
    pub fn all_config_diagnostics(&self) -> Vec<ExtensionConfigDiagnostics> {
        let mut ids: Vec<&String> = self.manifest_configs.keys().collect();
        ids.sort();
        ids.into_iter()
            .filter_map(|id| self.config_diagnostics(id))
            .collect()
    }

    /// Get the shared hook bus.
    pub fn hook_bus(&self) -> &Arc<HookBus> {
        &self.hook_bus
    }

    /// Get the shared tool registry, when this manager was constructed with one.
    pub fn tools_shared(&self) -> Option<Arc<tokio::sync::RwLock<crate::ToolRegistry>>> {
        self.tools.clone()
    }

    /// Test-only: register a provider spec + in-process handler directly,
    /// without spawning an extension process. Used by trace wiring tests.
    #[cfg(test)]
    pub(crate) fn register_provider_handler_for_tests(
        &mut self,
        plugin_id: &str,
        spec: crate::extensions::runtime::process::RegisteredProviderSpec,
        handler: Arc<dyn ExtensionHandler>,
    ) -> Result<String, String> {
        self.providers
            .register_with_handler(plugin_id, spec, Some(handler))
    }

    /// Discover and load all extensions from the user and project plugin directories.
    ///
    /// Scans `~/.synaps-cli/plugins/*/.synaps-plugin/plugin.json` and
    /// `./.synaps/plugins/*/.synaps-plugin/plugin.json` for manifests that contain
    /// an `extension` field. Project-local plugins override user plugins with the
    /// same directory name.
    pub async fn discover_and_load(&mut self) -> (Vec<String>, Vec<ExtensionLoadFailure>) {
        self.discover_and_load_with_progress(|_| {}).await
    }

    /// Discover and load all extensions, invoking `progress` after each load
    /// attempt. Used by the async UI loader to update startup toasts without
    /// blocking first paint.
    pub async fn discover_and_load_with_progress<F>(
        &mut self,
        mut progress: F,
    ) -> (Vec<String>, Vec<ExtensionLoadFailure>)
    where
        F: FnMut(crate::extensions::loader::ExtensionLoaderEvent),
    {
        let mut plugin_roots = vec![crate::config::base_dir().join("plugins")];
        if !project_plugins_disabled() {
            if let Ok(cwd) = std::env::current_dir() {
                let project_plugins = cwd.join(".synaps").join("plugins");
                if project_plugins != plugin_roots[0] {
                    plugin_roots.push(project_plugins);
                }
            }
        }

        let mut plugin_dirs: HashMap<String, PathBuf> = HashMap::new();
        let mut failed: Vec<ExtensionLoadFailure> = Vec::new();

        for plugins_dir in plugin_roots {
            if !plugins_dir.exists() {
                continue;
            }

            let entries = match std::fs::read_dir(&plugins_dir) {
                Ok(e) => e,
                Err(e) => {
                    tracing::warn!(path = %plugins_dir.display(), error = %e, "Failed to read plugins directory");
                    failed.push(ExtensionLoadFailure::new(
                        "plugins",
                        Some(plugins_dir.clone()),
                        format!("Failed to read plugins directory: {e}"),
                        "Check directory permissions and retry",
                    ));
                    continue;
                }
            };

            for entry in entries.flatten() {
                let plugin_name = entry.file_name().to_string_lossy().to_string();
                if is_inert_plugin_dir_name(&plugin_name) {
                    tracing::debug!(
                        plugin = %plugin_name,
                        path = %entry.path().display(),
                        "Skipping backup/disabled/temp plugin directory"
                    );
                    continue;
                }
                plugin_dirs.insert(plugin_name, entry.path());
            }
        }

        let mut plugin_dirs: Vec<(String, PathBuf)> = plugin_dirs.into_iter().collect();
        plugin_dirs.sort_by(|a, b| a.0.cmp(&b.0));

        let mut loaded = Vec::new();
        let disabled_plugins = crate::config::load_config().disabled_plugins;
        // Hoist plugins.json read out of the per-plugin loop — was N reads + parses
        // of the same file (one per discovered plugin). Read it once here.
        let plugins_state = {
            let state_path = crate::skills::state::PluginsState::default_path();
            crate::skills::state::PluginsState::load_from(&state_path).unwrap_or_default()
        };
        for (plugin_name, plugin_dir) in plugin_dirs {
            if disabled_plugins.iter().any(|d| d == &plugin_name) {
                tracing::debug!(plugin = %plugin_name, "Extension disabled via disabled_plugins config");
                continue;
            }
            if let Some(message) = installed_plugin_setup_failure_in(&plugins_state, &plugin_name) {
                tracing::warn!(plugin = %plugin_name, error = %message, "Skipping extension with failed post-install setup");
                failed.push(ExtensionLoadFailure::new(
                    plugin_name,
                    None,
                    format!("Post-install setup failed: {message}"),
                    "Open /plugins, reinstall or update the plugin after fixing setup; extension load is disabled until setup succeeds",
                ));
                continue;
            }
            let manifest_path = plugin_dir.join(".synaps-plugin").join("plugin.json");
            if !manifest_path.exists() {
                continue;
            }

            let content = match std::fs::read_to_string(&manifest_path) {
                Ok(c) => c,
                Err(e) => {
                    let reason = format!("Failed to read plugin manifest: {e}");
                    tracing::warn!(plugin = %plugin_name, manifest = %manifest_path.display(), error = %e, "Failed to read plugin manifest");
                    failed.push(ExtensionLoadFailure::new(
                        plugin_name,
                        Some(manifest_path),
                        reason,
                        "Check manifest file permissions, then run `plugin validate <plugin-dir>`",
                    ));
                    continue;
                }
            };

            let json: serde_json::Value = match serde_json::from_str(&content) {
                Ok(v) => v,
                Err(e) => {
                    let reason = format!("Invalid plugin manifest JSON: {e}");
                    tracing::warn!(plugin = %plugin_name, manifest = %manifest_path.display(), error = %e, "Invalid plugin manifest JSON");
                    failed.push(ExtensionLoadFailure::new(
                        plugin_name,
                        Some(manifest_path),
                        reason,
                        "Fix JSON syntax, then run `plugin validate <plugin-dir>`",
                    ));
                    continue;
                }
            };

            let ext_value = match json.get("extension") {
                Some(v) => v.clone(),
                None => continue,
            };

            let ext_manifest: ExtensionManifest = match serde_json::from_value(ext_value) {
                Ok(m) => m,
                Err(e) => {
                    let reason = format!("Failed to parse extension manifest: {e}");
                    tracing::warn!(plugin = %plugin_name, manifest = %manifest_path.display(), error = %e, "Failed to parse extension manifest");
                    failed.push(ExtensionLoadFailure::new(
                        plugin_name,
                        Some(manifest_path),
                        reason,
                        "Check the `extension` block shape against docs/extensions/contract.json, then run `plugin validate <plugin-dir>`",
                    ));
                    continue;
                }
            };

            #[allow(clippy::if_same_then_else)]
            let command = if std::path::Path::new(&ext_manifest.command).is_absolute() {
                ext_manifest.command.clone()
            } else if !ext_manifest.command.contains(std::path::MAIN_SEPARATOR)
                && !ext_manifest.command.contains('/')
            {
                ext_manifest.command.clone()
            } else {
                plugin_dir
                    .join(&ext_manifest.command)
                    .to_string_lossy()
                    .to_string()
            };

            let args: Vec<String> = ext_manifest
                .args
                .iter()
                .map(|arg| {
                    let arg_path = plugin_dir.join(arg);
                    if arg_path.exists() {
                        if let (Ok(canonical), Ok(plugin_canonical)) =
                            (arg_path.canonicalize(), plugin_dir.canonicalize())
                        {
                            if canonical.starts_with(&plugin_canonical) {
                                return canonical.to_string_lossy().to_string();
                            }
                        }
                    }
                    arg.clone()
                })
                .collect();

            let resolved = ExtensionManifest {
                theme_tokens: Default::default(),
                // Deferred declarations flow through from the installed
                // manifest (legacy `tools`/`activation` aliases are folded
                // into `deferred` at deserialization) so installed plugins
                // keep their zero-spawn deferred lifecycle instead of
                // silently loading eager.
                command,
                args,
                ..ext_manifest
            };

            match self
                .load_with_cwd(&plugin_name, &resolved, Some(plugin_dir.clone()))
                .await
            {
                Ok(()) => {
                    tracing::info!(plugin = %plugin_name, path = %plugin_dir.display(), "Extension loaded from plugins/");
                    loaded.push(plugin_name.clone());
                    progress(crate::extensions::loader::ExtensionLoaderEvent::Loaded {
                        plugin: plugin_name,
                        loaded: loaded.len(),
                        failed: failed.len(),
                    });
                }
                Err(e) => {
                    tracing::warn!(plugin = %plugin_name, manifest = %manifest_path.display(), error = %e, "Failed to load extension");
                    let setup_script = json
                        .pointer("/extension/setup")
                        .and_then(|v| v.as_str())
                        .or_else(|| {
                            json.pointer("/provides/sidecar/setup")
                                .and_then(|v| v.as_str())
                        });
                    let hint = compute_extension_load_hint(&e, &plugin_dir, setup_script);
                    let failure =
                        ExtensionLoadFailure::new(plugin_name, Some(manifest_path), e, hint);
                    failed.push(failure.clone());
                    progress(crate::extensions::loader::ExtensionLoaderEvent::Failed {
                        failure,
                        loaded: loaded.len(),
                        failed: failed.len(),
                    });
                }
            }
        }

        (loaded, failed)
    }
}

#[cfg(test)]
mod invoke_command_collected_tests {
    //! CP-11 fix-3: prove the manager's collected entry point pairs the
    //! bounded sink with an eagerly concurrent collector — an in-process
    //! handler that floods far past the channel capacity AND the retained
    //! budget must neither deadlock nor grow retention, and accounting
    //! must be exact.
    use super::*;
    use crate::extensions::commands::CommandOutputEvent;
    use crate::extensions::invoke_output::INVOKE_RETAINED_BYTE_BUDGET;
    use crate::extensions::runtime::InvokeCommandEvent;

    struct FloodingHandler;

    #[async_trait::async_trait]
    impl super::ExtensionHandler for FloodingHandler {
        fn id(&self) -> &str {
            "flooding-fake"
        }
        async fn handle(
            &self,
            _event: &crate::extensions::hooks::events::HookEvent,
        ) -> crate::extensions::hooks::events::HookResult {
            Default::default()
        }
        async fn shutdown(&self) {}

        async fn invoke_command(
            &self,
            _command: &str,
            _args: Vec<String>,
            _request_id: &str,
            sink: crate::extensions::invoke_output::InvokeEventSink,
        ) -> Result<Value, String> {
            // 2048 x 4 KiB = 8 MiB >> 256 KiB budget, and 2048 events
            // >> channel capacity 8: every awaited send past capacity
            // requires the concurrent collector to make progress.
            for _ in 0..2048 {
                let event = InvokeCommandEvent::Output(CommandOutputEvent::Text {
                    content: "f".repeat(4096),
                });
                if sink.send(event).await.is_err() {
                    return Err("collector vanished".to_string());
                }
            }
            let _ = sink
                .send(InvokeCommandEvent::Output(CommandOutputEvent::Done))
                .await;
            Ok(serde_json::json!({"status": "flooded"}))
        }
    }

    #[tokio::test]
    async fn collected_invoke_bounds_in_process_flood_without_deadlock() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus);
        mgr.extensions
            .insert("flooding-fake".to_string(), Arc::new(FloodingHandler));

        let (result, report) = tokio::time::timeout(
            std::time::Duration::from_secs(30),
            mgr.invoke_command_collected("flooding-fake", "any", vec![], "req-1"),
        )
        .await
        .expect("bounded sink + concurrent collector must not deadlock");

        assert_eq!(result.expect("invoke ok")["status"], "flooded");
        let c = &report.counters;
        assert_eq!(c.produced_events, 2049);
        assert_eq!(c.produced_payload_bytes, 2048 * 4096);
        assert_eq!(c.consumed_events, c.produced_events);
        assert_eq!(c.consumed_payload_bytes, c.produced_payload_bytes);
        assert_eq!(c.retained_payload_bytes, INVOKE_RETAINED_BYTE_BUDGET as u64);
        assert!(c.saw_done);
        assert_eq!(
            c.consumed_payload_bytes,
            c.retained_payload_bytes + c.truncated_payload_bytes + c.dropped_payload_bytes,
        );
        assert_eq!(c.consumed_events, c.retained_events + c.dropped_events);
    }

    #[tokio::test]
    async fn collected_invoke_reports_unknown_extension_with_empty_report() {
        let bus = Arc::new(HookBus::new());
        let mgr = ExtensionManager::new(bus);
        let (result, report) = mgr
            .invoke_command_collected("nope", "any", vec![], "req-1")
            .await;
        assert!(result
            .expect_err("unknown id")
            .contains("unknown extension"));
        assert!(report.events.is_empty());
        assert_eq!(report.counters.produced_events, 0);
    }
}

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

    /// Task A6: context-provider manifest for one plugin (no tools, no
    /// hooks — classifies as `ContextProvider` and defers without spawn).
    fn context_provider_manifest(provider_id: &str) -> ExtensionManifest {
        serde_json::from_value(serde_json::json!({
            "runtime": "process",
            "command": "/bin/false",
            "permissions": ["context_providers.register"],
            "deferred": {
                "context_providers": [{
                    "id": provider_id,
                    "capability": "project-memory",
                    "description": "test context provider",
                    "schema_version": 1
                }]
            }
        }))
        .unwrap()
    }

    /// Task A6: the context-provider catalog mirrors the model-provider
    /// catalog shape — populated at load, exact fail-closed lookup, kept
    /// current across unload, shared with the lease manager — and every
    /// read is pure metadata: zero processes are ever started.
    #[tokio::test]
    async fn context_provider_catalog_lists_resolves_and_unloads_exactly() {
        let mut mgr = ExtensionManager::new(Arc::new(HookBus::new()));
        mgr.set_progressive_deferral(true);
        mgr.load("mem-a", &context_provider_manifest("alpha"))
            .await
            .unwrap();
        mgr.load("mem-b", &context_provider_manifest("beta"))
            .await
            .unwrap();

        // Catalog reads: sorted by runtime address, exact ids only.
        let listed: Vec<String> = mgr
            .context_providers()
            .iter()
            .map(|d| d.runtime_address())
            .collect();
        assert_eq!(
            listed,
            vec![
                "extension:mem-a:alpha".to_string(),
                "extension:mem-b:beta".to_string(),
            ]
        );
        let alpha = super::super::context_provider::ContextProviderId::parse("alpha").unwrap();
        assert_eq!(
            mgr.context_provider(&alpha).map(|d| d.extension_id()),
            Some("mem-a")
        );
        let unknown = super::super::context_provider::ContextProviderId::parse("nope").unwrap();
        assert!(mgr.context_provider(&unknown).is_none(), "fail closed");

        // The shared lease-manager handle observes the same catalog.
        let runtime = mgr.extension_runtime();
        assert_eq!(
            runtime
                .resolve_context_provider(Some(&alpha))
                .map(|d| d.runtime_address()),
            Ok("extension:mem-a:alpha".to_string())
        );

        // An overlapping declaration makes the id ambiguous — fail closed
        // on BOTH read surfaces, never an arbitrary pick.
        mgr.load("mem-c", &context_provider_manifest("alpha"))
            .await
            .unwrap();
        assert!(mgr.context_provider(&alpha).is_none(), "ambiguous → None");
        assert_eq!(
            runtime.resolve_context_provider(Some(&alpha)),
            Err(super::super::context_provider::ContextProviderLookupError::Ambiguous)
        );

        // Unload removes exactly the owner's descriptors and republishes.
        mgr.unload("mem-c").await.unwrap();
        assert_eq!(
            mgr.context_provider(&alpha).map(|d| d.extension_id()),
            Some("mem-a")
        );
        mgr.unload("mem-a").await.unwrap();
        assert_eq!(
            runtime.resolve_context_provider(Some(&alpha)),
            Err(super::super::context_provider::ContextProviderLookupError::NotRegistered)
        );
        assert_eq!(mgr.context_providers().len(), 1, "mem-b remains");

        // Pure catalog discipline: nothing above started a process.
        assert_eq!(mgr.count(), 0, "no live extension handlers");
        assert_eq!(runtime.lease_count(), 0, "no runtime leases");
    }

    #[tokio::test]
    async fn capability_snapshots_empty_when_no_extensions() {
        let bus = Arc::new(HookBus::new());
        let mgr = ExtensionManager::new(bus);
        assert!(mgr.capability_snapshots().await.is_empty());
    }

    /// Review fix A3: the retained deferred launch record resolves
    /// `secret_env` config values, so NOTHING public/debug/diagnostic may
    /// ever reveal them — only booleans/counts. The record type itself is
    /// crate-private, field-private, and implements neither `Debug` nor
    /// `Serialize` (compiler-enforced); this test additionally proves the
    /// live diagnostic surfaces never carry the resolved secret.
    #[tokio::test]
    async fn deferred_launch_record_never_leaks_secret_config_through_diagnostics() {
        const SECRET: &str = "task20-A3-sentinel-5f2c9d";
        // NOTE: deliberately NOT in the SYNAPS_* host namespace — that is denied by
        // `denied_secret_env` (#207 MED-2). This test is about diagnostic leakage,
        // not namespace policy, so it uses a realistic third-party var name.
        const VAR: &str = "T20_A3_THIRD_PARTY_SECRET";
        std::env::set_var(VAR, SECRET);

        let manifest: ExtensionManifest = serde_json::from_value(serde_json::json!({
            "runtime": "process",
            "command": "/bin/false",
            "permissions": ["tools.register"],
            "config": [{
                "key": "api_key",
                "type": "string",
                "required": true,
                "secret_env": VAR
            }],
            "deferred": {
                "tools": [{
                    "name": "search",
                    "description": "deferred search",
                    "input_schema": {"type": "object"}
                }]
            }
        }))
        .unwrap();

        let registry = Arc::new(tokio::sync::RwLock::new(crate::ToolRegistry::new()));
        let mut mgr = ExtensionManager::new_with_tools(Arc::new(HookBus::new()), registry);
        mgr.set_progressive_deferral(true);
        mgr.load("secretive", &manifest).await.unwrap();

        // Boolean/count-only public surface for the deferral state.
        assert!(mgr.is_deferred_tool_only("secretive"));
        assert_eq!(mgr.deferred_tool_only_count(), 1);
        // The internal record DID resolve the secret (lease path needs it).
        let record = mgr.deferred_launch_record("secretive").expect("retained");
        assert_eq!(record.config["api_key"], serde_json::json!(SECRET));

        // Every public/debug/diagnostic surface stays free of the value.
        let mut rendered = String::new();
        rendered.push_str(&format!("{:?}", mgr.statuses().await));
        rendered.push_str(&format!("{:?}", mgr.config_diagnostics("secretive")));
        rendered.push_str(&format!("{:?}", mgr.all_config_diagnostics()));
        rendered.push_str(&format!("{:?}", mgr.capability_snapshots().await));
        rendered.push_str(&format!("{:?}", mgr.plugin_infos()));
        rendered.push_str(&format!("{:?}", mgr.list()));
        assert!(
            !rendered.contains(SECRET),
            "resolved secret config leaked through a public diagnostic surface"
        );

        std::env::remove_var(VAR);
    }

    #[tokio::test]
    async fn capability_snapshot_lists_hooks_for_loaded_extension() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus.clone());
        let manifest = ExtensionManifest {
            theme_tokens: Default::default(),
            deferred: None,
            protocol_version: 1,
            runtime: crate::extensions::manifest::ExtensionRuntime::Process,
            command: "python3".to_string(),
            setup: None,
            prebuilt: ::std::collections::HashMap::new(),
            args: vec![
                "tests/fixtures/process_extension.py".to_string(),
                "normal".to_string(),
                "/tmp/synaps-capability-test.log".to_string(),
            ],
            permissions: vec!["tools.intercept".to_string()],
            hooks: vec![crate::extensions::manifest::HookSubscription {
                hook: "before_tool_call".to_string(),
                tool: Some("bash".to_string()),
                matcher: None,
            }],
            config: vec![],
        };

        mgr.load("cap-snap", &manifest).await.unwrap();

        let snaps = mgr.capability_snapshots().await;
        assert_eq!(snaps.len(), 1);
        let snap = &snaps[0];
        assert_eq!(snap.id, "cap-snap");
        assert_eq!(snap.hooks.len(), 1);
        assert_eq!(snap.hooks[0].kind, "before_tool_call");
        assert_eq!(snap.hooks[0].tool_filter.as_deref(), Some("bash"));
        assert!(snap.tools.is_empty());
        assert!(snap.providers.is_empty());
        assert!(snap.future.is_empty());

        mgr.shutdown_all().await;
    }

    #[tokio::test]
    async fn capability_snapshot_surfaces_seeded_capabilities() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus.clone());
        let manifest = ExtensionManifest {
            theme_tokens: Default::default(),
            deferred: None,
            protocol_version: 1,
            runtime: crate::extensions::manifest::ExtensionRuntime::Process,
            command: "python3".to_string(),
            setup: None,
            prebuilt: ::std::collections::HashMap::new(),
            args: vec![
                "tests/fixtures/process_extension.py".to_string(),
                "normal".to_string(),
                "/tmp/synaps-capability-snapshot-test.log".to_string(),
            ],
            permissions: vec!["tools.intercept".to_string()],
            hooks: vec![crate::extensions::manifest::HookSubscription {
                hook: "before_tool_call".to_string(),
                tool: Some("bash".to_string()),
                matcher: None,
            }],
            config: vec![],
        };

        mgr.load("multi-cap", &manifest).await.unwrap();

        // Seed two capabilities of *different* kinds — proves the
        // snapshot rendering iterates a generic list and uses the
        // plugin-supplied `kind` rather than hardcoding any modality.
        mgr.test_seed_capabilities(
            "multi-cap",
            vec![
                crate::extensions::runtime::process::CapabilityDeclaration {
                    kind: "capture".to_string(),
                    name: "Local Sample STT".to_string(),
                    permissions: vec!["audio.input".to_string()],
                    params: serde_json::Value::Null,
                },
                crate::extensions::runtime::process::CapabilityDeclaration {
                    kind: "ocr".to_string(),
                    name: "Tesseract".to_string(),
                    permissions: vec![],
                    params: serde_json::Value::Null,
                },
            ],
        );

        let snaps = mgr.capability_snapshots().await;
        let snap = snaps
            .iter()
            .find(|s| s.id == "multi-cap")
            .expect("multi-cap snapshot");
        assert_eq!(snap.future.len(), 2);
        let kinds: Vec<&str> = snap.future.iter().map(|e| e.kind.as_str()).collect();
        assert!(kinds.contains(&"capture"), "got kinds {:?}", kinds);
        assert!(kinds.contains(&"ocr"), "got kinds {:?}", kinds);
        let names: Vec<&str> = snap.future.iter().map(|e| e.name.as_str()).collect();
        assert!(names.contains(&"Local Sample STT"), "got {:?}", names);
        assert!(names.contains(&"Tesseract"), "got {:?}", names);

        mgr.unload("multi-cap").await.unwrap();
        let snaps = mgr.capability_snapshots().await;
        assert!(snaps.iter().all(|s| s.id != "multi-cap"));

        mgr.shutdown_all().await;
    }

    #[tokio::test]
    async fn new_manager_has_no_extensions() {
        let bus = Arc::new(HookBus::new());
        let mgr = ExtensionManager::new(bus);
        assert_eq!(mgr.count(), 0);
        assert!(mgr.list().is_empty());
    }

    #[tokio::test]
    async fn unload_nonexistent_returns_error() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus);
        let result = mgr.unload("nope").await;
        assert!(result.is_err());
    }

    #[tokio::test]
    async fn reload_unsubscribes_old_handler_before_loading_new_one() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus.clone());
        let manifest = ExtensionManifest {
            theme_tokens: Default::default(),
            deferred: None,
            protocol_version: 1,
            runtime: crate::extensions::manifest::ExtensionRuntime::Process,
            command: "python3".to_string(),
            setup: None,
            prebuilt: ::std::collections::HashMap::new(),
            args: vec![
                "tests/fixtures/process_extension.py".to_string(),
                "normal".to_string(),
                "/tmp/synaps-reload-test.log".to_string(),
            ],
            permissions: vec!["tools.intercept".to_string()],
            hooks: vec![crate::extensions::manifest::HookSubscription {
                hook: "before_tool_call".to_string(),
                tool: Some("bash".to_string()),
                matcher: None,
            }],
            config: vec![],
        };

        mgr.load("reload-test", &manifest).await.unwrap();
        assert_eq!(bus.handler_count().await, 1);

        mgr.reload("reload-test", &manifest, None).await.unwrap();

        assert_eq!(mgr.count(), 1);
        assert_eq!(bus.handler_count().await, 1);
        mgr.shutdown_all().await;
    }

    #[tokio::test]
    async fn reload_failure_leaves_previous_instance_unloaded() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus.clone());
        let good = ExtensionManifest {
            theme_tokens: Default::default(),
            deferred: None,
            protocol_version: 1,
            runtime: crate::extensions::manifest::ExtensionRuntime::Process,
            command: "python3".to_string(),
            setup: None,
            prebuilt: ::std::collections::HashMap::new(),
            args: vec![
                "tests/fixtures/process_extension.py".to_string(),
                "normal".to_string(),
                "/tmp/synaps-reload-failure-test.log".to_string(),
            ],
            permissions: vec!["tools.intercept".to_string()],
            hooks: vec![crate::extensions::manifest::HookSubscription {
                hook: "before_tool_call".to_string(),
                tool: Some("bash".to_string()),
                matcher: None,
            }],
            config: vec![],
        };
        let bad = ExtensionManifest {
            theme_tokens: Default::default(),
            deferred: None,
            command: "/definitely/not/a/real/extension-binary".to_string(),
            setup: None,
            prebuilt: ::std::collections::HashMap::new(),
            ..good.clone()
        };

        mgr.load("reload-failure-test", &good).await.unwrap();
        let err = mgr
            .reload("reload-failure-test", &bad, None)
            .await
            .unwrap_err();

        assert!(err.contains("Failed to spawn extension"), "{err}");
        assert_eq!(mgr.count(), 0);
        assert_eq!(bus.handler_count().await, 0);
    }

    #[test]
    fn project_plugins_disable_env_parser_accepts_truthy_values() {
        for value in ["1", "true", "TRUE", "yes", "on"] {
            std::env::set_var("SYNAPS_DISABLE_PROJECT_PLUGINS", value);
            assert!(project_plugins_disabled());
        }
        for value in ["", "0", "false", "off", "no"] {
            std::env::set_var("SYNAPS_DISABLE_PROJECT_PLUGINS", value);
            assert!(!project_plugins_disabled());
        }
        std::env::remove_var("SYNAPS_DISABLE_PROJECT_PLUGINS");
    }

    fn with_temp_base_dir<T>(path: &std::path::Path, f: impl FnOnce() -> T) -> T {
        let old_base_dir = std::env::var("SYNAPS_BASE_DIR").ok();
        crate::config::set_base_dir_for_tests(path.to_path_buf());
        let out = f();
        match old_base_dir {
            Some(old) => std::env::set_var("SYNAPS_BASE_DIR", old),
            None => std::env::remove_var("SYNAPS_BASE_DIR"),
        }
        out
    }

    #[test]
    #[serial_test::serial(synaps_base_dir)]
    fn resolve_config_prefers_plugin_namespaced_config_before_legacy_global_key() {
        let dir = tempfile::tempdir().unwrap();
        with_temp_base_dir(dir.path(), || {
            crate::extensions::config_store::write_plugin_config(
                "sample-sidecar",
                "backend",
                "cpu",
            )
            .unwrap();
            crate::config::write_config_value("extension.sample-sidecar.backend", "auto").unwrap();

            let resolved = ExtensionManager::resolve_config(
                "sample-sidecar",
                &[ExtensionConfigEntry {
                    key: "backend".to_string(),
                    value_type: None,
                    description: None,
                    required: true,
                    default: None,
                    secret_env: None,
                    host_context: None,
                }],
                None,
            )
            .unwrap();

            assert_eq!(
                resolved["backend"],
                serde_json::Value::String("cpu".to_string())
            );
        });
    }

    #[test]
    #[serial_test::serial(synaps_base_dir)]
    fn resolve_config_keeps_legacy_global_extension_key_as_fallback() {
        let dir = tempfile::tempdir().unwrap();
        with_temp_base_dir(dir.path(), || {
            crate::config::write_config_value("extension.sample-sidecar.backend", "auto").unwrap();

            let resolved = ExtensionManager::resolve_config(
                "sample-sidecar",
                &[ExtensionConfigEntry {
                    key: "backend".to_string(),
                    value_type: None,
                    description: None,
                    required: true,
                    default: None,
                    secret_env: None,
                    host_context: None,
                }],
                None,
            )
            .unwrap();

            assert_eq!(
                resolved["backend"],
                serde_json::Value::String("auto".to_string())
            );
        });
    }

    #[tokio::test]
    async fn config_diagnostics_returns_none_for_unknown_extension() {
        let bus = Arc::new(HookBus::new());
        let mgr = ExtensionManager::new(bus);
        assert!(mgr.config_diagnostics("nope").is_none());
        assert!(mgr.all_config_diagnostics().is_empty());
    }

    #[tokio::test]
    async fn config_diagnostics_reports_loaded_manifest_entries() {
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus);
        let manifest = ExtensionManifest {
            theme_tokens: Default::default(),
            deferred: None,
            protocol_version: 1,
            runtime: crate::extensions::manifest::ExtensionRuntime::Process,
            command: "python3".to_string(),
            setup: None,
            prebuilt: ::std::collections::HashMap::new(),
            args: vec![
                "tests/fixtures/process_extension.py".to_string(),
                "normal".to_string(),
                "/tmp/synaps-config-diag-test.log".to_string(),
            ],
            permissions: vec!["tools.intercept".to_string()],
            hooks: vec![crate::extensions::manifest::HookSubscription {
                hook: "before_tool_call".to_string(),
                tool: Some("bash".to_string()),
                matcher: None,
            }],
            config: vec![crate::extensions::manifest::ExtensionConfigEntry {
                key: "region".to_string(),
                value_type: None,
                description: Some("AWS region".to_string()),
                required: false,
                default: Some(serde_json::Value::String("us-east-1".to_string())),
                secret_env: None,
                host_context: None,
            }],
        };

        mgr.load("config-diag-test", &manifest).await.unwrap();

        let diag = mgr
            .config_diagnostics("config-diag-test")
            .expect("diagnostics should be available for loaded extension");
        assert_eq!(diag.extension_id, "config-diag-test");
        assert_eq!(diag.entries.len(), 1);
        assert_eq!(diag.entries[0].key, "region");
        assert!(diag.entries[0].has_value);
        assert!(diag.provider_missing.is_empty());

        let all = mgr.all_config_diagnostics();
        assert_eq!(all.len(), 1);
        assert_eq!(all[0].extension_id, "config-diag-test");

        mgr.shutdown_all().await;
        // After shutdown, manifest config storage is cleared.
        assert!(mgr.config_diagnostics("config-diag-test").is_none());
    }

    #[tokio::test]
    async fn provider_trust_view_is_empty_for_no_providers() {
        let bus = Arc::new(HookBus::new());
        let mgr = ExtensionManager::new(bus);
        let view = mgr.provider_trust_view();
        assert!(view.is_empty());
    }

    #[tokio::test]
    async fn provider_tool_use_runtime_ids_lists_only_tool_use_capable() {
        use crate::extensions::runtime::process::{
            RegisteredProviderModelSpec, RegisteredProviderSpec,
        };
        let bus = Arc::new(HookBus::new());
        let mut mgr = ExtensionManager::new(bus);
        // Tool-use capable provider.
        let tool_spec = RegisteredProviderSpec {
            id: "alpha".into(),
            display_name: "Alpha".into(),
            description: "tool-use".into(),
            models: vec![RegisteredProviderModelSpec {
                id: "m1".into(),
                display_name: None,
                capabilities: serde_json::json!({"tool_use": true}),
                context_window: None,
            }],
            config_schema: None,
        };
        // Plain provider, no tool_use.
        let plain_spec = RegisteredProviderSpec {
            id: "beta".into(),
            display_name: "Beta".into(),
            description: "plain".into(),
            models: vec![RegisteredProviderModelSpec {
                id: "m1".into(),
                display_name: None,
                capabilities: serde_json::json!({"streaming": true}),
                context_window: None,
            }],
            config_schema: None,
        };
        mgr.providers.register("plug", tool_spec).unwrap();
        mgr.providers.register("plug", plain_spec).unwrap();
        let ids = mgr.provider_tool_use_runtime_ids();
        assert_eq!(ids, vec!["plug:alpha".to_string()]);
    }

    // ---- compute_extension_load_hint --------------------------------

    #[test]
    fn hint_missing_binary_with_declared_setup_points_at_script() {
        let hint = compute_extension_load_hint(
            "Failed to spawn extension 'sample-sidecar': No such file or directory (os error 2)",
            std::path::Path::new("/home/u/.synaps-cli/plugins/sample-sidecar"),
            Some("scripts/setup.sh"),
        );
        assert!(
            hint.contains("Extension binary missing"),
            "missing-binary case should be flagged: {hint}"
        );
        assert!(
            hint.contains("/home/u/.synaps-cli/plugins/sample-sidecar"),
            "hint should include the plugin dir: {hint}"
        );
        assert!(
            hint.contains("setup=scripts/setup.sh"),
            "hint should show sanitized setup path without copy-paste shell command: {hint}"
        );
    }

    #[test]
    fn hint_missing_binary_without_declared_setup_falls_back_to_generic() {
        let hint = compute_extension_load_hint(
            "Failed to spawn extension 'foo': No such file or directory (os error 2)",
            std::path::Path::new("/x/y"),
            None,
        );
        assert!(
            hint.contains("plugin validate"),
            "no setup declared → generic hint: {hint}"
        );
        assert!(
            !hint.contains("Extension binary missing"),
            "should not falsely promise a setup script: {hint}"
        );
    }

    #[test]
    fn hint_other_error_with_declared_setup_falls_back_to_generic() {
        let hint = compute_extension_load_hint(
            "Extension 'foo' must subscribe to at least one hook or request a registration permission",
            std::path::Path::new("/x/y"),
            Some("scripts/setup.sh"),
        );
        // Setup script is declared, but the error is *not* a missing
        // binary — running the script wouldn't help. Fall back to the
        // generic hint so we don't mislead the user.
        assert!(hint.contains("plugin validate"), "got {hint}");
        assert!(!hint.contains("Extension binary missing"), "got {hint}");
    }

    #[test]
    fn hint_recognises_os_error_2_format() {
        // Older / cross-platform error formats may include the kernel
        // errno but not the "No such file or directory" English text.
        let hint = compute_extension_load_hint(
            "spawn failed (os error 2)",
            std::path::Path::new("/p"),
            Some("setup.sh"),
        );
        assert!(hint.contains("Extension binary missing"), "got {hint}");
    }

    // ── bugfix: installed-plugin backup discovery ───────────────────────────

    /// Minimal deferred tool-only plugin.json (Axel-shaped: `memory_fetch`
    /// takes a single `{id}`), written under `<dir>/.synaps-plugin/`.
    fn write_installed_plugin(plugins_root: &std::path::Path, dir_name: &str) {
        let plugin_dir = plugins_root.join(dir_name).join(".synaps-plugin");
        std::fs::create_dir_all(&plugin_dir).unwrap();
        let manifest = serde_json::json!({
            "extension": {
                "runtime": "process",
                "command": "/bin/false",
                "permissions": ["tools.register"],
                "deferred": {
                    "tools": [{
                        "name": "memory_fetch",
                        "description": "Fetch one memory by exact ID.",
                        "input_schema": {
                            "type": "object",
                            "properties": {"id": {"type": "string"}},
                            "required": ["id"]
                        }
                    }]
                }
            }
        });
        std::fs::write(
            plugin_dir.join("plugin.json"),
            serde_json::to_string_pretty(&manifest).unwrap(),
        )
        .unwrap();
    }

    /// Regression (user incident, session 20260720-141844-fb5a): `plugins/`
    /// held the installed `axel-memory-manager` AND timestamped copies
    /// (`axel-memory-manager.backup.<TS>`). Discovery treated every
    /// directory with a valid manifest as a live plugin, so the backups
    /// were loaded and their binaries executed alongside the real one.
    /// Backup/disabled/temp directory names must never be discovered: only
    /// active plugins load, backup tools are never cataloged, and (since a
    /// backup that is never loaded owns no lease) a backup process can
    /// never be spawned.
    #[tokio::test]
    #[serial_test::serial(synaps_base_dir)]
    async fn discover_and_load_skips_backup_and_disabled_plugin_directories() {
        let base = crate::test_env::BaseDirGuard::new();
        let _no_project_plugins =
            crate::test_env::EnvVarGuard::set("SYNAPS_DISABLE_PROJECT_PLUGINS", "1");
        let plugins_root = base.path().join("plugins");

        write_installed_plugin(&plugins_root, "axel-memory-manager");
        // The two timestamped backups from the incident, the updater's own
        // `.update-backup` convention, and common manual conventions.
        for inert in [
            "axel-memory-manager.backup.20260719210053",
            "axel-memory-manager.backup.20260720101416",
            "axel-memory-manager.update-backup",
            "axel-memory-manager.backup",
            "axel-memory-manager.bak",
            "axel-memory-manager.disabled",
            "axel-memory-manager.old",
            "axel-memory-manager.orig",
            "axel-memory-manager.tmp",
        ] {
            write_installed_plugin(&plugins_root, inert);
        }
        // Dotted names that are NOT backups stay discoverable.
        write_installed_plugin(&plugins_root, "com.example.notes");

        let registry = Arc::new(tokio::sync::RwLock::new(
            crate::ToolRegistry::without_subagent(),
        ));
        let mut mgr = ExtensionManager::new_with_tools(Arc::new(HookBus::new()), registry.clone());
        mgr.set_progressive_deferral(true);

        let (mut loaded, failed) = mgr.discover_and_load().await;
        loaded.sort();

        assert!(failed.is_empty(), "no failures expected: {failed:?}");
        assert_eq!(
            loaded,
            vec![
                "axel-memory-manager".to_string(),
                "com.example.notes".to_string()
            ],
            "backup/disabled/temp plugin directories must not be discovered"
        );

        let reg = registry.read().await;
        assert!(reg.get("axel-memory-manager:memory_fetch").is_some());
        assert!(reg.get("com.example.notes:memory_fetch").is_some());
        assert!(
            reg.get("axel-memory-manager.backup.20260719210053:memory_fetch")
                .is_none(),
            "backup plugin tools must never be cataloged"
        );
        assert!(reg
            .get("axel-memory-manager.backup.20260720101416:memory_fetch")
            .is_none());
        drop(reg);

        // Deferred lifecycle + no backup load ⇒ zero live extension
        // processes were started by discovery.
        assert_eq!(mgr.count(), 0, "discovery must spawn nothing");
    }

    /// The exclusion helper is name-based and conservative: it must catch
    /// known backup/disabled/temp conventions without swallowing
    /// legitimate dotted or hyphenated plugin names.
    #[test]
    fn inert_plugin_dir_name_filter_is_exact() {
        for inert in [
            "axel-memory-manager.backup.20260720101416",
            "axel-memory-manager.BACKUP.1",
            "axel-memory-manager.backup",
            "axel-memory-manager.update-backup",
            "axel-memory-manager.bak",
            "axel-memory-manager.disabled",
            "axel-memory-manager.old",
            "axel-memory-manager.orig",
            "axel-memory-manager.tmp",
            "axel-memory-manager.temp",
        ] {
            assert!(
                is_inert_plugin_dir_name(inert),
                "{inert} should be excluded from discovery"
            );
        }
        for live in [
            "axel-memory-manager",
            "com.example.notes",
            "backup-manager",
            "old-school-tools",
            "tmpfs-inspector",
            "my.backups.viewer",
        ] {
            assert!(
                !is_inert_plugin_dir_name(live),
                "{live} is a legitimate plugin name and must stay discoverable"
            );
        }
    }
}

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

    fn entry(key: &str, secret_env: &str) -> ExtensionConfigEntry {
        serde_json::from_value(serde_json::json!({
            "key": key,
            "type": "string",
            "secret_env": secret_env,
        }))
        .unwrap()
    }

    /// #207 MED-2: a manifest must not be able to name the host's own
    /// provider credential and have it handed over at initialize.
    #[test]
    fn host_provider_credentials_are_rejected() {
        for var in [
            "ANTHROPIC_API_KEY",
            "OPENAI_API_KEY",
            "AWS_SECRET_ACCESS_KEY",
            "anthropic_api_key", // case-insensitive
        ] {
            let err = ExtensionManager::resolve_config("evil", &[entry("k", var)], None)
                .expect_err(&format!("{var} must be rejected"));
            assert!(err.contains("host credential"), "unexpected error: {err}");
        }
    }

    /// The whole SYNAPS_* namespace is the host's, except this extension's
    /// own scoped override.
    #[test]
    fn host_namespace_is_rejected_but_own_scope_is_allowed() {
        let err = ExtensionManager::resolve_config("evil", &[entry("k", "SYNAPS_BROKER_TOKEN")], None)
            .expect_err("host namespace must be rejected");
        assert!(err.contains("host namespace"), "unexpected error: {err}");

        // an extension's own scoped var is fine
        assert!(
            ExtensionManager::denied_secret_env("SYNAPS_EXTENSION_MY_PLUGIN_TOKEN", "my-plugin")
                .is_none()
        );
    }

    /// Third-party secrets an extension legitimately needs still work.
    #[test]
    fn third_party_secrets_are_still_allowed() {
        for var in ["GITHUB_TOKEN", "STRIPE_KEY", "MY_SERVICE_TOKEN"] {
            assert!(
                ExtensionManager::denied_secret_env(var, "some-ext").is_none(),
                "{var} should be allowed"
            );
        }
    }

    #[test]
    fn empty_secret_env_is_rejected() {
        assert!(ExtensionManager::denied_secret_env("   ", "x").is_some());
    }
}