a few more updates

Author: Hirogen

src/ColumnizerLib/IPluginContext.cs

@@ -0,0 +1,28 @@
+namespace LogExpert;
+
+/// <summary>
+/// Provides context information to plugins during initialization.
+/// </summary>
+public interface IPluginContext
+{
+    /// <summary>
+    /// Logger for the plugin to use for diagnostic output.
+    /// </summary>
+    ILogExpertLogger Logger { get; }
+    
+    /// <summary>
+    /// Directory where the plugin assembly is located.
+    /// </summary>
+    string PluginDirectory { get; }
+    
+    /// <summary>
+    /// Version of the host application (LogExpert).
+    /// </summary>
+    Version HostVersion { get; }
+    
+    /// <summary>
+    /// Directory where the plugin can store configuration files.
+    /// Typically %APPDATA%\LogExpert\Plugins\{PluginName}\
+    /// </summary>
+    string ConfigurationDirectory { get; }
+}

src/ColumnizerLib/IPluginLifecycle.cs

@@ -11,44 +11,17 @@ public interface IPluginLifecycle
     /// Use this to initialize resources, load configuration, etc.
     /// </summary>
     /// <param name="context">Context providing information about the host environment</param>
-    void Initialize(IPluginContext context);
-    
+    void Initialize (IPluginContext context);
+
     /// <summary>
     /// Called when the application is shutting down.
     /// Use this to cleanup resources, save state, etc.
     /// </summary>
-    void Shutdown();
-    
+    void Shutdown ();
+
     /// <summary>
     /// Called when the plugin should reload its configuration.
     /// Use this to refresh settings without restarting the application.
     /// </summary>
-    void Reload();
-}
-
-/// <summary>
-/// Provides context information to plugins during initialization.
-/// </summary>
-public interface IPluginContext
-{
-    /// <summary>
-    /// Logger for the plugin to use for diagnostic output.
-    /// </summary>
-    ILogExpertLogger Logger { get; }
-    
-    /// <summary>
-    /// Directory where the plugin assembly is located.
-    /// </summary>
-    string PluginDirectory { get; }
-    
-    /// <summary>
-    /// Version of the host application (LogExpert).
-    /// </summary>
-    Version HostVersion { get; }
-    
-    /// <summary>
-    /// Directory where the plugin can store configuration files.
-    /// Typically %APPDATA%\LogExpert\Plugins\{PluginName}\
-    /// </summary>
-    string ConfigurationDirectory { get; }
-}
+    void Reload ();
+}

src/LogExpert.Resources/Resources.Designer.cs

@@ -2270,4 +2270,16 @@ Continue?</value>
   <data name="LogExpert_Common_UI_Button_Save" xml:space="preserve">
     <value>&amp;Save</value>
   </data>
+  <data name="PluginRegistry_PluginLoadingProgress_ValidatingPluginSecurityAndManifest" xml:space="preserve">
+    <value>Validating plugin security and manifest</value>
+  </data>
+  <data name="PluginRegistry_PluginLoadingProgress_FailedValidationNotTrustedOrInvalidManifest" xml:space="preserve">
+    <value>Failed validation (not trusted or invalid manifest)</value>
+  </data>
+  <data name="PluginRegistry_PluginLoadingProgress_LoadingPluginAssembly" xml:space="preserve">
+    <value>Loading plugin assembly</value>
+  </data>
+  <data name="PluginRegistry_PluginLoadingProgress_FailedToLoadPluginAssemblyTimeoutOrError" xml:space="preserve">
+    <value>Failed to load plugin assembly (timeout or error)</value>
+  </data>
 </root>

src/LogExpert.Resources/Resources.resx

@@ -1,14 +1,14 @@
-using NUnit.Framework;
-
 using LogExpert.PluginRegistry;
 
+using NUnit.Framework;
+
 namespace LogExpert.Tests.PluginRegistry;
 
 [TestFixture]
 public class PluginManifestVersionParsingTests
 {
     [Test]
-    public void Validate_WithVersionRequirementWithSpaces_ShouldPass()
+    public void Validate_WithVersionRequirementWithSpaces_ShouldPass ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -31,7 +31,7 @@ public void Validate_WithVersionRequirementWithSpaces_ShouldPass()
     }
 
     [Test]
