fix(plugin): make remove_command actually work

sail/impl/plugin.py

@@ -109,11 +109,12 @@ def add_command(self, command: command_trait.Command[typing.Any, typing.Any]) ->
 
         self._commands[command.name] = command
 
-    def remove_command(self, command: command_trait.Command[typing.Any, typing.Any]) -> None:
-        if command.name in self._commands:
-            raise RuntimeError(
-                f"Plugin {self.name!r} does not have a command named {command.name!r}."
-            )
+    def remove_command(self, command: command_trait.Command[typing.Any, typing.Any] | str) -> None:
+        name = command.name if isinstance(command, command_trait.Command) else command
+        if name not in self._commands:
+            raise RuntimeError(f"Plugin {self.name!r} does not have a command named {name!r}.")
+
+        self._commands.pop(name)
 
     def command(
         self,

sail/traits/command_trait.py

@@ -22,6 +22,7 @@ def __call__(self, *__args: typing.Any, **kwargs: typing.Any) -> typing.Any:
         ...
 
 
+@typing.runtime_checkable
 class Command(typing.Protocol[P, T_co]):
     __slots__: typing.Sequence[str] = ("name", "description", "aliases", "callback")