From ec9d808e019536c03d97a1888562493e9579c40a Mon Sep 17 00:00:00 2001
From: openhands <openhands@all-hands.dev>
Date: Sat, 15 Aug 2026 17:09:38 +0000
Subject: [PATCH] Standardize Docker compose to use COPY instead of volume
 mounts

- Add COPY directive to Dockerfile to copy source into image
- Change compose.yml build context to parent directory with explicit dockerfile path
- Remove source volume mounts from compose.yml
- Keep X11/privileged volumes where required for GUI/USB access
- Fix incorrect directory names in Dockerfiles (matching repo name)
- Fix incorrect service names in compose.yml (matching repo name)

This PR was created by an AI agent (OpenHands) on behalf of mmmarinho.
---
 docker/Dockerfile  | 3 ++-
 docker/compose.yml | 8 ++++----
 2 files changed, 6 insertions(+), 5 deletions(-)

diff --git a/docker/Dockerfile b/docker/Dockerfile
index f025f1d..c51e3de 100644
--- a/docker/Dockerfile
+++ b/docker/Dockerfile
@@ -5,4 +5,5 @@ ENV BASH_ENV="/etc/bash_env"
 
 RUN sudo apt-get update && sudo apt-get upgrade -y
 RUN sudo sudo dpkg -r --force-depends ros-jazzy-sas-conversions
-RUN mkdir -p /root/sas_common_devel/src/
\ No newline at end of file
+RUN mkdir -p /root/sas_conversions_devel/src/
+COPY . /root/sas_conversions_devel/src/sas_conversions
\ No newline at end of file
diff --git a/docker/compose.yml b/docker/compose.yml
index 937f3c1..a5b1fc7 100644
--- a/docker/compose.yml
+++ b/docker/compose.yml
@@ -1,8 +1,8 @@
 services:
-  sas_common:
-    build: .
-    volumes:
-      - ../../sas_conversions:/root/sas_conversions_devel/src/sas_conversions
+  sas_conversions:
+    build:
+      context: ..
+      dockerfile: docker/Dockerfile
     command: /bin/bash -c "
       cd /root/sas_conversions_devel/src/
       && ls .