chocolatito-code 1.6.14 → 1.6.15

package/dist/servidor/servidor.d.ts

@@ -36,6 +36,11 @@ export interface OpcionesServidor {
     enRed?: boolean;
     /** Para pruebas: token fijo en vez de uno aleatorio. */
     token?: string;
+    /**
+     * Un nombre de dominio mas que se acepta como Host, para entrar por un tunel
+     * (cloudflared, ngrok). Uno concreto, no cualquiera.
+     */
+    anfitrionAutorizado?: string;
     /**
      * La carpeta de trabajo, leida en cada peticion y no copiada al arrancar.
      * Con /cd el agente se cambia de carpeta a mitad de sesion, y lo que se sube
@@ -58,9 +63,72 @@ export interface ServidorWeb {
  * Host valido: localhost o una IP literal. Ver el punto 2 de la cabecera.
  * Se le quita el puerto y los corchetes de IPv6 antes de mirarlo.
  */
-export declare function anfitrionAceptable(host: string | undefined): boolean;
+/**
+ * Host valido: localhost, una IP literal, o EL NOMBRE QUE SE HAYA AUTORIZADO.
+ *
+ * Lo tercero existe por un caso real: el README dice desde el primer dia que
+ * para entrar desde fuera de casa se use Tailscale o cloudflared... y con
+ * cloudflared esto devolvia 401. Tailscale funciona porque reparte IPs, que
+ * pasan la comprobacion; un tunel llega siempre con un nombre de dominio.
+ * O sea que el programa prometia algo que el mismo rechazaba.
+ *
+ * Se arregla SIN quitar la cerradura. No es "acepta cualquier dominio": es
+ * "acepta exactamente este, el que me acabas de decir". Contra el DNS rebinding
+ * -una web cualquiera que apunta su dominio a tu 127.0.0.1 y usa el navegador de
+ * la victima de puente- la defensa sigue entera, porque ese ataque llega con el
+ * dominio del atacante y no con el que tu autorizaste.
+ */
+export declare function anfitrionAceptable(host: string | undefined, autorizado?: string): boolean;
+/**
+ * De lo que pegue el usuario, saca el nombre.
+ *
+ * Nadie copia "abc-def.trycloudflare.com": se copia el enlace entero con su
+ * https:// delante, y a veces con la barra final. Exigir el nombre pelado seria
+ * un 401 que nadie sabria explicar.
+ */
+export declare function nombreDeAnfitrion(entrada: string | undefined): string;
 /** Origin, si viene, tiene que ser el mismo sitio. Ver el punto 3. */
-export declare function origenAceptable(origin: string | undefined, host: string | undefined): boolean;
+/**
+ * Origin, si viene, tiene que ser el mismo sitio. Ver el punto 3.
+ *
+ * CON UN TUNEL DELANTE, "EL MISMO SITIO" NO ES EL MISMO TEXTO
+ *
+ * Se vio en la primera prueba de verdad desde un movil con datos: la pagina
+ * cargaba y el WebSocket rebotaba con 401. El motivo es que cloudflared
+ * REESCRIBE la cabecera Host con la del destino -127.0.0.1:4700-, mientras que
+ * el navegador manda el Origin que el ve, que es el del tunel. Comparados a
+ * pelo no coinciden nunca, asi que la sesion se quedaba muda: el usuario
+ * escribia y no pasaba nada.
+ *
+ * Por eso el dominio autorizado vale tambien como Origin. Y sigue siendo una
+ * cerradura: se acepta EXACTAMENTE ese, el que se escribio en --tunel.
+ */
+export declare function origenAceptable(origin: string | undefined, host: string | undefined, autorizado?: string): boolean;
+/**
+ * EL TOKEN TIENE QUE SOBREVIVIR A UNA RECARGA, Y LA URL NO LO HACE
+ *
+ * DE DONDE SALE: primera prueba con un movil de verdad. El enlace se abre bien,
+ * la pagina quita el token de la barra de direcciones -para que no se quede en
+ * el historial ni salga en una captura- y a la primera recarga se acabo la
+ * sesion: la peticion de la propia pagina ya no lleva token, el servidor
+ * contesta 401 "No." y ahi no hay javascript que valga, porque la pagina ni
+ * llega a cargarse. En el terminal salieron trece rechazos seguidos.
+ *
+ * Y en un movil recargar no es raro: es tirar hacia abajo sin querer, y sobre
+ * todo es lo que hace Android solo cuando la pestana se queda de fondo. O sea,
+ * exactamente lo que hay que hacer para probar las notificaciones.
+ *
+ * Se arregla como se arregla en cualquier sitio: al entrar con el token bueno se
+ * deja una cookie, y a partir de ahi el navegador la manda sola en todo -la
+ * pagina, el socket, las imagenes-.
+ *
+ *  - SameSite=Strict: una web ajena no puede hacer que tu navegador la mande.
+ *    Es la misma defensa que el Origin, por otro camino.
+ *  - HttpOnly: no la lee ningun javascript, ni el nuestro.
+ *  - Sin Max-Age: se muere al cerrar el navegador, como la sesion.
+ */
+export declare const COOKIE_DE_SESION = "chocolatito_web";
+export declare function tokenDeLaCookie(cabecera: string | undefined): string;
 export declare function tokenDeLaPeticion(url: string | undefined): string;
 export declare function iniciarServidorWeb(opciones?: OpcionesServidor): Promise<ServidorWeb>;
 /** Las IP de la maquina en la red local, para abrirla desde el movil. */

package/dist/servidor/servidor.js

@@ -7,7 +7,7 @@ import chalk from "chalk";
 import { WebSocketServer, WebSocket } from "ws";
 import { paginaHtml } from "./pagina.js";
 import { LIMITE_DE_SUBIDA, avisoDeSubida, guardarSubida, rutaServible, tipoDelArchivo, } from "./archivos.js";
-import { alEscribirWeb, anotarWeb, historialWeb } from "./captura.js";
+import { alEscribirWeb, anotarWeb, cerrarLineaWeb, historialWeb } from "./captura.js";
 import { cancelarPermisosAbiertos, entregarEntrada, fijarAvisoDeTurno, fijarEmisorDePermiso, responderPermiso, } from "./puente.js";
 const PUERTO_POR_DEFECTO = 4700;
 /**
@@ -39,7 +39,22 @@ function mismoToken(a, b) {
  * Host valido: localhost o una IP literal. Ver el punto 2 de la cabecera.
  * Se le quita el puerto y los corchetes de IPv6 antes de mirarlo.
  */
-export function anfitrionAceptable(host) {
+/**
+ * Host valido: localhost, una IP literal, o EL NOMBRE QUE SE HAYA AUTORIZADO.
+ *
+ * Lo tercero existe por un caso real: el README dice desde el primer dia que
+ * para entrar desde fuera de casa se use Tailscale o cloudflared... y con
+ * cloudflared esto devolvia 401. Tailscale funciona porque reparte IPs, que
+ * pasan la comprobacion; un tunel llega siempre con un nombre de dominio.
+ * O sea que el programa prometia algo que el mismo rechazaba.
+ *
+ * Se arregla SIN quitar la cerradura. No es "acepta cualquier dominio": es
+ * "acepta exactamente este, el que me acabas de decir". Contra el DNS rebinding
+ * -una web cualquiera que apunta su dominio a tu 127.0.0.1 y usa el navegador de
+ * la victima de puente- la defensa sigue entera, porque ese ataque llega con el
+ * dominio del atacante y no con el que tu autorizaste.
+ */
+export function anfitrionAceptable(host, autorizado) {
     if (!host)
         return false;
     let nombre = host.trim();
@@ -56,19 +71,99 @@ export function anfitrionAceptable(host) {
     }
     if (nombre === "localhost")
         return true;
+    if (autorizado && nombre.toLowerCase() === autorizado.toLowerCase())
+        return true;
     return /^[0-9.]+$/.test(nombre) || /^[0-9a-fA-F:]+$/.test(nombre);
 }
+/**
+ * De lo que pegue el usuario, saca el nombre.
+ *
+ * Nadie copia "abc-def.trycloudflare.com": se copia el enlace entero con su
+ * https:// delante, y a veces con la barra final. Exigir el nombre pelado seria
+ * un 401 que nadie sabria explicar.
+ */
+export function nombreDeAnfitrion(entrada) {
+    if (!entrada)
+        return "";
+    const limpio = entrada.trim();
+    try {
+        if (/^[a-z]+:\/\//i.test(limpio))
+            return new URL(limpio).hostname;
+    }
+    catch { }
+    return limpio.replace(/^\/+|\/+$/g, "").split("/")[0].split(":")[0];
+}
 /** Origin, si viene, tiene que ser el mismo sitio. Ver el punto 3. */
-export function origenAceptable(origin, host) {
+/**
+ * Origin, si viene, tiene que ser el mismo sitio. Ver el punto 3.
+ *
+ * CON UN TUNEL DELANTE, "EL MISMO SITIO" NO ES EL MISMO TEXTO
+ *
+ * Se vio en la primera prueba de verdad desde un movil con datos: la pagina
+ * cargaba y el WebSocket rebotaba con 401. El motivo es que cloudflared
+ * REESCRIBE la cabecera Host con la del destino -127.0.0.1:4700-, mientras que
+ * el navegador manda el Origin que el ve, que es el del tunel. Comparados a
+ * pelo no coinciden nunca, asi que la sesion se quedaba muda: el usuario
+ * escribia y no pasaba nada.
+ *
+ * Por eso el dominio autorizado vale tambien como Origin. Y sigue siendo una
+ * cerradura: se acepta EXACTAMENTE ese, el que se escribio en --tunel.
+ */
+export function origenAceptable(origin, host, autorizado) {
     if (!origin)
         return true;
     try {
-        return new URL(origin).host === host;
+        const u = new URL(origin);
+        if (u.host === host)
+            return true;
+        return !!autorizado && u.hostname.toLowerCase() === autorizado.toLowerCase();
     }
     catch {
         return false;
     }
 }
+/**
+ * EL TOKEN TIENE QUE SOBREVIVIR A UNA RECARGA, Y LA URL NO LO HACE
+ *
+ * DE DONDE SALE: primera prueba con un movil de verdad. El enlace se abre bien,
+ * la pagina quita el token de la barra de direcciones -para que no se quede en
+ * el historial ni salga en una captura- y a la primera recarga se acabo la
+ * sesion: la peticion de la propia pagina ya no lleva token, el servidor
+ * contesta 401 "No." y ahi no hay javascript que valga, porque la pagina ni
+ * llega a cargarse. En el terminal salieron trece rechazos seguidos.
+ *
+ * Y en un movil recargar no es raro: es tirar hacia abajo sin querer, y sobre
+ * todo es lo que hace Android solo cuando la pestana se queda de fondo. O sea,
+ * exactamente lo que hay que hacer para probar las notificaciones.
+ *
+ * Se arregla como se arregla en cualquier sitio: al entrar con el token bueno se
+ * deja una cookie, y a partir de ahi el navegador la manda sola en todo -la
+ * pagina, el socket, las imagenes-.
+ *
+ *  - SameSite=Strict: una web ajena no puede hacer que tu navegador la mande.
+ *    Es la misma defensa que el Origin, por otro camino.
+ *  - HttpOnly: no la lee ningun javascript, ni el nuestro.
+ *  - Sin Max-Age: se muere al cerrar el navegador, como la sesion.
+ */
+export const COOKIE_DE_SESION = "chocolatito_web";
+export function tokenDeLaCookie(cabecera) {
+    if (!cabecera)
+        return "";
+    for (const trozo of cabecera.split(";")) {
+        const igual = trozo.indexOf("=");
+        if (igual < 0)
+            continue;
+        if (trozo.slice(0, igual).trim() !== COOKIE_DE_SESION)
+            continue;
+        try {
+            return decodeURIComponent(trozo.slice(igual + 1).trim());
+        }
+        catch {
+            return "";
+        }
+    }
+    return "";
+}
 export function tokenDeLaPeticion(url) {
     if (!url)
         return "";
@@ -89,9 +184,16 @@ export async function iniciarServidorWeb(opciones = {}) {
     const sesion = sesionNueva();
     const clientes = new Set();
     const wss = new WebSocketServer({ noServer: true });
-    const autorizada = (req) => anfitrionAceptable(req.headers.host) &&
-        origenAceptable(req.headers.origin, req.headers.host) &&
-        mismoToken(tokenDeLaPeticion(req.url), token);
+    const tunel = nombreDeAnfitrion(opciones.anfitrionAutorizado);
+    const autorizada = (req) => {
+        if (!anfitrionAceptable(req.headers.host, tunel))
+            return false;
+        if (!origenAceptable(req.headers.origin, req.headers.host, tunel))
+            return false;
+        // El de la URL al entrar; el de la cookie en todo lo demas.
+        return (mismoToken(tokenDeLaPeticion(req.url), token) ||
+            mismoToken(tokenDeLaCookie(req.headers.cookie), token));
+    };
     const carpetaDeTrabajo = opciones.carpeta || (() => process.cwd());
     const servidor = http.createServer((req, res) => {
         const ruta = (req.url || "/").split("?")[0];
@@ -134,6 +236,8 @@ export async function iniciarServidorWeb(opciones = {}) {
         res.writeHead(200, {
             "content-type": "text/html; charset=utf-8",
             "cache-control": "no-store",
+            // Path=/ para que valga tambien en /ws, /archivo y /subir.
+            "set-cookie": `${COOKIE_DE_SESION}=${encodeURIComponent(token)}; Path=/; HttpOnly; SameSite=Strict`,
             // La pagina no tiene que salir de aqui ni meterse en ningun marco ajeno.
             "x-frame-options": "DENY",
             "referrer-policy": "no-referrer",
@@ -142,7 +246,7 @@ export async function iniciarServidorWeb(opciones = {}) {
             // esta misma casa. blob: para la vista previa de lo que aun no se ha
             // subido. Nada de terceros: la pagina sigue sin pedirle nada a nadie.
             "default-src 'none'; style-src 'unsafe-inline'; script-src 'unsafe-inline'; " +
-                "img-src 'self' blob: data:; media-src 'self' blob:; connect-src 'self' ws: wss:",
+                "img-src 'self' blob: data:; font-src 'self' data:; media-src 'self' blob:; connect-src 'self' ws: wss:",
         });
         res.end(paginaHtml());
     });
@@ -325,6 +429,11 @@ ${m.texto}` : aviso;
     fijarEmisorDePermiso((peticion) => aTodos({ t: "permiso", ...peticion }) > 0);
     // El agente se puso a esperar: para el movil, eso es "ya esta".
     fijarAvisoDeTurno(() => {
+        // Primero se suelta lo que quede a medias. La ultima linea de una respuesta
+        // casi nunca trae salto de linea -el modelo acaba con un punto-, asi que sin
+        // esto el final de CADA respuesta se quedaba esperando un salto que no llega
+        // nunca: justo la frase que el usuario esta esperando leer.
+        cerrarLineaWeb();
         aTodos({ t: "turno" });
     });
     const puerto = await new Promise((resolve, reject) => {

package/dist/ui/comandos.js

@@ -26,6 +26,7 @@
 export const SLASH_COMMANDS = [
     { name: "/mcp", description: "Ver servidores MCP conectados y sus herramientas" },
     { name: "/undo", description: "Deshacer las últimas modificaciones de archivos de la sesión" },
+    { name: "/rewind", description: "Volver a como estaba antes de un turno (sobrevive a cerrar la sesión)", args: "[n]" },
     { name: "/export", description: "Exportar bitácora de la sesión a archivo Markdown", args: "[nombre.md]" },
     { name: "/plan", description: "Alternar Modo Plan (solo lectura e investigación previa)" },
     { name: "/resume", description: "Reanudar una sesión anterior guardada", args: "[id]" },

package/extension/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "minimum_chrome_version": "102",
   "name": "Chocolatito Code",
-  "version": "1.6.14",
+  "version": "1.6.15",
   "description": "Deja que Chocolatito Code trabaje dentro de tu Chrome, en segundo plano y con tus sesiones ya iniciadas.",
   "permissions": [
     "tabs",

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "chocolatito-code",
-  "version": "1.6.14",
+  "version": "1.6.15",
   "description": "Agente autónomo de programación para la terminal, con control real del ordenador y del navegador. Desarrollado por Chocolatito.",
   "type": "module",
   "bin": {