Invalid response to the workspace/SemanticTokens/refresh notification

[AmadeusW]

When debugging VS I was innoudated with JsonRpc warnings that No target methods are registered that match "NotificationRequested" (see azdo 2341995)
When monitoring traffic between the LSP Client and Server I noticed that the response to workspace/SemanticTokens/refresh notification has invalid json; I think for this to work correctly (and not produce warnings), the response should be "method": "workspace/SemanticTokens/refresh". Currently, it is "method": "NotificationReceived" with "params.MethodName": "workspace/SemanticTokens/refresh"

[davidwengier]

@AmadeusW what made you decide this is Razor? I can't see where this is us. We send that notification here:
https://github.com/dotnet/razor/blob/main/src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/WorkspaceSemanticTokensRefreshNotifier.cs#L49

Which is just a straight call to JsonRPC's NotifyAsync method:
https://github.com/dotnet/razor/blob/main/src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/ClientConnection.cs#L45

Indeed the string "NotificationReceived" doesn't appear in our repo anywhere. I'm also seeing these same messages logged for the Roslyn server too:

Are you sure this is coming from us? I could see this happening if something is sending this message to us, as we don't support that method, so the log message makes sense. We send it to the client, but perhaps there is something out there that is blasting that message to all clients? The method that I'm debugging in that screenshot is "DispatchIncomingRequest"

I couldn't see a way to identify any of that in the debugger though.