hyperactor_config/

attrs.rs

/*
 * Copyright (c) Meta Platforms, Inc. and affiliates.
 * All rights reserved.
 *
 * This source code is licensed under the BSD-style license found in the
 * LICENSE file in the root directory of this source tree.
 */

//! Attribute dictionary for type-safe, heterogeneous key-value storage with serde support.
//!
//! This module provides `Attrs`, a type-safe dictionary that can store heterogeneous values
//! and serialize/deserialize them using serde. All stored values must implement
//! `AttrValue` to ensure the entire dictionary can be serialized.
//!
//! Keys are automatically registered at compile time using the `declare_attrs!` macro and the
//! inventory crate, eliminating the need for manual registry management.
//!
//! # Basic Usage
//!
//! ```
//! use std::time::Duration;
//!
//! use hyperactor_config::attrs::Attrs;
//! use hyperactor_config::attrs::declare_attrs;
//!
//! // Declare keys with their associated types
//! declare_attrs! {
//!    /// Request timeout
//!    attr TIMEOUT: Duration;
//!
//!   /// Maximum retry count
//!   attr MAX_RETRIES: u32 = 3;  // with default value
//! }
//!
//! let mut attrs = Attrs::new();
//! attrs.set(TIMEOUT, Duration::from_secs(30));
//!
//! assert_eq!(attrs.get(TIMEOUT), Some(&Duration::from_secs(30)));
//! assert_eq!(attrs.get(MAX_RETRIES), Some(&3));
//! ```
//!
//! # Serialization
//!
//! `Attrs` can be serialized to and deserialized automatically:
//!
//! ```
//! use std::time::Duration;
//!
//! use hyperactor_config::attrs::Attrs;
//! use hyperactor_config::attrs::declare_attrs;
//!
//! declare_attrs! {
//!   /// Request timeout
//!   pub attr TIMEOUT: Duration;
//! }
//!
//! let mut attrs = Attrs::new();
//! attrs.set(TIMEOUT, Duration::from_secs(30));
//!
//! // Serialize to JSON
//! let json = serde_json::to_string(&attrs).unwrap();
//!
//! // Deserialize from JSON (no manual registry needed!)
//! let deserialized: Attrs = serde_json::from_str(&json).unwrap();
//!
//! assert_eq!(deserialized.get(TIMEOUT), Some(&Duration::from_secs(30)));
//! ```
//!
//! ## Meta attributes
//!
//! An attribute can be assigned a set of attribute values,
//! associated with the attribute key. These are specified by
//! @-annotations in the `declare_attrs!` macro:
//!
//! ```
//! use std::time::Duration;
//!
//! use hyperactor_config::attrs::Attrs;
//! use hyperactor_config::attrs::declare_attrs;
//!
//! declare_attrs! {
//!   /// Is experimental?
//!   pub attr EXPERIMENTAL: bool;
//!
//!   /// Request timeout
//!   @meta(EXPERIMENTAL = true)
//!   pub attr TIMEOUT: Duration;
//! }
//!
//! assert!(TIMEOUT.attrs().get(EXPERIMENTAL).unwrap());
//! ```
//!
//! Meta attributes can be used to provide more generic functionality
//! on top of the basic attributes. For example, a library can use
//! meta-attributes to specify the behavior of an attribute.

use std::any::Any;
use std::collections::HashMap;
use std::fmt::Display;
use std::ops::Index;
use std::ops::IndexMut;
use std::str::FromStr;
use std::sync::LazyLock;

use chrono::DateTime;
use chrono::Utc;
use erased_serde::Deserializer as ErasedDeserializer;
use erased_serde::Serialize as ErasedSerialize;
use serde::Deserialize;
use serde::Deserializer;
use serde::Serialize;
use serde::Serializer;
use serde::de::DeserializeOwned;
use serde::de::MapAccess;
use serde::de::Visitor;
use serde::ser::SerializeMap;
use typeuri::Named;

use crate::flattrs::Flattrs;

// Information about an attribute key, used for automatic registration.
// This needs to be public to be accessible from other crates, but it is
// not part of the public API.
#[doc(hidden)]
pub struct AttrKeyInfo {
    /// Name of the key
    pub name: &'static str,
    /// Unique hash for the key (FNV-1a hash of name)
    pub key_hash: u64,
    /// Function to get the type hash of the associated value type
    pub typehash: fn() -> u64,
    /// Deserializer function that deserializes directly from any deserializer
    pub deserialize_erased:
        fn(&mut dyn ErasedDeserializer) -> Result<Box<dyn SerializableValue>, erased_serde::Error>,
    /// Deserializer function for bincode bytes (used by Flattrs)
    pub deserialize_bincode:
        fn(&[u8]) -> Result<Box<dyn SerializableValue>, bincode::error::DecodeError>,
    /// Meta-attributes.
    pub meta: &'static LazyLock<Attrs>,
    /// Display an attribute value using AttrValue::display.
    pub display: fn(&dyn SerializableValue) -> String,
    /// Parse an attribute value using AttrValue::parse.
    pub parse: fn(&str) -> Result<Box<dyn SerializableValue>, anyhow::Error>,
    /// Default value for the attribute, if any.
    pub default: Option<&'static dyn SerializableValue>,
    /// A reference to the relevant key object with the associated
    /// type parameter erased. Can be downcast to a concrete Key<T>.
    pub erased: &'static dyn ErasedKey,
}

inventory::collect!(AttrKeyInfo);

/// Look up a key info by its hash using the global registry.
///
/// Returns `None` if no key with this hash is registered.
/// Uses a lazy-initialized hash map for O(1) lookup after first access.
pub fn lookup_key_info(key_hash: u64) -> Option<&'static AttrKeyInfo> {
    static KEYS_BY_HASH: std::sync::LazyLock<std::collections::HashMap<u64, &'static AttrKeyInfo>> =
        std::sync::LazyLock::new(|| {
            inventory::iter::<AttrKeyInfo>()
                .map(|info| (info.key_hash, info))
                .collect()
        });
    KEYS_BY_HASH.get(&key_hash).copied()
}

/// Look up a key info by name using the global registry.
///
/// Returns `None` if no key with this name is registered.
/// Uses a lazy-initialized hash map for O(1) lookup after first access.
pub fn lookup_key_info_by_name(name: &str) -> Option<&'static AttrKeyInfo> {
    static KEYS_BY_NAME: std::sync::LazyLock<
        std::collections::HashMap<&'static str, &'static AttrKeyInfo>,
    > = std::sync::LazyLock::new(|| {
        inventory::iter::<AttrKeyInfo>()
            .map(|info| (info.name, info))
            .collect()
    });
    KEYS_BY_NAME.get(name).copied()
}

declare_attrs! {
    /// Meta-attribute marker for operation context carried on
    /// envelope headers. Attrs declared with
    /// `@meta(OPERATION_CONTEXT_HEADER = true)` are stamped onto
    /// outgoing request envelopes and copied onto reply envelopes
    /// by the marker-driven helpers
    /// (`stamp_marked_attrs_into_flattrs`, `copy_marked_flattrs`).
    ///
    /// The vocabulary describes which user operation a message
    /// belongs to, so consumers such as the undeliverable-
    /// abandonment log can name the operation. `OPERATION_*` names
    /// the operation; it does not imply request/reply direction.
    pub attr OPERATION_CONTEXT_HEADER: bool;
}

