rust_keypaths/

lib.rs

use std::sync::{Arc, Mutex, RwLock};
use std::marker::PhantomData;
use std::any::{Any, TypeId};
use std::rc::Rc;
use std::cell::RefCell;

#[cfg(feature = "tagged")]
use tagged_core::Tagged;

// ========== HELPER MACROS FOR KEYPATH CREATION ==========

/// Macro to create a `KeyPath` (readable, non-optional)
/// 
/// # Examples
/// 
/// ```rust
/// use rust_keypaths::keypath;
/// 
/// struct User { name: String, address: Address }
/// struct Address { street: String }
/// 
/// // Using a closure with type annotation
/// let kp = keypath!(|u: &User| &u.name);
/// 
/// // Nested field access
/// let kp = keypath!(|u: &User| &u.address.street);
/// 
/// // Or with automatic type inference
/// let kp = keypath!(|u| &u.name);
/// ```
#[macro_export]
macro_rules! keypath {
    // Accept a closure directly
    ($closure:expr) => {
        $crate::KeyPath::new($closure)
    };
}

/// Macro to create an `OptionalKeyPath` (readable, optional)
/// 
/// # Examples
/// 
/// ```rust
/// use rust_keypaths::opt_keypath;
/// 
/// struct User { metadata: Option<String>, address: Option<Address> }
/// struct Address { street: String }
/// 
/// // Using a closure with type annotation
/// let kp = opt_keypath!(|u: &User| u.metadata.as_ref());
/// 
/// // Nested field access through Option
/// let kp = opt_keypath!(|u: &User| u.address.as_ref().map(|a| &a.street));
/// 
/// // Or with automatic type inference
/// let kp = opt_keypath!(|u| u.metadata.as_ref());
/// ```
#[macro_export]
macro_rules! opt_keypath {
    // Accept a closure directly
    ($closure:expr) => {
        $crate::OptionalKeyPath::new($closure)
    };
}

/// Macro to create a `WritableKeyPath` (writable, non-optional)
/// 
/// # Examples
/// 
/// ```rust
/// use rust_keypaths::writable_keypath;
/// 
/// struct User { name: String, address: Address }
/// struct Address { street: String }
/// 
/// // Using a closure with type annotation
/// let kp = writable_keypath!(|u: &mut User| &mut u.name);
/// 
/// // Nested field access
/// let kp = writable_keypath!(|u: &mut User| &mut u.address.street);
/// 
/// // Or with automatic type inference
/// let kp = writable_keypath!(|u| &mut u.name);
/// ```
#[macro_export]
macro_rules! writable_keypath {
    // Accept a closure directly
    ($closure:expr) => {
        $crate::WritableKeyPath::new($closure)
    };
}

/// Macro to create a `WritableOptionalKeyPath` (writable, optional)
/// 
/// # Examples
/// 
/// ```rust
/// use rust_keypaths::writable_opt_keypath;
/// 
/// struct User { metadata: Option<String>, address: Option<Address> }
/// struct Address { street: String }
/// 
/// // Using a closure with type annotation
/// let kp = writable_opt_keypath!(|u: &mut User| u.metadata.as_mut());
/// 
/// // Nested field access through Option
/// let kp = writable_opt_keypath!(|u: &mut User| u.address.as_mut().map(|a| &mut a.street));
/// 
/// // Or with automatic type inference
/// let kp = writable_opt_keypath!(|u| u.metadata.as_mut());
/// ```
#[macro_export]
macro_rules! writable_opt_keypath {
    // Accept a closure directly
    ($closure:expr) => {
        $crate::WritableOptionalKeyPath::new($closure)
    };
}

// ========== BASE KEYPATH TYPES ==========

// Base KeyPath
#[derive(Clone)]
pub struct KeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> &'r Value,
{
    getter: F,
    _phantom: PhantomData<(Root, Value)>,
}

impl<Root, Value, F> KeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> &'r Value,
{
    pub fn new(getter: F) -> Self {
        Self {
            getter,
            _phantom: PhantomData,
        }
    }
    
