From 0b606721dd4b37549150ed9b1c5d368264f0ca1c Mon Sep 17 00:00:00 2001
From: TBNilles <thomas@nilles.net>
Date: Sun, 7 Jun 2026 14:55:57 -0400
Subject: [PATCH] feat(model-manager): run container as host UID/GID

Downloads previously landed in models/ owned by root because the
container ran as root. Add `user: "${PUID:-1000}:${PGID:-1000}"` to the
model-manager service and PUID/PGID to .env.example so downloaded models
are owned by the host user. Defaults to 1000:1000.

Note: existing root-owned files under models/ and sparkyui-data/ must be
chowned once (e.g. via a one-off root container) when upgrading.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
---
 .env.example       | 5 +++++
 docker-compose.yml | 4 ++++
 2 files changed, 9 insertions(+)

diff --git a/.env.example b/.env.example
index e01c52b..57226c9 100644
--- a/.env.example
+++ b/.env.example
@@ -31,3 +31,8 @@ COMFYUIMINI_REF=main
 # Model Manager - StabilityMatrix-style model download/management UI
 # Access at http://<host>:8189
 MODEL_MANAGER_PORT=8189
+
+# Run the Model Manager container as this host user/group so downloaded
+# models are owned by you (not root). Set to your `id -u` / `id -g`.
+PUID=1000
+PGID=1000
diff --git a/docker-compose.yml b/docker-compose.yml
index 0f1c1c5..c59668c 100644
--- a/docker-compose.yml
+++ b/docker-compose.yml
@@ -138,6 +138,10 @@ services:
     image: sparkyui-model-manager:latest
     container_name: model-manager
 
+    # Run as the host user so downloaded models are owned by you, not root.
+    # Defaults to 1000:1000; override via PUID/PGID in .env if needed.
+    user: "${PUID:-1000}:${PGID:-1000}"
+
     ports:
       - "${MODEL_MANAGER_PORT:-8189}:8189"