feat!: improve repr for Member, User (#862)

* fix: always make repr return a str for Member

* feat!: reprs for User, Member > strs

This makes the repr now use the attrs-generated one.
I doubt people were relying on this feature besides for string conversion, so we should be fine.

* feat: add some basic variables to the repr

Author: AstreaTSS

interactions/api/models/member.py

@@ -35,8 +35,8 @@ class Member(ClientSerializerMixin):
     :ivar Optional[str] communication_disabled_until?: How long until they're unmuted, if any.
     """
 
-    user: Optional[User] = field(converter=User, default=None, add_client=True)
-    nick: Optional[str] = field(default=None)
+    user: Optional[User] = field(converter=User, default=None, add_client=True, repr=True)
+    nick: Optional[str] = field(default=None, repr=True)
     _avatar: Optional[str] = field(default=None, discord_name="avatar")
     roles: List[int] = field()
     joined_at: datetime = field(converter=datetime.fromisoformat)
@@ -54,8 +54,8 @@ class Member(ClientSerializerMixin):
     )  # TODO: Investigate what this is for when documented by Discord.
     flags: int = field()  # TODO: Investigate what this is for when documented by Discord.
 
-    def __repr__(self) -> str:
-        return self.name
+    def __str__(self) -> str:
+        return self.name or ""
 
     @property
     def avatar(self) -> Optional[str]:

interactions/api/models/user.py

@@ -30,11 +30,11 @@ class User(ClientSerializerMixin):
     :ivar Optional[UserFlags] public_flags?: The user's public flags
     """
 
-    id: Snowflake = field(converter=Snowflake)
-    username: str = field()
-    discriminator: str = field()
+    id: Snowflake = field(converter=Snowflake, repr=True)
+    username: str = field(repr=True)
+    discriminator: str = field(repr=True)
     avatar: Optional[str] = field(default=None)
-    bot: Optional[bool] = field(default=None)
+    bot: Optional[bool] = field(default=None, repr=True)
     system: Optional[bool] = field(default=None)
     mfa_enabled: Optional[bool] = field(default=None)
     banner: Optional[str] = field(default=None)
@@ -48,7 +48,7 @@ class User(ClientSerializerMixin):
     public_flags: Optional[UserFlags] = field(converter=UserFlags, default=None)
     bio: Optional[str] = field(default=None)
 
-    def __repr__(self) -> str:
+    def __str__(self) -> str:
         return self.username
 
     def has_public_flag(self, flag: Union[UserFlags, int]) -> bool:

interactions/api/models/user.pyi

@@ -24,7 +24,7 @@ class User(ClientSerializerMixin):
     premium_type: Optional[int]
     public_flags: Optional[UserFlags]
     bio: Optional[str]
-    def __repr__(self) -> str: ...
+    def __str__(self) -> str: ...
     def has_public_flag(self, flag: Union[UserFlags, int]) -> bool: ...
     @property
     def mention(self) -> str: ...