runmat-runtime 0.6.0

//! MATLAB-compatible timer handle objects.
//!
//! Timer objects are handle objects with MATLAB timer properties. RunMat keeps
//! timers in a per-thread registry so `timerfind` can discover them after the
//! constructor returns. Lifecycle methods execute callbacks on the current VM
//! path rather than a background event thread; this keeps callback execution
//! deterministic across native and wasm hosts.

use std::cell::{Cell, RefCell};
use std::collections::HashMap;
#[cfg(not(target_arch = "wasm32"))]
use std::time::Duration;

#[cfg(test)]
use once_cell::sync::Lazy;
#[cfg(test)]
use std::sync::Mutex;

use runmat_builtins::{
    Access, BuiltinCompletionPolicy, BuiltinDescriptor, BuiltinErrorDescriptor, BuiltinOutputMode,
    BuiltinParamArity, BuiltinParamDescriptor, BuiltinParamType, BuiltinSignatureDescriptor,
    CellArray, ClassDef, HandleRef, MethodDef, ObjectInstance, PropertyDef, StructValue, Value,
};
use runmat_macros::runtime_builtin;

use crate::builtins::common::spec::{
    BroadcastSemantics, BuiltinFusionSpec, BuiltinGpuSpec, ConstantStrategy, GpuOpKind,
    ReductionNaN, ResidencyPolicy, ShapeRequirements,
};
use crate::{build_runtime_error, BuiltinResult, RuntimeError};

const TIMER_CLASS: &str = "timer";
const BUILTIN_TIMER: &str = "timer";
const TIMER_METHOD_START: &str = "__runmat_timer_start";
const TIMER_METHOD_STARTAT: &str = "__runmat_timer_startat";
const TIMER_METHOD_STOP: &str = "__runmat_timer_stop";
const TIMER_METHOD_WAIT: &str = "__runmat_timer_wait";
const TIMER_METHOD_DELETE: &str = "__runmat_timer_delete";

const CALLBACK_PROPS: [&str; 4] = ["TimerFcn", "StartFcn", "StopFcn", "ErrorFcn"];
const STRING_PROPS: [&str; 5] = [
    "BusyMode",
    "ExecutionMode",
    "Name",
    "Tag",
    "ObjectVisibility",
];
const NUMERIC_PROPS: [&str; 3] = ["Period", "StartDelay", "TasksToExecute"];
const READONLY_PROPS: [&str; 5] = [
    "AveragePeriod",
    "InstantPeriod",
    "Running",
    "TasksExecuted",
    "Type",
];

thread_local! {
    static TIMER_REGISTRY: RefCell<Vec<HandleRef>> = const { RefCell::new(Vec::new()) };
    static TIMER_COUNTER: Cell<usize> = const { Cell::new(0) };
}

#[cfg(test)]
static TIMER_TEST_LOCK: Lazy<Mutex<()>> = Lazy::new(|| Mutex::new(()));

const TIMER_OUTPUT: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "t",
    ty: BuiltinParamType::Any,
    arity: BuiltinParamArity::Required,
    default: None,
    description: "Timer handle object.",
}];

const TIMER_INPUTS: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "Name,Value",
    ty: BuiltinParamType::Any,
    arity: BuiltinParamArity::Variadic,
    default: None,
    description: "Timer properties such as TimerFcn, StartDelay, Period, ExecutionMode, Name, Tag, and UserData.",
}];

const TIMER_SIGNATURES: [BuiltinSignatureDescriptor; 2] = [
    BuiltinSignatureDescriptor {
        label: "t = timer",
        inputs: &[],
        outputs: &TIMER_OUTPUT,
    },
    BuiltinSignatureDescriptor {
        label: "t = timer(Name, Value, ...)",
        inputs: &TIMER_INPUTS,
        outputs: &TIMER_OUTPUT,
    },
];

const TIMERFIND_OUTPUT: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "timers",
    ty: BuiltinParamType::Any,
    arity: BuiltinParamArity::Required,
    default: None,
    description: "Matching timer handle, empty cell row, or cell row of timer handles.",
}];

const TIMERFIND_INPUTS: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "Name,Value",
    ty: BuiltinParamType::Any,
    arity: BuiltinParamArity::Variadic,
    default: None,
    description: "Property filters matched by exact value.",
}];

const TIMERFIND_SIGNATURES: [BuiltinSignatureDescriptor; 2] = [
    BuiltinSignatureDescriptor {
        label: "timers = timerfind",
        inputs: &[],
        outputs: &TIMERFIND_OUTPUT,
    },
    BuiltinSignatureDescriptor {
        label: "timers = timerfind(Name, Value, ...)",
        inputs: &TIMERFIND_INPUTS,
        outputs: &TIMERFIND_OUTPUT,
    },
];

const TIMERFINDALL_SIGNATURES: [BuiltinSignatureDescriptor; 2] = [
    BuiltinSignatureDescriptor {
        label: "timers = timerfindall",
        inputs: &[],
        outputs: &TIMERFIND_OUTPUT,
    },
    BuiltinSignatureDescriptor {
        label: "timers = timerfindall(Name, Value, ...)",
        inputs: &TIMERFIND_INPUTS,
        outputs: &TIMERFIND_OUTPUT,
    },
];

const TIMER_METHOD_OUTPUT: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "status",
    ty: BuiltinParamType::NumericScalar,
    arity: BuiltinParamArity::Optional,
    default: None,
    description: "Zero on success.",
}];

const TIMER_METHOD_INPUTS: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "t",
    ty: BuiltinParamType::Any,
    arity: BuiltinParamArity::Required,
    default: None,
    description: "Timer handle or cell array of timer handles.",
}];

const TIMER_METHOD_SIGNATURES: [BuiltinSignatureDescriptor; 1] = [BuiltinSignatureDescriptor {
    label: "status = method(t)",
    inputs: &TIMER_METHOD_INPUTS,
    outputs: &TIMER_METHOD_OUTPUT,
}];

const TIMER_SETTER_OUTPUT: [BuiltinParamDescriptor; 1] = [BuiltinParamDescriptor {
    name: "obj",
    ty: BuiltinParamType::Any,
    arity: BuiltinParamArity::Required,
    default: None,
    description: "Updated timer object.",
}];

