don't set plugin to fail state if we threw InvalidPluginOperationExce…

…ption

These are supposed to indicate to the user that they called a function at the wrong point in time. We don't want to actually mutate the state in that case.

Dalamud/Plugin/Internal/Exceptions/InternalPluginStateException.cs

@@ -0,0 +1,16 @@
+namespace Dalamud.Plugin.Internal.Exceptions;
+
+/// <summary>
+/// An exception to be thrown when policy blocks a plugin from loading.
+/// </summary>
+internal class InternalPluginStateException : InvalidPluginOperationException
+{
+    /// <summary>
+    /// Initializes a new instance of the <see cref="InternalPluginStateException"/> class.
+    /// </summary>
+    /// <param name="message">The message to associate with this exception.</param>
+    public InternalPluginStateException(string message)
+        : base(message)
+    {
+    }
+}

Dalamud/Plugin/Internal/Types/LocalPlugin.cs

@@ -281,7 +281,7 @@ public async Task LoadAsync(PluginLoadReason reason, bool reloading = false)
                 case PluginState.Unloaded:
                     if (this.instance is not null)
                     {
-                        throw new InvalidPluginOperationException(
+                        throw new InternalPluginStateException(
                             "Plugin should have been unloaded but instance is not cleared");
                     }
 
@@ -413,7 +413,9 @@ public async Task LoadAsync(PluginLoadReason reason, bool reloading = false)
         }
         catch (Exception ex)
         {
-            this.State = PluginState.LoadError;
+            // These are "user errors", we don't want to mark the plugin as failed
+            if (ex is not InvalidPluginOperationException)
+                this.State = PluginState.LoadError;
 
             // If a precondition fails, don't record it as an error, as it isn't really. 
             if (ex is PluginPreconditionFailedException)
@@ -476,7 +478,10 @@ public async Task UnloadAsync(PluginLoaderDisposalMode disposalMode = PluginLoad
         }
         catch (Exception ex)
         {
-            this.State = PluginState.UnloadError;
+            // These are "user errors", we don't want to mark the plugin as failed
+            if (ex is not InvalidPluginOperationException)
+                this.State = PluginState.UnloadError;
+
             Log.Error(ex, "Error while unloading {PluginName}", this.InternalName);
 
             throw;