@saulwade/swl-ses 1.4.0 → 1.4.2

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 };