@hoilab/ada-cli 0.84.21 → 0.85.0

package/dist/cli/review-command.js

@@ -0,0 +1,168 @@
+/**
+ * `ada-cli review` — revisa un diff y ancla cada hallazgo al código.
+ *
+ * La pantalla de bienvenida ya ofrecía «Revisa mis cambios sin commitear», pero
+ * eso era una frase que iba al modelo sin nada detrás: sin contrato de salida,
+ * sin criba y sin forma de sacar los hallazgos del terminal. Aquí hay las tres
+ * cosas.
+ *
+ * Lo que hace que sirva es el ANCLAJE: un hallazgo cuya evidencia no aparece en
+ * el diff se descarta. Un revisor que inventa una línea plausible es peor que
+ * no tener revisor, porque cuesta tiempo comprobar que no existe.
+ *
+ * Y lo que lo hace vendible es `--sarif`: sin él los hallazgos se quedan en la
+ * terminal de quien lanzó el comando y no entran en el proceso de seguridad de
+ * la empresa.
+ */
+import { execFileSync } from "node:child_process";
+import { writeFileSync } from "node:fs";
+import { resolve } from "node:path";
+import { puntuacionDe } from "../core/revision/hallazgo.js";
+import { cribar, extraerCandidatos, PROMPT_REVISION } from "../core/revision/revisor.js";
+import { aSarif } from "../core/revision/sarif.js";
+const AYUDA = `Uso:
+  ada-cli review [--staged] [--base <ref>] [--model <p/m>] [--sarif <f>] [--json]
+
+  --staged          Revisa lo que está en el índice (git diff --cached)
+  --base <ref>      Revisa contra una referencia (p. ej. main)
+  --sarif <fich>    Escribe los hallazgos en SARIF 2.1.0, para GitHub Code
+                    Scanning o Azure DevOps
+  --model <p/m>     Revisa con un modelo concreto (proveedor/modelo)
+  --json            Escribe los hallazgos en JSON por stdout
+
+Sin opciones revisa los cambios sin commitear (git diff).
+
+Cada hallazgo lleva la línea exacta que lo justifica, y los que no se pueden
+anclar en el diff se descartan: un revisor que inventa cuesta más tiempo del
+que ahorra.
+
+Sale con código 1 si hay algún hallazgo crítico o alto, para que lo pueda
+mirar un CI.`;
+/** El diff que toca revisar, según las opciones. */
+function obtenerDiff(args, cwd) {
+    const base = args.indexOf("--base");
+    const argumentos = base >= 0 && args[base + 1]
+        ? ["diff", `${args[base + 1]}...HEAD`]
+        : args.includes("--staged")
+            ? ["diff", "--cached"]
+            : ["diff"];
+    // `-U5` da contexto suficiente para que el modelo entienda el cambio sin
+    // mandarle el fichero entero, que es lo que dispara el coste.
+    return execFileSync("git", [...argumentos, "-U5"], { cwd, encoding: "utf8", maxBuffer: 32 * 1024 * 1024 });
+}
+const COLOR = {
+    critica: "\u001b[31m",
+    alta: "\u001b[31m",
+    media: "\u001b[33m",
+    baja: "\u001b[36m",
+    informativa: "\u001b[90m",
+};
+const FIN = "\u001b[0m";
+function imprimir(hallazgos) {
+    for (const h of hallazgos) {
+        const color = COLOR[h.severidad] ?? "";
+        console.log(`${color}${h.severidad.toUpperCase()}${FIN}  ${h.titulo}`);
+        console.log(`  ${h.ruta}:${h.lineaInicio}${h.lineaFin && h.lineaFin !== h.lineaInicio ? `-${h.lineaFin}` : ""}`);
+        console.log(`  ${h.resumen}`);
+        console.log(`  │ ${h.evidencia.split("\n").join("\n  │ ")}`);
+        if (h.remedio)
+            console.log(`  → ${h.remedio}`);
+        console.log("");
+    }
+}
+/** Devuelve true si consumió los argumentos. */
+export async function runReviewCommand(args, deps) {
+    if (args[0] !== "review")
+        return false;
+    if (args.includes("--help") || args.includes("-h")) {
+        console.log(AYUDA);
+        return true;
+    }
+    const cwd = deps.cwd ?? process.cwd();
+    let diff;
+    try {
+        diff = obtenerDiff(args, cwd);
+    }
+    catch (error) {
+        console.error(`No se pudo obtener el diff: ${error instanceof Error ? error.message : String(error)}`);
+        process.exitCode = 1;
+        return true;
+    }
+    if (diff.trim() === "") {
+        // Distinto de "revisado y limpio". Si no hay diff, no se ha revisado
+        // nada, y decir "sin hallazgos" sería tranquilizar sobre algo que no ha
+        // pasado.
+        console.log("No hay cambios que revisar.");
+        return true;
+    }
+    let salida;
+    try {
+        salida = await deps.revisar(PROMPT_REVISION, diff);
+    }
+    catch (error) {
+        // Un fallo del modelo tiene que verse como fallo, con código 1, y no como
+        // una revisión limpia. Se midió: sobre un fallo sembrado, cuatro de cinco
+        // ejecuciones decían "no ha encontrado nada" cuando lo que pasaba era un
+        // 429 del proveedor. Y una traza de Node tampoco sirve: quien lanza esto
+        // desde un CI necesita el motivo en una línea.
+        console.error(`La revisión no se pudo completar.\n${error instanceof Error ? error.message : String(error)}`);
+        process.exitCode = 1;
+        return true;
+    }
+    const extraccion = extraerCandidatos(salida);
+    if (!extraccion.ok) {
+        // Una respuesta ilegible NO es un diff limpio. La guarda de arriba solo
+        // mira si el proveedor falló o cortó; un modelo que contesta prosa
+        // ("No puedo ayudarte con eso.") o un JSON roto pasaba entera y salía
+        // como "El revisor no ha encontrado nada en este diff." con código 0.
+        // Se midió: las dos salidas eran idénticas byte a byte.
+        console.error(`La revisión no se pudo completar.\n${extraccion.motivo}`);
+        console.error("NO se ha revisado el diff. Esto no significa que esté limpio.");
+        process.exitCode = 1;
+        return true;
+    }
+    const candidatos = extraccion.candidatos;
+    const { aceptados, descartados } = cribar(candidatos, diff);
+    if (args.includes("--json")) {
+        console.log(JSON.stringify(aceptados, null, 2));
+    }
+    else if (aceptados.length > 0) {
+        imprimir(aceptados);
+    }
+    else if (candidatos.length === 0) {
+        // El revisor miró y no vio nada. Distinto de lo de abajo, y distinto de
+        // "no había diff": las tres se leen igual si se juntan en un mensaje.
+        console.log("El revisor no ha encontrado nada en este diff.");
+    }
+    else {
+        // «Descartados», no «no se pudieron anclar»: el anclaje es solo uno de los
+        // motivos. Un hallazgo sin `lineaInicio` se tiraba con este mismo mensaje,
+        // que culpaba al anclaje de un fallo de forma y mandaba a mirar el sitio
+        // equivocado. Comprobado ejecutando el comando con un hallazgo sin línea.
+        console.log(`Ninguno de los ${candidatos.length} hallazgos pasó la criba; todos descartados.`);
+    }
+    // Los descartes se cuentan Y SE MOTIVAN, no se esconden: sin el motivo no hay
+    // forma de saber si el revisor está afinado, si el modelo inventa evidencias
+    // o si está devolviendo hallazgos mal formados. Agrupados por motivo para que
+    // diez descartes no tapen la pantalla.
+    if (descartados.length > 0) {
+        const porMotivo = new Map();
+        for (const d of descartados)
+            porMotivo.set(d.motivo, (porMotivo.get(d.motivo) ?? 0) + 1);
+        console.error(`${descartados.length} hallazgo(s) descartado(s):`);
+        for (const [motivo, cuantos] of porMotivo)
+            console.error(`  ${cuantos}x ${motivo}`);
+    }
+    const indice = args.indexOf("--sarif");
+    if (indice >= 0 && args[indice + 1]) {
+        const destino = resolve(cwd, args[indice + 1]);
+        writeFileSync(destino, `${JSON.stringify(aSarif(aceptados, { raizRepo: cwd }), null, 2)}\n`, "utf8");
+        console.log(`SARIF escrito en ${destino}`);
+    }
+    // Código 1 solo con lo grave: un CI que falle por un hallazgo informativo se
+    // desactiva la primera semana.
+    if (aceptados.some((h) => puntuacionDe(h.severidad) >= puntuacionDe("alta")))
+        process.exitCode = 1;
+    return true;
+}
+//# sourceMappingURL=review-command.js.map

