fix: explicit chown

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "rooz"
-version = "0.154.3"
+version = "0.155.0"
 edition = "2024"
 
 [dependencies]

src/api/exec.rs

@@ -324,6 +324,13 @@ echo '[install] {}: {}'
         Ok(())
     }
 
+    pub async fn default_uid(&self, container_id: &str) -> Result<i32, AnyError> {
+        let output = self
+            .output("default-uid", container_id, None, Some(vec!["id", "-u"]))
+            .await?;
+        Ok(output.trim().parse::<i32>()?)
+    }
+
     pub async fn ensure_user(&self, container_id: &str) -> Result<(), AnyError> {
         let ensure_user_cmd = inject(
             format!(

src/api/workspace/enter.rs

@@ -45,6 +45,7 @@ impl<'a> WorkspaceApi<'a> {
         shell: Option<Vec<&str>>,
         container_name: Option<&str>,
         root: bool,
+        chown: bool,
     ) -> Result<String, AnyError> {
         let container_name = container_name.unwrap_or(constants::DEFAULT_CONTAINER_NAME);
         let enter_labels = Labels::from(&[
@@ -101,25 +102,28 @@ impl<'a> WorkspaceApi<'a> {
                     self.api.exec.ensure_user(container_id).await?;
                 }
 
-                let real_mounts = if is_work_container {
-                    &config.real_mounts
-                } else {
-                    &config.sidecars[container_name].real_mounts
-                };
-
-                let chown_uid = if is_work_container {
-                    &config.uid
-                } else {
-                    &config.sidecars[container_name]
-                        .uid
-                        .unwrap_or_else(|| panic!("TODO: read default uid from the image"))
-                };
-
-                for (target, _) in real_mounts {
-                    self.api
-                        .exec
-                        .chown(&container_id, chown_uid, target.as_str())
-                        .await?;
+                if chown {
+                    let real_mounts = if is_work_container {
+                        &config.real_mounts
+                    } else {
+                        &config.sidecars[container_name].real_mounts
+                    };
+
+                    let chown_uid = if is_work_container {
+                        config.uid
+                    } else {
+                        match config.sidecars[container_name].uid {
+                            Some(uid) => uid,
+                            None => self.api.exec.default_uid(container_id).await?,
+                        }
+                    };
+
+                    for (target, _) in real_mounts {
+                        self.api
+                            .exec
+                            .chown(&container_id, &chown_uid, target.as_str())
+                            .await?;
+                    }
                 }
             }
 

src/cli.rs

@@ -199,6 +199,11 @@ pub struct EnterParams {
     pub work_dir: Option<String>,
     #[arg(short, long)]
     pub container: Option<String>,
+    #[arg(
+        long,
+        help = "Chowns the container's volume mounts to the user the container runs as"
+    )]
+    pub chown: bool,
 }
 
 #[derive(Parser, Debug)]

src/cmd/new.rs

@@ -458,6 +458,7 @@ impl<'a> WorkspaceApi<'a> {
                 Some(cfg.shell.iter().map(|v| v.as_str()).collect::<Vec<_>>()),
                 None,
                 root,
+                false,
             )
             .await?;
 

src/main.rs

@@ -166,6 +166,7 @@ async fn main() -> Result<(), AnyError> {
                     root,
                     work_dir,
                     container,
+                    chown,
                 }),
             ..
         } => {
@@ -176,6 +177,7 @@ async fn main() -> Result<(), AnyError> {
                     shell.as_deref().map(|v| vec![v.as_ref()]),
                     container.as_deref(),
                     root,
+                    chown,
                 )
                 .await?;
         }