[sakuga-agent]

Not critical: one screen of the console, no route. These are follow-ups of #316 and #331, which were merged while their last findings were being answered.

1. A name typed while a refused one was on its way is sent after it

Found by the lite review of 5001a4f on #316. When the name on its way was refused, the refusal returned before it read the field. A newer text typed, then left, meanwhile stayed unsent, with no alert. The refusal now sends that newer text in its turn, as a success already did (#316, d288774). A call with no answer goes through the same branch.

Test: « what is typed while a refused name is on its way is sent after the refusal ». It fails on main.

2. The line of a list of the extension that was not read goes when a later read succeeds

Found by the balanced review of 7f60c91 on #331. After a disconnection whose read again failed, a later read again that succeeded left the line « Fruitback did not answer. The browsers of the extension could not be read. » on screen. A read that succeeds now takes it away.

Test: « the line of a list of the extension that was not read goes when a later read succeeds ». It fails on main.

Proof

e2e/console-account.spec.ts passes locally, 28 tests; console unit tests 123. No sentence is added.

🤖 Generated with Claude Code

---

Sakuga Review · Score de confiance : 4/5

Fusionable après un regard sur le finding mineur : les deux corrections de saisie et de ligne d'erreur tiennent sur les chemins lus, et le seul écart porte sur des rangées d'extensions possiblement périmées après une lecture réussie.

Constats

1.  Liste des extensions périmée quand la relecture confirme la déconnexion refusée apps/console/app/routes/account.tsx:192
   Une lecture des extensions échoue : la ligne d'erreur s'affiche, et la liste garde ses anciennes rangées. Une déconnexion renvoie « unknown », la relecture réussit et montre toujours la rangée, donc la branche de blocage s'exécute : la ligne d'erreur part, mais setExtensions n'est pas appelé. Une rangée déjà terminée ailleurs reste affichée sans aucun avertissement. Correction : Dans la branche de blocage, mettre à jour la liste avec connected.data avant de retourner setExtensionStuck, pour que les rangées suivent la lecture qui a réussi.

Résumé

La PR corrige deux comportements de la page Compte de la console : une saisie de nom tapée pendant qu'un nom refusé est en route part après le refus, et la ligne « liste non lue » des extensions disparaît quand une relecture réussit. Les chemins de saveName et de disconnectExtension ont été relus en entier, avec le test e2e ajouté. Non vérifié : le corps de renameMe, endExtensionBrowser et extensionBrowsers, qui ne figurent pas dans le message.

Non relu

• apps/console/app/api.ts: renameMe, endExtensionBrowser, extensionBrowsers et leurs réponses non montrés, donc le comportement exact des statuts 404 et 502
• apps/console/app/problems.ts: seule la fonction problemOf est dans le contexte

_{Revues (2) · Dernier commit relu : chore(console): run the checks of #332 again (F... · Niveau lite · anthropic/claude-haiku-5.5 · 0,004 $}

[kento-review]

Score de confiance : 4/5 · ✅ Rien de bloquant · Mergeable : les deux corrections se lisent correctement dans le code montré, et aucun défaut confirmé n'a été trouvé. Le score reste à 4 car les fonctions renameMe, endExtensionBrowser et extensionBrowsers ne sont pas dans le diff.

_{Niveau lite · anthropic/claude-haiku-5.5 · 0,003 $ · Le résumé, le diagramme et la liste des constats sont à la fin de la description de la PR.}

[kento-review]

Score de confiance : 4/5 · ⚠️ Rien de bloquant · Fusionable après un regard sur le finding mineur : les deux corrections de saisie et de ligne d'erreur tiennent sur les chemins lus, et le seul écart porte sur des rangées d'extensions possiblement périmées après une lecture réussie.

_{Niveau lite · anthropic/claude-haiku-5.5 · 0,004 $ · Le résumé, le diagramme et la liste des constats sont à la fin de la description de la PR.}