Hi @soncodi, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

I've signed the CLA.

Our typescript grammar uses scopes
punctuation.definition.template-expression.begin.ts
and
punctuation.definition.template-expression.end.ts
for the interpolation in strings.
The PR should use that.

@aeschli Thanks. I adjusted the selectors, also including .js

I'd suggest to just have the common prefix: punctuation.definition.template-expression.
The inner part might also get colored, but this will be fixed by #3008 in the January build

@aeschli Selectors updated.

@aeschli Any update on this PR?

I think you can remove the template.variable rule. It's not a scope that is produced by the TypeScript grammar. This used to match before because we matched colors rules against a flattened list of all scopes on the stack. Now with the work for #3008, we match only if a coloring rule is a prefix of a scope on the stack.
In the January build you can examine the scope stack with a new command Developer: Inspect TM Scopes:

I'll remove the template.variable rule as discussed.
@soncodi Thanks for the PR!

Thank you @aeschli