@saulwade/swl-ses 1.4.1 → 1.5.0

package/scripts/lib/toml-merge.js

@@ -0,0 +1,204 @@
+'use strict';
+
+/**
+ * Merge mínimo de secciones TOML — zero-deps.
+ *
+ * Opera sobre el formato concreto de `~/.codex/config.toml` (OpenAI Codex CLI):
+ *   [mcp_servers.<name>]
+ *   command = "node"
+ *   args = ["...", "..."]
+ *   [mcp_servers.<name>.env]
+ *   KEY = "value"
+ *
+ * Reglas:
+ *  - NUNCA reescribe el archivo entero. Solo agrega o reemplaza la sección
+ *    `[mcp_servers.<name>]` indicada (incluyendo sub-tabla `.env`).
+ *  - Si el archivo no existe lo crea con permisos 0600.
+ *  - Si el archivo tiene comentarios o secciones desconocidas, las preserva.
+ *  - Bloque "owned" delimitado por marcadores `# SWL-MCP-BEGIN <name>` /
+ *    `# SWL-MCP-END <name>` para permitir actualizaciones idempotentes.
+ *
+ * NO soporta:
+ *  - TOML completo (arrays multilínea complejos, inline tables, etc.)
+ *  - Multi-section nested tables fuera de `mcp_servers.*`
+ *
+ * Si en el futuro Codex publica un comando idempotente `codex mcp add --idempotent`
+ * podemos delegarle el trabajo y deprecar este helper.
+ *
+ * @module scripts/lib/toml-merge
+ */
+
+const fs = require('fs');
+const path = require('path');
+
+const { atomicWriteSync } = require('../../hooks/lib/atomic-write');
+
+const BEGIN_PREFIX = '# SWL-MCP-BEGIN';
+const END_PREFIX = '# SWL-MCP-END';
+
+/**
+ * Serializa un servidor MCP en bloque TOML.
+ *
+ * @param {string} name - Nombre del server (identificador en config.toml).
+ * @param {{ command: string, args?: string[], env?: Record<string,string> }} cfg
+ * @returns {string} Bloque TOML con marcadores.
+ */
+function serializarServidor(name, cfg) {
+  if (!name || typeof name !== 'string' || !/^[a-zA-Z0-9_-]+$/.test(name)) {
+    throw new Error(`Nombre de MCP server inválido: "${name}". Solo [a-zA-Z0-9_-].`);
+  }
+  if (!cfg || typeof cfg.command !== 'string' || cfg.command.length === 0) {
+    throw new Error(`MCP server "${name}" requiere "command" string no vacío.`);
+  }
+
+  const lineas = [];
+  lineas.push(`${BEGIN_PREFIX} ${name}`);
+  lineas.push(`[mcp_servers.${name}]`);
+  lineas.push(`command = ${tomlString(cfg.command)}`);
+
+  if (Array.isArray(cfg.args) && cfg.args.length > 0) {
+    const items = cfg.args.map(tomlString).join(', ');
+    lineas.push(`args = [${items}]`);
+  }
+
+  if (cfg.env && typeof cfg.env === 'object' && Object.keys(cfg.env).length > 0) {
+    lineas.push('');
+    lineas.push(`[mcp_servers.${name}.env]`);
+    for (const [k, v] of Object.entries(cfg.env)) {
+      if (typeof v !== 'string') {
+        throw new Error(`MCP server "${name}" env.${k} debe ser string (recibido: ${typeof v}).`);
+      }
+      // Solo claves alfanuméricas/_ en env names — TOML restricción razonable
+      if (!/^[A-Za-z_][A-Za-z0-9_]*$/.test(k)) {
+        throw new Error(`MCP server "${name}" env key inválido: "${k}".`);
+      }
+      lineas.push(`${k} = ${tomlString(v)}`);
+    }
+  }
+
+  lineas.push(`${END_PREFIX} ${name}`);
+  return lineas.join('\n');
+}
+
+/**
+ * Escapa un string para TOML usando comillas dobles.
+ * No usa basic-string raw para evitar pitfalls con backslashes.
+ */
+function tomlString(s) {
+  // TOML basic string: escape backslash, doblequote, control chars.
+  const escapado = String(s)
+    .replace(/\\/g, '\\\\')
+    .replace(/"/g, '\\"')
+    .replace(/\n/g, '\\n')
+    .replace(/\r/g, '\\r')
+    .replace(/\t/g, '\\t');
+  return `"${escapado}"`;
+}
+
+/**
+ * Inserta o actualiza un servidor MCP en `config.toml`.
+ *
+ * Idempotente: si el bloque ya existe con el mismo contenido, no escribe.
+ * Si existe con contenido distinto, lo reemplaza dentro de los marcadores.
+ * Si no existe, lo agrega al final del archivo precedido de línea en blanco.
+ *
+ * @param {string} rutaConfig - Path absoluto a config.toml (típicamente ~/.codex/config.toml).
+ * @param {string} name - Nombre del MCP server (clave de [mcp_servers.<name>]).
+ * @param {{ command: string, args?: string[], env?: Record<string,string> }} cfg
+ * @returns {{ accion: 'creado' | 'agregado' | 'actualizado' | 'sin-cambios', ruta: string }}
+ */
+function upsertMcpServer(rutaConfig, name, cfg) {
+  const bloqueNuevo = serializarServidor(name, cfg);
+
+  let existente = '';
+  let existia = false;
+  try {
+    existente = fs.readFileSync(rutaConfig, 'utf-8');
+    existia = true;
+  } catch (err) {
+    if (err.code !== 'ENOENT') throw err;
+    // Archivo no existe — se crea
+  }
+
+  const beginTag = `${BEGIN_PREFIX} ${name}`;
+  const endTag = `${END_PREFIX} ${name}`;
+
+  if (!existia) {
+    const dir = path.dirname(rutaConfig);
+    fs.mkdirSync(dir, { recursive: true });
+    const contenido = `# config.toml de Codex CLI — gestionado parcialmente por swl-ses\n# Bloques delimitados por SWL-MCP-BEGIN/END son regenerables.\n\n${bloqueNuevo}\n`;
+    atomicWriteSync(rutaConfig, contenido, 'utf8', { mode: 0o600 });
+    return { accion: 'creado', ruta: rutaConfig };
+  }
+
+  const idxBegin = existente.indexOf(beginTag);
+  const idxEnd = existente.indexOf(endTag);
+
+  // Caso 1: no existe el bloque → append al final
+  if (idxBegin === -1 || idxEnd === -1) {
+    const separador = existente.endsWith('\n') ? '\n' : '\n\n';
+    const contenido = existente + separador + bloqueNuevo + '\n';
+    atomicWriteSync(rutaConfig, contenido, 'utf8', { mode: 0o600 });
+    return { accion: 'agregado', ruta: rutaConfig };
+  }
+
+  // Caso 2: existe — comparar contenido
+  // Calculamos fin REAL incluyendo el endTag completo.
+  const finBloqueExistente = idxEnd + endTag.length;
+  const bloqueExistente = existente.slice(idxBegin, finBloqueExistente);
+
+  if (bloqueExistente === bloqueNuevo) {
+    return { accion: 'sin-cambios', ruta: rutaConfig };
+  }
+
+  const contenido = existente.slice(0, idxBegin) + bloqueNuevo + existente.slice(finBloqueExistente);
+  atomicWriteSync(rutaConfig, contenido, 'utf8', { mode: 0o600 });
+  return { accion: 'actualizado', ruta: rutaConfig };
+}
+
+/**
+ * Elimina un bloque MCP server por nombre. Idempotente: si no existe, no hace nada.
+ *
+ * @param {string} rutaConfig
+ * @param {string} name
+ * @returns {{ accion: 'eliminado' | 'sin-cambios', ruta: string }}
+ */
+function removeMcpServer(rutaConfig, name) {
+  let existente = '';
+  try {
+    existente = fs.readFileSync(rutaConfig, 'utf-8');
+  } catch (err) {
+    if (err.code === 'ENOENT') return { accion: 'sin-cambios', ruta: rutaConfig };
+    throw err;
+  }
+
+  const beginTag = `${BEGIN_PREFIX} ${name}`;
+  const endTag = `${END_PREFIX} ${name}`;
+  const idxBegin = existente.indexOf(beginTag);
+  const idxEnd = existente.indexOf(endTag);
+
+  if (idxBegin === -1 || idxEnd === -1) {
+    return { accion: 'sin-cambios', ruta: rutaConfig };
+  }
+
+  // Quitar bloque + posible newline previo y posterior (cosmético)
+  let inicio = idxBegin;
+  let fin = idxEnd + endTag.length;
+  // Comer un \n previo si existe (no comer doble \n para preservar separadores entre secciones)
+  if (inicio > 0 && existente[inicio - 1] === '\n') inicio--;
+  if (fin < existente.length && existente[fin] === '\n') fin++;
+
+  const contenido = existente.slice(0, inicio) + existente.slice(fin);
+  atomicWriteSync(rutaConfig, contenido, 'utf8', { mode: 0o600 });
+  return { accion: 'eliminado', ruta: rutaConfig };
+}
+
+module.exports = {
+  upsertMcpServer,
+  removeMcpServer,
+  serializarServidor,
+  // Para tests
+  _tomlString: tomlString,
+  BEGIN_PREFIX,
+  END_PREFIX,
+};

package/scripts/lib/transformadores/base.js

@@ -8,27 +8,61 @@ class TransformadorBase {
     this.runtime = runtimeConfig;
   }
 
-  // Parsea frontmatter YAML de un archivo markdown
+  // Parsea frontmatter YAML de un archivo markdown.
+  // Maneja CRLF (Windows) y LF (Unix). Soporta valores multi-línea con `>` y `|`.
   // Retorna { frontmatter: object, cuerpo: string }
   parsearFrontmatter(contenido) {
-    const match = contenido.match(/^---\n([\s\S]*?)\n---\n([\s\S]*)$/);
+    // Normalizar a LF para que el regex funcione con archivos Windows (CRLF).
+    const norm = String(contenido).replace(/\r\n/g, '\n').replace(/\r/g, '\n');
+    const match = norm.match(/^---\n([\s\S]*?)\n---\n?([\s\S]*)$/);
     if (!match) return { frontmatter: {}, cuerpo: contenido };
 
     const yamlStr = match[1];
     const cuerpo = match[2];
     const frontmatter = {};
 
-    for (const linea of yamlStr.split('\n')) {
-      const idx = linea.indexOf(':');
-      if (idx === -1) continue;
-      const clave = linea.slice(0, idx).trim();
-      let valor = linea.slice(idx + 1).trim();
-      // Booleans
+    const lineas = yamlStr.split('\n');
+    let i = 0;
+    while (i < lineas.length) {
+      const linea = lineas[i];
+      const idxColon = linea.indexOf(':');
+      if (idxColon === -1) { i++; continue; }
+      const clave = linea.slice(0, idxColon).trim();
+      let valorRaw = linea.slice(idxColon + 1).trim();
+
+      // Multi-línea con `>` (folded) o `|` (literal):
+      // capturar líneas indentadas siguientes y colapsarlas a single-line para `>`,
+      // mantener saltos para `|`. Soporta el patrón estándar de YAML en SWL agents.
+      if (valorRaw === '>' || valorRaw === '|') {
+        const folded = valorRaw === '>';
+        const partes = [];
+        i++;
+        while (i < lineas.length) {
+          const sig = lineas[i];
+          // Línea no indentada y no vacía → fin del bloque multi-línea.
+          if (sig.length > 0 && !/^\s/.test(sig)) break;
+          // Línea vacía dentro del bloque → mantener como separador.
+          if (sig.trim().length === 0) {
+            partes.push('');
+            i++;
+            continue;
+          }
+          partes.push(sig.replace(/^\s+/, ''));
+          i++;
+        }
+        frontmatter[clave] = folded
+          ? partes.filter(p => p.length > 0).join(' ').trim()
+          : partes.join('\n').trim();
+        continue;
+      }
+
+      // Valor inline simple
+      let valor = valorRaw;
       if (valor === 'true') valor = true;
       else if (valor === 'false') valor = false;
-      // Numbers
       else if (/^\d+$/.test(valor)) valor = parseInt(valor, 10);
       frontmatter[clave] = valor;
+      i++;
     }
 
     return { frontmatter, cuerpo };