Skip to content

fix(tui): el paso 3 del setup vuelve a tener campo para la API key - #38

Merged
borjaperfra merged 1 commit into
mainfrom
fix/tui-setup-key-field
Oct 5, 2026
Merged

borjaperfra merged 1 commit into
mainfrom
fix/tui-setup-key-field

Conversation

@borjaperfra

Copy link
Copy Markdown
Contributor

Reportado: "al intentar meter la api key no hay espacio para ponerla en el setup". Dos causas, las dos en el paso 3 del setup guiado:

  • Terminal bajo (menos de unas 19 filas, lo normal en el terminal del IDE): el banner iba siempre arriba y View recorta lo que no cabe por abajo, así que lo que se perdía era el campo de la key. Ahora renderWizard solo pone el banner si deja sitio: primero se quita la mascota y después el logo. El paso en el que estás siempre se ve.
  • Key rechazada por el cluster: el campo se cerraba al pulsar enter y no volvía a abrirse, así que quedaba el error sin sitio para pegar otra key. El pie seguía diciendo "enter to continue", pero enter no hacía nada. Ahora el campo se reabre vacío debajo del error, y enter en el paso 3 también lo abre, salvo mientras se está comprobando la key.

Pruebas

  • Dos tests nuevos en config_test.go: el campo sale a 80x14, 80x18, 120x22 y 120x30, y una key rechazada deja el campo abierto. Los dos fallan con el código anterior.
  • Probado en el binario real (Linux, tmux en WSL), con un cambio local no incluido en el PR que arranca el wizard en el paso 3 para no depender de un magic link:
    • antes, a 80x16: el paso 3 sale sin campo;
    • ahora, a 80x12, 80x16 y 120x30: el campo sale; al pegar una key falsa aparece the cluster refused this key con el campo abierto debajo, y acepta lo que se teclea.
  • go vet limpio y la suite completa en verde.

🤖 Generated with Claude Code

En un terminal bajo, menos de unas 19 filas, el banner iba arriba y View
cortaba por abajo lo que no cabia: lo que se caia era el campo de la key.
Ahora el banner solo entra si deja sitio, primero sin la mascota y luego
sin el logo.

Y si el cluster rechazaba la key, el campo se quedaba cerrado bajo el
error; el pie decia "enter to continue" y enter no hacia nada. Ahora se
vuelve a abrir, y enter en el paso 3 tambien lo abre.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@borjaperfra
borjaperfra merged commit 89413cd into main Oct 5, 2026
6 checks passed
@borjaperfra
borjaperfra deleted the fix/tui-setup-key-field branch October 5, 2026 11:58
@borjaperfra borjaperfra mentioned this pull request Oct 5, 2026
borjaperfra added a commit that referenced this pull request Oct 5, 2026
Entrega lo mergeado desde la 0.1.22:

- #38 el paso 3 del setup vuelve a tener campo para la API key, en
  terminales bajos y despues de que el cluster rechace una key

Co-authored-by: borjaperfra <borjaperfra@users.noreply.github.com>
Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants