Fix linter

Author: cadivus

packages/amplify_lints/lib/amplify_lints.dart

@@ -3,10 +3,21 @@
 
 /// Custom lint rules for Amplify Flutter packages.
 ///
-/// This package provides an analyzer plugin that is automatically discovered
-/// by the Dart Analysis Server when enabled in `analysis_options.yaml`.
+/// This package provides custom lint rules using `custom_lint` that are
+/// automatically discovered by the Dart Analysis Server when `custom_lint`
+/// is enabled in `analysis_options.yaml` and this package is a dependency.
 library amplify_lints;
 
-export 'src/fixes/add_license_header_fix.dart' show AddLicenseHeaderFix;
-export 'src/lints/missing_license_header.dart' show MissingLicenseHeader;
-export 'src/plugin.dart' show AmplifyLintsPlugin;
+import 'package:custom_lint_builder/custom_lint_builder.dart';
+
+import 'src/lints/missing_license_header.dart';
+
+/// Creates the plugin instance for custom_lint to discover.
+PluginBase createPlugin() => _AmplifyLintsPlugin();
+
+class _AmplifyLintsPlugin extends PluginBase {
+  @override
+  List<LintRule> getLintRules(CustomLintConfigs configs) => [
+    MissingLicenseHeader(),
+  ];
+}

packages/amplify_lints/lib/app.yaml

@@ -1,10 +1,8 @@
 include: package:flutter_lints/flutter.yaml
 
-plugins:
-  amplify_lints:
-    path: ../
-
 analyzer:
+  plugins:
+    - custom_lint
   language:
     strict-casts: true
     strict-inference: true

packages/amplify_lints/lib/library.yaml

@@ -1,10 +1,8 @@
 include: package:lints/recommended.yaml
 
-plugins:
-  amplify_lints:
-    path: ../
-
 analyzer:
+  plugins:
+    - custom_lint
   language:
     strict-casts: true
     strict-inference: true

packages/amplify_lints/lib/src/lints/missing_license_header.dart

@@ -1,12 +1,8 @@
 // Copyright Amazon.com, Inc. or its affiliates. All Rights Reserved.
 // SPDX-License-Identifier: Apache-2.0
 
-import 'package:analyzer/analysis_rule/analysis_rule.dart';
-import 'package:analyzer/analysis_rule/rule_context.dart';
-import 'package:analyzer/analysis_rule/rule_visitor_registry.dart';
-import 'package:analyzer/dart/ast/ast.dart';
-import 'package:analyzer/dart/ast/visitor.dart';
-import 'package:analyzer/error/error.dart';
+import 'package:analyzer/error/listener.dart';
+import 'package:custom_lint_builder/custom_lint_builder.dart';
 
 /// The expected license header lines.
 const licenseHeader = [
@@ -29,54 +25,32 @@ const licenseHeader = [
 /// ```dart
 /// library my_library;
 /// ```
-class MissingLicenseHeader extends AnalysisRule {
+class MissingLicenseHeader extends DartLintRule {
   /// Creates a new [MissingLicenseHeader] lint rule.
-  MissingLicenseHeader()
-    : super(
-        name: 'missing_license_header',
-        description:
-            'Dart files must start with the Amazon copyright and '
-            'Apache-2.0 license header.',
-      );
+  MissingLicenseHeader() : super(code: _code);
 
-  /// The lint code reported by this rule.
-  static const LintCode code = LintCode(
-    'missing_license_header',
-    'Dart files must start with the Amazon copyright and Apache-2.0 '
-        'license header.',
+  static const _code = LintCode(
+    name: 'missing_license_header',
+    problemMessage:
+        'Dart files must start with the Amazon copyright and '
+        'Apache-2.0 license header.',
     correctionMessage: 'Add the license header to the top of this file.',
   );
 
   @override
-  LintCode get diagnosticCode => code;
-
-  @override
-  void registerNodeProcessors(
-    RuleVisitorRegistry registry,
-    RuleContext context,
+  void run(
+    CustomLintResolver resolver,
+    ErrorReporter reporter,
+    CustomLintContext context,
   ) {
-    final visitor = _Visitor(this, context);
-    registry.addCompilationUnit(this, visitor);
-  }
-}
-
-class _Visitor extends SimpleAstVisitor<void> {
-  _Visitor(this.rule, this.context);
-
-  final AnalysisRule rule;
-  final RuleContext context;
-
-  @override
-  void visitCompilationUnit(CompilationUnit node) {
-    final unit = context.currentUnit;
-    if (unit == null) return;
+    context.registry.addCompilationUnit((node) {
+      final content = resolver.source.contents.data;
 
-    // Use the content provided by the analysis context (no need to read
-    // the file from disk).
-    if (_hasLicenseHeader(unit.content)) return;
+      if (_hasLicenseHeader(content)) return;
 
-    // Report the lint on the first token in the file.
-    rule.reportAtToken(node.beginToken);
+      // Report the lint on the first token in the file.
+      reporter.atToken(node.beginToken, _code);
+    });
   }
 }
 

packages/amplify_lints/pubspec.yaml

@@ -9,13 +9,12 @@ environment:
   sdk: ^3.9.0
 
 dependencies:
-  analysis_server_plugin: any
   analyzer: any
   analyzer_plugin: any
+  custom_lint: any
+  custom_lint_builder: any
   flutter_lints: ^6.0.0
   lints: ^6.0.0
 
 dev_dependencies:
-  analyzer_testing: ^0.1.0
   test: ^1.24.0
-  test_reflective_loader: ^0.2.0