refactor!: change certain events args order (#238)

* Make them Optional

* Allow/Deny changed to Strings in v8

* Revert "Allow/Deny changed to Strings in v8"

This reverts commit 3ffb55705c.

* Revert "Revert "Allow/Deny changed to Strings in v8""

This reverts commit 4fb7ef271e.

* Revert "Revert "Revert "Allow/Deny changed to Strings in v8"""

This reverts commit d5a020e307.

docs/src/stepbystep/createevent.md

@@ -102,7 +102,7 @@ Now that we have fully covered events, it would be a good time to get some pract
     cachedEmojis: Emoji[],
   ) => unknown;
   guildMemberAdd?: (guild: Guild, member: Member) => unknown;
-  guildMemberRemove?: (guild: Guild, member: Member | UserPayload) => unknown;
+  guildMemberRemove?: (guild: Guild, user: UserPayload, member?: Member) => unknown;
   guildMemberUpdate?: (
     guild: Guild,
     member: Member,
@@ -110,7 +110,7 @@ Now that we have fully covered events, it would be a good time to get some pract
   ) => unknown;
   heartbeat?: () => unknown;
   messageCreate?: (message: Message) => unknown;
-  messageDelete?: (message: Message | PartialMessage) => unknown;
+  messageDelete?: (partial: PartialMessage, message?: Message) => unknown;
   messageUpdate?: (message: Message, cachedMessage: OldMessage) => unknown;
   nicknameUpdate?: (
     guild: Guild,
@@ -126,14 +126,16 @@ Now that we have fully covered events, it would be a good time to get some pract
   rawGateway?: (data: unknown) => unknown;
   ready?: () => unknown;
   reactionAdd?: (
-    message: Message | MessageReactionPayload,
+    payload: MessageReactionPayload
     emoji: ReactionPayload,
     userID: string,
+    message?: Message,
   ) => unknown;
   reactionRemove?: (
-    message: Message | MessageReactionPayload,
+    payload: MessageReactionPayload,
     emoji: ReactionPayload,
     userID: string,
+    message?: Message,
   ) => unknown;
   reactionRemoveAll?: (data: BaseMessageReactionPayload) => unknown;
   reactionRemoveEmoji?: (data: MessageReactionRemoveEmojiPayload) => unknown;

src/controllers/members.ts

@@ -35,10 +35,7 @@ export async function handleInternalGuildMemberRemove(data: DiscordPayload) {
 
   guild.memberCount--;
   const member = await cacheHandlers.get("members", payload.user.id);
-  eventHandlers.guildMemberRemove?.(
-    guild,
-    member || payload.user,
-  );
+  eventHandlers.guildMemberRemove?.(guild, payload.user, member);
 
   member?.guilds.delete(guild.id);
   if (member && !member.guilds.size) cacheHandlers.delete("members", member.id);

src/controllers/messages.ts

@@ -52,8 +52,8 @@ export async function handleInternalMessageDelete(data: DiscordPayload) {
   if (!channel) return;
 
   eventHandlers.messageDelete?.(
-    await cacheHandlers.get("messages", payload.id) ||
-      { id: payload.id, channel },
+    { id: payload.id, channel },
+    await cacheHandlers.get("messages", payload.id),
   );
 
   cacheHandlers.delete("messages", payload.id);
@@ -68,7 +68,8 @@ export async function handleInternalMessageDeleteBulk(data: DiscordPayload) {
 
   payload.ids.forEach(async (id) => {
     eventHandlers.messageDelete?.(
-      await cacheHandlers.get("messages", id) || { id, channel },
+      { id, channel },
+      await cacheHandlers.get("messages", id),
     );
     cacheHandlers.delete("messages", id);
   });

src/controllers/reactions.ts

@@ -52,9 +52,10 @@ export async function handleInternalMessageReactionAdd(data: DiscordPayload) {
   };
 
   eventHandlers.reactionAdd?.(
-    message || uncachedOptions,
+    uncachedOptions,
     payload.emoji,
     payload.user_id,
+    message,
   );
 }
 
@@ -107,9 +108,10 @@ export async function handleInternalMessageReactionRemove(
   };
 
   eventHandlers.reactionRemove?.(
-    message || uncachedOptions,
+    uncachedOptions,
     payload.emoji,
     payload.user_id,
+    message,
   );
 }
 

src/types/options.ts

@@ -78,7 +78,11 @@ export interface EventHandlers {
   debug?: (args: DebugArg) => unknown;
   dispatchRequirements?: (data: DiscordPayload, shardID: number) => unknown;
   guildBanAdd?: (guild: Guild, user: UserPayload, member?: Member) => unknown;
-  guildBanRemove?: (guild: Guild, user: UserPayload, member?: Member) => unknown;
+  guildBanRemove?: (
+    guild: Guild,
+    user: UserPayload,
+    member?: Member,
+  ) => unknown;
   guildCreate?: (guild: Guild) => unknown;
   guildLoaded?: (guild: Guild) => unknown;
   guildUpdate?: (guild: Guild, changes: GuildUpdateChange[]) => unknown;
@@ -89,7 +93,11 @@ export interface EventHandlers {
     cachedEmojis: Emoji[],
   ) => unknown;
   guildMemberAdd?: (guild: Guild, member: Member) => unknown;
-  guildMemberRemove?: (guild: Guild, member: Member | UserPayload) => unknown;
+  guildMemberRemove?: (
+    guild: Guild,
+    user: UserPayload,
+    member?: Member,
+  ) => unknown;
   guildMemberUpdate?: (
     guild: Guild,
     member: Member,
@@ -97,7 +105,7 @@ export interface EventHandlers {
   ) => unknown;
   heartbeat?: () => unknown;
   messageCreate?: (message: Message) => unknown;
-  messageDelete?: (message: Message | PartialMessage) => unknown;
+  messageDelete?: (partial: PartialMessage, message?: Message) => unknown;
   messageUpdate?: (message: Message, cachedMessage: OldMessage) => unknown;
   nicknameUpdate?: (
     guild: Guild,
@@ -113,14 +121,16 @@ export interface EventHandlers {
   rawGateway?: (data: unknown) => unknown;
   ready?: () => unknown;
   reactionAdd?: (
-    message: Message | MessageReactionUncachedPayload,
+    payload: MessageReactionUncachedPayload,
     emoji: ReactionPayload,
     userID: string,
+    message?: Message,
   ) => unknown;
   reactionRemove?: (
-    message: Message | MessageReactionUncachedPayload,
+    payload: MessageReactionUncachedPayload,
     emoji: ReactionPayload,
     userID: string,
+    message?: Message,
   ) => unknown;
   reactionRemoveAll?: (data: BaseMessageReactionPayload) => unknown;
   reactionRemoveEmoji?: (data: MessageReactionRemoveEmojiPayload) => unknown;