cleanup: dead code removal, duplicate column fixes, broken icon fix, horizontal scroll for docked views, code quality audit, cancellation and scheduled task fixes, UI polish

.editorconfig

@@ -167,6 +167,9 @@ dotnet_naming_style.type_parameter_style.required_prefix = T
 
 # .NET diagnostic rules
 
+# IDE0005: Remove unnecessary usings
+dotnet_diagnostic.IDE0005.severity = warning
+
 # CA1001: Types that own disposable fields should be disposable
 dotnet_diagnostic.CA1001.severity = warning
 

.gemini/GEMINI.md

@@ -38,7 +38,7 @@ If `indexing_status: "started"`: Search will work shortly. **DO NOT fall back to
 ## ContextStream v0.4.x (Consolidated Domain Tools)
 
 v0.4.x uses ~11 consolidated domain tools for ~75% token reduction vs previous versions.
-Rules Version: 0.1.61
+Rules Version: 0.1.86
 
 ### Required Every Message
 

CLAUDE.md

@@ -38,7 +38,7 @@ If `indexing_status: "started"`: Search will work shortly. **DO NOT fall back to
 ## ContextStream v0.4.x (Consolidated Domain Tools)
 
 v0.4.x uses ~11 consolidated domain tools for ~75% token reduction vs previous versions.
-Rules Version: 0.1.61
+Rules Version: 0.1.86
 
 ### Required Every Message
 

src/S7Tools.Core/Commands/BaseCommandHandler.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Threading;
-using System.Threading.Tasks;
 using Microsoft.Extensions.Logging;
 
 namespace S7Tools.Core.Commands;

src/S7Tools.Core/Commands/ClearLogsCommand.cs

@@ -1,5 +1,3 @@
-using S7Tools.Core.Commands;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/DeleteLogEntryCommand.cs

@@ -1,5 +1,3 @@
-using S7Tools.Core.Commands;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/ExportLogsCommand.cs

@@ -1,6 +1,3 @@
-using System;
-using S7Tools.Core.Commands;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/FilterLogsCommand.cs

@@ -1,6 +1,3 @@
-using System;
-using S7Tools.Core.Commands;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/ICommand.cs

@@ -1,6 +1,3 @@
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/ICommandDispatcher.cs

@@ -1,6 +1,3 @@
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/ICommandHandler.cs

@@ -1,6 +1,3 @@
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Commands/ImportLogsCommand.cs

@@ -1,5 +1,3 @@
-using S7Tools.Core.Commands;
-
 namespace S7Tools.Core.Commands;
 
 /// <summary>

src/S7Tools.Core/Exceptions/ConfigurationException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/ConnectionException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/DefaultProfileDeletionException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/DialogParentNotFoundException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/DuplicateProfileNameException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/MemoryRegionException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/PartialDumpException.cs

@@ -0,0 +1,28 @@
+using S7Tools.Core.Models;
+
+namespace S7Tools.Core.Exceptions;
+
+/// <summary>
+/// Exception thrown when a dump operation was interrupted (cancelled or failed)
+/// but partial data was already written to disk and is available for recovery.
+/// </summary>
+public class PartialDumpException : MemoryDumpException
+{
+    /// <summary>
+    /// Gets the partial <see cref="BootloaderResult"/> containing the paths of
+    /// any files that were fully or partially written before the interruption.
+    /// </summary>
+    public BootloaderResult PartialResult { get; }
+
+    /// <summary>
+    /// Initializes a new instance of the <see cref="PartialDumpException"/> class.
+    /// </summary>
+    /// <param name="message">The error message.</param>
+    /// <param name="innerException">The exception that caused the interruption.</param>
+    /// <param name="partialResult">The partial result with saved file paths.</param>
+    public PartialDumpException(string message, Exception innerException, BootloaderResult partialResult)
+        : base(message, innerException)
+    {
+        PartialResult = partialResult ?? throw new ArgumentNullException(nameof(partialResult));
+    }
+}

src/S7Tools.Core/Exceptions/ProfileException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/ProfileNotFoundException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/ReadOnlyProfileModificationException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/S7ToolsException.cs

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Exceptions/ValidationException.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-
 namespace S7Tools.Core.Exceptions;
 
 /// <summary>

src/S7Tools.Core/Factories/BaseFactory.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading;
-using System.Threading.Tasks;
 using Microsoft.Extensions.Logging;
 
 namespace S7Tools.Core.Factories;

src/S7Tools.Core/Factories/IFactory.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Factories;
 
 /// <summary>

src/S7Tools.Core/Interfaces/Logging/IStructuredLogger.cs

