fix(settings): un seul contrôle de langue, un seul écrivain#263
Merged
Conversation
The Settings language selector could not save anything but French, and even
that changed nothing on screen. It offered fr-BE / fr-FR / en-GB while the
server validated z.enum(LOCALES), which holds neither fr-FR nor en-GB — so
every choice but fr-BE failed validation with a generic error toast. And
updateProfileAction wrote users.locale without touching the NEXT_LOCALE cookie
or revalidating the root layout, so an accepted fr-BE moved a database column
while the app stayed in the previous language: the rendered locale comes from
the URL prefix alone.
One preference, two divergent writers. setLocaleAction did the whole job —
cookie, database, revalidate, navigate — and updateProfileAction did a third of
it, badly.
ProfileCard now renders the LocaleSwitcher that already exists: a FR | EN
segmented control built on LOCALES_VISIBLE, already audited for a11y, already
wired to the canonical path. locale leaves profileUpdateSchema and
updateProfileAction, leaving setLocaleAction as the single writer. Repairing
the bespoke Select would have meant writing a third implementation of the same
control; reusing this one removes the double submit, the duplicated
localeOptions keys and the initial-value problem in one move.
The control sits outside the form on purpose. It persists immediately and
navigates, which remounts the card — inside the form it implied "Save" applied
to it, and a name typed but not yet saved would vanish on a switch with no
explanation.
ui-auditor caught a regression this change introduced: at >=1024px the page
also mounts the header's switcher (its hidden lg:flex block is in the DOM and
focusable), so two radiogroups both announced "Changer de langue" became
indistinguishable in a screen reader's element list. LocaleSwitcher now takes an
optional labelledById, and the Settings one is named by its visible "Langue"
text. Header and MoreSheet instances are untouched — without the prop the
original behaviour stands. Accessible name and visible label now match exactly
rather than merely overlap.
Two near-identical error keys had to be told apart. settings.locale.invalid lost
its only emitter with the schema field and is gone from all five message files;
errors.locale.invalid is emitted by setLocaleAction and is preserved. Confusing
them would have broken the switcher's own error path. The localeOptions group
went too — it duplicated ui.localeSwitcher.options, and that duplication is
precisely how fr-FR and en-GB drifted away from the server enum.
Three things deliberately did not move. users.locale is still written by
setLocaleAction and still read by i18n/request.ts when requestLocale is absent —
it is the cross-device fallback, not dead weight. profileUpdateSchema stays a
plain z.object and must never become .strict(): during a deploy, tabs on the old
bundle keep posting locale, and Zod stripping it is what makes them degrade
gracefully. And all three edits ship in one commit, because the schema carried
.default('fr-BE') — split apart, every profile save would have silently reset
the user's language.
The new action test pins the payload actually sent to Supabase, the only place
this could come back unnoticed now that the schema no longer types locale.
Verified falsifiable: reintroducing the locale write turns two specs red. The
two schema specs that asserted the old contract are rewritten rather than
deleted, so the change of contract leaves a trace.
No Playwright spec covered this control, and the authenticated ones auto-skip in
CI, so a green pipeline proves nothing here. A seeded spec now covers it and
says so in its header; run locally at 3/3 on iPhone 14.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
github-actions
Bot
added
area:settings
Contributor
|
🧙 Sourcery examine votre pull request ! Conseils et commandesInteragir avec Sourcery
Personnaliser votre expérienceAccédez à votre tableau de bord pour :
Obtenir de l’aide
Original review guide in English🧙 Sourcery is reviewing your pull request! Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
Contributor
There was a problem hiding this comment.
Sorry @thierryvm, you have reached your weekly rate limit of 500000 diff characters.
Please try again later or upgrade to continue using Sourcery
thierryvm
added a commit
that referenced
this pull request
Jul 26, 2026
…SEO (#265) Handoff de fin de session. Documentation uniquement, 1 fichier. **Livré** : #261 (redirections localisées), #262 (registre des destinations de nav), #263 (sélecteur de langue), #264 (SEO) — toutes mergées. **Reste à faire, ordonné** : le lot 2 de la refonte UX en tête, avec les trois limites que le lot 1 s'est explicitement interdit de traiter reprises comme entrées plutôt que laissées à re-découvrir. Puis la reconnexion fantôme, sans diagnostic à ce jour. Puis les dettes tracées. **Frictions d'environnement consignées** parce qu'elles ont coûté du temps deux fois chacune : `playwright.config.ts` ne charge pas `.env.local`, `e2e:auth` sert un build prod où le rate limit échoue en fermé sur l'Upstash factice, la variable est `E2E_BASE_URL`, et le port 3000 appartient à un autre projet. Miroir dans le vault Obsidian, slug vérifié via `_index.md`.
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
1 participant
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.
Ferme un défaut préexistant remonté par
i18n-auditorpendant #258, avec la simplification que tu as demandée : FR - EN, sans précision régionale.Le défaut
Le sélecteur de langue des Réglages ne pouvait rien enregistrer d'autre que le français, et même ça ne changeait rien à l'écran.
fr-BE,fr-FR,en-GB, alors que le serveur validez.enum(LOCALES)qui ne contient nifr-FRnien-GB→ toute autre sélection échouait avec un toast d'erreur générique.updateProfileActionécrivaitusers.localesans toucher au cookie ni revalider : depuis fix(i18n): résolution de langue déterministe — le préfixe d'URL décide, plus le cookie #258 la langue affichée vient du préfixe d'URL, donc la colonne bougeait et l'app restait dans la langue précédente.Une préférence, deux écrivains divergents :
setLocaleActionfaisait tout le travail,updateProfileActionen faisait un tiers, mal.Le correctif
La carte Profil rend désormais le
LocaleSwitcherqui existe déjà — segmented control FR | EN, déjà audité a11y, déjà branché sur le bon chemin.setLocaleActiondevient le seul écrivain.Réparer le
<Select>maison revenait à écrire une troisième implémentation du même contrôle. Réutiliser l'existant supprime d'un coup le double submit, les clés dupliquées et la question de la valeur initiale.Le contrôle sort du formulaire : il persiste immédiatement et navigue, ce qui remonte la carte. À l'intérieur, il laissait croire que « Enregistrer » s'y appliquait — et un nom saisi non sauvegardé disparaissait au changement de langue.
Ce que l'audit UI a rattrapé
ui-auditora relevé une régression que j'introduisais : à ≥1024px la page monte aussi le switcher du header, donc deuxradiogroupannoncés « Changer de langue », indiscernables dans la liste des éléments d'un lecteur d'écran.Corrigé plutôt que laissé en arbitrage : le champ des Réglages nomme son groupe par son libellé visible « Langue ». Header et MoreSheet inchangés. Bénéfice au passage — nom accessible et texte visible sont désormais identiques (WCAG 2.5.3) au lieu de simplement se recouper.
Le piège des deux clés d'erreur
settings.locale.invalidperdait son émetteur → supprimée des 5 fichiers.errors.locale.invalidest émise parsetLocaleAction→ préservée. Les confondre cassait le chemin d'erreur du switcher.i18n-auditora vérifié les deux : ✅ GO, parité intacte.Preuve
npm run testtypecheck·lintsettings-locale-fieldiPhone 14["FR","EN"], ancien<Select>absentFalsifiabilité vérifiée : en réintroduisant l'écriture de
locale, deux specs passent au rouge.Restent en P2, non traités
Pas d'
aria-describedby« s'applique immédiatement » ; relation libellé/nom accessible non verrouillée par un test pour les autres instances ;CardTitlerend un<div>et jamais un vrai titre (concerne toute l'app, à tracer séparément).Rapport :
docs/prs/PR-settings-locale-select-report.mdSummary by Sourcery
Utiliser le
LocaleSwitcherpartagé comme unique contrôle de préférence de langue et supprimer la gestion de la locale du flux de mise à jour du profil.New Features:
LocaleSwitcherconfigurable qui peut être étiqueté par un élément visible externe.LocaleSwitcherpartagé.Bug Fixes:
localedans la payload du profil.Enhancements:
radiogroupun nom accessible distinct basé sur une étiquette visible.Documentation:
Tests:
updateProfileActionafin de s’assurer qu’il n’écrit jamais la locale et qu’il valide la gestion du nom d’affichage.Chores:
Original summary in English
Summary by Sourcery
Use the shared LocaleSwitcher as the single language preference control and remove locale handling from the profile update flow.
New Features:
Bug Fixes:
Enhancements:
Documentation:
Tests:
Chores: