fix: port webpack fix for __webpack_module__ name collision (#12956)

* Initial plan

* feat: port webpack fix for __webpack_module__ name collision

- Implement pre_declarator hook to detect 'module' variable declarations
- Implement pre_statement hook to detect 'module' function/class declarations
- Add test cases for collision scenarios

Co-authored-by: SyMind <19852293+SyMind@users.noreply.github.com>

* fix: use fixed internal variable names for consistency

Co-authored-by: SyMind <19852293+SyMind@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: SyMind <19852293+SyMind@users.noreply.github.com>
Co-authored-by: Cong-Cong <dacongsama@live.com>

Author: Copilot

crates/rspack_plugin_javascript/src/parser_plugin/api_plugin.rs

@@ -1,16 +1,16 @@
-use rspack_core::{ConstDependency, RuntimeGlobals, RuntimeRequirementsDependency};
+use rspack_core::{ConstDependency, ModuleArgument, RuntimeGlobals, RuntimeRequirementsDependency};
 use rspack_error::{Error, Severity};
 use rspack_util::SpanExt;
 use swc_core::{
   common::{Span, Spanned},
-  ecma::ast::{CallExpr, Ident, UnaryExpr},
+  ecma::ast::{CallExpr, Ident, Pat, UnaryExpr},
 };
 
 use crate::{
   dependency::{ModuleArgumentDependency, RequireMainDependency},
   parser_plugin::JavascriptParserPlugin,
   utils::eval::{self, BasicEvaluatedExpression},
-  visitors::{JavascriptParser, create_traceable_error},
+  visitors::{JavascriptParser, Statement, VariableDeclaration, create_traceable_error},
 };
 
 fn expression_not_supported(
@@ -340,6 +340,48 @@ impl JavascriptParserPlugin for APIPlugin {
     None
   }
 
+  fn pre_declarator(
+    &self,
+    parser: &mut JavascriptParser,
+    declarator: &swc_core::ecma::ast::VarDeclarator,
+    _declaration: VariableDeclaration<'_>,
+  ) -> Option<bool> {
+    // Check if we're at top level scope and the declarator is a simple identifier named "module"
+    if parser.is_top_level_scope()
+      && let Pat::Ident(ident) = &declarator.name
+      && ident.id.sym.as_ref() == "module"
+    {
+      parser.build_info.module_argument = ModuleArgument::RspackModule;
+    }
+    None
+  }
+
+  fn pre_statement(&self, parser: &mut JavascriptParser, stmt: Statement) -> Option<bool> {
+    // Check if we're at top level scope
+    if parser.is_top_level_scope() {
+      match stmt {
+        Statement::Fn(fn_decl) => {
+          // Check for function declaration named "module"
+          if let Some(ident) = fn_decl.ident()
+            && ident.sym.as_ref() == "module"
+          {
+            parser.build_info.module_argument = ModuleArgument::RspackModule;
+          }
+        }
+        Statement::Class(class_decl) => {
+          // Check for class declaration named "module"
+          if let Some(ident) = class_decl.ident()
+            && ident.sym.as_ref() == "module"
+          {
+            parser.build_info.module_argument = ModuleArgument::RspackModule;
+          }
+        }
+        _ => {}
+      }
+    }
+    None
+  }
+
   fn call(
     &self,
     parser: &mut JavascriptParser,

tests/rspack-test/normalCases/parsing/webpack-module-collision/index.js

@@ -0,0 +1,20 @@
+it("should handle __webpack_module__.id when user declares 'module' variable", () => {
+	const moduleId = require("./with-var-collision").default;
+	expect(typeof moduleId).toMatch(/^(string|number)$/);
+});
+
+it("should handle __webpack_module__.id when user declares 'module' function", () => {
+	const moduleId = require("./with-function-collision").default;
+	expect(typeof moduleId).toMatch(/^(string|number)$/);
+});
+
+it("should handle __webpack_module__.id when user declares 'module' class", () => {
+	const moduleId = require("./with-class-collision").default;
+	expect(typeof moduleId).toMatch(/^(string|number)$/);
+});
+
+it("should handle __webpack_module__ when user declares 'module' variable", () => {
+	const mod = require("./with-module-var-collision").default;
+	expect(mod.exports).toBeTypeOf("object");
+	expect(typeof mod.id).toMatch(/^(string|number)$/);
+});

tests/rspack-test/normalCases/parsing/webpack-module-collision/with-class-collision.js

@@ -0,0 +1,8 @@
+console.log(__webpack_module__.id);
+class module {
+	constructor() {
+		this.name = 'test';
+	}
+}
+new module();
+export default __webpack_module__.id;

tests/rspack-test/normalCases/parsing/webpack-module-collision/with-function-collision.js

@@ -0,0 +1,6 @@
+console.log(__webpack_module__.id);
+function module() {
+	return 'test';
+}
+module();
+export default __webpack_module__.id;

tests/rspack-test/normalCases/parsing/webpack-module-collision/with-module-var-collision.js

@@ -0,0 +1,4 @@
+console.log(__webpack_module__);
+const module = 'layout';
+module;
+export default __webpack_module__;

tests/rspack-test/normalCases/parsing/webpack-module-collision/with-var-collision.js

@@ -0,0 +1,4 @@
+console.log(__webpack_module__.id);
+const module = 'layout';
+module;
+export default __webpack_module__.id;