Don't mark node as available before we have a session id

Author: duncte123

settings.gradle.kts

@@ -39,7 +39,7 @@ fun VersionCatalogBuilder.common() {
 }
 
 fun VersionCatalogBuilder.discordLibs() {
-    library("jda", "net.dv8tion", "JDA").version("5.0.0-beta.11")
+    library("jda", "net.dv8tion", "JDA").version("5.1.0")
     library("d4j", "com.discord4j", "discord4j-core").version("3.2.3")
 }
 

src/main/kotlin/dev/arbjerg/lavalink/internal/LavalinkSocket.kt

@@ -38,7 +38,6 @@ class LavalinkSocket(private val node: LavalinkNode) : WebSocketListener(), Clos
 
     override fun onOpen(webSocket: WebSocket, response: Response) {
         logger.info("${node.name} has been connected!")
-        node.available = true
         open = true
         reconnectsAttempted = 0
     }
@@ -52,6 +51,7 @@ class LavalinkSocket(private val node: LavalinkNode) : WebSocketListener(), Clos
             Message.Op.Ready -> {
                 val sessionId = (event as Message.ReadyEvent).sessionId
                 node.sessionId = sessionId
+                node.available = true
                 logger.info("${node.name} is ready with session id $sessionId")
 
                 node.playerCache.values.forEach { player ->

testbot/src/main/java/me/duncte123/testbot/JDAListener.java

@@ -68,6 +68,13 @@ public void onReady(@NotNull ReadyEvent event) {
                         "The identifier of the song you want to play",
                         true
                     ),
+                Commands.slash("play-file", "Play a song from a file")
+                    .addOption(
+                        OptionType.ATTACHMENT,
+                        "file",
+                        "the file to play",
+                        true
+                    ),
                 Commands.slash("karaoke", "Turn karaoke on or off")
                     .addSubcommands(
                         new SubcommandData("on", "Turn karaoke on"),
@@ -214,6 +221,24 @@ public void onSlashCommandInteraction(@NotNull SlashCommandInteractionEvent even
 
                 break;
             }
+            case "play-file": {
+                // We are already connected, go ahead and play
+                if (guild.getSelfMember().getVoiceState().inAudioChannel()) {
+                    event.deferReply(false).queue();
+                } else {
+                    // Connect to VC first
+                    joinHelper(event);
+                }
+
+                final var file = event.getOption("file").getAsAttachment();
+                final long guildId = guild.getIdLong();
+                final Link link = this.client.getOrCreateLink(guildId);
+                final var mngr = this.getOrCreateMusicManager(guildId);
+
+                link.loadItem(file.getUrl()).subscribe(new AudioLoader(event, mngr));
+
+                break;
+            }
             // Required plugin: https://github.com/DuncteBot/java-timed-lyrics
             case "lyrics": {
                 final Link link = this.client.getOrCreateLink(guild.getIdLong());

testbot/src/main/java/me/duncte123/testbot/Main.java

@@ -65,14 +65,21 @@ public static void main(String[] args) throws InterruptedException {
 
     private static void registerLavalinkNodes(LavalinkClient client) {
         List.of(
-            client.addNode(
-                new NodeOptions.Builder()
-                    .setName("optiplex")
-                    .setServerUri("ws://optiplex.local.duncte123.lgbt")
-                    .setPassword("youshallnotpass")
-                    .build()
-            ),
-
+//            client.addNode(
+//                new NodeOptions.Builder()
+//                    .setName("localhost")
+//                    .setServerUri("ws://localhost")
+//                    .setPassword("youshallnotpass")
+//                    .build()
+//            )
+//            client.addNode(
+//                new NodeOptions.Builder()
+//                    .setName("optiplex")
+//                    .setServerUri("ws://optiplex.local.duncte123.lgbt")
+//                    .setPassword("youshallnotpass")
+//                    .build()
+//            ),
+//
             client.addNode(
                 new NodeOptions.Builder()
                     .setName("pi")