Make loader files async

Author: rscarson

src/inner_runtime.rs

@@ -36,7 +36,7 @@ impl Default for InnerRuntimeOptions {
             extensions: Default::default(),
             default_entrypoint: Default::default(),
             timeout: Duration::MAX,
-            module_cache: Some(Box::new(DefaultModuleCacheProvider::default())),
+            module_cache: Some(Box::<DefaultModuleCacheProvider>::default()),
         }
     }
 }

src/lib.rs

@@ -1,3 +1,4 @@
+#![warn(clippy::all, clippy::pedantic, clippy::nursery, clippy::cargo)]
 //! This crate is meant to provide a quick and simple way to integrate a runtime javacript or typescript component from within rust.
 //!
 //! - **By default, the code being run is entirely sandboxed from the host, having no filesystem or network access.**

src/module_loader.rs

@@ -46,7 +46,7 @@ impl ModuleCacheProvider for DefaultModuleCacheProvider {
     fn get(&self, specifier: &ModuleSpecifier) -> Option<ModuleSource> {
         let cache = self.0.borrow();
         let source = cache.get(specifier)?;
-        Some(Self::clone_source(&self, specifier, source))
+        Some(Self::clone_source(self, specifier, source))
     }
 }
 
@@ -62,7 +62,7 @@ impl ModuleLoader for RustyLoader {
         referrer: &str,
         _kind: deno_core::ResolutionKind,
     ) -> Result<ModuleSpecifier, anyhow::Error> {
-        let url = deno_core::resolve_import(specifier, &referrer)?;
+        let url = deno_core::resolve_import(specifier, referrer)?;
         if referrer == "." {
             self.whitelist_add(url.as_str());
         }
@@ -165,7 +165,7 @@ impl RustyLoader {
     ) -> Result<ModuleSource, deno_core::error::AnyError> {
         let cache_provider = cache_provider.as_ref().as_ref().map(|p| p.as_ref());
         match cache_provider.map(|p| p.get(&module_specifier)) {
-            Some(Some(source)) => return Ok(source),
+            Some(Some(source)) => Ok(source),
             _ => {
                 let module_type = if module_specifier.path().ends_with(".json") {
                     ModuleType::Json
@@ -193,7 +193,7 @@ impl RustyLoader {
     ) -> Result<ModuleSource, deno_core::error::AnyError> {
         let cache_provider = cache_provider.as_ref().as_ref().map(|p| p.as_ref());
         match cache_provider.map(|p| p.get(&module_specifier)) {
-            Some(Some(source)) => return Ok(source),
+            Some(Some(source)) => Ok(source),
             _ => {
                 let module_type = if module_specifier.path().ends_with(".json") {
                     ModuleType::Json
@@ -204,7 +204,7 @@ impl RustyLoader {
                 let path = module_specifier.to_file_path().map_err(|_| {
                     anyhow!("Provided module specifier \"{module_specifier}\" is not a file URL.")
                 })?;
-                let code = std::fs::read_to_string(path)?;
+                let code = tokio::fs::read_to_string(path).await?;
                 let code = transpiler::transpile(&module_specifier, &code)?;
 
                 Ok(ModuleSource::new(

src/traits.rs

@@ -10,7 +10,7 @@ pub trait ToModuleSpecifier {
 
 impl ToModuleSpecifier for str {
     fn to_module_specifier(&self) -> Result<ModuleSpecifier, Error> {
-        resolve_path(self, &current_dir()?).map_err(|e| Error::from(e))
+        resolve_path(self, &current_dir()?).map_err(Error::from)
     }
 }
 

src/transpiler.rs

@@ -27,7 +27,7 @@ fn should_transpile(media_type: &MediaType) -> bool {
         | MediaType::Dcts
         | MediaType::Tsx => true,
 
-        _ => return false,
+        _ => false,
     }
 }
 

src/utilities.rs

@@ -54,7 +54,7 @@ pub fn validate(javascript: &str) -> Result<bool, Error> {
     let mut runtime = Runtime::new(Default::default())?;
     match runtime.load_modules(&module, vec![]) {
         Ok(_) => Ok(true),
-        Err(e) if matches!(e, Error::Runtime(_)) => Ok(false),
+        Err(Error::Runtime(_)) => Ok(false),
         Err(e) => Err(e),
     }
 }