Patch group profile image

ios/Podfile.lock

@@ -99,8 +99,8 @@ EXTERNAL SOURCES:
     :path: ".symlinks/plugins/workmanager_apple/ios"
 
 SPEC CHECKSUMS:
-  audio_session: 9bb7f6c970f21241b19f5a3658097ae459681ba0
-  emoji_picker_flutter: ece213fc274bdddefb77d502d33080dc54e616cc
+  audio_session: 19e9480dbdd4e5f6c4543826b2e8b0e4ab6145fe
+  emoji_picker_flutter: 8e50ec5caac456a23a78637e02c6293ea0ac8771
   Flutter: cabc95a1d2626b1b06e7179b784ebcf0c0cde467
   flutter_local_notifications: df98d66e515e1ca797af436137b4459b160ad8c9
   flutter_native_splash: df59bb2e1421aa0282cb2e95618af4dcb0c56c29

lib/config/providers/group_provider.dart

@@ -164,6 +164,9 @@ class GroupsNotifier extends Notifier<GroupsState> {
       // Load and cache group types for synchronous access
       await _loadGroupTypesForAllGroups(groups);
 
+      // Load and cache group image paths
+      await _loadGroupImagePaths(groups);
+
       // Now calculate display names with member data available
       await _calculateDisplayNames(groups);
 
@@ -622,6 +625,56 @@ class GroupsNotifier extends Notifier<GroupsState> {
     }
   }
 
