@nicnocquee/dataqueue 1.32.0 → 1.34.0

package/src/install-mcp-command.test.ts

@@ -0,0 +1,216 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import {
+  runInstallMcp,
+  upsertMcpConfig,
+  InstallMcpDeps,
+} from './install-mcp-command.js';
+
+describe('upsertMcpConfig', () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('creates new config file when it does not exist', () => {
+    // Setup
+    const deps = {
+      existsSync: vi.fn(() => false),
+      readFileSync: vi.fn(),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMcpConfig(
+      '/path/mcp.json',
+      'dataqueue',
+      { command: 'npx', args: ['dataqueue-cli', 'mcp'] },
+      deps,
+    );
+
+    // Assert
+    const written = JSON.parse(deps.writeFileSync.mock.calls[0][1] as string);
+    expect(written.mcpServers.dataqueue).toEqual({
+      command: 'npx',
+      args: ['dataqueue-cli', 'mcp'],
+    });
+  });
+
+  it('adds to existing config without overwriting other servers', () => {
+    // Setup
+    const existing = JSON.stringify({
+      mcpServers: { other: { command: 'other' } },
+    });
+    const deps = {
+      existsSync: vi.fn(() => true),
+      readFileSync: vi.fn(() => existing),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMcpConfig(
+      '/path/mcp.json',
+      'dataqueue',
+      { command: 'npx', args: ['dataqueue-cli', 'mcp'] },
+      deps,
+    );
+
+    // Assert
+    const written = JSON.parse(deps.writeFileSync.mock.calls[0][1] as string);
+    expect(written.mcpServers.other).toEqual({ command: 'other' });
+    expect(written.mcpServers.dataqueue).toEqual({
+      command: 'npx',
+      args: ['dataqueue-cli', 'mcp'],
+    });
+  });
+
+  it('overwrites existing dataqueue entry', () => {
+    // Setup
+    const existing = JSON.stringify({
+      mcpServers: { dataqueue: { command: 'old' } },
+    });
+    const deps = {
+      existsSync: vi.fn(() => true),
+      readFileSync: vi.fn(() => existing),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMcpConfig('/path/mcp.json', 'dataqueue', { command: 'new' }, deps);
+
+    // Assert
+    const written = JSON.parse(deps.writeFileSync.mock.calls[0][1] as string);
+    expect(written.mcpServers.dataqueue).toEqual({ command: 'new' });
+  });
+
+  it('handles malformed JSON in existing file', () => {
+    // Setup
+    const deps = {
+      existsSync: vi.fn(() => true),
+      readFileSync: vi.fn(() => 'not json'),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMcpConfig('/path/mcp.json', 'dataqueue', { command: 'npx' }, deps);
+
+    // Assert
+    const written = JSON.parse(deps.writeFileSync.mock.calls[0][1] as string);
+    expect(written.mcpServers.dataqueue).toEqual({ command: 'npx' });
+  });
+});
+
+describe('runInstallMcp', () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  function makeDeps(overrides: Partial<InstallMcpDeps> = {}): InstallMcpDeps {
+    return {
+      log: vi.fn(),
+      error: vi.fn(),
+      exit: vi.fn(),
+      cwd: '/project',
+      readFileSync: vi.fn(() => '{}'),
+      writeFileSync: vi.fn(),
+      mkdirSync: vi.fn(),
+      existsSync: vi.fn(() => false),
+      ...overrides,
+    };
+  }
+
+  it('installs MCP config for Cursor (option 1)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '1' });
+
+    // Act
+    await runInstallMcp(deps);
+
+    // Assert
+    expect(deps.mkdirSync).toHaveBeenCalledWith('/project/.cursor', {
+      recursive: true,
+    });
+    const written = JSON.parse(
+      (deps.writeFileSync as ReturnType<typeof vi.fn>).mock
+        .calls[0][1] as string,
+    );
+    expect(written.mcpServers.dataqueue.command).toBe('npx');
+    expect(written.mcpServers.dataqueue.args).toEqual(['dataqueue-cli', 'mcp']);
+  });
+
+  it('installs MCP config for Claude Code (option 2)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '2' });
+
+    // Act
+    await runInstallMcp(deps);
+
+    // Assert
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/project/.mcp.json',
+      expect.any(String),
+    );
+  });
+
+  it('installs MCP config for VS Code (option 3)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '3' });
+
+    // Act
+    await runInstallMcp(deps);
+
+    // Assert
+    expect(deps.mkdirSync).toHaveBeenCalledWith('/project/.vscode', {
+      recursive: true,
+    });
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      expect.stringContaining('.vscode/mcp.json'),
+      expect.any(String),
+    );
+  });
+
+  it('exits with error for invalid choice', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '99' });
+
+    // Act
+    await runInstallMcp(deps);
+
+    // Assert
+    expect(deps.error).toHaveBeenCalledWith(
+      expect.stringContaining('Invalid choice'),
+    );
+    expect(deps.exit).toHaveBeenCalledWith(1);
+  });
+
+  it('handles install errors', async () => {
+    // Setup
+    const deps = makeDeps({
+      selectedClient: '1',
+      writeFileSync: vi.fn(() => {
+        throw new Error('permission denied');
+      }),
+    });
+
+    // Act
+    await runInstallMcp(deps);
+
+    // Assert
+    expect(deps.error).toHaveBeenCalledWith(
+      'Failed to install MCP config:',
+      expect.any(Error),
+    );
+    expect(deps.exit).toHaveBeenCalledWith(1);
+  });
+
+  it('logs done message on success', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '1' });
+
+    // Act
+    await runInstallMcp(deps);
+
+    // Assert
+    expect(deps.log).toHaveBeenCalledWith(
+      expect.stringContaining('npx dataqueue-cli mcp'),
+    );
+  });
+});

