nodenetcdf 4.9.32 → 4.9.33

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "nodenetcdf",
-  "version": "4.9.32",
+  "version": "4.9.33",
   "description": "Read and write NodeNetCDF files",
   "main": "./build/Release/nodenetcdf.node",
   "types": "./index.d.ts",

package/src/File.cpp

@@ -174,11 +174,27 @@ void File::ToJSON(const v8::FunctionCallbackInfo<v8::Value> &args)
     v8::Isolate *isolate = args.GetIsolate();
     v8::Local<v8::Context> context = isolate->GetCurrentContext();
     
-    // Get the root group and serialize it
+    // Get the root group
     v8::Local<v8::String> rootProp = v8::String::NewFromUtf8(isolate, "root", v8::NewStringType::kNormal).ToLocalChecked();
     v8::Local<v8::Value> root = args.Holder()->Get(context, rootProp).ToLocalChecked();
     
-    // The root group's toJSON will automatically serialize all children recursively
+    // Call toJSON on the root group to get proper array conversion
+    if (root->IsObject())
+    {
+        v8::Local<v8::Object> rootObj = root->ToObject(context).ToLocalChecked();
+        v8::Local<v8::String> toJSONProp = v8::String::NewFromUtf8(isolate, "toJSON", v8::NewStringType::kNormal).ToLocalChecked();
+        v8::Local<v8::Value> toJSONMethod = rootObj->Get(context, toJSONProp).ToLocalChecked();
+        
+        if (toJSONMethod->IsFunction())
+        {
+            v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSONMethod);
+            v8::Local<v8::Value> result = toJSONFunc->Call(context, rootObj, 0, nullptr).ToLocalChecked();
+            args.GetReturnValue().Set(result);
+            return;
+        }
+    }
+    
+    // Fallback: return root as-is
     args.GetReturnValue().Set(root);
 }
 } // namespace nodenetcdfjs

package/src/Group.cpp

@@ -525,69 +525,121 @@ void Group::ToJSON(const v8::FunctionCallbackInfo<v8::Value> &args)
         }
     }
     
-    // Convert dimensions object to array
+    // Convert dimensions object to array, calling toJSON on each item
     v8::Local<v8::Value> dimensions = args.Holder()->Get(context, dimensions_str).ToLocalChecked();
     if (dimensions->IsObject() && !dimensions->IsNull())
     {
         v8::Local<v8::Object> dimsObj = dimensions->ToObject(context).ToLocalChecked();
         v8::Local<v8::Array> propNames = dimsObj->GetOwnPropertyNames(context).ToLocalChecked();
         v8::Local<v8::Array> dimsArray = v8::Array::New(isolate, propNames->Length());
+        v8::Local<v8::String> toJSONStr = v8::String::NewFromUtf8Literal(isolate, "toJSON");
         
         for (uint32_t i = 0; i < propNames->Length(); i++)
         {
             v8::Local<v8::Value> key = propNames->Get(context, i).ToLocalChecked();
             v8::Local<v8::Value> value = dimsObj->Get(context, key).ToLocalChecked();
+            
+            // Call toJSON if available to ensure proper serialization
+            if (value->IsObject())
+            {
+                v8::Local<v8::Object> valueObj = value->ToObject(context).ToLocalChecked();
+                v8::Local<v8::Value> toJSON = valueObj->Get(context, toJSONStr).ToLocalChecked();
+                if (toJSON->IsFunction())
+                {
+                    v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSON);
+                    value = toJSONFunc->Call(context, valueObj, 0, nullptr).ToLocalChecked();
+                }
+            }
             (void)dimsArray->Set(context, i, value);
         }
         (void)json->CreateDataProperty(context, dimensions_str, dimsArray);
     }
     
