@saulwade/swl-ses 1.4.1 → 1.5.0

package/scripts/mcp-server/telemetry.js

@@ -0,0 +1,78 @@
+'use strict';
+
+/**
+ * Telemetría opt-in del swl-mcp-server v1.0.0 (ADR-0019 Sub-fase 3).
+ *
+ * Cuando `SWL_MCP_METRICS=1` está activo, cada `tools/call` se registra en
+ * `.planning/evolucion/mcp-metrics.jsonl` con:
+ *   { ts, tool, durationMs, ok, error?, baseDir }
+ *
+ * Defaults:
+ *   - SWL_MCP_METRICS no set → no se persiste nada (zero-cost).
+ *   - Se usa append-only JSONL (alta frecuencia → atomicWriteJSON sería ineficiente).
+ *   - Si el filesystem no permite escritura (p.ej. baseDir read-only), el error
+ *     se traga silenciosamente — la telemetría NUNCA debe romper el server.
+ *
+ * Zero-deps.
+ *
+ * @module scripts/mcp-server/telemetry
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+/**
+ * Construye un grabador de telemetría desde el entorno actual.
+ *
+ * @param {object} [opciones]
+ * @param {string} opciones.baseDir - Directorio raíz del proyecto SWL (donde está .planning/).
+ * @param {object} [opciones.env] - Sustituible para tests.
+ * @returns {{ habilitada: boolean, registrar: Function, ruta: string|null }}
+ */
+function construirTelemetria(opciones = {}) {
+  const env = opciones.env || process.env;
+  const habilitada = env.SWL_MCP_METRICS === '1' || env.SWL_MCP_METRICS === 'true';
+  const baseDir = opciones.baseDir;
+
+  if (!habilitada || !baseDir) {
+    return {
+      habilitada: false,
+      registrar: () => {}, // noop
+      ruta: null,
+    };
+  }
+
+  const dirEvolucion = path.join(baseDir, '.planning', 'evolucion');
+  const ruta = path.join(dirEvolucion, 'mcp-metrics.jsonl');
+
+  // Asegurar que el directorio existe — si falla, deshabilitamos silenciosamente.
+  let escribible = true;
+  try {
+    fs.mkdirSync(dirEvolucion, { recursive: true });
+  } catch (err) {
+    escribible = false;
+  }
+
+  return {
+    habilitada: escribible,
+    registrar: (evento) => {
+      if (!escribible) return;
+      try {
+        const linea = JSON.stringify({
+          ts: new Date().toISOString(),
+          ...evento,
+        }) + '\n';
+        fs.appendFileSync(ruta, linea, { encoding: 'utf-8' });
+      } catch {
+        // Telemetría nunca debe romper el server — error silencioso.
+        // Tras 5 errores consecutivos podríamos deshabilitar, pero no
+        // implementamos eso hasta que aparezca como problema real.
+      }
+    },
+    ruta,
+  };
+}
+
+module.exports = {
+  construirTelemetria,
+};

package/scripts/migrar-csv-a-array.js

