chore: upgrade to analyzer 8

chore: replace usage of deprecated members

Author: andrew-bekhiet-solid

lib/src/common/parameters/excluded_identifiers_list_parameter.dart

@@ -55,7 +55,7 @@ class ExcludedIdentifiersListParameter {
 
   /// Returns whether the target node should be ignored during analysis.
   bool shouldIgnore(Declaration node) {
-    final declarationName = node.declaredFragment?.name2;
+    final declarationName = node.declaredFragment?.name;
 
     final excludedItem = exclude.firstWhereOrNull(
       (e) {

lib/src/lints/avoid_debug_print_in_release/avoid_debug_print_in_release_rule.dart

@@ -61,7 +61,7 @@ your `debugPrint` call in a `!kReleaseMode` check.""",
   @override
   void run(
     CustomLintResolver resolver,
-    ErrorReporter reporter,
+    DiagnosticReporter reporter,
     CustomLintContext context,
   ) {
     context.registry.addFunctionExpressionInvocation(
@@ -102,7 +102,7 @@ your `debugPrint` call in a `!kReleaseMode` check.""",
   void _checkIdentifier({
     required Identifier identifier,
     required AstNode node,
-    required ErrorReporter reporter,
+    required DiagnosticReporter reporter,
   }) {
     if (!_isDebugPrintNode(identifier)) {
       return;
@@ -137,7 +137,7 @@ your `debugPrint` call in a `!kReleaseMode` check.""",
   /// Handles variable assignment and declaration
   void _handleVariableAssignmentDeclaration({
     required AstNode node,
-    required ErrorReporter reporter,
+    required DiagnosticReporter reporter,
   }) {
     final rightOperand = _getRightOperand(node.childEntities.toList());
 
@@ -159,10 +159,10 @@ your `debugPrint` call in a `!kReleaseMode` check.""",
       case PrefixedIdentifier():
         final prefix = node.prefix.name;
         name = node.name.replaceAll('$prefix.', '');
-        sourcePath = node.element?.library2?.uri.toString() ?? '';
+        sourcePath = node.element?.library?.uri.toString() ?? '';
       case SimpleIdentifier():
         name = node.name;
-        sourcePath = node.element?.library2?.uri.toString() ?? '';
+        sourcePath = node.element?.library?.uri.toString() ?? '';
 
       default:
         return false;
@@ -193,10 +193,10 @@ your `debugPrint` call in a `!kReleaseMode` check.""",
         final prefix = node.prefix.name;
 
         name = node.name.replaceAll('$prefix.', '');
-        sourcePath = node.element?.library2?.uri.toString() ?? '';
+        sourcePath = node.element?.library?.uri.toString() ?? '';
       case SimpleIdentifier():
         name = node.name;
-        sourcePath = node.element?.library2?.uri.toString() ?? '';
+        sourcePath = node.element?.library?.uri.toString() ?? '';
       default:
         return false;
     }

lib/src/lints/avoid_final_with_getter/avoid_final_with_getter_rule.dart

@@ -1,4 +1,4 @@
-import 'package:analyzer/error/error.dart' as error;
+import 'package:analyzer/diagnostic/diagnostic.dart';
 import 'package:analyzer/error/listener.dart';
 import 'package:analyzer/source/source_range.dart';
 import 'package:custom_lint_builder/custom_lint_builder.dart';
@@ -56,7 +56,7 @@ class AvoidFinalWithGetterRule extends SolidLintRule {
   @override
   void run(
     CustomLintResolver resolver,
-    ErrorReporter reporter,
+    DiagnosticReporter reporter,
     CustomLintContext context,
   ) {
     context.registry.addCompilationUnit((node) {

lib/src/lints/avoid_final_with_getter/fixes/avoid_final_with_getter_fix.dart

@@ -6,8 +6,8 @@ class _FinalWithGetterFix extends DartFix {
     CustomLintResolver resolver,
     ChangeReporter reporter,
     CustomLintContext context,
-    error.AnalysisError analysisError,
-    List<error.AnalysisError> others,
+    Diagnostic analysisError,
+    List<Diagnostic> others,
   ) {
     context.registry.addMethodDeclaration((node) {
       if (analysisError.sourceRange.intersects(node.sourceRange)) {

lib/src/lints/avoid_final_with_getter/visitors/avoid_final_with_getter_visitor.dart

@@ -1,6 +1,6 @@
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/visitor.dart';
-import 'package:analyzer/dart/element/element2.dart';
+import 'package:analyzer/dart/element/element.dart';
 import 'package:solid_lints/src/lints/avoid_final_with_getter/visitors/getter_variable_visitor.dart';
 
 /// A visitor that checks for final private fields with getters.
@@ -17,7 +17,7 @@ class AvoidFinalWithGetterVisitor extends RecursiveAstVisitor<void> {
         case MethodDeclaration(
           isGetter: true,
           declaredFragment: ExecutableFragment(
-            element: ExecutableElement2(
+            element: ExecutableElement(
               isAbstract: false,
               isPublic: true,
             )

lib/src/lints/avoid_final_with_getter/visitors/getter_variable_visitor.dart

@@ -1,6 +1,6 @@
 import 'package:analyzer/dart/ast/ast.dart';
 import 'package:analyzer/dart/ast/visitor.dart';
-import 'package:analyzer/dart/element/element2.dart';
+import 'package:analyzer/dart/element/element.dart';
 
 /// A visitor that checks the association of the getter with
 /// the final private variable
@@ -31,14 +31,13 @@ class GetterVariableVisitor extends RecursiveAstVisitor<void> {
 
 extension on MethodDeclaration {
   int? get getterReferenceId {
-    if (body is! ExpressionFunctionBody) return null;
-
-    final expression = (body as ExpressionFunctionBody).expression;
-    if (expression is SimpleIdentifier) {
-      final element = expression.element;
-      if (element is PropertyAccessorElement2) {
-        return element.variable3?.id;
-      }
+    if (body
+        case ExpressionFunctionBody(
+          expression: SimpleIdentifier(
+            element: PropertyAccessorElement(:final variable)
+          )
+        )) {
+      return variable.id;
     }
 
     return null;

lib/src/lints/avoid_global_state/avoid_global_state_rule.dart

@@ -57,7 +57,7 @@ class AvoidGlobalStateRule extends SolidLintRule {
   @override
   void run(
     CustomLintResolver resolver,
-    ErrorReporter reporter,
+    DiagnosticReporter reporter,
     CustomLintContext context,
   ) {
     context.registry.addTopLevelVariableDeclaration(
@@ -77,7 +77,7 @@ class AvoidGlobalStateRule extends SolidLintRule {
 extension on VariableDeclaration {
   bool get isMutable => !isFinal && !isConst;
 
-  bool get isPrivate => declaredElement2?.isPrivate ?? false;
+  bool get isPrivate => declaredFragment?.element.isPrivate ?? false;
 
   bool get isPublicMutable => isMutable && !isPrivate;
 }

lib/src/lints/avoid_late_keyword/avoid_late_keyword_rule.dart

@@ -71,7 +71,7 @@ class AvoidLateKeywordRule extends SolidLintRule<AvoidLateKeywordParameters> {
   @override
   void run(
     CustomLintResolver resolver,
-    ErrorReporter reporter,
+    DiagnosticReporter reporter,
     CustomLintContext context,
   ) {
     context.registry.addVariableDeclaration((node) {

lib/src/lints/avoid_non_null_assertion/avoid_non_null_assertion_rule.dart

@@ -60,7 +60,7 @@ class AvoidNonNullAssertionRule extends SolidLintRule {
   @override
   void run(
     CustomLintResolver resolver,
-    ErrorReporter reporter,
+    DiagnosticReporter reporter,
     CustomLintContext context,
   ) {
     context.registry.addPostfixExpression((node) {

lib/src/lints/avoid_returning_widgets/avoid_returning_widgets_rule.dart

@@ -77,7 +77,7 @@ class AvoidReturningWidgetsRule
   @override
   void run(
     CustomLintResolver resolver,
-    ErrorReporter reporter,
+    DiagnosticReporter reporter,
     CustomLintContext context,
   ) {
     context.registry.addDeclaration((node) {