formualizer_eval/

traits.rs

use crate::engine::lookup_index_cache::{LookupAxis, LookupIndex};
use crate::engine::range_view::RangeView;
use crate::engine::row_visibility::VisibilityMaskMode;
pub use crate::function::Function;
use crate::interpreter::Interpreter;
use crate::reference::CellRef;
use formualizer_common::{
    LiteralValue,
    error::{ExcelError, ExcelErrorKind},
};
use std::any::Any;
use std::borrow::Cow;
use std::fmt::Debug;
use std::sync::Arc;

use formualizer_parse::parser::{ASTNode, ASTNodeType, ReferenceType, TableSpecifier};

/// Actual provider object behind a potential function-as-value reference.
/// Unknown name-binding metadata remains explicit; no callback is invoked.
#[derive(Clone)]
pub(crate) struct FunctionValueReferenceIntent {
    pub function: Arc<dyn Function>,
    pub binding_absent: bool,
}

#[derive(Clone, Debug, Eq, PartialEq)]
pub struct ReferenceInfo {
    /// Excel-style 1-based index of the first sheet covered by the reference.
    pub first_sheet_index: Option<usize>,
    /// Number of sheets covered by the reference (`1` for ordinary references, `N` for 3D refs).
    pub sheet_count: Option<usize>,
    /// Top-left / first cell addressed by the reference, when it resolves to a concrete cell.
    pub first_cell: Option<CellRef>,
}

/* ───────────────────────────── Range ───────────────────────────── */

pub trait Range: Debug + Send + Sync {
    fn get(&self, row: usize, col: usize) -> Result<LiteralValue, ExcelError>;
    fn dimensions(&self) -> (usize, usize);

    fn is_sparse(&self) -> bool {
        false
    }

    // Handle infinite ranges (A:A, 1:1)
    fn is_infinite(&self) -> bool {
        false
    }

    fn materialise(&self) -> Cow<'_, [Vec<LiteralValue>]> {
        Cow::Owned(
            (0..self.dimensions().0)
                .map(|r| {
                    (0..self.dimensions().1)
                        .map(|c| self.get(r, c).unwrap_or(LiteralValue::Empty))
                        .collect()
                })
                .collect(),
        )
    }

    fn iter_cells<'a>(&'a self) -> Box<dyn Iterator<Item = LiteralValue> + 'a> {
        let (rows, cols) = self.dimensions();
        Box::new((0..rows).flat_map(move |r| (0..cols).map(move |c| self.get(r, c).unwrap())))
    }
    fn iter_rows<'a>(&'a self) -> Box<dyn Iterator<Item = Vec<LiteralValue>> + 'a> {
        let (rows, cols) = self.dimensions();
        Box::new((0..rows).map(move |r| (0..cols).map(|c| self.get(r, c).unwrap()).collect()))
    }

    /* down-cast hook for SIMD back-ends */
    fn as_any(&self) -> &dyn Any;
}

/* blanket dyn passthrough */
impl Range for Box<dyn Range> {
    fn get(&self, r: usize, c: usize) -> Result<LiteralValue, ExcelError> {
        (**self).get(r, c)
    }
    fn dimensions(&self) -> (usize, usize) {
        (**self).dimensions()
    }
    fn is_sparse(&self) -> bool {
        (**self).is_sparse()
    }
    fn materialise(&self) -> Cow<'_, [Vec<LiteralValue>]> {
        (**self).materialise()
    }
    fn iter_cells<'a>(&'a self) -> Box<dyn Iterator<Item = LiteralValue> + 'a> {
        (**self).iter_cells()
    }
    fn iter_rows<'a>(&'a self) -> Box<dyn Iterator<Item = Vec<LiteralValue>> + 'a> {
        (**self).iter_rows()
    }
    fn as_any(&self) -> &dyn Any {
        (**self).as_any()
    }
}

/* ────────────────────── ArgumentHandle helpers ───────────────────── */

pub type CowValue<'a> = Cow<'a, LiteralValue>;

pub trait CustomCallable: Send + Sync {
    fn arity(&self) -> usize;

    fn invoke<'ctx>(
        &self,
        interp: &Interpreter<'ctx>,
        args: &[LiteralValue],
    ) -> Result<CalcValue<'ctx>, ExcelError>;

    /// Invoke with each argument's optional bound reference alongside the
    /// already-materialized values.
    ///
    /// `references[i]` is the spreadsheet reference argument `i` was written
    /// as, when it was written as one; `None` otherwise, and a short slice is
    /// read as `None` for every missing position. A callable that binds its
    /// arguments to locals uses this to bind a range-valued parameter as
    /// [`crate::interpreter::LocalBinding::ValueWithReference`], so a by-ref
    /// argument slot inside the body sees the range rather than a lifted
    /// array.
    ///
    /// Defaults to [`Self::invoke`], so an implementor that does not care
    /// about references needs no change.
    fn invoke_with_references<'ctx>(
        &self,
        interp: &Interpreter<'ctx>,
        args: &[LiteralValue],
        references: &[Option<ReferenceType>],
    ) -> Result<CalcValue<'ctx>, ExcelError> {
        let _ = references;
        self.invoke(interp, args)
    }
}

#[derive(Clone)]
pub enum CalcValue<'a> {
    Scalar(LiteralValue),
    /// Scalar carrying an eval-internal number-format annotation.
    AnnotatedScalar(LiteralValue, crate::format::FormatId),
    Range(RangeView<'a>),
    Callable(Arc<dyn CustomCallable>),
}

/// The result of resolving an argument where either a reference or a value is accepted.
///
/// Reference-shaped syntax is resolved without first evaluating it as a value.
/// All other syntax is evaluated through [`ArgumentHandle::value`] and retains
/// its `CalcValue` discriminant.
#[derive(Clone)]
pub(crate) enum ResolvedArgument<'a> {
    Range(RangeView<'a>),
    ReferenceError(ExcelError),
    Value(CalcValue<'a>),
}

impl std::fmt::Debug for ResolvedArgument<'_> {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            Self::Range(view) => f.debug_tuple("Range").field(view).finish(),
            Self::ReferenceError(error) => f.debug_tuple("ReferenceError").field(error).finish(),
            Self::Value(value) => f.debug_tuple("Value").field(value).finish(),
        }
    }
}

impl<'a> std::fmt::Debug for CalcValue<'a> {
    fn fmt(&self, f: &mut std::fmt::Formatter<'_>) -> std::fmt::Result {
        match self {
            CalcValue::Scalar(v) => f.debug_tuple("Scalar").field(v).finish(),
            CalcValue::AnnotatedScalar(v, format) => f
                .debug_tuple("AnnotatedScalar")
                .field(v)
                .field(format)
                .finish(),
            CalcValue::Range(rv) => {
                let (r, c) = rv.dims();
                f.debug_tuple("Range").field(&(r, c)).finish()
            }
            CalcValue::Callable(_) => f.write_str("Callable(<opaque>)"),
        }
    }
}

// Thread-local instrumentation keeps serial publication regressions deterministic
// without counting materialization performed by unrelated parallel tests.
#[cfg(test)]
thread_local! {
    pub(crate) static RANGE_MATERIALIZED_CELLS: std::cell::Cell<usize> = const { std::cell::Cell::new(0) };
}

impl<'a> CalcValue<'a> {
    pub fn into_literal(self) -> LiteralValue {
        match self {
            CalcValue::Scalar(s) | CalcValue::AnnotatedScalar(s, _) => s,
            CalcValue::Range(rv) => {
                let (rows, cols) = rv.dims();
                #[cfg(test)]
                RANGE_MATERIALIZED_CELLS.with(|count| count.set(count.get() + rows * cols));
                if rows == 1 && cols == 1 {
                    rv.get_cell(0, 0)
                } else if let Some(error) = rv.materialization_error() {
                    LiteralValue::Error(error)
                } else {
                    let mut data = Vec::with_capacity(rows);
                    for row_idx in 0..rows {
                        let mut row = Vec::with_capacity(cols);
                        for col_idx in 0..cols {
                            row.push(rv.get_cell(row_idx, col_idx));
                        }
                        data.push(row);
                    }
                    LiteralValue::Array(data)
                }
            }
            CalcValue::Callable(_) => LiteralValue::Error(
                ExcelError::new(ExcelErrorKind::Calc).with_message("LAMBDA value must be invoked"),
            ),
        }
    }

    pub fn as_scalar(&self) -> Option<&LiteralValue> {
        match self {
            CalcValue::Scalar(s) | CalcValue::AnnotatedScalar(s, _) => Some(s),
            _ => None,
        }
    }

    pub fn format_id(&self) -> Option<crate::format::FormatId> {
        match self {
            CalcValue::AnnotatedScalar(_, format) => Some(*format),
            _ => None,
        }
    }

    pub fn with_format(self, format: Option<crate::format::FormatId>) -> Self {
        let format = format.filter(|id| *id != crate::format::FormatId::GENERAL);
        match (self, format) {
            (CalcValue::Scalar(value) | CalcValue::AnnotatedScalar(value, _), Some(id)) => {
                CalcValue::AnnotatedScalar(value, id)
            }
            (CalcValue::AnnotatedScalar(value, _), None) => CalcValue::Scalar(value),
            (other, _) => other,
        }
    }

    pub fn into_scalar_parts(self) -> (LiteralValue, Option<crate::format::FormatId>) {
        match self {
            CalcValue::Scalar(value) => (value, None),
            CalcValue::AnnotatedScalar(value, format) => (value, Some(format)),
            other => (other.into_literal(), None),
        }
    }

    pub fn as_range(&self) -> Option<&RangeView<'a>> {
        match self {
            CalcValue::Range(r) => Some(r),
            _ => None,
        }
    }

    pub fn as_callable(&self) -> Option<&Arc<dyn CustomCallable>> {
        match self {
            CalcValue::Callable(c) => Some(c),
            _ => None,
        }
    }

    pub fn into_owned(self) -> LiteralValue {
        self.into_literal()
    }
}

impl From<CalcValue<'_>> for LiteralValue {
    fn from(val: CalcValue<'_>) -> Self {
        val.into_literal()
    }
}

impl<'a> PartialEq<LiteralValue> for CalcValue<'a> {
    fn eq(&self, other: &LiteralValue) -> bool {
        match self {
            CalcValue::Scalar(s) | CalcValue::AnnotatedScalar(s, _) => s == other,
            CalcValue::Range(rv) => match other {
                LiteralValue::Array(arr) => {
                    let (rows, cols) = rv.dims();
                    if arr.len() != rows {
                        return false;
                    }
                    for (r, row) in arr.iter().enumerate() {
                        if row.len() != cols {
                            return false;
                        }
                        for (c, cell) in row.iter().enumerate() {
                            if &rv.get_cell(r, c) != cell {
                                return false;
                            }
                        }
                    }
                    true
                }
                _ => {
                    let (rows, cols) = rv.dims();
                    rows == 1 && cols == 1 && &rv.get_cell(0, 0) == other
                }
            },
            CalcValue::Callable(_) => false,
        }
    }
}

impl<'a> PartialEq<CalcValue<'a>> for LiteralValue {
    fn eq(&self, other: &CalcValue<'a>) -> bool {
        other == self
    }
}

pub enum EvaluatedArg<'a> {
    LiteralValue(CowValue<'a>),
    Range(Box<dyn Range>),
}

enum ArgumentExpr<'a> {
    Ast(&'a ASTNode),
    Arena {
        id: crate::engine::arena::AstNodeId,
        data_store: &'a crate::engine::arena::DataStore,
        sheet_registry: &'a crate::engine::sheet_registry::SheetRegistry,
    },
}

pub struct ArgumentHandle<'a, 'b> {
    expr: ArgumentExpr<'a>,
    interp: &'a Interpreter<'b>,
    caches: std::cell::OnceCell<Box<ArgumentCaches<'b>>>,
    /// Validation and builtin evaluation both read arguments. Memoization keeps
    /// nested calls linear instead of re-evaluating a leaf 2^depth times.
    /// Handles are per-call/per-evaluation; value_with_env deliberately bypasses
    /// this memo because its local environment may change.
    /// Scalar results/errors are hot; AST/reference/resolution caches stay cold.
    cached_value: std::cell::OnceCell<Result<MemoizedValue<'b>, ExcelError>>,
}

