@saulwade/swl-ses 2.3.0 → 2.4.1

package/hooks/calidad-pre-commit.js

@@ -375,6 +375,11 @@ function verificarTodosSinTicket(rutaRelativa, contenido) {
   // Construir palabra clave dinámicamente para evitar auto-detección
   const keyword = 'TO' + 'DO';
   const patronSinTicket = new RegExp(`\\b${keyword}\\b(?!\\s*[:(]\\s*(?:#\\d+|[A-Z]+-\\d+|https?:\\/\\/))`);
+  // En Markdown, "TODO" solo cuenta como marcador con sintaxis explícita
+  // (TODO: / TODO() — la palabra española "TODO" en prosa ("en TODO destino",
+  // "tumbaba TODO el release") nunca la lleva. La blacklist de verbos previa
+  // era frágil: cada frase nueva la esquivaba (falsos positivos 2026-07-03).
+  const patronSinTicketMd = new RegExp(`\\b${keyword}\\s*[:(](?!\\s*(?:#\\d+|[A-Z]+-\\d+|https?:\\/\\/))`);
   const patronConTicket = new RegExp(`\\b${keyword}\\s*[:(]\\s*(?:#\\d+|[A-Z]+-\\d+|https?:\\/\\/)`);
   const patronNosec = /\/\/\s*nosec/;
 
@@ -391,15 +396,12 @@ function verificarTodosSinTicket(rutaRelativa, contenido) {
     if (dentroDeCodigoMd) continue;
     // Excluir líneas marcadas con // nosec o <!-- nosec -->
     if (patronNosec.test(linea) || /<!--\s*nosec\s*-->/.test(linea)) continue;
-    // En Markdown, excluir "TODO" dentro de backticks (código inline) o como palabra española
+    // En Markdown, excluir "TODO" dentro de backticks (código inline)
     if (esMd && /`[^`]*TODO[^`]*`/.test(linea)) continue;
-    if (esMd && /\b(capturar|cargar|cubrir|cumplir|verificar|pasar|incluir|tener)\s+TODO\b/i.test(linea)) continue;
-    // En Markdown, excluir líneas descriptivas que mencionan TODO como concepto (ej: "detecta TODO sin ticket")
-    if (esMd && /TODO\s+sin\s+ticket/i.test(linea)) continue;
     // Si tiene referencia a ticket, es válido
     if (patronConTicket.test(linea)) continue;
-    // Si no tiene referencia, es una violación
-    if (patronSinTicket.test(linea)) {
+    // Si no tiene referencia, es una violación (en .md solo el marcador explícito)
+    if ((esMd ? patronSinTicketMd : patronSinTicket).test(linea)) {
       const tag = '[TO' + 'DO]';
       mensajes.push(
         `  ${tag} ${rutaRelativa}:${i + 1} — sin referencia a ticket/issue`
@@ -1009,10 +1011,11 @@ function ejecutarCoverage(fuentes, runner) {
 
 let inputRaw = '';
 
-// Solo enganchar stdin cuando el hook se ejecuta directamente (no al hacer
-// `require()` desde un test). Permite testear las funciones puras sin disparar
-// el flujo de stdin.
-if (require.main === module) {
+// Solo enganchar stdin cuando el hook corre como entrypoint: ejecución directa
+// O vía el wrapper de settings.json `node -e "require('./hooks/...')"`, donde
+// require.main es undefined. Al hacer `require()` desde un test, require.main
+// es el runner (definido y ≠ este módulo) → no engancha stdin.
+if (!require.main || require.main === module) {
 
 process.stdin.on('data', chunk => {
   inputRaw += chunk;
@@ -1182,7 +1185,7 @@ process.stdin.on('end', () => {
   }
 });
 
-} // fin if (require.main === module)
+} // fin gate de entrypoint (directo o wrapper node -e)
 
 // ─── Coverage diferencial opt-in (gate G2-cov, Fase 11) ──────────────────────
 //
@@ -1265,6 +1268,7 @@ module.exports = {
   extraerMensajeCommit,
   esPrefijoExentoDeTests,
   verificarTestPresence,
+  verificarTodosSinTicket,
   detectarRunnerCoverage,
   coverageDiferencial,
 };

package/hooks/captura-acciones-post.js

@@ -134,8 +134,9 @@ function procesarTurno(data, baseDir) {
   }
 }
 
-// Listeners de stdin SOLO en modo CLI (evita colgar la suite de tests).
-if (require.main === module) {
+// Listeners de stdin SOLO en modo entrypoint: ejecución directa o wrapper
+// `node -e "require(...)"` (require.main undefined). Evita colgar los tests.
+if (!require.main || require.main === module) {
   let inputRaw = '';
   process.stdin.on('data', (c) => { inputRaw += c; });
   process.stdin.on('end', () => {

package/hooks/captura-acciones-session.js

@@ -138,8 +138,9 @@ function inspeccionar(baseDir) {
   return { encolados: n };
 }
 
-// Listeners de stdin SOLO en modo CLI.
-if (require.main === module) {
+// Listeners de stdin SOLO en modo entrypoint: ejecución directa o wrapper
+// `node -e "require(...)"` (require.main undefined). Evita colgar los tests.
+if (!require.main || require.main === module) {
   let inputRaw = '';
   process.stdin.on('data', (c) => { inputRaw += c; });
   process.stdin.on('end', () => {

package/hooks/ciclo-evolucion-subagente.js

@@ -1,26 +1,26 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Hook: ciclo-evolucion-subagente.js
- * Tipo: SubagentStop (async: true — fire-and-forget)
- *
- * Entry point del ciclo de evolución en el evento SubagentStop. Delega a la
- * sub-etapa de auto-evolución (antes `auto-evolucion.js`), que analiza el
- * payload del subagente, registra en el log y emite nudges de evolución
- * (fallos/volumen/loop/drift). Fase 11, D-12.
- *
- * Nunca bloquea (siempre exit 0). Zero-deps fuera de hooks/lib/.
- */
-
-const cicloEvolucion = require('./lib/ciclo-evolucion');
-
-let inputRaw = '';
-process.stdin.on('data', chunk => { inputRaw += chunk; });
-process.stdin.on('end', () => {
-  try {
-    cicloEvolucion.ejecutarSubagentStop(inputRaw);
-  } catch {
-    // El hook nunca bloquea por error interno.
-  }
-});
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * Hook: ciclo-evolucion-subagente.js
+ * Tipo: SubagentStop (async: true — fire-and-forget)
+ *
+ * Entry point del ciclo de evolución en el evento SubagentStop. Delega a la
+ * sub-etapa de auto-evolución (antes `auto-evolucion.js`), que analiza el
+ * payload del subagente, registra en el log y emite nudges de evolución
+ * (fallos/volumen/loop/drift). Fase 11, D-12.
+ *
+ * Nunca bloquea (siempre exit 0). Zero-deps fuera de hooks/lib/.
+ */
+
+const cicloEvolucion = require('./lib/ciclo-evolucion');
+
+let inputRaw = '';
+process.stdin.on('data', chunk => { inputRaw += chunk; });
+process.stdin.on('end', () => {
+  try {
+    cicloEvolucion.ejecutarSubagentStop(inputRaw);
+  } catch {
+    // El hook nunca bloquea por error interno.
+  }
+});

package/hooks/ciclo-evolucion.js

@@ -1,26 +1,26 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Hook: ciclo-evolucion.js
- * Tipo: Stop (async: true — fire-and-forget)
- *
- * Entry point del ciclo de evolución en el evento Stop. Ejecuta en UN solo
- * proceso las sub-etapas que antes eran dos hooks separados
- * (`metricas-evolucion.js` + `actualizar-perfil-usuario.js`), reduciendo de
- * dos arranques de Node por sesión a uno (Fase 11, D-12).
- *
- * Nunca bloquea (siempre exit 0). Zero-deps fuera de hooks/lib/.
- */
-
-const cicloEvolucion = require('./lib/ciclo-evolucion');
-
-let inputRaw = '';
-process.stdin.on('data', chunk => { inputRaw += chunk; });
-process.stdin.on('end', () => {
-  try {
-    cicloEvolucion.ejecutarStop(inputRaw);
-  } catch {
-    // El hook nunca bloquea por error interno.
-  }
-});
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * Hook: ciclo-evolucion.js
+ * Tipo: Stop (async: true — fire-and-forget)
+ *
+ * Entry point del ciclo de evolución en el evento Stop. Ejecuta en UN solo
+ * proceso las sub-etapas que antes eran dos hooks separados
+ * (`metricas-evolucion.js` + `actualizar-perfil-usuario.js`), reduciendo de
+ * dos arranques de Node por sesión a uno (Fase 11, D-12).
+ *
+ * Nunca bloquea (siempre exit 0). Zero-deps fuera de hooks/lib/.
+ */
+
+const cicloEvolucion = require('./lib/ciclo-evolucion');
+
+let inputRaw = '';
+process.stdin.on('data', chunk => { inputRaw += chunk; });
+process.stdin.on('end', () => {
+  try {
+    cicloEvolucion.ejecutarStop(inputRaw);
+  } catch {
+    // El hook nunca bloquea por error interno.
+  }
+});

package/hooks/claudemd-bloat-detector.js

@@ -1,161 +1,161 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * Hook: claudemd-bloat-detector.js
- * Tipo: PostToolUse  (aplica a: Write, Edit, MultiEdit)
- *
- * Ejecuta `scripts/auditar-claudemd.js` contra archivos `CLAUDE.md`
- * recién modificados y emite un nudge a `.planning/evolution/nudges.jsonl`
- * si el veredicto es WARN o ERROR.
- *
- * Aplica ADR-0016 (best practices Anthropic "The CLAUDE.md file"):
- * detecta inflación (líneas excesivas, bullets monolíticos, secciones
- * canónicas ausentes, ausencia de @references) y sugiere intervención
- * con `/swl:claudemd refactor`.
- *
- * Opt-out: SWL_CLAUDEMD_BLOAT=0 desactiva completamente el hook.
- *
- * Comportamiento:
- *   - Nunca bloquea operaciones (exit code 0 siempre)
- *   - Solo emite nudge cuando veredicto != OK — ruido mínimo
- *   - Solo se dispara con archivos cuyo basename sea exactamente CLAUDE.md
- *   - Respeta exclusiones: temp/, node_modules/, respositorios-git/
- *
- * Formato del nudge:
- *   {
- *     id: string,
- *     kind: "claudemd-bloat",
- *     target: "documentador-swl",
- *     source: "hooks/claudemd-bloat-detector.js",
- *     message: "...",
- *     data: { archivo, veredicto, lineas, hallazgos_count },
- *     ts: ISO,
- *     accionado: false
- *   }
- */
-
-const fs = require('fs');
-const path = require('path');
-const crypto = require('crypto');
-
-// ─── Opt-out global ───────────────────────────────────────────────────────
-if (process.env.SWL_CLAUDEMD_BLOAT === '0') {
-  process.exit(0);
-}
-
-let hookInput = '';
-try {
-  hookInput = fs.readFileSync(0, 'utf-8');
-} catch (_) {
-  process.exit(0);
-}
-
-let evento;
-try {
-  evento = JSON.parse(hookInput);
-} catch (_) {
-  process.exit(0);
-}
-
-const toolName = evento?.tool_name;
-const toolInput = evento?.tool_input;
-
-if (!toolName || !['Write', 'Edit', 'MultiEdit'].includes(toolName)) {
-  process.exit(0);
-}
-
-const filePath = toolInput?.file_path;
-if (!filePath) {
-  process.exit(0);
-}
-
-// Solo CLAUDE.md (basename exacto, case-sensitive)
-const basename = path.basename(filePath);
-if (basename !== 'CLAUDE.md') {
-  process.exit(0);
-}
-
-const pathNormalized = filePath.replace(/\\/g, '/');
-const RUTAS_EXCLUIDAS = [
-  '/temp/',
-  '/node_modules/',
-  '/respositorios-git/',
-  '/.planning/',
-];
-if (RUTAS_EXCLUIDAS.some((excluida) => pathNormalized.includes(excluida))) {
-  process.exit(0);
-}
-
-// El archivo debe existir
-if (!fs.existsSync(filePath)) {
-  process.exit(0);
-}
-
-// ─── Ejecutar auditor (módulo, no subproceso) ─────────────────────────────
-const CWD = process.cwd();
-const auditorPath = path.join(CWD, 'scripts', 'auditar-claudemd.js');
-if (!fs.existsSync(auditorPath)) {
-  // No hay auditor instalado en este destino; salir silenciosamente
-  process.exit(0);
-}
-
-let resultado;
-try {
-  const { auditar } = require(auditorPath);
-  resultado = auditar(filePath);
-} catch (_) {
-  // Cualquier error del auditor: salir silenciosamente, no romper el hook
-  process.exit(0);
-}
-
-// Solo emitir nudge si veredicto != OK
-if (!resultado || resultado.veredicto === 'OK') {
-  process.exit(0);
-}
-
-// ─── Construir nudge ──────────────────────────────────────────────────────
-const rutaRelativa = path.relative(CWD, filePath).replace(/\\/g, '/');
-const topHallazgos = (resultado.hallazgos || [])
-  .slice(0, 3)
-  .map((h) => `  - [${h.severidad}] ${h.mensaje}`)
-  .join('\n');
-
-const nudge = {
-  id: crypto.randomBytes(8).toString('hex'),
-  kind: 'claudemd-bloat',
-  target: 'documentador-swl',
-  source: 'hooks/claudemd-bloat-detector.js',
-  message:
-    `[claudemd] ${rutaRelativa} veredicto: ${resultado.veredicto} ` +
-    `(${resultado.hallazgos.length} hallazgos)\n` +
-    topHallazgos + '\n' +
-    `  Ejecutar \`/swl:claudemd audit\` para detalle, ` +
-    `\`/swl:claudemd refactor\` para sugerencias de extracción.`,
-  data: {
-    archivo: rutaRelativa,
-    veredicto: resultado.veredicto,
-    lineas: resultado.metricas?.lineas,
-    secciones_ausentes: resultado.metricas?.secciones_ausentes || [],
-    tiene_at_references: resultado.metricas?.tiene_at_references,
-    hallazgos_count: resultado.hallazgos.length,
-  },
-  ts: new Date().toISOString(),
-  accionado: false,
-  accionado_ts: null,
-  accionado_por: null,
-};
-
-// ─── Persistir a nudges.jsonl ─────────────────────────────────────────────
-try {
-  const nudgesPath = path.join(CWD, '.planning', 'evolution', 'nudges.jsonl');
-  const nudgesDir = path.dirname(nudgesPath);
-  if (!fs.existsSync(nudgesDir)) {
-    fs.mkdirSync(nudgesDir, { recursive: true });
-  }
-  fs.appendFileSync(nudgesPath, JSON.stringify(nudge) + '\n', 'utf-8');
-} catch (_) {
-  // No fallar el hook por error de escritura
-}
-
-process.exit(0);
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * Hook: claudemd-bloat-detector.js
+ * Tipo: PostToolUse  (aplica a: Write, Edit, MultiEdit)
+ *
+ * Ejecuta `scripts/auditar-claudemd.js` contra archivos `CLAUDE.md`
+ * recién modificados y emite un nudge a `.planning/evolution/nudges.jsonl`
+ * si el veredicto es WARN o ERROR.
+ *
+ * Aplica ADR-0016 (best practices Anthropic "The CLAUDE.md file"):
+ * detecta inflación (líneas excesivas, bullets monolíticos, secciones
+ * canónicas ausentes, ausencia de @references) y sugiere intervención
+ * con `/swl:claudemd refactor`.
+ *
+ * Opt-out: SWL_CLAUDEMD_BLOAT=0 desactiva completamente el hook.
+ *
+ * Comportamiento:
+ *   - Nunca bloquea operaciones (exit code 0 siempre)
+ *   - Solo emite nudge cuando veredicto != OK — ruido mínimo
+ *   - Solo se dispara con archivos cuyo basename sea exactamente CLAUDE.md
+ *   - Respeta exclusiones: temp/, node_modules/, respositorios-git/
+ *
+ * Formato del nudge:
+ *   {
+ *     id: string,
+ *     kind: "claudemd-bloat",
+ *     target: "documentador-swl",
+ *     source: "hooks/claudemd-bloat-detector.js",
+ *     message: "...",
+ *     data: { archivo, veredicto, lineas, hallazgos_count },
+ *     ts: ISO,
+ *     accionado: false
+ *   }
+ */
+
+const fs = require('fs');
+const path = require('path');
+const crypto = require('crypto');
+
+// ─── Opt-out global ───────────────────────────────────────────────────────
+if (process.env.SWL_CLAUDEMD_BLOAT === '0') {
+  process.exit(0);
+}
+
+let hookInput = '';
+try {
+  hookInput = fs.readFileSync(0, 'utf-8');
+} catch (_) {
+  process.exit(0);
+}
+
+let evento;
+try {
+  evento = JSON.parse(hookInput);
+} catch (_) {
+  process.exit(0);
+}
+
+const toolName = evento?.tool_name;
+const toolInput = evento?.tool_input;
+
+if (!toolName || !['Write', 'Edit', 'MultiEdit'].includes(toolName)) {
+  process.exit(0);
+}
+
+const filePath = toolInput?.file_path;
+if (!filePath) {
+  process.exit(0);
+}
+
+// Solo CLAUDE.md (basename exacto, case-sensitive)
+const basename = path.basename(filePath);
+if (basename !== 'CLAUDE.md') {
+  process.exit(0);
+}
+
+const pathNormalized = filePath.replace(/\\/g, '/');
+const RUTAS_EXCLUIDAS = [
+  '/temp/',
+  '/node_modules/',
+  '/respositorios-git/',
+  '/.planning/',
+];
+if (RUTAS_EXCLUIDAS.some((excluida) => pathNormalized.includes(excluida))) {
+  process.exit(0);
+}
+
+// El archivo debe existir
+if (!fs.existsSync(filePath)) {
+  process.exit(0);
+}
+
+// ─── Ejecutar auditor (módulo, no subproceso) ─────────────────────────────
+const CWD = process.cwd();
+const auditorPath = path.join(CWD, 'scripts', 'auditar-claudemd.js');
+if (!fs.existsSync(auditorPath)) {
+  // No hay auditor instalado en este destino; salir silenciosamente
+  process.exit(0);
+}
+
+let resultado;
+try {
+  const { auditar } = require(auditorPath);
+  resultado = auditar(filePath);
+} catch (_) {
+  // Cualquier error del auditor: salir silenciosamente, no romper el hook
+  process.exit(0);
+}
+
+// Solo emitir nudge si veredicto != OK
+if (!resultado || resultado.veredicto === 'OK') {
+  process.exit(0);
+}
+
+// ─── Construir nudge ──────────────────────────────────────────────────────
+const rutaRelativa = path.relative(CWD, filePath).replace(/\\/g, '/');
+const topHallazgos = (resultado.hallazgos || [])
+  .slice(0, 3)
+  .map((h) => `  - [${h.severidad}] ${h.mensaje}`)
+  .join('\n');
+
+const nudge = {
+  id: crypto.randomBytes(8).toString('hex'),
+  kind: 'claudemd-bloat',
+  target: 'documentador-swl',
+  source: 'hooks/claudemd-bloat-detector.js',
+  message:
+    `[claudemd] ${rutaRelativa} veredicto: ${resultado.veredicto} ` +
+    `(${resultado.hallazgos.length} hallazgos)\n` +
+    topHallazgos + '\n' +
+    `  Ejecutar \`/swl:claudemd audit\` para detalle, ` +
+    `\`/swl:claudemd refactor\` para sugerencias de extracción.`,
+  data: {
+    archivo: rutaRelativa,
+    veredicto: resultado.veredicto,
+    lineas: resultado.metricas?.lineas,
+    secciones_ausentes: resultado.metricas?.secciones_ausentes || [],
+    tiene_at_references: resultado.metricas?.tiene_at_references,
+    hallazgos_count: resultado.hallazgos.length,
+  },
+  ts: new Date().toISOString(),
+  accionado: false,
+  accionado_ts: null,
+  accionado_por: null,
+};
+
+// ─── Persistir a nudges.jsonl ─────────────────────────────────────────────
+try {
+  const nudgesPath = path.join(CWD, '.planning', 'evolution', 'nudges.jsonl');
+  const nudgesDir = path.dirname(nudgesPath);
+  if (!fs.existsSync(nudgesDir)) {
+    fs.mkdirSync(nudgesDir, { recursive: true });
+  }
+  fs.appendFileSync(nudgesPath, JSON.stringify(nudge) + '\n', 'utf-8');
+} catch (_) {
+  // No fallar el hook por error de escritura
+}
+
+process.exit(0);