From 85029e611eef3673892e137b779907028f898b31 Mon Sep 17 00:00:00 2001
From: Mikayla Hutchinson <m.j.hutchinson@gmail.com>
Date: Sun, 16 Jun 2024 21:35:16 -0400
Subject: [PATCH] LSP Protocol: Suppress obsoletion propagation

---
 .../Protocol/Protocol/DocumentSymbolParams.cs             | 8 +++++++-
 .../Extensions/Converters/VSExtensionUtilities.cs         | 2 ++
 .../Protocol/Protocol/Extensions/VSSymbolInformation.cs   | 5 ++++-
 src/LanguageServer/Protocol/Protocol/Hover.cs             | 2 ++
 .../Internal/Converters/VSInternalExtensionUtilities.cs   | 2 ++
 .../Protocol/Protocol/Internal/VSInternalHover.cs         | 2 ++
 src/LanguageServer/Protocol/Protocol/Methods.Document.cs  | 2 ++
 src/LanguageServer/Protocol/Protocol/Methods.Workspace.cs | 2 ++
 .../Protocol/Protocol/WorkspaceSymbolParams.cs            | 7 ++++++-
 9 files changed, 29 insertions(+), 3 deletions(-)

diff --git a/src/LanguageServer/Protocol/Protocol/DocumentSymbolParams.cs b/src/LanguageServer/Protocol/Protocol/DocumentSymbolParams.cs
index 91d3d74ebc085..dae611b223974 100644
--- a/src/LanguageServer/Protocol/Protocol/DocumentSymbolParams.cs
+++ b/src/LanguageServer/Protocol/Protocol/DocumentSymbolParams.cs
@@ -12,7 +12,11 @@ namespace Roslyn.LanguageServer.Protocol;
 /// See the <see href="https://microsoft.github.io/language-server-protocol/specifications/specification-current/#documentSymbolParams">Language Server Protocol specification</see> for additional information.
 /// </para>
 /// </summary>
-internal class DocumentSymbolParams : ITextDocumentParams, IWorkDoneProgressParams, IPartialResultParams<SumType<SymbolInformation[], DocumentSymbol[]>>
+internal class DocumentSymbolParams
+    : ITextDocumentParams, IWorkDoneProgressParams,
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this class is not
+      IPartialResultParams<SumType<SymbolInformation[], DocumentSymbol[]>>
+#pragma warning restore CS0618
 {
     /// <summary>
     /// Gets or sets the text document.
@@ -32,5 +36,7 @@ public TextDocumentIdentifier TextDocument
     /// <inheritdoc/>
     [JsonPropertyName(Methods.PartialResultTokenName)]
     [JsonIgnore(Condition = JsonIgnoreCondition.WhenWritingNull)]
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this property is not
     public IProgress<SumType<SymbolInformation[], DocumentSymbol[]>>? PartialResultToken { get; set; }
+#pragma warning restore CS0618
 }
diff --git a/src/LanguageServer/Protocol/Protocol/Extensions/Converters/VSExtensionUtilities.cs b/src/LanguageServer/Protocol/Protocol/Extensions/Converters/VSExtensionUtilities.cs
index 0cd191528ca56..4db11ecf2f68b 100644
--- a/src/LanguageServer/Protocol/Protocol/Extensions/Converters/VSExtensionUtilities.cs
+++ b/src/LanguageServer/Protocol/Protocol/Extensions/Converters/VSExtensionUtilities.cs
@@ -34,7 +34,9 @@ private static void AddConverters(IList<JsonConverter> converters)
         TryAddConverter<Diagnostic, VSDiagnostic>();
         TryAddConverter<Location, VSLocation>();
         TryAddConverter<ServerCapabilities, VSServerCapabilities>();
+#pragma warning disable CS0618 // SymbolInformation is obsolete but we need the converter regardless
         TryAddConverter<SymbolInformation, VSSymbolInformation>();
+#pragma warning restore CS0618
         TryAddConverter<TextDocumentIdentifier, VSTextDocumentIdentifier>();
 
         void TryAddConverter<TBase, TExtension>()
diff --git a/src/LanguageServer/Protocol/Protocol/Extensions/VSSymbolInformation.cs b/src/LanguageServer/Protocol/Protocol/Extensions/VSSymbolInformation.cs
index 84d77d40b6219..49d524328e2e1 100644
--- a/src/LanguageServer/Protocol/Protocol/Extensions/VSSymbolInformation.cs
+++ b/src/LanguageServer/Protocol/Protocol/Extensions/VSSymbolInformation.cs
@@ -9,7 +9,10 @@ namespace Roslyn.LanguageServer.Protocol
     /// <summary>
     /// <see cref="VSSymbolInformation"/> extends <see cref="SymbolInformation"/> providing additional properties used by Visual Studio.
     /// </summary>
-    internal class VSSymbolInformation : SymbolInformation
+    internal class VSSymbolInformation
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this class is not (yet)
+        : SymbolInformation
+#pragma warning restore
     {
         /// <summary>
         /// Gets or sets the icon associated with the symbol. If specified, this icon is used instead of <see cref="SymbolKind" />.
diff --git a/src/LanguageServer/Protocol/Protocol/Hover.cs b/src/LanguageServer/Protocol/Protocol/Hover.cs
index 82874f9c10149..0f73a4a751d67 100644
--- a/src/LanguageServer/Protocol/Protocol/Hover.cs
+++ b/src/LanguageServer/Protocol/Protocol/Hover.cs
@@ -19,7 +19,9 @@ internal class Hover
         /// </summary>
         [JsonPropertyName("contents")]
         [JsonRequired]
+#pragma warning disable CS0618 // MarkedString is obsolete but this property is not
         public SumType<string, MarkedString, SumType<string, MarkedString>[], MarkupContent> Contents { get; set; }
+#pragma warning restore CS0618
 
         /// <summary>
         /// An optional range inside a text document that is used to visualize the applicable
diff --git a/src/LanguageServer/Protocol/Protocol/Internal/Converters/VSInternalExtensionUtilities.cs b/src/LanguageServer/Protocol/Protocol/Internal/Converters/VSInternalExtensionUtilities.cs
index 15e162c6c13eb..d16f7f7ca20e0 100644
--- a/src/LanguageServer/Protocol/Protocol/Internal/Converters/VSInternalExtensionUtilities.cs
+++ b/src/LanguageServer/Protocol/Protocol/Internal/Converters/VSInternalExtensionUtilities.cs
@@ -41,7 +41,9 @@ private static void AddConverters(IList<JsonConverter> converters)
         AddOrReplaceConverter<Location, VSInternalLocation>();
         AddOrReplaceConverter<VSProjectContext, VSInternalProjectContext>();
         AddOrReplaceConverter<ServerCapabilities, VSInternalServerCapabilities>();
+#pragma warning disable CS0618 // SymbolInformation is obsolete but we need the converter regardless
         AddOrReplaceConverter<SymbolInformation, VSInternalSymbolInformation>();
+#pragma warning restore CS0618
         AddOrReplaceConverter<ReferenceParams, VSInternalReferenceParams>();
         AddOrReplaceConverter<SignatureInformation, VSInternalSignatureInformation>();
         AddOrReplaceConverter<TextDocumentClientCapabilities, VSInternalTextDocumentClientCapabilities>();
diff --git a/src/LanguageServer/Protocol/Protocol/Internal/VSInternalHover.cs b/src/LanguageServer/Protocol/Protocol/Internal/VSInternalHover.cs
index 44fa7428eb2d4..d7df82cad3c5b 100644
--- a/src/LanguageServer/Protocol/Protocol/Internal/VSInternalHover.cs
+++ b/src/LanguageServer/Protocol/Protocol/Internal/VSInternalHover.cs
@@ -29,6 +29,8 @@ internal class VSInternalHover : Hover
         /// </remarks>
         [JsonPropertyName("contents")]
         [JsonRequired]
+#pragma warning disable CS0618 // MarkedString is obsolete but this property is not
         public new SumType<SumType<string, MarkedString>, SumType<string, MarkedString>[], MarkupContent>? Contents { get; set; }
+#pragma warning restore CS0618
     }
 }
