fix(designer): Complex array items to only be required if ancestors are all required

[Eric-B-Wu]

Fixes #5084

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Comments suppressed due to low confidence (7)

libs/designer-ui/src/lib/arrayeditor/util/util.ts:228

• Removing the error parameter from the catch block can make debugging harder. Consider keeping the error parameter for better error handling.

catch {

libs/designer-ui/src/lib/arrayeditor/util/util.ts:273

• Removing the error parameter from the catch block can make debugging harder. Consider keeping the error parameter for better error handling.

catch {

libs/designer-ui/src/lib/arrayeditor/util/util.ts:105

• [nitpick] The function name handleArrayItems could be more descriptive. Consider renaming it to something like handleArrayItemsInComplexArray.

const handleArrayItems = (

libs/designer-ui/src/lib/arrayeditor/util/util.ts:130

• [nitpick] The function name handleSimpleItem could be more descriptive. Consider renaming it to something like handleSimpleItemInComplexArray.

const handleSimpleItem = (

libs/designer-ui/src/lib/arrayeditor/util/util.ts:150

• [nitpick] The function name handleArrayOfComplexItems could be more descriptive. Consider renaming it to something like handleArrayOfComplexItemsInArray.

const handleArrayOfComplexItems = (

libs/designer-ui/src/lib/arrayeditor/util/util.ts:31

• Ensure that the new behavior introduced by the isRequired parameter is adequately covered by tests.

export const getOneDimensionalSchema = (itemSchema: ArrayItemSchema, isRequired: boolean): ItemSchemaItemProps[] => {

libs/designer-ui/src/lib/arrayeditor/index.tsx:136

• Ensure that the new isRequired behavior is covered by tests.

const objectValue = parseComplexItems(newItems, itemSchema, isRequired, castParameter, suppressCastingForSerialize);