Move value module from builtins (#674)

boa/examples/classes.rs

@@ -1,10 +1,10 @@
 use boa::{
-    builtins::{property::Attribute, value::Value},
+    builtins::property::Attribute,
     class::{Class, ClassBuilder},
     exec::Interpreter,
     forward_val,
     realm::Realm,
-    Finalize, Result, Trace,
+    Finalize, Result, Trace, Value,
 };
 
 // We create a new struct that is going to represent a person.

boa/src/builtins/array/mod.rs

@@ -17,9 +17,9 @@ use crate::{
     builtins::{
         object::{ObjectData, PROTOTYPE},
         property::{Attribute, Property},
-        value::{same_value_zero, Value},
     },
     exec::Interpreter,
+    value::{same_value_zero, Value},
     BoaProfiler, Result,
 };
 use std::{

boa/src/builtins/bigint/conversions.rs

@@ -1,9 +1,6 @@
 use super::BigInt;
 
-use crate::{
+use crate::{builtins::Number, exec::Interpreter, Value};
-    builtins::{Number, Value},
-    exec::Interpreter,
-};
 use num_traits::cast::{FromPrimitive, ToPrimitive};
 
 use std::convert::TryFrom;

boa/src/builtins/bigint/mod.rs

@@ -16,9 +16,9 @@ use crate::{
     builtins::{
         function::{make_builtin_fn, make_constructor_fn},
         object::ObjectData,
-        value::{RcBigInt, Value},
     },
     exec::Interpreter,
+    value::{RcBigInt, Value},
     BoaProfiler, Result,
 };
 

boa/src/builtins/boolean/mod.rs

@@ -13,11 +13,7 @@
 mod tests;
 
 use super::function::{make_builtin_fn, make_constructor_fn};
-use crate::{
+use crate::{builtins::object::ObjectData, exec::Interpreter, BoaProfiler, Result, Value};
-    builtins::{object::ObjectData, value::Value},
-    exec::Interpreter,
-    BoaProfiler, Result,
-};
 
 /// Boolean implementation.
 #[derive(Debug, Clone, Copy)]

boa/src/builtins/boolean/tests.rs

@@ -1,4 +1,4 @@
-use crate::{builtins::value::same_value, exec::Interpreter, forward, forward_val, realm::Realm};
+use crate::{exec::Interpreter, forward, forward_val, realm::Realm, value::same_value};
 
 /// Test the correct type is returned from call and construct
 #[allow(clippy::unwrap_used)]

boa/src/builtins/console/mod.rs

@@ -17,11 +17,9 @@
 mod tests;
 
 use crate::{
-    builtins::{
+    builtins::function::make_builtin_fn,
-        function::make_builtin_fn,
-        value::{display_obj, RcString, Value},
-    },
     exec::Interpreter,
+    value::{display_obj, RcString, Value},
     BoaProfiler, Result,
 };
 use rustc_hash::FxHashMap;

boa/src/builtins/console/tests.rs

@@ -1,8 +1,4 @@
-use crate::{
+use crate::{builtins::console::formatter, exec::Interpreter, realm::Realm, Value};
-    builtins::{console::formatter, value::Value},
-    exec::Interpreter,
-    realm::Realm,
-};
 
 #[test]
 fn formatter_no_args_is_empty_string() {

boa/src/builtins/date/mod.rs

@@ -5,10 +5,9 @@ use crate::{
     builtins::{
         function::{make_builtin_fn, make_constructor_fn},
         object::ObjectData,
-        value::PreferredType,
-        Value,
     },
-    BoaProfiler, Interpreter, Result,
+    value::PreferredType,
+    BoaProfiler, Interpreter, Result, Value,
 };
 use chrono::{prelude::*, Duration, LocalResult};
 use gc::{unsafe_empty_trace, Finalize, Trace};

boa/src/builtins/date/tests.rs

@@ -1,9 +1,6 @@
 #![allow(clippy::zero_prefixed_literal)]
 
-use crate::{
+use crate::{builtins::object::ObjectData, forward, forward_val, Interpreter, Realm, Value};
-    builtins::{object::ObjectData, Value},
-    forward, forward_val, Interpreter, Realm,
-};
 use chrono::prelude::*;
 
 // NOTE: Javascript Uses 0-based months, where chrono uses 1-based months. Many of the assertions look wrong because of

boa/src/builtins/error/mod.rs

@@ -14,11 +14,10 @@ use crate::{
     builtins::{
         function::{make_builtin_fn, make_constructor_fn},
         object::ObjectData,
-        value::Value,
     },
     exec::Interpreter,
     profiler::BoaProfiler,
-    Result,
+    Result, Value,
 };
 
 pub(crate) mod range;

boa/src/builtins/error/range.rs

@@ -10,12 +10,10 @@
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/RangeError
 
 use crate::{
-    builtins::{
+    builtins::{function::make_builtin_fn, function::make_constructor_fn, object::ObjectData},
-        function::make_builtin_fn, function::make_constructor_fn, object::ObjectData, value::Value,
-    },
     exec::Interpreter,
     profiler::BoaProfiler,
-    Result,
+    Result, Value,
 };
 
 /// JavaScript `RangeError` impleentation.

boa/src/builtins/error/reference.rs

@@ -10,12 +10,10 @@
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/ReferenceError
 
 use crate::{
-    builtins::{
+    builtins::{function::make_builtin_fn, function::make_constructor_fn, object::ObjectData},
-        function::make_builtin_fn, function::make_constructor_fn, object::ObjectData, value::Value,
-    },
     exec::Interpreter,
     profiler::BoaProfiler,
-    Result,
+    Result, Value,
 };
 
 #[derive(Debug, Clone, Copy)]

boa/src/builtins/error/syntax.rs

@@ -12,12 +12,10 @@
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/SyntaxError
 
 use crate::{
-    builtins::{
+    builtins::{function::make_builtin_fn, function::make_constructor_fn, object::ObjectData},
-        function::make_builtin_fn, function::make_constructor_fn, object::ObjectData, value::Value,
-    },
     exec::Interpreter,
     profiler::BoaProfiler,
-    Result,
+    Result, Value,
 };
 
 /// JavaScript `SyntaxError` impleentation.

boa/src/builtins/error/type.rs

@@ -16,11 +16,9 @@
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/TypeError
 
 use crate::{
-    builtins::{
+    builtins::{function::make_builtin_fn, function::make_constructor_fn, object::ObjectData},
-        function::make_builtin_fn, function::make_constructor_fn, object::ObjectData, value::Value,
-    },
     exec::Interpreter,
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 
 /// JavaScript `TypeError` implementation.

boa/src/builtins/function/mod.rs

@@ -15,13 +15,12 @@ use crate::{
     builtins::{
         object::{Object, ObjectData, PROTOTYPE},
         property::{Attribute, Property},
-        value::Value,
         Array,
     },
     environment::lexical_environment::Environment,
     exec::Interpreter,
-    syntax::ast::node::{FormalParameter, RcStatementList},
+    syntax::ast::node::{statement_list::RcStatementList, FormalParameter},
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 use bitflags::bitflags;
 use gc::{unsafe_empty_trace, Finalize, Trace};

boa/src/builtins/global_this/mod.rs

@@ -10,7 +10,7 @@
 //! [spec]: https://tc39.es/ecma262/#sec-globalthis
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/globalThis
 
-use crate::{builtins::value::Value, BoaProfiler, Interpreter};
+use crate::{BoaProfiler, Interpreter, Value};
 
 #[cfg(test)]
 mod tests;

boa/src/builtins/infinity/mod.rs

@@ -12,7 +12,7 @@
 #[cfg(test)]
 mod tests;
 
-use crate::{builtins::value::Value, BoaProfiler, Interpreter};
+use crate::{BoaProfiler, Interpreter, Value};
 
 /// JavaScript global `Infinity` property.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]

boa/src/builtins/json/mod.rs

@@ -17,10 +17,9 @@ use crate::{
     builtins::{
         function::make_builtin_fn,
         property::{Property, PropertyKey},
-        value::Value,
     },
     exec::Interpreter,
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 use serde_json::{self, Value as JSONValue};
 

boa/src/builtins/json/tests.rs

@@ -1,8 +1,6 @@
 use crate::{
-    builtins::{object::PROTOTYPE, value::same_value},
+    builtins::object::PROTOTYPE, exec::Interpreter, forward, forward_val, realm::Realm,
-    exec::Interpreter,
+    value::same_value,
-    forward, forward_val,
-    realm::Realm,
 };
 
 #[test]

boa/src/builtins/map/mod.rs

@@ -5,10 +5,9 @@ use crate::{
     builtins::{
         object::{ObjectData, PROTOTYPE},
         property::{Attribute, Property},
-        value::Value,
     },
     exec::Interpreter,
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 use ordered_map::OrderedMap;
 

boa/src/builtins/math/mod.rs

@@ -11,11 +11,7 @@
 //! [spec]: https://tc39.es/ecma262/#sec-math-object
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Math
 
-use crate::{
+use crate::{builtins::function::make_builtin_fn, exec::Interpreter, BoaProfiler, Result, Value};
-    builtins::{function::make_builtin_fn, value::Value},
-    exec::Interpreter,
-    BoaProfiler, Result,
-};
 use std::f64;
 
 #[cfg(test)]

boa/src/builtins/mod.rs

@@ -20,7 +20,6 @@ pub mod regexp;
 pub mod string;
 pub mod symbol;
 pub mod undefined;
-pub mod value;
 
 pub(crate) use self::{
     array::Array,
@@ -40,9 +39,8 @@ pub(crate) use self::{
     string::String,
     symbol::Symbol,
     undefined::Undefined,
-    value::Value,
 };
-use crate::Interpreter;
+use crate::{Interpreter, Value};
 
 /// Initializes builtin objects and functions
 #[inline]

boa/src/builtins/nan/mod.rs

@@ -13,7 +13,7 @@
 #[cfg(test)]
 mod tests;
 
-use crate::{builtins::value::Value, BoaProfiler, Interpreter};
+use crate::{BoaProfiler, Interpreter, Value};
 
 /// JavaScript global `NaN` property.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]

boa/src/builtins/number/mod.rs

@@ -16,9 +16,8 @@
 use super::{
     function::{make_builtin_fn, make_constructor_fn},
     object::ObjectData,
-    value::AbstractRelation,
 };
-use crate::{builtins::value::Value, exec::Interpreter, BoaProfiler, Result};
+use crate::{exec::Interpreter, value::AbstractRelation, BoaProfiler, Result, Value};
 use num_traits::float::FloatCore;
 
 mod conversions;

boa/src/builtins/object/gcobject.rs

@@ -4,15 +4,14 @@
 
 use super::{Object, PROTOTYPE};
 use crate::{
-    builtins::{
+    builtins::function::{
-        function::{create_unmapped_arguments_object, BuiltInFunction, Function, NativeFunction},
+        create_unmapped_arguments_object, BuiltInFunction, Function, NativeFunction,
-        Value,
     },
     environment::{
         function_environment_record::BindingStatus, lexical_environment::new_function_environment,
     },
-    syntax::ast::node::statement_list::RcStatementList,
+    syntax::ast::node::RcStatementList,
-    Executable, Interpreter, Result,
+    Executable, Interpreter, Result, Value,
 };
 use gc::{Finalize, Gc, GcCell, GcCellRef, GcCellRefMut, Trace};
 use std::{

boa/src/builtins/object/internal_methods.rs

@@ -5,13 +5,14 @@
 //!
 //! [spec]: https://tc39.es/ecma262/#sec-ordinary-object-internal-methods-and-internal-slots
 
-use crate::builtins::{
+use crate::{
-    object::Object,
+    builtins::{
-    property::{Attribute, Property, PropertyKey},
+        object::Object,
+        property::{Attribute, Property, PropertyKey},
+    },
     value::{same_value, Value},
+    BoaProfiler,
 };
-use crate::BoaProfiler;
-
 impl Object {
     /// Check if object has property.
     ///

boa/src/builtins/object/iter.rs

@@ -1,4 +1,5 @@
-use super::{Object, Property, PropertyKey, RcString, RcSymbol};
+use super::{Object, Property, PropertyKey};
+use crate::value::{RcString, RcSymbol};
 use std::{collections::hash_map, iter::FusedIterator};
 
 impl Object {

boa/src/builtins/object/mod.rs

@@ -15,13 +15,13 @@
 
 use crate::{
     builtins::{
-        function::Function,
+        function::{make_builtin_fn, make_constructor_fn, Function},
         map::ordered_map::OrderedMap,
         property::{Property, PropertyKey},
-        value::{RcBigInt, RcString, RcSymbol, Value},
         BigInt, Date, RegExp,
     },
     exec::Interpreter,
+    value::{same_value, RcBigInt, RcString, RcSymbol, Value},
     BoaProfiler, Result,
 };
 use gc::{Finalize, Trace};
@@ -29,9 +29,6 @@ use rustc_hash::FxHashMap;
 use std::fmt::{Debug, Display, Error, Formatter};
 use std::{any::Any, result::Result as StdResult};
 
-use super::function::{make_builtin_fn, make_constructor_fn};
-use crate::builtins::value::same_value;
-
 mod gcobject;
 mod internal_methods;
 mod iter;

boa/src/builtins/property/mod.rs

@@ -14,9 +14,7 @@
 //! [mdn]: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Object/defineProperty
 //! [section]: https://tc39.es/ecma262/#sec-property-attributes
 
-use crate::builtins::value::RcString;
+use crate::value::{RcString, RcSymbol, Value};
-use crate::builtins::value::RcSymbol;
-use crate::builtins::Value;
 use gc::{Finalize, Trace};
 use std::convert::TryFrom;
 use std::fmt;

boa/src/builtins/regexp/mod.rs

@@ -13,12 +13,9 @@ use regex::Regex;
 
 use super::function::{make_builtin_fn, make_constructor_fn};
 use crate::{
-    builtins::{
+    builtins::{object::ObjectData, property::Property},
-        object::ObjectData,
-        property::Property,
-        value::{RcString, Value},
-    },
     exec::Interpreter,
+    value::{RcString, Value},
     BoaProfiler, Result,
 };
 use gc::{unsafe_empty_trace, Finalize, Trace};

boa/src/builtins/string/mod.rs

@@ -17,10 +17,10 @@ use crate::{
     builtins::{
         object::{Object, ObjectData},
         property::Property,
-        value::{RcString, Value},
         RegExp,
     },
     exec::Interpreter,
+    value::{RcString, Value},
     BoaProfiler, Result,
 };
 use regex::Regex;

boa/src/builtins/symbol/mod.rs

@@ -20,11 +20,9 @@ mod tests;
 
 use super::function::{make_builtin_fn, make_constructor_fn};
 use crate::{
-    builtins::{
+    builtins::property::{Attribute, Property},
-        property::{Attribute, Property},
-        value::{RcString, RcSymbol, Value},
-    },
     exec::Interpreter,
+    value::{RcString, RcSymbol, Value},
     BoaProfiler, Result,
 };
 use gc::{Finalize, Trace};

boa/src/builtins/undefined/mod.rs

@@ -12,7 +12,7 @@
 #[cfg(test)]
 mod tests;
 
-use crate::{builtins::value::Value, BoaProfiler, Interpreter};
+use crate::{BoaProfiler, Interpreter, Value};
 
 /// JavaScript global `undefined` property.
 #[derive(Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash)]

boa/src/class.rs

@@ -3,12 +3,12 @@
 //! Native classes are implemented through the [`Class`][class-trait] trait.
 //! ```
 //!# use boa::{
-//!#    builtins::{property::Attribute, value::Value},
+//!#    builtins::property::Attribute,
 //!#    class::{Class, ClassBuilder},
 //!#    exec::Interpreter,
 //!#    forward_val,
 //!#    realm::Realm,
-//!#    Finalize, Result, Trace,
+//!#    Finalize, Value, Result, Trace,
 //!# };
 //!#
 //! // This does not have to be an enum it can also be a struct.
@@ -67,9 +67,8 @@ use crate::{
         function::{BuiltInFunction, Function, FunctionFlags, NativeFunction},
         object::{GcObject, NativeObject, Object, ObjectData, PROTOTYPE},
         property::{Attribute, Property, PropertyKey},
-        Value,
     },
-    Interpreter, Result,
+    Interpreter, Result, Value,
 };
 use std::fmt::Debug;
 

boa/src/environment/declarative_environment_record.rs

@@ -6,11 +6,11 @@
 //! More info:  [ECMA-262 sec-declarative-environment-records](https://tc39.es/ecma262/#sec-declarative-environment-records)
 
 use crate::{
-    builtins::value::Value,
     environment::{
         environment_record_trait::EnvironmentRecordTrait,
         lexical_environment::{Environment, EnvironmentType},
     },
+    Value,
 };
 use gc::{Finalize, Trace};
 use rustc_hash::FxHashMap;

boa/src/environment/environment_record_trait.rs

@@ -9,8 +9,8 @@
 //! There are 5 Environment record kinds. They all have methods in common, these are implemented as a the `EnvironmentRecordTrait`
 //!
 use crate::{
-    builtins::value::Value,
     environment::lexical_environment::{Environment, EnvironmentType},
+    Value,
 };
 use gc::{Finalize, Trace};
 use std::fmt::Debug;

boa/src/environment/function_environment_record.rs

@@ -9,12 +9,13 @@
 //! More info: <https://tc39.es/ecma262/#sec-function-environment-records>
 
 use crate::{
-    builtins::{object::GcObject, value::Value},
+    builtins::object::GcObject,
     environment::{
         declarative_environment_record::DeclarativeEnvironmentRecordBinding,
         environment_record_trait::EnvironmentRecordTrait,
         lexical_environment::{Environment, EnvironmentType},
     },
+    Value,
 };
 use gc::{unsafe_empty_trace, Finalize, Trace};
 use rustc_hash::FxHashMap;

boa/src/environment/global_environment_record.rs

@@ -8,16 +8,14 @@
 //! More info:  <https://tc39.es/ecma262/#sec-global-environment-records>
 
 use crate::{
-    builtins::{
+    builtins::property::{Attribute, Property},
-        property::{Attribute, Property},
-        Value,
-    },
     environment::{
         declarative_environment_record::DeclarativeEnvironmentRecord,
         environment_record_trait::EnvironmentRecordTrait,
         lexical_environment::{Environment, EnvironmentType},
         object_environment_record::ObjectEnvironmentRecord,
     },
+    Value,
 };
 use gc::{Finalize, Trace};
 use rustc_hash::FxHashSet;

boa/src/environment/lexical_environment.rs

@@ -6,7 +6,7 @@
 //! This is the entrypoint to lexical environments.
 
 use crate::{
-    builtins::{object::GcObject, value::Value},
+    builtins::object::GcObject,
     environment::{
         declarative_environment_record::DeclarativeEnvironmentRecord,
         environment_record_trait::EnvironmentRecordTrait,
@@ -14,7 +14,7 @@ use crate::{
         global_environment_record::GlobalEnvironmentRecord,
         object_environment_record::ObjectEnvironmentRecord,
     },
-    BoaProfiler,
+    BoaProfiler, Value,
 };
 use gc::{Gc, GcCell};
 use rustc_hash::{FxHashMap, FxHashSet};

boa/src/environment/object_environment_record.rs

@@ -7,14 +7,12 @@
 //! More info:  [Object Records](https://tc39.es/ecma262/#sec-object-environment-records)
 
 use crate::{
-    builtins::{
+    builtins::property::{Attribute, Property},
-        property::{Attribute, Property},
-        value::Value,
-    },
     environment::{
         environment_record_trait::EnvironmentRecordTrait,
         lexical_environment::{Environment, EnvironmentType},
     },
+    Value,
 };
 use gc::{Finalize, Trace};
 

boa/src/exec/array/mod.rs

@@ -2,9 +2,9 @@
 
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::{Array, Value},
+    builtins::Array,
     syntax::ast::node::{ArrayDecl, Node},
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 
 impl Executable for ArrayDecl {

boa/src/exec/block/mod.rs

@@ -2,8 +2,8 @@
 
 use super::{Executable, Interpreter, InterpreterState};
 use crate::{
-    builtins::value::Value, environment::lexical_environment::new_declarative_environment,
+    environment::lexical_environment::new_declarative_environment, syntax::ast::node::Block,
-    syntax::ast::node::Block, BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 
 impl Executable for Block {

boa/src/exec/break_node/mod.rs

@@ -1,8 +1,7 @@
 use super::{Executable, Interpreter, InterpreterState};
 use crate::{
-    builtins::value::Value,
     syntax::ast::node::{Break, Continue},
-    Result,
+    Result, Value,
 };
 
 #[cfg(test)]

boa/src/exec/call/mod.rs

@@ -1,7 +1,7 @@
 use super::{Executable, Interpreter, InterpreterState};
 use crate::{
-    builtins::value::{Type, Value},
     syntax::ast::node::{Call, Node},
+    value::{Type, Value},
     BoaProfiler, Result,
 };
 

boa/src/exec/conditional/mod.rs

@@ -1,8 +1,7 @@
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::Value,
     syntax::ast::node::{ConditionalOp, If},
-    Result,
+    Result, Value,
 };
 use std::borrow::Borrow;
 

boa/src/exec/declaration/mod.rs

@@ -2,12 +2,12 @@
 
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::{function::FunctionFlags, Value},
+    builtins::function::FunctionFlags,
     environment::lexical_environment::VariableScope,
     syntax::ast::node::{
         ArrowFunctionDecl, ConstDeclList, FunctionDecl, FunctionExpr, LetDeclList, VarDeclList,
     },
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 
 impl Executable for FunctionDecl {

boa/src/exec/field/mod.rs

@@ -1,7 +1,7 @@
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::value::{Type, Value},
     syntax::ast::node::{GetConstField, GetField},
+    value::{Type, Value},
     Result,
 };
 

boa/src/exec/identifier/mod.rs

@@ -1,5 +1,5 @@
 use super::{Executable, Interpreter};
-use crate::{builtins::value::Value, syntax::ast::node::identifier::Identifier, Result};
+use crate::{syntax::ast::node::identifier::Identifier, Result, Value};
 
 impl Executable for Identifier {
     fn run(&self, interpreter: &mut Interpreter) -> Result<Value> {

boa/src/exec/iteration/mod.rs

@@ -2,10 +2,9 @@
 
 use super::{Executable, Interpreter, InterpreterState};
 use crate::{
-    builtins::value::Value,
     environment::lexical_environment::new_declarative_environment,
     syntax::ast::node::{DoWhileLoop, ForLoop, WhileLoop},
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 use std::borrow::Borrow;
 

boa/src/exec/mod.rs

@@ -28,7 +28,6 @@ use crate::{
         function::{Function, FunctionFlags, NativeFunction},
         object::{GcObject, Object, ObjectData, PROTOTYPE},
         property::{Property, PropertyKey},
-        value::{PreferredType, RcString, RcSymbol, Type, Value},
         Console, Symbol,
     },
     class::{Class, ClassBuilder},
@@ -37,6 +36,7 @@ use crate::{
         constant::Const,
         node::{FormalParameter, Node, RcStatementList, StatementList},
     },
+    value::{PreferredType, RcString, RcSymbol, Type, Value},
     BoaProfiler, Result,
 };
 use std::result::Result as StdResult;

boa/src/exec/new/mod.rs

@@ -1,5 +1,5 @@
 use super::{Executable, Interpreter};
-use crate::{builtins::Value, syntax::ast::node::New, BoaProfiler, Result};
+use crate::{syntax::ast::node::New, BoaProfiler, Result, Value};
 
 impl Executable for New {
     fn run(&self, interpreter: &mut Interpreter) -> Result<Value> {

boa/src/exec/object/mod.rs

@@ -2,10 +2,9 @@
 
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::value::Value,
     syntax::ast::node::MethodDefinitionKind,
     syntax::ast::node::{Object, PropertyDefinition},
-    Result,
+    Result, Value,
 };
 
 use std::borrow::Borrow;

boa/src/exec/operator/mod.rs

@@ -4,13 +4,12 @@ mod tests;
 
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::value::Value,
     environment::lexical_environment::VariableScope,
     syntax::ast::{
         node::{Assign, BinOp, Node, UnaryOp},
         op::{self, AssignOp, BitOp, CompOp, LogOp, NumOp},
     },
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 
 impl Executable for Assign {

boa/src/exec/return_smt/mod.rs

@@ -1,5 +1,5 @@
 use super::{Executable, Interpreter, InterpreterState};
-use crate::{builtins::value::Value, syntax::ast::node::Return, Result};
+use crate::{syntax::ast::node::Return, Result, Value};
 
 impl Executable for Return {
     fn run(&self, interpreter: &mut Interpreter) -> Result<Value> {

boa/src/exec/spread/mod.rs

@@ -1,5 +1,5 @@
 use super::{Executable, Interpreter};
-use crate::{builtins::value::Value, syntax::ast::node::Spread, Result};
+use crate::{syntax::ast::node::Spread, Result, Value};
 
 impl Executable for Spread {
     fn run(&self, interpreter: &mut Interpreter) -> Result<Value> {

boa/src/exec/statement_list.rs

@@ -1,7 +1,7 @@
 //! Statement list execution.
 
 use super::{Executable, Interpreter, InterpreterState};
-use crate::{builtins::value::Value, syntax::ast::node::StatementList, BoaProfiler, Result};
+use crate::{syntax::ast::node::StatementList, BoaProfiler, Result, Value};
 
 impl Executable for StatementList {
     fn run(&self, interpreter: &mut Interpreter) -> Result<Value> {

boa/src/exec/switch/mod.rs

@@ -1,5 +1,5 @@
 use super::{Executable, Interpreter, InterpreterState};
-use crate::{builtins::value::Value, syntax::ast::node::Switch, Result};
+use crate::{syntax::ast::node::Switch, Result, Value};
 
 #[cfg(test)]
 mod tests;

boa/src/exec/tests.rs

@@ -1,10 +1,4 @@
-use crate::{
+use crate::{builtins::Number, exec, exec::Interpreter, forward, forward_val, realm::Realm, Value};
-    builtins::{Number, Value},
-    exec,
-    exec::Interpreter,
-    forward, forward_val,
-    realm::Realm,
-};
 
 #[test]
 fn function_declaration_returns_undefined() {

boa/src/exec/throw/mod.rs

@@ -1,5 +1,5 @@
 use super::{Executable, Interpreter};
-use crate::{builtins::value::Value, syntax::ast::node::Throw, Result};
+use crate::{syntax::ast::node::Throw, Result, Value};
 
 impl Executable for Throw {
     #[inline]

boa/src/exec/try_node/mod.rs

@@ -2,10 +2,9 @@
 
 use super::{Executable, Interpreter};
 use crate::{
-    builtins::value::Value,
     environment::lexical_environment::{new_declarative_environment, VariableScope},
     syntax::ast::node::Try,
-    BoaProfiler, Result,
+    BoaProfiler, Result, Value,
 };
 
 #[cfg(test)]

boa/src/lib.rs

@@ -41,8 +41,9 @@ pub mod exec;
 pub mod profiler;
 pub mod realm;
 pub mod syntax;
+pub mod value;
 
-use crate::{builtins::value::Value, syntax::ast::node::StatementList};
+use crate::syntax::ast::node::StatementList;
 pub use crate::{
     exec::{Executable, Interpreter},
     profiler::BoaProfiler,
@@ -51,6 +52,7 @@ pub use crate::{
         lexer::Lexer,
         parser::{ParseError, Parser},
     },
+    value::Value,
 };
 use std::result::Result as StdResult;
 

boa/src/realm.rs

@@ -5,14 +5,13 @@
 //! A realm is represented in this implementation as a Realm struct with the fields specified from the spec.
 
 use crate::{
-    builtins::value::Value,
     environment::{
         declarative_environment_record::DeclarativeEnvironmentRecord,
         global_environment_record::GlobalEnvironmentRecord,
         lexical_environment::LexicalEnvironment,
         object_environment_record::ObjectEnvironmentRecord,
     },
-    BoaProfiler,
+    BoaProfiler, Value,
 };
 use gc::{Gc, GcCell};
 use rustc_hash::{FxHashMap, FxHashSet};

boa/src/value/mod.rs

@@ -5,8 +5,8 @@
 #[cfg(test)]
 mod tests;
 
-use super::number::{f64_to_int32, f64_to_uint32};
 use crate::builtins::{
+    number::{f64_to_int32, f64_to_uint32},
     object::{GcObject, Object, ObjectData, PROTOTYPE},
     property::{Attribute, Property, PropertyKey},
     BigInt, Number,
@@ -641,7 +641,7 @@ impl Value {
     /// # Examples
     ///
     /// ```
-    /// use boa::builtins::value::Value;
+    /// use boa::Value;
     ///
     /// let value = Value::number(3);
     ///

boa/src/value/operations.rs

@@ -1,8 +1,5 @@
 use super::*;
-use crate::builtins::{
+use crate::builtins::number::{f64_to_int32, f64_to_uint32, Number};
-    number::{f64_to_int32, f64_to_uint32, Number},
-    value::PreferredType,
-};
 
 impl Value {
     #[inline]