chore: update rust edition to 2024 and rust-version to 1.93 (#64)

Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: sonesuke

Cargo.toml

@@ -1,8 +1,8 @@
 [package]
 name = "google-patent-cli"
 version = "0.1.5"
-edition = "2021"
-rust-version = "1.81"
+edition = "2024"
+rust-version = "1.93"
 
 [profile.release]
 lto = "fat"

src/cli/mod.rs

@@ -281,10 +281,12 @@ mod tests {
         ])
         .expect("Cli parsing success in test");
 
-        std::env::set_var("HOME", temp_dir.path());
-        std::env::set_var("XDG_CONFIG_HOME", temp_dir.path());
-        std::env::set_var("APPDATA", temp_dir.path());
-        std::env::set_var("USERPROFILE", temp_dir.path());
+        unsafe {
+            std::env::set_var("HOME", temp_dir.path());
+            std::env::set_var("XDG_CONFIG_HOME", temp_dir.path());
+            std::env::set_var("APPDATA", temp_dir.path());
+            std::env::set_var("USERPROFILE", temp_dir.path());
+        }
 
         let res = run_app(cli).await;
         assert!(res.is_ok());

src/core/config.rs

@@ -67,11 +67,11 @@ impl Config {
         }
 
         // Priority 2: CI environment
-        if env::var("CI").is_ok() || env::var("GITHUB_ACTIONS").is_ok() {
-            if let Some(path) = detect_chrome_path() {
-                // CI typically runs in containers/VMs, so add --no-sandbox
-                return (Some(path), vec!["--no-sandbox".to_string()]);
-            }
+        if (env::var("CI").is_ok() || env::var("GITHUB_ACTIONS").is_ok())
+            && let Some(path) = detect_chrome_path()
+        {
+            // CI typically runs in containers/VMs, so add --no-sandbox
+            return (Some(path), vec!["--no-sandbox".to_string()]);
         }
 
         // Priority 3: Auto-detection
@@ -117,13 +117,12 @@ fn is_running_in_container() -> bool {
     }
 
     // Check /proc/1/cgroup for container indicators
-    if let Ok(content) = fs::read_to_string("/proc/1/cgroup") {
-        if content.contains("docker")
+    if let Ok(content) = fs::read_to_string("/proc/1/cgroup")
+        && (content.contains("docker")
             || content.contains("kubepods")
-            || content.contains("containerd")
-        {
-            return true;
-        }
+            || content.contains("containerd"))
+    {
+        return true;
     }
 
     false

src/core/models.rs

@@ -178,27 +178,27 @@ impl SearchOptions {
         let mut url_str = url.to_string();
 
         // Manually append assignee parameter if present
-        if let Some(assignees) = &self.assignee {
-            if !assignees.is_empty() {
-                let encoded_assignees: Vec<String> = assignees
-                    .iter()
-                    .map(|a| {
-                        // Encode each assignee value, including quotes, using form_urlencoded logic
-                        let quoted = format!("\"{}\"", a);
-                        url::form_urlencoded::byte_serialize(quoted.as_bytes()).collect::<String>()
-                    })
-                    .collect();
-
-                // Determine if we need to add '?' or '&'
-                let separator = if !url_str.contains('?') {
-                    "?"
-                } else if url_str.ends_with('?') {
-                    ""
-                } else {
-                    "&"
-                };
-                url_str.push_str(&format!("{}assignee={}", separator, encoded_assignees.join(",")));
-            }
+        if let Some(assignees) = &self.assignee
+            && !assignees.is_empty()
+        {
+            let encoded_assignees: Vec<String> = assignees
+                .iter()
+                .map(|a| {
+                    // Encode each assignee value, including quotes, using form_urlencoded logic
+                    let quoted = format!("\"{}\"", a);
+                    url::form_urlencoded::byte_serialize(quoted.as_bytes()).collect::<String>()
+                })
+                .collect();
+
+            // Determine if we need to add '?' or '&'
+            let separator = if !url_str.contains('?') {
+                "?"
+            } else if url_str.ends_with('?') {
+                ""
+            } else {
+                "&"
+            };
+            url_str.push_str(&format!("{}assignee={}", separator, encoded_assignees.join(",")));
         }
 
         // Manual check for empty params (after constructing)

src/core/patent_search.rs

@@ -225,11 +225,7 @@ fn parse_single_patent_result(
                 })
             })
             .collect();
-        if parsed.is_empty() {
-            None
-        } else {
-            Some(parsed)
-        }
+        if parsed.is_empty() { None } else { Some(parsed) }
     });
 
     // Parse claims
@@ -244,11 +240,7 @@ fn parse_single_patent_result(
                 })
             })
             .collect();
-        if parsed.is_empty() {
-            None
-        } else {
-            Some(parsed)
-        }
+        if parsed.is_empty() { None } else { Some(parsed) }
     });
 
     // Parse images
@@ -262,11 +254,7 @@ fn parse_single_patent_result(
                 })
             })
             .collect();
-        if parsed.is_empty() {
-            None
-        } else {
-            Some(parsed)
-        }
+        if parsed.is_empty() { None } else { Some(parsed) }
     });
 
     let filing_date = result["filing_date"].as_str().map(String::from);
@@ -302,10 +290,10 @@ fn parse_single_patent_result(
 fn parse_search_results(results: serde_json::Value, limit: Option<usize>) -> Result<SearchResult> {
     let mut sr: SearchResult = serde_json::from_value(results)?;
 
-    if let Some(limit) = limit {
-        if sr.patents.len() > limit {
-            sr.patents.truncate(limit);
-        }
+    if let Some(limit) = limit
+        && sr.patents.len() > limit
+    {
+        sr.patents.truncate(limit);
     }
 
     Ok(sr)

src/mcp/mod.rs

@@ -3,13 +3,14 @@ use crate::core::models::SearchOptions;
 use crate::core::models::{Patent, SearchResult};
 use crate::core::patent_search::{PatentSearch, PatentSearcher};
 use rmcp::{
+    ErrorData, RoleServer, ServerHandler, ServiceExt,
     handler::server::{tool::ToolRouter, wrapper::Parameters},
     model::{
         ErrorCode, Implementation, ProtocolVersion, ServerCapabilities, ServerInfo, ToolsCapability,
     },
-    schemars::{self, schema_for, JsonSchema},
+    schemars::{self, JsonSchema, schema_for},
     service::{NotificationContext, RequestContext},
-    tool, tool_handler, tool_router, ErrorData, RoleServer, ServerHandler, ServiceExt,
+    tool, tool_handler, tool_router,
 };
 use serde::{Deserialize, Serialize};
 use serde_json::Value;