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

New issue

Closed

opened 2026-07-31 21:05:11 +00:00 by mathieu · 3 comments

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

Member

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

Member

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

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

mathieu commented

2026-09-23 12:37:39 +00:00

Author

Owner

Résolu sur main par 6c646da (intégré dans e3896f5). Décodeur Subscribe réutilisé après l’acquittement ; test événement groupé avec le handshake et tests IPC sous -race validés. La PR #38 revendique également des corrections déjà intégrées ; son traitement reste indépendant.

Résolu sur main par 6c646da (intégré dans e3896f5). Décodeur Subscribe réutilisé après l’acquittement ; test événement groupé avec le handshake et tests IPC sous -race validés. La PR #38 revendique également des corrections déjà intégrées ; son traitement reste indépendant.

mathieu closed this issue

2026-09-23 12:37:39 +00:00