chore: upgrade to Rust edition 2024

Edition 2024 makes set_var/remove_var unsafe. Env-var tests are safe
because nextest runs each test in its own process. Also applies new
rustfmt import ordering and assert macro formatting rules.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: matthiasdebernardini

Cargo.toml

@@ -1,7 +1,7 @@
 [package]
 name = "wokhei"
 version = "0.3.0"
-edition = "2021"
+edition = "2024"
 rust-version = "1.93.1"
 authors = ["matthiasdebernardini"]
 license = "MIT"

src/dtag.rs

@@ -1,4 +1,4 @@
-use nostr_sdk::hashes::{sha256, Hash};
+use nostr_sdk::hashes::{Hash, sha256};
 
 /// Compute a deterministic 8-hex-char suffix from a preimage string.
 fn suffix(preimage: &str) -> String {

src/error.rs

@@ -229,10 +229,12 @@ mod tests {
     fn non_relay_errors_are_not_retryable() {
         assert!(!AppError::KeysNotFound { path: "/x".into() }.retryable());
         assert!(!AppError::RelayRejected { reason: "x".into() }.retryable());
-        assert!(!AppError::HeaderNotFound {
-            event_id: "x".into()
-        }
-        .retryable());
+        assert!(
+            !AppError::HeaderNotFound {
+                event_id: "x".into()
+            }
+            .retryable()
+        );
         assert!(!AppError::HeaderMissingDTag.retryable());
         assert!(!AppError::InvalidEventId { id: "x".into() }.retryable());
         assert!(!AppError::NoResults.retryable());

src/main.rs

@@ -554,35 +554,38 @@ mod tests {
         j["result"]["v"].as_str().unwrap().to_string()
     }
 
+    // SAFETY: These env-var tests run in separate processes via nextest,
+    // so there is no concurrent thread that could observe a partial write.
+
     #[test]
     fn resolve_relay_default_fallback() {
-        std::env::remove_var("WOKHEI_RELAY");
+        unsafe { std::env::remove_var("WOKHEI_RELAY") };
         let exec = relay_cli().run_argv(["test", "c"]);
         assert!(exec.envelope().ok());
         assert_eq!(relay_result(&exec), "ws://localhost:7777");
     }
 
     #[test]
     fn resolve_relay_flag_override() {
-        std::env::remove_var("WOKHEI_RELAY");
+        unsafe { std::env::remove_var("WOKHEI_RELAY") };
         let exec = relay_cli().run_argv(["test", "c", "--relay=ws://custom:1234"]);
         assert!(exec.envelope().ok());
         assert_eq!(relay_result(&exec), "ws://custom:1234");
     }
 
     #[test]
     fn resolve_relay_env_var() {
-        std::env::set_var("WOKHEI_RELAY", "ws://envrelay:5555");
+        unsafe { std::env::set_var("WOKHEI_RELAY", "ws://envrelay:5555") };
         let exec = relay_cli().run_argv(["test", "c"]);
         assert_eq!(relay_result(&exec), "ws://envrelay:5555");
-        std::env::remove_var("WOKHEI_RELAY");
+        unsafe { std::env::remove_var("WOKHEI_RELAY") };
     }
 
     #[test]
     fn resolve_relay_flag_beats_env() {
-        std::env::set_var("WOKHEI_RELAY", "ws://envrelay:5555");
+        unsafe { std::env::set_var("WOKHEI_RELAY", "ws://envrelay:5555") };
         let exec = relay_cli().run_argv(["test", "c", "--relay=ws://flagrelay:9999"]);
         assert_eq!(relay_result(&exec), "ws://flagrelay:9999");
-        std::env::remove_var("WOKHEI_RELAY");
+        unsafe { std::env::remove_var("WOKHEI_RELAY") };
     }
 }