Escape in regexp

Author: blakeembrey

src/index.spec.ts

@@ -221,9 +221,20 @@ describe("javascript-stringify", () => {
       });
 
       describe("RegExp", () => {
-        it("should stringify as shorthand", test(/[abc]/gi, "/[abc]/gi"));
+        it(
+          "should stringify as shorthand",
+          test(/[abc]/gi, "new RegExp('[abc]', 'gi')"),
+        );
 
-        it("should escape slashes", test(new RegExp("a/b"), "/a\\/b/"));
+        it(
+          "should escape slashes",
+          test(new RegExp("a/b"), "new RegExp('a\\\\/b')"),
+        );
+
+        it(
+          "should escape html characters",
+          test(new RegExp("<!--"), "new RegExp('\\u003c!--')"),
+        );
       });
 
       describe("Number", () => {

src/object.ts

@@ -92,7 +92,10 @@ const OBJECT_TYPES: Record<string, ToString> = {
   "[object Map]": (map: Map<any, any>, space: string, next: Next) => {
     return `new Map(${next(Array.from(map))})`;
   },
-  "[object RegExp]": String,
+  "[object RegExp]": (value: RegExp, space: string, next: Next) => {
+    if (!value.flags) return `new RegExp(${next(value.source)})`;
+    return `new RegExp(${next(value.source)}, ${next(value.flags)})`;
+  },
   "[object global]": globalToString,
   "[object Window]": globalToString,
 };

src/stringify.ts

@@ -16,7 +16,7 @@ const PRIMITIVE_TYPES: Record<string, ToString> = {
     if (key !== undefined) return `Symbol.for(${next(key)})`;
 
     // ES2018 `Symbol.description`.
-    return `Symbol(${next((value as any).description)})`;
+    return `Symbol(${next(value.description)})`;
   },
   bigint: (value: bigint, space: string, next: Next) => {
     return `BigInt(${next(String(value))})`;