commit 3c23c865df94291fe6f2df7bc0e227edb83f81e5 · bretton.dev/coves-mobile

-4

lib/main.dart

···

       60
       60
        
                     (context) => FeedProvider(

     

       61
       61
        
                       authProvider,

     

       62
       62
        
                       voteProvider: context.read<VoteProvider>(),

     

       63
       63
       -
                       voteService: voteService,

     

       64
       63
        
                     ),

     

       65
       64
        
                 update: (context, auth, vote, previous) {

     

       66
       65
        
                   // Reuse existing provider to maintain state across rebuilds

     
···

       68
       67
        
                       FeedProvider(

     

       69
       68
        
                         auth,

     

       70
       69
        
                         voteProvider: vote,

     

       71
       71
       -
                         voteService: voteService,

     

       72
       70
        
                       );

     

       73
       71
        
                 },

     

       74
       72
        
               ),

     
···

       81
       79
        
                     (context) => CommentsProvider(

     

       82
       80
        
                       authProvider,

     

       83
       81
        
                       voteProvider: context.read<VoteProvider>(),

     

       84
       84
       -
                       voteService: voteService,

     

       85
       82
        
                     ),

     

       86
       83
        
                 update: (context, auth, vote, previous) {

     

       87
       84
        
                   // Reuse existing provider to maintain state across rebuilds

     
···

       89
       86
        
                       CommentsProvider(

     

       90
       87
        
                         auth,

     

       91
       88
        
                         voteProvider: vote,

     

       92
       92
       -
                         voteService: voteService,

     

       93
       89
        
                       );

     

       94
       90
        
                 },

     

       95
       91
        
               ),

+30 -17

lib/providers/comments_provider.dart

···

       3
       3
        
       import 'package:flutter/foundation.dart';

     

       4
       4
        
       import '../models/comment.dart';

     

       5
       5
        
       import '../services/coves_api_service.dart';

     

       6
       6
       -
       import '../services/vote_service.dart';

     

       7
       6
        
       import 'auth_provider.dart';

     

       8
       7
        
       import 'vote_provider.dart';

     

       9
       8
        
       

     
···

       20
       19
        
           this._authProvider, {

     

       21
       20
        
           CovesApiService? apiService,

     

       22
       21
        
           VoteProvider? voteProvider,

     

       23
       23
       -
           VoteService? voteService,

     

       24
       24
       -
         }) : _voteProvider = voteProvider,

     

       25
       25
       -
              _voteService = voteService {

     

       22
       22
       +
         }) : _voteProvider = voteProvider {

     

       26
       23
        
           // Use injected service (for testing) or create new one (for production)

     

       27
       24
        
           // Pass token getter, refresh handler, and sign out handler to API service

     

       28
       25
        
           // for automatic fresh token retrieval and automatic token refresh on 401

     
···

       62
       59
        
         final AuthProvider _authProvider;

     

       63
       60
        
         late final CovesApiService _apiService;

     

       64
       61
        
         final VoteProvider? _voteProvider;

     

       65
       65
       -
         final VoteService? _voteService;

     

       66
       62
        
       

     

       67
       63
        
         // Track previous auth state to detect transitions

     

       68
       64
        
         bool _wasAuthenticated = false;

     
···

       195
       191
        
               debugPrint('✅ Comments loaded: ${_comments.length} comments total');

     

       196
       192
        
             }

     

       197
       193
        
       

     

       198
       198
       -
             // Load initial vote state from PDS (only if authenticated)

     

       199
       199
       -
             if (_authProvider.isAuthenticated &&

     

       200
       200
       -
                 _voteProvider != null &&

     

       201
       201
       -
                 _voteService != null) {

     

       202
       202
       -
               try {

     

       203
       203
       -
                 final userVotes = await _voteService.getUserVotes();

     

       204
       204
       -
                 _voteProvider.loadInitialVotes(userVotes);

     

       205
       205
       -
               } on Exception catch (e) {

     

       206
       206
       -
                 if (kDebugMode) {

     

       207
       207
       -
                   debugPrint('⚠️ Failed to load vote state: $e');

     

       208
       208
       -
                 }

     

       209
       209
       -
                 // Don't fail the comments load if vote loading fails

     

       194
       194
       +
             // Initialize vote state from viewer data in comments response

     

       195
       195
       +
             if (_authProvider.isAuthenticated && _voteProvider != null) {

     

       196
       196
       +
               if (refresh) {

     

       197
       197
       +
                 // On refresh, initialize all comments - server data is truth

     

       198
       198
       +
                 _comments.forEach(_initializeCommentVoteState);

     

       199
       199
       +
               } else {

     

       200
       200
       +
                 // On pagination, only initialize newly fetched comments to avoid

     

       201
       201
       +
                 // overwriting optimistic vote state on existing comments

     

       202
       202
       +
                 response.comments.forEach(_initializeCommentVoteState);

     

       210
       203
        
               }

     

       211
       204
        
             }

     

       212
       205
        
       

     
···

       338
       331
        
             }

     

       339
       332
        
             rethrow;

     

       340
       333
        
           }

     

       334
       334
       +
         }

     

       335
       335
       +
       

     

       336
       336
       +
         /// Initialize vote state for a comment and its replies recursively

     

       337
       337
       +
         ///

     

       338
       338
       +
         /// Extracts viewer vote data from comment and initializes VoteProvider state.

     

       339
       339
       +
         /// Handles nested replies recursively.

     

       340
       340
       +
         ///

     

       341
       341
       +
         /// IMPORTANT: Always calls setInitialVoteState, even when viewer.vote is

     

       342
       342
       +
         /// null. This ensures that if a user removed their vote on another device,

     

       343
       343
       +
         /// the local state is cleared on refresh.

     

       344
       344
       +
         void _initializeCommentVoteState(ThreadViewComment threadComment) {

     

       345
       345
       +
           final viewer = threadComment.comment.viewer;

     

       346
       346
       +
           _voteProvider!.setInitialVoteState(

     

       347
       347
       +
             postUri: threadComment.comment.uri,

     

       348
       348
       +
             voteDirection: viewer?.vote,

     

       349
       349
       +
             voteUri: viewer?.voteUri,

     

       350
       350
       +
           );

     

       351
       351
       +
       

     

       352
       352
       +
           // Recursively initialize vote state for replies

     

       353
       353
       +
           threadComment.replies?.forEach(_initializeCommentVoteState);

     

       341
       354
        
         }

     

       342
       355
        
       

     

       343
       356
        
         /// Retry loading after error