package/src/install-mcp-command.ts

@@ -0,0 +1,185 @@
+import fs from 'fs';
+import path from 'path';
+import readline from 'readline';
+
+export interface InstallMcpDeps {
+  log?: (...args: unknown[]) => void;
+  error?: (...args: unknown[]) => void;
+  exit?: (code: number) => void;
+  cwd?: string;
+  readFileSync?: (p: string, enc: BufferEncoding) => string;
+  writeFileSync?: (p: string, data: string) => void;
+  mkdirSync?: (p: string, opts?: fs.MakeDirectoryOptions) => void;
+  existsSync?: (p: string) => boolean;
+  /** Override for selecting the client (skips interactive prompt). */
+  selectedClient?: string;
+}
+
+interface McpClientConfig {
+  label: string;
+  install: (
+    deps: Required<
+      Pick<
+        InstallMcpDeps,
+        | 'cwd'
+        | 'readFileSync'
+        | 'writeFileSync'
+        | 'mkdirSync'
+        | 'existsSync'
+        | 'log'
+      >
+    >,
+  ) => void;
+}
+
+/**
+ * Merges the dataqueue MCP server config into an existing JSON config file.
+ *
+ * @param filePath - Path to the MCP config file.
+ * @param serverKey - Key name for the server entry.
+ * @param serverConfig - Server configuration object.
+ * @param deps - Injectable file system functions.
+ */
+export function upsertMcpConfig(
+  filePath: string,
+  serverKey: string,
+  serverConfig: Record<string, unknown>,
+  deps: {
+    readFileSync: (p: string, enc: BufferEncoding) => string;
+    writeFileSync: (p: string, data: string) => void;
+    existsSync: (p: string) => boolean;
+  },
+): void {
+  let config: Record<string, unknown> = {};
+
+  if (deps.existsSync(filePath)) {
+    try {
+      config = JSON.parse(deps.readFileSync(filePath, 'utf-8'));
+    } catch {
+      config = {};
+    }
+  }
+
+  if (!config.mcpServers || typeof config.mcpServers !== 'object') {
+    config.mcpServers = {};
+  }
+
+  (config.mcpServers as Record<string, unknown>)[serverKey] = serverConfig;
+  deps.writeFileSync(filePath, JSON.stringify(config, null, 2) + '\n');
+}
+
+const MCP_SERVER_CONFIG = {
+  command: 'npx',
+  args: ['dataqueue-cli', 'mcp'],
+};
+
+const MCP_CLIENTS: Record<string, McpClientConfig> = {
+  '1': {
+    label: 'Cursor',
+    install: (deps) => {
+      const configDir = path.join(deps.cwd, '.cursor');
+      deps.mkdirSync(configDir, { recursive: true });
+      const configFile = path.join(configDir, 'mcp.json');
+      upsertMcpConfig(configFile, 'dataqueue', MCP_SERVER_CONFIG, deps);
+      deps.log(`  ✓ .cursor/mcp.json`);
+    },
+  },
+  '2': {
+    label: 'Claude Code',
+    install: (deps) => {
+      const configFile = path.join(deps.cwd, '.mcp.json');
+      upsertMcpConfig(configFile, 'dataqueue', MCP_SERVER_CONFIG, deps);
+      deps.log(`  ✓ .mcp.json`);
+    },
+  },
+  '3': {
+    label: 'VS Code (Copilot)',
+    install: (deps) => {
+      const configDir = path.join(deps.cwd, '.vscode');
+      deps.mkdirSync(configDir, { recursive: true });
+      const configFile = path.join(configDir, 'mcp.json');
+      upsertMcpConfig(configFile, 'dataqueue', MCP_SERVER_CONFIG, deps);
+      deps.log(`  ✓ .vscode/mcp.json`);
+    },
+  },
+  '4': {
+    label: 'Windsurf',
+    install: (deps) => {
+      const homeDir = process.env.HOME || process.env.USERPROFILE || '';
+      const configFile = path.join(
+        homeDir,
+        '.codeium',
+        'windsurf',
+        'mcp_config.json',
+      );
+      deps.mkdirSync(path.dirname(configFile), { recursive: true });
+      upsertMcpConfig(configFile, 'dataqueue', MCP_SERVER_CONFIG, deps);
+      deps.log(`  ✓ ~/.codeium/windsurf/mcp_config.json`);
+    },
+  },
+};
+
+/**
+ * Installs the DataQueue MCP server config for the selected AI client.
+ *
+ * @param deps - Injectable dependencies for testing.
+ */
+export async function runInstallMcp({
+  log = console.log,
+  error = console.error,
+  exit = (code: number) => process.exit(code),
+  cwd = process.cwd(),
+  readFileSync = fs.readFileSync,
+  writeFileSync = fs.writeFileSync,
+  mkdirSync = fs.mkdirSync,
+  existsSync = fs.existsSync,
+  selectedClient,
+}: InstallMcpDeps = {}): Promise<void> {
+  log('DataQueue MCP Server Installer\n');
+  log('Select your AI client:\n');
+
+  for (const [key, client] of Object.entries(MCP_CLIENTS)) {
+    log(`  ${key}) ${client.label}`);
+  }
+  log('');
+
+  let choice = selectedClient;
+
+  if (!choice) {
+    const rl = readline.createInterface({
+      input: process.stdin,
+      output: process.stdout,
+    });
+
+    choice = await new Promise<string>((resolve) => {
+      rl.question('Enter choice (1-4): ', (answer) => {
+        rl.close();
+        resolve(answer.trim());
+      });
+    });
+  }
+
+  const client = MCP_CLIENTS[choice];
+  if (!client) {
+    error(`Invalid choice: "${choice}". Expected 1-4.`);
+    exit(1);
+    return;
+  }
+
+  log(`\nInstalling MCP config for ${client.label}...`);
+
+  try {
+    client.install({
+      cwd,
+      readFileSync,
+      writeFileSync,
+      mkdirSync,
+      existsSync,
+      log,
+    });
+    log('\nDone! The MCP server will run via: npx dataqueue-cli mcp');
+  } catch (err) {
+    error('Failed to install MCP config:', err);
+    exit(1);
+  }
+}

