Be consistent with error suppression

Avoiding pragmas wherever possible.

Author: andyleejordan

.editorconfig

@@ -54,11 +54,11 @@ dotnet_diagnostic.CA1068.severity = error
 # CA1501: Avoid excessive inheritance
 dotnet_diagnostic.CA1501.severity = error
 # CA1502: Avoid excessive complexity
-dotnet_diagnostic.CA1502.severity = warning
+dotnet_diagnostic.CA1502.severity = silent
 # CA1505: Avoid unmaintainable code
 dotnet_diagnostic.CA1505.severity = error
 # CA1506: Avoid excessive class coupling
-dotnet_diagnostic.CA1506.severity = warning
+dotnet_diagnostic.CA1506.severity = silent
 # CA1507: Use nameof in place of string
 dotnet_diagnostic.CA1507.severity = error
 # CA1508: Avoid dead conditional code

src/PowerShellEditorServices.Hosting/Commands/StartEditorServicesCommand.cs

@@ -215,7 +215,7 @@ protected override void BeginProcessing()
         }
 #pragma warning restore IDE0022
 
-        [System.Diagnostics.CodeAnalysis.SuppressMessage("Design", "CA1031:Do not catch general exception types", Justification = "Uses ThrowTerminatingError() instead")]
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD002:Avoid problematic synchronous waits", Justification = "We have to wait here, it's the whole program.")]
         protected override void EndProcessing()
         {
             _logger.Log(PsesLogLevel.Diagnostic, "Beginning EndProcessing block");
@@ -232,9 +232,7 @@ protected override void EndProcessing()
                 using EditorServicesLoader psesLoader = EditorServicesLoader.Create(_logger, editorServicesConfig, SessionDetailsPath, _loggerUnsubscribers);
                 _logger.Log(PsesLogLevel.Verbose, "Loading EditorServices");
                 // Synchronously start editor services and wait here until it shuts down.
-#pragma warning disable VSTHRD002
                 psesLoader.LoadAndRunEditorServicesAsync().GetAwaiter().GetResult();
-#pragma warning restore VSTHRD002
             }
             catch (Exception e)
             {

src/PowerShellEditorServices.Hosting/EditorServicesLoader.cs

@@ -382,19 +382,18 @@ private void ValidateConfiguration()
             }
         }
 
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Performance", "CA1825:Avoid zero-length array allocations", Justification = "Cannot use Array.Empty, since it must work in net452")]
         private static Version GetPSVersion()
         {
             // In order to read the $PSVersionTable variable,
             // we are forced to create a new runspace to avoid concurrency issues,
             // which is expensive.
             // Rather than do that, we instead go straight to the source,
             // which is a static property, internal in WinPS and public in PS 6+
-#pragma warning disable CA1825
             return typeof(PSObject).Assembly
                 .GetType("System.Management.Automation.PSVersionInfo")
                 .GetMethod("get_PSVersion", BindingFlags.Static | BindingFlags.Public | BindingFlags.NonPublic)
-                .Invoke(null, new object[0] /* Cannot use Array.Empty, since it must work in net452 */) as Version;
-#pragma warning restore CA1825
+                .Invoke(null, new object[0]) as Version;
         }
     }
 }

src/PowerShellEditorServices.Hosting/Internal/EditorServicesRunner.cs

