Revert compact_dict in transformed update_task payloads

Keep compact_dict for simple pass-through payloads, but restore explicit field-by-field construction in update_task (sync/async) where values require date and datetime transformations.

Author: goncalossilva

todoist_api_python/api.py

@@ -350,7 +350,7 @@ def add_task_quick(
         )
         return Task.from_dict(task_data)
 
-    def update_task(
+    def update_task(  # noqa: PLR0912
         self,
         task_id: str,
         *,
@@ -396,27 +396,37 @@ def update_task(
         """
         endpoint = get_api_url(f"{TASKS_PATH}/{task_id}")
 
-        data = compact_dict(
-            content=content,
-            description=description,
-            labels=labels,
-            priority=priority,
-            due_string=due_string,
-            due_lang=due_lang,
-            due_date=format_date(due_date) if due_date is not None else None,
-            due_datetime=(
-                format_datetime(due_datetime) if due_datetime is not None else None
-            ),
-            assignee_id=assignee_id,
-            day_order=day_order,
-            collapsed=collapsed,
-            duration=duration,
-            duration_unit=duration_unit,
-            deadline_date=(
-                format_date(deadline_date) if deadline_date is not None else None
-            ),
-            deadline_lang=deadline_lang,
-        )
+        data: dict[str, Any] = {}
+        if content is not None:
+            data["content"] = content
+        if description is not None:
+            data["description"] = description
+        if labels is not None:
+            data["labels"] = labels
+        if priority is not None:
+            data["priority"] = priority
+        if due_string is not None:
+            data["due_string"] = due_string
+        if due_lang is not None:
+            data["due_lang"] = due_lang
+        if due_date is not None:
+            data["due_date"] = format_date(due_date)
+        if due_datetime is not None:
+            data["due_datetime"] = format_datetime(due_datetime)
+        if assignee_id is not None:
+            data["assignee_id"] = assignee_id
+        if day_order is not None:
+            data["day_order"] = day_order
+        if collapsed is not None:
+            data["collapsed"] = collapsed
+        if duration is not None:
+            data["duration"] = duration
+        if duration_unit is not None:
+            data["duration_unit"] = duration_unit
+        if deadline_date is not None:
+            data["deadline_date"] = format_date(deadline_date)
+        if deadline_lang is not None:
+            data["deadline_lang"] = deadline_lang
 
         task_data: dict[str, Any] = post(
             self._client,

todoist_api_python/api_async.py

@@ -374,7 +374,7 @@ async def add_task_quick(
         )
         return Task.from_dict(task_data)
 
-    async def update_task(
+    async def update_task(  # noqa: PLR0912
         self,
         task_id: str,
         *,
@@ -420,27 +420,37 @@ async def update_task(
         """
         endpoint = get_api_url(f"{TASKS_PATH}/{task_id}")
 
-        data = compact_dict(
-            content=content,
-            description=description,
-            labels=labels,
-            priority=priority,
-            due_string=due_string,
-            due_lang=due_lang,
-            due_date=format_date(due_date) if due_date is not None else None,
-            due_datetime=(
-                format_datetime(due_datetime) if due_datetime is not None else None
-            ),
-            assignee_id=assignee_id,
-            day_order=day_order,
-            collapsed=collapsed,
-            duration=duration,
-            duration_unit=duration_unit,
-            deadline_date=(
-                format_date(deadline_date) if deadline_date is not None else None
-            ),
-            deadline_lang=deadline_lang,
-        )
+        data: dict[str, Any] = {}
+        if content is not None:
+            data["content"] = content
+        if description is not None:
+            data["description"] = description
+        if labels is not None:
+            data["labels"] = labels
+        if priority is not None:
+            data["priority"] = priority
+        if due_string is not None:
+            data["due_string"] = due_string
+        if due_lang is not None:
+            data["due_lang"] = due_lang
+        if due_date is not None:
+            data["due_date"] = format_date(due_date)
+        if due_datetime is not None:
+            data["due_datetime"] = format_datetime(due_datetime)
+        if assignee_id is not None:
+            data["assignee_id"] = assignee_id
+        if day_order is not None:
+            data["day_order"] = day_order
+        if collapsed is not None:
+            data["collapsed"] = collapsed
+        if duration is not None:
+            data["duration"] = duration
+        if duration_unit is not None:
+            data["duration_unit"] = duration_unit
+        if deadline_date is not None:
+            data["deadline_date"] = format_date(deadline_date)
+        if deadline_lang is not None:
+            data["deadline_lang"] = deadline_lang
 
         task_data: dict[str, Any] = await post_async(
             self._client,