lint: misc

Author: fulleni

lib/src/config/environment_config.dart

@@ -1,6 +1,5 @@
-import 'dart:io';
-import 'package:logging/logging.dart';
 import 'package:dotenv/dotenv.dart';
+import 'package:logging/logging.dart';
 
 /// {@template environment_config}
 /// A utility class for accessing environment variables.

lib/src/services/auth_service.dart

@@ -531,22 +531,20 @@ class AuthService {
       }
 
       // 3. Clear any pending sign-in codes for the user's email (if they had one).
-      if (userToDelete.email != null) {
-        try {
-          await _verificationCodeStorageService.clearSignInCode(
-            userToDelete.email!,
-          );
-          _log.info(
-            'Cleared sign-in code for email ${userToDelete.email}.',
-          );
-        } catch (e) {
-          // Log but don't fail deletion if clearing codes fails
-          _log.warning(
-            'Warning: Failed to clear sign-in code for email ${userToDelete.email}: $e',
-          );
-        }
+      try {
+        await _verificationCodeStorageService.clearSignInCode(
+          userToDelete.email,
+        );
+        _log.info(
+          'Cleared sign-in code for email ${userToDelete.email}.',
+        );
+      } catch (e) {
+        // Log but don't fail deletion if clearing codes fails
+        _log.warning(
+          'Warning: Failed to clear sign-in code for email ${userToDelete.email}: $e',
+        );
       }
-
+    
       // TODO(fulleni): Add logic here to delete or anonymize other
       // user-related data (e.g., settings, content) from other repositories
       // once those features are implemented.

lib/src/services/database_seeding_service.dart

@@ -224,7 +224,7 @@ class DatabaseSeedingService {
           // `headquarters_country_id` column and then remove the original
           // nested object from the parameters to avoid a "superfluous
           // variable" error.
-          params['headquarters_country_id'] = source.headquarters?.id;
+          params['headquarters_country_id'] = source.headquarters.id;
           params.remove('headquarters');
 
           // Ensure optional fields exist for the postgres driver.

lib/src/services/default_user_preference_limit_service.dart

@@ -1,6 +1,7 @@
 import 'package:ht_api/src/services/user_preference_limit_service.dart';
 import 'package:ht_data_repository/ht_data_repository.dart';
 import 'package:ht_shared/ht_shared.dart';
+import 'package:logging/logging.dart';
 
 /// {@template default_user_preference_limit_service}
 /// Default implementation of [UserPreferenceLimitService] that enforces limits
@@ -10,9 +11,12 @@ class DefaultUserPreferenceLimitService implements UserPreferenceLimitService {
   /// {@macro default_user_preference_limit_service}
   const DefaultUserPreferenceLimitService({
     required HtDataRepository<RemoteConfig> remoteConfigRepository,
-  }) : _remoteConfigRepository = remoteConfigRepository;
+    required Logger log,
+  })  : _remoteConfigRepository = remoteConfigRepository,
+        _log = log;
 
   final HtDataRepository<RemoteConfig> _remoteConfigRepository;
+  final Logger _log;
 
   // Assuming a fixed ID for the RemoteConfig document
   static const String _remoteConfigId = 'remote_config';
@@ -67,7 +71,9 @@ class DefaultUserPreferenceLimitService implements UserPreferenceLimitService {
       rethrow;
     } catch (e) {
       // Catch unexpected errors
-      print('Error checking limit for user ${user.id}, itemType $itemType: $e');
+      _log.severe(
+        'Error checking limit for user ${user.id}, itemType $itemType: $e',
+      );
       throw const OperationFailedException(
         'Failed to check user preference limits.',
       );
@@ -139,7 +145,7 @@ class DefaultUserPreferenceLimitService implements UserPreferenceLimitService {
       rethrow;
     } catch (e) {
       // Catch unexpected errors
-      print('Error checking update limits for user ${user.id}: $e');
+      _log.severe('Error checking update limits for user ${user.id}: $e');
       throw const OperationFailedException(
         'Failed to check user preference update limits.',
       );

lib/src/services/token_blacklist_service.dart

@@ -1,8 +1,8 @@
 import 'dart:async';
 
 import 'package:ht_shared/ht_shared.dart';
-import 'package:meta/meta.dart';
 import 'package:logging/logging.dart';
+import 'package:meta/meta.dart';
 
 /// {@template token_blacklist_service}
 /// Defines the interface for a service that manages a blacklist of
@@ -51,8 +51,7 @@ class InMemoryTokenBlacklistService implements TokenBlacklistService {
   /// - [cleanupInterval]: How often the service checks for and removes
   ///   expired token IDs. Defaults to 1 hour.
   InMemoryTokenBlacklistService({
-    Duration cleanupInterval = const Duration(hours: 1),
-    required Logger log,
+    required Logger log, Duration cleanupInterval = const Duration(hours: 1),
   }) : _log = log {
     _cleanupTimer = Timer.periodic(cleanupInterval, (_) async {
       try {

routes/api/v1/data/index.dart

@@ -12,7 +12,7 @@ import '../../../_middleware.dart'; // Assuming RequestId is here
 String _camelToSnake(String input) {
   return input
       .replaceAllMapped(
-        RegExp(r'(?<!^)(?=[A-Z])'),
+        RegExp('(?<!^)(?=[A-Z])'),
         (match) => '_${match.group(0)}',
       )
       .toLowerCase();