[P1][notify] Retirer la notification recording persistante si le terminal échoue #31

New issue

Open

opened 2026-07-31 21:05:15 +00:00 by mathieu · 2 comments

mathieu commented

2026-07-31 21:05:15 +00:00

Owner

Constat

La notification CaptureReady n'expire pas. Si son remplacement terminal D-Bus échoue, la connexion ferme sans CloseNotification (internal/ui/notify.go:121-203, :278-331), laissant potentiellement « microphone actif » affiché après la session.

Tâche

Ajouter CloseNotification(ctx,id) au client et retirer, sous budget borné, le dernier ID non terminal si aucun remplacement terminal n'a été accepté.

Critères d'acceptation

Erreur/timeout terminal retire la notification persistante.
Close sans terminal la retire aussi.
Un remplacement terminal réussi reste visible selon son timeout.
L'échec de cleanup reste best-effort et borné.

Validation

go test -race ./internal/ui ./internal/session

## Constat La notification `CaptureReady` n'expire pas. Si son remplacement terminal D-Bus échoue, la connexion ferme sans `CloseNotification` (`internal/ui/notify.go:121-203`, `:278-331`), laissant potentiellement « microphone actif » affiché après la session. ## Tâche Ajouter `CloseNotification(ctx,id)` au client et retirer, sous budget borné, le dernier ID non terminal si aucun remplacement terminal n'a été accepté. ## Critères d'acceptation - Erreur/timeout terminal retire la notification persistante. - Close sans terminal la retire aussi. - Un remplacement terminal réussi reste visible selon son timeout. - L'échec de cleanup reste best-effort et borné. ## Validation `go test -race ./internal/ui ./internal/session`