/// Returns `Some(true)` when the declared attribute key with this
/// name carries the given bool meta marker with value `true`,
/// `Some(false)` when it carries the marker with value `false`, and
/// `None` for unknown names or declared keys that do not carry the
/// marker at all.
///
/// Generic "is this attr a member of the category declared by this
/// marker?" primitive. The lower layer does not know what category
/// the marker names — the caller supplies the category marker that
/// defines its vocabulary (e.g. `ATTRIBUTION_HEADER` for attribution
/// data carried on envelope headers).
pub fn is_attr_marked_with(name: &str, marker: Key<bool>) -> Option<bool> {
    let info = lookup_key_info_by_name(name)?;
    info.meta.get(marker).copied()
}

/// Copy every entry from `attrs` onto `dst` whose declared
/// attribute key carries the bool meta marker `marker` with value
/// `true`. Overwrite semantics (see `Flattrs::set_serialized`) — a
/// later write for the same `key_hash` is the value returned by
/// `Flattrs::get`. Entries whose declared key lacks the marker are
/// silently skipped; entries whose key is not declared are also
/// silently skipped.
///
/// Generic category-filter stamp. The lower layer does not know what
/// category the marker names; callers pass the marker key that
/// defines their vocabulary (e.g. `ATTRIBUTION_HEADER`). This is
/// the single mechanism every `Attrs → Flattrs` stamp for a category
/// should go through, so category membership is declared in one
/// place (the meta annotation on each key) and enforced in one
/// place (this helper).
pub fn stamp_marked_attrs_into_flattrs(dst: &mut Flattrs, attrs: &Attrs, marker: Key<bool>) {
    for (name, value) in attrs.iter() {
        if is_attr_marked_with(name, marker) != Some(true) {
            continue;
        }
        dst.set_serialized(fnv1a_hash(name.as_bytes()), &value.serialize_bincode());
    }
}

/// Copy every entry from `src` to `dst` whose declared attribute
/// key carries the bool meta marker `marker` with value `true`.
/// Overwrite semantics (see `Flattrs::set_serialized`). Entries
/// whose declared key lacks the marker — or whose key_hash does
/// not resolve to a declared key in the current binary's
/// inventory — are silently skipped.
///
/// Generic category-filter `Flattrs → Flattrs` copy. The companion
/// of `stamp_marked_attrs_into_flattrs` for callers that already
/// have a source `Flattrs` in hand (forwarding, hoisting) rather
/// than an in-memory `Attrs`.
pub fn copy_marked_flattrs(dst: &mut Flattrs, src: &Flattrs, marker: Key<bool>) {
    for (key_hash, value) in src.iter() {
        let Some(info) = lookup_key_info(key_hash) else {
            continue;
        };
        if info.meta.get(marker).copied() != Some(true) {
            continue;
        }
        dst.set_serialized(key_hash, value);
    }
}

/// Returns the set of all declared attribute key names that carry
/// the given bool meta marker (set to `true`) in the attrs
/// inventory linked into the current binary.
///
/// Generic category-membership enumeration. Used by consumers that
/// maintain explicit hard-coded category sets (for auditability)
/// to validate the set matches the declaration-driven vocabulary;
/// drift between the two is a bug.
pub fn marked_attr_names(marker: Key<bool>) -> std::collections::HashSet<&'static str> {
    inventory::iter::<AttrKeyInfo>()
        .filter(|info| info.meta.get(marker).copied() == Some(true))
        .map(|info| info.name)
        .collect()
}

/// A typed key for the attribute dictionary.
///
/// Each key is associated with a specific type T and has a unique name.
/// Keys are typically created using the `declare_attrs!` macro which ensures they have
/// static lifetime and automatically registers them for serialization.
pub struct Key<T: 'static> {
    name: &'static str,
    default_value: Option<&'static T>,
    attrs: &'static LazyLock<Attrs>,
}

impl<T> Key<T> {
    /// Returns the name of this key.
    pub fn name(&self) -> &'static str {
        self.name
    }

    /// Returns a unique ID for this key, computed as FNV-1a hash of the name.
    ///
    /// This ID is stable and can be used for efficient wire transmission
    /// instead of the full key name string.
    pub const fn key_hash(&self) -> u64 {
        fnv1a_hash(self.name.as_bytes())
    }
}

/// Compute FNV-1a hash of a byte slice (const-compatible).
///
/// FNV-1a is a simple, fast hash with good distribution properties.
/// We use 64-bit version for low collision probability.
pub const fn fnv1a_hash(bytes: &[u8]) -> u64 {
    const FNV_OFFSET_BASIS: u64 = 0xcbf29ce484222325;
    const FNV_PRIME: u64 = 0x100000001b3;

    let mut hash = FNV_OFFSET_BASIS;
    let mut i = 0;
    while i < bytes.len() {
        hash ^= bytes[i] as u64;
        hash = hash.wrapping_mul(FNV_PRIME);
        i += 1;
    }
    hash
}

impl<T: Named + 'static> Key<T> {
    /// Creates a new key with the given name.
    pub const fn new(
        name: &'static str,
        default_value: Option<&'static T>,
        attrs: &'static LazyLock<Attrs>,
    ) -> Self {
        Self {
            name,
            default_value,
            attrs,
        }
    }

    /// Returns a reference to the default value for this key, if one exists.
    pub fn default(&self) -> Option<&'static T> {
        self.default_value
    }

    /// Returns whether this key has a default value.
    pub fn has_default(&self) -> bool {
        self.default_value.is_some()
    }

    /// Returns the type hash of the associated value type.
    pub fn typehash(&self) -> u64 {
        T::typehash()
    }

    /// The attributes associated with this key.
    pub fn attrs(&self) -> &'static LazyLock<Attrs> {
        self.attrs
    }
}

impl<T: 'static> Clone for Key<T> {
    fn clone(&self) -> Self {
        // Use Copy.
        *self
    }
}

impl<T: 'static> Copy for Key<T> {}

/// A trait for type-erased keys.
pub trait ErasedKey: Any + Send + Sync + 'static {
    /// The name of the key.
    fn name(&self) -> &'static str;

    /// The unique ID of the key (FNV-1a hash of name).
    fn key_hash(&self) -> u64;

    /// The typehash of the key's associated type.
    fn typehash(&self) -> u64;

    /// The typename of the key's associated type.
    fn typename(&self) -> &'static str;
}

impl dyn ErasedKey {
    /// Downcast a type-erased key to a specific key type.
    pub fn downcast_ref<T: Named + 'static>(&'static self) -> Option<&'static Key<T>> {
        (self as &dyn Any).downcast_ref::<Key<T>>()
    }
}

impl<T: AttrValue> ErasedKey for Key<T> {
    fn name(&self) -> &'static str {
        self.name
    }

    fn key_hash(&self) -> u64 {
        self.key_hash()
    }

    fn typehash(&self) -> u64 {
        T::typehash()
    }

    fn typename(&self) -> &'static str {
        T::typename()
    }
}

// Enable attr[key] syntax.
impl<T: AttrValue> Index<Key<T>> for Attrs {
    type Output = T;

    fn index(&self, key: Key<T>) -> &Self::Output {
        self.get(key).unwrap()
    }
}