package/src/install-rules-command.test.ts

@@ -0,0 +1,218 @@
+import { describe, it, expect, vi, afterEach } from 'vitest';
+import {
+  runInstallRules,
+  upsertMarkedSection,
+  InstallRulesDeps,
+} from './install-rules-command.js';
+
+describe('upsertMarkedSection', () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('creates file with markers when file does not exist', () => {
+    // Setup
+    const deps = {
+      existsSync: vi.fn(() => false),
+      readFileSync: vi.fn(),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMarkedSection('/path/file.md', 'content here', deps);
+
+    // Assert
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/path/file.md',
+      expect.stringContaining('<!-- DATAQUEUE RULES START -->'),
+    );
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/path/file.md',
+      expect.stringContaining('content here'),
+    );
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/path/file.md',
+      expect.stringContaining('<!-- DATAQUEUE RULES END -->'),
+    );
+  });
+
+  it('replaces existing marked section', () => {
+    // Setup
+    const existing =
+      'before\n<!-- DATAQUEUE RULES START -->\nold content\n<!-- DATAQUEUE RULES END -->\nafter';
+    const deps = {
+      existsSync: vi.fn(() => true),
+      readFileSync: vi.fn(() => existing),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMarkedSection('/path/file.md', 'new content', deps);
+
+    // Assert
+    const written = deps.writeFileSync.mock.calls[0][1] as string;
+    expect(written).toContain('before\n');
+    expect(written).toContain('new content');
+    expect(written).not.toContain('old content');
+    expect(written).toContain('\nafter');
+  });
+
+  it('appends to file when no markers exist', () => {
+    // Setup
+    const deps = {
+      existsSync: vi.fn(() => true),
+      readFileSync: vi.fn(() => '# Existing content'),
+      writeFileSync: vi.fn(),
+    };
+
+    // Act
+    upsertMarkedSection('/path/file.md', 'new content', deps);
+
+    // Assert
+    const written = deps.writeFileSync.mock.calls[0][1] as string;
+    expect(written).toContain('# Existing content');
+    expect(written).toContain('new content');
+  });
+});
+
+describe('runInstallRules', () => {
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  function makeDeps(
+    overrides: Partial<InstallRulesDeps> = {},
+  ): InstallRulesDeps {
+    return {
+      log: vi.fn(),
+      error: vi.fn(),
+      exit: vi.fn(),
+      cwd: '/project',
+      readFileSync: vi.fn(() => '# Rule content'),
+      writeFileSync: vi.fn(),
+      appendFileSync: vi.fn(),
+      mkdirSync: vi.fn(),
+      existsSync: vi.fn(() => false),
+      rulesSourceDir: '/pkg/ai/rules',
+      ...overrides,
+    };
+  }
+
+  it('installs rules for Cursor (option 1)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '1' });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.mkdirSync).toHaveBeenCalledWith('/project/.cursor/rules', {
+      recursive: true,
+    });
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      expect.stringContaining('.cursor/rules/dataqueue-basic.mdc'),
+      expect.any(String),
+    );
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      expect.stringContaining('.cursor/rules/dataqueue-advanced.mdc'),
+      expect.any(String),
+    );
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      expect.stringContaining('.cursor/rules/dataqueue-react-dashboard.mdc'),
+      expect.any(String),
+    );
+  });
+
+  it('installs rules for Claude Code (option 2)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '2' });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/project/CLAUDE.md',
+      expect.stringContaining('<!-- DATAQUEUE RULES START -->'),
+    );
+  });
+
+  it('installs rules for AGENTS.md (option 3)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '3' });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/project/AGENTS.md',
+      expect.stringContaining('<!-- DATAQUEUE RULES START -->'),
+    );
+  });
+
+  it('installs rules for GitHub Copilot (option 4)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '4' });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.mkdirSync).toHaveBeenCalledWith('/project/.github', {
+      recursive: true,
+    });
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/project/.github/copilot-instructions.md',
+      expect.stringContaining('<!-- DATAQUEUE RULES START -->'),
+    );
+  });
+
+  it('installs rules for Windsurf (option 5)', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '5' });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.writeFileSync).toHaveBeenCalledWith(
+      '/project/CONVENTIONS.md',
+      expect.stringContaining('<!-- DATAQUEUE RULES START -->'),
+    );
+  });
+
+  it('exits with error for invalid choice', async () => {
+    // Setup
+    const deps = makeDeps({ selectedClient: '99' });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.error).toHaveBeenCalledWith(
+      expect.stringContaining('Invalid choice'),
+    );
+    expect(deps.exit).toHaveBeenCalledWith(1);
+  });
+
+  it('handles install errors', async () => {
+    // Setup
+    const deps = makeDeps({
+      selectedClient: '1',
+      readFileSync: vi.fn(() => {
+        throw new Error('file not found');
+      }),
+    });
+
+    // Act
+    await runInstallRules(deps);
+
+    // Assert
+    expect(deps.error).toHaveBeenCalledWith(
+      'Failed to install rules:',
+      expect.any(Error),
+    );
+    expect(deps.exit).toHaveBeenCalledWith(1);
+  });
+});