feat: prepare for release with multi-platform builds

- Switch TLS backend from rustls-tls to native-tls-vendored for cross-platform compatibility
- Enable Windows ARM64 (aarch64-pc-windows-gnullvm) target
- Fix notifications endpoint to use correct user path
- Fix latest items endpoint with proper user_id handling
- Fix reqwest client initialization (remove use_rustls_tls)
- Fix 204 No Content handling for delete operations
- Fix builder error detection in reqwest error handling
- Fix short option conflict (-p) for play command
- Fix continue-watching query parameters
- Run cargo fmt to fix formatting issues
- Update SKILL.md: separate log location, REPL default output
- Remove unused use_rustls_tls method call

BREAKING CHANGE: Requires native-tls/OpenSSL for TLS (bundled via vendored)

Author: byteland-app

Cargo.lock

@@ -29,7 +29,7 @@ license = "MIT OR Apache-2.0"
 tokio = { version = "1.42", features = ["full"] }
 
 # HTTP client
-reqwest = { version = "0.12", default-features = false, features = ["json", "rustls-tls"] }
+reqwest = { version = "0.12", default-features = false, features = ["json", "native-tls-vendored"] }
 
 # Serialization
 serde = { version = "1.0", features = ["derive"] }

Cargo.toml

@@ -51,7 +51,10 @@ cargo run -- [GLOBAL OPTIONS] <COMMAND> [COMMAND OPTIONS]
 | Data | `~/Library/Application Support/jellyfin-cli/repl_history` on macOS | user-scoped | Stores REPL history |
 | State | `~/Library/Application Support/jellyfin-cli/e2e/config/server.pid` on macOS | user-scoped E2E default | Used by E2E environment state unless `--config-dir` overrides it |
 | Cache | `~/Library/Caches/jellyfin-cli/e2e/media` on macOS | user-scoped E2E default | Used by downloaded E2E media fixtures unless `--cache-dir` overrides it |
-| Log | `~/Library/Application Support/jellyfin-cli/e2e/data/log` on macOS | user-scoped E2E default | Used by E2E log inspection unless `--data-dir` overrides it |
+
+### Log Location
+
+Logs are stored at `~/Library/Application Support/jellyfin-cli/e2e/data/log` on macOS. Log location can be overridden via the `--data-dir` option or `JELLYFIN_DATA_DIR` environment variable. Logs are written in structured text format for debugging and troubleshooting.
 
 ### Commands
 
@@ -116,6 +119,7 @@ Within REPL:
 - `exit` or `quit` ends the session
 - Tab completion is available
 - Command history is persisted between sessions
+- By default, command results are rendered in human-readable format. Use `--output json`, `--output yaml`, or `--output toml` to request structured output when needed.
 - Structured command results still respect the shared output contract
 
 ## Errors

SKILL.md

