Apply Qodo /improve pass 2: case-insensitive device-path matching

Windows HID device paths are case-insensitive and casing can vary across
enumerations; match paths with OrdinalIgnoreCase in both ConnectByPathAsync and
the bootloader wait (consistent with the serial-filter comparison). Real device
paths never differ only by case, so this cannot mis-target.

Declined the redundant path re-validation suggestion (importance 2): the
enumerated device is already matched by targetDevicePath in
WaitForBootloaderDeviceAsync, so re-checking before connect adds nothing.

Tests: +1 (case-insensitive match).

Claude-Session: https://claude.ai/code/session_01V5TeJuikExLKXzxvYcQNxB

Author: cptkoolbeenz

src/Daqifi.Core.Tests/Communication/Transport/HidLibraryTransportTests.cs

@@ -124,6 +124,20 @@ public async Task ConnectByPathAsync_WithMatchingPath_OpensThatExactDevice()
         Assert.Equal(1, deviceB.OpenCount);
     }
 
+    [Fact]
+    public async Task ConnectByPathAsync_MatchesPathCaseInsensitively()
+    {
+        // Windows HID device paths are case-insensitive; casing can vary across enumerations.
+        var device = new FakeHidTransportDevice(0x04D8, 0x003C, "PATH-A", string.Empty);
+        var platform = new FakeHidPlatform([device]);
+        using var transport = new HidLibraryTransport(platform);
+
+        await transport.ConnectByPathAsync("path-a");
+
+        Assert.True(transport.IsConnected);
+        Assert.Equal(1, device.OpenCount);
+    }
+
     [Fact]
     public async Task ConnectByPathAsync_WhenNoDeviceMatchesPath_ThrowsIOException()
     {

src/Daqifi.Core/Communication/Transport/HidLibraryTransport.cs

@@ -123,8 +123,11 @@ public async Task ConnectByPathAsync(string devicePath, CancellationToken cancel
             throw new ArgumentException("Device path cannot be null or empty.", nameof(devicePath));
         }
 
+        // Windows HID device paths are case-insensitive and their casing can vary across enumerations,
+        // so match case-insensitively (real device paths never differ only by case, so this can't
+        // mis-target). Mirrors the OrdinalIgnoreCase used for the serial filter in ConnectAsync.
         await ConnectMatchingDeviceAsync(
-            candidate => string.Equals(candidate.DevicePath, devicePath, StringComparison.Ordinal),
+            candidate => string.Equals(candidate.DevicePath, devicePath, StringComparison.OrdinalIgnoreCase),
             $"Path={devicePath}",
             cancellationToken).ConfigureAwait(false);
     }

src/Daqifi.Core/Firmware/FirmwareUpdateService.cs

@@ -1113,7 +1113,7 @@ private async Task<HidDeviceInfo> WaitForBootloaderDeviceAsync(
             var match = targetDevicePath == null
                 ? devices.FirstOrDefault()
                 : devices.FirstOrDefault(d =>
-                    string.Equals(d.DevicePath, targetDevicePath, StringComparison.Ordinal));
+                    string.Equals(d.DevicePath, targetDevicePath, StringComparison.OrdinalIgnoreCase));
             if (match != null)
             {
                 return match;