Merge pull request 'fix: remove venv volume mount conflicting with entrypoint.sh' (#49) from fix/remove-venv-mount into master

Reviewed-on: #49

ai/compose.yml

@@ -32,13 +32,16 @@ services:
         - default
     container_name: hermes
     entrypoint: ["/bin/bash", "-c",
-      "bash /opt/data/hermes-tools/install.sh && exec /usr/bin/tini -g -- /opt/hermes/docker/entrypoint.sh \"$@\"",
+      "bash /opt/data/hermes-tools/install.sh && bash /opt/data/hermes-tools/run-multi-gateways.sh && exec /usr/bin/tini -g -- /opt/hermes/docker/entrypoint.sh \"$@\"",
       "hermes-entrypoint"]
     restart: always
-    # Gateway run enables the internal API server on port 8642
-    command: gateway run
     environment:
       - OLLAMA_HOST=http://ollama:11434
+      - HERMES_DASHBOARD=1
+      # Multi-profile: comma-separated list of profiles to run as gateways.
+      # The entrypoint reads this and starts one gateway per profile.
+      # Add profiles here when they exist on disk (e.g. default,researcher,writer)
+      - HERMES_PROFILES=ashley,claire,finn,matt,paul
       - API_SERVER_ENABLED=true
       - API_SERVER_PORT=8642
       - API_SERVER_HOST=0.0.0.0
@@ -66,6 +69,30 @@ services:
       - "26"
     networks:
       - ai_backend
+      - ai_net
+    labels:
+      - "traefik.enable=true"
+      - "traefik.docker.network=ai_net"
+
+      # Router for HTTP + redirection to HTTPS
+      - "traefik.http.routers.hermes-web-http.rule=Host(`hermes.lazyworkhorse.net`)"
+      - "traefik.http.routers.hermes-web-http.entrypoints=web"
+      - "traefik.http.routers.hermes-web-http.middlewares=redirect-to-https"
+
+      # Router for HTTPS with TLS — protected by Authelia
+      - "traefik.http.routers.hermes-web-https.rule=Host(`hermes.lazyworkhorse.net`)"
+      - "traefik.http.routers.hermes-web-https.entrypoints=websecure"
+      - "traefik.http.routers.hermes-web-https.tls=true"
+      - "traefik.http.routers.hermes-web-https.tls.certresolver=njalla"
+      - "traefik.http.routers.hermes-web-https.middlewares=hermes-auth"
+
+      # Authelia forwardAuth
+      - "traefik.http.middlewares.hermes-auth.forwardauth.address=http://authelia:9091/api/verify?rd=https://auth.lazyworkhorse.net/"
+      - "traefik.http.middlewares.hermes-auth.forwardauth.trustforwardheader=true"
+      - "traefik.http.middlewares.hermes-auth.forwardauth.authresponseheaders=X-Forwarded-User,X-Forwarded-Groups"
+
+      # Service Loadbalancer (dashboard port 9119)
+      - "traefik.http.services.hermes-web.loadbalancer.server.port=9119"
 
   syncthing:
     image: syncthing/syncthing:latest

ai/hermes-workspace/entrypoint-combined.sh

@@ -1,154 +0,0 @@
-#!/bin/bash
-# ── Hermes Workspace Combined Entrypoint ──
-# Waits for the Hermes gateway container (hermes:8642) to become healthy,
-# then starts the Hermes Workspace web UI in the foreground.
-# Supports graceful shutdown via SIGTERM/SIGINT.
-# ──────────────────────────────────────────
-
-set -euo pipefail
-
-# ── Configuration ──────────────────────────────────────────────
-GATEWAY_HOST="${GATEWAY_HOST:-hermes}"
-GATEWAY_PORT="${GATEWAY_PORT:-8642}"
-GATEWAY_URL="http://${GATEWAY_HOST}:${GATEWAY_PORT}"
-
-HEALTH_ENDPOINT="${HEALTH_ENDPOINT:-/health}"
-MAX_RETRIES="${HEALTH_MAX_RETRIES:-60}"
-RETRY_INTERVAL="${HEALTH_RETRY_INTERVAL:-2}"
-
-WORKSPACE_DIR="${WORKSPACE_DIR:-/workspace}"
-WORKSPACE_ENTRY="${WORKSPACE_ENTRY:-server-entry.js}"
-
-PID_FILE="${PID_FILE:-/tmp/workspace.pid}"
-
-# ── Logging ────────────────────────────────────────────────────
-log_info()  { echo "[$(date '+%Y-%m-%d %H:%M:%S')] [INFO]  $*"; }
-log_warn()  { echo "[$(date '+%Y-%m-%d %H:%M:%S')] [WARN]  $*"; }
-log_error() { echo "[$(date '+%Y-%m-%d %H:%M:%S')] [ERROR] $*"; }
-
-# ── Graceful Shutdown ──────────────────────────────────────────
-_workspace_pid=""
-_shutting_down=false
-
-cleanup() {
-    if [ "$_shutting_down" = true ]; then
-        return
-    fi
-    _shutting_down=true
-
-    log_info "Shutdown signal received, cleaning up..."
-
-    # Stop workspace process if running
-    if [ -n "$_workspace_pid" ] && kill -0 "$_workspace_pid" 2>/dev/null; then
-        log_info "Stopping workspace (PID: $_workspace_pid)..."
-        kill -TERM "$_workspace_pid" 2>/dev/null || true
-
-        # Give it time to shut down gracefully
-        local wait_sec=10
-        while kill -0 "$_workspace_pid" 2>/dev/null && [ "$wait_sec" -gt 0 ]; do
-            sleep 1
-            wait_sec=$((wait_sec - 1))
-        done
-
-        # Force kill if still running
-        if kill -0 "$_workspace_pid" 2>/dev/null; then
-            log_warn "Workspace did not shut down gracefully, force killing..."
-            kill -KILL "$_workspace_pid" 2>/dev/null || true
-        fi
-    fi
-
-    # Clean up PID file
-    [ -f "$PID_FILE" ] && rm -f "$PID_FILE"
-
-    log_info "Shutdown complete."
-    exit 0
-}
-
-# Trap termination signals for graceful shutdown
-trap cleanup SIGTERM SIGINT
-
-# ── Gateway Health Check ───────────────────────────────────────
-wait_for_gateway() {
-    local url="${GATEWAY_URL}${HEALTH_ENDPOINT}"
-    local retries="$MAX_RETRIES"
-    local interval="$RETRY_INTERVAL"
-    local attempt=0
-
-    log_info "Waiting for Hermes gateway at ${GATEWAY_URL}..."
-    log_info "Max retries: ${retries}, interval: ${interval}s"
-
-    while [ "$attempt" -lt "$retries" ]; do
-        attempt=$((attempt + 1))
-
-        if curl -fsS "${url}" >/dev/null 2>&1; then
-            log_info "Gateway is healthy after ${attempt} attempt(s) (${GATEWAY_URL})"
-            return 0
-        fi
-
-        if [ "$attempt" -lt "$retries" ]; then
-            log_info "Gateway not ready yet (attempt ${attempt}/${retries}), retrying in ${interval}s..."
-            sleep "$interval"
-        fi
-    done
-
-    log_error "Gateway did not become healthy after ${retries} attempts ($((retries * interval))s)"
-    return 1
-}
-
-# ── Workspace Startup ──────────────────────────────────────────
-start_workspace() {
-    local entry="${WORKSPACE_DIR}/${WORKSPACE_ENTRY}"
-
-    if [ ! -d "$WORKSPACE_DIR" ]; then
-        log_error "Workspace directory not found: ${WORKSPACE_DIR}"
-        return 1
-    fi
-
-    if [ ! -f "$entry" ]; then
-        log_error "Workspace entry point not found: ${entry}"
-        return 1
-    fi
-
-    log_info "Starting Hermes Workspace web UI..."
-    log_info "  Directory: ${WORKSPACE_DIR}"
-    log_info "  Entry:     ${entry}"
-
-    cd "$WORKSPACE_DIR"
-
-    # Start workspace in background so we can trap signals
-    node --max-old-space-size=2048 "${entry}" &
-    _workspace_pid=$!
-    echo "$_workspace_pid" > "$PID_FILE"
-
-    log_info "Workspace started (PID: ${_workspace_pid})"
-
-    # Wait for workspace process
-    wait "$_workspace_pid"
-    local exit_code=$?
-
-    log_info "Workspace exited with code ${exit_code}"
-    return "$exit_code"
-}
-
-# ── Main ───────────────────────────────────────────────────────
-main() {
-    log_info "=== Hermes Workspace Combined Entrypoint ==="
-    log_info "Gateway:   ${GATEWAY_URL}"
-    log_info "Workspace: ${WORKSPACE_DIR}/${WORKSPACE_ENTRY}"
-    log_info "PID file:  ${PID_FILE}"
-
-    # Wait for gateway to be healthy
-    if ! wait_for_gateway; then
-        log_warn "Proceeding without confirmed gateway health..."
-    fi
-
-    # Start the workspace
-    start_workspace
-    local exit_code=$?
-
-    log_info "Entrypoint exiting with code ${exit_code}"
-    return "$exit_code"
-}
-
-# Run main; exit with its return code
-main "$@"

ai/hermes/Dockerfile

@@ -20,16 +20,10 @@ RUN --mount=type=ssh \
     GIT_SSH_COMMAND='ssh -p 2222 -o StrictHostKeyChecking=no' \
     git clone --depth 1 --branch main \
     git@code.lazyworkhorse.net:gortium/hermes-agent.git fork && \
-    rsync -a --delete fork/ /opt/hermes/ \
-      --exclude node_modules \
-      --exclude .venv \
-      --exclude .git && \
+    rm -rf fork/node_modules fork/.venv fork/.git && \
+    cp -a fork/. /opt/hermes/ && \
     rm -rf /tmp/fork /root/.ssh/
 
-# ---------- Rebuild web UI ----------
-# Source files changed; node_modules (from base image) reused.
-RUN cd /opt/hermes && npm run build
-
 # ---------- Reinstall Python package (editable) ----------
 # Picks up source changes from our fork.
 RUN . /opt/hermes/.venv/bin/activate && \
@@ -40,6 +34,7 @@ USER root
 RUN apt-get update && \
     apt-get install -y --no-install-recommends \
         libportaudio2 ca-certificates poppler-utils imagemagick \
+        libolm-dev \
         texlive-latex-base texlive-latex-extra texlive-fonts-recommended \
         texlive-xetex texlive-science \
         qemu-user-static binfmt-support emacs-nox && \
@@ -48,6 +43,12 @@ RUN apt-get update && \
 # ---------- UV ----------
 COPY --chmod=0755 --from=ghcr.io/astral-sh/uv:latest /uv /usr/local/bin/
 
+# ---------- Matrix bridge + extra pip deps ----------
+# Previously installed inline at container startup and persisted via volume mount.
+# Now baked into the image so the fragile venv volume mount can be removed.
+RUN . /opt/hermes/.venv/bin/activate && \
+    uv pip install --no-cache-dir 'mautrix[encryption]' openai
+
 # ---------- Piper TTS ----------
 RUN . /opt/hermes/.venv/bin/activate && \
     uv pip install --no-cache-dir piper-tts sounddevice numpy && \
@@ -75,10 +76,6 @@ os.remove(tgz)
 print('himalaya v1.2.0 installed')
 PYEOF
 
-# ---------- Install himalaya-ro wrapper ----------
-COPY --chmod=0755 himalaya-ro.sh /usr/local/bin/himalaya-ro
-
-
 # ---------- Runtime ----------
 USER hermes
 ENV HERMES_HOME=/opt/data
@@ -88,6 +85,7 @@ ENV CHROME_EXECUTABLE=/opt/hermes/.playwright/chromium/chrome-linux/chrome
 
 # Ensure tools directory and toolsets.py are writable by the hermes runtime user
 # so custom tools can be injected from the persistent volume at startup.
+USER root
 RUN chown -R hermes:hermes /opt/hermes/tools /opt/hermes/toolsets.py
 
 VOLUME [ "/opt/data" ]

ai/hermes/himalaya-ro.sh

@@ -1,73 +0,0 @@
-#!/usr/bin/env bash
-# ─────────────────────────────────────────────────────────────
-# himalaya-ro — Read-only wrapper for himalaya
-#
-# Blocks destructive commands and logs audit trail.
-# Pass-through for read-only commands (list, read, search).
-#
-# Usage:  himalaya-ro [options] <command> [args...]
-#
-# Install: place in PATH before the real himalaya, or use
-#          `ln -sf himalaya-ro /usr/local/bin/himalaya`
-# ─────────────────────────────────────────────────────────────
-set -o pipefail
-
-# ── Configuration ───────────────────────────────────────────
-HIMALAYA_BIN="${HIMALAYA_BIN:-/usr/local/bin/himalaya}"
-AUDIT_LOG="${HIMALAYA_AUDIT_LOG:-/var/log/himalaya-audit.log}"
-
-# ── Destructive commands we block ──────────────────────────
-BLOCKED_CMDS=(
-  "message move"
-  "message delete"
-  "message copy"
-  "flag add"
-  "flag remove"
-  "folder create"
-  "folder delete"
-  "folder rename"
-  "template send"
-  "account configure"
-  "account delete"
-)
-
-# ── Determine the subcommand being invoked ─────────────────
-# Strip leading options (--account, --output, etc.) to find the verb
-ARGS=()
-SKIP_NEXT=false
-for arg in "$@"; do
-  if $SKIP_NEXT; then
-    SKIP_NEXT=false
-    continue
-  fi
-  if [[ "$arg" == --* ]]; then
-    case "$arg" in
-      --account|--output|--page|--page-size|--folder|--color|--format)
-        SKIP_NEXT=true ;;
-    esac
-    continue
-  fi
-  ARGS+=("$arg")
-done
-
-# Build subcommand string and check against blocklist
-CMD_STR=""
-for ((i=0; i<${#ARGS[@]}; i++)); do
-  if [ -z "$CMD_STR" ]; then
-    CMD_STR="${ARGS[$i]}"
-  else
-    CMD_STR="$CMD_STR ${ARGS[$i]}"
-  fi
-  for blocked in "${BLOCKED_CMDS[@]}"; do
-    if [[ "$CMD_STR" == "$blocked" ]]; then
-      TS=$(date '+%Y-%m-%d %H:%M:%S')
-      echo "[AUDIT] $TS BLOCKED: himalaya $*" >> "$AUDIT_LOG"
-      echo "ERROR: Command 'himalaya $CMD_STR ...' is blocked by read-only policy." >&2
-      echo "       Audit log: $AUDIT_LOG" >&2
-      exit 100
-    fi
-  done
-done
-
-# ── Allow pass-through ─────────────────────────────────────
-exec "$HIMALAYA_BIN" "$@"