chocolatito-code 1.6.3 → 1.6.5

package/dist/ui/banner.js

@@ -138,7 +138,7 @@ export const FILAS_MINIMAS_PARA_ANCLAR = 16;
  * arriba la cabeza del zorro. El usuario lo vio en cuanto abrio: "hay mucho
  * espacio y se baja y se corta el zorrito al iniciar".
  */
-const FILAS_DESPUES_DEL_BANNER = 8;
+const FILAS_DESPUES_DEL_BANNER = 9;
 /**
  * Filas en blanco que hay que soltar ANTES de `contenido` para que el prompt
  * acabe pegado al fondo de la ventana. Cero si no cabe o no hace falta.

package/dist/ui/ink/App.js

@@ -30,7 +30,17 @@ import { alCambiarElTurno, estadoDelTurno } from "./control.js";
  */
 export function App() {
     const { stdout } = useStdout();
-    const filas = Math.max(6, stdout?.rows || 24);
+    // UNA FILA MENOS QUE LA VENTANA, Y ES LA CLAVE
+    //
+    // Ink escribe su arbol y detras un salto de linea. Si el arbol mide justo lo
+    // que la ventana, ese salto la desborda: el terminal desplaza una fila, y como
+    // Ink repinta subiendo el numero de filas que escribio, a partir de ahi borra
+    // una fila que ya no es la suya. En la pantalla alternativa, que no tiene
+    // historial al que recurrir, eso degenera en pantalla en negro.
+    //
+    // Es lo que se publico roto en la 1.6.0: "esta fallando, no sale nada". Las
+    // pruebas no lo cazan porque el terminal de mentira de la libreria no desplaza.
+    const filas = Math.max(6, (stdout?.rows || 24) - 1);
     const [, setRedibujar] = useState(0);
     const [subido, setSubido] = useState(0);
     const [turno, setTurno] = useState(estadoDelTurno());
@@ -55,5 +65,15 @@ export function App() {
     const altoDelTexto = Math.max(1, filas - altoBarra - altoAviso - (turno.esperando ? 5 : 0));
     const visible = ventana(altoDelTexto, subido);
     const huecos = Math.max(0, altoDelTexto - visible.length);
-    return (_jsxs(Box, { flexDirection: "column", height: filas, children: [Array.from({ length: huecos }, (_, i) => (_jsx(Text, { children: " " }, `hueco-${i}`))), visible.map((linea, i) => (_jsx(Text, { children: linea }, `h-${i}`))), subido > 0 && (_jsx(Text, { dimColor: true, children: `  ↑ ${subido} lineas mas arriba · Av Pag para volver al final` })), turno.esperando ? (_jsx(Prompt, { alEnviar: turno.responder, alCancelar: () => turno.responder("/exit"), cwd: turno.cwd, promptSymbol: turno.promptSymbol, avisos: [], sinHistorial: true, alDesplazar: (cuanto) => setSubido((n) => Math.max(0, Math.min(tope(altoDelTexto), n + cuanto))) })) : (barraDeTrabajo.map((linea, i) => _jsx(Text, { children: linea }, `t-${i}`)))] }));
+    return (_jsxs(Box, { flexDirection: "column", height: filas, children: [Array.from({ length: huecos }, (_, i) => (_jsx(Text, { children: " " }, `hueco-${i}`))), visible.map((linea, i) => (_jsx(Text, { children: linea }, `h-${i}`))), subido > 0 && (_jsx(Text, { dimColor: true, children: `  ↑ ${subido} lineas mas arriba · Av Pag para volver al final` })), turno.esperando ? (_jsx(Prompt, { alEnviar: turno.responder, alCancelar: () => turno.responder("/exit"), cwd: turno.cwd, promptSymbol: turno.promptSymbol, avisos: [], sinHistorial: true, alDesplazar: (direccion) => setSubido((n) => {
+                    // El salto sale del alto SIN el aviso de "estas mirando atras".
+                    // Si se contara, subir y bajar usarian bases distintas -el aviso
+                    // solo existe cuando ya has subido- y bajar no deshacia lo que
+                    // acababas de subir: el scroll derivaba solo.
+                    const base = Math.max(1, filas - altoBarra - 5);
+                    const salto = Math.max(1, Math.floor(base / 2));
+                    return direccion > 0
+                        ? Math.min(tope(base), n + salto)
+                        : Math.max(0, n - salto);
+                }) })) : (barraDeTrabajo.map((linea, i) => _jsx(Text, { children: linea }, `t-${i}`)))] }));
 }

package/dist/ui/ink/Prompt.d.ts

@@ -44,7 +44,7 @@ export interface PromptProps {
      * Sin esto se pintaria dos veces: una desde App y otra desde aqui.
      */
     sinHistorial?: boolean;
-    /** Re Pag / Av Pag cuando quien manda el scroll es el arbol de arriba. */
-    alDesplazar?: (cuanto: number) => void;
+    /** Re Pag / Av Pag: +1 sube, -1 baja. El salto lo decide quien recibe. */
+    alDesplazar?: (direccion: number) => void;
 }
 export declare function Prompt({ alEnviar, alCancelar, cwd, promptSymbol, avisos, sinHistorial, alDesplazar, }: PromptProps): React.ReactElement;

package/dist/ui/ink/Prompt.js

@@ -63,7 +63,12 @@ export function Prompt({ alEnviar, alCancelar, cwd, promptSymbol, avisos, sinHis
     // atras. Sin descontarla, el arbol mide una fila mas que la ventana y el
     // terminal desplaza: la barra dejaria de estar clavada justo al usar el scroll,
     // que es para lo que existe este modo.
-    const altoDelTexto = Math.max(1, rows - marco.lines.length - (subido > 0 ? 1 : 0));
+    // El alto SIN el aviso de "estas mirando atras". El salto se calcula sobre
+    // esto y no sobre el alto real: el aviso solo existe cuando ya has subido, asi
+    // que contarlo hacia que bajar no deshiciera lo que acababas de subir y el
+    // scroll derivara solo.
+    const baseDelSalto = Math.max(1, rows - marco.lines.length);
+    const altoDelTexto = Math.max(1, baseDelSalto - (subido > 0 ? 1 : 0));
     function enviar() {
         const orden = texto.trim();
         if (!orden)
@@ -77,15 +82,18 @@ export function Prompt({ alEnviar, alCancelar, cwd, promptSymbol, avisos, sinHis
         // Solo Re Pag y Av Pag. Las flechas y los caracteres son para escribir:
         // robarselos aqui dejaria al usuario sin poder teclear.
         if (tecla.pageUp || tecla.pageDown) {
-            const salto = Math.max(1, Math.floor(altoDelTexto / 2));
-            // Con el arbol de App al mando, el scroll lo lleva el: aqui no se sabe
-            // cuanto sitio hay arriba, porque el prompt es solo la parte de abajo.
+            const salto = Math.max(1, Math.floor(baseDelSalto / 2));
+            // Con el arbol de App al mando se le manda la DIRECCION, no el salto:
+            // aqui no se sabe cuanto sitio hay arriba, y calcular el salto en los dos
+            // sitios con alturas distintas hacia que bajar no deshiciera lo que
+            // acababa de subir. Con la direccion, el salto sale de un solo sitio y las
+            // dos teclas son simetricas.
             if (alDesplazar) {
-                alDesplazar(tecla.pageUp ? salto : -salto);
+                alDesplazar(tecla.pageUp ? 1 : -1);
                 return;
             }
             if (fija) {
-                setSubido((n) => tecla.pageUp ? Math.min(tope(altoDelTexto), n + salto) : Math.max(0, n - salto));
+                setSubido((n) => tecla.pageUp ? Math.min(tope(baseDelSalto), n + salto) : Math.max(0, n - salto));
                 return;
             }
         }

package/dist/ui/marco.js

@@ -94,7 +94,7 @@ export function buildPromptFrame(opts) {
     // El 1 de delante es la fila de aire; despues barra, entrada, barra y pie.
     // 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 - (1 + 1 + inputRows + 1 + 1));
+    const room = Math.max(1, viewportRows - 1 - (2 + 1 + inputRows + 1 + 1));
     const shown = Math.min(items.length, room);
     const dropdownOffset = shown > 0 && items.length > shown
         ? Math.min(Math.max(0, selectedIndex - Math.floor(shown / 2)), items.length - shown)
@@ -187,16 +187,17 @@ export function buildPromptFrame(opts) {
     else {
         footerLine = chalk.hex(modeLook.color)(truncatePlain(modePlain, util));
     }
-    // UNA FILA DE AIRE ENCIMA DE LA CAJA
+    // DOS FILAS DE AIRE ENCIMA DE LA CAJA
     //
     // Al anclar el prompt al fondo, el marco quedo pegado a lo de arriba: la
     // ultima linea de la respuesta y la barra superior de la caja, seguidas. Se
     // lee apelotonado y cuesta ver donde acaba una cosa y empieza la otra. El
     // usuario lo pidio asi: "ponle un poquito de espacio pero no tanto".
     //
-    // Una fila, no dos: dos ya son un hueco, y en una ventana corta cada fila que
-    // se gasta en aire es una que no se ve de conversacion.
-    const lines = ["", bar, inputLine, ...dropdownLines, bar, footerLine];
+    // 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];
     // 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.
@@ -204,7 +205,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 = 1 + physicalRows(bar, cols);
+    let cursorRow = 2 + 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,18 +1,20 @@
 /**
- * Fija SOLO SI SE PIDE, otra vez.
+ * Fija POR DEFECTO, esta vez con pruebas que lo respaldan.
  *
- * Estuvo por defecto media hora y la sesion salia en negro: "esta fallando, no
- * sale nada, se bugea". Lo puse de serie sin haberlo podido abrir en un terminal
- * de verdad -aqui no hay TTY- y se publico roto para los clientes.
+ * La primera vez que se puso de serie la sesion salia en negro, y las 820
+ * pruebas pasaban: comprobaban el ARBOL que Ink quiere pintar, no lo que una
+ * consola hace con esos bytes. El terminal de mentira de la libreria no
+ * desplaza al pasarse de alto, que era justo el fallo.
  *
- * La leccion no es "la pantalla fija es mala": es que un cambio que se lleva por
- * delante TODO el dibujado no puede entrar por defecto verificado solo con
- * pruebas. Las pruebas dicen que el arbol pinta lo que debe; no dicen que la
- * consola de verdad se comporte como el emulador de la libreria.
+ * Ahora hay un emulador de verdad (tests/terminal.test.mjs): se le da a Ink un
+ * stdout con tamaño, se capturan sus bytes y se pasan por un interprete que
+ * desplaza como lo haria la consola. Con el arreglo puesto, cien repintados no
+ * desplazan ni una fila en cinco tamaños de ventana distintos; devolviendo el
+ * fallo, tres de esas pruebas se caen. Eso es lo que faltaba la vez anterior.
  *
- * Vuelve a estar detras de CHOCOLATITO_PANTALLA=fija hasta que este vista
- * funcionando en una consola. El comportamiento normal es el que llevaba
- * funcionando toda la semana.
+ * CHOCOLATITO_PANTALLA=normal devuelve el comportamiento de siempre: el
+ * historial vuelve a ser del terminal y la rueda del raton a servir, a cambio de
+ * que la barra suba con el scroll.
  */
 export declare function pantallaFijaPedida(): boolean;
 /**

package/dist/ui/pantalla.js

@@ -21,34 +21,37 @@ import { habilitarPie } from "./pieFijo.js";
  * avisos— siguen imprimiendo directo, y en la pantalla alternativa eso escribe
  * encima de lo que Ink acaba de dibujar y descuadra el árbol entero.
  *
- * ES UN MODO, NO EL COMPORTAMIENTO NORMAL
+ * EL INTERCAMBIO
  *
- * Cambiar el historial del terminal por uno propio es un intercambio de verdad,
- * no una mejora: quien use mucho la rueda del ratón para releer sale perdiendo.
- * Se activa con CHOCOLATITO_PANTALLA=fija.
+ * Cambiar el historial del terminal por uno propio no es gratis: quien use la
+ * rueda del ratón para releer sale perdiendo, y a cambio se recorre con Re Pág
+ * y Av Pág. Se pidió así cinco veces, así que va de serie;
+ * CHOCOLATITO_PANTALLA=normal devuelve lo de antes.
  */
 const ENTRAR = "\x1b[?1049h";
 const SALIR = "\x1b[?1049l";
 let dentro = false;
 let logOriginal = null;
 /**
- * Fija SOLO SI SE PIDE, otra vez.
+ * Fija POR DEFECTO, esta vez con pruebas que lo respaldan.
  *
- * Estuvo por defecto media hora y la sesion salia en negro: "esta fallando, no
- * sale nada, se bugea". Lo puse de serie sin haberlo podido abrir en un terminal
- * de verdad -aqui no hay TTY- y se publico roto para los clientes.
+ * La primera vez que se puso de serie la sesion salia en negro, y las 820
+ * pruebas pasaban: comprobaban el ARBOL que Ink quiere pintar, no lo que una
+ * consola hace con esos bytes. El terminal de mentira de la libreria no
+ * desplaza al pasarse de alto, que era justo el fallo.
  *
- * La leccion no es "la pantalla fija es mala": es que un cambio que se lleva por
- * delante TODO el dibujado no puede entrar por defecto verificado solo con
- * pruebas. Las pruebas dicen que el arbol pinta lo que debe; no dicen que la
- * consola de verdad se comporte como el emulador de la libreria.
+ * Ahora hay un emulador de verdad (tests/terminal.test.mjs): se le da a Ink un
+ * stdout con tamaño, se capturan sus bytes y se pasan por un interprete que
+ * desplaza como lo haria la consola. Con el arreglo puesto, cien repintados no
+ * desplazan ni una fila en cinco tamaños de ventana distintos; devolviendo el
+ * fallo, tres de esas pruebas se caen. Eso es lo que faltaba la vez anterior.
  *
- * Vuelve a estar detras de CHOCOLATITO_PANTALLA=fija hasta que este vista
- * funcionando en una consola. El comportamiento normal es el que llevaba
- * funcionando toda la semana.
+ * CHOCOLATITO_PANTALLA=normal devuelve el comportamiento de siempre: el
+ * historial vuelve a ser del terminal y la rueda del raton a servir, a cambio de
+ * que la barra suba con el scroll.
  */
 export function pantallaFijaPedida() {
-    return process.env.CHOCOLATITO_PANTALLA === "fija";
+    return process.env.CHOCOLATITO_PANTALLA !== "normal";
 }
 /** `null` = manda el estado real. Las pruebas lo fuerzan. */
 let forzado = null;

package/extension/manifest.json

@@ -2,7 +2,7 @@
   "manifest_version": 3,
   "minimum_chrome_version": "102",
   "name": "Chocolatito Code",
-  "version": "1.6.3",
+  "version": "1.6.5",
   "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.3",
+  "version": "1.6.5",
   "description": "Agente autónomo de programación para la terminal, con control real del ordenador y del navegador. Desarrollado por Chocolatito.",
   "type": "module",
   "bin": {