pybricksdev/connections: Drop ssh connection.

This was used to automate motor benchmarking on ev3dev. This no longer applies, and better tools exist to run code on ev3dev.

Author: laurensvalk

CHANGELOG.md

@@ -15,6 +15,9 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 ### Removed
 - Removed `REPLHub`. This was used for non-Pybricks MicroPython boards, but
   `mpremote` should be used for this.
+- Removed `EV3Connection`. This was used for ev3dev connections over ssh,
+  making it possible to automate motor benchmarking on ev3dev. This use case
+  no longer applies and other tools can be used to run code on ev3dev.
 
 ## [1.2.0] - 2025-07-11
 

docs/conf.py

@@ -102,10 +102,8 @@
 # -- Options for autodoc extension -------------------------------------------
 
 autodoc_mock_imports = [
-    "aioserial",
     "appdirs",
     "argcomplete",
-    "asyncssh",
     "bleak",
     "mpy_cross_v5",
     "mpy_cross_v6",

pybricksdev/cli/__init__.py

@@ -130,7 +130,7 @@ def add_parser(self, subparsers: argparse._SubParsersAction):
             "conntype",
             metavar="<connection type>",
             help="connection type: %(choices)s",
-            choices=["ble", "usb", "ssh"],
+            choices=["ble", "usb"],
         )
         parser.add_argument(
             "file",
@@ -143,8 +143,7 @@ def add_parser(self, subparsers: argparse._SubParsersAction):
             "--name",
             metavar="<name>",
             required=False,
-            help="hostname or IP address for SSH connection; "
-            "Bluetooth device name or Bluetooth address for BLE connection; "
+            help="Bluetooth device name or Bluetooth address for BLE connection; "
             "serial port name for USB connection",
         )
 
@@ -173,17 +172,7 @@ def add_parser(self, subparsers: argparse._SubParsersAction):
     async def run(self, args: argparse.Namespace):
 
         # Pick the right connection
-        if args.conntype == "ssh":
-            from pybricksdev.connections.ev3dev import EV3Connection
-
-            # So it's an ev3dev
-            if args.name is None:
-                print("--name is required for SSH connections", file=sys.stderr)
-                exit(1)
-
-            device_or_address = socket.gethostbyname(args.name)
-            hub = EV3Connection(device_or_address)
-        elif args.conntype == "ble":
+        if args.conntype == "ble":
             from pybricksdev.ble import find_device as find_ble
             from pybricksdev.connections.pybricks import PybricksHubBLE
 
@@ -250,7 +239,7 @@ def add_parser(self, subparsers: argparse._SubParsersAction):
             "conntype",
             metavar="<connection type>",
             help="connection type: %(choices)s",
-            choices=["ble", "usb", "ssh"],
+            choices=["ble", "usb"],
         )
         parser.add_argument(
             "file",
@@ -263,24 +252,13 @@ def add_parser(self, subparsers: argparse._SubParsersAction):
             "--name",
             metavar="<name>",
             required=False,
-            help="hostname or IP address for SSH connection; "
-            "Bluetooth device name or Bluetooth address for BLE connection; "
+            help="Bluetooth device name or Bluetooth address for BLE connection; "
             "serial port name for USB connection",
         )
 
     async def run(self, args: argparse.Namespace):
         # Pick the right connection
-        if args.conntype == "ssh":
-            from pybricksdev.connections.ev3dev import EV3Connection
-
-            # So it's an ev3dev
-            if args.name is None:
-                print("--name is required for SSH connections", file=sys.stderr)
-                exit(1)
-
-            device_or_address = socket.gethostbyname(args.name)
-            hub = EV3Connection(device_or_address)
-        elif args.conntype == "ble":
+        if args.conntype == "ble":
             from pybricksdev.ble import find_device as find_ble
             from pybricksdev.connections.pybricks import PybricksHubBLE
 

pybricksdev/connections/ev3dev.py

@@ -28,9 +28,7 @@ include = [
 pybricksdev = 'pybricksdev.cli:main'
 
 [tool.poetry.dependencies]
-aioserial = ">=1.3.0"
 argcomplete = ">=1.11.1"
-asyncssh = ">=2.2.1"
 bleak = ">=0.22.0"
 mpy-cross-v5 = ">=1.0.0"
 python = ">=3.10,<3.14"

pyproject.toml

@@ -147,75 +147,6 @@ async def test_download_usb(self):
             mock_hub.download.assert_called_once()
             mock_hub.disconnect.assert_called_once()
 
-    @pytest.mark.asyncio
-    async def test_download_ssh(self):
-        """Test running the download command with SSH connection."""
-        # Create a mock hub
-        mock_hub = AsyncMock()
-        mock_hub.download = AsyncMock()
-
-        # Set up mocks using ExitStack
-        with contextlib.ExitStack() as stack:
-            # Create and manage temporary file
-            temp = stack.enter_context(
-                tempfile.NamedTemporaryFile(suffix=".py", mode="w+", delete=False)
-            )
-            temp.write("print('test')")
-            temp_path = temp.name
-            stack.callback(os.unlink, temp_path)
-
-            # Create args
-            args = argparse.Namespace(
-                conntype="ssh",
-                file=open(temp_path, "r"),
-                name="ev3dev.local",
-            )
-
-            mock_hub_class = stack.enter_context(
-                patch(
-                    "pybricksdev.connections.ev3dev.EV3Connection",
-                    return_value=mock_hub,
-                )
-            )
-            stack.enter_context(
-                patch("socket.gethostbyname", return_value="192.168.1.1")
-            )
-
-            # Run the command
-            download = Download()
-            await download.run(args)
-
-            # Verify the hub was created and used correctly
-            mock_hub_class.assert_called_once_with("192.168.1.1")
-            mock_hub.connect.assert_called_once()
-            mock_hub.download.assert_called_once()
-            mock_hub.disconnect.assert_called_once()
-
-    @pytest.mark.asyncio
-    async def test_download_ssh_no_name(self):
-        """Test that SSH connection requires a name."""
-        # Set up mocks using ExitStack
-        with contextlib.ExitStack() as stack:
-            # Create and manage temporary file
-            temp = stack.enter_context(
-                tempfile.NamedTemporaryFile(suffix=".py", mode="w+", delete=False)
-            )
-            temp.write("print('test')")
-            temp_path = temp.name
-            stack.callback(os.unlink, temp_path)
-
-            # Create args without name
-            args = argparse.Namespace(
-                conntype="ssh",
-                file=open(temp_path, "r"),
-                name=None,
-            )
-
-            # Run the command and verify it exits
-            download = Download()
-            with pytest.raises(SystemExit, match="1"):
-                await download.run(args)
-
     @pytest.mark.asyncio
     async def test_download_stdin(self):
         """Test running the download command with stdin input."""