@lobehub/chat 0.159.10 → 0.159.12

package/src/store/chat/slices/message/action.test.ts

@@ -1,14 +1,18 @@
+import * as lobeUIModules from '@lobehub/ui';
 import { act, renderHook, waitFor } from '@testing-library/react';
 import useSWR, { mutate } from 'swr';
 import { Mock, afterEach, beforeEach, describe, expect, it, vi } from 'vitest';
 
 import { LOADING_FLAT } from '@/const/message';
 import { DEFAULT_AGENT_CONFIG } from '@/const/settings';
+import { TraceEventType } from '@/const/trace';
 import { chatService } from '@/services/chat';
 import { messageService } from '@/services/message';
 import { topicService } from '@/services/topic';
+import { useAgentStore } from '@/store/agent';
 import { agentSelectors } from '@/store/agent/selectors';
 import { chatSelectors } from '@/store/chat/selectors';
+import { messageMapKey } from '@/store/chat/slices/message/utils';
 import { sessionMetaSelectors } from '@/store/session/selectors';
 import { ChatMessage } from '@/types/message';
 
@@ -34,6 +38,7 @@ vi.mock('@/services/message', () => ({
 }));
 vi.mock('@/services/topic', () => ({
   topicService: {
+    createTopic: vi.fn(() => Promise.resolve()),
     removeTopic: vi.fn(() => Promise.resolve()),
   },
 }));
