fix(cli): rendre stdin annulable et doctor non destructif #41

Open

hermes wants to merge 2 commits from fix/cli-doctor-cancellation into fix/session-event-invariants

pull from: fix/cli-doctor-cancellation

merge into: mathieu:fix/session-event-invariants

mathieu:main

mathieu:fix/text-pipeline-budget

mathieu:fix/ui-lifecycle

mathieu:fix/delivery-correctness

mathieu:fix/stt-correctness

mathieu:fix/build-test-gates

mathieu:fix/session-event-invariants

mathieu:fix/audio-lifecycle

mathieu:fix/config-policy

mathieu:fix/wav-portable-memfd

mathieu:fix/ipc-subscribe-decoder

Conversation 0 Commits 2 Files changed 6 +419 -45

hermes commented 2026-08-07 17:29:12 +00:00

First-time contributor

Copy link

Résumé

• lecture stdin cleanup bornée et annulable avec contrat CancelRead, sans goroutine laissée en arrière-plan
• annulation dictate avant STT: abort capture, terminal cancelled, code 1, aucun provider/livraison
• os.Stdin fermé seulement lorsqu’il est possédé
• doctor sonde un vrai remplacement atomique via fichier sibling sans ouvrir ni modifier la destination; symlink/FIFO/répertoire refusés et probes nettoyés

Closes #1
Closes #2

Validation

• tests CLI d’annulation/ownership sous race, 100 répétitions
• tests doctor de destination/probe sous race, 100 répétitions
• git diff --check

Signaux foreground

Le chemin dictate distingue maintenant les signaux de premier plan (arrêt puis traitement) d’une annulation de contexte API, avec stdin non bloquant et tests dédiés.

## Résumé - lecture stdin cleanup bornée et annulable avec contrat `CancelRead`, sans goroutine laissée en arrière-plan - annulation dictate avant STT: abort capture, terminal `cancelled`, code 1, aucun provider/livraison - `os.Stdin` fermé seulement lorsqu’il est possédé - doctor sonde un vrai remplacement atomique via fichier sibling sans ouvrir ni modifier la destination; symlink/FIFO/répertoire refusés et probes nettoyés Closes #1 Closes #2 ## Validation - tests CLI d’annulation/ownership sous race, 100 répétitions - tests doctor de destination/probe sous race, 100 répétitions - `git diff --check` ## Signaux foreground Le chemin `dictate` distingue maintenant les signaux de premier plan (arrêt puis traitement) d’une annulation de contexte API, avec stdin non bloquant et tests dédiés.

hermes added 1 commit 2026-08-07 17:29:12 +00:00

fix(cli): make stdin cancellation and doctor probes safe 181b85022f

hermes referenced this pull request 2026-08-07 17:29:13 +00:00

[P1][cli] Rendre la lecture stdin de cleanup annulable #1

hermes referenced this pull request 2026-08-07 17:29:13 +00:00

[P1][doctor] Tester la vraie capacité de remplacement atomique du sink fichier #2

hermes force-pushed fix/cli-doctor-cancellation from 181b85022f to b484e44c54 2026-08-07 17:54:30 +00:00 Compare

hermes changed target branch from main to fix/session-event-invariants 2026-08-07 18:02:08 +00:00

hermes referenced this pull request 2026-08-07 18:02:37 +00:00

[P1][cli] Rendre la lecture stdin de cleanup annulable #1

hermes referenced this pull request 2026-08-07 18:02:38 +00:00

[P1][doctor] Tester la vraie capacité de remplacement atomique du sink fichier #2

hermes added 1 commit 2026-08-07 18:16:39 +00:00

fix(cli): treat foreground signals as dictate stop e5dbc486f0

hermes force-pushed fix/cli-doctor-cancellation from e5dbc486f0 to 9d7dae7b73 2026-08-07 18:18:04 +00:00 Compare

hermes force-pushed fix/cli-doctor-cancellation from 9d7dae7b73 to b5b71dec87 2026-08-07 18:28:56 +00:00 Compare

Some checks failed

Hide checks

CI / amd64 / Go 1.23.12 (push) Has been cancelled

Details

CI / amd64 / Go 1.26.5 (push) Has been cancelled

Details

CI / arm64 cross-build / Go 1.23.12 (push) Has been cancelled

Details

CI / arm64 cross-build / Go 1.26.5 (push) Has been cancelled

Details

CI / amd64 / Go 1.23.12 (pull_request) Has been cancelled

Details

CI / amd64 / Go 1.26.5 (pull_request) Has been cancelled

Details

CI / arm64 cross-build / Go 1.23.12 (pull_request) Has been cancelled

Details

CI / arm64 cross-build / Go 1.26.5 (pull_request) Has been cancelled

Details

This pull request can be merged automatically.

You are not authorized to merge this pull request.

View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.

git fetch -u origin fix/cli-doctor-cancellation:fix/cli-doctor-cancellation

git switch fix/cli-doctor-cancellation

Merge

Merge the changes and update on Forgejo.

Warning: The "Autodetect manual merge" setting is not enabled for this repository, you will have to mark this pull request as manually merged afterwards.

git switch fix/session-event-invariants

git merge --no-ff fix/cli-doctor-cancellation

git switch fix/cli-doctor-cancellation

git rebase fix/session-event-invariants

git switch fix/session-event-invariants

git merge --ff-only fix/cli-doctor-cancellation

git switch fix/cli-doctor-cancellation

git rebase fix/session-event-invariants

git switch fix/session-event-invariants

git merge --no-ff fix/cli-doctor-cancellation

git switch fix/session-event-invariants

git merge --squash fix/cli-doctor-cancellation

git switch fix/session-event-invariants

git merge --ff-only fix/cli-doctor-cancellation

git switch fix/session-event-invariants

git merge fix/cli-doctor-cancellation

git push origin fix/session-event-invariants

Sign in to join this conversation.

Reviewers

No reviewers

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

1 participant

Notifications

Subscribe

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!41

No description provided.