[Dev][Build] .NET 8 Upgrade

.github/workflows/DevHome-CI.yml

@@ -15,7 +15,7 @@ jobs:
         configuration: [Release, Debug]
         platform: [x64, x86, arm64]
         os: [windows-latest]
-        dotnet-version: ['6.0.x']
+        dotnet-version: ['8.0.x']
         exclude:
         - configuration: Debug
           platform: x64
@@ -77,11 +77,11 @@ jobs:
 
     - name: DevHome UnitTests
       if: ${{ matrix.platform != 'arm64' }}
-      run: cmd /c "$env:VSDevTestCmd" /Platform:${{ matrix.platform }} test\\bin\\${{ matrix.platform }}\\${{ matrix.configuration }}\\net6.0-windows10.0.22000.0\\DevHome.Test.dll
+      run: cmd /c "$env:VSDevTestCmd" /Platform:${{ matrix.platform }} test\\bin\\${{ matrix.platform }}\\${{ matrix.configuration }}\\net8.0-windows10.0.22000.0\\DevHome.Test.dll
 
     - name: Tools UnitTests
       if: ${{ matrix.platform != 'arm64' }}
       run: |
-        foreach ($UnitTestPath in (Get-ChildItem "tools\\*\\*UnitTest\\bin\\${{ matrix.platform }}\\${{ matrix.configuration }}\\net6.0-windows10.0.22000.0\\*.UnitTest.dll")) {
+        foreach ($UnitTestPath in (Get-ChildItem "tools\\*\\*UnitTest\\bin\\${{ matrix.platform }}\\${{ matrix.configuration }}\\net8.0-windows10.0.22000.0\\*.UnitTest.dll")) {
           cmd /c "$env:VSDevTestCmd" /Platform:${{ matrix.platform }} $UnitTestPath.FullName
         }

CoreWidgetProvider/CoreWidgetProvider.csproj

@@ -12,16 +12,16 @@
     <Nullable>enable</Nullable>
     <CsWinRTEnabled>false</CsWinRTEnabled>
     <Platforms>x86;x64;arm64</Platforms>
-    <RuntimeIdentifiers>win10-x86;win10-x64;win10-arm64</RuntimeIdentifiers>
+    <RuntimeIdentifiers>win-x86;win-x64;win-arm64</RuntimeIdentifiers>
     <PublishProfileFullPath Condition="'$(BuildingInsideVisualStudio)' != 'True'">$(SolutionDir)\src\Properties\PublishProfiles\win10-$(Platform).pubxml</PublishProfileFullPath>
   </PropertyGroup>
 
   <ItemGroup>
     <PackageReference Include="Microsoft.Management.Infrastructure" Version="2.0.0" />
-    <PackageReference Include="Microsoft.Windows.CsWin32" Version="0.2.206-beta">
+    <PackageReference Include="Microsoft.Windows.CsWin32" Version="0.3.49-beta">
       <PrivateAssets>all</PrivateAssets>
     </PackageReference>
-    <PackageReference Include="System.Diagnostics.PerformanceCounter" Version="7.0.0" />
+    <PackageReference Include="System.Diagnostics.PerformanceCounter" Version="8.0.0" />
   </ItemGroup>
 
   <ItemGroup>

CoreWidgetProvider/Helpers/CPUStats.cs

@@ -5,15 +5,15 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class CPUStats : IDisposable
+internal sealed class CPUStats : IDisposable
 {
     // CPU counters
     private readonly PerformanceCounter procPerf = new ("Processor Information", "% Processor Utility", "_Total");
     private readonly PerformanceCounter procPerformance = new ("Processor Information", "% Processor Performance", "_Total");
     private readonly PerformanceCounter procFrequency = new ("Processor Information", "Processor Frequency", "_Total");
     private readonly Dictionary<Process, PerformanceCounter> cpuCounters = new ();
 
-    internal class ProcessStats
+    internal sealed class ProcessStats
     {
         public Process? Process { get; set; }
 
@@ -31,7 +31,12 @@ internal class ProcessStats
     public CPUStats()
     {
         CpuUsage = 0;
-        ProcessCPUStats = new ProcessStats[3] { new ProcessStats(), new ProcessStats(), new ProcessStats() };
+        ProcessCPUStats =
+        [
+            new ProcessStats(),
+            new ProcessStats(),
+            new ProcessStats()
+        ];
 
         InitCPUPerfCounters();
     }

CoreWidgetProvider/Helpers/ChartHelper.cs

@@ -7,7 +7,7 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class ChartHelper
+internal sealed class ChartHelper
 {
     public enum ChartType
     {

CoreWidgetProvider/Helpers/DataManager.cs

@@ -5,7 +5,7 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class DataManager : IDisposable
+internal sealed class DataManager : IDisposable
 {
     private readonly SystemData systemData;
     private readonly DataType dataType;

CoreWidgetProvider/Helpers/GPUStats.cs

@@ -7,14 +7,14 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class GPUStats : IDisposable
+internal sealed class GPUStats : IDisposable
 {
     // GPU counters
     private readonly Dictionary<int, List<PerformanceCounter>> gpuCounters = new ();
 
     private readonly List<Data> stats = new ();
 
-    public class Data
+    public sealed class Data
     {
         public string? Name { get; set; }
 
@@ -79,12 +79,13 @@ public void GetGPUPerfCounters()
                         continue;
                     }
 
-                    if (!gpuCounters.ContainsKey(phys))
+                    if (!gpuCounters.TryGetValue(phys, out var value))
                     {
-                        gpuCounters.Add(phys, new ());
+                        value = new ();
+                        gpuCounters.Add(phys, value);
                     }
 
-                    gpuCounters[phys].Add(counter);
+                    value.Add(counter);
                 }
             }
         }

CoreWidgetProvider/Helpers/IconLoader.cs

@@ -10,13 +10,14 @@ public class IconLoader
     public static string GetIconAsBase64(string filename)
     {
         Log.Logger()?.ReportDebug(nameof(IconLoader), $"Asking for icon: {filename}");
-        if (!Base64ImageRegistry.ContainsKey(filename))
+        if (!Base64ImageRegistry.TryGetValue(filename, out var value))
         {
-            Base64ImageRegistry.Add(filename, ConvertIconToDataString(filename));
+            value = ConvertIconToDataString(filename);
+            Base64ImageRegistry.Add(filename, value);
             Log.Logger()?.ReportDebug(nameof(IconLoader), $"The icon {filename} was converted and is now stored.");
         }
 
-        return Base64ImageRegistry[filename];
+        return value;
     }
 
     private static string ConvertIconToDataString(string fileName)

CoreWidgetProvider/Helpers/MemoryStats.cs

@@ -7,7 +7,7 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class MemoryStats : IDisposable
+internal sealed class MemoryStats : IDisposable
 {
     private readonly PerformanceCounter memCommitted = new ("Memory", "Committed Bytes", string.Empty);
     private readonly PerformanceCounter memCached = new ("Memory", "Cache Bytes", string.Empty);

CoreWidgetProvider/Helpers/NetworkStats.cs

@@ -5,15 +5,15 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class NetworkStats : IDisposable
+internal sealed class NetworkStats : IDisposable
 {
     private readonly Dictionary<string, List<PerformanceCounter>> networkCounters = new ();
 
     private Dictionary<string, Data> NetworkUsages { get; set; } = new Dictionary<string, Data>();
 
     private Dictionary<string, List<float>> NetChartValues { get; set; } = new Dictionary<string, List<float>>();
 
-    public class Data
+    public sealed class Data
     {
         public float Usage
         {
@@ -114,12 +114,12 @@ public Data GetNetworkUsage(int networkIndex)
         }
 
         var currNetworkName = NetChartValues.ElementAt(networkIndex).Key;
-        if (!NetworkUsages.ContainsKey(currNetworkName))
+        if (!NetworkUsages.TryGetValue(currNetworkName, out var value))
         {
             return new Data();
         }
 
-        return NetworkUsages[currNetworkName];
+        return value;
     }
 
     public int GetPrevNetworkIndex(int networkIndex)

CoreWidgetProvider/Helpers/Resources.cs

@@ -51,8 +51,8 @@ public static string ReplaceIdentifers(string str, string[] resourceIdentifiers,
     // These are all the string identifiers that appear in widgets.
     public static string[] GetWidgetResourceIdentifiers()
     {
-        return new string[]
-        {
+        return
+        [
             "Widget_Template/Loading",
             "Widget_Template_Tooltip/Submit",
             "SSH_Widget_Template/Name",
@@ -89,6 +89,6 @@ public static string[] GetWidgetResourceIdentifiers()
             "CPUUsage_Widget_Template/End_Process",
             "Widget_Template_Button/Save",
             "Widget_Template_Button/Cancel",
-        };
+        ];
     }
 }

CoreWidgetProvider/Helpers/SystemData.cs

@@ -3,7 +3,7 @@
 
 namespace CoreWidgetProvider.Helpers;
 
-internal class SystemData : IDisposable
+internal sealed class SystemData : IDisposable
 {
     public static MemoryStats MemStats { get; set; } = new MemoryStats();
 

CoreWidgetProvider/Widgets/CoreWidget.cs

@@ -125,10 +125,10 @@ public virtual string GetData(WidgetPageState page)
 
     protected string GetTemplateForPage(WidgetPageState page)
     {
-        if (Template.ContainsKey(page))
+        if (Template.TryGetValue(page, out var value))
         {
             Log.Logger()?.ReportDebug(Name, ShortId, $"Using cached template for {page}");
-            return Template[page];
+            return value;
         }
 
         try

CoreWidgetProvider/Widgets/SSHWalletWidget.cs

@@ -13,17 +13,17 @@
 
 namespace CoreWidgetProvider.Widgets;
 
-internal class SSHWalletWidget : CoreWidget
+internal sealed class SSHWalletWidget : CoreWidget
 {
-    protected static readonly string DefaultConfigFile = Environment.GetFolderPath(Environment.SpecialFolder.UserProfile) + "\\.ssh\\config";
+    private static readonly string DefaultConfigFile = Environment.GetFolderPath(Environment.SpecialFolder.UserProfile) + "\\.ssh\\config";
 
     private static readonly Regex HostRegex = new (@"^Host\s+(\S*)\s*$", RegexOptions.Compiled | RegexOptions.IgnoreCase | RegexOptions.Multiline);
 
     private FileSystemWatcher? FileWatcher { get; set; }
 
-    protected static readonly new string Name = nameof(SSHWalletWidget);
+    private static readonly new string Name = nameof(SSHWalletWidget);
 
-    protected string ConfigFile
+    private string ConfigFile
     {
         get => State();
 
@@ -401,7 +401,7 @@ private void SetConfigure()
     }
 }
 
-internal class DataPayload
+internal sealed class DataPayload
 {
     public string? ConfigFile
     {
@@ -411,6 +411,6 @@ internal class DataPayload
 
 [JsonSourceGenerationOptions(WriteIndented = true)]
 [JsonSerializable(typeof(DataPayload))]
-internal partial class SourceGenerationContext : JsonSerializerContext
+internal sealed partial class SourceGenerationContext : JsonSerializerContext
 {
 }

CoreWidgetProvider/Widgets/SystemCPUUsageWidget.cs

@@ -8,11 +8,11 @@
 using Microsoft.Windows.Widgets.Providers;
 
 namespace CoreWidgetProvider.Widgets;
-internal class SystemCPUUsageWidget : CoreWidget, IDisposable
+internal sealed class SystemCPUUsageWidget : CoreWidget, IDisposable
 {
     private static Dictionary<string, string> Templates { get; set; } = new ();
 
-    protected static readonly new string Name = nameof(SystemCPUUsageWidget);
+    private static readonly new string Name = nameof(SystemCPUUsageWidget);
 
     private readonly DataManager dataManager;
 

CoreWidgetProvider/Widgets/SystemGPUUsageWidget.cs

@@ -9,11 +9,11 @@
 using Microsoft.Windows.Widgets.Providers;
 
 namespace CoreWidgetProvider.Widgets;
-internal class SystemGPUUsageWidget : CoreWidget, IDisposable
+internal sealed class SystemGPUUsageWidget : CoreWidget, IDisposable
 {
     private static Dictionary<string, string> Templates { get; set; } = new ();
 
-    protected static readonly new string Name = nameof(SystemGPUUsageWidget);
+    private static readonly new string Name = nameof(SystemGPUUsageWidget);
 
     private readonly DataManager dataManager;
 

CoreWidgetProvider/Widgets/SystemMemoryWidget.cs

@@ -8,11 +8,11 @@
 
 namespace CoreWidgetProvider.Widgets;
 
-internal class SystemMemoryWidget : CoreWidget, IDisposable
+internal sealed class SystemMemoryWidget : CoreWidget, IDisposable
 {
     private static Dictionary<string, string> Templates { get; set; } = new ();
 
-    protected static readonly new string Name = nameof(SystemMemoryWidget);
+    private static readonly new string Name = nameof(SystemMemoryWidget);
 
     private readonly DataManager dataManager;
 

CoreWidgetProvider/Widgets/SystemNetworkUsageWidget.cs

@@ -8,13 +8,13 @@
 using Microsoft.Windows.Widgets.Providers;
 
 namespace CoreWidgetProvider.Widgets;
-internal class SystemNetworkUsageWidget : CoreWidget, IDisposable
+internal sealed class SystemNetworkUsageWidget : CoreWidget, IDisposable
 {
     private static Dictionary<string, string> Templates { get; set; } = new ();
 
     private int networkIndex;
 
-    protected static readonly new string Name = nameof(SystemNetworkUsageWidget);
+    private static readonly new string Name = nameof(SystemNetworkUsageWidget);
 
     private readonly DataManager dataManager;
 

CoreWidgetProvider/Widgets/WidgetImplFactory.cs

@@ -5,7 +5,7 @@
 using Microsoft.Windows.Widgets.Providers;
 
 namespace CoreWidgetProvider.Widgets;
-internal class WidgetImplFactory<T> : IWidgetImplFactory
+internal sealed class WidgetImplFactory<T> : IWidgetImplFactory
     where T : WidgetImpl, new()
 {
     public WidgetImpl Create(WidgetContext widgetContext, string state)