portacode 1.4.13.dev0__py3-none-any.whl → 1.4.13.dev1__py3-none-any.whl

portacode/_version.py

@@ -28,7 +28,7 @@ version_tuple: VERSION_TUPLE
 commit_id: COMMIT_ID
 __commit_id__: COMMIT_ID
 
-__version__ = version = '1.4.13.dev0'
-__version_tuple__ = version_tuple = (1, 4, 13, 'dev0')
+__version__ = version = '1.4.13.dev1'
+__version_tuple__ = version_tuple = (1, 4, 13, 'dev1')
 
 __commit_id__ = commit_id = None

portacode/connection/handlers/WEBSOCKET_PROTOCOL.md

@@ -27,6 +27,8 @@ The Portacode server acts as a **routing middleman** between client sessions and
 
 - **`source_client_session`** (Server → Device): Server **adds this** when forwarding client commands to devices (so device knows which client sent the command and can target responses back). Clients never include this field.
 
+Device handlers that want to respond only to the session that issued a given command can rely on the helper defined in `portacode/connection/handlers/base.py` (`BaseHandler.send_response_to_source_session`). It wraps the boilerplate of reusing `request_id`/`trace`, injecting `client_sessions=[source_client_session]`, and sending via the control channel so the handler does not need to manually reconstruct routing metadata for each reply.
+
 This document describes the complete protocol for communicating with devices through the server, guiding app developers on how to get their client sessions to communicate with devices.
 
 ## Table of Contents

portacode/connection/handlers/base.py

@@ -88,6 +88,42 @@ class BaseHandler(ABC):
             if reply_channel:
                 payload["reply_channel"] = reply_channel
             await self.control_channel.send(payload)