/// Keep the common scalar memo inline without paying for RangeView in every handle.
enum MemoizedValue<'b> {
    Scalar(LiteralValue, Option<crate::format::FormatId>),
    Other(Box<CalcValue<'b>>),
}

impl<'b> MemoizedValue<'b> {
    fn new(value: CalcValue<'b>) -> Self {
        match value {
            CalcValue::Scalar(value) => Self::Scalar(value, None),
            CalcValue::AnnotatedScalar(value, format) => Self::Scalar(value, Some(format)),
            other => Self::Other(Box::new(other)),
        }
    }

    fn value(&self) -> CalcValue<'b> {
        match self {
            Self::Scalar(value, None) => CalcValue::Scalar(value.clone()),
            Self::Scalar(value, Some(format)) => CalcValue::AnnotatedScalar(value.clone(), *format),
            Self::Other(value) => (**value).clone(),
        }
    }
}

#[derive(Default)]
struct ArgumentCaches<'b> {
    cached_ast: std::cell::OnceCell<ASTNode>,
    cached_ref: std::cell::OnceCell<ReferenceType>,
    cached_reference_or_value:
        std::cell::OnceCell<Result<crate::function::FunctionResolution<'b>, ExcelError>>,
    cached_resolved: std::cell::OnceCell<Result<ResolvedArgument<'b>, ExcelError>>,
}

