bamboo_engine/

session_repository.rs

//! Canonical session coordinator owned by the framework.
//!
//! [`SessionRepository`] bundles the three tiers a Bamboo session lives in — the
//! in-memory [`SessionCache`], the durable [`Storage`], and the
//! merge-on-write [`LockedSessionStore`] — and provides the one canonical
//! load/save coordination (cache → storage → backfill, and dual-write).
//!
//! This is a *framework* capability, not a server one: previously the
//! coordination lived only as inherent methods on the server's `AppState`,
//! which meant anything outside the HTTP server (the SDK, in-process embedders)
//! could not load or persist sessions consistently. `SessionRepository` lets any
//! caller that holds the three tiers share the exact same behaviour; the
//! server's `AppState` now delegates to it.

use std::sync::Arc;

use bamboo_agent_core::storage::Storage;
use bamboo_agent_core::Session;
use bamboo_domain::RetrievalWindowCheckpointOutcome;
use bamboo_storage::LockedSessionStore;

use crate::{read_cached_session, SessionCache};

#[path = "session_repository_root_inbox.rs"]
mod root_inbox;

#[path = "session_repository_parent_outcome.rs"]
mod parent_outcome;
pub use parent_outcome::{ParentOutcomeRoute, ParentOutcomeWriter};

#[cfg(test)]
#[path = "session_repository_admission_tests.rs"]
mod admission_tests;

#[cfg(test)]
type PostDurableHook = Arc<dyn Fn(&str, &str) + Send + Sync>;

/// Framework-owned coordinator over a session's cache / storage / persistence
/// tiers. Cheap to clone (all fields are `Arc`).
#[derive(Clone)]
pub struct SessionRepository {
    cache: SessionCache,
    storage: Arc<dyn Storage>,
    persistence: Arc<LockedSessionStore>,
    root_actor_directory: Option<Arc<dyn bamboo_domain::ActorDirectoryPort>>,
    root_actor_owner: Option<(
        Arc<dyn bamboo_domain::ActorDirectoryPort>,
        Arc<bamboo_domain::RootActorRuntimeWrite>,
    )>,
    root_inbox_receipt: Option<bamboo_domain::SessionInboxReceipt>,
    root_actor_writers:
        Arc<dashmap::DashMap<String, std::sync::Weak<bamboo_domain::RootActorRuntimeWrite>>>,
    // Always present in production builds too; only focused tests set it via
    // the #[cfg(test)] builder below.
    root_lease_override: Option<std::time::Duration>,
    #[cfg(test)]
    post_durable_hook: Option<PostDurableHook>,
}

impl SessionRepository {
    /// Publish a committed Child repair while its Storage guards are held.
    /// A cache belonging to another birth/run/launch/input is left untouched;
    /// this observation grants neither execution nor parent delivery authority.
    pub fn publish_reconciled_child(cache: &SessionCache, expected: &Session, saved: &Session) {
        fn runtime_id(session: &Session) -> Option<Option<String>> {
            let mirror = session
                .metadata
                .get("agent.runtime.state")
                .map(|raw| serde_json::from_str::<bamboo_domain::AgentRuntimeState>(raw))
                .transpose()
                .ok()?;
            let typed = session.agent_runtime_state.as_ref();
            if typed
                .zip(mirror.as_ref())
                .is_some_and(|(a, b)| a.run_id != b.run_id)
            {
                return None;
            }
            Some(typed.or(mirror.as_ref()).map(|s| s.run_id.clone()))
        }
        fn inputs(session: &Session) -> Option<serde_json::Value> {
            serde_json::to_value(
                session
                    .messages
                    .iter()
                    .filter(|m| m.role == bamboo_domain::Role::User)
                    .collect::<Vec<_>>(),
            )
            .ok()
        }
        if saved.kind != bamboo_domain::SessionKind::Child {
            return;
        }
        if let Some(entry) = cache.get(&saved.id) {
            // SessionSnapshot::update retries against a concurrent publication.
            // Leave a vacant cache vacant: loading canonical Storage remains
            // correct, while a check-then-insert could replace a new birth.
            entry.update(|current| {
                if current.id != saved.id
                    || current.kind != saved.kind
                    || current.root_session_id != saved.root_session_id
                    || current.authority_identity != saved.authority_identity
                    || !current.execution_compatibility().execution_allowed
                    || current.created_at != saved.created_at
                    || current.parent_created_at != saved.parent_created_at
                    || current.parent_session_id != saved.parent_session_id
                    || current.project_id_meta() != saved.project_id_meta()
                    || current.host_run_id() != saved.host_run_id()
                    || current.child_launch_generation() != saved.child_launch_generation()
                    || current.session_inbox_admission() != saved.session_inbox_admission()
                    || current.check_conversation_generation(saved).is_err()
                    || runtime_id(current).is_none()
                    || runtime_id(current) != runtime_id(expected)
                    || inputs(current).is_none()
                    || inputs(current) != inputs(expected)
                    || current.has_runtime_wait()
                    || (crate::runtime::runner::state_bridge::read_runtime_state(current)
                        .is_some_and(|s| s.status == bamboo_domain::AgentStatusState::Idle)
                        && !crate::runtime::runner::state_bridge::read_runtime_state(expected)
                            .is_some_and(|s| s.status == bamboo_domain::AgentStatusState::Idle))
                    || current.last_run_status().as_deref() == Some("pending")
                    || current.metadata.get("runtime.child_completion_source_v1")
                        != expected.metadata.get("runtime.child_completion_source_v1")
                {
                    return;
                }
                // Only the actual repair fields change. Preserve newer UI,
                // transcript, human engagement and accounting observations.
                if let Some(status) = saved.last_run_status() {
                    current.set_last_run_status(&status);
                }
                match saved.last_run_error() {
                    Some(error) => current.set_last_run_error(error),
                    None => current.clear_last_run_error(),
                }
                if let Some(final_state) = saved.agent_runtime_state.as_ref() {
                    if current.agent_runtime_state.is_none() {
                        current.agent_runtime_state =
                            crate::runtime::runner::state_bridge::read_runtime_state(current);
                    }
                    let state = current
                        .agent_runtime_state
                        .get_or_insert_with(|| final_state.clone());
                    state.status = final_state.status;
                    state.tools.active_tool_calls = final_state.tools.active_tool_calls;
                    if let Ok(raw) = serde_json::to_string(state) {
                        current.metadata.insert("agent.runtime.state".into(), raw);
                    }
                }
                for key in [
                    "runtime.reconciled_activation_fence",
                    "runtime.reconciled_terminal_generation",
                ] {
                    if let Some(marker) = saved.metadata.get(key) {
                        current.metadata.insert(key.into(), marker.clone());
                    }
                }
            });
        }
    }

    pub fn new(
        cache: SessionCache,
        storage: Arc<dyn Storage>,
        persistence: Arc<LockedSessionStore>,
    ) -> Self {
        Self {
            cache,
            storage,
            persistence,
            root_actor_directory: None,
            root_actor_owner: None,
            root_inbox_receipt: None,
            root_actor_writers: Arc::new(Default::default()),
            root_lease_override: None,
            #[cfg(test)]
            post_durable_hook: None,
        }
    }

    async fn bind_root_actor(
        &self,
        session: &Session,
        run_id: &str,
        for_compaction: bool,
    ) -> std::io::Result<Option<bamboo_domain::RootActorExecutionBinding>> {
        let repository = self.clone();
        let session = session.clone();
        let run_id = run_id.to_string();
        // The complete claim/start handoff owns its state if the adapter stops
        // waiting. An unobserved returned binding drops its exact-fence cleanup.
        tokio::spawn(async move {
            repository
                .bind_root_actor_inner(&session, &run_id, for_compaction)
                .await
        })
        .await
        .map_err(|error| root_actor_binding_error(format!("Root binding task failed: {error}")))?
    }

    async fn bind_root_actor_inner(
        &self,
        session: &Session,
        run_id: &str,
        for_compaction: bool,
    ) -> std::io::Result<Option<bamboo_domain::RootActorExecutionBinding>> {
        if session.kind != bamboo_domain::SessionKind::Root
            || !session.authority_identity.is_ordinary()
        {
            return Ok(None);
        }
        let Some(directory) = self.root_actor_directory.clone() else {
            return Ok(None);
        };
        if !self.storage.supports_root_actor_runtime_write() {
            return Err(std::io::Error::new(
                std::io::ErrorKind::Unsupported,
                "Root Actor execution requires a fenced storage writer",
            ));
        }
        let Some(durable) = self.storage.load_root_authority(&session.id).await? else {
            return Err(root_actor_binding_error(
                "Root must be persisted before activation",
            ));
        };
        if session.created_at != durable.created_at
            || session.id != durable.id
            || session.project_id_meta() != durable.project_id_meta()
            || session.authority_identity != durable.authority_identity
        {
            return Err(root_actor_binding_error(
                "Root identity changed before activation",
            ));
        }
        session
            .check_conversation_generation(&durable)
            .map_err(root_actor_binding_error)?;
        // A longer Root lease absorbs slow durable writes (multi-MB session
        // transcripts serialized under the actor transaction lock) so a busy
        // host cannot expire a live turn between renewal ticks. Renewal still
        // runs every lease/3, and a genuine takeover keeps rejecting the old
        // fence. Focused tests may shorten this through the test-only override.
        let lease_duration = self
            .root_lease_override
            .unwrap_or_else(|| std::time::Duration::from_secs(60));
        let lease_seconds = chrono::Duration::from_std(lease_duration).expect("bounded Root lease");
        let now = chrono::Utc::now();
        let claimed = directory
            .claim_root_activation_observed(
                &bamboo_domain::ActorActivationClaim {
                    actor_id: session.id.clone(),
                    run_id: run_id.to_string(),
                    lease_owner: if for_compaction {
                        bamboo_domain::new_root_compaction_lease_owner()
                    } else {
                        format!("root-execution-{}", uuid::Uuid::new_v4())
                    },
                    lease_expires_at: now + lease_seconds,
                    inbox_generation: session
                        .session_inbox_admission()
                        .map_or(0, |a| a.last_admitted_sequence),
                    placement_ref: None,
                    now,
                },
                session,
            )
            .await
            .map_err(root_actor_binding_error)?;
        let owner = bamboo_domain::RootActorRuntimeWrite {
            fence: claimed.fence(),
            created_at: durable.created_at,
            conversation_generation: session.conversation_generation_observation(),
        };
        let bound = self.bind_root_response_writer(owner.clone())?;
        let live_owner = bound
            .root_actor_owner
            .as_ref()
            .expect("bound Root owner")
            .1
            .clone();
        let abandon_directory = directory.clone();
        let abandon_owner = owner.clone();
        let terminal_cache = self.cache.clone();
        let terminal_publisher: bamboo_domain::storage::RootActorRuntimePublisher =
            Arc::new(move |saved| {
                terminal_cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            });
        let abandon_publisher = terminal_publisher.clone();
        let binding = bamboo_domain::RootActorExecutionBinding::new(
            Arc::new(bound),
            directory.clone(),
            owner,
            lease_duration,
            move || {
                if let Ok(handle) = tokio::runtime::Handle::try_current() {
                    handle.spawn(async move {
                        let _ = abandon_directory
                            .finish_root_activation_observed(
                                &abandon_owner,
                                chrono::Utc::now(),
                                bamboo_domain::ActorActivationFinish::Cancelled,
                                Some(abandon_publisher),
                            )
                            .await;
                    });
                }
            },
        )
        .with_terminal_publisher(terminal_publisher);
        directory
            .start_activation(&binding.owner.fence, chrono::Utc::now())
            .await
            .map_err(root_actor_binding_error)?;
        // Keep only a weak route to this exact execution's capability. Host
        // parent-outcome adapters never reconstruct an owner from observation.
        if !for_compaction {
            self.root_actor_writers
                .insert(session.id.clone(), Arc::downgrade(&live_owner));
        }
        Ok(Some(binding))
    }

    /// Build a response coordinator from the exact reserved execution's
    /// capability. This does not claim or observe a different Actor owner.
    pub(crate) fn bind_root_response_writer(
        &self,
        owner: bamboo_domain::RootActorRuntimeWrite,
    ) -> std::io::Result<Self> {
        let directory = self.root_actor_directory.clone().ok_or_else(|| {
            root_actor_binding_error("Root response requires its Host Actor directory")
        })?;
        let host_cache = self.cache.clone();
        let publisher: bamboo_domain::RootActorRuntimePublisher = Arc::new(move |saved| {
            host_cache.insert(
                saved.id.clone(),
                Arc::new(crate::SessionSnapshot::new(saved.clone())),
            );
        });
        let bound_store = self
            .persistence
            .bind_root_actor_writer(owner.clone(), publisher)?;
        // Deferred repository callbacks can only update this execution-private
        // cache. The shared Host cache is published inside V2's physical guard.
        let mut bound = Self::new(
            Arc::new(Default::default()),
            self.storage.clone(),
            Arc::new(bound_store),
        );
        bound.root_actor_owner = Some((directory.clone(), Arc::new(owner)));
        bound.root_actor_writers = self.root_actor_writers.clone();
        Ok(bound)
    }

    /// Preserve the exact inherited wait carried by the reserved successor.
    pub(crate) fn bind_root_response_writer_with_inherited(
        &self,
        owner: bamboo_domain::RootActorRuntimeWrite,
        inherited: Option<bamboo_domain::InheritedChildWait>,
    ) -> std::io::Result<Self> {
        let mut bound = self.bind_root_response_writer(owner)?;
        if let Some(inherited) = inherited {
            bound.persistence = Arc::new(bound.persistence.bind_inherited_child_wait(inherited));
        }
        Ok(bound)
    }

