add some additional special chars and use string.raw

Author: james-elicx

packages/open-next/src/utils/regex.ts

@@ -14,8 +14,8 @@ export function getCrossPlatformPathRegex(
   opts: { escape: boolean } = { escape: true },
 ) {
   const newExpr = (
-    opts.escape ? regex.replace(/([[\]().*+?^$|])/g, "\\$1") : regex
-  ).replaceAll("/", "(?:\\/|\\\\)");
+    opts.escape ? regex.replace(/([[\]().*+?^$|{}\\])/g, "\\$1") : regex
+  ).replaceAll("/", String.raw`(?:\/|\\)`);
 
   return new RegExp(newExpr, "g");
 }

packages/tests-unit/tests/utils/regex.test.ts

@@ -1,6 +1,6 @@
 import { getCrossPlatformPathRegex } from "@opennextjs/aws/utils/regex.js";
 
-const specialChars = "^([123]+|[123]*)?$";
+const specialChars = "^([123]+|[123]{1,3})*\\?$";
 
 describe("getCrossPlatformPathRegex", () => {
   it("should return a regex without escaping characters", () => {
@@ -11,12 +11,14 @@ describe("getCrossPlatformPathRegex", () => {
   it("should always create cross-platform separators", () => {
     [true, false].forEach((v) => {
       const regexp = getCrossPlatformPathRegex("test/path", { escape: v });
-      expect(regexp.source).toEqual("test(?:\\/|\\\\)path");
+      expect(regexp.source).toEqual(String.raw`test(?:\/|\\)path`);
     });
   });
 
   it("should return a regex with escaped characters", () => {
     const regexp = getCrossPlatformPathRegex(specialChars, { escape: true });
-    expect(regexp.source).toEqual("\\^\\(\\[123\\]\\+\\|\\[123\\]\\*\\)\\?\\$");
+    expect(regexp.source).toEqual(
+      String.raw`\^\(\[123\]\+\|\[123\]\{1,3\}\)\*\\\?\$`,
+    );
   });
 });