valentina-python-client 2.4.1__tar.gz → 2.5.0__tar.gz

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: valentina-python-client
-Version: 2.4.1
+Version: 2.5.0
 Summary: Async Python client library for the Valentina Noir API
 Author: Nate Landau
 Author-email: Nate Landau <github@natenate.org>

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/pyproject.toml

@@ -16,7 +16,7 @@
     name = "valentina-python-client"
     readme = "README.md"
     requires-python = ">=3.13"
-    version = "2.4.1"
+    version = "2.5.0"
 
     [project.optional-dependencies]
         testing = ["polyfactory>=3.3.0"]

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/__init__.py

@@ -116,4 +116,4 @@ __all__ = (
     "users_service",
 )
 
-__version__ = "2.4.1"
+__version__ = "2.5.0"

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/_sync/services/base.py

@@ -168,7 +168,7 @@ class SyncBaseService:
         config = self._client._config
         max_attempts = config.max_retries + 1 if config.auto_retry_rate_limit else 1
         retry_statuses = config.retry_statuses
-        request_logger = logger.bind(method=method, url=path)
+        request_logger = logger.bind(method=method, url=path, params=params)
         request_logger.trace("Send request")
         last_error: RateLimitError | ServerError | None = None
         for attempt in range(max_attempts):
@@ -196,7 +196,7 @@ class SyncBaseService:
                 time.sleep(delay)
                 continue
             try:
-                self._raise_for_status(response, method, path)
+                self._raise_for_status(response, method, path, params=params)
                 self._log_success_response(response, request_logger)
                 return response
             except RateLimitError as e:
@@ -260,13 +260,17 @@ class SyncBaseService:
             if header_id:
                 response_data["request_id"] = header_id
 
-    def _raise_for_status(self, response: httpx.Response, method: str, url: str) -> None:
+    def _raise_for_status(
+        self, response: httpx.Response, method: str, url: str, params: dict[str, Any] | None = None
+    ) -> None:
         """Raise appropriate exception for error responses.
 
         Args:
             response: The HTTP response to check.
             method: The HTTP method of the request.
             url: The URL path of the request.
+            params: The query parameters of the request, logged to disambiguate
+                paginated calls that share the same path.
 
         Raises:
             AuthenticationError: For 401 responses.
@@ -289,7 +293,11 @@ class SyncBaseService:
             message = response.text or f"HTTP {status_code}"
         self._inject_request_id_fallback(response_data, response)
         error_logger = logger.bind(
-            method=method, url=url, status=status_code, request_id=response_data.get("request_id")
+            method=method,
+            url=url,
+            params=params,
+            status=status_code,
+            request_id=response_data.get("request_id"),
         )
         if status_code == 429:
             retry_after = self._parse_retry_after(response)
@@ -584,20 +592,23 @@ class SyncBaseService:
         limit: int = DEFAULT_PAGE_LIMIT,
         offset: int = 0,
         params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> PaginatedResponse[dict[str, Any]]:
         """Make a paginated GET request.
 
         Args:
             path: API endpoint path.
-            limit: Maximum number of items to return (0-100, default 10).
+            limit: Maximum number of items to return (default 10).
             offset: Number of items to skip from the beginning (default 0).
             params: Additional query parameters.
+            max_limit: Upper bound the limit is clamped to (default 100). Reference/catalog
+                endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Returns:
             A PaginatedResponse containing the items and pagination metadata.
         """
         request_params = {
-            "limit": min(max(limit, 0), MAX_PAGE_LIMIT),
+            "limit": min(max(limit, 0), max_limit),
             "offset": max(offset, 0),
             **(params or {}),
         }
@@ -612,20 +623,25 @@ class SyncBaseService:
         limit: int = DEFAULT_PAGE_LIMIT,
         offset: int = 0,
         params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> PaginatedResponse[T]:
         """Make a paginated GET request and parse items into the given model class.
 
         Args:
             path: API endpoint path.
             model_class: Pydantic model class to validate each item into.
-            limit: Maximum number of items to return (0-100, default 10).
+            limit: Maximum number of items to return (default 10).
             offset: Number of items to skip from the beginning (default 0).
             params: Additional query parameters.
