@luquimbo/bi-superpowers 3.1.1 → 4.1.0

package/bin/commands/install.test.js

@@ -1,289 +0,0 @@
-/**
- * Unit tests for the install command
- *
- * Run with: npm test
- */
-
-const { test, describe, beforeEach, afterEach } = require('node:test');
-const assert = require('node:assert');
-const fs = require('fs');
-const path = require('path');
-const os = require('os');
-
-const installCommand = require('./install');
-const { parseArgs, detectAgents, copySkillDir, formatFsError, AGENTS, UNIVERSAL_DIR } =
-  installCommand;
-
-describe('install command - parseArgs', () => {
-  test('parses --yes/-y flag', () => {
-    assert.strictEqual(parseArgs(['--yes']).isYes, true);
-    assert.strictEqual(parseArgs(['-y']).isYes, true);
-    assert.strictEqual(parseArgs([]).isYes, false);
-  });
-
-  test('parses --all flag', () => {
-    assert.strictEqual(parseArgs(['--all']).isAll, true);
-    assert.strictEqual(parseArgs([]).isAll, false);
-  });
-
-  test('parses multiple --agent flags', () => {
-    const opts = parseArgs(['--agent', 'claude-code', '--agent', 'codex']);
-    assert.deepStrictEqual(opts.agentFlags, ['claude-code', 'codex']);
-  });
-
-  test('parses short -a flag', () => {
-    const opts = parseArgs(['-a', 'claude-code', '-a', 'kilo']);
-    assert.deepStrictEqual(opts.agentFlags, ['claude-code', 'kilo']);
-  });
-
-  test('ignores --agent without value (end of args)', () => {
-    const opts = parseArgs(['--agent']);
-    assert.deepStrictEqual(opts.agentFlags, []);
-  });
-
-  test('ignores --agent when next arg is another flag', () => {
-    const opts = parseArgs(['--agent', '--yes']);
-    assert.deepStrictEqual(opts.agentFlags, []);
-    assert.strictEqual(opts.isYes, true);
-  });
-
-  test('combines all flags correctly', () => {
-    const opts = parseArgs(['--all', '--yes', '-a', 'claude-code']);
-    assert.strictEqual(opts.isAll, true);
-    assert.strictEqual(opts.isYes, true);
-    assert.deepStrictEqual(opts.agentFlags, ['claude-code']);
-  });
-});
-
-describe('install command - AGENTS registry', () => {
-  test('has exactly 5 supported agents', () => {
-    assert.strictEqual(Object.keys(AGENTS).length, 5);
-  });
-
-  test('includes the 5 officially supported agents', () => {
-    const ids = Object.keys(AGENTS);
-    assert.ok(ids.includes('github-copilot'));
-    assert.ok(ids.includes('claude-code'));
-    assert.ok(ids.includes('codex'));
-    assert.ok(ids.includes('gemini-cli'));
-    assert.ok(ids.includes('kilo'));
-  });
-
-  test('every agent has name and dir fields', () => {
-    for (const [id, agent] of Object.entries(AGENTS)) {
-      assert.ok(agent.name, `${id} missing name`);
-      assert.ok(agent.dir, `${id} missing dir`);
-      assert.ok(agent.dir.startsWith('.'), `${id} dir should start with .`);
-    }
-  });
-
-  test('UNIVERSAL_DIR is .agents/skills', () => {
-    assert.strictEqual(UNIVERSAL_DIR, '.agents/skills');
-  });
-
-  test('codex uses UNIVERSAL_DIR', () => {
-    assert.strictEqual(AGENTS.codex.dir, UNIVERSAL_DIR);
-  });
-});
-
-describe('install command - detectAgents', () => {
-  let tempDir;
-
-  beforeEach(() => {
-    tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-install-test-'));
-  });
-
-  afterEach(() => {
-    fs.rmSync(tempDir, { recursive: true, force: true });
-  });
-
-  test('returns empty array when no agents are installed', () => {
-    const detected = detectAgents(tempDir);
-    assert.deepStrictEqual(detected, []);
-  });
-
-  test('detects claude-code when .claude exists', () => {
-    fs.mkdirSync(path.join(tempDir, '.claude'), { recursive: true });
-    const detected = detectAgents(tempDir);
-    assert.ok(detected.includes('claude-code'));
-  });
-
-  test('detects multiple agents', () => {
-    fs.mkdirSync(path.join(tempDir, '.claude'), { recursive: true });
-    fs.mkdirSync(path.join(tempDir, '.copilot'), { recursive: true });
-    fs.mkdirSync(path.join(tempDir, '.gemini'), { recursive: true });
-    const detected = detectAgents(tempDir);
-    assert.ok(detected.includes('claude-code'));
-    assert.ok(detected.includes('github-copilot'));
-    assert.ok(detected.includes('gemini-cli'));
-  });
-});
-
-describe('install command - copySkillDir', () => {
-  let srcDir;
-  let destDir;
-
-  beforeEach(() => {
-    const tempDir = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-install-copy-'));
-    srcDir = path.join(tempDir, 'src');
-    destDir = path.join(tempDir, 'dest');
-    fs.mkdirSync(srcDir, { recursive: true });
-  });
-
-  afterEach(() => {
-    fs.rmSync(path.dirname(srcDir), { recursive: true, force: true });
-  });
-
-  test('copies a single file', () => {
-    fs.writeFileSync(path.join(srcDir, 'SKILL.md'), '# Test');
-    copySkillDir(srcDir, destDir);
-    assert.strictEqual(fs.readFileSync(path.join(destDir, 'SKILL.md'), 'utf8'), '# Test');
-  });
-
-  test('creates destination directory if missing', () => {
-    fs.writeFileSync(path.join(srcDir, 'SKILL.md'), 'content');
-    copySkillDir(srcDir, destDir);
-    assert.ok(fs.existsSync(destDir));
-  });
-
-  test('copies nested directories recursively', () => {
-    fs.writeFileSync(path.join(srcDir, 'SKILL.md'), 'top');
-    fs.mkdirSync(path.join(srcDir, 'references'));
-    fs.writeFileSync(path.join(srcDir, 'references', 'notes.md'), 'nested');
-    fs.mkdirSync(path.join(srcDir, 'scripts'));
-    fs.writeFileSync(path.join(srcDir, 'scripts', 'run.sh'), '#!/bin/sh');
-
-    copySkillDir(srcDir, destDir);
-
-    assert.strictEqual(fs.readFileSync(path.join(destDir, 'SKILL.md'), 'utf8'), 'top');
-    assert.strictEqual(
-      fs.readFileSync(path.join(destDir, 'references', 'notes.md'), 'utf8'),
-      'nested'
-    );
-    assert.strictEqual(
-      fs.readFileSync(path.join(destDir, 'scripts', 'run.sh'), 'utf8'),
-      '#!/bin/sh'
-    );
-  });
-});
-
-describe('install command - formatFsError', () => {
-  test('includes the context and error message', () => {
-    const err = new Error('oops');
-    err.code = 'UNKNOWN';
-    const msg = formatFsError(err, 'Failed');
-    assert.ok(msg.includes('Failed'));
-    assert.ok(msg.includes('oops'));
-  });
-
-  test('adds hint for EACCES', () => {
-    const err = new Error('permission denied');
-    err.code = 'EACCES';
-    const msg = formatFsError(err, 'Failed');
-    assert.ok(msg.toLowerCase().includes('permiso'));
-  });
-
-  test('adds hint for EPERM (Windows admin)', () => {
-    const err = new Error('not permitted');
-    err.code = 'EPERM';
-    const msg = formatFsError(err, 'Failed');
-    assert.ok(msg.toLowerCase().includes('admin'));
-  });
-
-  test('adds hint for ENOSPC', () => {
-    const err = new Error('no space');
-    err.code = 'ENOSPC';
-    const msg = formatFsError(err, 'Failed');
-    assert.ok(msg.toLowerCase().includes('espacio'));
-  });
-});
-
-describe('install command - module exports', () => {
-  test('module default export is a function', () => {
-    assert.strictEqual(typeof installCommand, 'function');
-  });
-
-  test('exposes internal helpers for testing', () => {
-    assert.strictEqual(typeof installCommand.parseArgs, 'function');
-    assert.strictEqual(typeof installCommand.detectAgents, 'function');
-    assert.strictEqual(typeof installCommand.copySkillDir, 'function');
-    assert.strictEqual(typeof installCommand.formatFsError, 'function');
-  });
-});
-
-describe('install command - integration: --all --yes', () => {
-  let tempHome;
-  let tempPkg;
-  let origHome;
-
-  beforeEach(() => {
-    tempHome = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-install-int-home-'));
-    tempPkg = fs.mkdtempSync(path.join(os.tmpdir(), 'bi-install-int-pkg-'));
-
-    // Minimal fake package layout: skills/<name>/SKILL.md + launcher file
-    const skillsDir = path.join(tempPkg, 'skills');
-    for (const skillName of ['project-kickoff', 'pbi-connect']) {
-      const dir = path.join(skillsDir, skillName);
-      fs.mkdirSync(dir, { recursive: true });
-      fs.writeFileSync(
-        path.join(dir, 'SKILL.md'),
-        `---\nname: ${skillName}\ndescription: fake skill for test\n---\n# ${skillName}\n`
-      );
-    }
-    fs.mkdirSync(path.join(tempPkg, 'bin', 'mcp'), { recursive: true });
-    fs.writeFileSync(
-      path.join(tempPkg, 'bin', 'mcp', 'powerbi-modeling-launcher.js'),
-      '// fake launcher\n'
-    );
-
-    origHome = os.homedir;
-    os.homedir = () => tempHome;
-  });
-
-  afterEach(() => {
-    os.homedir = origHome;
-    fs.rmSync(tempHome, { recursive: true, force: true });
-    fs.rmSync(tempPkg, { recursive: true, force: true });
-  });
-
-  test('installs skills and writes MCP config for all 5 agents', async () => {
-    const origExitCode = process.exitCode;
-    try {
-      await installCommand(['--all', '--yes'], {
-        packageDir: tempPkg,
-        version: '9.9.9-test',
-      });
-    } finally {
-      // Reset exit code so subsequent tests aren't affected
-      process.exitCode = origExitCode;
-    }
-
-    // Skills installed at universal path
-    assert.ok(
-      fs.existsSync(path.join(tempHome, '.agents', 'skills', 'project-kickoff', 'SKILL.md'))
-    );
-    assert.ok(fs.existsSync(path.join(tempHome, '.agents', 'skills', 'pbi-connect', 'SKILL.md')));
-
-    // MCP configs written for all 5 agents
-    const expectedMcpFiles = [
-      path.join(tempHome, '.claude.json'),
-      path.join(tempHome, '.copilot', 'mcp-config.json'),
-      path.join(tempHome, '.codex', 'config.toml'),
-      path.join(tempHome, '.gemini', 'settings.json'),
-      path.join(tempHome, '.kilo', 'mcp_settings.json'),
-    ];
-    for (const filePath of expectedMcpFiles) {
-      assert.ok(fs.existsSync(filePath), `expected MCP config at ${filePath}`);
-    }
-
-    // Claude Code config has both servers under mcpServers
-    const claudeConfig = JSON.parse(fs.readFileSync(path.join(tempHome, '.claude.json'), 'utf8'));
-    assert.ok(claudeConfig.mcpServers['powerbi-modeling']);
-    assert.ok(claudeConfig.mcpServers['microsoft-learn']);
-
-    // Codex TOML has both sections
-    const codexToml = fs.readFileSync(path.join(tempHome, '.codex', 'config.toml'), 'utf8');
-    assert.ok(codexToml.includes('[mcp_servers.powerbi-modeling]'));
-    assert.ok(codexToml.includes('[mcp_servers.microsoft-learn]'));
-  });
-});

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/commands/pull.js

