✨ Support static methods for commands (fix #2)

VCF.Core/CommandRegistry.cs

@@ -243,8 +243,12 @@ static void HandleAfterExecute(ICommandContext ctx, ChatCommand command)
 				}
 			}
 
-			// construct command class with context if declared
-			var instance = command.Constructor == null ? Activator.CreateInstance(command.Method.DeclaringType) : command.Constructor.Invoke(new[] { ctx });
+			object instance = null;
+			// construct command's type with context if declared only in a non-static class and on a non-static method
+			if (!command.Method.IsStatic && !(command.Method.DeclaringType.IsAbstract && command.Method.DeclaringType.IsSealed))
+			{
+				instance = command.Constructor == null ? Activator.CreateInstance(command.Method.DeclaringType) : command.Constructor.Invoke(new[] { ctx });
+			}
 
 			if (!HandleCanExecute(ctx, command)) return null; // todo: need better return type
 			HandleBeforeExecute(ctx, command);

VCF.Tests/StaticCommandsTests.cs

@@ -0,0 +1,46 @@
+using FakeItEasy;
+using NUnit.Framework;
+using VampireCommandFramework;
+
+namespace VCF.Tests;
+
+public class StaticCommandsTests
+{
+	[SetUp]
+	public void Setup()
+	{
+		Log.Instance = new BepInEx.Logging.ManualLogSource("Test");
+		CommandRegistry.Reset();
+	}
+
+	[Test]
+	public void Static_Class_Static_Method_Command()
+	{
+		CommandRegistry.RegisterCommandType(typeof(StaticClassTestCommands));
+		Assert.IsNotNull(CommandRegistry.Handle(A.Fake<ICommandContext>(), ".test"));
+	}
+
+	[Test]
+	public void Regular_Class_Static_Method_Command()
+	{
+		CommandRegistry.RegisterCommandType(typeof(RegularClassTestCommands));
+		Assert.IsNotNull(CommandRegistry.Handle(A.Fake<ICommandContext>(), ".test"));
+	}
+
+	public static class StaticClassTestCommands
+	{
+		[ChatCommand("test")]
+		public static void Test(ICommandContext ctx) { }
+	}
+
+	public class RegularClassTestCommands
+	{
+		public RegularClassTestCommands(ICommandContext ctx)
+		{
+			Assert.Fail("Should not run this.");
+		}
+
+		[ChatCommand("test")]
+		public static void Test(ICommandContext ctx) { }
+	}
+}