fix: the moduleIds should be deterministic if the env is production and format is mf

Author: 2heal1

.changeset/smooth-parents-pull.md

@@ -0,0 +1,5 @@
+---
+'@rslib/core': patch
+---
+
+fix: the moduleIds should be deterministic if the env is production and format is mf

packages/core/src/config.ts

@@ -53,6 +53,7 @@ import {
   calcLongestCommonPath,
   checkMFPlugin,
   color,
+  composeMFPlugin,
   isEmptyObject,
   isObject,
   nodeBuiltInModules,
@@ -1077,6 +1078,7 @@ const composeExternalHelpersConfig = (
 
 async function composeLibRsbuildConfig(config: LibConfig, configPath: string) {
   checkMFPlugin(config);
+  const mfConfig = composeMFPlugin(config);
   const rootPath = dirname(configPath);
   const pkgJson = readPackageJson(rootPath);
   const { compilerOptions } = await loadTsconfig(
@@ -1160,6 +1162,7 @@ async function composeLibRsbuildConfig(config: LibConfig, configPath: string) {
   );
 
   return mergeRsbuildConfig(
+    mfConfig,
     formatConfig,
     shimsConfig,
     externalHelpersConfig,

packages/core/src/utils/helper.ts

@@ -1,7 +1,7 @@
 import fs from 'node:fs';
 import fsP from 'node:fs/promises';
 import path from 'node:path';
-import type { RsbuildPlugins } from '@rsbuild/core';
+import type { RsbuildConfig, RsbuildPlugins } from '@rsbuild/core';
 import color from 'picocolors';
 
 import type { LibConfig, PkgJson } from '../types';
@@ -201,4 +201,20 @@ export function checkMFPlugin(config: LibConfig): boolean {
   return added;
 }
 
+export function composeMFPlugin(config: LibConfig): RsbuildConfig {
+  if (config.format !== 'mf' || process.env.NODE_ENV !== 'production') {
+    return {};
+  }
+
+  return {
+    tools: {
+      rspack: {
+        optimization: {
+          moduleIds: 'deterministic',
+        },
+      },
+    },
+  };
+}
+
 export { color };

tests/integration/minify/index.test.ts

@@ -59,3 +59,31 @@ test('minify is disabled by the user, bar and baz should not be minified', async
     }]);"
   `);
 });
+
+test('enable minify and set NODE_ENV to production, the moduleIds should be the deterministic', async () => {
+  const fixturePath = join(__dirname, 'mf/config');
+  const prevNodeEnv = process.env.NODE_ENV;
+  process.env.NODE_ENV = 'production';
+  const { mfExposeEntry } = await buildAndGetResults({ fixturePath });
+  expect(mfExposeEntry).toMatchInlineSnapshot(`
+    ""use strict";
+    (globalThis['disable_minify'] = globalThis['disable_minify'] || []).push([["249"], {
+    "163": (function (__unused_webpack_module, __webpack_exports__, __webpack_require__) {
+    __webpack_require__.r(__webpack_exports__);
+    __webpack_require__.d(__webpack_exports__, {
+      foo: function() { return foo; }
+    });
+    const foo = ()=>{};
+    const bar = ()=>{};
+    const baz = ()=>{
+        return bar();
+    };
+
+
+    }),
+
+    }]);"
+  `);
+
+  process.env.NODE_ENV = prevNodeEnv;
+});