chore: add napi::tokio_runtime::block_on (#12358)

* chore: add napi::tokio_runtime::block_on

* chore: add allow to exist code

Author: stormslowly

clippy.toml

@@ -12,5 +12,6 @@ disallowed-methods = [
   { path = "std::mem::forget", reason = "future::scope is unsafe when used with forget" },
   { path = "futures::executor::block_on", reason = "block_on can cause deadlock easily" },
   { path = "async_std::task::block_on", reason = "block_on can cause deadlock easily" },
-  { path = "pollster::block_on", reason = "block_on can cause deadlock easily" }
+  { path = "pollster::block_on", reason = "block_on can cause deadlock easily" },
+  { path = "napi::tokio_runtime::block_on", reason = "block_on can cause deadlock easily" }
 ]

crates/rspack_binding_api/src/fs_node/write.rs

@@ -188,6 +188,7 @@ impl ReadableFileSystem for NodeFileSystem {
   }
   #[instrument(skip(self), level = "debug")]
   fn read_sync(&self, path: &Utf8Path) -> Result<Vec<u8>> {
+    #[allow(clippy::disallowed_methods)]
     block_on(self.read(path))
   }
 
@@ -210,6 +211,7 @@ impl ReadableFileSystem for NodeFileSystem {
 
   #[instrument(skip(self), level = "debug")]
   fn metadata_sync(&self, path: &Utf8Path) -> Result<FileMetadata> {
+    #[allow(clippy::disallowed_methods)]
     block_on(self.metadata(path))
   }
 
@@ -265,6 +267,7 @@ impl ReadableFileSystem for NodeFileSystem {
   }
   #[instrument(skip(self), level = "debug")]
   fn read_dir_sync(&self, dir: &Utf8Path) -> Result<Vec<String>> {
+    #[allow(clippy::disallowed_methods)]
     block_on(ReadableFileSystem::read_dir(self, dir))
   }
   #[instrument(skip(self), level = "debug")]

crates/rspack_binding_api/src/resolver.rs

@@ -53,6 +53,7 @@ impl JsResolver {
 impl JsResolver {
   #[napi]
   pub fn resolve_sync(&self, path: String, request: String) -> napi::Result<Either<String, ()>> {
+    #[allow(clippy::disallowed_methods)]
     block_on(async {
       match self.resolver.resolve(Path::new(&path), &request).await {
         Ok(rspack_core::ResolveResult::Resource(resource)) => Ok(Either::A(resource.full_path())),