diff --git a/src/LanguageServer/Protocol/Protocol/Methods.Document.cs b/src/LanguageServer/Protocol/Protocol/Methods.Document.cs
index 7614db428cf47..cf9ee709397e9 100644
--- a/src/LanguageServer/Protocol/Protocol/Methods.Document.cs
+++ b/src/LanguageServer/Protocol/Protocol/Methods.Document.cs
@@ -144,7 +144,9 @@ partial class Methods
     /// <summary>
     /// Strongly typed message object for 'textDocument/documentSymbol'.
     /// </summary>
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this property is not
     public static readonly LspRequest<DocumentSymbolParams, SumType<SymbolInformation[], DocumentSymbol[]>?> TextDocumentDocumentSymbol = new(TextDocumentDocumentSymbolName);
+#pragma warning restore CS0618
 
     /// <summary>
     /// Method name for 'textDocument/semanticTokens'.
diff --git a/src/LanguageServer/Protocol/Protocol/Methods.Workspace.cs b/src/LanguageServer/Protocol/Protocol/Methods.Workspace.cs
index b8e80b72864d7..811c75796666c 100644
--- a/src/LanguageServer/Protocol/Protocol/Methods.Workspace.cs
+++ b/src/LanguageServer/Protocol/Protocol/Methods.Workspace.cs
@@ -23,7 +23,9 @@ partial class Methods
     /// <summary>
     /// Strongly typed message object for 'workspace/symbol'.
     /// </summary>
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this property is not
     public static readonly LspRequest<WorkspaceSymbolParams, SumType<SymbolInformation[], WorkspaceSymbol[]>?> WorkspaceSymbol = new(WorkspaceSymbolName);
+#pragma warning restore CS0618
 
     /// <summary>
     /// Method name for 'workspaceSymbol/resolve'.
diff --git a/src/LanguageServer/Protocol/Protocol/WorkspaceSymbolParams.cs b/src/LanguageServer/Protocol/Protocol/WorkspaceSymbolParams.cs
index 954760df677dc..aa00f30785f83 100644
--- a/src/LanguageServer/Protocol/Protocol/WorkspaceSymbolParams.cs
+++ b/src/LanguageServer/Protocol/Protocol/WorkspaceSymbolParams.cs
@@ -13,7 +13,10 @@ namespace Roslyn.LanguageServer.Protocol
     /// See the <see href="https://microsoft.github.io/language-server-protocol/specifications/specification-current/#workspaceSymbolParams">Language Server Protocol specification</see> for additional information.
     /// </para>
     /// </summary>
-    internal class WorkspaceSymbolParams : IPartialResultParams<SumType<SymbolInformation[], WorkspaceSymbol[]>>, IWorkDoneProgressParams
+    internal class WorkspaceSymbolParams
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this class is not
+        : IPartialResultParams<SumType<SymbolInformation[], WorkspaceSymbol[]>>, IWorkDoneProgressParams
+#pragma warning restore CS0618
     {
         /// <summary>
         /// Gets or sets the query (a non-empty string).
@@ -25,7 +28,9 @@ internal class WorkspaceSymbolParams : IPartialResultParams<SumType<SymbolInform
         /// <inheritdoc/>
         [JsonPropertyName(Methods.PartialResultTokenName)]
         [JsonIgnore(Condition = JsonIgnoreCondition.WhenWritingNull)]
+#pragma warning disable CS0618 // SymbolInformation is obsolete but this property is not
         public IProgress<SumType<SymbolInformation[], WorkspaceSymbol[]>>? PartialResultToken { get; set; }
+#pragma warning restore CS0618
 
         /// <inheritdoc/>
         [JsonPropertyName(Methods.WorkDoneTokenName)]