@@ -1,5 +1,3 @@
-using System;
-using System.Collections.Generic;
 using Microsoft.Extensions.Logging;
 
 namespace S7Tools.Core.Interfaces.Logging;

src/S7Tools.Core/Interfaces/Services/IBootloaderService.cs

@@ -10,7 +10,7 @@ namespace S7Tools.Core.Interfaces.Services;
 /// </summary>
 public interface IBootloaderService
 {
-/// <summary>
+    /// <summary>
     /// Performs a complete memory dump operation on the PLC.
     /// Orchestrates socat bridge setup, power cycling, handshake, stager installation, and memory dumping.
     /// </summary>

src/S7Tools.Core/Interfaces/Services/IPowerSupplyService.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Threading;
-using System.Threading.Tasks;
 using S7Tools.Core.Models;
 
 namespace S7Tools.Core.Interfaces.Services;

src/S7Tools.Core/Interfaces/Services/IProfileBase.cs

@@ -1,6 +1,3 @@
-using System;
-using System.Collections.Generic;
-
 namespace S7Tools.Core.Interfaces.Services;
 
 /// <summary>

src/S7Tools.Core/Interfaces/Services/IProfileManager.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Interfaces.Services;
 
 /// <summary>

src/S7Tools.Core/Interfaces/Services/IProfileValidator.cs

@@ -1,8 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Interfaces.Services;
 
 /// <summary>

src/S7Tools.Core/Interfaces/Services/ISerialPortService.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Threading;
-using System.Threading.Tasks;
 using S7Tools.Core.Models;
 
 namespace S7Tools.Core.Interfaces.Services;

src/S7Tools.Core/Interfaces/Services/ISocatService.cs

@@ -1,7 +1,3 @@
-using System;
-using System.Collections.Generic;
-using System.Threading;
-using System.Threading.Tasks;
 using S7Tools.Core.Models;
 
 namespace S7Tools.Core.Interfaces.Services;

src/S7Tools.Core/Interfaces/Services/ITaskLogDataStore.cs

@@ -1,5 +1,3 @@
-using System;
-using System.Collections.Generic; // Added for IEnumerable<LogModel>
 using System.Collections.Specialized;
 using S7Tools.Core.Models;
 

src/S7Tools.Core/Interfaces/Services/ITaskScheduler.cs

@@ -1,5 +1,4 @@
 using S7Tools.Core.Models.Jobs;
-using S7Tools.Core.Validation;
 
 namespace S7Tools.Core.Interfaces.Services;
 

src/S7Tools.Core/Interfaces/Services/IUnifiedProfileDialogService.cs

@@ -1,5 +1,3 @@
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Interfaces.Services;
 
 /// <summary>

src/S7Tools.Core/Interfaces/Shell/IShellCommandExecutor.cs

@@ -1,7 +1,3 @@
-using System.Collections.Generic;
-using System.Threading;
-using System.Threading.Tasks;
-
 namespace S7Tools.Core.Interfaces.Shell;
 
 /// <summary>

src/S7Tools.Core/Models/BootloaderResult.cs

@@ -1,6 +1,3 @@
-
-using System.Collections.Generic;
-
 namespace S7Tools.Core.Models;
 
 /// <summary>

src/S7Tools.Core/Models/Configuration/PathConfiguration.cs

@@ -131,8 +131,8 @@ public bool Validate()
                 }
 
                 // Check that all computed paths are valid
-                string[] paths = new[]
-                {
+                string[] paths =
+                [
                     ResourcesDirectory,
                     AppSettingsPath,
                     ProfilesDirectory,
@@ -141,7 +141,7 @@ public bool Validate()
                     TasksPath,
                     PayloadsDirectory,
                     DumpsDirectory
-                };
+                ];
 
                 foreach (string? path in paths)
                 {

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

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Models.Jobs
 {
     /// <summary>

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

@@ -1,5 +1,3 @@
-using System;
-using System.Collections.Generic;
 using System.ComponentModel.DataAnnotations;
 using System.Text.Json.Serialization;
 using S7Tools.Core.Constants;

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

@@ -1,4 +1,3 @@
-using System;
 using System.ComponentModel.DataAnnotations;
 
 namespace S7Tools.Core.Models.Jobs;

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

@@ -1,5 +1,3 @@
-using System;
-using System.Collections.Generic;
 using System.ComponentModel.DataAnnotations;
 using S7Tools.Core.Interfaces.Services;
 

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

@@ -1,5 +1,3 @@
-using System;
-
 namespace S7Tools.Core.Models.Jobs;
 
 /// <summary>