fix: make file parameter required in upload methods

Address code review feedback: change `file` from `Optional[str]` to
`str` in all upload method overrides to prevent runtime errors when
None is passed to the multipart upload helper.

Made-with: Cursor

Author: itsmeadi

getstream/chat/async_client.py

@@ -32,7 +32,7 @@ async def upload_channel_file(
         self,
         type: str,
         id: str,
-        file: Optional[str] = None,
+        file: str,
         user: Optional[OnlyUserID] = None,
     ) -> StreamResponse[UploadChannelFileResponse]:
         form_fields = []
@@ -51,7 +51,7 @@ async def upload_channel_image(
         self,
         type: str,
         id: str,
-        file: Optional[str] = None,
+        file: str,
         upload_sizes: Optional[List[ImageSize]] = None,
         user: Optional[OnlyUserID] = None,
     ) -> StreamResponse[UploadChannelResponse]:

getstream/chat/client.py

@@ -32,7 +32,7 @@ def upload_channel_file(
         self,
         type: str,
         id: str,
-        file: Optional[str] = None,
+        file: str,
         user: Optional[OnlyUserID] = None,
     ) -> StreamResponse[UploadChannelFileResponse]:
         form_fields = []
@@ -51,7 +51,7 @@ def upload_channel_image(
         self,
         type: str,
         id: str,
-        file: Optional[str] = None,
+        file: str,
         upload_sizes: Optional[List[ImageSize]] = None,
         user: Optional[OnlyUserID] = None,
     ) -> StreamResponse[UploadChannelResponse]:

getstream/common/async_client.py

@@ -24,7 +24,7 @@ def __init__(self, api_key: str, base_url, token, timeout, user_agent=None):
 
     @telemetry.operation_name("getstream.api.common.upload_file")
     async def upload_file(
-        self, file: Optional[str] = None, user: Optional[OnlyUserID] = None
+        self, file: str, user: Optional[OnlyUserID] = None
     ) -> StreamResponse[FileUploadResponse]:
         form_fields = []
         if user is not None:
@@ -39,7 +39,7 @@ async def upload_file(
     @telemetry.operation_name("getstream.api.common.upload_image")
     async def upload_image(
         self,
-        file: Optional[str] = None,
+        file: str,
         upload_sizes: Optional[List[ImageSize]] = None,
         user: Optional[OnlyUserID] = None,
     ) -> StreamResponse[ImageUploadResponse]:

getstream/common/client.py

@@ -24,7 +24,7 @@ def __init__(self, api_key: str, base_url, token, timeout, user_agent=None):
 
     @telemetry.operation_name("getstream.api.common.upload_file")
     def upload_file(
-        self, file: Optional[str] = None, user: Optional[OnlyUserID] = None
+        self, file: str, user: Optional[OnlyUserID] = None
     ) -> StreamResponse[FileUploadResponse]:
         form_fields = []
         if user is not None:
@@ -39,7 +39,7 @@ def upload_file(
     @telemetry.operation_name("getstream.api.common.upload_image")
     def upload_image(
         self,
-        file: Optional[str] = None,
+        file: str,
         upload_sizes: Optional[List[ImageSize]] = None,
         user: Optional[OnlyUserID] = None,
     ) -> StreamResponse[ImageUploadResponse]: