chore(release): release version 1.19.0 (minor)

Cargo.toml

@@ -4,7 +4,7 @@ resolver = "2"
 
 [workspace.package]
 edition = "2021"
-version = "1.18.0"
+version = "1.19.0"
 authors = ["Block <[email protected]>"]
 license = "Apache-2.0"
 repository = "https://github.com/block/goose"

bin/cargo

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/cargo-clippy

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/cargo-fmt

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/cargo-miri

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/clippy-driver

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rls

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rust-analyzer

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rust-gdb

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rust-gdbgui

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rust-lldb

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rustc

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rustdoc

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rustfmt

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

bin/rustup

@@ -1 +1 @@
-.rustup-1.25.2.pkg
+.rustup-1.28.2.pkg

crates/goose-cli/src/logging.rs

@@ -45,8 +45,8 @@ fn setup_logging_internal(
         result = (|| {
             let log_dir = goose::logging::prepare_log_directory("cli", true)?;
             let timestamp = chrono::Local::now().format("%Y%m%d_%H%M%S").to_string();
-            let log_filename = if name.is_some() {
-                format!("{}-{}.log", timestamp, name.unwrap())
+            let log_filename = if let Some(n) = name {
+                format!("{}-{}.log", timestamp, n)
             } else {
                 format!("{}.log", timestamp)
             };

crates/goose-mcp/src/computercontroller/platform/mod.rs

@@ -1,5 +1,8 @@
+#[cfg(target_os = "linux")]
 mod linux;
+#[cfg(target_os = "macos")]
 mod macos;
+#[cfg(target_os = "windows")]
 mod windows;
 
 #[cfg(target_os = "windows")]

crates/goose-mcp/src/developer/tests/test_diff.rs

@@ -184,9 +184,8 @@ new file mode 100644
             // mpatch may add a trailing newline
             let content = std::fs::read_to_string(&file_path).unwrap();
             assert!(content == "different\ncontent" || content == "different\ncontent\n");
-        } else {
+        } else if let Err(err) = result {
             // Or it might return an error
-            let err = result.unwrap_err();
             assert!(
                 err.message.contains("diff")
                     || err.message.contains("version")
@@ -213,8 +212,7 @@ new file mode 100644
 
         // The behavior might be different with patcher - it might create the file
         // or it might fail. Let's check what happens.
-        if result.is_err() {
-            let err = result.unwrap_err();
+        if let Err(err) = result {
             // Could be "Failed to read" or similar
             assert!(err.message.contains("Failed") || err.message.contains("exist"));
         } else {

crates/goose-mcp/src/developer/text_editor.rs

@@ -483,16 +483,16 @@ pub fn format_file_content(
     };
 
     let language = lang::get_language_identifier(path);
-    if view_range.is_some() {
+    if let Some((start, end)) = view_range {
         formatdoc! {"
             ### {path} (lines {start}-{end})
             ```{language}
             {content}
             ```
             ",
             path=path.display(),
-            start=view_range.unwrap().0,
-            end=if view_range.unwrap().1 == -1 { "end".to_string() } else { view_range.unwrap().1.to_string() },
+            start=start,
+            end=if end == -1 { "end".to_string() } else { end.to_string() },
             language=language,
             content=display_content,
         }

crates/goose-server/src/logging.rs

@@ -15,8 +15,8 @@ use goose::tracing::{langfuse_layer, otlp_layer};
 pub fn setup_logging(name: Option<&str>) -> Result<()> {
     let log_dir = goose::logging::prepare_log_directory("server", true)?;
     let timestamp = chrono::Local::now().format("%Y%m%d_%H%M%S").to_string();
-    let log_filename = if name.is_some() {
-        format!("{}-{}.log", timestamp, name.unwrap())
+    let log_filename = if let Some(n) = name {
+        format!("{}-{}.log", timestamp, n)
     } else {
         format!("{}.log", timestamp)
     };

crates/goose/src/agents/extension_manager.rs

@@ -782,9 +782,9 @@ impl ExtensionManager {
         let extension_name = params.get("extension_name").and_then(|v| v.as_str());
 
         // If extension name is provided, we can just look it up
-        if extension_name.is_some() {
+        if let Some(ext_name) = extension_name {
             let read_result = self
-                .read_resource(uri, extension_name.unwrap(), cancellation_token.clone())
+                .read_resource(uri, ext_name, cancellation_token.clone())
                 .await?;
 
             let mut result = Vec::new();
@@ -1288,7 +1288,7 @@ mod tests {
     use rmcp::model::ListToolsResult;
     use rmcp::model::ReadResourceResult;
     use rmcp::model::ServerNotification;
-    use serde_json::json;
+
     use tokio::sync::mpsc;
 
     impl ExtensionManager {

crates/goose/src/conversation/mod.rs

@@ -544,7 +544,7 @@ mod tests {
 
     #[test]
     fn test_valid_conversation() {
-        let all_messages = vec![
+        let all_messages = [
             Message::user().with_text("Can you help me search for something?"),
             Message::assistant()
                 .with_text("I'll help you search.")

crates/goose/src/permission/permission_store.rs

@@ -84,8 +84,7 @@ impl ToolPermissionStore {
         self.permissions.get(&key).and_then(|records| {
             records
                 .iter()
-                .filter(|record| record.expiry.is_none_or(|exp| exp > Utc::now().timestamp()))
-                .next_back()
+                .rfind(|record| record.expiry.is_none_or(|exp| exp > Utc::now().timestamp()))
                 .map(|record| record.allowed)
         })
     }