fix: user externals should have higher priority

Author: fi3ework

packages/core/src/config.ts

@@ -1391,14 +1391,14 @@ async function composeLibRsbuildConfig(
     // #region Externals configs
     // The order of the externals config should come in the following order:
     // 1. `externalsWarnConfig` should come before other externals config to touch the externalized modules first.
-    // 2. The externals config in `bundlelessExternalConfig` should present after other externals config as
+    // 2. `userExternalsConfig` should present later to override the externals config of the ahead ones.
+    // 3. The externals config in `bundlelessExternalConfig` should present after other externals config as
     //    it relies on other externals config to bail out the externalized modules first then resolve
     //    the correct path for relative imports.
-    // 3. `userExternalsConfig` should present later to override the externals config of the ahead ones.
     externalsWarnConfig,
+    userExternalsConfig,
     autoExternalConfig,
     targetExternalsConfig,
-    userExternalsConfig,
     bundlelessExternalConfig,
     // #endregion
     entryConfig,

pnpm-lock.yaml

@@ -74,3 +74,20 @@ test('require ESM from CJS', async () => {
   const bazValue = await baz();
   expect(bazValue).toBe('baz');
 });
+
+test('user externals', async () => {
+  // Ensure the priority of user externals higher than others.
+  // - memfs: userExternalsConfig > targetExternalsConfig
+  // - lodash-es/zip: userExternalsConfig > autoExternalConfig
+  const fixturePath = join(__dirname, 'user-externals');
+  const { entries } = await buildAndGetResults({ fixturePath });
+  expect(entries.esm).toMatchInlineSnapshot(
+    `
+    "import * as __WEBPACK_EXTERNAL_MODULE_memfs__ from "memfs";
+    import * as __WEBPACK_EXTERNAL_MODULE_lodash__ from "lodash";
+    import * as __WEBPACK_EXTERNAL_MODULE_lodash_es_zip_b8981481__ from "lodash-es/zip";
+    console.log(__WEBPACK_EXTERNAL_MODULE_memfs__["default"], __WEBPACK_EXTERNAL_MODULE_lodash__["default"].add, __WEBPACK_EXTERNAL_MODULE_lodash_es_zip_b8981481__["default"]);
+    "
+  `,
+  );
+});

tests/integration/externals/index.test.ts

@@ -0,0 +1,12 @@
+{
+  "name": "externals-user-externals-test",
+  "version": "1.0.0",
+  "private": true,
+  "type": "module",
+  "dependencies": {
+    "lodash": "^4.17.21"
+  },
+  "devDependencies": {
+    "@types/lodash": "^4.17.14"
+  }
+}

tests/integration/externals/user-externals/package.json

@@ -0,0 +1,9 @@
+import { defineConfig } from '@rslib/core';
+import { generateBundleEsmConfig } from 'test-helper';
+
+export default defineConfig({
+  lib: [generateBundleEsmConfig()],
+  output: {
+    externals: { 'lodash/zip': 'lodash-es/zip', 'node:fs': 'memfs' },
+  },
+});

tests/integration/externals/user-externals/rslib.config.ts

@@ -0,0 +1,5 @@
+import fs from 'node:fs';
+import lodash from 'lodash';
+import zip from 'lodash/zip';
+
+console.log(fs, lodash.add, zip);