@luquimbo/bi-superpowers 2.0.1 → 3.0.1

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

@@ -38,8 +38,6 @@ describe('claude-plugin generator', () => {
       const skills = [
         makeSkill('project-kickoff', 'Project Kickoff'),
         makeSkill('pbi-connect', 'PBI Connect'),
-        makeSkill('dax', 'DAX'),
-        makeSkill('excel-formulas', 'Excel Formulas'),
       ];
 
       await claudePlugin.generate(tempDir, skills, {
@@ -52,7 +50,7 @@ describe('claude-plugin generator', () => {
       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', 'dax', 'SKILL.md');
+      const skillPath = path.join(tempDir, 'skills', 'pbi-connect', 'SKILL.md');
 
       assert.ok(fs.existsSync(pluginManifestPath));
       assert.ok(fs.existsSync(pluginMcpPath));
@@ -67,21 +65,21 @@ describe('claude-plugin generator', () => {
       assert.strictEqual(pluginManifest.name, 'bi-superpowers');
       assert.strictEqual(pluginManifest.version, '9.9.9');
 
-      assert.ok(pluginMcp['powerbi-remote']);
-      assert.ok(pluginMcp['fabric-mcp-server']);
+      // 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.strictEqual(pluginMcp['powerbi-remote'].type, 'http');
-      assert.strictEqual(pluginMcp['fabric-mcp-server'].command, 'npx');
+      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('description: "Project analysis and planning"'));
       assert.ok(commandContent.includes('Generated by BI Agent Superpowers'));
-      assert.ok(skillContent.includes('name: "dax"'));
+      assert.ok(skillContent.includes('name: "pbi-connect"'));
       assert.ok(skillContent.includes('version: "9.9.9"'));
     } finally {
       fs.rmSync(tempDir, { recursive: true, force: true });
@@ -92,7 +90,7 @@ describe('claude-plugin generator', () => {
     const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-superpowers-plugin-'));
 
     try {
-      const skills = [makeSkill('dax', 'DAX')];
+      const skills = [makeSkill('pbi-connect', 'PBI Connect')];
 
       await claudePlugin.generate(tempDir, skills, {
         packageDir: '/tmp/bi-superpowers',
@@ -100,7 +98,10 @@ describe('claude-plugin generator', () => {
         libraryPrefix: '.bi-superpowers/library',
       });
 
-      const skillContent = fs.readFileSync(path.join(tempDir, 'skills', 'dax', 'SKILL.md'), 'utf8');
+      const skillContent = fs.readFileSync(
+        path.join(tempDir, 'skills', 'pbi-connect', 'SKILL.md'),
+        'utf8'
+      );
 
       assert.ok(skillContent.includes('.bi-superpowers/library/snippets/example.md'));
     } finally {

package/bin/lib/mcp-config.js

@@ -0,0 +1,287 @@
+/**
+ * Multi-Agent MCP Configuration Writer
+ * =====================================
+ *
+ * Writes the bi-superpowers MCP config (2 servers: powerbi-modeling-mcp
+ * + microsoft-learn) into each supported agent's expected config location
+ * at the user level (home directory).
+ *
+ * Each agent has its own config format and path. This module normalizes
+ * the write operation across all 5 supported agents:
+ *
+ *   | Agent           | Path                                  | Format            |
+ *   |-----------------|---------------------------------------|-------------------|
+ *   | Claude Code     | ~/.claude.json                        | JSON mcpServers   |
+ *   | GitHub Copilot  | ~/.copilot/mcp-config.json            | JSON servers      |
+ *   | Codex           | ~/.codex/config.toml                  | TOML mcp_servers  |
+ *   | Gemini CLI      | ~/.gemini/settings.json               | JSON mcpServers   |
+ *   | Kilo Code       | ~/.kilocode/mcp_settings.json         | JSON mcpServers   |
+ *
+ * The launcher path for the Power BI Modeling MCP is resolved to an
+ * absolute path at install time so it works in agents that don't
+ * support Claude Code's `${CLAUDE_PLUGIN_ROOT}` variable.
+ *
+ * @module lib/mcp-config
+ */
+
+const fs = require('fs');
+const path = require('path');
+const os = require('os');
+
+const MICROSOFT_LEARN_URL = 'https://learn.microsoft.com/api/mcp';
+const MODELING_SERVER_NAME = 'powerbi-modeling';
+const LEARN_SERVER_NAME = 'microsoft-learn';
+
+/**
+ * Resolves the absolute path to the Power BI Modeling MCP launcher
+ * inside the installed npm package.
+ */
+function getLauncherAbsolutePath(packageDir) {
+  return path.join(packageDir, 'bin', 'mcp', 'powerbi-modeling-launcher.js');
+}
+
+/**
+ * Safely read a JSON file. Returns null if missing or unparseable.
+ */
+function readJsonSafe(filePath) {
+  if (!fs.existsSync(filePath)) return null;
+  try {
+    return JSON.parse(fs.readFileSync(filePath, 'utf8'));
+  } catch (_) {
+    return null;
+  }
+}
+
+/**
+ * Reject symlink attacks before writing to a file. We use lstatSync so
+ * we detect the link itself (not what it points to). If a user home has
+ * `~/.claude.json` pointing to `/etc/passwd`, we refuse to write.
+ * @throws {Error} if filePath exists and is a symbolic link
+ */
+function assertNotSymlink(filePath) {
+  if (!fs.existsSync(filePath)) return;
+  if (fs.lstatSync(filePath).isSymbolicLink()) {
+    throw new Error(
+      `Refusing to write MCP config: ${filePath} is a symbolic link. ` +
+        'Remove the symlink and re-run the install.'
+    );
+  }
+}
+
+/**
+ * Write a JSON file, creating parent dirs if needed.
+ * Refuses to overwrite symbolic links for safety.
+ */
+function writeJson(filePath, data) {
+  assertNotSymlink(filePath);
+  const dir = path.dirname(filePath);
+  if (!fs.existsSync(dir)) {
+    fs.mkdirSync(dir, { recursive: true });
+  }
+  fs.writeFileSync(filePath, JSON.stringify(data, null, 2) + '\n');
+}
+
+/**
+ * Escape a string for use inside TOML double-quoted strings.
+ * Covers backslash, quote, and the control characters that TOML spec
+ * requires escaping in basic strings: \b \t \n \f \r
+ *
+ * Note: `\b` in a JS regex is word-boundary, NOT backspace (0x08).
+ * We use \x08 explicitly to match the backspace character itself.
+ */
+function tomlEscape(str) {
+  return (
+    String(str)
+      .replace(/\\/g, '\\\\')
+      .replace(/"/g, '\\"')
+      // eslint-disable-next-line no-control-regex
+      .replace(/\x08/g, '\\b')
+      .replace(/\t/g, '\\t')
+      .replace(/\n/g, '\\n')
+      .replace(/\f/g, '\\f')
+      .replace(/\r/g, '\\r')
+  );
+}
+
+/**
+ * Escape a string for safe use inside a regex pattern (as a literal).
+ */
+function escapeRegex(str) {
+  return String(str).replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
+}
+
+// ============================================
+// CLAUDE CODE
+// ============================================
+// Writes ~/.claude.json, adding to `mcpServers`. Preserves other user-level
+// config in that file (projects, settings, etc.).
+function writeClaudeCodeConfig(packageDir) {
+  const configPath = path.join(os.homedir(), '.claude.json');
+  const existing = readJsonSafe(configPath) || {};
+  const mcpServers = { ...(existing.mcpServers || {}) };
+
+  mcpServers[MODELING_SERVER_NAME] = {
+    command: 'node',
+    args: [getLauncherAbsolutePath(packageDir)],
+  };
+  mcpServers[LEARN_SERVER_NAME] = {
+    type: 'http',
+    url: MICROSOFT_LEARN_URL,
+  };
+
+  writeJson(configPath, { ...existing, mcpServers });
+  return configPath;
+}
+
+// ============================================
+// GITHUB COPILOT CLI
+// ============================================
+// Writes ~/.copilot/mcp-config.json. Copilot uses `servers` (NOT mcpServers)
+// and each server has an explicit `type` field.
+function writeCopilotConfig(packageDir) {
+  const configPath = path.join(os.homedir(), '.copilot', 'mcp-config.json');
+  const existing = readJsonSafe(configPath) || {};
+  const servers = { ...(existing.servers || {}) };
+
+  servers[MODELING_SERVER_NAME] = {
+    type: 'stdio',
+    command: 'node',
+    args: [getLauncherAbsolutePath(packageDir)],
+  };
+  servers[LEARN_SERVER_NAME] = {
+    type: 'http',
+    url: MICROSOFT_LEARN_URL,
+  };
+
+  writeJson(configPath, { ...existing, servers });
+  return configPath;
+}
+
+// ============================================
+// CODEX (OpenAI) — TOML format
+// ============================================
+// Writes ~/.codex/config.toml, appending [mcp_servers.*] sections.
+// Preserves existing content by removing only our own sections before
+// appending the fresh ones.
+function writeCodexConfig(packageDir) {
+  const configPath = path.join(os.homedir(), '.codex', 'config.toml');
+  assertNotSymlink(configPath);
+  const launcher = getLauncherAbsolutePath(packageDir);
+
+  let existing = '';
+  if (fs.existsSync(configPath)) {
+    existing = fs.readFileSync(configPath, 'utf8');
+  }
+
+  // Remove any previous bi-superpowers MCP sections so re-running install
+  // doesn't duplicate them. Strips each section from its header to the
+  // next [ header or EOF. Server names are escaped for regex safety even
+  // though current values are literal — defensive against future renames.
+  const namePattern = [MODELING_SERVER_NAME, LEARN_SERVER_NAME].map(escapeRegex).join('|');
+  const stripPattern = new RegExp(
+    `\\n?\\[mcp_servers\\.(${namePattern})\\][\\s\\S]*?(?=\\n\\[|$)`,
+    'g'
+  );
+  existing = existing.replace(stripPattern, '');
+
+  const newSections =
+    `\n\n[mcp_servers.${MODELING_SERVER_NAME}]\n` +
+    'command = "node"\n' +
+    `args = ["${tomlEscape(launcher)}"]\n` +
+    `\n[mcp_servers.${LEARN_SERVER_NAME}]\n` +
+    `url = "${MICROSOFT_LEARN_URL}"\n`;
+
+  const content = existing.trimEnd() + newSections;
+
+  const dir = path.dirname(configPath);
+  if (!fs.existsSync(dir)) {
+    fs.mkdirSync(dir, { recursive: true });
+  }
+  fs.writeFileSync(configPath, content);
+  return configPath;
+}
+
+// ============================================
+// GEMINI CLI
+// ============================================
+// Writes ~/.gemini/settings.json. Gemini uses `mcpServers` and `httpUrl`
+// (not `url`) for HTTP transports.
+function writeGeminiConfig(packageDir) {
+  const configPath = path.join(os.homedir(), '.gemini', 'settings.json');
+  const existing = readJsonSafe(configPath) || {};
+  const mcpServers = { ...(existing.mcpServers || {}) };
+
+  mcpServers[MODELING_SERVER_NAME] = {
+    command: 'node',
+    args: [getLauncherAbsolutePath(packageDir)],
+  };
+  mcpServers[LEARN_SERVER_NAME] = {
+    httpUrl: MICROSOFT_LEARN_URL,
+  };
+
+  writeJson(configPath, { ...existing, mcpServers });
+  return configPath;
+}
+
+// ============================================
+// KILO CODE
+// ============================================
+// Writes ~/.kilocode/mcp_settings.json. Kilo uses `mcpServers` and `url`
+// for HTTP transports.
+function writeKiloConfig(packageDir) {
+  const configPath = path.join(os.homedir(), '.kilocode', 'mcp_settings.json');
+  const existing = readJsonSafe(configPath) || {};
+  const mcpServers = { ...(existing.mcpServers || {}) };
+
+  mcpServers[MODELING_SERVER_NAME] = {
+    command: 'node',
+    args: [getLauncherAbsolutePath(packageDir)],
+  };
+  mcpServers[LEARN_SERVER_NAME] = {
+    url: MICROSOFT_LEARN_URL,
+  };
+
+  writeJson(configPath, { ...existing, mcpServers });
+  return configPath;
+}
+
+/**
+ * Registry mapping agent IDs to their MCP config writers.
+ */
+const MCP_WRITERS = {
+  'claude-code': writeClaudeCodeConfig,
+  'github-copilot': writeCopilotConfig,
+  codex: writeCodexConfig,
+  'gemini-cli': writeGeminiConfig,
+  kilo: writeKiloConfig,
+};
+
+/**
+ * Write MCP config for a specific agent.
+ * @param {string} agentId - One of the supported agent IDs
+ * @param {string} packageDir - Absolute path to the installed package
+ * @returns {string|null} The config path written, or null if unknown agent
+ * @throws {Error} If the write fails
+ */
+function writeMcpConfigForAgent(agentId, packageDir) {
+  const writer = MCP_WRITERS[agentId];
+  if (!writer) {
+    return null;
+  }
+  return writer(packageDir);
+}
+
+module.exports = {
+  writeMcpConfigForAgent,
+  MCP_WRITERS,
+  MODELING_SERVER_NAME,
+  LEARN_SERVER_NAME,
+  MICROSOFT_LEARN_URL,
+  // Exported for testing
+  readJsonSafe,
+  writeJson,
+  tomlEscape,
+  escapeRegex,
+  assertNotSymlink,
+  getLauncherAbsolutePath,
+};

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

@@ -0,0 +1,273 @@
+/**
+ * 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, '.kilocode', '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}`);
+    });
+  }
+
+  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.js

@@ -2,21 +2,20 @@
  * Official Microsoft MCP Configuration Helpers
  * ============================================
  *
- * Single source of truth for the Claude Code plugin and legacy adapter
- * configurations that point at the official Microsoft Power BI / Fabric
- * MCP servers.
+ * Single source of truth for the MCP servers that bi-superpowers ships.
+ * Currently ships 2 servers, both from Microsoft:
+ *
+ * - powerbi-modeling-mcp: local stdio launcher that talks to Power BI Desktop
+ *   via XMLA on localhost. Requires Power BI Desktop running with a model open.
+ * - microsoft-learn: HTTP MCP that pipes Microsoft Learn docs into the agent.
  *
  * @module lib/microsoft-mcp
  */
 
 const path = require('path');
 
-const REMOTE_POWERBI_URL = 'https://api.fabric.microsoft.com/v1/mcp/powerbi';
-const FABRIC_MCP_PACKAGE = '@microsoft/fabric-mcp@latest';
 const MICROSOFT_LEARN_URL = 'https://learn.microsoft.com/api/mcp';
 const MODELING_SERVER_NAME = 'powerbi-modeling-mcp';
-const REMOTE_SERVER_NAME = 'powerbi-remote';
-const FABRIC_SERVER_NAME = 'fabric-mcp-server';
 const LEARN_SERVER_NAME = 'microsoft-learn';
 const ABSOLUTE_LAUNCHER_MODE = 'absolute';
 const PLUGIN_ROOT_LAUNCHER_MODE = 'plugin-root';
@@ -51,15 +50,6 @@ function createPluginMcpConfig(options = {}) {
   const launcherPath = resolveModelingLauncherPath(options);
 
   return {
-    [REMOTE_SERVER_NAME]: {
-      type: 'http',
-      url: REMOTE_POWERBI_URL,
-    },
-    [FABRIC_SERVER_NAME]: {
-      type: 'stdio',
-      command: 'npx',
-      args: ['-y', FABRIC_MCP_PACKAGE, 'server', 'start', '--mode', 'all'],
-    },
     [MODELING_SERVER_NAME]: {
       type: 'stdio',
       command: 'node',
@@ -170,12 +160,8 @@ function mergeMcpConfig(existingConfig, newConfig, format) {
 module.exports = {
   ABSOLUTE_LAUNCHER_MODE,
   PLUGIN_ROOT_LAUNCHER_MODE,
-  REMOTE_POWERBI_URL,
-  FABRIC_MCP_PACKAGE,
   MICROSOFT_LEARN_URL,
   MODELING_SERVER_NAME,
-  REMOTE_SERVER_NAME,
-  FABRIC_SERVER_NAME,
   LEARN_SERVER_NAME,
   resolveModelingLauncherPath,
   createPluginMcpConfig,