@@ -304,6 +304,7 @@ private void WriteStartupBanner()
             _config.PSHost.UI.WriteLine(_config.StartupBanner);
         }
 
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD110:Observe result of async calls", Justification = "Intentionally fire and forget.")]
         private void DebugServer_OnSessionEnded(object sender, EventArgs args)
         {
             _logger.Log(PsesLogLevel.Verbose, "Debug session ended, restarting debug service...");

src/PowerShellEditorServices/Extensions/EditorContext.cs

@@ -92,9 +92,8 @@ public void SetSelection(
         /// Sets a selection in the host editor's active buffer.
         /// </summary>
         /// <param name="selectionRange">The range of the selection.</param>
-        #pragma warning disable VSTHRD002
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD002:Avoid problematic synchronous waits", Justification = "Supporting synchronous API.")]
         public void SetSelection(FileRange selectionRange) => editorOperations.SetSelectionAsync(selectionRange.ToBufferRange()).Wait();
-        #pragma warning restore VSTHRD002
 
         #endregion
     }

src/PowerShellEditorServices/Extensions/EditorObject.cs

@@ -115,13 +115,12 @@ internal EditorObject(
         /// at the time this method is invoked.
         /// </summary>
         /// <returns>A instance of the EditorContext class.</returns>
-        #pragma warning disable VSTHRD002, VSTHRD104
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD002:Avoid problematic synchronous waits", Justification = "Supporting synchronous API.")]
         public EditorContext GetEditorContext() => _editorOperations.GetEditorContextAsync().Result;
-        #pragma warning restore VSTHRD002, VSTHRD104
 
         internal void SetAsStaticInstance()
         {
-            EditorObject.Instance = this;
+            Instance = this;
             s_editorObjectReady.TrySetResult(true);
         }
     }

src/PowerShellEditorServices/Extensions/FileContext.cs

@@ -208,14 +208,13 @@ public void InsertText(
         /// </summary>
         /// <param name="textToInsert">The text string to insert.</param>
         /// <param name="insertRange">The buffer range which will be replaced by the string.</param>
-        #pragma warning disable VSTHRD002
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD002:Avoid problematic synchronous waits", Justification = "Supporting synchronous API.")]
         public void InsertText(string textToInsert, IFileRange insertRange)
         {
             editorOperations
                 .InsertTextAsync(scriptFile.DocumentUri.ToString(), textToInsert, insertRange.ToBufferRange())
                 .Wait();
         }
-        #pragma warning restore VSTHRD002
 
         #endregion
 
@@ -233,7 +232,7 @@ public void InsertText(string textToInsert, IFileRange insertRange)
         /// the path where the file should be saved,
         /// including the file name with extension as the leaf
         /// </param>
-        #pragma warning disable VSTHRD002
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD002:Avoid problematic synchronous waits", Justification = "Supporting synchronous API.")]
         public void SaveAs(string newFilePath)
         {
             // Do some validation here so that we can provide a helpful error if the path won't work
@@ -248,7 +247,6 @@ public void SaveAs(string newFilePath)
 
             editorOperations.SaveFileAsync(scriptFile.FilePath, newFilePath).Wait();
         }
-        #pragma warning restore VSTHRD002
 
         #endregion
     }

src/PowerShellEditorServices/GlobalSuppressions.cs

@@ -1,4 +1,6 @@
-#pragma warning disable IDE0073
+// Copyright (c) Microsoft Corporation.
+// Licensed under the MIT License.
+
 #if NET5_0_OR_GREATER
 [assembly: System.Runtime.CompilerServices.TypeForwardedTo(typeof(System.Runtime.CompilerServices.IsExternalInit))]
 #else

src/PowerShellEditorServices/IsExternalInit.cs

@@ -70,9 +70,7 @@ public PsesLanguageServer(
         /// cref="PsesServiceCollectionExtensions.AddPsesLanguageServices"/>.
         /// </remarks>
         /// <returns>A task that completes when the server is ready and listening.</returns>
-#pragma warning disable CA1506 // Coupling complexity we don't care about
         public async Task StartAsync()
-#pragma warning restore CA1506
         {
             LanguageServer = await OmniSharp.Extensions.LanguageServer.Server.LanguageServer.From(options =>
             {

src/PowerShellEditorServices/Server/PsesLanguageServer.cs

@@ -17,6 +17,7 @@ namespace Microsoft.PowerShell.EditorServices.Server
 {
     internal static class PsesServiceCollectionExtensions
     {
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "VSTHRD110:Observe result of async calls", Justification = "Using lazy initialization.")]
         public static IServiceCollection AddPsesLanguageServices(
             this IServiceCollection collection,
             HostStartupInfo hostStartupInfo)
@@ -48,9 +49,7 @@ public static IServiceCollection AddPsesLanguageServices(
                         // is ready, it will be available. NOTE: We cannot await this because it
                         // uses a lazy initialization to avoid a race with the dependency injection
                         // framework, see the EditorObject class for that!
-#pragma warning disable VSTHRD110
                         extensionService.InitializeAsync();
-#pragma warning restore VSTHRD110
                         return extensionService;
                     })
                 .AddSingleton<AnalysisService>();

src/PowerShellEditorServices/Server/PsesServiceCollectionExtensions.cs

@@ -35,9 +35,7 @@ public LegacyReadLine(
             _onIdleAction = onIdleAction;
         }
 
-#pragma warning disable CA1502 // Cyclomatic complexity we don't care about
         public override string ReadLine(CancellationToken cancellationToken)
-#pragma warning restore CA1502
         {
             string inputBeforeCompletion = null;
             string inputAfterCompletion = null;

src/PowerShellEditorServices/Services/PowerShell/Console/LegacyReadLine.cs

@@ -29,9 +29,7 @@ namespace Microsoft.PowerShell.EditorServices.Services.PowerShell.Host
     using Microsoft.PowerShell.EditorServices.Server;
     using OmniSharp.Extensions.DebugAdapter.Protocol.Server;
 
-#pragma warning disable CA1506 // Coupling complexity we don't care about
     internal class PsesInternalHost : PSHost, IHostSupportsInteractiveSession, IRunspaceContext, IInternalPowerShellExecutionService
-#pragma warning restore CA1506
     {
         internal const string DefaultPrompt = "> ";
 

src/PowerShellEditorServices/Services/PowerShell/Host/PsesInternalHost.cs

@@ -41,6 +41,7 @@ public DidChangeWatchedFilesHandler(
     public DidChangeWatchedFilesRegistrationOptions GetRegistrationOptions(
         DidChangeWatchedFilesCapability capability,
         ClientCapabilities clientCapabilities)
+#pragma warning disable CS8601 // Possible null reference assignment (it's from the library).
         => new()
         {
             Watchers = new[]
@@ -52,6 +53,7 @@ public DidChangeWatchedFilesRegistrationOptions GetRegistrationOptions(
                 },
             },
         };
+#pragma warning restore CS8601 // Possible null reference assignment.
 
     public Task<Unit> Handle(DidChangeWatchedFilesParams request, CancellationToken cancellationToken)
     {

src/PowerShellEditorServices/Services/TextDocument/Handlers/DidChangeWatchedFilesHandler.cs

@@ -25,9 +25,7 @@ internal static class TokenOperations
         /// <summary>
         /// Extracts all of the unique foldable regions in a script given the list tokens
         /// </summary>
-#pragma warning disable CA1502 // Cyclomatic complexity we don't care about
         internal static FoldingReferenceList FoldableReferences(Token[] tokens)
-#pragma warning restore CA1502
         {
             FoldingReferenceList refList = new();
 

src/PowerShellEditorServices/Services/TextDocument/TokenOperations.cs

@@ -23,9 +23,8 @@ public class StdioServerProcess : ServerProcess
         /// <summary>
         ///     The current server process (if any).
         /// </summary>
-#pragma warning disable CA2213
+        [System.Diagnostics.CodeAnalysis.SuppressMessage("Usage", "CA2213:Disposable fields should be disposed", Justification = "It is diposed but with a lock.")]
         private Process _serverProcess;
-#pragma warning restore CA2213
 
         /// <summary>
         ///     Create a new <see cref="StdioServerProcess"/>.