From 903a1e45e4202ecda5f563054b8e9b43f1dd9305 Mon Sep 17 00:00:00 2001
From: Claude <noreply@anthropic.com>
Date: Fri, 16 Oct 2026 15:35:06 +0000
Subject: [PATCH] WiFi: drop stale netplan wifis entries before apply

netplan apply waits ~90s for every wifis: interface in every file and then
fails if one doesn't exist (e.g. installer config naming wlan0 while the
adapter is wlp1s0). Remove such entries (backup kept as *.kiosk-bak) before
applying, in both the menu and kiosk-wifi-helper.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01A97KUggSE3rBL8veDM5Mr4
---
 menus/wifi.sh                                 | 35 +++++++++++++++++++
 .../files/usr/local/bin/kiosk-wifi-helper     | 35 +++++++++++++++++++
 2 files changed, 70 insertions(+)

diff --git a/menus/wifi.sh b/menus/wifi.sh
index 8ce9efe..ac17415 100644
--- a/menus/wifi.sh
+++ b/menus/wifi.sh
@@ -34,6 +34,40 @@
 
 WIFI_NETPLAN_FILE="$NETPLAN_DIR/60-kiosk-wifi.yaml"
 
+# netplan apply starts a netplan-wpa-<iface> unit for every wifis: entry
+# in every file and waits for that device - 90s, then the whole apply
+# fails - if one names an interface this machine doesn't have (the
+# installer's own config can carry e.g. "wlan0" while the adapter here is
+# "wlp1s0"). Take such entries out of the other netplan files first
+# (originals kept as *.kiosk-bak, which netplan ignores).
+wifi_drop_stale_entries() {
+    local f
+    for f in "$NETPLAN_DIR"/*.yaml; do
+        [[ -f "$f" && "$f" != "$WIFI_NETPLAN_FILE" ]] || continue
+        python3 -I - "$f" <<'PYEOF' 2>/dev/null || true
+import os, shutil, sys, yaml
+p = sys.argv[1]
+try:
+    d = yaml.safe_load(open(p)) or {}
+    w = (d.get("network") or {}).get("wifis") or {}
+    stale = [n for n in w if not os.path.exists("/sys/class/net/" + str(n))]
+    if stale:
+        shutil.copy2(p, p + ".kiosk-bak")
+        for n in stale:
+            del w[n]
+        if not w:
+            del d["network"]["wifis"]
+        mode = os.stat(p).st_mode & 0o777
+        with open(p, "w") as fh:
+            yaml.safe_dump(d, fh, default_flow_style=False)
+        os.chmod(p, mode)
+        print("Removed stale WiFi entries from %s: %s" % (p, ", ".join(map(str, stale))))
+except Exception:
+    pass
+PYEOF
+    done
+}
+
 wifi_menu() {
     local rc=0
     wifi_configure || rc=$?
@@ -293,6 +327,7 @@ WATCHEOF
     sudo chmod 0600 "$netplan_file"
     rm -f "$temp_plan"
 
+    sudo bash -c "$(declare -f wifi_drop_stale_entries); NETPLAN_DIR='$NETPLAN_DIR' WIFI_NETPLAN_FILE='$netplan_file' wifi_drop_stale_entries"
     echo "Applying configuration..."
     local netplan_log
     netplan_log=$(mktemp)
diff --git a/provision/files/usr/local/bin/kiosk-wifi-helper b/provision/files/usr/local/bin/kiosk-wifi-helper
index 9968543..6c1ab92 100755
--- a/provision/files/usr/local/bin/kiosk-wifi-helper
+++ b/provision/files/usr/local/bin/kiosk-wifi-helper
@@ -168,6 +168,40 @@ do_scan() {
         | {ok: true, iface: $i, networks: .}' < "$TMP_SCAN"
 }
 
+# netplan apply starts a netplan-wpa-<iface> unit for every wifis: entry
+# in every file and waits for that device - 90s, then the whole apply
+# fails - if one names an interface this machine doesn't have (the
+# installer's own config can carry e.g. "wlan0" while the adapter here is
+# "wlp1s0"). Take such entries out of the other netplan files first
+# (originals kept as *.kiosk-bak, which netplan ignores).
+drop_stale_entries() {
+    local f
+    for f in "$NETPLAN_DIR"/*.yaml; do
+        [[ -f "$f" && "$f" != "$WIFI_NETPLAN_FILE" ]] || continue
+        python3 -I - "$f" <<'PYEOF' 2>/dev/null || true
+import os, shutil, sys, yaml
+p = sys.argv[1]
+try:
+    d = yaml.safe_load(open(p)) or {}
+    w = (d.get("network") or {}).get("wifis") or {}
+    stale = [n for n in w if not os.path.exists("/sys/class/net/" + str(n))]
+    if stale:
+        shutil.copy2(p, p + ".kiosk-bak")
+        for n in stale:
+            del w[n]
+        if not w:
+            del d["network"]["wifis"]
+        mode = os.stat(p).st_mode & 0o777
+        with open(p, "w") as fh:
+            yaml.safe_dump(d, fh, default_flow_style=False)
+        os.chmod(p, mode)
+        print("Removed stale WiFi entries from %s: %s" % (p, ", ".join(map(str, stale))))
+except Exception:
+    pass
+PYEOF
+    done
+}
+
 do_connect() {
     need_iface
     local input ssid password
@@ -219,6 +253,7 @@ do_connect() {
     chmod 0600 "$new"
     mv -f "$new" "$WIFI_NETPLAN_FILE"
 
+    drop_stale_entries >/dev/null
     local err
     if ! err=$(netplan apply 2>&1); then
         restore "$backup"