Skip to content

feat(providers): guiar la conexión de cuentas adicionales - #47

Merged
egdev6 merged 2 commits into
mainfrom
feat/new-account-flow
Oct 10, 2026
Merged

egdev6 merged 2 commits into
mainfrom
feat/new-account-flow

Conversation

@egdev6

@egdev6 egdev6 commented Oct 10, 2026

Copy link
Copy Markdown
Owner

Issue vinculada

Closes #46

Tipo

  • Nueva funcionalidad (type:feature).

Resumen

  • Guía de tres pasos para nombrar y conectar cuentas adicionales CLI, con carpeta personalizada en Avanzado.
  • «Guardar e iniciar sesión» exige persistencia confirmada: no abre el CLI tras un fallo o un guardado concurrente ni oculta el error original. Un fallo independiente de inicio con Windows no bloquea una configuración ya persistida.
  • Detección periódica del fichero de credenciales con indicador neutral: su presencia no confirma sesión válida ni garantiza consumo en el widget.

Cambios

Archivo Cambio
src/Settings.tsx Pasos, sondeo, guardado booleano, bloqueo busy/saving e indicador de presencia.
src/lib/account-login.ts Solo abre login tras confirmación explícita del guardado requerido; cuenta persistida no vuelve a guardarse.
src/lib/account-login.test.ts 8 casos de bloqueo, orden, guardado diferido y errores.
src/lib/settings-save.ts Confirma configuración guardada o sin cambios, independientemente de autostart.
src/lib/settings-save.test.ts 3 regresiones adicionales de confirmación de persistencia.
src/lib/api.ts Nombre de API de presencia; mantiene el comando IPC account_session.
src-tauri/src/lib.rs Comando de presencia, helper puro y 3 tests de archivo/cuenta.
src/theme.css Estilos de pasos.
docs/account-onboarding.md Guía y límites de la detección de credenciales.

9 archivos, +389/−20 frente a main: una unidad funcional de onboarding con pruebas y documentación. No se cambia el refresco OAuth, secretos, dependencias ni infraestructura de release.

Test plan

Comprobaciones repetidas por un verificador independiente sobre el resultado final:

  • RED observado: 4 fallos / 22 éxitos; GREEN focalizado: 26/26.
  • npm test: 154 Vitest + 46 Node, sin fallos.
  • npm run build: TypeScript y Vite correctos.
  • cargo test --offline --locked en src-tauri: 51 Rust, sin fallos.
  • Total: 251 tests, 14 nuevos.
  • Sonda independiente de handlers reales con IPC/UI simulados: 5/5 (fallo de config, guardado en curso, éxito, autostart fallido con config guardada y desmontaje).
  • Readback independiente sin defectos adicionales encontrados; git diff --check limpio.
  • Validación visual/DOM y accesibilidad del nuevo flujo.
  • Windows, wrapper IPC de Tauri y login/aislamiento real del CLI con credenciales.

LSP: 7 archivos consultados, 2 confirmados limpios y 5 inconclusos. TypeScript/build y compilación/tests Rust pasan; cero diagnósticos recibidos no prueba limpieza de los cinco inconclusos. Shellcheck no aplica (sin scripts shell modificados).

Evaluación y riesgos

ASSESS inicialmente no pudo evaluar los archivos nuevos sin rastrear. Tras el staging normal autorizado para este commit, la evaluación nativa sí resolvió: riesgo medio, reviewDue: false (under_budget), perfil de writer grande; no requiere otro verificador y ya se realizó verificación independiente. No equivale a aprobación/cierre de revisión nativa.

Riesgo residual: coordinación asíncrona del guardado y contrato compartido de Ajustes. Los tests y la sonda no validan autenticación real ni comportamiento nativo en Windows. El PR queda borrador, sin merge ni release.

Checklist

  • Issue feat: guiar la conexión de cuentas adicionales CLI #46 verificada con status:approved.
  • Tipo seleccionado: feature; exactamente una etiqueta type:feature solicitada al crear el PR.
  • Cambios de comportamiento con pruebas de regresión y documentación.
  • Commits convencionales y sin trailers Co-Authored-By.
  • Commit/push y PR borrador autorizados por el propietario.
  • Checks de CI del PR completos y verificados tras la creación.
  • Prueba funcional real en Windows/CLI antes de considerarlo listo para merge.

Preserve save diagnostics, prevent concurrent save/login races and distinguish credential-file presence from session validity.

Refs #46
@egdev6 egdev6 added the type:feature New functionality label Oct 10, 2026
@egdev6
egdev6 marked this pull request as ready for review October 10, 2026 10:25
@egdev6
egdev6 merged commit 5d06d7e into main Oct 10, 2026
@egdev6
egdev6 deleted the feat/new-account-flow branch October 10, 2026 10:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type:feature New functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

feat: guiar la conexión de cuentas adicionales CLI

1 participant