[P2][sink] Ne garder qu’un seul propriétaire du garde at-most-once #28

New issue

Closed

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

mathieu commented

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

Owner

Constat

sink.New retourne déjà Once(delivery), puis NewOrchestrator enveloppe à nouveau le même sink (internal/sink/sink.go:104-108, :182-208, internal/session/orchestrator.go:111-116). Deux sync.Once et deux états d'erreur protègent la même opération.

Tâche

Attribuer explicitement la garantie à l'orchestrateur, qui possède l'état de livraison et couvre aussi les sinks injectés. Retourner le sink composé directement depuis sink.New.

Critères d'acceptation

Une session contient un seul garde at-most-once.
Les sinks injectés gardent la garantie.
dictate reste naturellement single-shot.
L'ordre CopyRaw/Deliver ne change pas.

Validation

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

## Constat `sink.New` retourne déjà `Once(delivery)`, puis `NewOrchestrator` enveloppe à nouveau le même sink (`internal/sink/sink.go:104-108`, `:182-208`, `internal/session/orchestrator.go:111-116`). Deux `sync.Once` et deux états d'erreur protègent la même opération. ## Tâche Attribuer explicitement la garantie à l'orchestrateur, qui possède l'état de livraison et couvre aussi les sinks injectés. Retourner le sink composé directement depuis `sink.New`. ## Critères d'acceptation - Une session contient un seul garde at-most-once. - Les sinks injectés gardent la garantie. - `dictate` reste naturellement single-shot. - L'ordre CopyRaw/Deliver ne change pas. ## Validation `go test -race ./internal/sink ./internal/session ./internal/cli ./internal/acceptance`

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

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

fix(delivery): rendre les effets single-shot et bornés #40

hermes commented

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

Member

Correctif proposé dans la PR #40 (mathieu/murmur#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

Member

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

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

mathieu closed this issue

2026-09-11 19:47:24 +00:00