diff --git a/handlers.go b/handlers.go index 940172d..2ac4b2b 100644 --- a/handlers.go +++ b/handlers.go @@ -28,38 +28,27 @@ func ready(s *discordgo.Session, event *discordgo.Ready) { findAndDeleteUnusedMessages() removeOldCommandFromAllGuilds(s) var existingCommandNames []string - for _, guild := range event.Guilds { - existingCommands, err := s.ApplicationCommands(s.State.User.ID, guild.ID) + existingCommands, err := s.ApplicationCommands(s.State.User.ID, "") + if err != nil { + logrus.Errorf("error fetching existing global commands: %v", err) + } else { for _, existingCommand := range existingCommands { existingCommandNames = append(existingCommandNames, existingCommand.Name) } - if err != nil { - logrus.Errorf("error fetching existing commands for guild %s: %v", guild.Name, err) - continue - } - for _, command := range commands { - if !slices.Contains(existingCommandNames, command.Definition.Name) || slices.Contains(os.Args, "--update="+command.Definition.Name) { - cmd, err := s.ApplicationCommandCreate(s.State.User.ID, guild.ID, &command.Definition) - logrus.Infof("Added command '%s'", cmd.Name) - if err != nil { - logrus.Error("error creating command,", err) - continue - } + } + for _, command := range commands { + if !slices.Contains(existingCommandNames, command.Definition.Name) || slices.Contains(os.Args, "--update="+command.Definition.Name) || slices.Contains(os.Args, "--update=all") { + cmd, err := s.ApplicationCommandCreate(s.State.User.ID, "", &command.Definition) + if err != nil { + logrus.Errorf("error creating global command '%s': %v", cmd.Name, err) + } else { + logrus.Infof("Added global command '%s'", cmd.Name) } } } logrus.Info("Successfully started the Bot!") } -func guildCreate(s *discordgo.Session, event *discordgo.GuildCreate) { - for _, command := range commands { - _, err := s.ApplicationCommandCreate(s.State.User.ID, event.Guild.ID, &command.Definition) - if err != nil { - logrus.Errorf("error creating command for guild %s: %v", event.Guild.Name, err) - } - } -} - func interactionCreate(s *discordgo.Session, i *discordgo.InteractionCreate) { for _, command := range commands { switch i.Type { @@ -103,12 +92,9 @@ func interactionCreate(s *discordgo.Session, i *discordgo.InteractionCreate) { } func removeOldCommandFromAllGuilds(s *discordgo.Session) { - for _, guild := range s.State.Guilds { - existingCommands, err := s.ApplicationCommands(s.State.User.ID, guild.ID) - if err != nil { - logrus.Errorf("error fetching existing commands for guild %s: %v\n", guild.Name, err) - continue - } + existingCommands, err := s.ApplicationCommands(s.State.User.ID, "") + if err != nil { + logrus.Errorf("error fetching existing commands: %v\n", err) var commandIDs []string for _, command := range commands { commandIDs = append(commandIDs, command.Definition.Name) @@ -116,9 +102,9 @@ func removeOldCommandFromAllGuilds(s *discordgo.Session) { for _, existingCommand := range existingCommands { if !slices.Contains(commandIDs, existingCommand.Name) { logrus.Infof("Deleting command '%s'", existingCommand.Name) - err := s.ApplicationCommandDelete(s.State.User.ID, guild.ID, existingCommand.ID) + err := s.ApplicationCommandDelete(s.State.User.ID, "", existingCommand.ID) if err != nil { - logrus.Errorf("error deleting command %s for guild %s: %v", existingCommand.Name, guild.Name, err) + logrus.Errorf("error deleting command %s: %v", existingCommand.Name, err) } } } diff --git a/main.go b/main.go index 869955d..46f3537 100644 --- a/main.go +++ b/main.go @@ -42,7 +42,6 @@ func main() { } bot.Identify.Intents = discordgo.IntentsGuildMessages | discordgo.IntentsGuilds | discordgo.IntentMessageContent | discordgo.IntentGuildMembers bot.AddHandler(ready) - bot.AddHandler(guildCreate) bot.AddHandler(interactionCreate) bot.AddHandler(messageCreate) bot.AddHandler(messageDelete)