    pub fn get<'r>(&self, root: &'r Root) -> &'r Value {
        (self.getter)(root)
}

    // Instance methods for unwrapping containers (automatically infers Target from Value::Target)
    // Box<T> -> T
    pub fn for_box<Target>(self) -> KeyPath<Root, Target, impl for<'r> Fn(&'r Root) -> &'r Target + 'static>
    where
        Value: std::ops::Deref<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        KeyPath {
            getter: move |root: &Root| {
                getter(root).deref()
            },
            _phantom: PhantomData,
        }
    }
    
    // Arc<T> -> T
    pub fn for_arc<Target>(self) -> KeyPath<Root, Target, impl for<'r> Fn(&'r Root) -> &'r Target + 'static>
    where
        Value: std::ops::Deref<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        KeyPath {
            getter: move |root: &Root| {
                getter(root).deref()
            },
            _phantom: PhantomData,
        }
    }
    
    // Rc<T> -> T
    pub fn for_rc<Target>(self) -> KeyPath<Root, Target, impl for<'r> Fn(&'r Root) -> &'r Target + 'static>
    where
        Value: std::ops::Deref<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        KeyPath {
            getter: move |root: &Root| {
                getter(root).deref()
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Arc<Root> when Value is Sized (not a container)
    pub fn for_arc_root(self) -> OptionalKeyPath<Arc<Root>, Value, impl for<'r> Fn(&'r Arc<Root>) -> Option<&'r Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |arc: &Arc<Root>| {
                Some(getter(arc.as_ref()))
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Box<Root> when Value is Sized (not a container)
    pub fn for_box_root(self) -> OptionalKeyPath<Box<Root>, Value, impl for<'r> Fn(&'r Box<Root>) -> Option<&'r Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |boxed: &Box<Root>| {
                Some(getter(boxed.as_ref()))
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Rc<Root> when Value is Sized (not a container)
    pub fn for_rc_root(self) -> OptionalKeyPath<Rc<Root>, Value, impl for<'r> Fn(&'r Rc<Root>) -> Option<&'r Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |rc: &Rc<Root>| {
                Some(getter(rc.as_ref()))
            },
            _phantom: PhantomData,
        }
    }
    
    /// Adapt this keypath to work with Result<Root, E> instead of Root
    /// This unwraps the Result and applies the keypath to the Ok value
    pub fn for_result<E>(self) -> OptionalKeyPath<Result<Root, E>, Value, impl for<'r> Fn(&'r Result<Root, E>) -> Option<&'r Value> + 'static>
    where
        F: 'static,
        Root: 'static,
        Value: 'static,
        E: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |result: &Result<Root, E>| {
                result.as_ref().ok().map(|root| getter(root))
            },
            _phantom: PhantomData,
        }
    }
    
    /// Convert a KeyPath to OptionalKeyPath for chaining
    /// This allows non-optional keypaths to be chained with then()
    pub fn to_optional(self) -> OptionalKeyPath<Root, Value, impl for<'r> Fn(&'r Root) -> Option<&'r Value> + 'static>
    where
        F: 'static,
    {
        let getter = self.getter;
        OptionalKeyPath::new(move |root: &Root| Some(getter(root)))
    }
    
    /// Execute a closure with a reference to the value inside an Option
    pub fn with_option<Callback, R>(&self, option: &Option<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        option.as_ref().map(|root| {
            let value = self.get(root);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside a Result
    pub fn with_result<Callback, R, E>(&self, result: &Result<Root, E>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        result.as_ref().ok().map(|root| {
            let value = self.get(root);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside a Box
    pub fn with_box<Callback, R>(&self, boxed: &Box<Root>, f: Callback) -> R
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        let value = self.get(boxed);
        f(value)
    }
    
    /// Execute a closure with a reference to the value inside an Arc
    pub fn with_arc<Callback, R>(&self, arc: &Arc<Root>, f: Callback) -> R
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        let value = self.get(arc);
        f(value)
    }
    
    /// Execute a closure with a reference to the value inside an Rc
    pub fn with_rc<Callback, R>(&self, rc: &Rc<Root>, f: Callback) -> R
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        let value = self.get(rc);
        f(value)
    }
    
    /// Execute a closure with a reference to the value inside a RefCell
    pub fn with_refcell<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        refcell.try_borrow().ok().map(|borrow| {
            let value = self.get(&*borrow);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside a Mutex
    pub fn with_mutex<Callback, R>(&self, mutex: &Mutex<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        mutex.lock().ok().map(|guard| {
            let value = self.get(&*guard);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside an RwLock
    pub fn with_rwlock<Callback, R>(&self, rwlock: &RwLock<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        rwlock.read().ok().map(|guard| {
            let value = self.get(&*guard);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside an Arc<RwLock<Root>>
    pub fn with_arc_rwlock<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        arc_rwlock.read().ok().map(|guard| {
            let value = self.get(&*guard);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside an Arc<Mutex<Root>>
    pub fn with_arc_mutex<Callback, R>(&self, arc_mutex: &Arc<Mutex<Root>>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        arc_mutex.lock().ok().map(|guard| {
            let value = self.get(&*guard);
            f(value)
        })
    }
    
    #[cfg(feature = "tagged")]
    /// Adapt this keypath to work with Tagged<Root, Tag> instead of Root
    /// This unwraps the Tagged wrapper and applies the keypath to the inner value
    pub fn for_tagged<Tag>(self) -> KeyPath<Tagged<Root, Tag>, Value, impl for<'r> Fn(&'r Tagged<Root, Tag>) -> &'r Value + 'static>
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        F: 'static,
        Root: 'static,
        Value: 'static,
        Tag: 'static,
    {
        use std::ops::Deref;
        let getter = self.getter;
        
        KeyPath {
            getter: move |tagged: &Tagged<Root, Tag>| {
                getter(tagged.deref())
            },
            _phantom: PhantomData,
        }
    }
    
    #[cfg(feature = "tagged")]
    /// Execute a closure with a reference to the value inside a Tagged
    /// This avoids cloning by working with references directly
    pub fn with_tagged<Tag, Callback, R>(&self, tagged: &Tagged<Root, Tag>, f: Callback) -> R
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        Callback: FnOnce(&Value) -> R,
    {
        use std::ops::Deref;
        let value = self.get(tagged.deref());
        f(value)
    }
    
    /// Adapt this keypath to work with Option<Root> instead of Root
    /// This converts the KeyPath to an OptionalKeyPath and unwraps the Option
    pub fn for_option(self) -> OptionalKeyPath<Option<Root>, Value, impl for<'r> Fn(&'r Option<Root>) -> Option<&'r Value> + 'static>
    where
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |opt: &Option<Root>| {
                opt.as_ref().map(|root| getter(root))
            },
            _phantom: PhantomData,
        }
    }
    
}

// Extension methods for KeyPath to support Arc<RwLock> and Arc<Mutex> directly
impl<Root, Value, F> KeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> &'r Value,
{
    /// Execute a closure with a reference to the value inside an Arc<RwLock<Root>>
    /// This is a convenience method that works directly with Arc<RwLock<T>>
    pub fn with_arc_rwlock_direct<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        arc_rwlock.read().ok().map(|guard| {
            let value = self.get(&*guard);
            f(value)
        })
    }
    
    /// Execute a closure with a reference to the value inside an Arc<Mutex<Root>>
    /// This is a convenience method that works directly with Arc<Mutex<T>>
    pub fn with_arc_mutex_direct<Callback, R>(&self, arc_mutex: &Arc<Mutex<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        arc_mutex.lock().ok().map(|guard| {
            let value = self.get(&*guard);
            f(value)
        })
    }
}

// Utility function for slice access (kept as standalone function)
pub fn for_slice<T>() -> impl for<'r> Fn(&'r [T], usize) -> Option<&'r T> {
    |slice: &[T], index: usize| slice.get(index)
}

// Container access utilities
pub mod containers {
    use super::{OptionalKeyPath, WritableOptionalKeyPath, KeyPath, WritableKeyPath};
    use std::collections::{HashMap, BTreeMap, HashSet, BTreeSet, VecDeque, LinkedList, BinaryHeap};
    use std::sync::{Mutex, RwLock, Weak as StdWeak, Arc};
    use std::rc::{Weak as RcWeak, Rc};
    use std::ops::{Deref, DerefMut};

    #[cfg(feature = "parking_lot")]
    use parking_lot::{Mutex as ParkingMutex, RwLock as ParkingRwLock};

    #[cfg(feature = "tagged")]
    use tagged_core::Tagged;

    /// Create a keypath for indexed access in Vec<T>
    pub fn for_vec_index<T>(index: usize) -> OptionalKeyPath<Vec<T>, T, impl for<'r> Fn(&'r Vec<T>) -> Option<&'r T>> {
        OptionalKeyPath::new(move |vec: &Vec<T>| vec.get(index))
    }

    /// Create a keypath for indexed access in VecDeque<T>
    pub fn for_vecdeque_index<T>(index: usize) -> OptionalKeyPath<VecDeque<T>, T, impl for<'r> Fn(&'r VecDeque<T>) -> Option<&'r T>> {
        OptionalKeyPath::new(move |deque: &VecDeque<T>| deque.get(index))
    }

    /// Create a keypath for indexed access in LinkedList<T>
    pub fn for_linkedlist_index<T>(index: usize) -> OptionalKeyPath<LinkedList<T>, T, impl for<'r> Fn(&'r LinkedList<T>) -> Option<&'r T>> {
        OptionalKeyPath::new(move |list: &LinkedList<T>| {
            list.iter().nth(index)
        })
    }

    /// Create a keypath for key-based access in HashMap<K, V>
    pub fn for_hashmap_key<K, V>(key: K) -> OptionalKeyPath<HashMap<K, V>, V, impl for<'r> Fn(&'r HashMap<K, V>) -> Option<&'r V>>
    where
        K: std::hash::Hash + Eq + Clone + 'static,
        V: 'static,
    {
        OptionalKeyPath::new(move |map: &HashMap<K, V>| map.get(&key))
    }

    /// Create a keypath for key-based access in BTreeMap<K, V>
    pub fn for_btreemap_key<K, V>(key: K) -> OptionalKeyPath<BTreeMap<K, V>, V, impl for<'r> Fn(&'r BTreeMap<K, V>) -> Option<&'r V>>
    where
        K: Ord + Clone + 'static,
        V: 'static,
    {
        OptionalKeyPath::new(move |map: &BTreeMap<K, V>| map.get(&key))
    }

    /// Create a keypath for getting a value from HashSet<T> (returns Option<&T>)
    pub fn for_hashset_get<T>(value: T) -> OptionalKeyPath<HashSet<T>, T, impl for<'r> Fn(&'r HashSet<T>) -> Option<&'r T>>
    where
        T: std::hash::Hash + Eq + Clone + 'static,
    {
        OptionalKeyPath::new(move |set: &HashSet<T>| set.get(&value))
    }

    /// Create a keypath for checking membership in BTreeSet<T>
    pub fn for_btreeset_get<T>(value: T) -> OptionalKeyPath<BTreeSet<T>, T, impl for<'r> Fn(&'r BTreeSet<T>) -> Option<&'r T>>
    where
        T: Ord + Clone + 'static,
    {
        OptionalKeyPath::new(move |set: &BTreeSet<T>| set.get(&value))
    }

    /// Create a keypath for peeking at the top of BinaryHeap<T>
    pub fn for_binaryheap_peek<T>() -> OptionalKeyPath<BinaryHeap<T>, T, impl for<'r> Fn(&'r BinaryHeap<T>) -> Option<&'r T>>
    where
        T: Ord + 'static,
    {
        OptionalKeyPath::new(|heap: &BinaryHeap<T>| heap.peek())
    }

    // ========== WRITABLE VERSIONS ==========

    /// Create a writable keypath for indexed access in Vec<T>
    pub fn for_vec_index_mut<T>(index: usize) -> WritableOptionalKeyPath<Vec<T>, T, impl for<'r> Fn(&'r mut Vec<T>) -> Option<&'r mut T>> {
        WritableOptionalKeyPath::new(move |vec: &mut Vec<T>| vec.get_mut(index))
    }

    /// Create a writable keypath for indexed access in VecDeque<T>
    pub fn for_vecdeque_index_mut<T>(index: usize) -> WritableOptionalKeyPath<VecDeque<T>, T, impl for<'r> Fn(&'r mut VecDeque<T>) -> Option<&'r mut T>> {
        WritableOptionalKeyPath::new(move |deque: &mut VecDeque<T>| deque.get_mut(index))
    }

    /// Create a writable keypath for indexed access in LinkedList<T>
    pub fn for_linkedlist_index_mut<T>(index: usize) -> WritableOptionalKeyPath<LinkedList<T>, T, impl for<'r> Fn(&'r mut LinkedList<T>) -> Option<&'r mut T>> {
        WritableOptionalKeyPath::new(move |list: &mut LinkedList<T>| {
            // LinkedList doesn't have get_mut, so we need to iterate
            let mut iter = list.iter_mut();
            iter.nth(index)
        })
    }

    /// Create a writable keypath for key-based access in HashMap<K, V>
    pub fn for_hashmap_key_mut<K, V>(key: K) -> WritableOptionalKeyPath<HashMap<K, V>, V, impl for<'r> Fn(&'r mut HashMap<K, V>) -> Option<&'r mut V>>
    where
        K: std::hash::Hash + Eq + Clone + 'static,
        V: 'static,
    {
        WritableOptionalKeyPath::new(move |map: &mut HashMap<K, V>| map.get_mut(&key))
    }

    /// Create a writable keypath for key-based access in BTreeMap<K, V>
    pub fn for_btreemap_key_mut<K, V>(key: K) -> WritableOptionalKeyPath<BTreeMap<K, V>, V, impl for<'r> Fn(&'r mut BTreeMap<K, V>) -> Option<&'r mut V>>
    where
        K: Ord + Clone + 'static,
        V: 'static,
    {
        WritableOptionalKeyPath::new(move |map: &mut BTreeMap<K, V>| map.get_mut(&key))
    }

    /// Create a writable keypath for getting a mutable value from HashSet<T>
    /// Note: HashSet doesn't support mutable access to elements, but we provide it for consistency
    pub fn for_hashset_get_mut<T>(value: T) -> WritableOptionalKeyPath<HashSet<T>, T, impl for<'r> Fn(&'r mut HashSet<T>) -> Option<&'r mut T>>
    where
        T: std::hash::Hash + Eq + Clone + 'static,
    {
        WritableOptionalKeyPath::new(move |set: &mut HashSet<T>| {
            // HashSet doesn't have get_mut, so we need to check and return None
            // This is a limitation of HashSet's design
            if set.contains(&value) {
                // We can't return a mutable reference to the value in the set
                // This is a fundamental limitation of HashSet
                None
            } else {
                None
            }
        })
    }

    /// Create a writable keypath for getting a mutable value from BTreeSet<T>
    /// Note: BTreeSet doesn't support mutable access to elements, but we provide it for consistency
    pub fn for_btreeset_get_mut<T>(value: T) -> WritableOptionalKeyPath<BTreeSet<T>, T, impl for<'r> Fn(&'r mut BTreeSet<T>) -> Option<&'r mut T>>
    where
        T: Ord + Clone + 'static,
    {
        WritableOptionalKeyPath::new(move |set: &mut BTreeSet<T>| {
            // BTreeSet doesn't have get_mut, so we need to check and return None
            // This is a limitation of BTreeSet's design
            if set.contains(&value) {
                // We can't return a mutable reference to the value in the set
                // This is a fundamental limitation of BTreeSet
                None
            } else {
                None
            }
        })
    }

    /// Create a writable keypath for peeking at the top of BinaryHeap<T>
    /// Note: BinaryHeap.peek_mut() returns PeekMut which is a guard type.
    /// Due to Rust's borrowing rules, we cannot return &mut T directly from PeekMut.
    /// This function returns None as BinaryHeap doesn't support direct mutable access
    /// through keypaths. Use heap.peek_mut() directly for mutable access.
    pub fn for_binaryheap_peek_mut<T>() -> WritableOptionalKeyPath<BinaryHeap<T>, T, impl for<'r> Fn(&'r mut BinaryHeap<T>) -> Option<&'r mut T>>
    where
        T: Ord + 'static,
    {
        // BinaryHeap.peek_mut() returns PeekMut which is a guard type that owns the mutable reference.
        // We cannot return &mut T from it due to lifetime constraints.
        // This is a fundamental limitation - use heap.peek_mut() directly instead.
        WritableOptionalKeyPath::new(|_heap: &mut BinaryHeap<T>| {
            None
        })
    }

    // ========== SYNCHRONIZATION PRIMITIVES ==========
    // Note: Mutex and RwLock return guards that own the lock, not references.
    // We cannot create keypaths that return references from guards due to lifetime constraints.
    // These helper functions are provided for convenience, but direct lock()/read()/write() calls are recommended.

    /// Helper function to lock a Mutex<T> and access its value
    /// Returns None if the mutex is poisoned
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn lock_mutex<T>(mutex: &Mutex<T>) -> Option<std::sync::MutexGuard<'_, T>> {
        mutex.lock().ok()
    }

    /// Helper function to read-lock an RwLock<T> and access its value
    /// Returns None if the lock is poisoned
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn read_rwlock<T>(rwlock: &RwLock<T>) -> Option<std::sync::RwLockReadGuard<'_, T>> {
        rwlock.read().ok()
    }

    /// Helper function to write-lock an RwLock<T> and access its value
    /// Returns None if the lock is poisoned
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn write_rwlock<T>(rwlock: &RwLock<T>) -> Option<std::sync::RwLockWriteGuard<'_, T>> {
        rwlock.write().ok()
    }

    /// Helper function to lock an Arc<Mutex<T>> and access its value
    /// Returns None if the mutex is poisoned
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn lock_arc_mutex<T>(arc_mutex: &Arc<Mutex<T>>) -> Option<std::sync::MutexGuard<'_, T>> {
        arc_mutex.lock().ok()
    }

    /// Helper function to read-lock an Arc<RwLock<T>> and access its value
    /// Returns None if the lock is poisoned
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn read_arc_rwlock<T>(arc_rwlock: &Arc<RwLock<T>>) -> Option<std::sync::RwLockReadGuard<'_, T>> {
        arc_rwlock.read().ok()
    }

    /// Helper function to write-lock an Arc<RwLock<T>> and access its value
    /// Returns None if the lock is poisoned
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn write_arc_rwlock<T>(arc_rwlock: &Arc<RwLock<T>>) -> Option<std::sync::RwLockWriteGuard<'_, T>> {
        arc_rwlock.write().ok()
    }

    /// Helper function to upgrade a Weak<T> to Arc<T>
    /// Returns None if the Arc has been dropped
    /// Note: This returns an owned Arc, not a reference, so it cannot be used in keypaths directly
    pub fn upgrade_weak<T>(weak: &StdWeak<T>) -> Option<Arc<T>> {
        weak.upgrade()
    }

    /// Helper function to upgrade an Rc::Weak<T> to Rc<T>
    /// Returns None if the Rc has been dropped
    /// Note: This returns an owned Rc, not a reference, so it cannot be used in keypaths directly
    pub fn upgrade_rc_weak<T>(weak: &RcWeak<T>) -> Option<Rc<T>> {
        weak.upgrade()
    }

    #[cfg(feature = "parking_lot")]
    /// Helper function to lock a parking_lot::Mutex<T> and access its value
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn lock_parking_mutex<T>(mutex: &ParkingMutex<T>) -> parking_lot::MutexGuard<'_, T> {
        mutex.lock()
    }

    #[cfg(feature = "parking_lot")]
    /// Helper function to read-lock a parking_lot::RwLock<T> and access its value
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn read_parking_rwlock<T>(rwlock: &ParkingRwLock<T>) -> parking_lot::RwLockReadGuard<'_, T> {
        rwlock.read()
    }

    #[cfg(feature = "parking_lot")]
    /// Helper function to write-lock a parking_lot::RwLock<T> and access its value
    /// Note: This returns a guard, not a reference, so it cannot be used in keypaths directly
    pub fn write_parking_rwlock<T>(rwlock: &ParkingRwLock<T>) -> parking_lot::RwLockWriteGuard<'_, T> {
        rwlock.write()
    }

    #[cfg(feature = "tagged")]
    /// Create a keypath for accessing the inner value of Tagged<Tag, T>
    /// Tagged implements Deref, so we can access the inner value directly
    pub fn for_tagged<Tag, T>() -> KeyPath<Tagged<Tag, T>, T, impl for<'r> Fn(&'r Tagged<Tag, T>) -> &'r T>
    where
        Tagged<Tag, T>: std::ops::Deref<Target = T>,
        Tag: 'static,
        T: 'static,
    {
        KeyPath::new(|tagged: &Tagged<Tag, T>| tagged.deref())
    }

    #[cfg(feature = "tagged")]
    /// Create a writable keypath for accessing the inner value of Tagged<Tag, T>
    /// Tagged implements DerefMut, so we can access the inner value directly
    pub fn for_tagged_mut<Tag, T>() -> WritableKeyPath<Tagged<Tag, T>, T, impl for<'r> Fn(&'r mut Tagged<Tag, T>) -> &'r mut T>
    where
        Tagged<Tag, T>: std::ops::DerefMut<Target = T>,
        Tag: 'static,
        T: 'static,
    {
        WritableKeyPath::new(|tagged: &mut Tagged<Tag, T>| tagged.deref_mut())
    }
}

