docs: cleaning up unnecessary memo-style comments from the Dart files within the feature directories in lib/.

Author: fulleni

lib/account/view/account_page.dart

@@ -1,14 +1,12 @@
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:go_router/go_router.dart';
-// Import the User model and AuthenticationStatus enum
 import 'package:ht_authentication_client/ht_authentication_client.dart';
 import 'package:ht_authentication_repository/ht_authentication_repository.dart';
 import 'package:ht_main/account/bloc/account_bloc.dart';
-import 'package:ht_main/app/bloc/app_bloc.dart'; // Needed for AppBloc and AppState access
-// Remove direct import of app_state.dart when using part of
-import 'package:ht_main/l10n/l10n.dart'; // Added import
-import 'package:ht_main/router/routes.dart'; // Needed for route names
+import 'package:ht_main/app/bloc/app_bloc.dart';
+import 'package:ht_main/l10n/l10n.dart';
+import 'package:ht_main/router/routes.dart';
 
 /// {@template account_page}
 /// Page widget for the Account feature.
@@ -115,7 +113,7 @@ class _AccountView extends StatelessWidget {
           _authenticationStatusToString(
             context,
             user.authenticationStatus,
-          ), // Pass context
+          ),
           style: textTheme.bodyMedium?.copyWith(
             color: theme.colorScheme.secondary,
           ),
