@opencode-cloud/core 10.3.0 → 11.0.0

package/Cargo.toml

@@ -1,8 +1,8 @@
 [package]
 name = "opencode-cloud-core"
-version = "10.3.0"
+version = "11.0.0"
 edition = "2024"
-rust-version = "1.88"
+rust-version = "1.89"
 license = "MIT"
 repository = "https://github.com/pRizz/opencode-cloud"
 homepage = "https://github.com/pRizz/opencode-cloud"

package/package.json

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

package/src/config/validation.rs

@@ -96,26 +96,26 @@ pub fn validate_config(config: &Config) -> Result<Vec<ValidationWarning>, Valida
     }
 
     // Legacy auth fields present
-    if let Some(ref username) = config.auth_username {
-        if !username.is_empty() {
-            warnings.push(ValidationWarning {
-                field: "auth_username".to_string(),
-                message: "Legacy auth fields present; consider using 'occ user add' instead"
-                    .to_string(),
-                fix_command: "occ config set auth_username ''".to_string(),
-            });
-        }
+    if let Some(ref username) = config.auth_username
+        && !username.is_empty()
+    {
+        warnings.push(ValidationWarning {
+            field: "auth_username".to_string(),
+            message: "Legacy auth fields present; consider using 'occ user add' instead"
+                .to_string(),
+            fix_command: "occ config set auth_username ''".to_string(),
+        });
     }
 
-    if let Some(ref password) = config.auth_password {
-        if !password.is_empty() {
-            warnings.push(ValidationWarning {
-                field: "auth_password".to_string(),
-                message: "Legacy auth fields present; consider using 'occ user add' instead"
-                    .to_string(),
-                fix_command: "occ config set auth_password ''".to_string(),
-            });
-        }
+    if let Some(ref password) = config.auth_password
+        && !password.is_empty()
+    {
+        warnings.push(ValidationWarning {
+            field: "auth_password".to_string(),
+            message: "Legacy auth fields present; consider using 'occ user add' instead"
+                .to_string(),
+            fix_command: "occ config set auth_password ''".to_string(),
+        });
     }
 
     Ok(warnings)

package/src/docker/Dockerfile

@@ -36,6 +36,10 @@ FROM ubuntu:24.04 AS base
 
 # OCI Labels for image metadata
 LABEL org.opencontainers.image.title="opencode-cloud"
+# NOTE: This exact label format is parsed by scripts/extract-oci-description.py
+# (called from .github/workflows/docker-publish.yml) to populate multi-arch
+# manifest annotations for GHCR. If you change this line (format, quoting, or
+# key), update the extraction logic too.
 LABEL org.opencontainers.image.description="AI-assisted development environment with opencode"
 LABEL org.opencontainers.image.url="https://github.com/pRizz/opencode-cloud"
 LABEL org.opencontainers.image.source="https://github.com/pRizz/opencode-cloud"

package/src/docker/progress.rs

@@ -210,17 +210,17 @@ impl ProgressReporter {
 
         // Check if we should throttle this update
         if !is_step_message {
-            if let Some(last) = self.last_update_by_id.get(id) {
-                if now.duration_since(*last) < SPINNER_UPDATE_THROTTLE {
-                    return; // Throttle: too soon since last update
-                }
+            if let Some(last) = self.last_update_by_id.get(id)
+                && now.duration_since(*last) < SPINNER_UPDATE_THROTTLE
+            {
+                return; // Throttle: too soon since last update
             }
 
             // Skip if message is identical to last one
-            if let Some(last_msg) = self.last_message_by_id.get(id) {
-                if last_msg == message {
-                    return;
-                }
+            if let Some(last_msg) = self.last_message_by_id.get(id)
+                && last_msg == message
+            {
+                return;
             }
         }
 

package/src/docker/state.rs

@@ -71,10 +71,10 @@ pub fn load_state() -> Option<ImageState> {
 
 /// Clear image state (e.g., after image removal)
 pub fn clear_state() -> anyhow::Result<()> {
-    if let Some(path) = get_state_path() {
-        if path.exists() {
-            std::fs::remove_file(&path)?;
-        }
+    if let Some(path) = get_state_path()
+        && path.exists()
+    {
+        std::fs::remove_file(&path)?;
     }
     Ok(())
 }

package/src/host/schema.rs

@@ -139,10 +139,10 @@ impl HostConfig {
         let mut parts = vec!["ssh".to_string()];
 
         // Port (if non-default)
-        if let Some(port) = self.port {
-            if port != 22 {
-                parts.push(format!("-p {port}"));
-            }
+        if let Some(port) = self.port
+            && port != 22
+        {
+            parts.push(format!("-p {port}"));
         }
 
         // Identity file

package/src/host/ssh_config.rs

@@ -137,21 +137,21 @@ pub fn write_ssh_config_entry(
     })?;
 
     // Ensure .ssh directory exists with proper permissions
-    if let Some(ssh_dir) = config_path.parent() {
-        if !ssh_dir.exists() {
-            fs::create_dir_all(ssh_dir).map_err(|e| {
-                HostError::SshConfigWrite(format!("Failed to create .ssh directory: {e}"))
-            })?;
+    if let Some(ssh_dir) = config_path.parent()
+        && !ssh_dir.exists()
+    {
+        fs::create_dir_all(ssh_dir).map_err(|e| {
+            HostError::SshConfigWrite(format!("Failed to create .ssh directory: {e}"))
+        })?;
 
-            // Set directory permissions to 700 on Unix
-            #[cfg(unix)]
-            {
-                use std::os::unix::fs::PermissionsExt;
-                let perms = fs::Permissions::from_mode(0o700);
-                fs::set_permissions(ssh_dir, perms).map_err(|e| {
-                    HostError::SshConfigWrite(format!("Failed to set .ssh permissions: {e}"))
-                })?;
-            }
+        // Set directory permissions to 700 on Unix
+        #[cfg(unix)]
+        {
+            use std::os::unix::fs::PermissionsExt;
+            let perms = fs::Permissions::from_mode(0o700);
+            fs::set_permissions(ssh_dir, perms).map_err(|e| {
+                HostError::SshConfigWrite(format!("Failed to set .ssh permissions: {e}"))
+            })?;
         }
     }
 
@@ -164,10 +164,10 @@ pub fn write_ssh_config_entry(
     if let Some(u) = user {
         entry.push_str(&format!("    User {u}\n"));
     }
-    if let Some(p) = port {
-        if p != 22 {
-            entry.push_str(&format!("    Port {p}\n"));
-        }
+    if let Some(p) = port
+        && p != 22
+    {
+        entry.push_str(&format!("    Port {p}\n"));
     }
     if let Some(key) = identity_file {
         entry.push_str(&format!("    IdentityFile {key}\n"));

package/src/host/storage.rs

@@ -54,11 +54,11 @@ pub fn save_hosts(hosts: &HostsFile) -> Result<(), HostError> {
         .ok_or_else(|| HostError::SaveFailed("Could not determine hosts file path".to_string()))?;
 
     // Ensure config directory exists
-    if let Some(parent) = hosts_path.parent() {
-        if !parent.exists() {
-            fs::create_dir_all(parent)
-                .map_err(|e| HostError::SaveFailed(format!("Failed to create directory: {e}")))?;
-        }
+    if let Some(parent) = hosts_path.parent()
+        && !parent.exists()
+    {
+        fs::create_dir_all(parent)
+            .map_err(|e| HostError::SaveFailed(format!("Failed to create directory: {e}")))?;
     }
 
     // Create backup if file exists