refactor(schema): extract removeConditionalFieldsFromRequired to reduce nesting complexity

Extract the nested logic for removing conditional fields from required arrays
into a separate helper function to satisfy the nestif linter (complexity was 10).

Co-Authored-By: Martin Vasko <Matovidlo2@gmail.com>

Author: devin-ai-integration[bot]

internal/pkg/encoding/json/schema/schema.go

@@ -214,29 +214,7 @@ func NormalizeSchema(schema []byte) ([]byte, error) {
 		}
 
 		// Remove conditionally required fields from the required array
-		if requiredVal, found := parentObj.Get("required"); found {
-			if requiredArr, ok := requiredVal.([]any); ok {
-				newRequired := make([]any, 0, len(requiredArr))
-				conditionalFields := make(map[string]bool)
-				for _, req := range reqs {
-					conditionalFields[req.fieldName] = true
-				}
-				for _, field := range requiredArr {
-					if fieldStr, ok := field.(string); ok {
-						if !conditionalFields[fieldStr] {
-							newRequired = append(newRequired, field)
-						}
-					} else {
-						newRequired = append(newRequired, field)
-					}
-				}
-				if len(newRequired) > 0 {
-					parentObj.Set("required", newRequired)
-				} else {
-					parentObj.Delete("required")
-				}
-			}
-		}
+		removeConditionalFieldsFromRequired(parentObj, reqs)
 
 		// Generate if/then/else constructs for each conditional requirement
 		allOfItems := make([]any, 0, len(reqs))
@@ -268,6 +246,37 @@ func NormalizeSchema(schema []byte) ([]byte, error) {
 	return normalized, nil
 }
 
+// removeConditionalFieldsFromRequired removes conditionally required fields from the required array.
+func removeConditionalFieldsFromRequired(parentObj *orderedmap.OrderedMap, reqs []conditionalRequirement) {
+	requiredVal, found := parentObj.Get("required")
+	if !found {
+		return
+	}
+	requiredArr, ok := requiredVal.([]any)
+	if !ok {
+		return
+	}
+
+	conditionalFields := make(map[string]bool)
+	for _, req := range reqs {
+		conditionalFields[req.fieldName] = true
+	}
+
+	newRequired := make([]any, 0, len(requiredArr))
+	for _, field := range requiredArr {
+		fieldStr, ok := field.(string)
+		if !ok || !conditionalFields[fieldStr] {
+			newRequired = append(newRequired, field)
+		}
+	}
+
+	if len(newRequired) > 0 {
+		parentObj.Set("required", newRequired)
+	} else {
+		parentObj.Delete("required")
+	}
+}
+
 // getObjectAtPath returns the orderedmap at the given path string.
 func getObjectAtPath(m *orderedmap.OrderedMap, pathStr string) *orderedmap.OrderedMap {
 	if pathStr == "" {