refactor: replace cjson with internal comment-stripping JSON parser

package.json

@@ -120,7 +120,6 @@
     "async-lock": "1.4.1",
     "body-parser": "^1.19.0",
     "chokidar": "^3.6.0",
-    "cjson": "^0.3.1",
     "cli-table3": "0.6.5",
     "colorette": "^2.0.19",
     "commander": "^5.1.0",

src/loadCJSON.spec.ts

@@ -8,13 +8,26 @@
 
   it("should return parsed JSON on success", () => {
     const filePath = path.join(fixturesDir, "valid.cjson");
     const result = loadCJSON(filePath);
     expect(result).to.deep.equal({ key: "value" });
   });
 
+  it("should correctly parse comments inside string values and block comments", () => {
+    const filePath = path.join(fixturesDir, "complex.cjson");
+    const result = loadCJSON(filePath);
+    expect(result).to.deep.equal({
+      url: "https://example.com/foo",
+      comment_in_string: "this is // not a comment",
+      block_in_string: "this is /* not a block */ comment",
+      nested: {
+        key: "value",
+      },
+    });
+  });
+
   it("should throw FirebaseError on ENOENT", () => {
     const filePath = path.join(fixturesDir, "nonexistent.cjson");
     expect(() => loadCJSON(filePath)).to.throw(
       FirebaseError,
       "File " + filePath + " does not exist",
     );
@@ -22,7 +35,7 @@
 
   it("should throw FirebaseError on parse error", () => {
     const filePath = path.join(fixturesDir, "invalid.cjson");
     expect(() => loadCJSON(filePath)).to.throw(
       FirebaseError,
       new RegExp(`Parse Error in ${filePath}`),
     );

src/loadCJSON.ts

@@ -1,16 +1,30 @@
+import * as fs from "fs";
 import { FirebaseError } from "./error";
-import * as cjson from "cjson";
 
 /**
  * Loads CJSON from given path.
  */
 export function loadCJSON(path: string): any {
+  let content: string;
   try {
-    return cjson.load(path);
+    content = fs.readFileSync(path, "utf8");
   } catch (e: any) {
     if (e.code === "ENOENT") {
       throw new FirebaseError(`File ${path} does not exist`);
     }
+    throw e;
+  }
+
+  try {
+    const stripped = content.replace(
+      /("([^"\\]|\\.)*")|(\/\*[\s\S]*?\*\/|\/\/.*)/g,
+      (match, string) => {
+        if (string) return string;
+        return "";
+      },
+    );
+    return JSON.parse(stripped);
+  } catch (e: any) {
     throw new FirebaseError(`Parse Error in ${path}:\n\n${e.message}`);
   }
 }

src/test/fixtures/loadCJSON/complex.cjson

@@ -0,0 +1,11 @@
+{
+  // Line comment at start
+  "url": "https://example.com/foo",
+  "comment_in_string": "this is // not a comment",
+  "block_in_string": "this is /* not a block */ comment",
+  /* Multi-line
+     block comment */
+  "nested": {
+    "key": "value" // trailing line comment
+  }
+}