@tishlang/tish-format 2.16.13 → 2.35.0

package/crates/tish_vm/src/vm.rs

@@ -180,28 +180,65 @@ pub fn all_compiled_capabilities() -> HashSet<String> {
 fn get_builtin_export(enabled: &HashSet<String>, spec: &str, export_name: &str) -> Option<Value> {
     #[cfg(feature = "fs")]
     if spec == "tish:fs" && cap_allows(enabled, "fs") {
+        use tishlang_runtime::fs_ext as fx;
         return match export_name {
-            "readFile" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::read_file(args)
-            })),
-            "writeFile" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::write_file(args)
-            })),
-            "fileExists" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::file_exists(args)
-            })),
-            "isDir" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::is_dir(args)
-            })),
-            "readDir" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::read_dir(args)
-            })),
-            "mkdir" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::mkdir(args)
-            })),
-            "readFileBytes" => Some(Value::native(|args: &[Value]| {
-                tishlang_runtime::read_file_bytes(args)
-            })),
+            // tish names → the dual fs_ext functions (sync, or Node callback if a fn is passed)
+            "readFile" => Some(Value::native(|a: &[Value]| fx::read_file(a))),
+            "writeFile" => Some(Value::native(|a: &[Value]| fx::write_file(a))),
+            "fileExists" => Some(Value::native(|a: &[Value]| tishlang_runtime::file_exists(a))),
+            "isDir" => Some(Value::native(|a: &[Value]| tishlang_runtime::is_dir(a))),
+            "readDir" => Some(Value::native(|a: &[Value]| fx::readdir(a))),
+            "readFileBytes" => Some(Value::native(|a: &[Value]| fx::read_file_bytes(a))),
+            // Node-compatible names + new methods (sync) — issue #122
+            "readFileSync" => Some(Value::native(|a: &[Value]| fx::read_file(a))),
+            "readFileBytesSync" => Some(Value::native(|a: &[Value]| fx::read_file_bytes(a))),
+            "writeFileSync" => Some(Value::native(|a: &[Value]| fx::write_file(a))),
+            "appendFile" | "appendFileSync" => Some(Value::native(|a: &[Value]| fx::append_file(a))),
+            "existsSync" => Some(Value::native(|a: &[Value]| fx::exists(a))),
+            "mkdir" | "mkdirSync" => Some(Value::native(|a: &[Value]| fx::mkdir(a))),
+            "readdir" | "readdirSync" => Some(Value::native(|a: &[Value]| fx::readdir(a))),
+            "stat" | "statSync" => Some(Value::native(|a: &[Value]| fx::stat(a))),
+            "lstat" | "lstatSync" => Some(Value::native(|a: &[Value]| fx::lstat(a))),
+            "rm" | "rmSync" => Some(Value::native(|a: &[Value]| fx::rm(a))),
+            "rmdir" | "rmdirSync" => Some(Value::native(|a: &[Value]| fx::rmdir(a))),
+            "unlink" | "unlinkSync" => Some(Value::native(|a: &[Value]| fx::unlink(a))),
+            "rename" | "renameSync" => Some(Value::native(|a: &[Value]| fx::rename(a))),
+            "copyFile" | "copyFileSync" => Some(Value::native(|a: &[Value]| fx::copy_file(a))),
+            "realpath" | "realpathSync" => Some(Value::native(|a: &[Value]| fx::realpath(a))),
+            "readlink" | "readlinkSync" => Some(Value::native(|a: &[Value]| fx::readlink(a))),
+            "truncate" | "truncateSync" => Some(Value::native(|a: &[Value]| fx::truncate(a))),
+            "mkdtemp" | "mkdtempSync" => Some(Value::native(|a: &[Value]| fx::mkdtemp(a))),
+            "cp" | "cpSync" => Some(Value::native(|a: &[Value]| fx::cp(a))),
+            "access" | "accessSync" => Some(Value::native(|a: &[Value]| fx::access(a))),
+            "constants" => Some(fx::constants()),
+            _ => None,
+        };
+    }
+    #[cfg(feature = "fs")]
+    if spec == "tish:fs/promises" && cap_allows(enabled, "fs") {
+        use tishlang_runtime::fs_ext as fx;
+        return match export_name {
+            "readFile" => Some(Value::native(|a: &[Value]| fx::read_file_promise(a))),
+            "readFileBytes" => Some(Value::native(|a: &[Value]| fx::read_file_bytes_promise(a))),
+            "writeFile" => Some(Value::native(|a: &[Value]| fx::write_file_promise(a))),
+            "appendFile" => Some(Value::native(|a: &[Value]| fx::append_file_promise(a))),
+            "mkdir" => Some(Value::native(|a: &[Value]| fx::mkdir_promise(a))),
+            "readdir" => Some(Value::native(|a: &[Value]| fx::readdir_promise(a))),
+            "stat" => Some(Value::native(|a: &[Value]| fx::stat_promise(a))),
+            "lstat" => Some(Value::native(|a: &[Value]| fx::lstat_promise(a))),
+            "rm" => Some(Value::native(|a: &[Value]| fx::rm_promise(a))),
+            "rmdir" => Some(Value::native(|a: &[Value]| fx::rmdir_promise(a))),
+            "unlink" => Some(Value::native(|a: &[Value]| fx::unlink_promise(a))),
+            "rename" => Some(Value::native(|a: &[Value]| fx::rename_promise(a))),
+            "copyFile" => Some(Value::native(|a: &[Value]| fx::copy_file_promise(a))),
+            "realpath" => Some(Value::native(|a: &[Value]| fx::realpath_promise(a))),
+            "readlink" => Some(Value::native(|a: &[Value]| fx::readlink_promise(a))),
+            "truncate" => Some(Value::native(|a: &[Value]| fx::truncate_promise(a))),
+            "mkdtemp" => Some(Value::native(|a: &[Value]| fx::mkdtemp_promise(a))),
+            "cp" => Some(Value::native(|a: &[Value]| fx::cp_promise(a))),
+            "access" => Some(Value::native(|a: &[Value]| fx::access_promise(a))),
+            "exists" => Some(Value::native(|a: &[Value]| fx::exists_promise(a))),
+            "constants" => Some(fx::constants()),
             _ => None,
         };
     }
