Ryan Brandenburg

> @NTaylorMullen, @ryanbrandenburg could this be the root cause of the infamous [Disco Colors issue](https://developercommunity.visualstudio.com/t/Disco-Colors-Persist-again/1671094?space=8&q=disco+colors)? It seems totally plausible to me, but @NTaylorMullen is probably our expert on our Threading...

The Roslyn work for this is accomplished here https://github.com/dotnet/roslyn/pull/62924.

This is hypothetically resolved now that CLaSP is merged. I will close and we can re-open if the issue re-cures.

@arkalyanms yeah it looks like @davidwengier @allisonchou and I don't have access to view those alerts.

Likely problem spot: https://github.com/dotnet/razor-tooling/blob/main/src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/Semantic/Services/TagHelperSemanticRangeVisitor.cs#L406-L420 Edit: This is not the problem.

It's actually a bit worse than that: Things really go off the rails if you include html after the fact there.

Looking at the syntax tree here we have another instance of a SyntaxTree which is shorter than the document. Seemingly something about the self-closing `` confuses the parent causing the...

https://github.com/dotnet/aspnetcore/issues/39764

You're totally correct that this is a gap, so I'm going to leave this open for collecting user feedback, if you feel strongly about this give the issue a 👍....

This is a bit more complicated in the general case. For script you're mostly correct (though we would argue that it ought to be the html instead of the TagHelper...