Rybrande/o#upgrade v2 (#2439)

O# upgrade

Author: ryanbrandenburg

eng/Signing.props

@@ -15,10 +15,10 @@
   <ItemGroup>
     <FileSignInfo Include="Newtonsoft.Json.dll" CertificateName="3PartySHA2" />
     <FileSignInfo Include="MediatR.dll" CertificateName="3PartySHA2" />
-    <FileSignInfo Include="MediatR.Extensions.Microsoft.DependencyInjection.dll" CertificateName="3PartySHA2" />
     <FileSignInfo Include="OmniSharp.Extensions.JsonRpc.dll" CertificateName="3PartySHA2" />
     <FileSignInfo Include="OmniSharp.Extensions.LanguageProtocol.dll" CertificateName="3PartySHA2" />
     <FileSignInfo Include="OmniSharp.Extensions.LanguageServer.dll" CertificateName="3PartySHA2" />
+    <FileSignInfo Include="OmniSharp.Extensions.LanguageServer.Shared.dll" CertificateName="3PartySHA2" />
   </ItemGroup>
 
   <ItemGroup>

eng/Versions.props

@@ -122,7 +122,7 @@
     <MoqPackageVersion>4.10.0</MoqPackageVersion>
     <!-- STOP!!! We need to reference the version of JSON that our HOSTS supprt. -->
     <NewtonsoftJsonPackageVersion>12.0.2</NewtonsoftJsonPackageVersion>
-    <OmniSharpExtensionsLanguageServerPackageVersion>0.14.2</OmniSharpExtensionsLanguageServerPackageVersion>
+    <OmniSharpExtensionsLanguageServerPackageVersion>0.18.0-beta0081</OmniSharpExtensionsLanguageServerPackageVersion>
     <OmniSharpMSBuildPackageVersion>1.33.0</OmniSharpMSBuildPackageVersion>
     <SystemPrivateUriPackageVersion>4.3.2</SystemPrivateUriPackageVersion>
     <SystemCompositionPackageVersion>1.0.31.0</SystemCompositionPackageVersion>

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer.Common/LanguageServerConstants.cs

@@ -7,14 +7,19 @@ public static class LanguageServerConstants
     {
         public const string ProjectConfigurationFile = "project.razor.json";
 
-        public const string RazorSemanticTokensEndpoint = "textDocument/semanticTokens";
+        // Semantic "Legacy" endpoints refer to an old LSP spec version, needed for now until VS reacts.
+        public const string LegacyRazorSemanticTokensEndpoint = "textDocument/semanticTokens";
 
-        public const string RazorSemanticTokensEditEndpoint = "textDocument/semanticTokens/edits";
+        public const string LegacyRazorSemanticTokensEditEndpoint = "textDocument/semanticTokens/edits";
 
-        public const string RazorSemanticTokensRangeEndpoint = "textDocument/semanticTokens/range";
+        public const string LegacyRazorSemanticTokensRangeEndpoint = "textDocument/semanticTokens/range";
 
         public const string RazorSemanticTokensLegendEndpoint = "_ms_/textDocument/semanticTokensLegend";
 
+        public const string RazorSemanticTokensEditEndpoint = "textDocument/semanticTokens/full/delta";
+
+        public const string RazorSemanticTokensEndpoint = "textDocument/semanticTokens/full";
+
         public const string SemanticTokensProviderName = "semanticTokensProvider";
 
         public const string RazorRangeFormattingEndpoint = "razor/rangeFormatting";

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/CodeActions/CodeActionEndpoint.cs

@@ -14,9 +14,9 @@
 using Microsoft.CodeAnalysis.Razor.Workspaces;
 using Microsoft.CodeAnalysis.Text;
 using OmniSharp.Extensions.LanguageServer.Protocol.Client.Capabilities;
+using OmniSharp.Extensions.LanguageServer.Protocol.Document;
 using OmniSharp.Extensions.LanguageServer.Protocol.Models;
 using OmniSharp.Extensions.LanguageServer.Protocol.Server;
-using LanguageServerInstance = OmniSharp.Extensions.LanguageServer.Server.LanguageServer;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.CodeActions
 {
@@ -25,8 +25,8 @@ internal class CodeActionEndpoint : ICodeActionHandler
         private readonly IEnumerable<RazorCodeActionProvider> _providers;
         private readonly ForegroundDispatcher _foregroundDispatcher;
         private readonly DocumentResolver _documentResolver;
-        private readonly ILanguageServer _languageServer;
         private readonly LanguageServerFeatureOptions _languageServerFeatureOptions;
+        private readonly IClientLanguageServer _languageServer;
 
         private CodeActionCapability _capability;
 
@@ -36,7 +36,7 @@ public CodeActionEndpoint(
             IEnumerable<RazorCodeActionProvider> providers,
             ForegroundDispatcher foregroundDispatcher,
             DocumentResolver documentResolver,
-            ILanguageServer languageServer,
+            IClientLanguageServer languageServer,
             LanguageServerFeatureOptions languageServerFeatureOptions)
         {
             _providers = providers ?? throw new ArgumentNullException(nameof(providers));
@@ -63,8 +63,7 @@ public void SetCapability(CodeActionCapability capability)
         {
             _capability = capability;
 
-            var languageServerInstance = _languageServer as LanguageServerInstance;
-            var extendableClientCapabilities = languageServerInstance?.ClientSettings?.Capabilities as ExtendableClientCapabilities;
+            var extendableClientCapabilities = _languageServer.ClientSettings?.Capabilities as ExtendableClientCapabilities;
             _supportsCodeActionResolve = extendableClientCapabilities?.SupportsCodeActionResolve ?? false;
         }
 

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/CodeActions/ComponentAccessibilityCodeActionProvider.cs

@@ -15,6 +15,7 @@
 using Microsoft.AspNetCore.Razor.LanguageServer.Common;
 using Microsoft.CodeAnalysis.Razor;
 using Microsoft.VisualStudio.Editor.Razor;
+using OmniSharp.Extensions.LanguageServer.Protocol;
 using OmniSharp.Extensions.LanguageServer.Protocol.Models;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.CodeActions
@@ -216,7 +217,7 @@ private static WorkspaceEdit CreateRenameTagEdit(RazorCodeActionContext context,
 
             return new WorkspaceEdit
             {
-                Changes = new Dictionary<Uri, IEnumerable<TextEdit>> {
+                Changes = new Dictionary<DocumentUri, IEnumerable<TextEdit>> {
                     [context.Request.TextDocument.Uri] = changes,
                 }
             };

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/CodeActions/Models/AddUsingsCodeActionParams.cs

@@ -1,13 +1,13 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System;
+using OmniSharp.Extensions.LanguageServer.Protocol;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.CodeActions.Models
 {
     internal sealed class AddUsingsCodeActionParams
     {
-        public Uri Uri { get; set; }
+        public DocumentUri Uri { get; set; }
         public string Namespace { get; set; }
     }
 }

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/CodeActions/Models/CreateComponentCodeActionParams.cs

@@ -1,13 +1,13 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System;
+using OmniSharp.Extensions.LanguageServer.Protocol;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.CodeActions.Models
 {
     internal sealed class CreateComponentCodeActionParams
     {
-        public Uri Uri { get; set; }
+        public DocumentUri Uri { get; set; }
         public string Path { get; set; }
     }
 }

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/CodeActions/Models/ExtractToCodeBehindCodeActionParams.cs

@@ -1,13 +1,13 @@
 // Copyright (c) .NET Foundation. All rights reserved.
 // Licensed under the Apache License, Version 2.0. See License.txt in the project root for license information.
 
-using System;
+using OmniSharp.Extensions.LanguageServer.Protocol;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.CodeActions.Models
 {
     internal sealed class ExtractToCodeBehindCodeActionParams
     {
-        public Uri Uri { get; set; }
+        public DocumentUri Uri { get; set; }
         public int ExtractStart { get; set; }
         public int ExtractEnd { get; set; }
         public int RemoveStart { get; set; }

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/Completion/DefaultTagHelperDescriptionFactory.cs

@@ -8,7 +8,7 @@
 using System.Text.RegularExpressions;
 using Microsoft.CodeAnalysis.Razor.Completion;
 using OmniSharp.Extensions.LanguageServer.Protocol.Models;
-using OmniSharp.Extensions.LanguageServer.Server;
+using OmniSharp.Extensions.LanguageServer.Protocol.Server;
 using RazorAttributeDescriptionInfo = Microsoft.CodeAnalysis.Razor.Completion.AttributeDescriptionInfo;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.Completion
@@ -36,17 +36,21 @@ internal class DefaultTagHelperDescriptionFactory : TagHelperDescriptionFactory
             [typeof(decimal).FullName] = "decimal",
         };
 
-        public DefaultTagHelperDescriptionFactory(ILanguageServer languageServer)
+        private readonly Lazy<ILanguageServer> _server;
+
+        // Need to have a lazy server here because if we try to resolve the server it creates types which create a DefaultTagHelperDescriptionFactory, and we end up StackOverflowing.
+        // This lazy can be avoided in the future by using an upcoming ILanguageServerSettings interface, but it doesn't exist/work yet.
+        public DefaultTagHelperDescriptionFactory(Lazy<ILanguageServer> languageServer)
         {
             if (languageServer is null)
             {
                 throw new NotImplementedException(nameof(languageServer));
             }
 
-            LanguageServer = languageServer;
+            _server = languageServer;
         }
 
-        public ILanguageServer LanguageServer { get; }
+        public ILanguageServer LanguageServer => _server.Value;
 
         public override bool TryCreateDescription(ElementDescriptionInfo elementDescriptionInfo, out MarkupContent tagHelperDescription)
         {

src/Razor/src/Microsoft.AspNetCore.Razor.LanguageServer/Completion/RazorCompletionEndpoint.cs

@@ -15,8 +15,8 @@
 using Microsoft.CodeAnalysis.Text;
 using Microsoft.Extensions.Logging;
 using OmniSharp.Extensions.LanguageServer.Protocol.Client.Capabilities;
+using OmniSharp.Extensions.LanguageServer.Protocol.Document;
 using OmniSharp.Extensions.LanguageServer.Protocol.Models;
-using OmniSharp.Extensions.LanguageServer.Protocol.Server;
 
 namespace Microsoft.AspNetCore.Razor.LanguageServer.Completion
 {