chocolatito-code 1.6.6 → 1.6.8

package/dist/ui/ink/transcripcion.js

@@ -0,0 +1,180 @@
+/**
+ * LA CONVERSACIÓN, EN CAMINO AL HISTORIAL DEL TERMINAL
+ *
+ * QUÉ SUSTITUYE, Y POR QUÉ
+ *
+ * Antes esto era `historial.ts`: un buffer en memoria con las últimas 5000
+ * líneas, que hacía falta porque la pantalla alternativa deja al terminal sin
+ * historial propio. Aquí no hay pantalla alternativa. Lo que se escribe entra en
+ * el historial DEL TERMINAL —el de verdad, el de la rueda del ratón— a través de
+ * `<Static>` de Ink, y esto es solo la cinta transportadora que lo lleva hasta
+ * allí.
+ *
+ * POR QUÉ HAY QUE PARTIR EN LÍNEAS
+ *
+ * `<Static>` emite un elemento y lo da por escrito para siempre: una vez fuera,
+ * ni Ink ni nadie puede volver a tocarlo, porque ya es texto del terminal. Así
+ * que solo puede entrar lo que está terminado. El modelo, en cambio, escribe a
+ * chorros que cortan a mitad de palabra.
+ *
+ * Aquí se separan las dos cosas: lo que ya tiene su salto de línea pasa a
+ * `<Static>` y se vuelve permanente; el trozo que cuelga se queda en `resto` y
+ * lo pinta la zona de abajo, que sí se repinta. Cuando llega el salto, el resto
+ * entero cruza al otro lado. Visto desde fuera es una línea que se va
+ * escribiendo, sin saltos ni parpadeos.
+ *
+ * LAS DOS REGLAS DE <STATIC>, QUE NO SON NEGOCIABLES
+ *
+ * Están medidas en `node_modules/ink/build/components/Static.js:11-20`, que es
+ * todo el mecanismo: guarda un índice, renderiza `items.slice(indice)` y en el
+ * commit siguiente adelanta el índice a `items.length`. No compara nada.
+ *
+ *   1. La lista SOLO crece. Un `splice`, un `filter` o un vaciado hacen retroceder
+ *      el índice, y todo lo que se añada por encima se REIMPRIME: la conversación
+ *      sale duplicada en el historial del terminal y ya no hay forma de borrarla.
+ *   2. Cada elemento necesita `key` propia y estable. React reutiliza nodos por
+ *      posición, y la lista que ve cambia de longitud en cada commit; sin `key`
+ *      acabas emitiendo el texto de otro. Por eso cada línea lleva su `id`.
+ *
+ * Y POR QUÉ EXISTE `base`
+ *
+ * Cuando el árbol se desmonta y se vuelve a montar —un diálogo de permisos que
+ * necesita el teclado para él solo— el `<Static>` nuevo empieza de cero y
+ * volvería a escribir la conversación entera. Pero en la pantalla normal esas
+ * líneas siguen ahí, en el terminal, que no se ha borrado. `base` marca por
+ * dónde iba, y el montaje nuevo solo recibe lo que aún no se ha pintado.
+ */
+/** Lo que se ha ido escribiendo, en orden. Solo se añade al final. */
+let lineas = [];
+/** Cuántas de esas ya están escritas en el terminal. Solo cambia al remontar. */
+let base = 0;
+/** El trozo sin terminar. Todavía no puede cruzar a <Static>. */
+let resto = "";
+let siguienteId = 1;
+const oyentes = [];
+function avisar() {
+    for (const cb of oyentes) {
+        try {
+            cb();
+        }
+        catch { }
+    }
+}
+/**
+ * Entra texto del agente. Acepta trozos a media línea, que es lo normal.
+ *
+ * El `\r` suelto se descarta con su línea: en un flujo de markdown no es un
+ * retorno de carro de verdad, viene de un `\r\n` partido por la mitad entre dos
+ * trozos, y dejarlo pasar mete una línea vacía por cada salto.
+ */
+export function anotar(texto) {
+    if (!texto)
+        return;
+    const completo = resto + String(texto).replace(/\r\n/g, "\n");
+    const partes = completo.split("\n");
+    resto = partes.pop() ?? "";
+    for (const l of partes) {
+        lineas.push({ id: siguienteId++, texto: l });
+    }
+    avisar();
+}
+/** Lo último que se entregó, para no rehacerlo en cada dibujado. */
+let entregado = [];
+let entregadoLargo = -1;
+let entregadoBase = -1;
+/**
+ * Las líneas que <Static> todavía no ha escrito.
+ *
+ * DEVUELVE UN ARRAY NUEVO CADA VEZ QUE HAY ALGO NUEVO, Y NO ES UN CAPRICHO
+ *
+ * Este fue el fallo que dejó la conversación entera sin escribirse, y costó
+ * encontrarlo porque no parecía un fallo: la primera versión devolvía el array
+ * interno tal cual, al que `anotar` le hace `push`.
+ *
+ * <Static> memoriza lo que va a emitir con `useMemo(() => items.slice(index),
+ * [items, index])` (Static.js:12-14). Con la MISMA referencia de array y el
+ * mismo índice, React da por buena la memoria y devuelve lo de antes: el `[]`
+ * del primer dibujado. El array crecía, la pantalla no. Nada de emitir de más
+ * ni de menos: sencillamente no se emitía NUNCA.
+ *
+ * Así que la referencia tiene que cambiar cuando cambia el contenido. Se
+ * reconstruye solo cuando ha crecido de verdad —una vez por línea terminada, no
+ * una por dibujado—, que a treinta fotogramas por segundo es la diferencia entre
+ * reconstruirlo treinta veces por segundo o ninguna mientras nadie escribe.
+ */
+export function pendientes() {
+    if (lineas.length !== entregadoLargo || base !== entregadoBase) {
+        entregado = lineas.slice(base);
+        entregadoLargo = lineas.length;
+        entregadoBase = base;
+    }
+    return entregado;
+}
+/** El trozo que cuelga, el que pinta la zona de abajo. */
+export function colgando() {
+    return resto;
+}
+/**
+ * "Lo de hasta aquí ya está en el terminal."
+ *
+ * Se llama SOLO justo antes de desmontar el árbol. Llamarlo con el árbol vivo
+ * encogería la lista que ve <Static>, que es la regla 1 de arriba.
+ */
+export function darPorPintado() {
+    base = lineas.length;
+}
+export function alCambiar(cb) {
+    oyentes.push(cb);
+    return () => {
+        const i = oyentes.indexOf(cb);
+        if (i >= 0)
+            oyentes.splice(i, 1);
+    };
+}
+/** Para las pruebas. En producción la conversación no se tira nunca. */
+export function vaciarTranscripcion() {
+    lineas = [];
+    base = 0;
+    resto = "";
+    siguienteId = 1;
+    entregado = [];
+    entregadoLargo = -1;
+    entregadoBase = -1;
+}
+/** Cuántas líneas terminadas van. Para las pruebas y para medir. */
+export function cuantasLineas() {
+    return lineas.length;
+}
+const SOLO_COLOR = /\x1b\[[0-9;]*m/g;
+/**
+ * Recorta el trozo que cuelga para que quepa en las filas que le tocan.
+ *
+ * NO ES COSMÉTICO, ES LA RED DE SEGURIDAD DEL RENDIMIENTO
+ *
+ * En Windows, si el bloque de abajo llega a medir lo que la ventana, Ink deja de
+ * repintar por diferencias y pasa a `clearTerminal` + reimprimir la conversación
+ * ENTERA en cada fotograma (ink.js:100-102 y 769). Con el spinner a 30 por
+ * segundo eso es parpadeo, historial perdido y coste creciente con la sesión.
+ *
+ * El caso que lo dispara: un párrafo largo sin un solo salto de línea. Mientras
+ * no lo termine, ese párrafo entero es "lo que cuelga".
+ *
+ * Cuando hay que recortar se quitan también los colores: cortar una cadena por
+ * el medio parte los códigos de escape y lo que sale es basura de colores
+ * enganchada al resto de la sesión. Es un trozo que en cuanto llegue su salto de
+ * línea se reemplaza por la línea entera y bien pintada, así que el recorte no
+ * dura ni un parpadeo.
+ */
+export function recortarCola(texto, cols, filas) {
+    if (!texto)
+        return [];
+    const ancho = Math.max(1, cols);
+    const techo = Math.max(0, filas);
+    if (techo === 0)
+        return [];
+    const visible = texto.replace(SOLO_COLOR, "");
+    if (Math.ceil(visible.length / ancho) <= techo)
+        return [texto];
+    const pelado = visible.slice(-(ancho * techo));
+    return [pelado];
+}

package/dist/ui/interrupt.js

@@ -177,7 +177,10 @@ export class InterruptWatcher {
                 }
                 else {
                     this.stop();
-                    process.stdout.write("\n");
+                    // `dibujar` y no `process.stdout.write`: con el arbol montado, una
+                    // escritura cruda es una fila que Ink no cuenta, y el borrado del marco
+                    // al salir se come una linea de la conversacion.
+                    dibujar("\n");
                     process.exit(0);
                 }
                 return;

package/dist/ui/loopPrompt.d.ts

@@ -5,13 +5,12 @@ export interface RespuestaDeBucle {
     indicacion?: string;
 }
 /**
- * Que hacer cuando el agente lleva tres vueltas haciendo exactamente lo mismo.
+ * Se pide el terminal prestado antes de abrir esto.
  *
- * Sin terminal —CI, `chocolatito "tarea"`, un cron— se PARA. Es lo contrario de
- * lo que hace el dialogo de permisos, que ante la duda deniega para no tocar
- * nada, y el motivo es que aqui el riesgo es el opuesto: no hay nada que
- * proteger del agente, hay una cuota que proteger de un bucle que nadie esta
- * mirando. Parar no mata el turno: le devuelve al modelo un mensaje explicando
- * el atasco, y el turno sigue.
+ * Con el árbol de Ink montado hay un dueño de la pantalla, y esta función es de
+ * las que necesitan el teclado para ellas solas: escribe directo a stdout y abre
+ * su propia lectura de teclas. Sin el préstamo, las dos cuentas de filas se pisan
+ * y el marco de Ink no se recupera. Sin árbol, `conElTerminalPrestado` ejecuta la
+ * tarea tal cual y no cuesta nada. Ver ui/ink/prestamo.ts.
  */
 export declare function preguntarPorBucle(aviso: AvisoDeBucle): Promise<RespuestaDeBucle>;

package/dist/ui/loopPrompt.js

@@ -1,6 +1,7 @@
 import readline from "node:readline";
 import process from "node:process";
 import chalk from "chalk";
+import { conElTerminalPrestado } from "./ink/prestamo.js";
 /**
  * Que hacer cuando el agente lleva tres vueltas haciendo exactamente lo mismo.
  *
@@ -11,7 +12,7 @@ import chalk from "chalk";
  * mirando. Parar no mata el turno: le devuelve al modelo un mensaje explicando
  * el atasco, y el turno sigue.
  */
-export async function preguntarPorBucle(aviso) {
+async function preguntarPorBucleDirecto(aviso) {
     const cabecera = chalk.yellow(`\n↻  Parece un bucle: `) +
         chalk.white(`"${aviso.herramienta}"`) +
         chalk.yellow(` lleva ${aviso.repeticiones} llamadas identicas con el mismo resultado.`);
@@ -57,3 +58,15 @@ export async function preguntarPorBucle(aviso) {
         });
     });
 }