// OptionalKeyPath for Option<T>
#[derive(Clone)]
pub struct OptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> Option<&'r Value>,
{
    getter: F,
    _phantom: PhantomData<(Root, Value)>,
}

impl<Root, Value, F> OptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> Option<&'r Value>,
{
    pub fn new(getter: F) -> Self {
        Self {
            getter,
            _phantom: PhantomData,
        }
    }
    
    pub fn get<'r>(&self, root: &'r Root) -> Option<&'r Value> {
        (self.getter)(root)
    }
    
    // Swift-like operator for chaining OptionalKeyPath
    pub fn then<SubValue, G>(
        self,
        next: OptionalKeyPath<Value, SubValue, G>,
    ) -> OptionalKeyPath<Root, SubValue, impl for<'r> Fn(&'r Root) -> Option<&'r SubValue>>
    where
        G: for<'r> Fn(&'r Value) -> Option<&'r SubValue>,
        F: 'static,
        G: 'static,
        Value: 'static,
    {
        let first = self.getter;
        let second = next.getter;
        
        OptionalKeyPath::new(move |root: &Root| {
            first(root).and_then(|value| second(value))
        })
    }
    
    // Instance methods for unwrapping containers from Option<Container<T>>
    // Option<Box<T>> -> Option<&T> (type automatically inferred from Value::Target)
    pub fn for_box<Target>(self) -> OptionalKeyPath<Root, Target, impl for<'r> Fn(&'r Root) -> Option<&'r Target> + 'static>
    where
        Value: std::ops::Deref<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |root: &Root| {
                getter(root).map(|boxed| boxed.deref())
            },
            _phantom: PhantomData,
        }
    }
    
    // Option<Arc<T>> -> Option<&T> (type automatically inferred from Value::Target)
    pub fn for_arc<Target>(self) -> OptionalKeyPath<Root, Target, impl for<'r> Fn(&'r Root) -> Option<&'r Target> + 'static>
    where
        Value: std::ops::Deref<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |root: &Root| {
                getter(root).map(|arc| arc.deref())
            },
            _phantom: PhantomData,
        }
    }
    
    // Option<Rc<T>> -> Option<&T> (type automatically inferred from Value::Target)
    pub fn for_rc<Target>(self) -> OptionalKeyPath<Root, Target, impl for<'r> Fn(&'r Root) -> Option<&'r Target> + 'static>
    where
        Value: std::ops::Deref<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |root: &Root| {
                getter(root).map(|rc| rc.deref())
            },
            _phantom: PhantomData,
        }
    }
    
    #[cfg(feature = "tagged")]
    /// Adapt this keypath to work with Tagged<Root, Tag> instead of Root
    /// This unwraps the Tagged wrapper and applies the keypath to the inner value
    pub fn for_tagged<Tag>(self) -> OptionalKeyPath<Tagged<Root, Tag>, Value, impl for<'r> Fn(&'r Tagged<Root, Tag>) -> Option<&'r Value> + 'static>
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        F: 'static,
        Root: 'static,
        Value: 'static,
        Tag: 'static,
    {
        use std::ops::Deref;
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |tagged: &Tagged<Root, Tag>| {
                getter(tagged.deref())
            },
            _phantom: PhantomData,
        }
    }
    
    #[cfg(feature = "tagged")]
    /// Execute a closure with a reference to the value inside a Tagged
    /// This avoids cloning by working with references directly
    pub fn with_tagged<Tag, Callback, R>(&self, tagged: &Tagged<Root, Tag>, f: Callback) -> Option<R>
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        use std::ops::Deref;
        self.get(tagged.deref()).map(|value| f(value))
    }
    
    /// Adapt this keypath to work with Option<Root> instead of Root
    /// This unwraps the Option and applies the keypath to the inner value
    pub fn for_option(self) -> OptionalKeyPath<Option<Root>, Value, impl for<'r> Fn(&'r Option<Root>) -> Option<&'r Value> + 'static>
    where
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |opt: &Option<Root>| {
                opt.as_ref().and_then(|root| getter(root))
            },
            _phantom: PhantomData,
        }
    }
    
    /// Adapt this keypath to work with Result<Root, E> instead of Root
    /// This unwraps the Result and applies the keypath to the Ok value
    pub fn for_result<E>(self) -> OptionalKeyPath<Result<Root, E>, Value, impl for<'r> Fn(&'r Result<Root, E>) -> Option<&'r Value> + 'static>
    where
        F: 'static,
        Root: 'static,
        Value: 'static,
        E: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |result: &Result<Root, E>| {
                result.as_ref().ok().and_then(|root| getter(root))
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Arc<Root> when Value is Sized (not a container)
    pub fn for_arc_root(self) -> OptionalKeyPath<Arc<Root>, Value, impl for<'r> Fn(&'r Arc<Root>) -> Option<&'r Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |arc: &Arc<Root>| {
                getter(arc.as_ref())
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Rc<Root> when Value is Sized (not a container)
    pub fn for_rc_root(self) -> OptionalKeyPath<Rc<Root>, Value, impl for<'r> Fn(&'r Rc<Root>) -> Option<&'r Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        OptionalKeyPath {
            getter: move |rc: &Rc<Root>| {
                getter(rc.as_ref())
            },
            _phantom: PhantomData,
        }
    }
    
    /// Execute a closure with a reference to the value inside an Option
    pub fn with_option<Callback, R>(&self, option: &Option<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        option.as_ref().and_then(|root| {
            self.get(root).map(|value| f(value))
        })
    }
    
    /// Execute a closure with a reference to the value inside a Mutex
    pub fn with_mutex<Callback, R>(&self, mutex: &Mutex<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        mutex.lock().ok().and_then(|guard| {
            self.get(&*guard).map(|value| f(value))
        })
    }
    
    /// Execute a closure with a reference to the value inside an RwLock
    pub fn with_rwlock<Callback, R>(&self, rwlock: &RwLock<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        rwlock.read().ok().and_then(|guard| {
            self.get(&*guard).map(|value| f(value))
        })
    }
    
    /// Execute a closure with a reference to the value inside an Arc<RwLock<Root>>
    pub fn with_arc_rwlock<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        arc_rwlock.read().ok().and_then(|guard| {
            self.get(&*guard).map(|value| f(value))
        })
    }
    
    /// Execute a closure with a reference to the value inside an Arc<RwLock<Root>>
    /// This is a convenience method that works directly with Arc<RwLock<T>>
    /// Unlike with_arc_rwlock, this doesn't require F: Clone
    pub fn with_arc_rwlock_direct<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        arc_rwlock.read().ok().and_then(|guard| {
            self.get(&*guard).map(|value| f(value))
        })
    }
    
    /// Execute a closure with a reference to the value inside an Arc<Mutex<Root>>
    pub fn with_arc_mutex<Callback, R>(&self, arc_mutex: &Arc<Mutex<Root>>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&Value) -> R,
    {
        arc_mutex.lock().ok().and_then(|guard| {
            self.get(&*guard).map(|value| f(value))
        })
    }
}


// WritableKeyPath for mutable access
#[derive(Clone)]
pub struct WritableKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> &'r mut Value,
{
    getter: F,
    _phantom: PhantomData<(Root, Value)>,
}

impl<Root, Value, F> WritableKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> &'r mut Value,
{
    pub fn new(getter: F) -> Self {
        Self {
            getter,
            _phantom: PhantomData,
        }
    }
    
    pub fn get_mut<'r>(&self, root: &'r mut Root) -> &'r mut Value {
        (self.getter)(root)
    }
    
    /// Adapt this keypath to work with Result<Root, E> instead of Root
    /// This unwraps the Result and applies the keypath to the Ok value
    pub fn for_result<E>(self) -> WritableOptionalKeyPath<Result<Root, E>, Value, impl for<'r> Fn(&'r mut Result<Root, E>) -> Option<&'r mut Value> + 'static>
    where
        F: 'static,
        Root: 'static,
        Value: 'static,
        E: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |result: &mut Result<Root, E>| {
                result.as_mut().ok().map(|root| getter(root))
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Box<Root> when Value is Sized (not a container)
    pub fn for_box_root(self) -> WritableKeyPath<Box<Root>, Value, impl for<'r> Fn(&'r mut Box<Root>) -> &'r mut Value + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableKeyPath {
            getter: move |boxed: &mut Box<Root>| {
                getter(boxed.as_mut())
            },
            _phantom: PhantomData,
        }
    }
    
    /// Convert a WritableKeyPath to WritableOptionalKeyPath for chaining
    /// This allows non-optional writable keypaths to be chained with then()
    pub fn to_optional(self) -> WritableOptionalKeyPath<Root, Value, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut Value> + 'static>
    where
        F: 'static,
    {
        let getter = self.getter;
        WritableOptionalKeyPath::new(move |root: &mut Root| Some(getter(root)))
    }
    
    // Instance methods for unwrapping containers (automatically infers Target from Value::Target)
    // Box<T> -> T
    pub fn for_box<Target>(self) -> WritableKeyPath<Root, Target, impl for<'r> Fn(&'r mut Root) -> &'r mut Target + 'static>
    where
        Value: std::ops::DerefMut<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableKeyPath {
            getter: move |root: &mut Root| {
                getter(root).deref_mut()
            },
            _phantom: PhantomData,
        }
    }
    
    // Arc<T> -> T (Note: Arc doesn't support mutable access, but we provide it for consistency)
    // This will require interior mutability patterns
    pub fn for_arc<Target>(self) -> WritableKeyPath<Root, Target, impl for<'r> Fn(&'r mut Root) -> &'r mut Target + 'static>
    where
        Value: std::ops::DerefMut<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableKeyPath {
            getter: move |root: &mut Root| {
                getter(root).deref_mut()
            },
            _phantom: PhantomData,
        }
    }
    
    // Rc<T> -> T (Note: Rc doesn't support mutable access, but we provide it for consistency)
    // This will require interior mutability patterns
    pub fn for_rc<Target>(self) -> WritableKeyPath<Root, Target, impl for<'r> Fn(&'r mut Root) -> &'r mut Target + 'static>
    where
        Value: std::ops::DerefMut<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableKeyPath {
            getter: move |root: &mut Root| {
                getter(root).deref_mut()
            },
            _phantom: PhantomData,
        }
    }
    
    /// Execute a closure with a mutable reference to the value inside a Box
    pub fn with_box_mut<Callback, R>(&self, boxed: &mut Box<Root>, f: Callback) -> R
    where
        F: Clone,
        Callback: FnOnce(&mut Value) -> R,
    {
        let value = self.get_mut(boxed);
        f(value)
    }
    
    /// Execute a closure with a mutable reference to the value inside a Result
    pub fn with_result_mut<Callback, R, E>(&self, result: &mut Result<Root, E>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&mut Value) -> R,
    {
        result.as_mut().ok().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }
    
    /// Execute a closure with a mutable reference to the value inside an Option
    pub fn with_option_mut<Callback, R>(&self, option: &mut Option<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&mut Value) -> R,
    {
        option.as_mut().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }
    
    /// Execute a closure with a mutable reference to the value inside a RefCell
    pub fn with_refcell_mut<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&mut Value) -> R,
    {
        refcell.try_borrow_mut().ok().map(|mut borrow| {
            let value = self.get_mut(&mut *borrow);
            f(value)
        })
    }
    
    /// Execute a closure with a mutable reference to the value inside a Mutex
    pub fn with_mutex_mut<Callback, R>(&self, mutex: &mut Mutex<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&mut Value) -> R,
    {
        mutex.get_mut().ok().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }
    
    /// Execute a closure with a mutable reference to the value inside an RwLock
    pub fn with_rwlock_mut<Callback, R>(&self, rwlock: &mut RwLock<Root>, f: Callback) -> Option<R>
    where
        F: Clone,
        Callback: FnOnce(&mut Value) -> R,
    {
        rwlock.write().ok().map(|mut guard| {
            let value = self.get_mut(&mut *guard);
            f(value)
        })
    }
}

