fix: cjs bare exports expression detect

crates/rspack_plugin_javascript/src/visitors/dependency/common_js_export_scanner.rs

@@ -96,6 +96,13 @@ impl Visit for CommonJsExportDependencyScanner<'_> {
     {
       return;
     }
+    // var a = exports/module.exports/this;
+    // Object.setPrototypeOf(exports/module.exports/this, a);
+    // ...
+    if self.is_exports_or_module_exports_or_this_expr(expr) {
+      self.bailout();
+      return;
+    }
     expr.visit_children_with(self);
   }
 
@@ -116,10 +123,14 @@ impl Visit for CommonJsExportDependencyScanner<'_> {
           self.stmt_level == 1 && self.last_stmt_is_expr_stmt,
           &assign_expr.right,
         );
+        assign_expr.right.visit_children_with(self);
+        return;
       }
       // exports.xxx = 1;
       if self.is_exports_member_expr_start(expr) {
         self.enable();
+        assign_expr.right.visit_children_with(self);
+        return;
       }
       if self.is_exports_or_module_exports_or_this_expr(expr) {
         self.enable();
@@ -135,14 +146,10 @@ impl Visit for CommonJsExportDependencyScanner<'_> {
           // this = {};
           self.bailout();
         }
+        assign_expr.right.visit_children_with(self);
+        return;
       }
     }
-    // var a = exports;
-    // var a = module.exports;
-    // var a = this;
-    if self.is_exports_or_module_exports_or_this_expr(&assign_expr.right) {
-      self.bailout();
-    }
     assign_expr.visit_children_with(self);
   }
 
@@ -151,15 +158,32 @@ impl Visit for CommonJsExportDependencyScanner<'_> {
       // Object.defineProperty(exports, "__esModule", { value: true });
       // Object.defineProperty(module.exports, "__esModule", { value: true });
       // Object.defineProperty(this, "__esModule", { value: true });
-      if expr_matcher::is_object_define_property(expr) && let Some(ExprOrSpread { expr, .. }) = call_expr.args.get(0) && let Some(ExprOrSpread { expr: box Expr::Lit(Lit::Str(str)), .. }) = call_expr.args.get(1) && &str.value == "__esModule" && let Some(value) = get_value_of_property_description(&call_expr.args.get(2)) &&  self.is_exports_or_module_exports_or_this_expr(expr) {
+      if expr_matcher::is_object_define_property(expr)
+      && let Some(ExprOrSpread { expr, .. }) = call_expr.args.get(0)
+      && self.is_exports_or_module_exports_or_this_expr(expr)
+      && let Some(arg2) = call_expr.args.get(2) {
         self.enable();
-        self.check_namespace(self.stmt_level == 1, value);
+
+        if let Some(ExprOrSpread { expr: box Expr::Lit(Lit::Str(str)), .. }) = call_expr.args.get(1)
+        && &str.value == "__esModule"
+        && let Some(value) = get_value_of_property_description(arg2) {
+          self.check_namespace(self.stmt_level == 1, value);
+        }
+
+        self.enter_call += 1;
+        arg2.visit_children_with(self);
+        self.enter_call -= 1;
+        return;
       }
       // exports()
       // module.exports()
       // this()
       if self.is_exports_or_module_exports_or_this_expr(expr) {
         self.bailout();
+        self.enter_call += 1;
+        call_expr.args.visit_children_with(self);
+        self.enter_call -= 1;
+        return;
       }
     }
     self.enter_call += 1;
@@ -241,13 +265,11 @@ impl<'a> CommonJsExportDependencyScanner<'a> {
   }
 }
 
