@luquimbo/bi-superpowers 2.0.1 → 3.0.0

package/bin/utils/mcp-detect.js

@@ -12,11 +12,7 @@ const fs = require('fs');
 const path = require('path');
 const os = require('os');
 const { spawnSync } = require('child_process');
-const {
-  PLUGIN_ROOT_LAUNCHER_MODE,
-  REMOTE_POWERBI_URL,
-  createMcpConfigForFormat,
-} = require('../lib/microsoft-mcp');
+const { PLUGIN_ROOT_LAUNCHER_MODE, createMcpConfigForFormat } = require('../lib/microsoft-mcp');
 
 /**
  * Common installation paths for the Power BI Modeling MCP
@@ -159,8 +155,8 @@ function getMcpVersion(exePath) {
 }
 
 /**
- * Get full MCP status
- * @returns {Object} Status object with local and remote info
+ * Get full MCP status for the local Power BI Modeling MCP launcher.
+ * @returns {Object} Status object
  */
 function getMcpStatus() {
   const localPath = findLocalMcp();
@@ -174,17 +170,6 @@ function getMcpStatus() {
       available: os.platform() === 'win32',
       envVars: MODELING_MCP_ENV_VARS,
     },
-    remote: {
-      available: true, // Always available (requires auth)
-      url: REMOTE_POWERBI_URL,
-      requiresAuth: true,
-      description: 'Power BI Remote MCP (Fabric)',
-    },
-    fabric: {
-      available: true,
-      package: '@microsoft/fabric-mcp@latest',
-      description: 'Microsoft Fabric MCP Server',
-    },
   };
 }
 
