lr-shuttle 0.2.3__py3-none-any.whl → 0.2.4__py3-none-any.whl

{lr_shuttle-0.2.3.dist-info → lr_shuttle-0.2.4.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: lr-shuttle
-Version: 0.2.3
+Version: 0.2.4
 Summary: CLI and Python client for host-side of json based serial communication with embedded device bridge.
 Author-email: Jonas Estberger <jonas.estberger@lumenradio.com>
 License: MIT

{lr_shuttle-0.2.3.dist-info → lr_shuttle-0.2.4.dist-info}/RECORD

@@ -1,18 +1,18 @@
-shuttle/cli.py,sha256=i75UW_r_v2AbVTMA1X4u_LnVf3fQz5Xc00IjbxNZwN8,93515
+shuttle/cli.py,sha256=lou_JvWNoMXxoyXyz5MSIyAUZEUAe0CmXaBnl_gqLMY,93507
 shuttle/constants.py,sha256=GUlAg3iEuPxLQ2mDCvlv5gVXHnlawl_YeLtaUSqsnPM,757
 shuttle/flash.py,sha256=9ph23MHL40SjKZoL38Sbd3JbykGb-ECxvzBzCIjAues,4492
-shuttle/prodtest.py,sha256=1q-1dZYtrWBpI_e0jPgROVGbb_42Y0q0DIxDoo4MWUk,8020
-shuttle/serial_client.py,sha256=WjGanUAL16qw2RZcCjHjYMKHsk-B6zY3cMeS0gPtPHE,17650
+shuttle/prodtest.py,sha256=nI8k2OndhqsOv8BMtXwfcpGEdmHU7ywbIgMuW49EULU,8006
+shuttle/serial_client.py,sha256=bUpTs6MmJkpYBgtNYZZ0EYaybkLlrM7MlhWxHLQPh3U,18185
 shuttle/timo.py,sha256=1K18y0QtDF2lw2Abeok9PgrpPUiCEbQdGQXOQik75Hw,16481
 shuttle/firmware/__init__.py,sha256=KRXyz3xJ2GIB473tCHAky3DdPIQb78gX64Qn-uu55To,120
 shuttle/firmware/esp32c5/__init__.py,sha256=U2xXnb80Wv8EJaJ6Tv9iev1mVlpoaEeqsNmjmEtxdFQ,41
 shuttle/firmware/esp32c5/boot_app0.bin,sha256=-UxdeGp6j6sGrF0Q4zvzdxGmaXY23AN1WeoZzEEKF_A,8192
-shuttle/firmware/esp32c5/devboard.ino.bin,sha256=eCkSHtcBBbLRvHQmkvXR1jRoMMqMTMEXaXMv8GMIsD8,1098256
+shuttle/firmware/esp32c5/devboard.ino.bin,sha256=CxWJQYlbL7zu9nkUzBQ0PQQET4XHZKFoxMzm9tZonqk,1099040
 shuttle/firmware/esp32c5/devboard.ino.bootloader.bin,sha256=LPU51SdUwebYemCZb5Pya-wGe7RC4UXrkRmBnsHePp0,20784
 shuttle/firmware/esp32c5/devboard.ino.partitions.bin,sha256=FIuVnL_xw4qo4dXAup1hLFSZe5ReVqY_QSI-72UGU6E,3072
 shuttle/firmware/esp32c5/manifest.json,sha256=CPOegfEK4PTtI6UPeohuUKkJNeg0t8aWntEczpoxYt4,480
-lr_shuttle-0.2.3.dist-info/METADATA,sha256=_hp5-NmIdtKB2wZIQww4oPpXCe3mDHqevScN3G9uj4Y,13611
-lr_shuttle-0.2.3.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
-lr_shuttle-0.2.3.dist-info/entry_points.txt,sha256=obqdFPgvQLB1_EWcnD9ch8HjQRlNVT_pdB_EidDRDco,44
-lr_shuttle-0.2.3.dist-info/top_level.txt,sha256=PtNxNQQdya-Xs8DYublNTBTa8c1TrtfEpQ0lUd_OeZY,8
-lr_shuttle-0.2.3.dist-info/RECORD,,
+lr_shuttle-0.2.4.dist-info/METADATA,sha256=EEgzPGFjeZ3L04elKT8VcUkiRjm0CyUn9dAN5Cq4_IQ,13611
+lr_shuttle-0.2.4.dist-info/WHEEL,sha256=_zCd3N1l69ArxyTb8rzEoP9TpbYXkqRFSNOD5OuxnTs,91
+lr_shuttle-0.2.4.dist-info/entry_points.txt,sha256=obqdFPgvQLB1_EWcnD9ch8HjQRlNVT_pdB_EidDRDco,44
+lr_shuttle-0.2.4.dist-info/top_level.txt,sha256=PtNxNQQdya-Xs8DYublNTBTa8c1TrtfEpQ0lUd_OeZY,8
+lr_shuttle-0.2.4.dist-info/RECORD,,

shuttle/cli.py

@@ -107,7 +107,9 @@ for entry in PRODTEST_TX_POWER_LEVELS:
     for alias in entry["aliases"]:
         PRODTEST_TX_POWER_ALIASES[alias.lower()] = entry["value"]
     PRODTEST_TX_POWER_ALIASES[str(entry["value"])] = entry["value"]
-PRODTEST_TX_POWER_CANONICAL = [entry["aliases"][0] for entry in PRODTEST_TX_POWER_LEVELS]
+PRODTEST_TX_POWER_CANONICAL = [
+    entry["aliases"][0] for entry in PRODTEST_TX_POWER_LEVELS
+]
 
 _HOST_PORT_PATTERN = re.compile(r"^[A-Za-z0-9_.-]+:\d+$")
 _IPV6_HOST_PORT_PATTERN = re.compile(r"^\[[0-9A-Fa-f:]+\]:\d+$")
@@ -159,9 +161,7 @@ def _resolve_prodtest_power_choice(value: str) -> Tuple[int, Dict[str, str]]:
             resolved = parsed
     if resolved is None:
         allowed = ", ".join(PRODTEST_TX_POWER_CANONICAL)
-        raise typer.BadParameter(
-            f"Power must be one of: {allowed} or an index 0-7"
-        )
+        raise typer.BadParameter(f"Power must be one of: {allowed} or an index 0-7")
     return resolved, PRODTEST_TX_POWER_META[resolved]
 
 
@@ -259,9 +259,7 @@ def _resolve_uart_payload(
 def _normalize_port(port: str) -> str:
     trimmed = port.strip()
     if not trimmed:
-        raise typer.BadParameter(
-            "Serial port is required (use --port or SHUTTLE_PORT)"
-        )
+        raise typer.BadParameter("Serial port is required (use --port or SHUTTLE_PORT)")
     if "://" in trimmed:
         return trimmed
     if trimmed.startswith("/") or trimmed.startswith("\\"):
@@ -1568,7 +1566,9 @@ def prodtest_ping(
         raise typer.Exit(1)
 
     if len(responses) != len(sequence):
-        console.print("[red]Prodtest command halted before completing all SPI phases[/]")
+        console.print(
+            "[red]Prodtest command halted before completing all SPI phases[/]"
+        )
         raise typer.Exit(1)
 
     command_response, payload_response = responses
@@ -1583,9 +1583,7 @@ def prodtest_ping(
         command_label="spi.xfer (prodtest payload)",
     )
 
-    rx_bytes = _decode_hex_response(
-        payload_response, label="prodtest ping (payload)"
-    )
+    rx_bytes = _decode_hex_response(payload_response, label="prodtest ping (payload)")
     if not rx_bytes or rx_bytes[0] != 0x2D:  # ord('-')
         console.print(
             "[red]Ping failed: expected '-' (0x2D), got: "
@@ -1623,9 +1621,7 @@ def prodtest_antenna(
         antenna_value = PRODTEST_ANTENNA_CHOICES[normalized]
     except KeyError as exc:
         allowed = ", ".join(sorted(PRODTEST_ANTENNA_CHOICES))
-        raise typer.BadParameter(
-            f"Antenna must be one of: {allowed}"
-        ) from exc
+        raise typer.BadParameter(f"Antenna must be one of: {allowed}") from exc
 
     sequence = [prodtest.select_antenna(antenna_value)]
     responses = _execute_timo_sequence(
@@ -1767,7 +1763,9 @@ def prodtest_hw_device_id(
         raise typer.Exit(1)
 
     if len(responses) != len(sequence):
-        console.print("[red]Prodtest command halted before completing all SPI phases[/]")
+        console.print(
+            "[red]Prodtest command halted before completing all SPI phases[/]"
+        )
         raise typer.Exit(1)
 
     result_response = responses[-1]
@@ -1858,11 +1856,11 @@ def prodtest_serial_number(
             result_response,
             command_label="spi.xfer (prodtest payload)",
         )
-        rx_bytes = _decode_hex_response(
-            result_response, label="prodtest serial-number"
-        )
+        rx_bytes = _decode_hex_response(result_response, label="prodtest serial-number")
         if len(rx_bytes) < prodtest.SERIAL_NUMBER_LEN:
-            console.print("[red]Prodtest serial-number response shorter than expected[/]")
+            console.print(
+                "[red]Prodtest serial-number response shorter than expected[/]"
+            )
             raise typer.Exit(1)
         serial_bytes = rx_bytes[-prodtest.SERIAL_NUMBER_LEN :]
         console.print(f"Serial number: {_format_hex(serial_bytes.hex())}")
@@ -2476,7 +2474,9 @@ def wifi_cfg_command(
     if parsed_gateway is not None:
         network_payload["gateway"] = parsed_gateway
 
-    dns_entries = [entry for entry in (parsed_dns_primary, parsed_dns_secondary) if entry]
+    dns_entries = [
+        entry for entry in (parsed_dns_primary, parsed_dns_secondary) if entry
+    ]
     if dns_entries:
         network_payload["dns"] = dns_entries
 

shuttle/prodtest.py

@@ -62,9 +62,7 @@ def command(
 ) -> dict:
     """Build an NDJSON-ready spi.xfer payload for a prodtest command."""
 
-    return timo.command_payload(
-        _build_command_bytes(opcode, arguments), params=params
-    )
+    return timo.command_payload(_build_command_bytes(opcode, arguments), params=params)
 
 
 def reset() -> dict:

shuttle/serial_client.py

@@ -229,10 +229,23 @@ class NDJSONSerialClient:
     ):
         try:
             self._serial = serial.serial_for_url(
-                url=port, baudrate=baudrate, timeout=timeout
+                url=port,
+                baudrate=baudrate,
+                timeout=timeout,
+                do_not_open=True,
             )
+        except SerialException as exc:  # pragma: no cover - hardware specific
+            raise ShuttleSerialError(f"Unable to initialize {port}: {exc}") from exc
+
+        try:
+            if getattr(self._serial, "open", None) is not None:
+                if not getattr(self._serial, "is_open", False):
+                    self._serial.open()
         except SerialException as exc:  # pragma: no cover - hardware specific
             raise ShuttleSerialError(f"Unable to open {port}: {exc}") from exc
+        except AttributeError:
+            # Test stubs without an open() method are already "connected"
+            pass
         self._serial.reset_input_buffer()
         self._lock = threading.Lock()
         self._pending: Dict[int, CommandFuture] = {}