Emit error when missing comma

[dragon-archer]

Currently, this extension will silently accept items without comma

while the builtinjson mode will show a prominent error

How about implementing this by using a simple sementic highlight or some textmate rules like:

{
    "match": "\\}\\s*\\{",
    "name": "invalid.illegal.xxx"
}

Or perhaps you'd like to wait for the official VS Code Team to support using custom textmate rules together with the builtin mode together (As I've seen in your other issues)

[RedCMD]

yeah, same as in #3
I'm waiting on VSCode to allow custom extensions to enable the builtin json extension
microsoft/vscode#198025
microsoft/vscode#198583

if you want
there is code already for displaying Treesitter parse errors
you'll just need to add the import and enable the Diagnostics
it wont show what type of error, other than it being an error
(reports on false postive json errors are welcomed)

TmLanguage-Syntax-Highlighter/out/extension.js

Lines 5 to 6 in bcb5d10

	const TreeSitter_1 = require("./TreeSitter");
	const tokenColorCustomizations_1 = require("./tokenColorCustomizations");

const DiagnosticCollection_1 = require("./DiagnosticCollection");

TmLanguage-Syntax-Highlighter/out/extension.js

Line 20 in bcb5d10

// initDiagnostics(context);

(0, DiagnosticCollection_1.initDiagnostics)(context);

TmLanguage-Syntax-Highlighter/out/DiagnosticCollection.js

Line 33 in bcb5d10

if (false) {

and prob disable the regex errors (many false postives)

TmLanguage-Syntax-Highlighter/out/DiagnosticCollection.js

Line 49 in bcb5d10

if (true) {

I could look into making it a config option to enable/disable error reporting
tho I'll prob have it disabled by default for now

[RedCMD]

or the other way is manually enabling the builtin json-language-features extension

\extensions\json-language-features\client\dist\node\jsonClientMain.js
just involves adding the languageId to the DocumentSelector list "jsonc","json-textmate"

and "activationEvents"
\extensions\json-language-features\package.js
"onLanguage:json-textmate"

you would also get schema support at the same time

[dragon-archer]

Thanks, I turned on the builtin feature, though I think it might be better with a config option in this extension

[RedCMD]

Now available in the full release 2.3.0 for VSCode 1.87.0