Template literals with substitutions are auto-completed with too many backticks

[thebinarysearchtree]

Does this issue occur when all extensions are disabled?: Yes

• VS Code Version: 1.60
• OS Version: MacOS Big Sur

This only started occurring recently.

test.mov

[hediet]

This is difficult to solve nicely.

• Because of Allow auto-close quotation marks per file type and global #25658, the backtick does not autoclose, as it touches the word sql.
• Because } is not a word and not a string-token, the second backtick is autoclosed.

Maybe the backtick should be autoclosed if the opening backtick was tokenized as string (overriding any word rule)? @alexdima

[thebinarysearchtree]

This bug also occurs in Python, Ruby, and Swift. It does not occur in PHP, c#, and a couple of other languages. Strangely, those languages report the token type as a string, which makes no sense to me.

[alexdima]

This has been fixed by PR #135197

[alexdima]

The PR #135197 improves things, and makes it unlikely that this case is hit, but this case remains:

Pressing ` will insert two backticks here:

const t2 = `something ${t1}|

[alexr00]

Verified that in the example the first backtick is autocompleted, so I don't need to add a backtick at the problematic location.