fix: inject middleware correctly when there's multiple plugin instances (#186)

Author: sapphi-red

.changeset/brave-phones-read.md

@@ -0,0 +1,5 @@
+---
+'vite-plugin-static-copy': patch
+---
+
+fix a bug that the content was not sent when multiple vite-plugin-static-copy instance was used

src/serve.ts

@@ -123,14 +123,14 @@ export const servePlugin = ({
       return () => {
         // insert serveStaticCopyMiddleware before viteServePublicMiddleware
         // if viteServePublicMiddleware didn't exist use transformMiddleware instead
-        middlewares.use(serveStaticCopyMiddleware(config, fileMap))
+        const middleware = serveStaticCopyMiddleware(config, fileMap)
+        middlewares.use(middleware)
         const targetMiddlewareIndex = findMiddlewareIndex(middlewares.stack, [
           'viteServePublicMiddleware',
           'viteTransformMiddleware',
         ])
-        const serveStaticCopyMiddlewareIndex = findMiddlewareIndex(
-          middlewares.stack,
-          'viteServeStaticCopyMiddleware',
+        const serveStaticCopyMiddlewareIndex = middlewares.stack.findIndex(
+          (item) => item.handle === middleware,
         )
 
         const serveStaticCopyMiddlewareItem = middlewares.stack.splice(

test/fixtures/vite.config.ts

@@ -148,5 +148,13 @@ export default defineConfig({
         { src: 'eexist/*', dest: 'Eexist' },
       ],
     }),
+    viteStaticCopy({
+      targets: [
+        {
+          src: 'foo.txt',
+          dest: 'fixture1-1',
+        },
+      ],
+    }),
   ],
 })

test/testcases.ts

@@ -151,6 +151,11 @@ export const testcases: Record<string, Testcase[]> = {
       src: './eexist/b/1.txt',
       dest: '/eexist/b/1.txt',
     },
+    {
+      name: 'multiple plugin instance',
+      src: './foo.txt',
+      dest: '/fixture1-1/foo.txt',
+    },
   ],
   'vite.absolute.config.ts': [
     {