group-array with required parent fails for empty array, despite minItems: 0 #247

Activity

1. sandrina-p

   added the

   Bug

   issue type on Jan 16, 2026
2. sandrina-p

   added

   bugSomething isn't workingSomething isn't working

   on Jan 16, 2026
3. sandrina-p commented on Jan 16, 2026

   

   sandrina-p

   on Jan 16, 2026

   Collaborator

   More actions

   Hi @mariada-ta! That's indeed a bug. We currently don't have bandwidth to fix but, could you give it a try? It's somewhere in the validateSchema function

   👍React with 👍1Reacted by mariada-ta
4. mariada-ta commented on Jan 22, 2026

   

   mariada-ta

   on Jan 22, 2026

   Author

   More actions

   I looked into the problem and would have fixed it like this:
   File: src/validation/schema.ts
   Line: 241 - 263

    if (schema.required && isObjectValue(value)) {
       const missingKeys = schema.required.filter((key: string) => {
         const fieldValue = value[key]
         const fieldSchema = schema.properties?.[key]
         // Field is considered missing if:
         // - it's undefined OR
         // - it's null AND treatNullAsUndefined option is true
         // - it's an array/object and it's empty
         if (Array.isArray(fieldValue)) {
           if (fieldSchema && getSchemaType(fieldSchema) === 'array') {
             const minItems = (fieldSchema as any).minItems ?? 0
             return fieldValue.length < minItems
           }
           return fieldValue.length === 0
         }
   
         if (isObjectValue(fieldValue)) {
           return Object.keys(fieldValue).length === 0
         }
   
         return fieldValue === undefined || (fieldValue === null && options.treatNullAsUndefined)
       })
   

   I added a check that looks to see if there is a minItems attribute in the property and, if so, whether the element meets the length requirement. This means that an empty array with minItems: 0 is now correct.
5. spiderocious commented on May 26, 2026

   

   spiderocious

   on May 26, 2026

   More actions

   Hi
   I'd like to take a crack at this one. I see @mariada-ta proposed a fix back in January but there's no PR open yet. @mariada-ta are you still working on it? If not, I'll pick it up. My plan is to fix the required check in validateSchema so an empty array no longer counts as a missing field (array length should be governed by minItems/maxItems, not required), and add tests covering the group-array + minItems: 0 case from the issue.