-    // Convert variables object to array
+    // Convert variables object to array, calling toJSON on each item
     v8::Local<v8::Value> variables = args.Holder()->Get(context, variables_str).ToLocalChecked();
     if (variables->IsObject() && !variables->IsNull())
     {
         v8::Local<v8::Object> varsObj = variables->ToObject(context).ToLocalChecked();
         v8::Local<v8::Array> propNames = varsObj->GetOwnPropertyNames(context).ToLocalChecked();
         v8::Local<v8::Array> varsArray = v8::Array::New(isolate, propNames->Length());
+        v8::Local<v8::String> toJSONStr = v8::String::NewFromUtf8Literal(isolate, "toJSON");
         
         for (uint32_t i = 0; i < propNames->Length(); i++)
         {
             v8::Local<v8::Value> key = propNames->Get(context, i).ToLocalChecked();
             v8::Local<v8::Value> value = varsObj->Get(context, key).ToLocalChecked();
+            
+            // Call toJSON if available to ensure proper serialization
+            if (value->IsObject())
+            {
+                v8::Local<v8::Object> valueObj = value->ToObject(context).ToLocalChecked();
+                v8::Local<v8::Value> toJSON = valueObj->Get(context, toJSONStr).ToLocalChecked();
+                if (toJSON->IsFunction())
+                {
+                    v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSON);
+                    value = toJSONFunc->Call(context, valueObj, 0, nullptr).ToLocalChecked();
+                }
+            }
             (void)varsArray->Set(context, i, value);
         }
         (void)json->CreateDataProperty(context, variables_str, varsArray);
     }
     
-    // Convert attributes object to array
+    // Convert attributes object to array, calling toJSON on each item
     v8::Local<v8::Value> attributes = args.Holder()->Get(context, attributes_str).ToLocalChecked();
     if (attributes->IsObject() && !attributes->IsNull())
     {
         v8::Local<v8::Object> attrsObj = attributes->ToObject(context).ToLocalChecked();
         v8::Local<v8::Array> propNames = attrsObj->GetOwnPropertyNames(context).ToLocalChecked();
         v8::Local<v8::Array> attrsArray = v8::Array::New(isolate, propNames->Length());
+        v8::Local<v8::String> toJSONStr = v8::String::NewFromUtf8Literal(isolate, "toJSON");
         
         for (uint32_t i = 0; i < propNames->Length(); i++)
         {
             v8::Local<v8::Value> key = propNames->Get(context, i).ToLocalChecked();
             v8::Local<v8::Value> value = attrsObj->Get(context, key).ToLocalChecked();
+            
+            // Call toJSON if available to ensure proper serialization
+            if (value->IsObject())
+            {
+                v8::Local<v8::Object> valueObj = value->ToObject(context).ToLocalChecked();
+                v8::Local<v8::Value> toJSON = valueObj->Get(context, toJSONStr).ToLocalChecked();
+                if (toJSON->IsFunction())
+                {
+                    v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSON);
+                    value = toJSONFunc->Call(context, valueObj, 0, nullptr).ToLocalChecked();
+                }
+            }
             (void)attrsArray->Set(context, i, value);
         }
         (void)json->CreateDataProperty(context, attributes_str, attrsArray);
     }
     
-    // Convert subgroups object to array
+    // Convert subgroups object to array, calling toJSON on each item
     v8::Local<v8::Value> subgroups = args.Holder()->Get(context, subgroups_str).ToLocalChecked();
     if (subgroups->IsObject() && !subgroups->IsNull())
     {
         v8::Local<v8::Object> subgrpsObj = subgroups->ToObject(context).ToLocalChecked();
         v8::Local<v8::Array> propNames = subgrpsObj->GetOwnPropertyNames(context).ToLocalChecked();
         v8::Local<v8::Array> subgrpsArray = v8::Array::New(isolate, propNames->Length());
+        v8::Local<v8::String> toJSONStr = v8::String::NewFromUtf8Literal(isolate, "toJSON");
         
         for (uint32_t i = 0; i < propNames->Length(); i++)
         {
             v8::Local<v8::Value> key = propNames->Get(context, i).ToLocalChecked();
             v8::Local<v8::Value> value = subgrpsObj->Get(context, key).ToLocalChecked();
+            
+            // Call toJSON if available to ensure proper serialization
+            if (value->IsObject())
+            {
+                v8::Local<v8::Object> valueObj = value->ToObject(context).ToLocalChecked();
+                v8::Local<v8::Value> toJSON = valueObj->Get(context, toJSONStr).ToLocalChecked();
+                if (toJSON->IsFunction())
+                {
+                    v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSON);
+                    value = toJSONFunc->Call(context, valueObj, 0, nullptr).ToLocalChecked();
+                }
+            }
             (void)subgrpsArray->Set(context, i, value);
         }
         (void)json->CreateDataProperty(context, subgroups_str, subgrpsArray);

package/src/Variable.cpp

