🐛 Use JSON5 to parse manifest.json (closes #216)

Author: vialoh

package-lock.json

@@ -56,6 +56,7 @@
     "clean-webpack-plugin": "^4.0.0",
     "colors": "^1.4.0",
     "cross-env": "^7.0.3",
+    "json5": "^2.2.3",
     "lodash": "^4.17.21",
     "minimist": "^1.2.6",
     "useragent": "^2.3.0",

package.json

@@ -1,5 +1,6 @@
 import { readFileSync } from "fs";
 import { flatMapDeep } from "lodash";
+import JSON5 from "json5";
 import { Compiler, Entry } from "webpack";
 import { bgScriptEntryErrorMsg, bgScriptManifestRequiredMsg } from "../messages/errors";
 
@@ -8,8 +9,8 @@ export function extractEntries(
   manifestPath: string,
   webpackOutput: Compiler["options"]["output"] = {},
 ): IEntriesOption {
-  const manifestJson = JSON.parse(readFileSync(manifestPath).toString()) as IExtensionManifest;
-  const { background, content_scripts } = manifestJson;
+  const manifestJson = JSON5.parse(readFileSync(manifestPath).toString()) as IExtensionManifest;
+  const { background, content_scripts: contentScripts } = manifestJson;
   const { filename } = webpackOutput;
 
   if (!filename) {
@@ -31,9 +32,9 @@ export function extractEntries(
     throw new TypeError(bgScriptEntryErrorMsg.get());
   }
 
-  const contentEntries: unknown = content_scripts
+  const contentEntries: unknown = contentScripts
     ? flatMapDeep(Object.keys(webpackEntry), (entryName) =>
-        content_scripts.map(({ js }) =>
+        contentScripts.map(({ js }) =>
           js.map((contentItem) => contentItem.replace(toRemove, "")).filter((contentItem) => contentItem === entryName),
         ),
       )