fix: range error on chat drawer

This commit is contained in:
cogwheel0
2025-08-28 23:05:27 +05:30
parent 3d8948c84a
commit f65cf33c59
4 changed files with 75 additions and 46 deletions

View File

@@ -442,8 +442,11 @@ final conversationsProvider = FutureProvider<List<Conversation>>((ref) async {
conversationMap[conversation.id] = conversation.copyWith(
folderId: folderIdToUse,
);
final _idPreview = conversation.id.length > 8
? conversation.id.substring(0, 8)
: conversation.id;
foundation.debugPrint(
'DEBUG: Updated conversation ${conversation.id.substring(0, 8)} with folderId: $folderIdToUse (explicit: ${explicitFolderId != null})',
'DEBUG: Updated conversation $_idPreview with folderId: $folderIdToUse (explicit: ${explicitFolderId != null})',
);
} else {
conversationMap[conversation.id] = conversation;
@@ -505,8 +508,11 @@ final conversationsProvider = FutureProvider<List<Conversation>>((ref) async {
// Use map to prevent duplicates - this will overwrite if ID already exists
conversationMap[toAdd.id] = toAdd;
existingIds.add(toAdd.id);
final _idPreview = toAdd.id.length > 8
? toAdd.id.substring(0, 8)
: toAdd.id;
foundation.debugPrint(
'DEBUG: Added missing conversation from folder fetch: ${toAdd.id.substring(0, 8)} -> folder ${folder.id}',
'DEBUG: Added missing conversation from folder fetch: $_idPreview -> folder ${folder.id}',
);
} else {
// Create a minimal placeholder if not returned by folder API
@@ -521,8 +527,11 @@ final conversationsProvider = FutureProvider<List<Conversation>>((ref) async {
// Use map to prevent duplicates
conversationMap[convId] = placeholder;
existingIds.add(convId);
final _idPreview = convId.length > 8
? convId.substring(0, 8)
: convId;
foundation.debugPrint(
'DEBUG: Added placeholder conversation for missing ID: ${convId.substring(0, 8)} -> folder ${folder.id}',
'DEBUG: Added placeholder conversation for missing ID: $_idPreview -> folder ${folder.id}',
);
}
}

View File

@@ -417,9 +417,11 @@ class ApiService {
debugPrint(
'🔍 DEBUG: Sample chat data fields: ${chatData.keys.toList()}',
);
debugPrint(
'🔍 DEBUG: Sample chat data: ${chatData.toString().substring(0, 200)}...',
);
final _sampleStr = chatData.toString();
final _preview = _sampleStr.length > 200
? _sampleStr.substring(0, 200)
: _sampleStr;
debugPrint('🔍 DEBUG: Sample chat data: $_preview...');
}
final conversation = _parseOpenWebUIChat(chatData);
@@ -498,9 +500,8 @@ class ApiService {
// Debug logging for folder assignment
if (folderId != null) {
debugPrint(
'🔍 DEBUG: Conversation ${id.substring(0, 8)} has folderId: $folderId',
);
final _idPreview = id.length > 8 ? id.substring(0, 8) : id;
debugPrint('🔍 DEBUG: Conversation $_idPreview has folderId: $folderId');
}
debugPrint(
@@ -3020,9 +3021,11 @@ class ApiService {
} else if (response.data is Map) {
DebugLogger.log(' Object keys: ${(response.data as Map).keys}');
}
DebugLogger.log(
' Sample data: ${response.data.toString().substring(0, 200)}...',
);
final _dataStr = response.data.toString();
final _dataPreview = _dataStr.length > 200
? _dataStr.substring(0, 200)
: _dataStr;
DebugLogger.log(' Sample data: $_dataPreview...');
} catch (e) {
debugPrint('$endpoint - Error: $e');
}

View File

@@ -198,23 +198,32 @@ class _ChatsDrawerState extends ConsumerState<ChatsDrawer> {
// Build sections
final pinned = list.where((c) => c.pinned == true).toList();
final regular = list
.where(
(c) =>
c.pinned != true &&
c.archived != true &&
(c.folderId == null || c.folderId!.isEmpty),
)
.toList();
final foldered = list
.where(
(c) =>
c.pinned != true &&
c.archived != true &&
c.folderId != null &&
c.folderId!.isNotEmpty,
)
.toList();
// Determine which folder IDs actually exist from the API
final foldersState = ref.watch(foldersProvider);
final availableFolderIds = foldersState.maybeWhen(
data: (folders) => folders.map((f) => f.id).toSet(),
orElse: () => <String>{},
);
// Conversations that reference a non-existent/unknown folder should not disappear.
// Treat those as regular until the folders list is available and contains the ID.
final regular = list.where((c) {
final hasFolder = (c.folderId != null && c.folderId!.isNotEmpty);
final folderKnown = hasFolder && availableFolderIds.contains(c.folderId);
return c.pinned != true &&
c.archived != true &&
(!hasFolder || !folderKnown);
}).toList();
final foldered = list.where((c) {
final hasFolder = (c.folderId != null && c.folderId!.isNotEmpty);
return c.pinned != true &&
c.archived != true &&
hasFolder &&
availableFolderIds.contains(c.folderId);
}).toList();
final archived = list.where((c) => c.archived == true).toList();
return Scrollbar(
@@ -342,23 +351,30 @@ class _ChatsDrawerState extends ConsumerState<ChatsDrawer> {
}
final pinned = list.where((c) => c.pinned == true).toList();
final regular = list
.where(
(c) =>
c.pinned != true &&
c.archived != true &&
(c.folderId == null || c.folderId!.isEmpty),
)
.toList();
final foldered = list
.where(
(c) =>
c.pinned != true &&
c.archived != true &&
c.folderId != null &&
c.folderId!.isNotEmpty,
)
.toList();
// For search results, apply the same folder safety logic
final foldersState = ref.watch(foldersProvider);
final availableFolderIds = foldersState.maybeWhen(
data: (folders) => folders.map((f) => f.id).toSet(),
orElse: () => <String>{},
);
final regular = list.where((c) {
final hasFolder = (c.folderId != null && c.folderId!.isNotEmpty);
final folderKnown = hasFolder && availableFolderIds.contains(c.folderId);
return c.pinned != true &&
c.archived != true &&
(!hasFolder || !folderKnown);
}).toList();
final foldered = list.where((c) {
final hasFolder = (c.folderId != null && c.folderId!.isNotEmpty);
return c.pinned != true &&
c.archived != true &&
hasFolder &&
availableFolderIds.contains(c.folderId);
}).toList();
final archived = list.where((c) => c.archived == true).toList();
return Scrollbar(