fix: rename.stripBase with parent relative path (#247)

Author: sapphi-red

.changeset/heavy-candies-post.md

@@ -0,0 +1,21 @@
+---
+'vite-plugin-static-copy': patch
+---
+
+Fix `rename.stripBase` to work correctly with `../` paths
+
+Previously, `stripBase` counted `..` as directory segments, causing incorrect output paths when copying from parent directories.
+
+```js
+{ src: '../../src/pages/**/*.html', dest: 'dist/', rename: { stripBase: 2 } }
+```
+
+**Before**: `dist/src/pages/events/test.html`
+**After**: `dist/events/test.html`
+
+```js
+{ src: '../../src/pages/**/*.html', dest: 'dist/', rename: { stripBase: true } }
+```
+
+**Before**: `dist/src/pages/events/test.html`
+**After**: `dist/test.html`

src/options.ts

@@ -63,8 +63,12 @@ export type Target = {
    * ```js
    * // Copies src/pages/events/test.html to dist/events/test.html
    * { src: 'src/pages/**\/*.html', dest: 'dist/', rename: { stripBase: 2 } }
+   * // Copies ../../src/pages/events/test.html to dist/events/test.html
+   * { src: '../../src/pages/**\/*.html', dest: 'dist/', rename: { stripBase: 2 } }
    * // Copies src/pages/events/test.html to dist/test.html
    * { src: 'src/pages/**\/*.html', dest: 'dist/', rename: { stripBase: true } }
+   * // Copies ../../src/pages/events/test.html to dist/test.html
+   * { src: '../../src/pages/**\/*.html', dest: 'dist/', rename: { stripBase: true } }
    * // Copies src/pages/events/test.html to dist/src/pages/events/test2.html
    * { src: 'src/pages/**\/*.html', dest: 'dist/', rename: (name, ext) => `${name}2.${ext}` }
    * // Copies src/pages/events/test.html to dist/pages/events/test.html

src/utils.ts

@@ -244,11 +244,11 @@ export const collectCopyTargets = async (
 
       const { base, dir } = path.parse(relativeMatchedPath)
 
+      const dirClean = dir.replace(/^(?:\.\.\/)+/, '')
       let destDir: string
       if (!dir) {
         destDir = dest
       } else {
-        const dirClean = dir.replace(/^(?:\.\.\/)+/, '')
         const destClean = `${dest}/${dirClean}`.replace(/^\/+|\/+$/g, '')
         destDir = destClean
       }
@@ -258,7 +258,7 @@ export const collectCopyTargets = async (
         dest: path.join(
           destDir,
           rename
-            ? await renameTarget(base, rename, absoluteMatchedPath, dir)
+            ? await renameTarget(base, rename, absoluteMatchedPath, dirClean)
             : base,
         ),
         transform,

test/fixtures/vite.config.ts

@@ -200,6 +200,16 @@ export default defineConfig({
         },
         { src: 'eexist/**/*', dest: 'eexist', rename: { stripBase: 1 } },
         { src: 'eexist/**/*', dest: 'Eexist', rename: { stripBase: 1 } },
+        {
+          src: '../fixtures2/**/*.txt',
+          dest: 'fixture21',
+          rename: { stripBase: true },
+        },
+        {
+          src: '../fixtures2/**/*.txt',
+          dest: 'fixture22',
+          rename: { stripBase: 1 },
+        },
       ],
     }),
     viteStaticCopy({

test/fixtures2/nested/qux.txt

@@ -0,0 +1 @@
+qux

test/testcases.ts

@@ -212,6 +212,26 @@ export const testcases: Record<string, Testcase[]> = {
       src: './to-flat/deep/b.txt',
       dest: '/fixture20/b.txt',
     },
+    {
+      name: 'parent dir stripBase true flattens (1)',
+      src: '../fixtures2/baz.txt',
+      dest: '/fixture21/baz.txt',
+    },
+    {
+      name: 'parent dir stripBase true flattens (2)',
+      src: '../fixtures2/nested/qux.txt',
+      dest: '/fixture21/qux.txt',
+    },
+    {
+      name: 'parent dir stripBase 1 (1)',
+      src: '../fixtures2/baz.txt',
+      dest: '/fixture22/baz.txt',
+    },
+    {
+      name: 'parent dir stripBase 1 (2)',
+      src: '../fixtures2/nested/qux.txt',
+      dest: '/fixture22/nested/qux.txt',
+    },
     {
       name: 'parallel copy to same dir (1)',
       src: './eexist/a/1.txt',