replx 1.6__tar.gz → 1.6.1__tar.gz

{replx-1.6/replx.egg-info → replx-1.6.1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: replx
-Version: 1.6
+Version: 1.6.1
 Summary: replx is a fast, modern MicroPython CLI: turbo REPL, robust file sync (put/get), project install, mpy-cross integration, and smart port discovery.
 Author-email: "chanmin.park" <devcamp@gmail.com>
 License-Expression: MIT

{replx-1.6 → replx-1.6.1}/replx/__init__.py

@@ -1,5 +1,5 @@
 __all__ = ["__version__", "get_version", "__description__"]
-__version__ = "1.6"
+__version__ = "1.6.1"
 __description__ = "Fast, modern MicroPython CLI with REPL, file sync, install, and smart port detection."
 __author__ = "PlanX Lab Development Team"
 

{replx-1.6 → replx-1.6.1}/replx/cli/agent/client/core.py

@@ -2,6 +2,7 @@ import os
 import sys
 import socket
 import time
+import tempfile
 from typing import Dict, Any, Optional, Callable
 
 from replx.utils.constants import DEFAULT_AGENT_PORT, AGENT_HOST, MAX_UDP_SIZE
@@ -327,7 +328,18 @@ class AgentClient:
             cmd.append(str(port))
 
         proc = None
+        stderr_file = None
+        stderr_path = None
+
         if background:
+            try:
+                fd, stderr_path = tempfile.mkstemp(prefix='replx_agent_', suffix='.err')
+                os.close(fd)
+                stderr_file = open(stderr_path, 'w', encoding='utf-8')
+            except Exception:
+                stderr_file = None
+                stderr_path = None
+
             if sys.platform == 'win32':
                 startupinfo = subprocess.STARTUPINFO()
                 startupinfo.dwFlags |= subprocess.STARTF_USESHOWWINDOW
@@ -337,27 +349,48 @@ class AgentClient:
                     cmd,
                     creationflags=subprocess.CREATE_NEW_PROCESS_GROUP | subprocess.DETACHED_PROCESS,
                     stdout=subprocess.DEVNULL,
-                    stderr=subprocess.DEVNULL,
+                    stderr=stderr_file if stderr_file else subprocess.DEVNULL,
                     startupinfo=startupinfo
                 )
             else:
                 proc = subprocess.Popen(
                     cmd,
                     stdout=subprocess.DEVNULL,
-                    stderr=subprocess.DEVNULL,
+                    stderr=stderr_file if stderr_file else subprocess.DEVNULL,
                     stdin=subprocess.DEVNULL,
                     start_new_session=True,
                     close_fds=True
                 )
+
+            if stderr_file:
+                stderr_file.close()
+                stderr_file = None
         else:
             proc = subprocess.Popen(cmd)
 
-        for _ in range(100):
-            time.sleep(0.1)
-            if proc is not None and proc.poll() is not None:
-                raise RuntimeError(f"Failed to start agent (process exited with code {proc.returncode})")
-            if AgentClient.is_agent_running(port=port):
-                return True
+        try:
+            for _ in range(100):
+                time.sleep(0.1)
+                if proc is not None and proc.poll() is not None:
+                    detail = ''
+                    if stderr_path:
+                        try:
+                            with open(stderr_path, 'r', encoding='utf-8', errors='replace') as f:
+                                detail = f.read().strip()
+                        except Exception:
+                            pass
+                    msg = f"Failed to start agent (process exited with code {proc.returncode})"
+                    if detail:
+                        msg += f"\n{detail}"
+                    raise RuntimeError(msg)
+                if AgentClient.is_agent_running(port=port):
+                    return True
+        finally:
+            if stderr_path:
+                try:
+                    os.unlink(stderr_path)
+                except Exception:
+                    pass
 
         raise RuntimeError("Failed to start agent (timeout)")
 

{replx-1.6 → replx-1.6.1}/replx/cli/agent/server/connection_manager.py

@@ -455,9 +455,6 @@ class ConnectionManager:
 
             conn = self._connections.pop(key)
 
-        # Remember whether a detached (user) script was running before clearing the flag.
-        # If so, the device is executing paste-mode code on its UART, NOT in raw REPL.
-        # Sending CTRL_B in that state would inject \x02 into the running script's stdin.
         was_detached = conn.is_detached()
         conn.stop_detached()
 
@@ -474,8 +471,6 @@ class ConnectionManager:
                 if transport:
                     try:
                         if not was_detached:
-                            # Only send CTRL_B to leave raw REPL when no user script
-                            # was running; otherwise the byte contaminates the script stdin.
                             transport.write(CTRL_B)
                             time.sleep(0.1 if sys.platform == 'win32' else 0.05)
                     except Exception:

