logger, helper for dupe network lookup logic, crates.io docs and UI on saved connections

CHANGELOG.md

@@ -9,20 +9,24 @@
 - **nmrs-gui**: Renamed crate from `nmrs-ui` to `nmrs-gui`
 
 ### Added
-- Core: `StateReason` enum and `reason_to_error()` for mapping NetworkManager failure codes to typed errors ([#82](https://github.com/cachebag/nmrs/issues/82))
+- Core: `StateReason` enum and `reason_to_error()` for mapping NetworkManager failure codes to typed errors ([#82](https://github.com/cachebag/nmrs/issues/82), [#85](https://github.com/cachebag/nmrs/issues/85))
 - Core: Comprehensive documentation across all modules ([#82](https://github.com/cachebag/nmrs/issues/82))
+- Core: Logging support via `log` crate facade ([#87](https://github.com/cachebag/nmrs/issues/87))
 - UI: Pre-defined themes (Catppuccin, Dracula, Gruvbox, Nord, Tokyo) ([#106](https://github.com/cachebag/nmrs/issues/106))
 - CLI: `--version` flag with build hash extraction ([#108](https://github.com/cachebag/nmrs/issues/108))
 
 ### Changed
 - Core: Decomposed `connect()` into smaller helper functions ([#81](https://github.com/cachebag/nmrs/issues/81))
 - Core: Extracted disconnect + wait logic to unified helper ([#79](https://github.com/cachebag/nmrs/issues/79))
 - Core: Unified state polling logic ([#80](https://github.com/cachebag/nmrs/issues/80))
+- Core: Eliminated network lookup duplication via shared helper function ([#83](https://github.com/cachebag/nmrs/issues/83))
+- Core: Replaced `eprintln!` with structured logging (`debug!`, `info!`, `warn!`, `error!`) ([#87](https://github.com/cachebag/nmrs/issues/87))
 
 ### Fixed
-- Core: Auth error mapping now properly distinguishes supplicant failures, DHCP errors, and timeouts ([#82](https://github.com/cachebag/nmrs/issues/82), [#116](https://github.com/cachebag/nmrs/issues/116))
+- Core: Auth error mapping now properly distinguishes supplicant failures, DHCP errors, and timeouts ([#82](https://github.com/cachebag/nmrs/issues/82), [#85](https://github.com/cachebag/nmrs/issues/85), [#116](https://github.com/cachebag/nmrs/issues/116))
 - Core: `bitrate` property now fetches real connection speeds ([#110](https://github.com/cachebag/nmrs/issues/110))
 - UI: Re-aligned refresh button ([#111](https://github.com/cachebag/nmrs/issues/111))
+- UI: Show connection status when connecting with saved credentials ([#61](https://github.com/cachebag/nmrs/issues/61))
 
 ## [0.3.0-beta] - 2025-12-08
 ### Fixed

nmrs-gui/Cargo.toml

@@ -4,6 +4,7 @@ version = "0.4.0"
 edition = "2024"
 
 [dependencies]
+log = "0.4.29"
 nmrs = { path = "../nmrs" }
 tokio = { version = "1.47.1", features = ["full"] }
 gtk = { version = "0.10.1", package = "gtk4" }

nmrs-gui/src/ui/connect.rs

@@ -3,6 +3,7 @@ use gtk::{
     ApplicationWindow, Box as GtkBox, Dialog, Entry, EventControllerKey, Label, Orientation,
     prelude::*,
 };
+use log::{debug, error};
 use nmrs::{
     NetworkManager,
     models::{EapMethod, EapOptions, Phase2, WifiSecurity},
@@ -23,7 +24,7 @@ pub fn connect_modal(
         if let Some(current) = nm.current_ssid().await
             && current == ssid_owned
         {
-            println!("Already connected to {current}, skipping modal");
+            debug!("Already connected to {current}, skipping modal");
             return;
         }
 
@@ -128,17 +129,17 @@ fn draw_connect_modal(
                     WifiSecurity::WpaPsk { psk: pwd }
                 };
 
-                println!("Calling nm.connect() for '{ssid}'");
+                debug!("Calling nm.connect() for '{ssid}'");
                 match nm.connect(&ssid, creds).await {
                     Ok(_) => {
-                        println!("nm.connect() succeeded!");
+                        debug!("nm.connect() succeeded!");
                         status.set_text("✓ Connected!");
                         on_success();
                         glib::timeout_future_seconds(1).await;
                         dialog.close();
                     }
                     Err(err) => {
-                        eprintln!("nm.connect() failed: {err}");
+                        error!("nm.connect() failed: {err}");
                         let err_str = err.to_string().to_lowercase();
                         if err_str.contains("authentication")
                             || err_str.contains("supplicant")

nmrs-gui/src/ui/networks.rs

@@ -126,10 +126,14 @@ impl NetworkRowController {
                     let have = nm_c.has_saved_connection(&ssid_c).await.unwrap_or(false);
 
                     if have {
+                        status_c.set_text(&format!("Connecting to {}...", ssid_c));
                         window_c.set_sensitive(false);
                         let creds = WifiSecurity::WpaPsk { psk: "".into() };
                         match nm_c.connect(&ssid_c, creds).await {
-                            Ok(_) => on_success_c(),
+                            Ok(_) => {
+                                status_c.set_text("");
+                                on_success_c();
+                            }
                             Err(e) => status_c.set_text(&format!("Failed to connect: {e}")),
                         }
                         window_c.set_sensitive(true);
@@ -143,10 +147,14 @@ impl NetworkRowController {
                         );
                     }
                 } else {
+                    status_c.set_text(&format!("Connecting to {}...", ssid_c));
                     window_c.set_sensitive(false);
                     let creds = WifiSecurity::Open;
                     match nm_c.connect(&ssid_c, creds).await {
-                        Ok(_) => on_success_c(),
+                        Ok(_) => {
+                            status_c.set_text("");
+                            on_success_c();
+                        }
                         Err(e) => status_c.set_text(&format!("Failed to connect: {e}")),
                     }
                     window_c.set_sensitive(true);

nmrs/Cargo.toml

@@ -2,8 +2,15 @@
 name = "nmrs"
 version = "0.4.0"
 edition = "2024"
+description = "A Rust library for NetworkManager over D-Bus"
+license = "MIT"
+repository = "https://github.com/cachebag/nmrs"
+documentation = "https://docs.rs/nmrs"
+keywords = ["networkmanager", "dbus", "wifi", "linux", "networking"]
+categories = ["api-bindings", "asynchronous"]
 
 [dependencies]
+log = "0.4.29"
 zbus = "5.11.0"
 zvariant = "5.7.0"
 serde = { version = "1", features = ["derive"] }