Skip to content

feat: use cosmic-settings-daemon audio API#198

Open
mmstick wants to merge 6 commits into
masterfrom
varlink
Open

feat: use cosmic-settings-daemon audio API#198
mmstick wants to merge 6 commits into
masterfrom
varlink

Conversation

@mmstick
Copy link
Copy Markdown
Member

@mmstick mmstick commented May 12, 2026

Requires pop-os/cosmic-settings-daemon#135

This changes the behavior of cosmic-osd confirm-headphones to enable the settings daemon to take over the responsibility of selecting and applying profiles and their routes on behalf of cosmic-osd. The settings daemon will call cosmic-osd confirm-headphones --device ID to request a dialog and then cosmic-osd may use the device ID as an input to either SelectHeadphoneProfile or SelectHeadsetProfile. The settings daemon can then make better decisions about what to do since it's already tracking everything.

This will also drop the dependency on libpipewire, libpulse, and the cosmic-settings pulse subscription. The audio server events for the volume OSD notifications will now come from the settings-daemon's varlink API as well.


  • I have disclosed use of any AI generated code in my commit messages.
    • If you are using an LLM, and do not fully understand the changes it is making to the code base, do not create a PR.
    • In our experience, AI generated code often results in overly complex code that lacks enough context for a proper fix or feature inclusion. This results in considerably longer code reviews. Due to this, AI authored or partially authored PRs may be closed without comment.
  • I understand these changes in full and will be able to respond to review comments.
  • My change is accurately described in the commit message.
  • My contribution is tested and working as described.
  • I have read the Developer Certificate of Origin and certify my contribution under its conditions.

@jklgrasso
Copy link
Copy Markdown

I guess I missed that this requires another PR. It looks like pop-os/cosmic-settings-daemon#135 still has some work going on. I'll unassign myself for now

@jklgrasso jklgrasso removed their assignment May 15, 2026
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.

2 participants