fix(chat): address all review findings for bulk resolve feature (EVO-1011)

- fix(i18n): replace hardcoded PT-BR toasts with t() calls in all 6 locales
- fix(bulk-resolve): consume success_ids/failed_ids for per-item failure surfacing
- fix(selection): clear selection when user opens a conversation row
- fix(permission): guard handleBulkResolve with can('conversations','update') check
- fix(context): use reloadCurrentFilters after bulk resolve instead of raw updateConversation
- fix(selection): cap bulk selection at 200 items client-side
- fix(i18n): wire chatSidebar.selected key into bulk action toolbar
- fix(lint): remove eslint-disable on onClearSelection useEffect dep
- fix(a11y): add aria-label to conversation row Checkbox
- fix(checkbox): use checked boolean in onCheckedChange handler
- fix(arch): move bulkResolve call from conversationAPI to chatService facade
- test: add vitest spec for chatService.bulkResolve and selection cap logic
- fix(layout): sidebar width expands to w-96 only when bulk toolbar is visible

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: marcelogorutuba

src/components/chat/chat-sidebar/ChatSidebar.tsx

@@ -158,8 +158,7 @@ const ChatSidebar = ({
 
   useEffect(() => {
     onClearSelection();
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [showArchived]);
+  }, [showArchived, onClearSelection]);
 
   // Pipeline state
   const [allPipelines, setAllPipelines] = useState<Pipeline[]>([]);
@@ -901,7 +900,7 @@ const ChatSidebar = ({
       data-tour="chat-sidebar"
       className={`
         ${mobileView === 'list' ? 'flex' : 'hidden'} md:flex
-        w-full md:w-96 border-r bg-card/50 flex-col h-full
+        w-full ${selectedConversationIds.size > 0 ? 'md:w-96' : 'md:w-80'} border-r bg-card/50 flex-col h-full
       `}
     >
       {/* Search and Filter Header */}
@@ -997,7 +996,8 @@ const ChatSidebar = ({
               {selectedConversationIds.size}{' '}
               {selectedConversationIds.size === 1
                 ? t('chatSidebar.conversation')
-                : t('chatSidebar.conversations')}
+                : t('chatSidebar.conversations')}{' '}
+              {t('chatSidebar.selected')}
             </span>
             <Button
               variant="ghost"
@@ -1097,7 +1097,13 @@ const ChatSidebar = ({
                       >
                         <Checkbox
                           checked={selectedConversationIds.has(String(conversation.display_id))}
-                          onCheckedChange={() => onToggleSelect(String(conversation.display_id))}
+                          onCheckedChange={(checked: boolean | 'indeterminate') => {
+                            const isSelected = selectedConversationIds.has(String(conversation.display_id));
+                            if ((checked === true && !isSelected) || (checked === false && isSelected)) {
+                              onToggleSelect(String(conversation.display_id));
+                            }
+                          }}
+                          aria-label={t('chatSidebar.selectConversation')}
                           className="bg-white dark:bg-zinc-700 border-2 border-zinc-400 dark:border-zinc-500 data-[state=checked]:bg-primary data-[state=checked]:border-primary"
                         />
                       </div>

src/i18n/locales/en/chat.json

@@ -290,7 +290,10 @@
       "assignAgent": "Assign Team Member",
       "assignTeam": "Assign Team",
       "assignTag": "Assign Label",
-      "deleteConversation": "Delete conversation"
+      "deleteConversation": "Delete conversation",
+      "bulkResolveSuccess": "{{count}} conversation(s) resolved",
+      "bulkResolvePartialSuccess": "{{success}} resolved, {{failed}} failed",
+      "bulkResolveError": "Error resolving conversations"
     },
     "contactNoName": "Contact without name",
     "status": "Status:",
@@ -303,6 +306,7 @@
     "conversation": "conversation",
     "conversations": "conversations",
     "selected": "selected",
+    "selectConversation": "Select conversation",
     "filter": "filter",
     "filters": "filters",
     "filtersButton": "Filters",

src/i18n/locales/es/chat.json

@@ -285,7 +285,10 @@
       "assignAgent": "Asignar Agente",
       "assignTeam": "Asignar Equipo",
       "assignTag": "Asignar Etiqueta",
-      "deleteConversation": "Eliminar conversación"
+      "deleteConversation": "Eliminar conversación",
+      "bulkResolveSuccess": "{{count}} conversación(es) resuelta(s)",
+      "bulkResolvePartialSuccess": "{{success}} resuelta(s), {{failed}} fallida(s)",
+      "bulkResolveError": "Error al resolver conversaciones"
     },
     "contactNoName": "Contacto sin nombre",
     "status": "Estado:",
@@ -298,6 +301,7 @@
     "conversation": "conversación",
     "conversations": "conversaciones",
     "selected": "seleccionada(s)",
+    "selectConversation": "Seleccionar conversación",
     "filter": "filtro",
     "filters": "filtros",
     "filtersButton": "Filtros",

src/i18n/locales/fr/chat.json

@@ -285,7 +285,10 @@
       "assignAgent": "Assigner un Agent",
       "assignTeam": "Assigner une Équipe",
       "assignTag": "Assigner une Étiquette",
-      "deleteConversation": "Supprimer la conversation"
+      "deleteConversation": "Supprimer la conversation",
+      "bulkResolveSuccess": "{{count}} conversation(s) résolue(s)",
+      "bulkResolvePartialSuccess": "{{success}} résolue(s), {{failed}} échouée(s)",
+      "bulkResolveError": "Erreur lors de la résolution des conversations"
     },
     "contactNoName": "Contact sans nom",
     "status": "Statut :",
@@ -298,6 +301,7 @@
     "conversation": "conversation",
     "conversations": "conversations",
     "selected": "sélectionnée(s)",
+    "selectConversation": "Sélectionner la conversation",
     "filter": "filtre",
     "filters": "filtres",
     "filtersButton": "Filtres",

src/i18n/locales/it/chat.json

@@ -285,7 +285,10 @@
       "assignAgent": "Assegna Agente",
       "assignTeam": "Assegna Team",
       "assignTag": "Assegna Etichetta",
-      "deleteConversation": "Elimina conversazione"
+      "deleteConversation": "Elimina conversazione",
+      "bulkResolveSuccess": "{{count}} conversazione/i risolta/e",
+      "bulkResolvePartialSuccess": "{{success}} risolta/e, {{failed}} fallita/e",
+      "bulkResolveError": "Errore nella risoluzione delle conversazioni"
     },
     "contactNoName": "Contatto senza nome",
     "status": "Stato:",
@@ -298,6 +301,7 @@
     "conversation": "conversazione",
     "conversations": "conversazioni",
     "selected": "selezionata/e",
+    "selectConversation": "Seleziona conversazione",
     "filter": "filtro",
     "filters": "filtri",
     "filtersButton": "Filtri",

src/i18n/locales/pt-BR/chat.json

@@ -290,7 +290,10 @@
       "assignAgent": "Atribuir Atendente",
       "assignTeam": "Atribuir time",
       "assignTag": "Atribuir etiqueta",
-      "deleteConversation": "Deletar conversa"
+      "deleteConversation": "Deletar conversa",
+      "bulkResolveSuccess": "{{count}} conversa(s) resolvida(s)",
+      "bulkResolvePartialSuccess": "{{success}} resolvida(s), {{failed}} falhada(s)",
+      "bulkResolveError": "Erro ao resolver conversas em lote"
     },
     "contactNoName": "Contato sem nome",
     "status": "Status:",
@@ -303,6 +306,7 @@
     "conversation": "conversa",
     "conversations": "conversas",
     "selected": "selecionada(s)",
+    "selectConversation": "Selecionar conversa",
     "filter": "filtro",
     "filters": "filtros",
     "filtersButton": "Filtros",

src/i18n/locales/pt/chat.json

@@ -285,7 +285,10 @@
       "assignAgent": "Atribuir Atendente",
       "assignTeam": "Atribuir time",
       "assignTag": "Atribuir etiqueta",
-      "deleteConversation": "Deletar conversa"
+      "deleteConversation": "Deletar conversa",
+      "bulkResolveSuccess": "{{count}} conversa(s) resolvida(s)",
+      "bulkResolvePartialSuccess": "{{success}} resolvida(s), {{failed}} falhada(s)",
+      "bulkResolveError": "Erro ao resolver conversas em lote"
     },
     "contactNoName": "Contato sem nome",
     "status": "Status:",