@@ -235,7 +220,7 @@ function getModelingMcpError() {
   if (os.platform() !== 'win32') {
     return [
       'Power BI Modeling MCP is only available on Windows.',
-      'You can still use the official powerbi-remote and fabric-mcp-server entries on macOS/Linux.',
+      'You still get the microsoft-learn MCP on macOS/Linux.',
       'If you are on Windows and extracted the server manually, set BI_SUPERPOWERS_POWERBI_MODELING_MCP_PATH.',
     ].join('\n');
   }
@@ -270,5 +255,4 @@ module.exports = {
   getExplicitMcpPath,
   getModelingMcpError,
   MODELING_MCP_ENV_VARS,
-  REMOTE_POWERBI_URL,
 };

package/bin/utils/mcp-detect.test.js

@@ -19,38 +19,23 @@ describe('MCP Detection Utilities', () => {
     assert.strictEqual(typeof mcpDetect.getModelingMcpError, 'function');
   });
 
-  test('exports official remote url and env vars', () => {
-    assert.strictEqual(typeof mcpDetect.REMOTE_POWERBI_URL, 'string');
-    assert.strictEqual(
-      mcpDetect.REMOTE_POWERBI_URL,
-      'https://api.fabric.microsoft.com/v1/mcp/powerbi'
-    );
+  test('exports env vars for local MCP path override', () => {
     assert.ok(Array.isArray(mcpDetect.MODELING_MCP_ENV_VARS));
     assert.ok(mcpDetect.MODELING_MCP_ENV_VARS.includes('BI_SUPERPOWERS_POWERBI_MODELING_MCP_PATH'));
   });
 });
 
 describe('getMcpStatus', () => {
-  test('returns expected structure', () => {
+  test('returns expected structure for the local MCP only', () => {
     const status = mcpDetect.getMcpStatus();
 
     assert.ok(status.local);
-    assert.ok(status.remote);
-    assert.ok(status.fabric);
-
     assert.ok('installed' in status.local);
     assert.ok('path' in status.local);
     assert.ok('version' in status.local);
     assert.ok('platform' in status.local);
     assert.ok('available' in status.local);
     assert.ok(Array.isArray(status.local.envVars));
-
-    assert.strictEqual(status.remote.available, true);
-    assert.strictEqual(status.remote.url, mcpDetect.REMOTE_POWERBI_URL);
-    assert.strictEqual(status.remote.requiresAuth, true);
-
-    assert.strictEqual(status.fabric.available, true);
-    assert.strictEqual(status.fabric.package, '@microsoft/fabric-mcp@latest');
   });
 });
 
@@ -71,35 +56,26 @@ describe('getInstallInstructions', () => {
 });
 
 describe('generateMcpConfig', () => {
-  test('generates plugin format with official Microsoft MCPs', () => {
+  test('generates plugin format with 2 official Microsoft MCPs', () => {
     const config = mcpDetect.generateMcpConfig({}, 'plugin');
 
-    assert.ok(config['powerbi-remote']);
-    assert.ok(config['fabric-mcp-server']);
     assert.ok(config['powerbi-modeling-mcp']);
+    assert.ok(config['microsoft-learn']);
+    assert.strictEqual(Object.keys(config).length, 2);
 
-    assert.strictEqual(config['powerbi-remote'].type, 'http');
-    assert.strictEqual(config['powerbi-remote'].url, mcpDetect.REMOTE_POWERBI_URL);
-    assert.strictEqual(config['fabric-mcp-server'].command, 'npx');
-    assert.deepStrictEqual(config['fabric-mcp-server'].args, [
-      '-y',
-      '@microsoft/fabric-mcp@latest',
-      'server',
-      'start',
-      '--mode',
-      'all',
-    ]);
     assert.strictEqual(config['powerbi-modeling-mcp'].type, 'stdio');
     assert.strictEqual(config['powerbi-modeling-mcp'].command, 'node');
     assert.ok(config['powerbi-modeling-mcp'].args[0].includes('powerbi-modeling-launcher.js'));
+
+    assert.strictEqual(config['microsoft-learn'].type, 'http');
+    assert.strictEqual(config['microsoft-learn'].url, 'https://learn.microsoft.com/api/mcp');
   });
 
   test('generates claude format with nested mcpServers object', () => {
     const config = mcpDetect.generateMcpConfig({}, 'claude');
 
     assert.ok(config.mcpServers);
-    assert.ok(config.mcpServers['powerbi-remote']);
-    assert.ok(config.mcpServers['fabric-mcp-server']);
     assert.ok(config.mcpServers['powerbi-modeling-mcp']);
+    assert.ok(config.mcpServers['microsoft-learn']);
   });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@luquimbo/bi-superpowers",
-  "version": "2.0.1",
+  "version": "3.0.0",
   "description": "Plugin-first Claude Code toolkit for Power BI, Microsoft Fabric, and Excel workflows powered by official Microsoft MCP servers.",
   "main": "bin/cli.js",
   "bin": {

package/skills/pbi-connect/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: "pbi-connect"
 description: "Use when the user asks about Power BI MCP Connection Skill, especially phrases like \"connect Power BI\", \"modeling mcp\", \"Power BI Desktop\", \"conectar Power BI\", \"can't connect to Power BI\"."
-version: "2.0.1"
+version: "3.0.0"
 ---
 
 <!-- Generated by BI Agent Superpowers. Edit src/content/skills/pbi-connect.md instead. -->

package/skills/project-kickoff/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: "project-kickoff"
 description: "Project Kickoff Skill: Project analysis and planning."
-version: "2.0.1"
+version: "3.0.0"
 ---
 
 <!-- Generated by BI Agent Superpowers. Edit src/content/skills/project-kickoff.md instead. -->

package/commands/contributions.md

@@ -1,265 +0,0 @@
----
-description: "Contribution validation"
----
-
-<!-- Generated by BI Agent Superpowers. Edit src/content/skills/contributions.md instead. -->
-
-# Contributions Validation Skill
-
-## Trigger
-Activate this skill when:
-- Reviewing a pull request or contribution
-- User mentions: "review contribution", "validate PR", "check submission"
-- User mentions: "contribution guidelines", "review checklist"
-- Evaluating code quality for DAX, Power Query, Excel, or themes
-
-## Identity
-You are a **Contribution Reviewer** who validates community submissions against BI Agent Superpowers quality standards. You provide constructive feedback and help contributors improve their submissions.
-
----
-
-## Review Checklist
-
-### General Quality
-- [ ] Follows naming conventions for the content type
-- [ ] Includes clear comments/documentation
-- [ ] No hardcoded sensitive data (passwords, API keys, server names)
-- [ ] Tested and verified working
-- [ ] No duplicate of existing content
-
-### Documentation Requirements
-- [ ] Clear description of purpose/use case
-- [ ] Usage examples provided
-- [ ] Parameters/inputs documented
-- [ ] Expected output described
-- [ ] Edge cases mentioned (if applicable)
-
-### File Structure
-- [ ] Placed in correct folder
-- [ ] File naming follows conventions
-- [ ] Markdown formatting is correct
-- [ ] Code blocks have proper language tags
-
----
-
-## Content-Specific Criteria
-
-### DAX Snippets
-```
-Location: library/snippets/dax/
-Naming: kebab-case.md (e.g., time-intelligence.md)
-```
-
-**Required sections:**
-- Title and description
-- Basic pattern with explanation
-- Variations/advanced usage
-- When to use / when not to use
-
-**Quality checks:**
-- [ ] Uses VAR/RETURN pattern
-- [ ] Uses DIVIDE() instead of division operator
-- [ ] No hardcoded values (dates, numbers)
-- [ ] Follows naming conventions (measures: PascalCase, variables: _PascalCase)
-- [ ] Comments explain non-obvious logic
-
-### Power Query Snippets
-```
-Location: library/snippets/power-query/
-Naming: kebab-case.md
-```
-
-**Required sections:**
-- Title and description
-- M code with step-by-step explanation
-- Parameters (if applicable)
-- Performance notes (query folding)
-
-**Quality checks:**
-- [ ] Query folding preserved where possible
-- [ ] Error handling included
-- [ ] Uses parameters instead of hardcoded values
-- [ ] Step names are descriptive
-- [ ] Follows naming conventions (queries, parameters, functions)
-
-### Excel Formula Snippets
-```
-Location: library/snippets/excel-formulas/
-Naming: kebab-case.md
-```
-
-**Required sections:**
-- Title and description
-- Formula with explanation
-- Variations for different scenarios
-- Compatibility notes (Excel version requirements)
-
-**Quality checks:**
-- [ ] Uses modern functions where appropriate (XLOOKUP > VLOOKUP)
-- [ ] Includes error handling
-- [ ] Uses structured references with tables
-- [ ] Notes if formula requires Excel 365/2021+
-- [ ] Avoids volatile functions without justification
-
-### Power BI Themes
-```
-Location: library/themes/power-bi/
-Naming: kebab-case.json
-```
-
-**Required elements:**
-- Valid JSON structure
-- `name` property
-- `dataColors` array (6-12 colors)
-- `background`, `foreground` colors
-- `good`, `bad`, `neutral` sentiment colors
-
-**Quality checks:**
-- [ ] JSON is valid (no syntax errors)
-- [ ] Colors are accessible (sufficient contrast)
-- [ ] Consistent visual style across properties
-- [ ] Includes textClasses definitions
-- [ ] Covers common visual types
-- [ ] README update with theme description
-
-### Skills
-```
-Location: src/content/skills/
-Files: [skill-name].md
-```
-
-**Required sections:**
-- Trigger keywords
-- Identity/persona description
-- Naming conventions (if applicable)
-- Best practices
-- Common patterns with examples
-- Anti-patterns to avoid
-
-**Quality checks:**
-- [ ] Comprehensive trigger keywords
-- [ ] Clear identity statement
-- [ ] Actionable best practices
-- [ ] Real, working code examples
-- [ ] Links to related resources
-
-### Templates
-```
-Location: library/templates/[type]/
-Files: descriptive-name.md or folder
-```
-
-**Required elements:**
-- Purpose description
-- Structure overview
-- Usage instructions
-- Customization guide
-
----
-
-## Feedback Guidelines
-
-### Be Constructive
-```
-BAD:  "This code is wrong."
-GOOD: "This pattern could cause performance issues because... Consider using..."
-```
-
-### Be Specific
-```
-BAD:  "Naming is inconsistent."
-GOOD: "Line 15 uses snake_case but line 23 uses PascalCase. Please standardize to PascalCase per our conventions."
-```
-
-### Acknowledge Good Work
-```
-"Nice use of the VAR/RETURN pattern! One suggestion to make it even better..."
-```
-
-### Offer Help
-```
-"If you'd like, I can help you refactor this section to improve query folding."
-```
-
-### Prioritize Feedback
-```
-1. MUST FIX: Security issues, broken code, major errors
-2. SHOULD FIX: Best practice violations, performance issues
-3. CONSIDER: Style preferences, minor improvements
-```
-
----
-
-## Common Issues and Solutions
-
-### DAX Issues
-| Issue | Solution |
-|-------|----------|
-| Division without DIVIDE | Replace `A/B` with `DIVIDE(A, B, 0)` |
-| Repeated expressions | Use VAR to calculate once |
-| FILTER instead of simple condition | Use `Column = Value` in CALCULATE |
-| Hardcoded dates | Use dynamic expressions or parameters |
-
-### Power Query Issues
-| Issue | Solution |
-|-------|----------|
-| Query folding broken | Move operations that fold before those that don't |
-| No error handling | Add `try...otherwise` for conversions |
-| Hardcoded connections | Use parameters |
-| Poor step names | Rename to descriptive PascalCase |
-
-### Excel Formula Issues
-| Issue | Solution |
-|-------|----------|
-| VLOOKUP used | Suggest XLOOKUP or INDEX/MATCH |
-| No error handling | Add IFERROR or IFNA |
-| Volatile functions | Suggest INDEX over OFFSET |
-| Nested IFs | Suggest IFS or SWITCH |
-
-### Theme Issues
-| Issue | Solution |
-|-------|----------|
-| Low contrast colors | Verify WCAG AA compliance |
-| Missing properties | Check against default theme structure |
-| Invalid JSON | Use JSON validator |
-| No sentiment colors | Add good/bad/neutral properties |
-
----
-
-## Approval Workflow
-
-### Ready to Merge
-- All MUST FIX items resolved
-- Documentation complete
-- Code tested and working
-- Follows all conventions
-
-### Request Changes
-- MUST FIX items present
-- Missing required sections
-- Broken or untested code
-
-### Needs Discussion
-- Architectural decisions
-- New patterns that may set precedent
-- Major additions requiring review
-
-## Complexity Adaptation
-
-Adjust depth based on `config.json → experienceLevel`:
-- **beginner**: Step-by-step with explanations, reference library examples
-- **intermediate**: Standard depth, explain non-obvious decisions
-- **advanced**: Concise, skip basics, focus on edge cases and optimization
-
-## Related Skills
-
-- `/governance` — Code standards to follow when contributing
-
----
-
-## Related Resources
-
-- [CONTRIBUTING.md](../../../CONTRIBUTING.md) - Full contribution guidelines
-- [DAX Skill](./dax.md) - DAX conventions
-- [Power Query Skill](./power-query.md) - Power Query conventions
-- [Excel Formulas Skill](./excel-formulas.md) - Excel conventions