+            max_limit: Upper bound the limit is clamped to (default 100). Reference/catalog
+                endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Returns:
             A PaginatedResponse containing validated model instances.
         """
-        response = self._get_paginated(path, limit=limit, offset=offset, params=params)
+        response = self._get_paginated(
+            path, limit=limit, offset=offset, params=params, max_limit=max_limit
+        )
         return PaginatedResponse(
             items=[model_class.model_validate(item) for item in response.items],
             limit=response.limit,
@@ -634,7 +650,12 @@ class SyncBaseService:
         )
 
     def _iter_all_pages(
-        self, path: str, *, limit: int = MAX_PAGE_LIMIT, params: dict[str, Any] | None = None
+        self,
+        path: str,
+        *,
+        limit: int = MAX_PAGE_LIMIT,
+        params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> Iterator[dict[str, Any]]:
         """Iterate through all pages of a paginated endpoint.
 
@@ -645,6 +666,8 @@ class SyncBaseService:
             path: API endpoint path.
             limit: Items per page (default 100 for efficiency).
             params: Additional query parameters.
+            max_limit: Upper bound the per-page limit is clamped to (default 100).
+                Reference/catalog endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Yields:
             Individual items from the paginated response.
@@ -657,7 +680,9 @@ class SyncBaseService:
         """
         offset = 0
         while True:
-            page = self._get_paginated(path, limit=limit, offset=offset, params=params)
+            page = self._get_paginated(
+                path, limit=limit, offset=offset, params=params, max_limit=max_limit
+            )
             for item in page.items:
                 yield item
             if not page.has_more:
@@ -665,7 +690,12 @@ class SyncBaseService:
             offset = page.next_offset
 
     def _get_all(
-        self, path: str, *, limit: int = MAX_PAGE_LIMIT, params: dict[str, Any] | None = None
+        self,
+        path: str,
+        *,
+        limit: int = MAX_PAGE_LIMIT,
+        params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> list[dict[str, Any]]:
         """Fetch all items from a paginated endpoint.
 
@@ -677,8 +707,13 @@ class SyncBaseService:
             path: API endpoint path.
             limit: Items per page (default 100 for efficiency).
             params: Additional query parameters.
+            max_limit: Upper bound the per-page limit is clamped to (default 100).
+                Reference/catalog endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Returns:
             A list of all items from all pages.
         """
-        return [item for item in self._iter_all_pages(path, limit=limit, params=params)]
+        return [
+            item
+            for item in self._iter_all_pages(path, limit=limit, params=params, max_limit=max_limit)
+        ]

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/_sync/services/character_blueprint.py

@@ -5,7 +5,13 @@ from collections.abc import Iterator
 from typing import TYPE_CHECKING
 
 from vclient._sync.services.base import SyncBaseService
-from vclient.constants import DEFAULT_PAGE_LIMIT, BlueprintTraitOrderBy, CharacterClass, GameVersion
+from vclient.constants import (
+    DEFAULT_PAGE_LIMIT,
+    MAX_REFERENCE_PAGE_LIMIT,
+    BlueprintTraitOrderBy,
+    CharacterClass,
+    GameVersion,
+)
 from vclient.endpoints import Endpoints
 from vclient.models import (
     CharacterConcept,
@@ -58,6 +64,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             SheetSection,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version, character_class=character_class),
         )
 
@@ -84,6 +91,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all character blueprint sections."""
         for section in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_SECTIONS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version, character_class=character_class),
         ):
             yield SheetSection.model_validate(section)
@@ -110,6 +119,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             TraitCategory,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version, section_id=section_id, character_class=character_class
             ),
@@ -140,6 +150,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all character blueprint categories."""
         for category in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_CATEGORIES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version, section_id=section_id, character_class=character_class
             ),
@@ -168,6 +180,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             TraitSubcategory,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version, category_id=category_id, character_class=character_class
             ),
@@ -198,6 +211,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all character blueprint subcategories."""
         for subcategory in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_SUBCATEGORIES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version, category_id=category_id, character_class=character_class
             ),
@@ -232,6 +247,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             Trait,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 character_class=character_class,
                 category_id=category_id,
@@ -282,6 +298,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all character blueprint traits."""
         for trait in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_TRAITS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 character_class=character_class,
                 category_id=category_id,
@@ -306,7 +324,11 @@ class SyncCharacterBlueprintService(SyncBaseService):
     ) -> PaginatedResponse[CharacterConcept]:
         """Get a paginated page of character concepts."""
         return self._get_paginated_as(
-            self._format_endpoint(Endpoints.CONCEPTS), CharacterConcept, limit=limit, offset=offset
+            self._format_endpoint(Endpoints.CONCEPTS),
+            CharacterConcept,
+            limit=limit,
+            offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
         )
 
     def list_all_concepts(self) -> list[CharacterConcept]:
@@ -315,7 +337,11 @@ class SyncCharacterBlueprintService(SyncBaseService):
 
     def iter_all_concepts(self) -> Iterator[CharacterConcept]:
         """Iterate through all character concepts."""
-        for concept in self._iter_all_pages(self._format_endpoint(Endpoints.CONCEPTS)):
+        for concept in self._iter_all_pages(
+            self._format_endpoint(Endpoints.CONCEPTS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
+        ):
             yield CharacterConcept.model_validate(concept)
 
     def get_concept(self, *, concept_id: str) -> CharacterConcept:
@@ -336,6 +362,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             VampireClan,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         )
 
@@ -351,6 +378,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all vampire clans."""
         for clan in self._iter_all_pages(
             self._format_endpoint(Endpoints.VAMPIRE_CLANS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         ):
             yield VampireClan.model_validate(clan)
@@ -375,6 +404,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             WerewolfAuspice,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         )
 
