Skip to main content

harn_vm/vm/
dispatch.rs

1use std::future::Future;
2use std::rc::Rc;
3
4use crate::value::{ErrorCategory, VmBuiltinFn, VmClosure, VmError, VmValue};
5use crate::BuiltinId;
6
7use super::{
8    CallArgs, ScopeSpan, Vm, VmBuiltinArity, VmBuiltinDispatch, VmBuiltinEntry, VmBuiltinKind,
9    VmBuiltinMetadata,
10};
11
12impl Vm {
13    fn builtin_span_kind(name: &str) -> Option<crate::tracing::SpanKind> {
14        match name {
15            "llm_call" | "llm_stream" | "llm_stream_call" | "agent_loop" | "agent_turn" => {
16                Some(crate::tracing::SpanKind::LlmCall)
17            }
18            "mcp_call" => Some(crate::tracing::SpanKind::ToolCall),
19            _ => None,
20        }
21    }
22
23    fn is_runtime_context_builtin(name: &str) -> bool {
24        matches!(
25            name,
26            "runtime_context"
27                | "task_current"
28                | "runtime_context_values"
29                | "runtime_context_get"
30                | "runtime_context_set"
31                | "runtime_context_clear"
32        )
33    }
34
35    fn resolve_sync_builtin_id_or_name(
36        &self,
37        direct_id: Option<BuiltinId>,
38        name: &str,
39    ) -> Option<Result<VmBuiltinFn, VmError>> {
40        if crate::autonomy::needs_async_side_effect_enforcement(name)
41            || Self::is_runtime_context_builtin(name)
42        {
43            return None;
44        }
45
46        let dispatch = if let Some(id) = direct_id {
47            self.builtins_by_id
48                .get(&id)
49                .filter(|entry| entry.name.as_ref() == name)
50                .map(|entry| entry.dispatch.clone())
51        } else {
52            None
53        }
54        .or_else(|| {
55            self.builtins
56                .get(name)
57                .cloned()
58                .map(VmBuiltinDispatch::Sync)
59        });
60
61        let Some(dispatch) = dispatch else {
62            if self.async_builtins.contains_key(name) || self.bridge.is_some() {
63                return None;
64            }
65            let all_builtins = self
66                .builtins
67                .keys()
68                .chain(self.async_builtins.keys())
69                .map(|s| s.as_str());
70            return Some(
71                if let Some(suggestion) = crate::value::closest_match(name, all_builtins) {
72                    Err(VmError::Runtime(format!(
73                        "Undefined builtin: {name} (did you mean `{suggestion}`?)"
74                    )))
75                } else {
76                    Err(VmError::UndefinedBuiltin(name.to_string()))
77                },
78            );
79        };
80
81        match dispatch {
82            VmBuiltinDispatch::Sync(builtin) => Some(Ok(builtin)),
83            VmBuiltinDispatch::Async(_) => None,
84        }
85    }
86
87    fn validate_sync_builtin_args(&self, name: &str, args: &[VmValue]) -> Result<(), VmError> {
88        if self.denied_builtins.contains(name) {
89            return Err(VmError::CategorizedError {
90                message: format!("Tool '{name}' is not permitted."),
91                category: ErrorCategory::ToolRejected,
92            });
93        }
94        crate::orchestration::enforce_current_policy_for_builtin(name, args)?;
95        crate::typecheck::validate_builtin_call(name, args, None)
96    }
97
98    fn index_builtin_id(&mut self, name: &str, dispatch: VmBuiltinDispatch) {
99        let id = BuiltinId::from_name(name);
100        if self.builtin_id_collisions.contains(&id) {
101            return;
102        }
103        if let Some(existing) = self.builtins_by_id.get(&id) {
104            if existing.name.as_ref() != name {
105                Rc::make_mut(&mut self.builtins_by_id).remove(&id);
106                Rc::make_mut(&mut self.builtin_id_collisions).insert(id);
107                return;
108            }
109        }
110        Rc::make_mut(&mut self.builtins_by_id).insert(
111            id,
112            VmBuiltinEntry {
113                name: Rc::from(name),
114                dispatch,
115            },
116        );
117    }
118
119    fn refresh_builtin_id(&mut self, name: &str) {
120        if let Some(builtin) = self.builtins.get(name).cloned() {
121            self.index_builtin_id(name, VmBuiltinDispatch::Sync(builtin));
122        } else if let Some(async_builtin) = self.async_builtins.get(name).cloned() {
123            self.index_builtin_id(name, VmBuiltinDispatch::Async(async_builtin));
124        } else {
125            let id = BuiltinId::from_name(name);
126            if self
127                .builtins_by_id
128                .get(&id)
129                .is_some_and(|entry| entry.name.as_ref() == name)
130            {
131                Rc::make_mut(&mut self.builtins_by_id).remove(&id);
132            }
133        }
134    }
135
136    /// Register a sync builtin function.
137    pub fn register_builtin<F>(&mut self, name: &str, f: F)
138    where
139        F: Fn(&[VmValue], &mut String) -> Result<VmValue, VmError> + 'static,
140    {
141        Rc::make_mut(&mut self.builtins).insert(name.to_string(), Rc::new(f));
142        Rc::make_mut(&mut self.builtin_metadata)
143            .insert(name.to_string(), VmBuiltinMetadata::sync(name.to_string()));
144        self.refresh_builtin_id(name);
145    }
146
147    /// Register a sync builtin function with discoverable metadata.
148    pub fn register_builtin_with_metadata<F>(&mut self, metadata: VmBuiltinMetadata, f: F)
149    where
150        F: Fn(&[VmValue], &mut String) -> Result<VmValue, VmError> + 'static,
151    {
152        let name = metadata.name().to_string();
153        Rc::make_mut(&mut self.builtins).insert(name.clone(), Rc::new(f));
154        Rc::make_mut(&mut self.builtin_metadata)
155            .insert(name.clone(), metadata.with_kind(VmBuiltinKind::Sync));
156        self.refresh_builtin_id(&name);
157    }
158
159    /// Register a `VmBuiltinDef` (the shape emitted by `#[harn_builtin]`).
160    /// Registers the primary name plus each declared alias, sharing the
161    /// same handler. `runtime_only` defs skip the parser-side publish (the
162    /// vm-side registration still happens). `parser_only` defs skip the
163    /// vm-side registration entirely (handler is `None`).
164    pub fn register_builtin_def(&mut self, def: &'static crate::stdlib::macros::VmBuiltinDef) {
165        use crate::stdlib::macros::VmBuiltinHandler;
166        if def.parser_only {
167            return;
168        }
169        // Derive arity from the parsed `BuiltinSignature` so the discoverable
170        // metadata layer (harn explain, alignment-test metadata check) keeps
171        // parity with the pre-macro DSL builder.
172        let arity = arity_from_sig(&def.sig);
173        let names = std::iter::once(def.sig.name).chain(def.aliases.iter().copied());
174        for name in names {
175            match def.handler {
176                VmBuiltinHandler::Sync(f) => {
177                    let meta = builtin_def_metadata(def, name, arity, VmBuiltinKind::Sync);
178                    self.register_builtin_with_metadata(meta, f);
179                }
180                VmBuiltinHandler::Async(f) => {
181                    let meta = builtin_def_metadata(def, name, arity, VmBuiltinKind::Async);
182                    // Wrap the function pointer that already returns an
183                    // AsyncBuiltinFuture so register_async_builtin_with_metadata's
184                    // generic bound `F: Fn(Vec<VmValue>) -> Fut + 'static` is met.
185                    self.register_async_builtin_with_metadata(meta, f);
186                }
187                VmBuiltinHandler::None => {
188                    // Parser-only, but reached here despite parser_only=false.
189                    // This is a configuration bug.
190                    panic!(
191                        "VmBuiltinHandler::None for {name:?} without parser_only=true \
192                         on its BuiltinDef"
193                    );
194                }
195            }
196        }
197    }
198
199    /// Remove a sync builtin (so an async version can take precedence).
200    pub fn unregister_builtin(&mut self, name: &str) {
201        Rc::make_mut(&mut self.builtins).remove(name);
202        if self.async_builtins.contains_key(name) {
203            Rc::make_mut(&mut self.builtin_metadata).insert(
204                name.to_string(),
205                VmBuiltinMetadata::async_builtin(name.to_string()),
206            );
207        } else {
208            Rc::make_mut(&mut self.builtin_metadata).remove(name);
209        }
210        self.refresh_builtin_id(name);
211    }
212
213    /// Register an async builtin function. The handler receives the explicit
214    /// [`crate::vm::AsyncBuiltinCtx`] threaded by the dispatch loop.
215    pub fn register_async_builtin<F, Fut>(&mut self, name: &str, f: F)
216    where
217        F: Fn(crate::vm::AsyncBuiltinCtx, Vec<VmValue>) -> Fut + 'static,
218        Fut: Future<Output = Result<VmValue, VmError>> + 'static,
219    {
220        Rc::make_mut(&mut self.async_builtins).insert(
221            name.to_string(),
222            Rc::new(move |ctx, args| Box::pin(f(ctx, args))),
223        );
224        Rc::make_mut(&mut self.builtin_metadata).insert(
225            name.to_string(),
226            VmBuiltinMetadata::async_builtin(name.to_string()),
227        );
228        self.refresh_builtin_id(name);
229    }
230
231    /// Register an async builtin function with discoverable metadata. The
232    /// handler receives the explicit [`crate::vm::AsyncBuiltinCtx`].
233    pub fn register_async_builtin_with_metadata<F, Fut>(
234        &mut self,
235        metadata: VmBuiltinMetadata,
236        f: F,
237    ) where
238        F: Fn(crate::vm::AsyncBuiltinCtx, Vec<VmValue>) -> Fut + 'static,
239        Fut: Future<Output = Result<VmValue, VmError>> + 'static,
240    {
241        let name = metadata.name().to_string();
242        Rc::make_mut(&mut self.async_builtins).insert(
243            name.clone(),
244            Rc::new(move |ctx, args| Box::pin(f(ctx, args))),
245        );
246        Rc::make_mut(&mut self.builtin_metadata)
247            .insert(name.clone(), metadata.with_kind(VmBuiltinKind::Async));
248        self.refresh_builtin_id(&name);
249    }
250
251    pub(crate) fn registered_builtin_id(&self, name: &str) -> Option<BuiltinId> {
252        let id = BuiltinId::from_name(name);
253        if self
254            .builtins_by_id
255            .get(&id)
256            .is_some_and(|entry| entry.name.as_ref() == name)
257        {
258            Some(id)
259        } else {
260            None
261        }
262    }
263
264    /// Invoke a closure inline against the existing VM frame stack.
265    ///
266    /// Dispatch path for every callback-taking method on lists/dicts/sets
267    /// (`.map`, `.filter`, `.reduce`, `.each`, `.sort_by`, …) via
268    /// [`call_callable_value`]. The closure's frame is pushed onto
269    /// `self.frames` using the same machinery as `Op::Call`, and the
270    /// shared dispatch loop ([`Vm::drive_until_frame_depth`]) drains the
271    /// sub-execution back to the caller's depth.
272    ///
273    /// This avoids the per-invocation `Pin<Box<dyn Future>>` heap
274    /// allocation a recursive `async fn` would require — the recursion
275    /// cycle (closure → `.map` → callback → closure) is broken instead at
276    /// [`Vm::call_method`], which keeps a single boxed future per
277    /// method-call site rather than per callback element.
278    ///
279    /// Exception handlers are saved and cleared before the sub-execution
280    /// so an unhandled throw inside the body propagates as a Rust
281    /// `Result::Err` to the caller's dispatch loop. Iterators, deadlines,
282    /// and frames are scoped by `CallFrame::saved_iterator_depth` and the
283    /// per-frame deadline tags.
284    pub(crate) async fn call_closure(
285        &mut self,
286        closure: &VmClosure,
287        args: &[VmValue],
288    ) -> Result<VmValue, VmError> {
289        self.call_closure_args(closure, CallArgs::Slice(args)).await
290    }
291
292    pub(crate) async fn call_closure_args(
293        &mut self,
294        closure: &VmClosure,
295        args: CallArgs<'_>,
296    ) -> Result<VmValue, VmError> {
297        let saved_handlers = std::mem::take(&mut self.exception_handlers);
298        let active_context = (!crate::step_runtime::is_tracked_function(&closure.func.name))
299            .then(crate::step_runtime::take_active_context);
300
301        let target_frame_depth = self.frames.len();
302        let frame_result = self.push_closure_frame_args(closure, &args);
303        drop(args);
304        let result = match frame_result {
305            Ok(()) => self.drive_until_frame_depth(target_frame_depth).await,
306            Err(e) => Err(e),
307        };
308
309        self.exception_handlers = saved_handlers;
310        if let Some(ctx) = active_context {
311            crate::step_runtime::restore_active_context(ctx);
312        }
313
314        result
315    }
316
317    /// Invoke a value as a callable. Supports `VmValue::Closure` and
318    /// `VmValue::BuiltinRef`, so builtin names passed by reference (e.g.
319    /// `dict.rekey(snake_to_camel)`) dispatch through the same code path as
320    /// user-defined closures.
321    pub(crate) async fn call_callable_value(
322        &mut self,
323        callable: &VmValue,
324        args: &[VmValue],
325    ) -> Result<VmValue, VmError> {
326        self.call_callable_args(callable, CallArgs::Slice(args))
327            .await
328    }
329
330    pub(crate) async fn call_callable_owned(
331        &mut self,
332        callable: &VmValue,
333        args: Vec<VmValue>,
334    ) -> Result<VmValue, VmError> {
335        self.call_callable_args(callable, CallArgs::Owned(args))
336            .await
337    }
338
339    pub(crate) async fn call_callable_zero(
340        &mut self,
341        callable: &VmValue,
342    ) -> Result<VmValue, VmError> {
343        self.call_callable_args(callable, CallArgs::Empty).await
344    }
345
346    pub(crate) async fn call_callable_one(
347        &mut self,
348        callable: &VmValue,
349        arg: &VmValue,
350    ) -> Result<VmValue, VmError> {
351        self.call_callable_args(callable, CallArgs::One(arg)).await
352    }
353
354    pub(crate) async fn call_callable_two(
355        &mut self,
356        callable: &VmValue,
357        first: &VmValue,
358        second: &VmValue,
359    ) -> Result<VmValue, VmError> {
360        self.call_callable_args(callable, CallArgs::Two(first, second))
361            .await
362    }
363
364    pub(crate) async fn call_callable_args(
365        &mut self,
366        callable: &VmValue,
367        args: CallArgs<'_>,
368    ) -> Result<VmValue, VmError> {
369        match callable {
370            VmValue::Closure(closure) => self.call_closure_args(closure, args).await,
371            VmValue::BuiltinRef(name) => {
372                if !crate::autonomy::needs_async_side_effect_enforcement(name) {
373                    if let Some(result) = self.call_sync_builtin_by_ref_args(name, &args) {
374                        return result;
375                    }
376                }
377                self.call_named_builtin(name, args.into_vec()).await
378            }
379            VmValue::BuiltinRefId { id, name } => {
380                if let Some(result) =
381                    self.try_call_sync_builtin_id_or_name_args(Some(*id), name, &args)
382                {
383                    return result;
384                }
385                self.call_builtin_id_or_name(*id, name, args.into_vec())
386                    .await
387            }
388            other => Err(VmError::TypeError(format!(
389                "expected callable, got {}",
390                other.type_name()
391            ))),
392        }
393    }
394
395    fn call_sync_builtin_by_ref_args(
396        &mut self,
397        name: &str,
398        args: &CallArgs<'_>,
399    ) -> Option<Result<VmValue, VmError>> {
400        self.try_call_sync_builtin_id_or_name_args(None, name, args)
401    }
402
403    /// Returns true if `v` is callable via `call_callable_value`.
404    pub(crate) fn is_callable_value(v: &VmValue) -> bool {
405        matches!(
406            v,
407            VmValue::Closure(_) | VmValue::BuiltinRef(_) | VmValue::BuiltinRefId { .. }
408        )
409    }
410
411    /// Public wrapper for `call_closure`, used by the MCP server to invoke
412    /// tool handler closures from outside the VM execution loop.
413    pub async fn call_closure_pub(
414        &mut self,
415        closure: &VmClosure,
416        args: &[VmValue],
417    ) -> Result<VmValue, VmError> {
418        self.cancel_grace_instructions_remaining = None;
419        self.call_closure(closure, args).await
420    }
421
422    /// Resolve a named builtin: sync builtins → async builtins → bridge → error.
423    /// Used by Call, TailCall, and Pipe handlers to avoid duplicating this lookup.
424    pub(crate) async fn call_named_builtin(
425        &mut self,
426        name: &str,
427        args: Vec<VmValue>,
428    ) -> Result<VmValue, VmError> {
429        self.call_builtin_impl(name, args, None).await
430    }
431
432    pub(crate) async fn call_builtin_id_or_name(
433        &mut self,
434        id: BuiltinId,
435        name: &str,
436        args: Vec<VmValue>,
437    ) -> Result<VmValue, VmError> {
438        self.call_builtin_impl(name, args, Some(id)).await
439    }
440
441    pub(crate) fn try_call_sync_builtin_id_or_name_args(
442        &mut self,
443        direct_id: Option<BuiltinId>,
444        name: &str,
445        args: &CallArgs<'_>,
446    ) -> Option<Result<VmValue, VmError>> {
447        if self.denied_builtins.contains(name) {
448            return Some(Err(VmError::CategorizedError {
449                message: format!("Tool '{name}' is not permitted."),
450                category: ErrorCategory::ToolRejected,
451            }));
452        }
453        let builtin = match self.resolve_sync_builtin_id_or_name(direct_id, name)? {
454            Ok(builtin) => builtin,
455            Err(error) => return Some(Err(error)),
456        };
457        let _span =
458            Self::builtin_span_kind(name).map(|kind| ScopeSpan::new(kind, name.to_string()));
459        if let Err(error) = args.with_slice(|slice| self.validate_sync_builtin_args(name, slice)) {
460            return Some(Err(error));
461        }
462
463        Some(args.with_slice(|slice| builtin(slice, &mut self.output)))
464    }
465
466    pub(crate) fn try_call_sync_builtin_id_or_name_from_stack_args(
467        &mut self,
468        direct_id: Option<BuiltinId>,
469        name: &str,
470        args_start: usize,
471    ) -> Option<Result<VmValue, VmError>> {
472        if self.denied_builtins.contains(name) {
473            return Some(Err(VmError::CategorizedError {
474                message: format!("Tool '{name}' is not permitted."),
475                category: ErrorCategory::ToolRejected,
476            }));
477        }
478        let builtin = match self.resolve_sync_builtin_id_or_name(direct_id, name)? {
479            Ok(builtin) => builtin,
480            Err(error) => return Some(Err(error)),
481        };
482        if args_start > self.stack.len() {
483            return Some(Err(VmError::Runtime(
484                "call argument stack underflow".to_string(),
485            )));
486        }
487
488        let _span =
489            Self::builtin_span_kind(name).map(|kind| ScopeSpan::new(kind, name.to_string()));
490        let args = &self.stack[args_start..];
491        if let Err(error) = self.validate_sync_builtin_args(name, args) {
492            return Some(Err(error));
493        }
494
495        Some(builtin(args, &mut self.output))
496    }
497
498    async fn call_builtin_impl(
499        &mut self,
500        name: &str,
501        args: Vec<VmValue>,
502        direct_id: Option<BuiltinId>,
503    ) -> Result<VmValue, VmError> {
504        // Auto-trace LLM calls and tool calls.
505        let _span =
506            Self::builtin_span_kind(name).map(|kind| ScopeSpan::new(kind, name.to_string()));
507
508        // Sandbox check: deny builtins blocked by --deny/--allow flags.
509        if self.denied_builtins.contains(name) {
510            return Err(VmError::CategorizedError {
511                message: format!("Tool '{name}' is not permitted."),
512                category: ErrorCategory::ToolRejected,
513            });
514        }
515        let autonomy = if crate::autonomy::needs_async_side_effect_enforcement(name) {
516            crate::autonomy::enforce_builtin_side_effect_boxed(name, &args).await?
517        } else {
518            None
519        };
520        if let Some(crate::autonomy::AutonomyDecision::Skip(value)) = autonomy {
521            return Ok(value);
522        }
523        if !matches!(
524            autonomy,
525            Some(crate::autonomy::AutonomyDecision::AllowApproved)
526        ) {
527            crate::orchestration::enforce_current_policy_for_builtin(name, &args)?;
528        }
529        crate::typecheck::validate_builtin_call(name, &args, None)?;
530
531        if let Some(result) =
532            crate::runtime_context::dispatch_runtime_context_builtin(self, name, &args)
533        {
534            return result;
535        }
536
537        if let Some(id) = direct_id {
538            if let Some(entry) = self.builtins_by_id.get(&id).cloned() {
539                if entry.name.as_ref() == name {
540                    return self.call_builtin_entry(entry.dispatch, args).await;
541                }
542            }
543        }
544
545        if let Some(builtin) = self.builtins.get(name).cloned() {
546            self.call_builtin_entry(VmBuiltinDispatch::Sync(builtin), args)
547                .await
548        } else if let Some(async_builtin) = self.async_builtins.get(name).cloned() {
549            self.call_builtin_entry(VmBuiltinDispatch::Async(async_builtin), args)
550                .await
551        } else if let Some(bridge) = &self.bridge {
552            crate::orchestration::enforce_current_policy_for_bridge_builtin(name)?;
553            let args_json: Vec<serde_json::Value> =
554                args.iter().map(crate::llm::vm_value_to_json).collect();
555            let result = bridge
556                .call(
557                    "builtin_call",
558                    serde_json::json!({"name": name, "args": args_json}),
559                )
560                .await?;
561            Ok(crate::bridge::json_result_to_vm_value(&result))
562        } else {
563            let all_builtins = self
564                .builtins
565                .keys()
566                .chain(self.async_builtins.keys())
567                .map(|s| s.as_str());
568            if let Some(suggestion) = crate::value::closest_match(name, all_builtins) {
569                return Err(VmError::Runtime(format!(
570                    "Undefined builtin: {name} (did you mean `{suggestion}`?)"
571                )));
572            }
573            Err(VmError::UndefinedBuiltin(name.to_string()))
574        }
575    }
576
577    async fn call_builtin_entry(
578        &mut self,
579        dispatch: VmBuiltinDispatch,
580        args: Vec<VmValue>,
581    ) -> Result<VmValue, VmError> {
582        match dispatch {
583            VmBuiltinDispatch::Sync(builtin) => builtin(&args, &mut self.output),
584            VmBuiltinDispatch::Async(async_builtin) => {
585                // Bind a fresh child VM as the async-builtin context for the
586                // duration of this future, threading the explicit ctx handle
587                // into the handler. Drain any output VM-side closures
588                // forwarded into the ctx back to the parent.
589                let (result, captured) =
590                    crate::vm::run_async_builtin_with(self.child_vm(), |ctx| {
591                        async_builtin(ctx, args)
592                    })
593                    .await;
594                if !captured.is_empty() {
595                    self.output.push_str(&captured);
596                }
597                result
598            }
599        }
600    }
601}
602
603/// Build the discoverable [`VmBuiltinMetadata`] for one entry of a
604/// `#[harn_builtin]`-emitted `VmBuiltinDef`, threading the optional
605/// category / doc / signature_text fields without duplicating the chain
606/// across the Sync / Async dispatch arms in `register_builtin_def`.
607fn builtin_def_metadata(
608    def: &'static crate::stdlib::macros::VmBuiltinDef,
609    name: &'static str,
610    arity: VmBuiltinArity,
611    kind: VmBuiltinKind,
612) -> VmBuiltinMetadata {
613    let mut meta = match kind {
614        VmBuiltinKind::Sync => VmBuiltinMetadata::sync_static(name),
615        VmBuiltinKind::Async => VmBuiltinMetadata::async_static(name),
616    }
617    .arity(arity);
618    if let Some(category) = def.category {
619        meta = meta.category_static(category);
620    }
621    if let Some(doc) = def.doc {
622        meta = meta.doc_static(doc);
623    }
624    if let Some(sig_text) = def.signature_text {
625        meta = meta.signature_static(sig_text);
626    } else {
627        // Builtins declared via `sig_expr = …` (a canonical
628        // `harn_builtin_meta::signatures` const) carry no human-typed `sig`
629        // string, so render the parsed signature back through its `Display`
630        // impl. `Display` round-trips through the macro sig grammar (enforced
631        // by the signature-text drift test), so `harn explain` / LSP hover
632        // still surface an accurate, canonical signature.
633        meta = meta.signature_owned(format!("{}", def.sig));
634    }
635    meta
636}
637
638/// Derive a [`VmBuiltinArity`] from a parsed [`BuiltinSignature`]. Required
639/// params count toward the floor; optional params and `has_rest` widen the
640/// ceiling. Returns `Variadic` for `(...args: any)`-shaped sigs that have
641/// no required params, matching how the DSL builder previously declared
642/// `Variadic` explicitly.
643fn arity_from_sig(sig: &harn_builtin_meta::BuiltinSignature) -> VmBuiltinArity {
644    let required = sig.params.iter().filter(|p| !p.optional).count();
645    let total = sig.params.len();
646    if sig.has_rest {
647        if required == 0 {
648            VmBuiltinArity::Variadic
649        } else {
650            VmBuiltinArity::Min(required)
651        }
652    } else if required == total {
653        VmBuiltinArity::Exact(total)
654    } else {
655        VmBuiltinArity::Range {
656            min: required,
657            max: total,
658        }
659    }
660}