update VPN protocol version; fmt & lint

Author: spikecurtis

Coder-Desktop/VPNLib/Speaker.swift

@@ -89,7 +89,9 @@ public actor Speaker<SendMsg: RPCMessage & Message, RecvMsg: RPCMessage & Messag
     /// Does the VPN Protocol handshake and validates the result
     public func handshake() async throws(HandshakeError) {
         let hndsh = Handshaker(writeFD: writeFD, dispatch: dispatch, queue: queue, role: role)
-        // ignore the version for now because we know it can only be 1.0
+        // ignore the version for now because we know it can only be 1.0 or 1.1.
+        // 1.1 adds support for telemetry to StartRequest, but since setting these
+        // fields won't adversely affect a 1.0 speaker, we set them regardless.
         try _ = await hndsh.handshake()
     }
 
@@ -178,7 +180,7 @@ actor Handshaker {
 
     init(writeFD: FileHandle, dispatch: DispatchIO, queue: DispatchQueue,
          role: ProtoRole,
-         versions: [ProtoVersion] = [.init(1, 0)])
+         versions: [ProtoVersion] = [.init(1, 1)])
     {
         self.writeFD = writeFD
         self.dispatch = dispatch

Coder-Desktop/VPNLib/TelemetryEnricher.swift

@@ -23,8 +23,8 @@ public struct TelemetryEnricher {
         var req = original
         req.deviceOs = "macOS"
         req.deviceID = deviceID
-        if version != nil {
-            req.coderDesktopVersion = version!
+        if let version {
+            req.coderDesktopVersion = version
         }
         return req
     }

Coder-Desktop/VPNLibTests/SpeakerTests.swift

@@ -36,7 +36,7 @@ struct SpeakerTests: Sendable {
     @Test func handshake() async throws {
         async let v = handshaker.handshake()
         try await uut.handshake()
-        #expect(try await v == ProtoVersion(1, 0))
+        #expect(try await v == ProtoVersion(1, 1))
     }
 
     @Test func handleSingleMessage() async throws {

Coder-Desktop/VPNLibTests/TelemetryEnricherTests.swift

@@ -16,7 +16,7 @@ struct TelemetryEnricherTests {
         #expect(try enriched.coderDesktopVersion.contains(Regex(#"^\d+\.\d+\.\d+$"#)))
         let deviceID = enriched.deviceID
         #expect(!deviceID.isEmpty)
-        
+
         // check we get the same deviceID from a new enricher
         let enricher1 = TelemetryEnricher()
         enriched = enricher1.enrich(original)