fix: set filename dynamically using a function

Author: Timeless0911

packages/core/src/config.ts

@@ -855,25 +855,29 @@ const composeAutoExtensionConfig = (
   };
 
   const hash = getHash();
-
-  const updatedConfig: EnvironmentConfig = {
-    output: {
-      filename: {
-        js: `[name]${hash}${jsExtension}`,
-        ...config.output?.filename,
-      },
-    },
-  };
-
-  const updatedJsExtension =
-    typeof updatedConfig.output?.filename?.js === 'string' &&
-    updatedConfig.output?.filename?.js
-      ? extname(updatedConfig.output.filename.js)
+  const defaultJsFilename = `[name]${hash}${jsExtension}`;
+  const userJsFilename = config.output?.filename?.js;
+
+  // will be returned to use in redirect feature
+  // only support string type for now since we can not get the return value of function
+  const finalJsExtension =
+    typeof userJsFilename === 'string' && userJsFilename
+      ? extname(userJsFilename)
       : jsExtension;
 
+  const finalConfig = userJsFilename
+    ? {}
+    : {
+        output: {
+          filename: {
+            js: defaultJsFilename,
+          },
+        },
+      };
+
   return {
-    config: updatedConfig,
-    jsExtension: updatedJsExtension,
+    config: finalConfig,
+    jsExtension: finalJsExtension,
     dtsExtension,
   };
 };

tests/integration/auto-extension/__fixtures__/src/image.png

@@ -1,5 +1,9 @@
 import { extname, join } from 'node:path';
-import { buildAndGetResults, queryContent } from 'test-helper';
+import {
+  buildAndGetResults,
+  generateFileTree,
+  queryContent,
+} from 'test-helper';
 import { describe, expect, test } from 'vitest';
 
 describe('autoExtension: true', () => {
@@ -52,6 +56,22 @@ describe('should respect output.filename.js and output.filenameHash to override
     expect(entryFiles.cjs1).toMatchInlineSnapshot(
       `"<ROOT>/tests/integration/auto-extension/type-commonjs/config-override/dist/cjs-override-filename-hash/index.df02628a.js"`,
     );
+
+    // override different file types with function
+    const fileTree = generateFileTree(
+      join(fixturePath, './dist/cjs-override-filename-function'),
+    );
+    expect(fileTree).toMatchInlineSnapshot(`
+      {
+        "bar-image.js": "<ROOT>/tests/integration/auto-extension/type-commonjs/config-override/dist/cjs-override-filename-function/bar-image.js",
+        "bar-index.js": "<ROOT>/tests/integration/auto-extension/type-commonjs/config-override/dist/cjs-override-filename-function/bar-index.js",
+        "static": {
+          "image": {
+            "foo-image.png": "<ROOT>/tests/integration/auto-extension/type-commonjs/config-override/dist/cjs-override-filename-function/static/image/foo-image.png",
+          },
+        },
+      }
+    `);
   });
 
   test('type is module', async () => {
@@ -71,6 +91,22 @@ describe('should respect output.filename.js and output.filenameHash to override
     expect(entryFiles.cjs1).toMatchInlineSnapshot(
       `"<ROOT>/tests/integration/auto-extension/type-module/config-override/dist/cjs-override-filename-hash/index.df02628a.js"`,
     );
+
+    // override different file types with function
+    const fileTree = generateFileTree(
+      join(fixturePath, './dist/esm-override-filename-function'),
+    );
+    expect(fileTree).toMatchInlineSnapshot(`
+      {
+        "bar-image.js": "<ROOT>/tests/integration/auto-extension/type-module/config-override/dist/esm-override-filename-function/bar-image.js",
+        "bar-index.js": "<ROOT>/tests/integration/auto-extension/type-module/config-override/dist/esm-override-filename-function/bar-index.js",
+        "static": {
+          "image": {
+            "foo-image.png": "<ROOT>/tests/integration/auto-extension/type-module/config-override/dist/esm-override-filename-function/static/image/foo-image.png",
+          },
+        },
+      }
+    `);
   });
 });
 

tests/integration/auto-extension/index.test.ts

@@ -41,6 +41,30 @@ export default defineConfig({
         },
       },
     }),
+    generateBundleCjsConfig({
+      output: {
+        filename: {
+          image: () => {
+            return 'foo-[name][ext]';
+          },
+          js: () => {
+            return 'bar-[name].js';
+          },
+        },
+        distPath: {
+          root: './dist/cjs-override-filename-function',
+        },
+      },
+      bundle: false,
+      source: {
+        entry: {
+          index: [
+            '../../__fixtures__/src/index.ts',
+            '../../__fixtures__/src/image.png',
+          ],
+        },
+      },
+    }),
   ],
   source: {
     entry: {

tests/integration/auto-extension/type-commonjs/config-override/rslib.config.ts

@@ -41,6 +41,30 @@ export default defineConfig({
         },
       },
     }),
+    generateBundleEsmConfig({
+      output: {
+        filename: {
+          image: () => {
+            return 'foo-[name][ext]';
+          },
+          js: () => {
+            return 'bar-[name].js';
+          },
+        },
+        distPath: {
+          root: './dist/esm-override-filename-function',
+        },
+      },
+      bundle: false,
+      source: {
+        entry: {
+          index: [
+            '../../__fixtures__/src/index.ts',
+            '../../__fixtures__/src/image.png',
+          ],
+        },
+      },
+    }),
   ],
   source: {
     entry: {