diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
index c1b27eb2a9a..8aa0c43cc28 100644
--- a/.github/workflows/ci.yml
+++ b/.github/workflows/ci.yml
@@ -190,7 +190,6 @@ jobs:
       - name: Check Dependency Version
         if: steps.changes.outputs.src == 'true'
         run: pnpm run check-dependency-version
-  
 
   lint-website:
     name: Lint and format website
@@ -316,7 +315,8 @@ jobs:
           echo 'debug = false' >> Cargo.toml
 
       - name: Run test
-        run: cargo test --workspace -- --nocapture
+        # reason for excluding https://github.com/napi-rs/napi-rs/issues/2200
+        run: cargo test --workspace --exclude rspack_binding_options --exclude rspack_node -- --nocapture
 
   run_benchmark:
     name: Run benchmark
diff --git a/Cargo.lock b/Cargo.lock
index 9d437c049ef..fa493e59bd7 100644
--- a/Cargo.lock
+++ b/Cargo.lock
@@ -29,9 +29,9 @@ checksum = "f26201604c87b1e01bd3d98f8d5d9a8fcbb815e8cedb41ffccbeb4bf593a35fe"
 
 [[package]]
 name = "ahash"
-version = "0.7.6"
+version = "0.7.8"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "fcb51a0695d8f838b1ee009b3fbf66bda078cd64590202a864a8f3e8c4315c47"
+checksum = "891477e0c6a8957309ee5c45a6368af3ae14bb510732d2684ffa19af310920f9"
 dependencies = [
  "getrandom",
  "once_cell",
@@ -1417,7 +1417,7 @@ version = "0.12.3"
 source = "registry+https://github.com/rust-lang/crates.io-index"
 checksum = "8a9ee70c43aaf417c914396645a0fa852624801b24ebb7ae78fe8272889ac888"
 dependencies = [
- "ahash 0.7.6",
+ "ahash 0.7.8",
 ]
 
 [[package]]
@@ -6110,9 +6110,9 @@ dependencies = [
 
 [[package]]
 name = "time"
-version = "0.3.34"
+version = "0.3.36"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "c8248b6521bb14bc45b4067159b9b6ad792e2d6d754d6c41fb50e29fefe38749"
+checksum = "5dfd88e563464686c916c7e46e623e520ddc6d79fa6641390f2e3fa86e83e885"
 dependencies = [
  "deranged",
  "itoa",
@@ -6131,9 +6131,9 @@ checksum = "ef927ca75afb808a4d64dd374f00a2adf8d0fcff8e7b184af886c3c87ec4a3f3"
 
 [[package]]
 name = "time-macros"
-version = "0.2.17"
+version = "0.2.18"
 source = "registry+https://github.com/rust-lang/crates.io-index"
-checksum = "7ba3a3ef41e6672a2f0f001392bb5dcd3ff0a9992d618ca761a11c3121547774"
+checksum = "3f252a68540fde3a3877aeea552b832b40ab9a69e318efd078774a01ddee1ccf"
 dependencies = [
  "num-conv",
  "time-core",
diff --git a/crates/rspack_collections/src/ukey.rs b/crates/rspack_collections/src/ukey.rs
index 0f1b8650890..1aa1ef7bef2 100644
--- a/crates/rspack_collections/src/ukey.rs
+++ b/crates/rspack_collections/src/ukey.rs
@@ -226,7 +226,7 @@ where
   <Item as DatabaseItem>::ItemUkey: Eq + Hash + Debug,
 {
   pub fn add(&mut self, item: Item) -> &mut Item {
-    debug_assert!(self.inner.get(&item.ukey()).is_none());
+    debug_assert!(!self.inner.contains_key(&item.ukey()));
     let ukey = item.ukey();
     self.inner.entry(ukey).or_insert(item)
   }
diff --git a/crates/rspack_core/src/build_chunk_graph/code_splitter.rs b/crates/rspack_core/src/build_chunk_graph/code_splitter.rs
index b5003995d93..84014cf44ae 100644
--- a/crates/rspack_core/src/build_chunk_graph/code_splitter.rs
+++ b/crates/rspack_core/src/build_chunk_graph/code_splitter.rs
@@ -1014,10 +1014,10 @@ Or do you want to use the entrypoints '{name}' and '{runtime}' independently on
       .chunk_group_by_ukey
       .expect_get_mut(&cgi.chunk_group);
 
-    if chunk_group
+    #[allow(clippy::map_entry)]
+    if !chunk_group
       .module_pre_order_indices
-      .get(&item.module)
-      .is_none()
+      .contains_key(&item.module)
     {
       chunk_group
         .module_pre_order_indices
@@ -1058,10 +1058,10 @@ Or do you want to use the entrypoints '{name}' and '{runtime}' independently on
       .chunk_group_by_ukey
       .expect_get_mut(&cgi.chunk_group);
 
-    if chunk_group
+    #[allow(clippy::map_entry)]
+    if !chunk_group
       .module_post_order_indices
-      .get(&item.module)
-      .is_none()
+      .contains_key(&item.module)
     {
       chunk_group
         .module_post_order_indices
diff --git a/crates/rspack_core/src/chunk_graph/chunk_graph_chunk.rs b/crates/rspack_core/src/chunk_graph/chunk_graph_chunk.rs
index c0df7d34fcd..447197d42f3 100644
--- a/crates/rspack_core/src/chunk_graph/chunk_graph_chunk.rs
+++ b/crates/rspack_core/src/chunk_graph/chunk_graph_chunk.rs
@@ -76,10 +76,9 @@ impl ChunkGraph {
     old_module_id: &ModuleIdentifier,
     new_module_id: &ModuleIdentifier,
   ) {
-    if self
+    if !self
       .chunk_graph_module_by_module_identifier
-      .get(new_module_id)
-      .is_none()
+      .contains_key(new_module_id)
     {
       let new_chunk_graph_module = ChunkGraphModule::new();
       self
diff --git a/crates/rspack_core/src/compiler/compilation.rs b/crates/rspack_core/src/compiler/compilation.rs
index af4aaf5f5af..1a816db12ea 100644
--- a/crates/rspack_core/src/compiler/compilation.rs
+++ b/crates/rspack_core/src/compiler/compilation.rs
@@ -602,9 +602,9 @@ impl Compilation {
   /// Get sorted errors based on the factors as follows in order:
   /// - module identifier
   /// - error offset
-  /// Rspack assumes for each offset, there is only one error.
-  /// However, when it comes to the case that there are multiple errors with the same offset,
-  /// the order of these errors will not be guaranteed.
+  ///   Rspack assumes for each offset, there is only one error.
+  ///   However, when it comes to the case that there are multiple errors with the same offset,
+  ///   the order of these errors will not be guaranteed.
   pub fn get_errors_sorted(&self) -> impl Iterator<Item = &Diagnostic> {
     let get_offset = |d: &dyn rspack_error::miette::Diagnostic| {
       d.labels()
@@ -630,9 +630,9 @@ impl Compilation {
   /// Get sorted warnings based on the factors as follows in order:
   /// - module identifier
   /// - error offset
-  /// Rspack assumes for each offset, there is only one error.
-  /// However, when it comes to the case that there are multiple errors with the same offset,
-  /// the order of these errors will not be guaranteed.
+  ///   Rspack assumes for each offset, there is only one error.
+  ///   However, when it comes to the case that there are multiple errors with the same offset,
+  ///   the order of these errors will not be guaranteed.
   pub fn get_warnings_sorted(&self) -> impl Iterator<Item = &Diagnostic> {
     let get_offset = |d: &dyn rspack_error::miette::Diagnostic| {
       d.labels()
diff --git a/crates/rspack_core/src/compiler/module_executor/mod.rs b/crates/rspack_core/src/compiler/module_executor/mod.rs
index e0f75689581..012bea692b4 100644
--- a/crates/rspack_core/src/compiler/module_executor/mod.rs
+++ b/crates/rspack_core/src/compiler/module_executor/mod.rs
@@ -62,11 +62,7 @@ impl ModuleExecutor {
     make_artifact.diagnostics = Default::default();
     make_artifact.has_module_graph_change = false;
 
-    make_artifact = if let Ok(artifact) = update_module_graph(compilation, make_artifact, params) {
-      artifact
-    } else {
-      MakeArtifact::default()
-    };
+    make_artifact = update_module_graph(compilation, make_artifact, params).unwrap_or_default();
 
     let mut ctx = MakeTaskContext::new(compilation, make_artifact);
     let (event_sender, event_receiver) = unbounded_channel();
diff --git a/crates/rspack_core/src/exports_info.rs b/crates/rspack_core/src/exports_info.rs
index 68bef1161b2..754a006db9d 100644
--- a/crates/rspack_core/src/exports_info.rs
+++ b/crates/rspack_core/src/exports_info.rs
@@ -79,9 +79,7 @@ impl ExportsInfoId {
   }
 
   pub fn is_export_provided(&self, names: &[Atom], mg: &ModuleGraph) -> Option<ExportProvided> {
-    let Some(name) = names.first() else {
-      return None;
-    };
+    let name = names.first()?;
     let info = self.get_read_only_export_info(name, mg);
     if let Some(exports_info) = info.exports_info
       && names.len() > 1
@@ -256,9 +254,7 @@ impl ExportsInfoId {
     if names.len() == 1 {
       return Some(export_info);
     }
-    let Some(exports_info) = export_info.exports_info else {
-      return None;
-    };
+    let exports_info = export_info.exports_info?;
     exports_info.get_read_only_export_info_recursive(&names[1..], mg)
   }
 
@@ -456,10 +452,7 @@ impl ExportsInfoId {
           return Some(UsedName::Vec(names));
         }
         let export_info = self.get_read_only_export_info(&names[0], mg);
-        let x = export_info.get_used_name(Some(&names[0]), runtime);
-        let Some(x) = x else {
-          return None;
-        };
+        let x = export_info.get_used_name(Some(&names[0]), runtime)?;
         let names_len = names.len();
         let mut arr = if x == names[0] && names.len() == 1 {
           names.clone()
@@ -473,9 +466,7 @@ impl ExportsInfoId {
           && export_info.get_used(runtime) == UsageState::OnlyPropertiesUsed
         {
           let nested = exports_info.get_used_name(mg, runtime, UsedName::Vec(names[1..].to_vec()));
-          let Some(nested) = nested else {
-            return None;
-          };
+          let nested = nested?;
           arr.extend(match nested {
             UsedName::Str(name) => vec![name],
             UsedName::Vec(names) => names,
diff --git a/crates/rspack_core/src/lib.rs b/crates/rspack_core/src/lib.rs
index 3206ff1c58c..ddb6dfce3d0 100644
--- a/crates/rspack_core/src/lib.rs
+++ b/crates/rspack_core/src/lib.rs
@@ -5,7 +5,6 @@
 #![feature(anonymous_lifetime_in_impl_trait)]
 #![feature(hash_raw_entry)]
 #![feature(option_get_or_insert_default)]
-#![feature(slice_group_by)]
 
 use std::{fmt, sync::Arc};
 mod dependencies_block;
diff --git a/crates/rspack_core/src/module_factory.rs b/crates/rspack_core/src/module_factory.rs
index e3198d094c4..347f043e225 100644
--- a/crates/rspack_core/src/module_factory.rs
+++ b/crates/rspack_core/src/module_factory.rs
@@ -39,7 +39,7 @@ impl ModuleFactoryCreateData {
   }
 
   pub fn add_file_dependencies(&mut self, files: impl IntoIterator<Item = PathBuf>) {
-    self.file_dependencies.extend(files.into_iter());
+    self.file_dependencies.extend(files);
   }
 
   pub fn add_context_dependency(&mut self, context: PathBuf) {
@@ -47,7 +47,7 @@ impl ModuleFactoryCreateData {
   }
 
   pub fn add_context_dependencies(&mut self, contexts: impl IntoIterator<Item = PathBuf>) {
-    self.context_dependencies.extend(contexts.into_iter());
+    self.context_dependencies.extend(contexts);
   }
 
   pub fn add_missing_dependency(&mut self, missing: PathBuf) {
diff --git a/crates/rspack_core/src/module_graph/mod.rs b/crates/rspack_core/src/module_graph/mod.rs
index e91577afd2b..c020add6430 100644
--- a/crates/rspack_core/src/module_graph/mod.rs
+++ b/crates/rspack_core/src/module_graph/mod.rs
@@ -272,9 +272,7 @@ impl<'a> ModuleGraph<'a> {
     connection_id: &ConnectionId,
     force: bool,
   ) -> Option<BuildDependency> {
-    let Some(connection) = self.connection_by_connection_id(connection_id) else {
-      return None;
-    };
+    let connection = self.connection_by_connection_id(connection_id)?;
     let module_identifier = *connection.module_identifier();
     let original_module_identifier = connection.original_module_identifier;
     let dependency_id = connection.dependency_id;
diff --git a/crates/rspack_core/src/update_hash.rs b/crates/rspack_core/src/update_hash.rs
index a50de02a22b..7b7f47bc70d 100644
--- a/crates/rspack_core/src/update_hash.rs
+++ b/crates/rspack_core/src/update_hash.rs
@@ -2,6 +2,7 @@ use std::hash::Hasher;
 
 use crate::{Compilation, RuntimeSpec};
 
+#[allow(dead_code)]
 pub struct UpdateHashContext<'a> {
   pub compilation: &'a Compilation,
   pub runtime: Option<&'a RuntimeSpec>,
diff --git a/crates/rspack_fs_node/Cargo.toml b/crates/rspack_fs_node/Cargo.toml
index 444e044a157..f959b39fb4a 100644
--- a/crates/rspack_fs_node/Cargo.toml
+++ b/crates/rspack_fs_node/Cargo.toml
@@ -21,3 +21,6 @@ rspack_napi = { path = "../rspack_napi" }
 
 [build-dependencies]
 napi-build = { workspace = true }
+
+[lints.rust]
+unexpected_cfgs = { level = "warn", check-cfg = ['cfg(node)'] }
diff --git a/crates/rspack_loader_runner/src/loader.rs b/crates/rspack_loader_runner/src/loader.rs
index 86ff6b176c8..c602a2ea549 100644
--- a/crates/rspack_loader_runner/src/loader.rs
+++ b/crates/rspack_loader_runner/src/loader.rs
@@ -233,6 +233,7 @@ pub(crate) mod test {
 
   use super::{Loader, LoaderItem};
 
+  #[allow(dead_code)]
   pub(crate) struct Custom;
 
   #[async_trait::async_trait]
@@ -243,6 +244,7 @@ pub(crate) mod test {
     }
   }
 
+  #[allow(dead_code)]
   pub(crate) struct Custom2;
   #[async_trait::async_trait]
   impl Loader<()> for Custom2 {}
@@ -252,6 +254,7 @@ pub(crate) mod test {
     }
   }
 
+  #[allow(dead_code)]
   pub(crate) struct Builtin;
   #[async_trait::async_trait]
   impl Loader<()> for Builtin {}
diff --git a/crates/rspack_loader_swc/src/compiler.rs b/crates/rspack_loader_swc/src/compiler.rs
index 2486b413d65..cb2a42bf960 100644
--- a/crates/rspack_loader_swc/src/compiler.rs
+++ b/crates/rspack_loader_swc/src/compiler.rs
@@ -525,13 +525,7 @@ impl SwcCompiler {
         Ok(r) => r,
         Err(_err) => {
           // Load original source map if possible
-          match read_file_sourcemap(data_url) {
-            Ok(v) => v,
-            Err(_) => {
-              // tracing::error!("failed to read input source map: {:?}", err);
-              None
-            }
-          }
+          read_file_sourcemap(data_url).unwrap_or(None)
         }
       }
     };
diff --git a/crates/rspack_macros_test/tests/runtime_module.rs b/crates/rspack_macros_test/tests/runtime_module.rs
index f657040edb2..7188b83b328 100644
--- a/crates/rspack_macros_test/tests/runtime_module.rs
+++ b/crates/rspack_macros_test/tests/runtime_module.rs
@@ -5,6 +5,7 @@ use rspack_core::{rspack_sources::Source, Compilation};
 use rspack_error::Result;
 use rspack_macros::impl_runtime_module;
 
+#[allow(dead_code)]
 #[test]
 fn with_generic() {
   #[impl_runtime_module]
diff --git a/crates/rspack_plugin_javascript/src/ast/parse.rs b/crates/rspack_plugin_javascript/src/ast/parse.rs
index 5eb7ba3f734..ca517d00d35 100644
--- a/crates/rspack_plugin_javascript/src/ast/parse.rs
+++ b/crates/rspack_plugin_javascript/src/ast/parse.rs
@@ -25,11 +25,11 @@ fn module_type_to_is_module(value: &ModuleType) -> IsModule {
 
 /// Why this helper function design like this?
 /// 1. `swc_ecma_parser` could return ast with some errors which are recoverable
-/// or warning (though swc defined them as errors), but the parser at here should
-/// be non-error-tolerant.
+///    or warning (though swc defined them as errors), but the parser at here should
+///    be non-error-tolerant.
 ///
 /// 2. We can't convert to [rspack_error::Error] at this point, because there is
-/// no `path` and `source`
+///    no `path` and `source`
 pub fn parse_js(
   fm: Arc<SourceFile>,
   target: EsVersion,
diff --git a/crates/rspack_plugin_javascript/src/dependency/commonjs/common_js_export_require_dependency.rs b/crates/rspack_plugin_javascript/src/dependency/commonjs/common_js_export_require_dependency.rs
index a0bf1fb1519..e9b85cde295 100644
--- a/crates/rspack_plugin_javascript/src/dependency/commonjs/common_js_export_require_dependency.rs
+++ b/crates/rspack_plugin_javascript/src/dependency/commonjs/common_js_export_require_dependency.rs
@@ -189,9 +189,7 @@ impl Dependency for CommonJsExportRequireDependency {
       let Some(name) = self.names.first() else {
         unreachable!();
       };
-      let Some(from) = mg.connection_by_dependency(&self.id) else {
-        return None;
-      };
+      let from = mg.connection_by_dependency(&self.id)?;
       Some(ExportsSpec {
         exports: ExportsOfExportsSpec::Array(vec![ExportNameOrSpec::ExportSpec(ExportSpec {
           name: name.to_owned(),
@@ -208,9 +206,7 @@ impl Dependency for CommonJsExportRequireDependency {
         ..Default::default()
       })
     } else if self.names.is_empty() {
-      let Some(from) = mg.connection_by_dependency(&self.id) else {
-        return None;
-      };
+      let from = mg.connection_by_dependency(&self.id)?;
       if let Some(reexport_info) = self.get_star_reexports(mg, None, from.module_identifier()) {
         Some(ExportsSpec {
           exports: ExportsOfExportsSpec::Array(
diff --git a/crates/rspack_plugin_javascript/src/dependency/esm/harmony_export_imported_specifier_dependency.rs b/crates/rspack_plugin_javascript/src/dependency/esm/harmony_export_imported_specifier_dependency.rs
index 9e89aa91b6c..63e5600e90f 100644
--- a/crates/rspack_plugin_javascript/src/dependency/esm/harmony_export_imported_specifier_dependency.rs
+++ b/crates/rspack_plugin_javascript/src/dependency/esm/harmony_export_imported_specifier_dependency.rs
@@ -921,9 +921,7 @@ impl HarmonyExportImportedSpecifierDependency {
         &potential_conflicts.names[potential_conflicts.names_slice
           ..potential_conflicts.dependency_indices[potential_conflicts.dependency_index]],
       );
-      let Some(imported_module) = module_graph.get_module_by_dependency_id(&self.id) else {
-        return None;
-      };
+      let imported_module = module_graph.get_module_by_dependency_id(&self.id)?;
       let exports_info = module_graph.get_exports_info(&imported_module.identifier());
       let mut conflicts: IndexMap<&str, Vec<&Atom>, BuildHasherDefault<FxHasher>> =
         IndexMap::default();
diff --git a/crates/rspack_plugin_javascript/src/dependency/esm/harmony_import_dependency.rs b/crates/rspack_plugin_javascript/src/dependency/esm/harmony_import_dependency.rs
index 80701b5dcbb..5550495a4ad 100644
--- a/crates/rspack_plugin_javascript/src/dependency/esm/harmony_import_dependency.rs
+++ b/crates/rspack_plugin_javascript/src/dependency/esm/harmony_import_dependency.rs
@@ -207,10 +207,7 @@ pub fn harmony_import_dependency_get_linking_error<T: ModuleDependency>(
   additional_msg: String,
   should_error: bool,
 ) -> Option<Diagnostic> {
-  let Some(imported_module) = module_graph.get_module_by_dependency_id(module_dependency.id())
-  else {
-    return None;
-  };
+  let imported_module = module_graph.get_module_by_dependency_id(module_dependency.id())?;
   if !imported_module.get_diagnostics().is_empty() {
     return None;
   }
diff --git a/crates/rspack_plugin_javascript/src/lib.rs b/crates/rspack_plugin_javascript/src/lib.rs
index 8284889d696..f131cd226a5 100644
--- a/crates/rspack_plugin_javascript/src/lib.rs
+++ b/crates/rspack_plugin_javascript/src/lib.rs
@@ -2,6 +2,7 @@
 #![feature(if_let_guard)]
 #![feature(let_chains)]
 #![feature(box_patterns)]
+#![feature(trait_upcasting)]
 #![recursion_limit = "256"]
 
 pub mod ast;
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/common_js_exports_parse_plugin.rs b/crates/rspack_plugin_javascript/src/parser_plugin/common_js_exports_parse_plugin.rs
index e29a1d2687b..416fa399ef5 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/common_js_exports_parse_plugin.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/common_js_exports_parse_plugin.rs
@@ -32,7 +32,7 @@ fn get_member_expression_info<E: ExprLike>(
     None => is_module_exports_member_expr_start(expr),
   };
   expr.as_member().and_then(|expr: &MemberExpr| {
-    let Some(members) = parser
+    let members = parser
       .get_member_expression_info(expr, AllowedMemberTypes::Expression)
       .and_then(|info| match info {
         MemberExpressionInfo::Call(_) => None,
@@ -44,10 +44,7 @@ fn get_member_expression_info<E: ExprLike>(
           .skip(if is_module_exports_start { 1 } else { 0 })
           .map(|n| n.to_owned())
           .collect::<Vec<_>>()
-      })
-    else {
-      return None;
-    };
+      })?;
     match expr.obj {
       box Expr::Call(_) => Some(members),
       box Expr::Ident(_) => Some(members),
@@ -378,11 +375,7 @@ impl JavascriptParserPlugin for CommonJsExportsParserPlugin {
 
     let handle_remaining = |parser: &mut JavascriptParser, base: ExportsBase| {
       let is_module_exports_start = matches!(base, ExportsBase::ModuleExports);
-      let Some(remaining) =
-        get_member_expression_info(parser, left_expr, Some(is_module_exports_start))
-      else {
-        return None;
-      };
+      let remaining = get_member_expression_info(parser, left_expr, Some(is_module_exports_start))?;
 
       if (remaining.is_empty() || remaining.first().is_some_and(|i| i != "__esModule"))
         && parser.is_require_call_expr(&assign_expr.right)
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/common_js_imports_parse_plugin.rs b/crates/rspack_plugin_javascript/src/parser_plugin/common_js_imports_parse_plugin.rs
index 832c9229250..268f8a60c64 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/common_js_imports_parse_plugin.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/common_js_imports_parse_plugin.rs
@@ -88,9 +88,7 @@ impl CommonJsImportsParserPlugin {
       return None;
     }
 
-    let Some((members, first_arg, loc)) = extract_require_call_info(parser, mem_expr) else {
-      return None;
-    };
+    let (members, first_arg, loc) = extract_require_call_info(parser, mem_expr)?;
 
     let param = parser.evaluate_expression(&first_arg.expr);
     param.is_string().then(|| {
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/compatibility_plugin.rs b/crates/rspack_plugin_javascript/src/parser_plugin/compatibility_plugin.rs
index 65bf805c26b..a8930b25b81 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/compatibility_plugin.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/compatibility_plugin.rs
@@ -117,12 +117,8 @@ impl JavascriptParserPlugin for CompatibilityPlugin {
   }
 
   fn pre_statement(&self, parser: &mut JavascriptParser, stmt: Statement) -> Option<bool> {
-    let Some(fn_decl) = stmt.as_function_decl() else {
-      return None;
-    };
-    let Some(ident) = fn_decl.ident() else {
-      return None;
-    };
+    let fn_decl = stmt.as_function_decl()?;
+    let ident = fn_decl.ident()?;
     let name = ident.sym.as_str();
     if name != RuntimeGlobals::REQUIRE.name() {
       None
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/const/if_stmt.rs b/crates/rspack_plugin_javascript/src/parser_plugin/const/if_stmt.rs
index 0e06bf089bf..da36a88741e 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/const/if_stmt.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/const/if_stmt.rs
@@ -132,9 +132,7 @@ fn get_hoisted_declarations<'a>(
 
 pub fn statement_if(scanner: &mut JavascriptParser, stmt: &IfStmt) -> Option<bool> {
   let param = scanner.evaluate_expression(&stmt.test);
-  let Some(boolean) = param.as_bool() else {
-    return None;
-  };
+  let boolean = param.as_bool()?;
   if !param.could_have_side_effects() {
     scanner
       .presentational_dependencies
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/const/logic_expr.rs b/crates/rspack_plugin_javascript/src/parser_plugin/const/logic_expr.rs
index 8820f5e6bab..accdee19247 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/const/logic_expr.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/const/logic_expr.rs
@@ -16,9 +16,7 @@ pub fn expression_logic_operator(scanner: &mut JavascriptParser, expr: &BinExpr)
   if expr.op == BinaryOp::LogicalAnd || expr.op == BinaryOp::LogicalOr {
     let param = scanner.evaluate_expression(&expr.left);
     let boolean = param.as_bool();
-    let Some(boolean) = boolean else {
-      return None;
-    };
+    let boolean = boolean?;
     let keep_right = if boolean {
       expr.op == BinaryOp::LogicalAnd
     } else {
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/define_plugin/parser.rs b/crates/rspack_plugin_javascript/src/parser_plugin/define_plugin/parser.rs
index 56107a852e5..eccbbd87ea5 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/define_plugin/parser.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/define_plugin/parser.rs
@@ -367,7 +367,7 @@ impl JavascriptParserPlugin for DefineParserPlugin {
       && let Some(on_evaluate_typeof) = &record.on_evaluate_typeof
     {
       return on_evaluate_typeof(record, parser, expr.span.real_lo(), expr.span.hi.0);
-    } else if self.walk_data.object_define_record.get(for_name).is_some() {
+    } else if self.walk_data.object_define_record.contains_key(for_name) {
       return Some(evaluate_to_string(
         "object".to_string(),
         expr.span.real_lo(),
@@ -406,7 +406,7 @@ impl JavascriptParserPlugin for DefineParserPlugin {
       && let Some(on_typeof) = &record.on_typeof
     {
       return on_typeof(record, parser, expr.span.real_lo(), expr.span.real_hi());
-    } else if self.walk_data.object_define_record.get(for_name).is_some() {
+    } else if self.walk_data.object_define_record.contains_key(for_name) {
       debug_assert!(!parser.in_short_hand);
       parser.presentational_dependencies.push(Box::new(dep(
         parser,
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/exports_info_api_plugin.rs b/crates/rspack_plugin_javascript/src/parser_plugin/exports_info_api_plugin.rs
index 8fc5e4fa8ce..c35f5c5c0df 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/exports_info_api_plugin.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/exports_info_api_plugin.rs
@@ -17,7 +17,7 @@ impl JavascriptParserPlugin for ExportsInfoApiPlugin {
     member_expr: &swc_core::ecma::ast::MemberExpr,
     _name: &str,
   ) -> Option<bool> {
-    let Some((members, root)) = parser
+    let (members, root) = parser
       .get_member_expression_info(member_expr, AllowedMemberTypes::Expression)
       .and_then(|info| match info {
         MemberExpressionInfo::Call(_) => None,
@@ -28,10 +28,8 @@ impl JavascriptParserPlugin for ExportsInfoApiPlugin {
             None
           }
         }
-      })
-    else {
-      return None;
-    };
+      })?;
+
     let len = members.len();
     if len >= 1 && root == WEBPACK_EXPORTS_INFO && parser.is_unresolved_ident(WEBPACK_EXPORTS_INFO)
     {
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/import_parser_plugin.rs b/crates/rspack_plugin_javascript/src/parser_plugin/import_parser_plugin.rs
index 0901eafc924..edb0162a5ed 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/import_parser_plugin.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/import_parser_plugin.rs
@@ -26,9 +26,7 @@ impl JavascriptParserPlugin for ImportParserPlugin {
     let Callee::Import(import_call) = &node.callee else {
       unreachable!()
     };
-    let Some(dyn_imported) = node.args.first() else {
-      return None;
-    };
+    let dyn_imported = node.args.first()?;
     if dyn_imported.spread.is_some() {
       return None;
     }
diff --git a/crates/rspack_plugin_javascript/src/parser_plugin/trait.rs b/crates/rspack_plugin_javascript/src/parser_plugin/trait.rs
index 4e886fd8313..f64d2dc2ed2 100644
--- a/crates/rspack_plugin_javascript/src/parser_plugin/trait.rs
+++ b/crates/rspack_plugin_javascript/src/parser_plugin/trait.rs
@@ -17,7 +17,7 @@ type KeepRight = bool;
 pub trait JavascriptParserPlugin {
   /// Return:
   /// - `Some(true)` signifies the termination of the current
-  /// statement's visit during the pre-walk phase.
+  ///   statement's visit during the pre-walk phase.
   /// - Other return values imply that the walk operation ought to continue
   fn pre_statement(&self, _parser: &mut JavascriptParser, _stmt: Statement) -> Option<bool> {
     None
diff --git a/crates/rspack_plugin_javascript/src/plugin/mod.rs b/crates/rspack_plugin_javascript/src/plugin/mod.rs
index 9e3fbb74eb4..812e2a76e25 100644
--- a/crates/rspack_plugin_javascript/src/plugin/mod.rs
+++ b/crates/rspack_plugin_javascript/src/plugin/mod.rs
@@ -683,7 +683,7 @@ impl JsPlugin {
           continue;
         };
 
-        if renamed_inline_modules.get(m_identifier).is_some() {
+        if renamed_inline_modules.contains_key(m_identifier) {
           if let Some(source) = renamed_inline_modules.get(m_identifier) {
             rendered_module = source.clone();
           };
diff --git a/crates/rspack_plugin_javascript/src/utils/eval/eval_new_expr.rs b/crates/rspack_plugin_javascript/src/utils/eval/eval_new_expr.rs
index 2efd7e6acba..3b38cc0d3fb 100644
--- a/crates/rspack_plugin_javascript/src/utils/eval/eval_new_expr.rs
+++ b/crates/rspack_plugin_javascript/src/utils/eval/eval_new_expr.rs
@@ -10,9 +10,7 @@ pub fn eval_new_expression(
   scanner: &mut JavascriptParser,
   expr: &NewExpr,
 ) -> Option<BasicEvaluatedExpression> {
-  let Some(ident) = expr.callee.as_ident() else {
-    return None;
-  };
+  let ident = expr.callee.as_ident()?;
   if ident.sym.as_str() != "RegExp" {
     // FIXME: call hooks
     return None;
@@ -35,9 +33,7 @@ pub fn eval_new_expression(
   }
 
   let evaluated_reg_exp = scanner.evaluate_expression(&arg1.expr);
-  let Some(reg_exp) = evaluated_reg_exp.as_string() else {
-    return None;
-  };
+  let reg_exp = evaluated_reg_exp.as_string()?;
 
   let flags = if let Some(arg2) = args.get(1) {
     if arg2.spread.is_some() {
diff --git a/crates/rspack_plugin_javascript/src/utils/eval/eval_unary_expr.rs b/crates/rspack_plugin_javascript/src/utils/eval/eval_unary_expr.rs
index ac9bc43e102..5f5c15072af 100644
--- a/crates/rspack_plugin_javascript/src/utils/eval/eval_unary_expr.rs
+++ b/crates/rspack_plugin_javascript/src/utils/eval/eval_unary_expr.rs
@@ -113,9 +113,7 @@ pub fn eval_unary_expression(
     UnaryOp::TypeOf => eval_typeof(scanner, expr),
     UnaryOp::Bang => {
       let arg = scanner.evaluate_expression(&expr.arg);
-      let Some(boolean) = arg.as_bool() else {
-        return None;
-      };
+      let boolean = arg.as_bool()?;
       let mut eval = BasicEvaluatedExpression::with_range(expr.span().real_lo(), expr.span_hi().0);
       eval.set_bool(!boolean);
       eval.set_side_effects(arg.could_have_side_effects());
@@ -123,9 +121,7 @@ pub fn eval_unary_expression(
     }
     UnaryOp::Tilde => {
       let arg = scanner.evaluate_expression(&expr.arg);
-      let Some(number) = arg.as_int() else {
-        return None;
-      };
+      let number = arg.as_int()?;
       let mut eval = BasicEvaluatedExpression::with_range(expr.span().real_lo(), expr.span_hi().0);
       eval.set_number(!number as f64);
       eval.set_side_effects(arg.could_have_side_effects());
@@ -133,9 +129,7 @@ pub fn eval_unary_expression(
     }
     UnaryOp::Minus | UnaryOp::Plus => {
       let arg = scanner.evaluate_expression(&expr.arg);
-      let Some(number) = arg.as_number() else {
-        return None;
-      };
+      let number = arg.as_number()?;
       let res = match &expr.op {
         UnaryOp::Minus => -number,
         UnaryOp::Plus => number,
diff --git a/crates/rspack_plugin_javascript/src/utils/object_properties.rs b/crates/rspack_plugin_javascript/src/utils/object_properties.rs
index 4c02a9a79f3..6640730c457 100644
--- a/crates/rspack_plugin_javascript/src/utils/object_properties.rs
+++ b/crates/rspack_plugin_javascript/src/utils/object_properties.rs
@@ -20,9 +20,7 @@ pub fn get_value_by_obj_prop<'a>(obj: &'a ObjectLit, field: &'a str) -> Option<&
 }
 
 pub fn get_literal_str_by_obj_prop<'a>(obj: &'a ObjectLit, field: &'a str) -> Option<&'a Str> {
-  let Some(lit) = get_value_by_obj_prop(obj, field).and_then(|e| e.as_lit()) else {
-    return None;
-  };
+  let lit = get_value_by_obj_prop(obj, field).and_then(|e| e.as_lit())?;
   match lit {
     Lit::Str(str) => Some(str),
     _ => None,
@@ -30,9 +28,7 @@ pub fn get_literal_str_by_obj_prop<'a>(obj: &'a ObjectLit, field: &'a str) -> Op
 }
 
 pub fn get_bool_by_obj_prop<'a>(obj: &'a ObjectLit, field: &'a str) -> Option<&'a Bool> {
-  let Some(lit) = get_value_by_obj_prop(obj, field).and_then(|e| e.as_lit()) else {
-    return None;
-  };
+  let lit = get_value_by_obj_prop(obj, field).and_then(|e| e.as_lit())?;
   match lit {
     Lit::Bool(bool) => Some(bool),
     _ => None,
@@ -40,9 +36,7 @@ pub fn get_bool_by_obj_prop<'a>(obj: &'a ObjectLit, field: &'a str) -> Option<&'
 }
 
 pub fn get_regex_by_obj_prop<'a>(obj: &'a ObjectLit, field: &'a str) -> Option<&'a Regex> {
-  let Some(lit) = get_value_by_obj_prop(obj, field).and_then(|e| e.as_lit()) else {
-    return None;
-  };
+  let lit = get_value_by_obj_prop(obj, field).and_then(|e| e.as_lit())?;
   match lit {
     Lit::Regex(regexp) => Some(regexp),
     _ => None,
diff --git a/crates/rspack_plugin_javascript/src/visitors/dependency/parser/mod.rs b/crates/rspack_plugin_javascript/src/visitors/dependency/parser/mod.rs
index fa6fcc4506a..8404ecc3384 100644
--- a/crates/rspack_plugin_javascript/src/visitors/dependency/parser/mod.rs
+++ b/crates/rspack_plugin_javascript/src/visitors/dependency/parser/mod.rs
@@ -427,16 +427,12 @@ impl<'parser> JavascriptParser<'parser> {
   }
 
   pub fn get_mut_variable_info(&mut self, name: &str) -> Option<&mut VariableInfo> {
-    let Some(id) = self.definitions_db.get(self.definitions, name) else {
-      return None;
-    };
+    let id = self.definitions_db.get(self.definitions, name)?;
     Some(self.definitions_db.expect_get_mut_variable(id))
   }
 
   pub fn get_variable_info(&mut self, name: &str) -> Option<&VariableInfo> {
-    let Some(id) = self.definitions_db.get(self.definitions, name) else {
-      return None;
-    };
+    let id = self.definitions_db.get(self.definitions, name)?;
     Some(self.definitions_db.expect_get_variable(id))
   }
 
@@ -573,16 +569,12 @@ impl<'parser> JavascriptParser<'parser> {
         if !allowed_types.contains(AllowedMemberTypes::CallExpression) {
           return None;
         }
-        let Some(root_name) = expr.callee.get_root_name() else {
-          return None;
-        };
-        let Some(FreeInfo {
+        let root_name = expr.callee.get_root_name()?;
+        let FreeInfo {
           name: resolved_root,
           info: root_info,
-        }) = self.get_free_info_from_variable(&root_name)
-        else {
-          return None;
-        };
+        } = self.get_free_info_from_variable(&root_name)?;
+
         let callee_name = object_and_members_to_name(resolved_root, &members);
         members.reverse();
         members_optionals.reverse();
@@ -602,16 +594,13 @@ impl<'parser> JavascriptParser<'parser> {
         if !allowed_types.contains(AllowedMemberTypes::Expression) {
           return None;
         }
-        let Some(root_name) = object.get_root_name() else {
-          return None;
-        };
-        let Some(FreeInfo {
+        let root_name = object.get_root_name()?;
+
+        let FreeInfo {
           name: resolved_root,
           info: root_info,
-        }) = self.get_free_info_from_variable(&root_name)
-        else {
-          return None;
-        };
+        } = self.get_free_info_from_variable(&root_name)?;
+
         let name = object_and_members_to_name(resolved_root, &members);
         members.reverse();
         members_optionals.reverse();
diff --git a/crates/rspack_plugin_javascript/src/visitors/dependency/util.rs b/crates/rspack_plugin_javascript/src/visitors/dependency/util.rs
index 774b2cfb992..7182e0ba398 100644
--- a/crates/rspack_plugin_javascript/src/visitors/dependency/util.rs
+++ b/crates/rspack_plugin_javascript/src/visitors/dependency/util.rs
@@ -35,6 +35,7 @@ pub fn collect_destructuring_assignment_properties(
   }
 }
 
+#[allow(dead_code)]
 pub(crate) mod expr_like {
   use std::any::Any;
 
@@ -255,9 +256,7 @@ pub fn extract_require_call_info(
   };
 
   // call require() with no param
-  let Some(first_arg) = args.first() else {
-    return None;
-  };
+  let first_arg = args.first()?;
 
   let loc = DependencyLocation::new(expr.span().real_lo(), expr.span().real_hi(), None);
 
diff --git a/crates/rspack_plugin_library/src/assign_library_plugin.rs b/crates/rspack_plugin_library/src/assign_library_plugin.rs
index 4dc4ffbc9c9..a99bbf17ecc 100644
--- a/crates/rspack_plugin_library/src/assign_library_plugin.rs
+++ b/crates/rspack_plugin_library/src/assign_library_plugin.rs
@@ -498,7 +498,7 @@ fn access_with_init(accessor: &[String], existing_length: usize, init_last: bool
     props_so_far.push(accessor[i].clone());
     current = format!(
       "({current}{} = {base}{} || {{}})",
-      property_access(&vec![&accessor[i]], 0),
+      property_access(vec![&accessor[i]], 0),
       property_access(&props_so_far, 0)
     );
     i += 1;
diff --git a/crates/rspack_plugin_library/src/module_library_plugin.rs b/crates/rspack_plugin_library/src/module_library_plugin.rs
index 08e1eeaedb2..2a987c34010 100644
--- a/crates/rspack_plugin_library/src/module_library_plugin.rs
+++ b/crates/rspack_plugin_library/src/module_library_plugin.rs
@@ -85,7 +85,7 @@ fn render_startup(
     let var_name = format!("__webpack_exports__{}", to_identifier(info_name));
     source.add(RawSource::from(format!(
       "var {var_name} = __webpack_exports__{};\n",
-      property_access(&vec![used_name], 0)
+      property_access(vec![used_name], 0)
     )));
     exports.push(format!("{var_name} as {}", info_name));
   }
diff --git a/crates/rspack_plugin_library/src/system_library_plugin.rs b/crates/rspack_plugin_library/src/system_library_plugin.rs
index e8e308a4a55..fd6adb10f16 100644
--- a/crates/rspack_plugin_library/src/system_library_plugin.rs
+++ b/crates/rspack_plugin_library/src/system_library_plugin.rs
@@ -23,6 +23,7 @@ struct SystemLibraryPluginParsed<'a> {
 }
 
 #[derive(Debug, Default)]
+#[allow(dead_code)]
 struct SystemLibraryJavascriptModulesPluginPlugin;
 
 #[plugin]
diff --git a/crates/rspack_plugin_mf/src/sharing/consume_shared_plugin.rs b/crates/rspack_plugin_mf/src/sharing/consume_shared_plugin.rs
index 3aeaa5702f0..5556c73b24f 100644
--- a/crates/rspack_plugin_mf/src/sharing/consume_shared_plugin.rs
+++ b/crates/rspack_plugin_mf/src/sharing/consume_shared_plugin.rs
@@ -117,9 +117,7 @@ fn get_required_version_from_description_file(
   data: serde_json::Value,
   package_name: &str,
 ) -> Option<ConsumeVersion> {
-  let Some(data) = data.as_object() else {
-    return None;
-  };
+  let data = data.as_object()?;
   let get_version_from_dependencies = |dependencies: &str| {
     data
       .get(dependencies)
diff --git a/crates/rspack_plugin_wasm/src/dependency/wasm_import_dependency.rs b/crates/rspack_plugin_wasm/src/dependency/wasm_import_dependency.rs
index cd2a6613d00..0778e2b4901 100644
--- a/crates/rspack_plugin_wasm/src/dependency/wasm_import_dependency.rs
+++ b/crates/rspack_plugin_wasm/src/dependency/wasm_import_dependency.rs
@@ -6,6 +6,7 @@ use swc_core::ecma::atoms::Atom;
 
 use crate::WasmNode;
 
+#[allow(dead_code)]
 #[derive(Debug, Clone)]
 pub struct WasmImportDependency {
   id: DependencyId,
diff --git a/rust-toolchain.toml b/rust-toolchain.toml
index 8a00cf72b62..a9f7244565d 100644
--- a/rust-toolchain.toml
+++ b/rust-toolchain.toml
@@ -2,4 +2,4 @@
 profile = "default"
 # Use nightly for better access to the latest Rust features.
 # This date is aligned to stable release dates.
-channel = "nightly-2023-12-28" # v1.75.0
+channel = "nightly-2024-06-07" # v1.80.0