Refactor dependency injection to use default bindings map

Extracted default bindings and annotated bindings into separate helper methods to streamline dependency management. Updated `MinecraftGuiceModule` to utilize these maps, reducing repetitive code and improving maintainability. Introduced a constant for the plugin logger key to ensure consistency.

Author: Newspicel

src/main/kotlin/dev/newspicel/di/DIJavaPlugin.kt

@@ -10,11 +10,14 @@ import dev.newspicel.di.serverevents.OnEnable
 import dev.newspicel.di.serverevents.OnLoad
 import dev.newspicel.di.serverevents.ServerEventsHelper
 import net.kyori.adventure.text.Component.text
+import org.bukkit.Server
 import org.bukkit.command.CommandSender
+import org.bukkit.plugin.Plugin
 import org.bukkit.plugin.java.JavaPlugin
 import org.reflections.Reflections
 import org.reflections.util.ConfigurationBuilder
 import java.util.logging.Level
+import java.util.logging.Logger
 import kotlin.reflect.KClass
 import kotlin.reflect.KParameter
 import kotlin.reflect.full.findAnnotations
@@ -124,6 +127,24 @@ abstract class DIJavaPlugin : JavaPlugin() {
         }
     }
 
+    protected fun getDefaultBindings(): Map<KClass<*>, Any> {
+        return mapOf(
+            JavaPlugin::class to this,
+            GuiceJavaPlugin::class to this,
+            DIJavaPlugin::class to this,
+            Plugin::class to this,
+            this::class to this,
+            Server::class to server,
+            ClassLoader::class to classLoader,
+        )
+    }
+
+    protected fun getDefaultAnnotatedBindings(): Map<KClass<*>, Pair<String, Any>> {
+        return mapOf(
+            Logger::class to (PLUGIN_LOGGER_KEY to logger),
+        )
+    }
+
     // Lock this stuff down
     final override fun onTabComplete(sender: CommandSender, command: org.bukkit.command.Command, alias: String, args: Array<out String>): List<String> {
         sender.sendMessage(text("An error occurred while executing this command. Please contact the server administrators."))

src/main/kotlin/dev/newspicel/di/Extensions.kt

@@ -2,4 +2,6 @@ package dev.newspicel.di
 
 import java.util.logging.Logger
 
-fun logger(plugin: DIJavaPlugin) = lazy { plugin.getExistingBinding(Logger::class, "pluginLogger")!! }
+const val PLUGIN_LOGGER_KEY = "pluginLogger"
+
+fun logger(plugin: DIJavaPlugin) = lazy { plugin.getExistingBinding(Logger::class, PLUGIN_LOGGER_KEY)!! }

src/main/kotlin/dev/newspicel/di/GuiceJavaPlugin.kt

@@ -3,10 +3,6 @@ package dev.newspicel.di
 import com.google.inject.*
 import com.google.inject.name.Named
 import com.google.inject.name.Names
-import org.bukkit.Server
-import org.bukkit.plugin.Plugin
-import org.bukkit.plugin.java.JavaPlugin
-import java.util.logging.Logger
 import kotlin.reflect.KClass
 import kotlin.reflect.full.findAnnotations
 import kotlin.reflect.jvm.javaConstructor
@@ -16,7 +12,7 @@ abstract class GuiceJavaPlugin : DIJavaPlugin() {
     private lateinit var injector: Injector
 
     override fun startDependencyInjection() {
-        val module = MinecraftGuiceModule(this, stereotypesClasses, classLoader)
+        val module = MinecraftGuiceModule(getDefaultBindings(), getDefaultAnnotatedBindings(), stereotypesClasses)
 
         try {
             injector = Guice.createInjector(module) ?: error("Could not create Guice injector")
@@ -65,21 +61,18 @@ abstract class GuiceJavaPlugin : DIJavaPlugin() {
 }
 
 class MinecraftGuiceModule(
-    private val plugin: GuiceJavaPlugin,
+    private val defaultBindings: Map<KClass<*>, Any>,
+    private val defaultAnnotatedBindings: Map<KClass<*>, Pair<String, Any>>,
     private val allStereotypes: List<KClass<*>>,
-    private val classLoader: ClassLoader,
 ) : AbstractModule() {
-    @Suppress("UNCHECKED_CAST")
     override fun configure() {
-        bind(JavaPlugin::class.java).toInstance(plugin)
-        bind(GuiceJavaPlugin::class.java).toInstance(plugin)
-        bind(DIJavaPlugin::class.java).toInstance(plugin)
-        bind(Plugin::class.java).toInstance(plugin)
-        bind(plugin.javaClass).toInstance(plugin)
-        bind(Server::class.java).toInstance(plugin.server)
-        bind(ClassLoader::class.java).toInstance(classLoader)
+        defaultBindings.forEach { (type, instance) ->
+            bind(type.java as Class<Any>).toInstance(instance)
+        }
 
-        bind(Logger::class.java).annotatedWith(Names.named("pluginLogger")).toInstance(plugin.logger)
+        defaultAnnotatedBindings.forEach { (type, instance) ->
+            bind(type.java as Class<Any>).annotatedWith(Names.named(instance.first)).toInstance(instance.second)
+        }
 
         allStereotypes.forEach { stereotype ->
             val type = stereotype.java as Class<Any>