@@ -298,6 +301,7 @@
     "conversation": "conversa",
     "conversations": "conversas",
     "selected": "selecionada(s)",
+    "selectConversation": "Selecionar conversa",
     "filter": "filtro",
     "filters": "filtros",
     "filtersButton": "Filtros",

src/pages/Customer/Chat/Chat.tsx

@@ -46,7 +46,7 @@ import type { AssignmentOption, AssignmentType } from '@/components/chat/assignm
 import { labelsService } from '@/services/contacts/labelsService';
 import { useAppDataStore } from '@/store/appDataStore';
 import type { Label } from '@/types/settings';
-import { conversationAPI } from '@/services/conversations/conversationService';
+import chatService from '@/services/chat/chatService';
 
 const ContactSidebar = React.lazy(() => import('@/components/chat/contact-sidebar/ContactSidebar'));
 
@@ -194,12 +194,14 @@ const Chat = () => {
     await filterHandlers.reloadCurrentFilters();
   }, [filterHandlers]);
 
+  const MAX_BULK_SELECTION = 200;
+
   const handleToggleConversationSelection = useCallback((displayId: string) => {
     setSelectedConversationIds(prev => {
       const next = new Set(prev);
       if (next.has(displayId)) {
         next.delete(displayId);
-      } else {
+      } else if (next.size < MAX_BULK_SELECTION) {
         next.add(displayId);
       }
       return next;
@@ -212,24 +214,33 @@ const Chat = () => {
 
   const handleBulkResolve = useCallback(async () => {
     if (selectedConversationIds.size === 0) return;
+    if (!can('conversations', 'update')) {
+      toast.error(t('messages.noPermissionSend'));
+      return;
+    }
     const displayIds = Array.from(selectedConversationIds);
-    const resolvedConversations = conversations.state.conversations
-      .filter(c => selectedConversationIds.has(String(c.display_id)));
     setIsBulkResolving(true);
     try {
-      await conversationAPI.bulkResolve(displayIds);
-      toast.success(`${displayIds.length} conversa(s) resolvida(s)`);
+      const result = await chatService.bulkResolve(displayIds);
       setSelectedConversationIds(new Set());
-      for (const conv of resolvedConversations) {
-        conversations.updateConversation({ ...conv, status: 'resolved' });
+      if (result.failed_ids.length === 0) {
+        toast.success(t('chatHeader.actions.bulkResolveSuccess', { count: result.success_ids.length }));
+      } else if (result.success_ids.length > 0) {
+        toast.warning(t('chatHeader.actions.bulkResolvePartialSuccess', {
+          success: result.success_ids.length,
+          failed: result.failed_ids.length,
+        }));
+      } else {
+        toast.error(t('chatHeader.actions.bulkResolveError'));
       }
+      await reloadCurrentFilters();
     } catch (error) {
       console.error('Bulk resolve error:', error);
-      toast.error('Erro ao resolver conversas em lote');
+      toast.error(t('chatHeader.actions.bulkResolveError'));
     } finally {
       setIsBulkResolving(false);
     }
-  }, [selectedConversationIds, conversations]);
+  }, [selectedConversationIds, can, reloadCurrentFilters, t]);
 
   // 🔄 CARREGAMENTO SIMPLES: Apenas carregar mensagens quando conversa muda
   useEffect(() => {
@@ -699,6 +710,8 @@ const Chat = () => {
       return;
     }
 
+    setSelectedConversationIds(new Set());
+
     // 🔒 MARCAR NAVEGAÇÃO MANUAL para evitar URL sync
     isManualNavigationRef.current = true;
 

src/services/chat/chatService.bulkResolve.spec.ts

@@ -0,0 +1,85 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import chatService from './chatService';
+import api from '@/services/core/api';
+
+vi.mock('@/services/core/api', () => ({
+  default: {
+    post: vi.fn(),
+    get: vi.fn(),
+    put: vi.fn(),
+    patch: vi.fn(),
+    delete: vi.fn(),
+  },
+}));
+
+vi.mock('@/utils/retry/retryHelper', () => ({
+  withRetry: <T>(op: () => Promise<T>) => op(),
+}));
+
+describe('chatService.bulkResolve', () => {
+  const postMock = vi.mocked(api.post);
+
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  it('sends POST /bulk_actions with correct payload', async () => {
+    postMock.mockResolvedValue({
+      data: { data: { success_ids: [1, 2], failed_ids: [] } },
+    } as never);
+
+    await chatService.bulkResolve(['1', '2']);
+
+    expect(postMock).toHaveBeenCalledWith('/bulk_actions', {
+      type: 'Conversation',
+      ids: ['1', '2'],
+      fields: { status: 'resolved' },
+    });
+  });
+
+  it('returns success_ids and failed_ids from response', async () => {
+    postMock.mockResolvedValue({
+      data: { data: { success_ids: [10, 11], failed_ids: [12] } },
+    } as never);
+
+    const result = await chatService.bulkResolve(['10', '11', '12']);
+
+    expect(result.success_ids).toEqual([10, 11]);
+    expect(result.failed_ids).toEqual([12]);
+  });
+
+  it('returns empty arrays when response data is absent', async () => {
+    postMock.mockResolvedValue({ data: {} } as never);
+
+    const result = await chatService.bulkResolve(['1']);
+
+    expect(result.success_ids).toEqual([]);
+    expect(result.failed_ids).toEqual([]);
+  });
+
+  it('caps selection toggle at MAX_BULK_SELECTION (200) items', () => {
+    const MAX = 200;
+    let selection = new Set<string>();
+
+    const toggle = (id: string) => {
+      const next = new Set(selection);
+      if (next.has(id)) {
+        next.delete(id);
+      } else if (next.size < MAX) {
+        next.add(id);
+      }
+      selection = next;
+    };
+
+    for (let i = 0; i < MAX + 10; i++) {
+      toggle(String(i));
+    }
+    expect(selection.size).toBe(MAX);
+
+    toggle('0');
+    expect(selection.size).toBe(MAX - 1);
+
+    toggle('9999');
+    expect(selection.size).toBe(MAX);
+  });
+});

src/services/chat/chatService.ts

@@ -281,6 +281,16 @@ class ChatService {
     );
     return extractData<Message>(response);
   }
+
+  async bulkResolve(displayIds: string[]): Promise<{ success_ids: number[]; failed_ids: number[] }> {
+    const response = await api.post('/bulk_actions', {
+      type: 'Conversation',
+      ids: displayIds,
+      fields: { status: 'resolved' },
+    });
+    const data = response.data?.data;
+    return data ?? { success_ids: [], failed_ids: [] };
+  }
 }
 
 // Export singleton instance