impl<'a, 'b> ArgumentHandle<'a, 'b> {
    fn caches(&self) -> &ArgumentCaches<'b> {
        self.caches
            .get_or_init(|| Box::new(ArgumentCaches::default()))
    }

    pub(crate) fn new(node: &'a ASTNode, interp: &'a Interpreter<'b>) -> Self {
        Self {
            expr: ArgumentExpr::Ast(node),
            interp,
            caches: std::cell::OnceCell::new(),
            cached_value: std::cell::OnceCell::new(),
        }
    }

    pub(crate) fn new_arena(
        id: crate::engine::arena::AstNodeId,
        interp: &'a Interpreter<'b>,
        data_store: &'a crate::engine::arena::DataStore,
        sheet_registry: &'a crate::engine::sheet_registry::SheetRegistry,
    ) -> Self {
        Self {
            expr: ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            },
            interp,
            caches: std::cell::OnceCell::new(),
            cached_value: std::cell::OnceCell::new(),
        }
    }

    /// Workbook date system in force for the evaluation this argument belongs to.
    ///
    /// Lets value-collecting helpers resolve date literals to serials without
    /// threading a `DateSystem` (or the whole `FunctionContext`) through every
    /// call site.
    pub(crate) fn date_system(&self) -> crate::engine::DateSystem {
        self.interp.context.date_system()
    }

    /// Returns whether this handle represents an explicitly omitted argument slot.
    ///
    /// This is false for absent arguments, explicit empty text, and blank references.
    pub fn is_omitted(&self) -> bool {
        match &self.expr {
            ArgumentExpr::Ast(node) => matches!(node.node_type, ASTNodeType::Omitted),
            ArgumentExpr::Arena { id, data_store, .. } => matches!(
                data_store.get_node(*id),
                Some(crate::engine::arena::AstNodeData::Omitted)
            ),
        }
    }

    /// The reference this argument spells directly, found without evaluating
    /// anything: a cell or range reference, a defined name, or a LET/LAMBDA
    /// local bound to a range. `None` for a computed argument (a function
    /// call, an operator, a literal) and for a reference that fails to
    /// resolve.
    ///
    /// This is not [`Self::as_reference`]: its arena arm resolves through
    /// `reference_for_eval`, which evaluates a `Function` or `BinaryOp` node
    /// (such as `OFFSET(...)` or `A1:A3*1`), and its AST arm returns the
    /// written reference without the current family offset. Here the node kind
    /// is checked first, so nothing is evaluated, and a bare reference resolves
    /// the same way on the AST and arena paths.
    pub(crate) fn bare_reference(&self) -> Option<ReferenceType> {
        let is_reference = match &self.expr {
            ArgumentExpr::Ast(node) => matches!(node.node_type, ASTNodeType::Reference { .. }),
            ArgumentExpr::Arena { id, data_store, .. } => matches!(
                data_store.get_node(*id),
                Some(crate::engine::arena::AstNodeData::Reference { .. })
            ),
        };
        if !is_reference {
            return None;
        }
        self.as_reference_or_eval().ok()
    }

    /// Inspect an actual registered function reference in a higher-order value
    /// position. Ordinary argument evaluation and errors remain unchanged.
    pub(crate) fn function_value_reference_intent(&self) -> Option<FunctionValueReferenceIntent> {
        self.function_value_reference_intent_using(self.interp)
    }

    fn function_value_reference_intent_using(
        &self,
        interp: &crate::interpreter::Interpreter<'_>,
    ) -> Option<FunctionValueReferenceIntent> {
        let name = match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference {
                    reference: ReferenceType::NamedRange(name),
                    ..
                } => name.as_str(),
                _ => return None,
            },
            ArgumentExpr::Arena { id, data_store, .. } => {
                let crate::engine::arena::AstNodeData::Reference { ref_type, .. } =
                    data_store.get_node(*id)?
                else {
                    return None;
                };
                let crate::engine::arena::CompactRefType::NamedRange(name) =
                    data_store.compact_ref(*ref_type)
                else {
                    return None;
                };
                data_store.resolve_ast_string(*name)
            }
        };
        if interp.has_local_name(name) {
            return None;
        }
        let presence = interp
            .context
            .named_binding_presence(name, interp.current_sheet());
        if presence == Some(true) {
            return None;
        }
        let function = interp.ast_function_arc("", name)?;
        Some(FunctionValueReferenceIntent {
            function,
            binding_absent: presence == Some(false),
        })
    }

    /// Metadata query for an incrementally constructed lexical environment.
    pub(crate) fn function_value_reference_intent_in_env(
        &self,
        env: &crate::interpreter::LocalEnv,
    ) -> Option<FunctionValueReferenceIntent> {
        self.function_value_reference_intent_using(&self.interp.with_local_env(env.clone()))
    }

    /// The spill rectangle of the anchor this argument names, as `ANCHORARRAY`
    /// takes it: the argument must be written as a reference (a single cell,
    /// a name, or a LET/LAMBDA local bound to one), and is resolved through
    /// the same context hook as the `#` operator. Anything else is `#REF!`.
    pub(crate) fn spill_reference(&self) -> Result<ReferenceType, ExcelError> {
        match &self.expr {
            ArgumentExpr::Ast(node) => self.interp.ast_spill_reference(node),
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => self
                .interp
                .arena_spill_reference(*id, data_store, sheet_registry),
        }
    }

    /// Returns whether this argument resolves as a spreadsheet reference rather than a value.
    ///
    /// This uses the interpreter's reference-resolution path, so reference-returning functions
    /// are included only when they actually produce a reference. A computed array remains a value
    /// even though both it and a cell range are represented by [`CalcValue::Range`].
    pub(crate) fn has_reference_semantics(&self) -> bool {
        self.reference_attempt().is_some()
    }

    /// Return whether this argument's syntax may produce a spreadsheet reference.
    ///
    /// Unlike [`Self::has_reference_semantics`], this does not evaluate a
    /// reference-returning function to discover which arm it selects.
    pub(crate) fn may_return_reference(&self) -> bool {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => match reference {
                    // A local is a reference only when it was bound to one.
                    ReferenceType::NamedRange(name)
                        if self.interp.resolve_local_name(name).is_some() =>
                    {
                        self.interp.resolve_local_bound_reference(name).is_some()
                    }
                    _ => true,
                },
                ASTNodeType::BinaryOp { op, .. } => *op == formualizer_parse::BinaryOperator::Range,
                ASTNodeType::UnaryOp { op, .. } => *op == formualizer_parse::UnaryOperator::Spill,
                ASTNodeType::Function { name, .. } => self
                    .interp
                    .ast_function_capabilities(name)
                    .is_some_and(|caps| caps.contains(crate::function::FnCaps::RETURNS_REFERENCE)),
                _ => false,
            },
            ArgumentExpr::Arena { id, data_store, .. } => match data_store.get_node(*id) {
                Some(crate::engine::arena::AstNodeData::Reference { ref_type, .. }) => {
                    let ref_type = data_store.compact_ref(*ref_type);
                    match ref_type {
                        // Same rule as the AST branch above.
                        crate::engine::arena::CompactRefType::NamedRange(name_id)
                            if self
                                .interp
                                .resolve_local_name(data_store.resolve_ast_string(*name_id))
                                .is_some() =>
                        {
                            self.interp
                                .resolve_local_bound_reference(
                                    data_store.resolve_ast_string(*name_id),
                                )
                                .is_some()
                        }
                        _ => true,
                    }
                }
                Some(crate::engine::arena::AstNodeData::BinaryOp { operator, .. }) => {
                    *operator == formualizer_parse::BinaryOperator::Range
                }
                Some(crate::engine::arena::AstNodeData::UnaryOp { operator, .. }) => {
                    *operator == formualizer_parse::UnaryOperator::Spill
                }
                Some(crate::engine::arena::AstNodeData::Function { name_id, .. }) => self
                    .interp
                    .arena_function_capabilities(*name_id, data_store)
                    .is_some_and(|caps| caps.contains(crate::function::FnCaps::RETURNS_REFERENCE)),
                _ => false,
            },
        }
    }

    pub fn value(&self) -> Result<crate::traits::CalcValue<'b>, ExcelError> {
        let result = self
            .cached_value
            .get_or_init(|| self.compute_value().map(MemoizedValue::new))
            .as_ref()
            .map(MemoizedValue::value)
            .map_err(Clone::clone);
        if let Some(collector) = self.interp.diagnostics {
            collector.input(&result);
        }
        result
    }

    /// Inspect a spreadsheet error regardless of whether evaluation returned it
    /// as a value or as an error. Operation failures must remain failures.
    ///
    /// This boundary is opt-in: ordinary value consumers (including N/T) retain
    /// their existing propagation behavior.
    pub(crate) fn value_for_error_inspection(&self) -> Result<LiteralValue, ExcelError> {
        fn is_spreadsheet_error(error: &ExcelError) -> bool {
            matches!(
                error.kind,
                ExcelErrorKind::Null
                    | ExcelErrorKind::Ref
                    | ExcelErrorKind::Name
                    | ExcelErrorKind::Value
                    | ExcelErrorKind::Div
                    | ExcelErrorKind::Na
                    | ExcelErrorKind::Num
                    | ExcelErrorKind::Error
                    | ExcelErrorKind::NImpl
                    | ExcelErrorKind::Spill
                    | ExcelErrorKind::Calc
                    | ExcelErrorKind::Circ
            ) && matches!(
                error.extra(),
                formualizer_common::ExcelErrorExtra::None
                    | formualizer_common::ExcelErrorExtra::Spill { .. }
            )
        }

        let value = match self.value() {
            Ok(value) => value.into_literal(),
            Err(error) if is_spreadsheet_error(&error) => LiteralValue::Error(error),
            Err(error) => return Err(error),
        };
        if let LiteralValue::Error(error) = &value
            && !is_spreadsheet_error(error)
        {
            return Err(error.clone());
        }
        Ok(value)
    }

    /// Resolves a scalar that is about to be coerced to text.
    ///
    /// Omitted arguments materialize as numeric zero through `value()`, which is
    /// correct for Any/numeric consumers and aggregates. Text consumers must use
    /// this boundary so omission becomes empty text without changing explicit 0.
    pub(crate) fn value_for_text(&self) -> Result<crate::traits::CalcValue<'b>, ExcelError> {
        if self.is_omitted() {
            Ok(crate::traits::CalcValue::Scalar(LiteralValue::Text(
                String::new(),
            )))
        } else {
            self.value()
        }
    }

    pub(crate) fn resolve_once_for_text(&self) -> Result<ResolvedArgument<'b>, ExcelError> {
        if self.is_omitted() {
            Ok(ResolvedArgument::Value(crate::traits::CalcValue::Scalar(
                LiteralValue::Text(String::new()),
            )))
        } else {
            self.resolve_once()
        }
    }

    fn compute_value(&self) -> Result<crate::traits::CalcValue<'b>, ExcelError> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Literal(v) => Ok(crate::traits::CalcValue::Scalar(v.clone())),
                // With no schema-level text policy, Number(0) is the neutral Any-policy
                // materialization. Text consumers resolve through `value_for_text`.
                ASTNodeType::Omitted => {
                    Ok(crate::traits::CalcValue::Scalar(LiteralValue::Number(0.0)))
                }
                _ => self.interp.evaluate_ast_uncached(node),
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                if matches!(
                    data_store.get_node(*id),
                    Some(crate::engine::arena::AstNodeData::Omitted)
                ) {
                    Ok(crate::traits::CalcValue::Scalar(LiteralValue::Number(0.0)))
                } else {
                    self.interp
                        .evaluate_arena_ast_inner(*id, data_store, sheet_registry)
                }
            }
        }
    }

    pub fn value_with_env(
        &self,
        env: crate::interpreter::LocalEnv,
    ) -> Result<crate::traits::CalcValue<'b>, ExcelError> {
        let scoped = self.interp.with_local_env(env);
        let result = match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Literal(v) => Ok(crate::traits::CalcValue::Scalar(v.clone())),
                ASTNodeType::Omitted => {
                    Ok(crate::traits::CalcValue::Scalar(LiteralValue::Number(0.0)))
                }
                _ => scoped.evaluate_ast_uncached(node),
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                if matches!(
                    data_store.get_node(*id),
                    Some(crate::engine::arena::AstNodeData::Omitted)
                ) {
                    Ok(crate::traits::CalcValue::Scalar(LiteralValue::Number(0.0)))
                } else {
                    scoped.evaluate_arena_ast_inner(*id, data_store, sheet_registry)
                }
            }
        };
        if let Some(collector) = self.interp.diagnostics {
            collector.input(&result);
        }
        result
    }

    pub fn current_env(&self) -> crate::interpreter::LocalEnv {
        self.interp.local_env().clone()
    }

    /// The spreadsheet reference this binding expression names, evaluated under
    /// `env`, so a LET/LAMBDA local can keep the range it was bound to.
    ///
    /// Only syntactic reference expressions qualify: a computed value has no
    /// reference to preserve, and a reference-returning function is left to the
    /// ordinary by-ref path rather than evaluated a second time at bind time.
    pub(crate) fn bound_reference_in_env(
        &self,
        env: &crate::interpreter::LocalEnv,
    ) -> Option<ReferenceType> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => {
                    if let ReferenceType::NamedRange(name) = reference
                        && let Some(binding) = env.lookup(name)
                    {
                        // Rebinding a local: carry its reference forward, if any.
                        return match binding {
                            crate::interpreter::LocalBinding::ValueWithReference {
                                reference,
                                ..
                            } => Some(reference),
                            _ => None,
                        };
                    }
                    self.interp.reference_for_current_offset(reference).ok()
                }
                ASTNodeType::BinaryOp { op, .. }
                    if *op == formualizer_parse::BinaryOperator::Range =>
                {
                    self.interp
                        .with_local_env(env.clone())
                        .evaluate_ast_as_reference(node)
                        .ok()
                }
                ASTNodeType::UnaryOp { op, expr }
                    if *op == formualizer_parse::UnaryOperator::Spill =>
                {
                    self.interp
                        .with_local_env(env.clone())
                        .ast_spill_reference(expr)
                        .ok()
                }
                _ => None,
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => match data_store.get_node(*id) {
                Some(crate::engine::arena::AstNodeData::Reference { ref_type, .. }) => {
                    let ref_type = data_store.compact_ref(*ref_type);
                    if let crate::engine::arena::CompactRefType::NamedRange(name_id) = ref_type
                        && let Some(binding) = env.lookup(data_store.resolve_ast_string(*name_id))
                    {
                        return match binding {
                            crate::interpreter::LocalBinding::ValueWithReference {
                                reference,
                                ..
                            } => Some(reference),
                            _ => None,
                        };
                    }
                    let reference =
                        data_store.reconstruct_reference_type_for_eval(ref_type, sheet_registry);
                    self.interp.reference_for_current_offset(&reference).ok()
                }
                Some(crate::engine::arena::AstNodeData::BinaryOp { operator, .. })
                    if *operator == formualizer_parse::BinaryOperator::Range =>
                {
                    self.interp
                        .with_local_env(env.clone())
                        .evaluate_arena_ast_as_reference(*id, data_store, sheet_registry)
                        .ok()
                }
                Some(crate::engine::arena::AstNodeData::UnaryOp { operator, expr_id })
                    if *operator == formualizer_parse::UnaryOperator::Spill =>
                {
                    self.interp
                        .with_local_env(env.clone())
                        .arena_spill_reference(*expr_id, data_store, sheet_registry)
                        .ok()
                }
                _ => None,
            },
        }
    }

    pub fn inline_array_literal(&self) -> Result<Option<Vec<Vec<LiteralValue>>>, ExcelError> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Literal(LiteralValue::Array(arr)) => Ok(Some(arr.clone())),
                _ => Ok(None),
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                let node = data_store.get_node(*id).ok_or_else(|| {
                    ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                })?;
                match node {
                    crate::engine::arena::AstNodeData::Literal(vref) => {
                        match data_store.retrieve_value(*vref) {
                            LiteralValue::Array(arr) => Ok(Some(arr)),
                            _ => Ok(None),
                        }
                    }
                    _ => {
                        // preserve existing behavior: only a literal array (not a computed array)
                        // is treated as "inline array literal".
                        let _ = sheet_registry;
                        Ok(None)
                    }
                }
            }
        }
    }

    /// Resolve a name that a LET/LAMBDA local may shadow, on the by-ref path.
    ///
    /// `Some(Ok(reference))` when the local was bound to a reference expression.
    /// That includes a bare workbook defined name: a local bound to `K` carries
    /// `NamedRange("K")`, and the consumer resolves that workbook name exactly as
    /// it would resolve `K` written in place, whatever `K` is defined as.
    /// `Some(Err(#VALUE!))` when the local was bound to a computed value, so the
    /// consumer's own error path runs instead of the workbook-name route (which
    /// would report the local's spelling as an undefined name). A consumer that
    /// has a value fallback (MATCH) takes it; one that does not (OFFSET, ROW,
    /// COLUMN) answers `#VALUE!`, as Excel does. `None` when the name is not
    /// locally bound and the workbook-name route is correct.
    ///
    /// [`Self::reference_attempt`] and [`Self::may_return_reference`] apply the
    /// same rule, so a reference-returning selector (`IF`, `CHOOSE`) passes a
    /// range-bound local through as its range. A local bound to a computed value
    /// stays on the value path everywhere, which keeps `=LET(p,FALSE,OR(p))` a
    /// boolean.
    fn local_named_reference(&self, name: &str) -> Option<Result<ReferenceType, ExcelError>> {
        // `None` here sends the name down the ordinary workbook-name route.
        self.interp.resolve_local_name(name)?;
        Some(
            self.interp
                .resolve_local_bound_reference(name)
                .ok_or_else(|| {
                    ExcelError::new(ExcelErrorKind::Value)
                        .with_message("LET/LAMBDA local is not a reference")
                }),
        )
    }

    fn reference_for_eval(&self) -> Result<ReferenceType, ExcelError> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => {
                    // A LET/LAMBDA local shadows any workbook name of the same
                    // spelling, so it must never take the named-range route.
                    if let ReferenceType::NamedRange(name) = reference
                        && let Some(local) = self.local_named_reference(name)
                    {
                        return local;
                    }
                    self.interp.reference_for_current_offset(reference)
                }
                ASTNodeType::Function { .. } | ASTNodeType::BinaryOp { .. } => {
                    self.interp.evaluate_ast_as_reference(node)
                }
                ASTNodeType::UnaryOp { op, expr }
                    if *op == formualizer_parse::UnaryOperator::Spill =>
                {
                    self.interp.ast_spill_reference(expr)
                }
                _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                    .with_message("Expected a reference (by-ref argument)")),
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                let node = data_store.get_node(*id).ok_or_else(|| {
                    ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                })?;
                match node {
                    crate::engine::arena::AstNodeData::Reference { ref_type, .. } => {
                        let ref_type = data_store.compact_ref(*ref_type);
                        // Same local-shadowing rule as the AST branch above.
                        if let crate::engine::arena::CompactRefType::NamedRange(name_id) = ref_type
                            && let Some(local) =
                                self.local_named_reference(data_store.resolve_ast_string(*name_id))
                        {
                            return local;
                        }
                        let reference = data_store
                            .reconstruct_reference_type_for_eval(ref_type, sheet_registry);
                        self.interp.reference_for_current_offset(&reference)
                    }
                    crate::engine::arena::AstNodeData::Function { .. }
                    | crate::engine::arena::AstNodeData::BinaryOp { .. } => self
                        .interp
                        .evaluate_arena_ast_as_reference(*id, data_store, sheet_registry),
                    crate::engine::arena::AstNodeData::UnaryOp { operator, expr_id }
                        if *operator == formualizer_parse::UnaryOperator::Spill =>
                    {
                        self.interp
                            .arena_spill_reference(*expr_id, data_store, sheet_registry)
                    }
                    _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                        .with_message("Expected a reference (by-ref argument)")),
                }
            }
        }
    }

    fn function_resolution_attempt(
        &self,
    ) -> Option<Result<crate::function::FunctionResolution<'b>, ExcelError>> {
        match &self.expr {
            ArgumentExpr::Ast(node) => {
                let ASTNodeType::Function { name, args } = &node.node_type else {
                    return None;
                };
                if !self
                    .interp
                    .ast_function_capabilities(name)
                    .is_some_and(|caps| caps.contains(crate::function::FnCaps::RETURNS_REFERENCE))
                {
                    return None;
                }
                let fun = match self.interp.ast_function(name) {
                    Some(fun) => fun,
                    None => {
                        return Some(Err(ExcelError::new(ExcelErrorKind::Name)
                            .with_message(format!("Unknown function: {name}"))));
                    }
                };
                let handles: Vec<_> = args
                    .iter()
                    .map(|arg| ArgumentHandle::new(arg, self.interp))
                    .collect();
                let ctx = DefaultFunctionContext::new_with_sheet(
                    self.interp.context,
                    None,
                    self.interp.current_sheet(),
                );
                let _origin = self.interp.reference_diagnostic_scope();
                Some(self.interp.diagnostic_dispatch(
                    name,
                    &*fun,
                    fun.reports_known_limitations(),
                    || fun.resolve_reference_or_value(&handles, &ctx, &|| self.value()),
                    crate::operation_diagnostics::is_unimplemented_resolution,
                ))
            }
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                let node = match data_store.get_node(*id) {
                    Some(node) => node,
                    None => {
                        return Some(Err(
                            ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                        ));
                    }
                };
                let crate::engine::arena::AstNodeData::Function { name_id, .. } = node else {
                    return None;
                };
                let name = data_store.resolve_ast_string(*name_id);
                if !self
                    .interp
                    .ast_function_capabilities(name)
                    .is_some_and(|caps| caps.contains(crate::function::FnCaps::RETURNS_REFERENCE))
                {
                    return None;
                }
                let fun = match self.interp.ast_function(name) {
                    Some(fun) => fun,
                    None => {
                        return Some(Err(ExcelError::new(ExcelErrorKind::Name)
                            .with_message(format!("Unknown function: {name}"))));
                    }
                };
                let args = match data_store.get_args(*id) {
                    Some(args) => args,
                    None => {
                        return Some(Err(ExcelError::new(ExcelErrorKind::Value)
                            .with_message("Missing function args")));
                    }
                };
                let handles: Vec<_> = args
                    .iter()
                    .copied()
                    .map(|arg_id| {
                        ArgumentHandle::new_arena(arg_id, self.interp, data_store, sheet_registry)
                    })
                    .collect();
                let ctx = DefaultFunctionContext::new_with_sheet(
                    self.interp.context,
                    None,
                    self.interp.current_sheet(),
                );
                let _origin = self.interp.reference_diagnostic_scope();
                Some(self.interp.diagnostic_dispatch(
                    name,
                    &*fun,
                    fun.reports_known_limitations(),
                    || fun.resolve_reference_or_value(&handles, &ctx, &|| self.value()),
                    crate::operation_diagnostics::is_unimplemented_resolution,
                ))
            }
        }
    }

    fn reference_attempt(&self) -> Option<Result<ReferenceType, ExcelError>> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => {
                    // A LET/LAMBDA local shadows any workbook name of the same
                    // spelling, so it never takes the named-range route. A
                    // local bound to a range is that range; any other local
                    // resolves on the value path.
                    if let ReferenceType::NamedRange(name) = reference
                        && self.interp.resolve_local_name(name).is_some()
                    {
                        return self.interp.resolve_local_bound_reference(name).map(Ok);
                    }
                    Some(self.interp.reference_for_current_offset(reference))
                }
                ASTNodeType::BinaryOp { op, .. }
                    if *op == formualizer_parse::BinaryOperator::Range =>
                {
                    Some(self.interp.evaluate_ast_as_reference(node))
                }
                ASTNodeType::UnaryOp { op, expr }
                    if *op == formualizer_parse::UnaryOperator::Spill =>
                {
                    Some(self.interp.ast_spill_reference(expr))
                }
                ASTNodeType::Function { name, .. }
                    if self
                        .interp
                        .ast_function_capabilities(name)
                        .is_some_and(|caps| {
                            caps.contains(crate::function::FnCaps::RETURNS_REFERENCE)
                        }) =>
                {
                    self.interp.try_evaluate_ast_as_reference(node)
                }
                _ => None,
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                let node = match data_store.get_node(*id) {
                    Some(node) => node,
                    None => {
                        return Some(Err(
                            ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                        ));
                    }
                };
                match node {
                    crate::engine::arena::AstNodeData::Reference { ref_type, .. } => {
                        let ref_type = data_store.compact_ref(*ref_type);
                        // Same local-shadowing rule as the AST branch above.
                        if let crate::engine::arena::CompactRefType::NamedRange(name_id) = ref_type
                        {
                            let name = data_store.resolve_ast_string(*name_id);
                            if self.interp.resolve_local_name(name).is_some() {
                                return self.interp.resolve_local_bound_reference(name).map(Ok);
                            }
                        }
                        let reference = data_store
                            .reconstruct_reference_type_for_eval(ref_type, sheet_registry);
                        Some(self.interp.reference_for_current_offset(&reference))
                    }
                    crate::engine::arena::AstNodeData::BinaryOp { operator, .. }
                        if *operator == formualizer_parse::BinaryOperator::Range =>
                    {
                        Some(self.interp.evaluate_arena_ast_as_reference(
                            *id,
                            data_store,
                            sheet_registry,
                        ))
                    }
                    crate::engine::arena::AstNodeData::UnaryOp { operator, expr_id }
                        if *operator == formualizer_parse::UnaryOperator::Spill =>
                    {
                        Some(self.interp.arena_spill_reference(
                            *expr_id,
                            data_store,
                            sheet_registry,
                        ))
                    }
                    crate::engine::arena::AstNodeData::Function { name_id, .. } => {
                        if self
                            .interp
                            .arena_function_capabilities(*name_id, data_store)
                            .is_some_and(|caps| {
                                caps.contains(crate::function::FnCaps::RETURNS_REFERENCE)
                            })
                        {
                            self.interp.try_evaluate_arena_ast_as_reference(
                                *id,
                                data_store,
                                sheet_registry,
                            )
                        } else {
                            None
                        }
                    }
                    _ => None,
                }
            }
        }
    }

    pub(crate) fn resolve_reference_or_value(
        &self,
    ) -> Result<crate::function::FunctionResolution<'b>, ExcelError> {
        let resolved = self
            .caches()
            .cached_reference_or_value
            .get_or_init(|| self.compute_reference_or_value())
            .clone();
        if let Ok(crate::function::FunctionResolution::Value(value)) = &resolved {
            let _ = self.cached_value.set(Ok(MemoizedValue::new(value.clone())));
        }
        resolved
    }

    fn compute_reference_or_value(
        &self,
    ) -> Result<crate::function::FunctionResolution<'b>, ExcelError> {
        if let Some(result) = self.function_resolution_attempt() {
            return result;
        }
        if let Some(reference) = self.reference_attempt() {
            return Ok(match reference {
                Ok(reference) => crate::function::FunctionResolution::Reference(reference),
                Err(error) => crate::function::FunctionResolution::ReferenceError(error),
            });
        }
        self.value().map(crate::function::FunctionResolution::Value)
    }

    /// Resolve this argument once without using a failed range conversion as type dispatch.
    ///
    /// Direct references and the `:` operator take the reference path. Functions
    /// with `RETURNS_REFERENCE` first attempt reference evaluation, but fall back
    /// to their cached value when `eval_reference` returns `None`.
    pub(crate) fn resolve_once(&self) -> Result<ResolvedArgument<'b>, ExcelError> {
        let result = self
            .caches()
            .cached_resolved
            .get_or_init(|| self.compute_resolved_argument())
            .clone();
        if let Some(collector) = self.interp.diagnostics {
            match &result {
                Ok(ResolvedArgument::Range(_)) => collector.ambiguous_input(),
                Ok(ResolvedArgument::Value(value)) => collector.input_value(value),
                Ok(ResolvedArgument::ReferenceError(e)) | Err(e) => collector.input_error(e),
            }
        }
        result
    }

    /// Attaches the context's cancellation token and materialisation budget.
    pub(crate) fn with_context_cancel_token(&self, view: RangeView<'b>) -> RangeView<'b> {
        let view = view.with_cell_budget(self.interp.context.materialized_array_cells());
        match self.interp.context.cancellation_token() {
            Some(token) => view.with_cancel_token(Some(token)),
            None => view,
        }
    }

    fn compute_resolved_argument(&self) -> Result<ResolvedArgument<'b>, ExcelError> {
        if let ArgumentExpr::Arena { id, data_store, .. } = &self.expr
            && let Some(result) = self.interp.resolve_arena_argument_range(*id, data_store)
        {
            return match result {
                Ok(view) => Ok(ResolvedArgument::Range(
                    self.with_context_cancel_token(view),
                )),
                Err(error) if error.kind == ExcelErrorKind::Cancelled => Err(error),
                Err(error) => Ok(ResolvedArgument::ReferenceError(error)),
            };
        }
        let value = match self.resolve_reference_or_value()? {
            crate::function::FunctionResolution::Reference(reference) => {
                return match self
                    .interp
                    .context
                    .resolve_range_view(&reference, self.interp.current_sheet())
                {
                    Ok(view) => Ok(ResolvedArgument::Range(
                        self.with_context_cancel_token(view),
                    )),
                    Err(error) if error.kind == ExcelErrorKind::Cancelled => Err(error),
                    Err(error) => Ok(ResolvedArgument::ReferenceError(error)),
                };
            }
            crate::function::FunctionResolution::ReferenceError(error)
                if error.kind == ExcelErrorKind::Cancelled =>
            {
                return Err(error);
            }
            crate::function::FunctionResolution::ReferenceError(error) => {
                return Ok(ResolvedArgument::ReferenceError(error));
            }
            crate::function::FunctionResolution::Value(value) => value,
        };

        match value {
            CalcValue::Range(view) => Ok(ResolvedArgument::Range(
                self.with_context_cancel_token(view),
            )),
            CalcValue::Scalar(LiteralValue::Array(rows)) => {
                let view = RangeView::try_from_owned_rows(
                    rows,
                    self.interp.context.date_system(),
                    self.interp.context.cancellation_token(),
                )?;
                Ok(ResolvedArgument::Range(view))
            }
            other => Ok(ResolvedArgument::Value(other)),
        }
    }

    /// `#NUM!` when a `rows x cols` copy exceeds the context's materialisation
    /// budget.
    fn check_materialized_cells(&self, rows: usize, cols: usize) -> Result<(), ExcelError> {
        match crate::engine::resource_ledger::materialized_cells_exceed(
            rows,
            cols,
            self.interp.context.materialized_array_cells(),
        ) {
            Some(error) => Err(error),
            None => Ok(()),
        }
    }

    pub fn range(&self) -> Result<Box<dyn Range>, ExcelError> {
        if let Some(collector) = self.interp.diagnostics {
            collector.ambiguous_input();
        }
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => {
                    // Prefer RangeView since it has explicit current-sheet context.
                    let reference = self.interp.reference_for_current_offset(reference)?;
                    let view = self
                        .interp
                        .context
                        .resolve_range_view(&reference, self.interp.current_sheet())?;
                    let (rows, cols) = view.dims();
                    self.check_materialized_cells(rows, cols)?;
                    let mut out: Vec<Vec<LiteralValue>> = Vec::with_capacity(rows);
                    view.for_each_row(&mut |row| {
                        let row_data: Vec<LiteralValue> = (0..cols)
                            .map(|c| row.get(c).cloned().unwrap_or(LiteralValue::Empty))
                            .collect();
                        out.push(row_data);
                        Ok(())
                    })?;
                    Ok(Box::new(InMemoryRange::new(out)))
                }
                ASTNodeType::Function { .. } | ASTNodeType::BinaryOp { .. } => {
                    let reference = self.reference_for_eval()?;
                    let view = self
                        .interp
                        .context
                        .resolve_range_view(&reference, self.interp.current_sheet())?;
                    let (rows, cols) = view.dims();
                    self.check_materialized_cells(rows, cols)?;
                    let mut out: Vec<Vec<LiteralValue>> = Vec::with_capacity(rows);
                    view.for_each_row(&mut |row| {
                        let row_data: Vec<LiteralValue> = (0..cols)
                            .map(|c| row.get(c).cloned().unwrap_or(LiteralValue::Empty))
                            .collect();
                        out.push(row_data);
                        Ok(())
                    })?;
                    Ok(Box::new(InMemoryRange::new(out)))
                }
                ASTNodeType::Array(rows) => {
                    let mut materialized = Vec::new();
                    for row in rows {
                        let mut materialized_row = Vec::new();
                        for cell in row {
                            materialized_row
                                .push(self.interp.evaluate_ast_uncached(cell)?.into_literal());
                        }
                        materialized.push(materialized_row);
                    }
                    Ok(Box::new(InMemoryRange::new(materialized)))
                }
                _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                    .with_message(format!("Expected a range, got {:?}", node.node_type))),
            },
            ArgumentExpr::Arena { id, data_store, .. } => {
                let node = data_store.get_node(*id).ok_or_else(|| {
                    ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                })?;

                match node {
                    crate::engine::arena::AstNodeData::Reference { .. }
                    | crate::engine::arena::AstNodeData::Function { .. }
                    | crate::engine::arena::AstNodeData::BinaryOp { .. } => {
                        let reference = self.reference_for_eval()?;
                        let view = self
                            .interp
                            .context
                            .resolve_range_view(&reference, self.interp.current_sheet())?;
                        let (rows, cols) = view.dims();
                        self.check_materialized_cells(rows, cols)?;
                        let mut out: Vec<Vec<LiteralValue>> = Vec::with_capacity(rows);
                        view.for_each_row(&mut |row| {
                            let row_data: Vec<LiteralValue> = (0..cols)
                                .map(|c| row.get(c).cloned().unwrap_or(LiteralValue::Empty))
                                .collect();
                            out.push(row_data);
                            Ok(())
                        })?;
                        Ok(Box::new(InMemoryRange::new(out)))
                    }
                    crate::engine::arena::AstNodeData::Array { .. } => {
                        let (rows, cols, elements) =
                            data_store.get_array_elems(*id).ok_or_else(|| {
                                ExcelError::new(ExcelErrorKind::Value).with_message("Invalid array")
                            })?;
                        let rows_usize = rows as usize;
                        let cols_usize = cols as usize;
                        let mut materialized: Vec<Vec<LiteralValue>> =
                            Vec::with_capacity(rows_usize);
                        for r in 0..rows_usize {
                            let mut row = Vec::with_capacity(cols_usize);
                            for c in 0..cols_usize {
                                let idx = r * cols_usize + c;
                                let elem_id = elements.get(idx).copied().ok_or_else(|| {
                                    ExcelError::new(ExcelErrorKind::Value)
                                        .with_message("Invalid array")
                                })?;
                                let v = self.interp.evaluate_arena_ast_inner(
                                    elem_id,
                                    data_store,
                                    self.sheet_registry(),
                                )?;
                                row.push(v.into_literal());
                            }
                            materialized.push(row);
                        }
                        Ok(Box::new(InMemoryRange::new(materialized)))
                    }
                    _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                        .with_message("Argument cannot be interpreted as a range.")),
                }
            }
        }
    }

    fn sheet_registry(&self) -> &crate::engine::sheet_registry::SheetRegistry {
        match &self.expr {
            ArgumentExpr::Ast(_) => {
                // Not needed; used only in arena flows.
                unreachable!("sheet_registry only used for arena ArgumentHandle")
            }
            ArgumentExpr::Arena { sheet_registry, .. } => sheet_registry,
        }
    }

    /// Resolve this argument to a [`RangeView`].
    ///
    /// Delegates to [`Self::resolve_once`] so reference-shaped and computed
    /// arguments share one cached resolution path. A reference keeps its lazy
    /// view, while a computed argument (`B1:B3="x"`, `SEQUENCE(3)`, `{1,2}`)
    /// resolves through the same single evaluation the rest of argument
    /// preparation uses instead of being rejected for not being a reference.
    pub fn range_view(&self) -> Result<RangeView<'b>, ExcelError> {
        match self.resolve_once()? {
            ResolvedArgument::Range(view) => Ok(view),
            // A genuine reference failure (`OFFSET(A1,-1,0)`) stays an error
            // rather than being masked by re-evaluating the node as a value.
            ResolvedArgument::ReferenceError(error) => Err(error),
            // `resolve_once` already folds range-shaped and array values into
            // `Range`, so these two arms are defensive. They still apply the
            // cancellation token so the invariant changing could never silently
            // drop cancellation on this path.
            ResolvedArgument::Value(CalcValue::Range(view)) => {
                Ok(self.with_context_cancel_token(view))
            }
            ResolvedArgument::Value(CalcValue::Scalar(LiteralValue::Array(rows))) => {
                RangeView::try_from_owned_rows(
                    rows,
                    self.interp.context.date_system(),
                    self.interp.context.cancellation_token(),
                )
            }
            ResolvedArgument::Value(_) => Err(ExcelError::new(ExcelErrorKind::Ref)
                .with_message("Argument cannot be interpreted as a range.")),
        }
    }

    /// Resolve this argument to a [`RangeView`], promoting a scalar to a 1x1 view.
    ///
    /// Excel treats a scalar handed to a range-consuming function as a 1x1 array,
    /// so `=TRANSPOSE(2)` is `2` rather than an error. [`Self::range_view`] rejects
    /// scalars, and a function that wants the Excel behaviour opts in here.
    ///
    /// This is deliberately *not* the behaviour of `range_view` itself. Several
    /// builtins use a `range_view` failure as type dispatch, where "scalar" and
    /// "1x1 range" mean genuinely different things -- `MEDIAN(TRUE)` is `1`
    /// because a direct scalar is coerced while a range cell of the same type is
    /// skipped, and a D-function's scalar criteria argument is an error rather
    /// than an empty criteria block that matches every row. Promoting inside
    /// `range_view` would silently change those answers. The rule for choosing
    /// between the two: a function that distinguishes a scalar argument from a
    /// 1x1 range keeps `range_view`.
    ///
    /// An error scalar propagates as an error rather than becoming a 1x1 view
    /// containing it, so `=TRANSPOSE(NA())` is `#N/A` instead of being masked as
    /// `#REF!`.
    pub fn range_view_or_scalar(&self) -> Result<RangeView<'b>, ExcelError> {
        match self.resolve_once()? {
            ResolvedArgument::Range(view) => Ok(view),
            ResolvedArgument::ReferenceError(error) => Err(error),
            ResolvedArgument::Value(CalcValue::Range(view)) => {
                Ok(self.with_context_cancel_token(view))
            }
            ResolvedArgument::Value(CalcValue::Scalar(LiteralValue::Array(rows))) => {
                RangeView::try_from_owned_rows(
                    rows,
                    self.interp.context.date_system(),
                    self.interp.context.cancellation_token(),
                )
            }
            // Preserve the argument's own error instead of reporting the shape
            // mismatch that rejecting it would produce.
            ResolvedArgument::Value(CalcValue::Scalar(LiteralValue::Error(error))) => Err(error),
            ResolvedArgument::Value(
                CalcValue::Scalar(scalar) | CalcValue::AnnotatedScalar(scalar, _),
            ) => RangeView::try_from_owned_rows(
                vec![vec![scalar]],
                self.interp.context.date_system(),
                self.interp.context.cancellation_token(),
            ),
            // A lambda is not a value that can stand in for a 1x1 array.
            ResolvedArgument::Value(CalcValue::Callable(_)) => {
                Err(ExcelError::new(ExcelErrorKind::Ref)
                    .with_message("Argument cannot be interpreted as a range."))
            }
        }
    }

    pub fn value_or_range(&self) -> Result<EvaluatedArg<'_>, ExcelError> {
        self.range().map(EvaluatedArg::Range).or_else(|_| {
            self.value()
                .map(|cv| EvaluatedArg::LiteralValue(Cow::Owned(cv.into_literal())))
        })
    }

    /// Lazily iterate values for this argument in row-major expansion order.
    /// - Reference: stream via RangeView (row-major)
    /// - Array literal: evaluate each element lazily per cell
    /// - Scalar/other expressions: a single value
    pub fn lazy_values_owned(
        &'a self,
    ) -> Result<Box<dyn Iterator<Item = LiteralValue> + 'a>, ExcelError> {
        // Reference-shaped syntax usually names a range, but a bare LET/LAMBDA
        // local has the same syntax and may be bound to a scalar. Resolve once
        // and stream whichever shape came back instead of insisting on a range.
        fn resolved_values<'a, 'b>(
            handle: &'a ArgumentHandle<'a, 'b>,
        ) -> Result<Box<dyn Iterator<Item = LiteralValue> + 'a>, ExcelError> {
            match handle.resolve_once()? {
                ResolvedArgument::Range(view) => {
                    let mut values = Vec::new();
                    view.for_each_cell(&mut |value| {
                        values.push(value.clone());
                        Ok(())
                    })?;
                    Ok(Box::new(values.into_iter()))
                }
                ResolvedArgument::ReferenceError(error) => Err(error),
                ResolvedArgument::Value(value) => {
                    Ok(Box::new(std::iter::once(value.into_literal())))
                }
            }
        }

        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { .. } => resolved_values(self),
                ASTNodeType::Array(rows) => {
                    struct ArrayEvalIter<'a, 'b> {
                        rows: &'a [Vec<ASTNode>],
                        r: usize,
                        c: usize,
                        interp: &'a Interpreter<'b>,
                    }
                    impl<'a, 'b> Iterator for ArrayEvalIter<'a, 'b> {
                        type Item = LiteralValue;
                        fn next(&mut self) -> Option<Self::Item> {
                            if self.rows.is_empty() {
                                return None;
                            }
                            let rows = self.rows;
                            let mut r = self.r;
                            let mut c = self.c;
                            if r >= rows.len() {
                                return None;
                            }
                            let node = &rows[r][c];
                            // advance indices
                            c += 1;
                            if c >= rows[r].len() {
                                r += 1;
                                c = 0;
                            }
                            self.r = r;
                            self.c = c;
                            match self.interp.evaluate_ast_uncached(node) {
                                Ok(cv) => Some(cv.into_literal()),
                                Err(e) => Some(LiteralValue::Error(e)),
                            }
                        }
                    }
                    let it = ArrayEvalIter {
                        rows,
                        r: 0,
                        c: 0,
                        interp: self.interp,
                    };
                    Ok(Box::new(it))
                }
                _ => {
                    // Single value expression
                    let v = self.value()?.into_literal();
                    Ok(Box::new(std::iter::once(v)))
                }
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                let node = data_store.get_node(*id).ok_or_else(|| {
                    ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                })?;

                match node {
                    crate::engine::arena::AstNodeData::Reference { .. } => resolved_values(self),
                    crate::engine::arena::AstNodeData::Array { .. } => {
                        let (rows, cols, elements) =
                            data_store.get_array_elems(*id).ok_or_else(|| {
                                ExcelError::new(ExcelErrorKind::Value).with_message("Invalid array")
                            })?;

                        struct ArenaArrayEvalIter<'a, 'b> {
                            elements: &'a [crate::engine::arena::AstNodeId],
                            idx: usize,
                            interp: &'a Interpreter<'b>,
                            data_store: &'a crate::engine::arena::DataStore,
                            sheet_registry: &'a crate::engine::sheet_registry::SheetRegistry,
                        }

                        impl<'a, 'b> Iterator for ArenaArrayEvalIter<'a, 'b> {
                            type Item = LiteralValue;

                            fn next(&mut self) -> Option<Self::Item> {
                                let id = self.elements.get(self.idx).copied()?;
                                self.idx += 1;
                                match self.interp.evaluate_arena_ast_inner(
                                    id,
                                    self.data_store,
                                    self.sheet_registry,
                                ) {
                                    Ok(cv) => Some(cv.into_literal()),
                                    Err(e) => Some(LiteralValue::Error(e)),
                                }
                            }
                        }

                        let _ = (rows, cols);
                        let it = ArenaArrayEvalIter {
                            elements,
                            idx: 0,
                            interp: self.interp,
                            data_store,
                            sheet_registry,
                        };
                        Ok(Box::new(it))
                    }
                    _ => {
                        let v = self.interp.evaluate_arena_ast_inner(
                            *id,
                            data_store,
                            sheet_registry,
                        )?;
                        Ok(Box::new(std::iter::once(v.into_literal())))
                    }
                }
            }
        }
    }

    pub fn ast(&self) -> &ASTNode {
        match &self.expr {
            ArgumentExpr::Ast(node) => node,
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => self.caches().cached_ast.get_or_init(|| {
                data_store
                    .retrieve_ast(*id, sheet_registry)
                    .unwrap_or_else(|| ASTNode {
                        node_type: ASTNodeType::Literal(LiteralValue::Error(
                            ExcelError::new(ExcelErrorKind::Value)
                                .with_message("Missing formula AST"),
                        )),
                        source_token: None,
                        contains_volatile: false,
                    })
            }),
        }
    }

    /// Returns the raw reference from the AST when this argument is a reference.
    /// This does not evaluate the reference or materialize values.
    pub fn as_reference(&self) -> Result<&ReferenceType, ExcelError> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => {
                    // Same local-shadowing rule as `reference_for_eval`.
                    if let ReferenceType::NamedRange(name) = reference
                        && let Some(local) = self.local_named_reference(name)
                    {
                        let bound = local?;
                        return Ok(self.caches().cached_ref.get_or_init(|| bound));
                    }
                    Ok(reference)
                }
                _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                    .with_message("Expected a reference (by-ref argument)")),
            },
            ArgumentExpr::Arena { .. } => {
                let reference = self.reference_for_eval()?;
                Ok(self.caches().cached_ref.get_or_init(|| reference))
            }
        }
    }

    /// Returns a `ReferenceType` if this argument is a reference or a function that
    /// can yield a reference via `eval_reference`. Materializes no values.
    pub fn as_reference_or_eval(&self) -> Result<ReferenceType, ExcelError> {
        match &self.expr {
            ArgumentExpr::Ast(node) => match &node.node_type {
                ASTNodeType::Reference { reference, .. } => {
                    // Same local-shadowing rule as `reference_for_eval`.
                    if let ReferenceType::NamedRange(name) = reference
                        && let Some(local) = self.local_named_reference(name)
                    {
                        return local;
                    }
                    self.interp.reference_for_current_offset(reference)
                }
                ASTNodeType::Function { .. } | ASTNodeType::BinaryOp { .. } => {
                    self.interp.evaluate_ast_as_reference(node)
                }
                ASTNodeType::UnaryOp { op, expr }
                    if *op == formualizer_parse::UnaryOperator::Spill =>
                {
                    self.interp.ast_spill_reference(expr)
                }
                _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                    .with_message("Argument is not a reference")),
            },
            ArgumentExpr::Arena {
                id,
                data_store,
                sheet_registry,
            } => {
                let node = data_store.get_node(*id).ok_or_else(|| {
                    ExcelError::new(ExcelErrorKind::Value).with_message("Missing AST node")
                })?;

                match node {
                    crate::engine::arena::AstNodeData::Reference { .. } => {
                        self.reference_for_eval()
                    }
                    crate::engine::arena::AstNodeData::Function { .. }
                    | crate::engine::arena::AstNodeData::BinaryOp { .. } => self
                        .interp
                        .evaluate_arena_ast_as_reference(*id, data_store, sheet_registry),
                    crate::engine::arena::AstNodeData::UnaryOp { operator, expr_id }
                        if *operator == formualizer_parse::UnaryOperator::Spill =>
                    {
                        self.interp
                            .arena_spill_reference(*expr_id, data_store, sheet_registry)
                    }
                    _ => Err(ExcelError::new(ExcelErrorKind::Ref)
                        .with_message("Argument is not a reference")),
                }
            }
        }
    }

    /* tiny validator helper for macro */
    pub fn matches_kind(&self, k: formualizer_common::ArgKind) -> Result<bool, ExcelError> {
        Ok(match k {
            formualizer_common::ArgKind::Any => true,
            formualizer_common::ArgKind::Range => self.range().is_ok(),
            formualizer_common::ArgKind::Number => matches!(
                self.value()?.into_literal(),
                LiteralValue::Number(_) | LiteralValue::Int(_)
            ),
            formualizer_common::ArgKind::Text => {
                matches!(self.value()?.into_literal(), LiteralValue::Text(_))
            }
            formualizer_common::ArgKind::Logical => {
                matches!(self.value()?.into_literal(), LiteralValue::Boolean(_))
            }
        })
    }
}

