cyclo-manager 0.2.0.dev1__tar.gz → 0.2.0.dev3__tar.gz

{cyclo_manager-0.2.0.dev1 → cyclo_manager-0.2.0.dev3}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cyclo-manager
-Version: 0.2.0.dev1
+Version: 0.2.0.dev3
 Summary: cyclo_manager CLI: pip-installable launcher for cyclo_manager server and UI containers. Run 'cyclo_manager up' to start Docker stack.
 License-Expression: Apache-2.0
 Requires-Python: >=3.10

{cyclo_manager-0.2.0.dev1 → cyclo_manager-0.2.0.dev3}/cyclo_host_agent/routers/repos.py

@@ -22,7 +22,6 @@ import asyncio
 import json
 import os
 import re
-import subprocess
 import urllib.request
 from pathlib import Path
 from typing import Optional
@@ -87,23 +86,20 @@ def _fmt_cmd(cmd: str, output: str) -> str:
 
 # ── git helpers ────────────────────────────────────────────────────────────────
 
-def _git_sync(args: list[str], cwd: Optional[Path] = None) -> tuple[int, str, str]:
-    try:
-        result = subprocess.run(
-            ['git', *args],
-            cwd=str(cwd) if cwd else None,
-            capture_output=True,
-            timeout=GIT_TIMEOUT,
-        )
-        return result.returncode, result.stdout.decode(), result.stderr.decode()
-    except subprocess.TimeoutExpired:
-        return 1, '', 'git timed out'
-    except Exception as e:
-        return 1, '', str(e)
-
-
 async def _git(args: list[str], cwd: Optional[Path] = None) -> tuple[int, str, str]:
-    return await asyncio.to_thread(_git_sync, args, cwd)
+    proc = await asyncio.create_subprocess_exec(
+        'git', *args,
+        cwd=str(cwd) if cwd else None,
+        stdout=asyncio.subprocess.PIPE,
+        stderr=asyncio.subprocess.PIPE,
+    )
+    try:
+        stdout, stderr = await asyncio.wait_for(proc.communicate(), timeout=GIT_TIMEOUT)
+    except asyncio.TimeoutError:
+        proc.kill()
+        await proc.wait()
+        raise
+    return proc.returncode, stdout.decode(), stderr.decode()
 
 
 async def _repo_info(repo_path: Path) -> RepoInfo:

{cyclo_manager-0.2.0.dev1 → cyclo_manager-0.2.0.dev3}/cyclo_manager.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: cyclo-manager
-Version: 0.2.0.dev1
+Version: 0.2.0.dev3
 Summary: cyclo_manager CLI: pip-installable launcher for cyclo_manager server and UI containers. Run 'cyclo_manager up' to start Docker stack.
 License-Expression: Apache-2.0
 Requires-Python: >=3.10

{cyclo_manager-0.2.0.dev1 → cyclo_manager-0.2.0.dev3}/cyclo_manager_cli/cli.py

@@ -33,6 +33,7 @@ import sys
 PYPI_PACKAGE = 'cyclo-manager'
 HOST_AGENT_SERVICE = 'cyclo_host_agent'
 HOST_AGENT_SOCKET = '/var/run/robotis/agent_sockets/host/host_agent.sock'
+HOST_AGENT_SOCKET_DIR = '/var/run/robotis/agent_sockets/host'
 
 # `cyclo_manager up` starts these immediately.
 COMPOSE_SERVICES_UP = ('cyclo_manager', 'ui')
@@ -53,24 +54,67 @@ def _packaged_config_path() -> Path:
     return _config_dir() / 'config.yml'
 
 
-def _check_host_agent() -> bool:
-    """Return True if the cyclo_host_agent service is already active."""
-    result = subprocess.run(
-        ['systemctl', 'is-active', f'{HOST_AGENT_SERVICE}.service'],
-        capture_output=True,
-        text=True,
-    )
-    return result.returncode == 0
+def _resolve_service_user() -> tuple[str, Path] | None:
+    """
+    Return (username, home) for the host agent systemd service.
 
+    Uses SUDO_USER when cyclo_manager up is invoked via sudo so the agent runs
+    as the login user (required for git repo scans under ~/...).
+    """
+    sudo_user = os.environ.get('SUDO_USER')
+    if sudo_user:
+        return sudo_user, Path(f'/home/{sudo_user}')
+    user = os.environ.get('USER') or os.getlogin()
+    if user == 'root':
+        print(
+            'Error: cyclo_manager up must run as a normal user (e.g. robotis), not root.',
+            file=sys.stderr,
+        )
+        return None
+    return user, Path.home()
 
 
-def _create_host_agent() -> int:
+def _setup_socket_dir(user: str) -> bool:
     """
-    Write cyclo_host_agent systemd service unit file and enable it.
+    Ensure the host agent UDS directory exists and is owned by the service user.
 
-    Runs as a persistent service (Restart=always); systemd starts it at boot
-    and restarts it automatically if it exits unexpectedly.
+    Only host/ is chowned so other agent sockets under agent_sockets/ are untouched.
+    Removes a stale socket file (e.g. left by a previous root-owned service).
     """
