fix: update default value handling in array and object decorators

Author: yakud

packages/chaingraph-types/src/decorator/array.decorator.ts

@@ -125,7 +125,7 @@ export function PortArrayObject<T extends ObjectPortSchemaInput>(
 ): PropertyDecorator {
   return PortArray({
     // Preset the item configuration for object elements with the provided schema.
-    itemConfig: { type: 'object', schema, defaultValue: {} },
+    itemConfig: { type: 'object', schema },
     ...config,
   })
 }

packages/chaingraph-types/src/node/__test__/complex-node.test.ts

@@ -100,16 +100,19 @@ type UserStatusOptionId = keyof typeof userStatusOptions
 export class TestUserAddress {
   @PortString({
     description: 'Street of the address',
+    defaultValue: '',
   })
   street?: string = ''
 
   @PortString({
     description: 'City of the address',
+    defaultValue: '',
   })
   city?: string = ''
 
   @PortString({
     description: 'State of the address',
+    defaultValue: '',
   })
   country?: string = 'EU'
 
@@ -135,21 +138,25 @@ export class TestUserObject {
 
   @PortString({
     description: 'Username of the user',
+    defaultValue: '',
   })
   username: string = ''
 
   @PortString({
     description: 'Name of the user',
+    defaultValue: '',
   })
   name: string = ''
 
   @PortNumber({
     description: 'Age of the user',
+    defaultValue: 0,
   })
   age: number = 0
 
   @PortNumber({
     description: 'Age of the user decimal',
+    defaultValue: 0,
   })
   ageDecimal: number = 0
 
@@ -167,6 +174,7 @@ export class TestUserObject {
       type: 'string',
       defaultValue: '',
     },
+    defaultValue: [],
   })
   emails: string[] = []
 
@@ -225,6 +233,7 @@ export class UserProfileNode extends BaseNode {
   // Case for infer schema from field value
   @Input() @PortObject({
     schema: TestUserObject,
+    defaultValue: new TestUserObject(),
   })
   user1: TestUserObject = new TestUserObject()
 
@@ -331,6 +340,7 @@ export class UserProfileNode extends BaseNode {
         defaultValue: new TestUserObject(),
       },
     },
+    defaultValue: [],
   })
   users2DArray: TestUserObject[][] = []
 
@@ -417,7 +427,9 @@ export class UserProfileNode extends BaseNode {
   numberArray?: number[]
 
   @Output()
-  @PortArrayObject(TestUserObject)
+  @PortArrayObject(TestUserObject, {
+    defaultValue: [],
+  })
   simpleObjectArray?: TestUserObject[] = []
 
   @Output()

packages/chaingraph-types/src/port/plugins/NumberPortPlugin.ts

@@ -231,7 +231,7 @@ export const NumberPortPlugin: IPortPlugin<'number'> = {
           )
         }
       }
-      return data || undefined
+      return data
     } catch (error) {
       throw new PortError(
         PortErrorType.SerializationError,

packages/chaingraph-types/src/port/plugins/ObjectPortPlugin.ts

@@ -378,7 +378,7 @@ export const ObjectPortPlugin: IPortPlugin<'object'> = {
 
   deserializeValue: (data: JSONValue, config: ObjectPortConfig): ObjectPortValue => {
     if (data === undefined || data === null) {
-      return {}
+      return config.required ? {} : undefined
     }
     try {
       // Expecting an object with a "value" property that contains the serialized fields.
@@ -398,6 +398,14 @@ export const ObjectPortPlugin: IPortPlugin<'object'> = {
       for (const [key, fieldConfig] of Object.entries(config.schema.properties)) {
         const fieldSerializedValue = serializedFields[key]
         if (fieldSerializedValue === undefined) {
+          const defaultVal = fieldConfig.defaultValue
+          if (defaultVal !== undefined) {
+            deserialized[key] = defaultVal
+          } else if (fieldConfig.required) {
+            // If the field is required but missing and has no default, set to undefined or handle as needed.
+            deserialized[key] = undefined
+          }
+
           // If the field is required, you might want to throw an error.
           // For now, we simply continue to the next field.
           continue