Add generated method accessors (#241)

Additional changes:

* Do not insert annotations as services
* Check for abstract modifier and interface flag on class-annotated services
* Explicitly prevent optional parameters in event listeners

Author: freya022

BotCommands-core/build.gradle.kts

@@ -33,6 +33,10 @@ dependencies {
     // Classpath scanning
     api(libs.classgraph)
 
+    api(projects.botCommandsMethodAccessors.core) // API due to opt-in annotation
+    implementation(projects.botCommandsMethodAccessors.kotlinReflect)
+    implementation(projects.botCommandsMethodAccessors.classfile)
+
     // -------------------- GLOBAL DEPENDENCIES --------------------
 
     api(libs.kotlinx.datetime)
@@ -146,6 +150,12 @@ dokka {
     }
 }
 
+java {
+    // ClassFile-based method accessors require Java 24+
+    // but the class is conditionally loaded
+    disableAutoTargetJvm()
+}
+
 kotlin {
     compilerOptions {
         freeCompilerArgs.addAll(

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/api/core/BotCommands.kt

@@ -1,5 +1,6 @@
 package io.github.freya022.botcommands.api.core
 
+import dev.freya02.botcommands.method.accessors.api.annotations.ExperimentalMethodAccessorsApi
 import io.github.freya022.botcommands.api.ReceiverConsumer
 import io.github.freya022.botcommands.api.commands.annotations.Command
 import io.github.freya022.botcommands.api.core.config.BConfig
@@ -28,6 +29,34 @@ import kotlin.time.measureTimedValue
 object BotCommands {
     private val logger = KotlinLogging.logger { }
 
+    /**
+     * If enabled, instructs the framework to prefer using improved reflection calls, with the following benefits:
+     * - Shorter stack traces in exceptions and the debugger
+     * - No [InvocationTargetExceptions][java.lang.reflect.InvocationTargetException]
+     * - Better performance
+     *
+     * This feature requires *running* on Java 24+, if your bot doesn't, this method has no effect.
+     */
+    @ExperimentalMethodAccessorsApi
+    @get:JvmStatic
+    @get:JvmName("isPreferClassFileAccessors")
+    var preferClassFileAccessors: Boolean = false
+        private set
+
+    /**
+     * Instructs the framework to prefer using improved reflection calls, with the following benefits:
+     * - Shorter stack traces in exceptions and the debugger
+     * - No [InvocationTargetExceptions][java.lang.reflect.InvocationTargetException]
+     * - Better performance
+     *
+     * This feature requires *running* on Java 24+, if your bot doesn't, this method has no effect.
+     */
+    @JvmStatic
+    @ExperimentalMethodAccessorsApi
+    fun preferClassFileAccessors() {
+        preferClassFileAccessors = true
+    }
+
     /**
      * Creates a new instance of the framework.
      *

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/ExecutableMixin.kt

@@ -14,6 +14,8 @@ internal interface ExecutableMixin : Executable {
         get() = eventFunction.kFunction
     val instance: Any
         get() = eventFunction.instance
+    val methodAccessor
+        get() = eventFunction.methodAccessor
 }
 
 @Suppress("NOTHING_TO_INLINE") //Don't want this to appear in stack trace
@@ -26,4 +28,4 @@ internal inline fun ExecutableMixin.requireUser(value: Boolean, lazyMessage: ()
     }
 
     requireAt(value, function, lazyMessage)
-}
+}

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/commands/application/context/message/MessageCommandInfoImpl.kt

@@ -25,7 +25,6 @@ import io.github.freya022.botcommands.internal.utils.*
 import net.dv8tion.jda.api.interactions.IntegrationType
 import net.dv8tion.jda.api.interactions.InteractionContextType
 import net.dv8tion.jda.api.interactions.commands.Command
-import kotlin.reflect.full.callSuspendBy
 
 internal class MessageCommandInfoImpl internal constructor(
     override val context: BContext,
@@ -67,7 +66,7 @@ internal class MessageCommandInfoImpl internal constructor(
         }
 
         val finalParameters = parameters.mapFinalParameters(event, optionValues)
-        function.callSuspendBy(finalParameters)
+        methodAccessor.callSuspend(finalParameters)
 
         return true
     }

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/commands/application/context/user/UserCommandInfoImpl.kt

@@ -25,7 +25,6 @@ import io.github.freya022.botcommands.internal.utils.*
 import net.dv8tion.jda.api.interactions.IntegrationType
 import net.dv8tion.jda.api.interactions.InteractionContextType
 import net.dv8tion.jda.api.interactions.commands.Command
-import kotlin.reflect.full.callSuspendBy
 
 internal class UserCommandInfoImpl internal constructor(
     override val context: BContext,
@@ -67,7 +66,7 @@ internal class UserCommandInfoImpl internal constructor(
         }
 
         val finalParameters = parameters.mapFinalParameters(event, optionValues)
-        function.callSuspendBy(finalParameters)
+        methodAccessor.callSuspend(finalParameters)
 
         return true
     }

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/commands/application/slash/SlashCommandInfoImpl.kt

@@ -1,5 +1,6 @@
 package io.github.freya022.botcommands.internal.commands.application.slash
 
+import dev.freya02.botcommands.method.accessors.internal.MethodArguments
 import io.github.freya022.botcommands.api.commands.INamedCommand
 import io.github.freya022.botcommands.api.commands.application.slash.GlobalSlashEvent
 import io.github.freya022.botcommands.api.commands.application.slash.GuildSlashEvent
@@ -32,8 +33,6 @@ import net.dv8tion.jda.api.events.interaction.command.SlashCommandInteractionEve
 import net.dv8tion.jda.api.interactions.Interaction
 import net.dv8tion.jda.api.interactions.commands.CommandInteractionPayload
 import net.dv8tion.jda.api.interactions.commands.OptionMapping
-import kotlin.reflect.KParameter
-import kotlin.reflect.full.callSuspendBy
 import kotlin.reflect.jvm.jvmErasure
 
 private val logger = KotlinLogging.logger { }
@@ -77,7 +76,7 @@ internal sealed class SlashCommandInfoImpl(
 
     internal suspend fun execute(event: GlobalSlashEvent): Boolean {
         val objects = getSlashOptions(event, parameters) ?: return false
-        function.callSuspendBy(objects)
+        methodAccessor.callSuspend(objects)
 
         return true
     }
@@ -90,7 +89,7 @@ internal sealed class SlashCommandInfoImpl(
 internal suspend fun <T> ExecutableMixin.getSlashOptions(
     event: T,
     parameters: List<AggregatedParameterMixin>
-): Map<KParameter, Any?>? where T : CommandInteractionPayload, T : Event {
+): MethodArguments? where T : CommandInteractionPayload, T : Event {
     val optionValues = parameters.mapOptions { option ->
         if (tryInsertOption(event, this, option) == InsertOptionResult.ABORT)
             return null

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/commands/application/slash/autocomplete/AutocompleteHandler.kt

@@ -25,7 +25,6 @@ import net.dv8tion.jda.api.events.interaction.command.CommandAutoCompleteInterac
 import net.dv8tion.jda.api.interactions.commands.Command
 import net.dv8tion.jda.api.interactions.commands.OptionType as JDAOptionType
 import net.dv8tion.jda.api.interactions.commands.build.OptionData
-import kotlin.reflect.full.callSuspendBy
 import kotlin.reflect.jvm.jvmErasure
 
 /**
@@ -97,7 +96,7 @@ internal class AutocompleteHandler(
             ?: return emptyList() //Autocomplete was triggered without all the required parameters being present
 
         val actualChoices: MutableList<Command.Choice> = arrayOfSize(25)
-        val suppliedChoices = choiceSupplier.apply(event, autocompleteInfo.function.callSuspendBy(objects))
+        val suppliedChoices = choiceSupplier.apply(event, autocompleteInfo.methodAccessor.callSuspend(objects))
         val autoCompleteQuery = event.focusedOption
 
         //If something is typed but there are no choices, don't display user input

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/commands/application/slash/autocomplete/AutocompleteInfoImpl.kt

@@ -18,6 +18,7 @@ internal class AutocompleteInfoImpl internal constructor(
     override val name: String? = builder.name
     internal val eventFunction = builder.function.toMemberParamFunction<CommandAutoCompleteInteractionEvent, _>(context)
     override val function get() = eventFunction.kFunction
+    internal val methodAccessor get() = eventFunction.methodAccessor
     override val mode: AutocompleteMode = builder.mode
     override val showUserInput: Boolean = builder.showUserInput
 

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/commands/text/TextCommandVariationImpl.kt

@@ -24,7 +24,6 @@ import io.github.freya022.botcommands.internal.parameters.CustomMethodOption
 import io.github.freya022.botcommands.internal.parameters.ServiceMethodOption
 import io.github.freya022.botcommands.internal.utils.*
 import net.dv8tion.jda.api.events.message.MessageReceivedEvent
-import kotlin.reflect.full.callSuspendBy
 import kotlin.reflect.jvm.jvmErasure
 
 internal class TextCommandVariationImpl internal constructor(
@@ -90,7 +89,7 @@ internal class TextCommandVariationImpl internal constructor(
     internal suspend fun execute(event: BaseCommandEvent, optionValues: Map<out OptionImpl, Any?>) {
         val finalParameters = parameters.mapFinalParameters(event, optionValues)
 
-        function.callSuspendBy(finalParameters)
+        methodAccessor.callSuspend(finalParameters)
     }
 
     /**

BotCommands-core/src/main/kotlin/io/github/freya022/botcommands/internal/components/handler/ComponentHandlerExecutor.kt

@@ -24,7 +24,6 @@ import net.dv8tion.jda.api.events.interaction.component.EntitySelectInteractionE
 import net.dv8tion.jda.api.events.interaction.component.GenericComponentInteractionCreateEvent
 import net.dv8tion.jda.api.events.interaction.component.StringSelectInteractionEvent
 import net.dv8tion.jda.api.interactions.components.selections.SelectMenuInteraction
-import kotlin.reflect.full.callSuspendBy
 import kotlin.reflect.jvm.jvmErasure
 
 private val logger = KotlinLogging.logger { }
@@ -88,7 +87,7 @@ internal class ComponentHandlerExecutor internal constructor(
                     return false
             }
 
-            function.callSuspendBy(parameters.mapFinalParameters(event, optionValues))
+            methodAccessor.callSuspend(parameters.mapFinalParameters(event, optionValues))
         }
         return true
     }
@@ -135,4 +134,4 @@ internal class ComponentHandlerExecutor internal constructor(
 
         return tryInsertNullableOption(value, option, optionMap)
     }
-}
+}