@saulwade/swl-ses 2.3.0 → 2.4.1

package/hooks/sugerir-regenerar-inventario.js

@@ -1,170 +1,171 @@
-#!/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 corre como entrypoint: directo o wrapper
+// `node -e "require(...)"` (require.main undefined). No al importar para tests.
+if (!require.main || require.main === module) {
+  ejecutarComoHook();
+}
+
+// Para tests unitarios
+module.exports = {
+  CARPETAS_INVENTARIADAS,
+  detectarCarpeta,
+  construirMensaje,
+  dentroDeCooldown,
+  COOLDOWN_MS,
+};