This also does not fix it, as it would highlight brackets in

const myStr = `(`;

This also does not fix it, as it would highlight brackets in

const myStr = `(`;

Got it. Probably other selector expression can make it.

@hediet I can't reproduce your case, I get the following result which bracket not highlight.

Ah, this one only marks quoted strings as strings. Template strings are not marked as strings. I fear this causes other problems, e.g. for the bracket auto-closing.

I see. I can't ensure other problems either as my poor understanding of the codebase.

The expression enclosed by ${ and } inside a template string is scoped with meta.embedded.line.js. That resets the token type to other (microsoft/vscode-textmate#54). An important difference from tokenTypes in package.json is that meta.embedded enables proper nesting. The token type is determined by the last meta.embedded or string scope.

So I think we can remove all these:

This leaves one problem. ${ and } don't have the meta.embedded.line.js scope, only meta.template.expression.js. We can target these symbols with punctuation.definition.template-expression, like that:

"punctuation.definition.template-expression": "other",

I'm not very familiar with JavaScript, I'm actually working on https://github.com/bmalehorn/vscode-fish, so I'm not totally sure this will work.

Here is how it looks like: