viberag 0.2.0 → 0.3.0

package/dist/cli/__tests__/mcp-setup.test.js

@@ -1,704 +0,0 @@
-/**
- * MCP Setup Integration Tests
- *
- * Tests for config generation, file creation, merging, and edge cases.
- */
-import { describe, it, expect, beforeEach, afterEach } from 'vitest';
-import fs from 'node:fs/promises';
-import path from 'node:path';
-import os from 'node:os';
-import { writeMcpConfig, generateMcpConfig, generateViberagConfig, generateOpenCodeViberagConfig, generateTomlConfig, readJsonConfig, mergeConfig, mergeTomlConfig, hasViberagConfig, hasViberagTomlConfig, isAlreadyConfigured, configExists, removeViberagConfig, removeViberagFromConfig, removeViberagFromTomlConfig, findConfiguredEditors, } from '../commands/mcp-setup.js';
-import { EDITORS, getEditor } from '../data/mcp-editors.js';
-/**
- * Create a temporary directory for test isolation.
- */
-async function createTempDir() {
-    const dir = await fs.mkdtemp(path.join(os.tmpdir(), 'viberag-mcp-test-'));
-    return {
-        dir,
-        cleanup: async () => {
-            await fs.rm(dir, { recursive: true, force: true });
-        },
-    };
-}
-/**
- * Write a test config file.
- */
-async function writeTestConfig(dir, relativePath, content) {
-    const fullPath = path.join(dir, relativePath);
-    const dirPath = path.dirname(fullPath);
-    await fs.mkdir(dirPath, { recursive: true });
-    const data = typeof content === 'string' ? content : JSON.stringify(content, null, 2);
-    await fs.writeFile(fullPath, data, 'utf-8');
-}
-/**
- * Read a test config file.
- */
-async function readTestConfig(dir, relativePath) {
-    const fullPath = path.join(dir, relativePath);
-    const content = await fs.readFile(fullPath, 'utf-8');
-    return JSON.parse(content);
-}
-// =============================================================================
-// Config Generation Tests
-// =============================================================================
-describe('MCP Config Generation', () => {
-    it('generateViberagConfig returns correct structure', () => {
-        const config = generateViberagConfig();
-        expect(config).toEqual({
-            command: 'npx',
-            args: ['-y', 'viberag-mcp'],
-        });
-    });
-    it('generateMcpConfig uses correct key for Claude Code', () => {
-        const editor = getEditor('claude-code');
-        const config = generateMcpConfig(editor);
-        expect(config).toHaveProperty('mcpServers');
-        expect(config.mcpServers.viberag).toBeDefined();
-    });
-    it('generateMcpConfig uses "mcp" key for OpenCode', () => {
-        const editor = getEditor('opencode');
-        const config = generateMcpConfig(editor);
-        expect(config).toHaveProperty('mcp');
-        expect(config.mcp.viberag).toBeDefined();
-    });
-    it('generateMcpConfig uses "servers" key for VS Code', () => {
-        const editor = getEditor('vscode');
-        const config = generateMcpConfig(editor);
-        expect(config).toHaveProperty('servers');
-        expect(config).not.toHaveProperty('mcpServers');
-    });
-    it('generateMcpConfig uses "context_servers" key for Zed', () => {
-        const editor = getEditor('zed');
-        const config = generateMcpConfig(editor);
-        expect(config).toHaveProperty('context_servers');
-        expect(config).not.toHaveProperty('mcpServers');
-    });
-    it('generateTomlConfig returns valid TOML', () => {
-        const toml = generateTomlConfig();
-        expect(toml).toContain('[mcp_servers.viberag]');
-        expect(toml).toContain('command = "npx"');
-        expect(toml).toContain('args = ["-y", "viberag-mcp"]');
-    });
-});
-// =============================================================================
-// TOML Config Tests (OpenAI Codex)
-// =============================================================================
-describe('TOML Config Functions', () => {
-    it('hasViberagTomlConfig detects viberag section', () => {
-        const toml = `[mcp_servers.viberag]
-command = "npx"
-args = ["-y", "viberag-mcp"]
-`;
-        expect(hasViberagTomlConfig(toml)).toBe(true);
-    });
-    it('hasViberagTomlConfig returns false when viberag missing', () => {
-        const toml = `[mcp_servers.other]
-command = "other-cmd"
-`;
-        expect(hasViberagTomlConfig(toml)).toBe(false);
-    });
-    it('hasViberagTomlConfig handles empty content', () => {
-        expect(hasViberagTomlConfig('')).toBe(false);
-    });
-    it('mergeTomlConfig appends viberag to existing config', () => {
-        const existing = `[mcp_servers.context7]
-command = "npx"
-args = ["context7-mcp"]
-`;
-        const merged = mergeTomlConfig(existing);
-        expect(merged).toContain('[mcp_servers.context7]');
-        expect(merged).toContain('[mcp_servers.viberag]');
-        expect(merged).toContain('args = ["-y", "viberag-mcp"]');
-    });
-    it('mergeTomlConfig does not duplicate if viberag exists', () => {
-        const existing = generateTomlConfig();
-        const merged = mergeTomlConfig(existing);
-        expect(merged).toBe(existing);
-    });
-    it('mergeTomlConfig handles empty file', () => {
-        const merged = mergeTomlConfig('');
-        expect(merged).toContain('[mcp_servers.viberag]');
-    });
-    it('removeViberagFromTomlConfig removes viberag section', () => {
-        const toml = `[mcp_servers.context7]
-command = "npx"
-args = ["context7-mcp"]
-
-[mcp_servers.viberag]
-command = "npx"
-args = ["-y", "viberag-mcp"]
-`;
-        const result = removeViberagFromTomlConfig(toml);
-        expect(result).not.toBeNull();
-        expect(result).toContain('[mcp_servers.context7]');
-        expect(result).not.toContain('[mcp_servers.viberag]');
-    });
-    it('removeViberagFromTomlConfig returns null if viberag not present', () => {
-        const toml = `[mcp_servers.context7]
-command = "npx"
-args = ["context7-mcp"]
-`;
-        const result = removeViberagFromTomlConfig(toml);
-        expect(result).toBeNull();
-    });
-    it('removeViberagFromTomlConfig preserves other sections', () => {
-        const toml = `[general]
-api_key = "test"
-
-[mcp_servers.viberag]
-command = "npx"
-args = ["-y", "viberag-mcp"]
-
-[other_section]
-value = 123
-`;
-        const result = removeViberagFromTomlConfig(toml);
-        expect(result).not.toBeNull();
-        expect(result).toContain('[general]');
-        expect(result).toContain('[other_section]');
-        expect(result).not.toContain('[mcp_servers.viberag]');
-    });
-});
-// =============================================================================
-// Project-Level Config Creation Tests
-// =============================================================================
-describe('Project-Level Config Creation', () => {
-    let ctx;
-    beforeEach(async () => {
-        ctx = await createTempDir();
-    });
-    afterEach(async () => {
-        await ctx.cleanup();
-    });
-    it('creates .mcp.json for Claude Code in empty project', async () => {
-        const editor = getEditor('claude-code');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.method).toBe('file-created');
-        expect(result.configPath).toBe(path.join(ctx.dir, '.mcp.json'));
-        const config = await readTestConfig(ctx.dir, '.mcp.json');
-        expect(config.mcpServers.viberag
-            .command).toBe('npx');
-    });
-    it('creates .vscode/mcp.json with parent directory', async () => {
-        const editor = getEditor('vscode');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.method).toBe('file-created');
-        // Verify .vscode directory was created
-        const vscodeDir = path.join(ctx.dir, '.vscode');
-        const stat = await fs.stat(vscodeDir);
-        expect(stat.isDirectory()).toBe(true);
-        const config = await readTestConfig(ctx.dir, '.vscode/mcp.json');
-        expect(config.servers.viberag).toBeDefined();
-    });
-    it('creates .cursor/mcp.json for Cursor', async () => {
-        const editor = getEditor('cursor');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.configPath).toContain('.cursor/mcp.json');
-        const config = await readTestConfig(ctx.dir, '.cursor/mcp.json');
-        expect(config.mcpServers.viberag).toBeDefined();
-    });
-    it('creates .roo/mcp.json for Roo Code', async () => {
-        const editor = getEditor('roo-code');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.configPath).toContain('.roo/mcp.json');
-        const config = await readTestConfig(ctx.dir, '.roo/mcp.json');
-        expect(config.mcpServers.viberag).toBeDefined();
-    });
-});
-// =============================================================================
-// Config Merging Tests
-// =============================================================================
-describe('Config Merging', () => {
-    let ctx;
-    beforeEach(async () => {
-        ctx = await createTempDir();
-    });
-    afterEach(async () => {
-        await ctx.cleanup();
-    });
-    it('merges viberag into existing mcpServers', async () => {
-        // Setup: Create existing config with another server
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: {
-                'other-server': {
-                    command: 'node',
-                    args: ['other.js'],
-                },
-            },
-        });
-        const editor = getEditor('claude-code');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.method).toBe('file-merged');
-        const config = (await readTestConfig(ctx.dir, '.mcp.json'));
-        // Both servers should exist
-        expect(config.mcpServers['other-server']).toBeDefined();
-        expect(config.mcpServers.viberag).toBeDefined();
-    });
-    it('preserves existing servers during merge', async () => {
-        const existingConfig = {
-            mcpServers: {
-                github: { command: 'npx', args: ['github-mcp'] },
-                filesystem: { command: 'npx', args: ['fs-mcp'] },
-            },
-        };
-        await writeTestConfig(ctx.dir, '.mcp.json', existingConfig);
-        const editor = getEditor('claude-code');
-        await writeMcpConfig(editor, 'project', ctx.dir);
-        const config = (await readTestConfig(ctx.dir, '.mcp.json'));
-        expect(Object.keys(config.mcpServers)).toHaveLength(3);
-        expect(config.mcpServers['github']).toBeDefined();
-        expect(config.mcpServers['filesystem']).toBeDefined();
-        expect(config.mcpServers['viberag']).toBeDefined();
-    });
-    it('merges into VS Code config with servers key', async () => {
-        await writeTestConfig(ctx.dir, '.vscode/mcp.json', {
-            servers: {
-                existing: { command: 'existing' },
-            },
-        });
-        const editor = getEditor('vscode');
-        await writeMcpConfig(editor, 'project', ctx.dir);
-        const config = (await readTestConfig(ctx.dir, '.vscode/mcp.json'));
-        expect(config.servers['existing']).toBeDefined();
-        expect(config.servers['viberag']).toBeDefined();
-    });
-    it('mergeConfig function works correctly', () => {
-        const existing = {
-            mcpServers: {
-                other: { command: 'other' },
-            },
-            someOtherKey: 'value',
-        };
-        const editor = getEditor('claude-code');
-        const merged = mergeConfig(existing, editor);
-        expect(merged.mcpServers['other']).toBeDefined();
-        expect(merged.mcpServers['viberag']).toBeDefined();
-        expect(merged['someOtherKey']).toBe('value');
-    });
-    it('mergeConfig handles empty servers object', () => {
-        const existing = { mcpServers: {} };
-        const editor = getEditor('claude-code');
-        const merged = mergeConfig(existing, editor);
-        expect(merged.mcpServers['viberag']).toBeDefined();
-    });
-    it('mergeConfig creates servers object if missing', () => {
-        const existing = { otherKey: 'value' };
-        const editor = getEditor('claude-code');
-        const merged = mergeConfig(existing, editor);
-        expect(merged.mcpServers['viberag']).toBeDefined();
-    });
-    it('generateMcpConfig works with OpenCode nested mcp structure', () => {
-        const existing = {
-            mcp: {
-                'other-server': { command: 'other' },
-            },
-            someOtherKey: 'value',
-        };
-        const editor = getEditor('opencode');
-        const merged = mergeConfig(existing, editor);
-        expect(merged.mcp['other-server']).toBeDefined();
-        expect(merged.mcp['viberag']).toBeDefined();
-        expect(merged['someOtherKey']).toBe('value');
-    });
-});
-// =============================================================================
-// OpenCode-Specific Tests
-// =============================================================================
-describe('OpenCode MCP Configuration', () => {
-    it('generateOpenCodeViberagConfig returns correct OpenCode format', () => {
-        const config = generateOpenCodeViberagConfig();
-        expect(config).toEqual({
-            type: 'local',
-            command: ['npx', '-y', 'viberag-mcp'],
-        });
-    });
-    it('generateMcpConfig uses OpenCode format for OpenCode editor', () => {
-        const editor = getEditor('opencode');
-        const config = generateMcpConfig(editor);
-        expect(config).toHaveProperty('mcp');
-        expect(config.mcp
-            .viberag.type).toBe('local');
-        expect(config.mcp.viberag.command).toEqual(['npx', '-y', 'viberag-mcp']);
-    });
-    it('generateMcpConfig does NOT use args key for OpenCode', () => {
-        const editor = getEditor('opencode');
-        const config = generateMcpConfig(editor);
-        const viberagConfig = config.mcp.viberag;
-        expect(viberagConfig).not.toHaveProperty('args');
-        expect(viberagConfig).toHaveProperty('command');
-    });
-    it('mergeConfig uses OpenCode format when merging for OpenCode', () => {
-        const existing = {
-            mcp: {
-                'other-server': { command: 'other' },
-            },
-        };
-        const editor = getEditor('opencode');
-        const merged = mergeConfig(existing, editor);
-        expect(merged.mcp['viberag']?.type).toBe('local');
-        expect(merged.mcp['viberag']?.command).toEqual([
-            'npx',
-            '-y',
-            'viberag-mcp',
-        ]);
-        expect(merged.mcp['other-server']).toBeDefined();
-    });
-    it('hasViberagConfig detects viberag in OpenCode config', () => {
-        const config = {
-            mcp: {
-                viberag: { type: 'local', command: ['npx', '-y', 'viberag-mcp'] },
-            },
-        };
-        const editor = getEditor('opencode');
-        expect(hasViberagConfig(config, editor)).toBe(true);
-    });
-    it('hasViberagConfig returns false for missing viberag in OpenCode', () => {
-        const config = {
-            mcp: {
-                other: { type: 'local', command: ['npx', 'other'] },
-            },
-        };
-        const editor = getEditor('opencode');
-        expect(hasViberagConfig(config, editor)).toBe(false);
-    });
-});
-// =============================================================================
-// Already Configured Detection Tests
-// =============================================================================
-describe('Already Configured Detection', () => {
-    let ctx;
-    beforeEach(async () => {
-        ctx = await createTempDir();
-    });
-    afterEach(async () => {
-        await ctx.cleanup();
-    });
-    it('hasViberagConfig returns true when viberag exists', () => {
-        const config = {
-            mcpServers: {
-                viberag: { command: 'npx', args: ['viberag-mcp'] },
-            },
-        };
-        const editor = getEditor('claude-code');
-        expect(hasViberagConfig(config, editor)).toBe(true);
-    });
-    it('hasViberagConfig returns false when viberag missing', () => {
-        const config = {
-            mcpServers: {
-                other: { command: 'other' },
-            },
-        };
-        const editor = getEditor('claude-code');
-        expect(hasViberagConfig(config, editor)).toBe(false);
-    });
-    it('isAlreadyConfigured detects existing viberag config', async () => {
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: {
-                viberag: { command: 'npx', args: ['viberag-mcp'] },
-            },
-        });
-        const editor = getEditor('claude-code');
-        const result = await isAlreadyConfigured(editor, 'project', ctx.dir);
-        expect(result).toBe(true);
-    });
-    it('isAlreadyConfigured returns false for empty project', async () => {
-        const editor = getEditor('claude-code');
-        const result = await isAlreadyConfigured(editor, 'project', ctx.dir);
-        expect(result).toBe(false);
-    });
-    it('writeMcpConfig reports already configured', async () => {
-        // Pre-configure viberag
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: {
-                viberag: { command: 'npx', args: ['viberag-mcp'] },
-            },
-        });
-        const editor = getEditor('claude-code');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.method).toBe('file-merged');
-        expect(result.error).toBe('Already configured');
-    });
-});
-// =============================================================================
-// Edge Cases Tests
-// =============================================================================
-describe('Edge Cases', () => {
-    let ctx;
-    beforeEach(async () => {
-        ctx = await createTempDir();
-    });
-    afterEach(async () => {
-        await ctx.cleanup();
-    });
-    it('handles empty existing config file', async () => {
-        await writeTestConfig(ctx.dir, '.mcp.json', {});
-        const editor = getEditor('claude-code');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        const config = (await readTestConfig(ctx.dir, '.mcp.json'));
-        expect(config.mcpServers['viberag']).toBeDefined();
-    });
-    it('handles malformed JSON gracefully', async () => {
-        // Write invalid JSON
-        const configPath = path.join(ctx.dir, '.mcp.json');
-        await fs.writeFile(configPath, '{ invalid json }', 'utf-8');
-        const editor = getEditor('claude-code');
-        const result = await writeMcpConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(false);
-        expect(result.error).toContain('parse');
-    });
-    it('readJsonConfig returns null for non-existent file', async () => {
-        const result = await readJsonConfig(path.join(ctx.dir, 'nonexistent.json'));
-        expect(result).toBeNull();
-    });
-    it('configExists returns false for non-existent file', async () => {
-        const result = await configExists(path.join(ctx.dir, 'nonexistent.json'));
-        expect(result).toBe(false);
-    });
-    it('configExists returns true for existing file', async () => {
-        await writeTestConfig(ctx.dir, 'test.json', {});
-        const result = await configExists(path.join(ctx.dir, 'test.json'));
-        expect(result).toBe(true);
-    });
-});
-// =============================================================================
-// Editor Configuration Data Tests
-// =============================================================================
-describe('Editor Configuration Data', () => {
-    it('JSON editors with canAutoCreate true support project scope', () => {
-        // JSON editors that can auto-create should support project scope
-        // TOML editors (like Codex) may only support global scope
-        const autoCreateJsonEditors = EDITORS.filter(e => e.canAutoCreate && e.configFormat === 'json');
-        for (const editor of autoCreateJsonEditors) {
-            expect(editor.supportsProject).toBe(true);
-        }
-    });
-    it('Codex editor supports TOML auto-create for global scope only', () => {
-        const codex = EDITORS.find(e => e.id === 'codex');
-        expect(codex).toBeDefined();
-        expect(codex.configFormat).toBe('toml');
-        expect(codex.canAutoCreate).toBe(true);
-        expect(codex.supportsGlobal).toBe(true);
-        expect(codex.supportsProject).toBe(false);
-    });
-    it('all editors have valid docsUrl', () => {
-        for (const editor of EDITORS) {
-            expect(editor.docsUrl).toMatch(/^https?:\/\//);
-        }
-    });
-    it('getEditor returns correct editor by id', () => {
-        const claude = getEditor('claude-code');
-        expect(claude?.name).toBe('Claude Code');
-        const vscode = getEditor('vscode');
-        expect(vscode?.name).toBe('VS Code Copilot');
-        const nonexistent = getEditor('nonexistent');
-        expect(nonexistent).toBeUndefined();
-    });
-    it('each editor has unique id', () => {
-        const ids = EDITORS.map(e => e.id);
-        const uniqueIds = new Set(ids);
-        expect(uniqueIds.size).toBe(ids.length);
-    });
-    it('editors with both scopes have correct paths', () => {
-        const dualScopeEditors = EDITORS.filter(e => e.supportsGlobal && e.supportsProject);
-        for (const editor of dualScopeEditors) {
-            // Project path should be set
-            expect(editor.projectConfigPath).toBeTruthy();
-            // Global path may be null (for UI-only global like VS Code, Roo Code)
-        }
-    });
-});
-// =============================================================================
-// Config Removal Tests
-// =============================================================================
-describe('Config Removal', () => {
-    let ctx;
-    beforeEach(async () => {
-        ctx = await createTempDir();
-    });
-    afterEach(async () => {
-        await ctx.cleanup();
-    });
-    it('removes viberag while preserving other servers', async () => {
-        // Setup: config with viberag and another server
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: {
-                viberag: { command: 'npx', args: ['viberag-mcp'] },
-                github: { command: 'npx', args: ['github-mcp'] },
-            },
-        });
-        const editor = getEditor('claude-code');
-        const result = await removeViberagConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        expect(result.editor).toBe('claude-code');
-        // File should still exist with github server preserved
-        const config = (await readTestConfig(ctx.dir, '.mcp.json'));
-        expect(config.mcpServers['github']).toBeDefined();
-        expect(config.mcpServers['viberag']).toBeUndefined();
-    });
-    it('removes viberag and keeps file with empty servers', async () => {
-        // Setup: config with only viberag
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: {
-                viberag: { command: 'npx', args: ['viberag-mcp'] },
-            },
-        });
-        const editor = getEditor('claude-code');
-        const result = await removeViberagConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        // File should still exist with empty mcpServers
-        const config = (await readTestConfig(ctx.dir, '.mcp.json'));
-        expect(config.mcpServers).toBeDefined();
-        expect(Object.keys(config.mcpServers)).toHaveLength(0);
-    });
-    it('returns error when viberag not configured', async () => {
-        // Setup: config without viberag
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: {
-                github: { command: 'npx', args: ['github-mcp'] },
-            },
-        });
-        const editor = getEditor('claude-code');
-        const result = await removeViberagConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(false);
-        expect(result.error).toContain('not configured');
-    });
-    it('returns error when config file does not exist', async () => {
-        const editor = getEditor('claude-code');
-        const result = await removeViberagConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(false);
-        expect(result.error).toContain('does not exist');
-    });
-    it('handles VS Code config with servers key', async () => {
-        await writeTestConfig(ctx.dir, '.vscode/mcp.json', {
-            servers: {
-                viberag: { command: 'npx', args: ['viberag-mcp'] },
-                other: { command: 'other' },
-            },
-        });
-        const editor = getEditor('vscode');
-        const result = await removeViberagConfig(editor, 'project', ctx.dir);
-        expect(result.success).toBe(true);
-        const config = (await readTestConfig(ctx.dir, '.vscode/mcp.json'));
-        expect(config.servers['other']).toBeDefined();
-        expect(config.servers['viberag']).toBeUndefined();
-    });
-});
-// =============================================================================
-// removeViberagFromConfig Helper Tests
-// =============================================================================
-describe('removeViberagFromConfig helper', () => {
-    it('returns null when servers key is missing', () => {
-        const existing = { otherKey: 'value' };
-        const editor = getEditor('claude-code');
-        const result = removeViberagFromConfig(existing, editor);
-        expect(result).toBeNull();
-    });
-    it('returns null when servers is not an object', () => {
-        const existing = { mcpServers: 'not-an-object' };
-        const editor = getEditor('claude-code');
-        const result = removeViberagFromConfig(existing, editor);
-        expect(result).toBeNull();
-    });
-    it('returns null when viberag not in servers', () => {
-        const existing = {
-            mcpServers: {
-                github: { command: 'github' },
-            },
-        };
-        const editor = getEditor('claude-code');
-        const result = removeViberagFromConfig(existing, editor);
-        expect(result).toBeNull();
-    });
-    it('returns modified config with viberag removed', () => {
-        const existing = {
-            mcpServers: {
-                viberag: { command: 'npx' },
-                github: { command: 'github' },
-            },
-            otherKey: 'preserved',
-        };
-        const editor = getEditor('claude-code');
-        const result = removeViberagFromConfig(existing, editor);
-        expect(result).not.toBeNull();
-        expect(result.mcpServers['viberag']).toBeUndefined();
-        expect(result.mcpServers['github']).toBeDefined();
-        expect(result.otherKey).toBe('preserved');
-    });
-    it('works with VS Code servers key', () => {
-        const existing = {
-            servers: {
-                viberag: { command: 'npx' },
-                other: { command: 'other' },
-            },
-        };
-        const editor = getEditor('vscode');
-        const result = removeViberagFromConfig(existing, editor);
-        expect(result).not.toBeNull();
-        expect(result.servers['viberag']).toBeUndefined();
-        expect(result.servers['other']).toBeDefined();
-    });
-});
-// =============================================================================
-// findConfiguredEditors Tests
-// =============================================================================
-describe('findConfiguredEditors', () => {
-    let ctx;
-    beforeEach(async () => {
-        ctx = await createTempDir();
-    });
-    afterEach(async () => {
-        await ctx.cleanup();
-    });
-    it('finds project-scope editors with viberag configured', async () => {
-        // Setup: Claude Code and Cursor both configured
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: { viberag: { command: 'npx' } },
-        });
-        await writeTestConfig(ctx.dir, '.cursor/mcp.json', {
-            mcpServers: { viberag: { command: 'npx' } },
-        });
-        const result = await findConfiguredEditors(ctx.dir);
-        expect(result.projectScope.length).toBeGreaterThanOrEqual(2);
-        const ids = result.projectScope.map(e => e.editor.id);
-        expect(ids).toContain('claude-code');
-        expect(ids).toContain('cursor');
-    });
-    it('returns empty project scope when no configs exist', async () => {
-        const result = await findConfiguredEditors(ctx.dir);
-        // Project scope should be empty since we didn't create any configs
-        expect(result.projectScope).toHaveLength(0);
-        // Global scope might contain editors if they're configured globally on this machine
-        // This is expected behavior - the function checks real global configs
-    });
-    it('does not include editors without viberag', async () => {
-        // Setup: Claude Code has viberag, Cursor has different server
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: { viberag: { command: 'npx' } },
-        });
-        await writeTestConfig(ctx.dir, '.cursor/mcp.json', {
-            mcpServers: { github: { command: 'npx' } },
-        });
-        const result = await findConfiguredEditors(ctx.dir);
-        const ids = result.projectScope.map(e => e.editor.id);
-        expect(ids).toContain('claude-code');
-        expect(ids).not.toContain('cursor');
-    });
-    it('ConfiguredEditorInfo includes scope information', async () => {
-        await writeTestConfig(ctx.dir, '.mcp.json', {
-            mcpServers: { viberag: { command: 'npx' } },
-        });
-        const result = await findConfiguredEditors(ctx.dir);
-        const claudeInfo = result.projectScope.find(e => e.editor.id === 'claude-code');
-        expect(claudeInfo).toBeDefined();
-        expect(claudeInfo?.scope).toBe('project');
-        expect(claudeInfo?.editor.name).toBe('Claude Code');
-    });
-});