fix(auth): decir cómo salir del prompt al que mandamos a la gente - #30
Merged
Merged
Conversation
Dos cosas que salieron revisando la #25 despues de mergearla. El mensaje del login dice que si el enlace no ha llegado vuelvas a lanzar el comando, y lo dice desde dentro de `Paste the link:`, que es un prompt que bloquea. La linea vacia que uno prueba para salir es la unica salida que no lo es: vuelve como `nothing pasted` y exit 1, que se lee como que el login ha fallado y no como que te has ido. Ahora dice ctrl-c, que es lo que hay. Y el test del panel afirmaba sobre "15 minutes" llamando a View sin fijar m.lay, asi que dependia del 80x24 por defecto y de donde m.wrapped partiera la linea: a 60 columnas sale "expires 15 / minutes" y el test falla con la pantalla correcta. Fija el tamano como hacen los demas tests del fichero y compara contra la pantalla con los saltos colapsados - que "15 minutes" sobreviva junto en una linea es cosa del renderer. Comprobado a 60, 80 y 120 columnas, y sigue fallando si LinkValidity cambia a 20 minutos. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
Merged
borjaperfra
added a commit
that referenced
this pull request
Sep 21, 2026
Entrega lo mergeado desde la 0.1.20: - #25 el enlace de login dice que vence, y fuera la pantalla que ya no se dibujaba - #27 el config.toml que Codex no abria, y un perfil por modelo con su ventana - #29 el tablero recoge tambien los PR de fuera - #30 como salir del prompt del enlace, y un test que no dependa del ancho del terminal Primer release que sale por si solo: desde la #28 el tag lo crea tag.yml al pasar CI en main, leyendo esta constante. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Los dos hallazgos de la revisión de #25, ya mergeada.
1. El consejo no se podía seguir
El mensaje del login dice que si el enlace no ha llegado vuelvas a lanzar el comando — y lo dice desde dentro de
Paste the link:, que es un prompt que bloquea. La línea vacía que uno prueba para salir es la única salida que no lo es: vuelve comonothing pastedy exit 1, que se lee como que el login ha fallado y no como que te has ido.Ahora dice
ctrl-c, que es lo que hay. No se toca el comportamiento de la línea vacía: sigue siendonothing pastedy exit 1, por las razones que #25 dejó escritas.2. El test dependía del ancho del terminal
La aserción sobre
"15 minutes"llamaba aView()sin fijarm.lay, así que se apoyaba en el 80x24 por defecto y en dóndem.wrappedpartiera la línea. A 60 columnas el renderer parteexpires 15 / minutesy el test falla con la pantalla correcta.Fija el tamaño como hacen los demás tests del fichero, y compara contra la pantalla con los saltos colapsados: que
"15 minutes"sobreviva junto en una línea es cosa del renderer, no lo que este test viene a vigilar.Comprobado
LinkValiditya 20 minutos lo hace fallar.go vetlimpio y la suite completa en verde.🤖 Generated with Claude Code