+    try:
+        subprocess.run(
+            ['sudo', 'mkdir', '-p', HOST_AGENT_SOCKET_DIR],
+            check=True,
+            capture_output=True,
+        )
+        subprocess.run(
+            ['sudo', 'chown', f'{user}:{user}', HOST_AGENT_SOCKET_DIR],
+            check=True,
+            capture_output=True,
+        )
+        subprocess.run(
+            ['sudo', 'rm', '-f', HOST_AGENT_SOCKET],
+            check=True,
+            capture_output=True,
+        )
+        return True
+    except subprocess.CalledProcessError as e:
+        print(f'Failed to set up host agent socket directory: {e}', file=sys.stderr)
+        return False
+
+
+def _ensure_host_agent() -> int:
+    """
+    Install or refresh cyclo_host_agent: systemd unit, socket dir, and service start.
+
+    Idempotent — safe to run on every `cyclo_manager up` (fixes upgraded units and
+    socket ownership after switching away from a root-owned service).
+    """
+    account = _resolve_service_user()
+    if account is None:
+        return 1
+
+    user, user_home = account
     agent_exe = shutil.which('cyclo_host_agent')
     if not agent_exe:
         print(
@@ -80,8 +124,9 @@ def _create_host_agent() -> int:
         )
         return 1
 
-    current_user = os.environ.get('SUDO_USER') or os.environ.get('USER') or os.getlogin()
-    user_home = Path(f'/home/{current_user}') if current_user else Path.home()
+    if not _setup_socket_dir(user):
+        return 1
+
     service_content = f"""\
 [Unit]
 Description=Cyclo Host Agent
@@ -89,7 +134,8 @@ After=network.target
 
 [Service]
 Type=simple
-User={current_user}
+User={user}
+Group={user}
 ExecStart={agent_exe}
 Environment=HOME={user_home}
 Restart=always
@@ -110,7 +156,8 @@ WantedBy=multi-user.target
             check=True,
         )
         subprocess.run(['sudo', 'systemctl', 'daemon-reload'], check=True)
-        subprocess.run(['sudo', 'systemctl', 'enable', '--now', f'{HOST_AGENT_SERVICE}.service'], check=True)
+        subprocess.run(['sudo', 'systemctl', 'enable', f'{HOST_AGENT_SERVICE}.service'], check=True)
+        subprocess.run(['sudo', 'systemctl', 'restart', f'{HOST_AGENT_SERVICE}.service'], check=True)
         print('Host agent service installed and started.')
         return 0
     except subprocess.CalledProcessError as e:
@@ -123,9 +170,8 @@ WantedBy=multi-user.target
 
 def cmd_up(args: argparse.Namespace) -> int:
     """Start API + UI; create zenoh and noVNC containers without starting them."""
-    if not _check_host_agent():
-        if _create_host_agent() != 0:
-            print('Warning: Failed to install host agent service.')
+    if _ensure_host_agent() != 0:
+        print('Warning: Failed to install host agent service.', file=sys.stderr)
 
     config_path = _packaged_config_path()
     if not config_path.is_file():

{cyclo_manager-0.2.0.dev1 → cyclo_manager-0.2.0.dev3}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "setuptools.build_meta"
 
 [project]
 name = "cyclo-manager"
-version = "0.2.0.dev1"
+version = "0.2.0.dev3"
 description = "cyclo_manager CLI: pip-installable launcher for cyclo_manager server and UI containers. Run 'cyclo_manager up' to start Docker stack."
 readme = "README.md"
 license = "Apache-2.0"

{cyclo_manager-0.2.0.dev1 → cyclo_manager-0.2.0.dev3}/setup.cfg

@@ -1,6 +1,6 @@
 [metadata]
 name = cyclo-manager
-version = 0.2.0dev
+version = 0.2.0de2
 
 [egg_info]
 egg_base = .