const TIMER_SETTER_INPUTS: [BuiltinParamDescriptor; 2] = [
    BuiltinParamDescriptor {
        name: "obj",
        ty: BuiltinParamType::Any,
        arity: BuiltinParamArity::Required,
        default: None,
        description: "Timer object receiver.",
    },
    BuiltinParamDescriptor {
        name: "value",
        ty: BuiltinParamType::Any,
        arity: BuiltinParamArity::Required,
        default: None,
        description: "Property value to validate and assign.",
    },
];

const TIMER_SETTER_SIGNATURES: [BuiltinSignatureDescriptor; 1] = [BuiltinSignatureDescriptor {
    label: "obj = set.Property(obj, value)",
    inputs: &TIMER_SETTER_INPUTS,
    outputs: &TIMER_SETTER_OUTPUT,
}];

const TIMER_ERROR_INVALID_INPUT: BuiltinErrorDescriptor = BuiltinErrorDescriptor {
    code: "RM.TIMER.INVALID_INPUT",
    identifier: Some("RunMat:timer:InvalidInput"),
    when: "A timer argument, property, or callback value has an unsupported type.",
    message: "timer: invalid input",
};

const TIMER_ERROR_INVALID_PROPERTY: BuiltinErrorDescriptor = BuiltinErrorDescriptor {
    code: "RM.TIMER.INVALID_PROPERTY",
    identifier: Some("RunMat:timer:InvalidProperty"),
    when: "A timer property name is unknown, read-only, or has an invalid value.",
    message: "timer: invalid property",
};

const TIMER_ERROR_INVALID_HANDLE: BuiltinErrorDescriptor = BuiltinErrorDescriptor {
    code: "RM.TIMER.INVALID_HANDLE",
    identifier: Some("RunMat:timer:InvalidHandle"),
    when: "A lifecycle method receives a non-timer or invalid timer handle.",
    message: "timer: invalid timer handle",
};

const TIMER_ERROR_CALLBACK: BuiltinErrorDescriptor = BuiltinErrorDescriptor {
    code: "RM.TIMER.CALLBACK",
    identifier: Some("RunMat:timer:CallbackFailed"),
    when: "A timer callback fails.",
    message: "timer: callback failed",
};

const TIMER_ERROR_GC: BuiltinErrorDescriptor = BuiltinErrorDescriptor {
    code: "RM.TIMER.GC",
    identifier: Some("RunMat:timer:GcFailure"),
    when: "A timer object cannot be allocated, rooted, read, or mutated.",
    message: "timer: object storage failed",
};

const TIMER_ERRORS: [BuiltinErrorDescriptor; 5] = [
    TIMER_ERROR_INVALID_INPUT,
    TIMER_ERROR_INVALID_PROPERTY,
    TIMER_ERROR_INVALID_HANDLE,
    TIMER_ERROR_CALLBACK,
    TIMER_ERROR_GC,
];

pub const TIMER_DESCRIPTOR: BuiltinDescriptor = BuiltinDescriptor {
    signatures: &TIMER_SIGNATURES,
    output_mode: BuiltinOutputMode::Fixed,
    completion_policy: BuiltinCompletionPolicy::Public,
    errors: &TIMER_ERRORS,
};

pub const TIMERFIND_DESCRIPTOR: BuiltinDescriptor = BuiltinDescriptor {
    signatures: &TIMERFIND_SIGNATURES,
    output_mode: BuiltinOutputMode::Fixed,
    completion_policy: BuiltinCompletionPolicy::Public,
    errors: &TIMER_ERRORS,
};

pub const TIMERFINDALL_DESCRIPTOR: BuiltinDescriptor = BuiltinDescriptor {
    signatures: &TIMERFINDALL_SIGNATURES,
    output_mode: BuiltinOutputMode::Fixed,
    completion_policy: BuiltinCompletionPolicy::Public,
    errors: &TIMER_ERRORS,
};

pub const TIMER_METHOD_DESCRIPTOR: BuiltinDescriptor = BuiltinDescriptor {
    signatures: &TIMER_METHOD_SIGNATURES,
    output_mode: BuiltinOutputMode::Fixed,
    completion_policy: BuiltinCompletionPolicy::HiddenInternal,
    errors: &TIMER_ERRORS,
};

pub const TIMER_SETTER_DESCRIPTOR: BuiltinDescriptor = BuiltinDescriptor {
    signatures: &TIMER_SETTER_SIGNATURES,
    output_mode: BuiltinOutputMode::Fixed,
    completion_policy: BuiltinCompletionPolicy::MethodOnly,
    errors: &TIMER_ERRORS,
};

#[runmat_macros::register_gpu_spec(builtin_path = "crate::builtins::timing::timer")]
pub const GPU_SPEC: BuiltinGpuSpec = BuiltinGpuSpec {
    name: "timer",
    op_kind: GpuOpKind::Custom("timer"),
    supported_precisions: &[],
    broadcast: BroadcastSemantics::None,
    provider_hooks: &[],
    constant_strategy: ConstantStrategy::InlineLiteral,
    residency: ResidencyPolicy::GatherImmediately,
    nan_mode: ReductionNaN::Include,
    two_pass_threshold: None,
    workgroup_size: None,
    accepts_nan_mode: false,
    notes: "Timer handles are host control-flow objects. Timer callbacks may call GPU-capable builtins, but timer itself has no provider kernel.",
};

#[runmat_macros::register_fusion_spec(builtin_path = "crate::builtins::timing::timer")]
pub const FUSION_SPEC: BuiltinFusionSpec = BuiltinFusionSpec {
    name: "timer",
    shape: ShapeRequirements::Any,
    constant_strategy: ConstantStrategy::InlineLiteral,
    elementwise: None,
    reduction: None,
    emits_nan: false,
    notes: "Timer object creation, discovery, and callback execution are side-effect boundaries and are excluded from fusion.",
};