// WritableOptionalKeyPath for failable mutable access
#[derive(Clone)]
pub struct WritableOptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> Option<&'r mut Value>,
{
    getter: F,
    _phantom: PhantomData<(Root, Value)>,
}

impl<Root, Value, F> WritableOptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> Option<&'r mut Value>,
{
    pub fn new(getter: F) -> Self {
        Self {
            getter,
            _phantom: PhantomData,
        }
    }
    
    pub fn get_mut<'r>(&self, root: &'r mut Root) -> Option<&'r mut Value> {
        (self.getter)(root)
    }
    
    // Swift-like operator for chaining WritableOptionalKeyPath
    pub fn then<SubValue, G>(
        self,
        next: WritableOptionalKeyPath<Value, SubValue, G>,
    ) -> WritableOptionalKeyPath<Root, SubValue, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut SubValue>>
    where
        G: for<'r> Fn(&'r mut Value) -> Option<&'r mut SubValue>,
        F: 'static,
        G: 'static,
        Value: 'static,
    {
        let first = self.getter;
        let second = next.getter;
        
        WritableOptionalKeyPath::new(move |root: &mut Root| {
            first(root).and_then(|value| second(value))
        })
    }
    
    // Instance methods for unwrapping containers from Option<Container<T>>
    // Option<Box<T>> -> Option<&mut T> (type automatically inferred from Value::Target)
    pub fn for_box<Target>(self) -> WritableOptionalKeyPath<Root, Target, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut Target> + 'static>
    where
        Value: std::ops::DerefMut<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |root: &mut Root| {
                getter(root).map(|boxed| boxed.deref_mut())
            },
            _phantom: PhantomData,
        }
    }
    
    // Option<Arc<T>> -> Option<&mut T> (type automatically inferred from Value::Target)
    pub fn for_arc<Target>(self) -> WritableOptionalKeyPath<Root, Target, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut Target> + 'static>
    where
        Value: std::ops::DerefMut<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |root: &mut Root| {
                getter(root).map(|arc| arc.deref_mut())
            },
            _phantom: PhantomData,
        }
    }
    
    // Option<Rc<T>> -> Option<&mut T> (type automatically inferred from Value::Target)
    pub fn for_rc<Target>(self) -> WritableOptionalKeyPath<Root, Target, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut Target> + 'static>
    where
        Value: std::ops::DerefMut<Target = Target>,
        F: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |root: &mut Root| {
                getter(root).map(|rc| rc.deref_mut())
            },
            _phantom: PhantomData,
        }
    }
    
    /// Adapt this keypath to work with Result<Root, E> instead of Root
    /// This unwraps the Result and applies the keypath to the Ok value
    pub fn for_result<E>(self) -> WritableOptionalKeyPath<Result<Root, E>, Value, impl for<'r> Fn(&'r mut Result<Root, E>) -> Option<&'r mut Value> + 'static>
    where
        F: 'static,
        Root: 'static,
        Value: 'static,
        E: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |result: &mut Result<Root, E>| {
                result.as_mut().ok().and_then(|root| getter(root))
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Box<Root> when Value is Sized (not a container)
    pub fn for_box_root(self) -> WritableOptionalKeyPath<Box<Root>, Value, impl for<'r> Fn(&'r mut Box<Root>) -> Option<&'r mut Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |boxed: &mut Box<Root>| {
                getter(boxed.as_mut())
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Arc<Root> when Value is Sized (not a container)
    pub fn for_arc_root(self) -> WritableOptionalKeyPath<Arc<Root>, Value, impl for<'r> Fn(&'r mut Arc<Root>) -> Option<&'r mut Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |arc: &mut Arc<Root>| {
                // Arc doesn't support mutable access without interior mutability
                // This will always return None, but we provide it for API consistency
                None
            },
            _phantom: PhantomData,
        }
    }
    
    // Overload: Adapt root type to Rc<Root> when Value is Sized (not a container)
    pub fn for_rc_root(self) -> WritableOptionalKeyPath<Rc<Root>, Value, impl for<'r> Fn(&'r mut Rc<Root>) -> Option<&'r mut Value> + 'static>
    where
        Value: Sized,
        F: 'static,
        Root: 'static,
        Value: 'static,
    {
        let getter = self.getter;
        
        WritableOptionalKeyPath {
            getter: move |rc: &mut Rc<Root>| {
                // Rc doesn't support mutable access without interior mutability
                // This will always return None, but we provide it for API consistency
                None
            },
            _phantom: PhantomData,
        }
    }
    
    // Static method for Option<T> -> Option<&mut T>
    pub fn for_option<T>() -> WritableOptionalKeyPath<Option<T>, T, impl for<'r> Fn(&'r mut Option<T>) -> Option<&'r mut T>> {
        WritableOptionalKeyPath::new(|opt: &mut Option<T>| opt.as_mut())
    }
}

// Enum-specific keypaths
/// EnumKeyPath - A keypath for enum variants that supports both extraction and embedding
/// Uses generic type parameters instead of dynamic dispatch for zero-cost abstraction
/// 
/// This struct serves dual purpose:
/// 1. As a concrete keypath instance: `EnumKeyPath<Enum, Variant, ExtractFn, EmbedFn>`
/// 2. As a namespace for static factory methods: `EnumKeyPath::readable_enum(...)`
pub struct EnumKeyPath<Enum = (), Variant = (), ExtractFn = fn(&()) -> Option<&()>, EmbedFn = fn(()) -> ()> 
where
    ExtractFn: for<'r> Fn(&'r Enum) -> Option<&'r Variant> + 'static,
    EmbedFn: Fn(Variant) -> Enum + 'static,
{
    extractor: OptionalKeyPath<Enum, Variant, ExtractFn>,
    embedder: EmbedFn,
}

impl<Enum, Variant, ExtractFn, EmbedFn> EnumKeyPath<Enum, Variant, ExtractFn, EmbedFn>
where
    ExtractFn: for<'r> Fn(&'r Enum) -> Option<&'r Variant> + 'static,
    EmbedFn: Fn(Variant) -> Enum + 'static,
{
    /// Create a new EnumKeyPath with extractor and embedder functions
    pub fn new(
        extractor: ExtractFn,
        embedder: EmbedFn,
    ) -> Self {
        Self {
            extractor: OptionalKeyPath::new(extractor),
            embedder,
        }
    }
    
    /// Extract the value from an enum variant
    pub fn get<'r>(&self, enum_value: &'r Enum) -> Option<&'r Variant> {
        self.extractor.get(enum_value)
    }
    
    /// Embed a value into the enum variant
    pub fn embed(&self, value: Variant) -> Enum {
        (self.embedder)(value)
    }
    
    /// Get the underlying OptionalKeyPath for composition
    pub fn as_optional(&self) -> &OptionalKeyPath<Enum, Variant, ExtractFn> {
        &self.extractor
    }
    
    /// Convert to OptionalKeyPath (loses embedding capability but gains composition)
    pub fn to_optional(self) -> OptionalKeyPath<Enum, Variant, ExtractFn> {
        self.extractor
    }
}

// Static factory methods for EnumKeyPath
impl EnumKeyPath {
    /// Create a readable enum keypath with both extraction and embedding
    /// Returns an EnumKeyPath that supports both get() and embed() operations
    pub fn readable_enum<Enum, Variant, ExtractFn, EmbedFn>(
        embedder: EmbedFn,
        extractor: ExtractFn,
    ) -> EnumKeyPath<Enum, Variant, ExtractFn, EmbedFn>
    where
        ExtractFn: for<'r> Fn(&'r Enum) -> Option<&'r Variant> + 'static,
        EmbedFn: Fn(Variant) -> Enum + 'static,
    {
        EnumKeyPath::new(extractor, embedder)
    }
    
    /// Extract from a specific enum variant
    pub fn for_variant<Enum, Variant, ExtractFn>(
        extractor: ExtractFn
    ) -> OptionalKeyPath<Enum, Variant, impl for<'r> Fn(&'r Enum) -> Option<&'r Variant>>
    where
        ExtractFn: Fn(&Enum) -> Option<&Variant>,
    {
        OptionalKeyPath::new(extractor)
    }
    
    /// Match against multiple variants (returns a tagged union)
    pub fn for_match<Enum, Output, MatchFn>(
        matcher: MatchFn
    ) -> KeyPath<Enum, Output, impl for<'r> Fn(&'r Enum) -> &'r Output>
    where
        MatchFn: Fn(&Enum) -> &Output,
    {
        KeyPath::new(matcher)
    }
    
    /// Extract from Result<T, E> - Ok variant
    pub fn for_ok<T, E>() -> OptionalKeyPath<Result<T, E>, T, impl for<'r> Fn(&'r Result<T, E>) -> Option<&'r T>> {
        OptionalKeyPath::new(|result: &Result<T, E>| result.as_ref().ok())
    }
    
    /// Extract from Result<T, E> - Err variant
    pub fn for_err<T, E>() -> OptionalKeyPath<Result<T, E>, E, impl for<'r> Fn(&'r Result<T, E>) -> Option<&'r E>> {
        OptionalKeyPath::new(|result: &Result<T, E>| result.as_ref().err())
    }
    
    /// Extract from Option<T> - Some variant
    pub fn for_some<T>() -> OptionalKeyPath<Option<T>, T, impl for<'r> Fn(&'r Option<T>) -> Option<&'r T>> {
        OptionalKeyPath::new(|opt: &Option<T>| opt.as_ref())
    }
    
    /// Extract from Option<T> - Some variant (alias for for_some for consistency)
    pub fn for_option<T>() -> OptionalKeyPath<Option<T>, T, impl for<'r> Fn(&'r Option<T>) -> Option<&'r T>> {
        OptionalKeyPath::new(|opt: &Option<T>| opt.as_ref())
    }
    
    /// Unwrap Box<T> -> T
    pub fn for_box<T>() -> KeyPath<Box<T>, T, impl for<'r> Fn(&'r Box<T>) -> &'r T> {
        KeyPath::new(|b: &Box<T>| b.as_ref())
    }
    
    /// Unwrap Arc<T> -> T
    pub fn for_arc<T>() -> KeyPath<Arc<T>, T, impl for<'r> Fn(&'r Arc<T>) -> &'r T> {
        KeyPath::new(|arc: &Arc<T>| arc.as_ref())
    }
    
    /// Unwrap Rc<T> -> T
    pub fn for_rc<T>() -> KeyPath<std::rc::Rc<T>, T, impl for<'r> Fn(&'r std::rc::Rc<T>) -> &'r T> {
        KeyPath::new(|rc: &std::rc::Rc<T>| rc.as_ref())
    }

    /// Unwrap Box<T> -> T (mutable)
    pub fn for_box_mut<T>() -> WritableKeyPath<Box<T>, T, impl for<'r> Fn(&'r mut Box<T>) -> &'r mut T> {
        WritableKeyPath::new(|b: &mut Box<T>| b.as_mut())
    }

    // Note: Arc<T> and Rc<T> don't support direct mutable access without interior mutability
    // (e.g., Arc<Mutex<T>> or Rc<RefCell<T>>). These methods are not provided as they
    // would require unsafe code or interior mutability patterns.
}

// Helper to create enum variant keypaths with type inference
pub fn variant_of<Enum, Variant, F>(extractor: F) -> OptionalKeyPath<Enum, Variant, F>
where
    F: for<'r> Fn(&'r Enum) -> Option<&'r Variant>,
{
    OptionalKeyPath::new(extractor)
}

// ========== PARTIAL KEYPATHS (Hide Value Type) ==========

/// PartialKeyPath - Hides the Value type but keeps Root visible
/// Useful for storing keypaths in collections without knowing the exact Value type
/// 
/// # Why PhantomData<Root>?
/// 
/// `PhantomData<Root>` is needed because:
/// 1. The `Root` type parameter is not actually stored in the struct (only used in the closure)
/// 2. Rust needs to know the generic type parameter for:
///    - Type checking at compile time
///    - Ensuring correct usage (e.g., `PartialKeyPath<User>` can only be used with `&User`)
///    - Preventing mixing different Root types
/// 3. Without `PhantomData`, Rust would complain that `Root` is unused
/// 4. `PhantomData` is zero-sized - it adds no runtime overhead
#[derive(Clone)]
pub struct PartialKeyPath<Root> {
    getter: Rc<dyn for<'r> Fn(&'r Root) -> &'r dyn Any>,
    value_type_id: TypeId,
    _phantom: PhantomData<Root>,
}

