From bb84c1f8b21b896bcb73f4ee7be0fab590b8571d Mon Sep 17 00:00:00 2001 From: Rishabh672003 <53911515+Rishabh672003@users.noreply.github.com> Date: Sun, 27 Sep 2026 18:52:01 +0530 Subject: [PATCH] fix(mpris): match fully qualified D-Bus signal names in the watcher godbus reports Signal.Name as "interface.member" (e.g. "org.freedesktop.DBus.Properties.PropertiesChanged"), but the watcher switch compared against bare member names, so no case ever matched and the PropertiesChanged, Seeked and NameOwnerChanged handlers never ran. AI-assisted. --- internal/plugins/mpris/watcher.go | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/internal/plugins/mpris/watcher.go b/internal/plugins/mpris/watcher.go index 23066b0..273b946 100644 --- a/internal/plugins/mpris/watcher.go +++ b/internal/plugins/mpris/watcher.go @@ -98,11 +98,11 @@ func (p *MPRISPlugin) runDBusWatcher(ctx context.Context) error { } switch sig.Name { - case "NameOwnerChanged": + case "org.freedesktop.DBus.NameOwnerChanged": p.handleNameOwnerChanged(sig, conn, uniqueToDisplay) - case "Seeked": + case "org.mpris.MediaPlayer2.Player.Seeked": p.handleSeeked(sig, uniqueToDisplay) - case "PropertiesChanged": + case "org.freedesktop.DBus.Properties.PropertiesChanged": p.handlePropertiesChanged(sig, uniqueToDisplay) } }