#[runtime_builtin(
    name = "timer",
    category = "timing",
    summary = "Create a MATLAB-compatible timer handle object.",
    keywords = "timer,callback,start,stop,wait,timerfind",
    descriptor(crate::builtins::timing::timer::TIMER_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub async fn timer_builtin(args: Vec<Value>) -> BuiltinResult<Value> {
    ensure_timer_class_registered();
    let mut object = default_timer_object(next_timer_name());
    apply_name_value_pairs(&mut object, &args, true)?;
    let target = runmat_gc::gc_allocate(Value::Object(object))
        .map_err(|err| timer_error(&TIMER_ERROR_GC, format!("timer: {err}")))?;
    runmat_gc::gc_add_root(target)
        .map_err(|err| timer_error(&TIMER_ERROR_GC, format!("timer: {err}")))?;
    let handle = HandleRef {
        class_name: TIMER_CLASS.to_string(),
        target,
        valid: true,
    };
    TIMER_REGISTRY.with(|registry| registry.borrow_mut().push(handle.clone()));
    Ok(Value::HandleObject(handle))
}

#[runtime_builtin(
    name = "timerfind",
    category = "timing",
    summary = "Find visible timer objects by property value.",
    keywords = "timerfind,timer,callback,handle",
    descriptor(crate::builtins::timing::timer::TIMERFIND_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub fn timerfind_builtin(args: Vec<Value>) -> BuiltinResult<Value> {
    find_timers(args, false)
}

#[runtime_builtin(
    name = "timerfindall",
    category = "timing",
    summary = "Find all timer objects by property value.",
    keywords = "timerfindall,timer,callback,handle",
    descriptor(crate::builtins::timing::timer::TIMERFINDALL_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub fn timerfindall_builtin(args: Vec<Value>) -> BuiltinResult<Value> {
    find_timers(args, true)
}

#[runtime_builtin(
    name = "__runmat_timer_start",
    category = "timing",
    summary = "Start a timer object.",
    keywords = "timer,start",
    sink = true,
    suppress_auto_output = true,
    descriptor(crate::builtins::timing::timer::TIMER_METHOD_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub async fn timer_start_builtin(value: Value) -> BuiltinResult<Value> {
    start_timer_values(&value, None).await?;
    Ok(Value::Num(0.0))
}

#[runtime_builtin(
    name = "__runmat_timer_startat",
    category = "timing",
    summary = "Start a timer object at a requested time.",
    keywords = "timer,startat",
    sink = true,
    suppress_auto_output = true,
    descriptor(crate::builtins::timing::timer::TIMER_METHOD_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub async fn timer_startat_builtin(value: Value, rest: Vec<Value>) -> BuiltinResult<Value> {
    let delay = rest
        .first()
        .map(|value| numeric_scalar(value, "startat"))
        .transpose()?
        .filter(|seconds| *seconds > 0.0);
    start_timer_values(&value, delay).await?;
    Ok(Value::Num(0.0))
}

#[runtime_builtin(
    name = "__runmat_timer_stop",
    category = "timing",
    summary = "Stop a timer object.",
    keywords = "timer,stop",
    sink = true,
    suppress_auto_output = true,
    descriptor(crate::builtins::timing::timer::TIMER_METHOD_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub async fn timer_stop_builtin(value: Value) -> BuiltinResult<Value> {
    stop_timer_values(&value, true).await?;
    Ok(Value::Num(0.0))
}

#[runtime_builtin(
    name = "__runmat_timer_wait",
    category = "timing",
    summary = "Wait until timer objects finish.",
    keywords = "timer,wait",
    sink = true,
    suppress_auto_output = true,
    descriptor(crate::builtins::timing::timer::TIMER_METHOD_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub async fn timer_wait_builtin(value: Value) -> BuiltinResult<Value> {
    wait_timer_values(&value).await?;
    Ok(Value::Num(0.0))
}

#[runtime_builtin(
    name = "__runmat_timer_delete",
    category = "timing",
    summary = "Delete a timer object.",
    keywords = "timer,delete",
    sink = true,
    suppress_auto_output = true,
    descriptor(crate::builtins::timing::timer::TIMER_METHOD_DESCRIPTOR),
    builtin_path = "crate::builtins::timing::timer"
)]
pub async fn timer_delete_builtin(value: Value) -> BuiltinResult<Value> {
    delete_timer_values(&value).await?;
    Ok(Value::Num(0.0))
}

macro_rules! timer_setter_builtin {
    ($fn_name:ident, $builtin_name:literal, $property:literal) => {
        #[runtime_builtin(
                            name = $builtin_name,
                            category = "timing",
                            summary = "Validate and assign a timer property.",
                            keywords = "timer,set,property",
                            descriptor(crate::builtins::timing::timer::TIMER_SETTER_DESCRIPTOR),
                            builtin_path = "crate::builtins::timing::timer"
                        )]
        pub async fn $fn_name(obj: Value, value: Value) -> BuiltinResult<Value> {
            set_timer_property_object(obj, $property, value).await
        }
    };
}

timer_setter_builtin!(timer_set_timer_fcn_builtin, "set.TimerFcn", "TimerFcn");
timer_setter_builtin!(timer_set_start_fcn_builtin, "set.StartFcn", "StartFcn");
timer_setter_builtin!(timer_set_stop_fcn_builtin, "set.StopFcn", "StopFcn");
timer_setter_builtin!(timer_set_error_fcn_builtin, "set.ErrorFcn", "ErrorFcn");
timer_setter_builtin!(timer_set_period_builtin, "set.Period", "Period");
timer_setter_builtin!(
    timer_set_start_delay_builtin,
    "set.StartDelay",
    "StartDelay"
);
timer_setter_builtin!(
    timer_set_tasks_to_execute_builtin,
    "set.TasksToExecute",
    "TasksToExecute"
);
timer_setter_builtin!(timer_set_busy_mode_builtin, "set.BusyMode", "BusyMode");
timer_setter_builtin!(
    timer_set_execution_mode_builtin,
    "set.ExecutionMode",
    "ExecutionMode"
);
timer_setter_builtin!(timer_set_name_builtin, "set.Name", "Name");
timer_setter_builtin!(timer_set_tag_builtin, "set.Tag", "Tag");
timer_setter_builtin!(
    timer_set_object_visibility_builtin,
    "set.ObjectVisibility",
    "ObjectVisibility"
);
timer_setter_builtin!(timer_set_user_data_builtin, "set.UserData", "UserData");

fn ensure_timer_class_registered() {
    if runmat_builtins::get_class(TIMER_CLASS).is_some() {
        return;
    }

    let mut methods = HashMap::new();
    for (name, function_name) in [
        ("start", TIMER_METHOD_START),
        ("startat", TIMER_METHOD_STARTAT),
        ("stop", TIMER_METHOD_STOP),
        ("wait", TIMER_METHOD_WAIT),
        ("delete", TIMER_METHOD_DELETE),
    ] {
        methods.insert(
            name.to_string(),
            MethodDef {
                name: name.to_string(),
                is_static: false,
                is_abstract: false,
                is_sealed: false,
                access: Access::Public,
                function_name: function_name.to_string(),
                implicit_class_argument: None,
            },
        );
    }

    let mut properties = HashMap::new();
    for name in CALLBACK_PROPS
        .into_iter()
        .chain(STRING_PROPS)
        .chain(NUMERIC_PROPS)
        .chain(READONLY_PROPS)
        .chain(["UserData"])
    {
        properties.insert(
            name.to_string(),
            PropertyDef {
                name: name.to_string(),
                is_static: false,
                is_constant: false,
                is_dependent: is_timer_mutable_property(name),
                get_access: Access::Public,
                set_access: if READONLY_PROPS.contains(&name) {
                    Access::Private
                } else {
                    Access::Public
                },
                default_value: None,
            },
        );
    }

    runmat_builtins::register_class(ClassDef {
        name: TIMER_CLASS.to_string(),
        parent: Some("handle".to_string()),
        properties,
        methods,
    });
}

fn is_timer_mutable_property(name: &str) -> bool {
    CALLBACK_PROPS.contains(&name)
        || STRING_PROPS.contains(&name)
        || NUMERIC_PROPS.contains(&name)
        || name == "UserData"
}

fn default_timer_object(name: String) -> ObjectInstance {
    let mut object = ObjectInstance::new(TIMER_CLASS.to_string());
    for prop in CALLBACK_PROPS {
        object
            .properties
            .insert(prop.to_string(), Value::String(String::new()));
    }
    object
        .properties
        .insert("Period".to_string(), Value::Num(1.0));
    object
        .properties
        .insert("StartDelay".to_string(), Value::Num(0.0));
    object
        .properties
        .insert("TasksToExecute".to_string(), Value::Num(1.0));
    object
        .properties
        .insert("BusyMode".to_string(), Value::String("drop".to_string()));
    object.properties.insert(
        "ExecutionMode".to_string(),
        Value::String("singleShot".to_string()),
    );
    object
        .properties
        .insert("Name".to_string(), Value::String(name));
    object
        .properties
        .insert("Tag".to_string(), Value::String(String::new()));
    object.properties.insert(
        "ObjectVisibility".to_string(),
        Value::String("on".to_string()),
    );
    object
        .properties
        .insert("UserData".to_string(), Value::Num(0.0));
    object
        .properties
        .insert("AveragePeriod".to_string(), Value::Num(f64::NAN));
    object
        .properties
        .insert("InstantPeriod".to_string(), Value::Num(f64::NAN));
    object
        .properties
        .insert("Running".to_string(), Value::String("off".to_string()));
    object
        .properties
        .insert("TasksExecuted".to_string(), Value::Num(0.0));
    object
        .properties
        .insert("Type".to_string(), Value::String(TIMER_CLASS.to_string()));
    object.properties.insert(
        crate::HANDLE_VALID_FLAG_PROPERTY.to_string(),
        Value::Bool(true),
    );
    object
}

fn next_timer_name() -> String {
    TIMER_COUNTER.with(|counter| {
        let next = counter.get() + 1;
        counter.set(next);
        format!("timer-{next}")
    })
}

fn apply_name_value_pairs(
    object: &mut ObjectInstance,
    args: &[Value],
    constructor: bool,
) -> BuiltinResult<()> {
    if !args.len().is_multiple_of(2) {
        return Err(timer_error(
            &TIMER_ERROR_INVALID_INPUT,
            "timer: name-value arguments must appear in pairs",
        ));
    }
    let mut idx = 0usize;
    while idx < args.len() {
        let name = canonical_property_name(&value_to_string(&args[idx])?)?;
        if READONLY_PROPS.contains(&name.as_str()) {
            return Err(timer_error(
                &TIMER_ERROR_INVALID_PROPERTY,
                format!("timer: property '{name}' is read-only"),
            ));
        }
        let value = normalize_property_value(&name, args[idx + 1].clone())?;
        if !constructor && running_is_on(object) && is_running_readonly_property(&name) {
            return Err(timer_error(
                &TIMER_ERROR_INVALID_PROPERTY,
                format!("timer: property '{name}' cannot be changed while Running is on"),
            ));
        }
        object.properties.insert(name, value);
        idx += 2;
    }
    Ok(())
}

async fn set_timer_property_object(
    obj: Value,
    property_name: &'static str,
    value: Value,
) -> BuiltinResult<Value> {
    let Value::Object(mut object) = obj else {
        return Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            format!("timer: set.{property_name} requires timer object receiver"),
        ));
    };
    if object.class_name != TIMER_CLASS {
        return Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            format!(
                "timer: set.{property_name} requires timer object receiver, got '{}'",
                object.class_name
            ),
        ));
    }
    apply_name_value_pairs(
        &mut object,
        &[Value::String(property_name.to_string()), value],
        false,
    )?;
    Ok(Value::Object(object))
}

