@luquimbo/bi-superpowers 4.1.6 → 5.0.1

package/skills/report-design/scripts/ensure-pbi-cli.sh

@@ -1,41 +0,0 @@
-#!/bin/bash
-# ensure-pbi-cli.sh — idempotent installer for pbi-cli-tool + pywin32
-# Run from any directory. Requires Python 3.10+ and pip on PATH.
-# Exit code 0 = ready, 1 = install failed
-
-set -e
-
-check_python() {
-  python --version 2>/dev/null | grep -qE "Python 3\.(1[0-9]|[2-9][0-9])" && return 0
-  python3 --version 2>/dev/null | grep -qE "Python 3\.(1[0-9]|[2-9][0-9])" && return 0
-  echo "❌ Python 3.10+ required. Install from Microsoft Store or python.org."
-  exit 1
-}
-
-check_pipx() {
-  pipx --version 2>/dev/null && return 0
-  python -m pipx --version 2>/dev/null && return 0
-  echo "📦 Installing pipx..."
-  pip install --user pipx 2>/dev/null || python -m pip install --user pipx
-  python -m pipx ensurepath
-  echo "⚠ Close and reopen your terminal, then re-run this script."
-  exit 1
-}
-
-check_pbi() {
-  pbi --version 2>/dev/null && return 0
-  echo "📦 Installing pbi-cli-tool..."
-  python -m pipx install pbi-cli-tool
-}
-
-inject_pywin32() {
-  python -m pipx inject pbi-cli-tool pywin32 2>/dev/null || true
-}
-
-echo "Checking pbi-cli-tool environment..."
-check_python
-check_pipx
-check_pbi
-inject_pywin32
-echo "✅ pbi-cli-tool is ready."
-pbi --version

package/skills/report-design/scripts/validate-pbir.js

