refactor: replace custom ILogger with Serilog + Microsoft.Extensions.Logging (#85)

* refactor: extract IProgressReporter from ILogger (#64)

Separate progress reporting (Progress, EndProgress, Finish) from logging
into a new IProgressReporter interface. This is the first step toward
replacing the custom ILogger with Microsoft.Extensions.Logging + Serilog.

- Add IProgressReporter interface and NullProgressReporter
- Remove Progress, EndProgress, Finish from ILogger and all implementations
- Remove dead BeginModule/EndModule from NullLogger
- Add ProgressReporter property to ConfuserParameters and ConfuserContext
- Replace PackerLogger (full ILogger decorator) with PackerProgressReporter
- Update all WithProgress call sites to use context.ProgressReporter
- Fix MSBuildLogger.Finish bug (was setting HasError=false on failure)
- Seal NullLogger class

* feature: add M.E.L abstraction and MelLoggerAdapter (#64)

Add Microsoft.Extensions.Logging.Abstractions to Confuser.Core
(netstandard2.0 compatible) and a MelLoggerAdapter that bridges
M.E.L ILogger to the internal Confuser.Core.ILogger interface.

This allows callers to pass a standard M.E.L logger (backed by
Serilog or any other provider) into ConfuserEngine without
changing any internal code yet.

* feature: replace CLI ConsoleLogger with Serilog (#64)

Wire up Serilog as the logging provider in the CLI via
Microsoft.Extensions.Logging and MelLoggerAdapter.

- Add Serilog, Serilog.Extensions.Logging, Serilog.Sinks.Console
- Delete custom ConsoleLogger — Serilog handles all console output
- Add --verbose (-v, -vv, -vvv) and --quiet (-q) CLI flags
- Default: Information level; -q: Warning; -v: Debug; -vv+: Verbose

* feature: replace GUI logger with Serilog FlowDocument sink (#64)

Wire up Serilog in the WPF GUI via a custom FlowDocumentSink that
renders color-coded log output to the protection log panel.

- Add Serilog and Serilog.Extensions.Logging to ConfuserEx
- Create FlowDocumentSink — custom Serilog sink for WPF Paragraph
- Remove ILogger from ProtectTabVM — now uses MelLoggerAdapter
- ProtectTabVM keeps only IProgressReporter (progress bar + finish)
- Delete ~40 lines of manual ILogger boilerplate

* chore: add local-ci.sh script mirroring GitHub Actions pipeline

Full local CI script that replicates lint.yml, ci.yml, and test.yml:
- lint: whitespace, style, and analyzer checks via dotnet format
- build: dotnet build for SDK projects + MSBuild.exe for C++/CLI
- test: discovers all *.Test.csproj, runs with coverage, summary
- package: creates CLI, GUI, and combined zip archives

Usage: ./scripts/local-ci.sh [lint|build|test|package|all]

* refactor: replace Confuser.Core.ILogger with Microsoft.Extensions.Logging.ILogger (#64)

Complete migration from the custom 13-method ILogger interface to the
standard M.E.L ILogger abstraction across all projects.

- Change ConfuserContext.Logger and ConfuserParameters.Logger to M.E.L ILogger
- Convert all ~80 call sites: Debug→LogDebug, Info→LogInformation,
  Warn→LogWarning, Error→LogError, *Exception→swap parameter order
- Rewrite MSBuildLogger as MSBuildMelLogger implementing M.E.L ILogger
- Rewrite XUnitLogger implementing M.E.L ILogger + IProgressReporter
- Remove MelLoggerAdapter (no longer needed — M.E.L is the native type)
- Delete Confuser.Core.ILogger, NullLogger (replaced by M.E.L NullLogger)
- Add test-results/ and coverage/ to .gitignore

* chore: fix import ordering to pass CI lint (#64)

* fix: dispose GUI logger factory after async protection completes (#64)

DoProtect used 'using var loggerFactory' which disposed the factory
(and the Serilog logger via dispose:true) as soon as DoProtect returned.
Since ConfuserEngine.Run executes asynchronously on a background thread,
this disposed the logger before the protection actually used it, silently
dropping log output mid-run. Move disposal into the ContinueWith
continuation so the factory lives for the full protection lifetime.

* test: fix flaky GUI protect test — deterministic tab navigation (#64)

Gui_ProtectSampleApp_ShowsSuccess intermittently failed at the Protect!
button lookup (~1 in 3 runs). Two root causes:

1. ByText("Protect!") ambiguously matched both the tab header and the
   Protect! button (they share the caption), so the wrong element could
   be clicked and the tab never actually got selected.
2. WPF virtualizes inactive tab content — the Protect! button does not
   enter the UIA tree until the tab is selected AND rendered. The 5s
   button-find timeout was too short under load.

Fix: match the tab by TabItem control type + name, Select() it and wait
for IsSelected, then find the button with a 15s timeout. 5/5 consecutive
full-suite runs now pass 3/3 (previously ~1/3 failed).

---------

Co-authored-by: RandomCrocodile <mawi@polosab.com>

Author: mcpolo99

.gitignore

@@ -39,4 +39,8 @@ packages/
 gh-pages/
 
 .idea/
-**/*out*
+**/*out*
+
+# Local CI artifacts
+test-results/
+coverage/

Confuser.CLI/Confuser.CLI.csproj

@@ -16,6 +16,9 @@
 
   <ItemGroup Label="Nuget Dependencies">
     <PackageReference Include="NDesk.Options.Core" Version="1.2.5" />
+    <PackageReference Include="Serilog" Version="4.*" />
+    <PackageReference Include="Serilog.Extensions.Logging" Version="9.*" />
+    <PackageReference Include="Serilog.Sinks.Console" Version="6.*" />
   </ItemGroup>
 
   <ItemGroup Label="Project Dependencies">

Confuser.CLI/Program.cs

@@ -6,7 +6,10 @@
 using System.Xml;
 using Confuser.Core;
 using Confuser.Core.Project;
+using Microsoft.Extensions.Logging;
 using NDesk.Options;
+using Serilog;
+using Serilog.Events;
 
 namespace Confuser.CLI {
 	internal class Program {
@@ -21,6 +24,8 @@ static int Main(string[] args) {
 			try {
 				bool noPause = false;
 				bool debug = false;
+				bool quiet = false;
+				int verbosity = 0;
 				string outDir = null;
 				string snKeyPath = null;
 				string snKeyPass = null;
@@ -48,6 +53,12 @@ static int Main(string[] args) {
 					}, {
 						"snkeypass=", "specifies strong name key password.",
 						value => { snKeyPass = value; }
+					}, {
+						"v|verbose", "increase verbosity (repeat for more: -v, -vv, -vvv).",
+						value => { verbosity++; }
+					}, {
+						"q|quiet", "only show warnings and errors.",
+						value => { quiet = (value != null); }
 					}
 				};
 
@@ -130,7 +141,7 @@ static int Main(string[] args) {
 					parameters.Project = proj;
 				}
 
-				int retVal = RunProject(parameters);
+				int retVal = RunProject(parameters, quiet, verbosity);
 
 				if (NeedPause() && !noPause) {
 					Console.WriteLine("Press any key to continue...");
@@ -192,14 +203,34 @@ static void LoadTemplateProject(string templatePath, ConfuserProject proj, List<
 					templateModules.Add(templateModule);
 		}
 
-		static int RunProject(ConfuserParameters parameters) {
-			var logger = new ConsoleLogger();
-			parameters.Logger = logger;
-
-			Console.Title = "ConfuserEx - Running...";
+		static int RunProject(ConfuserParameters parameters, bool quiet, int verbosity) {
+			var levelSwitch = quiet
+				? LogEventLevel.Warning
+				: verbosity >= 3 ? LogEventLevel.Verbose
+				: verbosity >= 2 ? LogEventLevel.Verbose
+				: verbosity >= 1 ? LogEventLevel.Debug
+				: LogEventLevel.Information;
+
+			Log.Logger = new LoggerConfiguration()
+				.MinimumLevel.Is(levelSwitch)
+				.WriteTo.Console(
+					outputTemplate: "[{Level:u4}] {Message:lj}{NewLine}{Exception}")
+				.CreateLogger();
+
+			using var loggerFactory = LoggerFactory.Create(builder =>
+				builder.AddSerilog(dispose: false));
+			var melLogger = loggerFactory.CreateLogger("ConfuserEx");
+
+			var progressReporter = new ConsoleProgressReporter();
+			parameters.Logger = melLogger;
+			parameters.ProgressReporter = progressReporter;
+
+			if (OperatingSystem.IsWindows())
+				Console.Title = "ConfuserEx - Running...";
 			ConfuserEngine.Run(parameters).GetAwaiter().GetResult();
 
-			return logger.ReturnValue;
+			Log.CloseAndFlush();
+			return progressReporter.ReturnValue;
 		}
 
 		static bool NeedPause() {
@@ -217,6 +248,8 @@ static void PrintUsage() {
 			WriteLine("    -debug     : specifies debug symbol generation.");
 			WriteLine("    -snkey     : specifies strong name key file path.");
 			WriteLine("    -snkeypass : specifies strong name key password.");
+			WriteLine("    -v|verbose : increase verbosity (-v debug, -vv trace).");
+			WriteLine("    -q|quiet   : only show warnings and errors.");
 		}
 
 		static void WriteLineWithColor(ConsoleColor color, string txt) {
@@ -234,57 +267,15 @@ static void WriteLine() {
 			Console.WriteLine();
 		}
 
-		class ConsoleLogger : ILogger {
+		class ConsoleProgressReporter : IProgressReporter {
 			readonly DateTime begin;
 
-			public ConsoleLogger() {
+			public ConsoleProgressReporter() {
 				begin = DateTime.Now;
 			}
 
 			public int ReturnValue { get; private set; }
 
-			public void Debug(string msg) {
-				WriteLineWithColor(ConsoleColor.Gray, "[DEBUG] " + msg);
-			}
-
-			public void DebugFormat(string format, params object[] args) {
-				WriteLineWithColor(ConsoleColor.Gray, "[DEBUG] " + string.Format(format, args));
-			}
-
-			public void Info(string msg) {
-				WriteLineWithColor(ConsoleColor.White, " [INFO] " + msg);
-			}
-
-			public void InfoFormat(string format, params object[] args) {
-				WriteLineWithColor(ConsoleColor.White, " [INFO] " + string.Format(format, args));
-			}
-
-			public void Warn(string msg) {
-				WriteLineWithColor(ConsoleColor.Yellow, " [WARN] " + msg);
-			}
-
-			public void WarnFormat(string format, params object[] args) {
-				WriteLineWithColor(ConsoleColor.Yellow, " [WARN] " + string.Format(format, args));
-			}
-
-			public void WarnException(string msg, Exception ex) {
-				WriteLineWithColor(ConsoleColor.Yellow, " [WARN] " + msg);
-				WriteLineWithColor(ConsoleColor.Yellow, "Exception: " + ex);
-			}
-
-			public void Error(string msg) {
-				WriteLineWithColor(ConsoleColor.Red, "[ERROR] " + msg);
-			}
-
-			public void ErrorFormat(string format, params object[] args) {
-				WriteLineWithColor(ConsoleColor.Red, "[ERROR] " + string.Format(format, args));
-			}
-
-			public void ErrorException(string msg, Exception ex) {
-				WriteLineWithColor(ConsoleColor.Red, "[ERROR] " + msg);
-				WriteLineWithColor(ConsoleColor.Red, "Exception: " + ex);
-			}
-
 			public void Progress(int progress, int overall) { }
 
 			public void EndProgress() { }

Confuser.Core/Confuser.Core.csproj

@@ -17,6 +17,7 @@
   <ItemGroup Label="Nuget Dependencies">
     <PackageReference Include="dnlib" Version="3.6.0" />
     <PackageReference Include="Microsoft.DiaSymReader.Native" Version="1.7.0" />
+    <PackageReference Include="Microsoft.Extensions.Logging.Abstractions" Version="9.*" />
     <PackageReference Include="Microsoft.Win32.Registry" Version="5.0.0" />
   </ItemGroup>
 

Confuser.Core/ConfuserContext.cs

@@ -4,6 +4,7 @@
 using Confuser.Core.Project;
 using dnlib.DotNet;
 using dnlib.DotNet.Writer;
+using Microsoft.Extensions.Logging;
 
 namespace Confuser.Core {
 	/// <summary>
@@ -18,7 +19,13 @@ public class ConfuserContext {
 		///     Gets the logger used for logging events.
 		/// </summary>
 		/// <value>The logger.</value>
-		public ILogger Logger { get; internal set; }
+		public Microsoft.Extensions.Logging.ILogger Logger { get; internal set; }
+
+		/// <summary>
+		///     Gets the progress reporter used for reporting protection progress.
+		/// </summary>
+		/// <value>The progress reporter.</value>
+		public IProgressReporter ProgressReporter { get; internal set; }
 
 		/// <summary>
 		///     Gets the project being processed.