fn canonical_property_name(name: &str) -> BuiltinResult<String> {
    let trimmed = name.trim();
    for prop in CALLBACK_PROPS
        .into_iter()
        .chain(STRING_PROPS)
        .chain(NUMERIC_PROPS)
        .chain(READONLY_PROPS)
        .chain(["UserData"])
    {
        if prop.eq_ignore_ascii_case(trimmed) {
            return Ok(prop.to_string());
        }
    }
    Err(timer_error(
        &TIMER_ERROR_INVALID_PROPERTY,
        format!("timer: unknown property '{trimmed}'"),
    ))
}

fn normalize_property_value(name: &str, value: Value) -> BuiltinResult<Value> {
    match name {
        "Period" => {
            let seconds = numeric_scalar(&value, name)?;
            if !seconds.is_finite() || seconds <= 0.001 {
                return Err(timer_error(
                    &TIMER_ERROR_INVALID_PROPERTY,
                    "timer: Period must be a finite scalar greater than 0.001",
                ));
            }
            Ok(Value::Num(seconds))
        }
        "StartDelay" => {
            let seconds = numeric_scalar(&value, name)?;
            if !seconds.is_finite() || seconds < 0.0 {
                return Err(timer_error(
                    &TIMER_ERROR_INVALID_PROPERTY,
                    "timer: StartDelay must be a finite non-negative scalar",
                ));
            }
            Ok(Value::Num(seconds))
        }
        "TasksToExecute" => {
            let count = numeric_scalar(&value, name)?;
            if !count.is_finite() || count < 1.0 || count.fract() != 0.0 {
                return Err(timer_error(
                    &TIMER_ERROR_INVALID_PROPERTY,
                    "timer: TasksToExecute must be a positive integer scalar",
                ));
            }
            Ok(Value::Num(count))
        }
        "BusyMode" => match_string_choice(&value, name, &["drop", "error", "queue"]),
        "ExecutionMode" => match_string_choice(
            &value,
            name,
            &["singleShot", "fixedRate", "fixedDelay", "fixedSpacing"],
        ),
        "ObjectVisibility" => match_string_choice(&value, name, &["on", "off"]),
        "Name" | "Tag" => Ok(Value::String(value_to_string(&value)?)),
        "TimerFcn" | "StartFcn" | "StopFcn" | "ErrorFcn" => normalize_callback(value, name),
        "UserData" => Ok(value),
        _ => Err(timer_error(
            &TIMER_ERROR_INVALID_PROPERTY,
            format!("timer: property '{name}' cannot be assigned"),
        )),
    }
}

