From 42d27c85ef1e1f1447eee447f3eff3e930f1ad72 Mon Sep 17 00:00:00 2001 From: Dmitrii Date: Wed, 22 Apr 2026 09:32:43 -0400 Subject: [PATCH] fix: use SUDO_USER for docker group to get the invoking user not root Co-Authored-By: Claude Sonnet 4.6 --- scripts/check_deps.sh | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/scripts/check_deps.sh b/scripts/check_deps.sh index 3fbffd3..8fabf23 100755 --- a/scripts/check_deps.sh +++ b/scripts/check_deps.sh @@ -46,11 +46,13 @@ if ! docker compose version >/dev/null 2>&1 && ! command -v docker-compose >/dev fi echo " [ok] docker-compose" -# Ensure current user is in docker group -if ! id | grep -q '(docker)'; then - echo " Adding $(id -un) to docker group..." - usermod -aG docker "$(id -un)" +# Ensure the invoking user is in the docker group (use SUDO_USER if available) +TARGET_USER="${SUDO_USER:-$(id -un)}" +if ! groups "$TARGET_USER" 2>/dev/null | grep -q '\bdocker\b'; then + echo " Adding $TARGET_USER to docker group..." + usermod -aG docker "$TARGET_USER" echo " NOTE: Log out and back in (or run 'newgrp docker') for docker group to take effect." fi +echo " [ok] docker group ($TARGET_USER)" echo "=== All dependencies OK ==="