@opencode-cloud/core 13.0.0 → 13.0.1

package/Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "opencode-cloud-core"
-version = "13.0.0"
+version = "13.0.1"
 edition = "2024"
 rust-version = "1.89"
 license = "MIT"
@@ -23,11 +23,11 @@ napi = ["dep:napi", "dep:napi-derive"]
 
 [dependencies]
 clap = { version = "4.5", features = ["derive"] }
-tokio = { version = "1.43", features = ["rt-multi-thread", "macros"] }
+tokio = { version = "1.49", features = ["rt-multi-thread", "macros"] }
 serde = { version = "1.0", features = ["derive"] }
 serde_json = "1.0"
 jsonc-parser = { version = "0.29", features = ["serde"] }
-directories = "5"
+directories = "6"
 thiserror = "2"
 anyhow = "1"
 tracing = "0.1"
@@ -35,18 +35,18 @@ console = "0.16"
 chrono = { version = "0.4", default-features = false, features = ["std", "clock"] }
 
 # NAPI dependencies (optional - only for Node bindings)
-napi = { version = "2", features = ["tokio_rt", "napi9"], optional = true }
-napi-derive = { version = "2", optional = true }
+napi = { version = "3", features = ["tokio_rt", "napi9"], optional = true }
+napi-derive = { version = "3", optional = true }
 
 # Docker integration
 bollard = { version = "0.20.1", features = ["chrono", "buildkit"] }
 futures-util = "0.3"
 tar = "0.4"
-flate2 = "1.0"
+flate2 = "1.1"
 tokio-retry = "0.3"
-indicatif = { version = "0.17", features = ["tokio", "futures"] }
+indicatif = { version = "0.18", features = ["tokio", "futures"] }
 http-body-util = "0.1"
-bytes = "1.9"
+bytes = "1.11"
 reqwest = { version = "0.13", default-features = false, features = ["rustls", "webpki-roots", "json"] }
 
 # Platform service management (macOS)

package/package.json

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

package/src/platform/mod.rs

@@ -85,9 +85,12 @@ pub trait ServiceManager: Send + Sync {
 
 /// Get the appropriate service manager for the current platform
 ///
+/// # Arguments
+/// * `boot_mode` - "user" for user-level service (default), "system" for system-level
+///
 /// Returns an error if the platform is not supported or if the
 /// service manager implementation is not yet available.
-pub fn get_service_manager() -> Result<Box<dyn ServiceManager>> {
+pub fn get_service_manager(boot_mode: &str) -> Result<Box<dyn ServiceManager>> {
     #[cfg(target_os = "linux")]
     {
         if !systemd::systemd_available() {
@@ -96,11 +99,11 @@ pub fn get_service_manager() -> Result<Box<dyn ServiceManager>> {
                  Service registration requires systemd as the init system."
             ));
         }
-        Ok(Box::new(systemd::SystemdManager::new("user")))
+        Ok(Box::new(systemd::SystemdManager::new(boot_mode)))
     }
     #[cfg(target_os = "macos")]
     {
-        Ok(Box::new(launchd::LaunchdManager::new("user")))
+        Ok(Box::new(launchd::LaunchdManager::new(boot_mode)))
     }
     #[cfg(not(any(target_os = "linux", target_os = "macos")))]
     {
@@ -164,7 +167,7 @@ mod tests {
 
     #[test]
     fn test_get_service_manager_behavior() {
-        let result = get_service_manager();
+        let result = get_service_manager("user");
 
         // On Linux with systemd: returns Ok(SystemdManager)
         // On Linux without systemd: returns Err (systemd not available)
@@ -188,4 +191,16 @@ mod tests {
             assert!(result.is_err());
         }
     }
+
+    #[test]
+    fn test_get_service_manager_respects_boot_mode() {
+        // Test that boot_mode parameter is passed through
+        let user_result = get_service_manager("user");
+        let system_result = get_service_manager("system");
+
+        // Both should either succeed or fail based on platform support,
+        // but they should not panic
+        let _ = user_result;
+        let _ = system_result;
+    }
 }

package/src/platform/systemd.rs

@@ -130,11 +130,37 @@ pub fn systemd_available() -> bool {
     Path::new("/run/systemd/system").exists()
 }
 
+/// Check if systemd user session is available for the current user
+///
+/// Returns true if XDG_RUNTIME_DIR is set and the user's systemd directory exists.
+/// This is needed for `systemctl --user` commands to work.
+pub fn systemd_user_session_available() -> bool {
+    if let Ok(runtime_dir) = std::env::var("XDG_RUNTIME_DIR") {
+        // Check if the user's systemd directory exists
+        Path::new(&runtime_dir).join("systemd").exists()
+    } else {
+        false
+    }
+}
+
 impl ServiceManager for SystemdManager {
     fn install(&self, config: &ServiceConfig) -> Result<InstallResult> {
-        // Check permissions for system-level installation
-        if !self.user_mode {
-            // Check if we can write to /etc/systemd/system/
+        // Check permissions and session availability based on mode
+        if self.user_mode {
+            // User-level installation requires an active systemd user session
+            if !systemd_user_session_available() {
+                return Err(anyhow!(
+                    "User-level systemd session not available.\n\
+                     This typically happens during cloud-init or when running as a \
+                     different user without an active login session.\n\n\
+                     Solutions:\n\
+                     1. Use system-level installation: occ config set boot_mode system\n\
+                     2. Run the command from an interactive login session\n\
+                     3. Ensure XDG_RUNTIME_DIR is set and the user has an active systemd session"
+                ));
+            }
+        } else {
+            // System-level installation requires root privileges
             let test_path = self.service_dir().join(".opencode-cloud-test");
             if fs::write(&test_path, "").is_err() {
                 return Err(anyhow!(