[fs] breaking: remove async_std crate and remove double allocation …

…in `resolver` field

Since both  and  allocate memory on the heap, we don't need both ;  works as well.

crates/fs/Cargo.toml

@@ -32,14 +32,12 @@ rust-version.workspace = true
 file-format = ["dep:infer", "dep:file-format"]
 serde_json = ["dep:serde_json"]
 serde_yaml = ["dep:serde_yaml"]
-async-std = ["dep:async-std"]
 default = ["file-format", "serde_json"]
 tracing = ["dep:tracing"]
 serde = ["dep:serde"]
 log = ["dep:log"]
 
 [dependencies]
-async-std = { version = "1.12.0", optional = true }
 async-trait = "0.1.80"
 bytes = "1.6.0"
 dirs = "5.0.1"

crates/fs/src/service.rs

@@ -30,21 +30,16 @@ use std::{
     sync::Arc,
     time::SystemTime,
 };
-
-#[cfg(not(feature = "async_std"))]
 use tokio::{fs, io::*};
 
-#[cfg(feature = "async_std")]
-use async_std::{fs, io::*};
-
 #[cfg(feature = "tracing")]
 use tracing::instrument;
 
 /// Represents an implementation of a [`StorageService`](remi::StorageService) for the
 /// local filesystem.
 #[derive(Clone)]
 pub struct StorageService {
-    resolver: Arc<Box<dyn ContentTypeResolver>>,
+    resolver: Arc<dyn ContentTypeResolver>,
     config: Config,
 }
 
@@ -57,14 +52,14 @@ impl StorageService {
     /// Creates a new [`StorageService`] instance with a provided configuration object.
     pub fn with_config(config: Config) -> StorageService {
         StorageService {
-            resolver: Arc::new(Box::new(default_resolver)),
+            resolver: Arc::new(default_resolver),
             config,
         }
     }
 
     /// Updates the given [`ContentTypeResolver`] to something else.
     pub fn with_resolver<R: ContentTypeResolver + 'static>(mut self, resolver: R) -> StorageService {
-        self.resolver = Arc::new(Box::new(resolver));
+        self.resolver = Arc::new(resolver);
         self
     }