@luquimbo/bi-superpowers 3.2.0 → 4.1.1

package/bin/commands/lint.test.js

@@ -1,103 +0,0 @@
-/**
- * Unit tests for the lint command (checkup)
- *
- * Run with: npm test
- */
-
-const { test, describe } = require('node:test');
-const assert = require('node:assert');
-const fs = require('fs');
-const path = require('path');
-
-// Mock skill content for testing
-const validSkillContent = `# Test Skill
-
-## Trigger
-Activate this skill when user mentions:
-- "test keyword"
-- "another trigger"
-
-## Identity
-You are a **Test Expert** who helps users with testing.
-
-## MANDATORY RULES
-1. **ALWAYS TEST FIRST.** Never skip validation.
-2. **USE ASSERTIONS.** Verify expected outcomes.
-
----
-
-## PHASE 0: Initial Assessment
-
-Start with basic questions.
-
-\`\`\`dax
-// Example DAX
-TestMeasure = SUM(Table[Column])
-\`\`\`
-`;
-
-const invalidSkillContent = `# Missing Sections
-
-This skill is missing required sections like Trigger and MANDATORY RULES.
-
-Some content here but no proper structure.
-`;
-
-describe('Lint Command', () => {
-  test('valid skill content should have required sections', () => {
-    const hasTrigger = /##\s+Trigger/i.test(validSkillContent);
-    const hasIdentity = /##\s+Identity/i.test(validSkillContent);
-    const hasMandatoryRules = /##\s+MANDATORY RULES/i.test(validSkillContent);
-
-    assert.strictEqual(hasTrigger, true, 'Should have Trigger section');
-    assert.strictEqual(hasIdentity, true, 'Should have Identity section');
-    assert.strictEqual(hasMandatoryRules, true, 'Should have MANDATORY RULES section');
-  });
-
-  test('invalid skill content should be missing required sections', () => {
-    const hasTrigger = /##\s+Trigger/i.test(invalidSkillContent);
-    const hasMandatoryRules = /##\s+MANDATORY RULES/i.test(invalidSkillContent);
-
-    assert.strictEqual(hasTrigger, false, 'Should not have Trigger section');
-    assert.strictEqual(hasMandatoryRules, false, 'Should not have MANDATORY RULES section');
-  });
-
-  test('skill should have H1 title', () => {
-    const hasH1 = /^#\s+.+/m.test(validSkillContent);
-    assert.strictEqual(hasH1, true, 'Should have H1 title');
-  });
-
-  test('code blocks should have language specifier', () => {
-    // Check for code blocks with language
-    const codeBlockWithLang = /```\w+/;
-    const hasLanguage = codeBlockWithLang.test(validSkillContent);
-    assert.strictEqual(hasLanguage, true, 'Code blocks should have language specifier');
-  });
-
-  test('trigger section should have quoted phrases', () => {
-    const triggerSection = validSkillContent.match(/##\s+Trigger[\s\S]*?(?=##|$)/i);
-    assert.ok(triggerSection, 'Should have Trigger section');
-
-    const hasQuotedPhrases = /[-*]\s+["'].+["']/.test(triggerSection[0]);
-    assert.strictEqual(hasQuotedPhrases, true, 'Trigger should have quoted phrases');
-  });
-});
-
-describe('File naming convention', () => {
-  test('kebab-case pattern should match valid names', () => {
-    const pattern = /^[a-z0-9-]+\.md$/;
-
-    assert.strictEqual(pattern.test('dax.md'), true);
-    assert.strictEqual(pattern.test('power-query.md'), true);
-    assert.strictEqual(pattern.test('data-model-design.md'), true);
-  });
-
-  test('kebab-case pattern should reject invalid names', () => {
-    const pattern = /^[a-z0-9-]+\.md$/;
-
-    assert.strictEqual(pattern.test('DAX.md'), false);
-    assert.strictEqual(pattern.test('PowerQuery.md'), false);
-    assert.strictEqual(pattern.test('my_skill.md'), false);
-    assert.strictEqual(pattern.test('skill.txt'), false);
-  });
-});

package/bin/lib/generators/claude-plugin.test.js

@@ -1,111 +0,0 @@
-/**
- * Tests for the native Claude Code plugin generator.
- */
-
-const { test, describe } = require('node:test');
-const assert = require('node:assert');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-const claudePlugin = require('./claude-plugin');
-
-function makeSkill(name, title) {
-  return {
-    name,
-    path: `/virtual/${name}.md`,
-    content: `# ${title}
-
-## Trigger
-- "${name}"
-
-## Identity
-You are the ${title} guide.
-
-## MANDATORY RULES
-1. Stay helpful.
-
-Reference: library/snippets/example.md
-`,
-  };
-}
-
-describe('claude-plugin generator', () => {
-  test('creates plugin manifest, mcp config, commands, and skills', async () => {
-    const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-superpowers-plugin-'));
-
-    try {
-      const skills = [
-        makeSkill('project-kickoff', 'Project Kickoff'),
-        makeSkill('pbi-connect', 'PBI Connect'),
-      ];
-
-      await claudePlugin.generate(tempDir, skills, {
-        packageDir: '/tmp/bi-superpowers',
-        version: '9.9.9',
-        usePluginRootLauncher: true,
-        libraryPrefix: 'library',
-      });
-
-      const pluginManifestPath = path.join(tempDir, '.claude-plugin', 'plugin.json');
-      const pluginMcpPath = path.join(tempDir, '.mcp.json');
-      const commandPath = path.join(tempDir, 'commands', 'project-kickoff.md');
-      const skillPath = path.join(tempDir, 'skills', 'pbi-connect', 'SKILL.md');
-
-      assert.ok(fs.existsSync(pluginManifestPath));
-      assert.ok(fs.existsSync(pluginMcpPath));
-      assert.ok(fs.existsSync(commandPath));
-      assert.ok(fs.existsSync(skillPath));
-
-      const pluginManifest = JSON.parse(fs.readFileSync(pluginManifestPath, 'utf8'));
-      const pluginMcp = JSON.parse(fs.readFileSync(pluginMcpPath, 'utf8'));
-      const commandContent = fs.readFileSync(commandPath, 'utf8');
-      const skillContent = fs.readFileSync(skillPath, 'utf8');
-
-      assert.strictEqual(pluginManifest.name, 'bi-superpowers');
-      assert.strictEqual(pluginManifest.version, '9.9.9');
-
-      // Only 2 MCPs ship: local Power BI Modeling and Microsoft Learn
-      assert.strictEqual(Object.keys(pluginMcp).length, 2);
-      assert.ok(pluginMcp['powerbi-modeling-mcp']);
-      assert.ok(pluginMcp['microsoft-learn']);
-      assert.strictEqual(pluginMcp['powerbi-modeling-mcp'].command, 'node');
-      assert.ok(
-        pluginMcp['powerbi-modeling-mcp'].args[0].includes(
-          '${CLAUDE_PLUGIN_ROOT}/bin/mcp/powerbi-modeling-launcher.js'
-        )
-      );
-      assert.strictEqual(pluginMcp['microsoft-learn'].type, 'http');
-      assert.strictEqual(pluginMcp['microsoft-learn'].url, 'https://learn.microsoft.com/api/mcp');
-
-      assert.ok(commandContent.includes('Generated by BI Agent Superpowers'));
-      assert.ok(skillContent.includes('name: "pbi-connect"'));
-      assert.ok(skillContent.includes('version: "9.9.9"'));
-    } finally {
-      fs.rmSync(tempDir, { recursive: true, force: true });
-    }
-  });
-
-  test('rewrites library references for generated project outputs', async () => {
-    const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-superpowers-plugin-'));
-
-    try {
-      const skills = [makeSkill('pbi-connect', 'PBI Connect')];
-
-      await claudePlugin.generate(tempDir, skills, {
-        packageDir: '/tmp/bi-superpowers',
-        version: '1.0.0',
-        libraryPrefix: '.bi-superpowers/library',
-      });
-
-      const skillContent = fs.readFileSync(
-        path.join(tempDir, 'skills', 'pbi-connect', 'SKILL.md'),
-        'utf8'
-      );
-
-      assert.ok(skillContent.includes('.bi-superpowers/library/snippets/example.md'));
-    } finally {
-      fs.rmSync(tempDir, { recursive: true, force: true });
-    }
-  });
-});

package/bin/lib/mcp-config.test.js

@@ -1,310 +0,0 @@
-/**
- * Tests for the multi-agent MCP config writer.
- */
-
-const { test, describe, beforeEach, afterEach } = require('node:test');
-const assert = require('node:assert');
-const fs = require('node:fs');
-const os = require('node:os');
-const path = require('node:path');
-
-const {
-  MCP_WRITERS,
-  MODELING_SERVER_NAME,
-  LEARN_SERVER_NAME,
-  MICROSOFT_LEARN_URL,
-  tomlEscape,
-  escapeRegex,
-  assertNotSymlink,
-  getLauncherAbsolutePath,
-} = require('./mcp-config');
-
-describe('mcp-config helpers', () => {
-  test('getLauncherAbsolutePath returns absolute launcher path', () => {
-    const pkgDir = '/tmp/fake-pkg';
-    const launcher = getLauncherAbsolutePath(pkgDir);
-    assert.strictEqual(launcher, path.join(pkgDir, 'bin', 'mcp', 'powerbi-modeling-launcher.js'));
-  });
-
-  test('tomlEscape escapes backslashes and quotes', () => {
-    assert.strictEqual(tomlEscape('plain'), 'plain');
-    assert.strictEqual(tomlEscape('C:\\path\\to\\file'), 'C:\\\\path\\\\to\\\\file');
-    assert.strictEqual(tomlEscape('say "hello"'), 'say \\"hello\\"');
-  });
-
-  test('tomlEscape escapes TOML control characters', () => {
-    assert.strictEqual(tomlEscape('line1\nline2'), 'line1\\nline2');
-    assert.strictEqual(tomlEscape('tab\there'), 'tab\\there');
-    assert.strictEqual(tomlEscape('cr\rhere'), 'cr\\rhere');
-    // \x08 = backspace, \f = form feed
-    assert.strictEqual(tomlEscape('\x08\f'), '\\b\\f');
-  });
-
-  test('escapeRegex escapes all regex metacharacters', () => {
-    assert.strictEqual(escapeRegex('plain'), 'plain');
-    assert.strictEqual(escapeRegex('a.b'), 'a\\.b');
-    assert.strictEqual(escapeRegex('a[b]c'), 'a\\[b\\]c');
-    assert.strictEqual(escapeRegex('a+b*c?'), 'a\\+b\\*c\\?');
-    assert.strictEqual(escapeRegex('(hello)'), '\\(hello\\)');
-  });
-});
-
-describe('assertNotSymlink', () => {
-  let tempDir;
-
-  beforeEach(() => {
-    tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-mcp-symlink-'));
-  });
-
-  afterEach(() => {
-    fs.rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  test('allows non-existent paths', () => {
-    assert.doesNotThrow(() => {
-      assertNotSymlink(path.join(tempDir, 'nope.json'));
-    });
-  });
-
-  test('allows regular files', () => {
-    const file = path.join(tempDir, 'regular.json');
-    fs.writeFileSync(file, '{}');
-    assert.doesNotThrow(() => assertNotSymlink(file));
-  });
-
-  test('throws on symbolic links', () => {
-    const target = path.join(tempDir, 'target.json');
-    const link = path.join(tempDir, 'link.json');
-    fs.writeFileSync(target, '{}');
-    fs.symlinkSync(target, link);
-
-    assert.throws(() => assertNotSymlink(link), /is a symbolic link/);
-  });
-});
-
-describe('MCP_WRITERS registry', () => {
-  test('covers all 5 supported agents', () => {
-    const ids = Object.keys(MCP_WRITERS).sort();
-    assert.deepStrictEqual(ids, ['claude-code', 'codex', 'gemini-cli', 'github-copilot', 'kilo']);
-  });
-
-  test('each writer is a function', () => {
-    for (const [id, writer] of Object.entries(MCP_WRITERS)) {
-      assert.strictEqual(typeof writer, 'function', `${id} writer should be a function`);
-    }
-  });
-});
-
-describe('MCP writers — JSON agents', () => {
-  let tempHome;
-  let origHome;
-  const fakePkgDir = '/tmp/fake-bi-superpowers';
-
-  beforeEach(() => {
-    tempHome = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-mcp-home-'));
-    origHome = os.homedir;
-    os.homedir = () => tempHome;
-  });
-
-  afterEach(() => {
-    os.homedir = origHome;
-    fs.rmSync(tempHome, { recursive: true, force: true });
-  });
-
-  test('claude-code writer produces mcpServers JSON at ~/.claude.json', () => {
-    const configPath = MCP_WRITERS['claude-code'](fakePkgDir);
-
-    assert.strictEqual(configPath, path.join(tempHome, '.claude.json'));
-    assert.ok(fs.existsSync(configPath));
-
-    const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    assert.ok(config.mcpServers);
-    assert.ok(config.mcpServers[MODELING_SERVER_NAME]);
-    assert.strictEqual(config.mcpServers[MODELING_SERVER_NAME].command, 'node');
-    assert.ok(
-      config.mcpServers[MODELING_SERVER_NAME].args[0].includes('powerbi-modeling-launcher.js')
-    );
-    assert.deepStrictEqual(config.mcpServers[LEARN_SERVER_NAME], {
-      type: 'http',
-      url: MICROSOFT_LEARN_URL,
-    });
-  });
-
-  test('github-copilot writer uses "servers" key (not mcpServers)', () => {
-    const configPath = MCP_WRITERS['github-copilot'](fakePkgDir);
-
-    assert.strictEqual(configPath, path.join(tempHome, '.copilot', 'mcp-config.json'));
-    const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    assert.ok(config.servers);
-    assert.strictEqual(config.mcpServers, undefined);
-    assert.strictEqual(config.servers[MODELING_SERVER_NAME].type, 'stdio');
-    assert.strictEqual(config.servers[LEARN_SERVER_NAME].type, 'http');
-  });
-
-  test('gemini-cli writer uses httpUrl (not url)', () => {
-    const configPath = MCP_WRITERS['gemini-cli'](fakePkgDir);
-
-    assert.strictEqual(configPath, path.join(tempHome, '.gemini', 'settings.json'));
-    const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    assert.ok(config.mcpServers);
-    assert.strictEqual(config.mcpServers[LEARN_SERVER_NAME].httpUrl, MICROSOFT_LEARN_URL);
-    assert.strictEqual(config.mcpServers[LEARN_SERVER_NAME].url, undefined);
-  });
-
-  test('kilo writer uses url (standard)', () => {
-    const configPath = MCP_WRITERS.kilo(fakePkgDir);
-
-    assert.strictEqual(configPath, path.join(tempHome, '.kilo', 'mcp_settings.json'));
-    const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    assert.ok(config.mcpServers);
-    assert.strictEqual(config.mcpServers[LEARN_SERVER_NAME].url, MICROSOFT_LEARN_URL);
-  });
-
-  test('claude-code writer preserves existing config', () => {
-    const configPath = path.join(tempHome, '.claude.json');
-    fs.writeFileSync(
-      configPath,
-      JSON.stringify({
-        otherSetting: 'value',
-        mcpServers: { 'existing-mcp': { command: 'echo' } },
-      })
-    );
-
-    MCP_WRITERS['claude-code'](fakePkgDir);
-
-    const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-    assert.strictEqual(config.otherSetting, 'value');
-    assert.ok(config.mcpServers['existing-mcp']);
-    assert.ok(config.mcpServers[MODELING_SERVER_NAME]);
-    assert.ok(config.mcpServers[LEARN_SERVER_NAME]);
-  });
-
-  // Re-install de-duplication for JSON writers — each should replace its
-  // own entries cleanly without duplicating them on repeated runs.
-  for (const agentId of ['claude-code', 'github-copilot', 'gemini-cli', 'kilo']) {
-    test(`${agentId} re-install replaces entries without duplicating`, () => {
-      // First install
-      MCP_WRITERS[agentId](fakePkgDir);
-      // Second install (same args)
-      const configPath = MCP_WRITERS[agentId](fakePkgDir);
-
-      const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-      const serversKey = agentId === 'github-copilot' ? 'servers' : 'mcpServers';
-      const servers = config[serversKey];
-
-      // Our 2 servers should each appear exactly once (via object keys)
-      assert.ok(servers[MODELING_SERVER_NAME], 'modeling server missing after re-install');
-      assert.ok(servers[LEARN_SERVER_NAME], 'learn server missing after re-install');
-
-      // And no stray duplicates of our keys
-      const ourKeys = Object.keys(servers).filter(
-        (k) => k === MODELING_SERVER_NAME || k === LEARN_SERVER_NAME
-      );
-      assert.strictEqual(ourKeys.length, 2, `expected 2 of our keys, found ${ourKeys.length}`);
-    });
-  }
-
-  // Cross-agent sanity check: verifies each JSON agent writes the HTTP
-  // URL under the correct key per its spec. Catches copy-paste errors
-  // where someone might accidentally use `url` for Gemini (which expects
-  // `httpUrl`) or vice versa.
-  test('each JSON agent uses the correct HTTP field name', () => {
-    const expectations = {
-      'claude-code': { key: 'url', wrapperKey: 'mcpServers' },
-      'github-copilot': { key: 'url', wrapperKey: 'servers' },
-      'gemini-cli': { key: 'httpUrl', wrapperKey: 'mcpServers' },
-      kilo: { key: 'url', wrapperKey: 'mcpServers' },
-    };
-
-    for (const [agentId, expected] of Object.entries(expectations)) {
-      // Reset home for each agent so configs don't leak between writes.
-      fs.rmSync(tempHome, { recursive: true, force: true });
-      fs.mkdirSync(tempHome, { recursive: true });
-
-      const configPath = MCP_WRITERS[agentId](fakePkgDir);
-      const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-      const learnEntry = config[expected.wrapperKey][LEARN_SERVER_NAME];
-
-      assert.strictEqual(
-        learnEntry[expected.key],
-        MICROSOFT_LEARN_URL,
-        `${agentId} should use "${expected.key}" for HTTP URL`
-      );
-
-      // And it should NOT use the wrong key (defensive check)
-      const wrongKey = expected.key === 'url' ? 'httpUrl' : 'url';
-      assert.strictEqual(
-        learnEntry[wrongKey],
-        undefined,
-        `${agentId} should NOT have "${wrongKey}"`
-      );
-    }
-  });
-
-  test('writeJson refuses to overwrite symlinked target', () => {
-    const realFile = path.join(tempHome, 'real-claude.json');
-    const linkFile = path.join(tempHome, '.claude.json');
-    fs.writeFileSync(realFile, '{"mcpServers":{}}');
-    fs.symlinkSync(realFile, linkFile);
-
-    assert.throws(() => MCP_WRITERS['claude-code'](fakePkgDir), /symbolic link/);
-
-    // The real file content must not have been touched.
-    const content = JSON.parse(fs.readFileSync(realFile, 'utf8'));
-    assert.deepStrictEqual(content, { mcpServers: {} });
-  });
-});
-
-describe('codex writer — TOML', () => {
-  let tempHome;
-  let origHome;
-
-  beforeEach(() => {
-    tempHome = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-mcp-codex-'));
-    origHome = os.homedir;
-    os.homedir = () => tempHome;
-  });
-
-  afterEach(() => {
-    os.homedir = origHome;
-    fs.rmSync(tempHome, { recursive: true, force: true });
-  });
-
-  test('writes TOML with [mcp_servers.*] sections', () => {
-    const configPath = MCP_WRITERS.codex('/tmp/fake-pkg');
-
-    assert.strictEqual(configPath, path.join(tempHome, '.codex', 'config.toml'));
-    const content = fs.readFileSync(configPath, 'utf8');
-
-    assert.ok(content.includes('[mcp_servers.powerbi-modeling]'));
-    assert.ok(content.includes('command = "node"'));
-    assert.ok(content.includes('[mcp_servers.microsoft-learn]'));
-    assert.ok(content.includes(`url = "${MICROSOFT_LEARN_URL}"`));
-  });
-
-  test('preserves existing non-MCP TOML content', () => {
-    const configPath = path.join(tempHome, '.codex', 'config.toml');
-    fs.mkdirSync(path.dirname(configPath), { recursive: true });
-    fs.writeFileSync(configPath, '[profile]\nname = "default"\n');
-
-    MCP_WRITERS.codex('/tmp/fake-pkg');
-
-    const content = fs.readFileSync(configPath, 'utf8');
-    assert.ok(content.includes('[profile]'));
-    assert.ok(content.includes('name = "default"'));
-    assert.ok(content.includes('[mcp_servers.powerbi-modeling]'));
-  });
-
-  test('re-running install replaces MCP sections, does not duplicate', () => {
-    MCP_WRITERS.codex('/tmp/fake-pkg');
-    MCP_WRITERS.codex('/tmp/fake-pkg');
-
-    const content = fs.readFileSync(path.join(tempHome, '.codex', 'config.toml'), 'utf8');
-
-    // Each section name should appear exactly once
-    const modelingCount = (content.match(/\[mcp_servers\.powerbi-modeling\]/g) || []).length;
-    const learnCount = (content.match(/\[mcp_servers\.microsoft-learn\]/g) || []).length;
-    assert.strictEqual(modelingCount, 1);
-    assert.strictEqual(learnCount, 1);
-  });
-});

package/bin/lib/microsoft-mcp.test.js

@@ -1,115 +0,0 @@
-/**
- * Tests for official Microsoft MCP config helpers.
- */
-
-const { test, describe } = require('node:test');
-const assert = require('node:assert');
-const path = require('node:path');
-
-const {
-  ABSOLUTE_LAUNCHER_MODE,
-  PLUGIN_ROOT_LAUNCHER_MODE,
-  MICROSOFT_LEARN_URL,
-  createPluginMcpConfig,
-  createMcpConfigForFormat,
-  mergeMcpConfig,
-  resolveModelingLauncherPath,
-} = require('./microsoft-mcp');
-
-describe('resolveModelingLauncherPath', () => {
-  test('uses plugin root placeholder for plugin builds', () => {
-    const launcherPath = resolveModelingLauncherPath({
-      launcherMode: PLUGIN_ROOT_LAUNCHER_MODE,
-    });
-
-    assert.strictEqual(launcherPath, '${CLAUDE_PLUGIN_ROOT}/bin/mcp/powerbi-modeling-launcher.js');
-  });
-
-  test('uses absolute package path for generated project configs', () => {
-    const launcherPath = resolveModelingLauncherPath({
-      packageDir: '/tmp/bi-superpowers',
-      launcherMode: ABSOLUTE_LAUNCHER_MODE,
-    });
-
-    assert.strictEqual(
-      launcherPath,
-      path.join('/tmp/bi-superpowers', 'bin', 'mcp', 'powerbi-modeling-launcher.js')
-    );
-  });
-});
-
-describe('createPluginMcpConfig', () => {
-  test('returns the 2 official Microsoft MCP defaults', () => {
-    const config = createPluginMcpConfig({
-      packageDir: '/tmp/bi-superpowers',
-      launcherMode: ABSOLUTE_LAUNCHER_MODE,
-    });
-
-    // Should have exactly 2 servers
-    assert.strictEqual(Object.keys(config).length, 2);
-
-    // powerbi-modeling-mcp: local stdio launcher
-    assert.strictEqual(config['powerbi-modeling-mcp'].type, 'stdio');
-    assert.strictEqual(config['powerbi-modeling-mcp'].command, 'node');
-    assert.ok(config['powerbi-modeling-mcp'].args[0].endsWith('powerbi-modeling-launcher.js'));
-
-    // microsoft-learn: HTTP MCP
-    assert.deepStrictEqual(config['microsoft-learn'], {
-      type: 'http',
-      url: MICROSOFT_LEARN_URL,
-    });
-  });
-
-  test('does not include the old removed servers', () => {
-    const config = createPluginMcpConfig({
-      packageDir: '/tmp/bi-superpowers',
-      launcherMode: ABSOLUTE_LAUNCHER_MODE,
-    });
-
-    assert.strictEqual(config['powerbi-remote'], undefined);
-    assert.strictEqual(config['fabric-mcp-server'], undefined);
-  });
-});
-
-describe('createMcpConfigForFormat', () => {
-  test('returns flat plugin config for plugin/cursor outputs', () => {
-    const pluginConfig = createMcpConfigForFormat('plugin');
-    const cursorConfig = createMcpConfigForFormat('cursor');
-
-    assert.ok(pluginConfig['powerbi-modeling-mcp']);
-    assert.ok(cursorConfig['microsoft-learn']);
-    assert.strictEqual(pluginConfig['microsoft-learn'].type, 'http');
-  });
-
-  test('returns nested mcpServers config for claude-like outputs', () => {
-    const config = createMcpConfigForFormat('claude');
-
-    assert.ok(config.mcpServers);
-    assert.ok(config.mcpServers['powerbi-modeling-mcp']);
-    assert.ok(config.mcpServers['microsoft-learn']);
-  });
-});
-
-describe('mergeMcpConfig', () => {
-  test('merges plugin configs without dropping existing entries', () => {
-    const merged = mergeMcpConfig(
-      { existing: { type: 'http', url: 'https://example.com' } },
-      { 'microsoft-learn': { type: 'http', url: MICROSOFT_LEARN_URL } },
-      'plugin'
-    );
-
-    assert.ok(merged.existing);
-    assert.ok(merged['microsoft-learn']);
-  });
-
-  test('merges claude configs inside mcpServers', () => {
-    const merged = mergeMcpConfig(
-      { mcpServers: { existing: { type: 'http', url: 'https://example.com' } } },
-      { mcpServers: { 'microsoft-learn': { type: 'http', url: MICROSOFT_LEARN_URL } } },
-      'claude'
-    );
-
-    assert.ok(merged.mcpServers.existing);
-    assert.ok(merged.mcpServers['microsoft-learn']);
-  });
-});