@@ -390,6 +420,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all werewolf auspices."""
         for auspice in self._iter_all_pages(
             self._format_endpoint(Endpoints.WEREWOLF_AUSPICES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         ):
             yield WerewolfAuspice.model_validate(auspice)
@@ -416,6 +448,7 @@ class SyncCharacterBlueprintService(SyncBaseService):
             WerewolfTribe,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         )
 
@@ -431,6 +464,8 @@ class SyncCharacterBlueprintService(SyncBaseService):
         """Iterate through all werewolf tribes."""
         for tribe in self._iter_all_pages(
             self._format_endpoint(Endpoints.WEREWOLF_TRIBES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         ):
             yield WerewolfTribe.model_validate(tribe)

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/_sync/services/dictionary.py

@@ -5,7 +5,7 @@ from collections.abc import Iterator
 from typing import TYPE_CHECKING
 
 from vclient._sync.services.base import SyncBaseService
-from vclient.constants import DEFAULT_PAGE_LIMIT
+from vclient.constants import DEFAULT_PAGE_LIMIT, MAX_REFERENCE_PAGE_LIMIT
 from vclient.endpoints import Endpoints
 from vclient.models import (
     DictionaryTerm,
@@ -48,6 +48,7 @@ class SyncDictionaryService(SyncBaseService):
             DictionaryTerm,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(term=term),
         )
 
@@ -55,11 +56,14 @@ class SyncDictionaryService(SyncBaseService):
         """Retrieve all dictionary terms."""
         return [term for term in self.iter_all(term=term)]
 
-    def iter_all(self, *, term: str | None = None, limit: int = 100) -> Iterator[DictionaryTerm]:
+    def iter_all(
+        self, *, term: str | None = None, limit: int = MAX_REFERENCE_PAGE_LIMIT
+    ) -> Iterator[DictionaryTerm]:
         """Iterate through all dictionary terms."""
         for item in self._iter_all_pages(
             self._format_endpoint(Endpoints.DICTIONARY_TERMS),
             limit=limit,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(term=term),
         ):
             yield DictionaryTerm.model_validate(item)

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/constants.py

@@ -20,6 +20,9 @@ IDEMPOTENT_HTTP_METHODS: frozenset[str] = frozenset({"GET", "PUT", "DELETE"})
 # Pagination defaults
 DEFAULT_PAGE_LIMIT = 10
 MAX_PAGE_LIMIT = 100
+# Reference/catalog list endpoints serve bounded seed data and allow fetching a
+# full catalog in one request, so they accept a higher per-request limit.
+MAX_REFERENCE_PAGE_LIMIT = 1000
 
 # Server log tail defaults
 DEFAULT_LOG_TAIL_LIMIT = 100

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/services/base.py

@@ -176,7 +176,9 @@ class BaseService:
         config = self._client._config  # noqa: SLF001
         max_attempts = config.max_retries + 1 if config.auto_retry_rate_limit else 1
         retry_statuses = config.retry_statuses
-        request_logger = logger.bind(method=method, url=path)
+        # Bind params so paginated requests are distinguishable in logs; without
+        # limit/offset, identical paths look like duplicate (thundering-herd) calls.
+        request_logger = logger.bind(method=method, url=path, params=params)
 
         # TRACE so the in-flight request is visible only when diagnosing hangs; the
         # "Request complete" DEBUG line below is the superset record for normal use.
@@ -211,7 +213,7 @@ class BaseService:
                 continue
 
             try:
-                self._raise_for_status(response, method, path)
+                self._raise_for_status(response, method, path, params=params)
                 self._log_success_response(response, request_logger)
                 return response  # noqa: TRY300
             except RateLimitError as e:
@@ -287,13 +289,21 @@ class BaseService:
             if header_id:
                 response_data["request_id"] = header_id
 
-    def _raise_for_status(self, response: httpx.Response, method: str, url: str) -> None:
+    def _raise_for_status(
+        self,
+        response: httpx.Response,
+        method: str,
+        url: str,
+        params: dict[str, Any] | None = None,
+    ) -> None:
         """Raise appropriate exception for error responses.
 
         Args:
             response: The HTTP response to check.
             method: The HTTP method of the request.
             url: The URL path of the request.
