@saulwade/swl-ses 1.4.1 → 1.4.2

package/scripts/validar-manifest.js

@@ -1,195 +1,231 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Gate de sincronización manifiestos ↔ disco.
- *
- * Verifica que cada archivo de componente presente en disco esté declarado
- * en manifiestos/modulos.json (fuente de verdad del instalador). Atrapa el
- * bug recurrente "archivo existe en el repo pero el instalador no lo copia
- * al destino" que causó v5.7.3 (check-update.js) y v5.7.4 (adoptar-proyecto.md,
- * contribuir.md).
- *
- * Convenciones del manifest (validadas vs disco):
- *   agentes/X.md                → declarado como 'agentes/X.md'
- *   comandos/swl/X.md           → declarado como 'comandos/swl/X.md'
- *   habilidades/X/SKILL.md      → declarado como 'habilidades/X' (solo el dir)
- *   hooks/X.js (ejecutable)     → declarado como 'hooks/X.js' + hooks-config.json
- *   reglas/**.md                → declarado como 'reglas/<path>.md'
- *   schemas/X.json              → declarado como 'schemas/X.json'
- *
- * Exclusiones intencionales:
- *   hooks/lib/   — helpers internos; se declaran solo las libs que un hook
- *                  copiado necesita en su módulo. No van 1:1 con disco.
- *   scripts/     — herramientas de desarrollo, no componentes distribuibles.
- *   schemas/     — definiciones JSON Schema para validación interna del repo;
- *                  no se copian al destino (los runtimes no las leen).
- *
- * Exit codes: 0 OK, 1 faltan declaraciones.
- *
- * Uso: node scripts/validar-manifest.js [--json]
- */
-
-const fs = require('fs');
-const path = require('path');
-
-const RAIZ = path.resolve(__dirname, '..');
-const MODULOS_PATH = path.join(RAIZ, 'manifiestos', 'modulos.json');
-const HOOKS_CONFIG_PATH = path.join(RAIZ, 'manifiestos', 'hooks-config.json');
-
-const salidaJson = process.argv.includes('--json');
-const errores = [];
-
-function log(msg) { if (!salidaJson) console.log(msg); }
-
-function recolectarDeclarados(modulos) {
-  const declarados = new Set();
-  for (const mod of Object.values(modulos.modulos || {})) {
-    for (const archivo of mod.archivos || []) {
-      declarados.add(archivo);
-    }
-  }
-  return declarados;
-}
-
-function listarArchivos(dirRel, extension, recursivo = false) {
-  const base = path.join(RAIZ, dirRel);
-  if (!fs.existsSync(base)) return [];
-  const resultado = [];
-  function caminar(actual) {
-    for (const entrada of fs.readdirSync(actual, { withFileTypes: true })) {
-      const abs = path.join(actual, entrada.name);
-      if (entrada.isDirectory() && recursivo) {
-        caminar(abs);
-      } else if (entrada.isFile() && entrada.name.endsWith(extension)) {
-        resultado.push(path.relative(RAIZ, abs).replace(/\\/g, '/'));
-      }
-    }
-  }
-  caminar(base);
-  return resultado;
-}
-
-function listarDirectoriosHabilidades() {
-  const base = path.join(RAIZ, 'habilidades');
-  if (!fs.existsSync(base)) return [];
-  const dirs = [];
-  for (const entrada of fs.readdirSync(base, { withFileTypes: true })) {
-    if (!entrada.isDirectory()) continue;
-    // Solo directorios que tienen SKILL.md son habilidades válidas
-    if (fs.existsSync(path.join(base, entrada.name, 'SKILL.md'))) {
-      dirs.push(`habilidades/${entrada.name}`);
-    }
-  }
-  return dirs;
-}
-
-function verificarCategoria(etiqueta, archivosDisco, declarados) {
-  const faltantes = archivosDisco.filter(a => !declarados.has(a));
-  if (faltantes.length === 0) {
-    log(`  [OK]    ${etiqueta}: ${archivosDisco.length}/${archivosDisco.length} declarados`);
-    return true;
-  }
-  log(`  [FALLA] ${etiqueta}: ${faltantes.length} sin declarar en modulos.json`);
-  for (const f of faltantes) {
-    log(`          → ${f}`);
-    errores.push({ categoria: etiqueta, archivo: f, tipo: 'no_declarado_modulos' });
-  }
-  return false;
-}
-
-function verificarHooksEjecutablesEnConfig(archivosHooks, hooksConfig) {
-  // hooks-config.json tiene la estructura { hooks: { "nombre.js": { event, matcher, ... } } }
-  const hooks = hooksConfig.hooks || hooksConfig;
-  const declaradosConfig = new Set(Object.keys(hooks));
-  const faltantes = archivosHooks.filter(a => {
-    const nombre = path.basename(a);
-    return !declaradosConfig.has(nombre);
-  });
-  if (faltantes.length === 0) {
-    log(`  [OK]    hooks-config.json: ${archivosHooks.length}/${archivosHooks.length} hooks ejecutables registrados`);
-    return true;
-  }
-  log(`  [FALLA] hooks-config.json: ${faltantes.length} hook(s) sin declarar`);
-  for (const f of faltantes) {
-    log(`          → ${f}`);
-    errores.push({ categoria: 'hooks-config.json', archivo: f, tipo: 'no_declarado_hooks_config' });
-  }
-  return false;
-}
-
-function verificarDeclarados1a1ExistenEnDisco(declarados) {
-  // Caso inverso: declarados en manifest pero no presentes en disco
-  const rotos = [];
-  for (const decl of declarados) {
-    // Habilidades son directorios
-    if (decl.startsWith('habilidades/') && !decl.includes('/SKILL.md')) {
-      const absDir = path.join(RAIZ, decl);
-      const skillMd = path.join(absDir, 'SKILL.md');
-      if (!fs.existsSync(skillMd)) rotos.push(decl);
-      continue;
-    }
-    const abs = path.join(RAIZ, decl);
-    if (!fs.existsSync(abs)) rotos.push(decl);
-  }
-  if (rotos.length === 0) {
-    log(`  [OK]    Referencias en modulos.json: ${declarados.size}/${declarados.size} existen en disco`);
-    return true;
-  }
-  log(`  [FALLA] modulos.json: ${rotos.length} ruta(s) rotas (declaradas pero no existen)`);
-  for (const r of rotos) {
-    log(`          → ${r}`);
-    errores.push({ categoria: 'modulos.json', archivo: r, tipo: 'declarado_no_existe' });
-  }
-  return false;
-}
-
-function main() {
-  log('Validación de sincronización manifiestos ↔ disco\n');
-
-  const modulos = JSON.parse(fs.readFileSync(MODULOS_PATH, 'utf-8'));
-  const hooksConfig = JSON.parse(fs.readFileSync(HOOKS_CONFIG_PATH, 'utf-8'));
-  const declarados = recolectarDeclarados(modulos);
-
-  const hooksEjecutables = listarArchivos('hooks', '.js').filter(a => !a.startsWith('hooks/lib/'));
-
-  const categorias = [
-    { etiqueta: 'comandos/swl/',       archivos: listarArchivos('comandos/swl', '.md') },
-    { etiqueta: 'hooks/ (ejecutables)', archivos: hooksEjecutables },
-    { etiqueta: 'agentes/',             archivos: listarArchivos('agentes', '.md') },
-    { etiqueta: 'habilidades/ (dirs)',  archivos: listarDirectoriosHabilidades() },
-    { etiqueta: 'reglas/',              archivos: listarArchivos('reglas', '.md', true) },
-  ];
-
-  let todoOk = true;
-  for (const cat of categorias) {
-    const ok = verificarCategoria(cat.etiqueta, cat.archivos, declarados);
-    if (!ok) todoOk = false;
-  }
-
-  const hooksOk = verificarHooksEjecutablesEnConfig(hooksEjecutables, hooksConfig);
-  if (!hooksOk) todoOk = false;
-
-  const inversoOk = verificarDeclarados1a1ExistenEnDisco(declarados);
-  if (!inversoOk) todoOk = false;
-
-  log('');
-  if (todoOk) {
-    log('✓ Todos los manifiestos están sincronizados con el disco.');
-  } else {
-    log(`✗ ${errores.length} problema(s) de sincronización.`);
-    log('');
-    log('  Cómo corregir:');
-    log('  - "no_declarado_modulos": agregar la ruta al módulo apropiado en manifiestos/modulos.json');
-    log('  - "no_declarado_hooks_config": registrar el hook en manifiestos/hooks-config.json');
-    log('  - "declarado_no_existe": eliminar la ruta de modulos.json (archivo borrado) o crear el archivo');
-  }
-
-  if (salidaJson) {
-    console.log(JSON.stringify({ ok: todoOk, totalErrores: errores.length, errores }, null, 2));
-  }
-
-  process.exit(todoOk ? 0 : 1);
-}
-
-main();
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * Gate de sincronización manifiestos ↔ disco.
+ *
+ * Verifica que cada archivo de componente presente en disco esté declarado
+ * en manifiestos/modulos.json (fuente de verdad del instalador). Atrapa el
+ * bug recurrente "archivo existe en el repo pero el instalador no lo copia
+ * al destino" que causó v5.7.3 (check-update.js) y v5.7.4 (adoptar-proyecto.md,
+ * contribuir.md).
+ *
+ * Convenciones del manifest (validadas vs disco):
+ *   agentes/X.md                → declarado como 'agentes/X.md'
+ *   comandos/swl/X.md           → declarado como 'comandos/swl/X.md'
+ *   habilidades/X/SKILL.md      → declarado como 'habilidades/X' (solo el dir)
+ *   hooks/X.js (ejecutable)     → declarado como 'hooks/X.js' + hooks-config.json
+ *   reglas/**.md                → declarado como 'reglas/<path>.md'
+ *   schemas/X.json              → declarado como 'schemas/X.json'
+ *
+ * Exclusiones intencionales:
+ *   hooks/lib/   — helpers internos; se declaran solo las libs que un hook
+ *                  copiado necesita en su módulo. No van 1:1 con disco.
+ *   scripts/     — herramientas de desarrollo, no componentes distribuibles.
+ *   schemas/     — definiciones JSON Schema para validación interna del repo;
+ *                  no se copian al destino (los runtimes no las leen).
+ *
+ * Exit codes: 0 OK, 1 faltan declaraciones.
+ *
+ * Uso: node scripts/validar-manifest.js [--json]
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+const RAIZ = path.resolve(__dirname, '..');
+const MODULOS_PATH = path.join(RAIZ, 'manifiestos', 'modulos.json');
+const HOOKS_CONFIG_PATH = path.join(RAIZ, 'manifiestos', 'hooks-config.json');
+const PERFILES_PATH = path.join(RAIZ, 'manifiestos', 'perfiles.json');
+
+const salidaJson = process.argv.includes('--json');
+const errores = [];
+
+function log(msg) { if (!salidaJson) console.log(msg); }
+
+function recolectarDeclarados(modulos) {
+  const declarados = new Set();
+  for (const mod of Object.values(modulos.modulos || {})) {
+    for (const archivo of mod.archivos || []) {
+      declarados.add(archivo);
+    }
+  }
+  return declarados;
+}
+
+function listarArchivos(dirRel, extension, recursivo = false) {
+  const base = path.join(RAIZ, dirRel);
+  if (!fs.existsSync(base)) return [];
+  const resultado = [];
+  function caminar(actual) {
+    for (const entrada of fs.readdirSync(actual, { withFileTypes: true })) {
+      const abs = path.join(actual, entrada.name);
+      if (entrada.isDirectory() && recursivo) {
+        caminar(abs);
+      } else if (entrada.isFile() && entrada.name.endsWith(extension)) {
+        resultado.push(path.relative(RAIZ, abs).replace(/\\/g, '/'));
+      }
+    }
+  }
+  caminar(base);
+  return resultado;
+}
+
+function listarDirectoriosHabilidades() {
+  const base = path.join(RAIZ, 'habilidades');
+  if (!fs.existsSync(base)) return [];
+  const dirs = [];
+  for (const entrada of fs.readdirSync(base, { withFileTypes: true })) {
+    if (!entrada.isDirectory()) continue;
+    // Solo directorios que tienen SKILL.md son habilidades válidas
+    if (fs.existsSync(path.join(base, entrada.name, 'SKILL.md'))) {
+      dirs.push(`habilidades/${entrada.name}`);
+    }
+  }
+  return dirs;
+}
+
+function verificarCategoria(etiqueta, archivosDisco, declarados) {
+  const faltantes = archivosDisco.filter(a => !declarados.has(a));
+  if (faltantes.length === 0) {
+    log(`  [OK]    ${etiqueta}: ${archivosDisco.length}/${archivosDisco.length} declarados`);
+    return true;
+  }
+  log(`  [FALLA] ${etiqueta}: ${faltantes.length} sin declarar en modulos.json`);
+  for (const f of faltantes) {
+    log(`          → ${f}`);
+    errores.push({ categoria: etiqueta, archivo: f, tipo: 'no_declarado_modulos' });
+  }
+  return false;
+}
+
+function verificarHooksEjecutablesEnConfig(archivosHooks, hooksConfig) {
+  // hooks-config.json tiene la estructura { hooks: { "nombre.js": { event, matcher, ... } } }
+  const hooks = hooksConfig.hooks || hooksConfig;
+  const declaradosConfig = new Set(Object.keys(hooks));
+  const faltantes = archivosHooks.filter(a => {
+    const nombre = path.basename(a);
+    return !declaradosConfig.has(nombre);
+  });
+  if (faltantes.length === 0) {
+    log(`  [OK]    hooks-config.json: ${archivosHooks.length}/${archivosHooks.length} hooks ejecutables registrados`);
+    return true;
+  }
+  log(`  [FALLA] hooks-config.json: ${faltantes.length} hook(s) sin declarar`);
+  for (const f of faltantes) {
+    log(`          → ${f}`);
+    errores.push({ categoria: 'hooks-config.json', archivo: f, tipo: 'no_declarado_hooks_config' });
+  }
+  return false;
+}
+
+function verificarModulosReferenciadosPorPerfil(modulos, perfiles) {
+  // Cada módulo declarado en modulos.json DEBE estar referenciado por al menos un perfil
+  // O marcado explícitamente como `opt_in: true` (feature opcional, no incluida en perfiles).
+  // De lo contrario, el instalador nunca copia sus archivos al destino aunque viajen en el tarball.
+  // Bug histórico: v1.4.1 publicó 'auditoria-profunda' (Opción C) pero quedó huérfano del perfil
+  // 'completo' — los 16 archivos del módulo viajaban en el tarball pero el instalador los ignoraba.
+  // Solución: agregar al perfil. Para opt-in declarativo: campo `opt_in: true` en el módulo.
+  const todosLosModulos = modulos.modulos || {};
+  const referenciados = new Set();
+  for (const perfil of Object.values(perfiles.perfiles || {})) {
+    for (const modId of perfil.modulos || []) {
+      referenciados.add(modId);
+    }
+  }
+  const huerfanos = Object.entries(todosLosModulos)
+    .filter(([id, mod]) => !referenciados.has(id) && !mod.opt_in)
+    .map(([id]) => id);
+  const declarados = Object.keys(todosLosModulos);
+  if (huerfanos.length === 0) {
+    log(`  [OK]    perfiles.json: ${declarados.length}/${declarados.length} módulos referenciados por algún perfil`);
+    return true;
+  }
+  log(`  [FALLA] perfiles.json: ${huerfanos.length} módulo(s) huérfano(s) (declarados en modulos.json pero sin perfil)`);
+  for (const h of huerfanos) {
+    log(`          → ${h}`);
+    errores.push({ categoria: 'perfiles.json', archivo: h, tipo: 'modulo_huerfano_sin_perfil' });
+  }
+  return false;
+}
+
+function verificarDeclarados1a1ExistenEnDisco(declarados) {
+  // Caso inverso: declarados en manifest pero no presentes en disco
+  const rotos = [];
+  for (const decl of declarados) {
+    // Habilidades son directorios
+    if (decl.startsWith('habilidades/') && !decl.includes('/SKILL.md')) {
+      const absDir = path.join(RAIZ, decl);
+      const skillMd = path.join(absDir, 'SKILL.md');
+      if (!fs.existsSync(skillMd)) rotos.push(decl);
+      continue;
+    }
+    const abs = path.join(RAIZ, decl);
+    if (!fs.existsSync(abs)) rotos.push(decl);
+  }
+  if (rotos.length === 0) {
+    log(`  [OK]    Referencias en modulos.json: ${declarados.size}/${declarados.size} existen en disco`);
+    return true;
+  }
+  log(`  [FALLA] modulos.json: ${rotos.length} ruta(s) rotas (declaradas pero no existen)`);
+  for (const r of rotos) {
+    log(`          → ${r}`);
+    errores.push({ categoria: 'modulos.json', archivo: r, tipo: 'declarado_no_existe' });
+  }
+  return false;
+}
+
+function main() {
+  log('Validación de sincronización manifiestos ↔ disco\n');
+
+  const modulos = JSON.parse(fs.readFileSync(MODULOS_PATH, 'utf-8'));
+  const hooksConfig = JSON.parse(fs.readFileSync(HOOKS_CONFIG_PATH, 'utf-8'));
+  const perfiles = JSON.parse(fs.readFileSync(PERFILES_PATH, 'utf-8'));
+  const declarados = recolectarDeclarados(modulos);
+
+  const hooksEjecutables = listarArchivos('hooks', '.js').filter(a => !a.startsWith('hooks/lib/'));
+
+  const categorias = [
+    { etiqueta: 'comandos/swl/',       archivos: listarArchivos('comandos/swl', '.md') },
+    { etiqueta: 'hooks/ (ejecutables)', archivos: hooksEjecutables },
+    { etiqueta: 'agentes/',             archivos: listarArchivos('agentes', '.md') },
+    { etiqueta: 'habilidades/ (dirs)',  archivos: listarDirectoriosHabilidades() },
+    { etiqueta: 'reglas/',              archivos: listarArchivos('reglas', '.md', true) },
+  ];
+
+  let todoOk = true;
+  for (const cat of categorias) {
+    const ok = verificarCategoria(cat.etiqueta, cat.archivos, declarados);
+    if (!ok) todoOk = false;
+  }
+
+  const hooksOk = verificarHooksEjecutablesEnConfig(hooksEjecutables, hooksConfig);
+  if (!hooksOk) todoOk = false;
+
+  const perfilOk = verificarModulosReferenciadosPorPerfil(modulos, perfiles);
+  if (!perfilOk) todoOk = false;
+
+  const inversoOk = verificarDeclarados1a1ExistenEnDisco(declarados);
+  if (!inversoOk) todoOk = false;
+
+  log('');
+  if (todoOk) {
+    log('✓ Todos los manifiestos están sincronizados con el disco.');
+  } else {
+    log(`✗ ${errores.length} problema(s) de sincronización.`);
+    log('');
+    log('  Cómo corregir:');
+    log('  - "no_declarado_modulos": agregar la ruta al módulo apropiado en manifiestos/modulos.json');
+    log('  - "no_declarado_hooks_config": registrar el hook en manifiestos/hooks-config.json');
+    log('  - "modulo_huerfano_sin_perfil": agregar el módulo al array `modulos` de al menos un perfil en manifiestos/perfiles.json');
+    log('  - "declarado_no_existe": eliminar la ruta de modulos.json (archivo borrado) o crear el archivo');
+  }
+
+  if (salidaJson) {
+    console.log(JSON.stringify({ ok: todoOk, totalErrores: errores.length, errores }, null, 2));
+  }
+
+  process.exit(todoOk ? 0 : 1);
+}
+
+main();