@saulwade/swl-ses 2.6.0 → 2.8.0

package/scripts/lib/toml-merge.js

@@ -1,204 +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,
-};
+'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/codex.js

@@ -24,6 +24,8 @@ const path = require('path');
 const TransformadorBase = require('./base');
 const { upsertMcpServer } = require('../toml-merge');
 
+const CONTROLES_TOML = /[\u0000-\u0008\u000B\u000C\u000E-\u001F\u007F]/u;
+
 class TransformadorCodex extends TransformadorBase {
   /**
    * Sub-fase 11 v1.5.0: Codex CLI SÍ acepta agentes individuales en formato
@@ -34,7 +36,6 @@ class TransformadorCodex extends TransformadorBase {
    *   - name (required)
    *   - description (required)
    *   - developer_instructions (required) — multi-line literal del cuerpo Markdown
-   *   - model (opcional) — heredado del frontmatter SWL si existe
    *   - sandbox_mode (opcional) — mapeado desde permisosEscritura=false → "read-only"
    *
    * El cuerpo del .md SWL completo (después del frontmatter) se inserta como
@@ -64,7 +65,6 @@ class TransformadorCodex extends TransformadorBase {
       name,
       description,
       developer_instructions: developerInstructions,
-      model: frontmatter.model || null,
       sandbox_mode: (frontmatter.permisosEscritura === false) ? 'read-only' : null,
     });
 
@@ -88,7 +88,6 @@ class TransformadorCodex extends TransformadorBase {
     lineas.push('');
     lineas.push(`name = ${this._tomlBasicString(campos.name)}`);
     lineas.push(`description = ${this._tomlBasicString(campos.description)}`);
-    if (campos.model) lineas.push(`model = ${this._tomlBasicString(campos.model)}`);
     if (campos.sandbox_mode) lineas.push(`sandbox_mode = ${this._tomlBasicString(campos.sandbox_mode)}`);
     lineas.push('');
     lineas.push(`developer_instructions = ${this._tomlMultiline(campos.developer_instructions)}`);
@@ -96,7 +95,9 @@ class TransformadorCodex extends TransformadorBase {
   }
 
   _tomlBasicString(s) {
-    const escapado = String(s)
+    const texto = String(s);
+    if (CONTROLES_TOML.test(texto)) throw new TypeError('Control TOML inválido');
+    const escapado = texto
       .replace(/\\/g, '\\\\')
       .replace(/"/g, '\\"')
       .replace(/\n/g, '\\n')
@@ -107,6 +108,7 @@ class TransformadorCodex extends TransformadorBase {
 
   _tomlMultiline(s) {
     const contenido = String(s);
+    if (CONTROLES_TOML.test(contenido)) throw new TypeError('Control TOML inválido');
     if (!contenido.includes("'''")) {
       // Literal multi-line: preserva contenido sin escapes.
       return `'''\n${contenido}\n'''`;
@@ -122,11 +124,13 @@ class TransformadorCodex extends TransformadorBase {
     // `.agents/skills/` como ubicaciones válidas — NO `~/.codex/skills/`.
     // El path `~/.codex/skills/` que apareció en Sub-fase 8 era de una sample
     // sub-doc del repo (no canónica). Ahora apunta al path correcto vía dirBase.
-    const dirBaseGlobal = path.join(require('os').homedir(), '.agents');
+    const dirBase = metadatos.esGlobal
+      ? path.join(require('os').homedir(), '.agents')
+      : '.agents';
     return {
       tipo: 'directorio',
       rutaRelativa: `skills/${metadatos.nombreDirectorio}`,
-      dirBase: dirBaseGlobal,
+      dirBase,
     };
   }
 
@@ -178,9 +182,12 @@ class TransformadorCodex extends TransformadorBase {
 
     for (const [hookFile, def] of Object.entries(hooksConfig)) {
       if (hookFilesFiltro && !hookFilesFiltro.has(hookFile)) continue;
-      if (!def || !Array.isArray(def.events)) continue;
+      if (!def) continue;
+      const events = Array.isArray(def.events)
+        ? def.events
+        : (typeof def.event === 'string' ? [def.event] : []);
 
-      for (const event of def.events) {
+      for (const event of events) {
         if (!EVENTOS_VALIDOS.has(event)) {
           descartados.push({ hookFile, event, motivo: 'evento no soportado por Codex' });
           continue;

package/scripts/lib/transformadores/gemini.js

@@ -2,6 +2,13 @@
 
 const TransformadorBase = require('./base');
 
+const EVENTOS_GEMINI = Object.freeze({
+  PreToolUse: 'BeforeTool',
+  PostToolUse: 'AfterTool',
+  SessionStart: 'SessionStart',
+});
+const CONTROLES_TOML = /[\u0000-\u0008\u000B\u000C\u000E-\u001F\u007F]/u;
+
 class TransformadorGemini extends TransformadorBase {
   transformarAgente(contenidoMd, metadatos) {
     // Gemini soporta frontmatter similar a Claude
@@ -12,7 +19,6 @@ class TransformadorGemini extends TransformadorBase {
     if (frontmatter.name) fmGemini.name = frontmatter.name;
     if (frontmatter.description) fmGemini.description = frontmatter.description;
     if (frontmatter.tools) fmGemini.tools = frontmatter.tools;
-    if (frontmatter.model) fmGemini.model = frontmatter.model;
 
     return {
       contenido: this.construirMarkdown(fmGemini, cuerpo),
@@ -35,12 +41,36 @@ class TransformadorGemini extends TransformadorBase {
   }
 
   transformarComando(contenidoMd, metadatos) {
+    const { frontmatter, cuerpo } = this.parsearFrontmatter(contenidoMd);
+    const nombreArchivo = metadatos.nombreArchivo.replace(/\.md$/iu, '.toml');
+    const descripcion = this._tomlString(frontmatter.description || '');
+    const prompt = this._tomlMultiline(String(cuerpo || '').trim());
     return {
-      contenido: contenidoMd,
-      rutaRelativa: `commands/swl/${metadatos.nombreArchivo}`,
+      contenido: `description = ${descripcion}\nprompt = ${prompt}\n`,
+      rutaRelativa: `commands/swl/${nombreArchivo}`,
     };
   }
 
+  _tomlString(valor) {
+    const texto = String(valor);
+    if (CONTROLES_TOML.test(texto)) throw new TypeError('Control TOML inválido');
+    const escapado = texto
+      .replace(/\\/gu, '\\\\')
+      .replace(/"/gu, '\\"')
+      .replace(/\n/gu, '\\n')
+      .replace(/\r/gu, '\\r')
+      .replace(/\t/gu, '\\t');
+    return `"${escapado}"`;
+  }
+
+  _tomlMultiline(valor) {
+    const texto = String(valor);
+    if (CONTROLES_TOML.test(texto)) throw new TypeError('Control TOML inválido');
+    const escapado = texto.replace(/\r\n/gu, '\n')
+      .replace(/\\/gu, '\\\\').replace(/"""/gu, '\\"""');
+    return `"""\n${escapado}\n"""`;
+  }
+
   generarArchivoInstrucciones(contexto) {
     const lineas = [];
     lineas.push('# GEMINI.md — Sistema SWL');
@@ -86,8 +116,52 @@ class TransformadorGemini extends TransformadorBase {
   }
 
   transformarHooks(hooksConfig, opciones) {
-    // Gemini usa settings.json con formato similar a Claude
-    return { formato: 'gemini-settings', config: hooksConfig };
+    if (!hooksConfig || typeof hooksConfig !== 'object') return null;
+    const opts = opciones || {};
+    const filtro = Array.isArray(opts.hookFiles) ? new Set(opts.hookFiles) : null;
+    const rutaHooksDir = opts.rutaHooksDir || 'hooks';
+    const grupos = {};
+    const descartados = [];
+
+    for (const [hookFile, definicion] of Object.entries(hooksConfig)) {
+      const eventos = this._eventosHook(definicion);
+      if ((filtro && !filtro.has(hookFile)) || eventos.length === 0) continue;
+      for (const evento of eventos) {
+        const eventoGemini = EVENTOS_GEMINI[evento];
+        if (!eventoGemini) {
+          descartados.push({ hookFile, event: evento, motivo: 'evento no soportado por Gemini' });
+          continue;
+        }
+        if (!grupos[eventoGemini]) grupos[eventoGemini] = [];
+        const entrada = this._crearHookGemini(hookFile, definicion, rutaHooksDir);
+        grupos[eventoGemini].push(entrada);
+      }
+    }
+
+    return {
+      formato: 'gemini-hooks-json',
+      destino: 'hooks/hooks.json',
+      config: { hooks: grupos },
+      descartados,
+    };
+  }
+
+  _eventosHook(definicion) {
+    if (Array.isArray(definicion?.events)) return definicion.events;
+    return typeof definicion?.event === 'string' ? [definicion.event] : [];
+  }
+
+  _crearHookGemini(hookFile, definicion, rutaHooksDir) {
+    const hook = {
+      name: hookFile.replace(/\.js$/iu, ''),
+      type: 'command',
+      command: `node ${rutaHooksDir}/${hookFile}`,
+      ...(definicion.timeout ? { timeout: definicion.timeout } : {}),
+    };
+    return {
+      ...(definicion.matcher ? { matcher: definicion.matcher } : {}),
+      hooks: [hook],
+    };
   }
 }