fix: throw on webhook member fetch (#236)

Co-authored-by: Nico <[email protected]>
zaida04 · Jul 4, 2023 · 4f283ec · 4f283ec
1 parent e9ae297
commit 4f283ec
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 0 deletions.
diff --git a/.changeset/early-hats-complain.md b/.changeset/early-hats-complain.md
@@ -0,0 +1,5 @@
+---
+"guilded.js": patch
+---
+
+Issue with member.fetch in which it fetches user even if it is a webhook
diff --git a/packages/guilded.js/lib/managers/global/MemberManager.ts b/packages/guilded.js/lib/managers/global/MemberManager.ts
@@ -27,6 +27,8 @@ export class GlobalMemberManager extends CacheableStructManager<string, Member>
      * @returns A Promise that resolves with the fetched member.
      */
     async fetch(serverId: string, userId: string, force?: boolean): Promise<Member> {
+        if (userId === "Ann6LewA") throw new Error("You cannot fetch a webhook as a member. The provided ID (Ann6LewA) is only given to webhooks.");
+
         const memberKey = buildMemberKey(serverId, userId);
         if (!force) {
             const existingMember = this.client.members.cache.get(memberKey);