[Bug]: Embedded icons display incorrect song data

[LanderVM]

Bug description

When skipping a song using the embedded icons, the Setup Channel displays the previous song’s data instead of the current one.

Note: This issue does NOT occur when using the /skip command. It was resolved in Issue #781.

Related reports from other users (now closed issues):
#781 (comment)

Steps to reproduce

1. Start playing two songs using /play songname.
2. Skip the current song using the ⏩ emoji reaction.
3. Notice that the correct song data appears briefly before being replaced with incorrect data.

Operating System

Windows 11

Node.js version

v22.14.0

java version

21

Application/Library version

v4.6.7 (commit: 3a7327a)

Error logs or screenshots

After skipping using the emoji reaction ⏩: The previous song’s data is displayed instead of the current song.

When using /play and interacting with the embed icons for a while: No song data is displayed.

Additional context

This issue is likely caused by the ⏩ embed icon and the /skip command using separate implementations. Since the /skip command was correctly fixed in a previous issue, it's possible that the embedded icon ⏩ still relies on outdated or different logic.

A solution would be to unify the implementation for both the /skip command and the embedded skip icon ⏩, so that updates apply to both.

[voidsyu]

I've tested it multiple times, and the correct data updates each time, regardless of whether you use the button or the interaction command

[LanderVM]

I've tested it multiple times, and the correct data updates each time, regardless of whether you use the button or the interaction command

How do you run the bot? I installed the latest main (3a7327a), added the .env file and ran the following commands:

npm i
npm run db:push
npm run build
npm start

Then, when playing a song and skipping to the next one, it updates to the correct song data but then reverts back to the previous data after a second:

GithubIssue.mp4

[voidsyu]

Did you rebuild? npm run build

[LanderVM]

Yes i did. Can you try the steps i did and see if you also have the problem? If not then its not the code thats the issue.

[Shirowwww]

I still got that issue on the latest build, why is it closed as completed ?

[voidsyu]

I still got that issue on the latest build, why is it closed as completed ?

I tested it many times and each time embed details is updating correctly, do you have any way to reproduce this problem? because I'm not encountering any such issue, no matter how many times I skip or go back to the previous song

[Shirowwww]

I tested it many times and each time embed details is updating correctly, do you have any way to reproduce this problem? because I'm not encountering any such issue, no matter how many times I skip or go back to the previous song

Exactly as @LanderVM described just above :

Then, when playing a song and skipping to the next one, it updates to the correct song data but then reverts back to the previous data after a second:

GithubIssue.mp4

It shows the new song for 2–3 seconds before showing the previous one again with the mention “Skipped by ...”. The only way to update it is to change the volume, pause, then play again, etc.

[infnibor]

I tested it many times and each time embed details is updating correctly, do you have any way to reproduce this problem? because I'm not encountering any such issue, no matter how many times I skip or go back to the previous song

Exactly as @LanderVM described just above :

Then, when playing a song and skipping to the next one, it updates to the correct song data but then reverts back to the previous data after a second:

GithubIssue.mp4

It shows the new song for 2–3 seconds before showing the previous one again with the mention “Skipped by ...”. The only way to update it is to change the volume, pause, then play again, etc.

Nonsense, it can be done differently, so that it works right away.
However, this requires changes to the file SetupButtons.ts

2025-06-24.13-44-40.mp4

[Shirowwww]

Nonsense, it can be done differently, so that it works right away. However, this requires changes to the file SetupButtons.ts

Well, did you actually change anything in the SetupButtons.ts file? Because even after deleting everything, reinstalling, and rebuilding on the latest commit, I’m still facing the exact same issue.
I'm not very experienced with code, so if you did make a change, would you mind showing how to do it?

[infnibor]

Nonsense, it can be done differently, so that it works right away. However, this requires changes to the file SetupButtons.ts

Well, did you actually change anything in the SetupButtons.ts file? Because even after deleting everything, reinstalling, and rebuilding on the latest commit, I’m still facing the exact same issue.
I'm not very experienced with code, so if you did make a change, would you mind showing how to do it?

No właśnie, czy faktycznie coś zmieniłeś w pliku SetupButtons.ts? Ponieważ nawet po usunięciu wszystkiego, ponownej instalacji i ponownym skompilowaniu na najnowszym zatwierdzeniu, nadal napotykam dokładnie ten sam problem. Nie mam dużego doświadczenia z kodem, więc gdybyś dokonał zmiany, czy mógłbyś pokazać, jak to zrobić?

I have it changed like that in my bot, but I don't use the latest code because it's a rebuilt old code, so the changes may differ in the new file.

[Shirowwww]

Ok, I applied this fix and it works for me now — yeah, it's manually updating the embed, but it works tho.

[voidsyu]

To fix this issue, we need to rewrite SetupSystem.ts there are a lot of issues

[Shirowwww]

@Shirowwww Try this version: git clone --branch test https://github.com/justeevil/lavamusic.git I've removed a few things "just for testing purposes" let me know if it works then I'll rewrite and fix it properly

Doesn't seem to be working — the first song doesn't display at all (but is playing), and when I skip, it only shows the skipped track (not the one currently playing).