feat(WebServer): filter /dev/ttyS* serial devices

Author: FASTSHIFT

Tools/WebServer/serial_utils.py

@@ -18,18 +18,25 @@
 from device_worker import start_worker, stop_worker
 
 
+def _is_hidden_serial_device(device_path):
+    """Return True if serial device should be hidden from UI list."""
+    return device_path.startswith("/dev/ttyS")
+
+
 def scan_serial_ports():
     """Scan for available serial ports."""
     ports = serial.tools.list_ports.comports()
     result = [
-        {"device": port.device, "description": port.description} for port in ports
+        {"device": port.device, "description": port.description}
+        for port in ports
+        if not _is_hidden_serial_device(port.device)
     ]
 
     # Also scan for CH341 USB serial devices which may not be detected by pyserial
     ch341_devices = glob.glob("/dev/ttyCH341USB*")
     existing_devices = {item["device"] for item in result}
     for dev in ch341_devices:
-        if dev not in existing_devices:
+        if dev not in existing_devices and not _is_hidden_serial_device(dev):
             result.append({"device": dev, "description": "CH341 USB Serial"})
 
     return result

Tools/WebServer/tests/test_routes.py

@@ -9,7 +9,7 @@
 
 import json
 import pytest
-from unittest.mock import MagicMock
+from unittest.mock import MagicMock, patch
 
 
 @pytest.fixture
@@ -131,6 +131,29 @@ def test_get_ports(self, client):
         assert data["success"] is True
         assert "ports" in data
 
+    @patch("serial_utils.serial.tools.list_ports.comports")
+    @patch("serial_utils.glob.glob")
+    def test_get_ports_filters_ttys(self, mock_glob, mock_comports, client):
+        """Test get ports response does not include /dev/ttyS* devices."""
+        mock_comports.return_value = [
+            MagicMock(device="/dev/ttyUSB0", description="USB Serial"),
+            MagicMock(device="/dev/ttyS2", description="Legacy UART"),
+            MagicMock(device="/dev/ttyACM0", description="CDC ACM"),
+        ]
+        mock_glob.return_value = ["/dev/ttyCH341USB0"]
+
+        response = client.get("/api/ports")
+        assert response.status_code == 200
+        data = response.get_json()
+        assert data["success"] is True
+
+        devices = [port["device"] for port in data["ports"]]
+        assert "/dev/ttyUSB0" in devices
+        assert "/dev/ttyACM0" in devices
+        assert "/dev/ttyCH341USB0" in devices
+        assert "/dev/ttyS2" not in devices
+        assert all(not device.startswith("/dev/ttyS") for device in devices)
+
 
 class TestStatusRoute:
     """Test status API route."""

Tools/WebServer/tests/test_serial_utils.py

@@ -24,6 +24,29 @@ def test_scan_serial_ports(self):
             assert "device" in port
             assert "description" in port
 
+    @patch("serial_utils.serial.tools.list_ports.comports")
+    @patch("serial_utils.glob.glob")
+    def test_scan_serial_ports_filters_ttys(self, mock_glob, mock_comports):
+        """Test /dev/ttyS* devices are filtered from scan result."""
+        from serial_utils import scan_serial_ports
+
+        mock_comports.return_value = [
+            MagicMock(device="/dev/ttyUSB0", description="USB Serial"),
+            MagicMock(device="/dev/ttyS0", description="Onboard UART"),
+            MagicMock(device="/dev/ttyACM0", description="CDC ACM"),
+            MagicMock(device="/dev/ttyS3", description="Legacy UART"),
+        ]
+        mock_glob.return_value = ["/dev/ttyCH341USB0"]
+
+        ports = scan_serial_ports()
+        devices = [port["device"] for port in ports]
+
+        assert "/dev/ttyUSB0" in devices
+        assert "/dev/ttyACM0" in devices
+        assert "/dev/ttyCH341USB0" in devices
+        assert "/dev/ttyS0" not in devices
+        assert "/dev/ttyS3" not in devices
+
 
 class TestSerialOpen:
     """Test serial_open function."""