mtr-cli 0.3.0__tar.gz → 2.0.0__tar.gz

{mtr_cli-0.3.0 → mtr_cli-2.0.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: mtr-cli
-Version: 0.3.0
+Version: 2.0.0
 Summary: A CLI tool for seamless local development and remote execution on GPU servers.
 Project-URL: Homepage, https://github.com/lecoan/mtremote
 Project-URL: Repository, https://github.com/lecoan/mtremote
@@ -20,7 +20,7 @@ Classifier: Topic :: Software Development :: Build Tools
 Classifier: Topic :: System :: Systems Administration
 Requires-Python: >=3.10
 Requires-Dist: click>=8.0.0
-Requires-Dist: paramiko>=2.11.0
+Requires-Dist: packaging>=21.0
 Requires-Dist: pyyaml>=6.0
 Requires-Dist: rich>=12.0.0
 Description-Content-Type: text/markdown
@@ -33,8 +33,7 @@ MTRemote 是一个专为 AI Infra 和 Python/C++ 混合开发设计的命令行
 
 *   **多服务器管理**：通过配置文件管理多个 GPU 节点，支持默认服务器 (Implicit/Explicit)。
 *   **智能同步引擎**：
-    *   **Rsync (推荐)**：调用系统 `rsync`，支持增量同步，速度极快。支持 `sshpass` 自动处理密码认证。
-    *   **SFTP (兼容)**：纯 Python 实现，适用于无 `rsync` 的环境，配置简单。
+    *   **Rsync**：调用系统 `rsync`，支持增量同步，速度极快。支持 `sshpass` 自动处理密码认证。
 *   **双向同步**：支持从远端下载文件/文件夹到本地（`--get` 参数）。
 *   **双模式交互 (Dual-Mode Interaction)**：
     *   **交互模式 (Interactive)**：自动检测 TTY，支持 PTY 分配、Raw Mode、Rich UI 动画。完美支持 `vim`, `ipython`, `pdb`, `htop`。
@@ -93,7 +92,7 @@ mtr --init
 
 ```yaml
 defaults:
-  sync: "rsync"  # 或 "sftp"
+  sync: "rsync"
   exclude: [".git/", "__pycache__/"]
   download_dir: "./downloads"  # 默认下载位置（可选）
 
@@ -125,6 +124,21 @@ mtr ipython
 mtr -s prod-node python train.py
 ```
 
+### ⚠️ 参数传递注意事项
+
+当执行的命令包含以 `-` 开头的参数时（如 `python -c`, `gcc -O2`），建议使用 `--` 作为分隔符，避免被误认为是 `mtr` 的选项：
+
+```bash
+# ❌ 错误：-s 会被当作 --server 的短选项
+mtr python3 -s
+
+# ✅ 正确：使用 -- 分隔符
+mtr -- python3 -s
+
+# ✅ 指定服务器时也使用 --
+mtr --server prod-node -- python3 -c "print('hello')"
+```
+
 ## 📖 命令行选项
 
 ```bash
@@ -169,21 +183,9 @@ mtr --enable-log --log-file ./debug.log python train.py
 mtr --no-tty python train.py > log.txt
 ```
 
-### 2. 使用 SFTP 模式
-如果本地或远程无法使用 rsync，可以在配置中指定 `sync: sftp`：
-
-```yaml
-servers:
-  win-server:
-    host: "10.0.0.9"
-    sync: "sftp"
-    password: "secret_password"
-```
-
-### 3. 密码认证
+### 2. 密码认证
 支持 SSH 密码认证，但推荐使用 SSH Key。
 *   **交互式 Shell**: 使用 `sshpass` 包装 `ssh -t` 命令。
-*   **SFTP**: 原生支持密码。
 *   **Rsync**: 需要本地安装 `sshpass` 工具才能使用密码认证。
 
 **密码认证依赖**: 使用密码认证时，必须安装 `sshpass`:
@@ -198,7 +200,7 @@ sudo apt-get install sshpass
 sudo yum install sshpass
 ```
 
-### 4. 从远端下载文件 (--get)
+### 3. 从远端下载文件 (--get)
 使用 `--get` 参数可以从远端服务器下载文件或文件夹到本地：
 
 ```bash
@@ -241,7 +243,7 @@ servers:
 3. 默认配置中的 `download_dir`
 4. 当前工作目录
 
-### 5. 调试日志 (--enable-log)
+### 4. 调试日志 (--enable-log)
 当遇到问题需要排查时，可以启用文件日志：
 
 ```bash

{mtr_cli-0.3.0 → mtr_cli-2.0.0}/README.md

@@ -6,8 +6,7 @@ MTRemote 是一个专为 AI Infra 和 Python/C++ 混合开发设计的命令行
 
 *   **多服务器管理**：通过配置文件管理多个 GPU 节点，支持默认服务器 (Implicit/Explicit)。
 *   **智能同步引擎**：
-    *   **Rsync (推荐)**：调用系统 `rsync`，支持增量同步，速度极快。支持 `sshpass` 自动处理密码认证。
-    *   **SFTP (兼容)**：纯 Python 实现，适用于无 `rsync` 的环境，配置简单。
+    *   **Rsync**：调用系统 `rsync`，支持增量同步，速度极快。支持 `sshpass` 自动处理密码认证。
 *   **双向同步**：支持从远端下载文件/文件夹到本地（`--get` 参数）。
 *   **双模式交互 (Dual-Mode Interaction)**：
     *   **交互模式 (Interactive)**：自动检测 TTY，支持 PTY 分配、Raw Mode、Rich UI 动画。完美支持 `vim`, `ipython`, `pdb`, `htop`。
@@ -66,7 +65,7 @@ mtr --init
 
 ```yaml
 defaults:
-  sync: "rsync"  # 或 "sftp"
+  sync: "rsync"
   exclude: [".git/", "__pycache__/"]
   download_dir: "./downloads"  # 默认下载位置（可选）
 
@@ -98,6 +97,21 @@ mtr ipython
 mtr -s prod-node python train.py
 ```
 
+### ⚠️ 参数传递注意事项
+
+当执行的命令包含以 `-` 开头的参数时（如 `python -c`, `gcc -O2`），建议使用 `--` 作为分隔符，避免被误认为是 `mtr` 的选项：
+
+```bash
+# ❌ 错误：-s 会被当作 --server 的短选项
+mtr python3 -s
+
+# ✅ 正确：使用 -- 分隔符
+mtr -- python3 -s
+
+# ✅ 指定服务器时也使用 --
+mtr --server prod-node -- python3 -c "print('hello')"
+```
+
 ## 📖 命令行选项
 
 ```bash
@@ -142,21 +156,9 @@ mtr --enable-log --log-file ./debug.log python train.py
 mtr --no-tty python train.py > log.txt
 ```
 
-### 2. 使用 SFTP 模式
-如果本地或远程无法使用 rsync，可以在配置中指定 `sync: sftp`：
-
-```yaml
-servers:
-  win-server:
-    host: "10.0.0.9"
-    sync: "sftp"
-    password: "secret_password"
-```
-
-### 3. 密码认证
+### 2. 密码认证
 支持 SSH 密码认证，但推荐使用 SSH Key。
 *   **交互式 Shell**: 使用 `sshpass` 包装 `ssh -t` 命令。
-*   **SFTP**: 原生支持密码。
 *   **Rsync**: 需要本地安装 `sshpass` 工具才能使用密码认证。
 
 **密码认证依赖**: 使用密码认证时，必须安装 `sshpass`:
@@ -171,7 +173,7 @@ sudo apt-get install sshpass
 sudo yum install sshpass
 ```
 
-### 4. 从远端下载文件 (--get)
+### 3. 从远端下载文件 (--get)
 使用 `--get` 参数可以从远端服务器下载文件或文件夹到本地：
 
 ```bash
@@ -214,7 +216,7 @@ servers:
 3. 默认配置中的 `download_dir`
 4. 当前工作目录
 
-### 5. 调试日志 (--enable-log)
+### 4. 调试日志 (--enable-log)
 当遇到问题需要排查时，可以启用文件日志：
 
 ```bash

mtr_cli-2.0.0/mtr/__init__.py

@@ -0,0 +1,3 @@
+"""MTRemote - A CLI tool for seamless local development and remote execution."""
+
+__version__ = "2.0.0"

{mtr_cli-0.3.0 → mtr_cli-2.0.0}/mtr/cli.py

@@ -4,20 +4,20 @@ from datetime import datetime
 
 import click
 
+from mtr import __version__
 from mtr.config import ConfigError, ConfigLoader
 from mtr.logger import LogLevel, get_logger, setup_logging
-from mtr.ssh import SSHClientWrapper, SSHError
-from mtr.sync import RsyncSyncer, SftpSyncer, SyncError
+from mtr.ssh import SSHError, run_ssh_command
+from mtr.sync import RsyncSyncer, SyncError
+from mtr.updater import UpdateChecker
 
 DEFAULT_CONFIG_TEMPLATE = """# MTRemote Configuration
 defaults:
-  # 默认同步引擎
-  # 选项: "rsync" (推荐), "sftp"
+  # 默认同步引擎 (仅支持 rsync)
   sync: "rsync"
 
-  # 是否尊重 .gitignore 文件（仅 rsync 模式支持）
+  # 是否尊重 .gitignore 文件
   # 设置为 true 时，rsync 会自动读取项目根目录的 .gitignore 并排除匹配的文件
-  # SFTP 模式不支持此选项，如启用会报错
   respect_gitignore: true
 
   exclude:
@@ -39,12 +39,9 @@ servers:
     # 预设命令 (可选)
     # pre_cmd: "source ~/.bashrc && conda activate myenv"
 
-    # 密码认证 (可选)
+    # 密码认证 (可选，需要安装 sshpass)
     # password: "secret"
 
-    # 强制同步引擎 (可选)
-    # sync: "sftp"
-
     # 该服务器的下载位置（可选，覆盖 defaults）
     # download_dir: "./backups/dev-node"
   """
@@ -71,6 +68,7 @@ def _init_config():
 
 
 @click.command(context_settings=dict(ignore_unknown_options=True, allow_extra_args=True))
+@click.version_option(version=__version__, prog_name="mtr-cli", help="Show version and exit.")
 @click.option("-s", "--server", help="Target server alias")
 @click.option("--sync/--no-sync", default=True, help="Enable/Disable code sync")
 @click.option("--dry-run", is_flag=True, help="Print commands without executing")
@@ -81,10 +79,36 @@ def _init_config():
 @click.option("--log-file", help="Path to log file (default: ./.mtr/logs/mtr_YYYYMMDD_HHMMSS.log)")
 @click.option("--get", "remote_get_path", help="Remote path to download from")
 @click.option("--to", "local_dest_path", help="Local destination path for download (optional)")
+@click.option("--no-check-update", is_flag=True, help="Disable update check")
 @click.argument("command", nargs=-1, type=click.UNPROCESSED)
-def cli(server, sync, dry_run, tty, init, enable_log, log_level, log_file, remote_get_path, local_dest_path, command):
+def cli(
+    server,
+    sync,
+    dry_run,
+    tty,
+    init,
+    enable_log,
+    log_level,
+    log_file,
+    remote_get_path,
+    local_dest_path,
+    no_check_update,
+    command,
+):
     """MTRemote: Sync and Execute code on remote server."""
 
+    # Check for updates (async, non-blocking)
+    update_message = None
+    if not no_check_update and not init:
+        checker = UpdateChecker()
+        # Try to get cached update message first (from previous check)
+        update_message = checker.get_cached_update_message()
+        # Trigger background check for next time
+        try:
+            checker.check()
+        except Exception:
+            pass  # Silently fail update check
+
     # Get logger instance (will be no-op if not setup)
     logger = get_logger()
 
@@ -197,71 +221,61 @@ def cli(server, sync, dry_run, tty, init, enable_log, log_level, log_file, remot
         # Determine engine
         engine = server_conf.get("sync", config.global_defaults.get("sync", "rsync"))
 
-        if engine == "rsync":
-            syncer = RsyncSyncer(
-                local_dir=local_dir,
-                remote_dir=remote_dir,
-                host=host,
-                user=user,
-                key_filename=key_filename,
-                password=password,
-                port=port,
-                exclude=exclude,
-                respect_gitignore=respect_gitignore,
-            )
-        elif engine == "sftp":
-            syncer = SftpSyncer(
-                local_dir=local_dir,
-                remote_dir=remote_dir,
-                host=host,
-                user=user,
-                key_filename=key_filename,
-                password=password,
-                port=port,
-                exclude=exclude,
-                respect_gitignore=respect_gitignore,
-            )
-        else:
+        # Check if SFTP is configured
+        if engine == "sftp":
+            logger.error("SFTP mode is no longer supported. Please use rsync.", module="mtr.cli")
             click.secho(
-                f"Warning: Sync engine '{engine}' not supported yet. Fallback/Skipping.",
-                fg="yellow",
+                "Error: SFTP mode has been removed. Please update your config to use 'sync: rsync'.",
+                fg="red",
+                err=True,
             )
-            syncer = None
+            sys.exit(1)
+
+        syncer = RsyncSyncer(
+            local_dir=local_dir,
+            remote_dir=remote_dir,
+            host=host,
+            user=user,
+            key_filename=key_filename,
+            password=password,
+            port=port,
+            exclude=exclude,
+            respect_gitignore=respect_gitignore,
+        )
+
+        try:
+            if dry_run:
+                click.echo(f"[DryRun] Would sync {local_dir} -> {remote_dir}")
+                logger.info(f"[DryRun] Would sync {local_dir} -> {remote_dir}", module="mtr.sync")
+            else:
+                if is_interactive and console:
+                    # TTY mode: single line real-time update using Rich Live
+                    from rich.live import Live
+                    from rich.text import Text
+
+                    with Live(Text("Starting sync...", style="blue"), refresh_per_second=10) as live:
 
-        if syncer:
-            try:
-                if dry_run:
-                    click.echo(f"[DryRun] Would sync {local_dir} -> {remote_dir}")
-                    logger.info(f"[DryRun] Would sync {local_dir} -> {remote_dir}", module="mtr.sync")
-                else:
-                    if is_interactive and console:
-                        # TTY mode: single line real-time update using Rich Live
-                        from rich.live import Live
-                        from rich.text import Text
-
-                        with Live(Text("Starting sync...", style="blue"), refresh_per_second=10) as live:
-
-                            def show_sync_progress(filename):
-                                # Get relative path for cleaner display
-                                rel_path = os.path.relpath(filename, local_dir)
-                                live.update(Text(f"Syncing: {rel_path}", style="blue"))
-
-                            syncer.sync(show_progress=True, progress_callback=show_sync_progress)
-                            live.update(Text("Sync completed!", style="green"))
-                    else:
-                        # no_tty mode: print each file on new line
                         def show_sync_progress(filename):
+                            # Get relative path for cleaner display
                             rel_path = os.path.relpath(filename, local_dir)
-                            click.echo(f"Syncing: {rel_path}")
+                            live.update(Text(f"Syncing: {rel_path}", style="blue"))
 
-                        click.secho("Syncing code...", fg="blue")
                         syncer.sync(show_progress=True, progress_callback=show_sync_progress)
-                        click.secho("Sync completed!", fg="green")
-                    logger.info(f"Sync completed: {local_dir} -> {remote_dir}", module="mtr.sync")
-            except SyncError as e:
-                logger.error(f"Sync failed: {e}", module="mtr.sync")
-                click.secho(f"Sync Failed: {e}", fg="red", err=True)
-                sys.exit(1)
+                        live.update(Text("Sync completed!", style="green"))
+                else:
+                    # no_tty mode: print each file on new line
+                    def show_sync_progress(filename):
+                        rel_path = os.path.relpath(filename, local_dir)
+                        click.echo(f"Syncing: {rel_path}")
+
+                    click.secho("Syncing code...", fg="blue")
+                    syncer.sync(show_progress=True, progress_callback=show_sync_progress)
+                    click.secho("Sync completed!", fg="green")
+                logger.info(f"Sync completed: {local_dir} -> {remote_dir}", module="mtr.sync")
+        except SyncError as e:
+            logger.error(f"Sync failed: {e}", module="mtr.sync")
+            click.secho(f"Sync Failed: {e}", fg="red", err=True)
+            sys.exit(1)
 
     # 3. Download from remote (if --get is specified)
     if remote_get_path:
@@ -293,37 +307,28 @@ def cli(server, sync, dry_run, tty, init, enable_log, log_level, log_file, remot
         # Determine engine
         engine = server_conf.get("sync", config.global_defaults.get("sync", "rsync"))
 
-        if engine == "rsync":
-            syncer = RsyncSyncer(
-                local_dir=".",  # Not used for download
-                remote_dir=".",  # Not used for download
-                host=host,
-                user=user,
-                key_filename=key_filename,
-                password=password,
-                port=port,
-                exclude=exclude,
-                respect_gitignore=respect_gitignore,
-            )
-        elif engine == "sftp":
-            syncer = SftpSyncer(
-                local_dir=".",  # Not used for download
-                remote_dir=".",  # Not used for download
-                host=host,
-                user=user,
-                key_filename=key_filename,
-                password=password,
-                port=port,
-                exclude=exclude,
-                respect_gitignore=respect_gitignore,
-            )
-        else:
+        # Check if SFTP is configured
+        if engine == "sftp":
+            logger.error("SFTP mode is no longer supported. Please use rsync.", module="mtr.cli")
             click.secho(
-                f"Warning: Sync engine '{engine}' not supported yet.",
-                fg="yellow",
+                "Error: SFTP mode has been removed. Please update your config to use 'sync: rsync'.",
+                fg="red",
+                err=True,
             )
             sys.exit(1)
 
+        syncer = RsyncSyncer(
+            local_dir=".",  # Not used for download
+            remote_dir=".",  # Not used for download
+            host=host,
+            user=user,
+            key_filename=key_filename,
+            password=password,
+            port=port,
+            exclude=exclude,
+            respect_gitignore=respect_gitignore,
+        )
+
         try:
             if dry_run:
                 click.echo(f"[DryRun] Would download {remote_get_path} -> {local_dest}")
@@ -369,43 +374,34 @@ def cli(server, sync, dry_run, tty, init, enable_log, log_level, log_file, remot
         click.echo(f"[DryRun] Would run on {host}: {remote_cmd} (workdir={remote_dir})")
         return
 
-    ssh = SSHClientWrapper(host, user, port=port, key_filename=key_filename, password=password)
     try:
-        ssh.connect()
-        logger.info(f"SSH connection established to {host}", module="mtr.ssh")
-
-        if is_interactive:
-            # Run interactive shell (full TTY support)
-            logger.info(f"Executing interactive command: {remote_cmd}", module="mtr.cli")
-            exit_code = ssh.run_interactive_shell(remote_cmd, workdir=remote_dir, pre_cmd=pre_cmd)
-            logger.info(f"Command completed with exit code: {exit_code}", module="mtr.cli")
-            sys.exit(exit_code)
-        else:
-            # Run stream mode (for scripts/pipes)
-            # pty=False ensures clean output for parsing (separates stdout/stderr if we implemented that,
-            # but currently streams merged or just stdout. Let's keep pty=False to avoid control chars)
-            logger.info(f"Executing command: {remote_cmd}", module="mtr.cli")
-            stream = ssh.exec_command_stream(remote_cmd, workdir=remote_dir, pre_cmd=pre_cmd, pty=False)
-
-            # Consume generator and print
-            exit_code = 0
-            try:
-                while True:
-                    line = next(stream)
-                    click.echo(line, nl=False)
-            except StopIteration as e:
-                exit_code = e.value
-
-            logger.info(f"Command completed with exit code: {exit_code}", module="mtr.cli")
-            sys.exit(exit_code)
+        # Execute command via SSH
+        logger.info(f"Executing command: {remote_cmd}", module="mtr.cli")
+        exit_code = run_ssh_command(
+            host=host,
+            user=user,
+            command=remote_cmd,
+            port=port,
+            key_filename=key_filename,
+            password=password,
+            workdir=remote_dir,
+            pre_cmd=pre_cmd,
+            tty=is_interactive,
+        )
+        logger.info(f"Command completed with exit code: {exit_code}", module="mtr.cli")
+
+        # Show update message if available
+        if update_message:
+            click.echo(update_message, err=True)
+        sys.exit(exit_code)
 
     except SSHError as e:
         logger.error(f"SSH error: {e}", module="mtr.ssh")
         click.secho(f"SSH Error: {e}", fg="red", err=True)
+        # Show update message if available even on error
+        if update_message:
+            click.echo(update_message, err=True)
         sys.exit(1)
-    finally:
-        logger.info("Closing SSH connection", module="mtr.ssh")
-        ssh.close()
 
 
 if __name__ == "__main__":

mtr_cli-2.0.0/mtr/ssh.py

@@ -0,0 +1,132 @@
+"""SSH utilities for MTRemote."""
+
+import os
+import shutil
+import subprocess
+from typing import Optional
+
+from mtr.logger import get_logger
+
+
+class SSHError(Exception):
+    """SSH-related errors."""
+
+    pass
+
+
+def _check_ssh_availability():
+    """Check if ssh command is available."""
+    if shutil.which("ssh") is None:
+        raise SSHError(
+            "SSH command not found. Please install OpenSSH client.\n"
+            "  macOS: brew install openssh\n"
+            "  Ubuntu/Debian: sudo apt-get install openssh-client\n"
+            "  CentOS/RHEL: sudo yum install openssh-clients"
+        )
+
+
+def _check_sshpass_availability():
+    """Check if sshpass command is available."""
+    if shutil.which("sshpass") is None:
+        raise SSHError(
+            "sshpass command not found. Please install sshpass for password authentication.\n"
+            "  macOS: brew install hudochenkov/sshpass/sshpass\n"
+            "  Ubuntu/Debian: sudo apt-get install sshpass\n"
+            "  CentOS/RHEL: sudo yum install sshpass"
+        )
+
+
+def _build_command(command: str, workdir: Optional[str] = None, pre_cmd: Optional[str] = None) -> str:
+    """Build the full command string with workdir and pre_cmd."""
+    parts = []
+    if workdir:
+        parts.append(f"cd {workdir}")
+    if pre_cmd:
+        parts.append(pre_cmd)
+    parts.append(command)
+    return " && ".join(parts)
+
+
+def run_ssh_command(
+    host: str,
+    user: str,
+    command: str,
+    port: int = 22,
+    key_filename: Optional[str] = None,
+    password: Optional[str] = None,
+    workdir: Optional[str] = None,
+    pre_cmd: Optional[str] = None,
+    tty: bool = True,
+) -> int:
+    """
+    Run a command on remote host via system SSH.
+
+    Args:
+        host: Remote host address
+        user: SSH username
+        command: Command to execute
+        port: SSH port (default: 22)
+        key_filename: Path to SSH private key
+        password: SSH password (requires sshpass)
+        workdir: Working directory on remote host
+        pre_cmd: Command to run before main command
+        tty: If True, use ssh -t for TTY allocation
+
+    Returns:
+        Exit code from the remote command
+
+    Raises:
+        SSHError: If SSH command fails or is not available
+    """
+    logger = get_logger()
+    mode_str = "interactive" if tty else "batch"
+    logger.info(f"Executing {mode_str} command via SSH: {command}", module="mtr.ssh")
+    logger.debug(f"Host: {host}, User: {user}, Port: {port}, TTY: {tty}", module="mtr.ssh")
+
+    # Check SSH availability
+    _check_ssh_availability()
+
+    # Check sshpass availability if password is used
+    if password and not key_filename:
+        _check_sshpass_availability()
+
+    # Build the full command
+    full_command = _build_command(command, workdir, pre_cmd)
+    logger.debug(f"Full command: {full_command}", module="mtr.ssh")
+
+    # Build SSH command
+    ssh_cmd = ["ssh"]
+
+    # Add -t flag for TTY mode
+    if tty:
+        ssh_cmd.append("-t")
+
+    # Port
+    if port != 22:
+        ssh_cmd.extend(["-p", str(port)])
+
+    # Key authentication
+    if key_filename:
+        ssh_cmd.extend(["-i", os.path.expanduser(key_filename)])
+
+    # Target host and command
+    target = f"{user}@{host}"
+    ssh_cmd.extend([target, full_command])
+
+    # Wrap with sshpass if password is provided
+    if password and not key_filename:
+        ssh_cmd = ["sshpass", "-p", password] + ssh_cmd
+
+    logger.debug(f"Executing: {' '.join(ssh_cmd)}", module="mtr.ssh")
+
+    # Run command
+    try:
+        result = subprocess.run(ssh_cmd)
+        logger.info(f"Command exited with code: {result.returncode}", module="mtr.ssh")
+        return result.returncode
+    except FileNotFoundError as e:
+        logger.error(f"Command not found: {e}", module="mtr.ssh")
+        raise SSHError(f"SSH command execution failed: {e}")
+    except Exception as e:
+        logger.error(f"SSH command failed: {e}", module="mtr.ssh")
+        raise SSHError(f"SSH command failed: {e}")