suggested changes done

Author: iimironchuk

lib/analysis_options.yaml

@@ -94,7 +94,6 @@ custom_lint:
     - prefer_first
     - prefer_last
     - prefer_match_file_name:
-      ignore_extensions: true
     - proper_super_calls
 
 linter:

lib/src/common/parameters/excluded_entities_list_parameter.dart

@@ -3,27 +3,32 @@ import 'package:analyzer/dart/ast/ast.dart';
 /// A model representing "exclude_entity" parameters for linting, defining
 /// identifiers (classes, mixins, enums, extensions) to be ignored during
 /// analysis.
-class ExcludedEntitiesListParameters {
+/// Supported entities:
+///   - class
+///   - mixin
+///   - extension
+///   - enum
+class ExcludedEntitiesListParameter {
   /// The parameter model
   final List<String> excludedEntityNames;
 
   /// A common parameter key for analysis_options.yaml
   static const String excludeEntityKey = 'exclude_entity';
 
-  /// Constructor for [ExcludedEntitiesListParameters] class
-  ExcludedEntitiesListParameters({
+  /// Constructor for [ExcludedEntitiesListParameter] class
+  ExcludedEntitiesListParameter({
     required this.excludedEntityNames,
   });
 
   /// Method for creating from json data
-  factory ExcludedEntitiesListParameters.fromJson(Map<String, dynamic> json) {
+  factory ExcludedEntitiesListParameter.fromJson(Map<String, dynamic> json) {
     final raw = json['exclude_entity'];
     if (raw is List) {
-      return ExcludedEntitiesListParameters(
+      return ExcludedEntitiesListParameter(
         excludedEntityNames: List<String>.from(raw),
       );
     }
-    return ExcludedEntitiesListParameters(
+    return ExcludedEntitiesListParameter(
       excludedEntityNames: [],
     );
   }

lib/src/lints/prefer_match_file_name/models/prefer_match_file_name_parameters.dart

@@ -4,7 +4,7 @@ import 'package:solid_lints/src/common/parameters/excluded_entities_list_paramet
 /// parameters
 class PreferMatchFileNameParameters {
   /// A list of entities that should be excluded from the lint.
-  final ExcludedEntitiesListParameters excludeEntity;
+  final ExcludedEntitiesListParameter excludeEntity;
 
   /// Constructor for [PreferMatchFileNameParameters] model
   const PreferMatchFileNameParameters({
@@ -14,6 +14,6 @@ class PreferMatchFileNameParameters {
   /// Method for creating from json data
   factory PreferMatchFileNameParameters.fromJson(Map<String, Object?> json) =>
       PreferMatchFileNameParameters(
-        excludeEntity: ExcludedEntitiesListParameters.fromJson(json),
+        excludeEntity: ExcludedEntitiesListParameter.fromJson(json),
       );
 }

lib/src/lints/prefer_match_file_name/visitors/prefer_match_file_name_visitor.dart

@@ -11,54 +11,38 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
 
   /// Iterable that contains the name of entity (or entities) that should
   /// be ignored
-  final ExcludedEntitiesListParameters excludedEntities;
+  final ExcludedEntitiesListParameter excludedEntities;
 
   /// Constructor of [PreferMatchFileNameVisitor] class
   PreferMatchFileNameVisitor({
     required this.excludedEntities,
   });
 
   /// List of all declarations
-  Iterable<DeclarationTokenInfo> get declarations => _declarations
-    ..sort(
-      // partition into public and private
-      // put public ones first
-      // each partition sorted by declaration order
-      (a, b) => _publicDeclarationsFirst(a, b) ?? _byDeclarationOrder(a, b),
-    );
-
-  // bool _shouldIgnore(Declaration node) {
-  //   if (excludedEntities.isEmpty) return false;
-  //
-  //   if (node is ClassDeclaration && excludedEntities.contains('class')) {
-  //     return true;
-  //   } else if (node is MixinDeclaration && excludedEntities.contains('mixin')) {
-  //     return true;
-  //   } else if (node is EnumDeclaration && excludedEntities.contains('enum')) {
-  //     return true;
-  //   } else if (node is ExtensionDeclaration &&
-  //       excludedEntities.contains('extension')) {
-  //     return true;
-  //   }
-  //
-  //   return false;
-  // }
+  Iterable<DeclarationTokenInfo> get declarations => _declarations.where(
+        (declaration) {
+          if (declaration.parent is Declaration) {
+            return !excludedEntities
+                .shouldIgnoreEntity(declaration.parent as Declaration);
+          }
+          return true;
+        },
+      ).toList()
+        ..sort(
+          (a, b) => _publicDeclarationsFirst(a, b) ?? _byDeclarationOrder(a, b),
+        );
 
   @override
   void visitClassDeclaration(ClassDeclaration node) {
     super.visitClassDeclaration(node);
 
-    if (excludedEntities.shouldIgnoreEntity(node)) return;
-
     _declarations.add((token: node.name, parent: node));
   }
 
   @override
   void visitExtensionDeclaration(ExtensionDeclaration node) {
     super.visitExtensionDeclaration(node);
 
-    if (excludedEntities.shouldIgnoreEntity(node)) return;
-
     final name = node.name;
     if (name != null) {
       _declarations.add((token: name, parent: node));
@@ -69,17 +53,13 @@ class PreferMatchFileNameVisitor extends RecursiveAstVisitor<void> {
   void visitMixinDeclaration(MixinDeclaration node) {
     super.visitMixinDeclaration(node);
 
-    if (excludedEntities.shouldIgnoreEntity(node)) return;
-
     _declarations.add((token: node.name, parent: node));
   }
 
   @override
   void visitEnumDeclaration(EnumDeclaration node) {
     super.visitEnumDeclaration(node);
 
-    if (excludedEntities.shouldIgnoreEntity(node)) return;
-
     _declarations.add((token: node.name, parent: node));
   }