Replace wpa_supplicant with NetworkManager for WiFi management

- Update WiFi module to use NetworkManager (nmcli) instead of wpa_supplicant
- Update command handlers to work with NetworkManager
- Add a new wifi-scan command to scan for available networks
- Add a new migrate_to_networkmanager.sh script to help with migration
- Update installation instructions and add NetworkManager to dependencies
- Add migration information to README

🤖 Generated with [Claude Code](https://claude.ai/code)
Co-Authored-By: Claude <noreply@anthropic.com>

Author: dbieber

README.md

@@ -66,6 +66,23 @@ See the [hardware guide](hardware.md) to know exactly what to buy.
 
 See the [installation instructions](installation.md) to get started.
 
+## Recent Updates
+
+### NetworkManager Support
+Go Note Go now uses NetworkManager instead of wpa_supplicant for WiFi management. 
+This provides a more robust networking solution with easier configuration options.
+
+If you're updating from a previous version, run the migration script:
+```bash
+sudo /path/to/GoNoteGo/scripts/migrate_to_networkmanager.sh
+```
+
+Benefits of NetworkManager:
+- Better handling of network connections
+- Improved stability and reliability
+- More configuration options
+- Better integration with modern Linux distributions
+
 ## History
 
 [Learn about Go Note Go's predecessor "Shh Shell" here.](https://davidbieber.com/projects/shh-shell/)

gonotego/command_center/wifi_commands.py

@@ -1,4 +1,4 @@
-"""WiFi command handlers for Go Note Go."""
+"""WiFi command handlers for Go Note Go using NetworkManager."""
 from gonotego.command_center import registry
 from gonotego.command_center import system_commands
 from gonotego.settings import wifi
@@ -11,117 +11,169 @@
 @register_command('wifi {} {}')
 @register_command('wpa {} {}')
 def add_wpa_wifi(ssid, psk):
-  if '"' in ssid or '"' in psk:
-    say('WiFi not set.')
-    return
-
-  # Load existing networks
-  networks = wifi.get_networks()
-
-  # Check if this network already exists
-  for network in networks:
-    if network.get('ssid') == ssid:
-      network['psk'] = psk
-      break
-  else:
-    # Network doesn't exist, add it
-    networks.append({'ssid': ssid, 'psk': psk})
-
-  # Save updated networks
-  wifi.save_networks(networks)
-  
-  # Update wpa_supplicant.conf
-  if wifi.update_wpa_supplicant_config():
-    wifi.reconfigure_wifi()
-    say(f'WiFi network {ssid} added.')
-  else:
-    say('Failed to update WiFi configuration.')
+    if '"' in ssid or '"' in psk:
+        say('WiFi not set.')
+        return
+
+    # Load existing networks
+    networks = wifi.get_networks()
+
+    # Check if this network already exists
+    for network in networks:
+        if network.get('ssid') == ssid:
+            network['psk'] = psk
+            break
+    else:
+        # Network doesn't exist, add it
+        networks.append({'ssid': ssid, 'psk': psk})
+
+    # Save updated networks
+    wifi.save_networks(networks)
+    
+    # Update NetworkManager connections
+    if wifi.update_network_connections():
+        wifi.reconfigure_wifi()
+        say(f'WiFi network {ssid} added.')
+    else:
+        say('Failed to update WiFi configuration.')
 
 
 @register_command('wifi {}')
 def add_wifi_no_psk(ssid):
-  if '"' in ssid:
-    say('WiFi not set.')
-    return
-  
-  # Load existing networks
-  networks = wifi.get_networks()
-  
-  # Check if this network already exists
-  for network in networks:
-    if network.get('ssid') == ssid:
-      network.pop('psk', None)  # Remove password if it exists
-      break
-  else:
-    # Network doesn't exist, add it
-    networks.append({'ssid': ssid})
-  
-  # Save updated networks
-  wifi.save_networks(networks)
-  
-  # Update wpa_supplicant.conf
-  if wifi.update_wpa_supplicant_config():
-    wifi.reconfigure_wifi()
-    say(f'Open WiFi network {ssid} added.')
-  else:
-    say('Failed to update WiFi configuration.')
+    if '"' in ssid:
+        say('WiFi not set.')
+        return
+    
+    # Load existing networks
+    networks = wifi.get_networks()
+    
+    # Check if this network already exists
+    for network in networks:
+        if network.get('ssid') == ssid:
+            network.pop('psk', None)  # Remove password if it exists
+            break
+    else:
+        # Network doesn't exist, add it
+        networks.append({'ssid': ssid})
+    
+    # Save updated networks
+    wifi.save_networks(networks)
+    
+    # Update NetworkManager connections
+    if wifi.update_network_connections():
+        wifi.reconfigure_wifi()
+        say(f'Open WiFi network {ssid} added.')
+    else:
+        say('Failed to update WiFi configuration.')
 
 
 @register_command('wifi-list')
 def list_wifi_networks():
-  networks = wifi.get_networks()
-  if not networks:
-    say('No WiFi networks configured.')
-    return
-  
-  network_list = [f"{i+1}. {network['ssid']}" for i, network in enumerate(networks)]
-  say('Configured WiFi networks: ' + ', '.join(network_list))
+    networks = wifi.get_networks()
+    if not networks:
+        say('No WiFi networks configured.')
+        return
+    
+    network_list = [f"{i+1}. {network['ssid']}" for i, network in enumerate(networks)]
+    say('Configured WiFi networks: ' + ', '.join(network_list))
 
 
 @register_command('wifi-migrate')
 def migrate_wifi_networks():
-  """Scan wpa_supplicant.conf and migrate existing networks to Redis."""
-  networks = wifi.migrate_networks_from_wpa_supplicant()
-  
-  if networks is None:
-    say('WiFi configuration file not found or error reading it.')
-    return
+    """Migrate existing networks from wpa_supplicant.conf to NetworkManager."""
+    networks = wifi.migrate_networks_from_wpa_supplicant()
 
-  # Save the extracted networks to Redis
-  if networks:
-    wifi.save_networks(networks)
-    say(f'Migrated {len(networks)} WiFi networks to settings.')
+    if networks is None:
+        say('WiFi configuration file not found or error reading it.')
+        return
+        
+    # Save the extracted networks to Redis
+    if networks:
+        wifi.save_networks(networks)
+        say(f'Migrated {len(networks)} WiFi networks to settings.')
+        
+        # Update NetworkManager connections
+        wifi.update_network_connections()
+        wifi.reconfigure_wifi()
+    else:
+        say('No WiFi networks found to migrate.')
+
+
+@register_command('wifi-nm-migrate')
+def migrate_from_networkmanager():
+    """Migrate existing NetworkManager connections to Go Note Go settings."""
+    networks = wifi.migrate_from_networkmanger()
 
-    # Update wpa_supplicant.conf
-    wifi.update_wpa_supplicant_config()
-  else:
-    say('No WiFi networks found to migrate.')
+    if networks is None:
+        say('Error reading NetworkManager connections.')
+        return
+        
+    # Save the extracted networks to Redis
+    if networks:
+        wifi.save_networks(networks)
+        say(f'Migrated {len(networks)} WiFi networks from NetworkManager to settings.')
+    else:
+        say('No WiFi networks found to migrate.')
 
 
 @register_command('wifi-remove {}')
 def remove_wifi_network(ssid):
-  networks = wifi.get_networks()
-  initial_count = len(networks)
-  
-  # Remove networks with matching SSID
-  networks = [network for network in networks if network.get('ssid') != ssid]
-  
-  if len(networks) < initial_count:
-    # Save updated networks
-    wifi.save_networks(networks)
+    networks = wifi.get_networks()
+    initial_count = len(networks)
+    
+    # Remove networks with matching SSID
+    networks = [network for network in networks if network.get('ssid') != ssid]
 
-    # Update wpa_supplicant.conf
-    if wifi.update_wpa_supplicant_config():
-      wifi.reconfigure_wifi()
-      say(f'WiFi network {ssid} removed.')
+    if len(networks) < initial_count:
+        # Save updated networks
+        wifi.save_networks(networks)
+        
+        # Update NetworkManager connections
+        if wifi.update_network_connections():
+            wifi.reconfigure_wifi()
+            say(f'WiFi network {ssid} removed.')
+        else:
+            say('Failed to update WiFi configuration.')
     else:
-      say('Failed to update WiFi configuration.')
-  else:
-    say(f'WiFi network {ssid} not found.')
+        say(f'WiFi network {ssid} not found.')
 
 
 @register_command('reconnect')
 @register_command('wifi-refresh')
 @register_command('wifi-reconfigure')
 def reconfigure_wifi():
-  wifi.reconfigure_wifi()
+    wifi.reconfigure_wifi()
+
+
+@register_command('wifi-scan')
+def scan_wifi_networks():
+    """Scan for available WiFi networks."""
+    try:
+        # Ensure the WiFi adapter is on
+        shell('sudo nmcli radio wifi on')
+        
+        # Scan for networks
+        result = shell('nmcli -t -f SSID,SIGNAL,SECURITY device wifi list')
+        
+        networks = []
+        for line in result.stdout.strip().split('\n'):
+            if line:
+                parts = line.split(':', 2)
+                if len(parts) >= 3 and parts[0]:  # Ensure SSID is not empty
+                    ssid, signal, security = parts
+                    networks.append((ssid, int(signal), security))
+        
+        # Sort by signal strength (descending)
+        networks.sort(key=lambda x: x[1], reverse=True)
+        
+        # Take the top 5 networks
+        top_networks = networks[:5]
+        
+        if top_networks:
+            network_list = [f"{i+1}. {ssid} ({signal}%)" 
+                           for i, (ssid, signal, _) in enumerate(top_networks)]
+            say('Available WiFi networks: ' + ', '.join(network_list))
+        else:
+            say('No WiFi networks found.')
+    except Exception as e:
+        say(f'Error scanning for WiFi networks: {str(e)}')

gonotego/settings/server.py

@@ -176,7 +176,7 @@ def do_POST(self):
             # Handle WiFi networks specially
             if key == 'WIFI_NETWORKS':
               wifi.save_networks(value)
-              wifi.update_wpa_supplicant_config()
+              wifi.update_network_connections()
               wifi.reconfigure_wifi()
             else:
               # For all other settings, just use settings.set