{replx-1.6 → replx-1.6.1}/replx/cli/agent/server/handlers/exec.py

@@ -204,11 +204,35 @@ class ExecCommandsMixin:
         if not conn or not conn.repl_protocol:
             send_error("Not connected")
             return
-        
+
         with conn.interactive.lock:
             if conn.interactive.active:
-                send_error("Interactive session already active on this connection")
-                return
+                thread = conn.interactive.thread
+                if thread is not None and thread.is_alive():
+                    conn.interactive.stop_requested = True
+                    try:
+                        conn.repl_protocol.interrupt()
+                    except Exception:
+                        pass
+                else:
+                    conn.interactive.active = False
+                    conn.interactive.thread = None
+
+        for _ in range(20):
+            with conn.interactive.lock:
+                if not conn.interactive.active:
+                    break
+            time.sleep(0.05)
+
+        with conn.interactive.lock:
+            if conn.interactive.active:
+                thread = conn.interactive.thread
+                if thread is None or not thread.is_alive():
+                    conn.interactive.active = False
+                    conn.interactive.thread = None
+                else:
+                    send_error("Interactive session already active on this connection")
+                    return
         
         repl = conn.repl_protocol
         if conn.is_detached():

{replx-1.6 → replx-1.6.1}/replx/cli/agent/server/handlers/session.py

@@ -71,7 +71,6 @@ class SessionCommandsMixin:
 
         self._add_connection(conn_key, new_conn)
 
