feat: json as a feature (#430)

Author: load1n9

nova_vm/Cargo.toml

@@ -19,12 +19,13 @@ oxc_syntax = { workspace = true }
 rand = { workspace = true }
 ryu-js = { workspace = true }
 small_string = { path = "../small_string" }
-sonic-rs = { workspace = true }
+sonic-rs = { workspace = true,  optional = true}
 wtf8 = { workspace = true }
 
 [features]
-default = ["math"]
+default = ["math", "json"]
 math = []
+json = ["sonic-rs"]
 
 typescript = []
 

nova_vm/src/ecmascript/builtins/structured_data.rs

@@ -5,5 +5,6 @@
 pub(crate) mod array_buffer_objects;
 pub(crate) mod atomics_object;
 pub(crate) mod data_view_objects;
+#[cfg(feature = "json")]
 pub(crate) mod json_object;
 pub(crate) mod shared_array_buffer_objects;

nova_vm/src/ecmascript/execution/realm.rs

@@ -977,16 +977,19 @@ pub(crate) fn set_default_global_bindings(
         define_property_or_throw(agent, global, name, desc)?;
 
         // 19.4.2 JSON
-        let name = PropertyKey::from(BUILTIN_STRING_MEMORY.JSON);
-        let value = agent.get_realm(realm_id).intrinsics().json();
-        let desc = PropertyDescriptor {
-            value: Some(value.into_value()),
-            writable: Some(true),
-            enumerable: Some(false),
-            configurable: Some(true),
-            ..Default::default()
-        };
-        define_property_or_throw(agent, global, name, desc)?;
+        #[cfg(feature = "json")]
+        {
+            let name = PropertyKey::from(BUILTIN_STRING_MEMORY.JSON);
+            let value = agent.get_realm(realm_id).intrinsics().json();
+            let desc = PropertyDescriptor {
+                value: Some(value.into_value()),
+                writable: Some(true),
+                enumerable: Some(false),
+                configurable: Some(true),
+                ..Default::default()
+            };
+            define_property_or_throw(agent, global, name, desc)?;
+        }
 
         // 19.4.3 Math
         #[cfg(feature = "math")]

nova_vm/src/ecmascript/execution/realm/intrinsics.rs

@@ -82,7 +82,6 @@ use crate::{
                     data_view_constructor::DataViewConstructor,
                     data_view_prototype::DataViewPrototype,
                 },
-                json_object::JSONObject,
                 shared_array_buffer_objects::{
                     shared_array_buffer_constructor::SharedArrayBufferConstructor,
                     shared_array_buffer_prototype::SharedArrayBufferPrototype,
@@ -142,10 +141,11 @@ use crate::{
     },
 };
 
+use super::RealmIdentifier;
 #[cfg(feature = "math")]
 use crate::ecmascript::builtins::numbers_and_dates::math_object::MathObject;
-
-use super::RealmIdentifier;
+#[cfg(feature = "json")]
+use crate::ecmascript::builtins::structured_data::json_object::JSONObject;
 
 #[derive(Debug, Clone)]
 pub(crate) struct Intrinsics {
@@ -294,6 +294,7 @@ impl Intrinsics {
         DataViewPrototype::create_intrinsic(agent, realm);
         DataViewConstructor::create_intrinsic(agent, realm);
         AtomicsObject::create_intrinsic(agent, realm);
+        #[cfg(feature = "json")]
         JSONObject::create_intrinsic(agent, realm);
         WeakRefPrototype::create_intrinsic(agent, realm);
         WeakRefConstructor::create_intrinsic(agent, realm);
@@ -940,6 +941,7 @@ impl Intrinsics {
             .into()
     }
 
+    #[cfg(feature = "json")]
     /// %JSON%
     pub(crate) fn json(&self) -> OrdinaryObject {
         IntrinsicObjectIndexes::JSONObject
@@ -1546,6 +1548,7 @@ impl HeapMarkAndSweep for Intrinsics {
         self.is_finite().mark_values(queues);
         self.is_nan().mark_values(queues);
         self.iterator_prototype().mark_values(queues);
+        #[cfg(feature = "json")]
         self.json().mark_values(queues);
         self.map_prototype_entries().mark_values(queues);
         self.map_prototype().mark_values(queues);