remove ItemArray type alias, update nestWrapper parameter to emscripten::val, and simplify array validation and processing.

Author: saumyaj3

libnest2d_js/libnest2d_js.cpp

@@ -32,10 +32,6 @@ EMSCRIPTEN_DECLARE_VAL_TYPE(PointList);
 EMSCRIPTEN_DECLARE_VAL_TYPE(ItemList);
 EMSCRIPTEN_DECLARE_VAL_TYPE(DoubleList);
 
-// Define a more specific type for Item arrays (JavaScript Array<Item>)
-// Represents: Item[] in JavaScript/TypeScript
-using ItemArray = emscripten::val; // This will be bound as Item[] in TypeScript
-
 // Helper function to convert JavaScript arrays to std::vector<Point>
 std::vector<Point> jsArrayToPointVector(const emscripten::val& jsArray) {
     std::vector<Point> vertices;
@@ -76,11 +72,8 @@ std::vector<double> jsArrayToVectorDouble(const emscripten::val& jsArray) {
 }
 
 // Wrapper function for nest() to handle JavaScript arrays
-size_t nestWrapper(ItemArray& jsItems, const Box& bin, long distance = 1, const NfpConfig& config = NfpConfig()) {
-    
-    if (!jsItems.isArray()) {
-        throw std::invalid_argument("First parameter must be an array of Items");
-    }
+// Wrapper function for nest() to handle JavaScript arrays
+size_t nestWrapper(emscripten::val jsItems, const Box& bin, long distance = 1, const NfpConfig& config = NfpConfig()) {
     // Convert JavaScript array to std::vector<Item>
     std::vector<Item> items;
     auto length = jsItems["length"].as<unsigned>();
@@ -95,18 +88,17 @@ size_t nestWrapper(ItemArray& jsItems, const Box& bin, long distance = 1, const
     if (distance <= 0) {
         distance = 1;
     }
-
     // Create nest config
     NestConfig<> nestConfig(config);
 
     // Call the nest function
     size_t result = nest(items, bin, distance, nestConfig);
-    
+
     // Copy results back to original JavaScript items
     for (size_t i = 0; i < items.size() && i < length; ++i) {
         jsItems.set(i, val(items[i]));
     }
-    
+
     return result;
 }