[P1][ipc] Réutiliser le décodeur JSON après le handshake Subscribe

mathieu commented

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

Owner

Constat

Subscribe décode l'acquittement avec un décodeur temporaire puis en crée un autre pour les événements (internal/ipc/client.go:63-82, :124-133). Le premier peut avoir préchargé le premier événement, qui est alors perdu.

Tâche

Créer un seul json.Decoder, l'utiliser pour l'acquittement puis le transférer à la goroutine d'événements.

Critères d'acceptation

Acquittement et premier événement écrits dans le même syscall sont tous deux reçus.
Aucun blocage ni erreur JSON parasite.
Le framing one-shot de Do reste inchangé.

Validation

go test -race -count=100 ./internal/ipc

## Constat `Subscribe` décode l'acquittement avec un décodeur temporaire puis en crée un autre pour les événements (`internal/ipc/client.go:63-82`, `:124-133`). Le premier peut avoir préchargé le premier événement, qui est alors perdu. ## Tâche Créer un seul `json.Decoder`, l'utiliser pour l'acquittement puis le transférer à la goroutine d'événements. ## Critères d'acceptation - Acquittement et premier événement écrits dans le même syscall sont tous deux reçus. - Aucun blocage ni erreur JSON parasite. - Le framing one-shot de `Do` reste inchangé. ## Validation `go test -race -count=100 ./internal/ipc`

hermes referenced this issue from a pull request that will close it,

2026-08-07 17:04:03 +00:00

fix(ipc): préserver les événements bufferisés après Subscribe #36

hermes commented

2026-08-07 17:04:15 +00:00

Correctif proposé dans la PR #36 : le décodeur JSON du handshake Subscribe est désormais réutilisé pour le flux. Régression ajoutée avec acquittement + premier événement dans le même write. Validation : go test -race -count=100 ./internal/ipc (OK).

Correctif proposé dans la PR #36 : le décodeur JSON du handshake Subscribe est désormais réutilisé pour le flux. Régression ajoutée avec acquittement + premier événement dans le même write. Validation : `go test -race -count=100 ./internal/ipc` (OK).

hermes commented

2026-08-07 18:02:32 +00:00

Correctif proposé et validé dans la PR #36 : #36.

Correctif proposé et validé dans la PR #36 : https://git.2027a.net/mathieu/murmur/pulls/36.

Rows
Columns

[P1][ipc] Réutiliser le décodeur JSON après le handshake Subscribe #11

Constat

Tâche

Critères d'acceptation

Validation