/* simple Vec-backed range */
#[derive(Debug, Clone)]
pub struct InMemoryRange {
    data: Vec<Vec<LiteralValue>>,
}
impl InMemoryRange {
    pub fn new(d: Vec<Vec<LiteralValue>>) -> Self {
        Self { data: d }
    }
}
impl Range for InMemoryRange {
    fn get(&self, r: usize, c: usize) -> Result<LiteralValue, ExcelError> {
        Ok(self
            .data
            .get(r)
            .and_then(|row| row.get(c))
            .cloned()
            .unwrap_or(LiteralValue::Empty))
    }
    fn dimensions(&self) -> (usize, usize) {
        (self.data.len(), self.data.first().map_or(0, |r| r.len()))
    }
    fn as_any(&self) -> &dyn Any {
        self
    }
}

/* ───────────────────────── Table abstraction ───────────────────────── */

pub trait Table: Debug + Send + Sync {
    fn get_cell(&self, row: usize, column: &str) -> Result<LiteralValue, ExcelError>;
    fn get_column(&self, column: &str) -> Result<Box<dyn Range>, ExcelError>;
    /// Ordered list of column names
    fn columns(&self) -> Vec<String> {
        vec![]
    }
    /// Number of data rows (excluding headers/totals)
    fn data_height(&self) -> usize {
        0
    }
    /// Whether the table has a header row
    fn has_headers(&self) -> bool {
        false
    }
    /// Whether the table has a totals row
    fn has_totals(&self) -> bool {
        false
    }
    /// Headers row as a 1xW range
    fn headers_row(&self) -> Option<Box<dyn Range>> {
        None
    }
    /// Totals row as a 1xW range, if present
    fn totals_row(&self) -> Option<Box<dyn Range>> {
        None
    }
    /// Entire data body as HxW range
    fn data_body(&self) -> Option<Box<dyn Range>> {
        None
    }
    fn clone_box(&self) -> Box<dyn Table>;
}
impl Table for Box<dyn Table> {
    fn get_cell(&self, r: usize, c: &str) -> Result<LiteralValue, ExcelError> {
        (**self).get_cell(r, c)
    }
    fn get_column(&self, c: &str) -> Result<Box<dyn Range>, ExcelError> {
        (**self).get_column(c)
    }
    fn columns(&self) -> Vec<String> {
        (**self).columns()
    }
    fn data_height(&self) -> usize {
        (**self).data_height()
    }
    fn has_headers(&self) -> bool {
        (**self).has_headers()
    }
    fn has_totals(&self) -> bool {
        (**self).has_totals()
    }
    fn headers_row(&self) -> Option<Box<dyn Range>> {
        (**self).headers_row()
    }
    fn totals_row(&self) -> Option<Box<dyn Range>> {
        (**self).totals_row()
    }
    fn data_body(&self) -> Option<Box<dyn Range>> {
        (**self).data_body()
    }
    fn clone_box(&self) -> Box<dyn Table> {
        (**self).clone_box()
    }
}

