Format JSON.parse string literals

[nickmccurdy]

Prettier 1.19.1
Playground link

--parser babel

Input:

JSON.parse('{"property":"value"}');
JSON.parse`{"property":"value"}`;

Output:

JSON.parse('{"property":"value"}');
JSON.parse`{"property":"value"}`;

Expected behavior:

JSON.parse('{ "property": "value" }');
JSON.parse`{ "property": "value" }`;

For more complex JavaScript objects, it can be faster and more reliable to use JSON.parse with an inline string rather than just using an object literal (explanation). It would be useful if Prettier could format strings passed directly to JSON.parse as embedded JSON, so they look like what would happen if you loaded and parsed a separate JSON file formatted with Prettier. Note that template strings work with JSON.parse in JavaScript, but we'd have to handle edge cases when formatting template variables.

[thorn0]

I suppose this issue can be considered a duplicate of #5908.

[nickmccurdy]

Would that also cover automatic support for JSON when JSON.parse is used?

[thorn0]

It definitely wouldn't be automatic. A special comment, an option or something else would be needed to trigger it. See #5588, #6626.

[thorn0]

Closing in favor of #5588