manifest.json parser too strict

[freaktechnik]

What is the current behavior?

The manifest.json is parsed using the json-parser package, which emits nicer errors, but does not support comments. This was discovered with an error when trying to sign an extension with comments in the manifest by @AntoineTurmel.

What is the expected or desired behavior?

The parser should be able to handle comments in the manifest.json file.

[freaktechnik]

While that is generally true, it does not apply to manifest.json and messages.json files. We sadly have to expect comments in them.

See also https://browserext.github.io/browserext/#pre-parsing

[tommai78101]

Ugh, I accidentally deleted my previous comment. Original comment:

Comments are not valid in all JSON files.

https://plus.google.com/+DouglasCrockfordEsq/posts/RK8qyGVaGSr

---

I see, so in this case, only manifest.json and messages.json files are exempt from it? I do not see in the current article saying the messages.json should also ignore comments. Or maybe I'm looking in the wrong place?