refactor: refactor plugin manager

Author: ShrBox

src/lse/Plugin.cpp

@@ -8,7 +8,9 @@
 
 namespace lse {
 
-Plugin::Plugin(const ll::mod::Manifest& manifest) : ll::mod::Mod(manifest) {}
+Plugin::Plugin(const ll::mod::Manifest& manifest) : ll::mod::Mod(std::move(manifest)) {}
+
+Plugin::~Plugin() { release(); }
 
 std::shared_ptr<ll::mod::Mod> Plugin::current() {
     return lse::getPluginManager().getMod(getEngineOwnData()->pluginName);

src/lse/Plugin.h

@@ -12,6 +12,7 @@ class Plugin : public ll::mod::Mod {
 
 public:
     Plugin(const ll::mod::Manifest& manifest);
+    ~Plugin();
 
     static std::shared_ptr<ll::mod::Mod> current();
 };

src/lse/PluginManager.cpp

@@ -58,6 +58,7 @@ auto LLSERemoveAllExportedFuncs(script::ScriptEngine* engine) -> bool;
 namespace lse {
 
 PluginManager::PluginManager() : ll::mod::ModManager(PluginManagerName) {}
+PluginManager::~PluginManager() = default;
 
 ll::Expected<> PluginManager::load(ll::mod::Manifest manifest) {
 #ifdef LEGACY_SCRIPT_ENGINE_BACKEND_PYTHON
@@ -207,9 +208,8 @@ ll::Expected<> PluginManager::load(ll::mod::Manifest manifest) {
         ExitEngineScope exit;
         plugin->onLoad([](ll::mod::Mod&) { return true; });
         plugin->onUnload([](ll::mod::Mod&) { return true; });
-        plugin->onEnable([](ll::mod::Mod&) { return true; });
-        plugin->onDisable([](ll::mod::Mod&) { return true; });
-        addMod(manifest.name, plugin);
+
+        return plugin->onLoad().transform([&, this] { addMod(manifest.name, plugin); });
     } catch (const Exception& e) {
         EngineScope engineScope(scriptEngine);
         auto        error =
@@ -227,7 +227,6 @@ ll::Expected<> PluginManager::load(ll::mod::Manifest manifest) {
 
         return error;
     }
-    return {};
 }
 
 ll::Expected<> PluginManager::unload(std::string_view name) {

src/lse/PluginManager.h

@@ -8,16 +8,10 @@
 
 namespace lse {
 
-class PluginManager : public ll::mod::ModManager {
+class PluginManager final : public ll::mod::ModManager {
 public:
     PluginManager();
-
-    PluginManager(const PluginManager&)                    = delete;
-    PluginManager(PluginManager&&)                         = delete;
-    auto operator=(const PluginManager&) -> PluginManager& = delete;
-    auto operator=(PluginManager&&) -> PluginManager&      = delete;
-
-    ~PluginManager() override = default;
+    ~PluginManager() override;
 
 private:
     ll::Expected<> load(ll::mod::Manifest manifest) override;