Repository navigation
fix(cv): messages de connexion/dossier exacts et accessibilité (BUG-01 à 06) - #35
Merged
Merged
Conversation
…1 à 06) Corrige les anomalies relevées par la recette Phase T (PR #34), sans toucher à la cryptographie, au format, au stockage, à la CSP ni au service worker : - BUG-01 : fenêtre Google fermée/bloquée (error_callback) → code « popup », message « Connexion Google annulée ou fenêtre bloquée… », état neutre ; - BUG-02 : autorisation Drive refusée ou décochée → code « scope », message expliquant que l'accès n'a pas été accordé ; plus jamais « session expirée » ni « synchronisation bloquée » sans session préalable ; - BUG-03 : sélecteur annulé → « Aucun dossier choisi. », pas d'erreur ; - BUG-04 : sur l'origine où Google Drive est configuré, seuls l'écran Google Drive et sa navigation sont affichés (autres routes redirigées, recherche rapide et raccourcis désactivés) ; premier rendu après lecture de la configuration pour ne jamais montrer d'état de démonstration ; - BUG-05 : « Restaurer cette version » décrit par un texte visible (aria-describedby) tant qu'il est désactivé ; - BUG-06 : « Annuler » retiré à l'étape de contrôle, comme le commentaire l'annonçait, au lieu d'être laissé désactivé sans explication. Tests : tests/e2e/messages-exacts.spec.ts (rouges sur main, verts ici) ; connected-drive.spec.ts suit la redirection de Réglages vers Google Drive. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01E5jwZntqAvJaRThsuEjgja
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.
Objet
Corrige les 6 anomalies relevées par la recette Phase T (PR #34, brouillon), avec le plus petit changement possible dans
src/. Aucun changement de cryptographie, de format, de stockage, de CSP, de service worker, de workflow ni de garde-fou CI.Bug → test → résultat
Tests :
tests/e2e/messages-exacts.spec.ts(suite e2e existante, toutes les projections Chromium/WebKit deplaywright.config.ts).551f3d9)ba98f84)error_callback→ codepopup; message « Connexion Google annulée ou fenêtre bloquée : réessaie (autorise les fenêtres pour ce site). », état « En attente »BUG-01 fenêtre Google fermée ou bloquée : message exact, aucune « session expirée »access_deniedou portéedrive.fileabsente → codescope; message « L’accès au dossier Google Drive n’a pas été accordé… »BUG-02 autorisation Google Drive refusée : l’accès non accordé est expliquécancelleddu Picker → « Aucun dossier choisi. », état neutreBUG-03 sélecteur Google annulé : état neutre « Aucun dossier choisi »#/drive, recherche rapide et raccourcis désactivés) ; premier rendu après lecture dedrive-config.jsonBUG-04 origine de production : aucun état ni sauvegarde de démonstrationaria-describedbyvers un texte visible « Coche cette confirmation pour activer… » (ou « Cette version est identique… »)BUG-05 « Restaurer cette version » désactivé : condition décrite par un texte visiblearia-describedbyabsentBUG-06 sauvegarde : « Annuler » n’est jamais laissé désactivé sans explicationTest existant adapté :
connected-drive.spec.tsouvrait Réglages puis cliquait le lien vers Google Drive ; sur l’origine connectée, Réglages redirige désormais vers#/drive(le test le vérifie).Preuves locales (Chromium ; WebKit absent de l’environnement local)
origin/main+ nouveau test : 6/6 échecs, chacun pour la raison attendue (voir tableau).npm test376/376 ✅,npm run typecheck✅,npm run build+npm run security✅,npm run test:e2e93 réussis, 7 ignorés (ignorés par conception : clavier bureau / mobile uniquement), 0 échec.Impact et risques
google-drive-session.js: seuls les codes d’erreur de rejet changent (popup,scope) ; jeton, portée, stockage et flux inchangés.drive-config.jsonest injoignable (hors ligne au premier chargement sur la production), l’origine se comporte comme une démo explicitement signalée (bandeau « Démonstration »).GoogleDriveSession.load).Retour arrière
Revert du commit
ba98f84(code seul ; aucune donnée ni migration).Generated by Claude Code