Repository navigation
front: complete local authentication and recovery - #25
Merged
Merged
Conversation
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.
[agent:contract]
Ce qui change
Upgrade local indépendant du Market Advisor, basé sur staging @ 3ed1704.
Mandat transversal expressément autorisé par le brief humain Market Intelligence + Authentication Upgrade ; politique générale inchangée. Aucun changement au contrat, marge, scénarios ou preuves. Aucune transaction Stellar et aucun déploiement public.
Revue
Deux sessions Codex neuves et indépendantes sans contexte auteur : PASS sur récupération/expiration/rejeu/concurrence, anciens comptes, double soumission et séparation local/keeper. Aucun problème actionnable. Le dernier ajout de tests de scénarios a reçu une revue indépendante légère : PASS. Revues automatisées, pas humaines.
Tests
Tests auth avec vrai PGlite en mémoire et Web Crypto : doublons concurrents/casse, compte antérieur, échec générique, faiblesse/confirmation, codes invalides/expirés/consommés, limite d'essais, resend, reset concurrent et reconnexion, maintenance dev.
Deux scénarios frontend complets : KMMF cure/release/withdraw et KBOND default/refus, avec signataire mock, horloge contrôlée, conservation et fetch interdit. Aucun appel Stellar.
Navigateur intégré réel : inscription, confirmation incorrecte, yeux, mauvais mot de passe, déconnexion, persistance après reload, récupération complète et reconnexion avec le nouveau mot de passe, compte demo ; desktop et 375 px vérifiés, aucune erreur console bloquante observée. Les parcours de scénarios ont été commencés dans le navigateur en mocks mais pas achevés intégralement après arrêt de l'automatisation.
Non testé : véritable masquage puis réaffichage d'un onglet. Computer Use a arrêté Chrome, son URL ne pouvant pas être déterminée avec assez de certitude. Aucun contournement tenté.
Premier lancement concurrent : timeout existant de 5 s sur keeper health/config. Relance complète séquentielle ci-dessus PASS sans changer le keeper. Avertissements existants : taille des chunks et eval PGlite. Pas de benchmark navigateur avant/après comparable ; assets PGlite inchangés (WASM 10.09 MB, data 6.30 MB).
Interfaces
Aucun changement d'interface partagée/API keeper. Périmètre frontend autorisé par le brief humain. Compatible avec feature/collateral-market-advisor : calcul git merge-tree sans conflit.
Coordination Nathan : la PR #24 front/liquid-glass-final modifie aussi le lock screen. Calcul sans modifier les fichiers : conflit app/src/auth/LockScreen.tsx. Intégrer les fonctions/formulaires dans sa direction artistique lors de la résolution ; sa branche reste intacte.
Documentation
README : mode local, récupération/stockage, maintenance dev, variables publiques et migration Supabase SMTP/redirects/RLS/admin protégé. AGENTS.md, interfaces et décisions consultés, sans modification de leur contrat de communication.
Checklist
PR pour revue, sans fusion.