@loop_ouroboros/mcp-hub-lite 1.2.4 → 1.2.6

package/dist/server/tests/unit/services/hub-tools.service.test.js

@@ -36,6 +36,7 @@ describe('HubToolsService', () => {
                         instances: [
                             {
                                 id: 'test-server-1-instance',
+                                index: 0,
                                 enabled: true,
                                 args: [],
                                 env: {},
@@ -62,6 +63,7 @@ describe('HubToolsService', () => {
                         instances: [
                             {
                                 id: 'test-server-2-instance',
+                                index: 0,
                                 enabled: true,
                                 args: [],
                                 env: {},
@@ -82,20 +84,18 @@ describe('HubToolsService', () => {
                 const server = mockServers.find((s) => s.name === name);
                 return server?.config;
             });
-            vi.mocked(mcpConnectionManager.getStatusByName).mockImplementation(() => {
-                return {
-                    connected: true,
-                    lastCheck: Date.now(),
-                    toolsCount: 0,
-                    resourcesCount: 0
-                };
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockImplementation(() => {
+                return [0];
+            });
+            vi.mocked(mcpConnectionManager.getStatus).mockImplementation(() => {
+                return { connected: true, lastCheck: Date.now(), toolsCount: 0, resourcesCount: 0 };
             });
             // Act
             const servers = await hubToolsService.listServers();
             // Assert
             expect(servers).toEqual({
                 'Test Server 1': 'File system operations',
-                'Test Server 2': 'Connected MCP server: Test Server 2'
+                'Test Server 2': 'Test Server 2 (You can check the tool list to understand its capabilities and update the description.)'
             });
             expect(hubManager.getAllServers).toHaveBeenCalledTimes(1);
         });
@@ -118,6 +118,7 @@ describe('HubToolsService', () => {
                         instances: [
                             {
                                 id: 'server1-instance',
+                                index: 0,
                                 enabled: true,
                                 args: [],
                                 env: {},
@@ -138,19 +139,17 @@ describe('HubToolsService', () => {
                 const server = mockServers.find((s) => s.name === name);
                 return server?.config;
             });
-            vi.mocked(mcpConnectionManager.getStatusByName).mockImplementation(() => {
-                return {
-                    connected: true,
-                    lastCheck: Date.now(),
-                    toolsCount: 0,
-                    resourcesCount: 0
-                };
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockImplementation(() => {
+                return [0];
+            });
+            vi.mocked(mcpConnectionManager.getStatus).mockImplementation(() => {
+                return { connected: true, lastCheck: Date.now(), toolsCount: 0, resourcesCount: 0 };
             });
             // Act
             const servers = await hubToolsService.listServers();
             // Assert
             expect(servers).toEqual({
-                server1: 'Connected MCP server: server1'
+                server1: 'server1 (You can check the tool list to understand its capabilities and update the description.)'
             });
         });
         it('should use provided description when available', async () => {
@@ -173,6 +172,7 @@ describe('HubToolsService', () => {
                         instances: [
                             {
                                 id: 'filesystem-instance',
+                                index: 0,
                                 enabled: true,
                                 args: [],
                                 env: {},
@@ -200,6 +200,7 @@ describe('HubToolsService', () => {
                         instances: [
                             {
                                 id: 'time-instance',
+                                index: 0,
                                 enabled: true,
                                 args: [],
                                 env: {},
@@ -220,13 +221,11 @@ describe('HubToolsService', () => {
                 const server = mockServers.find((s) => s.name === name);
                 return server?.config;
             });
-            vi.mocked(mcpConnectionManager.getStatusByName).mockImplementation(() => {
-                return {
-                    connected: true,
-                    lastCheck: Date.now(),
-                    toolsCount: 0,
-                    resourcesCount: 0
-                };
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockImplementation(() => {
+                return [0];
+            });
+            vi.mocked(mcpConnectionManager.getStatus).mockImplementation(() => {
+                return { connected: true, lastCheck: Date.now(), toolsCount: 0, resourcesCount: 0 };
             });
             // Act
             const servers = await hubToolsService.listServers();
@@ -256,6 +255,7 @@ describe('HubToolsService', () => {
                         instances: [
                             {
                                 id: 'connected-server-instance',
+                                index: 0,
                                 enabled: true,
                                 args: [],
                                 env: {},
@@ -303,21 +303,14 @@ describe('HubToolsService', () => {
                 const server = mockServers.find((s) => s.name === name);
                 return server?.config;
             });
-            vi.mocked(mcpConnectionManager.getStatusByName).mockImplementation((name) => {
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockImplementation((name) => {
                 if (name === 'Connected Server') {
-                    return {
-                        connected: true,
-                        lastCheck: Date.now(),
-                        toolsCount: 0,
-                        resourcesCount: 0
-                    };
+                    return [0];
                 }
-                return {
-                    connected: false,
-                    lastCheck: Date.now(),
-                    toolsCount: 0,
-                    resourcesCount: 0
-                };
+                return [];
+            });
+            vi.mocked(mcpConnectionManager.getStatus).mockImplementation(() => {
+                return { connected: true, lastCheck: Date.now(), toolsCount: 0, resourcesCount: 0 };
             });
             // Act
             const servers = await hubToolsService.listServers();
@@ -380,6 +373,7 @@ describe('HubToolsService', () => {
                 tagDefinitions: []
             });
             vi.mocked(mcpConnectionManager.getToolsByServerName).mockReturnValue(mockTools);
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockReturnValue([0]);
             // Act
             const result = await hubToolsService.listToolsInServer({ serverName });
             // Assert
@@ -392,7 +386,7 @@ describe('HubToolsService', () => {
         it('should throw error if server not found', async () => {
             // Arrange
             const serverName = 'Non-existent Server';
-            vi.mocked(mcpConnectionManager.getToolsByServerName).mockReturnValue([]);
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockReturnValue([]);
             // Act & Assert
             await expect(hubToolsService.listToolsInServer({ serverName })).rejects.toThrow(`Server not found: ${serverName}`);
         });
@@ -440,6 +434,7 @@ describe('HubToolsService', () => {
                 tagDefinitions: []
             });
             vi.mocked(mcpConnectionManager.getToolsByServerName).mockReturnValue(mockTools);
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockReturnValue([0]);
             // Act
             const tool = await hubToolsService.getTool({ serverName, toolName });
             // Assert
@@ -480,6 +475,8 @@ describe('HubToolsService', () => {
                 instances: [mockInstance],
                 tagDefinitions: []
             });
+            vi.mocked(mcpConnectionManager.getToolsByServerName).mockReturnValue(mockTools);
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockReturnValue([0]);
             vi.mocked(mcpConnectionManager.getTools).mockReturnValue(mockTools);
             // Act
             const tool = await hubToolsService.getTool({ serverName, toolName });
@@ -610,14 +607,15 @@ describe('HubToolsService', () => {
             // Assert - System tools under mcp-hub-lite
             expect(allTools).toHaveProperty('mcp-hub-lite');
             expect(Array.isArray(allTools['mcp-hub-lite'].tools)).toBe(true);
-            // Assert system tools - should have 5 tools now
+            // Assert system tools - should have 6 tools now
             const systemToolNames = allTools['mcp-hub-lite'].tools.map((t) => t.name);
             expect(systemToolNames).toContain('list_servers');
             expect(systemToolNames).toContain('list_tools');
             expect(systemToolNames).toContain('get_tool');
             expect(systemToolNames).toContain('call_tool');
             expect(systemToolNames).toContain('update_server_description');
-            expect(systemToolNames).toHaveLength(5);
+            expect(systemToolNames).toContain('list_tags');
+            expect(systemToolNames).toHaveLength(6);
             // Assert server tools - should have only name and description
             expect(allTools['Server 1'].tools).toEqual(expectedToolSummariesServer1);
             expect(allTools['Server 2'].tools).toEqual(expectedToolSummariesServer2);
@@ -680,6 +678,7 @@ describe('HubToolsService', () => {
             vi.mocked(mcpConnectionManager.getResources).mockReturnValue([
                 { uri: 'test://resource', name: 'Test Resource' }
             ]);
+            vi.mocked(mcpConnectionManager.getConnectedIndexes).mockReturnValue([0]);
             // Act
             const resources = await hubToolsService.listResources();
             // Assert - the resource list should include use-guide and at least the server resource
@@ -746,6 +745,7 @@ describe('HubToolsService', () => {
                 instances: [
                     {
                         id: 'test-instance',
+                        index: 0,
                         enabled: true,
                         args: [],
                         env: {},
@@ -783,7 +783,7 @@ describe('HubToolsService', () => {
                 lastHeartbeat: mockInstance.lastHeartbeat,
                 // @ts-expect-error - Accessing extra fields on mock
                 uptime: mockInstance.uptime,
-                description: `Connected MCP server: ${serverName}`
+                description: `${serverName} (You can check the tool list to understand its capabilities and update the description.)`
             });
         });
         it('should return tools list for tools URI', async () => {
@@ -791,6 +791,7 @@ describe('HubToolsService', () => {
             const serverName = 'Test Server';
             const mockInstance = {
                 id: '1',
+                index: 0,
                 enabled: true,
                 args: [],
                 env: {},
@@ -816,6 +817,7 @@ describe('HubToolsService', () => {
             const serverName = 'Test Server';
             const mockInstance = {
                 id: '1',
+                index: 0,
                 enabled: true,
                 args: [],
                 env: {},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@loop_ouroboros/mcp-hub-lite",
-  "version": "1.2.4",
+  "version": "1.2.6",
   "description": "A lightweight MCP management platform designed for independent developers",
   "license": "MIT",
   "author": "loop_ouroboros",