fix(#485): only output visible SSIDs

docs/src/api/models.md

@@ -186,7 +186,7 @@ let saved_vpns = snapshot.saved_vpn_map();
 let applet = snapshot.applet_summary();
 ```
 
-`wifi_groups()` groups access points by `(interface, ssid)`, keeps every BSSID,
+`wifi_groups()` groups visible access points by `(interface, ssid)`, keeps every BSSID,
 marks known networks from matching saved profiles, and marks the active group
 from typed active Wi-Fi connections.
 

nmrs/src/api/models/snapshot.rs

@@ -74,10 +74,12 @@ impl NetworkSnapshot {
     pub fn wifi_groups(&self) -> Vec<WifiNetworkGroup> {
         let mut grouped: HashMap<(String, String), Vec<AccessPoint>> = HashMap::new();
         for ap in &self.access_points {
-            grouped
-                .entry((ap.interface.clone(), ap.ssid.clone()))
-                .or_default()
-                .push(ap.clone());
+            if !ap.ssid.is_empty() {
+                grouped
+                    .entry((ap.interface.clone(), ap.ssid.clone()))
+                    .or_default()
+                    .push(ap.clone());
+            }
         }
 
         let mut groups = grouped
@@ -668,4 +670,19 @@ mod tests {
         assert_eq!(vpns["active-vpn"].kind, Some(VpnKind::Plugin));
         assert_eq!(vpns["wg-vpn"].kind, Some(VpnKind::WireGuard));
     }
+
+    #[test]
+    fn hidden_networks_not_shown() {
+        let snapshot = snapshot(
+            vec![
+                ap("wlan0", "", "AA:AA:AA:AA:AA:01", 70),
+                ap("wlan1", "Cafe", "BB:BB:BB:BB:BB:01", 90),
+            ],
+            Vec::new(),
+            Vec::new(),
+            Vec::new(),
+        );
+        let groups = snapshot.wifi_groups();
+        assert_eq!(groups.len(), 1);
+    }
 }