package/dist/cli/review-command.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"review-command.js","sourceRoot":"","sources":["../../src/cli/review-command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;;GAeG;AAEH,OAAO,EAAE,YAAY,EAAE,MAAM,oBAAoB,CAAC;AAClD,OAAO,EAAE,aAAa,EAAE,MAAM,SAAS,CAAC;AACxC,OAAO,EAAE,OAAO,EAAE,MAAM,WAAW,CAAC;AACpC,OAAO,EAAiB,YAAY,EAAE,MAAM,8BAA8B,CAAC;AAC3E,OAAO,EAAE,MAAM,EAAE,iBAAiB,EAAE,eAAe,EAAE,MAAM,6BAA6B,CAAC;AACzF,OAAO,EAAE,MAAM,EAAE,MAAM,2BAA2B,CAAC;AAEnD,MAAM,KAAK,GAAG;;;;;;;;;;;;;;;;;aAiBD,CAAC;AAEd,qDAAoD;AACpD,SAAS,WAAW,CAAC,IAAc,EAAE,GAAW,EAAU;IACzD,MAAM,IAAI,GAAG,IAAI,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IACpC,MAAM,UAAU,GACf,IAAI,IAAI,CAAC,IAAI,IAAI,CAAC,IAAI,GAAG,CAAC,CAAC;QAC1B,CAAC,CAAC,CAAC,MAAM,EAAE,GAAG,IAAI,CAAC,IAAI,GAAG,CAAC,CAAC,SAAS,CAAC;QACtC,CAAC,CAAC,IAAI,CAAC,QAAQ,CAAC,UAAU,CAAC;YAC1B,CAAC,CAAC,CAAC,MAAM,EAAE,UAAU,CAAC;YACtB,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC;IACd,yEAAyE;IACzE,8DAA8D;IAC9D,OAAO,YAAY,CAAC,KAAK,EAAE,CAAC,GAAG,UAAU,EAAE,KAAK,CAAC,EAAE,EAAE,GAAG,EAAE,QAAQ,EAAE,MAAM,EAAE,SAAS,EAAE,EAAE,GAAG,IAAI,GAAG,IAAI,EAAE,CAAC,CAAC;AAAA,CAC3G;AAED,MAAM,KAAK,GAA2B;IACrC,OAAO,EAAE,YAAY;IACrB,IAAI,EAAE,YAAY;IAClB,KAAK,EAAE,YAAY;IACnB,IAAI,EAAE,YAAY;IAClB,WAAW,EAAE,YAAY;CACzB,CAAC;AACF,MAAM,GAAG,GAAG,WAAW,CAAC;AAExB,SAAS,QAAQ,CAAC,SAAqB,EAAQ;IAC9C,KAAK,MAAM,CAAC,IAAI,SAAS,EAAE,CAAC;QAC3B,MAAM,KAAK,GAAG,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC;QACvC,OAAO,CAAC,GAAG,CAAC,GAAG,KAAK,GAAG,CAAC,CAAC,SAAS,CAAC,WAAW,EAAE,GAAG,GAAG,KAAK,CAAC,CAAC,MAAM,EAAE,CAAC,CAAC;QACvE,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,IAAI,IAAI,CAAC,CAAC,WAAW,GAAG,CAAC,CAAC,QAAQ,IAAI,CAAC,CAAC,QAAQ,KAAK,CAAC,CAAC,WAAW,CAAC,CAAC,CAAC,IAAI,CAAC,CAAC,QAAQ,EAAE,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,CAAC;QACjH,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC;QAC9B,OAAO,CAAC,GAAG,CAAC,SAAO,CAAC,CAAC,SAAS,CAAC,KAAK,CAAC,IAAI,CAAC,CAAC,IAAI,CAAC,UAAQ,CAAC,EAAE,CAAC,CAAC;QAC7D,IAAI,CAAC,CAAC,OAAO;YAAE,OAAO,CAAC,GAAG,CAAC,SAAO,CAAC,CAAC,OAAO,EAAE,CAAC,CAAC;QAC/C,OAAO,CAAC,GAAG,CAAC,EAAE,CAAC,CAAC;IACjB,CAAC;AAAA,CACD;AAQD,iDAAgD;AAChD,MAAM,CAAC,KAAK,UAAU,gBAAgB,CAAC,IAAc,EAAE,IAA0B,EAAoB;IACpG,IAAI,IAAI,CAAC,CAAC,CAAC,KAAK,QAAQ;QAAE,OAAO,KAAK,CAAC;IACvC,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QACpD,OAAO,CAAC,GAAG,CAAC,KAAK,CAAC,CAAC;QACnB,OAAO,IAAI,CAAC;IACb,CAAC;IAED,MAAM,GAAG,GAAG,IAAI,CAAC,GAAG,IAAI,OAAO,CAAC,GAAG,EAAE,CAAC;IACtC,IAAI,IAAY,CAAC;IACjB,IAAI,CAAC;QACJ,IAAI,GAAG,WAAW,CAAC,IAAI,EAAE,GAAG,CAAC,CAAC;IAC/B,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QAChB,OAAO,CAAC,KAAK,CAAC,+BAA+B,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QACvG,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;QACrB,OAAO,IAAI,CAAC;IACb,CAAC;IAED,IAAI,IAAI,CAAC,IAAI,EAAE,KAAK,EAAE,EAAE,CAAC;QACxB,qEAAqE;QACrE,yEAAwE;QACxE,UAAU;QACV,OAAO,CAAC,GAAG,CAAC,6BAA6B,CAAC,CAAC;QAC3C,OAAO,IAAI,CAAC;IACb,CAAC;IAED,IAAI,MAAc,CAAC;IACnB,IAAI,CAAC;QACJ,MAAM,GAAG,MAAM,IAAI,CAAC,OAAO,CAAC,eAAe,EAAE,IAAI,CAAC,CAAC;IACpD,CAAC;IAAC,OAAO,KAAK,EAAE,CAAC;QAChB,2EAA0E;QAC1E,4EAA0E;QAC1E,0EAAyE;QACzE,yEAAyE;QACzE,gDAA+C;QAC/C,OAAO,CAAC,KAAK,CAAC,uCAAsC,KAAK,YAAY,KAAK,CAAC,CAAC,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,CAAC,MAAM,CAAC,KAAK,CAAC,EAAE,CAAC,CAAC;QAC9G,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;QACrB,OAAO,IAAI,CAAC;IACb,CAAC;IACD,MAAM,UAAU,GAAG,iBAAiB,CAAC,MAAM,CAAC,CAAC;IAC7C,IAAI,CAAC,UAAU,CAAC,EAAE,EAAE,CAAC;QACpB,wEAAwE;QACxE,qEAAmE;QACnE,uEAAsE;QACtE,uEAAsE;QACtE,0DAAwD;QACxD,OAAO,CAAC,KAAK,CAAC,uCAAsC,UAAU,CAAC,MAAM,EAAE,CAAC,CAAC;QACzE,OAAO,CAAC,KAAK,CAAC,gEAA+D,CAAC,CAAC;QAC/E,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;QACrB,OAAO,IAAI,CAAC;IACb,CAAC;IACD,MAAM,UAAU,GAAG,UAAU,CAAC,UAAU,CAAC;IACzC,MAAM,EAAE,SAAS,EAAE,WAAW,EAAE,GAAG,MAAM,CAAC,UAAU,EAAE,IAAI,CAAC,CAAC;IAE5D,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,EAAE,CAAC;QAC7B,OAAO,CAAC,GAAG,CAAC,IAAI,CAAC,SAAS,CAAC,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC,CAAC,CAAC;IACjD,CAAC;SAAM,IAAI,SAAS,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QACjC,QAAQ,CAAC,SAAS,CAAC,CAAC;IACrB,CAAC;SAAM,IAAI,UAAU,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;QACpC,yEAAwE;QACxE,uEAAsE;QACtE,OAAO,CAAC,GAAG,CAAC,gDAAgD,CAAC,CAAC;IAC/D,CAAC;SAAM,CAAC;QACP,+EAA2E;QAC3E,2EAA2E;QAC3E,yEAAyE;QACzE,2EAA0E;QAC1E,OAAO,CAAC,GAAG,CAAC,kBAAkB,UAAU,CAAC,MAAM,+CAA8C,CAAC,CAAC;IAChG,CAAC;IAED,8EAA8E;IAC9E,8EAA6E;IAC7E,+EAA8E;IAC9E,uCAAuC;IACvC,IAAI,WAAW,CAAC,MAAM,GAAG,CAAC,EAAE,CAAC;QAC5B,MAAM,SAAS,GAAG,IAAI,GAAG,EAAkB,CAAC;QAC5C,KAAK,MAAM,CAAC,IAAI,WAAW;YAAE,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,MAAM,EAAE,CAAC,SAAS,CAAC,GAAG,CAAC,CAAC,CAAC,MAAM,CAAC,IAAI,CAAC,CAAC,GAAG,CAAC,CAAC,CAAC;QACzF,OAAO,CAAC,KAAK,CAAC,GAAG,WAAW,CAAC,MAAM,6BAA6B,CAAC,CAAC;QAClE,KAAK,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,IAAI,SAAS;YAAE,OAAO,CAAC,KAAK,CAAC,KAAK,OAAO,KAAK,MAAM,EAAE,CAAC,CAAC;IACrF,CAAC;IAED,MAAM,MAAM,GAAG,IAAI,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;IACvC,IAAI,MAAM,IAAI,CAAC,IAAI,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,EAAE,CAAC;QACrC,MAAM,OAAO,GAAG,OAAO,CAAC,GAAG,EAAE,IAAI,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC;QAC/C,aAAa,CAAC,OAAO,EAAE,GAAG,IAAI,CAAC,SAAS,CAAC,MAAM,CAAC,SAAS,EAAE,EAAE,QAAQ,EAAE,GAAG,EAAE,CAAC,EAAE,IAAI,EAAE,CAAC,CAAC,IAAI,EAAE,MAAM,CAAC,CAAC;QACrG,OAAO,CAAC,GAAG,CAAC,oBAAoB,OAAO,EAAE,CAAC,CAAC;IAC5C,CAAC;IAED,8EAA6E;IAC7E,+BAA+B;IAC/B,IAAI,SAAS,CAAC,IAAI,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,YAAY,CAAC,CAAC,CAAC,SAAS,CAAC,IAAI,YAAY,CAAC,MAAM,CAAC,CAAC;QAAE,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;IACnG,OAAO,IAAI,CAAC;AAAA,CACZ","sourcesContent":["/**\n * `ada-cli review` — revisa un diff y ancla cada hallazgo al código.\n *\n * La pantalla de bienvenida ya ofrecía «Revisa mis cambios sin commitear», pero\n * eso era una frase que iba al modelo sin nada detrás: sin contrato de salida,\n * sin criba y sin forma de sacar los hallazgos del terminal. Aquí hay las tres\n * cosas.\n *\n * Lo que hace que sirva es el ANCLAJE: un hallazgo cuya evidencia no aparece en\n * el diff se descarta. Un revisor que inventa una línea plausible es peor que\n * no tener revisor, porque cuesta tiempo comprobar que no existe.\n *\n * Y lo que lo hace vendible es `--sarif`: sin él los hallazgos se quedan en la\n * terminal de quien lanzó el comando y no entran en el proceso de seguridad de\n * la empresa.\n */\n\nimport { execFileSync } from \"node:child_process\";\nimport { writeFileSync } from \"node:fs\";\nimport { resolve } from \"node:path\";\nimport { type Hallazgo, puntuacionDe } from \"../core/revision/hallazgo.ts\";\nimport { cribar, extraerCandidatos, PROMPT_REVISION } from \"../core/revision/revisor.ts\";\nimport { aSarif } from \"../core/revision/sarif.ts\";\n\nconst AYUDA = `Uso:\n  ada-cli review [--staged] [--base <ref>] [--model <p/m>] [--sarif <f>] [--json]\n\n  --staged          Revisa lo que está en el índice (git diff --cached)\n  --base <ref>      Revisa contra una referencia (p. ej. main)\n  --sarif <fich>    Escribe los hallazgos en SARIF 2.1.0, para GitHub Code\n                    Scanning o Azure DevOps\n  --model <p/m>     Revisa con un modelo concreto (proveedor/modelo)\n  --json            Escribe los hallazgos en JSON por stdout\n\nSin opciones revisa los cambios sin commitear (git diff).\n\nCada hallazgo lleva la línea exacta que lo justifica, y los que no se pueden\nanclar en el diff se descartan: un revisor que inventa cuesta más tiempo del\nque ahorra.\n\nSale con código 1 si hay algún hallazgo crítico o alto, para que lo pueda\nmirar un CI.`;\n\n/** El diff que toca revisar, según las opciones. */\nfunction obtenerDiff(args: string[], cwd: string): string {\n\tconst base = args.indexOf(\"--base\");\n\tconst argumentos =\n\t\tbase >= 0 && args[base + 1]\n\t\t\t? [\"diff\", `${args[base + 1]}...HEAD`]\n\t\t\t: args.includes(\"--staged\")\n\t\t\t\t? [\"diff\", \"--cached\"]\n\t\t\t\t: [\"diff\"];\n\t// `-U5` da contexto suficiente para que el modelo entienda el cambio sin\n\t// mandarle el fichero entero, que es lo que dispara el coste.\n\treturn execFileSync(\"git\", [...argumentos, \"-U5\"], { cwd, encoding: \"utf8\", maxBuffer: 32 * 1024 * 1024 });\n}\n\nconst COLOR: Record<string, string> = {\n\tcritica: \"\\u001b[31m\",\n\talta: \"\\u001b[31m\",\n\tmedia: \"\\u001b[33m\",\n\tbaja: \"\\u001b[36m\",\n\tinformativa: \"\\u001b[90m\",\n};\nconst FIN = \"\\u001b[0m\";\n\nfunction imprimir(hallazgos: Hallazgo[]): void {\n\tfor (const h of hallazgos) {\n\t\tconst color = COLOR[h.severidad] ?? \"\";\n\t\tconsole.log(`${color}${h.severidad.toUpperCase()}${FIN}  ${h.titulo}`);\n\t\tconsole.log(`  ${h.ruta}:${h.lineaInicio}${h.lineaFin && h.lineaFin !== h.lineaInicio ? `-${h.lineaFin}` : \"\"}`);\n\t\tconsole.log(`  ${h.resumen}`);\n\t\tconsole.log(`  │ ${h.evidencia.split(\"\\n\").join(\"\\n  │ \")}`);\n\t\tif (h.remedio) console.log(`  → ${h.remedio}`);\n\t\tconsole.log(\"\");\n\t}\n}\n\nexport interface DependenciasRevision {\n\t/** Pide la revisión al modelo. Se inyecta para poder probar sin red. */\n\trevisar: (sistema: string, diff: string) => Promise<string>;\n\tcwd?: string;\n}\n\n/** Devuelve true si consumió los argumentos. */\nexport async function runReviewCommand(args: string[], deps: DependenciasRevision): Promise<boolean> {\n\tif (args[0] !== \"review\") return false;\n\tif (args.includes(\"--help\") || args.includes(\"-h\")) {\n\t\tconsole.log(AYUDA);\n\t\treturn true;\n\t}\n\n\tconst cwd = deps.cwd ?? process.cwd();\n\tlet diff: string;\n\ttry {\n\t\tdiff = obtenerDiff(args, cwd);\n\t} catch (error) {\n\t\tconsole.error(`No se pudo obtener el diff: ${error instanceof Error ? error.message : String(error)}`);\n\t\tprocess.exitCode = 1;\n\t\treturn true;\n\t}\n\n\tif (diff.trim() === \"\") {\n\t\t// Distinto de \"revisado y limpio\". Si no hay diff, no se ha revisado\n\t\t// nada, y decir \"sin hallazgos\" sería tranquilizar sobre algo que no ha\n\t\t// pasado.\n\t\tconsole.log(\"No hay cambios que revisar.\");\n\t\treturn true;\n\t}\n\n\tlet salida: string;\n\ttry {\n\t\tsalida = await deps.revisar(PROMPT_REVISION, diff);\n\t} catch (error) {\n\t\t// Un fallo del modelo tiene que verse como fallo, con código 1, y no como\n\t\t// una revisión limpia. Se midió: sobre un fallo sembrado, cuatro de cinco\n\t\t// ejecuciones decían \"no ha encontrado nada\" cuando lo que pasaba era un\n\t\t// 429 del proveedor. Y una traza de Node tampoco sirve: quien lanza esto\n\t\t// desde un CI necesita el motivo en una línea.\n\t\tconsole.error(`La revisión no se pudo completar.\\n${error instanceof Error ? error.message : String(error)}`);\n\t\tprocess.exitCode = 1;\n\t\treturn true;\n\t}\n\tconst extraccion = extraerCandidatos(salida);\n\tif (!extraccion.ok) {\n\t\t// Una respuesta ilegible NO es un diff limpio. La guarda de arriba solo\n\t\t// mira si el proveedor falló o cortó; un modelo que contesta prosa\n\t\t// (\"No puedo ayudarte con eso.\") o un JSON roto pasaba entera y salía\n\t\t// como \"El revisor no ha encontrado nada en este diff.\" con código 0.\n\t\t// Se midió: las dos salidas eran idénticas byte a byte.\n\t\tconsole.error(`La revisión no se pudo completar.\\n${extraccion.motivo}`);\n\t\tconsole.error(\"NO se ha revisado el diff. Esto no significa que esté limpio.\");\n\t\tprocess.exitCode = 1;\n\t\treturn true;\n\t}\n\tconst candidatos = extraccion.candidatos;\n\tconst { aceptados, descartados } = cribar(candidatos, diff);\n\n\tif (args.includes(\"--json\")) {\n\t\tconsole.log(JSON.stringify(aceptados, null, 2));\n\t} else if (aceptados.length > 0) {\n\t\timprimir(aceptados);\n\t} else if (candidatos.length === 0) {\n\t\t// El revisor miró y no vio nada. Distinto de lo de abajo, y distinto de\n\t\t// \"no había diff\": las tres se leen igual si se juntan en un mensaje.\n\t\tconsole.log(\"El revisor no ha encontrado nada en este diff.\");\n\t} else {\n\t\t// «Descartados», no «no se pudieron anclar»: el anclaje es solo uno de los\n\t\t// motivos. Un hallazgo sin `lineaInicio` se tiraba con este mismo mensaje,\n\t\t// que culpaba al anclaje de un fallo de forma y mandaba a mirar el sitio\n\t\t// equivocado. Comprobado ejecutando el comando con un hallazgo sin línea.\n\t\tconsole.log(`Ninguno de los ${candidatos.length} hallazgos pasó la criba; todos descartados.`);\n\t}\n\n\t// Los descartes se cuentan Y SE MOTIVAN, no se esconden: sin el motivo no hay\n\t// forma de saber si el revisor está afinado, si el modelo inventa evidencias\n\t// o si está devolviendo hallazgos mal formados. Agrupados por motivo para que\n\t// diez descartes no tapen la pantalla.\n\tif (descartados.length > 0) {\n\t\tconst porMotivo = new Map<string, number>();\n\t\tfor (const d of descartados) porMotivo.set(d.motivo, (porMotivo.get(d.motivo) ?? 0) + 1);\n\t\tconsole.error(`${descartados.length} hallazgo(s) descartado(s):`);\n\t\tfor (const [motivo, cuantos] of porMotivo) console.error(`  ${cuantos}x ${motivo}`);\n\t}\n\n\tconst indice = args.indexOf(\"--sarif\");\n\tif (indice >= 0 && args[indice + 1]) {\n\t\tconst destino = resolve(cwd, args[indice + 1]);\n\t\twriteFileSync(destino, `${JSON.stringify(aSarif(aceptados, { raizRepo: cwd }), null, 2)}\\n`, \"utf8\");\n\t\tconsole.log(`SARIF escrito en ${destino}`);\n\t}\n\n\t// Código 1 solo con lo grave: un CI que falle por un hallazgo informativo se\n\t// desactiva la primera semana.\n\tif (aceptados.some((h) => puntuacionDe(h.severidad) >= puntuacionDe(\"alta\"))) process.exitCode = 1;\n\treturn true;\n}\n"]}

