escape cwd before adding to pattern

closes #78
SuperchupuDev · Dec 23, 2024 · 34d6229 · 34d6229
1 parent d51e2b9
commit 34d6229
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 2 deletions.
diff --git a/src/index.ts b/src/index.ts
@@ -1,7 +1,7 @@
 import path, { posix } from 'node:path';
 import { type Options as FdirOptions, fdir } from 'fdir';
 import picomatch from 'picomatch';
-import { isDynamicPattern } from './utils.ts';
+import { escapePath, isDynamicPattern } from './utils.ts';
 
 export interface GlobOptions {
   absolute?: boolean;
@@ -41,7 +41,7 @@ function normalizePattern(
   }
 
   if (path.isAbsolute(result.replace(/\\(?=[()[\]{}!*+?@|])/g, ''))) {
-    result = posix.relative(cwd, result);
+    result = posix.relative(escapePath(cwd), result);
   } else {
     result = posix.normalize(result);
   }

diff --git a/test/index.test.ts b/test/index.test.ts
@@ -14,6 +14,7 @@ const fixture = await createFixture({
     'b.txt': 'b'
   },
   '.a/a/a.txt': 'a',
+  '.[a]/a.txt': 'a',
   '.deep/a/a/a.txt': 'a',
   '.symlink': {
     file: ({ symlink }) => symlink('../a/a.txt'),
@@ -142,6 +143,15 @@ test('fully handle absolute patterns', async () => {
   assert.deepEqual(files.sort(), ['../b/a.txt', 'a.txt']);
 });
 
+test('escaped absolute patterns', async () => {
+  const files = await glob({
+    patterns: [`${cwd.replaceAll('\\', '/')}.\\[a\\]/a.txt`],
+    absolute: true,
+    cwd: path.join(cwd, '.[a]')
+  });
+  assert.deepEqual(files.sort(), [`${cwd.replaceAll('\\', '/')}.[a]/a.txt`]);
+});
+
 test('leading ../', async () => {
   const files = await glob({ patterns: ['../b/*.txt'], cwd: path.join(cwd, 'a') });
   assert.deepEqual(files.sort(), ['../b/a.txt', '../b/b.txt']);