@@ -1,168 +1,168 @@
-#!/usr/bin/env node
-/**
- * Migración 3A — convierte campos CSV a array YAML inline en frontmatter de
- * agentes. Afecta `tools`, `skillsInvocables`, `skillsRestringidos`.
- *
- * Antes:  tools: Read, Write, Edit, Bash
- * Después: tools: [Read, Write, Edit, Bash]
- *
- * Razón: el schema declara `type: array` pero el uso real es CSV string. YAML
- * estándar NO convierte CSV a array — el parser produce una sola string. Esto
- * causó bugs históricos documentados en CLAUDE.md ("regla obligatoria CSV
- * estricto"). Migrar a array inline elimina la ambigüedad.
- *
- * Idempotente: skip si ya es array.
- *
- * Casos especiales detectados en agentes:
- *   - skillsInvocables: <ninguno>      → mantener literal
- *   - skillsRestringidos: ninguno      → convertir a [] (array vacío)
- *   - skillsRestringidos: \n  - x\n    → ya es array YAML multilinea, mantener
- *
- * Uso:
- *   node scripts/migrar-csv-a-array.js [--dry-run]
- *
- * Zero-dependencies. Compatible Windows (CRLF-safe), Node 18+.
- */
-
-'use strict';
-
-const fs = require('fs');
-const path = require('path');
-
-const AGENTES_DIR = path.join(__dirname, '..', 'agentes');
-const DRY_RUN = process.argv.includes('--dry-run');
-
-// Campos que migran de CSV a array
-const CAMPOS_CSV = ['tools', 'skillsInvocables', 'skillsRestringidos'];
-
-// Valores especiales que NO son listas y deben preservarse
-const VALORES_LITERAL = new Set(['ninguno', 'ninguna', '<ninguno>', '<ninguna>', 'none', 'null']);
-
-function parsearValorCSV(valorStr) {
-  // Detectar si ya es array inline `[a, b, c]`
-  if (/^\s*\[/.test(valorStr)) return null;
-
-  // Limpiar
-  const valor = valorStr.trim();
-  if (!valor) return [];
-
-  // Valor especial literal
-  if (VALORES_LITERAL.has(valor.toLowerCase())) return [];
-
-  // CSV split
-  return valor
-    .split(',')
-    .map((s) => s.trim())
-    .filter(Boolean);
-}
-
-function migrarFrontmatter(contenido) {
-  const match = contenido.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n/);
-  if (!match) return { contenido, cambios: 0 };
-
-  const eol = match[0].includes('\r\n') ? '\r\n' : '\n';
-  const fmCompleto = match[0];
-
-  let nuevoFm = fmCompleto;
-  let cambios = 0;
-
-  for (const campo of CAMPOS_CSV) {
-    // Patrón 2 PRIMERO: array YAML multilínea
-    //   skillsRestringidos:
-    //     - angular-moderno
-    //     - typescript-avanzado
-    // El campo está solo en su línea (sin valor) seguido de líneas con `  - X`
-    const re2 = new RegExp(`^(${campo}):[ \\t]*\\r?\\n((?:[ \\t]+-[ \\t]+[^\\r\\n]+\\r?\\n)+)`, 'm');
-    const m2 = nuevoFm.match(re2);
-    if (m2) {
-      const bloque = m2[2];
-      const items = bloque
-        .split(/\r?\n/)
-        .map((l) => l.trim())
-        .filter((l) => l.startsWith('- '))
-        .map((l) => l.slice(2).trim().replace(/^["']|["']$/g, ''));
-      if (items.length > 0) {
-        const arrayInline = `[${items.join(', ')}]`;
-        const reemplazo = `${campo}: ${arrayInline}${eol}`;
-        nuevoFm = nuevoFm.replace(re2, reemplazo);
-        cambios++;
-      }
-      continue; // procesado, no aplicar regex 1
-    }
-
-    // Patrón 1: campo en una sola línea con valor (CSV o array inline)
-    //   tools: Read, Write
-    //   skillsRestringidos: ninguno
-    //   tools: [Read, Write]   ← skip, ya es array
-    // CRÍTICO: usar [ \\t]* (NO \\s*) para no consumir newlines
-    const re1 = new RegExp(`^(${campo}):[ \\t]*([^\\r\\n]+)$`, 'm');
-    const m1 = nuevoFm.match(re1);
-    if (m1) {
-      const valorStr = m1[2];
-      const items = parsearValorCSV(valorStr);
-      if (items === null) continue; // ya es array, skip
-      const arrayInline = `[${items.join(', ')}]`;
-      const reemplazo = `${campo}: ${arrayInline}`;
-      if (m1[0] !== reemplazo) {
-        nuevoFm = nuevoFm.replace(re1, reemplazo);
-        cambios++;
-      }
-    }
-  }
-
-  if (cambios === 0) return { contenido, cambios: 0 };
-
-  return { contenido: contenido.replace(fmCompleto, nuevoFm), cambios };
-}
-
-function migrarAgente(archivo) {
-  const contenido = fs.readFileSync(archivo, 'utf-8');
-  const { contenido: nuevo, cambios } = migrarFrontmatter(contenido);
-
-  if (cambios === 0) {
-    return { archivo: path.basename(archivo), status: 'sin-cambios' };
-  }
-
-  if (DRY_RUN) {
-    return { archivo: path.basename(archivo), status: 'dry-run', cambios };
-  }
-
-  fs.writeFileSync(archivo, nuevo, 'utf-8');
-  return { archivo: path.basename(archivo), status: 'migrado', cambios };
-}
-
-function main() {
-  console.log(`Migrando CSV → array YAML${DRY_RUN ? ' (dry-run)' : ''}...\n`);
-
-  const archivos = fs
-    .readdirSync(AGENTES_DIR)
-    .filter((f) => f.endsWith('.md') && !f.startsWith('_'))
-    .map((f) => path.join(AGENTES_DIR, f));
-
-  const resultados = archivos.map(migrarAgente);
-
-  const stats = {
-    'sin-cambios': 0,
-    migrado: 0,
-    'dry-run': 0,
-    totalCambios: 0,
-  };
-
-  for (const r of resultados) {
-    stats[r.status] = (stats[r.status] || 0) + 1;
-    if (r.cambios) stats.totalCambios += r.cambios;
-  }
-
-  console.log(`Sin cambios: ${stats['sin-cambios']}`);
-  console.log(`Migrados: ${stats.migrado}`);
-  console.log(`Dry-run: ${stats['dry-run']}`);
-  console.log(`Total ediciones: ${stats.totalCambios}`);
-
-  process.exit(0);
-}
-
-if (require.main === module) {
-  main();
-}
-
-module.exports = { parsearValorCSV, migrarFrontmatter };
+#!/usr/bin/env node
+/**
+ * Migración 3A — convierte campos CSV a array YAML inline en frontmatter de
+ * agentes. Afecta `tools`, `skillsInvocables`, `skillsRestringidos`.
+ *
+ * Antes:  tools: Read, Write, Edit, Bash
+ * Después: tools: [Read, Write, Edit, Bash]
+ *
+ * Razón: el schema declara `type: array` pero el uso real es CSV string. YAML
+ * estándar NO convierte CSV a array — el parser produce una sola string. Esto
+ * causó bugs históricos documentados en CLAUDE.md ("regla obligatoria CSV
+ * estricto"). Migrar a array inline elimina la ambigüedad.
+ *
+ * Idempotente: skip si ya es array.
+ *
+ * Casos especiales detectados en agentes:
+ *   - skillsInvocables: <ninguno>      → mantener literal
+ *   - skillsRestringidos: ninguno      → convertir a [] (array vacío)
+ *   - skillsRestringidos: \n  - x\n    → ya es array YAML multilinea, mantener
+ *
+ * Uso:
+ *   node scripts/migrar-csv-a-array.js [--dry-run]
+ *
+ * Zero-dependencies. Compatible Windows (CRLF-safe), Node 18+.
+ */
+
+'use strict';
+
+const fs = require('fs');
+const path = require('path');
+
+const AGENTES_DIR = path.join(__dirname, '..', 'agentes');
+const DRY_RUN = process.argv.includes('--dry-run');
+
+// Campos que migran de CSV a array
+const CAMPOS_CSV = ['tools', 'skillsInvocables', 'skillsRestringidos'];
+
+// Valores especiales que NO son listas y deben preservarse
+const VALORES_LITERAL = new Set(['ninguno', 'ninguna', '<ninguno>', '<ninguna>', 'none', 'null']);
+
+function parsearValorCSV(valorStr) {
+  // Detectar si ya es array inline `[a, b, c]`
+  if (/^\s*\[/.test(valorStr)) return null;
+
+  // Limpiar
+  const valor = valorStr.trim();
+  if (!valor) return [];
+
+  // Valor especial literal
+  if (VALORES_LITERAL.has(valor.toLowerCase())) return [];
+
+  // CSV split
+  return valor
+    .split(',')
+    .map((s) => s.trim())
+    .filter(Boolean);
+}
+
+function migrarFrontmatter(contenido) {
+  const match = contenido.match(/^---\r?\n([\s\S]*?)\r?\n---\r?\n/);
+  if (!match) return { contenido, cambios: 0 };
+
+  const eol = match[0].includes('\r\n') ? '\r\n' : '\n';
+  const fmCompleto = match[0];
+
+  let nuevoFm = fmCompleto;
+  let cambios = 0;
+
+  for (const campo of CAMPOS_CSV) {
+    // Patrón 2 PRIMERO: array YAML multilínea
+    //   skillsRestringidos:
+    //     - angular-moderno
+    //     - typescript-avanzado
+    // El campo está solo en su línea (sin valor) seguido de líneas con `  - X`
+    const re2 = new RegExp(`^(${campo}):[ \\t]*\\r?\\n((?:[ \\t]+-[ \\t]+[^\\r\\n]+\\r?\\n)+)`, 'm');
+    const m2 = nuevoFm.match(re2);
+    if (m2) {
+      const bloque = m2[2];
+      const items = bloque
+        .split(/\r?\n/)
+        .map((l) => l.trim())
+        .filter((l) => l.startsWith('- '))
+        .map((l) => l.slice(2).trim().replace(/^["']|["']$/g, ''));
+      if (items.length > 0) {
+        const arrayInline = `[${items.join(', ')}]`;
+        const reemplazo = `${campo}: ${arrayInline}${eol}`;
+        nuevoFm = nuevoFm.replace(re2, reemplazo);
+        cambios++;
+      }
+      continue; // procesado, no aplicar regex 1
+    }
+
+    // Patrón 1: campo en una sola línea con valor (CSV o array inline)
+    //   tools: Read, Write
+    //   skillsRestringidos: ninguno
+    //   tools: [Read, Write]   ← skip, ya es array
+    // CRÍTICO: usar [ \\t]* (NO \\s*) para no consumir newlines
+    const re1 = new RegExp(`^(${campo}):[ \\t]*([^\\r\\n]+)$`, 'm');
+    const m1 = nuevoFm.match(re1);
+    if (m1) {
+      const valorStr = m1[2];
+      const items = parsearValorCSV(valorStr);
+      if (items === null) continue; // ya es array, skip
+      const arrayInline = `[${items.join(', ')}]`;
+      const reemplazo = `${campo}: ${arrayInline}`;
+      if (m1[0] !== reemplazo) {
+        nuevoFm = nuevoFm.replace(re1, reemplazo);
+        cambios++;
+      }
+    }
+  }
+
+  if (cambios === 0) return { contenido, cambios: 0 };
+
+  return { contenido: contenido.replace(fmCompleto, nuevoFm), cambios };
+}
+
+function migrarAgente(archivo) {
+  const contenido = fs.readFileSync(archivo, 'utf-8');
+  const { contenido: nuevo, cambios } = migrarFrontmatter(contenido);
+
+  if (cambios === 0) {
+    return { archivo: path.basename(archivo), status: 'sin-cambios' };
+  }
+
+  if (DRY_RUN) {
+    return { archivo: path.basename(archivo), status: 'dry-run', cambios };
+  }
+
+  fs.writeFileSync(archivo, nuevo, 'utf-8');
+  return { archivo: path.basename(archivo), status: 'migrado', cambios };
+}
+
+function main() {
+  console.log(`Migrando CSV → array YAML${DRY_RUN ? ' (dry-run)' : ''}...\n`);
+
+  const archivos = fs
+    .readdirSync(AGENTES_DIR)
+    .filter((f) => f.endsWith('.md') && !f.startsWith('_'))
+    .map((f) => path.join(AGENTES_DIR, f));
+
+  const resultados = archivos.map(migrarAgente);
+
+  const stats = {
+    'sin-cambios': 0,
+    migrado: 0,
+    'dry-run': 0,
+    totalCambios: 0,
+  };
+
+  for (const r of resultados) {
+    stats[r.status] = (stats[r.status] || 0) + 1;
+    if (r.cambios) stats.totalCambios += r.cambios;
+  }
+
+  console.log(`Sin cambios: ${stats['sin-cambios']}`);
+  console.log(`Migrados: ${stats.migrado}`);
+  console.log(`Dry-run: ${stats['dry-run']}`);
+  console.log(`Total ediciones: ${stats.totalCambios}`);
+
+  process.exit(0);
+}
+
+if (require.main === module) {
+  main();
+}
+
+module.exports = { parsearValorCSV, migrarFrontmatter };