viberag 0.1.0

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

@@ -0,0 +1,597 @@
+/**
+ * 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, hasViberagConfig, isAlreadyConfigured, configExists, removeViberagConfig, removeViberagFromConfig, 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: ['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 = ["viberag-mcp"]');
+    });
+});
+// =============================================================================
+// 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, 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, 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, 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, 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, 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, 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, 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, ctx.dir);
+        expect(result).toBe(true);
+    });
+    it('isAlreadyConfigured returns false for empty project', async () => {
+        const editor = getEditor('claude-code');
+        const result = await isAlreadyConfigured(editor, 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, 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, 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, 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('all project-scope editors have canAutoCreate true', () => {
+        const projectEditors = EDITORS.filter(e => e.scope === 'project');
+        for (const editor of projectEditors) {
+            expect(editor.canAutoCreate).toBe(true);
+        }
+    });
+    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);
+    });
+});
+// =============================================================================
+// 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, 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, 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, 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, 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, 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.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.id);
+        expect(ids).toContain('claude-code');
+        expect(ids).not.toContain('cursor');
+    });
+});

package/dist/cli/app.d.ts

@@ -0,0 +1,2 @@
+import React from 'react';
+export default function App(): React.JSX.Element;