// TODO: separately type keys with defaults, so that we can statically enforce that indexmut is only
// called on keys with defaults.
impl<T: AttrValue> IndexMut<Key<T>> for Attrs {
    fn index_mut(&mut self, key: Key<T>) -> &mut Self::Output {
        self.get_mut(key).unwrap()
    }
}

/// This trait must be implemented by all attribute values. In addition to enforcing
/// the supertrait `Named + Sized + Serialize + DeserializeOwned + Send + Sync + Clone`,
/// `AttrValue` requires that the type be representable in "display" format.
///
/// `AttrValue` includes its own `display` and `parse` so that behavior can be tailored
/// for attribute purposes specifically, allowing common types like `Duration` to be used
/// without modification.
///
/// This crate includes a derive macro for AttrValue, which uses the type's
/// `std::string::ToString` for display, and `std::str::FromStr` for parsing.
pub trait AttrValue:
    Named + Sized + Serialize + DeserializeOwned + Send + Sync + Clone + 'static
{
    /// Display the value, typically using [`std::fmt::Display`].
    /// This is called to show the output in human-readable form.
    fn display(&self) -> String;

    /// Parse a value from a string, typically using [`std::str::FromStr`].
    fn parse(value: &str) -> Result<Self, anyhow::Error>;
}

/// Marker trait for types that implement a "display" that never produces an
/// empty string. This allows Option<T> where T: DisplayNonEmpty to not require
/// a Some(...) wrapper, and can use the empty string as the "None" value.
trait DisplayNonEmpty {}

/// Macro to implement AttrValue for types that implement ToString and FromStr.
///
/// This macro provides a convenient way to implement AttrValue for types that already
/// have string conversion capabilities through the standard ToString and FromStr traits.
///
/// # Usage
///
/// ```ignore
/// impl_attrvalue!(i32, u64, f64);
/// ```
///
/// This will generate AttrValue implementations for i32, u64, and f64 that use
/// their ToString and FromStr implementations for display and parsing.
#[macro_export]
macro_rules! impl_attrvalue {
    ($($ty:ty),+ $(,)?) => {
        $(
            impl $crate::attrs::AttrValue for $ty {
                fn display(&self) -> String {
                    self.to_string()
                }

                fn parse(value: &str) -> Result<Self, anyhow::Error> {
                    value.parse().map_err(|e| anyhow::anyhow!("failed to parse {}: {}", stringify!($ty), e))
                }
            }
        )+
    };
}

#[macro_export]
macro_rules! impl_attrvalue_and_display_non_empty {
    ($($ty:ty),+ $(,)?) => {
        $(
            impl_attrvalue!($ty);
            impl $crate::attrs::DisplayNonEmpty for $ty {}
        )+
    };
}

// pub use impl_attrvalue;

// Implement AttrValue for common standard library types
impl_attrvalue_and_display_non_empty!(
    i8, i16, i32, i64, i128, isize, u8, u16, u32, u64, u128, usize, f32, f64,
);

// String has a display that can produce an empty string, so it cannot impl
// DisplayNonEmpty
impl_attrvalue!(
    String,
    std::net::IpAddr,
    std::net::Ipv4Addr,
    std::net::Ipv6Addr,
);

impl AttrValue for std::time::Duration {
    fn display(&self) -> String {
        humantime::format_duration(*self).to_string()
    }

    fn parse(value: &str) -> Result<Self, anyhow::Error> {
        Ok(humantime::parse_duration(value)?)
    }
}

// Durations never have an empty display.
impl DisplayNonEmpty for std::time::Duration {}

impl AttrValue for std::time::SystemTime {
    fn display(&self) -> String {
        let datetime: DateTime<Utc> = (*self).into();
        datetime.to_rfc3339()
    }

    fn parse(value: &str) -> Result<Self, anyhow::Error> {
        let datetime = DateTime::parse_from_rfc3339(value)?;
        Ok(datetime.into())
    }
}

// SystemTimes never have an empty display.
impl DisplayNonEmpty for std::time::SystemTime {}

impl<T> AttrValue for Vec<T>
where
    T: AttrValue,
{
    fn display(&self) -> String {
        serde_json::to_string(self).unwrap_or_else(|_| "[]".to_string())
    }

    fn parse(value: &str) -> Result<Self, anyhow::Error> {
        Ok(serde_json::from_str(value)?)
    }
}

impl<T, E> AttrValue for std::ops::Range<T>
where
    T: Named
        + Display
        + FromStr<Err = E>
        + Send
        + Sync
        + Serialize
        + DeserializeOwned
        + Clone
        + 'static,
    E: Into<anyhow::Error> + Send + Sync + 'static,
{
    fn display(&self) -> String {
        format!("{}..{}", self.start, self.end)
    }

    fn parse(value: &str) -> Result<Self, anyhow::Error> {
        let (start, end) = value.split_once("..").ok_or_else(|| {
            anyhow::anyhow!("expected range in format `start..end`, got `{}`", value)
        })?;
        let start = start.parse().map_err(|e: E| e.into())?;
        let end = end.parse().map_err(|e: E| e.into())?;
        Ok(start..end)
    }
}

impl AttrValue for bool {
    fn display(&self) -> String {
        if *self { 1.to_string() } else { 0.to_string() }
    }

    fn parse(value: &str) -> Result<Self, anyhow::Error> {
        let value = value.to_ascii_lowercase();
        match value.as_str() {
            "0" | "false" => Ok(false),
            "1" | "true" => Ok(true),
            _ => Err(anyhow::anyhow!(
                "expected `0`, `1`, `true` or `false`, got `{}`",
                value
            )),
        }
    }
}

impl DisplayNonEmpty for bool {}

impl DisplayNonEmpty for crate::NonZeroUsize {}

impl<T: AttrValue + DisplayNonEmpty> AttrValue for Option<T> {
    fn display(&self) -> String {
        match self {
            Some(value) => value.display(),
            None => String::new(),
        }
    }

    fn parse(value: &str) -> Result<Self, anyhow::Error> {
        if value.is_empty() {
            Ok(None)
        } else {
            Ok(Some(T::parse(value)?))
        }
    }
}

// Internal trait for type-erased serialization
#[doc(hidden)]
pub trait SerializableValue: Send + Sync {
    /// Get a reference to this value as Any for downcasting
    fn as_any(&self) -> &dyn Any;
    /// Get a mutable reference to this value as Any for downcasting
    fn as_any_mut(&mut self) -> &mut dyn Any;
    /// Get a reference to this value as an erased serializable trait object
    fn as_erased_serialize(&self) -> &dyn ErasedSerialize;
    /// Clone the underlying value, retaining dyn compatibility.
    fn cloned(&self) -> Box<dyn SerializableValue>;
    /// Display the value
    fn display(&self) -> String;
    /// Serialize to bincode bytes for wire transmission
    fn serialize_bincode(&self) -> Vec<u8>;
}

impl<T: AttrValue> SerializableValue for T {
    fn as_any(&self) -> &dyn Any {
        self
    }

    fn as_any_mut(&mut self) -> &mut dyn Any {
        self
    }

    fn as_erased_serialize(&self) -> &dyn ErasedSerialize {
        self
    }

    fn cloned(&self) -> Box<dyn SerializableValue> {
        Box::new(self.clone())
    }

    fn display(&self) -> String {
        self.display()
    }

    fn serialize_bincode(&self) -> Vec<u8> {
        bincode::serde::encode_to_vec(self, bincode::config::legacy())
            .expect("bincode serialization failed")
    }
}

