feat: add support for allowTrailingCommas option in JSON

[lumirlumir]

Prerequisites checklist

• I have read the contributing guidelines.

What is the purpose of this pull request?

Hello,

I've added support for the allowTrailingCommas option in JSON.

Code Explorer can now parse trailing commas in JSONC mode!

Please let me know if there's anything I might have missed.

What changes did you make? (Give an overview)

Before

After

Related Issues

Is there anything you'd like reviewers to focus on?

[netlify]

✅ Deploy Preview for eslint-code-explorer ready!

Name	Link
🔨 Latest commit	82e639b
🔍 Latest deploy log	https://app.netlify.com/sites/eslint-code-explorer/deploys/6819eab50c27820008ecd08d
😎 Deploy Preview	https://deploy-preview-96--eslint-code-explorer.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[nzakas]

Thanks for taking a look at this. There's a bit of complexity here: allowTrailingCommas is only valid for the JSONC language. For JSON, trailing commas are never allowed, and for JSON5 trailing commas are always allowed. So, we can't just have a radio button for "allow trailing commas" because it doesn't work properly in JSON and JSON5 modes. Can we maybe disable the radio button in those cases?

[amareshsm]

Thanks for taking a look at this. There's a bit of complexity here: allowTrailingCommas is only valid for the JSONC language. For JSON, trailing commas are never allowed, and for JSON5 trailing commas are always allowed. So, we can't just have a radio button for "allow trailing commas" because it doesn't work properly in JSON and JSON5 modes. Can we maybe disable the radio button in those cases?

Can we display the "Allow trailing commas" option only when the selected mode is JSONC?

[lumirlumir]

@nzakas @amareshsm

Thanks for the reviews!

I've added a new commit in 82e639b.

Now, allowTrailingCommas appears only when JSONC mode is enabled.

[nzakas]

LGTM. Would like @amareshsm to review before merging.

[amareshsm]

LGTM Thanks.