Fix repl failing to start on python 3.8, address review comments

Author: dhoomakethu

.gitignore

@@ -157,4 +157,4 @@ cython_debug/
 #  be found at https://github.com/github/gitignore/blob/main/Global/JetBrains.gitignore
 #  and can be added to the global gitignore or merged into this file.  For a more nuclear
 #  option (not recommended) you can uncomment the following to ignore the entire idea folder.
-#.idea/
+.idea/

poetry.lock

@@ -1 +1,3 @@
 """REPL (Read-Eval-Print Loop) tool for working with Modbus devices using the Pymodbus library."""
+__VERSION__ = "2.0.0"
+

pymodbus_repl/__init__.py

@@ -12,6 +12,7 @@
 from prompt_toolkit.styles import Style
 from pygments.lexers.python import PythonLexer
 from pymodbus import __version__ as pymodbus_version
+from pymodbus_repl import __VERSION__ as repl_version
 from pymodbus.exceptions import ParameterException
 from pymodbus.transaction import (
     ModbusAsciiFramer,
@@ -38,7 +39,7 @@
  |    |    \___  /    Y    (  <_> ) /_/ |    |    |   \  ___/|  |_> >  |__
  |____|    / ____\____|__  /\____/\____ | /\ |____|_  /\___  >   __/|____/
            \/            \/            \/ \/        \/     \/|__|
-                                        v1.3.1 - {pymodbus_version}
+                                                v{repl_version} - Pymodbus-{pymodbus_version}
 ----------------------------------------------------------------------------
 """
 

pymodbus_repl/client/main.py

@@ -8,6 +8,8 @@
 from prompt_toolkit.shortcuts import clear
 from prompt_toolkit.shortcuts.progress_bar import formatters
 from prompt_toolkit.styles import Style
+from pymodbus import __version__ as pymodbus_version
+from pymodbus_repl import __VERSION__ as repl_version
 
 
 TITLE = r"""
@@ -17,6 +19,7 @@
  |    |    \___  |  Y Y  (  <_> ) /_/ |  | \_\ \  |  /\___ \   /        \  ___/|  | \/\   /\  ___/|  | \/
  |____|    / ____|__|_|  /\____/\____ |  |___  /____//____  > /_______  /\___  >__|    \_/  \___  >__|
            \/          \/            \/      \/           \/          \/     \/                 \/
+                                                                                                v{}-Pymodbus{}
 """
 
 
@@ -112,7 +115,8 @@ def print_title():
         [len(t) for t in TITLE.split("\n")]
     )
     if col > max_len:
-        info(TITLE)
+        title = TITLE.format(repl_version, pymodbus_version)
+        info(title)
     else:
         print_formatted_text(
             HTML(f'<u><b><style color="green">{SMALL_TITLE}</style></b></u>')

pymodbus_repl/server/cli.py

@@ -1,15 +1,17 @@
 """Repl server main."""
-from __future__ import annotations
+# from __future__ import annotations
 
 import asyncio
 import contextlib
 import json
 import logging
 import sys
 from enum import Enum
-from pathlib import Path
 
+from typing import List, Optional
+from pathlib import Path
 import typer
+from typing_extensions import Annotated
 from pymodbus import pymodbus_apply_logging_config
 from pymodbus.framer.socket_framer import ModbusSocketFramer
 from pymodbus.logging import Log
@@ -50,7 +52,7 @@ class ModbusFramerTypes(str, Enum):
     binary = "binary"  # pylint: disable=invalid-name
 
 
-def _completer(incomplete: str, valid_values: list[str]) -> list[str]:
+def _completer(incomplete: str, valid_values: List[str]) -> List[str]:
     """Complete value."""
     completion = []
     for name in valid_values:
@@ -59,19 +61,19 @@ def _completer(incomplete: str, valid_values: list[str]) -> list[str]:
     return completion
 
 
-def framers(incomplete: str) -> list[str]:
+def framers(incomplete: str) -> List[str]:
     """Return an autocompleted list of supported clouds."""
     _framers = ["socket", "rtu", "tls", "ascii", "binary"]
     return _completer(incomplete, _framers)
 
 
-def servers(incomplete: str) -> list[str]:
+def servers(incomplete: str) -> List[str]:
     """Return an autocompleted list of supported clouds."""
     _servers = ["tcp", "serial", "tls", "udp"]
     return _completer(incomplete, _servers)
 
 
-def process_extra_args(extra_args: list[str], modbus_config: dict) -> dict:
+def process_extra_args(extra_args: List[str], modbus_config: dict) -> dict:
     """Process extra args passed to server."""
     options_stripped = [x.strip().replace("--", "") for x in extra_args[::2]]
     extra_args_dict = dict(list(zip(options_stripped, extra_args[1::2])))
@@ -139,12 +141,12 @@ def run(
         help="Modbus framer to use",
     ),
     modbus_port: str = typer.Option("5020", "--modbus-port", "-p", help="Modbus port"),
-    modbus_slave_id: list[int] = typer.Option(
-        [1], "--slave-id", "-u", help="Supported Modbus slave id's"
-    ),
-    modbus_config_path: Path = typer.Option(
-        None, help="Path to additional modbus server config"
-    ),
+    modbus_slave_id: Annotated[Optional[List[int]], typer.Option(
+        "--unit-id", "-u", help="Supported Modbus slave id's"
+    )] = [1],
+    modbus_config_path: Annotated[Path, typer.Option(
+        help="Path to additional modbus server config"
+    )] = None,
     randomize: int = typer.Option(
         0,
         "--random",

pymodbus_repl/server/main.py

@@ -1,6 +1,6 @@
 [tool.poetry]
 name = "pymodbus_repl"
-version = "0.1.0"
+version = "2.0.0"
 description = "REPL (Read-Eval-Print Loop) tool for working with Modbus devices using the Pymodbus library."
 authors = ["dhoomakethu <otlasanju@gmail.com>"]
 readme = "README.md"
@@ -11,7 +11,6 @@ python = "^3.8"
 typer = {extras = ["all"], version = "^0.9.0"}
 prompt-toolkit = "^3.0.43"
 pygments = "^2.17.2"
-pytest-xdist = "^3.5.0"
 
 [[tool.poetry.dependencies.aiohttp]]
 python="3.12"
@@ -30,6 +29,7 @@ include = ["pymodbus_repl"]
 pymodbus = "^3.6.4"
 ruff = "^0.2.1"
 coverage = "^7.4.1"
+pytest-xdist = "^3.5.0"
 pytest-cov = "^4.1.0"
 
 [build-system]
@@ -43,7 +43,7 @@ addopts = "--cov-report html"
 
 [tool.coverage.run]
 source = [
-    "pymodbus/"
+    "pymodbus_repl/"
 ]
 omit = ["examples/contrib/"]
 branch = true
@@ -56,6 +56,7 @@ exclude_also = [
     ]
 skip_covered = true
 fail_under = 70.0
+show_missing = true
 
 [tool.coverage.html]
 directory = "build/cov"