mathieu/murmur
Watch
1
1
Fork
You've already forked murmur
 0
Code Issues 35 Pull requests 11 Projects Releases Packages Wiki Activity Actions

[P2][sink] Détecter les short writes du sink stdout #27

New issue
Open
opened 2026-07-31 21:05:14 +00:00 by mathieu · 2 comments
mathieu commented 2026-07-31 21:05:14 +00:00
Owner
Copy link

Constat

Le sink stdout vérifie seulement l'erreur de fmt.Fprintln (internal/sink/sink.go:115-123). Un io.Writer peut retourner un write court sans erreur et la livraison tronquée est alors déclarée réussie.

Tâche

Construire le payload exact text+LF, l'écrire une fois et retourner sink_failed sur erreur ou short write.

Critères d'acceptation

  • Full write réussit.
  • Short nil, short+error et zero write échouent.
  • Le contrat garde exactement un LF terminal.

Validation

go test -race ./internal/sink ./internal/cli

## Constat Le sink stdout vérifie seulement l'erreur de `fmt.Fprintln` (`internal/sink/sink.go:115-123`). Un `io.Writer` peut retourner un write court sans erreur et la livraison tronquée est alors déclarée réussie. ## Tâche Construire le payload exact `text+LF`, l'écrire une fois et retourner `sink_failed` sur erreur ou short write. ## Critères d'acceptation - Full write réussit. - Short nil, short+error et zero write échouent. - Le contrat garde exactement un LF terminal. ## Validation `go test -race ./internal/sink ./internal/cli`
hermes commented 2026-08-07 17:15:07 +00:00
Copy link

Correctif proposé dans la PR #40 (#40). Les packages sysexec/clipboard/inject/sink/session passent sous race.

Correctif proposé dans la PR #40 (https://git.2027a.net/mathieu/murmur/pulls/40). Les packages sysexec/clipboard/inject/sink/session passent sous race.
hermes commented 2026-08-07 18:02:37 +00:00
Copy link

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

Correctif proposé et validé dans la PR #40 : https://git.2027a.net/mathieu/murmur/pulls/40.
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
main
fix/cli-doctor-cancellation
fix/text-pipeline-budget
fix/ui-lifecycle
fix/delivery-correctness
fix/stt-correctness
fix/build-test-gates
fix/session-event-invariants
fix/audio-lifecycle
fix/config-policy
fix/wav-portable-memfd
fix/ipc-subscribe-decoder
v0.1.0-rc1
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
mathieu/murmur#27
No description provided.