@@ -1,287 +0,0 @@
-/**
- * Pull Command - Sync from Original to Repo
- * ==========================================
- *
- * Pulls changes from the original BI file to the repo.
- * Extracts updated TMDL/content and creates a Git commit.
- *
- * Usage:
- *   super pull                   Pull current project
- *   super pull <project-name>    Pull specific project
- *   super pull --all             Pull all projects
- *
- * @module commands/pull
- */
-
-const fs = require('fs');
-const path = require('path');
-
-const git = require('../utils/git');
-const profiles = require('../utils/profiles');
-const pbix = require('../utils/pbix');
-const { getAllProjects, getProject, detectCurrentProject } = require('../utils/projects');
-
-/**
- * Parse command line arguments
- * @param {string[]} args - CLI arguments
- * @returns {Object} Parsed options
- */
-function parseArgs(args) {
-  const options = {
-    projectName: null,
-    all: false,
-    force: false,
-    help: false,
-  };
-
-  for (let i = 0; i < args.length; i++) {
-    const arg = args[i];
-
-    if (arg === '--all' || arg === '-a') {
-      options.all = true;
-    } else if (arg === '--force' || arg === '-f') {
-      options.force = true;
-    } else if (arg === '--help' || arg === '-h') {
-      options.help = true;
-    } else if (!arg.startsWith('-') && !options.projectName) {
-      options.projectName = arg;
-    }
-  }
-
-  return options;
-}
-
-/**
- * Show help message
- */
-function showHelp() {
-  console.log(`
-super pull - Traer cambios del archivo original al repo
-
-Uso:
-  super pull                   Pull del proyecto actual (si estás en la carpeta)
-  super pull <nombre-proyecto> Pull de un proyecto específico
-  super pull --all             Pull de todos los proyectos
-
-Opciones:
-  --all, -a        Pull de todos los proyectos
-  --force, -f      Forzar pull aunque no haya cambios detectados
-  --help, -h       Mostrar esta ayuda
-
-Ejemplos:
-  super pull sales-q4
-  super pull --all
-  super pull hr-dashboard --force
-`);
-}
-
-// Note: getAllProjects, getProject, and detectCurrentProject are imported from ../utils/projects
-
-/**
- * Pull changes for a single project
- * @param {Object} project - Project config
- * @param {boolean} force - Force pull even if no changes
- * @returns {Object} Result object
- */
-function pullProject(project, force = false) {
-  const result = {
-    success: false,
-    project: project.name,
-    message: '',
-    filesChanged: 0,
-  };
-
-  // Check if source file exists
-  const sourcePath = project.source?.path;
-  if (!sourcePath || !fs.existsSync(sourcePath)) {
-    result.message = `Archivo original no encontrado: ${sourcePath || 'no configurado'}`;
-    return result;
-  }
-
-  // Check if file has changed (by modification time or hash)
-  const currentHash = pbix.getFileHash(sourcePath);
-  const lastHash = project.source?.hash || '';
-
-  if (currentHash === lastHash && !force) {
-    result.success = true;
-    result.message = 'Sin cambios';
-    return result;
-  }
-
-  // Detect file type and extract
-  const fileInfo = pbix.detectFileType(sourcePath);
-
-  if (fileInfo.type === 'power-bi-project') {
-    // PBIP - extract TMDL
-    const definitionPath = pbix.getPbipDefinitionPath(sourcePath);
-
-    if (definitionPath) {
-      const targetDef = path.join(project.projectPath, 'definition');
-
-      // Compare directories to show changes
-      const changes = pbix.compareDirectories(targetDef, definitionPath);
-
-      // Copy new content
-      const extraction = pbix.extractPbipToRepo(sourcePath, project.projectPath);
-
-      if (extraction.success) {
-        result.success = true;
-        result.filesChanged =
-          changes.added.length + changes.modified.length + changes.deleted.length;
-        result.message = `Actualizado: +${changes.added.length} ~${changes.modified.length} -${changes.deleted.length}`;
-      } else {
-        result.message = extraction.error;
-      }
-    } else {
-      result.message = 'No se encontró carpeta definition en PBIP';
-    }
-  } else if (fileInfo.type === 'power-bi') {
-    // PBIX - just update modification time and hash
-    result.success = true;
-    result.message = 'Archivo .pbix actualizado (binario, sin diff detallado)';
-
-    // Update the README with latest info
-    const readmePath = path.join(project.projectPath, 'README.md');
-    if (fs.existsSync(readmePath)) {
-      let content = fs.readFileSync(readmePath, 'utf8');
-      const lastMod = pbix.getLastModified(sourcePath);
-
-      // Add update note
-      if (!content.includes('## Historial de Actualizaciones')) {
-        content += '\n\n## Historial de Actualizaciones\n';
-      }
-      content += `\n- ${lastMod?.toISOString()}: Pull desde archivo original`;
-      fs.writeFileSync(readmePath, content);
-    }
-  } else if (fileInfo.type === 'excel' || fileInfo.type === 'excel-macro') {
-    // Excel - update modification tracking
-    result.success = true;
-    result.message = 'Archivo Excel actualizado';
-  } else {
-    result.message = 'Tipo de archivo no soportado para pull';
-  }
-
-  // Update project.json with new hash and sync time
-  if (result.success) {
-    project.source.hash = currentHash;
-    project.source.lastSync = new Date().toISOString();
-
-    const configPath = path.join(project.projectPath, 'project.json');
-    fs.writeFileSync(configPath, JSON.stringify(project, null, 2));
-  }
-
-  return result;
-}
-
-/**
- * Main pull command handler
- */
-async function pullCommand(args, _config) {
-  const options = parseArgs(args);
-
-  if (options.help) {
-    showHelp();
-    return;
-  }
-
-  // Check if repo exists
-  const repoPath = profiles.getRepoPath();
-  if (!repoPath || !fs.existsSync(repoPath)) {
-    console.log(`
-No se encontró el repositorio de BI.
-
-Ejecuta primero:
-  super setup
-`);
-    process.exit(1);
-  }
-
-  console.log(`
-════════════════════════════════════════════════════════════════
-  Pull - Traer cambios de archivos originales
-════════════════════════════════════════════════════════════════
-`);
-
-  let projectsToPull = [];
-
-  if (options.all) {
-    // Pull all projects
-    projectsToPull = getAllProjects(repoPath);
-    console.log(`Proyectos encontrados: ${projectsToPull.length}\n`);
-  } else if (options.projectName) {
-    // Pull specific project
-    const project = getProject(repoPath, options.projectName);
-    if (!project) {
-      console.log(`✗ Proyecto no encontrado: ${options.projectName}`);
-      console.log('\nProyectos disponibles:');
-      getAllProjects(repoPath).forEach((p) => {
-        console.log(`  - ${p.name}`);
-      });
-      process.exit(1);
-    }
-    projectsToPull = [project];
-  } else {
-    // Try to detect current project
-    const current = detectCurrentProject(repoPath);
-    if (current) {
-      projectsToPull = [current];
-    } else {
-      // No project specified, show help
-      console.log('Especifica un proyecto o usa --all:\n');
-      console.log('  super pull <nombre-proyecto>');
-      console.log('  super pull --all\n');
-      console.log('Proyectos disponibles:');
-      getAllProjects(repoPath).forEach((p) => {
-        console.log(`  - ${p.name}`);
-      });
-      return;
-    }
-  }
-
-  if (projectsToPull.length === 0) {
-    console.log('No hay proyectos para actualizar.');
-    return;
-  }
-
-  // Pull each project
-  const results = [];
-
-  for (const project of projectsToPull) {
-    process.stdout.write(`  ${project.name}... `);
-
-    const result = pullProject(project, options.force);
-    results.push(result);
-
-    if (result.success) {
-      console.log(`✓ ${result.message}`);
-    } else {
-      console.log(`✗ ${result.message}`);
-    }
-  }
-
-  // Git commit if there were changes
-  const changedProjects = results.filter((r) => r.success && r.filesChanged > 0);
-
-  if (changedProjects.length > 0 && git.isGitRepo(repoPath)) {
-    console.log('\nGuardando cambios en Git...');
-
-    git.stageFiles(repoPath, '.');
-
-    const projectNames = changedProjects.map((r) => r.project).join(', ');
-    git.commit(repoPath, `Pull updates: ${projectNames}`);
-
-    console.log(`  ✓ Commit: "Pull updates: ${projectNames}"`);
-  }
-
-  // Summary
-  const successful = results.filter((r) => r.success).length;
-  const failed = results.filter((r) => !r.success).length;
-
-  console.log(`
-════════════════════════════════════════════════════════════════
-  Resumen: ${successful} exitosos, ${failed} fallidos
-════════════════════════════════════════════════════════════════
-`);
-}
-
-module.exports = pullCommand;