Update to .NET 9.0 and modernize C# syntax patterns

- Upgrade target framework from net8.0 to net9.0 across all projects
- Replace traditional null checks with ArgumentNullException.ThrowIfNull()
- Modernize collection initializers using [] syntax instead of new()
- Convert constructors to primary constructor syntax where appropriate
- Add global using statements to reduce code duplication
- Update bootloader service signatures to support enhanced progress reporting
- Fix clipboard service API usage for Avalonia compatibility
- Remove unused test placeholders and update test assertions

Author: efargas

Directory.Build.props

@@ -1,6 +1,6 @@
 <Project>
   <PropertyGroup>
-    <TargetFramework>net8.0</TargetFramework>
+    <TargetFramework>net9.0</TargetFramework>
     <Nullable>enable</Nullable>
     <ImplicitUsings>enable</ImplicitUsings>
     <TreatWarningsAsErrors>true</TreatWarningsAsErrors>
@@ -37,7 +37,8 @@
     <DebugSymbols>true</DebugSymbols>
     <DebugType>portable</DebugType>
     <Optimize>false</Optimize>
-    <!-- Avoid overly strict determinism which can sometimes confuse source mapping in certain toolchains -->
+    <!-- Avoid overly strict determinism which can sometimes confuse source mapping in certain
+    toolchains -->
     <Deterministic>false</Deterministic>
   </PropertyGroup>
-</Project>
+</Project>

src/S7Tools.Core/Exceptions/ValidationException.cs

@@ -27,7 +27,7 @@ public class ValidationException : S7ToolsException
     public ValidationException(string message)
         : base(message)
     {
-        ValidationErrors = new List<string> { message };
+        ValidationErrors = [message];
     }
 
     /// <summary>
@@ -37,7 +37,7 @@ public ValidationException(string message)
     public ValidationException(IEnumerable<string> validationErrors)
         : base(BuildMessage(validationErrors))
     {
-        ValidationErrors = validationErrors?.ToList() ?? new List<string>();
+        ValidationErrors = validationErrors?.ToList() ?? [];
     }
 
     /// <summary>
@@ -49,7 +49,7 @@ public ValidationException(string propertyName, string message)
         : base($"{propertyName}: {message}")
     {
         PropertyName = propertyName;
-        ValidationErrors = new List<string> { message };
+        ValidationErrors = [message];
     }
 
     /// <summary>
@@ -60,7 +60,7 @@ public ValidationException(string propertyName, string message)
     public ValidationException(string message, Exception innerException)
         : base(message, innerException)
     {
-        ValidationErrors = new List<string> { message };
+        ValidationErrors = [message];
     }
 
     /// <summary>

src/S7Tools.Core/GlobalUsings.cs

@@ -0,0 +1,7 @@
+global using System;
+global using System.Collections.Generic;
+global using System.Linq;
+global using System.IO;
+global using System.Threading;
+global using System.Threading.Tasks;
+global using System.Buffers.Binary;

src/S7Tools.Core/Models/BootloaderResult.cs

@@ -1,4 +1,4 @@
-using System.Collections.Generic;
+
 
 namespace S7Tools.Core.Models;
 

src/S7Tools.Core/Models/Jobs/ResourceLockChangedEventArgs.cs

@@ -33,7 +33,7 @@ public class ResourceLockChangedEventArgs : EventArgs
     /// <param name="jobId">The job ID (optional).</param>
     public ResourceLockChangedEventArgs(ResourceKey resource, ResourceLockAction action, int? jobId = null)
     {
-        ArgumentNullException.ThrowIfNull(resource);
+
         Resource = resource;
         Action = action;
         JobId = jobId;

src/S7Tools.Core/Models/MemoryBlock.cs

@@ -1,7 +1,4 @@
-using System;
-using System.Buffers.Binary;
-using System.Linq;
-using System.Text;
+
 
 namespace S7Tools.Core.Models;
 
@@ -20,7 +17,9 @@ namespace S7Tools.Core.Models;
     /// <summary>
     /// Gets the raw memory data.
     /// </summary>
+#pragma warning disable CA1819 // Properties should not return arrays
     public byte[] Data { get; }
+#pragma warning restore CA1819
 
     /// <summary>
     /// Initializes a new instance of the <see cref="MemoryBlock"/> struct.

src/S7Tools.Core/Models/SocatConfiguration.cs

@@ -332,7 +332,7 @@ public string GenerateCommand(string serialDevice)
         }
 
         // Build TCP-LISTEN part