/// A heterogeneous, strongly-typed attribute dictionary with serialization support.
///
/// This dictionary stores key-value pairs where:
/// - Keys are type-safe and must be predefined with their associated types
/// - Values must implement [`AttrValue`]
/// - The entire dictionary can be serialized to/from JSON automatically
///
/// # Type Safety
///
/// The dictionary enforces type safety at compile time. You cannot retrieve a value
/// with the wrong type, and the compiler will catch such errors.
///
/// # Serialization
///
/// The dictionary can be serialized using serde. During serialization, each value
/// is serialized with its key name. During deserialization, the automatically registered
/// key information is used to determine the correct type for each value.
pub struct Attrs {
    values: HashMap<&'static str, Box<dyn SerializableValue>>,
}

impl Attrs {
    /// Create a new empty attribute dictionary.
    pub fn new() -> Self {
        Self {
            values: HashMap::new(),
        }
    }

    /// Set a value for the given key.
    pub fn set<T: AttrValue>(&mut self, key: Key<T>, value: T) {
        self.values.insert(key.name, Box::new(value));
    }

    fn maybe_set_from_default<T: AttrValue>(&mut self, key: Key<T>) {
        if self.contains_key(key) {
            return;
        }
        let Some(default) = key.default() else { return };
        self.set(key, default.clone());
    }

    /// Get a value for the given key, returning None if not present. If the key has a default value,
    /// that is returned instead.
    pub fn get<T: AttrValue>(&self, key: Key<T>) -> Option<&T> {
        self.values
            .get(key.name)
            .and_then(|value| value.as_any().downcast_ref::<T>())
            .or_else(|| key.default())
    }

    /// Get a mutable reference to a value for the given key. If the key has a default value, it is
    /// first set, and then returned as a mutable reference.
    pub fn get_mut<T: AttrValue>(&mut self, key: Key<T>) -> Option<&mut T> {
        self.maybe_set_from_default(key);
        self.values
            .get_mut(key.name)
            .and_then(|value| value.as_any_mut().downcast_mut::<T>())
    }

    /// Remove a value for the given key, returning it if present.
    pub fn remove<T: AttrValue>(&mut self, key: Key<T>) -> bool {
        // TODO: return value (this is tricky because of the type erasure)
        self.values.remove(key.name).is_some()
    }

    /// Checks if the given key exists in the dictionary.
    pub fn contains_key<T: AttrValue>(&self, key: Key<T>) -> bool {
        self.values.contains_key(key.name)
    }

    /// Returns the number of key-value pairs in the dictionary.
    pub fn len(&self) -> usize {
        self.values.len()
    }

    /// Returns true if the dictionary is empty.
    pub fn is_empty(&self) -> bool {
        self.values.is_empty()
    }

    /// Iterate over all key-value pairs.
    ///
    /// Returns an iterator of (key_name, value) pairs.
    pub fn iter(&self) -> impl Iterator<Item = (&'static str, &dyn SerializableValue)> {
        self.values.iter().map(|(k, v)| (*k, v.as_ref()))
    }

    /// Clear all key-value pairs from the dictionary.
    pub fn clear(&mut self) {
        self.values.clear();
    }

    // Internal methods for config guard support
    /// Take a value by key name, returning the boxed value if present
    pub fn remove_value<T: 'static>(&mut self, key: Key<T>) -> Option<Box<dyn SerializableValue>> {
        self.values.remove(key.name)
    }

    /// Restore a value by key name
    pub fn insert_value<T: 'static>(&mut self, key: Key<T>, value: Box<dyn SerializableValue>) {
        self.values.insert(key.name, value);
    }

    /// Restore a value by key name
    pub fn insert_value_by_name_unchecked(
        &mut self,
        name: &'static str,
        value: Box<dyn SerializableValue>,
    ) {
        self.values.insert(name, value);
    }

    /// Internal getter by key name for explicitly-set values (no
    /// defaults).
    pub fn get_value_by_name(&self, name: &'static str) -> Option<&dyn SerializableValue> {
        self.values.get(name).map(|b| b.as_ref())
    }

    /// Merge all attributes from `other` into this set, consuming
    /// `other`.
    ///
    /// For each key in `other`, moves its value into `self`,
    /// overwriting any existing value for the same key.
    pub fn merge(&mut self, other: Attrs) {
        self.values.extend(other.values);
    }
}

impl Clone for Attrs {
    fn clone(&self) -> Self {
        let mut values = HashMap::new();
        for (key, value) in &self.values {
            values.insert(*key, value.cloned());
        }
        Self { values }
    }
}

impl std::fmt::Display for Attrs {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        let mut first = true;
        for (key, value) in &self.values {
            if first {
                first = false;
            } else {
                write!(f, ",")?;
            }
            write!(f, "{}={}", key, value.display())?
        }
        Ok(())
    }
}

impl std::fmt::Debug for Attrs {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        // Create a map of key names to their JSON representation for debugging
        let mut debug_map = std::collections::BTreeMap::new();
        for (key, value) in &self.values {
            match serde_json::to_string(value.as_erased_serialize()) {
                Ok(json) => {
                    debug_map.insert(*key, json);
                }
                Err(_) => {
                    debug_map.insert(*key, "<serialization error>".to_string());
                }
            }
        }

        f.debug_struct("Attrs").field("values", &debug_map).finish()
    }
}

impl Default for Attrs {
    fn default() -> Self {
        Self::new()
    }
}

impl Serialize for Attrs {
    fn serialize<S>(&self, serializer: S) -> Result<S::Ok, S::Error>
    where
        S: Serializer,
    {
        let mut map = serializer.serialize_map(Some(self.values.len()))?;

        for (key_name, value) in &self.values {
            map.serialize_entry(key_name, value.as_erased_serialize())?;
        }

        map.end()
    }
}

struct AttrsVisitor;

impl<'de> Visitor<'de> for AttrsVisitor {
    type Value = Attrs;

    fn expecting(&self, formatter: &mut std::fmt::Formatter) -> std::fmt::Result {
        formatter.write_str("a map of attribute keys to their serialized values")
    }

    fn visit_map<M>(self, mut access: M) -> Result<Self::Value, M::Error>
    where
        M: MapAccess<'de>,
    {
        static KEYS_BY_NAME: std::sync::LazyLock<HashMap<&'static str, &'static AttrKeyInfo>> =
            std::sync::LazyLock::new(|| {
                inventory::iter::<AttrKeyInfo>()
                    .map(|info| (info.name, info))
                    .collect()
            });
        let keys_by_name = &*KEYS_BY_NAME;

        let exe_name = std::env::current_exe()
            .ok()
            .map(|p| p.display().to_string())
            .unwrap_or_else(|| "<unknown-exe>".to_string());

        let mut attrs = Attrs::new();
        while let Some(key_name) = access.next_key::<String>()? {
            let Some(&key) = keys_by_name.get(key_name.as_str()) else {
                // We hit an attribute key that this binary doesn't
                // know about.
                //
                // In JSON we'd just deserialize that value into
                // `IgnoredAny` and move on. With bincode we *can't*
                // do that safely:
                //
                // - We don't know this key's value type.
                // - That means we don't know how many bytes to
                //   consume for the value.
                // - If we guess wrong or try `IgnoredAny`, bincode
                //   would need `Deserializer::deserialize_any()` to
                //   skip it, but bincode refuses because it can't
                //   know how many bytes to advance.
                //
                // Result: we cannot safely "skip" the unknown value
                // without risking desync of the remaining stream. So
                // we abort here and surface which key caused it, and
                // the caller must strip it before sending.
                access.next_value::<serde::de::IgnoredAny>().map_err(|_| {
                    serde::de::Error::custom(format!(
                        "unknown attr key '{}' on binary '{}'; \
                         this binary doesn't know this key and cannot skip its value safely under bincode",
                        key_name, exe_name,
                    ))
                })?;
                continue;
            };

            // Create a seed to deserialize the value using erased_serde
            let seed = ValueDeserializeSeed {
                deserialize_erased: key.deserialize_erased,
            };
            match access.next_value_seed(seed) {
                Ok(value) => {
                    attrs.values.insert(key.name, value);
                }
                Err(err) => {
                    return Err(serde::de::Error::custom(format!(
                        "failed to deserialize value for key {}: {}",
                        key_name, err
                    )));
                }
            }
        }

        Ok(attrs)
    }
}

