Repository navigation
feat(providers): guiar y comprobar conexiones multicuenta - #51
Merged
Merged
Conversation
egdev6
marked this pull request as ready for review
October 10, 2026 15:41
This was referenced Oct 10, 2026
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
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.
Related issue / Issue relacionada
Closes #50
Issue aprobada con
status:approved. Tipo:type:feature, elegido por el propietario.ODD: N/A, saneado directo autorizado fuera del piloto coordinado.
Scope / Alcance
Las cuentas adicionales consideran tanto sus propios cambios como la configuración heredada del proveedor antes de conectar. Los fallos de guardado conservan su diagnóstico; un fallo exclusivo de autostart no bloquea una configuración ya guardada. Polling y respuestas pendientes se descartan al desmontar.
Settings.tsx,account-health.tsy tests,api.ts: guías, estado, polling y secuencia de guardado/login.lib.rs,login.rs,providers/mod.rs: comandos coherentes, búsqueda CLI, PATH y consulta/rutas por cuenta; preserva los tres tests de presencia existentes.ProviderCard.tsx,accounts.ts,theme.css,docs/account-onboarding.md: errores visibles, requisitos y explicaciones sin promesas falsas de OAuth.11 archivos respecto a
main; cambio amplio en Ajustes, revisar por unidades frontend/backend/contratos. Sin cambios de versión, secretos, firma, instalación automática ni release.Coordination / Coordinación
type:feature, referenciaClosesy commit/push/PR. Escritores disjuntos frontend/backend y verificación independiente del conjunto.Acceptance and validation / Aceptación y validación
Criterios cumplidos en el alcance verificado: compila frontend/Rust; una sola apertura de login tras confirmación explícita cuando guardar es necesario; bloqueo tras fallo/ocupado; cambios heredados detectados; presencia ≠ validez; conserva formularios y tests; sin instalación automática.
npm testnpm run buildcargo test --offline --lockedgit diff --check274 tests automatizados + 9 probes, no pruebas Windows/DOM. La regresión de configuración heredada tuvo RED conductual y después GREEN; no se afirma un RED conductual aislado del antiguo doble-login o del caso de directorio en Rust.
No ejecutado / límites: DOM/browser y accesibilidad, clipboard/foco reales, Windows/cmd/PATHEXT nativos, CLI/OAuth/login o credenciales reales, aislamiento en Windows, Credential Manager, notificaciones/widget, firmas y rebuild Rust limpio. Los probes simulan IPC, no el bridge Tauri real. Codex principal conserva la resolución de home existente, aún sin prueba determinista de ese caso.
Pendientes antes de integrar: revisión humana y comprobaciones Windows aplicables. No se afirma CI verde sin checks observados; el repositorio solo declara el workflow manual de release, que no se ejecutó.
Esta PR no implica integración. Merge, release, secretos y firma requieren autorización separada.