Exception when a valid JSON file containing U+2028 is required

[Macil]

If you try to use Browserify to build a javascript bundle that calls require() to include a JSON file that contains a U+2028 (line separator) or U+2029 (paragraph separator) character, then Browserify throws an exception.

U+2028 and U+2029 characters are both valid inside strings in JSON, but not in javascript. JSON is not perfectly a Javascript subset. When Browserify transforms JSON files to javascript by prepending module.exports=, it also needs to transform all U+2028/9 characters (which would only be present inside strings) to \u2028 and \u2029. A well-placed .replace(/\u2028/g, '\\u2028').replace(/\u2029/g, '\\u2029'); should do the trick.

Here's an example repo showing the issue: https://github.com/AgentME/br-json-test

[Macil]

Fixed by #1211.

[jmm]

@agentme I just wanted to acknowledge this and say thank you for the report!