fn is_running_readonly_property(name: &str) -> bool {
    matches!(name, "BusyMode" | "ExecutionMode" | "StartDelay")
}

fn normalize_callback(value: Value, name: &str) -> BuiltinResult<Value> {
    match value {
        Value::String(_)
        | Value::CharArray(_)
        | Value::FunctionHandle(_)
        | Value::ExternalFunctionHandle(_)
        | Value::MethodFunctionHandle(_)
        | Value::BoundFunctionHandle { .. }
        | Value::Closure(_)
        | Value::Cell(_) => Ok(value),
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_PROPERTY,
            format!("timer: {name} must be text, a function handle, or a callback cell array, got {other:?}"),
        )),
    }
}

fn match_string_choice(value: &Value, name: &str, choices: &[&str]) -> BuiltinResult<Value> {
    let text = value_to_string(value)?;
    choices
        .iter()
        .find(|choice| choice.eq_ignore_ascii_case(text.trim()))
        .map(|choice| Value::String((*choice).to_string()))
        .ok_or_else(|| {
            timer_error(
                &TIMER_ERROR_INVALID_PROPERTY,
                format!("timer: invalid {name} value '{text}'"),
            )
        })
}

fn value_to_string(value: &Value) -> BuiltinResult<String> {
    match value {
        Value::String(text) => Ok(text.clone()),
        Value::CharArray(chars) if chars.rows == 1 => Ok(chars.data.iter().collect()),
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_INPUT,
            format!("timer: expected string scalar or character row, got {other:?}"),
        )),
    }
}

fn numeric_scalar(value: &Value, name: &str) -> BuiltinResult<f64> {
    match value {
        Value::Num(value) => Ok(*value),
        Value::Int(value) => Ok(value.to_f64()),
        Value::Tensor(tensor) if tensor.data.len() == 1 => Ok(tensor.data[0]),
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_INPUT,
            format!("timer: {name} must be a numeric scalar, got {other:?}"),
        )),
    }
}

async fn start_timer_values(value: &Value, override_delay: Option<f64>) -> BuiltinResult<()> {
    match value {
        Value::HandleObject(handle) if is_timer_handle(handle) => {
            start_one_timer(handle.clone(), override_delay).await
        }
        Value::Cell(cell) => {
            for item in &cell.data {
                Box::pin(start_timer_values(item, override_delay)).await?;
            }
            Ok(())
        }
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            format!("timer: expected timer handle, got {other:?}"),
        )),
    }
}

async fn stop_timer_values(value: &Value, call_stop: bool) -> BuiltinResult<()> {
    match value {
        Value::HandleObject(handle) if is_timer_handle(handle) => {
            stop_one_timer(handle, call_stop).await
        }
        Value::Cell(cell) => {
            for item in &cell.data {
                Box::pin(stop_timer_values(item, call_stop)).await?;
            }
            Ok(())
        }
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            format!("timer: expected timer handle, got {other:?}"),
        )),
    }
}

async fn wait_timer_values(value: &Value) -> BuiltinResult<()> {
    match value {
        Value::HandleObject(handle) if is_timer_handle(handle) => {
            if running_property(handle)? {
                stop_one_timer(handle, true).await?;
            }
            Ok(())
        }
        Value::Cell(cell) => {
            for item in &cell.data {
                Box::pin(wait_timer_values(item)).await?;
            }
            Ok(())
        }
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            format!("timer: expected timer handle, got {other:?}"),
        )),
    }
}

async fn delete_timer_values(value: &Value) -> BuiltinResult<()> {
    match value {
        Value::HandleObject(handle) if is_timer_handle(handle) => {
            stop_one_timer(handle, false).await?;
            crate::set_handle_valid(handle, false);
            TIMER_REGISTRY.with(|registry| registry.borrow_mut().retain(|h| h != handle));
            let _ = runmat_gc::gc_remove_root(handle.target);
            Ok(())
        }
        Value::Cell(cell) => {
            for item in &cell.data {
                Box::pin(delete_timer_values(item)).await?;
            }
            Ok(())
        }
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            format!("timer: expected timer handle, got {other:?}"),
        )),
    }
}

