fix

Author: TristanSpeakEasy

marshaller/unmarshaller.go

@@ -367,15 +367,16 @@ func unmarshalModel(ctx context.Context, parentName string, node *yaml.Node, str
 			// Direct field index lookup (eliminates map[string]Field allocation)
 			fieldIndex, ok := fieldMap.FieldIndexes[key]
 			if !ok {
-				if strings.HasPrefix(key, "x-") && extensionsField != nil {
+				switch {
+				case strings.HasPrefix(key, "x-") && extensionsField != nil:
 					// Lock access to extensionsField to prevent concurrent modification
 					extensionsMutex.Lock()
 					defer extensionsMutex.Unlock()
 					err := UnmarshalExtension(keyNode, valueNode, *extensionsField)
 					if err != nil {
 						return err
 					}
-				} else if embeddedMap != nil {
+				case embeddedMap != nil:
 					// Skip alias definitions - these are nodes where:
 					// 1. The value node has an anchor (e.g., &keyAlias)
 					// 2. The key is not an alias reference (doesn't start with *)
@@ -385,7 +386,7 @@ func unmarshalModel(ctx context.Context, parentName string, node *yaml.Node, str
 						return nil
 					}
 					jobMapContent[i/2] = append(jobMapContent[i/2], keyNode, valueNode)
-				} else {
+				default:
 					// This is an unknown property (not a recognized field, not an extension, not in embedded map)
 					unknownPropertiesMutex.Lock()
 					unknownProperties = append(unknownProperties, key)

openapi/cmd/sanitize.go

@@ -139,7 +139,7 @@ func sanitizeOpenAPI(ctx context.Context, processor *OpenAPIProcessor) error {
 		if err != nil {
 			return fmt.Errorf("failed to load config file: %w", err)
 		}
-		processor.PrintInfo(fmt.Sprintf("Using configuration from %s", sanitizeConfigFile))
+		processor.PrintInfo("Using configuration from " + sanitizeConfigFile)
 	}
 
 	// Perform the sanitization

openapi/sanitize.go

@@ -142,7 +142,7 @@ func LoadSanitizeConfig(r io.Reader) (*SanitizeOptions, error) {
 
 // LoadSanitizeConfigFromFile loads sanitize configuration from a YAML file.
 func LoadSanitizeConfigFromFile(path string) (*SanitizeOptions, error) {
-	f, err := os.Open(path)
+	f, err := os.Open(path) //nolint:gosec
 	if err != nil {
 		return nil, fmt.Errorf("failed to open config file: %w", err)
 	}
@@ -183,8 +183,7 @@ func removeExtensions(ctx context.Context, doc *OpenAPI, opts *SanitizeOptions)
 				// Collect keys to remove
 				keysToRemove := []string{}
 				for key := range ext.All() {
-					shouldRemove := false
-
+					var shouldRemove bool
 					if removeAll {
 						// Remove all extensions
 						shouldRemove = true
@@ -222,12 +221,8 @@ func removeUnknownProperties(ctx context.Context, doc *OpenAPI) error {
 	// We need specific matchers for wrapped types (Referenced*, JSONSchema)
 	for item := range Walk(ctx, doc) {
 		err := item.Match(Matcher{
-			Any: func(model any) error {
-				return cleanUnknownPropertiesFromModel(model)
-			},
-			Schema: func(schema *oas3.JSONSchema[oas3.Referenceable]) error {
-				return cleanUnknownPropertiesFromJSONSchema(schema)
-			},
+			Any:    cleanUnknownPropertiesFromModel,
+			Schema: cleanUnknownPropertiesFromJSONSchema,
 			// Handle all Referenced types by extracting their Object
 			ReferencedResponse: func(ref *ReferencedResponse) error {
 				if ref != nil && !ref.IsReference() && ref.Object != nil {