/// Helper struct to deserialize values using erased_serde
struct ValueDeserializeSeed {
    deserialize_erased:
        fn(&mut dyn ErasedDeserializer) -> Result<Box<dyn SerializableValue>, erased_serde::Error>,
}

impl<'de> serde::de::DeserializeSeed<'de> for ValueDeserializeSeed {
    type Value = Box<dyn SerializableValue>;

    fn deserialize<D>(self, deserializer: D) -> Result<Self::Value, D::Error>
    where
        D: serde::de::Deserializer<'de>,
    {
        let mut erased = <dyn erased_serde::Deserializer>::erase(deserializer);
        (self.deserialize_erased)(&mut erased).map_err(serde::de::Error::custom)
    }
}

impl<'de> Deserialize<'de> for Attrs {
    fn deserialize<D>(deserializer: D) -> Result<Self, D::Error>
    where
        D: Deserializer<'de>,
    {
        deserializer.deserialize_map(AttrsVisitor)
    }
}

// Converts an ASCII string to lowercase at compile time.
// Returns a const string with lowercase ASCII characters.
#[doc(hidden)]
pub const fn ascii_to_lowercase_const<const N: usize>(input: &str) -> [u8; N] {
    let bytes = input.as_bytes();
    let mut result = [0u8; N];
    let mut i = 0;

    while i < bytes.len() && i < N {
        let byte = bytes[i];
        if byte >= b'A' && byte <= b'Z' {
            result[i] = byte + 32; // Convert to lowercase
        } else {
            result[i] = byte;
        }
        i += 1;
    }

    result
}

// Macro to generate a const lowercase string at compile time
#[doc(hidden)]
#[macro_export]
macro_rules! const_ascii_lowercase {
    ($s:expr) => {{
        const INPUT: &str = $s;
        const LEN: usize = INPUT.len();
        const BYTES: [u8; LEN] = $crate::attrs::ascii_to_lowercase_const::<LEN>(INPUT);
        // Safety: We're converting ASCII to ASCII, so it's valid UTF-8
        unsafe { std::str::from_utf8_unchecked(&BYTES) }
    }};
}

/// Macro to check check that a trait is implemented, generating a
/// nice error message if it isn't.
#[doc(hidden)]
#[macro_export]
macro_rules! assert_impl {
    ($ty:ty, $trait:path) => {
        const _: fn() = || {
            fn check<T: $trait>() {}
            check::<$ty>();
        };
    };
}