/* ─────────────────────── Resolver super-trait ─────────────────────── */

pub trait ReferenceResolver: Send + Sync {
    fn resolve_cell_reference(
        &self,
        sheet: Option<&str>,
        row: u32,
        col: u32,
    ) -> Result<LiteralValue, ExcelError>;
}
pub trait RangeResolver: Send + Sync {
    fn resolve_range_reference(
        &self,
        sheet: Option<&str>,
        sr: Option<u32>,
        sc: Option<u32>,
        er: Option<u32>,
        ec: Option<u32>,
    ) -> Result<Box<dyn Range>, ExcelError>;
}
pub trait NamedRangeResolver: Send + Sync {
    fn resolve_named_range_reference(
        &self,
        name: &str,
    ) -> Result<Vec<Vec<LiteralValue>>, ExcelError>;
}
pub trait TableResolver: Send + Sync {
    fn resolve_table_reference(
        &self,
        tref: &formualizer_parse::parser::TableReference,
    ) -> Result<Box<dyn Table>, ExcelError>;
}

pub trait SourceResolver: Send + Sync {
    fn source_scalar_version(&self, _name: &str) -> Option<u64> {
        None
    }

    fn resolve_source_scalar(&self, name: &str) -> Result<LiteralValue, ExcelError> {
        Err(ExcelError::new(ExcelErrorKind::NImpl)
            .with_message(format!("Source scalar not supported: {name}")))
    }

