Fix gemini comments

Author: j-sanaa

packages/agentflow/src/atoms/ArrayInput.test.tsx

@@ -281,7 +281,32 @@ describe('ArrayInput', () => {
         expect(deleteButtons[1]).toBeDisabled()
     })
 
-    // Test 11: itemParameters prop overrides inputParam.array display flags
+    // Test 11: Type-specific defaults when no explicit default is provided
+    it('should initialize new items with type-appropriate defaults when no default is specified', () => {
+        const inputParamWithTypes: InputParam = {
+            id: 'typed-array',
+            name: 'testArray',
+            label: 'Test Item',
+            type: 'array',
+            array: [
+                { id: 'str', name: 'str', label: 'String', type: 'string' } as InputParam,
+                { id: 'num', name: 'num', label: 'Number', type: 'number' } as InputParam,
+                { id: 'bool', name: 'bool', label: 'Boolean', type: 'boolean' } as InputParam,
+                { id: 'arr', name: 'arr', label: 'Array', type: 'array' } as InputParam
+            ]
+        }
+
+        render(<ArrayInput inputParam={inputParamWithTypes} data={mockNodeData} onDataChange={mockOnDataChange} />)
+
+        fireEvent.click(screen.getByRole('button', { name: /Add Test Item/i }))
+
+        expect(mockOnDataChange).toHaveBeenCalledWith({
+            inputParam: inputParamWithTypes,
+            newValue: [{ str: '', num: 0, bool: false, arr: [] }]
+        })
+    })
+
+    // Test 12: itemParameters prop overrides inputParam.array display flags
     it('should use itemParameters prop for field visibility when provided, ignoring inputParam.array display flags', () => {
         // inputParam.array has both fields with no display flag (both would show)
         const dataWithItem: NodeData = {

packages/agentflow/src/atoms/ArrayInput.tsx

@@ -51,12 +51,28 @@ export function ArrayInput({ inputParam, data, disabled = false, onDataChange, i
 
     // Add new array item
     const handleAddItem = useCallback(() => {
-        // Initialize new item with default values
+        // Initialize new item with type-appropriate default values
         const newItem: Record<string, unknown> = {}
 
         if (inputParam.array) {
             for (const field of inputParam.array) {
-                newItem[field.name] = field.default ?? ''
+                if (field.default !== undefined) {
+                    newItem[field.name] = field.default
+                } else {
+                    switch (field.type) {
+                        case 'number':
+                            newItem[field.name] = 0
+                            break
+                        case 'boolean':
+                            newItem[field.name] = false
+                            break
+                        case 'array':
+                            newItem[field.name] = []
+                            break
+                        default:
+                            newItem[field.name] = ''
+                    }
+                }
             }
         }
 

packages/agentflow/src/features/node-editor/EditNodeDialog.test.tsx

@@ -44,11 +44,7 @@ jest.mock('@/atoms', () => ({
                     <button
                         data-testid={`add-${inputParam.name}`}
                         onClick={() => {
-                            const newItem = inputParam.array?.reduce((acc, field) => {
-                                acc[field.name] = field.default ?? ''
-                                return acc
-                            }, {} as Record<string, unknown>)
-                            onDataChange({ inputParam, newValue: [...currentArray, newItem || {}] })
+                            onDataChange({ inputParam, newValue: [...currentArray, { _mockAdded: true }] })
                         }}
                     >
                         Add {inputParam.label}
@@ -381,17 +377,13 @@ describe('EditNodeDialog', () => {
             expect(screen.getByTestId('change-connections-0')).toBeInTheDocument()
             expect(screen.getByTestId('change-connections-1')).toBeInTheDocument()
 
-            // Test Add operation - adds a new item with default values
+            // Test Add operation - appends a new item to the array
             const addButton = screen.getByTestId('add-connections')
             fireEvent.click(addButton)
 
             expect(mockUpdateNodeData).toHaveBeenCalledWith('node-1', {
                 inputValues: {
-                    connections: [
-                        { host: 'server1.com', port: 3000 },
-                        { host: 'server2.com', port: 8080 },
-                        { host: 'localhost', port: 5432 }
-                    ]
+                    connections: [{ host: 'server1.com', port: 3000 }, { host: 'server2.com', port: 8080 }, { _mockAdded: true }]
                 }
             })