async fn start_one_timer(handle: HandleRef, override_delay: Option<f64>) -> BuiltinResult<()> {
    ensure_valid_timer(&handle)?;
    let timer_fcn = property(&handle, "TimerFcn")?;
    if callback_is_empty(&timer_fcn) {
        return Err(timer_error(
            &TIMER_ERROR_INVALID_PROPERTY,
            "timer: TimerFcn must be set before starting a timer",
        ));
    }
    set_property(&handle, "Running", Value::String("on".to_string()))?;
    set_property(&handle, "TasksExecuted", Value::Num(0.0))?;
    let start_delay = override_delay.unwrap_or(numeric_property(&handle, "StartDelay")?);
    sleep_seconds(start_delay);
    if let Err(err) = run_callback(&handle, "StartFcn", "StartFcn").await {
        let _ = stop_one_timer(&handle, false).await;
        return Err(err);
    }

    let execution_mode = string_property(&handle, "ExecutionMode")?;
    let tasks = if execution_mode.eq_ignore_ascii_case("singleShot") {
        1usize
    } else {
        numeric_property(&handle, "TasksToExecute")?.max(1.0) as usize
    };
    let period = numeric_property(&handle, "Period")?;
    let mut last_fire: Option<runmat_time::Instant> = None;
    for idx in 0..tasks {
        if idx > 0 {
            sleep_seconds(period);
        }
        let now = runmat_time::Instant::now();
        if let Some(last) = last_fire {
            let instant_period = now.duration_since(last).as_secs_f64();
            set_property(&handle, "InstantPeriod", Value::Num(instant_period))?;
            update_average_period(&handle, instant_period, idx)?;
        }
        last_fire = Some(now);
        match run_callback(&handle, "TimerFcn", "TimerFcn").await {
            Ok(()) => {
                let executed = numeric_property(&handle, "TasksExecuted")? + 1.0;
                set_property(&handle, "TasksExecuted", Value::Num(executed))?;
            }
            Err(err) => {
                let _ = run_callback(&handle, "ErrorFcn", "ErrorFcn").await;
                let _ = stop_one_timer(&handle, true).await;
                return Err(timer_error(&TIMER_ERROR_CALLBACK, err.message()));
            }
        }
    }
    stop_one_timer(&handle, true).await
}

async fn stop_one_timer(handle: &HandleRef, call_stop: bool) -> BuiltinResult<()> {
    ensure_valid_timer(handle)?;
    let was_running = running_property(handle)?;
    set_property(handle, "Running", Value::String("off".to_string()))?;
    if call_stop && was_running {
        run_callback(handle, "StopFcn", "StopFcn").await?;
    }
    Ok(())
}

fn update_average_period(handle: &HandleRef, instant_period: f64, idx: usize) -> BuiltinResult<()> {
    let current = numeric_property(handle, "AveragePeriod").unwrap_or(f64::NAN);
    let average = if current.is_finite() {
        (current * (idx as f64 - 1.0) + instant_period) / idx as f64
    } else {
        instant_period
    };
    set_property(handle, "AveragePeriod", Value::Num(average))
}

async fn run_callback(
    handle: &HandleRef,
    property_name: &str,
    event_type: &str,
) -> BuiltinResult<()> {
    let callback = property(handle, property_name)?;
    if callback_is_empty(&callback) {
        return Ok(());
    }
    match callback {
        Value::String(text) if !text.trim().is_empty() => {
            return Err(timer_error(
                &TIMER_ERROR_CALLBACK,
                "timer: text callback execution requires dynamic-eval callback infrastructure; use a function handle callback",
            ));
        }
        Value::CharArray(chars) if !chars.data.is_empty() => {
            let text: String = chars.data.iter().collect();
            if !text.trim().is_empty() {
                return Err(timer_error(
                    &TIMER_ERROR_CALLBACK,
                    "timer: text callback execution requires dynamic-eval callback infrastructure; use a function handle callback",
                ));
            }
        }
        Value::Cell(cell) => {
            let Some(function) = cell.data.first().cloned() else {
                return Ok(());
            };
            let mut args = callback_base_args(handle, event_type);
            args.extend(cell.data.iter().skip(1).cloned());
            crate::call_feval_async_with_outputs(function, &args, 0).await?;
        }
        function @ (Value::FunctionHandle(_)
        | Value::ExternalFunctionHandle(_)
        | Value::MethodFunctionHandle(_)
        | Value::BoundFunctionHandle { .. }
        | Value::Closure(_)) => {
            let args = callback_base_args(handle, event_type);
            crate::call_feval_async_with_outputs(function, &args, 0).await?;
        }
        _ => {}
    }
    Ok(())
}

fn callback_base_args(handle: &HandleRef, event_type: &str) -> Vec<Value> {
    vec![Value::HandleObject(handle.clone()), timer_event(event_type)]
}

fn timer_event(event_type: &str) -> Value {
    let mut data = StructValue::new();
    data.insert(
        "time",
        Value::Num(runmat_time::duration_since_epoch().as_secs_f64()),
    );
    let mut event = StructValue::new();
    event.insert("Type", Value::String(event_type.to_string()));
    event.insert("Data", Value::Struct(data));
    Value::Struct(event)
}

fn callback_is_empty(value: &Value) -> bool {
    match value {
        Value::String(text) => text.trim().is_empty(),
        Value::CharArray(chars) => chars.data.iter().collect::<String>().trim().is_empty(),
        Value::Cell(cell) => cell.data.is_empty(),
        _ => false,
    }
}

fn sleep_seconds(seconds: f64) {
    if seconds <= 0.0 || !seconds.is_finite() {
        return;
    }
    #[cfg(not(target_arch = "wasm32"))]
    {
        std::thread::sleep(Duration::from_secs_f64(seconds));
    }
    #[cfg(target_arch = "wasm32")]
    {
        let _ = seconds;
    }
}