/// Declares attribute keys using a lazy_static! style syntax.
///
/// # Syntax
///
/// ```ignore
/// declare_attrs! {
///     /// Documentation for the key (default visibility).
///     attr KEY_NAME: Type = default_value;
///
///     /// Another key (default value is optional)
///     pub attr ANOTHER_KEY: AnotherType;
/// }
/// ```
///
/// # Arguments
///
/// * Optional visibility modifier (`pub`, `pub(crate)`, etc.)
/// * `attr` keyword (required)
/// * Key name (identifier)
/// * Type of values this key can store
/// * Optional default value
///
/// # Example
///
/// ```
/// use std::time::Duration;
///
/// use hyperactor_config::attrs::Attrs;
/// use hyperactor_config::attrs::declare_attrs;
///
/// declare_attrs! {
///     /// Timeout for RPC operations
///     pub attr TIMEOUT: Duration = Duration::from_secs(30);
///
///     /// Maximum number of retry attempts (no default specified)
///     attr MAX_RETRIES: u32;
/// }
///
/// let mut attrs = Attrs::new();
/// assert_eq!(attrs.get(TIMEOUT), Some(&Duration::from_secs(30)));
/// attrs.set(MAX_RETRIES, 5);
/// ```
#[macro_export]
macro_rules! declare_attrs {
    // Handle multiple attribute keys with optional default values and optional meta attributes
    ($(
        $(#[$attr:meta])*
        $(@meta($($meta_key:ident = $meta_value:expr),* $(,)?))*
        $vis:vis attr $name:ident: $type:ty $(= $default:expr)?;
    )*) => {
        $(
            $crate::declare_attrs! {
                @single
                $(@meta($($meta_key = $meta_value),*))*
                $(#[$attr])* ;
                $vis attr $name: $type $(= $default)?;
            }
        )*
    };

    // Handle single attribute key with default value and meta attributes
    (@single $(@meta($($meta_key:ident = $meta_value:expr),* $(,)?))* $(#[$attr:meta])* ; $vis:vis attr $name:ident: $type:ty = $default:expr;) => {
        $crate::assert_impl!($type, $crate::attrs::AttrValue);

        // Create a static default value
        $crate::paste! {
            static [<$name _DEFAULT>]: $type = $default;
            static [<$name _META_ATTRS>]: std::sync::LazyLock<$crate::attrs::Attrs> =
                std::sync::LazyLock::new(|| {
                    #[allow(unused_mut)]
                    let mut attrs = $crate::attrs::Attrs::new();
                    $($(
                        attrs.set($meta_key, $meta_value);
                    )*)*
                    attrs
                });
        }

        $(#[$attr])*
        $vis static $name: $crate::attrs::Key<$type> = {
            $crate::assert_impl!($type, $crate::attrs::AttrValue);

            const FULL_NAME: &str = concat!(std::module_path!(), "::", stringify!($name));
            const LOWER_NAME: &str = $crate::const_ascii_lowercase!(FULL_NAME);
            $crate::paste! {
                $crate::attrs::Key::new(
                    LOWER_NAME,
                    Some(&[<$name _DEFAULT>]),
                    $crate::paste! { &[<$name _META_ATTRS>] },
                )
            }
        };

        // Register the key for serialization
        $crate::submit! {
            $crate::attrs::AttrKeyInfo {
                name: {
                    const FULL_NAME: &str = concat!(std::module_path!(), "::", stringify!($name));
                    $crate::const_ascii_lowercase!(FULL_NAME)
                },
                key_hash: {
                    const FULL_NAME: &str = concat!(std::module_path!(), "::", stringify!($name));
                    const LOWER_NAME: &str = $crate::const_ascii_lowercase!(FULL_NAME);
                    $crate::attrs::fnv1a_hash(LOWER_NAME.as_bytes())
                },
                typehash: <$type as $crate::typeuri::Named>::typehash,
                deserialize_erased: |deserializer| {
                    let value: $type = erased_serde::deserialize(deserializer)?;
                    Ok(Box::new(value) as Box<dyn $crate::attrs::SerializableValue>)
                },
                deserialize_bincode: |bytes| {
                    let value: $type = $crate::bincode::serde::decode_from_slice(bytes, $crate::bincode::config::legacy()).map(|(v, _)| v)?;
                    Ok(Box::new(value) as Box<dyn $crate::attrs::SerializableValue>)
                },
                meta: $crate::paste! { &[<$name _META_ATTRS>] },
                display: |value: &dyn $crate::attrs::SerializableValue| {
                    let value = value.as_any().downcast_ref::<$type>().unwrap();
                    $crate::attrs::AttrValue::display(value)
                },
                parse: |value: &str| {
                    let value: $type = $crate::attrs::AttrValue::parse(value)?;
                    Ok(Box::new(value) as Box<dyn $crate::attrs::SerializableValue>)
                },
                default: Some($crate::paste! { &[<$name _DEFAULT>] }),
                erased: &$name,
            }
        }
    };

    // Handle single attribute key without default value but with meta attributes
    (@single $(@meta($($meta_key:ident = $meta_value:expr),* $(,)?))* $(#[$attr:meta])* ; $vis:vis attr $name:ident: $type:ty;) => {
        $crate::assert_impl!($type, $crate::attrs::AttrValue);

        $crate::paste! {
            static [<$name _META_ATTRS>]: std::sync::LazyLock<$crate::attrs::Attrs> =
            std::sync::LazyLock::new(|| {
                #[allow(unused_mut)]
                let mut attrs = $crate::attrs::Attrs::new();
                $($(
                    // Note: This assumes meta keys are already declared somewhere
                    // The user needs to ensure the meta keys exist and are in scope
                    attrs.set($meta_key, $meta_value);
                )*)*
                attrs
            });
        }

        $(#[$attr])*
        $vis static $name: $crate::attrs::Key<$type> = {
            const FULL_NAME: &str = concat!(std::module_path!(), "::", stringify!($name));
            const LOWER_NAME: &str = $crate::const_ascii_lowercase!(FULL_NAME);
            $crate::attrs::Key::new(LOWER_NAME, None, $crate::paste! { &[<$name _META_ATTRS>] })
        };


        // Register the key for serialization
        $crate::submit! {
            $crate::attrs::AttrKeyInfo {
                name: {
                    const FULL_NAME: &str = concat!(std::module_path!(), "::", stringify!($name));
                    $crate::const_ascii_lowercase!(FULL_NAME)
                },
                key_hash: {
                    const FULL_NAME: &str = concat!(std::module_path!(), "::", stringify!($name));
                    const LOWER_NAME: &str = $crate::const_ascii_lowercase!(FULL_NAME);
                    $crate::attrs::fnv1a_hash(LOWER_NAME.as_bytes())
                },
                typehash: <$type as $crate::typeuri::Named>::typehash,
                deserialize_erased: |deserializer| {
                    let value: $type = erased_serde::deserialize(deserializer)?;
                    Ok(Box::new(value) as Box<dyn $crate::attrs::SerializableValue>)
                },
                deserialize_bincode: |bytes| {
                    let value: $type = $crate::bincode::serde::decode_from_slice(bytes, $crate::bincode::config::legacy()).map(|(v, _)| v)?;
                    Ok(Box::new(value) as Box<dyn $crate::attrs::SerializableValue>)
                },
                meta: $crate::paste! { &[<$name _META_ATTRS>] },
                display: |value: &dyn $crate::attrs::SerializableValue| {
                    let value = value.as_any().downcast_ref::<$type>().unwrap();
                    $crate::attrs::AttrValue::display(value)
                },
                parse: |value: &str| {
                    let value: $type = $crate::attrs::AttrValue::parse(value)?;
                    Ok(Box::new(value) as Box<dyn $crate::attrs::SerializableValue>)
                },
                default: None,
                erased: &$name,
            }
        }
    };
}

pub use declare_attrs;

#[cfg(test)]
mod tests {
    use std::time::Duration;

    use super::*;

    declare_attrs! {
        attr TEST_TIMEOUT: Duration;
        attr TEST_COUNT: u32;
        @meta(TEST_COUNT = 42)
        pub attr TEST_NAME: String;
        attr TEST_OPTION_COUNT: Option<u32>;
        attr TEST_OPTION_TIMEOUT: Option<Duration>;
    }

    #[test]
    fn test_basic_operations() {
        let mut attrs = Attrs::new();

        // Test setting and getting values
        attrs.set(TEST_TIMEOUT, Duration::from_secs(5));
        attrs.set(TEST_COUNT, 42u32);
        attrs.set(TEST_NAME, "test".to_string());

        assert_eq!(attrs.get(TEST_TIMEOUT), Some(&Duration::from_secs(5)));
        assert_eq!(attrs.get(TEST_COUNT), Some(&42u32));
        assert_eq!(attrs.get(TEST_NAME), Some(&"test".to_string()));

        // Test contains_key
        assert!(attrs.contains_key(TEST_TIMEOUT));
        assert!(attrs.contains_key(TEST_COUNT));
        assert!(attrs.contains_key(TEST_NAME));

        // Test len
        assert_eq!(attrs.len(), 3);
        assert!(!attrs.is_empty());

        // Meta attribute:
        assert_eq!(TEST_NAME.attrs().get(TEST_COUNT).unwrap(), &42u32);
    }

    #[test]
    fn test_get_mut() {
        let mut attrs = Attrs::new();
        attrs.set(TEST_COUNT, 10u32);

        if let Some(count) = attrs.get_mut(TEST_COUNT) {
            *count += 5;
        }

        assert_eq!(attrs.get(TEST_COUNT), Some(&15u32));
    }

    #[test]
    fn test_remove() {
        let mut attrs = Attrs::new();
        attrs.set(TEST_COUNT, 42u32);

        let removed = attrs.remove(TEST_COUNT);
        assert!(removed);
        assert_eq!(attrs.get(TEST_COUNT), None);
        assert!(!attrs.contains_key(TEST_COUNT));
    }

    #[test]
    fn test_clear() {
        let mut attrs = Attrs::new();
        attrs.set(TEST_TIMEOUT, Duration::from_secs(1));
        attrs.set(TEST_COUNT, 42u32);

        attrs.clear();
        assert!(attrs.is_empty());
        assert_eq!(attrs.len(), 0);
    }

    #[test]
    fn test_key_properties() {
        assert_eq!(
            TEST_TIMEOUT.name(),
            "hyperactor_config::attrs::tests::test_timeout"
        );
    }

    #[test]
    fn test_serialization() {
        let mut attrs = Attrs::new();
        attrs.set(TEST_TIMEOUT, Duration::from_secs(5));
        attrs.set(TEST_COUNT, 42u32);
        attrs.set(TEST_NAME, "test".to_string());

        // Test serialization
        let serialized = serde_json::to_string(&attrs).expect("Failed to serialize");

        // The serialized string should contain the key names and their values
        assert!(serialized.contains("hyperactor_config::attrs::tests::test_timeout"));
        assert!(serialized.contains("hyperactor_config::attrs::tests::test_count"));
        assert!(serialized.contains("hyperactor_config::attrs::tests::test_name"));
    }

    #[test]
    fn test_deserialization() {
        // Create original attrs
        let mut original_attrs = Attrs::new();
        original_attrs.set(TEST_TIMEOUT, Duration::from_secs(5));
        original_attrs.set(TEST_COUNT, 42u32);
        original_attrs.set(TEST_NAME, "test".to_string());

        // Serialize
        let serialized = serde_json::to_string(&original_attrs).expect("Failed to serialize");

        // Deserialize (no manual registry needed!)
        let deserialized_attrs: Attrs =
            serde_json::from_str(&serialized).expect("Failed to deserialize");

        // Verify the deserialized values
        assert_eq!(
            deserialized_attrs.get(TEST_TIMEOUT),
            Some(&Duration::from_secs(5))
        );
        assert_eq!(deserialized_attrs.get(TEST_COUNT), Some(&42u32));
        assert_eq!(deserialized_attrs.get(TEST_NAME), Some(&"test".to_string()));
    }

    #[test]
    fn test_roundtrip_serialization() {
        // Create original attrs
        let mut original = Attrs::new();
        original.set(TEST_TIMEOUT, Duration::from_secs(10));
        original.set(TEST_COUNT, 5u32);
        original.set(TEST_OPTION_COUNT, Some(5u32));
        original.set(TEST_OPTION_TIMEOUT, None);
        original.set(TEST_NAME, "test-service".to_string());

        // Serialize
        let serialized = serde_json::to_string(&original).unwrap();

        // Deserialize
        let deserialized: Attrs = serde_json::from_str(&serialized).unwrap();

        // Verify round-trip worked
        assert_eq!(
            deserialized.get(TEST_TIMEOUT),
            Some(&Duration::from_secs(10))
        );
        assert_eq!(deserialized.get(TEST_COUNT), Some(&5u32));
        assert_eq!(deserialized.get(TEST_OPTION_COUNT), Some(&Some(5u32)));
        assert_eq!(deserialized.get(TEST_OPTION_TIMEOUT), Some(&None));
        assert_eq!(
            deserialized.get(TEST_NAME),
            Some(&"test-service".to_string())
        );
    }

    #[test]
    fn test_empty_attrs_serialization() {
        let attrs = Attrs::new();
        let serialized = serde_json::to_string(&attrs).unwrap();

        // Empty attrs should serialize to empty JSON object
        assert_eq!(serialized, "{}");

        let deserialized: Attrs = serde_json::from_str(&serialized).unwrap();

        assert!(deserialized.is_empty());
    }

    #[test]
    fn test_format_independence() {
        // Test that proves we're using the serializer directly, not JSON internally
        let mut attrs = Attrs::new();
        attrs.set(TEST_COUNT, 42u32);
        attrs.set(TEST_NAME, "test".to_string());

        // Serialize to different formats
        let json_output = serde_json::to_string(&attrs).unwrap();
        let yaml_output = serde_yaml::to_string(&attrs).unwrap();

        // JSON should have colons and quotes
        assert!(json_output.contains(":"));
        assert!(json_output.contains("\""));

        // JSON should serialize numbers as numbers, not strings
        assert!(json_output.contains("42"));
        assert!(!json_output.contains("\"42\""));

        // YAML should have colons but different formatting
        assert!(yaml_output.contains(":"));
        assert!(yaml_output.contains("42"));

        // YAML shouldn't quote simple strings or numbers
        assert!(!yaml_output.contains("\"42\""));

        // The outputs should be different (proving different serializers were used)
        assert_ne!(json_output, yaml_output);

        // Verify that both can be deserialized correctly
        let from_json: Attrs = serde_json::from_str(&json_output).unwrap();
        let from_yaml: Attrs = serde_yaml::from_str(&yaml_output).unwrap();

        assert_eq!(from_json.get(TEST_COUNT), Some(&42u32));
        assert_eq!(from_yaml.get(TEST_COUNT), Some(&42u32));
        assert_eq!(from_json.get(TEST_NAME), Some(&"test".to_string()));
        assert_eq!(from_yaml.get(TEST_NAME), Some(&"test".to_string()));
    }

    #[test]
    fn test_clone() {
        // Create original attrs with multiple types
        let mut original = Attrs::new();
        original.set(TEST_COUNT, 42u32);
        original.set(TEST_NAME, "test".to_string());
        original.set(TEST_TIMEOUT, std::time::Duration::from_secs(10));

        // Clone the attrs
        let cloned = original.clone();

        // Verify that the clone has the same values
        assert_eq!(cloned.get(TEST_COUNT), Some(&42u32));
        assert_eq!(cloned.get(TEST_NAME), Some(&"test".to_string()));
        assert_eq!(
            cloned.get(TEST_TIMEOUT),
            Some(&std::time::Duration::from_secs(10))
        );

        // Verify that modifications to the original don't affect the clone
        original.set(TEST_COUNT, 100u32);
        assert_eq!(original.get(TEST_COUNT), Some(&100u32));
        assert_eq!(cloned.get(TEST_COUNT), Some(&42u32)); // Clone should be unchanged

        // Verify that modifications to the clone don't affect the original
        let mut cloned_mut = cloned.clone();
        cloned_mut.set(TEST_NAME, "modified".to_string());
        assert_eq!(cloned_mut.get(TEST_NAME), Some(&"modified".to_string()));
        assert_eq!(original.get(TEST_NAME), Some(&"test".to_string())); // Original should be unchanged
    }

    #[test]
    fn test_debug_with_json() {
        let mut attrs = Attrs::new();
        attrs.set(TEST_COUNT, 42u32);
        attrs.set(TEST_NAME, "test".to_string());

        // Test that Debug implementation works and contains JSON representations
        let debug_output = format!("{:?}", attrs);

        // Should contain the struct name
        assert!(debug_output.contains("Attrs"));

        // Should contain JSON representations of the values
        assert!(debug_output.contains("42"));

        // Should contain the key names
        assert!(debug_output.contains("hyperactor_config::attrs::tests::test_count"));
        assert!(debug_output.contains("hyperactor_config::attrs::tests::test_name"));

        // For strings, the JSON representation should be the escaped version
        // Let's check that the test string is actually present in some form
        assert!(debug_output.contains("test"));
    }

    declare_attrs! {
        /// With default...
        attr TIMEOUT_WITH_DEFAULT: Duration = Duration::from_secs(10);

        /// Just to ensure visibilty is parsed.
        pub(crate) attr CRATE_LOCAL_ATTR: String;
    }

    #[test]
    fn test_defaults() {
        assert!(TIMEOUT_WITH_DEFAULT.has_default());
        assert!(!CRATE_LOCAL_ATTR.has_default());

        assert_eq!(
            Attrs::new().get(TIMEOUT_WITH_DEFAULT),
            Some(&Duration::from_secs(10))
        );
    }

    #[test]
    fn test_indexing() {
        let mut attrs = Attrs::new();

        assert_eq!(attrs[TIMEOUT_WITH_DEFAULT], Duration::from_secs(10));
        attrs[TIMEOUT_WITH_DEFAULT] = Duration::from_secs(100);
        assert_eq!(attrs[TIMEOUT_WITH_DEFAULT], Duration::from_secs(100));

        attrs.set(CRATE_LOCAL_ATTR, "test".to_string());
        assert_eq!(attrs[CRATE_LOCAL_ATTR], "test".to_string());
    }

    #[test]
    fn attrs_deserialize_unknown_key_is_error() {
        // Build a real Attrs, but inject a key that this binary does
        // NOT know about. We do that with
        // insert_value_by_name_unchecked(), which bypasses the
        // declare_attrs!/inventory registration path.
        //
        // Then:
        //   1. Serialize that Attrs with bincode (the real wire
        //      format).
        //   2. Attempt to bincode-deserialize those bytes back into
        //      Attrs.
        //
        // During deserialization, AttrsVisitor::visit_map() will:
        //   - read the key string
        //   - fail to find it in KEYS_BY_NAME (the compiled-in
        //     registry)
        //   - immediately error instead of trying to skip the value,
        //     because with bincode we can't safely consume an unknown
        //     typed value without risking stream desync.
        //
        // This reproduces exactly what happens when a parent proc
        // sends an Attrs containing a key the child binary wasn't
        // built with.

        // Definitely not declared in this crate's inventory:
        let bad_key: &'static str = "monarch_hyperactor::pytokio::unawaited_pytokio_traceback";

        // Make an Attrs that pretends to have that key. u32 already
        // implements AttrValue -> SerializableValue, so we can just
        // box a 0u32.
        let mut attrs = Attrs::new();
        attrs.insert_value_by_name_unchecked(bad_key, Box::new(0u32));

        // Serialize this Attrs using bincode (same codec we use on
        // the wire).
        let wire_bytes = bincode::serde::encode_to_vec(&attrs, bincode::config::legacy()).unwrap();

        // Now try to decode those bytes back into Attrs. This should
        // hit the unknown-key branch and return Err.
        let err =
            bincode::serde::decode_from_slice::<Attrs, _>(&wire_bytes, bincode::config::legacy())
                .map(|(v, _)| v)
                .expect_err("should error on unknown attr key");

        let exe_str = std::env::current_exe()
            .ok()
            .map(|p| p.display().to_string())
            .unwrap_or_else(|| "<unknown-exe>".to_string());
        let msg = format!("{err}");

        assert!(msg.contains("unknown attr key"), "got: {msg}");
        assert!(msg.contains(bad_key), "got: {msg}");
        assert!(msg.contains(&exe_str), "got: {msg}");
    }

    // Verify that Vec<T> values work as attribute values when T:
    // AttrValue.
    //
    // This checks two things:
    //  1. A Vec<String> can be stored and retrieved through the Attrs
    //    API.
    //  2. The value survives a JSON round-trip via serde
    //     serialization.
    #[test]
    fn test_vec_string_attr() {
        declare_attrs! {
            attr TEST_VEC: Vec<String>;
        }
        let mut attrs = Attrs::new();
        let original = vec!["a".to_string(), "b".to_string()];
        attrs.set(TEST_VEC, original.clone());
        assert_eq!(attrs.get(TEST_VEC), Some(&original));
        // Round-trip through Attrs JSON serialization
        let json = serde_json::to_string(&attrs).unwrap();
        let attrs2: Attrs = serde_json::from_str(&json).unwrap();
        assert_eq!(attrs2.get(TEST_VEC), Some(&original));
        // Round-trip through AttrValue display/parse
        let displayed = AttrValue::display(&original);
        let parsed: Vec<String> = AttrValue::parse(&displayed).unwrap();
        assert_eq!(parsed, original);
    }

    declare_attrs! {
        /// Test-local marker for the generic mechanism's
        /// membership / stamp / copy / enumeration tests. Scoped to
        /// the test module so the mechanism's coverage does not
        /// depend on any caller-defined marker vocabulary.
        pub attr TEST_GENERIC_MARKER: bool;

        /// Declared attr carrying the test-local marker.
        @meta(TEST_GENERIC_MARKER = true)
        pub attr TEST_MARKED_ATTR: String;

        /// Declared attr NOT carrying the test-local marker.
        pub attr TEST_UNMARKED_ATTR: String;
    }

    // Generic marker-parameterized membership check: the helper
    // does not know about any specific category; callers supply
    // the marker. Validates that declared keys with
    // `@meta(TEST_GENERIC_MARKER = true)` match and keys without
    // do not.
    #[test]
    fn test_is_attr_marked_with() {
        assert_eq!(
            is_attr_marked_with(TEST_MARKED_ATTR.name(), TEST_GENERIC_MARKER),
            Some(true),
            "marked key must report Some(true)",
        );
        assert_eq!(
            is_attr_marked_with(TEST_UNMARKED_ATTR.name(), TEST_GENERIC_MARKER),
            None,
            "unmarked key must report None (marker not present on its meta)",
        );
        assert_eq!(
            is_attr_marked_with(
                "hyperactor_config::attrs::tests::no_such_key_declared_anywhere",
                TEST_GENERIC_MARKER,
            ),
            None,
            "unknown names must report None",
        );
    }

    // Generic marker-parameterized Attrs → Flattrs stamp. Builds
    // an `Attrs` with one marked and one unmarked entry; stamps
    // via the generic helper with `TEST_GENERIC_MARKER`; asserts
    // only the marked entry lands on headers.
    #[test]
    fn test_stamp_marked_attrs_into_flattrs() {
        use crate::flattrs::Flattrs;

        let mut attrs = Attrs::new();
        attrs.set(TEST_MARKED_ATTR, "marked_value".to_string());
        attrs.set(TEST_UNMARKED_ATTR, "unmarked_value".to_string());

        let mut headers = Flattrs::new();
        stamp_marked_attrs_into_flattrs(&mut headers, &attrs, TEST_GENERIC_MARKER);

        assert_eq!(
            headers.get(TEST_MARKED_ATTR),
            Some("marked_value".to_string()),
        );
        assert_eq!(headers.get::<String>(TEST_UNMARKED_ATTR), None);
    }

    // Generic marker-parameterized Flattrs → Flattrs copy. The
    // source Flattrs gets populated with both marked and unmarked
    // values via typed `set` (by-key-hash stamping); the generic
    // copy resolves each key_hash through the inventory, filters by
    // marker, and copies only the marked entry.
    #[test]
    fn test_copy_marked_flattrs() {
        use crate::flattrs::Flattrs;

        let mut src = Flattrs::new();
        src.set(TEST_MARKED_ATTR, "marked".to_string());
        src.set(TEST_UNMARKED_ATTR, "unmarked".to_string());

        let mut dst = Flattrs::new();
        copy_marked_flattrs(&mut dst, &src, TEST_GENERIC_MARKER);

        assert_eq!(dst.get(TEST_MARKED_ATTR), Some("marked".to_string()),);
        assert_eq!(dst.get::<String>(TEST_UNMARKED_ATTR), None);
    }

    // Generic marker-parameterized enumeration. The test-local
    // `TEST_MARKED_ATTR` is marked, so it must appear; the
    // unmarked test attr must not.
    #[test]
    fn test_marked_attr_names_enumeration() {
        let names = marked_attr_names(TEST_GENERIC_MARKER);
        assert!(
            names.contains(TEST_MARKED_ATTR.name()),
            "marked test attr must appear in the vocabulary enumeration",
        );
        assert!(
            !names.contains(TEST_UNMARKED_ATTR.name()),
            "unmarked test attr must not appear in the vocabulary enumeration",
        );
    }
}