small things

Author: BRUNER Patrick

src/PluginRegistry/PluginManifest.cs

@@ -317,7 +317,7 @@ public bool IsCompatibleWith (Version logExpertVersion)
         catch (Exception ex) when (ex is ArgumentException or
                                          FormatException)
         {
-            _logger.Error(ex, "ArgumentException checking version compatibility for {Name}: '{Requirement}'. Details: {Message}", Name, Requires.LogExpert, ex.Message);
+            _logger.Error(ex, "ArgumentException/FormatException checking version compatibility for {Name}: '{Requirement}'. Details: {Message}", Name, Requires.LogExpert, ex.Message);
             return false; // Fail closed on error
         }
         catch (Exception ex)

src/PluginRegistry/PluginRegistry.cs

@@ -748,18 +748,12 @@ private bool LoadPluginAssembly (string dllName, PluginManifest? manifest)
             _logger.Debug("Checking type {TypeName} in assembly {AssemblyName}", type.FullName, assembly.FullName);
 
             // Check for ILogLineColumnizer
-            if (type.GetInterfaces().Any(i => i.FullName == typeof(ILogLineColumnizer).FullName))
+            if (type.GetInterfaces().Any(i => i.FullName == typeof(ILogLineColumnizer).FullName) &&
+                TryInstantiatePluginSafe(type, out var instance) &&
+                instance is ILogLineColumnizer columnizer)
             {
-                // Instantiate plugin safely with timeout
-                if (TryInstantiatePluginSafe(type, out var instance))
-                {
-                    // Process as ILogLineColumnizer
-                    if (instance is ILogLineColumnizer columnizer)
-                    {
-                        ProcessLoadedPlugin(columnizer, manifest, dllName);
-                        pluginLoadedCount++;
-                    }
-                }
+                ProcessLoadedPlugin(columnizer, manifest, dllName);
+                pluginLoadedCount++;
             }
 
             // Check for other plugin types (regardless of whether ILogLineColumnizer was found)
@@ -829,7 +823,6 @@ NotSupportedException or
         }
     }
 
-
     /// <summary>
     /// Processes a loaded plugin (either from cache or fresh load).
     /// </summary>

src/docs/PLUGIN_DEVELOPMENT_GUIDE.md

@@ -582,6 +582,6 @@ Contributions welcome! If you create a useful plugin, consider:
 
 ---
 
-**Happy Plugin Development!** ??
+**Happy Plugin Development!**
 
 *This guide is for LogExpert 1.11.0 and later.*

src/docs/PLUGIN_HASH_MANAGEMENT.md

@@ -52,7 +52,7 @@ To enable/disable:
 dotnet build src/PluginRegistry/LogExpert.PluginRegistry.csproj /t:GeneratePluginHashes --configuration Release
 ```
 
-**Important**: The target requires that plugins have already been bitten and are present in the output directory. Make sure to build the entire solution first:
+**Important**: The target requires that plugins have already been built and are present in the output directory. Make sure to build the entire solution first:
 
 ```powershell
 # 1. Build all projects (including plugins) in Release mode