@@ -537,6 +574,13 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
         "pow".into(),
         Value::native(|args: &[Value]| math_builtins::pow(args)),
     );
+    // `imul` delegates to the shared builtin (ToInt32 + wrapping i32 multiply) so the vm never
+    // diverges from interp/native (#247). Was previously absent from the vm `Math` object, so
+    // `Math.imul(...)` threw "Call of non-function" on the bytecode VM.
+    math.insert(
+        "imul".into(),
+        Value::native(|args: &[Value]| math_builtins::imul(args)),
+    );
     math.insert(
         "sin".into(),
         Value::native(|args: &[Value]| math_builtins::sin(args)),
@@ -597,13 +641,12 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
             Value::Number(n.acos())
         }),
     );
+    // hypot delegates to the shared builtin (variadic, NaN/Infinity-correct) so the vm never diverges
+    // from interp/native (#247). The old inline `filter_map(as_number)` dropped non-number args
+    // instead of propagating NaN.
     math.insert(
         "hypot".into(),
-        Value::native(|args: &[Value]| {
-            let nums: Vec<f64> = args.iter().filter_map(|v| v.as_number()).collect();
-            let sum_sq: f64 = nums.iter().map(|n| n * n).sum();
-            Value::Number(sum_sq.sqrt())
-        }),
+        Value::native(|args: &[Value]| math_builtins::hypot(args)),
     );
     // Hyperbolic, inverse-hyperbolic, cbrt and base-2/10 logs. Like the trig block above
     // these aren't in `math_builtins`, and on the wasm/native VM there is no host `Math`
