Skip to content

[ONB-2057] Wire browser flow into login command#50

Merged
FranciscaOtero merged 4 commits into
mainfrom
onb-2057-login-browser-flow
Jun 3, 2026
Merged

[ONB-2057] Wire browser flow into login command#50
FranciscaOtero merged 4 commits into
mainfrom
onb-2057-login-browser-flow

Conversation

@FranciscaOtero

@FranciscaOtero FranciscaOtero commented Jun 2, 2026

Copy link
Copy Markdown
Collaborator

Contexto

Ya teníamos la librería browser-login lista del PR anterior. Faltaba conectarla al comando login y dejar la metadata de la key visible en config.

Que hay de nuevo?

  • login ahora abre el browser para autenticar, con paste manual como fallback si el usuario prefiere pegar la key mientras espera
  • Flags nuevos --mode test|live (default test) y -y/--yes para skipear la confirmación de re-login
  • config muestra key_name y expires_at cuando la key viene del archivo, tanto en texto como en --json
  • FintocConfig extendido con key_name? y expires_at?, y resolveAuth los propaga

UX

  • fintoc login: abre el browser en modo test, muestra el link por si no abre solo y queda esperando el callback. En paralelo acepta paste manual, gana el primero.
  • fintoc login --mode live: igual pero en modo live, además persiste key_name y expires_at.
  • fintoc login con sesión activa: pide confirmación antes de continuar. Con --yes la skipea.
  • fintoc login --api-key sk_...: salta el browser y guarda la key directo (también es el camino para CI/non-TTY).
  • Cancelación, timeout, mode mismatch o denied muestran un hint con el comando para reintentar.

Tests

  • Tests unitarios
  • Pruebas en local con el dashboard

Created with Claude Code /create-pr command

@FranciscaOtero FranciscaOtero self-assigned this Jun 2, 2026
@FranciscaOtero FranciscaOtero force-pushed the onb-2057-login-browser-flow branch 5 times, most recently from 15c9c50 to d1a8805 Compare June 2, 2026 21:58
@FranciscaOtero FranciscaOtero marked this pull request as ready for review June 2, 2026 22:04
@FranciscaOtero FranciscaOtero requested review from daleal and tlarrain June 2, 2026 22:04

@daleal daleal left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Lo veo bien, dejo sugerencias menores

Comment thread src/commands/login.ts Outdated
Comment thread src/lib/auth.ts Outdated
@FranciscaOtero FranciscaOtero force-pushed the onb-2057-login-browser-flow branch from d1a8805 to 32fb930 Compare June 3, 2026 14:42
@FranciscaOtero FranciscaOtero merged commit 2df37a0 into main Jun 3, 2026
2 checks passed
@FranciscaOtero FranciscaOtero deleted the onb-2057-login-browser-flow branch June 3, 2026 14:44
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