-        # Pass original port to session
         self.session_manager.add_connection_to_session(
             ppid, port,
             as_foreground=as_foreground,
@@ -113,7 +112,6 @@ class SessionCommandsMixin:
             ports_to_close = list(session.get_all_connections())
             freed_port = "all"
         elif port:
-            # Find port in session
             found = None
             for conn in session.get_all_connections():
                 if conn == port:
@@ -141,8 +139,6 @@ class SessionCommandsMixin:
         if not self.session_manager.has_sessions():
             def delayed_shutdown():
                 time.sleep(0.3)
-                # cleanup() properly sets _stop_event to stop the asyncio loop
-                # and releases the UDP port so a new agent can bind it immediately.
                 self.cleanup()
 
             shutdown_thread = threading.Thread(target=delayed_shutdown, daemon=True)
@@ -178,7 +174,6 @@ class SessionCommandsMixin:
 
         old_fg = session.foreground
 
-        # Pass original port, not normalized
         session.add_connection(port, as_foreground=True)
 
         return {
@@ -189,7 +184,6 @@ class SessionCommandsMixin:
         }
 
     def _cmd_set_default(self, ctx: CommandContext, port: str = None, update_session: bool = False) -> dict:
-        """Set default port. If update_session=True, also update calling session's default_port."""
         port = port or ctx.explicit_port
         if port:
             self._set_default_port(port)

{replx-1.6 → replx-1.6.1}/replx/cli/app.py

@@ -513,7 +513,7 @@ def main():
             break
     first_nonopt = sys.argv[first_nonopt_idx] if first_nonopt_idx is not None else None
 
-    run_opts = {'-n', '--non-interactive', '-e', '--echo', '-d', '--device'}
+    run_opts = {'-n', '--non-interactive', '-e', '--echo', '-d', '--device', '--line', '--hex'}
     has_device_opt = bool(run_opts & {'-d', '--device'} & set(args))
     
     script_files = []
@@ -538,7 +538,7 @@ def main():
 
     if should_inject_run:
         opt_idx = next((i for i, a in enumerate(sys.argv[1:], 1) if a in run_opts), None)
-        insert_at = opt_idx if opt_idx is not None else script_arg_idx
+        insert_at = min(opt_idx, script_arg_idx) if opt_idx is not None else script_arg_idx
         sys.argv.insert(insert_at, 'run')
 
         first_nonopt_idx = next((i for i, a in enumerate(sys.argv[1:], 1) if not a.startswith('-')), None)

{replx-1.6 → replx-1.6.1}/replx/cli/commands/device.py

@@ -35,11 +35,6 @@ from ..app import app
 
 
 def _normalize_path_for_comparison(path: str) -> str:
-    """Normalize file path for comparison.
-    
-    Windows: case-insensitive
-    Linux/macOS: case-sensitive
-    """
     path = os.path.normpath(path)
     if sys.platform.startswith("win"):
         return path.lower()
@@ -48,12 +43,6 @@ def _normalize_path_for_comparison(path: str) -> str:
 
 
 def _serial_port_cmp_key(port: str) -> str:
-    """Comparison key for serial ports.
-
-    Policy:
-    - Display: keep OS-provided casing.
-    - Compare: on Windows, treat COM ports case-insensitively.
-    """
     if port is None:
         return ""
     p = str(port).strip()
@@ -61,10 +50,6 @@ def _serial_port_cmp_key(port: str) -> str:
 
 
 def _serial_port_display(port: str) -> str:
-    """Format a serial port name for display.
-
-    Requirement: on Windows, always show port names in uppercase.
-    """
     if port is None:
         return ""
     p = str(port).strip()
@@ -72,12 +57,6 @@ def _serial_port_display(port: str) -> str:
 
 
 def _resolve_os_serial_port_name(port: str) -> str:
-    """Resolve port name to the OS-enumerated spelling (best-effort).
-
-    This is mainly for Windows where users might type `com1` but the OS reports
-    `COM1`. We keep display as OS-provided and use case-insensitive comparison
-    separately.
-    """
     if not port:
         return port
     p = str(port).strip()
@@ -158,7 +137,6 @@ def _create_vscode_files_and_typehints(vscode_dir: str, core: str, device: str,
     
     extra_paths = []
     
-    # 1. MicroPython standard library typehints (included in replx tool)
     from replx.utils.device_info import is_std_micropython
     if is_std_micropython(core):
         comm_path = StoreManager.comm_typehints_path()
@@ -167,27 +145,22 @@ def _create_vscode_files_and_typehints(vscode_dir: str, core: str, device: str,
     if comm_path and os.path.isdir(comm_path):
         extra_paths.append(comm_path)
     
-    # 2. Core builtin typehints (included in replx tool)
     if core:
         core_builtin = StoreManager.core_typehints_path(core)
         if core_builtin and os.path.isdir(core_builtin):
             extra_paths.append(core_builtin)
     
-    # 3. Core library typehints (downloaded via 'replx pkg download')
     if core:
         core_lib_typehints = os.path.join(StoreManager.pkg_root(), "core", core, "typehints")
         if os.path.isdir(core_lib_typehints):
             extra_paths.append(core_lib_typehints)
     
-    # 4. Device builtin typehints (included in replx tool)
     if device:
         device_path = device_name_to_path(device)
         device_builtin = StoreManager.device_typehints_path(device_path)
         if device_builtin and os.path.isdir(device_builtin):
             extra_paths.append(device_builtin)
     
-    # 5. Device library typehints (downloaded via 'replx pkg download')
-    # Note: device name like 'ticle-lite' becomes 'ticle_lite' in filesystem
     if device:
         device_path = device_name_to_path(device)
         device_typehints = os.path.join(StoreManager.pkg_root(), "device", device_path, "typehints")
@@ -290,7 +263,6 @@ Run this once per project folder to set up your workspace.
     port = global_opts.get('port')
     agent_port = global_opts.get('agent_port')
 
-    # Store/use the OS-enumerated port spelling when possible.
     port = _resolve_os_serial_port_name(port)
 
     if not port:
@@ -306,7 +278,6 @@ Run this once per project folder to set up your workspace.
     
     env_path = _find_env_file()
     if agent_port is None:
-        # Use original port for connection lookup
         if env_path and port:
             existing_config = _get_connection_config(env_path, port)
             if existing_config and existing_config.get('agent_port'):
@@ -356,7 +327,6 @@ Run this once per project folder to set up your workspace.
                     env_path = os.path.join(vscode_dir, ".replx")
                     workspace = os.path.dirname(vscode_dir)
                     
-                    # Use original port for storing connection
                     _update_connection_config(
                         env_path, port,
                         version=version, core=core, device=device,
@@ -391,7 +361,6 @@ Run this once per project folder to set up your workspace.
                     )
                     raise typer.Exit()
                 else:
-                    # Use original port for session setup
                     try:
                         with AgentClient(port=agent_port, device_port=port) as client:
                             result = client.send_command(
@@ -410,7 +379,6 @@ Run this once per project folder to set up your workspace.
                         vscode_dir = _find_or_create_vscode_dir()
                         env_path = os.path.join(vscode_dir, ".replx")
                         
-                        # Use original port for storing connection
                         _update_connection_config(
                             env_path, port,
                             version=STATE.version, core=STATE.core, device=STATE.device,
@@ -478,7 +446,6 @@ Run this once per project folder to set up your workspace.
         )
         raise typer.Exit(1)
     
-    # Use original port for session setup
     try:
         with AgentClient(port=agent_port, device_port=port) as client:
             result = client.send_command('session_setup', 
@@ -542,7 +509,6 @@ Run this once per project folder to set up your workspace.
             except Exception:
                 pass
     
-    # Use original port for storing connection
     set_as_default = True
     
     if clean_mode:
@@ -1022,7 +988,6 @@ WIFI_STATUS_MESSAGES = {
 }
 
 def _wifi_status(client: AgentClient):
-    """Show WiFi status."""
     code = '''
 import network
 import json