-fn get_value_of_property_description<'a>(
-  expr_or_spread: &Option<&'a ExprOrSpread>,
-) -> Option<&'a Expr> {
-  if let Some(ExprOrSpread {
+fn get_value_of_property_description(expr_or_spread: &ExprOrSpread) -> Option<&Expr> {
+  if let ExprOrSpread {
     expr: box Expr::Object(ObjectLit { props, .. }),
     ..
-  }) = expr_or_spread
+  } = expr_or_spread
   {
     for prop in props {
       if let PropOrSpread::Prop(prop) = prop && let Prop::KeyValue(key_value_prop) = &**prop && let PropName::Ident(ident) = &key_value_prop.key && &ident.sym == "value" {

webpack-test/cases/json/default-default-with-concatenation/test.filter.js

@@ -1,4 +1,5 @@
+const { FilteredStatus } = require("../../../lib/util/filterUtil")
 
-module.exports = () => {return false}
+module.exports = () => {return [FilteredStatus.PARTIAL_PASS, "https://github.com/web-infra-dev/rspack/issues/3823"]}
 
 

webpack-test/cases/json/import-assertions-type-json/test.filter.js

@@ -1,4 +1,4 @@
 
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4358"}
 
 

webpack-test/cases/json/import-by-name-with-concatenation/test.filter.js

@@ -1,4 +1,5 @@
+const { FilteredStatus } = require("../../../lib/util/filterUtil")
 
-module.exports = () => {return false}
+module.exports = () => {return [FilteredStatus.PARTIAL_PASS, "https://github.com/web-infra-dev/rspack/issues/3823, https://github.com/web-infra-dev/rspack/issues/4323"]}
 
 

webpack-test/cases/json/import-by-name/test.filter.js

@@ -1,4 +1,4 @@
 
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4323"}
 
 

webpack-test/cases/json/import-with-default-with-concatenation/test.filter.js

@@ -1,4 +1,5 @@
+const { FilteredStatus } = require("../../../lib/util/filterUtil")
 
-module.exports = () => {return false}
+module.exports = () => {return [FilteredStatus.PARTIAL_PASS, "https://github.com/web-infra-dev/rspack/issues/3823"]}
 
 

webpack-test/cases/json/reexport/test.filter.js

@@ -1,4 +1,5 @@
+const { FilteredStatus } = require("../../../lib/util/filterUtil")
 
-module.exports = () => {return false}
+module.exports = () => {return [FilteredStatus.PARTIAL_PASS, "https://github.com/web-infra-dev/rspack/issues/4323"]}
 
 

webpack-test/cases/large/big-assets/index.js

@@ -1,4 +1,4 @@
-const createHash = require("../../../../lib/util/hash/xxhash64");
+const createHash = require("@rspack/core/dist/util/hash/xxhash64");
 const fs = require("fs");
 
 const h = url => {

webpack-test/cases/large/big-assets/test.filter.js

@@ -5,6 +5,6 @@ module.exports = function (config) {
 };
 
 */
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/3180"}
 
 

webpack-test/cases/large/many-replacements/test.filter.js

@@ -5,6 +5,6 @@ module.exports = function (config) {
 };
 
 */
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4396"}
 
 

webpack-test/cases/loaders/coffee-loader/test.filter.js

@@ -1,4 +1,4 @@
 
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4396"}
 
 

webpack-test/cases/loaders/context/test.filter.js

@@ -7,6 +7,6 @@ module.exports = config => {
 };
 
 */
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4397"}
 
 

webpack-test/cases/loaders/import-module/test.filter.js

@@ -5,6 +5,6 @@ module.exports = config => {
 };
 
 */
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4398"}
 
 

webpack-test/cases/loaders/pug-loader/test.filter.js

@@ -1,12 +1,5 @@
-
-/*
 const supportsRequireInModule = require("../../../helpers/supportsRequireInModule");
 
 module.exports = config => {
 	return !config.module || supportsRequireInModule();
 };
-
-*/
-module.exports = () => {return true}
-
-

webpack-test/cases/loaders/query/test.filter.js

@@ -1,5 +1,3 @@
+const { FilteredStatus } = require("../../../lib/util/filterUtil")
 
-// Didn't turn on every cases
-module.exports = () => {return true}
-
-
+module.exports = () => {return [FilteredStatus.PARTIAL_PASS, "https://github.com/web-infra-dev/rspack/issues/3180, https://github.com/web-infra-dev/rspack/issues/4397"]}

webpack-test/cases/loaders/resolve/test.filter.js

@@ -1,4 +1,4 @@
 
-module.exports = () => {return "Rspack does not support resolving a virtual resource with only loader available, see: query test"}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/3180"}
 
 

webpack-test/cases/loaders/utils/test.filter.js

@@ -1,4 +1,4 @@
 
-module.exports = () => {return "blocked by context support for loader"}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4397"}
 
 

webpack-test/cases/nonce/set-nonce/test.filter.js

@@ -1,4 +1,4 @@
 
-module.exports = () => {return false}
+module.exports = () => {return "https://github.com/web-infra-dev/rspack/issues/4401"}
 
 

webpack-test/scripts/test-metric-util.js

@@ -60,7 +60,8 @@ function renderAllTestsToMarkdown(jsonObj) {
 	const testResults = jsonObj["testResults"];
 	return testResults
 		.flatMap(testSuite => testSuite.assertionResults)
-		.map((test, index) => `${index + 1}. ${renderTestToMarkdown(test.fullName)}`)
+		// use 1\. to break GitHub markdown list auto ordering
+		.map((test, index) => `${index + 1}\. ${renderTestToMarkdown(test.fullName)}`)
 		.join('\n')
 }