-    public void Validate_WithVersionRequirementWithoutSpaces_ShouldPass()
+    public void Validate_WithVersionRequirementWithoutSpaces_ShouldPass ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -66,7 +66,7 @@ public void Validate_WithVersionRequirementWithoutSpaces_ShouldPass()
     [TestCase("~1.10.0")]
     [TestCase("^ 1.10.0")]
     [TestCase("^1.10.0")]
-    public void Validate_WithVariousVersionRequirementFormats_ShouldPass(string requirement)
+    public void Validate_WithVariousVersionRequirementFormats_ShouldPass (string requirement)
     {
         // Arrange
         var manifest = new PluginManifest
@@ -89,7 +89,7 @@ public void Validate_WithVariousVersionRequirementFormats_ShouldPass(string requ
     }
 
     [Test]
-    public void IsCompatibleWith_WithVersionRequirementWithSpaces_ShouldWorkCorrectly()
+    public void IsCompatibleWith_WithVersionRequirementWithSpaces_ShouldWorkCorrectly ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -111,7 +111,7 @@ public void IsCompatibleWith_WithVersionRequirementWithSpaces_ShouldWorkCorrectl
     }
 
     [Test]
-    public void IsCompatibleWith_WithCaretRange_ShouldAllowMinorUpdates()
+    public void IsCompatibleWith_WithCaretRange_ShouldAllowMinorUpdates ()
     {
         // Arrange - ^ allows minor and patch updates but not major
         var manifest = new PluginManifest
@@ -134,7 +134,7 @@ public void IsCompatibleWith_WithCaretRange_ShouldAllowMinorUpdates()
     }
 
     [Test]
-    public void IsCompatibleWith_WithTildeRange_ShouldAllowPatchUpdates()
+    public void IsCompatibleWith_WithTildeRange_ShouldAllowPatchUpdates ()
     {
         // Arrange - ~ allows patch updates but not minor or major
         var manifest = new PluginManifest
@@ -157,7 +157,7 @@ public void IsCompatibleWith_WithTildeRange_ShouldAllowPatchUpdates()
     }
 
     [Test]
-    public void IsCompatibleWith_WithGreaterThan_ShouldExcludeEqualVersion()
+    public void IsCompatibleWith_WithGreaterThan_ShouldExcludeEqualVersion ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -178,7 +178,7 @@ public void IsCompatibleWith_WithGreaterThan_ShouldExcludeEqualVersion()
     }
 
     [Test]
-    public void IsCompatibleWith_WithLessThan_ShouldExcludeEqualVersion()
+    public void IsCompatibleWith_WithLessThan_ShouldExcludeEqualVersion ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -200,7 +200,7 @@ public void IsCompatibleWith_WithLessThan_ShouldExcludeEqualVersion()
     }
 
     [Test]
-    public void IsCompatibleWith_WithNoRequirement_ShouldAlwaysBeCompatible()
+    public void IsCompatibleWith_WithNoRequirement_ShouldAlwaysBeCompatible ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -222,7 +222,7 @@ public void IsCompatibleWith_WithNoRequirement_ShouldAlwaysBeCompatible()
     }
 
     [Test]
-    public void IsCompatibleWith_WithEmptyRequirement_ShouldAlwaysBeCompatible()
+    public void IsCompatibleWith_WithEmptyRequirement_ShouldAlwaysBeCompatible ()
     {
         // Arrange
         var manifest = new PluginManifest
@@ -242,7 +242,7 @@ public void IsCompatibleWith_WithEmptyRequirement_ShouldAlwaysBeCompatible()
     }
 
     [Test]
-    public void DebugVersionParsing_WithExactInputFromUser()
+    public void DebugVersionParsing_WithExactInputFromUser ()
     {
         // Arrange - Test the EXACT input that's causing the issue
         var requirement = ">= 1.10.0";
@@ -273,7 +273,7 @@ public void DebugVersionParsing_WithExactInputFromUser()
     }
 
     [Test]
-    public void Manifest_Validate_WithSpacesInRequirement_ShouldNotThrow()
+    public void Manifest_Validate_WithSpacesInRequirement_ShouldNotThrow ()
     {
         // Arrange
         var manifest = new PluginManifest