TypeError: Expected string for scope, got undefined

[TheWitheredStriker]

This error keeps happening with most (though not all) of the grammars I'm testing. This includes grammars that don't even have a scope property, like this one.

I'm not sure why this happens, any ideas?

[Nixinova]

Will look into this, thanks for the report.

[Nixinova]

Looks like its an issue with parsing CSON grammar. If i convert your example to JSON then it works fine https://www.convertsimple.com/convert-cson-to-json/. Will see how I can fix this, but that is a workaround in the meantime.

[TheWitheredStriker]

It doesn't seem to exclusively be a CSON problem. It occurs with YAML too, like this one.

EDIT: I also tried converting the YAML to JSON and I still get the error.

[Nixinova]

Looking into it, the error seems to be here:

NovaLightshow/index.mjs

Line 36 in 1a0a549

const tree = starryNight.highlight(sample, grammarJson.scopeName);

When there is an unexpected scopeName property the code doesn't work.

For instance your latest example file:

https://github.com/mitranim/sublime-forth/blob/2293f5c0fd65a141e246880b7c8146affd94c29f/Forth.sublime-syntax#L2-L5

scope: source.forth

It says scope not scopeName.
So I'll have to change the code to take that into account.
Hopefully that fixes this

[Nixinova]

Try it now, hopefully fixed.

[TheWitheredStriker]

I don't get the error anymore, but when trying the Forth grammar, the code is displayed but not highlighted:

Trying the CSON Factor grammar up top also still throws the same error. Sorry!

[Nixinova]

Hmm that's interesting. Welp, will have to do fix attempt two.

[Nixinova]

Ah - this is a bug in the CSON parser, fabiospampinato/cson2json#1.

Seems like it can't handle mixed indents:

repository:
  escaped_characters:
    patterns: true
scope: "source.factor"
scopeName: "source.factor"

Will have to switch CSON parser.

[Nixinova]

It doesn't seem to exclusively be a CSON problem. It occurs with YAML too, like this one.

EDIT: I also tried converting the YAML to JSON and I still get the error.

The previous fix resolves this problem, but the original issue is with the CSON.