@@ -1,351 +0,0 @@
-#!/usr/bin/env node
-/**
- * validate-pbir.js — allowlist-driven PBIR validator.
- *
- * Complements `pbi report validate` (which does schema sanity) by enforcing
- * the native-visuals allowlist maintained in create-visual.js. Catches:
- *
- *   - visualType that's not a native PBIR type (e.g. `stackedBarChart`,
- *     typos) — Desktop renders these as "objeto visual personalizado" but
- *     the CLI validate passes them. This is the exact bug F1 that motivated
- *     the migration away from the CLI.
- *
- *   - bindings on roles that the type doesn't declare (e.g. `--bind size`
- *     on a funnelChart) — CLI validate doesn't catch these either.
- *
- *   - missing required roles (e.g. a barChart with no Category).
- *
- *   - visualContainer schema drift (wrong $schema URL or missing `name`).
- *
- * Usage:
- *   node validate-pbir.js <reportPath>
- *   node validate-pbir.js --json <reportPath>    # machine-readable output
- *
- * Exit codes:
- *   0  valid
- *   1  validation errors found
- *   2  I/O error (bad path, corrupt JSON)
- *
- * NOT a replacement for `pbi report validate` — run both. This validator
- * trusts that the .Report folder is structurally correct (has definition/,
- * pages/, etc.) and focuses on visual-level correctness that the CLI misses.
- */
-
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-
-const { NATIVE_VISUAL_TYPES, KNOWN_NON_NATIVE_TYPES } = require('./create-visual.js');
-
-const EXPECTED_VISUAL_SCHEMA =
-  'https://developer.microsoft.com/json-schemas/fabric/item/report/definition/visualContainer/2.7.0/schema.json';
-
-function fail(msg, code = 2) {
-  process.stderr.write(`validate-pbir: ${msg}\n`);
-  process.exit(code);
-}
-
-function parseArgs(argv) {
-  const out = {};
-  const positional = [];
-  for (let i = 0; i < argv.length; i += 1) {
-    const a = argv[i];
-    if (a === '--json') out.json = true;
-    else if (a === '-h' || a === '--help') out.help = true;
-    else if (a.startsWith('-')) fail(`unknown flag: ${a}`, 2);
-    else positional.push(a);
-  }
-  out.reportPath = positional[0];
-  return out;
-}
-
-function printHelp() {
-  process.stdout.write(
-    [
-      'Usage: validate-pbir.js [--json] <reportPath>',
-      '',
-      'Arguments:',
-      '  reportPath   Path to a .Report folder',
-      '',
-      'Options:',
-      '  --json       Emit JSON instead of human-readable output',
-      '  -h, --help   Show this help',
-      '',
-      'Exit codes:',
-      '  0  valid',
-      '  1  validation errors found',
-      '  2  I/O error',
-      '',
-    ].join('\n')
-  );
-}
-
-function listPages(reportPath) {
-  const pagesDir = path.join(reportPath, 'definition', 'pages');
-  if (!fs.existsSync(pagesDir)) {
-    fail(`pages directory not found: ${pagesDir}`, 2);
-  }
-  const out = [];
-  for (const entry of fs.readdirSync(pagesDir, { withFileTypes: true })) {
-    if (!entry.isDirectory()) continue;
-    const pageJsonPath = path.join(pagesDir, entry.name, 'page.json');
-    if (!fs.existsSync(pageJsonPath)) continue;
-    out.push({ name: entry.name, dir: path.join(pagesDir, entry.name) });
-  }
-  return out;
-}
-
-function listVisuals(pageDir) {
-  const visualsDir = path.join(pageDir, 'visuals');
-  if (!fs.existsSync(visualsDir)) return [];
-  const out = [];
-  for (const entry of fs.readdirSync(visualsDir, { withFileTypes: true })) {
-    if (!entry.isDirectory()) continue;
-    const visualJsonPath = path.join(visualsDir, entry.name, 'visual.json');
-    if (!fs.existsSync(visualJsonPath)) continue;
-    out.push({ name: entry.name, path: visualJsonPath });
-  }
-  return out;
-}
-
-function readJsonSafe(filePath) {
-  try {
-    const raw = fs.readFileSync(filePath, 'utf8');
-    return { data: JSON.parse(raw), err: null };
-  } catch (err) {
-    return { data: null, err };
-  }
-}
-
-// Extracts the set of roles that have at least one projection bound.
-function boundRoles(visualData) {
-  const qs = visualData && visualData.visual && visualData.visual.query && visualData.visual.query.queryState;
-  if (!qs || typeof qs !== 'object') return new Set();
-  const bound = new Set();
-  for (const [role, spec] of Object.entries(qs)) {
-    const projections = spec && spec.projections;
-    if (Array.isArray(projections) && projections.length > 0) bound.add(role);
-  }
-  return bound;
-}
-
-function validatePosition(position) {
-  const errors = [];
-  const requiredFields = ['x', 'y', 'z', 'height', 'width', 'tabOrder'];
-
-  if (!position || typeof position !== 'object') {
-    return requiredFields.map((field) => ({
-      severity: 'error',
-      rule: 'position-number',
-      field,
-      message: `position.${field} must be a finite number`,
-    }));
-  }
-
-  for (const field of requiredFields) {
-    if (!Number.isFinite(position[field])) {
-      errors.push({
-        severity: 'error',
-        rule: 'position-number',
-        field,
-        message: `position.${field} must be a finite number`,
-      });
-    }
-  }
-
-  return errors;
-}
-
-function validateVisual(visual) {
-  const errors = [];
-  const { data, path: filePath } = visual;
-
-  // Schema URL check.
-  if (data.$schema !== EXPECTED_VISUAL_SCHEMA) {
-    errors.push({
-      severity: 'warn',
-      rule: 'schema-url',
-      message: `unexpected $schema (got ${data.$schema || 'none'})`,
-    });
-  }
-
-  // `name` must exist and match the folder.
-  if (typeof data.name !== 'string' || !data.name) {
-    errors.push({ severity: 'error', rule: 'name', message: 'missing "name"' });
-  }
-
-  errors.push(...validatePosition(data.position));
-
-  const vt = data.visual && data.visual.visualType;
-  if (!vt) {
-    errors.push({ severity: 'error', rule: 'visual-type', message: 'missing visualType' });
-    return errors;
-  }
-
-  // Allowlist: known non-native (like stackedBarChart).
-  if (KNOWN_NON_NATIVE_TYPES[vt]) {
-    errors.push({
-      severity: 'error',
-      rule: 'non-native-type',
-      visualType: vt,
-      message: KNOWN_NON_NATIVE_TYPES[vt],
-    });
-    return errors;
-  }
-
-  const def = NATIVE_VISUAL_TYPES[vt];
-  if (!def) {
-    errors.push({
-      severity: 'error',
-      rule: 'unknown-type',
-      visualType: vt,
-      message: `visualType "${vt}" is not in the native allowlist. Run create-visual.js --list-types for the canonical list.`,
-    });
-    return errors;
-  }
-
-  // Check required roles are bound (skip for textbox — no roles).
-  if (vt !== 'textbox') {
-    const bound = boundRoles(data);
-    for (const [role, spec] of Object.entries(def.roles)) {
-      if (spec.required && !bound.has(role)) {
-        errors.push({
-          severity: 'error',
-          rule: 'missing-required-role',
-          visualType: vt,
-          role,
-          message: `required role "${role}" has no projections (visualType: ${vt})`,
-        });
-      }
-    }
-
-    // Check that every bound role is valid for this type.
-    for (const role of bound) {
-      if (!(role in def.roles)) {
-        errors.push({
-          severity: 'error',
-          rule: 'invalid-role',
-          visualType: vt,
-          role,
-          message: `role "${role}" is not valid for visualType ${vt}. Valid: ${Object.keys(def.roles).join(', ') || '(none)'}`,
-        });
-      }
-    }
-  }
-
-  return errors;
-}
-
-function main() {
-  const args = parseArgs(process.argv.slice(2));
-  if (args.help) {
-    printHelp();
-    return;
-  }
-  if (!args.reportPath) fail('reportPath is required. Run --help for usage.', 2);
-
-  const reportPath = path.resolve(args.reportPath);
-  if (!fs.existsSync(reportPath)) fail(`not found: ${reportPath}`, 2);
-
-  const pages = listPages(reportPath);
-  const results = {
-    reportPath,
-    pagesChecked: pages.length,
-    visualsChecked: 0,
-    errors: [],
-  };
-
-  for (const page of pages) {
-    const visuals = listVisuals(page.dir);
-    // Track visual.name per page so we can flag duplicates. PBI Desktop
-    // does not reject a duplicate-name pair on load, but downstream tooling
-    // that references visuals by name (filters, bookmarks, selections) gets
-    // ambiguous — validate-pbir treats it as a hard error.
-    const namesSeenOnPage = new Map();
-    for (const v of visuals) {
-      results.visualsChecked += 1;
-      const { data, err } = readJsonSafe(v.path);
-      if (err) {
-        results.errors.push({
-          page: page.name,
-          visual: v.name,
-          severity: 'error',
-          rule: 'bad-json',
-          message: `could not parse visual.json: ${err.message}`,
-        });
-        continue;
-      }
-
-      // Duplicate-name check (page-scoped).
-      const declaredName = data && typeof data.name === 'string' ? data.name : null;
-      if (declaredName) {
-        if (namesSeenOnPage.has(declaredName)) {
-          results.errors.push({
-            page: page.name,
-            visual: v.name,
-            severity: 'error',
-            rule: 'duplicate-name',
-            message: `visual name "${declaredName}" is also used by "${namesSeenOnPage.get(declaredName)}" on this page`,
-          });
-        } else {
-          namesSeenOnPage.set(declaredName, v.name);
-        }
-      }
-
-      const visualErrors = validateVisual({ data, path: v.path });
-      for (const e of visualErrors) {
-        results.errors.push({
-          page: page.name,
-          visual: v.name,
-          ...e,
-        });
-      }
-    }
-  }
-
-  const errorCount = results.errors.filter((e) => e.severity === 'error').length;
-  const warnCount = results.errors.filter((e) => e.severity === 'warn').length;
-  const valid = errorCount === 0;
-
-  if (args.json) {
-    process.stdout.write(JSON.stringify({ ...results, valid, errorCount, warnCount }, null, 2) + '\n');
-  } else {
-    const W = 77; // inner width between the | borders
-    const padRow = (label, value) => {
-      const text = `${label}: ${value}`;
-      return `| ${text}${' '.repeat(Math.max(0, W - text.length - 1))}|`;
-    };
-    const border = `+${'-'.repeat(W)}+`;
-    process.stdout.write(border + '\n');
-    process.stdout.write(padRow('valid', valid ? 'True' : 'False') + '\n');
-    process.stdout.write(padRow('pages_checked', String(results.pagesChecked)) + '\n');
-    process.stdout.write(padRow('visuals_checked', String(results.visualsChecked)) + '\n');
-    process.stdout.write(padRow('errors', String(errorCount)) + '\n');
-    process.stdout.write(padRow('warnings', String(warnCount)) + '\n');
-    process.stdout.write(border + '\n');
-    if (results.errors.length) {
-      process.stdout.write('\nIssues:\n');
-      for (const e of results.errors) {
-        const icon = e.severity === 'error' ? '✗' : '⚠';
-        const loc = `${e.page}/${e.visual}`;
-        process.stdout.write(`  ${icon} [${e.rule}] ${loc}: ${e.message}\n`);
-      }
-      process.stdout.write('\n');
-    }
-  }
-
-  process.exit(valid ? 0 : 1);
-}
-
-module.exports = {
-  validateVisual,
-  boundRoles,
-  listPages,
-  listVisuals,
-  parseArgs,
-};
-
-if (require.main === module) {
-  main();
-}

package/src/content/skills/pbi-connect.md

@@ -1,226 +0,0 @@
-# Power BI MCP Connection Skill
-
-## Trigger
-Activate this skill when user mentions:
-- "connect Power BI", "PBI connection", "MCP connection", "Power BI MCP"
-- "modeling mcp", "Power BI Modeling MCP"
-- "Power BI Desktop", "PBIP", "semantic model"
-- "conectar Power BI", "MCP de Power BI", "modeling mcp"
-- "can't connect to Power BI", "connection error", "MCP not working"
-
-## Identity
-You are a **Power BI MCP Connection Specialist**. Your job is to help the user connect their AI agent to Power BI Desktop using the official Microsoft MCP servers shipped with bi-superpowers, with a user-level install workflow that works across projects.
-
-## MANDATORY RULES
-1. **USER-LEVEL FIRST.** Prefer `super install --all --yes` or `super install --agent <agent> --yes`; this installs skills and MCP config under the user's home directory and applies across projects. `.mcp.json` is only for an optional repo-local Claude Code plugin.
-2. **OFFICIAL SERVERS ONLY.** Use `powerbi-modeling-mcp` (local) and `microsoft-learn` (HTTP). Do not invent or recommend unofficial MCPs.
-3. **WINDOWS LIMITATION.** Explain clearly that the local Modeling MCP is only available on Windows.
-4. **NO PORT INVENTION.** Do not suggest local port-based setups for the official Modeling MCP flow.
-5. **ONE QUESTION AT A TIME.** Follow the wizard pattern.
-
----
-
-## PHASE 0: Initial Triage
-
-Start with:
-
-```text
-POWER BI MCP CONNECTION
-=======================
-
-I'll help you connect your AI agent using the official Microsoft MCP servers.
-
-What do you need?
-
-1. Connect to Power BI Desktop on this machine (Windows)
-2. Verify that my agent MCP config is installed correctly
-3. Verify an optional local Claude Code plugin `.mcp.json`
-```
-
----
-
-## PHASE 1: Local Modeling MCP (Windows)
-
-If the user chooses option 1:
-
-```text
-LOCAL MODELING MCP
-==================
-
-This path uses Microsoft's official `powerbi-modeling-mcp` executable.
-
-Before we continue:
-
-1. Are you on Windows?
-2. Did you install the "Power BI Modeling MCP" extension in VS Code or Cursor?
-```
-
-### If the user is on Windows and installed the extension
-
-Guide them to the user-level install:
-
-1. Run `super install --all --yes`, or for one agent run `super install --agent codex --yes` / `super install --agent claude-code --yes` / etc.
-2. Confirm the agent config contains `powerbi-modeling-mcp` and `microsoft-learn`.
-3. Restart or refresh the AI agent so it reloads skills and MCP servers.
-
-Use these config locations:
-
-| Agent | Skill path | MCP config |
-| --- | --- | --- |
-| Claude Code | `~/.claude/skills` or `~/.agents/skills` | `~/.claude.json` |
-| GitHub Copilot | `~/.copilot/skills` | `~/.copilot/mcp-config.json` |
-| Codex | `~/.agents/skills` | `~/.codex/config.toml` |
-| Gemini CLI | `~/.gemini/skills` | `~/.gemini/settings.json` |
-| Kilo Code | `~/.kilo/skills` | `~/.kilo/mcp_settings.json` |
-
-Use this explanation:
-
-```text
-BI Agent Superpowers does not launch the Modeling MCP with `uvx`.
-Instead, it uses a local wrapper that finds the official Microsoft executable
-and starts it with `--start`.
-```
-
-If the user wants a config example, show:
-
-```toml
-[mcp_servers.powerbi-modeling-mcp]
-command = "node"
-args = ["<package-dir>/bin/mcp/powerbi-modeling-launcher.js"]
-
-[mcp_servers.microsoft-learn]
-url = "https://learn.microsoft.com/api/mcp"
-```
-
-### If the user installed the executable manually
-
-Tell them to set:
-
-```text
-BI_SUPERPOWERS_POWERBI_MODELING_MCP_PATH
-```
-
-Then re-run:
-
-```bash
-super install --all --yes
-```
-
-If they are intentionally maintaining a repo-local Claude Code plugin, they can run `super mcp-setup` inside that plugin project instead.
-
-### If the user is on macOS or Linux
-
-Say:
-
-```text
-The official local Power BI Modeling MCP is only available on Windows.
-
-You still have `microsoft-learn` (HTTP) available on every platform for
-docs, and you can work with Power BI files using the skills library.
-For live editing of a local semantic model, you need a Windows environment.
-```
-
----
-
-## PHASE 2: Verify Agent MCP Config
-
-If the user chooses option 2:
-
-Check the config for the agent they use:
-
-- Claude Code: `~/.claude.json`
-- GitHub Copilot: `~/.copilot/mcp-config.json`
-- Codex: `~/.codex/config.toml`
-- Gemini CLI: `~/.gemini/settings.json`
-- Kilo Code: `~/.kilo/mcp_settings.json`
-
-Confirm:
-
-- skills are installed under the agent's user-level skill directory
-- config includes `powerbi-modeling-mcp`
-- config includes `microsoft-learn`
-
-If anything is missing, recommend:
-
-```bash
-super install --agent <agent-id> --yes
-```
-
-Then restart or refresh the agent.
-
-## PHASE 3: Verify Optional Local Claude Code Plugin Config
-
-If the user chooses option 3, or explicitly says they use `super kickoff` / `claude --plugin-dir`:
-
-Check these files in order:
-
-1. `.claude-plugin/plugin.json`
-2. `.mcp.json`
-3. `.bi-superpowers.json`
-
-Confirm:
-
-- plugin name is `bi-superpowers`
-- `.mcp.json` includes `powerbi-modeling-mcp`
-- `.mcp.json` includes `microsoft-learn`
-
-If anything is missing, recommend:
-
-```bash
-super recharge
-super mcp-setup
-```
-
-Then:
-
-```bash
-claude --plugin-dir .
-```
-
----
-
-## TROUBLESHOOTING
-
-| Problem | What to do |
-| --- | --- |
-| Modeling MCP missing on Windows | Install the Microsoft extension in VS Code or Cursor |
-| Modeling MCP installed manually | Set `BI_SUPERPOWERS_POWERBI_MODELING_MCP_PATH` |
-| Agent not loading MCPs | Re-run `super install --agent <agent-id> --yes` and restart the agent |
-| Local Claude Code plugin not loading MCPs | Re-run `super mcp-setup` inside the plugin project and restart Claude Code |
-| macOS/Linux local modeling request | Use `microsoft-learn` for docs; live editing requires Windows |
-| User asks about Excel MCP | Explain Excel remains supported through skills and library content, not a default MCP |
-
----
-
-## ANTI-PATTERNS
-
-| Don't Do This | Why | Do This Instead |
-| --- | --- | --- |
-| Recommend `uvx` for Modeling MCP | Not the official Microsoft installation path | Use the official executable via the local launcher |
-| Ask the user to find a localhost port | Not required in the new flow | Use the official Modeling MCP launcher |
-| Run `super kickoff` for Codex/GitHub Copilot/Gemini/Kilo setup | `kickoff` creates repo-local Claude Code plugin files | Use `super install --agent <agent-id> --yes` |
-| Treat `.mcp.json` as the default install target | It is only for optional local Claude Code plugins | Use the agent's user-level MCP config |
-| Invent unofficial MCPs (remote, fabric, etc.) | This plugin only ships 2 official MCPs | Only use the 2 official MCPs we ship (`powerbi-modeling-mcp` and `microsoft-learn`) |
-
----
-
-## 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
-
-- `/project-kickoff` — Analyze a BI project and plan next steps
-
----
-
-## RELATED RESOURCES
-
-- [Power BI MCP overview](https://learn.microsoft.com/en-us/power-bi/developer/mcp/mcp-servers-overview)
-- [Power BI Modeling MCP on GitHub](https://github.com/microsoft/powerbi-modeling-mcp)
-- [Microsoft Learn MCP](https://learn.microsoft.com/en-us/training/support/mcp)