@saulwade/swl-ses 1.4.0 → 1.4.2

package/hooks/sugerir-regenerar-inventario.js

@@ -1,170 +1,170 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Hook: sugerir-regenerar-inventario.js
- * Tipo: PostToolUse  (aplica a: Write, Edit, MultiEdit)
- *
- * Cuando el agente edita o crea un componente registrado en INVENTARIO.md
- * (agentes, habilidades, comandos, hooks o reglas), sugiere — sin bloquear —
- * regenerar el inventario antes del próximo commit.
- *
- * Motivado por el drift recurrente detectado el 2026-04-29: tras agregar el
- * skill `harness-claude-code` el inventario quedó stale en 5 archivos canónicos
- * porque nadie regeneró `INVENTARIO.md` después de la edición. La regeneración
- * es 1 comando (`node scripts/generar-inventario.js`) pero es fácil olvidarla.
- *
- * Cooldown: una vez por sesión por carpeta afectada para no spammear cuando
- * el agente edita docenas de archivos seguidos. Estado persistente en
- * `.planning/inventario-aviso-state.json`.
- *
- * Resultado:
- *   - Edición sobre carpeta inventariada y sin cooldown → sugerencia en stdout (exit 0)
- *   - Edición sobre carpeta no inventariada            → sin output (exit 0)
- *   - Error interno                                    → sin output (nunca bloquear)
- *
- * Activo por defecto. Para silenciar: SWL_SUGERIR_REGEN_INVENTARIO=0.
- */
-
-const fs = require('fs');
-const path = require('path');
-
-// Carpetas cuyo cambio invalida los contadores de INVENTARIO.md
-const CARPETAS_INVENTARIADAS = [
-  { ruta: 'agentes/',      etiqueta: 'agentes' },
-  { ruta: 'habilidades/',  etiqueta: 'habilidades' },
-  { ruta: 'comandos/swl/', etiqueta: 'comandos' },
-  { ruta: 'hooks/',        etiqueta: 'hooks',  excluir: ['hooks/lib/'] },
-  { ruta: 'reglas/',       etiqueta: 'reglas' },
-];
-
-const STATE_FILE = path.join(process.cwd(), '.planning', 'inventario-aviso-state.json');
-const COOLDOWN_MS = 30 * 60 * 1000; // 30 min
-
-function normalizarSeparadores(p) {
-  return String(p).replace(/\\/g, '/');
-}
-
-function detectarCarpeta(rutaRaw) {
-  if (!rutaRaw) return null;
-  let normalizada = normalizarSeparadores(rutaRaw);
-
-  // Solo nos interesan rutas relativas al proyecto. Si es absoluta, intentamos
-  // hacerla relativa al CWD actual.
-  if (path.isAbsolute(normalizada)) {
-    const cwd = normalizarSeparadores(process.cwd());
-    if (normalizada.toLowerCase().startsWith(cwd.toLowerCase())) {
-      normalizada = normalizada.slice(cwd.length).replace(/^\/+/, '');
-    } else {
-      return null;
-    }
-  } else {
-    normalizada = normalizada.replace(/^\.\//, '');
-  }
-
-  for (const c of CARPETAS_INVENTARIADAS) {
-    if (normalizada.startsWith(c.ruta)) {
-      if (c.excluir && c.excluir.some(ex => normalizada.startsWith(ex))) continue;
-      return c.etiqueta;
-    }
-  }
-  return null;
-}
-
-function leerEstado() {
-  try {
-    return JSON.parse(fs.readFileSync(STATE_FILE, 'utf8'));
-  } catch {
-    return {};
-  }
-}
-
-function escribirEstado(estado) {
-  try {
-    fs.mkdirSync(path.dirname(STATE_FILE), { recursive: true });
-    const tmp = STATE_FILE + '.tmp';
-    fs.writeFileSync(tmp, JSON.stringify(estado, null, 2), 'utf8');
-    fs.renameSync(tmp, STATE_FILE);
-  } catch {
-    // Falla silenciosa — el cooldown no es crítico
-  }
-}
-
-function dentroDeCooldown(estado, etiqueta) {
-  const ultima = estado[etiqueta];
-  if (!ultima) return false;
-  return Date.now() - ultima < COOLDOWN_MS;
-}
-
-function marcarAviso(estado, etiqueta) {
-  estado[etiqueta] = Date.now();
-  escribirEstado(estado);
-}
-
-function construirMensaje(etiqueta) {
-  return (
-    '\n[swl-ses] Tocaste un archivo en `' + etiqueta + '/` — los contadores ' +
-    'de INVENTARIO.md pueden quedar stale.\n' +
-    '   Antes del próximo commit, ejecuta:\n' +
-    '     node scripts/generar-inventario.js\n' +
-    '   y verifica con:\n' +
-    '     node scripts/verificar-release.js\n' +
-    '   (silenciar con SWL_SUGERIR_REGEN_INVENTARIO=0)\n'
-  );
-}
-
-function extraerRuta(toolInput) {
-  if (!toolInput || typeof toolInput !== 'object') return null;
-  return (
-    toolInput.file_path ||
-    toolInput.path ||
-    toolInput.notebook_path ||
-    null
-  );
-}
-
-function ejecutarComoHook() {
-  let inputRaw = '';
-
-  process.stdin.on('data', chunk => { inputRaw += chunk; });
-
-  process.stdin.on('end', () => {
-    try {
-      if (process.env.SWL_SUGERIR_REGEN_INVENTARIO === '0') return;
-
-      const data = JSON.parse(inputRaw || '{}');
-      const toolName = String(data.tool_name || data.tool?.name || '');
-      const toolInput = data.tool_input || data.tool?.input || {};
-
-      if (!['Write', 'Edit', 'MultiEdit'].includes(toolName)) return;
-
-      const ruta = extraerRuta(toolInput);
-      if (!ruta) return;
-
-      const etiqueta = detectarCarpeta(ruta);
-      if (!etiqueta) return;
-
-      const estado = leerEstado();
-      if (dentroDeCooldown(estado, etiqueta)) return;
-
-      process.stdout.write(construirMensaje(etiqueta));
-      marcarAviso(estado, etiqueta);
-    } catch {
-      // Errores internos nunca bloquean
-    }
-  });
-}
-
-// Solo escuchar stdin cuando se ejecuta como entrypoint (no al importar para tests)
-if (require.main === module) {
-  ejecutarComoHook();
-}
-
-// Para tests unitarios
-module.exports = {
-  CARPETAS_INVENTARIADAS,
-  detectarCarpeta,
-  construirMensaje,
-  dentroDeCooldown,
-  COOLDOWN_MS,
-};
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * Hook: sugerir-regenerar-inventario.js
+ * Tipo: PostToolUse  (aplica a: Write, Edit, MultiEdit)
+ *
+ * Cuando el agente edita o crea un componente registrado en INVENTARIO.md
+ * (agentes, habilidades, comandos, hooks o reglas), sugiere — sin bloquear —
+ * regenerar el inventario antes del próximo commit.
+ *
+ * Motivado por el drift recurrente detectado el 2026-04-29: tras agregar el
+ * skill `harness-claude-code` el inventario quedó stale en 5 archivos canónicos
+ * porque nadie regeneró `INVENTARIO.md` después de la edición. La regeneración
+ * es 1 comando (`node scripts/generar-inventario.js`) pero es fácil olvidarla.
+ *
+ * Cooldown: una vez por sesión por carpeta afectada para no spammear cuando
+ * el agente edita docenas de archivos seguidos. Estado persistente en
+ * `.planning/inventario-aviso-state.json`.
+ *
+ * Resultado:
+ *   - Edición sobre carpeta inventariada y sin cooldown → sugerencia en stdout (exit 0)
+ *   - Edición sobre carpeta no inventariada            → sin output (exit 0)
+ *   - Error interno                                    → sin output (nunca bloquear)
+ *
+ * Activo por defecto. Para silenciar: SWL_SUGERIR_REGEN_INVENTARIO=0.
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+// Carpetas cuyo cambio invalida los contadores de INVENTARIO.md
+const CARPETAS_INVENTARIADAS = [
+  { ruta: 'agentes/',      etiqueta: 'agentes' },
+  { ruta: 'habilidades/',  etiqueta: 'habilidades' },
+  { ruta: 'comandos/swl/', etiqueta: 'comandos' },
+  { ruta: 'hooks/',        etiqueta: 'hooks',  excluir: ['hooks/lib/'] },
+  { ruta: 'reglas/',       etiqueta: 'reglas' },
+];
+
+const STATE_FILE = path.join(process.cwd(), '.planning', 'inventario-aviso-state.json');
+const COOLDOWN_MS = 30 * 60 * 1000; // 30 min
+
+function normalizarSeparadores(p) {
+  return String(p).replace(/\\/g, '/');
+}
+
+function detectarCarpeta(rutaRaw) {
+  if (!rutaRaw) return null;
+  let normalizada = normalizarSeparadores(rutaRaw);
+
+  // Solo nos interesan rutas relativas al proyecto. Si es absoluta, intentamos
+  // hacerla relativa al CWD actual.
+  if (path.isAbsolute(normalizada)) {
+    const cwd = normalizarSeparadores(process.cwd());
+    if (normalizada.toLowerCase().startsWith(cwd.toLowerCase())) {
+      normalizada = normalizada.slice(cwd.length).replace(/^\/+/, '');
+    } else {
+      return null;
+    }
+  } else {
+    normalizada = normalizada.replace(/^\.\//, '');
+  }
+
+  for (const c of CARPETAS_INVENTARIADAS) {
+    if (normalizada.startsWith(c.ruta)) {
+      if (c.excluir && c.excluir.some(ex => normalizada.startsWith(ex))) continue;
+      return c.etiqueta;
+    }
+  }
+  return null;
+}
+
+function leerEstado() {
+  try {
+    return JSON.parse(fs.readFileSync(STATE_FILE, 'utf8'));
+  } catch {
+    return {};
+  }
+}
+
+function escribirEstado(estado) {
+  try {
+    fs.mkdirSync(path.dirname(STATE_FILE), { recursive: true });
+    const tmp = STATE_FILE + '.tmp';
+    fs.writeFileSync(tmp, JSON.stringify(estado, null, 2), 'utf8');
+    fs.renameSync(tmp, STATE_FILE);
+  } catch {
+    // Falla silenciosa — el cooldown no es crítico
+  }
+}
+
+function dentroDeCooldown(estado, etiqueta) {
+  const ultima = estado[etiqueta];
+  if (!ultima) return false;
+  return Date.now() - ultima < COOLDOWN_MS;
+}
+
+function marcarAviso(estado, etiqueta) {
+  estado[etiqueta] = Date.now();
+  escribirEstado(estado);
+}
+
+function construirMensaje(etiqueta) {
+  return (
+    '\n[swl-ses] Tocaste un archivo en `' + etiqueta + '/` — los contadores ' +
+    'de INVENTARIO.md pueden quedar stale.\n' +
+    '   Antes del próximo commit, ejecuta:\n' +
+    '     node scripts/generar-inventario.js\n' +
+    '   y verifica con:\n' +
+    '     node scripts/verificar-release.js\n' +
+    '   (silenciar con SWL_SUGERIR_REGEN_INVENTARIO=0)\n'
+  );
+}
+
+function extraerRuta(toolInput) {
+  if (!toolInput || typeof toolInput !== 'object') return null;
+  return (
+    toolInput.file_path ||
+    toolInput.path ||
+    toolInput.notebook_path ||
+    null
+  );
+}
+
+function ejecutarComoHook() {
+  let inputRaw = '';
+
+  process.stdin.on('data', chunk => { inputRaw += chunk; });
+
+  process.stdin.on('end', () => {
+    try {
+      if (process.env.SWL_SUGERIR_REGEN_INVENTARIO === '0') return;
+
+      const data = JSON.parse(inputRaw || '{}');
+      const toolName = String(data.tool_name || data.tool?.name || '');
+      const toolInput = data.tool_input || data.tool?.input || {};
+
+      if (!['Write', 'Edit', 'MultiEdit'].includes(toolName)) return;
+
+      const ruta = extraerRuta(toolInput);
+      if (!ruta) return;
+
+      const etiqueta = detectarCarpeta(ruta);
+      if (!etiqueta) return;
+
+      const estado = leerEstado();
+      if (dentroDeCooldown(estado, etiqueta)) return;
+
+      process.stdout.write(construirMensaje(etiqueta));
+      marcarAviso(estado, etiqueta);
+    } catch {
+      // Errores internos nunca bloquean
+    }
+  });
+}
+
+// Solo escuchar stdin cuando se ejecuta como entrypoint (no al importar para tests)
+if (require.main === module) {
+  ejecutarComoHook();
+}
+
+// Para tests unitarios
+module.exports = {
+  CARPETAS_INVENTARIADAS,
+  detectarCarpeta,
+  construirMensaje,
+  dentroDeCooldown,
+  COOLDOWN_MS,
+};

package/hooks/validar-formato-post-subagente.js

@@ -1,140 +1,140 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Hook: validar-formato-post-subagente.js
- * Tipo: SubagentStop (async: true — fire-and-forget)
- *
- * Cuando un subagente SWL termina, valida que su output cumple con el
- * schema declarado en `manifiestos/agent-output-schemas.json`. Si el
- * agente no tiene schema declarado, el hook no hace nada — no asume
- * que todo agente debe seguir el formato compacto.
- *
- * Registra el resultado en `.planning/evolucion/formato-violaciones.jsonl`
- * para que `metricas-evolucion.js` calcule la tasa de violación por
- * agente y la incluya en el dashboard de calidad conductual.
- *
- * Schema simple: lista de patrones regex que DEBEN aparecer en el output.
- * Si todos matchean → conforme. Si falta alguno → violación con detalle.
- *
- * Opt-out: SWL_FORMATO_VALIDACION=0 desactiva el hook.
- *
- * El hook nunca bloquea — solo registra para análisis posterior.
- */
-
-const fs   = require('fs');
-const path = require('path');
-
-const CWD = process.cwd();
-const SCHEMAS_PATH = path.join(CWD, 'manifiestos', 'agent-output-schemas.json');
-const LOG_PATH     = path.join(CWD, '.planning', 'evolucion', 'formato-violaciones.jsonl');
-
-// ---------------------------------------------------------------------------
-// Carga del schema (una vez)
-// ---------------------------------------------------------------------------
-
-function cargarSchemas() {
-  try {
-    const raw = fs.readFileSync(SCHEMAS_PATH, 'utf8');
-    const parsed = JSON.parse(raw);
-    return parsed.schemas || {};
-  } catch {
-    return {};
-  }
-}
-
-// ---------------------------------------------------------------------------
-// Validación
-// ---------------------------------------------------------------------------
-
-/**
- * Compara el output contra el schema. Devuelve { violation, errors }.
- * @param {string} output - Texto del output del subagente
- * @param {object} schema - { patronesRequeridos: string[] }
- */
-function validarOutput(output, schema) {
-  const errors = [];
-  const patrones = Array.isArray(schema.patronesRequeridos)
-    ? schema.patronesRequeridos
-    : [];
-  for (const patron of patrones) {
-    // El schema puede declarar inline flags estilo (?im) por compatibilidad
-    // con regex POSIX. JavaScript no los soporta inline — los stripeamos
-    // y aplicamos 'im' siempre (case-insensitive + multiline).
-    const limpio = patron.replace(/^\(\?[imsxu]+\)/, '');
-    let regex;
-    try { regex = new RegExp(limpio, 'im'); }
-    catch { errors.push(`Patrón inválido en schema: ${patron}`); continue; }
-    if (!regex.test(output)) {
-      errors.push(`Falta patrón requerido: ${patron}`);
-    }
-  }
-  return { violation: errors.length > 0, errors };
-}
-
-/**
- * Extrae el output textual del subagente del payload del hook.
- * El payload del SubagentStop expone el resultado en distintos lugares
- * según versión de Claude Code; intentamos varias rutas.
- */
-function extraerOutput(data) {
-  const r = data.tool_response || data.tool_result || data.response || {};
-  // Posibles ubicaciones del output del subagente
-  if (typeof r.text === 'string') return r.text;
-  if (typeof r.content === 'string') return r.content;
-  if (Array.isArray(r.messages)) {
-    return r.messages
-      .map(m => (typeof m.content === 'string' ? m.content : ''))
-      .filter(Boolean)
-      .join('\n');
-  }
-  if (typeof r.output === 'string') return r.output;
-  return '';
-}
-
-function ensureDir() {
-  try { fs.mkdirSync(path.dirname(LOG_PATH), { recursive: true }); } catch { /* ignore */ }
-}
-
-// ---------------------------------------------------------------------------
-// Entrypoint
-// ---------------------------------------------------------------------------
-
-let inputRaw = '';
-process.stdin.on('data', chunk => { inputRaw += chunk; });
-
-process.stdin.on('end', () => {
-  try {
-    if (process.env.SWL_FORMATO_VALIDACION === '0') return;
-
-    const data = JSON.parse(inputRaw);
-    const subagentType =
-      data.tool_input?.subagent_type ||
-      data.tool?.input?.subagent_type ||
-      data.subagent_type ||
-      null;
-    if (!subagentType) return;
-
-    const schemas = cargarSchemas();
-    const schema = schemas[subagentType];
-    if (!schema) return; // agente sin schema → no se evalúa
-
-    const output = extraerOutput(data);
-    if (!output || output.length < 10) return; // output trivial — no evaluar
-
-    const { violation, errors } = validarOutput(output, schema);
-    const entry = {
-      ts: new Date().toISOString(),
-      sessionId: String(data.session_id || 'default'),
-      agente: subagentType,
-      violation,
-      errors,
-      outputLen: output.length,
-    };
-
-    ensureDir();
-    fs.appendFileSync(LOG_PATH, JSON.stringify(entry) + '\n', 'utf8');
-  } catch {
-    // SubagentStop hook nunca bloquea por error interno.
-  }
-});
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * Hook: validar-formato-post-subagente.js
+ * Tipo: SubagentStop (async: true — fire-and-forget)
+ *
+ * Cuando un subagente SWL termina, valida que su output cumple con el
+ * schema declarado en `manifiestos/agent-output-schemas.json`. Si el
+ * agente no tiene schema declarado, el hook no hace nada — no asume
+ * que todo agente debe seguir el formato compacto.
+ *
+ * Registra el resultado en `.planning/evolucion/formato-violaciones.jsonl`
+ * para que `metricas-evolucion.js` calcule la tasa de violación por
+ * agente y la incluya en el dashboard de calidad conductual.
+ *
+ * Schema simple: lista de patrones regex que DEBEN aparecer en el output.
+ * Si todos matchean → conforme. Si falta alguno → violación con detalle.
+ *
+ * Opt-out: SWL_FORMATO_VALIDACION=0 desactiva el hook.
+ *
+ * El hook nunca bloquea — solo registra para análisis posterior.
+ */
+
+const fs   = require('fs');
+const path = require('path');
+
+const CWD = process.cwd();
+const SCHEMAS_PATH = path.join(CWD, 'manifiestos', 'agent-output-schemas.json');
+const LOG_PATH     = path.join(CWD, '.planning', 'evolucion', 'formato-violaciones.jsonl');
+
+// ---------------------------------------------------------------------------
+// Carga del schema (una vez)
+// ---------------------------------------------------------------------------
+
+function cargarSchemas() {
+  try {
+    const raw = fs.readFileSync(SCHEMAS_PATH, 'utf8');
+    const parsed = JSON.parse(raw);
+    return parsed.schemas || {};
+  } catch {
+    return {};
+  }
+}
+
+// ---------------------------------------------------------------------------
+// Validación
+// ---------------------------------------------------------------------------
+
+/**
+ * Compara el output contra el schema. Devuelve { violation, errors }.
+ * @param {string} output - Texto del output del subagente
+ * @param {object} schema - { patronesRequeridos: string[] }
+ */
+function validarOutput(output, schema) {
+  const errors = [];
+  const patrones = Array.isArray(schema.patronesRequeridos)
+    ? schema.patronesRequeridos
+    : [];
+  for (const patron of patrones) {
+    // El schema puede declarar inline flags estilo (?im) por compatibilidad
+    // con regex POSIX. JavaScript no los soporta inline — los stripeamos
+    // y aplicamos 'im' siempre (case-insensitive + multiline).
+    const limpio = patron.replace(/^\(\?[imsxu]+\)/, '');
+    let regex;
+    try { regex = new RegExp(limpio, 'im'); }
+    catch { errors.push(`Patrón inválido en schema: ${patron}`); continue; }
+    if (!regex.test(output)) {
+      errors.push(`Falta patrón requerido: ${patron}`);
+    }
+  }
+  return { violation: errors.length > 0, errors };
+}
+
+/**
+ * Extrae el output textual del subagente del payload del hook.
+ * El payload del SubagentStop expone el resultado en distintos lugares
+ * según versión de Claude Code; intentamos varias rutas.
+ */
+function extraerOutput(data) {
+  const r = data.tool_response || data.tool_result || data.response || {};
+  // Posibles ubicaciones del output del subagente
+  if (typeof r.text === 'string') return r.text;
+  if (typeof r.content === 'string') return r.content;
+  if (Array.isArray(r.messages)) {
+    return r.messages
+      .map(m => (typeof m.content === 'string' ? m.content : ''))
+      .filter(Boolean)
+      .join('\n');
+  }
+  if (typeof r.output === 'string') return r.output;
+  return '';
+}
+
+function ensureDir() {
+  try { fs.mkdirSync(path.dirname(LOG_PATH), { recursive: true }); } catch { /* ignore */ }
+}
+
+// ---------------------------------------------------------------------------
+// Entrypoint
+// ---------------------------------------------------------------------------
+
+let inputRaw = '';
+process.stdin.on('data', chunk => { inputRaw += chunk; });
+
+process.stdin.on('end', () => {
+  try {
+    if (process.env.SWL_FORMATO_VALIDACION === '0') return;
+
+    const data = JSON.parse(inputRaw);
+    const subagentType =
+      data.tool_input?.subagent_type ||
+      data.tool?.input?.subagent_type ||
+      data.subagent_type ||
+      null;
+    if (!subagentType) return;
+
+    const schemas = cargarSchemas();
+    const schema = schemas[subagentType];
+    if (!schema) return; // agente sin schema → no se evalúa
+
+    const output = extraerOutput(data);
+    if (!output || output.length < 10) return; // output trivial — no evaluar
+
+    const { violation, errors } = validarOutput(output, schema);
+    const entry = {
+      ts: new Date().toISOString(),
+      sessionId: String(data.session_id || 'default'),
+      agente: subagentType,
+      violation,
+      errors,
+      outputLen: output.length,
+    };
+
+    ensureDir();
+    fs.appendFileSync(LOG_PATH, JSON.stringify(entry) + '\n', 'utf8');
+  } catch {
+    // SubagentStop hook nunca bloquea por error interno.
+  }
+});