impl<Root> PartialKeyPath<Root> {
    pub fn new<Value>(keypath: KeyPath<Root, Value, impl for<'r> Fn(&'r Root) -> &'r Value + 'static>) -> Self
    where
        Value: Any + 'static,
        Root: 'static,
    {
        let value_type_id = TypeId::of::<Value>();
        let getter = Rc::new(keypath.getter);
        
        Self {
            getter: Rc::new(move |root: &Root| {
                let value: &Value = getter(root);
                value as &dyn Any
            }),
            value_type_id,
            _phantom: PhantomData,
        }
    }
    
    /// Create a PartialKeyPath from a concrete KeyPath
    /// Alias for `new()` for consistency with `from()` pattern
    pub fn from<Value>(keypath: KeyPath<Root, Value, impl for<'r> Fn(&'r Root) -> &'r Value + 'static>) -> Self
    where
        Value: Any + 'static,
        Root: 'static,
    {
        Self::new(keypath)
    }
    
    pub fn get<'r>(&self, root: &'r Root) -> &'r dyn Any {
        (self.getter)(root)
    }
    
    /// Get the TypeId of the Value type
    pub fn value_type_id(&self) -> TypeId {
        self.value_type_id
    }
    
    /// Try to downcast the result to a specific type
    pub fn get_as<'a, Value: Any>(&self, root: &'a Root) -> Option<&'a Value> {
        if self.value_type_id == TypeId::of::<Value>() {
            self.get(root).downcast_ref::<Value>()
        } else {
            None
        }
    }
    
    /// Get a human-readable name for the value type
    /// Returns a string representation of the TypeId
    pub fn kind_name(&self) -> String {
        format!("{:?}", self.value_type_id)
    }
}

/// PartialOptionalKeyPath - Hides the Value type but keeps Root visible
/// Useful for storing optional keypaths in collections without knowing the exact Value type
/// 
/// # Why PhantomData<Root>?
/// 
/// See `PartialKeyPath` documentation for explanation of why `PhantomData` is needed.
#[derive(Clone)]
pub struct PartialOptionalKeyPath<Root> {
    getter: Rc<dyn for<'r> Fn(&'r Root) -> Option<&'r dyn Any>>,
    value_type_id: TypeId,
    _phantom: PhantomData<Root>,
}

impl<Root> PartialOptionalKeyPath<Root> {
    pub fn new<Value>(keypath: OptionalKeyPath<Root, Value, impl for<'r> Fn(&'r Root) -> Option<&'r Value> + 'static>) -> Self
    where
        Value: Any + 'static,
        Root: 'static,
    {
        let value_type_id = TypeId::of::<Value>();
        let getter = Rc::new(keypath.getter);
        
        Self {
            getter: Rc::new(move |root: &Root| {
                getter(root).map(|value: &Value| value as &dyn Any)
            }),
            value_type_id,
            _phantom: PhantomData,
        }
    }
    
    pub fn get<'r>(&self, root: &'r Root) -> Option<&'r dyn Any> {
        (self.getter)(root)
    }
    
    /// Get the TypeId of the Value type
    pub fn value_type_id(&self) -> TypeId {
        self.value_type_id
    }
    
    /// Try to downcast the result to a specific type
    pub fn get_as<'a, Value: Any>(&self, root: &'a Root) -> Option<Option<&'a Value>> {
        if self.value_type_id == TypeId::of::<Value>() {
            self.get(root).map(|any| any.downcast_ref::<Value>())
        } else {
            None
        }
    }
    
    /// Chain with another PartialOptionalKeyPath
    /// Note: This requires the Value type of the first keypath to match the Root type of the second
    /// For type-erased chaining, consider using AnyKeyPath instead
    pub fn then<MidValue>(
        self,
        next: PartialOptionalKeyPath<MidValue>,
    ) -> PartialOptionalKeyPath<Root>
    where
        MidValue: Any + 'static,
        Root: 'static,
    {
        let first = self.getter;
        let second = next.getter;
        let value_type_id = next.value_type_id;
        
        PartialOptionalKeyPath {
            getter: Rc::new(move |root: &Root| {
                first(root).and_then(|any| {
                    if let Some(mid_value) = any.downcast_ref::<MidValue>() {
                        second(mid_value)
                    } else {
                        None
                    }
                })
            }),
            value_type_id,
            _phantom: PhantomData,
        }
    }
}

/// PartialWritableKeyPath - Hides the Value type but keeps Root visible (writable)
/// 
/// # Why PhantomData<Root>?
/// 
/// See `PartialKeyPath` documentation for explanation of why `PhantomData` is needed.
#[derive(Clone)]
pub struct PartialWritableKeyPath<Root> {
    getter: Rc<dyn for<'r> Fn(&'r mut Root) -> &'r mut dyn Any>,
    value_type_id: TypeId,
    _phantom: PhantomData<Root>,
}

impl<Root> PartialWritableKeyPath<Root> {
    pub fn new<Value>(keypath: WritableKeyPath<Root, Value, impl for<'r> Fn(&'r mut Root) -> &'r mut Value + 'static>) -> Self
    where
        Value: Any + 'static,
        Root: 'static,
    {
        let value_type_id = TypeId::of::<Value>();
        let getter = Rc::new(keypath.getter);
        
        Self {
            getter: Rc::new(move |root: &mut Root| {
                let value: &mut Value = getter(root);
                value as &mut dyn Any
            }),
            value_type_id,
            _phantom: PhantomData,
        }
    }
    
    /// Create a PartialWritableKeyPath from a concrete WritableKeyPath
    /// Alias for `new()` for consistency with `from()` pattern
    pub fn from<Value>(keypath: WritableKeyPath<Root, Value, impl for<'r> Fn(&'r mut Root) -> &'r mut Value + 'static>) -> Self
    where
        Value: Any + 'static,
        Root: 'static,
    {
        Self::new(keypath)
    }
    
    pub fn get_mut<'r>(&self, root: &'r mut Root) -> &'r mut dyn Any {
        (self.getter)(root)
    }
    
    /// Get the TypeId of the Value type
    pub fn value_type_id(&self) -> TypeId {
        self.value_type_id
    }
    
    /// Try to downcast the result to a specific type
    pub fn get_mut_as<'a, Value: Any>(&self, root: &'a mut Root) -> Option<&'a mut Value> {
        if self.value_type_id == TypeId::of::<Value>() {
            self.get_mut(root).downcast_mut::<Value>()
        } else {
            None
        }
    }
}

/// PartialWritableOptionalKeyPath - Hides the Value type but keeps Root visible (writable optional)
/// 
/// # Why PhantomData<Root>?
/// 
/// See `PartialKeyPath` documentation for explanation of why `PhantomData` is needed.
#[derive(Clone)]
pub struct PartialWritableOptionalKeyPath<Root> {
    getter: Rc<dyn for<'r> Fn(&'r mut Root) -> Option<&'r mut dyn Any>>,
    value_type_id: TypeId,
    _phantom: PhantomData<Root>,
}

impl<Root> PartialWritableOptionalKeyPath<Root> {
    pub fn new<Value>(keypath: WritableOptionalKeyPath<Root, Value, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut Value> + 'static>) -> Self
    where
        Value: Any + 'static,
        Root: 'static,
    {
        let value_type_id = TypeId::of::<Value>();
        let getter = Rc::new(keypath.getter);
        
        Self {
            getter: Rc::new(move |root: &mut Root| {
                getter(root).map(|value: &mut Value| value as &mut dyn Any)
            }),
            value_type_id,
            _phantom: PhantomData,
        }
    }
    
    pub fn get_mut<'r>(&self, root: &'r mut Root) -> Option<&'r mut dyn Any> {
        (self.getter)(root)
    }
    
    /// Get the TypeId of the Value type
    pub fn value_type_id(&self) -> TypeId {
        self.value_type_id
    }
    
    /// Try to downcast the result to a specific type
    pub fn get_mut_as<'a, Value: Any>(&self, root: &'a mut Root) -> Option<Option<&'a mut Value>> {
        if self.value_type_id == TypeId::of::<Value>() {
            self.get_mut(root).map(|any| any.downcast_mut::<Value>())
        } else {
            None
        }
    }
}

// ========== ANY KEYPATHS (Hide Both Root and Value Types) ==========

/// AnyKeyPath - Hides both Root and Value types
/// Equivalent to Swift's AnyKeyPath
/// Useful for storing keypaths in collections without knowing either type
/// 
/// # Why No PhantomData?
/// 
/// Unlike `PartialKeyPath`, `AnyKeyPath` doesn't need `PhantomData` because:
/// - Both `Root` and `Value` types are completely erased
/// - We store `TypeId` instead for runtime type checking
/// - The type information is encoded in the closure's behavior, not the struct
/// - There's no generic type parameter to track at compile time
#[derive(Clone)]
pub struct AnyKeyPath {
    getter: Rc<dyn for<'r> Fn(&'r dyn Any) -> Option<&'r dyn Any>>,
    root_type_id: TypeId,
    value_type_id: TypeId,
}

impl AnyKeyPath {
    pub fn new<Root, Value>(keypath: OptionalKeyPath<Root, Value, impl for<'r> Fn(&'r Root) -> Option<&'r Value> + 'static>) -> Self
    where
        Root: Any + 'static,
        Value: Any + 'static,
    {
        let root_type_id = TypeId::of::<Root>();
        let value_type_id = TypeId::of::<Value>();
        let getter = keypath.getter;
        
        Self {
            getter: Rc::new(move |any: &dyn Any| {
                if let Some(root) = any.downcast_ref::<Root>() {
                    getter(root).map(|value: &Value| value as &dyn Any)
                } else {
                    None
                }
            }),
            root_type_id,
            value_type_id,
        }
    }
    
    /// Create an AnyKeyPath from a concrete OptionalKeyPath
    /// Alias for `new()` for consistency with `from()` pattern
    pub fn from<Root, Value>(keypath: OptionalKeyPath<Root, Value, impl for<'r> Fn(&'r Root) -> Option<&'r Value> + 'static>) -> Self
    where
        Root: Any + 'static,
        Value: Any + 'static,
    {
        Self::new(keypath)
    }
    
    pub fn get<'r>(&self, root: &'r dyn Any) -> Option<&'r dyn Any> {
        (self.getter)(root)
    }
    
    /// Get the TypeId of the Root type
    pub fn root_type_id(&self) -> TypeId {
        self.root_type_id
    }
    
    /// Get the TypeId of the Value type
    pub fn value_type_id(&self) -> TypeId {
        self.value_type_id
    }
    
    /// Try to get the value with type checking
    pub fn get_as<'a, Root: Any, Value: Any>(&self, root: &'a Root) -> Option<Option<&'a Value>> {
        if self.root_type_id == TypeId::of::<Root>() && self.value_type_id == TypeId::of::<Value>() {
            self.get(root as &dyn Any).map(|any| any.downcast_ref::<Value>())
        } else {
            None
        }
    }
    
    /// Get a human-readable name for the value type
    /// Returns a string representation of the TypeId
    pub fn kind_name(&self) -> String {
        format!("{:?}", self.value_type_id)
    }
}

/// AnyWritableKeyPath - Hides both Root and Value types (writable)
#[derive(Clone)]
pub struct AnyWritableKeyPath {
    getter: Rc<dyn for<'r> Fn(&'r mut dyn Any) -> Option<&'r mut dyn Any>>,
    root_type_id: TypeId,
    value_type_id: TypeId,
}