package/dist/cli/schedule-command.d.ts

@@ -0,0 +1,17 @@
+/**
+ * `ada-cli schedule` — trabajo recurrente sobre una sesión residente.
+ *
+ * El módulo `core/scheduler` existía completo y testeado, pero NADIE lo
+ * importaba: era código muerto mientras la tabla comparativa lo daba por
+ * capacidad disponible. Esto es su punto de entrada.
+ *
+ * Vive en el CLI y no como comando slash porque programar trabajo recurrente
+ * sin un proceso que siga vivo no significa nada: el reloj corre dentro de
+ * `ada-cli serve`, y sin servidor un job no se dispara. De paso, este fichero
+ * es nuestro y no cuesta presupuesto de sync.
+ */
+/** Convierte `30m`, `2h`, `1d` a milisegundos. Devuelve undefined si no encaja. */
+export declare function parseDuration(text: string): number | undefined;
+export declare function isScheduleCommand(args: string[]): boolean;
+export declare function runScheduleCommand(args: string[]): Promise<boolean>;
+//# sourceMappingURL=schedule-command.d.ts.map

package/dist/cli/schedule-command.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"schedule-command.d.ts","sourceRoot":"","sources":["../../src/cli/schedule-command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAIH,mFAAmF;AACnF,wBAAgB,aAAa,CAAC,IAAI,EAAE,MAAM,GAAG,MAAM,GAAG,SAAS,CAM9D;AAED,wBAAgB,iBAAiB,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAEzD;AAqBD,wBAAsB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,OAAO,CAAC,CA8EzE","sourcesContent":["/**\n * `ada-cli schedule` — trabajo recurrente sobre una sesión residente.\n *\n * El módulo `core/scheduler` existía completo y testeado, pero NADIE lo\n * importaba: era código muerto mientras la tabla comparativa lo daba por\n * capacidad disponible. Esto es su punto de entrada.\n *\n * Vive en el CLI y no como comando slash porque programar trabajo recurrente\n * sin un proceso que siga vivo no significa nada: el reloj corre dentro de\n * `ada-cli serve`, y sin servidor un job no se dispara. De paso, este fichero\n * es nuestro y no cuesta presupuesto de sync.\n */\n\nimport { getAgentDir } from \"../config.ts\";\n\n/** Convierte `30m`, `2h`, `1d` a milisegundos. Devuelve undefined si no encaja. */\nexport function parseDuration(text: string): number | undefined {\n\tconst match = /^(\\d+)(m|h|d)$/.exec(text.trim());\n\tif (!match) return undefined;\n\tconst value = Number(match[1]);\n\tif (!Number.isFinite(value) || value <= 0) return undefined;\n\treturn value * (match[2] === \"m\" ? 60_000 : match[2] === \"h\" ? 3_600_000 : 86_400_000);\n}\n\nexport function isScheduleCommand(args: string[]): boolean {\n\treturn args[0] === \"schedule\";\n}\n\nfunction help(): void {\n\tconsole.log(`Uso:\n  ada-cli schedule list\n  ada-cli schedule add <id> --every <30m|2h|1d> --runs <n> --expires <ISO> -- <prompt>\n  ada-cli schedule add <id> --at <ISO> --runs 1 --expires <ISO> -- <prompt>\n  ada-cli schedule rm <id>\n\n--runs y --expires son OBLIGATORIOS: un trabajo recurrente sin techo ni fecha de\ncaducidad es gasto indefinido, y nadie se acuerda de apagarlo.\n\nEl reloj corre dentro de 'ada-cli serve'. Sin servidor, nada se dispara.`);\n}\n\n/** Lee `--flag valor` de la lista de argumentos. */\nfunction flag(args: string[], name: string): string | undefined {\n\tconst index = args.indexOf(name);\n\treturn index >= 0 ? args[index + 1] : undefined;\n}\n\nexport async function runScheduleCommand(args: string[]): Promise<boolean> {\n\tif (!isScheduleCommand(args)) return false;\n\tconst action = args[1];\n\tif (!action || args.includes(\"--help\") || args.includes(\"-h\")) {\n\t\thelp();\n\t\treturn true;\n\t}\n\n\tconst { Scheduler } = await import(\"../core/scheduler/scheduler.ts\");\n\tconst scheduler = new Scheduler(getAgentDir());\n\n\tif (action === \"list\") {\n\t\tconst jobs = scheduler.list();\n\t\tif (jobs.length === 0) {\n\t\t\tconsole.log(\"No hay trabajos programados.\");\n\t\t\treturn true;\n\t\t}\n\t\tfor (const job of jobs) {\n\t\t\tconst every = job.intervalMs ? `cada ${Math.round(job.intervalMs / 60000)}min` : \"una vez\";\n\t\t\tconsole.log(\n\t\t\t\t`${job.id}  ${every}  próxima: ${job.nextRunAt}  ${job.runsUsed}/${job.maxRuns} ejecuciones  caduca: ${job.expiresAt}${job.enabled ? \"\" : \"  [desactivado]\"}`,\n\t\t\t);\n\t\t\tconsole.log(`    ${job.prompt.slice(0, 100)}`);\n\t\t}\n\t\treturn true;\n\t}\n\n\tif (action === \"rm\") {\n\t\tconst id = args[2];\n\t\tif (!id) {\n\t\t\tconsole.error(\"Falta el id. Usa 'ada-cli schedule list' para verlos.\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconsole.log(scheduler.remove(id) ? `Eliminado: ${id}` : `No existe: ${id}`);\n\t\treturn true;\n\t}\n\n\tif (action === \"add\") {\n\t\tconst id = args[2];\n\t\t// Todo lo que va tras `--` es el prompt, sin interpretar: así puede\n\t\t// contener guiones sin que se confundan con flags.\n\t\tconst separator = args.indexOf(\"--\", 3);\n\t\tconst prompt = separator >= 0 ? args.slice(separator + 1).join(\" \") : \"\";\n\t\tif (!id || !prompt) {\n\t\t\thelp();\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst every = flag(args, \"--every\");\n\t\tconst intervalMs = every ? parseDuration(every) : undefined;\n\t\tif (every && intervalMs === undefined) {\n\t\t\tconsole.error(`Intervalo no reconocido: '${every}'. Usa formato 30m, 2h o 1d.`);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst runs = Number(flag(args, \"--runs\"));\n\t\tconst expiresAt = flag(args, \"--expires\");\n\t\tif (!expiresAt) {\n\t\t\tconsole.error(\"--expires es obligatorio (fecha ISO).\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst firstRunAt = flag(args, \"--at\") ?? new Date(Date.now() + (intervalMs ?? 0)).toISOString();\n\t\tconst result = scheduler.add({ id, prompt, firstRunAt, intervalMs, maxRuns: runs, expiresAt });\n\t\tif (!result.ok) {\n\t\t\tconsole.error(result.error ?? \"no se pudo crear el trabajo\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconsole.log(`Programado '${id}'. Primera ejecución: ${firstRunAt}`);\n\t\tconsole.log(\"Recuerda que solo corre mientras 'ada-cli serve' esté levantado.\");\n\t\treturn true;\n\t}\n\n\thelp();\n\tprocess.exitCode = 1;\n\treturn true;\n}\n"]}

