@lobehub/lobehub 2.0.0-next.32 → 2.0.0-next.34

package/src/server/routers/lambda/__tests__/integration/message.integration.test.ts

@@ -72,7 +72,7 @@ describe('Message Router Integration Tests', () => {
     it('should create message with correct sessionId and topicId', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const result = await caller.createNewMessage({
+      const result = await caller.createMessage({
         content: 'Test message',
         role: 'user',
         sessionId: testSessionId,
@@ -111,7 +111,7 @@ describe('Message Router Integration Tests', () => {
         })
         .returning()) as any;
 
-      const result = await caller.createNewMessage({
+      const result = await caller.createMessage({
         content: 'Test message in thread',
         role: 'user',
         sessionId: testSessionId,
@@ -140,7 +140,7 @@ describe('Message Router Integration Tests', () => {
     it('should create message without topicId', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const result = await caller.createNewMessage({
+      const result = await caller.createMessage({
         content: 'Test message without topic',
         role: 'user',
         sessionId: testSessionId,
@@ -160,7 +160,7 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       await expect(
-        caller.createNewMessage({
+        caller.createMessage({
           content: 'Test message',
           role: 'user',
           sessionId: 'non-existent-session',
@@ -192,7 +192,7 @@ describe('Message Router Integration Tests', () => {
 
       // 尝试在 testSessionId 下创建消息，但使用 anotherTopic 的 ID
       await expect(
-        caller.createNewMessage({
+        caller.createMessage({
           content: 'Test message',
           role: 'user',
           sessionId: testSessionId,
@@ -207,13 +207,13 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建多个消息
-      const msg1Result = await caller.createNewMessage({
+      const msg1Result = await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      const msg2Result = await caller.createNewMessage({
+      const msg2Result = await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -228,7 +228,7 @@ describe('Message Router Integration Tests', () => {
         })
         .returning();
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message in another session',
         role: 'user',
         sessionId: anotherSession.id,
@@ -248,7 +248,7 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 在 topic 中创建消息
-      const msgInTopicResult = await caller.createNewMessage({
+      const msgInTopicResult = await caller.createMessage({
         content: 'Message in topic',
         role: 'user',
         sessionId: testSessionId,
@@ -256,7 +256,7 @@ describe('Message Router Integration Tests', () => {
       });
 
       // 在 session 中创建消息（不在 topic 中）
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message without topic',
         role: 'user',
         sessionId: testSessionId,
@@ -278,7 +278,7 @@ describe('Message Router Integration Tests', () => {
 
       // 创建多个消息
       for (let i = 0; i < 5; i++) {
-        await caller.createNewMessage({
+        await caller.createMessage({
           content: `Pagination test message ${i}`,
           role: 'user',
           sessionId: testSessionId,
@@ -331,7 +331,7 @@ describe('Message Router Integration Tests', () => {
         .returning();
 
       // 创建消息并设置 groupId
-      const msg1 = await caller.createNewMessage({
+      const msg1 = await caller.createMessage({
         content: 'Message 1 in group',
         role: 'assistant',
         sessionId: testSessionId,
@@ -343,7 +343,7 @@ describe('Message Router Integration Tests', () => {
         .where(eq(messages.id, msg1.id));
 
       // 创建不在 group 中的消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message without group',
         role: 'user',
         sessionId: testSessionId,
@@ -363,13 +363,13 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建多个消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 1',
         role: 'assistant',
         sessionId: testSessionId,
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -391,13 +391,13 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建消息
-      const msg1Result = await caller.createNewMessage({
+      const msg1Result = await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      const msg2Result = await caller.createNewMessage({
+      const msg2Result = await caller.createMessage({
         content: 'Message 2',
         role: 'user',
         sessionId: testSessionId,
@@ -419,19 +419,19 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建消息
-      const msg1Result = await caller.createNewMessage({
+      const msg1Result = await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      const msg2Result = await caller.createNewMessage({
+      const msg2Result = await caller.createMessage({
         content: 'Message 2',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      const msg3Result = await caller.createNewMessage({
+      const msg3Result = await caller.createMessage({
         content: 'Message 3',
         role: 'user',
         sessionId: testSessionId,
@@ -455,7 +455,7 @@ describe('Message Router Integration Tests', () => {
     it('should remove a single message', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msgResult = await caller.createNewMessage({
+      const msgResult = await caller.createMessage({
         content: 'Message to remove',
         role: 'user',
         sessionId: testSessionId,
@@ -475,13 +475,13 @@ describe('Message Router Integration Tests', () => {
     it('should return message list when sessionId is provided', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg1Result = await caller.createNewMessage({
+      const msg1Result = await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      const msg2Result = await caller.createNewMessage({
+      const msg2Result = await caller.createMessage({
         content: 'Message 2',
         role: 'user',
         sessionId: testSessionId,
@@ -504,7 +504,7 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建多个 session 和消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
@@ -518,7 +518,7 @@ describe('Message Router Integration Tests', () => {
         })
         .returning();
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'user',
         sessionId: anotherSession.id,
@@ -541,7 +541,7 @@ describe('Message Router Integration Tests', () => {
     it('should remove message query', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msgResult = await caller.createNewMessage({
+      const msgResult = await caller.createMessage({
         content: 'Message with query',
         role: 'user',
         sessionId: testSessionId,
@@ -575,13 +575,13 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建多个消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -605,7 +605,7 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 在 topic 中创建消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message in topic',
         role: 'user',
         sessionId: testSessionId,
@@ -613,7 +613,7 @@ describe('Message Router Integration Tests', () => {
       });
 
       // 在 session 中创建消息（不在 topic 中）
-      const msgOutsideTopicResult = await caller.createNewMessage({
+      const msgOutsideTopicResult = await caller.createMessage({
         content: 'Message outside topic',
         role: 'user',
         sessionId: testSessionId,
@@ -651,7 +651,7 @@ describe('Message Router Integration Tests', () => {
         .returning();
 
       // 创建消息并设置 groupId
-      const msg1 = await caller.createNewMessage({
+      const msg1 = await caller.createMessage({
         content: 'Message 1 in group',
         role: 'assistant',
         sessionId: testSessionId,
@@ -677,7 +677,7 @@ describe('Message Router Integration Tests', () => {
     it('should update message content', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const result = await caller.createNewMessage({
+      const result = await caller.createMessage({
         content: 'Original content',
         role: 'user',
         sessionId: testSessionId,
@@ -701,13 +701,13 @@ describe('Message Router Integration Tests', () => {
     it('should update message and return message list when sessionId is provided', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg1 = await caller.createNewMessage({
+      const msg1 = await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      const msg2 = await caller.createNewMessage({
+      const msg2 = await caller.createMessage({
         content: 'Message 2',
         role: 'user',
         sessionId: testSessionId,
@@ -733,13 +733,13 @@ describe('Message Router Integration Tests', () => {
     it('should search messages by keyword', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'This is a test message about TypeScript',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Another message about JavaScript',
         role: 'user',
         sessionId: testSessionId,
@@ -758,7 +758,7 @@ describe('Message Router Integration Tests', () => {
     it('should update message plugin state', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with plugin',
         role: 'assistant',
         sessionId: testSessionId,
@@ -794,7 +794,7 @@ describe('Message Router Integration Tests', () => {
     it('should update message RAG information', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with RAG',
         role: 'assistant',
         sessionId: testSessionId,
@@ -844,13 +844,13 @@ describe('Message Router Integration Tests', () => {
     it('should return message list when sessionId is provided', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg1 = await caller.createNewMessage({
+      const msg1 = await caller.createMessage({
         content: 'Message 1',
         role: 'assistant',
         sessionId: testSessionId,
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -895,7 +895,7 @@ describe('Message Router Integration Tests', () => {
     it('should update message metadata', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with metadata',
         role: 'user',
         sessionId: testSessionId,
@@ -921,7 +921,7 @@ describe('Message Router Integration Tests', () => {
     it('should update plugin error state', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with plugin error',
         role: 'assistant',
         sessionId: testSessionId,
@@ -953,7 +953,7 @@ describe('Message Router Integration Tests', () => {
     it('should return message list when sessionId is provided', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg1 = await caller.createNewMessage({
+      const msg1 = await caller.createMessage({
         content: 'Message 1',
         role: 'assistant',
         sessionId: testSessionId,
@@ -968,7 +968,7 @@ describe('Message Router Integration Tests', () => {
         type: 'default',
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -990,7 +990,7 @@ describe('Message Router Integration Tests', () => {
     it('should update plugin state', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with plugin state',
         role: 'assistant',
         sessionId: testSessionId,
@@ -1019,7 +1019,7 @@ describe('Message Router Integration Tests', () => {
     it('should update TTS information', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with TTS',
         role: 'assistant',
         sessionId: testSessionId,
@@ -1058,7 +1058,7 @@ describe('Message Router Integration Tests', () => {
     it('should delete TTS when value is false', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message with TTS to delete',
         role: 'assistant',
         sessionId: testSessionId,
@@ -1103,7 +1103,7 @@ describe('Message Router Integration Tests', () => {
     it('should update translation information', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Hello world',
         role: 'user',
         sessionId: testSessionId,
@@ -1131,7 +1131,7 @@ describe('Message Router Integration Tests', () => {
     it('should delete translation when value is false', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Hello world',
         role: 'user',
         sessionId: testSessionId,
@@ -1166,13 +1166,13 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建一些消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -1190,7 +1190,7 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建带有模型信息的消息
-      const msg = await caller.createNewMessage({
+      const msg = await caller.createMessage({
         content: 'Message from AI',
         role: 'assistant',
         sessionId: testSessionId,
@@ -1211,13 +1211,13 @@ describe('Message Router Integration Tests', () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
       // 创建消息
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
       });
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 2',
         role: 'assistant',
         sessionId: testSessionId,
@@ -1231,7 +1231,7 @@ describe('Message Router Integration Tests', () => {
     it('should count messages with date range', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Message 1',
         role: 'user',
         sessionId: testSessionId,
@@ -1251,7 +1251,7 @@ describe('Message Router Integration Tests', () => {
     it('should count words', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Hello world',
         role: 'user',
         sessionId: testSessionId,
@@ -1265,7 +1265,7 @@ describe('Message Router Integration Tests', () => {
     it('should count words with date range', async () => {
       const caller = messageRouter.createCaller(createTestContext(userId));
 
-      await caller.createNewMessage({
+      await caller.createMessage({
         content: 'Hello world test message',
         role: 'user',
         sessionId: testSessionId,

package/src/server/routers/lambda/message.ts

@@ -54,11 +54,11 @@ export const messageRouter = router({
       return ctx.messageModel.countWords(input);
     }),
 
-  createNewMessage: messageProcedure
+  createMessage: messageProcedure
     .input(CreateNewMessageParamsSchema.extend({ useGroup: z.boolean().optional() }))
     .mutation(async ({ input, ctx }) => {
       const { useGroup, ...params } = input;
-      return ctx.messageService.createNewMessage(params as any, { useGroup });
+      return ctx.messageService.createMessage(params as any, { useGroup });
     }),
 
   getHeatmaps: messageProcedure.query(async ({ ctx }) => {

package/src/server/services/mcp/deps/checkers/ManualInstallationChecker.test.ts

@@ -0,0 +1,162 @@
+import { describe, expect, it } from 'vitest';
+
+import { ManualInstallationChecker } from './ManualInstallationChecker';
+
+describe('ManualInstallationChecker', () => {
+  let checker: ManualInstallationChecker;
+
+  beforeEach(() => {
+    checker = new ManualInstallationChecker();
+  });
+
+  describe('checkPackageInstalled', () => {
+    it('should always return not installed for manual packages', async () => {
+      const result = await checker.checkPackageInstalled({ packageName: 'manual-package' });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: 'manual-package',
+      });
+    });
+
+    it('should handle packageName with repository URL', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: 'my-custom-tool',
+        repositoryUrlToClone: 'https://github.com/user/repo.git',
+      });
+
+      expect(result.installed).toBe(false);
+      expect(result.packageName).toBe('my-custom-tool');
+    });
+
+    it('should use repositoryUrlToClone as packageName when packageName is not provided', async () => {
+      const result = await checker.checkPackageInstalled({
+        repositoryUrlToClone: 'https://github.com/user/custom-tool.git',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: 'https://github.com/user/custom-tool.git',
+      });
+    });
+
+    it('should return empty packageName when neither packageName nor repositoryUrlToClone provided', async () => {
+      const result = await checker.checkPackageInstalled({});
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: '',
+      });
+    });
+
+    it('should handle undefined packageName', async () => {
+      const result = await checker.checkPackageInstalled({ packageName: undefined });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: '',
+      });
+    });
+
+    it('should handle empty string packageName', async () => {
+      const result = await checker.checkPackageInstalled({ packageName: '' });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: '',
+      });
+    });
+
+    it('should handle empty string repositoryUrlToClone as fallback', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: '',
+        repositoryUrlToClone: '',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: '',
+      });
+    });
+
+    it('should prioritize packageName over repositoryUrlToClone', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: 'my-tool',
+        repositoryUrlToClone: 'https://github.com/user/repo.git',
+      });
+
+      expect(result.packageName).toBe('my-tool');
+      expect(result.installed).toBe(false);
+    });
+
+    it('should handle complex package names', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: '@scope/package-name-with-special.chars_123',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: '@scope/package-name-with-special.chars_123',
+      });
+    });
+
+    it('should handle SSH repository URLs', async () => {
+      const result = await checker.checkPackageInstalled({
+        repositoryUrlToClone: 'git@github.com:user/repo.git',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: 'git@github.com:user/repo.git',
+      });
+    });
+
+    it('should handle local file paths', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: '/usr/local/custom-tool',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: '/usr/local/custom-tool',
+      });
+    });
+
+    it('should handle Windows-style paths', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: 'C:\\Program Files\\CustomTool',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: 'C:\\Program Files\\CustomTool',
+      });
+    });
+
+    it('should handle package names with unicode characters', async () => {
+      const result = await checker.checkPackageInstalled({
+        packageName: 'my-tool-日本語',
+      });
+
+      expect(result).toEqual({
+        installed: false,
+        packageName: 'my-tool-日本語',
+      });
+    });
+
+    it('should always return false for installed regardless of input', async () => {
+      const testCases = [
+        { packageName: 'package1' },
+        { packageName: 'package2', repositoryUrlToClone: 'https://example.com/repo.git' },
+        { repositoryUrlToClone: 'https://example.com/repo2.git' },
+        {},
+        { packageName: '' },
+      ];
+
+      for (const testCase of testCases) {
+        const result = await checker.checkPackageInstalled(testCase);
+        expect(result.installed).toBe(false);
+      }
+    });
+  });
+});