lint: misc

Author: fulleni

lib/src/config/server.dart

@@ -1,14 +1,14 @@
- import 'dart:io';
+import 'dart:io';
 
 import 'package:dart_frog/dart_frog.dart';
 import 'package:ht_api/src/config/environment_config.dart';
 import 'package:ht_api/src/rbac/permission_service.dart';
 import 'package:ht_api/src/services/auth_service.dart';
 import 'package:ht_api/src/services/auth_token_service.dart';
 import 'package:ht_api/src/services/dashboard_summary_service.dart';
+import 'package:ht_api/src/services/database_seeding_service.dart';
 import 'package:ht_api/src/services/default_user_preference_limit_service.dart';
 import 'package:ht_api/src/services/jwt_auth_token_service.dart';
-import 'package:ht_api/src/services/database_seeding_service.dart';
 import 'package:ht_api/src/services/token_blacklist_service.dart';
 import 'package:ht_api/src/services/user_preference_limit_service.dart';
 import 'package:ht_api/src/services/verification_code_storage_service.dart';
@@ -23,20 +23,20 @@ import 'package:postgres/postgres.dart';
 import 'package:uuid/uuid.dart';
 
 /// Global logger instance.
- final _log = Logger('ht_api');
+final _log = Logger('ht_api');
 
 /// Global PostgreSQL connection instance.
- late final Connection _connection;
+late final Connection _connection;
 
 /// Creates a data repository for a given type [T].
 ///
 /// This helper function centralizes the creation of repositories,
 /// ensuring they all use the same database connection and logger.
- HtDataRepository<T> _createRepository<T>({
+HtDataRepository<T> _createRepository<T>({
   required String tableName,
   required FromJson<T> fromJson,
   required ToJson<T> toJson,
- }) {
+}) {
   return HtDataRepository<T>(
     dataClient: HtDataPostgresClient<T>(
       connection: _connection,
@@ -46,7 +46,7 @@ import 'package:uuid/uuid.dart';
       log: _log,
     ),
   );
- }
+}
 
 /// The main entry point for the server.
 ///
@@ -98,7 +98,10 @@ Future<HttpServer> run(Handler handler, InternetAddress ip, int port) async {
   // `ON CONFLICT DO NOTHING`), so it's safe to run every time. This ensures
   // the database is always in a valid state, especially for first-time setup
   // in any environment.
-  final seedingService = DatabaseSeedingService(connection: _connection, log: _log);
+  final seedingService = DatabaseSeedingService(
+    connection: _connection,
+    log: _log,
+  );
   await seedingService.createTables();
   await seedingService.seedGlobalFixtureData();
   await seedingService.seedInitialAdminAndConfig();
@@ -136,10 +139,10 @@ Future<HttpServer> run(Handler handler, InternetAddress ip, int port) async {
   );
   final userContentPreferencesRepository =
       _createRepository<UserContentPreferences>(
-    tableName: 'user_content_preferences',
-    fromJson: UserContentPreferences.fromJson,
-    toJson: (p) => p.toJson(),
-  );
+        tableName: 'user_content_preferences',
+        fromJson: UserContentPreferences.fromJson,
+        toJson: (p) => p.toJson(),
+      );
   final appConfigRepository = _createRepository<AppConfig>(
     tableName: 'app_config',
     fromJson: AppConfig.fromJson,
@@ -212,17 +215,11 @@ Future<HttpServer> run(Handler handler, InternetAddress ip, int port) async {
       .use(provider<DashboardSummaryService>((_) => dashboardSummaryService))
       .use(provider<PermissionService>((_) => permissionService))
       .use(
-        provider<UserPreferenceLimitService>(
-          (_) => userPreferenceLimitService,
-        ),
+        provider<UserPreferenceLimitService>((_) => userPreferenceLimitService),
       );
 
   // 7. Start the server
-  final server = await serve(
-    finalHandler,
-    ip,
-    port,
-  );
+  final server = await serve(finalHandler, ip, port);
   _log.info('Server listening on port ${server.port}');
 
   // 8. Handle graceful shutdown
@@ -236,4 +233,4 @@ Future<HttpServer> run(Handler handler, InternetAddress ip, int port) async {
   });
 
   return server;
- }
+}

lib/src/registry/model_registry.dart

@@ -1,3 +1,5 @@
+// ignore_for_file: comment_references
+
 import 'package:dart_frog/dart_frog.dart';
 import 'package:ht_api/src/rbac/permissions.dart';
 import 'package:ht_data_client/ht_data_client.dart';

lib/src/services/auth_service.dart