impl AnyWritableKeyPath {
    pub fn new<Root, Value>(keypath: WritableOptionalKeyPath<Root, Value, impl for<'r> Fn(&'r mut Root) -> Option<&'r mut Value> + 'static>) -> Self
    where
        Root: Any + 'static,
        Value: Any + 'static,
    {
        let root_type_id = TypeId::of::<Root>();
        let value_type_id = TypeId::of::<Value>();
        let getter = keypath.getter;
        
        Self {
            getter: Rc::new(move |any: &mut dyn Any| {
                if let Some(root) = any.downcast_mut::<Root>() {
                    getter(root).map(|value: &mut Value| value as &mut dyn Any)
                } else {
                    None
                }
            }),
            root_type_id,
            value_type_id,
        }
    }
    
    pub fn get_mut<'r>(&self, root: &'r mut dyn Any) -> Option<&'r mut dyn Any> {
        (self.getter)(root)
    }
    
    /// Get the TypeId of the Root type
    pub fn root_type_id(&self) -> TypeId {
        self.root_type_id
    }
    
    /// Get the TypeId of the Value type
    pub fn value_type_id(&self) -> TypeId {
        self.value_type_id
    }
    
    /// Try to get the value with type checking
    pub fn get_mut_as<'a, Root: Any, Value: Any>(&self, root: &'a mut Root) -> Option<Option<&'a mut Value>> {
        if self.root_type_id == TypeId::of::<Root>() && self.value_type_id == TypeId::of::<Value>() {
            self.get_mut(root as &mut dyn Any).map(|any| any.downcast_mut::<Value>())
        } else {
            None
        }
    }
}

// Conversion methods from concrete keypaths to partial/any keypaths
impl<Root, Value, F> KeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> &'r Value + 'static,
    Root: 'static,
    Value: Any + 'static,
{
    /// Convert to PartialKeyPath (hides Value type)
    pub fn to_partial(self) -> PartialKeyPath<Root> {
        PartialKeyPath::new(self)
    }
    
    /// Alias for `to_partial()` - converts to PartialKeyPath
    pub fn to(self) -> PartialKeyPath<Root> {
        self.to_partial()
    }
}

impl<Root, Value, F> OptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> Option<&'r Value> + 'static,
    Root: Any + 'static,
    Value: Any + 'static,
{
    /// Convert to PartialOptionalKeyPath (hides Value type)
    pub fn to_partial(self) -> PartialOptionalKeyPath<Root> {
        PartialOptionalKeyPath::new(self)
    }
    
    /// Convert to AnyKeyPath (hides both Root and Value types)
    pub fn to_any(self) -> AnyKeyPath {
        AnyKeyPath::new(self)
    }
    
    /// Convert to PartialOptionalKeyPath (alias for `to_partial()`)
    pub fn to(self) -> PartialOptionalKeyPath<Root> {
        self.to_partial()
    }
}

impl<Root, Value, F> WritableKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> &'r mut Value + 'static,
    Root: 'static,
    Value: Any + 'static,
{
    /// Convert to PartialWritableKeyPath (hides Value type)
    pub fn to_partial(self) -> PartialWritableKeyPath<Root> {
        PartialWritableKeyPath::new(self)
    }
    
    /// Alias for `to_partial()` - converts to PartialWritableKeyPath
    pub fn to(self) -> PartialWritableKeyPath<Root> {
        self.to_partial()
    }
}

impl<Root, Value, F> WritableOptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> Option<&'r mut Value> + 'static,
    Root: Any + 'static,
    Value: Any + 'static,
{
    /// Convert to PartialWritableOptionalKeyPath (hides Value type)
    pub fn to_partial(self) -> PartialWritableOptionalKeyPath<Root> {
        PartialWritableOptionalKeyPath::new(self)
    }
    
    /// Convert to AnyWritableKeyPath (hides both Root and Value types)
    pub fn to_any(self) -> AnyWritableKeyPath {
        AnyWritableKeyPath::new(self)
    }
    
    /// Convert to PartialWritableOptionalKeyPath (alias for `to_partial()`)
    pub fn to(self) -> PartialWritableOptionalKeyPath<Root> {
        self.to_partial()
    }
}

#[cfg(test)]
mod tests {
    use super::*;
    use std::sync::atomic::{AtomicUsize, Ordering};
    use std::rc::Rc;

    // Global counter to track memory allocations/deallocations
    static ALLOC_COUNT: AtomicUsize = AtomicUsize::new(0);
    static DEALLOC_COUNT: AtomicUsize = AtomicUsize::new(0);

    // Type that panics on clone to detect unwanted cloning
    #[derive(Debug)]
    struct NoCloneType {
        id: usize,
        data: String,
    }

    impl NoCloneType {
        fn new(data: String) -> Self {
            ALLOC_COUNT.fetch_add(1, Ordering::SeqCst);
            Self {
                id: ALLOC_COUNT.load(Ordering::SeqCst),
                data,
            }
        }
    }

    impl Clone for NoCloneType {
        fn clone(&self) -> Self {
            eprintln!("[DEBUG] NoCloneType should not be cloned! ID: {}", self.id);
            unreachable!("NoCloneType should not be cloned! ID: {}", self.id);
        }
    }

    impl Drop for NoCloneType {
        fn drop(&mut self) {
            DEALLOC_COUNT.fetch_add(1, Ordering::SeqCst);
        }
    }

    // Helper functions for testing memory management
    fn reset_memory_counters() {
        ALLOC_COUNT.store(0, Ordering::SeqCst);
        DEALLOC_COUNT.store(0, Ordering::SeqCst);
    }

    fn get_alloc_count() -> usize {
        ALLOC_COUNT.load(Ordering::SeqCst)
    }

    fn get_dealloc_count() -> usize {
        DEALLOC_COUNT.load(Ordering::SeqCst)
    }

// Usage example
#[derive(Debug)]
struct User {
    name: String,
    metadata: Option<Box<UserMetadata>>,
    friends: Vec<Arc<User>>,
}

#[derive(Debug)]
struct UserMetadata {
    created_at: String,
}

fn some_fn() {
        let akash = User {
        name: "Alice".to_string(),
        metadata: Some(Box::new(UserMetadata {
            created_at: "2024-01-01".to_string(),
        })),
        friends: vec![
            Arc::new(User {
                name: "Bob".to_string(),
                metadata: None,
                friends: vec![],
            }),
        ],
    };
    
    // Create keypaths
    let name_kp = KeyPath::new(|u: &User| &u.name);
    let metadata_kp = OptionalKeyPath::new(|u: &User| u.metadata.as_ref());
    let friends_kp = KeyPath::new(|u: &User| &u.friends);
    
    // Use them
        println!("Name: {}", name_kp.get(&akash));
    
        if let Some(metadata) = metadata_kp.get(&akash) {
        println!("Has metadata: {:?}", metadata);
    }
    
    // Access first friend's name
        if let Some(first_friend) = akash.friends.get(0) {
        println!("First friend: {}", name_kp.get(first_friend));
    }
    
        // Access metadata through Box using for_box()
    let created_at_kp = KeyPath::new(|m: &UserMetadata| &m.created_at);
    
        if let Some(metadata) = akash.metadata.as_ref() {
            // Use for_box() to unwrap Box<UserMetadata> to &UserMetadata
            let boxed_metadata: &Box<UserMetadata> = metadata;
            let unwrapped = boxed_metadata.as_ref();
            println!("Created at: {:?}", created_at_kp.get(unwrapped));
        }
    }

    #[test]
    fn test_name() {
        some_fn();
    }
    
    #[test]
    fn test_no_cloning_on_keypath_operations() {
        reset_memory_counters();
        
        // Create a value that panics on clone
        let value = NoCloneType::new("test".to_string());
        let boxed = Box::new(value);
        
        // Create keypath - should not clone
        let kp = KeyPath::new(|b: &Box<NoCloneType>| b.as_ref());
        
        // Access value - should not clone
        let _ref = kp.get(&boxed);
        
        // Clone the keypath itself (this is allowed)
        let _kp_clone = kp.clone();
        
        // Access again - should not clone the value
        let _ref2 = _kp_clone.get(&boxed);
        
        // Verify no panics occurred (if we got here, no cloning happened)
        assert_eq!(get_alloc_count(), 1);
    }
    
    #[test]
    fn test_no_cloning_on_optional_keypath_operations() {
        reset_memory_counters();
        
        let value = NoCloneType::new("test".to_string());
        let opt = Some(Box::new(value));
        
        // Create optional keypath
        let okp = OptionalKeyPath::new(|o: &Option<Box<NoCloneType>>| o.as_ref());
        
        // Access - should not clone
        let _ref = okp.get(&opt);
        
        // Clone keypath (allowed)
        let _okp_clone = okp.clone();
        
        // Chain operations - should not clone values
        let chained = okp.then(OptionalKeyPath::new(|b: &Box<NoCloneType>| Some(b.as_ref())));
        let _ref2 = chained.get(&opt);
        
        assert_eq!(get_alloc_count(), 1);
    }
    
    #[test]
    fn test_memory_release() {
        reset_memory_counters();
        
        {
            let value = NoCloneType::new("test".to_string());
            let boxed = Box::new(value);
            let kp = KeyPath::new(|b: &Box<NoCloneType>| b.as_ref());
            
            // Use the keypath
            let _ref = kp.get(&boxed);
            
            // boxed goes out of scope here
        }
        
        // After drop, memory should be released
        // Note: This is a best-effort check since drop timing can vary
        assert_eq!(get_alloc_count(), 1);
        // Deallocation happens when the value is dropped
        // We can't reliably test exact timing, but we verify the counter exists
    }
    
    #[test]
    fn test_keypath_clone_does_not_clone_underlying_data() {
        reset_memory_counters();
        
        let value = NoCloneType::new("data".to_string());
        let rc_value = Rc::new(value);
        
        // Create keypath
        let kp = KeyPath::new(|r: &Rc<NoCloneType>| r.as_ref());
        
        // Clone keypath multiple times
        let kp1 = kp.clone();
        let kp2 = kp.clone();
        let kp3 = kp1.clone();
        
        // All should work without cloning the underlying data
        let _ref1 = kp.get(&rc_value);
        let _ref2 = kp1.get(&rc_value);
        let _ref3 = kp2.get(&rc_value);
        let _ref4 = kp3.get(&rc_value);
        
        // Only one allocation should have happened
        assert_eq!(get_alloc_count(), 1);
    }
    
    #[test]
    fn test_optional_keypath_chaining_no_clone() {
        reset_memory_counters();
        
        let value = NoCloneType::new("value1".to_string());
        
        struct Container {
            inner: Option<Box<NoCloneType>>,
        }
        
        let container = Container {
            inner: Some(Box::new(value)),
        };
        
        // Create chained keypath
        let kp1 = OptionalKeyPath::new(|c: &Container| c.inner.as_ref());
        let kp2 = OptionalKeyPath::new(|b: &Box<NoCloneType>| Some(b.as_ref()));
        
        // Chain them - should not clone
        let chained = kp1.then(kp2);
        
        // Use chained keypath
        let _result = chained.get(&container);
        
        // Should only have one allocation
        assert_eq!(get_alloc_count(), 1);
    }
    
    #[test]
    fn test_for_box_no_clone() {
        reset_memory_counters();
        
        let value = NoCloneType::new("test".to_string());
        let boxed = Box::new(value);
        let opt_boxed = Some(boxed);
        
        // Create keypath with for_box
        let kp = OptionalKeyPath::new(|o: &Option<Box<NoCloneType>>| o.as_ref());
        let unwrapped = kp.for_box();
        
        // Access - should not clone
        let _ref = unwrapped.get(&opt_boxed);
        
        assert_eq!(get_alloc_count(), 1);
    }
    
    // ========== MACRO USAGE EXAMPLES ==========
    
    #[derive(Debug, PartialEq)]
    struct TestUser {
        name: String,
        age: u32,
        metadata: Option<String>,
        address: Option<TestAddress>,
    }
    
    #[derive(Debug, PartialEq)]
    struct TestAddress {
        street: String,
        city: String,
        country: Option<TestCountry>,
    }
    
    #[derive(Debug, PartialEq)]
    struct TestCountry {
        name: String,
    }
    
    #[test]
    fn test_keypath_macro() {
        let user = TestUser {
            name: "Alice".to_string(),
            age: 30,
            metadata: None,
            address: None,
        };
        
        // Simple field access using closure
        let name_kp = keypath!(|u: &TestUser| &u.name);
        assert_eq!(name_kp.get(&user), "Alice");
        
        // Nested field access
        let user_with_address = TestUser {
            name: "Bob".to_string(),
            age: 25,
            metadata: None,
            address: Some(TestAddress {
                street: "123 Main St".to_string(),
                city: "New York".to_string(),
                country: None,
            }),
        };
        
        let street_kp = keypath!(|u: &TestUser| &u.address.as_ref().unwrap().street);
        assert_eq!(street_kp.get(&user_with_address), "123 Main St");
        
        // Deeper nesting
        let user_with_country = TestUser {
            name: "Charlie".to_string(),
            age: 35,
            metadata: None,
            address: Some(TestAddress {
                street: "456 Oak Ave".to_string(),
                city: "London".to_string(),
                country: Some(TestCountry {
                    name: "UK".to_string(),
                }),
            }),
        };
        
        let country_name_kp = keypath!(|u: &TestUser| &u.address.as_ref().unwrap().country.as_ref().unwrap().name);
        assert_eq!(country_name_kp.get(&user_with_country), "UK");
        
        // Fallback: using closure
        let age_kp = keypath!(|u: &TestUser| &u.age);
        assert_eq!(age_kp.get(&user), &30);
    }
    