package/dist/cli/schedule-command.js

@@ -0,0 +1,117 @@
+/**
+ * `ada-cli schedule` — trabajo recurrente sobre una sesión residente.
+ *
+ * El módulo `core/scheduler` existía completo y testeado, pero NADIE lo
+ * importaba: era código muerto mientras la tabla comparativa lo daba por
+ * capacidad disponible. Esto es su punto de entrada.
+ *
+ * Vive en el CLI y no como comando slash porque programar trabajo recurrente
+ * sin un proceso que siga vivo no significa nada: el reloj corre dentro de
+ * `ada-cli serve`, y sin servidor un job no se dispara. De paso, este fichero
+ * es nuestro y no cuesta presupuesto de sync.
+ */
+import { getAgentDir } from "../config.js";
+/** Convierte `30m`, `2h`, `1d` a milisegundos. Devuelve undefined si no encaja. */
+export function parseDuration(text) {
+    const match = /^(\d+)(m|h|d)$/.exec(text.trim());
+    if (!match)
+        return undefined;
+    const value = Number(match[1]);
+    if (!Number.isFinite(value) || value <= 0)
+        return undefined;
+    return value * (match[2] === "m" ? 60_000 : match[2] === "h" ? 3_600_000 : 86_400_000);
+}
+export function isScheduleCommand(args) {
+    return args[0] === "schedule";
+}
+function help() {
+    console.log(`Uso:
+  ada-cli schedule list
+  ada-cli schedule add <id> --every <30m|2h|1d> --runs <n> --expires <ISO> -- <prompt>
+  ada-cli schedule add <id> --at <ISO> --runs 1 --expires <ISO> -- <prompt>
+  ada-cli schedule rm <id>
+
+--runs y --expires son OBLIGATORIOS: un trabajo recurrente sin techo ni fecha de
+caducidad es gasto indefinido, y nadie se acuerda de apagarlo.
+
+El reloj corre dentro de 'ada-cli serve'. Sin servidor, nada se dispara.`);
+}
+/** Lee `--flag valor` de la lista de argumentos. */
+function flag(args, name) {
+    const index = args.indexOf(name);
+    return index >= 0 ? args[index + 1] : undefined;
+}
+export async function runScheduleCommand(args) {
+    if (!isScheduleCommand(args))
+        return false;
+    const action = args[1];
+    if (!action || args.includes("--help") || args.includes("-h")) {
+        help();
+        return true;
+    }
+    const { Scheduler } = await import("../core/scheduler/scheduler.js");
+    const scheduler = new Scheduler(getAgentDir());
+    if (action === "list") {
+        const jobs = scheduler.list();
+        if (jobs.length === 0) {
+            console.log("No hay trabajos programados.");
+            return true;
+        }
+        for (const job of jobs) {
+            const every = job.intervalMs ? `cada ${Math.round(job.intervalMs / 60000)}min` : "una vez";
+            console.log(`${job.id}  ${every}  próxima: ${job.nextRunAt}  ${job.runsUsed}/${job.maxRuns} ejecuciones  caduca: ${job.expiresAt}${job.enabled ? "" : "  [desactivado]"}`);
+            console.log(`    ${job.prompt.slice(0, 100)}`);
+        }
+        return true;
+    }
+    if (action === "rm") {
+        const id = args[2];
+        if (!id) {
+            console.error("Falta el id. Usa 'ada-cli schedule list' para verlos.");
+            process.exitCode = 1;
+            return true;
+        }
+        console.log(scheduler.remove(id) ? `Eliminado: ${id}` : `No existe: ${id}`);
+        return true;
+    }
+    if (action === "add") {
+        const id = args[2];
+        // Todo lo que va tras `--` es el prompt, sin interpretar: así puede
+        // contener guiones sin que se confundan con flags.
+        const separator = args.indexOf("--", 3);
+        const prompt = separator >= 0 ? args.slice(separator + 1).join(" ") : "";
+        if (!id || !prompt) {
+            help();
+            process.exitCode = 1;
+            return true;
+        }
+        const every = flag(args, "--every");
+        const intervalMs = every ? parseDuration(every) : undefined;
+        if (every && intervalMs === undefined) {
+            console.error(`Intervalo no reconocido: '${every}'. Usa formato 30m, 2h o 1d.`);
+            process.exitCode = 1;
+            return true;
+        }
+        const runs = Number(flag(args, "--runs"));
+        const expiresAt = flag(args, "--expires");
+        if (!expiresAt) {
+            console.error("--expires es obligatorio (fecha ISO).");
+            process.exitCode = 1;
+            return true;
+        }
+        const firstRunAt = flag(args, "--at") ?? new Date(Date.now() + (intervalMs ?? 0)).toISOString();
+        const result = scheduler.add({ id, prompt, firstRunAt, intervalMs, maxRuns: runs, expiresAt });
+        if (!result.ok) {
+            console.error(result.error ?? "no se pudo crear el trabajo");
+            process.exitCode = 1;
+            return true;
+        }
+        console.log(`Programado '${id}'. Primera ejecución: ${firstRunAt}`);
+        console.log("Recuerda que solo corre mientras 'ada-cli serve' esté levantado.");
+        return true;
+    }
+    help();
+    process.exitCode = 1;
+    return true;
+}
+//# sourceMappingURL=schedule-command.js.map

