Resolve copy paths relative to src in DomStack constructor

index.js

@@ -160,10 +160,11 @@ export class DomStack {
     this.#src = src
     this.#dest = dest
 
-    const copyDirs = opts?.copy ?? []
+    const copyDirs = (opts?.copy ?? []).map(dir => resolve(dir))
 
     this.opts = {
       ...opts,
+      copy: copyDirs,
       ignore: [
         ...DEFAULT_IGNORES,
         basename(dest),
@@ -172,12 +173,11 @@ export class DomStack {
       ],
     }
 
-    if (copyDirs && copyDirs.length > 0) {
+    if (copyDirs.length > 0) {
       const absDest = resolve(this.#dest)
       for (const copyDir of copyDirs) {
         // Copy dirs can be in the src dir (nested builds), but not in the dest dir.
-        const absCopyDir = resolve(copyDir)
-        const relToDest = relative(absDest, absCopyDir)
+        const relToDest = relative(absDest, copyDir)
         if (relToDest === '' || !relToDest.startsWith('..')) {
           throw new Error(`copyDir ${copyDir} is within the dest directory`)
         }

test-cases/constructor-copy-paths/index.test.js

@@ -0,0 +1,34 @@
+import { test } from 'node:test'
+import assert from 'node:assert'
+import { isAbsolute } from 'node:path'
+import { DomStack } from '../../index.js'
+
+test.describe('DomStack constructor - copy path resolution', () => {
+  test('resolves a relative copy path to an absolute path', () => {
+    const ds = new DomStack('/tmp/test-src', '/tmp/test-dest', {
+      copy: ['some-relative-copy-dir'],
+    })
+
+    assert.strictEqual(ds.opts.copy.length, 1, 'one copy entry')
+    assert.ok(isAbsolute(ds.opts.copy[0]), `copy path should be absolute, got: "${ds.opts.copy[0]}"`)
+  })
+
+  test('leaves an already-absolute copy path unchanged', () => {
+    const ds = new DomStack('/tmp/test-src', '/tmp/test-dest', {
+      copy: ['/absolute/copy/dir'],
+    })
+
+    assert.strictEqual(ds.opts.copy[0], '/absolute/copy/dir', 'absolute path is preserved')
+  })
+
+  test('resolves multiple mixed copy paths', () => {
+    const ds = new DomStack('/tmp/test-src', '/tmp/test-dest', {
+      copy: ['relative-dir', '/absolute/dir'],
+    })
+
+    assert.strictEqual(ds.opts.copy.length, 2, 'two copy entries')
+    for (const p of ds.opts.copy) {
+      assert.ok(isAbsolute(p), `each copy path should be absolute, got: "${p}"`)
+    }
+  })
+})