    #[test]
    fn test_opt_keypath_macro() {
        let user = TestUser {
            name: "Alice".to_string(),
            age: 30,
            metadata: Some("admin".to_string()),
            address: None,
        };
        
        // Simple Option field access using closure
        let metadata_kp = opt_keypath!(|u: &TestUser| u.metadata.as_ref());
        assert_eq!(metadata_kp.get(&user), Some(&"admin".to_string()));
        
        // None case
        let user_no_metadata = TestUser {
            name: "Bob".to_string(),
            age: 25,
            metadata: None,
            address: None,
        };
        assert_eq!(metadata_kp.get(&user_no_metadata), None);
        
        // Nested Option access
        let user_with_address = TestUser {
            name: "Charlie".to_string(),
            age: 35,
            metadata: None,
            address: Some(TestAddress {
                street: "789 Pine Rd".to_string(),
                city: "Paris".to_string(),
                country: None,
            }),
        };
        
        let street_kp = opt_keypath!(|u: &TestUser| u.address.as_ref().map(|a| &a.street));
        assert_eq!(street_kp.get(&user_with_address), Some(&"789 Pine Rd".to_string()));
        
        // Deeper nesting through Options
        let user_with_country = TestUser {
            name: "David".to_string(),
            age: 40,
            metadata: None,
            address: Some(TestAddress {
                street: "321 Elm St".to_string(),
                city: "Tokyo".to_string(),
                country: Some(TestCountry {
                    name: "Japan".to_string(),
                }),
            }),
        };
        
        let country_name_kp = opt_keypath!(|u: &TestUser| u.address.as_ref().and_then(|a| a.country.as_ref().map(|c| &c.name)));
        assert_eq!(country_name_kp.get(&user_with_country), Some(&"Japan".to_string()));
        
        // Fallback: using closure
        let metadata_kp2 = opt_keypath!(|u: &TestUser| u.metadata.as_ref());
        assert_eq!(metadata_kp2.get(&user), Some(&"admin".to_string()));
    }
    
    #[test]
    fn test_writable_keypath_macro() {
        let mut user = TestUser {
            name: "Alice".to_string(),
            age: 30,
            metadata: None,
            address: None,
        };
        
        // Simple field mutation using closure
        let name_kp = writable_keypath!(|u: &mut TestUser| &mut u.name);
        *name_kp.get_mut(&mut user) = "Bob".to_string();
        assert_eq!(user.name, "Bob");
        
        // Nested field mutation
        let mut user_with_address = TestUser {
            name: "Charlie".to_string(),
            age: 25,
            metadata: None,
            address: Some(TestAddress {
                street: "123 Main St".to_string(),
                city: "New York".to_string(),
                country: None,
            }),
        };
        
        let street_kp = writable_keypath!(|u: &mut TestUser| &mut u.address.as_mut().unwrap().street);
        *street_kp.get_mut(&mut user_with_address) = "456 Oak Ave".to_string();
        assert_eq!(user_with_address.address.as_ref().unwrap().street, "456 Oak Ave");
        
        // Deeper nesting
        let mut user_with_country = TestUser {
            name: "David".to_string(),
            age: 35,
            metadata: None,
            address: Some(TestAddress {
                street: "789 Pine Rd".to_string(),
                city: "London".to_string(),
                country: Some(TestCountry {
                    name: "UK".to_string(),
                }),
            }),
        };
        
        let country_name_kp = writable_keypath!(|u: &mut TestUser| &mut u.address.as_mut().unwrap().country.as_mut().unwrap().name);
        *country_name_kp.get_mut(&mut user_with_country) = "United Kingdom".to_string();
        assert_eq!(user_with_country.address.as_ref().unwrap().country.as_ref().unwrap().name, "United Kingdom");
        
        // Fallback: using closure
        let age_kp = writable_keypath!(|u: &mut TestUser| &mut u.age);
        *age_kp.get_mut(&mut user) = 31;
        assert_eq!(user.age, 31);
    }
    
    #[test]
    fn test_writable_opt_keypath_macro() {
        let mut user = TestUser {
            name: "Alice".to_string(),
            age: 30,
            metadata: Some("user".to_string()),
            address: None,
        };
        
        // Simple Option field mutation using closure
        let metadata_kp = writable_opt_keypath!(|u: &mut TestUser| u.metadata.as_mut());
        if let Some(metadata) = metadata_kp.get_mut(&mut user) {
            *metadata = "admin".to_string();
        }
        assert_eq!(user.metadata, Some("admin".to_string()));
        
        // None case - should return None
        let mut user_no_metadata = TestUser {
            name: "Bob".to_string(),
            age: 25,
            metadata: None,
            address: None,
        };
        assert_eq!(metadata_kp.get_mut(&mut user_no_metadata), None);
        
        // Nested Option access
        let mut user_with_address = TestUser {
            name: "Charlie".to_string(),
            age: 35,
            metadata: None,
            address: Some(TestAddress {
                street: "123 Main St".to_string(),
                city: "New York".to_string(),
                country: None,
            }),
        };
        
        let street_kp = writable_opt_keypath!(|u: &mut TestUser| u.address.as_mut().map(|a| &mut a.street));
        if let Some(street) = street_kp.get_mut(&mut user_with_address) {
            *street = "456 Oak Ave".to_string();
        }
        assert_eq!(user_with_address.address.as_ref().unwrap().street, "456 Oak Ave");
        
        // Deeper nesting through Options
        let mut user_with_country = TestUser {
            name: "David".to_string(),
            age: 40,
            metadata: None,
            address: Some(TestAddress {
                street: "789 Pine Rd".to_string(),
                city: "Tokyo".to_string(),
                country: Some(TestCountry {
                    name: "Japan".to_string(),
                }),
            }),
        };
        
        let country_name_kp = writable_opt_keypath!(|u: &mut TestUser| u.address.as_mut().and_then(|a| a.country.as_mut().map(|c| &mut c.name)));
        if let Some(country_name) = country_name_kp.get_mut(&mut user_with_country) {
            *country_name = "Nippon".to_string();
        }
        assert_eq!(user_with_country.address.as_ref().unwrap().country.as_ref().unwrap().name, "Nippon");
        
        // Fallback: using closure
        let metadata_kp2 = writable_opt_keypath!(|u: &mut TestUser| u.metadata.as_mut());
        if let Some(metadata) = metadata_kp2.get_mut(&mut user) {
            *metadata = "super_admin".to_string();
        }
        assert_eq!(user.metadata, Some("super_admin".to_string()));
    }
}

// ========== WithContainer Trait ==========

/// Trait for no-clone callback-based access to container types
/// Provides methods to execute closures with references to values inside containers
/// without requiring cloning of the values
pub trait WithContainer<Root, Value> {
    /// Execute a closure with a reference to the value inside an Arc
    fn with_arc<F, R>(&self, arc: &Arc<Root>, f: F) -> R
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a reference to the value inside a Box
    fn with_box<F, R>(&self, boxed: &Box<Root>, f: F) -> R
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside a Box
    fn with_box_mut<F, R>(&self, boxed: &mut Box<Root>, f: F) -> R
    where
        F: FnOnce(&mut Value) -> R;

    /// Execute a closure with a reference to the value inside an Rc
    fn with_rc<F, R>(&self, rc: &Rc<Root>, f: F) -> R
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a reference to the value inside a Result
    fn with_result<F, R, E>(&self, result: &Result<Root, E>, f: F) -> Option<R>
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside a Result
    fn with_result_mut<F, R, E>(&self, result: &mut Result<Root, E>, f: F) -> Option<R>
    where
        F: FnOnce(&mut Value) -> R;

    /// Execute a closure with a reference to the value inside an Option
    fn with_option<F, R>(&self, option: &Option<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside an Option
    fn with_option_mut<F, R>(&self, option: &mut Option<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&mut Value) -> R;

    /// Execute a closure with a reference to the value inside a RefCell
    fn with_refcell<F, R>(&self, refcell: &RefCell<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside a RefCell
    fn with_refcell_mut<F, R>(&self, refcell: &RefCell<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&mut Value) -> R;

    #[cfg(feature = "tagged")]
    /// Execute a closure with a reference to the value inside a Tagged
    fn with_tagged<F, R, Tag>(&self, tagged: &Tagged<Root, Tag>, f: F) -> R
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a reference to the value inside a Mutex
    fn with_mutex<F, R>(&self, mutex: &Mutex<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside a Mutex
    fn with_mutex_mut<F, R>(&self, mutex: &mut Mutex<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&mut Value) -> R;

    /// Execute a closure with a reference to the value inside an RwLock
    fn with_rwlock<F, R>(&self, rwlock: &RwLock<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside an RwLock
    fn with_rwlock_mut<F, R>(&self, rwlock: &mut RwLock<Root>, f: F) -> Option<R>
    where
        F: FnOnce(&mut Value) -> R;

    /// Execute a closure with a reference to the value inside an Arc<RwLock<Root>>
    fn with_arc_rwlock<F, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: F) -> Option<R>
    where
        F: FnOnce(&Value) -> R;

    /// Execute a closure with a mutable reference to the value inside an Arc<RwLock<Root>>
    fn with_arc_rwlock_mut<F, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: F) -> Option<R>
    where
        F: FnOnce(&mut Value) -> R;
}

// Implement WithContainer for KeyPath
impl<Root, Value, F> WithContainer<Root, Value> for KeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> &'r Value + Clone,
{
    fn with_arc<Callback, R>(&self, arc: &Arc<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_arc(arc, f)
    }

    fn with_box<Callback, R>(&self, boxed: &Box<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_box(boxed, f)
    }

    fn with_box_mut<Callback, R>(&self, _boxed: &mut Box<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        eprintln!("[DEBUG] KeyPath does not support mutable access - use WritableKeyPath instead");
        unreachable!("KeyPath does not support mutable access - use WritableKeyPath instead")
    }

    fn with_rc<Callback, R>(&self, rc: &Rc<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_rc(rc, f)
    }

    fn with_result<Callback, R, E>(&self, result: &Result<Root, E>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_result(result, f)
    }

    fn with_result_mut<Callback, R, E>(&self, _result: &mut Result<Root, E>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None
    }

    fn with_option<Callback, R>(&self, option: &Option<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_option(option, f)
    }

    fn with_option_mut<Callback, R>(&self, _option: &mut Option<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None
    }

    fn with_refcell<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_refcell(refcell, f)
    }

    fn with_refcell_mut<Callback, R>(&self, _refcell: &RefCell<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None
    }

    #[cfg(feature = "tagged")]
    fn with_tagged<Tag, Callback, R>(&self, tagged: &Tagged<Root, Tag>, f: Callback) -> R
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        Callback: FnOnce(&Value) -> R,
    {
        self.with_tagged(tagged, f)
    }

    fn with_mutex<Callback, R>(&self, mutex: &Mutex<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_mutex(mutex, f)
    }

    fn with_mutex_mut<Callback, R>(&self, _mutex: &mut Mutex<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None
    }

    fn with_rwlock<Callback, R>(&self, rwlock: &RwLock<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_rwlock(rwlock, f)
    }

    fn with_rwlock_mut<Callback, R>(&self, _rwlock: &mut RwLock<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None
    }

    fn with_arc_rwlock<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_arc_rwlock(arc_rwlock, f)
    }

    fn with_arc_rwlock_mut<Callback, R>(&self, _arc_rwlock: &Arc<RwLock<Root>>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None
    }
}

// Implement WithContainer for OptionalKeyPath - read-only operations only
impl<Root, Value, F> WithContainer<Root, Value> for OptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r Root) -> Option<&'r Value> + Clone,
{
    fn with_arc<Callback, R>(&self, arc: &Arc<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_arc(arc, f)
    }

