tighten our static analysis

Author: Leptopoda

analysis_options.yaml

@@ -5,3 +5,24 @@ include: package:lint/strict.yaml
 linter:
   rules:
     sort_pub_dependencies: false
+    prefer_final_parameters: false
+    discarded_futures: false
+    prefer_asserts_with_message: false
+    unawaited_futures: false
+    prefer_mixin: false
+    prefer_expression_function_bodies: false
+    only_throw_errors: false
+
+    always_put_control_body_on_new_line: true
+    use_key_in_widget_constructors: true
+    always_put_required_named_parameters_first: true
+    prefer_single_quotes: true
+    sort_constructors_first: true
+    omit_local_variable_types: true
+    prefer_int_literals: true
+    cascade_invocations: true
+    avoid_equals_and_hash_code_on_mutable_classes: true
+    avoid_types_on_closure_parameters: true
+    use_decorated_box: true
+    unnecessary_lambdas: true
+    prefer_foreach: true

lib/main.dart

@@ -94,7 +94,7 @@ class _AppState extends State<App> {
   Widget build(BuildContext context) {
     return ThemeModeHandler(
       manager: ThemeModeManager(),
-      builder: (ThemeMode themeMode) => MaterialApp(
+      builder: (themeMode) => MaterialApp(
         navigatorKey: IntentRepository().getNavigationKey(),
         themeMode: themeMode,
         theme: AppTheme.lightThemeData,
@@ -121,8 +121,8 @@ class _AppState extends State<App> {
                         SnackBar(
                           content: Text(
                             translate(
-                              "categories.errors.api_version_check_failed",
-                              args: {"error_msg": snapshot.error},
+                              'categories.errors.api_version_check_failed',
+                              args: {'error_msg': snapshot.error},
                             ),
                             style: TextStyle(
                               color: Theme.of(context).colorScheme.onError,
@@ -138,10 +138,10 @@ class _AppState extends State<App> {
                         SnackBar(
                           content: Text(
                             translate(
-                              "categories.errors.api_version_above_confirmed",
+                              'categories.errors.api_version_above_confirmed',
                               args: {
-                                "version":
-                                    "${snapshot.data!.major}.${snapshot.data!.minor}"
+                                'version':
+                                    '${snapshot.data!.major}.${snapshot.data!.minor}'
                               },
                             ),
                           ),

lib/src/blocs/authentication/authentication_bloc.dart

@@ -8,19 +8,18 @@ part 'authentication_state.dart';
 
 class AuthenticationBloc
     extends Bloc<AuthenticationEvent, AuthenticationState> {
-  final UserRepository userRepository = UserRepository();
-
   AuthenticationBloc() : super(AuthenticationState()) {
     on<AppStarted>(_mapAppStartedEventToState);
     on<LoggedIn>(_mapLoggedInEventToState);
     on<LoggedOut>(_mapLoggedOutEventToState);
   }
+  final UserRepository userRepository = UserRepository();
 
   Future<void> _mapAppStartedEventToState(
     AppStarted event,
     Emitter<AuthenticationState> emit,
   ) async {
-    final bool hasToken = await userRepository.hasAppAuthentication();
+    final hasToken = await userRepository.hasAppAuthentication();
 
     if (hasToken) {
       await userRepository.loadAppAuthentication();

lib/src/blocs/authentication/authentication_event.dart

@@ -12,9 +12,8 @@ class AppStarted extends AuthenticationEvent {
 }
 
 class LoggedIn extends AuthenticationEvent {
-  final AppAuthentication appAuthentication;
-
   const LoggedIn({required this.appAuthentication});
+  final AppAuthentication appAuthentication;
 
   @override
   List<Object> get props => [appAuthentication];

lib/src/blocs/authentication/authentication_state.dart

@@ -22,16 +22,15 @@ enum AuthenticationStatus {
 }
 
 class AuthenticationState extends Equatable {
-  final AuthenticationStatus status;
-  final String? error;
-
   const AuthenticationState({
     this.status = AuthenticationStatus.loading,
     this.error,
   }) : assert(
           (status != AuthenticationStatus.error && error == null) ||
               (status == AuthenticationStatus.error && error != null),
         );
+  final AuthenticationStatus status;
+  final String? error;
 
   @override
   List<Object?> get props => [status, error];

lib/src/blocs/categories/categories_bloc.dart

@@ -7,11 +7,10 @@ part 'categories_event.dart';
 part 'categories_state.dart';
 
 class CategoriesBloc extends Bloc<CategoriesEvent, CategoriesState> {
-  final DataRepository dataRepository = DataRepository();
-
   CategoriesBloc() : super(CategoriesState()) {
     on<CategoriesLoaded>(_mapCategoriesLoadedEventToState);
   }
+  final DataRepository dataRepository = DataRepository();
 
   Future<void> _mapCategoriesLoadedEventToState(
     CategoriesLoaded event,

lib/src/blocs/categories/categories_state.dart

@@ -8,11 +8,6 @@ enum CategoriesStatus {
 }
 
 class CategoriesState extends Equatable {
-  final CategoriesStatus status;
-  final String? error;
-  final Iterable<Category>? categories;
-  final Iterable<RecipeStub?>? recipes;
-
   CategoriesState({
     this.status = CategoriesStatus.loadInProgress,
     this.error,
@@ -33,6 +28,10 @@ class CategoriesState extends Equatable {
         assert(error != null && categories == null && recipes == null);
     }
   }
+  final CategoriesStatus status;
+  final String? error;
+  final Iterable<Category>? categories;
+  final Iterable<RecipeStub?>? recipes;
 
   @override
   List<Object?> get props => [status, error, categories];

lib/src/blocs/login/login_bloc.dart

@@ -9,14 +9,13 @@ part 'login_event.dart';
 part 'login_state.dart';
 
 class LoginBloc extends Bloc<LoginEvent, LoginState> {
-  final UserRepository userRepository = UserRepository();
-  final AuthenticationBloc authenticationBloc;
-
   LoginBloc({
     required this.authenticationBloc,
   }) : super(LoginState()) {
     on<LoginButtonPressed>(_mapLoginButtonPressedEventToState);
   }
+  final UserRepository userRepository = UserRepository();
+  final AuthenticationBloc authenticationBloc;
 
   Future<void> _mapLoginButtonPressedEventToState(
     LoginButtonPressed event,

lib/src/blocs/login/login_event.dart

@@ -8,19 +8,18 @@ abstract class LoginEvent extends Equatable {
 }
 
 class LoginButtonPressed extends LoginEvent {
-  final String serverURL;
-  final String username;
-  final String originalBasicAuth;
-  final bool isAppPassword;
-  final bool isSelfSignedCertificate;
-
   const LoginButtonPressed({
     required this.serverURL,
     required this.username,
     required this.originalBasicAuth,
     required this.isAppPassword,
     required this.isSelfSignedCertificate,
   });
+  final String serverURL;
+  final String username;
+  final String originalBasicAuth;
+  final bool isAppPassword;
+  final bool isSelfSignedCertificate;
 
   @override
   List<Object> get props =>

lib/src/blocs/login/login_state.dart

@@ -7,16 +7,15 @@ enum LoginStatus {
 }
 
 class LoginState extends Equatable {
-  final LoginStatus status;
-  final String? error;
-
   const LoginState({
     this.status = LoginStatus.initial,
     this.error,
   }) : assert(
           (status != LoginStatus.failure && error == null) ||
               (status == LoginStatus.failure && error != null),
         );
+  final LoginStatus status;
+  final String? error;
 
   @override
   List<Object?> get props => [status, error];