fix: New Analysis Errors found after NET10 update

Author: JustinGrote

src/PowerShellEditorServices/Services/CodeLens/ReferencesCodeLensProvider.cs

@@ -25,7 +25,6 @@ internal class ReferencesCodeLensProvider : ICodeLensProvider
         /// </summary>
         private readonly IDocumentSymbolProvider _symbolProvider;
         private readonly SymbolsService _symbolsService;
-        private readonly WorkspaceService _workspaceService;
 
         public static string Id => nameof(ReferencesCodeLensProvider);
 
@@ -38,11 +37,9 @@ internal class ReferencesCodeLensProvider : ICodeLensProvider
         /// <summary>
         /// Construct a new ReferencesCodeLensProvider for a given EditorSession.
         /// </summary>
-        /// <param name="workspaceService"></param>
         /// <param name="symbolsService"></param>
-        public ReferencesCodeLensProvider(WorkspaceService workspaceService, SymbolsService symbolsService)
+        public ReferencesCodeLensProvider(SymbolsService symbolsService)
         {
-            _workspaceService = workspaceService;
             _symbolsService = symbolsService;
             // TODO: Pull this from components
             _symbolProvider = new ScriptDocumentSymbolProvider();

src/PowerShellEditorServices/Services/PowerShell/Debugging/IPowerShellDebugContext.cs

@@ -13,13 +13,13 @@ internal interface IPowerShellDebugContext
 
         DebuggerStopEventArgs LastStopEventArgs { get; }
 
-        public bool IsDebuggingRemoteRunspace { get; set; }
+        bool IsDebuggingRemoteRunspace { get; set; }
 
-        public event Action<object, DebuggerStopEventArgs> DebuggerStopped;
+        event Action<object, DebuggerStopEventArgs> DebuggerStopped;
 
-        public event Action<object, DebuggerResumingEventArgs> DebuggerResuming;
+        event Action<object, DebuggerResumingEventArgs> DebuggerResuming;
 
-        public event Action<object, BreakpointUpdatedEventArgs> BreakpointUpdated;
+        event Action<object, BreakpointUpdatedEventArgs> BreakpointUpdated;
 
         void Continue();
 

src/PowerShellEditorServices/Services/PowerShell/Debugging/PowerShellDebugContext.cs

@@ -114,9 +114,9 @@ public void SetDebugResuming(DebuggerResumeAction debuggerResumeAction)
             // then this came over LSP and we need to set it.
             _psesHost.SetExit();
 
-            if (LastStopEventArgs is not null)
+            if (LastStopEventArgs is { } lastStopEventArgs)
             {
-                LastStopEventArgs.ResumeAction = debuggerResumeAction;
+                lastStopEventArgs.ResumeAction = debuggerResumeAction;
             }
 
             // We need to tell whatever is happening right now in the debug prompt to wrap up so we

src/PowerShellEditorServices/Services/PowerShell/Execution/SynchronousTask.cs

@@ -20,15 +20,15 @@ internal interface ISynchronousTask
 
     internal abstract class SynchronousTask<TResult> : ISynchronousTask
     {
-        private readonly TaskCompletionSource<TResult> _taskCompletionSource;
+        private TaskCompletionSource<TResult> _taskCompletionSource { get; }
 
-        private readonly CancellationToken _taskRequesterCancellationToken;
+        private CancellationToken _taskRequesterCancellationToken { get; }
 
-        private bool _executionCanceled;
+        private bool _executionCanceled { get; set; }
 
-        private TResult _result;
+        private TResult _result { get; set; }
 
-        private ExceptionDispatchInfo _exceptionInfo;
+        private ExceptionDispatchInfo _exceptionInfo { get; set; }
 
         protected SynchronousTask(
             ILogger logger,

src/PowerShellEditorServices/Services/Symbols/ReferenceTable.cs

@@ -22,7 +22,7 @@ internal sealed class ReferenceTable
 
     private readonly ConcurrentDictionary<string, ConcurrentBag<SymbolReference>> _symbolReferences = new(StringComparer.OrdinalIgnoreCase);
 
-    private bool _isInited;
+    private bool _isInited { get; set; }
 
     public ReferenceTable(ScriptFile parent) => _parent = parent;
 

src/PowerShellEditorServices/Services/Symbols/SymbolsService.cs

@@ -70,7 +70,7 @@ public SymbolsService(
             _codeLensProviders = new ConcurrentDictionary<string, ICodeLensProvider>();
             if (configurationService.CurrentSettings.EnableReferencesCodeLens)
             {
-                ReferencesCodeLensProvider referencesProvider = new(_workspaceService, this);
+                ReferencesCodeLensProvider referencesProvider = new(this);
                 _ = _codeLensProviders.TryAdd(referencesProvider.ProviderId, referencesProvider);
             }
 
@@ -495,7 +495,7 @@ void CloseUnopenedFiles()
                     return;
                 }
 
-                TryRegisterCodeLensProvider(new ReferencesCodeLensProvider(_workspaceService, this));
+                TryRegisterCodeLensProvider(new ReferencesCodeLensProvider(this));
                 return;
             }
 

src/PowerShellEditorServices/Services/Workspace/RemoteFileManagerService.cs

@@ -691,10 +691,7 @@ private void RemovePSEditFunction(IRunspaceInfo runspaceInfo)
             }
             try
             {
-                if (runspaceInfo.Runspace.Events != null)
-                {
-                    runspaceInfo.Runspace.Events.ReceivedEvents.PSEventReceived -= HandlePSEventReceivedAsync;
-                }
+                runspaceInfo.Runspace.Events.ReceivedEvents.PSEventReceived -= HandlePSEventReceivedAsync;
 
                 if (runspaceInfo.Runspace.RunspaceStateInfo.State == RunspaceState.Opened)
                 {