+
+    async def send_response_to_source_session(
+        self,
+        message: Dict[str, Any],
+        payload: Dict[str, Any],
+        reply_channel: Optional[str] = None,
+    ) -> None:
+        """Send a response directly to the client session that issued ``message``.
+
+        This bypasses ``ClientSessionManager`` routing and ensures only the session
+        whose ``source_client_session`` was injected by the server will receive the
+        response. The helper also copies ``request_id``/``trace`` timing data so tracing
+        and logging remain consistent.
+        """
+        response = dict(payload)
+        request_id = message.get("request_id")
+        if request_id and "request_id" not in response:
+            response["request_id"] = request_id
+
+        if "trace" in message and "request_id" in message and "trace" not in response:
+            trace_data = dict(message["trace"])
+            handler_complete_time = ntp_clock.now_ms()
+            if handler_complete_time is not None:
+                trace_data["handler_complete"] = handler_complete_time
+                if "client_send" in trace_data:
+                    trace_data["ping"] = handler_complete_time - trace_data["client_send"]
+            response["trace"] = trace_data
+
+        source_client_session = message.get("source_client_session")
+        if source_client_session:
+            response["client_sessions"] = [source_client_session]
+
+        if reply_channel:
+            response["reply_channel"] = reply_channel
+
+        await self.control_channel.send(response)
     
     async def send_error(self, message: str, reply_channel: Optional[str] = None, project_id: str = None) -> None:
         """Send an error response with client session awareness.
@@ -141,22 +177,16 @@ class AsyncHandler(BaseHandler):
                 if "request_id" in message and "request_id" not in response:
                     response["request_id"] = message["request_id"]
 
-                # Pass through trace from request to response (add to existing trace, don't create new one)
-                if "trace" in message and "request_id" in message:
-                    response["trace"] = dict(message["trace"])
+                if "trace" in message and "request_id" in message and "trace" not in response:
+                    trace_data = dict(message["trace"])
                     handler_complete_time = ntp_clock.now_ms()
                     if handler_complete_time is not None:
-                        response["trace"]["handler_complete"] = handler_complete_time
-                        # Update ping to show total time from client_send
-                        if "client_send" in response["trace"]:
-                            response["trace"]["ping"] = handler_complete_time - response["trace"]["client_send"]
-                        logger.info(f"✅ Handler completed: {message['request_id']} ({self.command_name})")
-
-                # Extract project_id from response for session targeting
-                project_id = response.get("project_id")
-                logger.info("handler: %s response project_id=%s, response=%s",
-                           self.command_name, project_id, response)
-                await self.send_response(response, reply_channel, project_id)
+                        trace_data["handler_complete"] = handler_complete_time
+                        if "client_send" in trace_data:
+                            trace_data["ping"] = handler_complete_time - trace_data["client_send"]
+                    response["trace"] = trace_data
+                    logger.info(f"✅ Handler completed: {message['request_id']} ({self.command_name})")
+
             else:
                 logger.info("handler: %s handled response transmission directly", self.command_name)
         except Exception as exc:
@@ -216,4 +246,4 @@ class SyncHandler(BaseHandler):
             logger.exception("Error in sync handler %s: %s", self.command_name, exc)
             # Extract project_id from original message for error targeting
             project_id = message.get("project_id")
-            await self.send_error(str(exc), reply_channel, project_id) 
+            await self.send_error(str(exc), reply_channel, project_id) 

portacode/connection/handlers/proxmox_infra.py

@@ -1261,6 +1261,7 @@ class CreateProxmoxContainerHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response
 
 
@@ -1390,6 +1391,7 @@ class StartPortacodeServiceHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response
 
 
@@ -1431,6 +1433,7 @@ class StartProxmoxContainerHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response
 
 
@@ -1478,6 +1481,7 @@ class StopProxmoxContainerHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response
 
 
@@ -1523,6 +1527,7 @@ class RemoveProxmoxContainerHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response
 
 
@@ -1548,6 +1553,7 @@ class ConfigureProxmoxInfraHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response
 
 
@@ -1568,4 +1574,5 @@ class RevertProxmoxInfraHandler(SyncHandler):
         }
         if client_sessions:
             response["client_sessions"] = client_sessions
+        response["_reply_to_source_session"] = True
         return response

{portacode-1.4.13.dev0.dist-info → portacode-1.4.13.dev1.dist-info}/METADATA

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: portacode
-Version: 1.4.13.dev0
+Version: 1.4.13.dev1
 Summary: Portacode CLI client and SDK
 Home-page: https://github.com/portacode/portacode
 Author: Meena Erian

{portacode-1.4.13.dev0.dist-info → portacode-1.4.13.dev1.dist-info}/RECORD

@@ -1,7 +1,7 @@
 portacode/README.md,sha256=4dKtpvR8LNgZPVz37GmkQCMWIr_u25Ao63iW56s7Ke4,775
 portacode/__init__.py,sha256=oB3sV1wXr-um-RXio73UG8E5Xx6cF2ZVJveqjNmC-vQ,1086
 portacode/__main__.py,sha256=jmHTGC1hzmo9iKJLv-SSYe9BSIbPPZ2IOpecI03PlTs,296
-portacode/_version.py,sha256=EoCeFU0-D6AjtQLTsyubZ8HHL9Z71w1CLhD054hnlNo,719
+portacode/_version.py,sha256=LRfIU5637o003YhvJfKkC-W6_vMYzK36mKyg9SlBZi0,719
 portacode/cli.py,sha256=mGLKoZ-T2FBF7IA9wUq0zyG0X9__-A1ao7gajjcVRH8,21828
 portacode/data.py,sha256=5-s291bv8J354myaHm1Y7CQZTZyRzMU3TGe5U4hb-FA,1591
 portacode/keypair.py,sha256=0OO4vHDcF1XMxCDqce61xFTlFwlTcmqe5HyGsXFEt7s,5838
@@ -14,15 +14,15 @@ portacode/connection/client.py,sha256=jtLb9_YufqPkzi9t8VQH3iz_JEMisbtY6a8L9U5wei
 portacode/connection/multiplex.py,sha256=L-TxqJ_ZEbfNEfu1cwxgJ5vUdyRzZjsMy2Kx1diiZys,5237
 portacode/connection/terminal.py,sha256=oyLPOVLPlUuN_eRvHPGazB51yi8W8JEF3oOEYxucGTE,45069
 portacode/connection/handlers/README.md,sha256=HsLZG1QK1JNm67HsgL6WoDg9nxzKXxwkc5fJPFJdX5g,12169
-portacode/connection/handlers/WEBSOCKET_PROTOCOL.md,sha256=7tBYNEY8EBGAPIMT606BqeHnyMOQIZVlQYpH7me26LY,97962
+portacode/connection/handlers/WEBSOCKET_PROTOCOL.md,sha256=2sFO9VsvZUl57RM6R4cgYzN4WYlIQLASlKMa7r0NMSc,98404
 portacode/connection/handlers/__init__.py,sha256=WSeBmi65GWFQPYt9M3E10rn0uZ_EPCJzNJOzSf2HZyw,2921
-portacode/connection/handlers/base.py,sha256=oENFb-Fcfzwk99Qx8gJQriEMiwSxwygwjOiuCH36hM4,10231
+portacode/connection/handlers/base.py,sha256=YKjXeS_PDbM6RyrSauZBJvmndU2bfvCMOwckYLVLn-U,11296
 portacode/connection/handlers/chunked_content.py,sha256=h6hXRmxSeOgnIxoU8CkmvEf2Odv-ajPrpHIe_W3GKcA,9251
 portacode/connection/handlers/diff_handlers.py,sha256=iYTIRCcpEQ03vIPKZCsMTE5aZbQw6sF04M3dM6rUV8Q,24477
 portacode/connection/handlers/file_handlers.py,sha256=nAJH8nXnX07xxD28ngLpgIUzcTuRwZBNpEGEKdRqohw,39507
 portacode/connection/handlers/project_aware_file_handlers.py,sha256=AqgMnDqX2893T2NsrvUSCwjN5VKj4Pb2TN0S_SuboOE,9803
 portacode/connection/handlers/project_state_handlers.py,sha256=v6ZefGW9i7n1aZLq2jOGumJIjYb6aHlPI4m1jkYewm8,1686
-portacode/connection/handlers/proxmox_infra.py,sha256=QxD5FOQqRLz1Z8nhycdm2O991TWIXPjSC589xASZADQ,57264
+portacode/connection/handlers/proxmox_infra.py,sha256=bZVlM6k--CyLEBbhPvFafHTN-cyryRxy86EG6PpXoII,57628
 portacode/connection/handlers/registry.py,sha256=qXGE60sYEWg6ZtVQzFcZ5YI2XWR6lMgw4hAL9x5qR1I,6181
 portacode/connection/handlers/session.py,sha256=uNGfiO_1B9-_yjJKkpvmbiJhIl6b-UXlT86UTfd6WYE,42219
 portacode/connection/handlers/system_handlers.py,sha256=AKh7IbwptlLYrbSw5f-DHigvlaKHsg9lDP-lkAUm8cE,10755
@@ -64,7 +64,7 @@ portacode/utils/__init__.py,sha256=NgBlWTuNJESfIYJzP_3adI1yJQJR0XJLRpSdVNaBAN0,3
 portacode/utils/diff_apply.py,sha256=4Oi7ft3VUCKmiUE4VM-OeqO7Gk6H7PF3WnN4WHXtjxI,15157
 portacode/utils/diff_renderer.py,sha256=S76StnQ2DLfsz4Gg0m07UwPfRp8270PuzbNaQq-rmYk,13850
 portacode/utils/ntp_clock.py,sha256=VqCnWCTehCufE43W23oB-WUdAZGeCcLxkmIOPwInYHc,2499
-portacode-1.4.13.dev0.dist-info/licenses/LICENSE,sha256=2FGbCnUDgRYuQTkB1O1dUUpu5CVAjK1j4_p6ack9Z54,1066
+portacode-1.4.13.dev1.dist-info/licenses/LICENSE,sha256=2FGbCnUDgRYuQTkB1O1dUUpu5CVAjK1j4_p6ack9Z54,1066
 test_modules/README.md,sha256=Do_agkm9WhSzueXjRAkV_xEj6Emy5zB3N3VKY5Roce8,9274
 test_modules/__init__.py,sha256=1LcbHodIHsB0g-g4NGjSn6AMuCoGbymvXPYLOb6Z7F0,53
 test_modules/test_device_online.py,sha256=QtYq0Dq9vME8Gp2O4fGSheqVf8LUtpsSKosXXk56gGM,1654
@@ -90,8 +90,8 @@ testing_framework/core/playwright_manager.py,sha256=Tw46qwxIhOFkS48C2IWIQHHNpEe-
 testing_framework/core/runner.py,sha256=j2QwNJmAxVBmJvcbVS7DgPJUKPNzqfLmt_4NNdaKmZU,19297
 testing_framework/core/shared_cli_manager.py,sha256=BESSNtyQb7BOlaOvZmm04T8Uezjms4KCBs2MzTxvzYQ,8790
 testing_framework/core/test_discovery.py,sha256=2FZ9fJ8Dp5dloA-fkgXoJ_gCMC_nYPBnA3Hs2xlagzM,4928
-portacode-1.4.13.dev0.dist-info/METADATA,sha256=NuHta1wZHQRrXvVF3tcjW6ybf3wz3ivf3ktRZrYrn3c,13051
-portacode-1.4.13.dev0.dist-info/WHEEL,sha256=qELbo2s1Yzl39ZmrAibXA2jjPLUYfnVhUNTlyF1rq0Y,92
-portacode-1.4.13.dev0.dist-info/entry_points.txt,sha256=lLUUL-BM6_wwe44Xv0__5NQ1BnAz6jWjSMFvZdWW3zU,48
-portacode-1.4.13.dev0.dist-info/top_level.txt,sha256=TGhTYUxfW8SyVZc_zGgzjzc24gGT7nSw8Qf73liVRKM,41
-portacode-1.4.13.dev0.dist-info/RECORD,,
+portacode-1.4.13.dev1.dist-info/METADATA,sha256=mSEfLnN_1l9Wjf0LHMvOp5Gai18-BsmIgQ-jD80vAVY,13051
+portacode-1.4.13.dev1.dist-info/WHEEL,sha256=qELbo2s1Yzl39ZmrAibXA2jjPLUYfnVhUNTlyF1rq0Y,92
+portacode-1.4.13.dev1.dist-info/entry_points.txt,sha256=lLUUL-BM6_wwe44Xv0__5NQ1BnAz6jWjSMFvZdWW3zU,48
+portacode-1.4.13.dev1.dist-info/top_level.txt,sha256=TGhTYUxfW8SyVZc_zGgzjzc24gGT7nSw8Qf73liVRKM,41
+portacode-1.4.13.dev1.dist-info/RECORD,,