valentina-python-client 2.4.0__tar.gz → 2.4.2__tar.gz

{valentina_python_client-2.4.0 → valentina_python_client-2.4.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.3
 Name: valentina-python-client
-Version: 2.4.0
+Version: 2.4.2
 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.0 → valentina_python_client-2.4.2}/pyproject.toml

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

{valentina_python_client-2.4.0 → valentina_python_client-2.4.2}/src/vclient/__init__.py

@@ -28,6 +28,9 @@ class _PropagateHandler(_logging.Handler):
 _logger.add(
     _PropagateHandler(),
     format="{message}",
+    # Forward TRACE too (e.g. the "Send request" line) and let the consumer's
+    # stdlib logging level decide what to show; the sink must not pre-filter it.
+    level="TRACE",
     filter=lambda record: record["name"].startswith("vclient"),
 )
 
@@ -113,4 +116,4 @@ __all__ = (
     "users_service",
 )
 
-__version__ = "2.4.0"
+__version__ = "2.4.2"

{valentina_python_client-2.4.0 → valentina_python_client-2.4.2}/src/vclient/_sync/services/base.py

@@ -168,8 +168,8 @@ 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.debug("Send request")
+        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):
             try:
@@ -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:
@@ -243,7 +243,7 @@ class SyncBaseService:
         header_request_id = response.headers.get(REQUEST_ID_HEADER)
         if header_request_id:
             success_bind["request_id"] = header_request_id
-        request_logger.bind(**success_bind).debug("Receive response")
+        request_logger.bind(**success_bind).debug("Request complete")
 
     @staticmethod
     def _inject_request_id_fallback(
@@ -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)

{valentina_python_client-2.4.0 → valentina_python_client-2.4.2}/src/vclient/services/base.py

@@ -176,9 +176,13 @@ 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)
 
-        request_logger.debug("Send request")
+        # 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.
+        request_logger.trace("Send request")
 
         last_error: RateLimitError | ServerError | None = None
 
@@ -209,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:
@@ -268,7 +272,7 @@ class BaseService:
         header_request_id = response.headers.get(REQUEST_ID_HEADER)
         if header_request_id:
             success_bind["request_id"] = header_request_id
-        request_logger.bind(**success_bind).debug("Receive response")
+        request_logger.bind(**success_bind).debug("Request complete")
 
     @staticmethod
     def _inject_request_id_fallback(
@@ -285,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.
@@ -317,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