Merge pull request #16 from cpfarhood/fix/happy-daemon-no-sudo

fix: remove sudo from happy daemon start
This commit was merged in pull request #16.
This commit is contained in:
2026-02-20 10:07:24 -05:00
committed by GitHub
+4 -5
View File
@@ -59,13 +59,12 @@ chown -R "$RUN_UID:$RUN_GID" "$WORKSPACE_DIR"
mkdir -p "$HOME"
chown "$RUN_UID:$RUN_GID" "$HOME"
# Start Happy Coder daemon as the app user so it can access user credentials
# (running as root means HOME=/root, Claude Code and Happy config would be missing)
# Start Happy Coder daemon. startapp.sh already runs as the app user (UID 1000),
# so no sudo needed — Happy/Claude Code will find credentials in the correct home dir.
echo "Starting Happy Coder..."
RUN_USER=$(id -nu "$RUN_UID" 2>/dev/null || echo "user")
sudo -u "$RUN_USER" -E sh -c "cd '$WORKSPACE_DIR' && happy daemon start" \
|| echo "Happy Coder daemon failed to start, continuing anyway..."
cd "$WORKSPACE_DIR"
happy daemon start || echo "Happy Coder daemon failed to start, continuing anyway..."
echo "Happy Coder daemon started"