package/dist/cli/schedule-command.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"schedule-command.js","sourceRoot":"","sources":["../../src/cli/schedule-command.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;GAWG;AAEH,OAAO,EAAE,WAAW,EAAE,MAAM,cAAc,CAAC;AAE3C,mFAAmF;AACnF,MAAM,UAAU,aAAa,CAAC,IAAY,EAAsB;IAC/D,MAAM,KAAK,GAAG,gBAAgB,CAAC,IAAI,CAAC,IAAI,CAAC,IAAI,EAAE,CAAC,CAAC;IACjD,IAAI,CAAC,KAAK;QAAE,OAAO,SAAS,CAAC;IAC7B,MAAM,KAAK,GAAG,MAAM,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,CAAC;IAC/B,IAAI,CAAC,MAAM,CAAC,QAAQ,CAAC,KAAK,CAAC,IAAI,KAAK,IAAI,CAAC;QAAE,OAAO,SAAS,CAAC;IAC5D,OAAO,KAAK,GAAG,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,MAAM,CAAC,CAAC,CAAC,KAAK,CAAC,CAAC,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,SAAS,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC;AAAA,CACvF;AAED,MAAM,UAAU,iBAAiB,CAAC,IAAc,EAAW;IAC1D,OAAO,IAAI,CAAC,CAAC,CAAC,KAAK,UAAU,CAAC;AAAA,CAC9B;AAED,SAAS,IAAI,GAAS;IACrB,OAAO,CAAC,GAAG,CAAC;;;;;;;;;yEAS4D,CAAC,CAAC;AAAA,CAC1E;AAED,oDAAoD;AACpD,SAAS,IAAI,CAAC,IAAc,EAAE,IAAY,EAAsB;IAC/D,MAAM,KAAK,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,CAAC,CAAC;IACjC,OAAO,KAAK,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,GAAG,CAAC,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;AAAA,CAChD;AAED,MAAM,CAAC,KAAK,UAAU,kBAAkB,CAAC,IAAc,EAAoB;IAC1E,IAAI,CAAC,iBAAiB,CAAC,IAAI,CAAC;QAAE,OAAO,KAAK,CAAC;IAC3C,MAAM,MAAM,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;IACvB,IAAI,CAAC,MAAM,IAAI,IAAI,CAAC,QAAQ,CAAC,QAAQ,CAAC,IAAI,IAAI,CAAC,QAAQ,CAAC,IAAI,CAAC,EAAE,CAAC;QAC/D,IAAI,EAAE,CAAC;QACP,OAAO,IAAI,CAAC;IACb,CAAC;IAED,MAAM,EAAE,SAAS,EAAE,GAAG,MAAM,MAAM,CAAC,gCAAgC,CAAC,CAAC;IACrE,MAAM,SAAS,GAAG,IAAI,SAAS,CAAC,WAAW,EAAE,CAAC,CAAC;IAE/C,IAAI,MAAM,KAAK,MAAM,EAAE,CAAC;QACvB,MAAM,IAAI,GAAG,SAAS,CAAC,IAAI,EAAE,CAAC;QAC9B,IAAI,IAAI,CAAC,MAAM,KAAK,CAAC,EAAE,CAAC;YACvB,OAAO,CAAC,GAAG,CAAC,8BAA8B,CAAC,CAAC;YAC5C,OAAO,IAAI,CAAC;QACb,CAAC;QACD,KAAK,MAAM,GAAG,IAAI,IAAI,EAAE,CAAC;YACxB,MAAM,KAAK,GAAG,GAAG,CAAC,UAAU,CAAC,CAAC,CAAC,QAAQ,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,UAAU,GAAG,KAAK,CAAC,KAAK,CAAC,CAAC,CAAC,SAAS,CAAC;YAC3F,OAAO,CAAC,GAAG,CACV,GAAG,GAAG,CAAC,EAAE,KAAK,KAAK,eAAc,GAAG,CAAC,SAAS,KAAK,GAAG,CAAC,QAAQ,IAAI,GAAG,CAAC,OAAO,yBAAyB,GAAG,CAAC,SAAS,GAAG,GAAG,CAAC,OAAO,CAAC,CAAC,CAAC,EAAE,CAAC,CAAC,CAAC,iBAAiB,EAAE,CAC7J,CAAC;YACF,OAAO,CAAC,GAAG,CAAC,OAAO,GAAG,CAAC,MAAM,CAAC,KAAK,CAAC,CAAC,EAAE,GAAG,CAAC,EAAE,CAAC,CAAC;QAChD,CAAC;QACD,OAAO,IAAI,CAAC;IACb,CAAC;IAED,IAAI,MAAM,KAAK,IAAI,EAAE,CAAC;QACrB,MAAM,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;QACnB,IAAI,CAAC,EAAE,EAAE,CAAC;YACT,OAAO,CAAC,KAAK,CAAC,uDAAuD,CAAC,CAAC;YACvE,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;YACrB,OAAO,IAAI,CAAC;QACb,CAAC;QACD,OAAO,CAAC,GAAG,CAAC,SAAS,CAAC,MAAM,CAAC,EAAE,CAAC,CAAC,CAAC,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC,CAAC,cAAc,EAAE,EAAE,CAAC,CAAC;QAC5E,OAAO,IAAI,CAAC;IACb,CAAC;IAED,IAAI,MAAM,KAAK,KAAK,EAAE,CAAC;QACtB,MAAM,EAAE,GAAG,IAAI,CAAC,CAAC,CAAC,CAAC;QACnB,qEAAoE;QACpE,mDAAmD;QACnD,MAAM,SAAS,GAAG,IAAI,CAAC,OAAO,CAAC,IAAI,EAAE,CAAC,CAAC,CAAC;QACxC,MAAM,MAAM,GAAG,SAAS,IAAI,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,KAAK,CAAC,SAAS,GAAG,CAAC,CAAC,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC,CAAC,CAAC,EAAE,CAAC;QACzE,IAAI,CAAC,EAAE,IAAI,CAAC,MAAM,EAAE,CAAC;YACpB,IAAI,EAAE,CAAC;YACP,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;YACrB,OAAO,IAAI,CAAC;QACb,CAAC;QACD,MAAM,KAAK,GAAG,IAAI,CAAC,IAAI,EAAE,SAAS,CAAC,CAAC;QACpC,MAAM,UAAU,GAAG,KAAK,CAAC,CAAC,CAAC,aAAa,CAAC,KAAK,CAAC,CAAC,CAAC,CAAC,SAAS,CAAC;QAC5D,IAAI,KAAK,IAAI,UAAU,KAAK,SAAS,EAAE,CAAC;YACvC,OAAO,CAAC,KAAK,CAAC,6BAA6B,KAAK,8BAA8B,CAAC,CAAC;YAChF,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;YACrB,OAAO,IAAI,CAAC;QACb,CAAC;QACD,MAAM,IAAI,GAAG,MAAM,CAAC,IAAI,CAAC,IAAI,EAAE,QAAQ,CAAC,CAAC,CAAC;QAC1C,MAAM,SAAS,GAAG,IAAI,CAAC,IAAI,EAAE,WAAW,CAAC,CAAC;QAC1C,IAAI,CAAC,SAAS,EAAE,CAAC;YAChB,OAAO,CAAC,KAAK,CAAC,uCAAuC,CAAC,CAAC;YACvD,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;YACrB,OAAO,IAAI,CAAC;QACb,CAAC;QACD,MAAM,UAAU,GAAG,IAAI,CAAC,IAAI,EAAE,MAAM,CAAC,IAAI,IAAI,IAAI,CAAC,IAAI,CAAC,GAAG,EAAE,GAAG,CAAC,UAAU,IAAI,CAAC,CAAC,CAAC,CAAC,WAAW,EAAE,CAAC;QAChG,MAAM,MAAM,GAAG,SAAS,CAAC,GAAG,CAAC,EAAE,EAAE,EAAE,MAAM,EAAE,UAAU,EAAE,UAAU,EAAE,OAAO,EAAE,IAAI,EAAE,SAAS,EAAE,CAAC,CAAC;QAC/F,IAAI,CAAC,MAAM,CAAC,EAAE,EAAE,CAAC;YAChB,OAAO,CAAC,KAAK,CAAC,MAAM,CAAC,KAAK,IAAI,6BAA6B,CAAC,CAAC;YAC7D,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;YACrB,OAAO,IAAI,CAAC;QACb,CAAC;QACD,OAAO,CAAC,GAAG,CAAC,eAAe,EAAE,0BAAyB,UAAU,EAAE,CAAC,CAAC;QACpE,OAAO,CAAC,GAAG,CAAC,mEAAkE,CAAC,CAAC;QAChF,OAAO,IAAI,CAAC;IACb,CAAC;IAED,IAAI,EAAE,CAAC;IACP,OAAO,CAAC,QAAQ,GAAG,CAAC,CAAC;IACrB,OAAO,IAAI,CAAC;AAAA,CACZ","sourcesContent":["/**\n * `ada-cli schedule` — trabajo recurrente sobre una sesión residente.\n *\n * El módulo `core/scheduler` existía completo y testeado, pero NADIE lo\n * importaba: era código muerto mientras la tabla comparativa lo daba por\n * capacidad disponible. Esto es su punto de entrada.\n *\n * Vive en el CLI y no como comando slash porque programar trabajo recurrente\n * sin un proceso que siga vivo no significa nada: el reloj corre dentro de\n * `ada-cli serve`, y sin servidor un job no se dispara. De paso, este fichero\n * es nuestro y no cuesta presupuesto de sync.\n */\n\nimport { getAgentDir } from \"../config.ts\";\n\n/** Convierte `30m`, `2h`, `1d` a milisegundos. Devuelve undefined si no encaja. */\nexport function parseDuration(text: string): number | undefined {\n\tconst match = /^(\\d+)(m|h|d)$/.exec(text.trim());\n\tif (!match) return undefined;\n\tconst value = Number(match[1]);\n\tif (!Number.isFinite(value) || value <= 0) return undefined;\n\treturn value * (match[2] === \"m\" ? 60_000 : match[2] === \"h\" ? 3_600_000 : 86_400_000);\n}\n\nexport function isScheduleCommand(args: string[]): boolean {\n\treturn args[0] === \"schedule\";\n}\n\nfunction help(): void {\n\tconsole.log(`Uso:\n  ada-cli schedule list\n  ada-cli schedule add <id> --every <30m|2h|1d> --runs <n> --expires <ISO> -- <prompt>\n  ada-cli schedule add <id> --at <ISO> --runs 1 --expires <ISO> -- <prompt>\n  ada-cli schedule rm <id>\n\n--runs y --expires son OBLIGATORIOS: un trabajo recurrente sin techo ni fecha de\ncaducidad es gasto indefinido, y nadie se acuerda de apagarlo.\n\nEl reloj corre dentro de 'ada-cli serve'. Sin servidor, nada se dispara.`);\n}\n\n/** Lee `--flag valor` de la lista de argumentos. */\nfunction flag(args: string[], name: string): string | undefined {\n\tconst index = args.indexOf(name);\n\treturn index >= 0 ? args[index + 1] : undefined;\n}\n\nexport async function runScheduleCommand(args: string[]): Promise<boolean> {\n\tif (!isScheduleCommand(args)) return false;\n\tconst action = args[1];\n\tif (!action || args.includes(\"--help\") || args.includes(\"-h\")) {\n\t\thelp();\n\t\treturn true;\n\t}\n\n\tconst { Scheduler } = await import(\"../core/scheduler/scheduler.ts\");\n\tconst scheduler = new Scheduler(getAgentDir());\n\n\tif (action === \"list\") {\n\t\tconst jobs = scheduler.list();\n\t\tif (jobs.length === 0) {\n\t\t\tconsole.log(\"No hay trabajos programados.\");\n\t\t\treturn true;\n\t\t}\n\t\tfor (const job of jobs) {\n\t\t\tconst every = job.intervalMs ? `cada ${Math.round(job.intervalMs / 60000)}min` : \"una vez\";\n\t\t\tconsole.log(\n\t\t\t\t`${job.id}  ${every}  próxima: ${job.nextRunAt}  ${job.runsUsed}/${job.maxRuns} ejecuciones  caduca: ${job.expiresAt}${job.enabled ? \"\" : \"  [desactivado]\"}`,\n\t\t\t);\n\t\t\tconsole.log(`    ${job.prompt.slice(0, 100)}`);\n\t\t}\n\t\treturn true;\n\t}\n\n\tif (action === \"rm\") {\n\t\tconst id = args[2];\n\t\tif (!id) {\n\t\t\tconsole.error(\"Falta el id. Usa 'ada-cli schedule list' para verlos.\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconsole.log(scheduler.remove(id) ? `Eliminado: ${id}` : `No existe: ${id}`);\n\t\treturn true;\n\t}\n\n\tif (action === \"add\") {\n\t\tconst id = args[2];\n\t\t// Todo lo que va tras `--` es el prompt, sin interpretar: así puede\n\t\t// contener guiones sin que se confundan con flags.\n\t\tconst separator = args.indexOf(\"--\", 3);\n\t\tconst prompt = separator >= 0 ? args.slice(separator + 1).join(\" \") : \"\";\n\t\tif (!id || !prompt) {\n\t\t\thelp();\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst every = flag(args, \"--every\");\n\t\tconst intervalMs = every ? parseDuration(every) : undefined;\n\t\tif (every && intervalMs === undefined) {\n\t\t\tconsole.error(`Intervalo no reconocido: '${every}'. Usa formato 30m, 2h o 1d.`);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst runs = Number(flag(args, \"--runs\"));\n\t\tconst expiresAt = flag(args, \"--expires\");\n\t\tif (!expiresAt) {\n\t\t\tconsole.error(\"--expires es obligatorio (fecha ISO).\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst firstRunAt = flag(args, \"--at\") ?? new Date(Date.now() + (intervalMs ?? 0)).toISOString();\n\t\tconst result = scheduler.add({ id, prompt, firstRunAt, intervalMs, maxRuns: runs, expiresAt });\n\t\tif (!result.ok) {\n\t\t\tconsole.error(result.error ?? \"no se pudo crear el trabajo\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconsole.log(`Programado '${id}'. Primera ejecución: ${firstRunAt}`);\n\t\tconsole.log(\"Recuerda que solo corre mientras 'ada-cli serve' esté levantado.\");\n\t\treturn true;\n\t}\n\n\thelp();\n\tprocess.exitCode = 1;\n\treturn true;\n}\n"]}

package/dist/cli/serve-command.d.ts

@@ -0,0 +1,72 @@
+/**
+ * Subcomandos que no abren el TUI: `serve`, `sessions`, `session`, `attach`,
+ * `acp`, `schedule`, `refine` y `telegram`.
+ *
+ * Se mantienen en un fichero propio y el enganche en `main.ts` es un import y
+ * una llamada, para no engordar la superficie de conflicto con upstream. `acp`
+ * entra aquí por lo mismo: añadirlo cuesta cero hunks nuevos contra upstream.
+ */
+/**
+ * Separa `--socket <ruta>` de los argumentos de la operación.
+ *
+ * Existe porque sin ella se colaban dentro: `session x rename --socket /tmp/s`
+ * acababa poniéndole a la sesión el nombre "--socket /tmp/s". Se exporta para
+ * poder probarlo sin levantar un servidor.
+ */
+export declare function parseSessionOpArgs(args: string[]): {
+    sessionId?: string;
+    op?: string;
+    rest: string[];
+};
+/**
+ * Por qué lo que ha llegado del modelo NO es una respuesta, o `null` si sí lo es.
+ *
+ * DOS formas de no serlo, y solo se miraba una:
+ *
+ *   - `vacia`: no ha llegado texto. Ésta ya se miraba.
+ *   - `cortada`: el proveedor se murió con el stream YA EMPEZADO. Llega texto,
+ *     así que la guarda de «texto vacío» no lo ve, y el trozo sale como si
+ *     fuera la respuesta buena.
+ *
+ * Reproducido con un servidor que emite un trozo y destruye el socket. La
+ * respuesta que devuelve el SDK es literalmente:
+ *
+ *     { stopReason: "error", errorMessage: "terminated",
+ *       texto: "reintenta las peti" }
+ *
+ * Y con eso, `ada-cli commit --hacer` dejó ESTO en la historia de git, con
+ * salida 0 y stderr vacío:
+ *
+ *     [main c3a409b] reintenta las peti
+ *      1 file changed, 7 insertions(+)
+ *
+ * `review` entra por la misma puerta: media revisión sale como la revisión
+ * entera. Es exactamente el fallo que esta función dice impedir, colándose por
+ * el lado que no se miraba.
+ *
+ * El trozo se DESCARTA entero a propósito: media revisión no es una revisión y
+ * medio titular no es un titular. `length` NO cuenta como corte — ahí el modelo
+ * terminó de hablar, aunque fuera por el tope de tokens, y rechazarlo sería un
+ * falso positivo que rompe la cara buena.
+ */
+export declare function porQueNoEsRespuesta(stopReason: string, texto: string): "vacia" | "cortada" | null;
+/**
+ * Los subcomandos que se despachan aquí. La fuente de verdad, exportada.
+ *
+ * Se exporta para que la prueba de la ayuda pueda recorrer ESTA lista y no una
+ * copia. Con la copia, añadir un subcomando nuevo sin documentarlo dejaba la
+ * prueba en verde: se comprobó añadiendo `fantasma` a la lista y sin tocar la
+ * ayuda, y pasaron 21 de 21. La prueba decía mirar la fuente de verdad y miraba
+ * su propia copia, que es el mismo defecto que dice prevenir.
+ */
+export declare const SUBCOMANDOS_SERVE: readonly string[];
+export declare function isServeCommand(args: string[]): boolean;
+export declare function printServeHelp(): void;
+/**
+ * Ejecuta el subcomando. Devuelve true si consumió los argumentos.
+ *
+ * Las dependencias pesadas se cargan de forma perezosa: un arranque normal del
+ * CLI no debe pagar el coste de importar el servidor.
+ */
+export declare function runServeCommand(args: string[]): Promise<boolean>;
+//# sourceMappingURL=serve-command.d.ts.map