@@ -1383,25 +1383,59 @@ void Variable::ToJSON(const v8::FunctionCallbackInfo<v8::Value> &args)
     (void)json->CreateDataProperty(context, type_str,
               v8::String::NewFromUtf8(isolate, type_name, v8::NewStringType::kInternalized).ToLocalChecked());
     
-    // Dimensions are already an array, just add them
+    // Dimensions are already an array, but call toJSON on each item
     v8::Local<v8::Value> dimensions = args.Holder()->Get(context, dimensions_str).ToLocalChecked();
     if (dimensions->IsArray())
     {
-        (void)json->CreateDataProperty(context, dimensions_str, dimensions);
+        v8::Local<v8::Array> dimsArray = v8::Local<v8::Array>::Cast(dimensions);
+        v8::Local<v8::Array> newDimsArray = v8::Array::New(isolate, dimsArray->Length());
+        v8::Local<v8::String> toJSONStr = v8::String::NewFromUtf8Literal(isolate, "toJSON");
+        
+        for (uint32_t i = 0; i < dimsArray->Length(); i++)
+        {
+            v8::Local<v8::Value> value = dimsArray->Get(context, i).ToLocalChecked();
+            
+            // Call toJSON if available to ensure proper serialization
+            if (value->IsObject())
+            {
+                v8::Local<v8::Object> valueObj = value->ToObject(context).ToLocalChecked();
+                v8::Local<v8::Value> toJSON = valueObj->Get(context, toJSONStr).ToLocalChecked();
+                if (toJSON->IsFunction())
+                {
+                    v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSON);
+                    value = toJSONFunc->Call(context, valueObj, 0, nullptr).ToLocalChecked();
+                }
+            }
+            (void)newDimsArray->Set(context, i, value);
+        }
+        (void)json->CreateDataProperty(context, dimensions_str, newDimsArray);
     }
     
-    // Convert attributes object to array
+    // Convert attributes object to array, calling toJSON on each item
     v8::Local<v8::Value> attributes = args.Holder()->Get(context, attributes_str).ToLocalChecked();
     if (attributes->IsObject() && !attributes->IsNull())
     {
         v8::Local<v8::Object> attrsObj = attributes->ToObject(context).ToLocalChecked();
         v8::Local<v8::Array> propNames = attrsObj->GetOwnPropertyNames(context).ToLocalChecked();
         v8::Local<v8::Array> attrsArray = v8::Array::New(isolate, propNames->Length());
+        v8::Local<v8::String> toJSONStr = v8::String::NewFromUtf8Literal(isolate, "toJSON");
         
         for (uint32_t i = 0; i < propNames->Length(); i++)
         {
             v8::Local<v8::Value> key = propNames->Get(context, i).ToLocalChecked();
             v8::Local<v8::Value> value = attrsObj->Get(context, key).ToLocalChecked();
+            
+            // Call toJSON if available to ensure proper serialization
+            if (value->IsObject())
+            {
+                v8::Local<v8::Object> valueObj = value->ToObject(context).ToLocalChecked();
+                v8::Local<v8::Value> toJSON = valueObj->Get(context, toJSONStr).ToLocalChecked();
+                if (toJSON->IsFunction())
+                {
+                    v8::Local<v8::Function> toJSONFunc = v8::Local<v8::Function>::Cast(toJSON);
+                    value = toJSONFunc->Call(context, valueObj, 0, nullptr).ToLocalChecked();
+                }
+            }
             (void)attrsArray->Set(context, i, value);
         }
         (void)json->CreateDataProperty(context, attributes_str, attrsArray);

package/test/json.js

@@ -164,11 +164,15 @@ describe('JSON Serialization', function() {
         it('should recursively serialize entire file structure', function() {
             var json = JSON.parse(JSON.stringify(file));
             
-            // File should serialize as root group with all children
+            // File should serialize as root group with all children as arrays
             expect(json).to.have.property('dimensions');
             expect(json).to.have.property('variables');
-            expect(json.variables).to.be.an('object');
-            expect(json.dimensions).to.be.an('object');
+            expect(json).to.have.property('attributes');
+            expect(json).to.have.property('subgroups');
+            expect(json.variables).to.be.an('array');
+            expect(json.dimensions).to.be.an('array');
+            expect(json.attributes).to.be.an('array');
+            expect(json.subgroups).to.be.an('array');
         });
     });
 });