+/**
+ * Se pide el terminal prestado antes de abrir esto.
+ *
+ * Con el árbol de Ink montado hay un dueño de la pantalla, y esta función es de
+ * las que necesitan el teclado para ellas solas: escribe directo a stdout y abre
+ * su propia lectura de teclas. Sin el préstamo, las dos cuentas de filas se pisan
+ * y el marco de Ink no se recupera. Sin árbol, `conElTerminalPrestado` ejecuta la
+ * tarea tal cual y no cuesta nada. Ver ui/ink/prestamo.ts.
+ */
+export function preguntarPorBucle(aviso) {
+    return conElTerminalPrestado(() => preguntarPorBucleDirecto(aviso));
+}

package/dist/ui/marco.d.ts

@@ -13,6 +13,7 @@ export interface PromptFrameOptions {
     cursorIndex?: number;
     items?: DropdownItem[];
     selectedIndex?: number;
+    taskWidget?: string;
 }
 export interface PromptFrame {
     /** Lineas LOGICAS. Cualquiera puede ocupar mas de una fila fisica. */
@@ -32,4 +33,6 @@ export interface PromptFrame {
  * aritmetica de filas se hace UNA vez sobre las mismas lineas que se pintan, en
  * lugar de estimarse por otro lado y descuadrarse.
  */
+/** Cuantas sugerencias se ensenan como mucho. Ver la nota en el desplegable. */
+export declare const MAXIMO_DEL_DESPLEGABLE = 8;
 export declare function buildPromptFrame(opts: PromptFrameOptions): PromptFrame;

package/dist/ui/marco.js

@@ -55,6 +55,8 @@ function truncatePlain(text, width) {
  * aritmetica de filas se hace UNA vez sobre las mismas lineas que se pintan, en
  * lugar de estimarse por otro lado y descuadrarse.
  */
+/** Cuantas sugerencias se ensenan como mucho. Ver la nota en el desplegable. */
+export const MAXIMO_DEL_DESPLEGABLE = 8;
 export function buildPromptFrame(opts) {
     const cols = Math.max(20, opts.cols || 80);
     const viewportRows = Math.max(6, opts.rows || 24);
@@ -95,7 +97,22 @@ export function buildPromptFrame(opts) {
     // Sin contarla, el desplegable pide una fila mas de las que caben y el marco
     // se sale por arriba de la ventana.
     const room = Math.max(1, viewportRows - 1 - (2 + 1 + inputRows + 1 + 1));
-    const shown = Math.min(items.length, room);
+    // Y ademas un tope propio, mas bajo que lo que cabe.
+    //
+    // POR QUE NO SE DEJA CRECER HASTA LLENAR LA VENTANA
+    //
+    // Con solo "/" hay dieciocho comandos. En una consola de 24 filas eso hacia un
+    // marco de veinte, o sea que al escribir UNA barra la caja se comia la pantalla
+    // entera y tapaba la conversacion; y a la letra siguiente se encogia a nueve.
+    // Cada tecla movia veinte filas arriba y abajo.
+    //
+    // Eso es feo, y ademas es el terreno donde aparecen los fallos de repintado: un
+    // marco que crece hasta el borde obliga al terminal a desplazar, y a partir de
+    // ahi Ink borra contando filas que ya no son suyas.
+    //
+    // Ocho es lo que se ve de un vistazo sin buscar. Y el desplegable filtra segun
+    // escribes, asi que llegar al que quieres son dos letras, no dieciocho lineas.
+    const shown = Math.min(items.length, room, MAXIMO_DEL_DESPLEGABLE);
     const dropdownOffset = shown > 0 && items.length > shown
         ? Math.min(Math.max(0, selectedIndex - Math.floor(shown / 2)), items.length - shown)
         : 0;
@@ -197,7 +214,9 @@ export function buildPromptFrame(opts) {
     // Dos, no una. Con una, la caja seguia leyendose pegada a lo de arriba: "pusiste
     // una linea de espacio, que sea bien pues". Con dos se separa de verdad, y es
     // el mismo aire que hay al arrancar entre el zorro y el aviso.
-    const lines = ["", "", bar, inputLine, ...dropdownLines, bar, footerLine];
+    const widgetLines = opts.taskWidget && opts.taskWidget.trim().length > 0 ? [opts.taskWidget] : [];
+    const widgetRows = widgetLines.reduce((n, l) => n + physicalRows(l, cols), 0);
+    const lines = ["", "", ...widgetLines, bar, inputLine, ...dropdownLines, bar, footerLine];
     // 4. Donde queda el cursor, en filas fisicas. Antes se hacia cursorCol =
     // total % cols: aplicaba el modulo pero nunca sumaba la fila, asi que en
     // cuanto la entrada se partia el acento se iba a escribir a otra linea.
@@ -205,7 +224,7 @@ export function buildPromptFrame(opts) {
     const segments = typed.split("\n");
     // La fila de aire de arriba cuenta: sin sumarla, el cursor se dibuja una fila
     // por encima de donde se escribe.
-    let cursorRow = 2 + physicalRows(bar, cols);
+    let cursorRow = 2 + widgetRows + physicalRows(bar, cols);
     for (let i = 0; i < segments.length - 1; i++) {
         cursorRow += Math.max(1, Math.ceil(((i === 0 ? symbolWidth : 0) + segments[i].length) / cols));
     }

package/dist/ui/pantalla.d.ts

@@ -1,30 +1,58 @@
 /**
- * Fija SOLO SI SE PIDE. Van dos veces que la pongo de serie y se rompe.
+ * De serie SÍ, desde la 1.6.7, y con salida de emergencia.
  *
- * La segunda: "ahora sale una barra y lo demas negro, escribo y se cierra".
+ * DOS VECES ESTUVO AQUÍ Y DOS VECES SALIÓ ROTO
  *
- * La regla que saco de esto, y que no pienso volver a saltarme: esto no vuelve
- * a ser lo normal hasta que el dueño confirme que lo ha visto funcionar en SU
- * consola con la variable puesta. Un emulador -por bueno que sea, y el de
- * tests/terminal.test.mjs lo es- prueba lo que sabe emular. Ha hecho falta que
- * falle dos veces para aprenderlo.
+ * La 1.6.0 salió en negro ("no sale nada, se bugea") y la 1.6.5 con "una barra y
+ * lo demás negro, escribo y se cierra". Las dos veces pasaban las 820 pruebas:
+ * un emulador prueba lo que sabe emular, y una consola de verdad hace cosas que
+ * no sabe.
+ *
+ * Esta vez el orden fue el otro: se entregó detrás de la variable, el dueño lo
+ * corrió en SU consola, y solo después pasó a ser lo normal. Lo hizo, y lo
+ * aprobó sabiendo lo que no le iba a gustar: que la caja no se queda clavada al
+ * hacer scroll. No se queda porque es texto del terminal, y en Claude Code
+ * tampoco se queda. Es el precio de recuperar la rueda del ratón y el historial.
+ *
+ * CHOCOLATITO_PANTALLA=normal devuelve la interfaz clásica. Existe porque una
+ * consola rara —una que no sea la suya ni la mía— puede seguir haciendo algo que
+ * no habíamos visto, y entonces el usuario necesita poder trabajar hoy, no
+ * esperar al parche.
  */
 export declare function pantallaFijaPedida(): boolean;
+export declare function forzarPantallaFija(valor: boolean | null): void;
+export declare function enPantallaFija(): boolean;
 /**
- * Enciende o apaga el modo a mano, SIN tocar el terminal.
+ * Enciende el modo de árbol único. Devuelve si de verdad se encendió.
+ *
+ * EL BOOLEANO NO SOBRA, Y SE VOLVIÓ IMPRESCINDIBLE AL PONERLO POR DEFECTO
+ *
+ * Sin terminal —una salida redirigida a un archivo, un `chocolatito "tarea"` en
+ * un script, CI— aquí no se entra. Pero quien llama montaba el árbol de todas
+ * formas, y Ink sin terminal escribe su marco como texto suelto: la salida del
+ * programa sale con la caja de escribir dentro.
  *
- * Las pruebas lo necesitan: entrar de verdad escribiría la secuencia de la
- * pantalla alternativa en la consola de quien corre las pruebas y le dejaría el
- * informe en una pantalla que desaparece al acabar.
+ * Mientras esto iba detrás de una variable no pasaba nunca, porque nadie la pone
+ * en un script. De serie, es el caso normal.
  */
-export declare function forzarPantallaFija(valor: boolean | null): void;
-export declare function enPantallaFija(): boolean;
-export declare function entrarEnPantallaFija(): void;
+export declare function entrarEnPantallaFija(): boolean;
+/**
+ * Manda `console` a la transcripción.
+ *
+ * Se exporta para poder probarlo: `entrarEnPantallaFija` no se puede llamar en
+ * las pruebas porque exige un terminal de verdad, y un parche de consola sin
+ * probar es de las cosas que fallan en silencio —lo que se pierde no se ve, que
+ * es justo el problema—.
+ */
+export declare function interceptarConsola(): void;
+export declare function devolverConsola(): void;
 /**
  * Devuelve el terminal como estaba.
  *
- * Se llama al salir del programa, y también si algo revienta: quedarse en la
- * pantalla alternativa deja al usuario en una consola en blanco sin su prompt,
- * y la única salida es cerrar la ventana.
+ * Aquí ya no hay ninguna pantalla que abandonar —esa era la versión de antes—,
+ * pero sí hay que devolver el conducto de dibujo y el pie: si el proceso sigue
+ * vivo después de esto (una prueba, una sesión que cambia de modo) y el dibujo
+ * se queda desviado, todo lo que se escriba va a una transcripción que ya no
+ * pinta nadie.
  */
 export declare function salirDePantallaFija(): void;