Add hpmute command for headphone output mute #2

Open

thebrianbug wants to merge 0 commits from thebrianbug/add-hpmute-command into main

thebrianbug commented

2026-02-24 19:02:46 +00:00

(Migrated from github.com)

Note

This PR was written with the help of an LLM (Claude).

Summary

Expose existing get_hp_mute() / set_hp_mute() methods as a new hpmute CLI command
Add toggle_hp_mute() helper method to the Wave3 class
Update usage docstring to document the new command

Usage

wave3ctl hpmute              Toggle headphone mute
wave3ctl hpmute on|off       Set headphone mute

Motivation

Setting volume 0 does not silence the headphone jack — audio still passes through. The headphone mute control (Entity 5, Mute selector) is already readable via discover and the backend methods exist, but there was no CLI command to use them.

Closes #1

Test plan

wave3ctl hpmute on — verify headphone jack goes silent
wave3ctl hpmute off — verify audio resumes
wave3ctl hpmute — verify toggle behavior
wave3ctl status — verify mute state displays correctly
wave3ctl --help — verify updated usage text

> [!NOTE] > This PR was written with the help of an LLM (Claude). ## Summary - Expose existing `get_hp_mute()` / `set_hp_mute()` methods as a new `hpmute` CLI command - Add `toggle_hp_mute()` helper method to the `Wave3` class - Update usage docstring to document the new command ## Usage ``` wave3ctl hpmute Toggle headphone mute wave3ctl hpmute on|off Set headphone mute ``` ## Motivation Setting `volume 0` does not silence the headphone jack — audio still passes through. The headphone mute control (Entity 5, Mute selector) is already readable via `discover` and the backend methods exist, but there was no CLI command to use them. Closes #1 ## Test plan - [ ] `wave3ctl hpmute on` — verify headphone jack goes silent - [ ] `wave3ctl hpmute off` — verify audio resumes - [ ] `wave3ctl hpmute` — verify toggle behavior - [ ] `wave3ctl status` — verify mute state displays correctly - [ ] `wave3ctl --help` — verify updated usage text

x4ndr0m3d4x (Migrated from github.com) reviewed

2026-02-24 19:02:46 +00:00

This pull request can be merged automatically.

This branch is out-of-date with the base branch

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 thebrianbug/add-hpmute-command:thebrianbug/add-hpmute-command

git switch thebrianbug/add-hpmute-command

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 main

git merge --no-ff thebrianbug/add-hpmute-command

git switch thebrianbug/add-hpmute-command

git rebase main

git switch main

git merge --ff-only thebrianbug/add-hpmute-command

git switch thebrianbug/add-hpmute-command

git rebase main

git switch main

git merge --no-ff thebrianbug/add-hpmute-command

git switch main

git merge --squash thebrianbug/add-hpmute-command