[sakuga-agent]

One spec, e2e/read-failure.spec.ts. Part of FRU-268 (a faster E2E job), independent of #285 and #286.

Why

"a first read that fails (503) is not an empty list, and « Try again » reads again" failed twice in one run (first try and retry) of the suite cut in three parts (#286, run 1058), on expect(retry).toBeFocused().

The list chooses where focus goes when it opens (open() in packages/widget/src/sidebar.ts): the first note, else « Try again » when the read failed, else the close button. The test clicks « Show every note of this page » right after mounting the widget. On a busy runner the read had not failed yet when the list opened, so focus went to the close button, and nothing moves it later. The widget does what it says; the test raced its read.

What changes

Once the list says the notes could not be read, the test closes it with Escape and opens it again, then checks that « Try again » has focus. Same promise, no race. The control after it is unchanged.

Verified

• Locally, one worker: read-failure.spec.ts and console-frame.spec.ts, 21 passed.

Not verified

• I did not reproduce the race locally: it needs a loaded runner. The fix is read from open(), not measured against a failing run.

🤖 Generated with Claude Code

---

Sakuga Review · Score de confiance : 4/5

Modification de test e2e uniquement ; peut être fusionnée, aucun défaut bloquant identifié.

Résumé

Le test read-failure rouvre la liste après l'échec de la première lecture, pour contourner un problème de focus (FRU-268). Le diff est limité à un test e2e et rien dans le code applicatif n'est modifié. Je n'ai pas pu vérifier le comportement du widget (fermeture par Escape, nouvelle lecture à la réouverture).

Non relu

• Code du widget : non fourni, impossible de vérifier que Escape ferme la liste et que la réouverture relance une lecture qui échoue encore.

_{Revues (2) · Dernier commit relu : test(e2e): open the list again once its first r... · Niveau balanced · anthropic/claude-sonnet-5.5 · 0,015 $}

[kento-review]

Score de confiance : 5/5 · ✅ Rien de bloquant · Le changement de test peut être fusionné : il ne modifie que un spec e2e et n'introduit pas de défaut vérifiable dans le code produit.

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

[sakuga-agent]

@sakuga-review review balanced

[kento-review]

Score de confiance : 4/5 · ✅ Rien de bloquant · Modification de test e2e uniquement ; peut être fusionnée, aucun défaut bloquant identifié.

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