Skip to content

feat(engine): play a distinct sound when a clip with an after part ends - #127

Merged
Palawizard merged 3 commits into
devfrom
feature/clip-finished-sound
Oct 27, 2026
Merged

Palawizard merged 3 commits into
devfrom
feature/clip-finished-sound

Conversation

@Palawizard

Copy link
Copy Markdown
Owner

Branches

  • feature/, fix/, chore/ → dev

Phase / tâche

v1.4.0 — retour 5 de Palawi : un son à part pour la fin du « après »

Ce qui change

  • chime::Sound::ClipFinished : La5 → Mi6 → La6 (candidat A choisi par Palawi après écoute : l'appui se « résout » à l'octave), même synthèse et même volume que le carillon. Le carillon d'appui ne change pas.
  • clip_saved porte origin (replay, post_roll, recording) : le shell joue le carillon pour un replay, le son « clip terminé » à la fin d'un « après », et plus rien pour un enregistrement (il a déjà son son de fin ; avant, le carillon de sauvegarde pouvait le couper, PlaySoundW asynchrone interrompant le son en cours). Toujours soumis à « Son de confirmation ».
  • CLI replay --chime suit la même règle ; find-chimes.py connaît le nouveau son.

Critères d'acceptation

  • Son « clip terminé » distinct de l'appui et des sons d'enregistrement, même timbre et volume
  • Test « tous les sons sont différents » (all_sounds_differ) + crête à ±15 % du carillon
  • Soumis au réglage « Son de confirmation »
  • Exclu des clips en « Tous les sons sauf PalaClips »

Comment ça a été testé

  • Automatique : cargo clippy --workspace --all-targets -- -D warnings, cargo test, npm test, npm run build
  • scripts/feedback-sounds.ps1 -Mode all (journal feedback-sounds-2026-10-27_22-33.txt) : mode « sortie précise » → « clip terminé » trouvé à 0,98 de corrélation, distinct de « sauvegarde » ; « Tous les sons sauf PalaClips » → 0 son de PalaClips, témoin d'un autre processus entendu ; pistes séparées → 0.

Docs mises à jour

  • SPEC RPL-10, ARCHITECTURE § IPC (CHANGELOG dans la PR de version)

@Palawizard
Palawizard merged commit ea6fab3 into dev Oct 27, 2026
1 check passed
@Palawizard
Palawizard deleted the feature/clip-finished-sound branch October 27, 2026 21:50
@Palawizard Palawizard mentioned this pull request Oct 28, 2026
1 task done
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant