portacode 2.2.20.dev0__tar.gz → 2.2.20.dev1__tar.gz

{portacode-2.2.20.dev0 → portacode-2.2.20.dev1}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: portacode
-Version: 2.2.20.dev0
+Version: 2.2.20.dev1
 Summary: Device agent for the AI platform that builds, tests, deploys, and operates real software
 Home-page: https://portacode.com/
 Author: Meena Erian

{portacode-2.2.20.dev0 → portacode-2.2.20.dev1}/portacode/_version.py

@@ -18,7 +18,7 @@ version_tuple: tuple[int | str, ...]
 commit_id: str | None
 __commit_id__: str | None
 
-__version__ = version = '2.2.20.dev0'
-__version_tuple__ = version_tuple = (2, 2, 20, 'dev0')
+__version__ = version = '2.2.20.dev1'
+__version_tuple__ = version_tuple = (2, 2, 20, 'dev1')
 
 __commit_id__ = commit_id = None

{portacode-2.2.20.dev0 → portacode-2.2.20.dev1}/portacode/connection/handlers/browser_handlers.py

@@ -8,6 +8,7 @@ import logging
 import os
 import re
 import shutil
+import sys
 import time
 from pathlib import Path
 from typing import Any, Dict
@@ -142,6 +143,25 @@ async def _node_path() -> str:
     return _NODE_PATH
 
 
+def _node_executable() -> str:
+    """Use Playwright's packaged Node runtime before any system installation."""
+    try:
+        import playwright
+
+        name = "node.exe" if sys.platform == "win32" else "node"
+        bundled = Path(playwright.__file__).resolve().parent / "driver" / name
+        if bundled.is_file() and os.access(bundled, os.X_OK):
+            return str(bundled)
+    except (ImportError, OSError, TypeError):
+        pass
+    system_node = shutil.which("node") or shutil.which("nodejs")
+    if system_node:
+        return system_node
+    raise RuntimeError(
+        "Unable to locate the Playwright Node runtime or a system node executable"
+    )
+
+
 class _BrowserWorker:
     def __init__(self, *, user: str, session_id: str, argv: list[str], env: dict[str, str]):
         self.user = user
@@ -323,7 +343,7 @@ class BrowserRunHandler(AsyncHandler):
         env = dict(os.environ)
         env["NODE_PATH"] = await _node_path()
         env.setdefault("PLAYWRIGHT_BROWSERS_PATH", "/opt/ms-playwright")
-        argv = wrap_argv_for_user(["node", "-e", _RUNNER], user, login=False)
+        argv = wrap_argv_for_user([_node_executable(), "-e", _RUNNER], user, login=False)
         worker = await _worker_for(
             user=user, session_id=session_id, argv=argv, env=env,
         )

{portacode-2.2.20.dev0 → portacode-2.2.20.dev1}/portacode/connection/handlers/test_browser_handlers.py

@@ -50,6 +50,36 @@ class BrowserRunHandlerTests(unittest.IsolatedAsyncioTestCase):
             os.pathsep.join(["/opt/portacode-playwright/node_modules", "/usr/lib/node_modules"]),
         )
 
+    def test_uses_node_runtime_bundled_with_python_playwright(self):
+        fake_package = SimpleNamespace(__file__="/opt/venv/playwright/__init__.py")
+        bundled = "/opt/venv/playwright/driver/node"
+        with (
+            patch.dict("sys.modules", {"playwright": fake_package}),
+            patch.object(browser_handlers.Path, "is_file", return_value=True),
+            patch.object(browser_handlers.os, "access", return_value=True),
+            patch.object(browser_handlers.shutil, "which") as which,
+        ):
+            self.assertEqual(browser_handlers._node_executable(), bundled)
+        which.assert_not_called()
+
+    def test_node_runtime_falls_back_to_system_nodejs(self):
+        with (
+            patch.dict("sys.modules", {"playwright": None}),
+            patch.object(
+                browser_handlers.shutil, "which",
+                side_effect=lambda name: "/usr/bin/nodejs" if name == "nodejs" else None,
+            ),
+        ):
+            self.assertEqual(browser_handlers._node_executable(), "/usr/bin/nodejs")
+
+    def test_node_runtime_has_clear_error_when_unavailable(self):
+        with (
+            patch.dict("sys.modules", {"playwright": None}),
+            patch.object(browser_handlers.shutil, "which", return_value=None),
+        ):
+            with self.assertRaisesRegex(RuntimeError, "Playwright Node runtime"):
+                browser_handlers._node_executable()
+
     async def test_reuses_worker_for_same_runtime_user_and_session(self):
         worker = AsyncMock()
         worker.run.return_value = {"ok": True, "steps": [], "screenshots": []}
@@ -60,6 +90,7 @@ class BrowserRunHandlerTests(unittest.IsolatedAsyncioTestCase):
             patch.object(browser_handlers, "mkdir_with_owner"),
             patch.object(browser_handlers, "wrap_argv_for_user", return_value=["node"]),
             patch.object(browser_handlers, "_node_path", AsyncMock(return_value="/modules")),
+            patch.object(browser_handlers, "_node_executable", return_value="/bundled/node"),
             patch.object(browser_handlers, "_worker_for", AsyncMock(return_value=worker)) as get_worker,
         ):
             first = await self.handler.execute(message)

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

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: portacode
-Version: 2.2.20.dev0
+Version: 2.2.20.dev1
 Summary: Device agent for the AI platform that builds, tests, deploys, and operates real software
 Home-page: https://portacode.com/
 Author: Meena Erian