Skip to content

fix(console): a delivery that Fruitback gave up says so on Notes (FRU-245) - #323

Merged
sakuga-agent[bot] merged 1 commit into
mainfrom
fru-sf-workspace/fru-245-gave-up
Oct 14, 2026
Merged

sakuga-agent[bot] merged 1 commit into
mainfrom
fru-sf-workspace/fru-245-gave-up

Conversation

@sakuga-agent

@sakuga-agent sakuga-agent Bot commented Oct 14, 2026 •

Copy link
Copy Markdown
Contributor

Not critical: one cell of the Notes page. Follow-up of #313, from the P2 of its lite review, agreed with the orchestrator.

What changes

A note that a REST address did not take carries delivery.state in the route of #306: waiting (the worker still tries) or gave-up (it stopped, FRU-122). The cell said « Not delivered yet » for both. A note the worker gave up now reads « Not delivered: Fruitback stopped trying », in the same accent, 13px/600. It will not arrive by itself. « Send again » stays and sends both, as before (POST …/notes/<id>/retry takes both).

French added

English French
Not delivered: Fruitback stopped trying Non livrée : Fruitback a cessé d’essayer

Proof

e2e/console-notes.spec.ts: « a delivery the worker gave up says so, and « Send again » still sends it ». A note of the second page is given up. Its cell reads the new sentence in the accent, a note still tried reads « Not delivered yet », and « Send again » sends the given-up one and takes the alert away. It fails on main. The spec passes locally, 17 tests; console unit tests 119.

Not verified

The look of the longer sentence beside « Send again » at 390 px; the cell wraps, as the others do.

🤖 Generated with Claude Code


Sakuga Review · Score de confiance : 4/5

Le changement peut être fusionné : il ne fait qu'ajouter un libellé conditionnel pour une livraison abandonnée. Le type de note.delivery.state (la valeur 'gave-up') n'est pas visible dans le contexte, donc sa compilation n'a pas pu être confirmée.

Résumé

La PR affiche « Non livrée : Fruitback a cessé d'essayer » sur une note dont le worker a abandonné la livraison, et garde « Envoyer à nouveau » disponible. Le libellé français est ajouté dans messages.fr.ts et un test e2e couvre les deux libellés ainsi que le renvoi. J'ai vérifié que la clé est une phrase anglaise complète, donc t retombe correctement sur l'anglais si la traduction manque. Je n'ai pas pu vérifier le type de delivery.state ni la logique de renvoi côté worker.

Diagramme
%%{init: {'theme': 'neutral'}}%%
flowchart LR
A["NoteRow"] --> B["note.delivery.state"]
B -->|gave-up| C["Not delivered: Fruitback stopped trying"]
B -->|autre| D["Not delivered yet"]
C --> E["Send again"]
D --> E
Loading
Non relu
  • apps/console/app/routes/notes.tsx: le type de note.delivery et la valeur 'gave-up' ne sont pas dans le contexte ; si la valeur n'existe pas, le typecheck échoue
  • apps/worker: la logique qui pose l'état gave-up et le renvoi (POST /retry) n'est pas dans le diff ; le comportement réel du renvoi n'a pas été vérifié

Revues (1) · Dernier commit relu : fix(console): a delivery that Fruitback gave up... · Niveau lite · anthropic/claude-haiku-5.5 · 0,003 $

…-245)

A note that an address did not take read « Not delivered yet », whether
the worker still tries or gave up (FRU-122). One it gave up now reads
« Not delivered: Fruitback stopped trying »: it will not arrive by itself.
« Send again » sends both, as before. The test fails on main.

Co-Authored-By: Claude Opus 5.5 <[email protected]>

@kento-review kento-review Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Score de confiance : 4/5 · ✅ Rien de bloquant · Le changement peut être fusionné : il ne fait qu'ajouter un libellé conditionnel pour une livraison abandonnée. Le type de note.delivery.state (la valeur 'gave-up') n'est pas visible dans le contexte, donc sa compilation n'a pas pu être confirmée.

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 kento-review Bot added the review: 4/5 Score de confiance de Sakuga Review (5 : prête à merger) label Oct 14, 2026
@sakuga-agent
sakuga-agent Bot merged commit bf3dddf into main Oct 14, 2026
16 checks passed
@sakuga-agent
sakuga-agent Bot deleted the fru-sf-workspace/fru-245-gave-up branch October 14, 2026 09:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

review: 4/5 Score de confiance de Sakuga Review (5 : prête à merger)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant