Refactor USB handling to improve connection management and error cleanup

lib/connector/meshcore_connector.dart

@@ -981,7 +981,6 @@ class MeshCoreConnector extends ChangeNotifier {
     _selfInfoRetryTimer = null;
     _hasReceivedDeviceInfo = false;
     _pendingInitialChannelSync = false;
-    _hasReceivedDeviceInfo = false;
   }
 
   bool get _shouldAutoReconnect =>

lib/services/usb_serial_frame_codec.dart

@@ -27,6 +27,7 @@ class UsbSerialDecodedPacket {
 
 class UsbSerialFrameDecoder {
   final List<int> _rxBuffer = <int>[];
+  int _startIndex = 0;
 
   List<UsbSerialDecodedPacket> ingest(Uint8List bytes) {
     if (bytes.isEmpty) {
@@ -37,34 +38,56 @@ class UsbSerialFrameDecoder {
     final packets = <UsbSerialDecodedPacket>[];
 
     while (true) {
-      if (_rxBuffer.isEmpty) {
+      if (_startIndex >= _rxBuffer.length) {
+        _rxBuffer.clear();
+        _startIndex = 0;
         return packets;
       }
 
-      if (_rxBuffer.first != usbSerialRxFrameStart &&
-          _rxBuffer.first != usbSerialTxFrameStart) {
-        _rxBuffer.removeAt(0);
+      if (_rxBuffer[_startIndex] != usbSerialRxFrameStart &&
+          _rxBuffer[_startIndex] != usbSerialTxFrameStart) {
+        _startIndex++;
+        _compactBufferIfNeeded();
         continue;
       }
 
-      if (_rxBuffer.length < usbSerialHeaderLength) {
+      final availableLength = _rxBuffer.length - _startIndex;
+      if (availableLength < usbSerialHeaderLength) {
+        _compactBufferIfNeeded(force: true);
         return packets;
       }
 
-      final payloadLength = _rxBuffer[1] | (_rxBuffer[2] << 8);
+      final payloadLength =
+          _rxBuffer[_startIndex + 1] | (_rxBuffer[_startIndex + 2] << 8);
       final packetLength = usbSerialHeaderLength + payloadLength;
-      if (_rxBuffer.length < packetLength) {
+      if (availableLength < packetLength) {
+        _compactBufferIfNeeded(force: true);
         return packets;
       }
 
-      final frameStart = _rxBuffer.first;
+      final frameStart = _rxBuffer[_startIndex];
       final payload = Uint8List.fromList(
-        _rxBuffer.sublist(usbSerialHeaderLength, packetLength),
+        _rxBuffer.sublist(
+          _startIndex + usbSerialHeaderLength,
+          _startIndex + packetLength,
+        ),
       );
-      _rxBuffer.removeRange(0, packetLength);
+      _startIndex += packetLength;
+      _compactBufferIfNeeded();
       packets.add(
         UsbSerialDecodedPacket(frameStart: frameStart, payload: payload),
       );
     }
   }
+
+  void _compactBufferIfNeeded({bool force = false}) {
+    if (_startIndex == 0) {
+      return;
+    }
+    if (!force && _startIndex < 1024 && _startIndex < (_rxBuffer.length ~/ 2)) {
+      return;
+    }
+    _rxBuffer.removeRange(0, _startIndex);
+    _startIndex = 0;
+  }
 }

lib/services/usb_serial_service_web.dart

@@ -88,8 +88,10 @@ class UsbSerialService {
 
       debugPrint('USB serial opened port=$_connectedPortName via Web Serial');
     } catch (error) {
+      await _cleanupFailedConnect();
       _status = UsbSerialStatus.disconnected;
       _connectedPortName = null;
+      _connectedPortKey = null;
       rethrow;
     }
   }
@@ -205,6 +207,37 @@ class UsbSerialService {
     return port.callMethod<JSPromise<JSAny?>>('open'.toJS, options).toDart;
   }
 
+  Future<void> _cleanupFailedConnect() async {
+    final reader = _reader;
+    final writer = _writer;
+    final port = _port;
+
+    _reader = null;
+    _writer = null;
+    _port = null;
+
+    if (reader != null) {
+      try {
+        await reader.callMethod<JSPromise<JSAny?>>('cancel'.toJS).toDart;
+      } catch (_) {
+        // Ignore cleanup errors after a failed connect.
+      }
+      _releaseLock(reader);
+    }
+
+    if (writer != null) {
+      _releaseLock(writer);
+    }
+
+    if (port != null) {
+      try {
+        await port.callMethod<JSPromise<JSAny?>>('close'.toJS).toDart;
+      } catch (_) {
+        // Ignore cleanup errors after a failed connect.
+      }
+    }
+  }
+
   JSObject? _getReader(JSObject port) {
     final readable = port.getProperty<JSAny?>('readable'.toJS);
     if (readable == null) {