@@ -161,8 +159,8 @@ class _AccountView extends StatelessWidget {
     final l10n = context.l10n;
     return ListTile(
       leading: const Icon(Icons.link), // Icon suggesting connection/linking
-      title: Text(l10n.accountConnectPrompt), // New l10n key needed
-      subtitle: Text(l10n.accountConnectBenefit), // New l10n key needed
+      title: Text(l10n.accountConnectPrompt),
+      subtitle: Text(l10n.accountConnectBenefit),
       isThreeLine: true, // Allow more space for subtitle
       trailing: const Icon(Icons.chevron_right),
       onTap: () {

lib/app/view/app.dart

@@ -3,11 +3,9 @@
 
 import 'dart:async';
 import 'package:firebase_dynamic_links/firebase_dynamic_links.dart'; // For dynamic links
-// Removed: import 'package:flex_color_scheme/flex_color_scheme.dart';
 import 'package:flutter/material.dart';
 import 'package:flutter_bloc/flutter_bloc.dart';
 import 'package:go_router/go_router.dart';
-// Removed: import 'package:google_fonts/google_fonts.dart';
 import 'package:ht_authentication_repository/ht_authentication_repository.dart';
 import 'package:ht_headlines_repository/ht_headlines_repository.dart';
 import 'package:ht_kv_storage_service/ht_kv_storage_service.dart'; // Import storage service
@@ -71,21 +69,15 @@ class _AppView extends StatefulWidget {
 }
 
 class _AppViewState extends State<_AppView> {
-  // Store the router and the status notifier
   late final GoRouter _router;
   late final ValueNotifier<AppStatus> _statusNotifier;
   StreamSubscription<PendingDynamicLinkData>? _linkSubscription;
 
   @override
   void initState() {
     super.initState();
-    // Get the AppBloc instance from the BlocProvider above
     final appBloc = context.read<AppBloc>();
-
-    // Create the ValueNotifier, initialized with the current status
     _statusNotifier = ValueNotifier<AppStatus>(appBloc.state.status);
-
-    // Create the router instance, passing the ValueNotifier as the listenable
     _router = createRouter(authStatusNotifier: _statusNotifier);
 
     // --- Initialize Deep Link Handling ---
@@ -105,11 +97,9 @@ class _AppViewState extends State<_AppView> {
     // Handle links received while the app is running
     _linkSubscription = FirebaseDynamicLinks.instance.onLink.listen(
       (pendingDynamicLinkData) {
-        // Handle link data
         _handleDynamicLink(pendingDynamicLinkData.link);
       },
       onError: (Object error) {
-        // Handle errors (e.g., log them)
         debugPrint('Dynamic Link Listener Error: $error');
       },
     );
@@ -122,7 +112,6 @@ class _AppViewState extends State<_AppView> {
       }
     } catch (e) {
       debugPrint('Error getting initial dynamic link: $e');
-      // Handle potential errors during initial link retrieval
     }
   }
 
@@ -136,14 +125,14 @@ class _AppViewState extends State<_AppView> {
     final authBloc = context.read<AuthenticationBloc>();
     final linkString = deepLink.toString();
 
-    debugPrint('Handling dynamic link: $linkString'); // Log received link
+    debugPrint('Handling dynamic link: $linkString');
 
     try {
       // The async gap happens here
       final isSignInLink = await authRepo.isSignInWithEmailLink(
         emailLink: linkString,
       );
-      debugPrint('Is sign-in link: $isSignInLink'); // Log validation result
+      debugPrint('Is sign-in link: $isSignInLink');
 
       // Check mounted again *after* the await before using BLoC/state
       if (!mounted) return;
@@ -153,9 +142,7 @@ class _AppViewState extends State<_AppView> {
         authBloc.add(
           AuthenticationSignInWithLinkAttempted(emailLink: linkString),
         );
-        debugPrint(
-          'Dispatched AuthenticationSignInWithLinkAttempted',
-        ); // Log dispatch
+        debugPrint('Dispatched AuthenticationSignInWithLinkAttempted');
       } else {
         // Handle other types of deep links if necessary
         debugPrint(

lib/authentication/bloc/authentication_bloc.dart

@@ -19,7 +19,6 @@ class AuthenticationBloc
   /// {@macro authentication_bloc}
   AuthenticationBloc({
     required HtAuthenticationRepository authenticationRepository,
-    // Remove kvStorageService from constructor
   }) : _authenticationRepository = authenticationRepository,
        super(AuthenticationInitial()) {
     on<AuthenticationSendSignInLinkRequested>(
@@ -70,7 +69,7 @@ class AuthenticationBloc
   /// Handles [AuthenticationSignInWithLinkAttempted] events.
   /// This assumes the event is dispatched after the app receives the deep link.
   Future<void> _onAuthenticationSignInWithLinkAttempted(
-    AuthenticationSignInWithLinkAttempted event, // Event no longer has email
+    AuthenticationSignInWithLinkAttempted event,
     Emitter<AuthenticationState> emit,
   ) async {
     emit(AuthenticationLoading()); // General loading for sign-in attempt

lib/authentication/bloc/authentication_event.dart

@@ -33,17 +33,14 @@ final class AuthenticationSendSignInLinkRequested extends AuthenticationEvent {
 final class AuthenticationSignInWithLinkAttempted extends AuthenticationEvent {
   /// {@macro authentication_sign_in_with_link_attempted}
   const AuthenticationSignInWithLinkAttempted({
-    // Remove email parameter
     required this.emailLink,
   });
 
-  // Remove email field
-
   /// The sign-in link received by the app.
   final String emailLink;
 
   @override
-  List<Object> get props => [emailLink]; // Remove email from props
+  List<Object> get props => [emailLink];
 }
 
 /// {@template authentication_google_sign_in_requested}

lib/authentication/view/authentication_page.dart

@@ -7,7 +7,7 @@ import 'package:ht_authentication_repository/ht_authentication_repository.dart';
 import 'package:ht_main/authentication/bloc/authentication_bloc.dart';
 import 'package:ht_main/l10n/l10n.dart';
 import 'package:ht_main/router/routes.dart';
-import 'package:ht_main/shared/constants/app_spacing.dart'; // Use shared constants
+import 'package:ht_main/shared/constants/app_spacing.dart'; 
 
 /// {@template authentication_page}
 /// Displays authentication options (Google, Email, Anonymous) based on context.
@@ -21,8 +21,7 @@ class AuthenticationPage extends StatelessWidget {
     required this.headline,
     required this.subHeadline,
     required this.showAnonymousButton,
-    required this.isLinkingContext, // Add this parameter
-    // this.iconData, // REMOVE optional icon data
+    required this.isLinkingContext,
     super.key,
   });
 
@@ -36,10 +35,7 @@ class AuthenticationPage extends StatelessWidget {
   final bool showAnonymousButton;
 
   /// Whether this page is being shown in the account linking context.
-  final bool isLinkingContext; // Add this field
-
-  /// Optional icon to display at the top of the page.
-  // final IconData? iconData; // REMOVE this field
+  final bool isLinkingContext;
 
   @override
   Widget build(BuildContext context) {
@@ -57,28 +53,24 @@ class AuthenticationPage extends StatelessWidget {
         headline: headline,
         subHeadline: subHeadline,
         showAnonymousButton: showAnonymousButton,
-        isLinkingContext: isLinkingContext, // Pass down the flag
-        // iconData: iconData, // REMOVE passing down icon data
+        isLinkingContext: isLinkingContext,
       ),
     );
   }
 }
 
-// Renamed from _AuthenticationView to follow convention
 class _AuthenticationView extends StatelessWidget {
   const _AuthenticationView({
     required this.headline,
     required this.subHeadline,
     required this.showAnonymousButton,
-    required this.isLinkingContext, // Add this parameter
-    // this.iconData, // REMOVE optional icon data
+    required this.isLinkingContext,
   });
 
   final String headline;
   final String subHeadline;
   final bool showAnonymousButton;
-  final bool isLinkingContext; // Add this field
-  // final IconData? iconData; // REMOVE this field
+  final bool isLinkingContext;
 
   @override
   Widget build(BuildContext context) {
@@ -130,7 +122,7 @@ class _AuthenticationView extends StatelessWidget {
             return Padding(
               padding: const EdgeInsets.all(
                 AppSpacing.paddingLarge,
-              ), // Use constant
+              ), 
               child: Center(
                 child: SingleChildScrollView(
                   child: Column(
@@ -139,35 +131,33 @@ class _AuthenticationView extends StatelessWidget {
                     crossAxisAlignment: CrossAxisAlignment.stretch,
                     children: [
                       // --- Hardcoded Icon ---
-                      // REMOVE if check: if (iconData != null) ...[
                       Padding(
                         padding: const EdgeInsets.only(
                           bottom: AppSpacing.xl,
                         ), // Spacing below icon
                         child: Icon(
                           Icons.security, // Hardcode the icon
                           size: (Theme.of(context).iconTheme.size ?? AppSpacing.xl) *
-                              3.0, // Use ?? with AppSpacing constant
+                              3.0, 
                           color: Theme.of(context)
                               .colorScheme
-                              .primary, // Use theme color
+                              .primary, 
                         ),
                       ),
                       const SizedBox(height: AppSpacing.lg), // Space between icon and headline
-                      // REMOVE closing bracket for if: ],
                       // --- Headline and Subheadline ---
                       Text(
                         headline,
                         style: textTheme.headlineMedium,
                         textAlign: TextAlign.center,
                       ),
-                      const SizedBox(height: AppSpacing.sm), // Use constant
+                      const SizedBox(height: AppSpacing.sm), 
                       Text(
                         subHeadline,
                         style: textTheme.bodyLarge,
                         textAlign: TextAlign.center,
                       ),
-                      const SizedBox(height: AppSpacing.xxl), // Use constant
+                      const SizedBox(height: AppSpacing.xxl), 
                       // --- Google Sign-In Button ---
                       ElevatedButton.icon(
                         icon: const Icon(
@@ -182,14 +172,14 @@ class _AuthenticationView extends StatelessWidget {
                                 ),
                         // Style adjustments can be made via ElevatedButtonThemeData
                       ),
-                      const SizedBox(height: AppSpacing.lg), // Use constant
+                      const SizedBox(height: AppSpacing.lg), 
                       // --- Email Sign-In Button ---
                       ElevatedButton(
                         // Consider an email icon
                         // icon: const Icon(Icons.email_outlined),
                         child: Text(
                           l10n.authenticationEmailSignInButton,
-                        ), // New l10n key needed
+                        ),
                         onPressed:
                             isLoading
                                 ? null
@@ -198,14 +188,14 @@ class _AuthenticationView extends StatelessWidget {
                                   // passing the linking context via 'extra'.
                                   context.goNamed(
                                     Routes.emailSignInName,
-                                    extra: isLinkingContext, // Pass the flag
+                                    extra: isLinkingContext,
                                   );
                                 },
                       ),
 
                       // --- Anonymous Sign-In Button (Conditional) ---
                       if (showAnonymousButton) ...[
-                        const SizedBox(height: AppSpacing.lg), // Use constant
+                        const SizedBox(height: AppSpacing.lg), 
                         OutlinedButton(
                           child: Text(l10n.authenticationAnonymousSignInButton),
                           onPressed:

lib/authentication/view/email_sign_in_page.dart

@@ -42,7 +42,7 @@ class _EmailSignInView extends StatelessWidget {
 
     return Scaffold(
       appBar: AppBar(
-        title: Text(l10n.emailSignInPageTitle), // New l10n key needed
+        title: Text(l10n.emailSignInPageTitle), 
         // Add a custom leading back button to control navigation based on context.
         leading: IconButton(
           icon: const Icon(Icons.arrow_back),
@@ -107,20 +107,22 @@ class _EmailSignInView extends StatelessWidget {
                         ), // Spacing below icon
                         child: Icon(
                           Icons.email_outlined, // Hardcoded icon
-                          size: (Theme.of(context).iconTheme.size ?? AppSpacing.xl) *
-                              3.0, // Use ?? with AppSpacing constant
+                          size:
+                              (Theme.of(context).iconTheme.size ??
+                                  AppSpacing.xl) *
+                              3.0, 
                           color:
                               Theme.of(
                                 context,
-                              ).colorScheme.primary, // Use theme color
+                              ).colorScheme.primary, 
                         ),
                       ),
                       const SizedBox(
                         height: AppSpacing.lg,
                       ), // Space between icon and text
                       // --- Explanation Text ---
                       Text(
-                        l10n.emailSignInExplanation, // New l10n key needed
+                        l10n.emailSignInExplanation, 
                         style: Theme.of(context).textTheme.bodyLarge,
                         textAlign: TextAlign.center,
                       ),
@@ -181,7 +183,7 @@ class _EmailLinkFormState extends State<_EmailLinkForm> {
           TextFormField(
             controller: _emailController,
             decoration: InputDecoration(
-              labelText: l10n.authenticationEmailLabel, // Re-use existing key
+              labelText: l10n.authenticationEmailLabel, 
               border: const OutlineInputBorder(),
               // Consider adding hint text if needed
             ),
@@ -191,9 +193,8 @@ class _EmailLinkFormState extends State<_EmailLinkForm> {
             enabled: !widget.isLoading,
             validator: (value) {
               if (value == null || value.isEmpty || !value.contains('@')) {
-                // Add a specific validation error message key if needed
                 return l10n
-                    .accountLinkingEmailValidationError; // Re-use for now
+                    .accountLinkingEmailValidationError;
               }
               return null;
             },
@@ -206,13 +207,13 @@ class _EmailLinkFormState extends State<_EmailLinkForm> {
             child:
                 widget.isLoading
                     ? const SizedBox(
-                      height: 24, // Consistent height
+                      height: 24,
                       width: 24,
                       child: CircularProgressIndicator(strokeWidth: 2),
                     )
                     : Text(
                       l10n.authenticationSendLinkButton,
-                    ), // Re-use existing key
+                    ),
           ),
         ],
       ),