diff --git a/pkg/workflow/safe_output_validation_config.go b/pkg/workflow/safe_output_validation_config.go index ba9926d8a1f..2e989427e05 100644 --- a/pkg/workflow/safe_output_validation_config.go +++ b/pkg/workflow/safe_output_validation_config.go @@ -251,7 +251,7 @@ var ValidationConfig = map[string]TypeValidationConfig{ "create_project": { DefaultMax: 1, Fields: map[string]FieldValidation{ - "title": {Required: false, Type: "string", Sanitize: true, MaxLength: 256}, + "title": {Type: "string", Sanitize: true, MaxLength: 256}, "owner": {Type: "string", Sanitize: true, MaxLength: 128}, "owner_type": {Type: "string", Enum: []string{"org", "user"}}, "item_url": {Type: "string", Sanitize: true, MaxLength: 512, Pattern: "^https://github\\.com/[^/]+/[^/]+/issues/\\d+", PatternError: "must be a full GitHub issue URL (e.g., https://github.com/owner/repo/issues/123)"}, @@ -274,14 +274,9 @@ var ValidationConfig = map[string]TypeValidationConfig{ // If enabledTypes is provided, returns only configs for the specified types func GetValidationConfigJSON(enabledTypes []string) (string, error) { safeOutputValidationLog.Printf("Getting validation config JSON for %d types", len(enabledTypes)) - var configToMarshal map[string]TypeValidationConfig - if len(enabledTypes) == 0 { - // Return all configs (backwards compatible) - safeOutputValidationLog.Print("Returning all validation configs") - configToMarshal = ValidationConfig - } else { - // Filter to only enabled types + configToMarshal := ValidationConfig + if len(enabledTypes) > 0 { safeOutputValidationLog.Printf("Filtering validation configs to enabled types: %v", enabledTypes) configToMarshal = make(map[string]TypeValidationConfig) for _, typeName := range enabledTypes { @@ -289,9 +284,10 @@ func GetValidationConfigJSON(enabledTypes []string) (string, error) { configToMarshal[typeName] = config } } + } else { + safeOutputValidationLog.Print("Returning all validation configs") } - // Use MarshalIndent for pretty-printed JSON to avoid merge issues data, err := json.MarshalIndent(configToMarshal, "", " ") if err != nil { safeOutputValidationLog.Printf("Failed to marshal validation config: %v", err)