    fn source_table_version(&self, _name: &str) -> Option<u64> {
        None
    }

    fn resolve_source_table(&self, name: &str) -> Result<Box<dyn Table>, ExcelError> {
        Err(ExcelError::new(ExcelErrorKind::NImpl)
            .with_message(format!("Source table not supported: {name}")))
    }
}

pub trait Resolver: ReferenceResolver + RangeResolver + NamedRangeResolver + TableResolver {
    fn resolve_range_like(&self, r: &ReferenceType) -> Result<Box<dyn Range>, ExcelError> {
        match r {
            ReferenceType::Range {
                sheet,
                start_row,
                start_col,
                end_row,
                end_col,
                ..
            } => self.resolve_range_reference(
                sheet.as_deref(),
                *start_row,
                *start_col,
                *end_row,
                *end_col,
            ),
            ReferenceType::External(_) => Err(ExcelError::new(ExcelErrorKind::NImpl)
                .with_message("External references are not supported by Resolver".to_string())),
            ReferenceType::Table(tref) => {
                let t = self.resolve_table_reference(tref)?;
                match &tref.specifier {
                    Some(TableSpecifier::Column(c)) => t.get_column(c),
                    Some(TableSpecifier::ColumnRange(start, end)) => {
                        // Build a rectangular range from start..=end columns in table order
                        let cols = t.columns();
                        let start_key = start.to_lowercase();
                        let end_key = end.to_lowercase();
                        let start_idx = cols.iter().position(|n| n.to_lowercase() == start_key);
                        let end_idx = cols.iter().position(|n| n.to_lowercase() == end_key);
                        if let (Some(mut si), Some(mut ei)) = (start_idx, end_idx) {
                            if si > ei {
                                std::mem::swap(&mut si, &mut ei);
                            }
                            // Materialize by stacking columns into a 2D array
                            let h = t.data_height();
                            let w = ei - si + 1;
                            let mut rows = vec![vec![LiteralValue::Empty; w]; h];
                            for (offset, ci) in (si..=ei).enumerate() {
                                let cname = &cols[ci];
                                let col_range = t.get_column(cname)?;
                                let (rh, _) = col_range.dimensions();
                                for (r, row) in rows.iter_mut().enumerate().take(h.min(rh)) {
                                    row[offset] = col_range.get(r, 0)?;
                                }
                            }
                            Ok(Box::new(InMemoryRange::new(rows)))
                        } else {
                            Err(ExcelError::new(ExcelErrorKind::Ref).with_message(
                                "Column range refers to unknown column(s)".to_string(),
                            ))
                        }
                    }
                    Some(TableSpecifier::SpecialItem(
                        formualizer_parse::parser::SpecialItem::Headers,
                    )) => {
                        if let Some(h) = t.headers_row() {
                            Ok(h)
                        } else {
                            Ok(Box::new(InMemoryRange::new(vec![])))
                        }
                    }
                    Some(TableSpecifier::SpecialItem(
                        formualizer_parse::parser::SpecialItem::Totals,
                    )) => {
                        if let Some(tr) = t.totals_row() {
                            Ok(tr)
                        } else {
                            Ok(Box::new(InMemoryRange::new(vec![])))
                        }
                    }
                    Some(TableSpecifier::SpecialItem(
                        formualizer_parse::parser::SpecialItem::Data,
                    )) => {
                        if let Some(body) = t.data_body() {
                            Ok(body)
                        } else {
                            Ok(Box::new(InMemoryRange::new(vec![])))
                        }
                    }
                    Some(TableSpecifier::SpecialItem(
                        formualizer_parse::parser::SpecialItem::All,
                    )) => {
                        // Equivalent to TableSpecifier::All handling
                        let mut out: Vec<Vec<LiteralValue>> = Vec::new();
                        if let Some(h) = t.headers_row() {
                            out.extend(h.iter_rows());
                        }
                        if let Some(body) = t.data_body() {
                            out.extend(body.iter_rows());
                        }
                        if let Some(tr) = t.totals_row() {
                            out.extend(tr.iter_rows());
                        }
                        Ok(Box::new(InMemoryRange::new(out)))
                    }
                    Some(TableSpecifier::SpecialItem(
                        formualizer_parse::parser::SpecialItem::ThisRow,
                    )) => Err(ExcelError::new(ExcelErrorKind::NImpl).with_message(
                        "@ (This Row) requires table-aware context; not yet supported".to_string(),
                    )),
                    Some(TableSpecifier::All) => {
                        // Concatenate headers (if any), data, totals (if any)
                        let mut out: Vec<Vec<LiteralValue>> = Vec::new();
                        if let Some(h) = t.headers_row() {
                            out.extend(h.iter_rows());
                        }
                        if let Some(body) = t.data_body() {
                            out.extend(body.iter_rows());
                        }
                        if let Some(tr) = t.totals_row() {
                            out.extend(tr.iter_rows());
                        }
                        Ok(Box::new(InMemoryRange::new(out)))
                    }
                    Some(TableSpecifier::Data) => {
                        if let Some(body) = t.data_body() {
                            Ok(body)
                        } else {
                            Ok(Box::new(InMemoryRange::new(vec![])))
                        }
                    }
                    // Defer complex combinations and row selectors for tranche 1
                    Some(TableSpecifier::Combination(_)) => Err(ExcelError::new(
                        ExcelErrorKind::NImpl,
                    )
                    .with_message("Complex structured references not yet supported".to_string())),
                    Some(TableSpecifier::Row(_)) => Err(ExcelError::new(ExcelErrorKind::NImpl)
                        .with_message("Row selectors (@/index) not yet supported".to_string())),
                    Some(TableSpecifier::Headers) | Some(TableSpecifier::Totals) => {
                        Err(ExcelError::new(ExcelErrorKind::NImpl).with_message(
                            "Legacy Headers/Totals variants not used; use SpecialItem".to_string(),
                        ))
                    }
                    None => Err(ExcelError::new(ExcelErrorKind::Ref).with_message(
                        "Table reference without specifier is unsupported".to_string(),
                    )),
                }
            }
            ReferenceType::NamedRange(n) => {
                let v = self.resolve_named_range_reference(n)?;
                Ok(Box::new(InMemoryRange::new(v)))
            }
            ReferenceType::Cell {
                sheet, row, col, ..
            } => {
                let v = self.resolve_cell_reference(sheet.as_deref(), *row, *col)?;
                Ok(Box::new(InMemoryRange::new(vec![vec![v]])))
            }
            ReferenceType::Cell3D { .. } | ReferenceType::Range3D { .. } => {
                Err(ExcelError::new(ExcelErrorKind::NImpl)
                    .with_message("3D references are not yet supported".to_string()))
            }
        }
    }
}

/* ───────────────────── EvaluationContext = Resolver+Fns ───────────── */

pub trait FunctionProvider: Send + Sync {
    fn get_function(&self, ns: &str, name: &str) -> Option<Arc<dyn Function>>;

    #[doc(hidden)]
    fn get_function_for_planning(&self, _ns: &str, _name: &str) -> Option<Arc<dyn Function>> {
        None
    }