package/dist/cli/serve-command.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"serve-command.d.ts","sourceRoot":"","sources":["../../src/cli/serve-command.ts"],"names":[],"mappings":"AAAA;;;;;;;GAOG;AAKH;;;;;;GAMG;AACH,wBAAgB,kBAAkB,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG;IAAE,SAAS,CAAC,EAAE,MAAM,CAAC;IAAC,EAAE,CAAC,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,EAAE,CAAA;CAAE,CAKtG;AAED;;;;;;;;;;;;;;;;;;;;;;;;;;;;;;GA8BG;AACH,wBAAgB,mBAAmB,CAAC,UAAU,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,GAAG,OAAO,GAAG,SAAS,GAAG,IAAI,CAIjG;AAyED;;;;;;;;GAQG;AACH,eAAO,MAAM,iBAAiB,EAAE,SAAS,MAAM,EAkC9C,CAAC;AAEF,wBAAgB,cAAc,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAEtD;AAED,wBAAgB,cAAc,IAAI,IAAI,CA8BrC;AAED;;;;;GAKG;AACH,wBAAsB,eAAe,CAAC,IAAI,EAAE,MAAM,EAAE,GAAG,OAAO,CAAC,OAAO,CAAC,CAiYtE","sourcesContent":["/**\n * Subcomandos que no abren el TUI: `serve`, `sessions`, `session`, `attach`,\n * `acp`, `schedule`, `refine` y `telegram`.\n *\n * Se mantienen en un fichero propio y el enganche en `main.ts` es un import y\n * una llamada, para no engordar la superficie de conflicto con upstream. `acp`\n * entra aquí por lo mismo: añadirlo cuesta cero hunks nuevos contra upstream.\n */\n\nimport { join } from \"node:path\";\nimport { getAgentDir } from \"../config.ts\";\n\n/**\n * Separa `--socket <ruta>` de los argumentos de la operación.\n *\n * Existe porque sin ella se colaban dentro: `session x rename --socket /tmp/s`\n * acababa poniéndole a la sesión el nombre \"--socket /tmp/s\". Se exporta para\n * poder probarlo sin levantar un servidor.\n */\nexport function parseSessionOpArgs(args: string[]): { sessionId?: string; op?: string; rest: string[] } {\n\tconst socketIndex = args.indexOf(\"--socket\");\n\tconst cleaned = socketIndex >= 0 ? args.filter((_, i) => i !== socketIndex && i !== socketIndex + 1) : args;\n\tconst [, sessionId, op, ...rest] = cleaned;\n\treturn { sessionId, op, rest };\n}\n\n/**\n * Por qué lo que ha llegado del modelo NO es una respuesta, o `null` si sí lo es.\n *\n * DOS formas de no serlo, y solo se miraba una:\n *\n *   - `vacia`: no ha llegado texto. Ésta ya se miraba.\n *   - `cortada`: el proveedor se murió con el stream YA EMPEZADO. Llega texto,\n *     así que la guarda de «texto vacío» no lo ve, y el trozo sale como si\n *     fuera la respuesta buena.\n *\n * Reproducido con un servidor que emite un trozo y destruye el socket. La\n * respuesta que devuelve el SDK es literalmente:\n *\n *     { stopReason: \"error\", errorMessage: \"terminated\",\n *       texto: \"reintenta las peti\" }\n *\n * Y con eso, `ada-cli commit --hacer` dejó ESTO en la historia de git, con\n * salida 0 y stderr vacío:\n *\n *     [main c3a409b] reintenta las peti\n *      1 file changed, 7 insertions(+)\n *\n * `review` entra por la misma puerta: media revisión sale como la revisión\n * entera. Es exactamente el fallo que esta función dice impedir, colándose por\n * el lado que no se miraba.\n *\n * El trozo se DESCARTA entero a propósito: media revisión no es una revisión y\n * medio titular no es un titular. `length` NO cuenta como corte — ahí el modelo\n * terminó de hablar, aunque fuera por el tope de tokens, y rechazarlo sería un\n * falso positivo que rompe la cara buena.\n */\nexport function porQueNoEsRespuesta(stopReason: string, texto: string): \"vacia\" | \"cortada\" | null {\n\tif (stopReason === \"error\" || stopReason === \"aborted\") return \"cortada\";\n\tif (texto.trim() === \"\") return \"vacia\";\n\treturn null;\n}\n\n/**\n * Una llamada al modelo configurado, lista para inyectar en un subcomando.\n *\n * Vive aquí y no en cada comando porque `review` y `commit` la necesitan igual,\n * y sobre todo porque la parte que importa —una respuesta vacía o cortada es un\n * FALLO, no un resultado— no puede depender de que cada comando se acuerde de\n * escribirla. Ya nos costó cuatro revisiones falsamente limpias: eran 429 del\n * proveedor que se leyeron como \"tu código está limpio\".\n *\n * `advertencia` es lo que cambia entre comandos: la frase que evita que el\n * usuario interprete el fallo como un veredicto.\n */\nfunction crearLlamadaAlModelo(\n\targs: string[],\n\tagentDir: string,\n\tadvertencia: string,\n): (sistema: string, usuario: string) => Promise<string> {\n\treturn async (sistema, usuario) => {\n\t\tconst { ModelRuntime } = await import(\"../core/model-runtime.ts\");\n\t\tconst { resolveCliModel } = await import(\"../core/model-resolver.ts\");\n\t\tconst { SettingsManager } = await import(\"../core/settings-manager.ts\");\n\t\tconst { homedir } = await import(\"node:os\");\n\t\tconst modelRuntime = await ModelRuntime.create();\n\t\t// El modelo de los ajustes, que es el que el usuario ya eligió. Sin esto\n\t\t// habría que pasarlo a mano en cada llamada, y un comando que exige\n\t\t// recordar un identificador de modelo no lo lanza nadie.\n\t\tconst ajustes = SettingsManager.create(homedir(), agentDir);\n\t\t// `--model proveedor/modelo` gana a los ajustes: un CI necesita fijar con\n\t\t// qué se ejecuta, y no depender de lo que tenga elegido quien lo configuró.\n\t\tconst iModelo = args.indexOf(\"--model\");\n\t\tconst spec = iModelo >= 0 ? args[iModelo + 1] : undefined;\n\t\tconst barra = spec?.indexOf(\"/\") ?? -1;\n\t\tconst { model } = resolveCliModel({\n\t\t\tmodelRuntime,\n\t\t\tcliProvider: barra > 0 ? spec?.slice(0, barra) : ajustes.getDefaultProvider(),\n\t\t\tcliModel: barra > 0 ? spec?.slice(barra + 1) : ajustes.getDefaultModel(),\n\t\t});\n\t\tif (!model) {\n\t\t\tthrow new Error(\"No hay modelo configurado. Abre `ada-cli` y elige uno con /login o /model.\");\n\t\t}\n\t\tconst { contentText } = await import(\"@earendil-works/pi-ai\");\n\t\tconst respuesta = await modelRuntime.completeSimple(model, {\n\t\t\tsystemPrompt: sistema,\n\t\t\tmessages: [{ role: \"user\", content: usuario }],\n\t\t} as never);\n\t\tconst texto = contentText(respuesta.content, \"\");\n\t\t// Ni vacía ni cortada. Lo encontró el propio `ada-cli review` revisando\n\t\t// este mismo diff, y tenía razón: cuatro de cinco ejecuciones sobre un\n\t\t// fallo sembrado decían \"el revisor no ha encontrado nada\" cuando lo que\n\t\t// pasaba era un 429 del proveedor.\n\t\tconst motivo = porQueNoEsRespuesta(respuesta.stopReason, texto);\n\t\tif (motivo) {\n\t\t\t// El error del proveedor, clasificado. Antes se le enseñaba al usuario el\n\t\t\t// volcado JSON entero —400 caracteres con tres cosas útiles dentro— y la\n\t\t\t// mayoría no lo lee: un error que no se lee se interpreta como \"esto no\n\t\t\t// funciona\".\n\t\t\tconst { clasificarError, explicarError } = await import(\"../core/errores/proveedor.ts\");\n\t\t\tconst clasificado = clasificarError({ mensaje: respuesta.errorMessage ?? respuesta.stopReason });\n\t\t\t// Decir CUÁNTO llegó y que se tira. Callarlo dejaría al usuario creyendo\n\t\t\t// que el proveedor no contestó nada, cuando lo que pasó es que contestó\n\t\t\t// a medias y hemos decidido no fiarnos.\n\t\t\tconst aMedias =\n\t\t\t\tmotivo === \"cortada\" && texto.trim() !== \"\"\n\t\t\t\t\t? `\\nLa respuesta llegó a medias (${texto.trim().length} caracteres) y se DESCARTA: media respuesta no es una respuesta.`\n\t\t\t\t\t: \"\";\n\t\t\tthrow new Error(`${explicarError(clasificado)}${aMedias}\\n${advertencia}`);\n\t\t}\n\t\treturn texto;\n\t};\n}\n\n/**\n * Los subcomandos que se despachan aquí. La fuente de verdad, exportada.\n *\n * Se exporta para que la prueba de la ayuda pueda recorrer ESTA lista y no una\n * copia. Con la copia, añadir un subcomando nuevo sin documentarlo dejaba la\n * prueba en verde: se comprobó añadiendo `fantasma` a la lista y sin tocar la\n * ayuda, y pasaron 21 de 21. La prueba decía mirar la fuente de verdad y miraba\n * su propia copia, que es el mismo defecto que dice prevenir.\n */\nexport const SUBCOMANDOS_SERVE: readonly string[] = [\n\t\"serve\",\n\t\"attach\",\n\t\"sessions\",\n\t\"session\",\n\t\"acp\",\n\t\"schedule\",\n\t\"refine\",\n\t\"telegram\",\n\t\"undo\",\n\t\"usage\",\n\t\"wiki\",\n\t\"audit\",\n\t\"review\",\n\t\"import\",\n\t\"discover\",\n\t\"hashline\",\n\t\"cleanse\",\n\t\"banco\",\n\t\"commit\",\n\t// `computer` faltaba mientras `runServeCommand` lo manejaba y `printServeHelp`\n\t// lo anunciaba: `ada-cli computer status` no llegaba nunca a su rama, se iba\n\t// al camino normal y salía \"No API key found\" con código 0 — capacidad\n\t// construida, documentada y desenchufada. Se perdió justamente porque la\n\t// lista vivía copiada en dos sitios; por eso ahora es UNA constante y la\n\t// guarda de `test/serve-command-lista.test.ts` la compara con el despacho.\n\t\"computer\",\n\t// `politica`/`policy`: el ámbito de configuración que el usuario no puede\n\t// editar. Va en la lista, en `ownsItsHelp` y en la ayuda a la vez, que son\n\t// los tres sitios donde `computer` se quedó a medias.\n\t\"politica\",\n\t\"policy\",\n\t\"telemetria\",\n\t\"telemetry\",\n];\n\nexport function isServeCommand(args: string[]): boolean {\n\treturn SUBCOMANDOS_SERVE.includes(args[0] ?? \"\");\n}\n\nexport function printServeHelp(): void {\n\tconsole.log(`Uso:\n  ada-cli serve [--socket <ruta>]   Hospeda sesiones residentes; el trabajo sobrevive al cierre del terminal\n  ada-cli serve --broker [--escuchar host:puerto] [--credenciales <archivo.json>]\n                                     Broker de credenciales: las claves salen del portátil de cada dev\n  ada-cli sessions                  Lista las sesiones conocidas\n  ada-cli attach <id> [--watch]     Reconecta a una sesión servida (--watch = solo mirar)\n  ada-cli session <id> <op>         compact | rename <nombre> | fork | kill | ops\n  ada-cli acp                       Habla Agent Client Protocol por stdio (Zed y otros editores)\n  ada-cli schedule <acción>         Trabajo recurrente (corre dentro de 'serve')\n  ada-cli refine [sessionId]        Extrae lecciones durables de una sesión al harness\n  ada-cli telegram                  Controla las sesiones residentes desde Telegram\n  ada-cli undo [--dry-run]          Deshace los cambios de ficheros del último turno\n  ada-cli usage [--days N]          Tokens y coste reales, por modelo, proyecto y día\n  ada-cli wiki <index|check>        Wiki del repositorio con citas que caducan\n  ada-cli computer <status|install> Motor de control del ordenador y sus permisos\n  ada-cli review [--sarif <f>]      Revisa el diff con hallazgos anclados al código y SARIF 2.1.0\n  ada-cli audit <...>               Registro de cumplimiento: solo lee ficheros, contesta aunque el resto esté caído\n  ada-cli politica                  Política del equipo: el único ámbito que el usuario no puede editar\n                                    (en inglés: ada-cli policy)\n  ada-cli telemetria [--cuerpo]     Exportación OTLP: qué sale de esta máquina, y qué no\n                                    (en inglés: ada-cli telemetry)\n  ada-cli import <...>              Trae sesiones y configuración de otras herramientas\n  ada-cli discover                  Lista MCP, skills y comandos que otras herramientas dejaron en el proyecto\n  ada-cli hashline <tag|apply>      Formato de edición anclado por hash de línea\n  ada-cli cleanse [--dry-run]       Detecta y arregla en paralelo los diagnósticos de lint/typecheck/test\n  ada-cli banco <generar|run|comparar>  Banco de fiabilidad de edición: casos, ejecución y comparación\n  ada-cli commit [--hacer]          Mensaje de commit de lo que hay en el índice, por map-reduce sobre el diff\n\nEl socket vive por defecto en <agentDir>/server.sock y solo lo puede leer tu usuario.`);\n}\n\n/**\n * Ejecuta el subcomando. Devuelve true si consumió los argumentos.\n *\n * Las dependencias pesadas se cargan de forma perezosa: un arranque normal del\n * CLI no debe pagar el coste de importar el servidor.\n */\nexport async function runServeCommand(args: string[]): Promise<boolean> {\n\tif (!isServeCommand(args)) return false;\n\n\t// La ayuda genérica solo cubre los subcomandos que NO traen la suya. Antes\n\t// se comprobaba primero y se comía a las demás: `refine --help` acababa\n\t// enseñando la ayuda de `serve`.\n\tconst ownsItsHelp = [\n\t\t\"acp\",\n\t\t\"schedule\",\n\t\t\"refine\",\n\t\t\"telegram\",\n\t\t\"undo\",\n\t\t\"usage\",\n\t\t\"wiki\",\n\t\t\"computer\",\n\t\t\"audit\",\n\t\t\"politica\",\n\t\t\"policy\",\n\t\t\"telemetria\",\n\t\t\"telemetry\",\n\t\t\"review\",\n\t\t\"import\",\n\t\t\"discover\",\n\t\t\"hashline\",\n\t\t\"cleanse\",\n\t\t\"banco\",\n\t\t\"commit\",\n\t].includes(args[0] ?? \"\");\n\tif (!ownsItsHelp && (args.includes(\"--help\") || args.includes(\"-h\"))) {\n\t\tprintServeHelp();\n\t\treturn true;\n\t}\n\n\tconst agentDir = getAgentDir();\n\n\t// `audit` va lo primero: solo lee ficheros, no necesita socket ni servidor,\n\t// y tiene que poder contestar aunque el resto esté caído — un registro de\n\t// cumplimiento que exige que el producto arranque no sirve de mucho.\n\tconst { runPoliticaCommand } = await import(\"./politica-command.ts\");\n\tif (await runPoliticaCommand(args)) return true;\n\n\tconst { runTelemetriaCommand } = await import(\"./telemetria-command.ts\");\n\tif (await runTelemetriaCommand(args, agentDir)) return true;\n\n\tconst { runAuditCommand } = await import(\"./audit-command.ts\");\n\tif (await runAuditCommand(args, agentDir)) return true;\n\n\t// `discover` tampoco necesita servidor: solo lee ficheros de otras\n\t// herramientas en el cwd (MCP, skills, comandos de barra).\n\tconst { runDiscoverCommand } = await import(\"./discover-command.ts\");\n\tif (await runDiscoverCommand(args)) return true;\n\n\t// `import` tampoco necesita servidor: lee ficheros de otras herramientas y\n\t// escribe los nuestros.\n\tif (args[0] === \"import\") {\n\t\tconst { runImportCommand } = await import(\"./import-command.ts\");\n\t\tconst { randomUUID } = await import(\"node:crypto\");\n\t\treturn runImportCommand(args, agentDir, () => randomUUID());\n\t}\n\n\t// Frontera de red. `audit`, `discover` e `import` solo leen y escriben\n\t// ficheros, y cargar `undici` para ellos cuesta 221 módulos medidos (`discover`\n\t// pasaba de 120 a 341). Todo lo que va DEBAJO puede hablar por red, así que el\n\t// dispatcher se configura aquí.\n\t//\n\t// El sentido en el que se estropea es el bueno a propósito: un subcomando nuevo\n\t// se añade debajo y recibe el dispatcher. Subirlo por encima de esta línea es\n\t// lo que hay que justificar, no bajarlo.\n\tconst { configureHttpDispatcher } = await import(\"../core/http-dispatcher.ts\");\n\tconfigureHttpDispatcher();\n\n\t// `cleanse` tampoco necesita servidor: detecta comprobadores del proyecto,\n\t// los ejecuta y lanza subagentes de reparación directamente.\n\tif (args[0] === \"cleanse\") {\n\t\tconst { runCleanseCommand } = await import(\"./cleanse-command.ts\");\n\t\treturn runCleanseCommand(args, agentDir);\n\t}\n\n\t// `banco` tampoco necesita servidor: `generar`/`comparar` solo leen y\n\t// escriben ficheros; `run` abre sus propias sesiones de agente puntuales.\n\tif (args[0] === \"banco\") {\n\t\tconst { runBancoCommand } = await import(\"./banco-command.ts\");\n\t\treturn runBancoCommand(args);\n\t}\n\n\t// `review` y `commit` necesitan el modelo, así que se les pasa la llamada ya\n\t// construida. La lógica vive en `core/revision/` y `core/commit/`, que se\n\t// pueden probar sin red.\n\tif (args[0] === \"review\") {\n\t\tconst { runReviewCommand } = await import(\"./review-command.ts\");\n\t\treturn await runReviewCommand(args, {\n\t\t\trevisar: crearLlamadaAlModelo(args, agentDir, \"NO se ha revisado el diff. Esto no significa que esté limpio.\"),\n\t\t});\n\t}\n\n\t// `commit` tampoco necesita servidor: lee el índice de git, resume cada\n\t// fichero por separado (map) y sintetiza el mensaje (reduce).\n\tif (args[0] === \"commit\") {\n\t\tconst { runCommitCommand } = await import(\"./commit-command.ts\");\n\t\treturn await runCommitCommand(args, {\n\t\t\tllamar: crearLlamadaAlModelo(args, agentDir, \"NO se ha generado ningún mensaje. El índice sigue intacto.\"),\n\t\t});\n\t}\n\n\t// `acp` va antes de resolver el socket: no usa ninguno, habla JSON-RPC por\n\t// stdio. Y sobre todo, no puede escribir NADA en stdout antes de arrancar —\n\t// stdout es el cable.\n\tif (args[0] === \"acp\") {\n\t\tif (args.includes(\"--help\") || args.includes(\"-h\")) {\n\t\t\tconsole.log(`Uso:\n  ada-cli acp [--model <provider/model>]    Habla Agent Client Protocol por stdio\n\nPensado para que lo lance un editor (Zed y otros clientes ACP), no para\nejecutarlo a mano: stdout es el cable JSON-RPC. Configúralo como comando del\nagente en tu editor.`);\n\t\t\treturn true;\n\t\t}\n\t\tconst { runAcpMode } = await import(\"../modes/acp/acp-mode.ts\");\n\t\tconst { createAgentSession } = await import(\"../core/sdk.ts\");\n\t\t// `--model provider/model`. Se parte por la PRIMERA barra: los ids de\n\t\t// modelo pueden llevar más (openrouter/stealth/ox-alpha).\n\t\tconst modelIndex = args.indexOf(\"--model\");\n\t\tconst modelSpec = modelIndex >= 0 ? args[modelIndex + 1] : undefined;\n\t\tlet model: import(\"@earendil-works/pi-ai\").Model<never> | undefined;\n\t\tif (modelSpec) {\n\t\t\tconst slash = modelSpec.indexOf(\"/\");\n\t\t\tif (slash <= 0) {\n\t\t\t\tprocess.stderr.write(`[acp] --model debe ser 'provider/model' (recibido: ${modelSpec}).\\n`);\n\t\t\t\tprocess.exitCode = 1;\n\t\t\t\treturn true;\n\t\t\t}\n\t\t\tconst { ModelRuntime } = await import(\"../core/model-runtime.ts\");\n\t\t\tconst runtime = await ModelRuntime.create({\n\t\t\t\tauthPath: join(agentDir, \"auth.json\"),\n\t\t\t\tmodelsPath: join(agentDir, \"models.json\"),\n\t\t\t});\n\t\t\tmodel = runtime.getModel(modelSpec.slice(0, slash), modelSpec.slice(slash + 1)) as never;\n\t\t\tif (!model) {\n\t\t\t\t// Fallar aquí y no más tarde: arrancar con el modelo equivocado y no\n\t\t\t\t// decirlo produce una prueba que parece pasar y no prueba nada.\n\t\t\t\tprocess.stderr.write(`[acp] modelo no encontrado: ${modelSpec}\\n`);\n\t\t\t\tprocess.exitCode = 1;\n\t\t\t\treturn true;\n\t\t\t}\n\t\t}\n\t\tawait runAcpMode({\n\t\t\tcreateSession: async ({ cwd }) => {\n\t\t\t\tconst { session } = await createAgentSession({ cwd, agentDir, model });\n\t\t\t\treturn session;\n\t\t\t},\n\t\t});\n\t\treturn true;\n\t}\n\n\t// Ni `schedule` ni `refine` usan el socket: uno escribe el fichero de jobs\n\t// que `serve` lee, y el otro trabaja sobre una sesión en disco.\n\tif (args[0] === \"schedule\") {\n\t\tconst { runScheduleCommand } = await import(\"./schedule-command.ts\");\n\t\treturn runScheduleCommand(args);\n\t}\n\tif (args[0] === \"refine\") {\n\t\tconst { runRefineCommand } = await import(\"./refine-command.ts\");\n\t\treturn runRefineCommand(args);\n\t}\n\tif (args[0] === \"telegram\") {\n\t\tconst { runTelegramCommand } = await import(\"./telegram-command.ts\");\n\t\treturn runTelegramCommand(args);\n\t}\n\tif (args[0] === \"undo\") {\n\t\tconst { runUndoCommand } = await import(\"./undo-command.ts\");\n\t\treturn runUndoCommand(args, agentDir);\n\t}\n\tif (args[0] === \"usage\") {\n\t\tconst { runUsageCommand } = await import(\"./usage-command.ts\");\n\t\treturn runUsageCommand(args, agentDir);\n\t}\n\tif (args[0] === \"wiki\") {\n\t\tconst { runWikiCommand } = await import(\"./wiki-command.ts\");\n\t\treturn runWikiCommand(args);\n\t}\n\tif (args[0] === \"computer\") {\n\t\tconst { runComputerCommand } = await import(\"./computer-command.ts\");\n\t\treturn runComputerCommand(args);\n\t}\n\tif (args[0] === \"hashline\") {\n\t\tconst { runHashlineCommand } = await import(\"./hashline-command.ts\");\n\t\treturn runHashlineCommand(args);\n\t}\n\n\t// `serve --broker`: modo alternativo, no usa el socket Unix de sesiones. Es\n\t// lo que hace viable el hueco sin construir un producto servidor nuevo — un\n\t// equipo que quiera centralizar credenciales levanta esto, y quien no\n\t// quiera no paga nada porque el flag no existe hasta que se pide.\n\tif (args[0] === \"serve\" && args.includes(\"--broker\")) {\n\t\tconst { CredencialPool } = await import(\"../core/credenciales/pool.ts\");\n\t\tconst { iniciarServidorBroker, DEFAULT_BROKER_BIND } = await import(\"../core/credenciales/servidor.ts\");\n\t\tconst { LibroDeSesion } = await import(\"../core/audit/libro.ts\");\n\t\tconst { readFile } = await import(\"node:fs/promises\");\n\n\t\t// Fallar cerrado en el propio CLI, antes de tocar el pool o el fichero:\n\t\t// un broker sin bearer no debe ni intentar arrancar. `iniciarServidorBroker`\n\t\t// repite esta comprobación (es la que de verdad importa, por si algún día\n\t\t// se llama sin pasar por este CLI) — dos capas, un solo mensaje.\n\t\tconst token = process.env.ADA_BROKER_TOKEN?.trim();\n\t\tif (!token) {\n\t\t\tconsole.error(\n\t\t\t\t\"Falta ADA_BROKER_TOKEN. El broker no arranca sin un bearer: escuchar sin auth expondría \" +\n\t\t\t\t\t\"las credenciales de todo el equipo a quien alcance el puerto.\\n\" +\n\t\t\t\t\t'Genera uno y expórtalo, por ejemplo: export ADA_BROKER_TOKEN=\"$(node -e \\'console.log(require(\"crypto\").randomBytes(24).toString(\"hex\"))\\')\"',\n\t\t\t);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\n\t\tconst iCredenciales = args.indexOf(\"--credenciales\");\n\t\tconst rutaCredenciales = iCredenciales >= 0 ? args[iCredenciales + 1] : undefined;\n\t\tif (!rutaCredenciales) {\n\t\t\tconsole.error(\n\t\t\t\t\"Falta --credenciales <archivo.json> con las credenciales del pool. Formato: \" +\n\t\t\t\t\t'{\"<proveedor>\": [{\"id\": \"...\", \"valor\": \"...\"}, ...]}',\n\t\t\t);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\n\t\tlet credencialesCrudo: string;\n\t\ttry {\n\t\t\tcredencialesCrudo = await readFile(rutaCredenciales, \"utf-8\");\n\t\t} catch (error) {\n\t\t\tconsole.error(\n\t\t\t\t`No se puede leer ${rutaCredenciales}: ${error instanceof Error ? error.message : String(error)}`,\n\t\t\t);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\n\t\tconst pool = new CredencialPool(JSON.parse(credencialesCrudo));\n\t\t// Sesión propia \"broker\": no es una sesión de agente, pero el libro es el\n\t\t// mismo mecanismo y el mismo directorio — es lo que hace auditable \"quién\n\t\t// usó qué cuenta y cuándo\" sin construir un segundo sistema de registro.\n\t\tconst libro = new LibroDeSesion(agentDir, \"broker\");\n\n\t\tconst iEscuchar = args.indexOf(\"--escuchar\");\n\t\tconst bind = iEscuchar >= 0 && args[iEscuchar + 1] ? args[iEscuchar + 1] : DEFAULT_BROKER_BIND;\n\n\t\ttry {\n\t\t\tconst servidor = await iniciarServidorBroker({ pool, token, libro, bind });\n\t\t\tconsole.log(`Broker de credenciales escuchando en ${servidor.url}`);\n\t\t\tconsole.log(\"Bearer requerido en cada petición salvo /v1/health. Ctrl-C para parar.\");\n\t\t\tawait new Promise<void>(() => {});\n\t\t} catch (error) {\n\t\t\tconsole.error(error instanceof Error ? error.message : String(error));\n\t\t\tprocess.exitCode = 1;\n\t\t}\n\t\treturn true;\n\t}\n\n\tconst socketIndex = args.indexOf(\"--socket\");\n\tconst { defaultSocketPath } = await import(\"../server/serve.ts\");\n\tconst socketPath = socketIndex >= 0 && args[socketIndex + 1] ? args[socketIndex + 1] : defaultSocketPath(agentDir);\n\n\tif (args[0] === \"sessions\") {\n\t\tconst { SessionManager } = await import(\"../core/session-manager.ts\");\n\t\tconst sessions = await SessionManager.list(process.cwd());\n\t\tif (sessions.length === 0) {\n\t\t\tconsole.log(\"No hay sesiones en este proyecto.\");\n\t\t\treturn true;\n\t\t}\n\t\tfor (const session of sessions) {\n\t\t\tconst name = session.name ? ` — ${session.name}` : \"\";\n\t\t\tconsole.log(\n\t\t\t\t`${session.id.slice(0, 8)}  ${session.modified.toISOString()}  ${session.messageCount} msgs${name}`,\n\t\t\t);\n\t\t}\n\t\treturn true;\n\t}\n\n\tif (args[0] === \"session\") {\n\t\t// `ada-cli session <id> <op> [args]` — compact/rename/fork/kill sobre una\n\t\t// sesión del servidor. Van por el comando genérico `extension`, así que\n\t\t// añadir una op nueva no toca este fichero ni el protocolo.\n\t\tconst { sessionId, op, rest } = parseSessionOpArgs(args);\n\t\tif (!sessionId || !op) {\n\t\t\tconsole.error(\"Uso: ada-cli session <id> <compact|rename|fork|kill|ops> [args]\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst { existsSync } = await import(\"node:fs\");\n\t\tif (!existsSync(socketPath)) {\n\t\t\tconsole.error(`No hay ningún servidor de Ada en ${socketPath}. Arráncalo con 'ada-cli serve'.`);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst payload =\n\t\t\top === \"rename\"\n\t\t\t\t? { name: rest.join(\" \") }\n\t\t\t\t: op === \"compact\" && rest.length > 0\n\t\t\t\t\t? { customInstructions: rest.join(\" \") }\n\t\t\t\t\t: {};\n\t\tconst { runExtensionOp } = await import(\"../server/extension-ops.ts\");\n\t\tconst result = await runExtensionOp({ socketPath, op, sessionId, payload });\n\t\tif (!result.ok) {\n\t\t\tconsole.error(result.error ?? \"la operación falló\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconsole.log(JSON.stringify(result.data, null, 2));\n\t\treturn true;\n\t}\n\n\tif (args[0] === \"attach\") {\n\t\tconst sessionId = args[1];\n\t\tif (!sessionId) {\n\t\t\tconsole.error(\"Falta el id de sesión. Usa 'ada-cli sessions' para verlos.\");\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst { existsSync } = await import(\"node:fs\");\n\t\tif (!existsSync(socketPath)) {\n\t\t\tconsole.error(`No hay ningún servidor de Ada en ${socketPath}. Arráncalo con 'ada-cli serve'.`);\n\t\t\tprocess.exitCode = 1;\n\t\t\treturn true;\n\t\t}\n\t\tconst { attach } = await import(\"../server/attach.ts\");\n\t\ttry {\n\t\t\t// --watch: solo mirar, sin abrir entrada. Útil para vigilar un goal\n\t\t\t// largo sin riesgo de escribir en él por accidente.\n\t\t\tawait attach({ socketPath, sessionId, readOnly: args.includes(\"--watch\") });\n\t\t} catch (error) {\n\t\t\tconsole.error(error instanceof Error ? error.message : String(error));\n\t\t\tprocess.exitCode = 1;\n\t\t}\n\t\treturn true;\n\t}\n\n\t// serve\n\tconst { AdaServerService } = await import(\"../server/pi-server-service.ts\");\n\tconst { serve } = await import(\"../server/serve.ts\");\n\tconst { createAgentSession } = await import(\"../core/sdk.ts\");\n\n\tconst service = new AdaServerService({\n\t\tagentDir,\n\t\tdefaultCwd: process.cwd(),\n\t\tcreateSession: async (options) => {\n\t\t\tconst { session } = await createAgentSession({\n\t\t\t\tcwd: options.cwd,\n\t\t\t\tagentDir,\n\t\t\t\tsessionManager: options.sessionManager as never,\n\t\t\t});\n\t\t\treturn session;\n\t\t},\n\t});\n\n\ttry {\n\t\tconst handle = await serve({\n\t\t\tservice,\n\t\t\tsocketPath,\n\t\t\tonError: (error) => console.error(`[serve] ${error.message}`),\n\t\t});\n\t\t// Reloj del scheduler: los trabajos recurrentes solo existen mientras haya\n\t\t// un proceso vivo que los dispare, y ese proceso es este.\n\t\tconst { Scheduler } = await import(\"../core/scheduler/scheduler.ts\");\n\t\tconst { startSchedulerTick } = await import(\"../server/scheduler-tick.ts\");\n\t\tconst scheduler = new Scheduler(agentDir);\n\t\tstartSchedulerTick({\n\t\t\tscheduler,\n\t\t\tonError: (error) => console.error(`[schedule] ${error.message}`),\n\t\t\tdispatch: async (job) => {\n\t\t\t\t// Sesión propia por job: dos trabajos recurrentes compartiendo\n\t\t\t\t// historial se contaminarían el contexto mutuamente.\n\t\t\t\tconst runtime = await service.createSession({ id: `schedule-${job.id}-${Date.now()}`, cwd: process.cwd() });\n\t\t\t\tawait runtime.prompt({ text: job.prompt });\n\t\t\t\treturn true;\n\t\t\t},\n\t\t});\n\n\t\tconsole.log(`Ada escuchando en ${handle.socketPath}`);\n\t\tconsole.log(\"El trabajo continúa aunque cierres este terminal. Ctrl-C para parar el servidor.\");\n\t\t// Mantener vivo el proceso: el servidor es el dueño de la ejecución.\n\t\tawait new Promise<void>(() => {});\n\t} catch (error) {\n\t\tconsole.error(error instanceof Error ? error.message : String(error));\n\t\tprocess.exitCode = 1;\n\t}\n\treturn true;\n}\n"]}