@vibescope/mcp-server 0.5.0 → 0.5.1

package/src/setup.test.ts

@@ -1,243 +1,243 @@
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-
-// Mock node:fs before importing setup module
-vi.mock('node:fs', () => ({
-	existsSync: vi.fn(),
-	readFileSync: vi.fn(),
-	writeFileSync: vi.fn(),
-	mkdirSync: vi.fn(),
-}));
-
-// Mock node:os
-vi.mock('node:os', () => ({
-	homedir: vi.fn(() => '/home/testuser'),
-	platform: vi.fn(() => 'darwin'),
-}));
-
-import { existsSync } from 'node:fs';
-import { homedir, platform } from 'node:os';
-import { detectIdes, generateMcpConfig, type IdeConfig } from './setup.js';
-
-describe('Setup module', () => {
-	beforeEach(() => {
-		vi.clearAllMocks();
-		vi.mocked(homedir).mockReturnValue('/home/testuser');
-		vi.mocked(platform).mockReturnValue('darwin');
-	});
-
-	describe('detectIdes', () => {
-		it('should always include Claude Code as detected', () => {
-			vi.mocked(existsSync).mockReturnValue(false);
-			const ides = detectIdes();
-			const claudeCode = ides.find(ide => ide.name === 'claude-code');
-
-			expect(claudeCode).toBeDefined();
-			expect(claudeCode?.detected).toBe(true);
-			expect(claudeCode?.configPath).toBe('.mcp.json');
-			expect(claudeCode?.configFormat).toBe('mcp-json');
-		});
-
-		it('should always include Gemini CLI as detected', () => {
-			vi.mocked(existsSync).mockReturnValue(false);
-			const ides = detectIdes();
-			const gemini = ides.find(ide => ide.name === 'gemini');
-
-			expect(gemini).toBeDefined();
-			expect(gemini?.detected).toBe(true);
-			expect(gemini?.configFormat).toBe('settings-json');
-		});
-
-		it('should detect Claude Desktop when directory exists', () => {
-			vi.mocked(existsSync).mockImplementation((path: unknown) => {
-				if (typeof path === 'string' && path.includes('Claude')) {
-					return true;
-				}
-				return false;
-			});
-
-			const ides = detectIdes();
-			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
-
-			expect(claudeDesktop).toBeDefined();
-			expect(claudeDesktop?.detected).toBe(true);
-		});
-
-		it('should not detect Claude Desktop when directory does not exist', () => {
-			vi.mocked(existsSync).mockReturnValue(false);
-
-			const ides = detectIdes();
-			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
-
-			expect(claudeDesktop).toBeDefined();
-			expect(claudeDesktop?.detected).toBe(false);
-		});
-
-		it('should detect Cursor when directory exists', () => {
-			vi.mocked(existsSync).mockImplementation((path: unknown) => {
-				if (typeof path === 'string' && path.includes('Cursor')) {
-					return true;
-				}
-				return false;
-			});
-
-			const ides = detectIdes();
-			const cursor = ides.find(ide => ide.name === 'cursor');
-
-			expect(cursor).toBeDefined();
-			expect(cursor?.detected).toBe(true);
-		});
-
-		it('should return all four IDE configs', () => {
-			vi.mocked(existsSync).mockReturnValue(false);
-			const ides = detectIdes();
-
-			expect(ides).toHaveLength(4);
-			expect(ides.map(ide => ide.name)).toEqual([
-				'claude-code',
-				'claude-desktop',
-				'cursor',
-				'gemini',
-			]);
-		});
-
-		it('should use correct config paths on macOS', () => {
-			vi.mocked(platform).mockReturnValue('darwin');
-			vi.mocked(homedir).mockReturnValue('/Users/testuser');
-			vi.mocked(existsSync).mockReturnValue(true);
-
-			const ides = detectIdes();
-
-			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
-			// Note: path.join uses OS separator, so we check for path components
-			expect(claudeDesktop?.configPath).toMatch(/Library.*Application Support.*Claude/);
-
-			const cursor = ides.find(ide => ide.name === 'cursor');
-			expect(cursor?.configPath).toMatch(/Library.*Application Support.*Cursor/);
-
-			const gemini = ides.find(ide => ide.name === 'gemini');
-			expect(gemini?.configPath).toMatch(/Users.*testuser.*\.gemini.*settings\.json/);
-		});
-
-		it('should use correct config paths on Windows', () => {
-			vi.mocked(platform).mockReturnValue('win32');
-			vi.mocked(homedir).mockReturnValue('C:\\Users\\testuser');
-			vi.mocked(existsSync).mockReturnValue(true);
-
-			const ides = detectIdes();
-
-			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
-			// Note: path.join uses OS separator, so we check for path components
-			expect(claudeDesktop?.configPath).toMatch(/AppData.*Roaming.*Claude/);
-
-			const cursor = ides.find(ide => ide.name === 'cursor');
-			expect(cursor?.configPath).toMatch(/AppData.*Roaming.*Cursor/);
-		});
-
-		it('should use correct config paths on Linux', () => {
-			vi.mocked(platform).mockReturnValue('linux');
-			vi.mocked(homedir).mockReturnValue('/home/testuser');
-			vi.mocked(existsSync).mockReturnValue(true);
-
-			const ides = detectIdes();
-
-			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
-			// Note: path.join uses OS separator, so we check for path components
-			expect(claudeDesktop?.configPath).toMatch(/\.config.*Claude/);
-
-			const cursor = ides.find(ide => ide.name === 'cursor');
-			expect(cursor?.configPath).toMatch(/\.config.*Cursor/);
-		});
-	});
-
-	describe('IdeConfig types', () => {
-		it('should have correct config format for MCP-based IDEs', () => {
-			vi.mocked(existsSync).mockReturnValue(true);
-			const ides = detectIdes();
-
-			const claudeCode = ides.find(ide => ide.name === 'claude-code');
-			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
-			const cursor = ides.find(ide => ide.name === 'cursor');
-
-			expect(claudeCode?.configFormat).toBe('mcp-json');
-			expect(claudeDesktop?.configFormat).toBe('mcp-json');
-			expect(cursor?.configFormat).toBe('mcp-json');
-		});
-
-		it('should have correct config format for Gemini CLI', () => {
-			vi.mocked(existsSync).mockReturnValue(true);
-			const ides = detectIdes();
-
-			const gemini = ides.find(ide => ide.name === 'gemini');
-			expect(gemini?.configFormat).toBe('settings-json');
-		});
-	});
-
-	describe('generateMcpConfig', () => {
-		it('should generate correct config structure for standard MCP IDEs', () => {
-			const claudeCodeIde: IdeConfig = {
-				name: 'claude-code',
-				displayName: 'Claude Code (CLI)',
-				configPath: '.mcp.json',
-				detected: true,
-				configFormat: 'mcp-json',
-			};
-
-			const config = generateMcpConfig('test-api-key', claudeCodeIde);
-			const mcpServers = config.mcpServers as Record<string, unknown>;
-			const vibescope = mcpServers.vibescope as Record<string, unknown>;
-
-			// Config uses shell wrapper to prefer global binary with npx fallback
-			expect(['cmd', 'bash']).toContain(vibescope.command);
-			const argsStr = JSON.stringify(vibescope.args);
-			expect(argsStr).toContain('vibescope-mcp');
-			expect(argsStr).toContain('npx');
-			expect((vibescope.env as Record<string, string>).VIBESCOPE_API_KEY).toBe('test-api-key');
-			// Standard MCP config should NOT have timeout/trust
-			expect(vibescope.timeout).toBeUndefined();
-			expect(vibescope.trust).toBeUndefined();
-		});
-
-		it('should include timeout and trust for Gemini CLI config', () => {
-			const geminiIde: IdeConfig = {
-				name: 'gemini',
-				displayName: 'Gemini CLI',
-				configPath: '~/.gemini/settings.json',
-				detected: true,
-				configFormat: 'settings-json',
-			};
-
-			const config = generateMcpConfig('test-api-key', geminiIde);
-			const mcpServers = config.mcpServers as Record<string, unknown>;
-			const vibescope = mcpServers.vibescope as Record<string, unknown>;
-
-			expect(['cmd', 'bash']).toContain(vibescope.command);
-			const argsStr = JSON.stringify(vibescope.args);
-			expect(argsStr).toContain('vibescope-mcp');
-			expect(argsStr).toContain('npx');
-			expect((vibescope.env as Record<string, string>).VIBESCOPE_API_KEY).toBe('test-api-key');
-			expect(vibescope.timeout).toBe(30000);
-			expect(vibescope.trust).toBe(true);
-		});
-
-		it('should use shell wrapper with global binary fallback', () => {
-			const ide: IdeConfig = {
-				name: 'claude-code',
-				displayName: 'Claude Code (CLI)',
-				configPath: '.mcp.json',
-				detected: true,
-				configFormat: 'mcp-json',
-			};
-
-			const config = generateMcpConfig('my-key', ide);
-			const mcpServers = config.mcpServers as Record<string, unknown>;
-			const vibescope = mcpServers.vibescope as Record<string, unknown>;
-			const args = vibescope.args as string[];
-
-			// Should check for global binary first, then fall back to npx
-			const shellCmd = args.join(' ');
-			expect(shellCmd).toContain('vibescope-mcp');
-			expect(shellCmd).toContain('npx');
-			expect(shellCmd).toContain('@vibescope/mcp-server@latest');
-		});
-	});
-});
+import { describe, it, expect, vi, beforeEach } from 'vitest';
+
+// Mock node:fs before importing setup module
+vi.mock('node:fs', () => ({
+	existsSync: vi.fn(),
+	readFileSync: vi.fn(),
+	writeFileSync: vi.fn(),
+	mkdirSync: vi.fn(),
+}));
+
+// Mock node:os
+vi.mock('node:os', () => ({
+	homedir: vi.fn(() => '/home/testuser'),
+	platform: vi.fn(() => 'darwin'),
+}));
+
+import { existsSync } from 'node:fs';
+import { homedir, platform } from 'node:os';
+import { detectIdes, generateMcpConfig, type IdeConfig } from './setup.js';
+
+describe('Setup module', () => {
+	beforeEach(() => {
+		vi.clearAllMocks();
+		vi.mocked(homedir).mockReturnValue('/home/testuser');
+		vi.mocked(platform).mockReturnValue('darwin');
+	});
+
+	describe('detectIdes', () => {
+		it('should always include Claude Code as detected', () => {
+			vi.mocked(existsSync).mockReturnValue(false);
+			const ides = detectIdes();
+			const claudeCode = ides.find(ide => ide.name === 'claude-code');
+
+			expect(claudeCode).toBeDefined();
+			expect(claudeCode?.detected).toBe(true);
+			expect(claudeCode?.configPath).toBe('.mcp.json');
+			expect(claudeCode?.configFormat).toBe('mcp-json');
+		});
+
+		it('should always include Gemini CLI as detected', () => {
+			vi.mocked(existsSync).mockReturnValue(false);
+			const ides = detectIdes();
+			const gemini = ides.find(ide => ide.name === 'gemini');
+
+			expect(gemini).toBeDefined();
+			expect(gemini?.detected).toBe(true);
+			expect(gemini?.configFormat).toBe('settings-json');
+		});
+
+		it('should detect Claude Desktop when directory exists', () => {
+			vi.mocked(existsSync).mockImplementation((path: unknown) => {
+				if (typeof path === 'string' && path.includes('Claude')) {
+					return true;
+				}
+				return false;
+			});
+
+			const ides = detectIdes();
+			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
+
+			expect(claudeDesktop).toBeDefined();
+			expect(claudeDesktop?.detected).toBe(true);
+		});
+
+		it('should not detect Claude Desktop when directory does not exist', () => {
+			vi.mocked(existsSync).mockReturnValue(false);
+
+			const ides = detectIdes();
+			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
+
+			expect(claudeDesktop).toBeDefined();
+			expect(claudeDesktop?.detected).toBe(false);
+		});
+
+		it('should detect Cursor when directory exists', () => {
+			vi.mocked(existsSync).mockImplementation((path: unknown) => {
+				if (typeof path === 'string' && path.includes('Cursor')) {
+					return true;
+				}
+				return false;
+			});
+
+			const ides = detectIdes();
+			const cursor = ides.find(ide => ide.name === 'cursor');
+
+			expect(cursor).toBeDefined();
+			expect(cursor?.detected).toBe(true);
+		});
+
+		it('should return all four IDE configs', () => {
+			vi.mocked(existsSync).mockReturnValue(false);
+			const ides = detectIdes();
+
+			expect(ides).toHaveLength(4);
+			expect(ides.map(ide => ide.name)).toEqual([
+				'claude-code',
+				'claude-desktop',
+				'cursor',
+				'gemini',
+			]);
+		});
+
+		it('should use correct config paths on macOS', () => {
+			vi.mocked(platform).mockReturnValue('darwin');
+			vi.mocked(homedir).mockReturnValue('/Users/testuser');
+			vi.mocked(existsSync).mockReturnValue(true);
+
+			const ides = detectIdes();
+
+			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
+			// Note: path.join uses OS separator, so we check for path components
+			expect(claudeDesktop?.configPath).toMatch(/Library.*Application Support.*Claude/);
+
+			const cursor = ides.find(ide => ide.name === 'cursor');
+			expect(cursor?.configPath).toMatch(/Library.*Application Support.*Cursor/);
+
+			const gemini = ides.find(ide => ide.name === 'gemini');
+			expect(gemini?.configPath).toMatch(/Users.*testuser.*\.gemini.*settings\.json/);
+		});
+
+		it('should use correct config paths on Windows', () => {
+			vi.mocked(platform).mockReturnValue('win32');
+			vi.mocked(homedir).mockReturnValue('C:\\Users\\testuser');
+			vi.mocked(existsSync).mockReturnValue(true);
+
+			const ides = detectIdes();
+
+			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
+			// Note: path.join uses OS separator, so we check for path components
+			expect(claudeDesktop?.configPath).toMatch(/AppData.*Roaming.*Claude/);
+
+			const cursor = ides.find(ide => ide.name === 'cursor');
+			expect(cursor?.configPath).toMatch(/AppData.*Roaming.*Cursor/);
+		});
+
+		it('should use correct config paths on Linux', () => {
+			vi.mocked(platform).mockReturnValue('linux');
+			vi.mocked(homedir).mockReturnValue('/home/testuser');
+			vi.mocked(existsSync).mockReturnValue(true);
+
+			const ides = detectIdes();
+
+			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
+			// Note: path.join uses OS separator, so we check for path components
+			expect(claudeDesktop?.configPath).toMatch(/\.config.*Claude/);
+
+			const cursor = ides.find(ide => ide.name === 'cursor');
+			expect(cursor?.configPath).toMatch(/\.config.*Cursor/);
+		});
+	});
+
+	describe('IdeConfig types', () => {
+		it('should have correct config format for MCP-based IDEs', () => {
+			vi.mocked(existsSync).mockReturnValue(true);
+			const ides = detectIdes();
+
+			const claudeCode = ides.find(ide => ide.name === 'claude-code');
+			const claudeDesktop = ides.find(ide => ide.name === 'claude-desktop');
+			const cursor = ides.find(ide => ide.name === 'cursor');
+
+			expect(claudeCode?.configFormat).toBe('mcp-json');
+			expect(claudeDesktop?.configFormat).toBe('mcp-json');
+			expect(cursor?.configFormat).toBe('mcp-json');
+		});
+
+		it('should have correct config format for Gemini CLI', () => {
+			vi.mocked(existsSync).mockReturnValue(true);
+			const ides = detectIdes();
+
+			const gemini = ides.find(ide => ide.name === 'gemini');
+			expect(gemini?.configFormat).toBe('settings-json');
+		});
+	});
+
+	describe('generateMcpConfig', () => {
+		it('should generate correct config structure for standard MCP IDEs', () => {
+			const claudeCodeIde: IdeConfig = {
+				name: 'claude-code',
+				displayName: 'Claude Code (CLI)',
+				configPath: '.mcp.json',
+				detected: true,
+				configFormat: 'mcp-json',
+			};
+
+			const config = generateMcpConfig('test-api-key', claudeCodeIde);
+			const mcpServers = config.mcpServers as Record<string, unknown>;
+			const vibescope = mcpServers.vibescope as Record<string, unknown>;
+
+			// Config uses shell wrapper to prefer global binary with npx fallback
+			expect(['cmd', 'bash']).toContain(vibescope.command);
+			const argsStr = JSON.stringify(vibescope.args);
+			expect(argsStr).toContain('vibescope-mcp');
+			expect(argsStr).toContain('npx');
+			expect((vibescope.env as Record<string, string>).VIBESCOPE_API_KEY).toBe('test-api-key');
+			// Standard MCP config should NOT have timeout/trust
+			expect(vibescope.timeout).toBeUndefined();
+			expect(vibescope.trust).toBeUndefined();
+		});
+
+		it('should include timeout and trust for Gemini CLI config', () => {
+			const geminiIde: IdeConfig = {
+				name: 'gemini',
+				displayName: 'Gemini CLI',
+				configPath: '~/.gemini/settings.json',
+				detected: true,
+				configFormat: 'settings-json',
+			};
+
+			const config = generateMcpConfig('test-api-key', geminiIde);
+			const mcpServers = config.mcpServers as Record<string, unknown>;
+			const vibescope = mcpServers.vibescope as Record<string, unknown>;
+
+			expect(['cmd', 'bash']).toContain(vibescope.command);
+			const argsStr = JSON.stringify(vibescope.args);
+			expect(argsStr).toContain('vibescope-mcp');
+			expect(argsStr).toContain('npx');
+			expect((vibescope.env as Record<string, string>).VIBESCOPE_API_KEY).toBe('test-api-key');
+			expect(vibescope.timeout).toBe(30000);
+			expect(vibescope.trust).toBe(true);
+		});
+
+		it('should use shell wrapper with global binary fallback', () => {
+			const ide: IdeConfig = {
+				name: 'claude-code',
+				displayName: 'Claude Code (CLI)',
+				configPath: '.mcp.json',
+				detected: true,
+				configFormat: 'mcp-json',
+			};
+
+			const config = generateMcpConfig('my-key', ide);
+			const mcpServers = config.mcpServers as Record<string, unknown>;
+			const vibescope = mcpServers.vibescope as Record<string, unknown>;
+			const args = vibescope.args as string[];
+
+			// Should check for global binary first, then fall back to npx
+			const shellCmd = args.join(' ');
+			expect(shellCmd).toContain('vibescope-mcp');
+			expect(shellCmd).toContain('npx');
+			expect(shellCmd).toContain('@vibescope/mcp-server@latest');
+		});
+	});
+});