Skip to content

Satellites : régler le volume à la voix - #38

Merged
lfpoulain merged 1 commit into
lfpoulain:betafrom
NeuTroNBZh:satellite-volume-voix
Nov 9, 2026
Merged

lfpoulain merged 1 commit into
lfpoulain:betafrom
NeuTroNBZh:satellite-volume-voix

Conversation

@NeuTroNBZh

Copy link
Copy Markdown
Contributor

Ticket : #37

But

« Parle moins fort », « parle plus fort », « mets le volume à 5 » pendant une conversation de satellite.

Ce qui change

  • server/satellite-volume.ts : outil satellite__volume (level de 0 à 10, ou change louder / quieter ; exactement l'un des deux) et setSatelliteVolume(), qui enregistre le volume (0 à 100 en base) puis appelle SatelliteHub.refresh(), ce qui envoie settings au satellite connecté. Deux crans par « plus fort » / « moins fort », bornés à 0 et 10 ; « déjà au maximum » est dit.
  • LiveTransport.attachSatelliteVolume(), sur le modèle d'attachSatelliteOutput(). L'outil n'est proposé que dans une conversation de satellite (voiceOnly), et il est appliqué au satellite de la conversation, jamais à un autre. Consignes ajoutées à la conversation et à la délégation quand l'outil est présent.
  • Documentation (docs/fonctionnalites/satellites.md) et CHANGELOG.md.

Vérifications

  • npm run check : OK.
  • satellite-volume.test.ts (nouveau), satellite-conversation.test.ts, live.test.ts, satellites.test.ts : 79/79. Les nouveaux tests couvrent : niveau, plus fort, moins fort, bornes ; 8 demandes invalides refusées sans rien changer ; appel routé avec l'id du satellite de la conversation ; outil absent d'une conversation du miroir.

Nouvel outil satellite__volume, proposé seulement dans une conversation
de satellite : level (0 à 10) ou change (louder, quieter, deux crans).
Le volume est enregistré comme depuis l'administration et envoyé au
satellite par settings (SatelliteHub.refresh). Branché par
LiveTransport.attachSatelliteVolume, comme la sortie audio.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Copilot AI balanced review requested due to automatic review settings November 8, 2026 22:44

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has insufficient remaining quota. Try requesting a review at a lower effort level.

@lfpoulain
lfpoulain merged commit a873c4b into lfpoulain:beta Nov 9, 2026
@lfpoulain

Copy link
Copy Markdown
Owner

Merci, fusionné dans beta et publié dans 0.3.0. Conflit de CHANGELOG.md résolu, tests relancés. J'ai particulièrement aimé la validation stricte et le test à 8 cas.

Deux points à regarder :

  • « mets le volume à 0 » règle le volume à 0 de façon persistante, partagé avec l'administration. À vérifier côté firmware si cela rend aussi muettes les alarmes, sinon à documenter.
  • La fiche du satellite dans l'administration ne se met à jour qu'au rechargement après un réglage vocal.

@NeuTroNBZh

Copy link
Copy Markdown
Contributor Author

Merci ! Les remarques de cette PR sont reprises dans #42 (avec celles de #26, #34, #36 et #38).

lfpoulain pushed a commit that referenced this pull request Nov 9, 2026
…délai

Remarques faites sur #26, #34, #36 et #38 (fusionnées dans 0.3.0) :
- #26 : la couleur demandée et la couleur relue sont comparées à pleine
  intensité (Home Assistant relit [128, 0, 128] en [255, 0, 255]) ; le
  faux Home Assistant des tests relit comme lui ; test de la route admin
  POST /control avec rgbColor ; architecture.md cite couleur et blanc.
- #34 : confidentialite.md, le bouton coupe le micro en veille et ferme
  la conversation pendant une conversation.
- #36 : le délai propre au satellite est pris à satelliteWake, un
  changement vaut pour la conversation suivante (satellites.md,
  administration.md).
- #38 : sonneries et voix partagent le volume du codec
  (board_set_volume → esp_codec_dev_set_out_vol) : au volume 0, l'outil
  le rappelle et la doc le dit ; la fiche du satellite suit un réglage
  vocal (resynchronisée sur la liste rechargée hors réglage en cours).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.

3 participants