Standardize code style across pagination implementation

- Make TYPE_CHECKING comments consistent across all files
- Remove redundant comments about circular imports
- Maintain consistent formatting and documentation style

Author: jpstroop

fitbit_client/resources/activity.py

@@ -23,13 +23,9 @@
 from fitbit_client.utils.types import JSONList
 from fitbit_client.utils.types import ParamDict
 
-# We use TYPE_CHECKING to avoid circular imports at runtime.
-# PaginatedIterator is only needed for type annotations, not for runtime code.
-# This pattern is recommended by the Python typing documentation:
-# https://docs.python.org/3/library/typing.html#typing.TYPE_CHECKING
+# Use TYPE_CHECKING to avoid circular imports
 if TYPE_CHECKING:
     # Local imports - only imported during type checking
-    # Local imports
     from fitbit_client.resources.pagination import PaginatedIterator
 
 
@@ -271,7 +267,6 @@ def get_activity_log_list(
             return cast(JSONDict, result)
 
         # Return as iterator if requested
-        # We use string literal type annotation 'PaginatedIterator' to avoid circular imports
         if as_iterator:
             return create_paginated_iterator(
                 response=cast(JSONDict, result),

fitbit_client/resources/electrocardiogram.py

@@ -17,8 +17,9 @@
 from fitbit_client.utils.types import JSONDict
 from fitbit_client.utils.types import ParamDict
 
+# Use TYPE_CHECKING to avoid circular imports
 if TYPE_CHECKING:
-    # Local imports
+    # Local imports - only imported during type checking
     from fitbit_client.resources.pagination import PaginatedIterator
 
 
@@ -128,7 +129,6 @@ def get_ecg_log_list(
             return cast(JSONDict, result)
 
         # Return as iterator if requested
-        # We use TYPE_CHECKING for PaginatedIterator type to avoid circular imports
         if as_iterator:
             return create_paginated_iterator(
                 response=cast(JSONDict, result),

fitbit_client/resources/irregular_rhythm_notifications.py

@@ -15,8 +15,9 @@
 from fitbit_client.utils.types import JSONDict
 from fitbit_client.utils.types import ParamDict
 
+# Use TYPE_CHECKING to avoid circular imports
 if TYPE_CHECKING:
-    # Local imports
+    # Local imports - only imported during type checking
     from fitbit_client.resources.pagination import PaginatedIterator
 
 
@@ -129,7 +130,6 @@ def get_irn_alerts_list(
             return cast(JSONDict, result)
 
         # Return as iterator if requested
-        # We use TYPE_CHECKING for PaginatedIterator type to avoid circular imports
         if as_iterator:
             return create_paginated_iterator(
                 response=cast(JSONDict, result),

fitbit_client/resources/pagination.py

@@ -17,9 +17,9 @@
 # Set up logging
 logger = logging.getLogger(__name__)
 
-# Avoid circular imports
+# Use TYPE_CHECKING to avoid circular imports
 if TYPE_CHECKING:
-    # Local imports
+    # Local imports - only imported during type checking
     from fitbit_client.resources.base import BaseResource
 
 

fitbit_client/resources/sleep.py

@@ -19,8 +19,9 @@
 from fitbit_client.utils.types import JSONDict
 from fitbit_client.utils.types import ParamDict
 
+# Use TYPE_CHECKING to avoid circular imports
 if TYPE_CHECKING:
-    # Local imports
+    # Local imports - only imported during type checking
     from fitbit_client.resources.pagination import PaginatedIterator
 
 
@@ -391,7 +392,6 @@ def get_sleep_log_list(
             return cast(JSONDict, result)
 
         # Return as iterator if requested
-        # We use TYPE_CHECKING for PaginatedIterator type to avoid circular imports
         if as_iterator:
             return create_paginated_iterator(
                 response=cast(JSONDict, result),