idk

Author: half-duplex

sopel/modules/version.py

@@ -49,14 +49,15 @@ def version(bot, trigger):
             return
 
         meta = bot.get_plugin_meta(plugin)
-        if meta["source"].startswith("sopel."):
-            version = "v" + release + " (built in)"
-        elif meta["version"] is None:
-            version = "(version unknown)"
+        if meta["version"] is None:
+            version = "(unknown)"
         else:
-            version = "v" + meta["version"]
+            version = "v" + str(meta["version"])
+
+        if meta["source"].startswith("sopel."):
+            version += " (built in)"
 
-        bot.say(plugin + " " + version)
+        bot.say(plugin + " " + str(version))
         return
 
     # Sopel version

sopel/plugins/handlers.py

@@ -49,8 +49,9 @@
 import inspect
 import itertools
 import os
+from typing import Optional
 
-from sopel import loader
+from sopel import __version__ as release, loader
 from . import exceptions
 
 try:
@@ -121,6 +122,15 @@ def get_meta_description(self) -> dict:
         """
         # TODO: change return type to a TypedDict when dropping py3.7
 
+    @abc.abstractmethod
+    def get_version(self):
+        """Retrieve the plugin's version.
+
+        :return: the plugin's version string
+        :rtype: str
+        """
+        raise NotImplementedError
+
     @abc.abstractmethod
     def is_loaded(self) -> bool:
         """Tell if the plugin is loaded or not.
@@ -292,9 +302,20 @@ def get_meta_description(self):
             'type': self.PLUGIN_TYPE,
             'name': self.name,
             'source': self.module_name,
-            'version': getattr(self._module, "__version__", None),
+            'version': self.get_version(),
         }
 
+    def get_version(self) -> Optional[str]:
+        """Retrieve the plugin's version.
+
+        :return: the plugin's version string
+        :rtype: Optional[str]
+        """
+        if hasattr(self._module, "__version__"):
+            return str(self._module.__version__)
+        if self.module_name.startswith("sopel."):
+            return release
+
     def load(self):
         """Load the plugin's module using :func:`importlib.import_module`.