application command registry saying it takes 40 seconds to initalise

the title
No description
35 Replies
iaqi
iaqiOP5mo ago
and it keeps saying it finds differences between api command and my commands every time i start the bot but i never change anything like that
fenix
fenix5mo ago
Do you have the command IDs that are registered in the ‘idHints’?
iaqi
iaqiOP5mo ago
i tried with and without and the same thing happens sapphire has been very buggy for some reason
Favna
Favna5mo ago
This is partially because of networking and partially because of comparisons. Setting idHints definitely should speed it up, but if you're really concerned about it then use bulk overwrite method.
iaqi
iaqiOP5mo ago
well im concerned because its saying its finding differences and my network is good unless its on discords side same thing happened when i had idHints enabled too
Favna
Favna5mo ago
Please show the code of a command that has this issue It is entirely possible that @vladdy 's comparison logic needs fixing
iaqi
iaqiOP5mo ago
okay
vladdy
vladdy5mo ago
i wouldn't put it past me xD
iaqi
iaqiOP5mo ago
No description
vladdy
vladdy5mo ago
hmm interesting
iaqi
iaqiOP5mo ago
its saying its finding differences for every command i do
iaqi
iaqiOP5mo ago
GitHub
GitHub - nicklvh/joewy: Multi-purpose discord bot made in TypeScrip...
Multi-purpose discord bot made in TypeScript, Discord.js and Sapphire - nicklvh/joewy
Favna
Favna5mo ago
Oh hm but that's a very simple one. I had half expected something complex.
iaqi
iaqiOP5mo ago
but its for every command not juse this one
vladdy
vladdy5mo ago
must be some builders shenanigans can you set the default overwrite behavior to verbose overwrite and send the debug logs?
iaqi
iaqiOP5mo ago
okay
iaqi
iaqiOP5mo ago
No description
iaqi
iaqiOP5mo ago
this is what it says i have no idea what integration types are
vladdy
vladdy5mo ago
well that helps me enough eugh ok i'll look into it unless someones faster
Favna
Favna5mo ago
The API or builders implicitly sets integration types?
vladdy
vladdy5mo ago
i think builders might? i need to check
Favna
Favna5mo ago
For now the immediate fix is probably to use bulk overwrite mode @iaqi Unless that conflicts with your bot setup but there are few scenarios where it does
iaqi
iaqiOP5mo ago
okay
kyra
kyra4mo ago
cc: @vladdy
iaqi
iaqiOP4mo ago
im still getting this issue btw
EvolutionX
EvolutionX3mo ago
@Boomeravna @vladdy the issue didn't exist until i changed .setDMPermission to .setContext([InteractionContextType.Guild])
No description
EvolutionX
EvolutionX3mo ago
No description
EvolutionX
EvolutionX3mo ago
understood, this is becuz the comparison algorithm is not updated to check dmpermission replacement setContexts does that mean im stuck with dmPermission :wtfummm: @iaqi did you do something?
vladdy
vladdy3mo ago
Wtff Ok I'll check Do y'all have some command samples
EvolutionX
EvolutionX3mo ago
just make a simple ping command with .setContext, you should get the same result in logs if you are busy I could try checking this file https://github.com/sapphiredev/framework/blob/71fc3a427a29319b6f0b9e115fceed9124b1625b/src/lib/utils/application-commands/ApplicationCommandRegistry.ts#L26 but if you can fix it, you'll obviously do it faster than me :pepeCry:
iaqi
iaqiOP3mo ago
wdym
EvolutionX
EvolutionX3mo ago
for the slow time of registries
iaqi
iaqiOP3mo ago
i just started using bulk overwrite instead
EvolutionX
EvolutionX3mo ago
💀 I'll get vladdy to fix it
Prosta4ok_ua
Prosta4ok_ua3mo ago
oh, i maybe have the same problem 👀

Did you find this page helpful?