g360-cli 1.9.0 → 1.10.1

package/src/commands/ingest.js

@@ -4,7 +4,6 @@
  * Procesa archivos ERP y genera maestro_ventas_crm.csv
  */
 
-import { Command } from 'commander';
 import chalk from 'chalk';
 import path from 'path';
 import { fileURLToPath } from 'url';
@@ -13,87 +12,83 @@ import fs from 'fs-extra';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
-export = (program: Command) => {
-  program
-    .command('ingest')
-    .argument('<input>', 'Archivo CSV/Excel o directorio con archivos ERP')
-    .option('-o, --output <archivo>', 'Ruta de salida', 'maestro_ventas_crm.csv')
-    .action(async (input, options) => {
-      console.log(chalk.blue(`\n🚀 Iniciando ingesta: ${input}`));
-
-      const inputPath = path.resolve(input);
-      const outputPath = path.resolve(options.output);
-
-      // Determinar si es archivo o directorio
-      try {
-        await fs.access(inputPath);
-      } catch {
-        console.error(chalk.red(`❌ Ruta no encontrada: ${inputPath}`));
-        process.exit(1);
-      }
+export async function ingest(input, options) {
+  const { output = 'maestro_ventas_crm.csv' } = options;
+  
+  console.log(chalk.blue(`\n🚀 Iniciando ingesta: ${input}`));
 
-      const stat = await fs.stat(inputPath);
-      let filepaths: string[];
-
-      if (stat.isFile()) {
-        filepaths = [inputPath];
-      } else if (stat.isDirectory()) {
-        console.log(chalk.gray(`📁 Escaneando directorio...`));
-        const { valid } = await scanDirectory(inputPath);
-        filepaths = valid.map((info: any) => info.path);
-        if (filepaths.length === 0) {
-          console.error(chalk.red('❌ No se encontraron archivos ERP válidos en el directorio'));
-          process.exit(1);
-        }
-        console.log(chalk.green(`   Encontrados ${filepaths.length} archivos válidos`));
-      } else {
-        console.error(chalk.red(`❌ Ruta no válida: ${inputPath}`));
-        process.exit(1);
-      }
+  const inputPath = path.resolve(input);
+  const outputPath = path.resolve(output);
+
+  // Determinar si es archivo o directorio
+  try {
+    await fs.access(inputPath);
+  } catch {
+    console.error(chalk.red(`❌ Ruta no encontrada: ${inputPath}`));
+    process.exit(1);
+  }
+
+  const stat = await fs.stat(inputPath);
+  let filepaths;
+
+  if (stat.isFile()) {
+    filepaths = [inputPath];
+  } else if (stat.isDirectory()) {
+    console.log(chalk.gray(`📁 Escaneando directorio...`));
+    const result = await scanDirectory(inputPath);
+    filepaths = result.valid.map(info => info.path);
+    if (filepaths.length === 0) {
+      console.error(chalk.red('❌ No se encontraron archivos ERP válidos en el directorio'));
+      process.exit(1);
+    }
+    console.log(chalk.green(`   Encontrados ${filepaths.length} archivos válidos`));
+  } else {
+    console.error(chalk.red(`❌ Ruta no válida: ${inputPath}`));
+    process.exit(1);
+  }
 
-      // Procesar archivos
-      console.log(chalk.blue(`\n⚙️  Procesando ${filepaths.length} archivo(s)...`));
-
-      try {
-        const combinedCsv = await runBatchIngest(filepaths);
-
-        // Escribir salida
-        await fs.ensureDir(path.dirname(outputPath));
-        await fs.writeFile(outputPath, combinedCsv, 'utf-8');
-        console.log(chalk.green(`\n✅ Ingesta completada: ${outputPath}`));
-
-        // Resumen por archivo
-        const lines = combinedCsv.split('\n');
-        const header = lines[0];
-        const dataLines = lines.filter(l => l && l !== header);
-        console.log(chalk.gray(`   Total filas: ${dataLines.length}`));
-
-        // Conteo por ARCHIVO_ORIGEN
-        const counts = new Map<string, number>();
-        for (const line of dataLines) {
-          const cols = line.split(',');
-          const idx = header.split(',').indexOf('ARCHIVO_ORIGEN');
-          if (idx !== -1 && cols[idx]) {
-            counts.set(cols[idx], (counts.get(cols[idx]) || 0) + 1);
-          }
-        }
-        if (counts.size > 0) {
-          console.log(chalk.cyan('\n📊 Resumen por archivo:'));
-          for (const [file, count] of counts) {
-            console.log(`   ${file}: ${count} filas`);
-          }
-        }
-
-      } catch (err: any) {
-        console.error(chalk.red('\n❌ Error durante la ingesta:'), err.message);
-        if (err.stdout) console.error(chalk.gray(err.stdout));
-        if (err.stderr) console.error(chalk.red(err.stderr));
-        process.exit(1);
+  // Procesar archivos
+  console.log(chalk.blue(`\n⚙️  Procesando ${filepaths.length} archivo(s)...`));
+
+  try {
+    const combinedCsv = await runBatchIngest(filepaths);
+
+    // Escribir salida
+    await fs.ensureDir(path.dirname(outputPath));
+    await fs.writeFile(outputPath, combinedCsv, 'utf-8');
+    console.log(chalk.green(`\n✅ Ingesta completada: ${outputPath}`));
+
+    // Resumen por archivo
+    const lines = combinedCsv.split('\n');
+    const header = lines[0];
+    const dataLines = lines.filter(l => l && l !== header);
+    console.log(chalk.gray(`   Total filas: ${dataLines.length}`));
+
+    // Conteo por ARCHIVO_ORIGEN
+    const counts = new Map();
+    for (const line of dataLines) {
+      const cols = line.split(',');
+      const idx = header.split(',').indexOf('ARCHIVO_ORIGEN');
+      if (idx !== -1 && cols[idx]) {
+        counts.set(cols[idx], (counts.get(cols[idx]) || 0) + 1);
       }
-    });
-};
+    }
+    if (counts.size > 0) {
+      console.log(chalk.cyan('\n📊 Resumen por archivo:'));
+      for (const [file, count] of counts) {
+        console.log(`   ${file}: ${count} filas`);
+      }
+    }
 
-async function scanDirectory(dir: string): Promise<{ valid: any[]; invalid: any[] }> {
+  } catch (err) {
+    console.error(chalk.red('\n❌ Error durante la ingesta:'), err.message);
+    if (err.stdout) console.error(chalk.gray(err.stdout));
+    if (err.stderr) console.error(chalk.red(err.stderr));
+    process.exit(1);
+  }
+}
+
+async function scanDirectory(dir) {
   const pyCode = `
 import sys
 sys.path.insert(0, '${path.join(__dirname, '..', 'py', 'src')}')
@@ -109,8 +104,8 @@ for i in invalid[:10]:
 
   const result = await runPython(pyCode);
   const lines = result.stdout.split('\n').filter(l => l.trim());
-  const valid: any[] = [];
-  const invalid: any[] = [];
+  const valid = [];
+  const invalid = [];
 
   for (const line of lines) {
     if (line.startsWith('VALID::')) {
@@ -125,7 +120,7 @@ for i in invalid[:10]:
   return { valid, invalid };
 }
 
-async function runBatchIngest(filepaths: string[]): Promise<string> {
+async function runBatchIngest(filepaths) {
   const pyCode = `
 import sys
 sys.path.insert(0, '${path.join(__dirname, '..', 'py', 'src')}')
@@ -135,7 +130,6 @@ import pandas as pd
 
 filepaths = [${JSON.stringify(filepaths).replace(/"/g, "'")}]
 df = batch_process_files([Path(p) for p in filepaths], merge_results=True)
-# Output as CSV to stdout
 sys.stdout.write(df.to_csv(index=False))
 `;
 
@@ -165,7 +159,7 @@ sys.stdout.write(df.to_csv(index=False))
   });
 }
 
-function runPython(code: string): Promise<{ stdout: string; stderr: string }> {
+function runPython(code) {
   return new Promise((resolve, reject) => {
     const pyExec = process.env.PYTHON || 'python3';
     const proc = spawn(pyExec, ['-c', code], {
@@ -190,4 +184,4 @@ function runPython(code: string): Promise<{ stdout: string; stderr: string }> {
       reject(new Error(`No se pudo ejecutar Python: ${err.message}`));
     });
   });
-}
+}

package/src/commands/init.js

@@ -40,10 +40,21 @@ export async function init(name, options) {
 
   const targetDir = path.join(process.cwd(), dir, name);
 
+  // Leer version del CLI desde package.json
+  const cliPkgPath = path.join(__dirname, '..', '..', 'package.json');
+  let cliVersion = '1.0.0';
+  try {
+    const cliPkg = fs.readJsonSync(cliPkgPath);
+    cliVersion = cliPkg.version || '1.0.0';
+  } catch {
+    // fallback si no se puede leer
+  }
+
   console.log(chalk.bold.cyan('\n🚀 G360 Project Initialization\n'));
   console.log(`Project: ${chalk.yellow(name)}`);
   console.log(`Template: ${chalk.blue(template)}`);
   console.log(`Skill: ${chalk.magenta(skill)}`);
+  console.log(`CLI Version: ${chalk.gray(cliVersion)}`);
   console.log(`Target: ${chalk.gray(targetDir)}\n`);
 
   let wantPortable = false;

package/src/commands/init.test.js

@@ -0,0 +1,19 @@
+/**
+ * @file init.test.js
+ * @description Tests para el comando init
+ */
+
+import { describe, it, expect } from 'vitest';
+
+describe('init command', () => {
+  it('should export init function', async () => {
+    const mod = await import('../commands/init.js');
+    expect(mod).toHaveProperty('init');
+    expect(typeof mod.init).toBe('function');
+  });
+
+  it('should accept name and options parameters', async () => {
+    const { init } = await import('../commands/init.js');
+    expect(init.length).toBe(2);
+  });
+});

package/src/commands/list.test.js

@@ -0,0 +1,132 @@
+/**
+ * @file list.test.js
+ * @description Tests para el comando list
+ */
+
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+import fs from 'fs-extra';
+import path from 'path';
+import { fileURLToPath } from 'url';
+import { list } from '../commands/list.js';
+
+const __dirname = path.dirname(fileURLToPath(import.meta.url));
+
+// Mock de fs-extra
+vi.mock('fs-extra', () => ({
+  default: {
+    existsSync: vi.fn(),
+    readdirSync: vi.fn(),
+    readJsonSync: vi.fn()
+  }
+}));
+
+describe('list command', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+  });
+
+  describe('list templates', () => {
+    it('should list available templates', async () => {
+      const mockTemplates = ['web-pwa', 'svelte-web', 'solid-web'];
+      fs.existsSync.mockReturnValue(true);
+      fs.readdirSync.mockReturnValue(mockTemplates);
+
+      await list('templates', { json: false });
+
+      expect(fs.readdirSync).toHaveBeenCalled();
+    });
+
+    it('should return empty array when templates directory does not exist', async () => {
+      fs.existsSync.mockReturnValue(false);
+
+      await list('templates', { json: false });
+
+      expect(fs.readdirSync).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('list skills', () => {
+    it('should list skills from g360-skills.json', async () => {
+      const mockSkills = {
+        skills: [
+          { name: 'corporativo', description: 'Proyectos corporativos', device: 'pc' },
+          { name: 'moderno', description: 'Herramientas modernas', device: 'movil' }
+        ]
+      };
+
+      fs.existsSync.mockReturnValue(true);
+      fs.readJsonSync.mockReturnValue(mockSkills);
+
+      await list('skills', { json: false });
+
+      expect(fs.readJsonSync).toHaveBeenCalled();
+    });
+
+    it('should handle missing g360-skills.json gracefully', async () => {
+      fs.existsSync.mockReturnValue(false);
+
+      await list('skills', { json: false });
+
+      expect(fs.readJsonSync).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('list snippets', () => {
+    it('should list snippets from snippets.json', async () => {
+      const mockSnippets = {
+        snippets: [
+          { name: 'cli-argparse-basic', description: 'Basic argparse CLI', language: 'python' },
+          { name: 'g360-button', description: 'G360 styled button', language: 'html' }
+        ]
+      };
+
+      fs.existsSync.mockReturnValue(true);
+      fs.readJsonSync.mockReturnValue(mockSnippets);
+
+      await list('snippets', { json: false });
+
+      expect(fs.readJsonSync).toHaveBeenCalled();
+    });
+
+    it('should handle missing snippets.json gracefully', async () => {
+      fs.existsSync.mockReturnValue(false);
+
+      await list('snippets', { json: false });
+
+      expect(fs.readJsonSync).not.toHaveBeenCalled();
+    });
+  });
+
+  describe('list all', () => {
+    it('should list all asset types', async () => {
+      const mockTemplates = ['web-pwa', 'svelte-web'];
+      const mockSkills = {
+        skills: [{ name: 'corporativo', description: 'Proyectos corporativos', device: 'pc' }]
+      };
+      const mockSnippets = {
+        snippets: [{ name: 'cli-argparse-basic', description: 'Basic argparse', language: 'python' }]
+      };
+
+      fs.existsSync.mockReturnValue(true);
+      fs.readdirSync.mockReturnValue(mockTemplates);
+      fs.readJsonSync.mockReturnValue(mockSkills).mockReturnValueOnce(mockSnippets);
+
+      await list('all', { json: false });
+
+      expect(fs.readdirSync).toHaveBeenCalled();
+      expect(fs.readJsonSync).toHaveBeenCalled();
+    });
+  });
+
+  describe('JSON output', () => {
+    it('should output JSON when requested', async () => {
+      const mockTemplates = ['web-pwa'];
+      fs.existsSync.mockReturnValue(true);
+      fs.readdirSync.mockReturnValue(mockTemplates);
+
+      await list('templates', { json: true });
+
+      expect(fs.readdirSync).toHaveBeenCalled();
+    });
+  });
+});

package/src/commands/scan.js

@@ -4,99 +4,87 @@
  * Escanea un directorio para detectar archivos ERP válidos.
  */
 
-import { Command } from 'commander';
 import chalk from 'chalk';
 import path from 'path';
 import { fileURLToPath } from 'url';
+import { spawn } from 'child_process';
 
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 
-export = (program: Command) => {
-  program
-    .command('scan')
-    .argument('<directorio>', 'Directorio a escanear')
-    .option('-r, --recursive', 'Buscar recursivamente', true)
-    .option('--min-score <n>', 'Puntuación mínima', '10')
-    .action(async (directorio, options) => {
-      console.log(chalk.blue(`\n🔍 Escaneando directorio: ${directorio}`));
+export async function scan(directory, options) {
+  const { recursive = true, minScore = 10 } = options;
+  
+  console.log(chalk.blue(`\n🔍 Escaneando directorio: ${directory}`));
 
-      const pyCode = `
+  const pyCode = `
 import sys
+sys.path.insert(0, '${path.join(__dirname, '..', 'py', 'src')}')
+from g360_core.scanner import find_erp_files_in_dir
 from pathlib import Path
-
-from g360_core.scanner import ERPScanner
-
-scanner = ERPScanner(min_valid_score=${options.minScore})
-files = scanner.scan_directory(Path('${directorio}'), recursive=${options.recursive})
-valid = scanner.get_valid_files(files)
-invalid = scanner.get_invalid_files(files)
-
-print(f"TOTAL={len(files)}")
-print(f"VALIDOS={len(valid)}")
-print(f"INVALIDOS={len(invalid)}")
-
-for erp_type, flist in scanner.group_by_erp_type(files).items():
-    if erp_type not in ('UNKNOWN', 'ERROR'):
-        print(f"TIPO::{erp_type}::{len(flist)}")
-        for info in flist[:5]:
-            print(f"FILE::{info.path.name}::{info.size_bytes}")
-
-for info in invalid[:10]:
-    print(f"INVALID::{info.path.name}::${info.error_msg}")
+import json
+
+valid, invalid = find_erp_files_in_dir(Path('${directory}'), recursive=${recursive})
+result = {
+    "valid": [{"path": str(v.path), "erp_type": v.erp_type, "size_bytes": v.size_bytes} for v in valid],
+    "invalid": [{"path": str(i.path), "error_msg": i.error_msg} for i in invalid],
+    "stats": {"total": len(valid) + len(invalid), "valid": len(valid), "invalid": len(invalid)}
+}
+print(json.dumps(result, indent=2))
 `;
 
-      try {
-        const { runPython } = await import('../lib/python_runner.js');
-        const result = await runPython(pyCode);
-        const lines = result.stdout.split('\n').filter(l => l.trim());
-
-        let total = 0, validCount = 0, invalidCount = 0;
-        const types = new Map();
-
-        for (const line of lines) {
-          if (line.startsWith('TOTAL=')) total = parseInt(line.split('=')[1]);
-          else if (line.startsWith('VALIDOS=')) validCount = parseInt(line.split('=')[1]);
-          else if (line.startsWith('INVALIDOS=')) invalidCount = parseInt(line.split('=')[1]);
-          else if (line.startsWith('TIPO::')) {
-            const [, tipo, count] = line.split('::');
-            types.set(tipo, parseInt(count));
-          }
-        }
-
-        console.log(chalk.gray(`\n📈 Resultados:`));
-        console.log(`   Total archivos: ${total}`);
-        console.log(chalk.green(`   Válidos: ${validCount}`));
-        console.log(chalk.red(`   Inválidos/Error: ${invalidCount}`));
-
-        if (types.size > 0) {
-          console.log(chalk.green('\n✅ Archivos válidos por tipo ERP:'));
-          for (const [tipo, count] of types) {
-            console.log(`  ${tipo}: ${count} archivos`);
-          }
-        }
+  try {
+    const result = await runPython(pyCode);
+    const data = JSON.parse(result.stdout);
+    
+    console.log(chalk.gray(`\n   Total archivos: ${data.stats.total}`));
+    console.log(chalk.green(`   Válidos: ${data.stats.valid}`));
+    console.log(chalk.red(`   Inválidos: ${data.stats.invalid}`));
+    
+    if (data.valid.length > 0) {
+      console.log(chalk.cyan('\n📋 Archivos válidos:'));
+      data.valid.forEach(f => {
+        console.log(chalk.gray(`   ${f.path} (${f.erp_type})`));
+      });
+    }
+    
+    if (data.invalid.length > 0) {
+      console.log(chalk.yellow('\n⚠️  Archivos inválidos:'));
+      data.invalid.forEach(f => {
+        console.log(chalk.red(`   ${f.path}: ${f.error_msg}`));
+      });
+    }
+    
+  } catch (err) {
+    console.error(chalk.red(`\n❌ Error escaneando: ${err.message}`));
+    if (err.stdout) console.log(chalk.gray(err.stdout));
+    if (err.stderr) console.error(chalk.red(err.stderr));
+    process.exit(1);
+  }
+}
+
+function runPython(code) {
+  return new Promise((resolve, reject) => {
+    const pyExec = process.env.PYTHON || 'python3';
+    const proc = spawn(pyExec, ['-c', code], {
+      stdio: ['ignore', 'pipe', 'pipe'],
+    });
 
-        if (invalidCount > 0) {
-          console.log(chalk.red('\n❌ Archivos con problemas:'));
-          let invalidLines = lines.filter(l => l.startsWith('INVALID::'));
-          for (const line of invalidLines.slice(0, 10)) {
-            const parts = line.split('::');
-            console.log(`  • ${parts[1]}: ${parts.slice(2).join('::')}`);
-          }
-          if (invalidLines.length > 10) {
-            console.log(`  ... y ${invalidLines.length - 10} más`);
-          }
-        }
+    let stdout = '';
+    let stderr = '';
 
-        if (validCount > 0) {
-          console.log(chalk.cyan('\n💡 Para procesar todos estos archivos:'));
-          console.log(`   g360 ingest "${directorio}" -o maestro_ventas_crm.csv`);
-        }
+    proc.stdout?.on('data', (data) => { stdout += data.toString(); });
+    proc.stderr?.on('data', (data) => { stderr += data.toString(); });
 
-      } catch (err: any) {
-        console.error(chalk.red('❌ Error durante escaneo:'), err.message);
-        if (err.stdout) console.error(chalk.gray(err.stdout));
-        if (err.stderr) console.error(chalk.red(err.stderr));
-        process.exit(1);
+    proc.on('close', (code) => {
+      if (code === 0) {
+        resolve({ stdout, stderr });
+      } else {
+        reject(new Error(stderr || `Python terminó con código ${code}`));
       }
     });
-};
+
+    proc.on('error', (err) => {
+      reject(new Error(`No se pudo ejecutar Python: ${err.message}`));
+    });
+  });
+}

package/src/commands/set-skill.js

@@ -12,11 +12,13 @@ import { fileURLToPath } from 'url';
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const SKILLS_PATH = path.resolve(__dirname, '../assets/config/g360-skills.json');
 
-export async function setSkill(skillName, options) {
-  const { verbose = false, cwd = process.cwd() } = options;
+export async function setSkill(skillName, options = {}) {
+  const { verbose = false, cwd = process.cwd(), force = false } = options;
   const isInternalCall = options.cwd !== undefined;
   
-  console.log(chalk.bold.cyan('\n🎨 G360 Skill Selector\n'));
+  if (!isInternalCall) {
+    console.log(chalk.bold.cyan('\n🎨 G360 Skill Selector\n'));
+  }
   
   // Cargar skills disponibles
   let skillsConfig;
@@ -43,9 +45,9 @@ export async function setSkill(skillName, options) {
   const skillJsonPath = path.join(cwd, 'skill.json');
   
   if (fs.existsSync(skillJsonPath)) {
-    console.log(chalk.yellow('⚠️  El proyecto ya tiene un skill configurado.'));
-    console.log(chalk.gray('Usar --force para sobrescribir'));
-    if (!options.force) {
+    if (!force) {
+      console.log(chalk.yellow('⚠️  El proyecto ya tiene un skill configurado.'));
+      console.log(chalk.gray('Usar --force para sobrescribir'));
       console.log(chalk.cyan('\nPara cambiar el skill:'));
       console.log(chalk.cyan('  g360 set-skill ') + skillName + chalk.cyan(' --force'));
       return;
@@ -65,17 +67,19 @@ export async function setSkill(skillName, options) {
     
     await fs.writeJson(skillJsonPath, skillData, { spaces: 2 });
     
-    console.log(chalk.green(`\n✅ Skill "${skillName}" configurado correctamente`));
-    console.log(chalk.gray('\nDetalles:'));
-    console.log(`  Device: ${skill.device}`);
-    console.log(`  Accent: ${skill.colors.accent}`);
-    console.log(`  Signature: ${skill.signature.mode}`);
-    
-    if (verbose) {
-      console.log(chalk.gray('\nColores:'));
-      Object.entries(skill.colors).forEach(([key, value]) => {
-        console.log(`  ${key}: ${value}`);
-      });
+    if (!isInternalCall) {
+      console.log(chalk.green(`\n✅ Skill "${skillName}" configurado correctamente`));
+      console.log(chalk.gray('\nDetalles:'));
+      console.log(`  Device: ${skill.device}`);
+      console.log(`  Accent: ${skill.colors.accent}`);
+      console.log(`  Signature: ${skill.signature.mode}`);
+
+      if (verbose) {
+        console.log(chalk.gray('\nColores:'));
+        Object.entries(skill.colors).forEach(([key, value]) => {
+          console.log(`  ${key}: ${value}`);
+        });
+      }
     }
     
   } catch (error) {