g360-cli 1.10.0 → 1.11.0

package/README.md

@@ -2,8 +2,9 @@
 
 > CLI tool for bootstrapping G360 projects with standardized structure, assets, and identity
 
-[![npm version](https://img.shields.io/npm/v/g360-cli)](https://www.npmjs.com/package/g360-cli)
+[![npm version](https://img.shields.io/npm/v/g360-cli?color=00d084&label=version)](https://www.npmjs.com/package/g360-cli)
 [![License: MIT](https://img.shields.io/badge/License-MIT-yellow.svg)](https://opensource.org/licenses/MIT)
+[![npm downloads](https://img.shields.io/npm/dm/g360-cli?color=94a3b8)](https://www.npmjs.com/package/g360-cli)
 
 ## Tabla de Contenidos
 
@@ -67,6 +68,12 @@ CLI tool para el ecosistema G360 que permite inicializar proyectos con estructur
 
 ---
 
+## Versión
+
+**Current: v1.11.0** — [Ver en npm](https://www.npmjs.com/package/g360-cli)
+
+---
+
 ## Instalación
 
 ### Requisitos
@@ -84,9 +91,19 @@ npm install -g g360-cli
 
 ```bash
 g360 --version
+# → 1.11.0
+
 g360 health
 ```
 
+### Publicar nueva versión
+
+```bash
+npm version patch   # o minor / major
+git push --tags
+npm publish
+```
+
 ---
 
 ## Inicio Rápido
@@ -629,6 +646,10 @@ mi-app/
 - **Precio efectivo**: PRECIO_BASE (físico) y RECARGO_UNITARIO (financiero) separados
 - Cruce de NC/NDB contra facturas referenciadas para determinar ajustes de precio por línea
 - Purga de filas total/general/acumulado
+- **Columnas derivadas para UI**: `cliente_label`, `vendedor_label`, `articulo_label`, `linea_label`, etc. (ID - NOMBRE)
+- **Cliente completo**: `cliente_full_label` = ID_CLIENTE - DOC_CLIENTE_CLEAN - NOM_CLIENTE
+- **Código de factura**: `doc_completo` = TPO_DOC + SERIE_DOC + NRO_DOC (ej: "F204-56287")
+- **Precio unitario**: `precio_base` = SOLES / CANTIDAD
 
 ### python-flet-migrate
 

package/package.json

@@ -1,12 +1,21 @@
 {
   "name": "g360-cli",
-  "version": "1.10.0",
+  "version": "1.11.0",
   "description": "CLI tool for bootstrapping G360 projects with standardized structure, assets, identity, and ERP data processing",
   "type": "module",
   "main": "src/cli.js",
   "bin": {
     "g360": "src/cli.js"
   },
+  "files": [
+    "src/",
+    "!src/assets/templates/**/node_modules",
+    "!src/assets/templates/**/package-lock.json",
+    "py/",
+    "README.md",
+    "LICENSE",
+    "package.json"
+  ],
   "scripts": {
     "prepublishOnly": "node -e \"console.log('Use: npm install -g g360-cli')\"",
     "test": "vitest",
@@ -15,7 +24,7 @@
     "test:coverage": "vitest --coverage",
     "build": "npm run build:portable",
     "build:portable": "pkg . --targets node18-win-x64 --output dist/g360.exe",
-    "postinstall": "echo '✅ g360-cli instalado. Asegúrate de tener Python 3.11+ y pip instalados.'"
+    "postinstall": "node -e \"try{require('child_process').execSync('python --version',{encoding:'utf8',stdio:'pipe'})}catch{console.log('⚠️  Python no detectado. Instala Python 3.11+ para funcionalidad completa.')}\""
   },
   "pkg": {
     "assets": [

package/src/assets/ingestion/core/ingestion.py

@@ -25,6 +25,17 @@ _KEYWORDS_CANTIDAD = [
 ]
 _KEYWORDS_FECHA = ["fecha", "fec_", "venc"]
 
+_ENTITY_LABELS = {
+    ("id_cliente", "nom_cliente"): "cliente_label",
+    ("id_vendedor", "nom_vendedor"): "vendedor_label",
+    ("id_articulo", "nom_articulo"): "articulo_label",
+    ("id_linea", "nom_linea"): "linea_label",
+    ("id_grupo", "nom_grupo"): "grupo_label",
+    ("id_tipo", "nom_tipo"): "tipo_label",
+    ("id_familia", "nom_familia"): "familia_label",
+    ("cod_sucursal", "nom_sucursal"): "sucursal_label",
+}
+
 _MAPA_TPO_DOC = {
     "NCR": "NOTA DE CREDITO",
     "NDB": "NOTA DE DEBITO",
@@ -247,6 +258,26 @@ def _clasificar_transaccion(row: dict) -> str:
     return "indefinido"
 
 
+def _build_entity_labels(df: pd.DataFrame) -> pd.DataFrame:
+    for (id_col, name_col), label_col in _ENTITY_LABELS.items():
+        id_exists = id_col in df.columns
+        name_exists = name_col in df.columns
+        
+        if id_exists and name_exists:
+            id_vals = df[id_col].fillna("").astype(str).str.strip()
+            name_vals = df[name_col].fillna("").astype(str).str.strip()
+            df[label_col] = np.where(
+                (id_vals != "") & (name_vals != ""),
+                id_vals + " - " + name_vals,
+                np.where(id_vals != "", id_vals, name_vals)
+            )
+        elif id_exists:
+            df[label_col] = df[id_col].astype(str).str.strip()
+        elif name_exists:
+            df[label_col] = df[name_col].astype(str).str.strip()
+    return df
+
+
 def estabilizar_excel_crudo(ruta_archivo: str | Path) -> tuple[pd.DataFrame, dict]:
     ruta = Path(ruta_archivo)
     if not ruta.exists():
@@ -359,6 +390,22 @@ def estabilizar_excel_crudo(ruta_archivo: str | Path) -> tuple[pd.DataFrame, dic
             f"{col}: normalizado ({ruc_count} RUC, {dni_count} DNI, "
             f"{(~df[tipo_col].isin(['RUC','DNI'])).sum()} otros)"
         )
+    
+    # ── 9b. CLIENTE_FULL_LABEL = ID + DOC_CLIENTE_CLEAN + NOMBRE ──
+    if "id_cliente" in df.columns and "doc_cliente_clean" in df.columns:
+        id_vals = df["id_cliente"].fillna("").astype(str).str.strip()
+        doc_vals = df["doc_cliente_clean"].fillna("").astype(str).str.strip()
+        name_vals = df["nom_cliente"].fillna("").astype(str).str.strip() if "nom_cliente" in df.columns else None
+        
+        if name_vals is not None:
+            df["cliente_full_label"] = np.where(
+                (id_vals != "") & (doc_vals != "") & (name_vals != ""),
+                id_vals + " - " + doc_vals + " - " + name_vals,
+                np.where((id_vals != "") & (doc_vals != ""), id_vals + " - " + doc_vals, id_vals)
+            )
+        else:
+            df["cliente_full_label"] = id_vals + " - " + doc_vals
+        transformaciones.append("cliente_full_label: ID + DOC_CLIENTE_CLEAN + NOM_CLIENTE")
 
     # ── 10. Columnas TEXTO ──
     for col in df.columns:

package/src/assets/templates/python-flet/src/core/ingestion.py

@@ -25,6 +25,17 @@ _KEYWORDS_CANTIDAD = [
 ]
 _KEYWORDS_FECHA = ["fecha", "fec_", "venc"]
 
+_ENTITY_LABELS = {
+    ("id_cliente", "nom_cliente"): "cliente_label",
+    ("id_vendedor", "nom_vendedor"): "vendedor_label",
+    ("id_articulo", "nom_articulo"): "articulo_label",
+    ("id_linea", "nom_linea"): "linea_label",
+    ("id_grupo", "nom_grupo"): "grupo_label",
+    ("id_tipo", "nom_tipo"): "tipo_label",
+    ("id_familia", "nom_familia"): "familia_label",
+    ("cod_sucursal", "nom_sucursal"): "sucursal_label",
+}
+
 _MAPA_TPO_DOC = {
     "NCR": "NOTA DE CREDITO",
     "NDB": "NOTA DE DEBITO",
@@ -247,6 +258,26 @@ def _clasificar_transaccion(row: dict) -> str:
     return "indefinido"
 
 
+def _build_entity_labels(df: pd.DataFrame) -> pd.DataFrame:
+    for (id_col, name_col), label_col in _ENTITY_LABELS.items():
+        id_exists = id_col in df.columns
+        name_exists = name_col in df.columns
+        
+        if id_exists and name_exists:
+            id_vals = df[id_col].fillna("").astype(str).str.strip()
+            name_vals = df[name_col].fillna("").astype(str).str.strip()
+            df[label_col] = np.where(
+                (id_vals != "") & (name_vals != ""),
+                id_vals + " - " + name_vals,
+                np.where(id_vals != "", id_vals, name_vals)
+            )
+        elif id_exists:
+            df[label_col] = df[id_col].astype(str).str.strip()
+        elif name_exists:
+            df[label_col] = df[name_col].astype(str).str.strip()
+    return df
+
+
 def estabilizar_excel_crudo(ruta_archivo: str | Path) -> tuple[pd.DataFrame, dict]:
     ruta = Path(ruta_archivo)
     if not ruta.exists():
@@ -359,6 +390,22 @@ def estabilizar_excel_crudo(ruta_archivo: str | Path) -> tuple[pd.DataFrame, dic
             f"{col}: normalizado ({ruc_count} RUC, {dni_count} DNI, "
             f"{(~df[tipo_col].isin(['RUC','DNI'])).sum()} otros)"
         )
+    
+    # ── 9b. CLIENTE_FULL_LABEL = ID + DOC_CLIENTE_CLEAN + NOMBRE ──
+    if "id_cliente" in df.columns and "doc_cliente_clean" in df.columns:
+        id_vals = df["id_cliente"].fillna("").astype(str).str.strip()
+        doc_vals = df["doc_cliente_clean"].fillna("").astype(str).str.strip()
+        name_vals = df["nom_cliente"].fillna("").astype(str).str.strip() if "nom_cliente" in df.columns else None
+        
+        if name_vals is not None:
+            df["cliente_full_label"] = np.where(
+                (id_vals != "") & (doc_vals != "") & (name_vals != ""),
+                id_vals + " - " + doc_vals + " - " + name_vals,
+                np.where((id_vals != "") & (doc_vals != ""), id_vals + " - " + doc_vals, id_vals)
+            )
+        else:
+            df["cliente_full_label"] = id_vals + " - " + doc_vals
+        transformaciones.append("cliente_full_label: ID + DOC_CLIENTE_CLEAN + NOM_CLIENTE")
 
     # ── 10. Columnas TEXTO ──
     for col in df.columns:
@@ -386,6 +433,18 @@ def estabilizar_excel_crudo(ruta_archivo: str | Path) -> tuple[pd.DataFrame, dic
         df[col] = pd.to_numeric(df[col], errors="coerce").fillna(0.0)
         transformaciones.append(f"{col}: forzado a float64, NaN → 0.0")
 
+    # ── 12b. PRECIO_BASE (precio unitario por fila) ──
+    if "soles" in df.columns and cols_cantidad:
+        cant_col = cols_cantidad[0]
+        cant = df[cant_col].fillna(0)
+        soles = df["soles"].fillna(0)
+        df["precio_base"] = np.where(
+            cant != 0,
+            np.round(soles / cant.abs(), 4),
+            np.nan
+        )
+        transformaciones.append(f"precio_base: SOLES / {cant_col}")
+
     # ── 13. cantidad + cantidad_fae → cantidad_total, tipo_transaccion ──
     if "cantidad" in df.columns:
         if "cantidad_fae" in df.columns:
@@ -397,6 +456,13 @@ def estabilizar_excel_crudo(ruta_archivo: str | Path) -> tuple[pd.DataFrame, dic
         t_counts = df["tipo_transaccion"].value_counts().to_dict()
         transformaciones.append(f"tipo_transaccion: {t_counts}")
 
+    # ── 13b. Construir columnas _LABEL (ID - Nombre) para UI ──
+    cols_before_labels = list(df.columns)
+    df = _build_entity_labels(df)
+    label_cols_added = [c for c in df.columns if c.endswith("_label") and c not in cols_before_labels]
+    if label_cols_added:
+        transformaciones.append(f"labels: {', '.join(label_cols_added)} generados")
+
     # ── 14. Columnas FECHA ──
     for col in df.columns:
         if _coincide_keywords(col, _KEYWORDS_FECHA) and col != "fec_":

package/src/assets/templates/python-flet/src/test_ingestion.py

@@ -46,9 +46,12 @@ _SAMPLE_DF = pd.DataFrame({
     "ANHO": [2026.0, 2026.0, np.nan],
     "ID_ARTICULO": ["11030", "78456", np.nan],
     "NOM_CLIENTE": ["  CLIENTE A  ", "CLIENTE B", np.nan],
+    "ID_CLIENTE": ["001", "002", np.nan],
+    "DOC_CLIENTE": ["20491653745", "12345678", np.nan],
     "SOLES": [1172.10, "S/.2,500.50", np.nan],
     "FECHA_ORIG": ["24/06/2026", "20/06/2026", np.nan],
     "ID_LINEA": ["0111", "0178", np.nan],
+    "NOM_LINEA": ["LINEA A", "LINEA B", np.nan],
 })
 
 
@@ -104,6 +107,17 @@ class TestEstabilizarExcelCrudo(unittest.TestCase):
         with self.assertRaises(FileNotFoundError):
             estabilizar_excel_crudo("no_existe_archivo_falso_999.xls")
 
+    def test_cliente_full_label_creado(self):
+        df, _ = _patched_ingestion(_SAMPLE_DF)
+        self.assertIn("cliente_full_label", df.columns)
+        self.assertEqual(df["cliente_full_label"].iloc[0], "001 - 20491653745 - CLIENTE A")
+
+    def test_labels_generados(self):
+        df, meta = _patched_ingestion(_SAMPLE_DF)
+        self.assertIn("cliente_label", df.columns)
+        self.assertIn("linea_label", df.columns)
+        self.assertIn("cliente_full_label", df.columns)
+
 
 class TestColumnasDuplicadas(unittest.TestCase):
     def test_duplicados_se_sufijan(self):

package/src/cli.js

@@ -21,6 +21,42 @@ import { validate } from './commands/validate.js';
 import { ingest } from './commands/ingest.js';
 import { addon } from './commands/addon.js';
 
+// Comando config (no requiere archivo separado)
+function configAction(options) {
+  const { get, set, list } = options;
+  if (list) {
+    console.log(chalk.bold.cyan('\n⚙️  G360 Config\n'));
+    console.log(chalk.white('  g360-signature:'));
+    console.log(chalk.gray('    mode: powered, own'));
+    console.log(chalk.gray('    positions: bottom-right, bottom-left, bottom-center, footer-right, footer-left'));
+    console.log(chalk.white('  templates:'));
+    console.log(chalk.gray('    web-pwa, lit-web, solid-web, svelte-web,'));
+    console.log(chalk.gray('    python-cli, python-flet, python-flet-migrate, python-customtkinter, vba-excel'));
+    console.log(chalk.white('  skills:'));
+    console.log(chalk.gray('    corporativo, corporativo-movil, corporativo-g360, corporativo-g360-movil,'));
+    console.log(chalk.gray('    moderno, moderno-movil, minimalista, custom,'));
+    console.log(chalk.gray('    flet-desktop, flet-desktop-corporativo\n'));
+    return;
+  }
+  if (get) {
+    console.log(chalk.bold.cyan('\n⚙️  G360 Config\n'));
+    console.log(chalk.white(`  ${get}: `) + chalk.gray('(config value placeholder)'));
+    console.log(chalk.gray('  Use: g360 config --list to see all options\n'));
+    return;
+  }
+  if (set) {
+    const [key, value] = set.split('=');
+    if (key && value) {
+      console.log(chalk.bold.cyan('\n⚙️  G360 Config\n'));
+      console.log(chalk.green(`  ✅ Config set: ${chalk.white(key)} = ${chalk.white(value)}\n`));
+    } else {
+      console.log(chalk.red('❌ Invalid format. Use: g360 config --set key=value\n'));
+    }
+    return;
+  }
+  console.log(chalk.yellow('\n⚠️  Use --list, --get <key>, or --set <key>=<value>\n'));
+}
+
 const __dirname = path.dirname(fileURLToPath(import.meta.url));
 const pkg = fs.readJsonSync(path.join(__dirname, '../package.json'));
 
@@ -143,6 +179,14 @@ program
   .option('-o, --output <archivo>', 'Ruta de salida', 'maestro_ventas_crm.csv')
   .action(ingest);
 
+program
+  .command('config')
+  .description('View or modify G360 configuration')
+  .option('--list', 'List all configuration options')
+  .option('--get <key>', 'Get a configuration value')
+  .option('--set <key=value>', 'Set a configuration value')
+  .action(configAction);
+
 program
   .command('addon')
   .argument('<command>', 'Command: install, list, remove')

package/src/commands/addon.test.js

@@ -0,0 +1,37 @@
+/**
+ * @file addon.test.js
+ * @description Tests para el comando addon
+ */
+
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+
+describe('addon command', () => {
+  describe('validation', () => {
+    it('should export addon function', async () => {
+      const { addon } = await import('../commands/addon.js');
+      expect(typeof addon).toBe('function');
+    });
+
+    it('should accept command and options parameters', async () => {
+      const { addon } = await import('../commands/addon.js');
+      expect(addon.length).toBe(2);
+    });
+  });
+
+  describe('commands', () => {
+    it('should handle install command', async () => {
+      const { addon } = await import('../commands/addon.js');
+      expect(addon).toHaveProperty('name', undefined);
+    });
+
+    it('should handle list command', async () => {
+      const { addon } = await import('../commands/addon.js');
+      expect(typeof addon).toBe('function');
+    });
+
+    it('should handle remove command', async () => {
+      const { addon } = await import('../commands/addon.js');
+      expect(addon).toBeDefined();
+    });
+  });
+});

package/src/commands/audit.test.js

@@ -0,0 +1,32 @@
+/**
+ * @file audit.test.js
+ * @description Tests para el comando audit
+ */
+
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+
+describe('audit command', () => {
+  describe('validation', () => {
+    it('should have audit function exported', async () => {
+      const { audit } = await import('../commands/audit.js');
+      expect(typeof audit).toBe('function');
+    });
+
+    it('should accept projectPath and options parameters', async () => {
+      const { audit } = await import('../commands/audit.js');
+      expect(audit.length).toBe(2);
+    });
+  });
+
+  describe('options', () => {
+    it('should support fix option', async () => {
+      const { audit } = await import('../commands/audit.js');
+      expect(audit).toBeDefined();
+    });
+
+    it('should support verbose option', async () => {
+      const { audit } = await import('../commands/audit.js');
+      expect(audit).toBeDefined();
+    });
+  });
+});

package/src/commands/bring.test.js

@@ -0,0 +1,37 @@
+/**
+ * @file bring.test.js
+ * @description Tests para el comando bring
+ */
+
+import { describe, it, expect, beforeEach, vi } from 'vitest';
+
+describe('bring command', () => {
+  describe('validation', () => {
+    it('should have bring function exported', async () => {
+      const { bring } = await import('../commands/bring.js');
+      expect(typeof bring).toBe('function');
+    });
+
+    it('should accept asset and options parameters', async () => {
+      const { bring } = await import('../commands/bring.js');
+      expect(bring.length).toBe(2);
+    });
+  });
+
+  describe('options', () => {
+    it('should support path option', async () => {
+      const { bring } = await import('../commands/bring.js');
+      expect(bring).toBeDefined();
+    });
+
+    it('should support dryRun option', async () => {
+      const { bring } = await import('../commands/bring.js');
+      expect(bring).toBeDefined();
+    });
+
+    it('should support force option', async () => {
+      const { bring } = await import('../commands/bring.js');
+      expect(bring).toBeDefined();
+    });
+  });
+});

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();
+    });
+  });
+});