-        var tcpOptions = new List<string>();
+        List<string> tcpOptions = [];
         if (EnableFork)
         {
             tcpOptions.Add("fork");
@@ -352,7 +352,7 @@ public string GenerateCommand(string serialDevice)
         command.Append($" {tcpPart}");
 
         // Build serial device part
-        var serialOptions = new List<string>();
+        List<string> serialOptions = [];
         if (SerialRawMode)
         {
             serialOptions.Add("raw");
@@ -382,7 +382,7 @@ public string GenerateCommand(string serialDevice)
     /// <returns>A list of validation error messages, or empty list if valid.</returns>
     public List<string> Validate()
     {
-        var errors = new List<string>();
+        List<string> errors = [];
 
         if (TcpPort is < 1 or > 65535)
         {
@@ -423,7 +423,7 @@ public List<string> Validate()
     /// <returns>A string describing the key configuration parameters.</returns>
     public override string ToString()
     {
-        var features = new List<string>();
+        List<string> features = [];
 
         if (Verbose)
         {

src/S7Tools.Core/Validation/BaseValidator.cs

@@ -34,7 +34,7 @@ public abstract class BaseValidator<T> : IValidator<T>
     protected BaseValidator(ILogger logger)
     {
         Logger = logger ?? throw new ArgumentNullException(nameof(logger));
-        Rules = new List<ValidationRule<T>>();
+        Rules = [];
         ConfigureRules();
     }
 
@@ -47,7 +47,7 @@ public virtual ValidationResult Validate(T instance)
             return ValidationResult.Failure("Instance", "Instance cannot be null", "NULL_INSTANCE");
         }
 
-        var errors = new List<ValidationError>();
+        List<ValidationError> errors = [];
 
         Logger.LogDebug("Starting validation for {Type}", typeof(T).Name);
 
@@ -84,7 +84,7 @@ public virtual async Task<ValidationResult> ValidateAsync(T instance, Cancellati
             return ValidationResult.Failure("Instance", "Instance cannot be null", "NULL_INSTANCE");
         }
 
-        var errors = new List<ValidationError>();
+        List<ValidationError> errors = [];
 
         Logger.LogDebug("Starting async validation for {Type}", typeof(T).Name);
 

src/S7Tools.Infrastructure.Logging/GlobalUsings.cs

@@ -0,0 +1,4 @@
+global using System;
+global using System.Collections.Generic;
+global using System.Linq;
+global using Microsoft.Extensions.Logging;

src/S7Tools/App.axaml.cs

@@ -1,22 +1,14 @@
-using System;
 using System.Reactive;
-using System.Threading.Tasks;
-using Avalonia;
-using Avalonia.Controls;
 using Avalonia.Controls.ApplicationLifetimes;
-using Avalonia.Markup.Xaml;
 using Microsoft.Extensions.DependencyInjection;
-using Microsoft.Extensions.Logging;
 using S7Tools.Core.Interfaces.Services;
 using S7Tools.Core.Models.Configuration;
 using S7Tools.Core.Resources;
 using S7Tools.Extensions;
 using S7Tools.Models;
 using S7Tools.Resources;
 using S7Tools.Services.Interfaces;
-using S7Tools.ViewModels;
 using S7Tools.ViewModels.Dialogs;
-using S7Tools.Views;
 using S7Tools.Views.Dialogs;
 using S7Tools.Views.Layout;