perf: parallel find new name in concatenated code generation

Author: LingyuCoder

crates/rspack_core/src/concatenated_module.rs

@@ -1116,7 +1116,7 @@ impl JsPlugin {
           let context = compilation.options.context.clone();
           let readable_identifier = module.readable_identifier(&context).to_string();
           let splitted_readable_identifier = split_readable_identifier(&readable_identifier);
-          let new_name = find_new_name(name, &all_used_names, &splitted_readable_identifier);
+          let new_name = find_new_name("", name, &all_used_names, &splitted_readable_identifier);
 
           for identifier in refs.iter() {
             let span = identifier.id.span();

crates/rspack_plugin_javascript/src/plugin/mod.rs

@@ -330,7 +330,7 @@ Object {
         main.js,
       ],
       filteredModules: undefined,
-      hash: cfe10b32fa6df1ab,
+      hash: f2679505795e0dc5,
       id: 909,
       idHints: Array [],
       initial: true,
@@ -718,7 +718,7 @@ Object {
   errorsCount: 0,
   filteredAssets: undefined,
   filteredModules: undefined,
-  hash: de56093745c27934,
+  hash: 2252b7b6fa0bc796,
   modules: Array [
     Object {
       assets: Array [],

packages/rspack-test-tools/tests/__snapshots__/StatsAPI.test.js.snap

@@ -58,7 +58,7 @@ runtime modules xx KiB
     [no exports]
     [used exports unknown]
   
-Rspack compiled successfully (cb468c46ad9d7f18)
+Rspack compiled successfully (b6a6c74437c67e0f)
 `;
 
 exports[`statsOutput statsOutput/builtin-swc-loader-parse-error should print correct stats for 1`] = `

packages/rspack-test-tools/tests/__snapshots__/StatsOutput.test.js.snap

@@ -4,15 +4,15 @@
 "./index.js": (function () {
 
 ;// CONCATENATED MODULE: ./answer.js
-const answer = 103330;
+const _0_answer = 103330;
 
 ;// CONCATENATED MODULE: ./index.js
 
-answer;
+_0_answer;
 
-function index_answer() {}
+function _1_answer() {}
 
-index_answer();
+_1_answer();
 
 function test() {}
 

packages/rspack-test-tools/tests/builtinCases/samples/concatenate-modules/__snapshots__/output.snap.txt

@@ -47,12 +47,14 @@ module.exports = {
 						const jsContent = assets[js].source().toString();
 
 						const preseveImport =
-							/import\simg_namespaceObject\sfrom ['"]\.\/static\/img\/img\.png['"]/.test(
+							/import\s_0_img_namespaceObject\sfrom ['"]\.\/static\/img\/img\.png['"]/.test(
 								jsContent
 							);
 						assert(preseveImport);
 						const hasExports =
-							/export\s{\simg_namespaceObject\sas\sdefault\s}/.test(jsContent);
+							/export\s{\s_0_img_namespaceObject\sas\sdefault\s}/.test(
+								jsContent
+							);
 						assert(hasExports);
 					});
 				});

packages/rspack-test-tools/tests/configCases/library/modern-module-asset-entry/rspack.config.js

@@ -30,7 +30,7 @@ module.exports = {
 				compilation.hooks.afterProcessAssets.tap("testcase", assets => {
 					const bundle = Object.values(assets)[0]._value;
 					expect(bundle)
-						.toContain(`var __webpack_exports__cjsInterop = (foo_default());
+						.toContain(`var __webpack_exports__cjsInterop = (_1_foo_default());
 export { external_module as defaultImport, namedImport, __webpack_exports__cjsInterop as cjsInterop };`);
 					expect(bundle).toContain(
 						'import external_module, { namedImport } from "external-module";'

packages/rspack-test-tools/tests/configCases/library/modern-module-export-char/rspack.config.js

@@ -62,13 +62,13 @@ __webpack_require__.o = (obj, prop) => (Object.prototype.hasOwnProperty.call(obj
 const foo = 'foo'
 
 // EXTERNAL MODULE: ./e/bar.cjs
-var bar = __webpack_require__(997);
-var bar_default = /*#__PURE__*/__webpack_require__.n(bar);
+var _1_bar = __webpack_require__(997);
+var _1_bar_default = /*#__PURE__*/__webpack_require__.n(_1_bar);
 ;// CONCATENATED MODULE: ./e/index.js
 
 
 
 
 
-var __webpack_exports__bar = (bar_default());
+var __webpack_exports__bar = (_1_bar_default());
 export { foo, __webpack_exports__bar as bar };

packages/rspack-test-tools/tests/configCases/library/modern-module-force-concaten/__snapshot__/e.js.txt

@@ -67,16 +67,16 @@ __webpack_require__.o = (obj, prop) => (Object.prototype.hasOwnProperty.call(obj
 /************************************************************************/
 
 // EXTERNAL MODULE: ./f/bar.cjs
-var bar = __webpack_require__(441);
-var bar_default = /*#__PURE__*/__webpack_require__.n(bar);
+var _0_bar = __webpack_require__(441);
+var _0_bar_default = /*#__PURE__*/__webpack_require__.n(_0_bar);
 ;// CONCATENATED MODULE: ./f/foo.js
 const foo = 'foo'
 
 ;// CONCATENATED MODULE: ./f/index.js
 
 
 
-const value = foo + (bar_default())
+const value = foo + (_0_bar_default())
 
 
 

packages/rspack-test-tools/tests/configCases/library/modern-module-force-concaten/__snapshot__/f.js.txt

@@ -60,12 +60,12 @@ __webpack_require__.o = (obj, prop) => (Object.prototype.hasOwnProperty.call(obj
 /************************************************************************/
 
 // EXTERNAL MODULE: ./g/foo.js
-var foo = __webpack_require__(974);
-var foo_default = /*#__PURE__*/__webpack_require__.n(foo);
+var _0_foo = __webpack_require__(974);
+var _0_foo_default = /*#__PURE__*/__webpack_require__.n(_0_foo);
 ;// CONCATENATED MODULE: ./g/index.js
 
 
 
 
-var __webpack_exports__foo = (foo_default());
+var __webpack_exports__foo = (_0_foo_default());
 export { __webpack_exports__foo as foo };