fn find_timers(args: Vec<Value>, include_invisible: bool) -> BuiltinResult<Value> {
    ensure_timer_class_registered();
    let filters = parse_filters(&args)?;
    let mut matches = Vec::new();
    TIMER_REGISTRY.with(|registry| {
        let mut registry = registry.borrow_mut();
        registry.retain(crate::is_handle_valid);
        for handle in registry.iter() {
            if !include_invisible
                && !string_property(handle, "ObjectVisibility")
                    .map(|value| value.eq_ignore_ascii_case("on"))
                    .unwrap_or(false)
            {
                continue;
            }
            if filters_match(handle, &filters).unwrap_or(false) {
                matches.push(Value::HandleObject(handle.clone()));
            }
        }
    });
    handles_to_result(matches)
}

fn parse_filters(args: &[Value]) -> BuiltinResult<Vec<(String, Value)>> {
    if !args.len().is_multiple_of(2) {
        return Err(timer_error(
            &TIMER_ERROR_INVALID_INPUT,
            "timerfind: name-value arguments must appear in pairs",
        ));
    }
    let mut out = Vec::new();
    let mut idx = 0usize;
    while idx < args.len() {
        let name = canonical_property_name(&value_to_string(&args[idx])?)?;
        out.push((name, normalize_find_value(args[idx + 1].clone())));
        idx += 2;
    }
    Ok(out)
}

fn normalize_find_value(value: Value) -> Value {
    match value {
        Value::CharArray(chars) if chars.rows == 1 => Value::String(chars.data.iter().collect()),
        other => other,
    }
}

fn filters_match(handle: &HandleRef, filters: &[(String, Value)]) -> BuiltinResult<bool> {
    for (name, expected) in filters {
        let actual = normalize_find_value(property(handle, name)?);
        if !timer_values_equal(&actual, expected) {
            return Ok(false);
        }
    }
    Ok(true)
}

fn timer_values_equal(lhs: &Value, rhs: &Value) -> bool {
    match (lhs, rhs) {
        (Value::String(a), Value::String(b)) => a == b,
        (Value::Num(a), Value::Num(b)) if a.is_nan() && b.is_nan() => true,
        (Value::Num(a), Value::Num(b)) => a == b,
        (Value::Bool(a), Value::Bool(b)) => a == b,
        _ => lhs == rhs,
    }
}

fn handles_to_result(handles: Vec<Value>) -> BuiltinResult<Value> {
    if handles.len() == 1 {
        Ok(handles.into_iter().next().unwrap())
    } else {
        let len = handles.len();
        CellArray::new(handles, 1, len)
            .map(Value::Cell)
            .map_err(|err| timer_error(&TIMER_ERROR_INVALID_INPUT, err))
    }
}

fn property(handle: &HandleRef, name: &str) -> BuiltinResult<Value> {
    ensure_valid_timer(handle)?;
    runmat_gc::gc_with_value(&handle.target, |target| {
        let Value::Object(object) = target else {
            return None;
        };
        object.properties.get(name).cloned()
    })
    .map_err(|err| timer_error(&TIMER_ERROR_GC, format!("timer: {err}")))?
    .ok_or_else(|| {
        timer_error(
            &TIMER_ERROR_INVALID_PROPERTY,
            format!("timer: missing property '{name}'"),
        )
    })
}

fn set_property(handle: &HandleRef, name: &str, value: Value) -> BuiltinResult<()> {
    ensure_valid_timer(handle)?;
    let updated = runmat_gc::gc_with_value_mut(&handle.target, |target| {
        let Value::Object(object) = target else {
            return false;
        };
        object.properties.insert(name.to_string(), value);
        true
    })
    .map_err(|err| timer_error(&TIMER_ERROR_GC, format!("timer: {err}")))?;
    if updated {
        Ok(())
    } else {
        Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            "timer: handle target is not an object",
        ))
    }
}

fn string_property(handle: &HandleRef, name: &str) -> BuiltinResult<String> {
    match property(handle, name)? {
        Value::String(text) => Ok(text),
        Value::CharArray(chars) if chars.rows == 1 => Ok(chars.data.iter().collect()),
        other => Err(timer_error(
            &TIMER_ERROR_INVALID_PROPERTY,
            format!("timer: property '{name}' must be text, got {other:?}"),
        )),
    }
}

fn numeric_property(handle: &HandleRef, name: &str) -> BuiltinResult<f64> {
    numeric_scalar(&property(handle, name)?, name)
}

fn running_property(handle: &HandleRef) -> BuiltinResult<bool> {
    Ok(string_property(handle, "Running")?.eq_ignore_ascii_case("on"))
}

fn running_is_on(object: &ObjectInstance) -> bool {
    matches!(
        object.properties.get("Running"),
        Some(Value::String(text)) if text.eq_ignore_ascii_case("on")
    )
}

fn is_timer_handle(handle: &HandleRef) -> bool {
    handle.class_name == TIMER_CLASS
}

fn ensure_valid_timer(handle: &HandleRef) -> BuiltinResult<()> {
    if is_timer_handle(handle) && crate::is_handle_valid(handle) {
        Ok(())
    } else {
        Err(timer_error(
            &TIMER_ERROR_INVALID_HANDLE,
            "timer: invalid or deleted timer handle",
        ))
    }
}

fn timer_error(error: &'static BuiltinErrorDescriptor, detail: impl AsRef<str>) -> RuntimeError {
    let detail = detail.as_ref();
    let message = if detail.is_empty() {
        error.message.to_string()
    } else {
        detail.to_string()
    };
    let mut builder = build_runtime_error(message).with_builtin(BUILTIN_TIMER);
    if let Some(identifier) = error.identifier {
        builder = builder.with_identifier(identifier);
    }
    builder.build()
}

#[cfg(test)]
pub(crate) fn reset_timer_state_for_tests() {
    TIMER_REGISTRY.with(|registry| {
        for handle in registry.borrow().iter() {
            let _ = runmat_gc::gc_remove_root(handle.target);
        }
        registry.borrow_mut().clear();
    });
    TIMER_COUNTER.with(|counter| counter.set(0));
}

#[cfg(test)]
mod tests {
    use super::*;
    use futures::executor::block_on;
    use runmat_builtins::CharArray;
    use std::sync::{Arc, Mutex};

