fix: fixed plugin lifecycle

Author: duruer

src/main/kotlin/co/statu/parsek/Main.kt

@@ -10,7 +10,6 @@ import io.vertx.core.VertxOptions
 import io.vertx.ext.web.Router
 import io.vertx.kotlin.coroutines.CoroutineVerticle
 import io.vertx.kotlin.coroutines.coAwait
-import kotlinx.coroutines.runBlocking
 import org.slf4j.LoggerFactory
 import org.springframework.context.annotation.AnnotationConfigApplicationContext
 
@@ -129,19 +128,11 @@ class Main : CoroutineVerticle() {
 
         pluginManager.loadPlugins()
 
-        logger.info("Enabling plugins")
-
-        pluginManager.getPluginWrappers().forEach {
-            pluginManager.enablePlugin(it.pluginId)
-        }
-
         pluginManager.startPlugins()
 
-        runBlocking {
-            pluginManager.getActivePlugins().forEach {
-                it.onStart()
-            }
-        }
+        logger.info("Initializing plugins")
+
+        pluginManager.initializePlugins()
     }
 
     private fun initDependencyInjection() {

src/main/kotlin/co/statu/parsek/PluginManager.kt

@@ -114,4 +114,20 @@ class PluginManager(importPaths: List<Path>) : DefaultPluginManager(importPaths)
 
         return super.disablePlugin(pluginId)
     }
+
+    fun getSortedPlugins(): List<PluginWrapper> {
+        val resolver = DependencyResolver(versionManager)
+        val result = resolver.resolve(plugins.values.map { it.descriptor })
+        return result.sortedPlugins.map { plugins[it]!! }
+    }
+
+    fun initializePlugins() {
+        runBlocking {
+            getSortedPlugins().forEach { wrapper ->
+                if (wrapper.pluginState == PluginState.STARTED) {
+                    (wrapper.plugin as? ParsekPlugin)?.onStart()
+                }
+            }
+        }
+    }
 }

src/main/kotlin/co/statu/parsek/api/ParsekPlugin.kt

@@ -6,9 +6,7 @@ import co.statu.parsek.PluginManager
 import co.statu.parsek.ReleaseStage
 import co.statu.parsek.api.event.PluginEventListener
 import io.vertx.core.Vertx
-import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.runBlocking
-import kotlinx.coroutines.withContext
 import org.pf4j.Plugin
 import org.pf4j.PluginState
 import org.slf4j.Logger
@@ -88,17 +86,12 @@ abstract class ParsekPlugin : Plugin() {
         pluginEventManager.unRegister(this, eventListener)
     }
 
-    @Deprecated("Use onStart method.")
+    @Deprecated("Use onStart method.", level = DeprecationLevel.HIDDEN)
     override fun start() {
     }
 
-    @Deprecated("Use onStop method.")
+    @Deprecated("Use onStop method.", level = DeprecationLevel.HIDDEN)
     override fun stop() {
-        runBlocking {
-            withContext(Dispatchers.IO) {
-                onStop()
-            }
-        }
     }
 
     internal fun load() {