fix: resolve include patterns to absolute paths in getAutoIncludes

Prevents adding duplicate patches include when the user specifies
the pattern with a relative prefix (./patches/**) or absolute path.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: sorccu

packages/cli/src/services/__tests__/util.spec.ts

@@ -32,6 +32,7 @@ describe('util', () => {
   })
 
   describe('getAutoIncludes()', () => {
+    const basePath = '/project'
     const makePm = (name: string): PackageManager => ({
       name,
       representativeLockfile: undefined,
@@ -43,27 +44,37 @@ describe('util', () => {
     })
 
     it('should return patches/*.patch for pnpm', () => {
-      const result = getAutoIncludes(makePm('pnpm'), [])
+      const result = getAutoIncludes(basePath, makePm('pnpm'), [])
       expect(result).toEqual(['patches/*.patch'])
     })
 
     it('should return empty for npm', () => {
-      const result = getAutoIncludes(makePm('npm'), [])
+      const result = getAutoIncludes(basePath, makePm('npm'), [])
       expect(result).toEqual([])
     })
 
     it('should return empty for yarn', () => {
-      const result = getAutoIncludes(makePm('yarn'), [])
+      const result = getAutoIncludes(basePath, makePm('yarn'), [])
       expect(result).toEqual([])
     })
 
     it('should skip when user already includes patches/*.patch', () => {
-      const result = getAutoIncludes(makePm('pnpm'), ['patches/*.patch'])
+      const result = getAutoIncludes(basePath, makePm('pnpm'), ['patches/*.patch'])
       expect(result).toEqual([])
     })
 
     it('should skip when user already includes a patches/ subpath', () => {
-      const result = getAutoIncludes(makePm('pnpm'), ['patches/some-patch.patch'])
+      const result = getAutoIncludes(basePath, makePm('pnpm'), ['patches/some-patch.patch'])
+      expect(result).toEqual([])
+    })
+
+    it('should skip when user includes ./patches/**', () => {
+      const result = getAutoIncludes(basePath, makePm('pnpm'), ['./patches/**'])
+      expect(result).toEqual([])
+    })
+
+    it('should skip when user includes absolute patches path', () => {
+      const result = getAutoIncludes(basePath, makePm('pnpm'), ['/project/patches/**'])
       expect(result).toEqual([])
     })
   })

packages/cli/src/services/util.ts

@@ -144,14 +144,16 @@ export function normalizeVersion (v?: string | undefined): string | undefined {
 }
 
 export function getAutoIncludes (
+  basePath: string,
   packageManager: PackageManager,
   existingIncludes: string[],
 ): string[] {
   const autoIncludes: string[] = []
 
   if (packageManager.name === 'pnpm') {
     const patchesPattern = 'patches/*.patch'
-    const alreadyIncluded = existingIncludes.some(p => p === patchesPattern || p.startsWith('patches/'))
+    const patchesDir = path.join(basePath, 'patches')
+    const alreadyIncluded = existingIncludes.some(p => path.resolve(basePath, p).startsWith(patchesDir))
     if (!alreadyIncluded) {
       autoIncludes.push(patchesPattern)
     }
@@ -212,7 +214,7 @@ export async function bundlePlayWrightProject (
     }
   }
 
-  const autoIncludes = getAutoIncludes(Session.packageManager, include)
+  const autoIncludes = getAutoIncludes(Session.basePath!, Session.packageManager, include)
   const effectiveIncludes = [...include, ...autoIncludes]
 
   const includedFiles = await findFilesWithPattern(