@@ -75,6 +80,48 @@ afterEach(() => {
 });
 
 describe('chatMessage actions', () => {
+  describe('addAIMessage', () => {
+    it('should return early if activeId is undefined', async () => {
+      useChatStore.setState({ activeId: undefined });
+      const { result } = renderHook(() => useChatStore());
+      const updateInputMessageSpy = vi.spyOn(result.current, 'updateInputMessage');
+
+      await act(async () => {
+        await result.current.addAIMessage();
+      });
+
+      expect(messageService.createMessage).not.toHaveBeenCalled();
+      expect(updateInputMessageSpy).not.toHaveBeenCalled();
+    });
+
+    it('should call internal_createMessage with correct parameters', async () => {
+      const inputMessage = 'Test input message';
+      useChatStore.setState({ inputMessage });
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.addAIMessage();
+      });
+
+      expect(messageService.createMessage).toHaveBeenCalledWith({
+        content: inputMessage,
+        role: 'assistant',
+        sessionId: mockState.activeId,
+        topicId: mockState.activeTopicId,
+      });
+    });
+
+    it('should call updateInputMessage with empty string', async () => {
+      const { result } = renderHook(() => useChatStore());
+      const updateInputMessageSpy = vi.spyOn(result.current, 'updateInputMessage');
+      await act(async () => {
+        await result.current.addAIMessage();
+      });
+
+      expect(updateInputMessageSpy).toHaveBeenCalledWith('');
+    });
+  });
+
   describe('deleteMessage', () => {
     it('deleteMessage should remove a message by id', async () => {
       const { result } = renderHook(() => useChatStore());
@@ -82,7 +129,13 @@ describe('chatMessage actions', () => {
       const deleteSpy = vi.spyOn(result.current, 'deleteMessage');
 
       act(() => {
-        useChatStore.setState({ messages: [{ id: messageId } as ChatMessage] });
+        useChatStore.setState({
+          activeId: 'session-id',
+          activeTopicId: undefined,
+          messagesMap: {
+            [messageMapKey('session-id')]: [{ id: messageId } as ChatMessage],
+          },
+        });
       });
       await act(async () => {
         await result.current.deleteMessage(messageId);
@@ -119,6 +172,36 @@ describe('chatMessage actions', () => {
     });
   });
 
+  describe('copyMessage', () => {
+    it('should call copyToClipboard with correct content', async () => {
+      const messageId = 'message-id';
+      const content = 'Test content';
+      const { result } = renderHook(() => useChatStore());
+      const copyToClipboardSpy = vi.spyOn(lobeUIModules, 'copyToClipboard');
+
+      await act(async () => {
+        await result.current.copyMessage(messageId, content);
+      });
+
+      expect(copyToClipboardSpy).toHaveBeenCalledWith(content);
+    });
+
+    it('should call internal_traceMessage with correct parameters', async () => {
+      const messageId = 'message-id';
+      const content = 'Test content';
+      const { result } = renderHook(() => useChatStore());
+      const internal_traceMessageSpy = vi.spyOn(result.current, 'internal_traceMessage');
+
+      await act(async () => {
+        await result.current.copyMessage(messageId, content);
+      });
+
+      expect(internal_traceMessageSpy).toHaveBeenCalledWith(messageId, {
+        eventType: TraceEventType.CopyMessage,
+      });
+    });
+  });
+
   describe('clearMessage', () => {
     beforeEach(() => {
       vi.clearAllMocks(); // 清除 mocks
@@ -157,7 +240,7 @@ describe('chatMessage actions', () => {
       expect(switchTopicSpy).toHaveBeenCalled();
 
       // 检查 activeTopicId 是否被清除，需要在状态更新后进行检查
-      expect(useChatStore.getState().activeTopicId).toBeUndefined();
+      expect(useChatStore.getState().activeTopicId).toBeNull();
     });
 
     it('should call removeTopic if there is an activeTopicId', async () => {
@@ -237,7 +320,6 @@ describe('chatMessage actions', () => {
         sessionId: mockState.activeId,
         topicId: mockState.activeTopicId,
       });
-      expect(result.current.refreshMessages).toHaveBeenCalled();
       expect(result.current.internal_coreProcessMessage).toHaveBeenCalled();
     });
 
@@ -265,9 +347,14 @@ describe('chatMessage actions', () => {
           useChatStore.setState({
             ...mockState,
             // Mock the currentChats selector to return a list that does not reach the threshold
-            messages: Array.from({ length: autoCreateTopicThreshold + 1 }, (_, i) => ({
-              id: `msg-${i}`,
-            })) as any,
+            messagesMap: {
+              [messageMapKey('session-id')]: Array.from(
+                { length: autoCreateTopicThreshold + 1 },
+                (_, i) => ({
+                  id: `msg-${i}`,
+                }),
+              ) as any,
+            },
             activeTopicId: undefined,
             saveToTopic: saveToTopicMock,
             switchTopic: switchTopicMock,
@@ -296,17 +383,23 @@ describe('chatMessage actions', () => {
         (messageService.createMessage as Mock).mockResolvedValue('new-message-id');
 
         // Mock saveToTopic to resolve with a topic id and switchTopic to switch to the new topic
-        const saveToTopicMock = vi.fn(() => Promise.resolve('new-topic-id'));
+        const createTopicMock = vi.fn(() => Promise.resolve('new-topic-id'));
         const switchTopicMock = vi.fn();
 
         act(() => {
           useChatStore.setState({
             ...mockState,
-            messages: Array.from({ length: autoCreateTopicThreshold }, (_, i) => ({
-              id: `msg-${i}`,
-            })) as any,
+            activeId: 'session_id',
+            messagesMap: {
+              [messageMapKey('session_id')]: Array.from(
+                { length: autoCreateTopicThreshold },
+                (_, i) => ({
+                  id: `msg-${i}`,
+                }),
+              ) as any,
+            },
             activeTopicId: undefined,
-            saveToTopic: saveToTopicMock,
+            createTopic: createTopicMock,
             switchTopic: switchTopicMock,
           });
         });
@@ -315,8 +408,33 @@ describe('chatMessage actions', () => {
           await result.current.sendMessage({ message });
         });
 
-        expect(saveToTopicMock).toHaveBeenCalled();
-        expect(switchTopicMock).toHaveBeenCalledWith('new-topic-id');
+        expect(createTopicMock).toHaveBeenCalled();
+        expect(switchTopicMock).toHaveBeenCalledWith('new-topic-id', true);
+      });
+
+      it('should not auto-create topic, if autoCreateTopic = false and reached topic threshold', async () => {
+        const { result } = renderHook(() => useChatStore());
+        act(() => {
+          useAgentStore.setState({
+            agentConfig: {
+              enableAutoCreateTopic: false,
+              autoCreateTopicThreshold: 1,
+            },
+          });
+          useChatStore.setState({
+            // Mock the currentChats selector to return a list that does not reach the threshold
+            messagesMap: {
+              [messageMapKey('inbox')]: [{ id: '1' }, { id: '2' }] as ChatMessage[],
+            },
+            activeTopicId: 'inbox',
+          });
+        });
+
+        await act(async () => {
+          await result.current.sendMessage({ message: 'test' });
+        });
+
+        expect(topicService.createTopic).not.toHaveBeenCalled();
       });
 
       it('should not auto-create topic if autoCreateTopicThreshold is not reached', async () => {
@@ -335,28 +453,185 @@ describe('chatMessage actions', () => {
         }));
 
         // Mock saveToTopic and switchTopic to simulate not being called
-        const saveToTopicMock = vi.fn();
+        const createTopicMock = vi.fn();
         const switchTopicMock = vi.fn();
 
         await act(async () => {
           useChatStore.setState({
             ...mockState,
-            // Mock the currentChats selector to return a list that does not reach the threshold
-            messages: Array.from({ length: autoCreateTopicThreshold - 2 }, (_, i) => ({
-              id: `msg-${i}`,
-            })) as any,
+            activeId: 'session_id',
+            messagesMap: {
+              // Mock the currentChats selector to return a list that does not reach the threshold
+              [messageMapKey('session_id')]: Array.from(
+                { length: autoCreateTopicThreshold - 3 },
+                (_, i) => ({
+                  id: `msg-${i}`,
+                }),
+              ) as any,
+            },
             activeTopicId: undefined,
-            saveToTopic: saveToTopicMock,
+            createTopic: createTopicMock,
             switchTopic: switchTopicMock,
           });
 
           await result.current.sendMessage({ message });
         });
 
-        expect(saveToTopicMock).not.toHaveBeenCalled();
+        expect(createTopicMock).not.toHaveBeenCalled();
         expect(switchTopicMock).not.toHaveBeenCalled();
       });
     });
+
+    it('should add user message and not call internal_coreProcessMessage if onlyAddUserMessage = true', async () => {
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.sendMessage({ message: 'test', onlyAddUserMessage: true });
+      });
+
+      expect(messageService.createMessage).toHaveBeenCalled();
+      expect(result.current.internal_coreProcessMessage).not.toHaveBeenCalled();
+    });
+
+    it('当 isWelcomeQuestion 为 true 时,正确地传递给 internal_coreProcessMessage', async () => {
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.sendMessage({ message: 'test', isWelcomeQuestion: true });
+      });
+
+      expect(result.current.internal_coreProcessMessage).toHaveBeenCalledWith(
+        expect.anything(),
+        expect.anything(),
+        { isWelcomeQuestion: true },
+      );
+    });
+
+    it('当只有文件而没有消息内容时,正确发送消息', async () => {
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.sendMessage({ message: '', files: [{ id: 'file-1' }] as any });
+      });
+
+      expect(messageService.createMessage).toHaveBeenCalledWith({
+        content: '',
+        files: ['file-1'],
+        role: 'user',
+        sessionId: 'session-id',
+        topicId: 'topic-id',
+      });
+    });
+
+    it('当同时有文件和消息内容时,正确发送消息并关联文件', async () => {
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.sendMessage({ message: 'test', files: [{ id: 'file-1' }] as any });
+      });
+
+      expect(messageService.createMessage).toHaveBeenCalledWith({
+        content: 'test',
+        files: ['file-1'],
+        role: 'user',
+        sessionId: 'session-id',
+        topicId: 'topic-id',
+      });
+    });
+
+    it('当 createMessage 抛出错误时,正确处理错误而不影响整个应用', async () => {
+      const { result } = renderHook(() => useChatStore());
+      vi.spyOn(messageService, 'createMessage').mockRejectedValue(
+        new Error('create message error'),
+      );
+
+      await expect(result.current.sendMessage({ message: 'test' })).rejects.toThrow(
+        'create message error',
+      );
+
+      expect(result.current.internal_coreProcessMessage).not.toHaveBeenCalled();
+    });
+
+    // it('自动创建主题成功后,正确地将消息复制到新主题,并删除之前的临时消息', async () => {
+    //   const { result } = renderHook(() => useChatStore());
+    //   act(() => {
+    //     useAgentStore.setState({
+    //       agentConfig: { enableAutoCreateTopic: true, autoCreateTopicThreshold: 1 },
+    //     });
+    //
+    //     useChatStore.setState({
+    //       // Mock the currentChats selector to return a list that does not reach the threshold
+    //       messagesMap: {
+    //         [messageMapKey('inbox')]: [{ id: '1' }, { id: '2' }] as ChatMessage[],
+    //       },
+    //       activeId: 'inbox',
+    //     });
+    //   });
+    //   vi.spyOn(topicService, 'createTopic').mockResolvedValue('new-topic');
+    //
+    //   await act(async () => {
+    //     await result.current.sendMessage({ message: 'test' });
+    //   });
+    //
+    //   expect(result.current.messagesMap[messageMapKey('inbox')]).toEqual([
+    //     // { id: '1' },
+    //     // { id: '2' },
+    //     // { id: 'temp-id', content: 'test', role: 'user' },
+    //   ]);
+    //   // expect(result.current.getMessages('session-id')).toEqual([]);
+    // });
+
+    // it('自动创建主题失败时,正确地处理错误,不会影响后续的消息发送', async () => {
+    //   const { result } = renderHook(() => useChatStore());
+    //   result.current.setAgentConfig({ enableAutoCreateTopic: true, autoCreateTopicThreshold: 1 });
+    //   result.current.setMessages([{ id: '1' }, { id: '2' }] as any);
+    //   vi.spyOn(topicService, 'createTopic').mockRejectedValue(new Error('create topic error'));
+    //
+    //   await act(async () => {
+    //     await result.current.sendMessage({ message: 'test' });
+    //   });
+    //
+    //   expect(result.current.getMessages('session-id')).toEqual([
+    //     { id: '1' },
+    //     { id: '2' },
+    //     { id: 'new-message-id', content: 'test', role: 'user' },
+    //   ]);
+    // });
+
+    // it('当 activeTopicId 不存在且 autoCreateTopic 为 true,但消息数量未达到阈值时,正确地总结主题标题', async () => {
+    //   const { result } = renderHook(() => useChatStore());
+    //   result.current.setAgentConfig({ enableAutoCreateTopic: true, autoCreateTopicThreshold: 10 });
+    //   result.current.setMessages([{ id: '1' }, { id: '2' }] as any);
+    //   result.current.setActiveTopic({ id: 'topic-1', title: '' });
+    //
+    //   await act(async () => {
+    //     await result.current.sendMessage({ message: 'test' });
+    //   });
+    //
+    //   expect(result.current.summaryTopicTitle).toHaveBeenCalledWith('topic-1', [
+    //     { id: '1' },
+    //     { id: '2' },
+    //     { id: 'new-message-id', content: 'test', role: 'user' },
+    //     { id: 'assistant-message', role: 'assistant' },
+    //   ]);
+    // });
+    //
+    // it('当 activeTopicId 存在且主题标题为空时,正确地总结主题标题', async () => {
+    //   const { result } = renderHook(() => useChatStore());
+    //   result.current.setActiveTopic({ id: 'topic-1', title: '' });
+    //   result.current.setMessages([{ id: '1' }, { id: '2' }] as any, 'session-id', 'topic-1');
+    //
+    //   await act(async () => {
+    //     await result.current.sendMessage({ message: 'test' });
+    //   });
+    //
+    //   expect(result.current.summaryTopicTitle).toHaveBeenCalledWith('topic-1', [
+    //     { id: '1' },
+    //     { id: '2' },
+    //     { id: 'new-message-id', content: 'test', role: 'user' },
+    //     { id: 'assistant-message', role: 'assistant' },
+    //   ]);
+    // });
   });
 
   describe('toggleMessageEditing action', () => {
@@ -391,10 +666,14 @@ describe('chatMessage actions', () => {
 
       act(() => {
         useChatStore.setState({
+          activeId: 'session-id',
+          activeTopicId: undefined,
           // Mock the currentChats selector to return a list that includes the message to be resent
-          messages: [
-            { id: messageId, role: 'user', content: 'Resend this message' } as ChatMessage,
-          ],
+          messagesMap: {
+            [messageMapKey('session-id')]: [
+              { id: messageId, role: 'user', content: 'Resend this message' } as ChatMessage,
+            ],
+          },
         });
       });
 
@@ -419,8 +698,12 @@ describe('chatMessage actions', () => {
 
       act(() => {
         useChatStore.setState({
+          activeId: 'session-id',
+          activeTopicId: undefined,
           // Mock the currentChats selector to return a list that does not include the message to be resent
-          messages: [],
+          messagesMap: {
+            [messageMapKey('session-id')]: [],
+          },
         });
       });
 
@@ -559,6 +842,31 @@ describe('chatMessage actions', () => {
     });
   });
 
+  describe('toggleMessageEditing', () => {
+    it('should update messageEditingIds correctly when enabling editing', () => {
+      const messageId = 'message-id';
+      const { result } = renderHook(() => useChatStore());
+
+      act(() => {
+        result.current.toggleMessageEditing(messageId, true);
+      });
+
+      expect(result.current.messageEditingIds).toContain(messageId);
+    });
+
+    it('should update messageEditingIds correctly when disabling editing', () => {
+      const messageId = 'message-id';
+      useChatStore.setState({ messageEditingIds: [messageId] });
+      const { result } = renderHook(() => useChatStore());
+
+      act(() => {
+        result.current.toggleMessageEditing(messageId, false);
+      });
+
+      expect(result.current.messageEditingIds).not.toContain(messageId);
+    });
+  });
+
   describe('refreshMessages action', () => {
     beforeEach(() => {
       vi.mock('swr', async () => {
@@ -663,6 +971,72 @@ describe('chatMessage actions', () => {
         });
       });
     });
+
+    it('should generate correct contextMessages for "user" role', async () => {
+      const messageId = 'message-id';
+      const messages = [
+        { id: 'msg-1', role: 'system' },
+        { id: messageId, role: 'user', meta: { avatar: '😀' } },
+        { id: 'msg-3', role: 'assistant' },
+      ];
+      act(() => {
+        useChatStore.setState({
+          messagesMap: {
+            [chatSelectors.currentChatKey(mockState as any)]: messages as ChatMessage[],
+          },
+        });
+      });
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.internal_resendMessage(messageId);
+      });
+
+      expect(result.current.internal_coreProcessMessage).toHaveBeenCalledWith(
+        messages.slice(0, 2),
+        messageId,
+        { traceId: undefined },
+      );
+    });
+
+    it('should generate correct contextMessages for "assistant" role', async () => {
+      const messageId = 'message-id';
+      const messages = [
+        { id: 'msg-1', role: 'system' },
+        { id: 'msg-2', role: 'user', meta: { avatar: '😀' } },
+        { id: messageId, role: 'assistant', parentId: 'msg-2' },
+      ];
+      useChatStore.setState({
+        messagesMap: {
+          [chatSelectors.currentChatKey(mockState as any)]: messages as ChatMessage[],
+        },
+      });
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.internal_resendMessage(messageId);
+      });
+
+      expect(result.current.internal_coreProcessMessage).toHaveBeenCalledWith(
+        messages.slice(0, 2),
+        'msg-2',
+        { traceId: undefined },
+      );
+    });
+
+    it('should return early if contextMessages is empty', async () => {
+      const messageId = 'message-id';
+      useChatStore.setState({
+        messagesMap: { [chatSelectors.currentChatKey(mockState as any)]: [] },
+      });
+      const { result } = renderHook(() => useChatStore());
+
+      await act(async () => {
+        await result.current.internal_resendMessage(messageId);
+      });
+
+      expect(result.current.internal_coreProcessMessage).not.toHaveBeenCalled();
+    });
   });
 
   describe('internal_toggleChatLoading', () => {
@@ -760,4 +1134,101 @@ describe('chatMessage actions', () => {
       expect(result.current.messageLoadingIds).not.toContain(messageId);
     });
   });
+
+  describe('stopGenerateMessage', () => {
+    it('should return early if abortController is undefined', () => {
+      act(() => {
+        useChatStore.setState({ abortController: undefined });
+      });
+
+      const { result } = renderHook(() => useChatStore());
+
+      const spy = vi.spyOn(result.current, 'internal_toggleChatLoading');
+
+      act(() => {
+        result.current.stopGenerateMessage();
+      });
+
+      expect(spy).not.toHaveBeenCalled();
+    });
+
+    it('should call abortController.abort()', () => {
+      const abortMock = vi.fn();
+      const abortController = { abort: abortMock } as unknown as AbortController;
+      act(() => {
+        useChatStore.setState({ abortController });
+      });
+      const { result } = renderHook(() => useChatStore());
+
+      act(() => {
+        result.current.stopGenerateMessage();
+      });
+
+      expect(abortMock).toHaveBeenCalled();
+    });
+
+    it('should call internal_toggleChatLoading with correct parameters', () => {
+      const abortController = new AbortController();
+      act(() => {
+        useChatStore.setState({ abortController });
+      });
+      const { result } = renderHook(() => useChatStore());
+      const spy = vi.spyOn(result.current, 'internal_toggleChatLoading');
+
+      act(() => {
+        result.current.stopGenerateMessage();
+      });
+
+      expect(spy).toHaveBeenCalledWith(false, undefined, expect.any(String));
+    });
+  });
+
+  describe('updateInputMessage', () => {
+    it('should not update state if message is the same as current inputMessage', () => {
+      const inputMessage = 'Test input message';
+      useChatStore.setState({ inputMessage });
+      const { result } = renderHook(() => useChatStore());
+
+      act(() => {
+        result.current.updateInputMessage(inputMessage);
+      });
+
+      expect(result.current.inputMessage).toBe(inputMessage);
+    });
+  });
+
+  describe('modifyMessageContent', () => {
+    it('should call internal_traceMessage with correct parameters before updating', async () => {
+      const messageId = 'message-id';
+      const content = 'Updated content';
+      const { result } = renderHook(() => useChatStore());
+
+      const spy = vi.spyOn(result.current, 'internal_traceMessage');
+      await act(async () => {
+        await result.current.modifyMessageContent(messageId, content);
+      });
+
+      expect(spy).toHaveBeenCalledWith(messageId, {
+        eventType: TraceEventType.ModifyMessage,
+        nextContent: content,
+      });
+    });
+
+    it('should call internal_updateMessageContent with correct parameters', async () => {
+      const messageId = 'message-id';
+      const content = 'Updated content';
+      const { result } = renderHook(() => useChatStore());
+
+      const spy = vi.spyOn(result.current, 'internal_traceMessage');
+
+      await act(async () => {
+        await result.current.modifyMessageContent(messageId, content);
+      });
+
+      expect(spy).toHaveBeenCalledWith(messageId, {
+        eventType: 'Modify Message',
+        nextContent: 'Updated content',
+      });
+    });
+  });
 });