Some fixes

src/provision.ts

@@ -5,8 +5,11 @@ export const startProvisioning = async (button: SerialLaunchButton) => {
   let port: SerialPort | undefined;
   try {
     port = await navigator.serial.requestPort();
-  } catch (err) {
-    console.error("User cancelled request", err);
+  } catch (err: any) {
+    if ((err as DOMException).name === "NotFoundError") {
+      return;
+    }
+    alert(`Error: ${err.message}`);
     return;
   }
 

src/serial-provision-dialog.ts

@@ -286,16 +286,19 @@ class SerialProvisionDialog extends LitElement {
   }
 
   private async _provision() {
-    this._hasProvisioned = true;
     this._busy = true;
     try {
-      // No need to do error handling because we listen for `error-changed` events
       await this._client!.provision(
         this._selectedSsid === -1
           ? this._inputSSID.value
           : this._ssids![this._selectedSsid].name,
         this._inputPassword.value
       );
+      this._hasProvisioned = true;
+      this._showProvisionForm = false;
+    } catch (err) {
+      // No need to do error handling because we listen for `error-changed` events
+      console.log(err);
     } finally {
       this._busy = false;
     }
@@ -343,7 +346,6 @@ class SerialProvisionDialog extends LitElement {
     const client = new ImprovSerial(this.port!, this.logger);
     client.addEventListener("state-changed", () => {
       this._state = "IMPROV-STATE";
-      this._showProvisionForm = false;
       this.requestUpdate();
     });
     client.addEventListener("error-changed", () => this.requestUpdate());

src/serial.ts

@@ -165,16 +165,20 @@ export class ImprovSerial extends EventTarget {
     this.nextUrl = response[0];
   }
 
-  public async scan() {
-    const ssids = await this._sendRPCWithMultipleResponses(
+  public async scan(): Promise<Ssid[]> {
+    const results = await this._sendRPCWithMultipleResponses(
       ImprovSerialRPCCommand.REQUEST_WIFI_NETWORKS,
       []
     );
-    return ssids.map(([name, rssi, secured]) => ({
+    const ssids = results.map(([name, rssi, secured]) => ({
       name,
       rssi: parseInt(rssi),
       secured: secured === "YES",
     }));
+    ssids.sort((a, b) =>
+      a.name.toLocaleLowerCase().localeCompare(b.name.toLocaleLowerCase())
+    );
+    return ssids;
   }
 
   private _sendRPC(command: ImprovSerialRPCCommand, data: number[]) {