Extract function to generate default request IDs

Author: goncalossilva

todoist_api_python/_core/utils.py

@@ -2,6 +2,7 @@
 
 import asyncio
 import sys
+import uuid
 from datetime import date, datetime, timezone
 from typing import TYPE_CHECKING, TypeVar, cast
 
@@ -69,3 +70,8 @@ def parse_datetime(datetime_str: str) -> datetime:
         datetime_str = datetime_str[:-1] + "+00:00"
         return datetime.fromisoformat(datetime_str)
     return datetime.fromisoformat(datetime_str)
+
+
+def default_request_id_fn() -> str:
+    """Generate random UUIDv4s as the default request ID."""
+    return str(uuid.uuid4())

todoist_api_python/api.py

@@ -1,7 +1,6 @@
 from __future__ import annotations
 
 import sys
-import uuid
 from collections.abc import Callable, Iterator
 from typing import TYPE_CHECKING, Annotated, Any, Literal, TypeVar
 from weakref import finalize
@@ -28,7 +27,11 @@
     get_api_url,
 )
 from todoist_api_python._core.http_requests import delete, get, post
-from todoist_api_python._core.utils import format_date, format_datetime
+from todoist_api_python._core.utils import (
+    default_request_id_fn,
+    format_date,
+    format_datetime,
+)
 from todoist_api_python.models import (
     Attachment,
     Collaborator,
@@ -95,7 +98,7 @@ class TodoistAPI:
     def __init__(
         self,
         token: str,
-        request_id_fn: Callable[[], str] | None = lambda: str(uuid.uuid4()),
+        request_id_fn: Callable[[], str] | None = default_request_id_fn,
         session: requests.Session | None = None,
     ) -> None:
         """

todoist_api_python/api_async.py

@@ -1,12 +1,15 @@
 from __future__ import annotations
 
 import sys
-import uuid
 from typing import TYPE_CHECKING, Annotated, Callable, Literal, TypeVar
 
 from annotated_types import Ge, Le, MaxLen, MinLen
 
-from todoist_api_python._core.utils import generate_async, run_async
+from todoist_api_python._core.utils import (
+    default_request_id_fn,
+    generate_async,
+    run_async,
+)
 from todoist_api_python.api import TodoistAPI
 
 if TYPE_CHECKING:
@@ -52,7 +55,7 @@ class TodoistAPIAsync:
     def __init__(
         self,
         token: str,
-        request_id_fn: Callable[[], str] | None = lambda: str(uuid.uuid4()),
+        request_id_fn: Callable[[], str] | None = default_request_id_fn,
         session: requests.Session | None = None,
     ) -> None:
         """