fix: escape glob path correctly

Author: benyap

src/steps/getFilesToProcess.ts

@@ -1,5 +1,5 @@
 import { resolve } from "path";
-import { sync } from "fast-glob";
+import { convertPathToPattern, sync } from "fast-glob";
 
 import { normalizePath } from "~/utils/path";
 
@@ -10,7 +10,7 @@ import { normalizePath } from "~/utils/path";
  * @param extensions A list of extensions to match.
  */
 export function getFilesToProcess(outPath: string, extensions: string[]) {
-  const normalizedOutPath = normalizePath(outPath);
+  const normalizedOutPath = convertPathToPattern(normalizePath(outPath));
 
   let glob = "*";
   if (extensions.length === 1) glob = `*.${extensions[0]}`;

test/fixtures/(special files*)/file \slash.js

@@ -0,0 +1 @@
+export {};

test/fixtures/(special files*)/file with (parentheses).js

@@ -0,0 +1 @@
+export {};

test/fixtures/(special files*)/file with space.js

@@ -0,0 +1 @@
+export {};

test/steps/getFilesToProcess.test.ts

@@ -9,20 +9,47 @@ describe("steps/getFilesToProcess", () => {
     const cwd = process.cwd();
     const testDirectory = join(cwd, "test/fixtures/files");
     const files = getFilesToProcess(testDirectory, ["js"]);
-    expect(files).toHaveLength(1);
+    expect(files.map((path) => path.replace(cwd, ""))).toMatchInlineSnapshot(`
+      [
+        "/test/fixtures/files/file.js",
+      ]
+    `);
   });
 
   it("gets files with multiple extensions correctly", () => {
     const cwd = process.cwd();
     const testDirectory = join(cwd, "test/fixtures/files");
     const files = getFilesToProcess(testDirectory, ["js", "ts"]);
-    expect(files).toHaveLength(2);
+    expect(files.map((path) => path.replace(cwd, ""))).toMatchInlineSnapshot(`
+      [
+        "/test/fixtures/files/file.js",
+        "/test/fixtures/files/file.ts",
+      ]
+    `);
+  });
+
+  it("gets files with paths that have special characters correctly", () => {
+    const cwd = process.cwd();
+    const testDirectory = join(cwd, "test/fixtures/(special files*)");
+    const files = getFilesToProcess(testDirectory, ["js", "ts"]);
+    expect(files.map((path) => path.replace(cwd, ""))).toMatchInlineSnapshot(`
+      [
+        "/test/fixtures/(special files*)/file \\slash.js",
+        "/test/fixtures/(special files*)/file with (parentheses).js",
+        "/test/fixtures/(special files*)/file with space.js",
+      ]
+    `);
   });
 
   it("gets nested files correctly", () => {
     const cwd = process.cwd();
     const testDirectory = join(cwd, "test/fixtures/files");
     const files = getFilesToProcess(testDirectory, ["tsx"]);
-    expect(files).toHaveLength(2);
+    expect(files.map((path) => path.replace(cwd, ""))).toMatchInlineSnapshot(`
+      [
+        "/test/fixtures/files/file.tsx",
+        "/test/fixtures/files/nested/file.tsx",
+      ]
+    `);
   });
 });