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

Reviewed-on: #49
feat: bake Matrix bridge deps into Docker image instead of volume mount
2026-05-22 17:05:52 +00:00 · 2026-05-22 13:04:49 -04:00 · 2026-05-22 13:03:51 -04:00
2 changed files with 7 additions and 2 deletions
--- a/ai/compose.yml
+++ b/ai/compose.yml
@@ -61,8 +61,6 @@ services:
      - /mnt/HoardingCow_docker_data/Syncthing/telos-ro:/opt/data/telos-ro:ro
      # Syncthing-shared inbox — write tasks here, they sync to user's laptop
      - /mnt/HoardingCow_docker_data/Syncthing/telos-rw:/opt/data/telos-rw:rw
-      # Persist Python venv across container recreation (Matrix bridge deps, etc.)
-      - /mnt/HoardingCow_docker_data/Hermes/venv:/opt/hermes/.venv
    devices:
      - /dev/kfd:/dev/kfd
      - /dev/dri:/dev/dri
--- a/ai/hermes/Dockerfile
+++ b/ai/hermes/Dockerfile
@@ -34,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 && \
@@ -42,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 && \
Author	SHA1	Message	Date
Thierry Pouplier	46f3b23a3f	Merge pull request 'fix: remove venv volume mount conflicting with entrypoint.sh' (#49 ) from fix/remove-venv-mount into master Some checks failed Build Hermes agent / build (push) Has been cancelled Details Build ollama (gfx906) / build (push) Has been cancelled Details Reviewed-on: #49	2026-05-22 17:05:52 +00:00
Hermes	567850bd13	feat: bake Matrix bridge deps into Docker image instead of volume mount Some checks failed Build Hermes agent / build (pull_request) Has been cancelled Details Build ollama (gfx906) / build (pull_request) Has been cancelled Details - Add libolm-dev system dep (required by mautrix[encryption]) - Add mautrix[encryption] + openai pip packages to build - These were previously installed inline at container startup and persisted via the fragile venv volume mount (now removed)	2026-05-22 13:04:49 -04:00
Hermes	2d59bb44c3	fix: remove venv volume mount conflicting with upstream entrypoint.sh Some checks failed Build Hermes agent / build (pull_request) Has been cancelled Details Build ollama (gfx906) / build (pull_request) Has been cancelled Details The volume mount /mnt/HoardingCow_docker_data/Hermes/venv overrides the container's built-in .venv with whatever is on the host. On a fresh start or after a clean build, an empty/missing venv directory causes entrypoint.sh line 62 (source .venv/bin/activate) to fail with set -e. The Docker image already builds a complete venv — persisting it on the host is unnecessary and fragile.	2026-05-22 13:03:51 -04:00