Fix XML documentation warnings

Author: andyleejordan

.editorconfig

@@ -36,6 +36,12 @@ dotnet_diagnostic.CS0414.severity = error
 dotnet_diagnostic.CS0618.severity = suggestion
 # CS0649: Uninitialized private or internal field declaration that is never assigned a value
 dotnet_diagnostic.CS0649.severity = error
+# CS1570: Parameter has no matching param tag in the XML comment
+dotnet_diagnostic.CS1570.severity = silent
+# CS1574: XML comment has cref attribute that could not be resolved.
+dotnet_diagnostic.CS1574.severity = suggestion
+# CS1591: Missing XML comment for publicly visible type or member
+dotnet_diagnostic.CS1591.severity = silent
 # CS1998: This async method lacks 'await' operators and will run synchronously
 dotnet_diagnostic.CS1998.severity = suggestion
 # CS4014: Consider applying the await operator to the result of the call

src/PowerShellEditorServices.Hosting/Configuration/HostLogger.cs

@@ -15,9 +15,9 @@ namespace Microsoft.PowerShell.EditorServices.Hosting
     /// User-facing log level for editor services configuration.
     /// </summary>
     /// <remarks>
-    /// The underlying values of this enum attempt to align to both <see
-    /// cref="Microsoft.Logging.Extensions.LogLevel"</see> and <see
-    /// cref="Serilog.Events.LogEventLevel"</see>.
+    /// The underlying values of this enum attempt to align to both
+    /// <see cref="Microsoft.Extensions.Logging.LogLevel" /> and
+    /// <see cref="Serilog.Events.LogEventLevel" />.
     /// </remarks>
     public enum PsesLogLevel
     {

src/PowerShellEditorServices.Hosting/EditorServicesLoader.cs

@@ -61,6 +61,7 @@ public static EditorServicesLoader Create(
         /// <param name="logger">The host logger to use.</param>
         /// <param name="hostConfig">The host configuration to start editor services with.</param>
         /// <param name="sessionFileWriter">The session file writer to write the session file with.</param>
+        /// <param name="loggersToUnsubscribe">The loggers to unsubscribe form writing to the terminal.</param>
         /// <returns></returns>
         public static EditorServicesLoader Create(
             HostLogger logger,

src/PowerShellEditorServices/Extensions/Api/EditorExtensionServiceProvider.cs

@@ -54,11 +54,6 @@ internal EditorExtensionServiceProvider(IServiceProvider serviceProvider)
         /// </summary>
         public ILanguageServerService LanguageServer { get; }
 
-        /// <summary>
-        /// Service providing document symbol provider registration.
-        /// </summary>
-        // public IDocumentSymbolService DocumentSymbols { get; }
-
         /// <summary>
         /// Service providing extension command registration and functionality.
         /// </summary>
@@ -93,7 +88,7 @@ internal EditorExtensionServiceProvider(IServiceProvider serviceProvider)
         /// <summary>
         /// Get an underlying service object from PSES by type name.
         /// </summary>
-        /// <param name="psesServiceFullTypeName">The full type name of the service to get.</param>
+        /// <param name="fullTypeName">The full type name of the service to get.</param>
         /// <param name="assemblyName">The assembly name from which the service comes.</param>
         /// <returns>The service object requested, or null if no service of that type name exists.</returns>
         /// <remarks>

src/PowerShellEditorServices/Extensions/EditorObject.cs

@@ -73,6 +73,7 @@ public class EditorObject
         /// <summary>
         /// Creates a new instance of the EditorObject class.
         /// </summary>
+        /// <param name="serviceProvider">The service provider?</param>
         /// <param name="extensionService">An ExtensionService which handles command registration.</param>
         /// <param name="editorOperations">An IEditorOperations implementation which handles operations in the host editor.</param>
         internal EditorObject(

src/PowerShellEditorServices/Extensions/FileContext.cs

@@ -122,7 +122,7 @@ public string GetText(FileRange bufferRange)
         /// <summary>
         /// Gets the file content in the specified range as an array of strings.
         /// </summary>
-        /// <param name="bufferRange">The buffer range for which content will be extracted.</param>
+        /// <param name="fileRange">The buffer range for which content will be extracted.</param>
         /// <returns>An array of strings, each representing a line in the file within the specified range.</returns>
         public string[] GetTextLines(FileRange fileRange) => scriptFile.GetLinesInRange(fileRange.ToBufferRange());
 

src/PowerShellEditorServices/Hosting/EditorServicesServerFactory.cs

@@ -36,7 +36,7 @@ internal sealed class EditorServicesServerFactory : IDisposable
         /// <remarks>
         /// <para>
         /// This can only be called once because it sets global state (the logger) and that call is
-        /// in <see cref="EditorServicesRunner"/>.
+        /// in <see cref="Hosting.EditorServicesRunner" />.
         /// </para>
         /// <para>
         /// TODO: Why is this a static function wrapping a constructor instead of just a
@@ -45,7 +45,7 @@ internal sealed class EditorServicesServerFactory : IDisposable
         /// </remarks>
         /// <param name="logPath">The path of the log file to use.</param>
         /// <param name="minimumLogLevel">The minimum log level to use.</param>
-        /// <returns></returns>
+        /// <param name="hostLogger">The host logger?</param>
         public static EditorServicesServerFactory Create(string logPath, int minimumLogLevel, IObservable<(int logLevel, string message)> hostLogger)
         {
             Log.Logger = new LoggerConfiguration()
@@ -78,7 +78,7 @@ public static EditorServicesServerFactory Create(string logPath, int minimumLogL
         /// Create the LSP server.
         /// </summary>
         /// <remarks>
-        /// This is only called once and that's in <see cref="EditorServicesRunner"/>.
+        /// This is only called once and that's in <see cref="Hosting.EditorServicesRunner"/>.
         /// </remarks>
         /// <param name="inputStream">The protocol transport input stream.</param>
         /// <param name="outputStream">The protocol transport output stream.</param>
@@ -94,7 +94,7 @@ public PsesLanguageServer CreateLanguageServer(
         /// Create the debug server given a language server instance.
         /// </summary>
         /// <remarks>
-        /// This is only called once and that's in <see cref="EditorServicesRunner"/>.
+        /// This is only called once and that's in <see cref="Hosting.EditorServicesRunner"/>.
         /// </remarks>
         /// <param name="inputStream">The protocol transport input stream.</param>
         /// <param name="outputStream">The protocol transport output stream.</param>
@@ -116,7 +116,7 @@ public PsesDebugServer CreateDebugServerWithLanguageServer(
         /// Create a new debug server based on an old one in an ended session.
         /// </summary>
         /// <remarks>
-        /// This is only called once and that's in <see cref="EditorServicesRunner"/>.
+        /// This is only called once and that's in <see cref="Hosting.EditorServicesRunner"/>.
         /// </remarks>
         /// <param name="inputStream">The protocol transport input stream.</param>
         /// <param name="outputStream">The protocol transport output stream.</param>

src/PowerShellEditorServices/Hosting/HostStartupInfo.cs

@@ -101,7 +101,7 @@ public sealed class HostStartupInfo
         /// The minimum log level of log events to be logged.
         /// </summary>
         /// <remarks>
-        /// This is cast to all of <see cref="PsesLogLevel"/>, <see
+        /// This is cast to all of <see cref="Hosting.PsesLogLevel"/>, <see
         /// cref="Microsoft.Extensions.Logging.LogLevel"/>, and <see
         /// cref="Serilog.Events.LogEventLevel"/>, hence it is an <c>int</c>.
         /// </remarks>
@@ -131,8 +131,7 @@ public sealed class HostStartupInfo
         /// </param>
         /// <param name="version">The host application's version.</param>
         /// <param name="psHost">The PowerShell host to use.</param>
-        /// <param name="allUsersProfilePath">The path to the shared profile.</param>
-        /// <param name="currentUsersProfilePath">The path to the user specific profile.</param>
+        /// <param name="profilePaths">The set of profile paths.</param>
         /// <param name="featureFlags">Flags of features to enable.</param>
         /// <param name="additionalModules">Names or paths of additional modules to import.</param>
         /// <param name="initialSessionState">The language mode inherited from the orginal PowerShell process. This will be used when creating runspaces so that we honor the same initialSessionState including allowed modules, cmdlets and language mode.</param>

src/PowerShellEditorServices/Services/Analysis/AnalysisService.cs

@@ -131,10 +131,8 @@ public AnalysisService(
         /// Sets up a script analysis run, eventually returning the result.
         /// </summary>
         /// <param name="filesToAnalyze">The files to run script analysis on.</param>
-        /// <param name="cancellationToken">A cancellation token to cancel this call with.</param>
         /// <returns>A task that finishes when script diagnostics have been published.</returns>
-        public void StartScriptDiagnostics(
-            ScriptFile[] filesToAnalyze)
+        public void StartScriptDiagnostics(ScriptFile[] filesToAnalyze)
         {
             if (!_configurationService.CurrentSettings.ScriptAnalysis.Enable)
             {
@@ -219,7 +217,7 @@ public async Task<string> GetCommentHelpText(string functionText, string helpLoc
         /// <summary>
         /// Get the most recent corrections computed for a given script file.
         /// </summary>
-        /// <param name="documentUri">The URI string of the file to get code actions for.</param>
+        /// <param name="uri">The URI string of the file to get code actions for.</param>
         /// <returns>A threadsafe readonly dictionary of the code actions of the particular file.</returns>
         public async Task<IReadOnlyDictionary<string, IEnumerable<MarkerCorrection>>> GetMostRecentCodeActionsForFileAsync(DocumentUri uri)
         {
@@ -245,7 +243,7 @@ public async Task<IReadOnlyDictionary<string, IEnumerable<MarkerCorrection>>> Ge
         /// <summary>
         /// Event subscription method to be run when PSES configuration has been updated.
         /// </summary>
-        /// <param name="sender">The sender of the configuration update event.</param>
+        /// <param name="_">The sender of the configuration update event.</param>
         /// <param name="settings">The new language server settings.</param>
         public void OnConfigurationUpdated(object _, LanguageServerSettings settings)
         {

src/PowerShellEditorServices/Services/DebugAdapter/Debugging/BreakpointApiUtils.cs

@@ -167,6 +167,7 @@ public static Breakpoint SetBreakpoint(Debugger debugger, BreakpointDetailsBase
         /// <param name="condition">The expression that needs to be true for the breakpoint to be triggered.</param>
         /// <param name="hitCondition">The amount of times this line should be hit til the breakpoint is triggered.</param>
         /// <param name="logMessage">The log message to write instead of calling 'break'. In VS Code, this is called a 'logPoint'.</param>
+        /// <param name="errorMessage">The error message we might return.</param>
         /// <returns>ScriptBlock</returns>
         public static ScriptBlock GetBreakpointActionScriptBlock(string condition, string hitCondition, string logMessage, out string errorMessage)
         {