portacode 1.4.18.dev1__tar.gz → 1.4.19.dev0__tar.gz

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/PKG-INFO

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

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/portacode/_version.py

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

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/portacode/connection/handlers/WEBSOCKET_PROTOCOL.md

@@ -408,7 +408,7 @@ Deletes a managed container from Proxmox (stopping it first if necessary) and re
 
 **Payload Fields:**
 
-*   `ctid` (string, required): Identifier of the container to delete.
+*   `ctid` (string, optional): Identifier of the container to delete. If omitted, the handler resolves the CTID from the managed container records using `child_device_id`.
 *   `child_device_id` (string, required): Dashboard `Device.id` that should own the container metadata being purged.
 
 **Responses:**

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/portacode/connection/handlers/base.py

@@ -62,16 +62,18 @@ class BaseHandler(ABC):
         # Get client session manager from context
         client_session_manager = self.context.get("client_session_manager")
 
+        bypass_session_gate = bool(payload.get("bypass_session_gate"))
         if client_session_manager and client_session_manager.has_interested_clients():
             # Get target sessions
             target_sessions = client_session_manager.get_target_sessions(project_id)
-            if not target_sessions:
+            if not target_sessions and not bypass_session_gate:
                 logger.debug("handler: No target sessions found, skipping response send")
                 return
 
             # Add session targeting information
             enhanced_payload = dict(payload)
-            enhanced_payload["client_sessions"] = target_sessions
+            if target_sessions:
+                enhanced_payload["client_sessions"] = target_sessions
 
             # Add backward compatibility reply_channel (first session if not provided)
             if not reply_channel:
@@ -89,15 +91,24 @@ class BaseHandler(ABC):
                 payload["reply_channel"] = reply_channel
             await self.control_channel.send(payload)
     
-    async def send_error(self, message: str, reply_channel: Optional[str] = None, project_id: str = None) -> None:
+    async def send_error(
+        self,
+        message: str,
+        reply_channel: Optional[str] = None,
+        project_id: str = None,
+        request_id: Optional[str] = None,
+    ) -> None:
         """Send an error response with client session awareness.
         
         Args:
             message: Error message
             reply_channel: Optional reply channel for backward compatibility
             project_id: Optional project filter for targeting specific sessions
+            request_id: Optional request_id to correlate error with a request
         """
         payload = {"event": "error", "message": message}
+        if request_id:
+            payload["request_id"] = request_id
         await self.send_response(payload, reply_channel, project_id)
 
 
@@ -163,7 +174,12 @@ class AsyncHandler(BaseHandler):
             logger.exception("handler: Error in async 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,
+                request_id=message.get("request_id"),
+            )
 
 
 class SyncHandler(BaseHandler):
@@ -216,4 +232,9 @@ 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,
+                request_id=message.get("request_id"),
+            )

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/portacode/connection/handlers/proxmox_infra.py

@@ -116,6 +116,7 @@ def _emit_progress_event(
         payload["details"] = details
     if on_behalf_of_device:
         payload["on_behalf_of_device"] = str(on_behalf_of_device)
+        payload["bypass_session_gate"] = True
 
     future = asyncio.run_coroutine_threadsafe(handler.send_response(payload), loop)
     future.add_done_callback(
@@ -1399,6 +1400,25 @@ def _parse_ctid(message: Dict[str, Any]) -> int:
     raise ValueError("ctid is required")
 
 
+def _resolve_vmid_for_device(device_id: str) -> int:
+    _initialize_managed_containers_state()
+    records = list(_MANAGED_CONTAINERS_STATE.get("records", {}).values())
+    for record in records:
+        record_device_id = record.get("device_id")
+        if record_device_id is None:
+            continue
+        if str(record_device_id) != str(device_id):
+            continue
+        vmid = record.get("vmid")
+        if vmid is None:
+            continue
+        try:
+            return int(str(vmid).strip())
+        except ValueError:
+            raise ValueError("ctid must be an integer") from None
+    raise ValueError("ctid is required for remove_proxmox_container")
+
+
 def _ensure_container_managed(
     proxmox: Any, node: str, vmid: int, *, device_id: Optional[str] = None
 ) -> Tuple[Dict[str, Any], Dict[str, Any]]:
@@ -2331,6 +2351,7 @@ class CreateProxmoxContainerHandler(SyncHandler):
                         "setup_steps": steps,
                         "device_id": device_id,
                         "on_behalf_of_device": device_id,
+                        "bypass_session_gate": True,
                         "service_installed": service_installed,
                         "request_id": request_id,
                     },
@@ -2357,7 +2378,6 @@ class CreateProxmoxContainerHandler(SyncHandler):
             "event": "proxmox_container_accepted",
             "success": True,
             "message": "Provisioning accepted; resources reserved.",
-            "device_id": device_id,
             "request_id": request_id,
         }
 
@@ -2557,10 +2577,14 @@ class RemoveProxmoxContainerHandler(SyncHandler):
         return "remove_proxmox_container"
 
     def execute(self, message: Dict[str, Any]) -> Dict[str, Any]:
-        vmid = _parse_ctid(message)
         child_device_id = (message.get("child_device_id") or "").strip()
         if not child_device_id:
             raise ValueError("child_device_id is required for remove_proxmox_container")
+        try:
+            vmid = _parse_ctid(message)
+        except ValueError:
+            vmid = _resolve_vmid_for_device(child_device_id)
+        request_id = message.get("request_id")
         config = _ensure_infra_configured()
         proxmox = _connect_proxmox(config)
         node = _get_node_from_config(config)
@@ -2582,6 +2606,9 @@ class RemoveProxmoxContainerHandler(SyncHandler):
                 "delete_exitstatus": delete_status.get("exitstatus"),
             },
             "status": "deleted",
+            "child_device_id": child_device_id,
+            "on_behalf_of_device": child_device_id,
+            "request_id": request_id,
             "infra": infra,
         }
 

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/portacode/connection/handlers/registry.py

@@ -106,12 +106,25 @@ class CommandRegistry:
         except Exception as exc:
             logger.exception("registry: Error dispatching command %s: %s", command_name, exc)
             # Send session-aware error response
-            await self._send_session_aware_error(str(exc), reply_channel, message.get("project_id"))
+            await self._send_session_aware_error(
+                str(exc),
+                reply_channel,
+                message.get("project_id"),
+                request_id=message.get("request_id"),
+            )
             return False
     
-    async def _send_session_aware_error(self, message: str, reply_channel: Optional[str] = None, project_id: str = None) -> None:
+    async def _send_session_aware_error(
+        self,
+        message: str,
+        reply_channel: Optional[str] = None,
+        project_id: str = None,
+        request_id: Optional[str] = None,
+    ) -> None:
         """Send an error response with client session awareness."""
         error_payload = {"event": "error", "message": message}
+        if request_id:
+            error_payload["request_id"] = request_id
         
         # Get client session manager from context
         client_session_manager = self.context.get("client_session_manager")
@@ -151,4 +164,4 @@ class CommandRegistry:
         
         # Update context for all existing handlers
         for handler in self._handlers.values():
-            handler.context = self.context 
+            handler.context = self.context 

{portacode-1.4.18.dev1 → portacode-1.4.19.dev0}/portacode.egg-info/PKG-INFO

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