+            params: The query parameters of the request, logged to disambiguate
+                paginated calls that share the same path.
 
         Raises:
             AuthenticationError: For 401 responses.
@@ -319,7 +329,11 @@ class BaseService:
         self._inject_request_id_fallback(response_data, response)
 
         error_logger = logger.bind(
-            method=method, url=url, status=status_code, request_id=response_data.get("request_id")
+            method=method,
+            url=url,
+            params=params,
+            status=status_code,
+            request_id=response_data.get("request_id"),
         )
 
         if status_code == 429:  # noqa: PLR2004
@@ -653,20 +667,23 @@ class BaseService:
         limit: int = DEFAULT_PAGE_LIMIT,
         offset: int = 0,
         params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> PaginatedResponse[dict[str, Any]]:
         """Make a paginated GET request.
 
         Args:
             path: API endpoint path.
-            limit: Maximum number of items to return (0-100, default 10).
+            limit: Maximum number of items to return (default 10).
             offset: Number of items to skip from the beginning (default 0).
             params: Additional query parameters.
+            max_limit: Upper bound the limit is clamped to (default 100). Reference/catalog
+                endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Returns:
             A PaginatedResponse containing the items and pagination metadata.
         """
         request_params = {
-            "limit": min(max(limit, 0), MAX_PAGE_LIMIT),  # Clamp to valid range
+            "limit": min(max(limit, 0), max_limit),  # Clamp to valid range
             "offset": max(offset, 0),
             **(params or {}),
         }
@@ -682,20 +699,25 @@ class BaseService:
         limit: int = DEFAULT_PAGE_LIMIT,
         offset: int = 0,
         params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> PaginatedResponse[T]:
         """Make a paginated GET request and parse items into the given model class.
 
         Args:
             path: API endpoint path.
             model_class: Pydantic model class to validate each item into.
-            limit: Maximum number of items to return (0-100, default 10).
+            limit: Maximum number of items to return (default 10).
             offset: Number of items to skip from the beginning (default 0).
             params: Additional query parameters.
+            max_limit: Upper bound the limit is clamped to (default 100). Reference/catalog
+                endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Returns:
             A PaginatedResponse containing validated model instances.
         """
-        response = await self._get_paginated(path, limit=limit, offset=offset, params=params)
+        response = await self._get_paginated(
+            path, limit=limit, offset=offset, params=params, max_limit=max_limit
+        )
         return PaginatedResponse(
             items=[model_class.model_validate(item) for item in response.items],
             limit=response.limit,
@@ -709,6 +731,7 @@ class BaseService:
         *,
         limit: int = MAX_PAGE_LIMIT,
         params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> AsyncIterator[dict[str, Any]]:
         """Iterate through all pages of a paginated endpoint.
 
@@ -719,6 +742,8 @@ class BaseService:
             path: API endpoint path.
             limit: Items per page (default 100 for efficiency).
             params: Additional query parameters.
+            max_limit: Upper bound the per-page limit is clamped to (default 100).
+                Reference/catalog endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Yields:
             Individual items from the paginated response.
@@ -737,6 +762,7 @@ class BaseService:
                 limit=limit,
                 offset=offset,
                 params=params,
+                max_limit=max_limit,
             )
 
             for item in page.items:
@@ -753,6 +779,7 @@ class BaseService:
         *,
         limit: int = MAX_PAGE_LIMIT,
         params: dict[str, Any] | None = None,
+        max_limit: int = MAX_PAGE_LIMIT,
     ) -> list[dict[str, Any]]:
         """Fetch all items from a paginated endpoint.
 
@@ -764,8 +791,15 @@ class BaseService:
             path: API endpoint path.
             limit: Items per page (default 100 for efficiency).
             params: Additional query parameters.
+            max_limit: Upper bound the per-page limit is clamped to (default 100).
+                Reference/catalog endpoints pass a higher bound (MAX_REFERENCE_PAGE_LIMIT).
 
         Returns:
             A list of all items from all pages.
         """
-        return [item async for item in self._iter_all_pages(path, limit=limit, params=params)]
+        return [
+            item
+            async for item in self._iter_all_pages(
+                path, limit=limit, params=params, max_limit=max_limit
+            )
+        ]

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/services/character_blueprint.py

@@ -5,6 +5,7 @@ from typing import TYPE_CHECKING
 
 from vclient.constants import (
     DEFAULT_PAGE_LIMIT,
+    MAX_REFERENCE_PAGE_LIMIT,
     BlueprintTraitOrderBy,
     CharacterClass,
     GameVersion,
@@ -65,6 +66,7 @@ class CharacterBlueprintService(BaseService):
             SheetSection,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version, character_class=character_class),
         )
 
@@ -91,6 +93,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all character blueprint sections."""
         async for section in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_SECTIONS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version, character_class=character_class),
         ):
             yield SheetSection.model_validate(section)
