@opencode-cloud/core 4.2.0 → 4.2.1

package/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "opencode-cloud-core"
-version = "4.2.0"
+version = "4.2.1"
 edition = "2024"
 rust-version = "1.88"
 license = "MIT"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@opencode-cloud/core",
-  "version": "4.2.0",
+  "version": "4.2.1",
   "description": "Core NAPI bindings for opencode-cloud (internal package)",
   "main": "index.js",
   "types": "index.d.ts",

package/src/config/mod.rs

@@ -15,7 +15,7 @@ use anyhow::{Context, Result};
 use jsonc_parser::parse_to_serde_value;
 
 pub use paths::{get_config_dir, get_config_path, get_data_dir, get_hosts_path, get_pid_path};
-pub use schema::{Config, validate_bind_address};
+pub use schema::{Config, default_mounts, validate_bind_address};
 pub use validation::{
     ValidationError, ValidationWarning, display_validation_error, display_validation_warning,
     validate_config,
@@ -75,6 +75,7 @@ pub fn load_config() -> Result<Config> {
             config_path.display()
         );
         let config = Config::default();
+        ensure_default_mount_dirs(&config)?;
         save_config(&config)?;
         return Ok(config);
     }
@@ -105,6 +106,7 @@ pub fn load_config() -> Result<Config> {
         )
     })?;
 
+    ensure_default_mount_dirs(&config)?;
     Ok(config)
 }
 
@@ -141,6 +143,36 @@ pub fn save_config(config: &Config) -> Result<()> {
     Ok(())
 }
 
+fn ensure_default_mount_dirs(config: &Config) -> Result<()> {
+    let defaults = default_mounts();
+    if defaults.is_empty() {
+        return Ok(());
+    }
+
+    for mount_str in &config.mounts {
+        if !defaults.contains(mount_str) {
+            continue;
+        }
+        let parsed = crate::docker::mount::ParsedMount::parse(mount_str)
+            .with_context(|| format!("Invalid default mount configured: {mount_str}"))?;
+        let path = parsed.host_path.as_path();
+        if path.exists() {
+            if !path.is_dir() {
+                return Err(anyhow::anyhow!(
+                    "Default mount path is not a directory: {}",
+                    path.display()
+                ));
+            }
+            continue;
+        }
+        fs::create_dir_all(path)
+            .with_context(|| format!("Failed to create mount directory: {}", path.display()))?;
+        tracing::info!("Created mount directory: {}", path.display());
+    }
+
+    Ok(())
+}
+
 #[cfg(test)]
 mod tests {
     use super::*;

package/src/config/schema.rs

@@ -2,6 +2,8 @@
 //!
 //! Defines the structure and defaults for the config.json file.
 
+use super::paths::{get_config_dir, get_data_dir};
+use crate::docker::volume::{MOUNT_CONFIG, MOUNT_PROJECTS, MOUNT_SESSION};
 use serde::{Deserialize, Serialize};
 use std::net::{IpAddr, Ipv4Addr};
 /// Main configuration structure for opencode-cloud
@@ -117,7 +119,7 @@ pub struct Config {
 
     /// Bind mounts to apply when starting the container
     /// Format: ["/host/path:/container/path", "/host:/mnt:ro"]
-    #[serde(default)]
+    #[serde(default = "default_mounts")]
     pub mounts: Vec<String>,
 }
 
@@ -173,6 +175,24 @@ fn default_update_check() -> String {
     "always".to_string()
 }
 
+pub fn default_mounts() -> Vec<String> {
+    let maybe_data_dir = get_data_dir();
+    let maybe_config_dir = get_config_dir();
+    let (Some(data_dir), Some(config_dir)) = (maybe_data_dir, maybe_config_dir) else {
+        return Vec::new();
+    };
+
+    let session_dir = data_dir.join("data");
+    let workspace_dir = data_dir.join("workspace");
+    let config_dir = config_dir.join("container");
+
+    vec![
+        format!("{}:{MOUNT_SESSION}", session_dir.display()),
+        format!("{}:{MOUNT_PROJECTS}", workspace_dir.display()),
+        format!("{}:{MOUNT_CONFIG}", config_dir.display()),
+    ]
+}
+
 /// Validate and parse a bind address string
 ///
 /// Accepts:
@@ -225,7 +245,7 @@ impl Default for Config {
             cockpit_enabled: default_cockpit_enabled(),
             image_source: default_image_source(),
             update_check: default_update_check(),
-            mounts: Vec::new(),
+            mounts: default_mounts(),
         }
     }
 }
@@ -309,7 +329,7 @@ mod tests {
         assert_eq!(config.rate_limit_attempts, 5);
         assert_eq!(config.rate_limit_window_seconds, 60);
         assert!(config.users.is_empty());
-        assert!(config.mounts.is_empty());
+        assert_eq!(config.mounts, default_mounts());
     }
 
     #[test]
@@ -693,7 +713,7 @@ mod tests {
     #[test]
     fn test_default_config_mounts_field() {
         let config = Config::default();
-        assert!(config.mounts.is_empty());
+        assert_eq!(config.mounts, default_mounts());
     }
 
     #[test]
@@ -717,9 +737,9 @@ mod tests {
 
     #[test]
     fn test_mounts_field_default_on_missing() {
-        // Old configs without mounts field should get empty vec
+        // Old configs without mounts field should get default mounts
         let json = r#"{"version": 1}"#;
         let config: Config = serde_json::from_str(json).unwrap();
-        assert!(config.mounts.is_empty());
+        assert_eq!(config.mounts, default_mounts());
     }
 }