    /// Monotonic revision for runtime function resolution and semantics used by
    /// compressed planning. Providers that cannot supply one fail closed.
    #[doc(hidden)]
    fn planning_semantic_revision(&self) -> Option<u64> {
        None
    }

    #[doc(hidden)]
    fn function_capabilities(&self, ns: &str, name: &str) -> Option<crate::function::FnCaps> {
        self.get_function(ns, name).map(|function| {
            crate::external_callbacks::metadata(
                function.as_ref(),
                crate::function::FnCaps::empty,
                || function.caps(),
            )
        })
    }

    fn function_semantic_identity(
        &self,
        ns: &str,
        name: &str,
        arity: usize,
    ) -> Option<crate::function_contract::FunctionSemanticIdentity> {
        crate::function_registry::resolve_semantic_identity(self, ns, name, arity)
    }
}

pub trait EvaluationContext: Resolver + FunctionProvider + SourceResolver {
    /// Optional request-event collector; does not certify retained-result coverage.
    fn operation_diagnostic_collector(
        &self,
    ) -> Option<&crate::operation_diagnostics::OperationDiagnosticCollector> {
        None
    }

    /// Get access to the shared thread pool for parallel evaluation
    /// Returns None if parallel evaluation is disabled or unavailable
    fn thread_pool(&self) -> Option<&Arc<rayon::ThreadPool>> {
        None
    }

    /// Returns the optional shared cancellation handle for this evaluation.
    ///
    /// Custom context authors may return a clone: clones share the same signal
    /// without allocating. Consumers should retrieve the handle once before a
    /// hot loop and poll [`crate::engine::CancelToken::is_cancelled`]
    /// periodically.
    fn cancellation_token(&self) -> Option<crate::engine::CancelToken> {
        None
    }

    /// Optional chunk size hint for streaming visitors.
    fn chunk_hint(&self) -> Option<usize> {
        None
    }

    /// Most cells a range or array value may have when it is materialised as
    /// a dense array (`evaluation_budgets.scratch.materialized_array_cells`);
    /// larger arrays evaluate to `#NUM!`. The default is 2^24 cells.
    fn materialized_array_cells(&self) -> u64 {
        crate::engine::resource_ledger::DEFAULT_MATERIALIZED_ARRAY_CELLS
    }

    /// Resolve a reference into a `RangeView` with clear bounds.
    /// Implementations should resolve un/partially bounded references using used-region.
    fn resolve_range_view<'c>(
        &'c self,
        _reference: &ReferenceType,
        _current_sheet: &str,
    ) -> Result<RangeView<'c>, ExcelError> {
        Err(ExcelError::new(ExcelErrorKind::NImpl))
    }

    /// Resolve a single-cell reference as a scalar value.
    ///
    /// Default implementation preserves existing reference semantics by routing through
    /// `resolve_range_view` and extracting a 1x1 value.
    fn resolve_cell_reference_value(
        &self,
        sheet: Option<&str>,
        row: u32,
        col: u32,
        current_sheet: &str,
    ) -> Result<LiteralValue, ExcelError> {
        let reference = ReferenceType::Cell {
            sheet: sheet.map(str::to_string),
            row,
            col,
            row_abs: true,
            col_abs: true,
        };
        let view = self.resolve_range_view(&reference, current_sheet)?;
        Ok(view.as_1x1().unwrap_or(LiteralValue::Empty))
    }

    /// Resolve the effective number-format annotation of a scalar cell read.
    fn resolve_cell_format(
        &self,
        _sheet: Option<&str>,
        _row: u32,
        _col: u32,
        _current_sheet: &str,
    ) -> Option<crate::format::FormatId> {
        None
    }

    /// A cell reference's value and format in one call: exactly
    /// `resolve_cell_reference_value` then `resolve_cell_format` (the
    /// default does that). Contexts that answer both from one lookup
    /// override it; wrappers that record reads need not.
    #[doc(hidden)]
    fn resolve_cell_reference_value_formatted(
        &self,
        sheet: Option<&str>,
        row: u32,
        col: u32,
        current_sheet: &str,
    ) -> Result<(LiteralValue, Option<crate::format::FormatId>), ExcelError> {
        let value = self.resolve_cell_reference_value(sheet, row, col, current_sheet)?;
        Ok((
            value,
            self.resolve_cell_format(sheet, row, col, current_sheet),
        ))
    }

    /// Resolve an interned format id to its reported class.
    fn format_class(
        &self,
        format: crate::format::FormatId,
    ) -> Option<formualizer_common::numfmt::FormatClass> {
        formualizer_common::numfmt::NumberFormat::builtin(format.0)
            .map(|format| format.class().clone())
    }

    /// Record a formula cell's derived scalar format during alternate scalar evaluation paths.
    fn record_cell_derived_format(
        &self,
        _sheet: &str,
        _row: u32,
        _col: u32,
        _format: Option<crate::format::FormatId>,
    ) {
    }

    /// Locale provider: invariant by default
    fn locale(&self) -> crate::locale::Locale {
        crate::locale::Locale::invariant()
    }

    /// Metadata-only binding presence in the current sheet scope. None means
    /// the resolver cannot attest presence or absence; values are never read.
    fn named_binding_presence(&self, _name: &str, _current_sheet: &str) -> Option<bool> {
        None
    }

    /// Number of active sheets in the workbook, if known.
    fn workbook_sheet_count(&self) -> Option<usize> {
        None
    }

    /// Excel-style 1-based active-sheet index for a sheet name, if known.
    fn sheet_index_by_name(&self, _sheet: &str) -> Option<usize> {
        None
    }

    /// Excel-style 1-based active-sheet index for the current formula sheet, if known.
    fn current_sheet_index(&self, current_sheet: &str) -> Option<usize> {
        self.sheet_index_by_name(current_sheet)
    }

    /// Inspect reference metadata without materializing referenced values.
    fn inspect_reference(
        &self,
        _reference: &ReferenceType,
        _current_sheet: &str,
    ) -> Result<Option<ReferenceInfo>, ExcelError> {
        Ok(None)
    }

    /// Whether a concrete cell stores a formula, without requiring its text.
    /// Existing contexts can keep supplying formula text via the fallback.
    fn has_formula_at_cell(&self, cell: CellRef) -> Result<bool, ExcelError> {
        self.formula_text_at_cell(cell).map(|text| text.is_some())
    }

    /// Retrieve formula text for a concrete cell, if that cell stores a formula.
    fn formula_text_at_cell(&self, _cell: CellRef) -> Result<Option<String>, ExcelError> {
        Ok(None)
    }

    /// Resolve a spill-range reference (`A1#`, `ANCHORARRAY(A1)`) to the
    /// anchor's current committed spill rectangle.
    ///
    /// `anchor` is the operand as written, relocated for the current cell:
    /// a single-cell [`ReferenceType::Cell`] or a [`ReferenceType::NamedRange`]
    /// that must name one cell. The result is a real sheet reference, so
    /// callers read it through the ordinary reference path.
    ///
    /// Policy: `#REF!` for any other operand and for an anchor without a
    /// current spill (a value, an empty cell, a scalar result, or a blocked or
    /// oversized spill). The default, for contexts without a spill registry,
    /// is `#REF!`.
    fn resolve_spill_reference(
        &self,
        _anchor: &ReferenceType,
        _current_sheet: &str,
    ) -> Result<ReferenceType, ExcelError> {
        Err(ExcelError::new(ExcelErrorKind::Ref)
            .with_message("Spill references are not available in this context"))
    }

    /// Clock provider for volatile date/time builtins.
    ///
    /// Default when `system-clock` feature is enabled: `SystemClock(Local)` for
    /// Excel-compatible wall-clock behaviour.
    ///
    /// Default when `system-clock` is **disabled** (portable wasm profile): a
    /// UTC epoch `FixedClock`. Implementors that need real wall-clock time should
    /// override this method and inject an appropriate `ClockProvider`.
    fn clock(&self) -> &dyn crate::timezone::ClockProvider {
        #[cfg(feature = "system-clock")]
        {
            static DEFAULT_CLOCK: std::sync::OnceLock<crate::timezone::SystemClock> =
                std::sync::OnceLock::new();
            DEFAULT_CLOCK.get_or_init(|| {
                crate::timezone::SystemClock::new(crate::timezone::TimeZoneSpec::default())
            })
        }
        #[cfg(not(feature = "system-clock"))]
        {
            static DEFAULT_CLOCK: std::sync::OnceLock<crate::timezone::FixedClock> =
                std::sync::OnceLock::new();
            DEFAULT_CLOCK.get_or_init(|| {
                crate::timezone::FixedClock::new(
                    chrono::DateTime::UNIX_EPOCH,
                    crate::timezone::TimeZoneSpec::Utc,
                )
            })
        }
    }

    /// Timezone spec for date/time functions.
    ///
    /// Default: derived from `clock()`.
    fn timezone(&self) -> &crate::timezone::TimeZoneSpec {
        self.clock().timezone()
    }

    /// Volatile granularity. Default Always for backwards compatibility.
    fn volatile_level(&self) -> VolatileLevel {
        VolatileLevel::Always
    }

    /// A stable workbook seed for RNG composition.
    fn workbook_seed(&self) -> u64 {
        0xF0F0_D0D0_AAAA_5555
    }

    /// Recalc epoch that increments on each full recalc when appropriate.
    fn recalc_epoch(&self) -> u64 {
        0
    }

    /* ─────────────── Future-proof IO/backends hooks (default no-op) ─────────────── */

    /// Optional: Return the min/max used rows for a set of columns on a sheet.
    /// When None, the backend does not provide used-region hints.
    fn used_rows_for_columns(
        &self,
        _sheet: &str,
        _start_col: u32,
        _end_col: u32,
    ) -> Option<(u32, u32)> {
        None
    }

    /// Optional: Return the min/max used columns for a set of rows on a sheet.
    /// When None, the backend does not provide used-region hints.
    fn used_cols_for_rows(
        &self,
        _sheet: &str,
        _start_row: u32,
        _end_row: u32,
    ) -> Option<(u32, u32)> {
        None
    }

    /// Optional: Physical sheet bounds (max rows, max cols) if known.
    fn sheet_bounds(&self, _sheet: &str) -> Option<(u32, u32)> {
        None
    }

    /// Monotonic identifier for the current data snapshot; increments on mutation.
    fn data_snapshot_id(&self) -> u64 {
        0
    }

    /// Backend capability advertisement for IO/adapters.
    fn backend_caps(&self) -> BackendCaps {
        BackendCaps::default()
    }

    // Flats removed

    /// Workbook date system selection (1900 vs 1904).
    /// Defaults to 1900 for compatibility.
    fn date_system(&self) -> crate::engine::DateSystem {
        crate::engine::DateSystem::Excel1900
    }

    /// Optional: Build or fetch an exact-match lookup index over an Arrow-backed view.
    /// Implementations should return None if not supported or unsafe.
    fn build_lookup_index(
        &self,
        _view: &RangeView<'_>,
        _axis: LookupAxis,
    ) -> Option<std::sync::Arc<LookupIndex>> {
        None
    }

    /// Optional: Build or fetch a cached boolean mask for a criterion over an Arrow-backed view.
    /// Implementations should return None if not supported.
    fn build_criteria_mask(
        &self,
        _view: &RangeView<'_>,
        _col_in_view: usize,
        _pred: &crate::args::CriteriaPredicate,
    ) -> Option<std::sync::Arc<arrow_array::BooleanArray>> {
        None
    }

    /// Optional: Build row-visibility mask aligned to `view` rows.
    /// Returns None if not supported by the underlying context.
    fn build_row_visibility_mask(
        &self,
        _view: &RangeView<'_>,
        _mode: VisibilityMaskMode,
    ) -> Option<std::sync::Arc<arrow_array::BooleanArray>> {
        None
    }

    /// Optional: the cells of a sheet-backed `view` whose formulas call
    /// SUBTOTAL (and AGGREGATE when `include_aggregate`), as sorted,
    /// disjoint `(column, first_row, last_row)` offset intervals within the
    /// view (rows inclusive). SUBTOTAL and AGGREGATE skip
    /// these cells (Excel ignores nested subtotals). `None` when the context
    /// does not track formulas.
    fn nested_subtotal_cells(
        &self,
        _view: &RangeView<'_>,
        _include_aggregate: bool,
    ) -> Option<Vec<(usize, usize, usize)>> {
        None
    }
}