    fn with_box<Callback, R>(&self, boxed: &Box<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_box(boxed, f)
    }

    fn with_box_mut<Callback, R>(&self, _boxed: &mut Box<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        eprintln!("[DEBUG] OptionalKeyPath does not support mutable access - use WritableOptionalKeyPath instead");
        unreachable!("OptionalKeyPath does not support mutable access - use WritableOptionalKeyPath instead")
    }

    fn with_rc<Callback, R>(&self, rc: &Rc<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_rc(rc, f)
    }

    fn with_result<Callback, R, E>(&self, result: &Result<Root, E>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_result(result, f)
    }

    fn with_result_mut<Callback, R, E>(&self, _result: &mut Result<Root, E>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None // OptionalKeyPath doesn't support mutable access
    }

    fn with_option<Callback, R>(&self, option: &Option<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_option(option, f)
    }

    fn with_option_mut<Callback, R>(&self, _option: &mut Option<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None // OptionalKeyPath doesn't support mutable access
    }

    fn with_refcell<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_refcell(refcell, f)
    }

    fn with_refcell_mut<Callback, R>(&self, _refcell: &RefCell<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None // OptionalKeyPath doesn't support mutable access
    }

    #[cfg(feature = "tagged")]
    fn with_tagged<Tag, Callback, R>(&self, tagged: &Tagged<Root, Tag>, f: Callback) -> R
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        Callback: FnOnce(&Value) -> R,
    {
        self.with_tagged(tagged, f)
    }

    fn with_mutex<Callback, R>(&self, mutex: &Mutex<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_mutex(mutex, f)
    }

    fn with_mutex_mut<Callback, R>(&self, _mutex: &mut Mutex<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None // OptionalKeyPath doesn't support mutable access
    }

    fn with_rwlock<Callback, R>(&self, rwlock: &RwLock<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_rwlock(rwlock, f)
    }

    fn with_rwlock_mut<Callback, R>(&self, _rwlock: &mut RwLock<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None // OptionalKeyPath doesn't support mutable access
    }

    fn with_arc_rwlock<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        self.with_arc_rwlock(arc_rwlock, f)
    }

    fn with_arc_rwlock_mut<Callback, R>(&self, _arc_rwlock: &Arc<RwLock<Root>>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        None // OptionalKeyPath doesn't support mutable access - use WritableOptionalKeyPath instead
    }
}

// Implement WithContainer for WritableKeyPath - supports all mutable operations
impl<Root, Value, F> WithContainer<Root, Value> for WritableKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> &'r mut Value,
{
    fn with_arc<Callback, R>(&self, _arc: &Arc<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Arc doesn't support mutable access without interior mutability
        // This method requires &mut Arc<Root> which we don't have
        eprintln!("[DEBUG] WritableKeyPath::with_arc requires &mut Arc<Root> or interior mutability");
        unreachable!("WritableKeyPath::with_arc requires &mut Arc<Root> or interior mutability")
    }

    fn with_box<Callback, R>(&self, boxed: &Box<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Box doesn't support getting mutable reference from immutable reference
        // This is a limitation - we'd need &mut Box<Root> for mutable access
        eprintln!("[DEBUG] WritableKeyPath::with_box requires &mut Box<Root> - use with_box_mut instead");
        unreachable!("WritableKeyPath::with_box requires &mut Box<Root> - use with_box_mut instead")
    }

    fn with_box_mut<Callback, R>(&self, boxed: &mut Box<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        let value = self.get_mut(boxed.as_mut());
        f(value)
    }

    fn with_rc<Callback, R>(&self, _rc: &Rc<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Rc doesn't support mutable access without interior mutability
        // This method requires &mut Rc<Root> which we don't have
        eprintln!("[DEBUG] WritableKeyPath::with_rc requires &mut Rc<Root> or interior mutability");
        unreachable!("WritableKeyPath::with_rc requires &mut Rc<Root> or interior mutability")
    }

    fn with_result<Callback, R, E>(&self, _result: &Result<Root, E>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // WritableKeyPath requires &mut Root, but we only have &Result<Root, E>
        // This is a limitation - use with_result_mut for mutable access
        None
    }

    fn with_result_mut<Callback, R, E>(&self, result: &mut Result<Root, E>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        result.as_mut().ok().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }

    fn with_option<Callback, R>(&self, _option: &Option<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // WritableKeyPath requires &mut Root, but we only have &Option<Root>
        // This is a limitation - use with_option_mut for mutable access
        None
    }

    fn with_option_mut<Callback, R>(&self, option: &mut Option<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        option.as_mut().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }

    fn with_refcell<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // RefCell doesn't allow getting mutable reference from immutable borrow
        // This is a limitation - we'd need try_borrow_mut for mutable access
        None
    }

    fn with_refcell_mut<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        refcell.try_borrow_mut().ok().map(|mut borrow| {
            let value = self.get_mut(&mut *borrow);
            f(value)
        })
    }

    #[cfg(feature = "tagged")]
    fn with_tagged<Tag, Callback, R>(&self, _tagged: &Tagged<Root, Tag>, _f: Callback) -> R
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        Callback: FnOnce(&Value) -> R,
    {
        // WritableKeyPath requires &mut Root, but we only have &Tagged<Root, Tag>
        // This is a limitation - Tagged doesn't support mutable access without interior mutability
        eprintln!("[DEBUG] WritableKeyPath::with_tagged requires &mut Tagged<Root, Tag> or interior mutability");
        unreachable!("WritableKeyPath::with_tagged requires &mut Tagged<Root, Tag> or interior mutability")
    }

    fn with_mutex<Callback, R>(&self, mutex: &Mutex<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        mutex.lock().ok().map(|mut guard| {
            let value = self.get_mut(&mut *guard);
            f(value)
        })
    }

    fn with_mutex_mut<Callback, R>(&self, mutex: &mut Mutex<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        // Mutex::get_mut returns Result<&mut Root, PoisonError>
        mutex.get_mut().ok().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }

    fn with_rwlock<Callback, R>(&self, rwlock: &RwLock<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // RwLock read guard doesn't allow mutable access
        // This is a limitation - we'd need write() for mutable access
        None
    }

    fn with_rwlock_mut<Callback, R>(&self, rwlock: &mut RwLock<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        // RwLock::get_mut returns Result<&mut Root, PoisonError>
        rwlock.get_mut().ok().map(|root| {
            let value = self.get_mut(root);
            f(value)
        })
    }

    fn with_arc_rwlock<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Arc<RwLock> read guard doesn't allow mutable access
        // This is a limitation - we'd need write() for mutable access
        None
    }

    fn with_arc_rwlock_mut<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        arc_rwlock.write().ok().map(|mut guard| {
            let value = self.get_mut(&mut *guard);
            f(value)
        })
    }
}

// Implement WithContainer for WritableOptionalKeyPath - supports all mutable operations
impl<Root, Value, F> WithContainer<Root, Value> for WritableOptionalKeyPath<Root, Value, F>
where
    F: for<'r> Fn(&'r mut Root) -> Option<&'r mut Value>,
{
    fn with_arc<Callback, R>(&self, _arc: &Arc<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Arc doesn't support mutable access without interior mutability
        // This method requires &mut Arc<Root> which we don't have
        eprintln!("[DEBUG] WritableOptionalKeyPath::with_arc requires &mut Arc<Root> or interior mutability");
        unreachable!("WritableOptionalKeyPath::with_arc requires &mut Arc<Root> or interior mutability")
    }

    fn with_box<Callback, R>(&self, _boxed: &Box<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        // WritableOptionalKeyPath requires &mut Root, but we only have &Box<Root>
        // This is a limitation - use with_box_mut for mutable access
        eprintln!("[DEBUG] WritableOptionalKeyPath::with_box requires &mut Box<Root> - use with_box_mut instead");
        unreachable!("WritableOptionalKeyPath::with_box requires &mut Box<Root> - use with_box_mut instead")
    }

    fn with_box_mut<Callback, R>(&self, boxed: &mut Box<Root>, f: Callback) -> R
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        if let Some(value) = self.get_mut(boxed.as_mut()) {
            f(value)
        } else {
            eprintln!("[DEBUG] WritableOptionalKeyPath failed to get value from Box");
            unreachable!("WritableOptionalKeyPath failed to get value from Box")
        }
    }

    fn with_rc<Callback, R>(&self, _rc: &Rc<Root>, _f: Callback) -> R
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Rc doesn't support mutable access without interior mutability
        // This method requires &mut Rc<Root> which we don't have
        eprintln!("[DEBUG] WritableOptionalKeyPath::with_rc requires &mut Rc<Root> or interior mutability");
        unreachable!("WritableOptionalKeyPath::with_rc requires &mut Rc<Root> or interior mutability")
    }

    fn with_result<Callback, R, E>(&self, _result: &Result<Root, E>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // WritableOptionalKeyPath requires &mut Root, but we only have &Result<Root, E>
        // This is a limitation - use with_result_mut for mutable access
        None
    }

    fn with_result_mut<Callback, R, E>(&self, result: &mut Result<Root, E>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        result.as_mut().ok().and_then(|root| {
            self.get_mut(root).map(|value| f(value))
        })
    }

    fn with_option<Callback, R>(&self, _option: &Option<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // WritableOptionalKeyPath requires &mut Root, but we only have &Option<Root>
        // This is a limitation - use with_option_mut for mutable access
        None
    }

    fn with_option_mut<Callback, R>(&self, option: &mut Option<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        option.as_mut().and_then(|root| {
            self.get_mut(root).map(|value| f(value))
        })
    }

    fn with_refcell<Callback, R>(&self, _refcell: &RefCell<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // RefCell doesn't allow getting mutable reference from immutable borrow
        // This is a limitation - we'd need try_borrow_mut for mutable access
        None
    }

    fn with_refcell_mut<Callback, R>(&self, refcell: &RefCell<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        refcell.try_borrow_mut().ok().and_then(|mut borrow| {
            self.get_mut(&mut *borrow).map(|value| f(value))
        })
    }

    #[cfg(feature = "tagged")]
    fn with_tagged<Tag, Callback, R>(&self, _tagged: &Tagged<Root, Tag>, _f: Callback) -> R
    where
        Tagged<Root, Tag>: std::ops::Deref<Target = Root>,
        Callback: FnOnce(&Value) -> R,
    {
        // WritableOptionalKeyPath requires &mut Root, but we only have &Tagged<Root, Tag>
        // This is a limitation - Tagged doesn't support mutable access without interior mutability
        eprintln!("[DEBUG] WritableOptionalKeyPath::with_tagged requires &mut Tagged<Root, Tag> or interior mutability");
        unreachable!("WritableOptionalKeyPath::with_tagged requires &mut Tagged<Root, Tag> or interior mutability")
    }

    fn with_mutex<Callback, R>(&self, mutex: &Mutex<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        mutex.lock().ok().and_then(|mut guard| {
            self.get_mut(&mut *guard).map(|value| f(value))
        })
    }

    fn with_mutex_mut<Callback, R>(&self, mutex: &mut Mutex<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        // Mutex::get_mut returns Result<&mut Root, PoisonError>
        mutex.get_mut().ok().and_then(|root| {
            self.get_mut(root).map(|value| f(value))
        })
    }

    fn with_rwlock<Callback, R>(&self, _rwlock: &RwLock<Root>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // RwLock read guard doesn't allow mutable access
        // This is a limitation - we'd need write() for mutable access
        None
    }

    fn with_rwlock_mut<Callback, R>(&self, rwlock: &mut RwLock<Root>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        // RwLock::get_mut returns Result<&mut Root, PoisonError>
        rwlock.get_mut().ok().and_then(|root| {
            self.get_mut(root).map(|value| f(value))
        })
    }

    fn with_arc_rwlock<Callback, R>(&self, _arc_rwlock: &Arc<RwLock<Root>>, _f: Callback) -> Option<R>
    where
        Callback: FnOnce(&Value) -> R,
    {
        // Arc<RwLock> read guard doesn't allow mutable access
        // This is a limitation - we'd need write() for mutable access
        None
    }

    fn with_arc_rwlock_mut<Callback, R>(&self, arc_rwlock: &Arc<RwLock<Root>>, f: Callback) -> Option<R>
    where
        Callback: FnOnce(&mut Value) -> R,
    {
        arc_rwlock.write().ok().and_then(|mut guard| {
            self.get_mut(&mut *guard).map(|value| f(value))
        })
    }
}