    /// Enable durable ordinary Root execution ownership in a Host whose
    /// storage implements the named final writer boundary.
    pub fn with_root_actor_directory(
        mut self,
        directory: Arc<dyn bamboo_domain::ActorDirectoryPort>,
    ) -> Self {
        self.root_actor_directory = Some(directory);
        self
    }

    #[cfg(test)]
    fn with_post_durable_hook(mut self, hook: PostDurableHook) -> Self {
        self.post_durable_hook = Some(hook);
        self
    }

    /// Test-only shortening of the Root activation lease. Focused port tests
    /// observe expiry-driven fencing without waiting out the production lease.
    #[cfg(test)]
    pub(crate) fn with_root_lease_override(mut self, duration: std::time::Duration) -> Self {
        self.root_lease_override = Some(duration);
        self
    }

    #[cfg(test)]
    fn run_post_durable_hook(&self, operation: &str, marker: &str) {
        if let Some(hook) = self.post_durable_hook.as_ref() {
            hook(operation, marker);
        }
    }

    pub fn cache(&self) -> &SessionCache {
        &self.cache
    }

    pub fn storage(&self) -> &Arc<dyn Storage> {
        &self.storage
    }

    pub fn persistence(&self) -> &Arc<LockedSessionStore> {
        &self.persistence
    }

    /// Load a session from the memory cache, falling back to durable storage
    /// (and back-filling the cache on a storage hit). `None` if absent in both.
    pub async fn load(&self, session_id: &str) -> Option<Session> {
        if let Some(session) = read_cached_session(&self.cache, session_id) {
            return Some(session);
        }

        let _guard = self.persistence.acquire_lock(session_id).await;
        if let Some(session) = read_cached_session(&self.cache, session_id) {
            return Some(session);
        }

        let loaded = self.storage.load_session(session_id).await;
        #[cfg(test)]
        self.run_post_durable_hook("load", session_id);
        match loaded {
            Ok(Some(session)) => {
                self.cache.insert(
                    session_id.to_string(),
                    Arc::new(crate::SessionSnapshot::new(session.clone())),
                );
                Some(session)
            }
            _ => None,
        }
    }

    /// Like [`load`](Self::load), but surfaces storage errors instead of
    /// swallowing them to `None`. Cache hit short-circuits; a storage hit
    /// back-fills the cache.
    pub async fn try_load(&self, session_id: &str) -> std::io::Result<Option<Session>> {
        if let Some(session) = read_cached_session(&self.cache, session_id) {
            return Ok(Some(session));
        }

        let _guard = self.persistence.acquire_lock(session_id).await;
        if let Some(session) = read_cached_session(&self.cache, session_id) {
            return Ok(Some(session));
        }

        let loaded = self.storage.load_session(session_id).await?;
        #[cfg(test)]
        self.run_post_durable_hook("try_load", session_id);
        if let Some(ref session) = loaded {
            self.cache.insert(
                session_id.to_string(),
                Arc::new(crate::SessionSnapshot::new(session.clone())),
            );
        }
        Ok(loaded)
    }

    /// Persist the session (merge-on-write) and refresh the cache, surfacing
    /// storage errors. Use [`save_and_cache`](Self::save_and_cache) for the
    /// fire-and-forget variant that logs and continues on failure.
    pub async fn save(&self, session: &mut Session) -> std::io::Result<()> {
        self.persistence
            .merge_save_runtime_and_publish(session, |saved, committed| {
                if committed {
                    #[cfg(test)]
                    self.run_post_durable_hook("save_full", &saved.id);
                    self.cache.insert(
                        saved.id.clone(),
                        Arc::new(crate::SessionSnapshot::new(saved.clone())),
                    );
                }
            })
            .await
    }

    /// Persist a blocked approval replay before starting its successor. A
    /// A save error may arrive after the rewritten result was committed. When
    /// the save was attempted, evict the old approval from cache under the
    /// same session lock so the next attempt reloads durable state.
    pub async fn save_replay_resolution(&self, session: &mut Session) -> std::io::Result<()> {
        self.persistence
            .merge_save_runtime_and_publish(session, |saved, committed| {
                if committed {
                    self.cache.insert(
                        saved.id.clone(),
                        Arc::new(crate::SessionSnapshot::new(saved.clone())),
                    );
                } else {
                    self.cache.remove(&saved.id);
                }
            })
            .await
    }

    /// Atomically mutate the latest durable runtime session and refresh the
    /// cache with the saved value. This is the safe path for narrow metadata
    /// indexes that can be updated concurrently with runner message writes.
    pub async fn update_runtime_session<F>(
        &self,
        session_id: &str,
        metadata_keys: &[&str],
        mutate: F,
    ) -> std::io::Result<Option<Session>>
    where
        F: FnOnce(&mut Session),
    {
        self.persistence
            .update_runtime_config_and_publish(session_id, mutate, |saved| {
                if let Some(cached) = self.cache.get(session_id) {
                    cached.update(|cached| {
                        for key in metadata_keys {
                            if let Some(value) = saved.metadata.get(*key) {
                                cached.metadata.insert((*key).to_string(), value.clone());
                            } else {
                                cached.metadata.remove(*key);
                            }
                        }
                    });
                }
            })
            .await
    }

    /// Load a session, creating a fresh `Session::new(id, model)` if absent.
    pub async fn load_or_create(&self, session_id: &str, model: &str) -> Session {
        if let Some(session) = self.load(session_id).await {
            return session;
        }
        Session::new(session_id.to_string(), model.to_string())
    }

    /// Load a session, reconciling the memory and storage copies via a
    /// preference heuristic: a live durable Root birth or mode epoch wins first;
    /// otherwise storage wins when it is strictly newer, or when it is the same
    /// age but still carries a pending question memory lost.
    ///
    /// The cache is refreshed cache-aside. Timestamps cannot make a stale
    /// Root birth or mode epoch outrank verified durable authority.
    pub async fn load_merged_checked(&self, session_id: &str) -> std::io::Result<Option<Session>> {
        let _guard = self.persistence.acquire_lock(session_id).await;
        let memory_session = read_cached_session(&self.cache, session_id);
        let storage_session = self.storage.load_session(session_id).await?;
        #[cfg(test)]
        self.run_post_durable_hook("load_merged", session_id);

        Ok(match (memory_session, storage_session) {
            (Some(memory), Some(storage)) => {
                let root_authority_preference =
                    root_authority_storage_preference(&memory, &storage);
                let prefer_storage = should_prefer_storage(&memory, &storage);
                let diverged = prefer_storage || memory.messages.len() != storage.messages.len();
                let chosen_len = if prefer_storage {
                    storage.messages.len()
                } else {
                    memory.messages.len()
                };
                macro_rules! merged_log {
                    ($level:ident) => {
                        tracing::$level!(
                            "[{}] load_session_merged: memory={} msgs (updated_at={}), storage={} msgs (updated_at={}), prefer_storage={} -> chose {} msgs",
                            session_id,
                            memory.messages.len(),
                            memory.updated_at,
                            storage.messages.len(),
                            storage.updated_at,
                            prefer_storage,
                            chosen_len,
                        )
                    };
                }
                if diverged {
                    merged_log!(debug);
                } else {
                    merged_log!(trace);
                }
                let memory_updated_at = memory.updated_at;
                let chosen = if prefer_storage { storage } else { memory };
                // Cache-aside refresh with a hard no-regression invariant: only
                // write back when we actually reconciled *to storage* (a memory
                // win is already the cached copy; re-inserting it would needlessly
                // replace a possibly-live Arc) AND the reconciled copy is not
                // older than what memory already holds, except when durable
                // storage proved a new Root birth or higher mode epoch. Those
                // authority changes must survive cross-process clock skew.
                if prefer_storage
                    && (root_authority_preference == Some(true)
                        || chosen.updated_at >= memory_updated_at)
                {
                    self.cache.insert(
                        session_id.to_string(),
                        Arc::new(crate::SessionSnapshot::new(chosen.clone())),
                    );
                }
                Some(chosen)
            }
            (Some(memory), None) => Some(memory),
            (None, Some(storage)) => {
                self.cache.insert(
                    session_id.to_string(),
                    Arc::new(crate::SessionSnapshot::new(storage.clone())),
                );
                Some(storage)
            }
            (None, None) => None,
        })
    }

    /// Compatibility wrapper for read paths where the historical contract
    /// treated a storage failure like absence. Mutating/recovery paths should
    /// use [`Self::load_merged_checked`] so they can preserve retry state.
    pub async fn load_merged(&self, session_id: &str) -> Option<Session> {
        match self.load_merged_checked(session_id).await {
            Ok(session) => session,
            Err(error) => {
                tracing::warn!(
                    "[{}] Failed to load merged session from storage: {}",
                    session_id,
                    error
                );
                read_cached_session(&self.cache, session_id)
            }
        }
    }

