👷 pages: refactor kwin debug console dbus call

use native dbus impl instead of depending on external dbus client

Author: vnepogodin

src/kwin_dbus.rs

@@ -0,0 +1,21 @@
+use anyhow::{Context, Result};
+use tokio::runtime::Runtime;
+use zbus::Connection;
+
+#[zbus::proxy(interface = "org.kde.KWin", default_service = "org.kde.KWin", default_path = "/KWin")]
+pub trait KWin {
+    /// showDebugConsole method
+    #[zbus(name = "showDebugConsole")]
+    fn show_debug_console(&self) -> zbus::Result<()>;
+}
+
+pub fn launch_kwin_debug_window() -> Result<()> {
+    let rt = Runtime::new().context("Failed to initialize tokio runtime")?;
+    rt.block_on(async move {
+        let connection = Connection::session().await?;
+        let kwin_client = KWinProxy::new(&connection).await?;
+        kwin_client.show_debug_console().await?;
+
+        anyhow::Ok(())
+    })
+}

src/main.rs

@@ -7,6 +7,7 @@ mod config;
 mod embed_data;
 mod gresource;
 mod installer;
+mod kwin_dbus;
 mod localization;
 mod logger;
 mod pages;

src/pages.rs

@@ -1,7 +1,7 @@
 use crate::application_browser::ApplicationBrowser;
 use crate::systemd_units::SystemdUnits;
 use crate::utils::PacmanWrapper;
-use crate::{fl, systemd_units, utils};
+use crate::{fl, kwin_dbus, systemd_units, utils};
 
 use std::boxed::Box;
 use std::fmt::Write;
@@ -262,10 +262,9 @@ fn get_nm_connections() -> Vec<String> {
 }
 
 fn launch_kwin_debug_window() {
-    let _ = Exec::cmd("qdbus6")
-        .args(&["org.kde.KWin", "/KWin", "org.kde.KWin.showDebugConsole"])
-        .join()
-        .unwrap();
+    if let Err(kwin_err) = kwin_dbus::launch_kwin_debug_window() {
+        error!("Failed to launch kwin debug window: {kwin_err}");
+    }
 }
 
 fn create_fixes_section(builder: &Builder) -> gtk::Box {