+  /// Load and cache group image paths for all groups
+  Future<void> _loadGroupImagePaths(List<Group> groups) async {
+    try {
+      final Map<String, String> groupImagePaths = Map<String, String>.from(
+        state.groupImagePaths ?? {},
+      );
+
+      final List<Future<void>> loadTasks = [];
+      final activePubkey = ref.read(activePubkeyProvider);
+      if (activePubkey == null || activePubkey.isEmpty) return;
+
+      for (final group in groups) {
+        loadTasks.add(
+          getGroupImagePath(
+                accountPubkey: activePubkey,
+                groupId: group.mlsGroupId,
+              )
+              .then((imagePath) {
+                if (imagePath != null && imagePath.isNotEmpty) {
+                  groupImagePaths[group.mlsGroupId] = imagePath;
+                }
+              })
+              .catchError((e) {
+                _logErrorSync('Failed to load image path for group ${group.mlsGroupId}', e);
+                // Skip this group if image loading fails
+              }),
+        );
+      }
+
+      // Execute all image path loading in parallel for better performance
+      await Future.wait(loadTasks);
+
+      // Update state with the cached image paths
+      state = state.copyWith(groupImagePaths: groupImagePaths);
+
+      _logger.info('GroupsProvider: Loaded image paths for ${groupImagePaths.length} groups');
+    } catch (e) {
+      // Log the full exception details with proper ApiError unpacking
+      String logMessage = 'GroupsProvider: Error loading group image paths - Exception: ';
+      if (e is ApiError) {
+        final errorDetails = await e.messageText();
+        logMessage += '$errorDetails (Type: ${e.runtimeType})';
+      } else {
+        logMessage += '$e (Type: ${e.runtimeType})';
+      }
+      _logger.severe(logMessage, e);
+      // Don't throw - we want to continue even if some image loading fails
+    }
+  }
+
   /// Load members for all groups (used during initial group loading)
   Future<void> _loadMembersForAllGroups(List<Group> groups) async {
     try {
@@ -805,6 +858,12 @@ class GroupsNotifier extends Notifier<GroupsState> {
     return state.groupTypes?[groupId];
   }
 
+  /// Get cached group image path synchronously
+  /// Returns null if group image path is not cached yet
+  String? getCachedGroupImagePath(String groupId) {
+    return state.groupImagePaths?[groupId];
+  }
+
   Future<bool> isCurrentUserAdmin(String groupId) async {
     try {
       final activePubkey = ref.read(activePubkeyProvider) ?? '';
@@ -895,6 +954,9 @@ class GroupsNotifier extends Notifier<GroupsState> {
         // Load and cache group types for new groups
         await _loadGroupTypesForAllGroups(actuallyNewGroups);
 
+        // Load and cache group image paths for new groups
+        await _loadGroupImagePaths(actuallyNewGroups);
+
         // Calculate display names for new groups
         await _calculateDisplayNamesForSpecificGroups(actuallyNewGroups);
 
@@ -1248,7 +1310,7 @@ extension GroupMemberUtils on GroupsNotifier {
 
   /// Get the display image for a group based on its type
   /// For direct messages, returns the other member's image
-  /// For regular groups, returns null (can be extended for group avatars)
+  /// For regular groups, returns the cached group image path
   String? getGroupDisplayImage(String groupId) {
     final group = findGroupById(groupId);
     if (group == null) return null;
@@ -1269,7 +1331,7 @@ extension GroupMemberUtils on GroupsNotifier {
       return otherMember?.imagePath;
     }
 
-    // For regular groups, could return group avatar in the future
-    return null;
+    // For regular groups, return the cached group image path
+    return getCachedGroupImagePath(groupId);
   }
 }

lib/config/states/group_state.dart

@@ -13,6 +13,7 @@ abstract class GroupsState with _$GroupsState {
     Map<String, List<User>>? groupAdmins, // groupId -> admins
     Map<String, String>? groupDisplayNames, // groupId -> display name
     Map<String, GroupType>? groupTypes, // groupId -> GroupType (cached for synchronous access)
+    Map<String, String>? groupImagePaths, // groupId -> image file path
     @Default(false) bool isLoading,
     String? error,
   }) = _GroupsState;

lib/ui/chat/chat_info/chat_info_screen.dart

@@ -9,6 +9,7 @@ import 'package:whitenoise/config/providers/chat_search_provider.dart';
 import 'package:whitenoise/config/providers/follow_provider.dart';
 import 'package:whitenoise/config/providers/follows_provider.dart';
 import 'package:whitenoise/config/providers/group_provider.dart';
+import 'package:whitenoise/config/states/group_state.dart';
 import 'package:whitenoise/domain/models/dm_chat_data.dart';
 import 'package:whitenoise/domain/models/user_model.dart';
 import 'package:whitenoise/domain/services/dm_chat_service.dart';

lib/ui/chat/chat_info/group_chat_info.dart

@@ -15,6 +15,8 @@ class _GroupChatInfoState extends ConsumerState<GroupChatInfo> {
   List<User> groupAdmins = [];
   bool isLoadingMembers = false;
   String? currentUserNpub;
+  String? groupImagePath;
+  ProviderSubscription<GroupsState>? _groupsSubscription;
 
   @override
   void initState() {
@@ -23,9 +25,25 @@ class _GroupChatInfoState extends ConsumerState<GroupChatInfo> {
       _loadGroup();
       _loadMembers();
       _loadCurrentUserNpub();
+
+      setState(() {
+        groupImagePath = ref.read(groupsProvider.notifier).getCachedGroupImagePath(widget.groupId);
+      });
+
+      _groupsSubscription = ref.listenManual(groupsProvider, (previous, next) {
+        if (mounted) {
+          _loadMembers();
+        }
+      });
     });
   }
 
+  @override
+  void dispose() {
+    _groupsSubscription?.close();
+    super.dispose();
+  }
+
   Future<void> _loadGroup() async {
     final groupDetails = ref.read(groupsProvider).groupsMap?[widget.groupId];
     if (groupDetails?.nostrGroupId != null) {
@@ -127,9 +145,6 @@ class _GroupChatInfoState extends ConsumerState<GroupChatInfo> {
   @override
   Widget build(BuildContext context) {
     final groupDetails = ref.watch(groupsProvider).groupsMap?[widget.groupId];
-    ref.listen(groupsProvider, (previous, next) {
-      _loadMembers();
-    });
     final isAdmin = groupAdmins.any((admin) {
       if (currentUserNpub == null) {
         return false;
@@ -146,7 +161,7 @@ class _GroupChatInfoState extends ConsumerState<GroupChatInfo> {
         children: [
           Gap(64.h),
           WnAvatar(
-            imageUrl: '',
+            imageUrl: groupImagePath ?? '',
             displayName: groupDetails?.name ?? 'chats.unknownGroup'.tr(),
             size: 96.w,
             showBorder: true,

lib/ui/chat/chat_screen.dart

@@ -382,7 +382,10 @@ class _ChatScreenState extends ConsumerState<ChatScreen> with WidgetsBindingObse
                                         image:
                                             groupType == GroupType.directMessage
                                                 ? otherUser?.displayImage ?? ''
-                                                : '',
+                                                : groupsNotifier.getCachedGroupImagePath(
+                                                      widget.groupId,
+                                                    ) ??
+                                                    '',
                                         onTap: () => context.push('/chats/${widget.groupId}/info'),
                                       );
                                     },

lib/ui/chat/widgets/chat_header_widget.dart

@@ -70,13 +70,18 @@ class _GroupChatHeaderState extends ConsumerState<GroupChatHeader> {
 
   @override
   Widget build(BuildContext context) {
+    final groupsNotifier = ref.watch(groupsProvider.notifier);
     return Container(
       padding: EdgeInsets.symmetric(horizontal: 24.w),
       child: Column(
         children: [
           Gap(32.h),
           WnAvatar(
-            imageUrl: '',
+            imageUrl:
+                groupsNotifier.getCachedGroupImagePath(
+                  widget.group.mlsGroupId,
+                ) ??
+                '',
             displayName: widget.group.name,
             size: 96.r,
             showBorder: true,