@saulwade/swl-ses 2.3.0 → 2.4.1

package/scripts/limpiar-artefactos-python.js

@@ -1,131 +1,131 @@
-#!/usr/bin/env node
-'use strict';
-
-/**
- * limpiar-artefactos-python.js — limpia caché Python antes de empaquetar.
- *
- * Se ejecuta como `prepack` antes de `npm pack` y `npm publish` para evitar
- * que artefactos locales (.pyc, __pycache__/) contaminen el tarball publicado.
- *
- * Reglas de seguridad (defensa en profundidad):
- *   1. Aborta si cwd no coincide con la raíz del package.json del repo.
- *   2. Profundidad máxima de recursión: 3 niveles desde la raíz.
- *   3. Allowlist explícita de directorios a EXCLUIR de la búsqueda
- *      (node_modules, .git, temp, .planning, respositorios-git, _userland).
- *   4. Solo elimina directorios cuyo nombre coincide exactamente con el
- *      conjunto cerrado: __pycache__, .pytest_cache, .mypy_cache, .ruff_cache.
- *   5. Solo elimina archivos sueltos con extensiones .pyc, .pyo.
- *   6. En CI no-interactivo respeta el flag SWL_PREPACK_DRY=1 (no borra,
- *      solo lista).
- *
- * Exit codes:
- *   0 — OK (limpieza ejecutada o nada que limpiar)
- *   1 — error de invariante (cwd incorrecto, package.json no encontrado)
- */
-
-const fs   = require('fs');
-const path = require('path');
-
-const { NOMBRES_VALIDOS } = require('./lib/paquetes-conocidos');
-const {
-  DIRS_ARTEFACTOS_PYTHON: DIRS_A_LIMPIAR,
-  EXTS_ARTEFACTOS_PYTHON: EXTS_A_LIMPIAR,
-} = require('./lib/artefactos-python');
-
-const PROFUNDIDAD_MAX = 3;
-const DIRS_EXCLUIDOS  = new Set([
-  'node_modules', '.git', 'temp', '.planning', 'respositorios-git',
-  '_userland', 'tests', '.github',
-]);
-
-const dryRun = process.env.SWL_PREPACK_DRY === '1';
-
-function log(msg) { process.stdout.write(`[prepack] ${msg}\n`); }
-function err(msg) { process.stderr.write(`[prepack] ERROR: ${msg}\n`); }
-
-/**
- * Verifica que el cwd actual contiene el package.json del repo swl-ses.
- * Esto evita que el script borre directorios en la máquina del usuario si
- * algún día se invocara desde un cwd incorrecto (ej. dentro de un tarball
- * extraído por npm en .npm/_cacache).
- */
-function verificarRaiz() {
-  const cwd = process.cwd();
-  const pkgPath = path.join(cwd, 'package.json');
-  if (!fs.existsSync(pkgPath)) {
-    err(`no existe package.json en cwd: ${cwd}`);
-    process.exit(1);
-  }
-  const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
-  if (!NOMBRES_VALIDOS.includes(pkg.name)) {
-    err(`package.json en ${cwd} no corresponde a swl-ses (name: ${pkg.name}). Abortando por seguridad.`);
-    process.exit(1);
-  }
-  return cwd;
-}
-
-/**
- * Recorre el árbol desde root con profundidad limitada y elimina los
- * artefactos Python detectados. No sigue symlinks.
- */
-function limpiar(root, profundidad = 0) {
-  if (profundidad > PROFUNDIDAD_MAX) return { dirs: 0, files: 0 };
-
-  let dirsEliminados = 0;
-  let filesEliminados = 0;
-
-  let entries;
-  try {
-    entries = fs.readdirSync(root, { withFileTypes: true });
-  } catch (e) {
-    err(`no se pudo leer ${root}: ${e.message}`);
-    return { dirs: 0, files: 0 };
-  }
-
-  for (const entry of entries) {
-    if (entry.isSymbolicLink()) continue;
-    const fullPath = path.join(root, entry.name);
-
-    if (entry.isDirectory()) {
-      if (DIRS_A_LIMPIAR.has(entry.name)) {
-        if (dryRun) {
-          log(`[dry-run] borraría dir: ${path.relative(process.cwd(), fullPath)}`);
-        } else {
-          fs.rmSync(fullPath, { recursive: true, force: true });
-          log(`borrado dir: ${path.relative(process.cwd(), fullPath)}`);
-        }
-        dirsEliminados++;
-      } else if (!DIRS_EXCLUIDOS.has(entry.name) && !entry.name.startsWith('.')) {
-        const sub = limpiar(fullPath, profundidad + 1);
-        dirsEliminados  += sub.dirs;
-        filesEliminados += sub.files;
-      }
-    } else if (entry.isFile()) {
-      const ext = path.extname(entry.name).toLowerCase();
-      if (EXTS_A_LIMPIAR.has(ext)) {
-        if (dryRun) {
-          log(`[dry-run] borraría archivo: ${path.relative(process.cwd(), fullPath)}`);
-        } else {
-          fs.rmSync(fullPath, { force: true });
-        }
-        filesEliminados++;
-      }
-    }
-  }
-
-  return { dirs: dirsEliminados, files: filesEliminados };
-}
-
-function main() {
-  const root = verificarRaiz();
-  const result = limpiar(root);
-
-  if (result.dirs === 0 && result.files === 0) {
-    log('sin artefactos Python que limpiar.');
-  } else {
-    const accion = dryRun ? 'detectados' : 'eliminados';
-    log(`${accion}: ${result.dirs} directorios + ${result.files} archivos.`);
-  }
-}
-
-main();
+#!/usr/bin/env node
+'use strict';
+
+/**
+ * limpiar-artefactos-python.js — limpia caché Python antes de empaquetar.
+ *
+ * Se ejecuta como `prepack` antes de `npm pack` y `npm publish` para evitar
+ * que artefactos locales (.pyc, __pycache__/) contaminen el tarball publicado.
+ *
+ * Reglas de seguridad (defensa en profundidad):
+ *   1. Aborta si cwd no coincide con la raíz del package.json del repo.
+ *   2. Profundidad máxima de recursión: 3 niveles desde la raíz.
+ *   3. Allowlist explícita de directorios a EXCLUIR de la búsqueda
+ *      (node_modules, .git, temp, .planning, respositorios-git, _userland).
+ *   4. Solo elimina directorios cuyo nombre coincide exactamente con el
+ *      conjunto cerrado: __pycache__, .pytest_cache, .mypy_cache, .ruff_cache.
+ *   5. Solo elimina archivos sueltos con extensiones .pyc, .pyo.
+ *   6. En CI no-interactivo respeta el flag SWL_PREPACK_DRY=1 (no borra,
+ *      solo lista).
+ *
+ * Exit codes:
+ *   0 — OK (limpieza ejecutada o nada que limpiar)
+ *   1 — error de invariante (cwd incorrecto, package.json no encontrado)
+ */
+
+const fs   = require('fs');
+const path = require('path');
+
+const { NOMBRES_VALIDOS } = require('./lib/paquetes-conocidos');
+const {
+  DIRS_ARTEFACTOS_PYTHON: DIRS_A_LIMPIAR,
+  EXTS_ARTEFACTOS_PYTHON: EXTS_A_LIMPIAR,
+} = require('./lib/artefactos-python');
+
+const PROFUNDIDAD_MAX = 3;
+const DIRS_EXCLUIDOS  = new Set([
+  'node_modules', '.git', 'temp', '.planning', 'respositorios-git',
+  '_userland', 'tests', '.github',
+]);
+
+const dryRun = process.env.SWL_PREPACK_DRY === '1';
+
+function log(msg) { process.stdout.write(`[prepack] ${msg}\n`); }
+function err(msg) { process.stderr.write(`[prepack] ERROR: ${msg}\n`); }
+
+/**
+ * Verifica que el cwd actual contiene el package.json del repo swl-ses.
+ * Esto evita que el script borre directorios en la máquina del usuario si
+ * algún día se invocara desde un cwd incorrecto (ej. dentro de un tarball
+ * extraído por npm en .npm/_cacache).
+ */
+function verificarRaiz() {
+  const cwd = process.cwd();
+  const pkgPath = path.join(cwd, 'package.json');
+  if (!fs.existsSync(pkgPath)) {
+    err(`no existe package.json en cwd: ${cwd}`);
+    process.exit(1);
+  }
+  const pkg = JSON.parse(fs.readFileSync(pkgPath, 'utf-8'));
+  if (!NOMBRES_VALIDOS.includes(pkg.name)) {
+    err(`package.json en ${cwd} no corresponde a swl-ses (name: ${pkg.name}). Abortando por seguridad.`);
+    process.exit(1);
+  }
+  return cwd;
+}
+
+/**
+ * Recorre el árbol desde root con profundidad limitada y elimina los
+ * artefactos Python detectados. No sigue symlinks.
+ */
+function limpiar(root, profundidad = 0) {
+  if (profundidad > PROFUNDIDAD_MAX) return { dirs: 0, files: 0 };
+
+  let dirsEliminados = 0;
+  let filesEliminados = 0;
+
+  let entries;
+  try {
+    entries = fs.readdirSync(root, { withFileTypes: true });
+  } catch (e) {
+    err(`no se pudo leer ${root}: ${e.message}`);
+    return { dirs: 0, files: 0 };
+  }
+
+  for (const entry of entries) {
+    if (entry.isSymbolicLink()) continue;
+    const fullPath = path.join(root, entry.name);
+
+    if (entry.isDirectory()) {
+      if (DIRS_A_LIMPIAR.has(entry.name)) {
+        if (dryRun) {
+          log(`[dry-run] borraría dir: ${path.relative(process.cwd(), fullPath)}`);
+        } else {
+          fs.rmSync(fullPath, { recursive: true, force: true });
+          log(`borrado dir: ${path.relative(process.cwd(), fullPath)}`);
+        }
+        dirsEliminados++;
+      } else if (!DIRS_EXCLUIDOS.has(entry.name) && !entry.name.startsWith('.')) {
+        const sub = limpiar(fullPath, profundidad + 1);
+        dirsEliminados  += sub.dirs;
+        filesEliminados += sub.files;
+      }
+    } else if (entry.isFile()) {
+      const ext = path.extname(entry.name).toLowerCase();
+      if (EXTS_A_LIMPIAR.has(ext)) {
+        if (dryRun) {
+          log(`[dry-run] borraría archivo: ${path.relative(process.cwd(), fullPath)}`);
+        } else {
+          fs.rmSync(fullPath, { force: true });
+        }
+        filesEliminados++;
+      }
+    }
+  }
+
+  return { dirs: dirsEliminados, files: filesEliminados };
+}
+
+function main() {
+  const root = verificarRaiz();
+  const result = limpiar(root);
+
+  if (result.dirs === 0 && result.files === 0) {
+    log('sin artefactos Python que limpiar.');
+  } else {
+    const accion = dryRun ? 'detectados' : 'eliminados';
+    log(`${accion}: ${result.dirs} directorios + ${result.files} archivos.`);
+  }
+}
+
+main();

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