···

       
87
       
 
       
    try {

     

       
88
       
 
       
      if (refresh) {

     

       
89
       
 
       
        _isLoading = true;

     

       
90
       
-
       
        _posts = [];

     

       
91
       
-
       
        _cursor = null;

     

       
92
       
-
       
        _hasMore = true;

     

       
93
       
 
       
        _error = null;

     

       
94
       
 
       
      } else {

     

       
95
       
 
       
        _isLoadingMore = true;

     
···

       
98
       
 
       


     

       
99
       
 
       
      final response = await fetcher();

     

       
100
       
 
       


     

       
0
       
 
       

     

       
101
       
 
       
      if (refresh) {

     

       
102
       
 
       
        _posts = response.feed;

     

       
103
       
 
       
      } else {

     

       
104
       
-
       
        _posts.addAll(response.feed);

     

       
0
       
 
       

     

       
0
       
 
       

     

       
105
       
 
       
      }

     

       
106
       
 
       


     

       
107
       
 
       
      _cursor = response.cursor;

     

···

       
87
       
 
       
    try {

     

       
88
       
 
       
      if (refresh) {

     

       
89
       
 
       
        _isLoading = true;

     

       
90
       
+
       
        // DON'T clear _posts, _cursor, or _hasMore yet

     

       
91
       
+
       
        // Keep existing data visible until refresh succeeds

     

       
92
       
+
       
        // This prevents transient failures from wiping the user's feed and pagination state

     

       
93
       
 
       
        _error = null;

     

       
94
       
 
       
      } else {

     

       
95
       
 
       
        _isLoadingMore = true;

     
···

       
98
       
 
       


     

       
99
       
 
       
      final response = await fetcher();

     

       
100
       
 
       


     

       
101
       
+
       
      // Only update state after successful fetch

     

       
102
       
 
       
      if (refresh) {

     

       
103
       
 
       
        _posts = response.feed;

     

       
104
       
 
       
      } else {

     

       
105
       
+
       
        // Create new list instance to trigger context.select rebuilds

     

       
106
       
+
       
        // Using spread operator instead of addAll to ensure reference changes

     

       
107
       
+
       
        _posts = [..._posts, ...response.feed];

     

       
108
       
 
       
      }

     

       
109
       
 
       


     

       
110
       
 
       
      _cursor = response.cursor;

     

···

       
66
       
 
       
    final isLoading = context.select<FeedProvider, bool>((p) => p.isLoading);

     

       
67
       
 
       
    final error = context.select<FeedProvider, String?>((p) => p.error);

     

       
68
       
 
       


     

       
69
       
-
       
    // IMPORTANT: This works because FeedProvider replaces the list (_posts = ...)

     

       
70
       
-
       
    // rather than mutating it in-place (_posts.addAll(...)).

     

       
71
       
-
       
    // If you change FeedProvider to use in-place mutations, this will break

     

       
72
       
-
       
    // because Lists use reference equality by default.

     

       
73
       
 
       
    final posts = context.select<FeedProvider, List<FeedViewPost>>(

     

       
74
       
 
       
      (p) => p.posts,

     

       
75
       
 
       
    );

     
···

       
104
       
 
       
    required bool isLoadingMore,

     

       
105
       
 
       
    required bool isAuthenticated,

     

       
106
       
 
       
  }) {

     

       
107
       
-
       
    // Loading state

     

       
108
       
-
       
    if (isLoading) {

     

       
109
       
 
       
      return const Center(

     

       
110
       
 
       
        child: CircularProgressIndicator(color: Color(0xFFFF6B35)),

     

       
111
       
 
       
      );

     

···

       
66
       
 
       
    final isLoading = context.select<FeedProvider, bool>((p) => p.isLoading);

     

       
67
       
 
       
    final error = context.select<FeedProvider, String?>((p) => p.error);

     

       
68
       
 
       


     

       
69
       
+
       
    // IMPORTANT: This relies on FeedProvider creating new list instances

     

       
70
       
+
       
    // (_posts = [..._posts, ...response.feed]) rather than mutating in-place.

     

       
71
       
+
       
    // context.select uses == for comparison, and Lists use reference equality,

     

       
72
       
+
       
    // so in-place mutations (_posts.addAll(...)) would not trigger rebuilds.

     

       
73
       
 
       
    final posts = context.select<FeedProvider, List<FeedViewPost>>(

     

       
74
       
 
       
      (p) => p.posts,

     

       
75
       
 
       
    );

     
···

       
104
       
 
       
    required bool isLoadingMore,

     

       
105
       
 
       
    required bool isAuthenticated,

     

       
106
       
 
       
  }) {

     

       
107
       
+
       
    // Loading state (only show full-screen loader for initial load, not refresh)

     

       
108
       
+
       
    if (isLoading && posts.isEmpty) {

     

       
109
       
 
       
      return const Center(

     

       
110
       
 
       
        child: CircularProgressIndicator(color: Color(0xFFFF6B35)),

     

       
111
       
 
       
      );