refactor: remove metadata cache provider

Author: josefinalliende

ios/Podfile.lock

@@ -86,7 +86,7 @@ EXTERNAL SOURCES:
 SPEC CHECKSUMS:
   audio_session: 9bb7f6c970f21241b19f5a3658097ae459681ba0
   emoji_picker_flutter: ece213fc274bdddefb77d502d33080dc54e616cc
-  Flutter: cabc95a1d2626b1b06e7179b784ebcf0c0cde467
+  Flutter: e0871f40cf51350855a761d2e70bf5af5b9b5de7
   flutter_native_splash: c32d145d68aeda5502d5f543ee38c192065986cf
   flutter_secure_storage: 1ed9476fba7e7a782b22888f956cce43e2c62f13
   image_picker_ios: 7fe1ff8e34c1790d6fff70a32484959f563a928a

lib/config/providers/chat_provider.dart

@@ -6,12 +6,12 @@ import 'package:logging/logging.dart';
 import 'package:whitenoise/config/providers/active_pubkey_provider.dart';
 import 'package:whitenoise/config/providers/auth_provider.dart';
 import 'package:whitenoise/config/providers/group_provider.dart';
+import 'package:whitenoise/config/services/group_messages_service.dart';
 import 'package:whitenoise/config/states/chat_state.dart';
 import 'package:whitenoise/domain/models/message_model.dart';
 import 'package:whitenoise/src/rust/api/error.dart' show ApiError;
 import 'package:whitenoise/src/rust/api/messages.dart';
 import 'package:whitenoise/src/rust/api/utils.dart';
-import 'package:whitenoise/utils/message_converter.dart';
 import 'package:whitenoise/utils/pubkey_formatter.dart';
 
 class ChatNotifier extends Notifier<ChatState> {
@@ -86,11 +86,12 @@ class ChatNotifier extends Notifier<ChatState> {
       // Sort messages by creation time (oldest first)
       aggregatedMessages.sort((a, b) => a.createdAt.compareTo(b.createdAt));
 
-      final messages = await MessageConverter.fromChatMessageList(
-        aggregatedMessages,
-        currentUserPublicKey: activePubkey,
+      final messages = await GroupMessagesService(
         groupId: groupId,
         ref: ref,
+      ).toMessageModels(
+        chatMessages: aggregatedMessages,
+        activePubkey: activePubkey,
       );
 
       state = state.copyWith(
@@ -178,7 +179,7 @@ class ChatNotifier extends Notifier<ChatState> {
       );
 
       // Build message cache from current messages for consistency
-      final messageCache = <String, ChatMessage>{};
+      final chatMessagesMap = <String, ChatMessage>{};
       for (final msg in currentMessages) {
         // Convert existing MessageModel back to ChatMessage for cache
         final chatMessage = ChatMessage(
@@ -194,15 +195,16 @@ class ChatNotifier extends Notifier<ChatState> {
           reactions: const ReactionSummary(byEmoji: [], userReactions: []),
           kind: msg.kind, // Use the actual message kind
         );
-        messageCache[msg.id] = chatMessage;
+        chatMessagesMap[msg.id] = chatMessage;
       }
 
-      final sentMessageModel = await MessageConverter.fromChatMessage(
-        sentChatMessage,
-        currentUserPublicKey: activePubkey,
+      final sentMessageModel = await GroupMessagesService(
         groupId: groupId,
         ref: ref,
-        messageCache: messageCache,
+      ).toMessageModel(
+        chatMessage: sentChatMessage,
+        activePubkey: activePubkey,
+        chatMessagesMap: chatMessagesMap,
       );
       final updatedMessages = [...currentMessages, sentMessageModel];
 
@@ -297,11 +299,13 @@ class ChatNotifier extends Notifier<ChatState> {
       );
 
       aggregatedMessages.sort((a, b) => a.createdAt.compareTo(b.createdAt));
-      final newMessages = await MessageConverter.fromChatMessageList(
-        aggregatedMessages,
-        currentUserPublicKey: activePubkey,
+
+      final newMessages = await GroupMessagesService(
         groupId: groupId,
         ref: ref,
+      ).toMessageModels(
+        chatMessages: aggregatedMessages,
+        activePubkey: activePubkey,
       );
 
       final currentMessages = state.groupMessages[groupId] ?? [];
@@ -622,7 +626,7 @@ class ChatNotifier extends Notifier<ChatState> {
       );
 
       // Build message cache from current messages for reply lookup
-      final messageCache = <String, ChatMessage>{};
+      final chatMessagesMap = <String, ChatMessage>{};
       for (final msg in currentMessages) {
         // Convert existing MessageModel back to ChatMessage for cache
         final chatMessage = ChatMessage(
@@ -638,15 +642,16 @@ class ChatNotifier extends Notifier<ChatState> {
           reactions: const ReactionSummary(byEmoji: [], userReactions: []),
           kind: msg.kind, // Use the actual message kind
         );
-        messageCache[msg.id] = chatMessage;
+        chatMessagesMap[msg.id] = chatMessage;
       }
 
-      final sentMessageModel = await MessageConverter.fromChatMessage(
-        sentChatMessage,
-        currentUserPublicKey: activePubkey,
+      final sentMessageModel = await GroupMessagesService(
         groupId: groupId,
         ref: ref,
-        messageCache: messageCache,
+      ).toMessageModel(
+        chatMessage: sentChatMessage,
+        activePubkey: activePubkey,
+        chatMessagesMap: chatMessagesMap,
       );
       final updatedMessages = [...currentMessages, sentMessageModel];