@@ -628,6 +671,10 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
     math_unary!("cbrt", cbrt);
     math_unary!("log2", log2);
     math_unary!("log10", log10);
+    math_unary!("expm1", exp_m1);
+    math_unary!("log1p", ln_1p);
+    math.insert("clz32".into(), Value::native(|a: &[Value]| math_builtins::clz32(a)));
+    math.insert("fround".into(), Value::native(|a: &[Value]| math_builtins::fround(a)));
     math.insert("PI".into(), Value::Number(std::f64::consts::PI));
     math.insert("E".into(), Value::Number(std::f64::consts::E));
     g.insert("Math".into(), value_object_from_map(math));
@@ -668,6 +715,18 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
         "decodeURI".into(),
         Value::native(|args: &[Value]| globals_builtins::decode_uri(args)),
     );
+    g.insert(
+        "encodeURIComponent".into(),
+        Value::native(|args: &[Value]| globals_builtins::encode_uri_component(args)),
+    );
+    g.insert(
+        "decodeURIComponent".into(),
+        Value::native(|args: &[Value]| globals_builtins::decode_uri_component(args)),
+    );
+    g.insert(
+        "structuredClone".into(),
+        Value::native(|args: &[Value]| globals_builtins::structured_clone(args)),
+    );
     g.insert(
         "htmlEscape".into(),
         Value::native(|args: &[Value]| {
@@ -769,6 +828,18 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
         "assign".into(),
         Value::native(|args: &[Value]| globals_builtins::object_assign(args)),
     );
+    object_methods.insert(
+        "is".into(),
+        Value::native(|args: &[Value]| globals_builtins::object_is(args)),
+    );
+    object_methods.insert(
+        "hasOwn".into(),
+        Value::native(|args: &[Value]| globals_builtins::object_has_own(args)),
+    );
+    object_methods.insert(
+        "getOwnPropertyNames".into(),
+        Value::native(|args: &[Value]| globals_builtins::object_keys(args)),
+    );
     object_methods.insert(
         "fromEntries".into(),
         Value::native(|args: &[Value]| globals_builtins::object_from_entries(args)),
@@ -794,6 +865,14 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
         "isArray".into(),
         Value::native(|args: &[Value]| globals_builtins::array_is_array(args)),
     );
+    array_static.insert(
+        "of".into(),
+        Value::native(|args: &[Value]| globals_builtins::array_of(args)),
+    );
+    array_static.insert(
+        "from".into(),
+        Value::native(|args: &[Value]| globals_builtins::array_from(args)),
+    );
     array_static.insert(
         Arc::from("__call"),
         Value::native(|args: &[Value]| construct_builtin::array_construct(args)),
@@ -810,12 +889,26 @@ fn init_globals(enabled: &HashSet<String>) -> ObjectMap {
     string_static.insert(Arc::from("__call"), string_convert_fn);
     g.insert("String".into(), value_object_from_map(string_static));
 
-    // Number(value) coercion as a callable global (issue #36).
+    // Number(value) coercion as a callable global (issue #36) + the `Number.*` statics.
     let mut number_static = ObjectMap::default();
     number_static.insert(
         Arc::from("__call"),
         Value::native(|args: &[Value]| globals_builtins::number_convert(args)),
     );
+    number_static.insert(Arc::from("isInteger"), Value::native(|a: &[Value]| globals_builtins::number_is_integer(a)));
+    number_static.insert(Arc::from("isSafeInteger"), Value::native(|a: &[Value]| globals_builtins::number_is_safe_integer(a)));
+    number_static.insert(Arc::from("isNaN"), Value::native(|a: &[Value]| globals_builtins::number_is_nan(a)));
+    number_static.insert(Arc::from("isFinite"), Value::native(|a: &[Value]| globals_builtins::number_is_finite(a)));
+    number_static.insert(Arc::from("parseInt"), Value::native(|a: &[Value]| globals_builtins::parse_int(a)));
+    number_static.insert(Arc::from("parseFloat"), Value::native(|a: &[Value]| globals_builtins::parse_float(a)));
+    number_static.insert(Arc::from("MAX_SAFE_INTEGER"), Value::Number(9_007_199_254_740_991.0));
+    number_static.insert(Arc::from("MIN_SAFE_INTEGER"), Value::Number(-9_007_199_254_740_991.0));
+    number_static.insert(Arc::from("EPSILON"), Value::Number(f64::EPSILON));
+    number_static.insert(Arc::from("MAX_VALUE"), Value::Number(f64::MAX));
+    number_static.insert(Arc::from("MIN_VALUE"), Value::Number(5e-324));
+    number_static.insert(Arc::from("POSITIVE_INFINITY"), Value::Number(f64::INFINITY));
+    number_static.insert(Arc::from("NEGATIVE_INFINITY"), Value::Number(f64::NEG_INFINITY));
+    number_static.insert(Arc::from("NaN"), Value::Number(f64::NAN));
     g.insert("Number".into(), value_object_from_map(number_static));
 
     // JSX / Lattish: stubs for bytecode VM when no DOM (e.g. console). Override via set_global in browser.
@@ -1383,7 +1476,7 @@ mod hof_fusion {
     /// (which would pass a `Value::Bool`, not a number). No-initial-value semantics match the generic
     /// path: absent init (`Value::Null`) with a non-empty array seeds `acc = data[0]` and scans from
     /// index 1; an empty array with no init throws (bail to the generic path, which raises it).
-    pub(super) fn reduce(arr: &Value, cb: &Value, init: &Value) -> Option<Value> {
+    pub(super) fn reduce(arr: &Value, cb: &Value, initial: Option<&Value>) -> Option<Value> {
         let nf = fused_numeric_fn(cb)?;
         if nf.arity() > 3 || nf.result_is_bool() {
             return None;
@@ -1392,11 +1485,11 @@ mod hof_fusion {
         let arity = nf.arity();
         // Determine seed. Reduce with no initial value AND an empty array is a TypeError in JS — let
         // the generic path produce that exact throw rather than replicate it here.
-        let (start, mut acc) = match init {
-            Value::Null if !data.is_empty() => (1usize, data[0]),
-            Value::Null => return None, // empty + no init → generic path throws
-            Value::Number(n) => (0usize, *n),
-            _ => return None, // non-numeric explicit init → generic path (acc wouldn't stay f64)
+        let (start, mut acc) = match initial {
+            None if !data.is_empty() => (1usize, data[0]),
+            None => return None,             // empty + no init → generic path throws
+            Some(Value::Number(n)) => (0usize, *n),
+            Some(_) => return None, // non-numeric/explicit-null init → generic path (acc wouldn't stay f64)
         };
         let mut args = [0.0f64; 3];
         for (i, &x) in data.iter().enumerate().skip(start) {
@@ -3823,13 +3916,21 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                         }),
                         "reduce" => make_native_fn(move |args| {
                             let cb = args.first().cloned().unwrap_or(Value::Null);
-                            let init = args.get(1).cloned().unwrap_or(Value::Null);
+                            let init = args.get(1).cloned(); // None = no initial value (empty array → throw)
                             #[cfg(not(target_arch = "wasm32"))]
-                            if let Some(v) = hof_fusion::reduce(&bv, &cb, &init) {
+                            if let Some(v) = hof_fusion::reduce(&bv, &cb, init.as_ref()) {
                                 return v;
                             }
-                            arr_builtins::reduce(&bv, &cb, &init)
+                            arr_builtins::reduce(&bv, &cb, init.as_ref())
                         }),
+                        "reduceRight" => make_native_fn(move |args| {
+                            let cb = args.first().cloned().unwrap_or(Value::Null);
+                            let init = args.get(1).cloned();
+                            arr_builtins::reduce_right(&bv, &cb, init.as_ref())
+                        }),
+                        "keys" => make_native_fn(move |_| arr_builtins::keys(&bv)),
+                        "values" => make_native_fn(move |_| arr_builtins::values(&bv)),
+                        "entries" => make_native_fn(move |_| arr_builtins::entries(&bv)),
                         "forEach" => make_native_fn(move |args| {
                             let cb = args.first().cloned().unwrap_or(Value::Null);
                             #[cfg(not(target_arch = "wasm32"))]
@@ -3879,6 +3980,21 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                             arr_builtins::flat_map(&bv, &cb)
                         }),
                         "reverse" => make_native_fn(move |_| arr_builtins::reverse(&bv)),
+                        "toReversed" => make_native_fn(move |_| arr_builtins::to_reversed(&bv)),
+                        "toSorted" => {
+                            make_native_fn(move |args| arr_builtins::to_sorted(&bv, args.first()))
+                        }
+                        "with" => make_native_fn(move |args| {
+                            let i = args.first().cloned().unwrap_or(Value::Null);
+                            let v = args.get(1).cloned().unwrap_or(Value::Null);
+                            arr_builtins::with(&bv, &i, &v)
+                        }),
+                        "toSpliced" => make_native_fn(move |args| {
+                            let start = args.first().cloned().unwrap_or(Value::Null);
+                            let dc = args.get(1).cloned();
+                            let items: Vec<Value> = args.iter().skip(2).cloned().collect();
+                            arr_builtins::to_spliced(&bv, &start, dc.as_ref(), &items)
+                        }),
                         "fill" => make_native_fn(move |args| {
                             let v = args.first().cloned().unwrap_or(Value::Null);
                             let s = args.get(1).cloned().unwrap_or(Value::Null);
@@ -3893,7 +4009,17 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                         "concat" => make_native_fn(move |args| arr_builtins::concat(&bv, args)),
                         "indexOf" => make_native_fn(move |args| {
                             let s = args.first().cloned().unwrap_or(Value::Null);
-                            arr_builtins::index_of(&bv, &s)
+                            arr_builtins::index_of(&bv, &s, args.get(1))
+                        }),
+                        "lastIndexOf" => make_native_fn(move |args| {
+                            let s = args.first().cloned().unwrap_or(Value::Null);
+                            arr_builtins::last_index_of(&bv, &s, args.get(1))
+                        }),
+                        "copyWithin" => make_native_fn(move |args| {
+                            let t = args.first().cloned().unwrap_or(Value::Null);
+                            let s = args.get(1).cloned().unwrap_or(Value::Null);
+                            let e = args.get(2).cloned().unwrap_or(Value::Null);
+                            arr_builtins::copy_within(&bv, &t, &s, &e)
                         }),
                         "includes" => make_native_fn(move |args| {
                             let s = args.first().cloned().unwrap_or(Value::Null);
@@ -3966,7 +4092,17 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                 }),
                 "indexOf" => make_native_fn(move |args: &[Value]| {
                     let search = args.first().unwrap_or(&Value::Null);
-                    arr_builtins::index_of(&Value::Array(a_clone.clone()), search)
+                    arr_builtins::index_of(&Value::Array(a_clone.clone()), search, args.get(1))
+                }),
+                "lastIndexOf" => make_native_fn(move |args: &[Value]| {
+                    let search = args.first().unwrap_or(&Value::Null);
+                    arr_builtins::last_index_of(&Value::Array(a_clone.clone()), search, args.get(1))
+                }),
+                "copyWithin" => make_native_fn(move |args: &[Value]| {
+                    let t = args.first().cloned().unwrap_or(Value::Null);
+                    let s = args.get(1).cloned().unwrap_or(Value::Null);
+                    let e = args.get(2).cloned().unwrap_or(Value::Null);
+                    arr_builtins::copy_within(&Value::Array(a_clone.clone()), &t, &s, &e)
                 }),
                 "includes" => make_native_fn(move |args: &[Value]| {
                     let search = args.first().unwrap_or(&Value::Null);
@@ -3995,14 +4131,22 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                 }),
                 "reduce" => make_native_fn(move |args: &[Value]| {
                     let cb = args.first().cloned().unwrap_or(Value::Null);
-                    let init = args.get(1).cloned().unwrap_or(Value::Null);
+                    let init = args.get(1).cloned(); // None = no initial value (empty array → throw)
                     let arr = Value::Array(a_clone.clone());
                     #[cfg(not(target_arch = "wasm32"))]
-                    if let Some(v) = hof_fusion::reduce(&arr, &cb, &init) {
+                    if let Some(v) = hof_fusion::reduce(&arr, &cb, init.as_ref()) {
                         return v;
                     }
-                    arr_builtins::reduce(&arr, &cb, &init)
+                    arr_builtins::reduce(&arr, &cb, init.as_ref())
                 }),
+                "reduceRight" => make_native_fn(move |args: &[Value]| {
+                    let cb = args.first().cloned().unwrap_or(Value::Null);
+                    let init = args.get(1).cloned();
+                    arr_builtins::reduce_right(&Value::Array(a_clone.clone()), &cb, init.as_ref())
+                }),
+                "keys" => make_native_fn(move |_: &[Value]| arr_builtins::keys(&Value::Array(a_clone.clone()))),
+                "values" => make_native_fn(move |_: &[Value]| arr_builtins::values(&Value::Array(a_clone.clone()))),
+                "entries" => make_native_fn(move |_: &[Value]| arr_builtins::entries(&Value::Array(a_clone.clone()))),
                 "forEach" => make_native_fn(move |args: &[Value]| {
                     let cb = args.first().cloned().unwrap_or(Value::Null);
                     let arr = Value::Array(a_clone.clone());
@@ -4070,6 +4214,28 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                         &items,
                     )
                 }),
+                "toReversed" => make_native_fn(move |_: &[Value]| {
+                    arr_builtins::to_reversed(&Value::Array(a_clone.clone()))
+                }),
+                "toSorted" => make_native_fn(move |args: &[Value]| {
+                    arr_builtins::to_sorted(&Value::Array(a_clone.clone()), args.first())
+                }),
+                "with" => make_native_fn(move |args: &[Value]| {
+                    let i = args.first().cloned().unwrap_or(Value::Null);
+                    let v = args.get(1).cloned().unwrap_or(Value::Null);
+                    arr_builtins::with(&Value::Array(a_clone.clone()), &i, &v)
+                }),
+                "toSpliced" => make_native_fn(move |args: &[Value]| {
+                    let start = args.first().unwrap_or(&Value::Null);
+                    let delete_count = args.get(1).map(|v| v as &Value);
+                    let items: Vec<Value> = args.get(2..).unwrap_or(&[]).to_vec();
+                    arr_builtins::to_spliced(
+                        &Value::Array(a_clone.clone()),
+                        start,
+                        delete_count,
+                        &items,
+                    )
+                }),
                 _ => return Err(format!("Property '{}' not found", key)),
             };
             Ok(Value::Function(method))
@@ -4136,6 +4302,16 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                 "trim" => make_native_fn(move |_args: &[Value]| {
                     str_builtins::trim(&Value::String(s_clone.clone()))
                 }),
+                "trimStart" => make_native_fn(move |_args: &[Value]| {
+                    str_builtins::trim_start(&Value::String(s_clone.clone()))
+                }),
+                "trimEnd" => make_native_fn(move |_args: &[Value]| {
+                    str_builtins::trim_end(&Value::String(s_clone.clone()))
+                }),
+                "normalize" => make_native_fn(move |args: &[Value]| {
+                    let form = args.first().cloned().unwrap_or(Value::Null);
+                    str_builtins::normalize(&Value::String(s_clone.clone()), &form)
+                }),
                 "toUpperCase" => make_native_fn(move |_args: &[Value]| {
                     str_builtins::to_upper_case(&Value::String(s_clone.clone()))
                 }),
@@ -4144,11 +4320,11 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                 }),
                 "startsWith" => make_native_fn(move |args: &[Value]| {
                     let search = args.first().unwrap_or(&Value::Null);
-                    str_builtins::starts_with(&Value::String(s_clone.clone()), search)
+                    str_builtins::starts_with(&Value::String(s_clone.clone()), search, args.get(1))
                 }),
                 "endsWith" => make_native_fn(move |args: &[Value]| {
                     let search = args.first().unwrap_or(&Value::Null);
-                    str_builtins::ends_with(&Value::String(s_clone.clone()), search)
+                    str_builtins::ends_with(&Value::String(s_clone.clone()), search, args.get(1))
                 }),
                 "replace" => make_native_fn(move |args: &[Value]| {
                     let search = args.first().unwrap_or(&Value::Null);
@@ -4176,6 +4352,11 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                     tishlang_runtime::string_match_regex(&Value::String(s_clone.clone()), re)
                 }),
                 #[cfg(feature = "regex")]
+                "matchAll" => make_native_fn(move |args: &[Value]| {
+                    let re = args.first().unwrap_or(&Value::Null);
+                    tishlang_runtime::string_match_all_regex(&Value::String(s_clone.clone()), re)
+                }),
+                #[cfg(feature = "regex")]
                 "search" => make_native_fn(move |args: &[Value]| {
                     let re = args.first().unwrap_or(&Value::Null);
                     tishlang_runtime::string_search_regex(&Value::String(s_clone.clone()), re)
@@ -4192,6 +4373,15 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                     let idx = args.first().unwrap_or(&Value::Null);
                     str_builtins::char_code_at(&Value::String(s_clone.clone()), idx)
                 }),
+                "codePointAt" => make_native_fn(move |args: &[Value]| {
+                    let idx = args.first().unwrap_or(&Value::Null);
+                    str_builtins::code_point_at(&Value::String(s_clone.clone()), idx)
+                }),
+                "substr" => make_native_fn(move |args: &[Value]| {
+                    let start = args.first().cloned().unwrap_or(Value::Null);
+                    let length = args.get(1).cloned().unwrap_or(Value::Null);
+                    str_builtins::substr(&Value::String(s_clone.clone()), &start, &length)
+                }),
                 "repeat" => make_native_fn(move |args: &[Value]| {
                     let count = args.first().unwrap_or(&Value::Null);
                     str_builtins::repeat(&Value::String(s_clone.clone()), count)
@@ -4219,6 +4409,14 @@ fn get_member(obj: &Value, key: &Arc<str>) -> Result<Value, String> {
                     let digits = args.first().unwrap_or(&Value::Null);
                     num_builtins::to_fixed(&Value::Number(n_val), digits)
                 }),
+                "toExponential" => make_native_fn(move |args: &[Value]| {
+                    let digits = args.first().unwrap_or(&Value::Null);
+                    num_builtins::to_exponential(&Value::Number(n_val), digits)
+                }),
+                "toPrecision" => make_native_fn(move |args: &[Value]| {
+                    let precision = args.first().unwrap_or(&Value::Null);
+                    num_builtins::to_precision(&Value::Number(n_val), precision)
+                }),
                 "toString" => make_native_fn(move |args: &[Value]| {
                     let radix = args.first().unwrap_or(&Value::Null);
                     num_builtins::to_string(&Value::Number(n_val), radix)
@@ -4333,12 +4531,13 @@ fn get_index(obj: &Value, idx: &Value) -> Result<Value, String> {
         Value::NumberArray(a) => {
             let i = match idx {
                 Value::Number(n) => *n as usize,
-                _ => {
-                    return Err(format!(
-                        "Array index must be number, got {}",
-                        idx.type_name()
-                    ))
-                }
+                // A canonical integer string key ("0","1",…) indexes the array (#432); any other key
+                // reads Null (JS reads a missing property as undefined) rather than throwing.
+                Value::String(s) => match tishlang_core::str_to_array_index(s) {
+                    Some(i) => i,
+                    None => return Ok(Value::Null),
+                },
+                _ => return Ok(Value::Null),
             };
             // NaN is used as the hole marker (sparse-array positions); reads return Null.
             Ok(a.borrow()
@@ -4355,12 +4554,12 @@ fn get_index(obj: &Value, idx: &Value) -> Result<Value, String> {
         Value::Array(a) => {
             let i = match idx {
                 Value::Number(n) => *n as usize,
-                _ => {
-                    return Err(format!(
-                        "Array index must be number, got {}",
-                        idx.type_name()
-                    ));
-                }
+                // Canonical integer string key → index (#432); any other key reads Null, not a throw.
+                Value::String(s) => match tishlang_core::str_to_array_index(s) {
+                    Some(i) => i,
+                    None => return Ok(Value::Null),
+                },
+                _ => return Ok(Value::Null),
             };
             Ok(a.borrow().get(i).cloned().unwrap_or(Value::Null))
         }
@@ -4369,10 +4568,9 @@ fn get_index(obj: &Value, idx: &Value) -> Result<Value, String> {
                 Value::Number(n) => {
                     let n = *n;
                     if n < 0.0 || n.fract() != 0.0 {
-                        return Err(format!(
-                            "String index must be non-negative integer, got {}",
-                            n
-                        ));
+                        // A negative / non-integer string index reads `null` (JS `undefined`), NOT a
+                        // throw — interp and native already return null; the vm must agree. (#437)
+                        return Ok(Value::Null);
                     }
                     n as usize
                 }
@@ -4387,7 +4585,8 @@ fn get_index(obj: &Value, idx: &Value) -> Result<Value, String> {
             // O(n) `chars().count()` pre-check (#203).
             match str_builtins::nth_char(s, i) {
                 Some(c) => Ok(Value::String(tishlang_core::ArcStr::from(c.to_string()))),
-                None => Err("Index out of bounds".to_string()),
+                // Past the end → `null` (JS `undefined`), matching interp/native, not a throw. (#437)
+                None => Ok(Value::Null),
             }
         }
         // A missing own property returns `null`, not a thrown error — matching dot reads
@@ -4447,6 +4646,12 @@ fn set_index(obj: &Value, idx: &Value, val: Value) -> Result<(), String> {
         Value::NumberArray(a) => {
             let i = match idx {
                 Value::Number(n) => *n as usize,
+                // Canonical integer string key → index (#432); a non-index string key can't be stored
+                // in a Vec-backed array, so the write is a no-op (rather than a throw).
+                Value::String(s) => match tishlang_core::str_to_array_index(s) {
+                    Some(i) => i,
+                    None => return Ok(()),
+                },
                 _ => {
                     return Err(format!(
                         "Array index must be number, got {}",
@@ -4487,6 +4692,11 @@ fn set_index(obj: &Value, idx: &Value, val: Value) -> Result<(), String> {
         Value::Array(a) => {
             let i = match idx {
                 Value::Number(n) => *n as usize,
+                // Canonical integer string key → index (#432); a non-index string key is a no-op write.
+                Value::String(s) => match tishlang_core::str_to_array_index(s) {
+                    Some(i) => i,
+                    None => return Ok(()),
+                },
                 _ => {
                     return Err(format!(
                         "Array index must be number, got {}",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@tishlang/tish-format",
-  "version": "2.16.13",
+  "version": "2.35.0",
   "description": "Tish code formatter",
   "license": "PIF",
   "repository": {