fix(street): Use assert_never, cast firebase task id to int when provider is mapillary, correct type hint

Author: ofr1tz

apps/project/models.py

@@ -689,7 +689,7 @@ class ProjectTask(FirebasePushResource):
     project_type_specifics = models.JSONField()
 
     # Type hints
-    id: int | str
+    id: int
     task_group_id: int
 
     # FIXME: Quick fix involves removing uniqueness constraint

project_types/street/api_calls.py

@@ -2,7 +2,7 @@
 from collections.abc import Hashable
 from concurrent.futures import ProcessPoolExecutor
 from functools import partial
-from typing import Any
+from typing import Any, assert_never
 from warnings import deprecated
 
 import mercantile  # type: ignore[reportMissingTypeStubs]
@@ -194,7 +194,7 @@ def download_and_process_tile(
         base = (provider.url or Config.PANORAMAX_API_LINK).rstrip("/")
         url = f"{base}/api/map/{z}/{x}/{y}.mvt"
     else:
-        raise Exception(f"Unknown provider {getattr(provider.name, 'value', '')}")
+        assert_never(provider.name)
 
     for _ in range(attempt_limit):
         try:

project_types/street/project.py

@@ -247,8 +247,13 @@ def compress_tasks_on_firebase(self) -> bool:
     @typing.override
     def get_task_specifics_for_firebase(self, task: ProjectTask):
         assert task.geometry is not None, "Task geometry must not be None"
+        image_provider = self.project_type_specifics.image_provider
+        task_id: int | str = task.firebase_id
+        if image_provider and image_provider.name == StreetImageProviderNameEnum.MAPILLARY:
+            task_id = int(task.firebase_id)
+
         return firebase_models.FbMappingTaskStreetCreateOnlyInput(
-            taskId=task.firebase_id,
+            taskId=task_id,
             groupId=task.task_group.firebase_id,
         )