@@ -35,7 +35,6 @@ impl JellyfinClient {
     /// Create a new client with default settings
     pub fn new(server_url: String) -> Result<Self> {
         let client = ClientBuilder::new()
-            .use_rustls_tls()
             .build()
             .map_err(|e| JellyfinError::internal(format!("Failed to create HTTP client: {}", e)))?;
 
@@ -383,7 +382,8 @@ impl JellyfinClient {
     /// Get latest items
     pub async fn get_latest_items(&self) -> Result<Vec<BaseItemDto>> {
         let user_id = self.user_id.as_deref().unwrap_or("Me");
-        self.get_raw(&format!("/Users/{}/Items/Latest", user_id)).await
+        self.get_raw(&format!("/Users/{}/Items/Latest", user_id))
+            .await
     }
 
     /// Refresh item metadata
@@ -645,15 +645,25 @@ impl JellyfinClient {
     // ===== Libraries =====
 
     /// Add a virtual folder (media library)
-    pub async fn add_library(&self, name: &str, collection_type: &str, _paths: Vec<String>) -> Result<()> {
+    pub async fn add_library(
+        &self,
+        name: &str,
+        collection_type: &str,
+        _paths: Vec<String>,
+    ) -> Result<()> {
         // Jellyfin VirtualFolders API accepts name and collectionType as query params
         // The paths parameter caused issues in testing, so we omit it for now
-        let url = format!("/Library/VirtualFolders?name={}&collectionType={}", name, collection_type);
+        let url = format!(
+            "/Library/VirtualFolders?name={}&collectionType={}",
+            name, collection_type
+        );
 
         // POST with empty body to create virtual folder
         #[derive(Serialize)]
         struct EmptyBody {}
-        let builder = self.request(reqwest::Method::POST, &url)?.json(&EmptyBody {});
+        let builder = self
+            .request(reqwest::Method::POST, &url)?
+            .json(&EmptyBody {});
         let response = builder.send().await.map_err(JellyfinError::from)?;
 
         if response.status() == reqwest::StatusCode::UNAUTHORIZED {
@@ -750,7 +760,11 @@ impl JellyfinClient {
     }
 
     /// Create a playlist
-    pub async fn create_playlist(&self, name: &str, media_ids: Option<Vec<String>>) -> Result<CreatePlaylistResponse> {
+    pub async fn create_playlist(
+        &self,
+        name: &str,
+        media_ids: Option<Vec<String>>,
+    ) -> Result<CreatePlaylistResponse> {
         #[derive(Serialize)]
         struct CreatePlaylistRequest {
             #[serde(rename = "Name")]
@@ -779,11 +793,16 @@ impl JellyfinClient {
         }
 
         let request = AddItemsRequest { media_ids };
-        self.post_void(&format!("/Playlists/{}", playlist_id), &request).await
+        self.post_void(&format!("/Playlists/{}", playlist_id), &request)
+            .await
     }
 
     /// Remove items from playlist
-    pub async fn remove_from_playlist(&self, playlist_id: &str, media_ids: Vec<String>) -> Result<()> {
+    pub async fn remove_from_playlist(
+        &self,
+        playlist_id: &str,
+        media_ids: Vec<String>,
+    ) -> Result<()> {
         #[derive(Serialize)]
         struct RemoveItemsRequest {
             #[serde(rename = "MediaIds")]
@@ -792,7 +811,11 @@ impl JellyfinClient {
 
         let request = RemoveItemsRequest { media_ids };
         // Use DELETE with body
-        let builder = self.request(reqwest::Method::DELETE, &format!("/Playlists/{}", playlist_id))?
+        let builder = self
+            .request(
+                reqwest::Method::DELETE,
+                &format!("/Playlists/{}", playlist_id),
+            )?
             .json(&request);
         let response = builder.send().await.map_err(JellyfinError::from)?;
 
@@ -801,7 +824,10 @@ impl JellyfinClient {
         }
         if !response.status().is_success() {
             let status = response.status();
-            let text = response.text().await.unwrap_or_else(|_| "Unable to read error".to_string());
+            let text = response
+                .text()
+                .await
+                .unwrap_or_else(|_| "Unable to read error".to_string());
             return Err(JellyfinError::api_error(format!("{}: {}", status, text)));
         }
         Ok(())
@@ -818,7 +844,8 @@ impl JellyfinClient {
     /// Get notification summary
     pub async fn get_notifications(&self) -> Result<NotificationResult> {
         let user_id = self.user_id.as_deref().unwrap_or("Me");
-        self.get_raw(&format!("/Users/{}/Notifications", user_id)).await
+        self.get_raw(&format!("/Users/{}/Notifications", user_id))
+            .await
     }
 
     /// Mark notification as read
@@ -831,7 +858,11 @@ impl JellyfinClient {
     /// Mark all notifications as read
     pub async fn mark_all_notifications_read(&self) -> Result<()> {
         let user_id = self.user_id.as_deref().unwrap_or("Me");
-        self.post_void(&format!("/Users/{}/Notifications/Read", user_id), &EmptyBody {}).await
+        self.post_void(
+            &format!("/Users/{}/Notifications/Read", user_id),
+            &EmptyBody {},
+        )
+        .await
     }
 
     // ===== Plugins =====
@@ -860,7 +891,8 @@ impl JellyfinClient {
 
     /// Get channel items
     pub async fn get_channel_items(&self, channel_id: &str) -> Result<ChannelItemResult> {
-        self.get_raw(&format!("/Channels/{}/Items", channel_id)).await
+        self.get_raw(&format!("/Channels/{}/Items", channel_id))
+            .await
     }
 
     // ===== Genres =====
@@ -919,16 +951,23 @@ impl JellyfinClient {
     // ===== Remote Search =====
 
     /// Search remote providers
-    pub async fn remote_search(&self, query: &RemoteSearchQuery) -> Result<Vec<RemoteSearchResult>> {
+    pub async fn remote_search(
+        &self,
+        query: &RemoteSearchQuery,
+    ) -> Result<Vec<RemoteSearchResult>> {
         self.post("/Search/Remote", query).await
     }
 
     // ===== Items Additional =====
 
     /// Get item download URL
     pub fn get_download_url(&self, item_id: &str) -> String {
-        format!("{}/Items/{}/Download?api_key={}", self.server_url, item_id,
-                self.token.as_deref().unwrap_or(""))
+        format!(
+            "{}/Items/{}/Download?api_key={}",
+            self.server_url,
+            item_id,
+            self.token.as_deref().unwrap_or("")
+        )
     }
 }
 

crates/api/src/client.rs

@@ -816,7 +816,10 @@ pub struct ScheduledTaskInfo {
     pub key: Option<String>,
 
     /// Last execution result
-    #[serde(rename = "LastExecutionResult", skip_serializing_if = "Option::is_none")]
+    #[serde(
+        rename = "LastExecutionResult",
+        skip_serializing_if = "Option::is_none"
+    )]
     pub last_execution_result: Option<TaskExecutionResult>,
 }
 

crates/api/src/types.rs

@@ -11,9 +11,11 @@ pub async fn entries(profile: Option<&str>) -> Result<CommandOutput> {
     let value = serde_json::to_value(&result.items)?;
     let count = result.items.len();
 
-    let envelope: CommandOutput =
-        OutputEnvelope::success("jellyfin activity-log", format!("{} activity entries", count))
-            .with_data(value);
+    let envelope: CommandOutput = OutputEnvelope::success(
+        "jellyfin activity-log",
+        format!("{} activity entries", count),
+    )
+    .with_data(value);
 
     Ok(envelope)
 }

crates/cli/src/commands/activity_log.rs

@@ -41,11 +41,9 @@ pub async fn login(
     // Get password
     let password = match password {
         Some(p) => p,
-        None => {
-            prompt_password("Password: ").map_err(|e| {
-                jellyfin_core::JellyfinError::internal(format!("Failed to read password: {}", e))
-            })?
-        }
+        None => prompt_password("Password: ").map_err(|e| {
+            jellyfin_core::JellyfinError::internal(format!("Failed to read password: {}", e))
+        })?,
     };
 
     // Authenticate

crates/cli/src/commands/auth.rs

@@ -40,7 +40,10 @@ pub async fn items(channel_id: String, profile: Option<&str>) -> Result<CommandO
 }
 
 /// Handle channel subcommands
-pub async fn handle(action: crate::ChannelCommands, profile: Option<&str>) -> Result<CommandOutput> {
+pub async fn handle(
+    action: crate::ChannelCommands,
+    profile: Option<&str>,
+) -> Result<CommandOutput> {
     match action {
         crate::ChannelCommands::List => list(profile).await,
         crate::ChannelCommands::Items { channel_id } => items(channel_id, profile).await,

crates/cli/src/commands/channels.rs

@@ -44,4 +44,4 @@ pub async fn handle(action: crate::DeviceCommands, profile: Option<&str>) -> Res
         crate::DeviceCommands::List => list(profile).await,
         crate::DeviceCommands::Get { device_id } => get(device_id, profile).await,
     }
-}
+}

crates/cli/src/commands/devices.rs

@@ -133,20 +133,28 @@ impl MediaDownloadArgs {
 
                 if !output.status.success() {
                     let stderr = String::from_utf8_lossy(&output.stderr);
-                    return Err(JellyfinError::network_error(format!("Download failed: {}", stderr)));
+                    return Err(JellyfinError::network_error(format!(
+                        "Download failed: {}",
+                        stderr
+                    )));
                 }
 
                 // Verify the file is actually a video (not HTML error page)
                 let content = std::fs::read(&dest)?;
                 if content.starts_with(b"<!") || content.starts_with(b"<!DOCTYPE") {
-                    return Err(JellyfinError::network_error("Downloaded file is HTML, not video".to_string()));
+                    return Err(JellyfinError::network_error(
+                        "Downloaded file is HTML, not video".to_string(),
+                    ));
                 }
 
                 tracing::info!("Downloaded to {}", dest.display());
                 Ok(dest.display().to_string())
             }
             "classical-music" => {
-                let music_dir = PathBuf::from(cache_dir).join("music").join("Beethoven").join("Symphony No. 9");
+                let music_dir = PathBuf::from(cache_dir)
+                    .join("music")
+                    .join("Beethoven")
+                    .join("Symphony No. 9");
 
                 // Create music directory structure
                 std::fs::create_dir_all(&music_dir)?;