/// Minimal backend capability descriptor for planning and adapters.
#[derive(Copy, Clone, Debug, Default)]
pub struct BackendCaps {
    /// Provides lazy access (// TODO REMOVE?)
    pub streaming: bool,
    /// Can compute used-region for rows/columns
    pub used_region: bool,
    /// Supports write-back mutations via external sink
    pub write: bool,
    /// Provides table metadata/streaming beyond basic column access
    pub tables: bool,
    /// May provide asynchronous/lazy remote streams (reserved)
    pub async_stream: bool,
}

/* ───────────────────── FunctionContext (narrow) ───────────────────── */

#[derive(Copy, Clone, Debug, Eq, PartialEq)]
pub enum VolatileLevel {
    /// Value can change at any edit; seed excludes recalc_epoch by default.
    Always,
    /// Value changes per recalculation; seed should include recalc_epoch.
    OnRecalc,
    /// Value changes per open; seed uses only workbook_seed.
    OnOpen,
}

/// Minimal context exposed to functions (no engine/graph APIs)
pub trait FunctionContext<'ctx> {
    /// Emit typed evidence on an executed unsupported path. Do not report a
    /// propagated input error as a new operation. No effect unless enabled.
    fn report_unavailable_operation(
        &self,
        _operation: crate::operation_diagnostics::UnavailableOperation,
    ) {
    }

    fn locale(&self) -> crate::locale::Locale;
    fn timezone(&self) -> &crate::timezone::TimeZoneSpec;
    fn clock(&self) -> &dyn crate::timezone::ClockProvider;
    fn thread_pool(&self) -> Option<&std::sync::Arc<rayon::ThreadPool>>;
    /// Returns the optional shared cancellation handle for this evaluation.
    ///
    /// Custom function authors should retrieve this once before a hot loop and
    /// poll [`crate::engine::CancelToken::is_cancelled`] periodically. Cloning
    /// the handle shares the same signal without allocating.
    fn cancellation_token(&self) -> Option<crate::engine::CancelToken>;
    fn chunk_hint(&self) -> Option<usize>;

    /// Most cells a range or array value may have when it is materialised as
    /// a dense array; larger arrays evaluate to `#NUM!`. See
    /// [`EvaluationContext::materialized_array_cells`].
    fn materialized_array_cells(&self) -> u64 {
        crate::engine::resource_ledger::DEFAULT_MATERIALIZED_ARRAY_CELLS
    }

    /// Current formula sheet name.
    fn current_sheet(&self) -> &str;

    fn workbook_sheet_count(&self) -> Option<usize> {
        None
    }

    fn sheet_index_by_name(&self, _sheet: &str) -> Option<usize> {
        None
    }

    fn current_sheet_index(&self) -> Option<usize> {
        self.sheet_index_by_name(self.current_sheet())
    }

    fn inspect_reference(
        &self,
        _reference: &ReferenceType,
    ) -> Result<Option<ReferenceInfo>, ExcelError> {
        Ok(None)
    }

    /// Whether a concrete cell stores a formula. The default preserves
    /// contexts that provide only formula text.
    fn has_formula_at_cell(&self, cell: CellRef) -> Result<bool, ExcelError> {
        self.formula_text_at_cell(cell).map(|text| text.is_some())
    }

    fn formula_text_at_cell(&self, _cell: CellRef) -> Result<Option<String>, ExcelError> {
        Ok(None)
    }

    fn volatile_level(&self) -> VolatileLevel;
    fn workbook_seed(&self) -> u64;
    fn recalc_epoch(&self) -> u64;
    fn current_cell(&self) -> Option<CellRef>;

    /// Resolve a reference into a RangeView using the underlying engine context.
    fn resolve_range_view(
        &self,
        _reference: &ReferenceType,
        _current_sheet: &str,
    ) -> Result<RangeView<'ctx>, ExcelError>;

    // Flats removed

    /// Deterministic RNG seeded for the current evaluation site and function salt.
    fn rng_for_current(&self, fn_salt: u64) -> rand::rngs::SmallRng {
        use crate::rng::{compose_seed, small_rng_from_lanes};
        let (sheet_id, row, col) = self
            .current_cell()
            .map(|c| (c.sheet_id as u32, c.coord.row(), c.coord.col()))
            .unwrap_or((0, 0, 0));
        // Include epoch only for OnRecalc
        let epoch = match self.volatile_level() {
            VolatileLevel::OnRecalc => self.recalc_epoch(),
            _ => 0,
        };
        let (l0, l1) = compose_seed(self.workbook_seed(), sheet_id, row, col, fn_salt, epoch);
        small_rng_from_lanes(l0, l1)
    }

    /// Workbook date system selection (1900 vs 1904).
    fn date_system(&self) -> crate::engine::DateSystem {
        crate::engine::DateSystem::Excel1900
    }

    /// Optional: Build or fetch an exact-match lookup index over an Arrow-backed view.
    /// Returns None if not supported by the underlying context.
    fn get_lookup_index(
        &self,
        _view: &RangeView<'_>,
        _axis: LookupAxis,
    ) -> Option<std::sync::Arc<LookupIndex>> {
        None
    }

    /// Optional: Build or fetch a cached boolean mask for a criterion over an Arrow-backed view.
    /// Returns None if not supported by the underlying context.
    fn get_criteria_mask(
        &self,
        _view: &RangeView<'_>,
        _col_in_view: usize,
        _pred: &crate::args::CriteriaPredicate,
    ) -> Option<std::sync::Arc<arrow_array::BooleanArray>> {
        None
    }

    /// Optional: Build row-visibility mask aligned to `view` rows.
    fn get_row_visibility_mask(
        &self,
        _view: &RangeView<'_>,
        _mode: VisibilityMaskMode,
    ) -> Option<std::sync::Arc<arrow_array::BooleanArray>> {
        None
    }

    /// Optional: cells of `view` holding nested SUBTOTAL (and AGGREGATE when
    /// `include_aggregate`) formulas; see
    /// [`EvaluationContext::nested_subtotal_cells`].
    fn get_nested_subtotal_cells(
        &self,
        _view: &RangeView<'_>,
        _include_aggregate: bool,
    ) -> Option<Vec<(usize, usize, usize)>> {
        None
    }
}

/// Default adapter that wraps an EvaluationContext and provides the narrow FunctionContext.
pub struct DefaultFunctionContext<'a> {
    pub base: &'a dyn EvaluationContext,
    pub current: Option<CellRef>,
    pub current_sheet: &'a str,
}

impl<'a> DefaultFunctionContext<'a> {
    pub fn new(
        base: &'a dyn EvaluationContext,
        current: Option<CellRef>,
        current_sheet: &'a str,
    ) -> Self {
        Self {
            base,
            current,
            current_sheet,
        }
    }

    pub fn new_with_sheet(
        base: &'a dyn EvaluationContext,
        current: Option<CellRef>,
        current_sheet: &'a str,
    ) -> Self {
        Self::new(base, current, current_sheet)
    }
}

impl<'a> FunctionContext<'a> for DefaultFunctionContext<'a> {
    fn report_unavailable_operation(
        &self,
        operation: crate::operation_diagnostics::UnavailableOperation,
    ) {
        if let Some(collector) = self.base.operation_diagnostic_collector() {
            collector.record(self.current, operation);
        }
    }

    fn locale(&self) -> crate::locale::Locale {
        self.base.locale()
    }

    fn current_sheet(&self) -> &str {
        self.current_sheet
    }

    fn workbook_sheet_count(&self) -> Option<usize> {
        self.base.workbook_sheet_count()
    }

    fn sheet_index_by_name(&self, sheet: &str) -> Option<usize> {
        self.base.sheet_index_by_name(sheet)
    }

    fn current_sheet_index(&self) -> Option<usize> {
        self.base.current_sheet_index(self.current_sheet)
    }

    fn inspect_reference(
        &self,
        reference: &ReferenceType,
    ) -> Result<Option<ReferenceInfo>, ExcelError> {
        self.base.inspect_reference(reference, self.current_sheet)
    }

    fn has_formula_at_cell(&self, cell: CellRef) -> Result<bool, ExcelError> {
        self.base.has_formula_at_cell(cell)
    }

    fn formula_text_at_cell(&self, cell: CellRef) -> Result<Option<String>, ExcelError> {
        self.base.formula_text_at_cell(cell)
    }

    fn timezone(&self) -> &crate::timezone::TimeZoneSpec {
        self.base.timezone()
    }

    fn clock(&self) -> &dyn crate::timezone::ClockProvider {
        self.base.clock()
    }
    fn thread_pool(&self) -> Option<&std::sync::Arc<rayon::ThreadPool>> {
        self.base.thread_pool()
    }
    fn cancellation_token(&self) -> Option<crate::engine::CancelToken> {
        self.base.cancellation_token()
    }
    fn chunk_hint(&self) -> Option<usize> {
        self.base.chunk_hint()
    }
    fn materialized_array_cells(&self) -> u64 {
        self.base.materialized_array_cells()
    }

    fn volatile_level(&self) -> VolatileLevel {
        self.base.volatile_level()
    }
    fn workbook_seed(&self) -> u64 {
        self.base.workbook_seed()
    }
    fn recalc_epoch(&self) -> u64 {
        self.base.recalc_epoch()
    }
    fn current_cell(&self) -> Option<CellRef> {
        self.current
    }

    fn resolve_range_view(
        &self,
        reference: &ReferenceType,
        current_sheet: &str,
    ) -> Result<RangeView<'a>, ExcelError> {
        self.base
            .resolve_range_view(reference, current_sheet)
            .map(|view| view.with_cell_budget(self.base.materialized_array_cells()))
    }

    // Flats removed

    fn date_system(&self) -> crate::engine::DateSystem {
        self.base.date_system()
    }

    fn get_lookup_index(
        &self,
        view: &RangeView<'_>,
        axis: LookupAxis,
    ) -> Option<std::sync::Arc<LookupIndex>> {
        self.base.build_lookup_index(view, axis)
    }

    fn get_criteria_mask(
        &self,
        view: &RangeView<'_>,
        col_in_view: usize,
        pred: &crate::args::CriteriaPredicate,
    ) -> Option<std::sync::Arc<arrow_array::BooleanArray>> {
        self.base.build_criteria_mask(view, col_in_view, pred)
    }

    fn get_row_visibility_mask(
        &self,
        view: &RangeView<'_>,
        mode: VisibilityMaskMode,
    ) -> Option<std::sync::Arc<arrow_array::BooleanArray>> {
        self.base.build_row_visibility_mask(view, mode)
    }

    fn get_nested_subtotal_cells(
        &self,
        view: &RangeView<'_>,
        include_aggregate: bool,
    ) -> Option<Vec<(usize, usize, usize)>> {
        self.base.nested_subtotal_cells(view, include_aggregate)
    }
}

#[cfg(test)]
mod argument_handle_size_tests {
    use super::*;
    #[expect(dead_code, reason = "layout-only regression fixture")]
    struct LegacyArgumentHandle<'a, 'b> {
        expr: ArgumentExpr<'a>,
        interp: &'a Interpreter<'b>,
        cached_ast: std::cell::OnceCell<ASTNode>,
        cached_ref: std::cell::OnceCell<ReferenceType>,
        cached_reference_or_value:
            std::cell::OnceCell<Result<crate::function::FunctionResolution<'b>, ExcelError>>,
        cached_resolved: std::cell::OnceCell<Result<ResolvedArgument<'b>, ExcelError>>,
        /// Memoized result of [`Self::value`]. `Function::dispatch` evaluates
        /// every argument once during schema validation and the function's `eval`
        /// evaluates it again — without this cache that re-entry compounds to
        /// 2^depth evaluations of the innermost node for nested non-short-circuit
        /// calls (measured: depth 12 ⇒ 4096 evaluations). The handle is created
        /// per call site and per evaluation, so the memo can never go stale
        /// across recalcs. `value_with_env` is intentionally NOT memoized (the
        /// local env changes the result).
        cached_value: std::cell::OnceCell<Result<crate::traits::CalcValue<'b>, ExcelError>>,
    }

    #[test]
    fn argument_handle_fits_inline_call_storage() {
        let before = std::mem::size_of::<LegacyArgumentHandle<'_, '_>>();
        let after = std::mem::size_of::<ArgumentHandle<'_, '_>>();
        assert_eq!(after, 80, "ArgumentHandle layout changed");
        assert!(after < before / 4, "cold caches must stay out of line");
    }
}