    #[test]
    fn timer_constructor_sets_defaults_and_name_values() {
        let _lock = TIMER_TEST_LOCK.lock().unwrap();
        reset_timer_state_for_tests();
        let value = block_on(timer_builtin(vec![
            Value::String("Name".into()),
            Value::String("solver".into()),
            Value::String("Tag".into()),
            Value::String("client".into()),
            Value::String("StartDelay".into()),
            Value::Num(0.25),
        ]))
        .expect("timer");
        let Value::HandleObject(handle) = value else {
            panic!("expected timer handle");
        };
        assert_eq!(string_property(&handle, "Name").unwrap(), "solver");
        assert_eq!(string_property(&handle, "Tag").unwrap(), "client");
        assert_eq!(numeric_property(&handle, "StartDelay").unwrap(), 0.25);
        assert_eq!(string_property(&handle, "Running").unwrap(), "off");
        assert!(crate::is_handle_valid(&handle));
    }

    #[test]
    fn timerfind_filters_visible_timers() {
        let _lock = TIMER_TEST_LOCK.lock().unwrap();
        reset_timer_state_for_tests();
        let _hidden = block_on(timer_builtin(vec![
            Value::String("Name".into()),
            Value::String("hidden".into()),
            Value::String("ObjectVisibility".into()),
            Value::String("off".into()),
        ]))
        .expect("hidden timer");
        let visible = block_on(timer_builtin(vec![
            Value::String("Name".into()),
            Value::String("visible".into()),
            Value::String("Tag".into()),
            Value::String("batch".into()),
        ]))
        .expect("visible timer");
        let found = timerfind_builtin(vec![
            Value::String("Tag".into()),
            Value::String("batch".into()),
        ])
        .expect("timerfind");
        assert_eq!(found, visible);
        let all = timerfindall_builtin(vec![]).expect("timerfindall");
        let Value::Cell(cell) = all else {
            panic!("expected cell row of all timers");
        };
        assert_eq!((cell.rows, cell.cols), (1, 2));
    }

    #[test]
    fn timer_start_runs_function_callback_and_updates_state() {
        let _lock = TIMER_TEST_LOCK.lock().unwrap();
        reset_timer_state_for_tests();
        let calls = Arc::new(Mutex::new(0usize));
        let invoker_calls = calls.clone();
        let _guard = crate::user_functions::install_semantic_function_invoker(Some(Arc::new(
            move |_function, args, requested_outputs| {
                assert_eq!(requested_outputs, 0);
                let args = args.to_vec();
                let invoker_calls = Arc::clone(&invoker_calls);
                Box::pin(async move {
                    assert_eq!(args.len(), 2);
                    let Value::HandleObject(handle) = &args[0] else {
                        panic!("expected timer handle");
                    };
                    assert!(is_timer_handle(handle));
                    let Value::Struct(event) = &args[1] else {
                        panic!("expected event struct");
                    };
                    assert!(event.fields.contains_key("Type"));
                    *invoker_calls.lock().unwrap() += 1;
                    Ok(Value::OutputList(Vec::new()))
                })
            },
        )));
        let timer = block_on(timer_builtin(vec![
            Value::String("TimerFcn".into()),
            Value::BoundFunctionHandle {
                name: "tick".into(),
                function: 1,
            },
            Value::String("ExecutionMode".into()),
            Value::String("fixedSpacing".into()),
            Value::String("TasksToExecute".into()),
            Value::Num(2.0),
            Value::String("Period".into()),
            Value::Num(0.002),
        ]))
        .expect("timer");
        block_on(timer_start_builtin(timer.clone())).expect("start");
        let Value::HandleObject(handle) = timer else {
            panic!("expected timer handle");
        };
        assert_eq!(*calls.lock().unwrap(), 2);
        assert_eq!(numeric_property(&handle, "TasksExecuted").unwrap(), 2.0);
        assert_eq!(string_property(&handle, "Running").unwrap(), "off");
    }

    #[test]
    fn timer_delete_invalidates_and_removes_from_find_results() {
        let _lock = TIMER_TEST_LOCK.lock().unwrap();
        reset_timer_state_for_tests();
        let timer = block_on(timer_builtin(vec![
            Value::String("TimerFcn".into()),
            Value::String("disp(1)".into()),
        ]))
        .expect("timer");
        let Value::HandleObject(handle) = timer.clone() else {
            panic!("expected timer handle");
        };
        block_on(timer_delete_builtin(timer)).expect("delete");
        assert!(!crate::is_handle_valid(&handle));
        let found = timerfindall_builtin(vec![]).expect("timerfindall");
        let Value::Cell(cell) = found else {
            panic!("expected empty cell");
        };
        assert_eq!((cell.rows, cell.cols), (1, 0));
    }

    #[test]
    fn timer_rejects_invalid_property_values() {
        let _lock = TIMER_TEST_LOCK.lock().unwrap();
        reset_timer_state_for_tests();
        let err = block_on(timer_builtin(vec![
            Value::CharArray(CharArray::new_row("Period")),
            Value::Num(0.0),
        ]))
        .unwrap_err();
        assert_eq!(
            err.identifier().map(str::to_string),
            Some("RunMat:timer:InvalidProperty".to_string())
        );
    }

    #[test]
    fn timer_dependent_setters_validate_post_construction_values() {
        let _lock = TIMER_TEST_LOCK.lock().unwrap();
        reset_timer_state_for_tests();
        let timer = block_on(timer_builtin(vec![])).expect("timer");
        let Value::HandleObject(handle) = timer else {
            panic!("expected timer handle");
        };
        let object = runmat_gc::gc_clone_value(&handle.target).expect("timer object");
        let err = block_on(set_timer_property_object(
            object.clone(),
            "Period",
            Value::Num(0.0),
        ))
        .unwrap_err();
        assert_eq!(
            err.identifier().map(str::to_string),
            Some("RunMat:timer:InvalidProperty".to_string())
        );

        let updated = block_on(set_timer_property_object(
            object,
            "Tag",
            Value::CharArray(CharArray::new_row("batch")),
        ))
        .expect("valid tag update");
        let Value::Object(updated) = updated else {
            panic!("expected updated timer object");
        };
        assert_eq!(
            updated.properties.get("Tag"),
            Some(&Value::String("batch".to_string()))
        );
    }
}