fix: add implicit work volume mounted at /work - user definition takes preference

Author: queil

src/api/volume.rs

@@ -105,7 +105,15 @@ impl<'a> VolumeApi<'a> {
         mounts: &HashMap<String, String>,
     ) -> HashMap<TargetPath, DataEntryVolumeSpec> {
         let mut result = HashMap::new();
-        for (target, source_key) in mounts {
+
+        let mut mount_entries: HashMap<String, String> = HashMap::new();
+        mount_entries.extend(mounts.clone());
+
+        if !mounts.values().any(|key| key == "work") {
+            mount_entries.insert("/work".to_string(), "work".to_string());
+        }
+
+        for (target, source_key) in mount_entries {
             let source_key = DataEntryKey(source_key.to_string());
             let source_exists = volumes.contains_key(&source_key);
             if !source_exists {
@@ -133,6 +141,12 @@ impl<'a> VolumeApi<'a> {
         let mut data_entries = vec![];
         data_entries.extend_from_slice(data.clone().into_entries().as_slice());
 
+        if !data.contains_key("work") {
+            data_entries.push(DataEntry::Dir {
+                name: "work".to_string(),
+            });
+        }
+
         let files_key = "files";
         let files_volume_spec = VolumeSpec {
             name: Self::volume_name(workspace_key, files_key),

src/model/volume.rs

@@ -86,7 +86,7 @@ impl RoozVolume {
             RoozVolume {
                 sharing: RoozVolumeSharing::Exclusive { key },
                 ..
-            } => format!("rooz_{}_{}", sanitize(&key), &role_segment),
+            } => format!("rooz-{}-{}", sanitize(&key), &role_segment),
             RoozVolume { .. } => format!("rooz_{}", &role_segment),
         }
     }