@@ -171,11 +171,7 @@ class AuthService {
         // Admin users must be provisioned out-of-band (e.g., via fixtures).
         final roles = [UserRoles.standardUser];
 
-        user = User(
-          id: _uuid.v4(),
-          email: email,
-          roles: roles,
-        );
+        user = User(id: _uuid.v4(), email: email, roles: roles);
         user = await _userRepository.create(item: user);
         print('Created new user: ${user.id} with roles: ${user.roles}');
 
@@ -197,7 +193,9 @@ class AuthService {
       }
     } on HtHttpException catch (e) {
       print('Error finding/creating user for $email: $e');
-      throw const OperationFailedException('Failed to find or create user account.');
+      throw const OperationFailedException(
+        'Failed to find or create user account.',
+      );
     } catch (e) {
       print('Unexpected error during user lookup/creation for $email: $e');
       throw const OperationFailedException('Failed to process user account.');
@@ -213,7 +211,7 @@ class AuthService {
       throw const OperationFailedException(
         'Failed to generate authentication token.',
       );
-    } 
+    }
   }
 
   /// Performs anonymous sign-in.
@@ -227,7 +225,7 @@ class AuthService {
     try {
       user = User(
         id: _uuid.v4(), // Generate new ID
-        roles: [UserRoles.guestUser], // Anonymous users are guest users
+        roles: const [UserRoles.guestUser], // Anonymous users are guest users
         email: null, // Anonymous users don't have an email initially
       );
       user = await _userRepository.create(item: user);
@@ -426,7 +424,7 @@ class AuthService {
       final updatedUser = User(
         id: anonymousUser.id, // Preserve original ID
         email: linkedEmail,
-        roles: [UserRoles.standardUser], // Now a permanent standard user
+        roles: const [UserRoles.standardUser], // Now a permanent standard user
       );
       final permanentUser = await _userRepository.update(
         id: updatedUser.id,

lib/src/services/database_seeding_service.dart

@@ -1,6 +1,6 @@
+import 'package:ht_shared/ht_shared.dart';
 import 'package:logging/logging.dart';
 import 'package:postgres/postgres.dart';
-import 'package:ht_shared/ht_shared.dart';
 
 /// {@template database_seeding_service}
 /// A service responsible for initializing the database schema and seeding it
@@ -137,11 +137,7 @@ class DatabaseSeedingService {
       }
       _log.info('Database schema creation completed successfully.');
     } on Object catch (e, st) {
-      _log.severe(
-        'An error occurred during database schema creation.',
-        e,
-        st,
-      );
+      _log.severe('An error occurred during database schema creation.', e, st);
       // Propagate as a standard exception for the server to handle.
       throw OperationFailedException(
         'Failed to initialize database schema: $e',
@@ -225,9 +221,7 @@ class DatabaseSeedingService {
         e,
         st,
       );
-      throw OperationFailedException(
-        'Failed to seed global fixture data: $e',
-      );
+      throw OperationFailedException('Failed to seed global fixture data: $e');
     }
   }
 
@@ -279,10 +273,7 @@ class DatabaseSeedingService {
             'VALUES (@id, @user_id, @display_settings, @language) '
             'ON CONFLICT (id) DO NOTHING',
           ),
-          parameters: {
-            ...adminSettings.toJson(),
-            'user_id': adminUser.id,
-          },
+          parameters: {...adminSettings.toJson(), 'user_id': adminUser.id},
         );
 
         await _connection.execute(
@@ -293,10 +284,7 @@ class DatabaseSeedingService {
             '@followed_sources, @followed_countries, @saved_headlines) '
             'ON CONFLICT (id) DO NOTHING',
           ),
-          parameters: {
-            ...adminPreferences.toJson(),
-            'user_id': adminUser.id,
-          },
+          parameters: {...adminPreferences.toJson(), 'user_id': adminUser.id},
         );
 
         await _connection.execute('COMMIT');

routes/_middleware.dart

@@ -1,6 +1,5 @@
 import 'package:dart_frog/dart_frog.dart';
 import 'package:ht_api/src/middlewares/error_handler.dart';
-import 'package:ht_api/src/middlewares/request_logger.dart';
 import 'package:uuid/uuid.dart';
 
 // --- Request ID Wrapper ---
@@ -54,16 +53,14 @@ Handler middleware(Handler handler) {
   // 3. Error Handler: Catches all errors and formats them into a standard
   //    JSON response.
   return handler
-      .use(
-        (innerHandler) {
-          return (context) {
-            // Read the singleton Uuid instance provided from server.dart.
-            final uuid = context.read<Uuid>();
-            final requestId = RequestId(uuid.v4());
-            return innerHandler(context.provide<RequestId>(() => requestId));
-          };
-        },
-      )
+      .use((innerHandler) {
+        return (context) {
+          // Read the singleton Uuid instance provided from server.dart.
+          final uuid = context.read<Uuid>();
+          final requestId = RequestId(uuid.v4());
+          return innerHandler(context.provide<RequestId>(() => requestId));
+        };
+      })
       .use(requestLogger())
       .use(errorHandler());
 }