@@ -122,6 +126,7 @@ class CharacterBlueprintService(BaseService):
             TraitCategory,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version,
                 section_id=section_id,
@@ -156,6 +161,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all character blueprint categories."""
         async for category in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_CATEGORIES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version,
                 section_id=section_id,
@@ -191,6 +198,7 @@ class CharacterBlueprintService(BaseService):
             TraitSubcategory,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version,
                 category_id=category_id,
@@ -225,6 +233,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all character blueprint subcategories."""
         async for subcategory in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_SUBCATEGORIES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 game_version=game_version,
                 category_id=category_id,
@@ -264,6 +274,7 @@ class CharacterBlueprintService(BaseService):
             Trait,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 character_class=character_class,
                 category_id=category_id,
@@ -314,6 +325,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all character blueprint traits."""
         async for trait in self._iter_all_pages(
             self._format_endpoint(Endpoints.BLUEPRINT_TRAITS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(
                 character_class=character_class,
                 category_id=category_id,
@@ -347,6 +360,7 @@ class CharacterBlueprintService(BaseService):
             CharacterConcept,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
         )
 
     async def list_all_concepts(self) -> list[CharacterConcept]:
@@ -357,6 +371,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all character concepts."""
         async for concept in self._iter_all_pages(
             self._format_endpoint(Endpoints.CONCEPTS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
         ):
             yield CharacterConcept.model_validate(concept)
 
@@ -383,6 +399,7 @@ class CharacterBlueprintService(BaseService):
             VampireClan,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         )
 
@@ -398,6 +415,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all vampire clans."""
         async for clan in self._iter_all_pages(
             self._format_endpoint(Endpoints.VAMPIRE_CLANS),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         ):
             yield VampireClan.model_validate(clan)
@@ -425,6 +444,7 @@ class CharacterBlueprintService(BaseService):
             WerewolfAuspice,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         )
 
@@ -442,6 +462,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all werewolf auspices."""
         async for auspice in self._iter_all_pages(
             self._format_endpoint(Endpoints.WEREWOLF_AUSPICES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         ):
             yield WerewolfAuspice.model_validate(auspice)
@@ -471,6 +493,7 @@ class CharacterBlueprintService(BaseService):
             WerewolfTribe,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         )
 
@@ -486,6 +509,8 @@ class CharacterBlueprintService(BaseService):
         """Iterate through all werewolf tribes."""
         async for tribe in self._iter_all_pages(
             self._format_endpoint(Endpoints.WEREWOLF_TRIBES),
+            limit=MAX_REFERENCE_PAGE_LIMIT,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(game_version=game_version),
         ):
             yield WerewolfTribe.model_validate(tribe)

{valentina_python_client-2.4.1 → valentina_python_client-2.5.0}/src/vclient/services/dictionary.py

@@ -3,7 +3,7 @@
 from collections.abc import AsyncIterator
 from typing import TYPE_CHECKING
 
-from vclient.constants import DEFAULT_PAGE_LIMIT
+from vclient.constants import DEFAULT_PAGE_LIMIT, MAX_REFERENCE_PAGE_LIMIT
 from vclient.endpoints import Endpoints
 from vclient.models import (
     DictionaryTerm,
@@ -53,6 +53,7 @@ class DictionaryService(BaseService):
             DictionaryTerm,
             limit=limit,
             offset=offset,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(term=term),
         )
 
@@ -61,12 +62,13 @@ class DictionaryService(BaseService):
         return [term async for term in self.iter_all(term=term)]
 
     async def iter_all(
-        self, *, term: str | None = None, limit: int = 100
+        self, *, term: str | None = None, limit: int = MAX_REFERENCE_PAGE_LIMIT
     ) -> AsyncIterator[DictionaryTerm]:
         """Iterate through all dictionary terms."""
         async for item in self._iter_all_pages(
             self._format_endpoint(Endpoints.DICTIONARY_TERMS),
             limit=limit,
+            max_limit=MAX_REFERENCE_PAGE_LIMIT,
             params=self._build_params(term=term),
         ):
             yield DictionaryTerm.model_validate(item)