    /// Persist the session (merge-on-write, preserving concurrent UI edits to
    /// the authoritative metadata group) and refresh the in-memory cache.
    pub async fn save_and_cache(&self, session: &mut Session) {
        let result = self
            .persistence
            .merge_save_runtime_and_publish(session, |saved, _| {
                #[cfg(test)]
                self.run_post_durable_hook("save_and_cache", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await;
        if let Err(error) = result {
            tracing::warn!("[{}] Failed to save session: {}", session.id, error);
        }
    }

    async fn refresh_cached_task_control_plane(&self, session_id: &str) {
        match self.storage.load_runtime_control_plane(session_id).await {
            Ok(Some(durable)) => {
                if let Some(cached) = self.cache.get(session_id) {
                    cached.update(|cached| adopt_task_control_plane(cached, &durable));
                }
            }
            Ok(None) => {}
            Err(error) => tracing::warn!(
                "[{}] Failed to refresh Task control-plane after write conflict: {}",
                session_id,
                error
            ),
        }
    }
}

fn adopt_task_control_plane(target: &mut Session, durable: &Session) {
    target.task_list = durable.task_list.clone();
    target
        .metadata
        .remove(bamboo_domain::session::runtime_metadata::keys::TASK_LIST_VERSION);
    if let Some(runtime_metadata) = target.runtime_metadata.as_mut() {
        runtime_metadata.task_list_version = None;
    }
    if target
        .runtime_metadata
        .as_ref()
        .is_some_and(bamboo_domain::session::SessionRuntimeMetadata::is_empty)
    {
        target.runtime_metadata = None;
    }
    if let Some(version) = durable.task_list_version_meta() {
        target.set_task_list_version_meta(version);
    }
}

fn should_prefer_storage(memory_session: &Session, storage_session: &Session) -> bool {
    if let Some(prefer_storage) = root_authority_storage_preference(memory_session, storage_session)
    {
        return prefer_storage;
    }
    // Never reconcile *backwards* to a strictly-older storage copy: if memory is
    // newer it is authoritative (e.g. it just answered and cleared a pending
    // question while storage still holds the stale one). Respecting `updated_at`
    // here is what stops `load_merged` from returning — and caching — stale data.
    if storage_session.updated_at < memory_session.updated_at {
        return false;
    }
    // Storage is same-age or newer: prefer it when strictly newer, or when it
    // still carries a pending question that the (same-age) memory copy lost, so
    // a genuine clarification is never dropped.
    storage_session.updated_at > memory_session.updated_at
        || (memory_session.pending_question.is_none() && storage_session.pending_question.is_some())
}

/// `load_session` has already checked that the storage copy is live. Its Root
/// birth therefore outranks any cached incarnation with the same ID, even if
/// the new lifetime reset the mode epoch or a process clock moved backwards.
/// Within one birth, only a higher durable epoch is an authority advance.
fn root_authority_storage_preference(
    memory_session: &Session,
    storage_session: &Session,
) -> Option<bool> {
    use bamboo_domain::SessionKind;

    if memory_session.kind != SessionKind::Root || storage_session.kind != SessionKind::Root {
        return None;
    }
    if memory_session.created_at != storage_session.created_at
        || memory_session.authority_identity != storage_session.authority_identity
    {
        return Some(true);
    }
    (memory_session.root_mode_transition_epoch != storage_session.root_mode_transition_epoch)
        .then_some(
            storage_session.root_mode_transition_epoch > memory_session.root_mode_transition_epoch,
        )
}

/// `SessionRepository` is the canonical `RuntimeSessionPersistence`: the runtime
/// can persist a session through the same coordinator (merge-on-write + cache
/// refresh) instead of a bespoke adapter.
#[async_trait::async_trait]
impl bamboo_domain::RuntimeSessionPersistence for SessionRepository {
    fn inherited_child_wait(&self) -> Option<bamboo_domain::InheritedChildWait> {
        bamboo_domain::RuntimeSessionPersistence::inherited_child_wait(self.persistence.as_ref())
    }

    fn bind_inherited_child_wait(
        &self,
        inherited: bamboo_domain::InheritedChildWait,
    ) -> std::io::Result<Arc<dyn bamboo_domain::RuntimeSessionPersistence>> {
        let mut bound = self.clone();
        bound.persistence = Arc::new(
            self.persistence
                .as_ref()
                .bind_inherited_child_wait(inherited),
        );
        Ok(Arc::new(bound))
    }

    fn root_actor_writer(&self) -> Option<bamboo_domain::RootActorRuntimeWrite> {
        self.root_actor_owner
            .as_ref()
            .map(|(_, owner)| (**owner).clone())
    }

    fn bind_root_inbox_receipt(
        &self,
        receipt: bamboo_domain::SessionInboxReceipt,
    ) -> std::io::Result<Arc<dyn bamboo_domain::RuntimeSessionPersistence>> {
        if self.root_actor_owner.is_none()
            || receipt.generation == 0
            || self
                .root_inbox_receipt
                .as_ref()
                .is_some_and(|current| current != &receipt)
        {
            return Err(root_actor_binding_error(
                "Root receipt requires its unchanged execution writer",
            ));
        }
        let mut bound = self.clone();
        bound.root_inbox_receipt = Some(receipt);
        Ok(Arc::new(bound))
    }

    fn root_inbox_receipt(&self) -> Option<&bamboo_domain::SessionInboxReceipt> {
        self.root_inbox_receipt.as_ref()
    }

    fn root_actor_execution_required(&self, session: &Session) -> bool {
        self.root_actor_directory.is_some()
            && session.kind == bamboo_domain::SessionKind::Root
            && session.authority_identity.is_ordinary()
    }

    async fn bind_root_actor_execution(
        &self,
        session: &Session,
        run_id: &str,
    ) -> std::io::Result<Option<bamboo_domain::RootActorExecutionBinding>> {
        self.bind_root_actor(session, run_id, false).await
    }

    async fn bind_root_actor_compaction(
        &self,
        session: &Session,
        run_id: &str,
    ) -> std::io::Result<Option<bamboo_domain::RootActorExecutionBinding>> {
        self.bind_root_actor(session, run_id, true).await
    }

    async fn save_runtime_session(&self, session: &mut Session) -> std::io::Result<()> {
        // Runtime authorization reads through this same cache. Refresh it even
        // when durable storage fails so a current activation can never observe
        // a previous run's skill allowlist. The error is still returned to the
        // caller and durable state remains unchanged.
        self.persistence
            .merge_save_runtime_and_publish(session, |saved, _| {
                #[cfg(test)]
                self.run_post_durable_hook("save_runtime_session", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    fn supports_bound_startup_session(&self) -> bool {
        bamboo_domain::RuntimeSessionPersistence::supports_bound_startup_session(
            self.persistence.as_ref(),
        )
    }

    async fn save_bound_startup_session(
        &self,
        session: &mut Session,
        expected: &Session,
    ) -> std::io::Result<()> {
        self.persistence
            .merge_save_bound_startup_and_publish(session, expected, |saved, _| {
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn save_finalized_runtime_session(&self, session: &mut Session) -> std::io::Result<()> {
        self.persistence
            .merge_save_finalized_runtime_and_publish(session, |saved, _| {
                #[cfg(test)]
                self.run_post_durable_hook("save_finalized_runtime_session", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn save_finalized_runtime_with_inherited_child_wait(
        &self,
        session: &mut Session,
        inherited: &bamboo_domain::session::runtime_state::WaitingForChildrenState,
    ) -> std::io::Result<()> {
        self.persistence
            .merge_save_inherited_child_wait_and_publish(session, inherited, |saved, _| {
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn seed_runtime_activation(&self, session: &mut Session) -> std::io::Result<()> {
        self.persistence
            .seed_runtime_activation_and_publish(session, |saved, committed| {
                #[cfg(test)]
                self.run_post_durable_hook("seed_runtime_activation", &saved.id);
                if committed {
                    self.cache.insert(
                        saved.id.clone(),
                        Arc::new(crate::SessionSnapshot::new(saved.clone())),
                    );
                }
            })
            .await
    }

    async fn record_permission_posture_activation(
        &self,
        session_id: &str,
        expected_audit_revision: Option<u64>,
        seed: &bamboo_domain::PermissionAuditSeed,
    ) -> std::io::Result<Option<Session>> {
        self.persistence
            .record_permission_posture_activation_and_publish(
                session_id,
                expected_audit_revision,
                seed,
                |saved| {
                    #[cfg(test)]
                    self.run_post_durable_hook("permission_posture_activation", &saved.id);
                    self.cache.insert(
                        saved.id.clone(),
                        Arc::new(crate::SessionSnapshot::new(saved.clone())),
                    );
                },
            )
            .await
    }

    async fn save_runtime_control_plane(&self, session: &mut Session) -> std::io::Result<()> {
        self.persistence
            .save_runtime_only_and_publish(session, |saved| {
                #[cfg(test)]
                self.run_post_durable_hook("save", &saved.id);

                // A control-plane snapshot may intentionally carry no messages
                // (for example, child Task synchronization loads the root's
                // runtime sidecar). Publish its fresh runtime fields without
                // replacing a cache-resident transcript with that empty
                // snapshot. SessionInbox admission is coupled to transcript
                // persistence and is therefore preserved alongside the cached
                // messages, matching the V2 sidecar overlay contract.
                if let Some(cached) = self.cache.get(&saved.id) {
                    cached.update(|cached| merge_runtime_control_plane_cache(cached, saved));
                }
            })
            .await
    }

    async fn load_runtime_control_plane(
        &self,
        session_id: &str,
    ) -> std::io::Result<Option<Session>> {
        bamboo_domain::RuntimeSessionPersistence::load_runtime_control_plane(
            self.persistence.as_ref(),
            session_id,
        )
        .await
    }

    async fn update_task_list_control_plane(
        &self,
        session_id: &str,
        task_list: &bamboo_domain::TaskList,
        version: &str,
    ) -> std::io::Result<bool> {
        let result = self
            .persistence
            .update_task_list_control_plane_and_publish(session_id, task_list, version, |_| {
                #[cfg(test)]
                self.run_post_durable_hook("task", version);

                // The durable transaction changed only Task-owned fields.
                // Mirror that same narrow patch into the cache so a
                // concurrent round/status/child transition already present
                // in memory cannot be replaced by a stale whole-control-
                // plane snapshot.
                if let Some(cached) = self.cache.get(session_id) {
                    cached.update(|cached| {
                        cached.set_task_list(task_list.clone());
                        cached.set_task_list_version_meta(version.to_string());
                    });
                }
            })
            .await;
        if result
            .as_ref()
            .is_err_and(|error| error.kind() == std::io::ErrorKind::WouldBlock)
        {
            // The storage final-CAS rejected this unconditional patch because
            // an independent writer won. Refresh the winner into the cache but
            // preserve the conflict result so Taskwrite never mistakes it for
            // the Ok(false) legacy-persistence fallback signal.
            self.refresh_cached_task_control_plane(session_id).await;
        }
        result
    }

    async fn update_task_list_control_plane_if_version(
        &self,
        session_id: &str,
        expected_version: &str,
        expected_task_list: &bamboo_domain::TaskList,
        task_list: &bamboo_domain::TaskList,
        version: &str,
    ) -> std::io::Result<bool> {
        self.persistence
            .update_task_list_control_plane_if_version_and_publish(
                session_id,
                expected_version,
                expected_task_list,
                task_list,
                version,
                |_| {
                    if let Some(cached) = self.cache.get(session_id) {
                        cached.update(|cached| {
                            cached.set_task_list(task_list.clone());
                            cached.set_task_list_version_meta(version.to_string());
                        });
                    }
                },
            )
            .await
    }

    async fn update_task_list_control_planes_if_version(
        &self,
        session_id: &str,
        shared_session_id: &str,
        expected_version: &str,
        expected_task_list: &bamboo_domain::TaskList,
        task_list: &bamboo_domain::TaskList,
        version: &str,
    ) -> std::io::Result<bool> {
        self.persistence
            .update_task_list_control_planes_if_version_and_publish(
                session_id,
                shared_session_id,
                expected_version,
                expected_task_list,
                task_list,
                version,
                |_, _| {
                    for id in [session_id, shared_session_id] {
                        if let Some(cached) = self.cache.get(id) {
                            cached.update(|cached| {
                                cached.set_task_list(task_list.clone());
                                cached.set_task_list_version_meta(version.to_string());
                            });
                        }
                    }
                },
            )
            .await
    }

    async fn checkpoint_live_bootstrap(
        &self,
        session: &Session,
        inbox: Arc<dyn bamboo_domain::SessionInboxPort>,
        gate: bamboo_domain::AdmissionGate,
    ) -> std::io::Result<bamboo_domain::LiveBootstrapProof> {
        self.persistence
            .checkpoint_live_bootstrap(session, inbox, gate)
            .await
    }

    async fn confirm_live_bootstrap(
        &self,
        proof: &bamboo_domain::LiveBootstrapProof,
        inbox: Arc<dyn bamboo_domain::SessionInboxPort>,
        publish: bamboo_domain::RootActorRuntimeEventPublisher,
    ) -> std::io::Result<()> {
        self.persistence
            .confirm_live_bootstrap(proof, inbox, publish)
            .await
    }

    async fn checkpoint_runtime_session(&self, session: &mut Session) -> std::io::Result<()> {
        // The execute-boundary checkpoint uses LockedSessionStore's atomic
        // append-safe transcript merge, then publishes that reconciled snapshot
        // to the runtime cache.  On failure, leave the existing cache alone: the
        // checkpoint may have failed to load the latest durable transcript, and
        // replacing a fresher cache entry with the stale runner snapshot would
        // set up a later SHRINK write.
        self.persistence
            .checkpoint_runtime_session_and_publish(session, |saved, committed| {
                #[cfg(test)]
                self.run_post_durable_hook("checkpoint", &saved.id);
                if committed {
                    self.cache.insert(
                        saved.id.clone(),
                        Arc::new(crate::SessionSnapshot::new(saved.clone())),
                    );
                }
            })
            .await
    }

    async fn claim_scoped_loop_controls(
        &self,
        session: &Session,
        inbox: Arc<dyn bamboo_domain::SessionInboxPort>,
        active_run_id: &str,
        limit: usize,
    ) -> std::io::Result<Vec<bamboo_domain::ScopedLoopControlClaim>> {
        use bamboo_domain::ScopedLoopControlClaim;
        if let Some((directory, owner)) = self.root_actor_owner.as_ref() {
            let inbox = self.storage.bind_root_actor_inbox(owner, inbox)?;
            let consumer = bamboo_domain::SessionInboxConsumerId::new();
            let request = root_inbox::lease_request(directory.as_ref(), owner, &consumer).await?;
            return inbox
                .claim_owned_scoped_loop_controls(&session.id, limit, Some(active_run_id), &request)
                .await
                .map(|claims| {
                    claims
                        .into_iter()
                        .map(ScopedLoopControlClaim::Owned)
                        .collect()
                })
                .map_err(std::io::Error::other);
        }
        inbox
            .claim_scoped_loop_controls(&session.id, limit, Some(active_run_id))
            .await
            .map(|claims| {
                claims
                    .into_iter()
                    .map(ScopedLoopControlClaim::Legacy)
                    .collect()
            })
            .map_err(std::io::Error::other)
    }

    async fn checkpoint_scoped_loop_control(
        &self,
        session: &mut Session,
        bindings: &[bamboo_domain::ScopedLoopControlBinding],
        activation: Arc<dyn bamboo_domain::SessionActivationPort>,
        input: Option<(
            Arc<dyn bamboo_domain::SessionInboxPort>,
            bamboo_domain::ScopedLoopControlClaim,
        )>,
    ) -> std::io::Result<()> {
        let _guard = self.persistence.acquire_lock(&session.id).await;
        let cache = self.cache.clone();
        let publish: bamboo_domain::RootActorRuntimePublisher = Arc::new(move |saved| {
            cache.insert(
                saved.id.clone(),
                Arc::new(crate::SessionSnapshot::new(saved.clone())),
            );
        });
        self.storage
            .checkpoint_scoped_loop(bamboo_domain::ScopedLoopControlCheckpoint {
                session,
                bindings,
                activation,
                owner: self
                    .root_actor_owner
                    .as_ref()
                    .map(|(_, owner)| owner.as_ref()),
                input,
                publish,
            })
            .await
    }

    async fn load_loop_control_checkpoint(
        &self,
        session_id: &str,
    ) -> std::io::Result<Option<Session>> {
        self.storage
            .load_execution_admission_snapshot(session_id)
            .await
    }

    async fn admit_root_inbox(
        &self,
        session: &mut Session,
        inbox: Arc<dyn bamboo_domain::SessionInboxPort>,
        active_run_id: Option<&str>,
    ) -> std::io::Result<Option<bamboo_domain::RootInboxAdmission>> {
        self.admit_owned_root_inbox(session, inbox, active_run_id)
            .await
    }

    async fn checkpoint_retrieval_window(
        &self,
        expected_base: &Session,
        staged: &mut Session,
    ) -> std::io::Result<RetrievalWindowCheckpointOutcome> {
        self.persistence
            .checkpoint_retrieval_window_and_publish(expected_base, staged, |saved| {
                #[cfg(test)]
                self.run_post_durable_hook("checkpoint_retrieval_window", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn checkpoint_prompt_rewrite(
        &self,
        expected_base: &Session,
        staged: &mut Session,
    ) -> std::io::Result<RetrievalWindowCheckpointOutcome> {
        self.persistence
            .checkpoint_prompt_rewrite_and_publish(expected_base, staged, |saved| {
                #[cfg(test)]
                self.run_post_durable_hook("checkpoint_prompt_rewrite", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn checkpoint_manual_archive_rejection(
        &self,
        expected_base: &Session,
        staged: &mut Session,
    ) -> std::io::Result<RetrievalWindowCheckpointOutcome> {
        self.persistence
            .checkpoint_manual_archive_rejection_and_publish(expected_base, staged, |saved| {
                #[cfg(test)]
                self.run_post_durable_hook("checkpoint_manual_archive_rejection", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn checkpoint_manual_archive_consumption(
        &self,
        expected_base: &Session,
        staged: &mut Session,
    ) -> std::io::Result<RetrievalWindowCheckpointOutcome> {
        self.persistence
            .checkpoint_manual_archive_consumption_and_publish(expected_base, staged, |saved| {
                #[cfg(test)]
                self.run_post_durable_hook("checkpoint_manual_archive_consumption", &saved.id);
                self.cache.insert(
                    saved.id.clone(),
                    Arc::new(crate::SessionSnapshot::new(saved.clone())),
                );
            })
            .await
    }

    async fn load_runtime_session(&self, session_id: &str) -> std::io::Result<Option<Session>> {
        if let Some((directory, owner)) = self.root_actor_owner.as_ref() {
            if session_id != owner.fence.actor_id {
                return Err(root_actor_binding_error(
                    "bound execution cannot load another Root",
                ));
            }
            directory
                .validate_fence(&owner.fence, chrono::Utc::now())
                .await
                .map_err(root_actor_binding_error)?;
            let loaded = self.storage.load_session(session_id).await?;
            let Some(snapshot) = loaded.as_ref() else {
                return Err(root_actor_binding_error("bound Root no longer exists"));
            };
            validate_bound_root_snapshot(owner, snapshot)?;
            directory
                .validate_fence(&owner.fence, chrono::Utc::now())
                .await
                .map_err(root_actor_binding_error)?;
            return Ok(loaded);
        }
        self.try_load(session_id).await
    }

    async fn load_runtime_admission_snapshot(
        &self,
        session_id: &str,
    ) -> std::io::Result<Option<Session>> {
        if let Some((directory, owner)) = self.root_actor_owner.as_ref() {
            if session_id != owner.fence.actor_id {
                return Err(root_actor_binding_error(
                    "bound execution cannot admit another Root",
                ));
            }
            directory
                .validate_fence(&owner.fence, chrono::Utc::now())
                .await
                .map_err(root_actor_binding_error)?;
            let loaded = self
                .storage
                .load_execution_admission_snapshot(session_id)
                .await?;
            let Some(snapshot) = loaded.as_ref() else {
                return Err(root_actor_binding_error("bound Root no longer exists"));
            };
            validate_bound_root_snapshot(owner, snapshot)?;
            // A takeover or delete/rebirth during the full durable read cannot
            // release even a previously materialized snapshot to this owner.
            directory
                .validate_fence(&owner.fence, chrono::Utc::now())
                .await
                .map_err(root_actor_binding_error)?;
            return Ok(loaded);
        }
        // Runtime/tool reads remain cache-first. An execution admission must
        // observe durable facts without publishing a stale or rejected clone.
        bamboo_domain::RuntimeSessionPersistence::load_runtime_admission_snapshot(
            self.persistence.as_ref(),
            session_id,
        )
        .await
    }

    async fn clear_legacy_pending_messages(
        &self,
        session_id: &str,
        expected: &[serde_json::Value],
    ) -> std::io::Result<bool> {
        // Do not use the trait default here: `load_runtime_session` is
        // cache-first, so a stale cache could erase a message concurrently
        // appended to the durable legacy queue. Delegate the compare-and-clear
        // to LockedSessionStore's single per-session critical section.
        self.persistence
            .clear_legacy_pending_messages_and_publish(session_id, expected, |latest| {
                #[cfg(test)]
                self.run_post_durable_hook("clear_legacy", session_id);
                self.cache.insert(
                    session_id.to_string(),
                    Arc::new(crate::SessionSnapshot::new(latest.clone())),
                );
            })
            .await
    }

    async fn append_token_usage_record(
        &self,
        session_id: &str,
        json_line: &str,
    ) -> std::io::Result<()> {
        self.storage
            .append_token_usage_record(session_id, json_line)
            .await
    }
}

/// Runtime-only callbacks do not observe Main-owned facts. Preserve the
/// existing cache projection only within the same Session lifetime and reset
/// generation; consumers requiring durable outcome truth must read Main.
fn merge_runtime_control_plane_cache(cached: &mut Session, saved: &Session) {
    let messages = cached.messages.clone();
    let provider_transcript = cached.provider_transcript.clone();
    let admission = cached
        .runtime_metadata
        .as_ref()
        .and_then(|metadata| metadata.session_inbox_admission.clone());
    let same_lifetime = cached.id == saved.id
        && cached.created_at == saved.created_at
        && cached.kind == saved.kind
        && cached.authority_identity == saved.authority_identity
        && cached.check_conversation_generation(saved).is_ok();
    let outcomes = if same_lifetime {
        cached
            .runtime_metadata
            .as_ref()
            .and_then(|metadata| metadata.supervised_root_outcomes.clone())
    } else {
        None
    };
    let mut refreshed = saved.clone();
    refreshed.messages = messages;
    refreshed.provider_transcript = provider_transcript;
    if let Some(admission) = admission {
        refreshed
            .runtime_metadata
            .get_or_insert_with(Default::default)
            .session_inbox_admission = Some(admission);
    } else if let Some(metadata) = refreshed.runtime_metadata.as_mut() {
        metadata.session_inbox_admission = None;
    }
    if let Some(outcomes) = outcomes {
        refreshed
            .runtime_metadata
            .get_or_insert_with(Default::default)
            .supervised_root_outcomes = Some(outcomes);
    } else if let Some(metadata) = refreshed.runtime_metadata.as_mut() {
        metadata.supervised_root_outcomes = None;
    }
    *cached = refreshed;
}

fn validate_bound_root_snapshot(
    owner: &bamboo_domain::RootActorRuntimeWrite,
    snapshot: &Session,
) -> std::io::Result<()> {
    if snapshot.id != owner.fence.actor_id
        || snapshot.created_at != owner.created_at
        || snapshot.kind != bamboo_domain::SessionKind::Root
        || !snapshot.authority_identity.is_ordinary()
    {
        return Err(root_actor_binding_error("bound Root identity changed"));
    }
    owner
        .conversation_generation
        .check_canonical(snapshot)
        .map_err(root_actor_binding_error)
}

fn root_actor_binding_error(reason: impl std::fmt::Display) -> std::io::Error {
    std::io::Error::new(
        std::io::ErrorKind::WouldBlock,
        bamboo_domain::SessionAuthorityConflict(format!("Root Actor execution rejected: {reason}")),
    )
}

#[cfg(test)]
mod tests {
    use super::*;
    use bamboo_agent_core::storage::Storage;
    use chrono::Utc;
    use std::collections::HashMap;
    use std::sync::atomic::{AtomicBool, Ordering};
    use std::sync::{Condvar, Mutex};
    use std::time::Duration;

    #[test]
    fn child_repair_cache_preserves_later_legacy_run_input_wait_and_ui() {
        use bamboo_domain::{AgentRuntimeState, AgentStatusState, Message};
        let parent = Session::new("repair-parent", "model");
        let mut expected = Session::new_child_of("repair-child", &parent, "model", "Child");
        expected.add_message(Message::user("original input"));
        expected.agent_runtime_state = Some(AgentRuntimeState::new("pipeline-A"));
        expected.agent_runtime_state.as_mut().unwrap().status = AgentStatusState::Running;
        expected.set_last_run_status("running");
        let mut saved = expected.clone();
        saved.set_last_run_status("cancelled");
        saved.synchronize_terminal_runtime_status();
        saved
            .metadata
            .insert("runtime.reconciled_terminal_generation".into(), "0".into());
        for case in [
            "typed-run",
            "legacy-run",
            "mixed-mirror",
            "input",
            "wait",
            "idle",
            "source",
        ] {
            let cache: SessionCache = Arc::default();
            let mut later = expected.clone();
            match case {
                "typed-run" => {
                    later.agent_runtime_state.as_mut().unwrap().run_id = "pipeline-B".into()
                }
                "legacy-run" | "mixed-mirror" => {
                    let mut b = AgentRuntimeState::new("pipeline-B");
                    b.status = AgentStatusState::Running;
                    later.metadata.insert(
                        "agent.runtime.state".into(),
                        serde_json::to_string(&b).unwrap(),
                    );
                    if case == "legacy-run" {
                        later.agent_runtime_state = None;
                    }
                }
                "input" => later.add_message(Message::user("new legacy input B")),
                "wait" => {
                    later.agent_runtime_state.as_mut().unwrap().status = AgentStatusState::Suspended
                }
                "idle" => {
                    later.agent_runtime_state.as_mut().unwrap().status = AgentStatusState::Idle
                }
                "source" => {
                    later.metadata.insert(
                        "runtime.child_completion_source_v1".into(),
                        "new unconfirmed source".into(),
                    );
                }
                _ => unreachable!(),
            }
            assert!(later.host_run_id().is_none());
            assert!(later.session_inbox_admission().is_none());
            let before = serde_json::to_value(&later).unwrap();
            cache.insert(
                later.id.clone(),
                Arc::new(crate::SessionSnapshot::new(later)),
            );
            SessionRepository::publish_reconciled_child(&cache, &expected, &saved);
            assert_eq!(
                serde_json::to_value(&*cache.get(&saved.id).unwrap().read()).unwrap(),
                before,
                "{case}"
            );
        }
        let cache: SessionCache = Arc::default();
        SessionRepository::publish_reconciled_child(&cache, &expected, &saved);
        assert!(!cache.contains_key(&saved.id));
        let mut current = expected.clone();
        current.title = "new UI title".into();
        current.add_message(Message::assistant("newer same-run output", None));
        current
            .agent_runtime_state
            .as_mut()
            .unwrap()
            .round
            .current_round = 99;
        cache.insert(
            current.id.clone(),
            Arc::new(crate::SessionSnapshot::new(current.clone())),
        );
        SessionRepository::publish_reconciled_child(&cache, &expected, &saved);
        let entry = cache.get(&saved.id).unwrap();
        let repaired = entry.read();
        assert_eq!(repaired.title, current.title);
        assert_eq!(
            repaired
                .metadata
                .get("runtime.reconciled_terminal_generation")
                .map(String::as_str),
            Some("0")
        );
        assert_eq!(repaired.messages.len(), current.messages.len());
        assert_eq!(
            repaired
                .agent_runtime_state
                .as_ref()
                .unwrap()
                .round
                .current_round,
            99
        );
        assert_eq!(
            repaired.agent_runtime_state.as_ref().unwrap().status,
            AgentStatusState::Cancelled
        );
        assert_eq!(
            repaired
                .agent_runtime_state
                .as_ref()
                .unwrap()
                .tools
                .active_tool_calls,
            0
        );
    }

    #[test]
    fn runtime_cache_preserves_only_same_lifetime_main_outcome_projection() {
        let mut cached = Session::new("outcome-cache", "synthetic-model");
        let owner = bamboo_domain::ActorActivationFence {
            schema_version: bamboo_domain::ACTOR_DIRECTORY_SCHEMA_VERSION,
            actor_id: cached.id.clone(),
            activation_id: "synthetic-activation".into(),
            attempt: 1,
            run_id: "synthetic-run".into(),
            lease_owner: "synthetic-owner".into(),
            lease_epoch: 1,
        };
        let mut outcomes =
            bamboo_domain::SupervisedRootOutcomes::new(&cached.id, cached.created_at).unwrap();
        outcomes.reserve(&owner, None).unwrap();
        outcomes
            .finalize(
                &owner,
                bamboo_domain::SupervisedRootOutcomeResult::Completed,
            )
            .unwrap();
        cached
            .runtime_metadata
            .get_or_insert_with(Default::default)
            .supervised_root_outcomes = Some(outcomes.clone());

        // The runtime callback carries a conflicting value, which is not Main.
        let mut saved = cached.clone();
        saved
            .runtime_metadata
            .as_mut()
            .unwrap()
            .supervised_root_outcomes
            .as_mut()
            .unwrap()
            .facts[0]
            .result = Some(bamboo_domain::SupervisedRootOutcomeResult::Failed);
        merge_runtime_control_plane_cache(&mut cached, &saved);
        assert_eq!(
            cached
                .runtime_metadata
                .as_ref()
                .unwrap()
                .supervised_root_outcomes,
            Some(outcomes)
        );

        // A reused ID is not the old Session's birth and cannot inherit facts.
        saved.created_at += chrono::Duration::seconds(1);
        merge_runtime_control_plane_cache(&mut cached, &saved);
        assert!(cached
            .runtime_metadata
            .as_ref()
            .unwrap()
            .supervised_root_outcomes
            .is_none());
    }

    #[derive(Default)]
    struct MapStorage {
        sessions: Mutex<HashMap<String, Session>>,
        fail_pair_commit: AtomicBool,
    }

    struct FailingSaveStorage {
        persisted: Mutex<Option<Session>>,
    }

    struct AmbiguousReplaySaveStorage {
        persisted: Mutex<Option<Session>>,
    }

    #[async_trait::async_trait]
    impl Storage for MapStorage {
        async fn save_session(&self, session: &Session) -> std::io::Result<()> {
            self.sessions
                .lock()
                .unwrap()
                .insert(session.id.clone(), session.clone());
            Ok(())
        }
        async fn load_session(&self, session_id: &str) -> std::io::Result<Option<Session>> {
            Ok(self.sessions.lock().unwrap().get(session_id).cloned())
        }
        async fn delete_session(&self, session_id: &str) -> std::io::Result<bool> {
            Ok(self.sessions.lock().unwrap().remove(session_id).is_some())
        }

        async fn save_task_control_plane_if_matches(
            &self,
            original: &Session,
            updated: &Session,
        ) -> std::io::Result<bool> {
            let mut sessions = self.sessions.lock().unwrap();
            let matches_original = sessions.get(&original.id).is_some_and(|current| {
                current.task_list_version_meta() == original.task_list_version_meta()
                    && serde_json::to_value(&current.task_list).ok()
                        == serde_json::to_value(&original.task_list).ok()
            });
            if !matches_original {
                return Ok(false);
            }
            let Some(current) = sessions.get(&original.id).cloned() else {
                return Ok(false);
            };
            let mut committed = current;
            committed.task_list = updated.task_list.clone();
            if let Some(version) = updated.task_list_version_meta() {
                committed.set_task_list_version_meta(version);
            }
            sessions.insert(committed.id.clone(), committed);
            Ok(true)
        }

        async fn save_task_control_planes_atomically(
            &self,
            first_original: &Session,
            first_updated: &Session,
            second_original: &Session,
            second_updated: &Session,
        ) -> std::io::Result<bool> {
            if self.fail_pair_commit.swap(false, Ordering::SeqCst) {
                return Err(std::io::Error::other(
                    "injected paired Task transaction failure",
                ));
            }
            let mut sessions = self.sessions.lock().unwrap();
            let matches_original = |current: Option<&Session>, original: &Session| {
                current.is_some_and(|current| {
                    current.task_list_version_meta() == original.task_list_version_meta()
                        && serde_json::to_value(&current.task_list).ok()
                            == serde_json::to_value(&original.task_list).ok()
                })
            };
            if !matches_original(sessions.get(&first_original.id), first_original)
                || !matches_original(sessions.get(&second_original.id), second_original)
            {
                return Ok(false);
            }
            sessions.insert(first_updated.id.clone(), first_updated.clone());
            sessions.insert(second_updated.id.clone(), second_updated.clone());
            Ok(true)
        }
    }

    #[async_trait::async_trait]
    impl Storage for FailingSaveStorage {
        async fn save_session(&self, _session: &Session) -> std::io::Result<()> {
            Err(std::io::Error::other("injected save failure"))
        }

        async fn load_session(&self, _session_id: &str) -> std::io::Result<Option<Session>> {
            Ok(self.persisted.lock().unwrap().clone())
        }

        async fn delete_session(&self, _session_id: &str) -> std::io::Result<bool> {
            Ok(false)
        }
    }

    #[async_trait::async_trait]
    impl Storage for AmbiguousReplaySaveStorage {
        async fn save_session(&self, session: &Session) -> std::io::Result<()> {
            *self.persisted.lock().unwrap() = Some(session.clone());
            Err(std::io::Error::other(
                "injected post-commit replay save error",
            ))
        }

        async fn load_session(&self, _session_id: &str) -> std::io::Result<Option<Session>> {
            Ok(self.persisted.lock().unwrap().clone())
        }

        async fn delete_session(&self, _session_id: &str) -> std::io::Result<bool> {
            Ok(false)
        }
    }

    fn test_repo(storage: Arc<dyn Storage>) -> SessionRepository {
        let cache: SessionCache = Arc::default();
        let persistence = Arc::new(LockedSessionStore::new(storage.clone()));
        SessionRepository::new(cache, storage, persistence)
    }

    fn cache_put(repo: &SessionRepository, session: &Session) {
        repo.cache().insert(
            session.id.clone(),
            Arc::new(crate::SessionSnapshot::new(session.clone())),
        );
    }

    #[tokio::test]
    async fn merged_root_prefers_live_recreated_birth_even_when_epoch_resets() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "root-recreated-same-id";
        let mut stale = Session::new(id, "stale");
        stale.root_mode_transition_epoch = 3;
        stale.updated_at += chrono::Duration::hours(1);
        cache_put(&repo, &stale);

        let mut recreated = Session::new(id, "recreated");
        recreated.created_at = stale.created_at + chrono::Duration::seconds(1);
        recreated.updated_at = stale.created_at + chrono::Duration::seconds(1);
        storage.save_session(&recreated).await.unwrap();

        let loaded = repo.load_merged_checked(id).await.unwrap().unwrap();
        assert_eq!(loaded.model, "recreated");
        assert_eq!(loaded.root_mode_transition_epoch, 0);
        assert_eq!(
            loaded.root_mode_birth_token(),
            recreated.root_mode_birth_token()
        );
        let cached = read_cached_session(repo.cache(), id).expect("new birth cached");
        assert_eq!(cached.model, "recreated");
        assert_eq!(
            cached.root_mode_birth_token(),
            recreated.root_mode_birth_token()
        );
    }

    #[tokio::test]
    async fn merged_root_caches_higher_durable_epoch_despite_clock_skew() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "root-mode-clock-skew";
        let mut stale = Session::new(id, "model");
        stale.updated_at += chrono::Duration::hours(1);
        cache_put(&repo, &stale);

        let mut durable = stale.clone();
        durable.root_mode_transition_epoch = 1;
        durable.root_orchestration_only = true;
        durable.root_tool_authority_revision = 1;
        durable.updated_at = durable.created_at;
        storage.save_session(&durable).await.unwrap();

        let loaded = repo.load_merged_checked(id).await.unwrap().unwrap();
        assert_eq!(loaded.root_mode_transition_epoch, 1);
        assert!(loaded.root_orchestration_only);
        let cached = read_cached_session(repo.cache(), id).expect("higher epoch cached");
        assert_eq!(cached.root_mode_transition_epoch, 1);
        assert!(cached.root_orchestration_only);
    }

    #[tokio::test]
    async fn runtime_merge_save_adopts_root_mode_terminal_tuple_even_for_policy_neutral_fence() {
        use bamboo_domain::{RootModeOperationAction, RootModeOperationRequest};

        for action in [
            RootModeOperationAction::Select,
            RootModeOperationAction::Recover,
        ] {
            let home = tempfile::tempdir().unwrap();
            let concrete = Arc::new(
                bamboo_storage::SessionStoreV2::new(home.path().into())
                    .await
                    .unwrap(),
            );
            let storage: Arc<dyn Storage> = concrete.clone();
            let repo = test_repo(storage.clone());
            let mut running = Session::new("runtime-root-mode-merge", "model");
            storage.save_session(&running).await.unwrap();
            cache_put(&repo, &running);
            let request = RootModeOperationRequest {
                session_id: running.id.clone(),
                operation_id: format!("0:{}", uuid::Uuid::new_v4()),
                birth_token: running.root_mode_birth_token(),
                expected_epoch: 0,
                requested_enabled: true,
                action,
            };
            storage.root_mode_operation(&request).await.unwrap();

            // This save can race a host mode operation before the next runtime
            // boundary. The production merge must preserve that durable tuple.
            running.add_message(bamboo_agent_core::Message::assistant("run result", None));
            repo.save(&mut running).await.unwrap();
            assert_eq!(running.root_mode_transition_epoch, 1);
            assert_eq!(running.root_mode_operations.len(), 1);
            assert_eq!(
                running.root_orchestration_only_enabled(),
                action == RootModeOperationAction::Select
            );
            let durable = storage.load_session(&running.id).await.unwrap().unwrap();
            assert_eq!(durable.root_mode_transition_epoch, 1);
            assert_eq!(durable.root_mode_operations, running.root_mode_operations);
            assert_eq!(durable.messages.last().unwrap().content, "run result");
            concrete.flush_search_index().await;
        }
    }

    fn task_list(session_id: &str, title: &str) -> bamboo_domain::TaskList {
        let now = Utc::now();
        bamboo_domain::TaskList {
            session_id: session_id.to_string(),
            title: title.to_string(),
            items: Vec::new(),
            created_at: now,
            updated_at: now,
        }
    }

    fn staged_retrieval_window_archive(expected: &Session) -> Session {
        let mut staged = expected.clone();
        let mut event = bamboo_domain::CompressionEvent::new(
            1,
            1,
            80.0,
            60.0,
            0,
            bamboo_domain::CompressionTriggerType::Auto,
            0.0,
            None,
            0,
        );
        event.kind = bamboo_domain::CompressionEventKind::RetrievalWindow;
        staged.messages[0].compressed = true;
        staged.messages[0].compressed_by_event_id = Some(event.id.clone());
        staged.compression_events.push(event);
        staged.reset_model_context_epoch(bamboo_domain::ModelContextResetReason::Compression);
        staged
    }

    #[tokio::test]
    async fn paired_task_cas_narrowly_updates_child_and_root_cache() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let root_id = "paired-cache-root";
        let child_id = "paired-cache-child";
        let expected_task_list = task_list(root_id, "old shared");

        let mut root = Session::new(root_id, "model");
        root.add_message(bamboo_agent_core::Message::user("root transcript"));
        root.metadata
            .insert("unrelated.root".to_string(), "keep".to_string());
        root.set_task_list(expected_task_list.clone());
        root.set_task_list_version_meta("1");
        storage.save_session(&root).await.unwrap();
        cache_put(&repo, &root);

        let mut child = Session::new_child(child_id, root_id, "model", "child");
        child.add_message(bamboo_agent_core::Message::user("child transcript"));
        child
            .metadata
            .insert("unrelated.child".to_string(), "keep".to_string());
        child.set_task_list(expected_task_list.clone());
        child.set_task_list_version_meta("1");
        storage.save_session(&child).await.unwrap();
        cache_put(&repo, &child);

        assert!(
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_planes_if_version(
                &repo,
                child_id,
                root_id,
                "1",
                &expected_task_list,
                &task_list(root_id, "evaluated"),
                "2",
            )
            .await
            .expect("paired cache CAS succeeds")
        );

        let cached_root = read_cached_session(repo.cache(), root_id).expect("cached root");
        let cached_child = read_cached_session(repo.cache(), child_id).expect("cached child");
        for (session, transcript, metadata_key) in [
            (&cached_root, "root transcript", "unrelated.root"),
            (&cached_child, "child transcript", "unrelated.child"),
        ] {
            assert_eq!(session.task_list_version_meta().as_deref(), Some("2"));
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("evaluated")
            );
            assert_eq!(session.messages.len(), 1);
            assert_eq!(session.messages[0].content, transcript);
            assert_eq!(
                session.metadata.get(metadata_key).map(String::as_str),
                Some("keep")
            );
        }
    }

    #[tokio::test]
    async fn failed_paired_task_transaction_does_not_publish_child_or_root_cache() {
        let concrete = Arc::new(MapStorage::default());
        let storage: Arc<dyn Storage> = concrete.clone();
        let repo = test_repo(storage.clone());
        let root_id = "paired-cache-failure-root";
        let child_id = "paired-cache-failure-child";
        let expected_task_list = task_list(root_id, "old shared");

        let mut root = Session::new(root_id, "model");
        root.add_message(bamboo_agent_core::Message::user("root transcript"));
        root.metadata
            .insert("unrelated.root".to_string(), "keep".to_string());
        root.set_task_list(expected_task_list.clone());
        root.set_task_list_version_meta("1");
        storage.save_session(&root).await.unwrap();
        cache_put(&repo, &root);

        let mut child = Session::new_child(child_id, root_id, "model", "child");
        child.add_message(bamboo_agent_core::Message::user("child transcript"));
        child
            .metadata
            .insert("unrelated.child".to_string(), "keep".to_string());
        child.set_task_list(expected_task_list.clone());
        child.set_task_list_version_meta("1");
        storage.save_session(&child).await.unwrap();
        cache_put(&repo, &child);

        concrete.fail_pair_commit.store(true, Ordering::SeqCst);
        let error =
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_planes_if_version(
                &repo,
                child_id,
                root_id,
                "1",
                &expected_task_list,
                &task_list(root_id, "must not publish"),
                "2",
            )
            .await
            .expect_err("paired durable transaction fails");
        assert!(error.to_string().contains("injected paired"));

        for (id, expected_title, transcript, metadata_key) in [
            (root_id, "old shared", "root transcript", "unrelated.root"),
            (
                child_id,
                "old shared",
                "child transcript",
                "unrelated.child",
            ),
        ] {
            let cached = read_cached_session(repo.cache(), id).expect("cached session remains");
            assert_eq!(cached.task_list_version_meta().as_deref(), Some("1"));
            assert_eq!(
                cached.task_list.as_ref().map(|list| list.title.as_str()),
                Some(expected_title)
            );
            assert_eq!(cached.messages[0].content, transcript);
            assert_eq!(
                cached.metadata.get(metadata_key).map(String::as_str),
                Some("keep")
            );

            let durable = storage.load_session(id).await.unwrap().unwrap();
            assert_eq!(durable.task_list_version_meta().as_deref(), Some("1"));
            assert_eq!(
                durable.task_list.as_ref().map(|list| list.title.as_str()),
                Some(expected_title)
            );
        }
    }

    fn durable_cache_fence(
        operation: impl Into<String>,
        marker: impl Into<String>,
    ) -> (
        PostDurableHook,
        tokio::sync::oneshot::Receiver<()>,
        Arc<(Mutex<bool>, Condvar)>,
    ) {
        let operation = operation.into();
        let marker = marker.into();
        let (started_tx, started_rx) = tokio::sync::oneshot::channel();
        let started_tx = Arc::new(Mutex::new(Some(started_tx)));
        let release = Arc::new((Mutex::new(false), Condvar::new()));
        let hook_release = release.clone();
        let hook: PostDurableHook = Arc::new(move |actual_operation, actual_marker| {
            if actual_operation != operation || actual_marker != marker {
                return;
            }
            if let Some(started_tx) = started_tx.lock().unwrap().take() {
                started_tx.send(()).expect("fence observer still present");
            }
            let (released, wake) = &*hook_release;
            let mut released = released.lock().unwrap();
            while !*released {
                released = wake.wait(released).unwrap();
            }
        });
        (hook, started_rx, release)
    }

    fn release_fence(release: &Arc<(Mutex<bool>, Condvar)>) {
        let (released, wake) = &**release;
        *released.lock().unwrap() = true;
        wake.notify_all();
    }

    async fn assert_second_write_waits_for_cache_publish<T>(
        first: tokio::task::JoinHandle<std::io::Result<T>>,
        mut second: tokio::task::JoinHandle<std::io::Result<T>>,
        release: Arc<(Mutex<bool>, Condvar)>,
    ) -> (T, T) {
        let second_before_release =
            tokio::time::timeout(Duration::from_millis(100), &mut second).await;
        let completed_before_release = second_before_release.is_ok();
        release_fence(&release);

        let first = first
            .await
            .expect("first writer joins")
            .expect("first writer succeeds");
        let second = match second_before_release {
            Ok(joined) => joined
                .expect("second writer joins")
                .expect("second writer succeeds"),
            Err(_) => second
                .await
                .expect("second writer joins")
                .expect("second writer succeeds"),
        };
        assert!(
            !completed_before_release,
            "the second write must remain behind the first write's durable-to-cache fence"
        );
        (first, second)
    }

    #[derive(Clone, Copy, Debug)]
    enum FullSaveRoute {
        InherentSave,
        SaveAndCache,
        RuntimePersistence,
    }

    impl FullSaveRoute {
        fn operation(self) -> &'static str {
            match self {
                Self::InherentSave => "save_full",
                Self::SaveAndCache => "save_and_cache",
                Self::RuntimePersistence => "save_runtime_session",
            }
        }

        fn name(self) -> &'static str {
            match self {
                Self::InherentSave => "inherent",
                Self::SaveAndCache => "save-and-cache",
                Self::RuntimePersistence => "runtime-persistence",
            }
        }
    }

    async fn assert_full_save_route_serializes_cache_publish(route: FullSaveRoute) {
        let temp = tempfile::tempdir().unwrap();
        let concrete_storage = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let storage: Arc<dyn Storage> = concrete_storage;
        let id = format!("root-full-cache-order-{}", route.name());
        let (hook, first_durable, release) = durable_cache_fence(route.operation(), id.clone());
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));

        let mut initial = Session::new(&id, "model");
        initial.add_message(bamboo_agent_core::Message::user("durable transcript"));
        initial.set_task_list(task_list(&id, "initial"));
        initial.set_task_list_version_meta("0");
        initial
            .metadata
            .insert("unrelated.runtime".to_string(), "keep".to_string());
        storage.save_session(&initial).await.unwrap();
        cache_put(&repo, &initial);

        let first_repo = repo.clone();
        let mut root_snapshot = initial.clone();
        root_snapshot.add_message(bamboo_agent_core::Message::assistant(
            "full-save transcript suffix",
            None,
        ));
        root_snapshot.set_task_list(task_list(&id, "root"));
        root_snapshot.set_task_list_version_meta("1");
        let first = tokio::spawn(async move {
            match route {
                FullSaveRoute::InherentSave => first_repo.save(&mut root_snapshot).await,
                FullSaveRoute::SaveAndCache => {
                    first_repo.save_and_cache(&mut root_snapshot).await;
                    Ok(())
                }
                FullSaveRoute::RuntimePersistence => {
                    bamboo_domain::RuntimeSessionPersistence::save_runtime_session(
                        first_repo.as_ref(),
                        &mut root_snapshot,
                    )
                    .await
                }
            }
        });
        first_durable
            .await
            .expect("root full durable write reached");

        let second_repo = repo.clone();
        let second_id = id.clone();
        let child_task_list = task_list(&id, "child");
        let second = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                second_repo.as_ref(),
                &second_id,
                &child_task_list,
                "2",
            )
            .await
            .map(|updated| assert!(updated, "root must exist"))
        });
        assert_second_write_waits_for_cache_publish(first, second, release).await;

        let durable = storage.load_session(&id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), &id).expect("cached root");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("2"),
                "{route:?} {tier} must retain the child transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("child"),
                "{route:?} {tier} must retain the child transaction"
            );
            assert_eq!(
                session
                    .metadata
                    .get("unrelated.runtime")
                    .map(String::as_str),
                Some("keep"),
                "{route:?} {tier} must preserve unrelated runtime state"
            );
            assert_eq!(
                session.messages.len(),
                2,
                "{route:?} {tier} must preserve the full-save transcript"
            );
        }
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn root_full_saves_and_child_task_patch_share_publish_order() {
        for route in [
            FullSaveRoute::InherentSave,
            FullSaveRoute::SaveAndCache,
            FullSaveRoute::RuntimePersistence,
        ] {
            assert_full_save_route_serializes_cache_publish(route).await;
        }
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn checkpoint_and_child_task_patch_share_publish_order() {
        let temp = tempfile::tempdir().unwrap();
        let concrete_storage = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let storage: Arc<dyn Storage> = concrete_storage;
        let id = "checkpoint-cache-order";
        let (hook, checkpoint_durable, release) = durable_cache_fence("checkpoint", id);
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));

        let mut initial = Session::new(id, "model");
        initial.add_message(bamboo_agent_core::Message::user("durable transcript"));
        initial.set_task_list(task_list(id, "initial"));
        initial.set_task_list_version_meta("0");
        initial
            .metadata
            .insert("unrelated.runtime".to_string(), "keep".to_string());
        storage.save_session(&initial).await.unwrap();
        cache_put(&repo, &initial);

        let checkpoint_repo = repo.clone();
        let mut checkpoint_snapshot = initial.clone();
        checkpoint_snapshot.add_message(bamboo_agent_core::Message::assistant(
            "checkpoint transcript suffix",
            None,
        ));
        checkpoint_snapshot.set_task_list(task_list(id, "checkpoint"));
        checkpoint_snapshot.set_task_list_version_meta("1");
        let checkpoint = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::checkpoint_runtime_session(
                checkpoint_repo.as_ref(),
                &mut checkpoint_snapshot,
            )
            .await
        });
        checkpoint_durable
            .await
            .expect("checkpoint durable write reached");

        let child_repo = repo.clone();
        let child_task_list = task_list(id, "child");
        let child_patch = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                child_repo.as_ref(),
                id,
                &child_task_list,
                "2",
            )
            .await
            .map(|updated| assert!(updated, "root must exist"))
        });
        assert_second_write_waits_for_cache_publish(checkpoint, child_patch, release).await;

        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("cached root");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("2"),
                "{tier} must retain the child transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("child"),
                "{tier} must retain the child transaction"
            );
            assert_eq!(
                session
                    .metadata
                    .get("unrelated.runtime")
                    .map(String::as_str),
                Some("keep"),
                "{tier} must preserve unrelated runtime state"
            );
            assert_eq!(
                session.messages.len(),
                2,
                "{tier} must preserve the checkpoint transcript"
            );
        }
    }

    #[derive(Clone, Copy, Debug)]
    enum CacheBackfillRoute {
        Load,
        TryLoad,
    }

    impl CacheBackfillRoute {
        fn operation(self) -> &'static str {
            match self {
                Self::Load => "load",
                Self::TryLoad => "try_load",
            }
        }

        fn name(self) -> &'static str {
            match self {
                Self::Load => "load",
                Self::TryLoad => "try-load",
            }
        }
    }

    async fn assert_cache_backfill_serializes_with_task_patch(route: CacheBackfillRoute) {
        let temp = tempfile::tempdir().unwrap();
        let concrete_storage = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let storage: Arc<dyn Storage> = concrete_storage;
        let id = format!("cache-backfill-order-{}", route.name());
        let (hook, loaded_old_durable, release) =
            durable_cache_fence(route.operation(), id.clone());
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));

        let mut initial = Session::new(&id, "model");
        initial.set_task_list(task_list(&id, "initial"));
        initial.set_task_list_version_meta("0");
        storage.save_session(&initial).await.unwrap();
        assert!(
            read_cached_session(repo.cache(), &id).is_none(),
            "the race requires a genuine cache miss"
        );

        let load_repo = repo.clone();
        let load_id = id.clone();
        let load = tokio::spawn(async move {
            let loaded = match route {
                CacheBackfillRoute::Load => load_repo.load(&load_id).await,
                CacheBackfillRoute::TryLoad => {
                    load_repo.try_load(&load_id).await.expect("storage load")
                }
            };
            assert!(loaded.is_some(), "seeded session must load");
            Ok(())
        });
        loaded_old_durable
            .await
            .expect("old durable snapshot loaded");

        let patch_repo = repo.clone();
        let patch_id = id.clone();
        let child_task_list = task_list(&id, "child");
        let patch = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                patch_repo.as_ref(),
                &patch_id,
                &child_task_list,
                "1",
            )
            .await
            .map(|updated| assert!(updated, "root must exist"))
        });
        assert_second_write_waits_for_cache_publish(load, patch, release).await;

        let durable = storage.load_session(&id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), &id).expect("backfilled cache");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("1"),
                "{route:?} {tier} must retain the child transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("child"),
                "{route:?} {tier} must retain the child transaction"
            );
        }
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn cache_miss_backfills_and_child_task_patch_share_publish_order() {
        for route in [CacheBackfillRoute::Load, CacheBackfillRoute::TryLoad] {
            assert_cache_backfill_serializes_with_task_patch(route).await;
        }
    }

    #[tokio::test]
    async fn cache_hits_do_not_wait_for_the_persistence_lock() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage);
        let id = "cache-hit-lock-free";
        let cached = Session::new(id, "cached-model");
        cache_put(&repo, &cached);
        let persistence_guard = repo.persistence().acquire_lock(id).await;

        let loaded = tokio::time::timeout(Duration::from_millis(100), repo.load(id)).await;
        let try_loaded = tokio::time::timeout(Duration::from_millis(100), repo.try_load(id)).await;
        drop(persistence_guard);

        assert_eq!(
            loaded
                .expect("cache hit must not wait for the persistence lock")
                .expect("cached session")
                .model,
            "cached-model"
        );
        assert_eq!(
            try_loaded
                .expect("fallible cache hit must not wait for the persistence lock")
                .expect("cache read succeeds")
                .expect("cached session")
                .model,
            "cached-model"
        );
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn load_merged_storage_refresh_and_child_task_patch_share_publish_order() {
        let temp = tempfile::tempdir().unwrap();
        let concrete_storage = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let storage: Arc<dyn Storage> = concrete_storage;
        let id = "load-merged-cache-order";
        let (hook, loaded_old_durable, release) = durable_cache_fence("load_merged", id);
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));

        let mut durable = Session::new(id, "model");
        durable.updated_at = Utc::now();
        durable.set_task_list(task_list(id, "initial"));
        durable.set_task_list_version_meta("0");
        storage.save_session(&durable).await.unwrap();

        let mut memory = durable.clone();
        memory.updated_at = durable.updated_at - chrono::Duration::seconds(1);
        memory.set_task_list(task_list(id, "memory"));
        cache_put(&repo, &memory);

        let load_repo = repo.clone();
        let load = tokio::spawn(async move {
            assert!(
                load_repo.load_merged(id).await.is_some(),
                "seeded session must load"
            );
            Ok(())
        });
        loaded_old_durable
            .await
            .expect("old durable snapshot loaded");

        let patch_repo = repo.clone();
        let child_task_list = task_list(id, "child");
        let patch = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                patch_repo.as_ref(),
                id,
                &child_task_list,
                "1",
            )
            .await
            .map(|updated| assert!(updated, "root must exist"))
        });
        assert_second_write_waits_for_cache_publish(load, patch, release).await;

        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("refreshed cache");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("1"),
                "{tier} must retain the child transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("child"),
                "{tier} must retain the child transaction"
            );
        }
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn legacy_clear_refresh_and_child_task_patch_share_publish_order() {
        let temp = tempfile::tempdir().unwrap();
        let concrete_storage = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let storage: Arc<dyn Storage> = concrete_storage;
        let id = "legacy-clear-cache-order";
        let expected = vec![serde_json::json!({"content": "legacy"})];
        let (hook, loaded_post_cas_snapshot, release) = durable_cache_fence("clear_legacy", id);
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));

        let mut initial = Session::new(id, "model");
        initial.set_pending_injected_messages(expected.clone());
        initial.set_task_list(task_list(id, "initial"));
        initial.set_task_list_version_meta("0");
        storage.save_session(&initial).await.unwrap();
        cache_put(&repo, &initial);

        let clear_repo = repo.clone();
        let clear_expected = expected.clone();
        let clear = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::clear_legacy_pending_messages(
                clear_repo.as_ref(),
                id,
                &clear_expected,
            )
            .await
            .map(|cleared| assert!(cleared, "legacy queue must match"))
        });
        loaded_post_cas_snapshot
            .await
            .expect("post-CAS snapshot loaded");

        let patch_repo = repo.clone();
        let child_task_list = task_list(id, "child");
        let patch = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                patch_repo.as_ref(),
                id,
                &child_task_list,
                "1",
            )
            .await
            .map(|updated| assert!(updated, "root must exist"))
        });
        assert_second_write_waits_for_cache_publish(clear, patch, release).await;

        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("refreshed cache");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("1"),
                "{tier} must retain the child transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("child"),
                "{tier} must retain the child transaction"
            );
            assert!(
                !session.has_pending_injected_messages(),
                "{tier} must retain the successful legacy clear"
            );
        }
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn concurrent_task_patches_publish_cache_in_durable_order() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let (hook, first_durable, release) = durable_cache_fence("task", "1");
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));
        let id = "concurrent-task-cache-order";
        let mut initial = Session::new(id, "model");
        initial.set_task_list(task_list(id, "initial"));
        initial.set_task_list_version_meta("0");
        storage.save_session(&initial).await.unwrap();
        cache_put(&repo, &initial);

        let first_repo = repo.clone();
        let first_task_list = task_list(id, "first");
        let first = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                first_repo.as_ref(),
                id,
                &first_task_list,
                "1",
            )
            .await
        });
        first_durable.await.expect("first durable write reached");

        let second_repo = repo.clone();
        let second_task_list = task_list(id, "second");
        let second = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                second_repo.as_ref(),
                id,
                &second_task_list,
                "2",
            )
            .await
        });
        let (first_updated, second_updated) =
            assert_second_write_waits_for_cache_publish(first, second, release).await;
        assert!(first_updated && second_updated);

        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("cached root");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("2"),
                "{tier} must retain the second transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("second"),
                "{tier} must retain the second transaction"
            );
        }
    }

    #[tokio::test(flavor = "multi_thread", worker_threads = 4)]
    async fn root_control_plane_save_and_child_task_patch_share_publish_order() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let (hook, root_durable, release) = durable_cache_fence("save", "root-control-plane-order");
        let repo = Arc::new(test_repo(storage.clone()).with_post_durable_hook(hook));
        let id = "root-control-plane-order";

        let mut initial = Session::new(id, "model");
        initial.add_message(bamboo_agent_core::Message::user("durable transcript"));
        initial.set_task_list(task_list(id, "initial"));
        initial.set_task_list_version_meta("0");
        initial
            .metadata
            .insert("unrelated.runtime".to_string(), "keep".to_string());
        storage.save_session(&initial).await.unwrap();
        cache_put(&repo, &initial);

        let root_repo = repo.clone();
        let mut root_snapshot = initial.clone();
        root_snapshot.set_task_list(task_list(id, "root"));
        root_snapshot.set_task_list_version_meta("1");
        let root_save = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::save_runtime_control_plane(
                root_repo.as_ref(),
                &mut root_snapshot,
            )
            .await
        });
        root_durable.await.expect("root durable write reached");

        let child_repo = repo.clone();
        let child_task_list = task_list(id, "child");
        let child_patch = tokio::spawn(async move {
            bamboo_domain::RuntimeSessionPersistence::update_task_list_control_plane(
                child_repo.as_ref(),
                id,
                &child_task_list,
                "2",
            )
            .await
            .map(|updated| {
                assert!(updated, "root must exist");
            })
        });
        assert_second_write_waits_for_cache_publish(root_save, child_patch, release).await;

        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("cached root");
        for (tier, session) in [("durable", durable), ("cache", cached)] {
            assert_eq!(
                session.task_list_version_meta().as_deref(),
                Some("2"),
                "{tier} must retain the child transaction"
            );
            assert_eq!(
                session.task_list.as_ref().map(|list| list.title.as_str()),
                Some("child"),
                "{tier} must retain the child transaction"
            );
            assert_eq!(
                session
                    .metadata
                    .get("unrelated.runtime")
                    .map(String::as_str),
                Some("keep"),
                "{tier} must preserve unrelated runtime state"
            );
            assert_eq!(
                session.messages.len(),
                1,
                "{tier} must preserve the transcript"
            );
        }
    }

    #[tokio::test]
    async fn stale_control_plane_save_keeps_checkpointed_ledger_in_durable_and_cache() {
        let temp = tempfile::tempdir().unwrap();
        let storage: Arc<dyn Storage> = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let repo = test_repo(storage.clone());
        let id = "control-plane-ledger-cache";
        let mut initial = Session::new(id, "model");
        initial.add_message(bamboo_agent_core::Message::user("durable transcript"));
        let assistant = bamboo_agent_core::Message::assistant("normalized", None);
        let anchor = assistant.id.clone();
        initial.add_message(assistant);
        let native_item = bamboo_domain::ProviderTranscriptItem::try_from_payload(
            bamboo_domain::ProviderFamily::OpenAi,
            bamboo_domain::ProviderProtocol::OpenAiResponsesV1,
            bamboo_domain::ProviderTranscriptOrigin::Provider,
            bamboo_domain::ProviderTranscriptAuthor::Model,
            serde_json::json!({
                "type":"tool_search_call","id":"tsc_cache_native","execution":"client","call_id":"cache_native",
                "status":"completed","arguments":{"query":"CACHE_NATIVE_PAYLOAD_SENTINEL"}
            }),
        )
        .unwrap();
        initial
            .append_provider_transcript_group(&anchor, None, vec![native_item])
            .unwrap();
        storage.save_session(&initial).await.unwrap();
        cache_put(&repo, &initial);
        let mut stale = initial.clone();

        let mut runner = initial;
        runner.model_context_state = Some(bamboo_domain::ModelContextState {
            state_revision: 1,
            prefix_epoch: 1,
            cache_scope_sha256: Some("scope".to_string()),
            transcript_item_sha256: vec!["runner-l1".to_string()],
            ..bamboo_domain::ModelContextState::default()
        });
        bamboo_domain::RuntimeSessionPersistence::checkpoint_runtime_session(&repo, &mut runner)
            .await
            .unwrap();

        stale
            .metadata
            .insert("runtime.suspend_reason".to_string(), "waiting".to_string());
        bamboo_domain::RuntimeSessionPersistence::save_runtime_control_plane(&repo, &mut stale)
            .await
            .unwrap();

        let expected = runner.model_context_state;
        let expected_native = runner.provider_transcript;
        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("cached session");
        for (tier, session) in [("durable", durable), ("cache", cached.clone())] {
            assert_eq!(session.model_context_state, expected, "tier={tier}");
            assert_eq!(
                session.provider_transcript, expected_native,
                "tier={tier} must retain the message-anchored native transcript"
            );
            assert_eq!(
                session
                    .metadata
                    .get("runtime.suspend_reason")
                    .map(String::as_str),
                Some("waiting"),
                "tier={tier}"
            );
            assert_eq!(session.messages.len(), 2, "tier={tier}");
        }

        let runtime_json =
            std::fs::read_to_string(temp.path().join("sessions").join(id).join("runtime.json"))
                .unwrap();
        assert!(!runtime_json.contains("CACHE_NATIVE_PAYLOAD_SENTINEL"));

        // Prove the cache projection cannot turn a runtime-only update into a
        // later durable loss when that cached value becomes a full checkpoint.
        let mut cache_writer = cached;
        bamboo_domain::RuntimeSessionPersistence::checkpoint_runtime_session(
            &repo,
            &mut cache_writer,
        )
        .await
        .unwrap();
        let restarted = storage.load_session(id).await.unwrap().unwrap();
        assert_eq!(restarted.provider_transcript, expected_native);
    }

    #[tokio::test]
    async fn narrow_runtime_metadata_transaction_preserves_live_and_durable_non_owned_state() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "narrow-metadata";
        let mut durable = Session::new(id, "durable-model");
        durable.add_message(bamboo_agent_core::Message::user("durable user turn"));
        durable
            .metadata
            .insert("external.durable".to_string(), "keep".to_string());
        storage.save_session(&durable).await.expect("seed durable");

        let mut live = durable.clone();
        live.add_message(bamboo_agent_core::Message::assistant(
            "in-flight assistant tool call",
            None,
        ));
        live.model = "live-model".to_string();
        live.metadata
            .insert("external.live".to_string(), "keep".to_string());
        cache_put(&repo, &live);

        repo.update_runtime_session(id, &["workflow.owned"], |latest| {
            latest
                .metadata
                .insert("workflow.owned".to_string(), "active".to_string());
        })
        .await
        .expect("transaction")
        .expect("session exists");

        let saved = storage
            .load_session(id)
            .await
            .expect("load durable")
            .expect("durable exists");
        assert_eq!(
            saved.messages.len(),
            1,
            "transaction never writes stale live messages"
        );
        assert_eq!(
            saved.metadata.get("external.durable").map(String::as_str),
            Some("keep")
        );
        assert_eq!(
            saved.metadata.get("workflow.owned").map(String::as_str),
            Some("active")
        );

        let cached = read_cached_session(repo.cache(), id).expect("live cache");
        assert_eq!(
            cached.messages.len(),
            2,
            "cache live tool call is not replaced"
        );
        assert_eq!(cached.model, "live-model");
        assert_eq!(
            cached.metadata.get("external.live").map(String::as_str),
            Some("keep")
        );
        assert_eq!(
            cached.metadata.get("workflow.owned").map(String::as_str),
            Some("active")
        );
    }

    /// Regression guard: a strictly-newer in-memory session (e.g. one that just
    /// answered and cleared its pending question) must win over a strictly-older
    /// storage copy that still carries the pending question — both in the value
    /// returned AND in the cache (no clobber).
    #[tokio::test]
    async fn load_merged_does_not_regress_to_older_storage() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "s1";

        let mut stale = Session::new(id.to_string(), "m");
        stale.set_pending_question(
            "tc1".into(),
            "kind".into(),
            "q?".into(),
            vec!["OK".into()],
            true,
        );
        stale.updated_at = Utc::now() - chrono::Duration::seconds(10);
        storage.save_session(&stale).await.unwrap();

        // This is a fresher snapshot of the same Root birth, not a recreated
        // Session with the same ID (whose live durable birth takes precedence).
        let mut fresh = stale.clone();
        fresh.clear_pending_question();
        fresh.updated_at = Utc::now();
        cache_put(&repo, &fresh);

        let merged = repo.load_merged(id).await.expect("session exists");
        assert!(
            merged.pending_question.is_none(),
            "must return the newer answered memory copy, not the stale storage one"
        );
        let cached = read_cached_session(repo.cache(), id).expect("cached");
        assert!(
            cached.pending_question.is_none(),
            "load_merged must never regress the cache to a stale storage copy"
        );
    }

    /// The pending-question recovery still works when storage is the same age:
    /// if memory lost a pending question that same-age storage retains, prefer
    /// storage so a genuine clarification is not dropped.
    #[tokio::test]
    async fn load_merged_recovers_pending_question_from_same_age_storage() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "s2";
        let ts = Utc::now();

        let mut with_pending = Session::new(id.to_string(), "m");
        with_pending.set_pending_question(
            "tc".into(),
            "k".into(),
            "q".into(),
            vec!["OK".into()],
            true,
        );
        with_pending.updated_at = ts;
        storage.save_session(&with_pending).await.unwrap();

        let mut lost = with_pending.clone();
        lost.clear_pending_question();
        lost.updated_at = ts;
        cache_put(&repo, &lost);

        let merged = repo.load_merged(id).await.expect("session exists");
        assert!(
            merged.pending_question.is_some(),
            "same-age storage carrying a pending question must still be recovered"
        );
    }

    #[tokio::test]
    async fn runtime_publish_refreshes_cache_even_when_storage_fails() {
        let id = "runtime-selection";
        let mut previous = Session::new(id.to_string(), "m");
        previous.metadata.insert(
            "skill_runtime_selected_skill_ids".to_string(),
            "[\"plan\"]".to_string(),
        );
        let storage: Arc<dyn Storage> = Arc::new(FailingSaveStorage {
            persisted: Mutex::new(Some(previous.clone())),
        });
        let repo = test_repo(storage.clone());
        cache_put(&repo, &previous);

        let mut current = previous.clone();
        current.metadata.insert(
            "skill_runtime_selected_skill_ids".to_string(),
            "[\"review\"]".to_string(),
        );
        current.updated_at = Utc::now();

        let result =
            bamboo_domain::RuntimeSessionPersistence::save_runtime_session(&repo, &mut current)
                .await;
        assert!(result.is_err(), "durable failure must still be surfaced");

        let cached = repo.load(id).await.expect("cached current session");
        assert_eq!(
            cached
                .metadata
                .get("skill_runtime_selected_skill_ids")
                .map(String::as_str),
            Some("[\"review\"]")
        );
        let allowlist = bamboo_skills::access_control::extract_skill_allowlist(&cached.metadata)
            .expect("runtime authorization allowlist");
        assert!(allowlist.contains("review"));
        assert!(!allowlist.contains("plan"));
        let durable = storage
            .load_session(id)
            .await
            .expect("load durable state")
            .expect("previous durable session");
        assert_eq!(
            durable
                .metadata
                .get("skill_runtime_selected_skill_ids")
                .map(String::as_str),
            Some("[\"plan\"]")
        );
    }

    #[tokio::test]
    async fn finalized_runtime_save_reconciles_late_child_completion_in_disk_and_cache() {
        use bamboo_domain::session::runtime_state::{
            AgentRuntimeState, AgentStatusState, ChildWaitPolicy, WaitingForChildrenState,
        };

        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "late-child-finalization";
        let mut parent = Session::new(id, "model");
        let mut runtime = AgentRuntimeState::new("parent-run");
        runtime.status = AgentStatusState::Suspended;
        let mut wait = WaitingForChildrenState::for_children(
            vec!["child-1".to_string()],
            ChildWaitPolicy::All,
            Utc::now(),
        );
        wait.registered_by_tool_call_id = Some("tc_wait".to_string());
        runtime.waiting_for_children = Some(wait);
        parent.agent_runtime_state = Some(runtime);
        parent.metadata.insert(
            "runtime.suspend_reason".to_string(),
            "waiting_for_children".to_string(),
        );
        parent.set_last_run_status("suspended");
        storage.save_session(&parent).await.unwrap();
        cache_put(&repo, &parent);

        let mut stale_runner = parent.clone();
        let mut completed = parent;
        let completed_runtime = completed.agent_runtime_state.as_mut().unwrap();
        completed_runtime.waiting_for_children = None;
        completed_runtime.status = AgentStatusState::Idle;
        completed_runtime.suspension = None;
        completed.metadata.remove("runtime.suspend_reason");
        let mut outcome = bamboo_agent_core::Message::user("completed child result");
        outcome.metadata = Some(serde_json::json!({
            "runtime_kind": "child_completion_resume"
        }));
        completed.add_message(outcome);
        storage.save_session(&completed).await.unwrap();

        bamboo_domain::RuntimeSessionPersistence::save_finalized_runtime_session(
            &repo,
            &mut stale_runner,
        )
        .await
        .unwrap();

        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("final snapshot published");
        for saved in [&durable, &cached] {
            assert!(saved
                .agent_runtime_state
                .as_ref()
                .unwrap()
                .waiting_for_children
                .is_none());
            assert!(!saved.metadata.contains_key("runtime.suspend_reason"));
            assert_eq!(saved.last_run_status().as_deref(), Some("completed"));
            assert!(saved
                .messages
                .iter()
                .any(|message| message.content == "completed child result"));
        }
    }

    #[tokio::test]
    async fn inherent_save_leaves_existing_cache_untouched_when_storage_fails() {
        let id = "inherent-save-failure";
        let previous = Session::new(id, "previous");
        let storage: Arc<dyn Storage> = Arc::new(FailingSaveStorage {
            persisted: Mutex::new(Some(previous.clone())),
        });
        let repo = test_repo(storage);
        cache_put(&repo, &previous);

        let mut current = previous.clone();
        current.model = "current".to_string();
        assert!(repo.save(&mut current).await.is_err());
        assert_eq!(
            read_cached_session(repo.cache(), id)
                .expect("existing cache")
                .model,
            "previous",
            "fallible inherent save must publish only after a durable commit"
        );
    }

    #[tokio::test]
    async fn blocked_replay_ambiguous_save_evicts_same_age_stale_approval_cache() {
        let id = "blocked-replay-ambiguous-save";
        let mut approved = Session::new(id, "model");
        approved.metadata.insert(
            crate::session_app::respond::PERMISSION_REEXECUTE_METADATA_KEY.into(),
            "call-1".into(),
        );
        let storage: Arc<dyn Storage> = Arc::new(AmbiguousReplaySaveStorage {
            persisted: Mutex::new(Some(approved.clone())),
        });
        let repo = test_repo(storage);
        cache_put(&repo, &approved);

        let mut resolved = approved.clone();
        resolved
            .metadata
            .remove(crate::session_app::respond::PERMISSION_REEXECUTE_METADATA_KEY);
        assert!(repo.save_replay_resolution(&mut resolved).await.is_err());
        assert!(read_cached_session(repo.cache(), id).is_none());
        let reloaded = repo.load_merged_checked(id).await.unwrap().unwrap();
        assert_eq!(reloaded.updated_at, approved.updated_at);
        assert!(!reloaded
            .metadata
            .contains_key(crate::session_app::respond::PERMISSION_REEXECUTE_METADATA_KEY));
    }

    #[tokio::test]
    async fn blocked_replay_precommit_save_error_reloads_retryable_approval() {
        let id = "blocked-replay-precommit-save";
        let mut approved = Session::new(id, "model");
        approved.metadata.insert(
            crate::session_app::respond::PERMISSION_REEXECUTE_METADATA_KEY.into(),
            "call-1".into(),
        );
        let storage: Arc<dyn Storage> = Arc::new(FailingSaveStorage {
            persisted: Mutex::new(Some(approved.clone())),
        });
        let repo = test_repo(storage);
        cache_put(&repo, &approved);

        let mut resolved = approved.clone();
        resolved
            .metadata
            .remove(crate::session_app::respond::PERMISSION_REEXECUTE_METADATA_KEY);
        assert!(repo.save_replay_resolution(&mut resolved).await.is_err());
        assert!(read_cached_session(repo.cache(), id).is_none());
        let reloaded = repo.load_merged_checked(id).await.unwrap().unwrap();
        assert_eq!(reloaded.updated_at, approved.updated_at);
        assert_eq!(
            reloaded
                .metadata
                .get(crate::session_app::respond::PERMISSION_REEXECUTE_METADATA_KEY)
                .map(String::as_str),
            Some("call-1")
        );
    }

    #[tokio::test]
    async fn save_and_cache_still_refreshes_cache_when_storage_fails() {
        let id = "save-and-cache-failure";
        let previous = Session::new(id, "previous");
        let storage: Arc<dyn Storage> = Arc::new(FailingSaveStorage {
            persisted: Mutex::new(Some(previous.clone())),
        });
        let repo = test_repo(storage);
        cache_put(&repo, &previous);

        let mut current = previous;
        current.model = "current".to_string();
        repo.save_and_cache(&mut current).await;
        assert_eq!(
            read_cached_session(repo.cache(), id)
                .expect("refreshed cache")
                .model,
            "current",
            "fire-and-forget save must retain its existing cache-on-failure behavior"
        );
    }

    #[tokio::test]
    async fn checkpoint_leaves_existing_cache_untouched_when_storage_fails() {
        let id = "checkpoint-failure";
        let previous = Session::new(id, "previous");
        let storage: Arc<dyn Storage> = Arc::new(FailingSaveStorage {
            persisted: Mutex::new(Some(previous.clone())),
        });
        let repo = test_repo(storage);
        cache_put(&repo, &previous);

        let mut current = previous.clone();
        current.model = "current".to_string();
        let result = bamboo_domain::RuntimeSessionPersistence::checkpoint_runtime_session(
            &repo,
            &mut current,
        )
        .await;
        assert!(result.is_err());
        assert_eq!(
            read_cached_session(repo.cache(), id)
                .expect("existing cache")
                .model,
            "previous",
            "checkpoint must publish only after a durable commit"
        );
    }

    #[tokio::test]
    async fn retrieval_window_checkpoint_publishes_the_exact_archived_snapshot() {
        let temp = tempfile::tempdir().unwrap();
        let storage: Arc<dyn Storage> = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .expect("SessionStoreV2"),
        );
        let repo = test_repo(storage.clone());
        let id = "retrieval-window-repository-checkpoint";
        let mut expected = Session::new(id, "model");
        expected.add_message(bamboo_agent_core::Message::user("archive candidate"));
        expected.add_message(bamboo_agent_core::Message::assistant("retain", None));
        storage.save_session(&expected).await.unwrap();
        cache_put(&repo, &expected);
        let mut staged = staged_retrieval_window_archive(&expected);

        let outcome = bamboo_domain::RuntimeSessionPersistence::checkpoint_retrieval_window(
            &repo,
            &expected,
            &mut staged,
        )
        .await
        .unwrap();

        assert_eq!(outcome, RetrievalWindowCheckpointOutcome::Committed);
        let durable = storage.load_session(id).await.unwrap().unwrap();
        let cached = read_cached_session(repo.cache(), id).expect("cached archived Session");
        for (tier, saved) in [("durable", durable), ("cache", cached)] {
            assert!(saved.messages[0].compressed, "tier={tier}");
            assert_eq!(
                saved.messages[0].compressed_by_event_id, staged.messages[0].compressed_by_event_id,
                "tier={tier}"
            );
            assert_eq!(
                serde_json::to_value(&saved).unwrap(),
                serde_json::to_value(&staged).unwrap(),
                "tier={tier}"
            );
        }
    }

    #[tokio::test]
    async fn legacy_clear_uses_durable_cas_and_never_erases_concurrent_append_from_stale_cache() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "legacy-cas-race";
        let expected = vec![serde_json::json!({"content": "first"})];

        let mut stale_cache = Session::new(id, "m");
        stale_cache.set_pending_injected_messages(expected.clone());
        cache_put(&repo, &stale_cache);

        let mut durable = stale_cache.clone();
        durable.set_pending_injected_messages(vec![
            serde_json::json!({"content": "first"}),
            serde_json::json!({"content": "concurrent"}),
        ]);
        storage.save_session(&durable).await.unwrap();

        let cleared = bamboo_domain::RuntimeSessionPersistence::clear_legacy_pending_messages(
            &repo, id, &expected,
        )
        .await
        .unwrap();
        assert!(!cleared, "the durable compare-and-clear must reject drift");
        assert_eq!(
            storage
                .load_session(id)
                .await
                .unwrap()
                .unwrap()
                .pending_injected_messages()
                .unwrap(),
            durable.pending_injected_messages().unwrap(),
            "the concurrent durable append must remain intact"
        );
        assert_eq!(
            read_cached_session(repo.cache(), id)
                .unwrap()
                .pending_injected_messages()
                .unwrap(),
            expected,
            "a failed CAS must not mutate the existing cache"
        );
    }

    #[tokio::test]
    async fn successful_legacy_clear_refreshes_stale_cache_from_durable_state() {
        let storage: Arc<dyn Storage> = Arc::new(MapStorage::default());
        let repo = test_repo(storage.clone());
        let id = "legacy-cas-success";
        let expected = vec![serde_json::json!({"content": "first"})];

        let mut stale_cache = Session::new(id, "stale-model");
        stale_cache.set_pending_injected_messages(expected.clone());
        cache_put(&repo, &stale_cache);

        let mut durable = Session::new(id, "durable-model");
        durable.set_pending_injected_messages(expected.clone());
        durable
            .metadata
            .insert("durable-only".to_string(), "keep".to_string());
        storage.save_session(&durable).await.unwrap();

        assert!(
            bamboo_domain::RuntimeSessionPersistence::clear_legacy_pending_messages(
                &repo, id, &expected,
            )
            .await
            .unwrap()
        );
        let cached = read_cached_session(repo.cache(), id).unwrap();
        assert!(!cached.has_pending_injected_messages());
        assert_eq!(cached.model, "durable-model");
        assert_eq!(
            cached.metadata.get("durable-only").map(String::as_str),
            Some("keep")
        );
    }

    #[tokio::test]
    async fn repository_final_save_publishes_host_checkpointed_parent_question() {
        use bamboo_domain::{
            FunctionCall, ParentQuestion, ParentQuestionCheckpointV1, PendingQuestion,
            PendingQuestionSource, ToolCall,
        };

        let temp = tempfile::tempdir().unwrap();
        let store = Arc::new(
            bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
                .await
                .unwrap(),
        );
        let storage: Arc<dyn Storage> = store.clone();
        let repo = test_repo(storage);
        let parent = Session::new("repository-question-parent", "model");
        store.save_session(&parent).await.unwrap();
        let mut child =
            Session::new_child_of("repository-question-child", &parent, "model", "assignment");
        child.add_message(bamboo_agent_core::Message::user("assignment"));
        child.set_last_run_status("running");
        store.save_session(&child).await.unwrap();
        cache_put(&repo, &child);

        let assistant = bamboo_agent_core::Message::assistant(
            "",
            Some(vec![ToolCall {
                id: "question-call".into(),
                tool_type: "function".into(),
                function: FunctionCall {
                    name: "AgenticQuestion".into(),
                    arguments: "{}".into(),
                },
            }]),
        );
        let result = bamboo_agent_core::Message::tool_result_with_status(
            "question-call",
            "Clarification needed: Which option?",
            true,
        );
        let pending = PendingQuestion {
            tool_call_id: "question-call".into(),
            tool_name: "AgenticQuestion".into(),
            question: "Which option?".into(),
            options: vec!["A".into(), "B".into()],
            allow_custom: true,
            source: PendingQuestionSource::AgenticClarification,
        };
        let observation = ParentQuestionCheckpointV1 {
            version: 1,
            prefix_message_count: child.messages.len(),
            prefix_digest: ParentQuestion::prefix_digest(&child.messages).unwrap(),
            suffix: vec![assistant, result.clone()],
            tool_call_id: pending.tool_call_id.clone(),
            tool_result_message_id: result.id.clone(),
            question_digest: ParentQuestion::question_digest(&pending, &result.id),
            pending,
        };
        let (_, question) = store
            .checkpoint_parent_question(&child, &parent, &observation)
            .await
            .unwrap()
            .unwrap();

        let mut old_run = child.clone();
        old_run.metadata.insert(
            "runtime.actor_parent_question_handoff".into(),
            "true".into(),
        );
        old_run.set_last_run_status("suspended");
        old_run.add_message(bamboo_agent_core::Message::assistant(
            "old terminal text",
            None,
        ));
        cache_put(&repo, &old_run);
        bamboo_domain::RuntimeSessionPersistence::save_runtime_session(&repo, &mut old_run)
            .await
            .unwrap();

        let cached = repo.load(&child.id).await.unwrap();
        let reopened = bamboo_storage::SessionStoreV2::new(temp.path().to_path_buf())
            .await
            .unwrap()
            .load_session(&child.id)
            .await
            .unwrap()
            .unwrap();
        for saved in [&cached, &reopened] {
            assert_eq!(saved.last_run_status().as_deref(), Some("suspended"));
            assert_eq!(saved.messages.len(), child.messages.len() + 2);
            assert!(!saved
                .messages
                .iter()
                .any(|message| message.content == "old terminal text"));
            assert_eq!(
                saved.pending_question.as_ref().unwrap().source,
                PendingQuestionSource::DirectParent
            );
            assert_eq!(
                ParentQuestion::for_pending(&parent, saved).unwrap().id,
                question.id
            );
        }
    }
}