fix(legacy)!: should rename x.[hash].js to x-legacy.[hash].js (#11599)

Co-authored-by: bluwy <[email protected]>

Author: aleen42

packages/plugin-legacy/src/index.ts

@@ -337,6 +337,11 @@ function viteLegacyPlugin(options: Options = {}): Plugin[] {
           if (fileName.includes('[name]')) {
             // [name]-[hash].[format] -> [name]-legacy-[hash].[format]
             fileName = fileName.replace('[name]', '[name]-legacy')
+          } else if (fileName.includes('[hash]')) {
+            // custom[hash].[format] -> [name]-legacy[hash].[format]
+            // custom-[hash].[format] -> [name]-legacy-[hash].[format]
+            // custom.[hash].[format] -> [name]-legacy.[hash].[format]
+            fileName = fileName.replace(/[.-]?\[hash\]/, '-legacy$&')
           } else {
             // entry.js -> entry-legacy.js
             fileName = fileName.replace(/(.+)\.(.+)/, '$1-legacy.$2')

playground/legacy/__tests__/legacy.spec.ts

@@ -97,6 +97,13 @@ describe.runIf(isBuild)('build', () => {
     expect(manifest['../../vite/legacy-polyfills-legacy'].src).toBe(
       '../../vite/legacy-polyfills-legacy',
     )
+    expect(manifest['custom0-legacy.js'].file).toMatch(
+      /chunk-X-legacy.\w{8}.js/,
+    )
+    expect(manifest['custom1-legacy.js'].file).toMatch(
+      /chunk-X-legacy-\w{8}.js/,
+    )
+    expect(manifest['custom2-legacy.js'].file).toMatch(/chunk-X-legacy\w{8}.js/)
     // modern polyfill
     expect(manifest['../../vite/legacy-polyfills']).toBeDefined()
     expect(manifest['../../vite/legacy-polyfills'].src).toBe(

playground/legacy/custom0.js

@@ -0,0 +1 @@
+export const foo = 'bar'

playground/legacy/custom1.js

@@ -0,0 +1 @@
+export const foo = 'bar'

playground/legacy/custom2.js

@@ -0,0 +1 @@
+export const foo = 'bar'

playground/legacy/main.js

@@ -3,6 +3,9 @@ import viteSvgPath from './vite.svg'
 import MyWorker from './worker?worker'
 
 async function run() {
+  await import('./custom0.js')
+  await import('./custom1.js')
+  await import('./custom2.js')
   const { fn } = await import('./async.js')
   fn()
 }

playground/legacy/vite.config.js

@@ -25,6 +25,10 @@ export default defineConfig({
         chunkFileNames(chunkInfo) {
           if (chunkInfo.name === 'immutable-chunk') {
             return `assets/${chunkInfo.name}.js`
+          } else if (/custom\d/.test(chunkInfo.name)) {
+            return `assets/chunk-X${
+              ['.', '-', ''][/custom(\d)/.exec(chunkInfo.name)[1]]
+            }[hash].js`
           }
           return `assets/chunk-[name].[hash].js`
         },