devrev-Python-SDK 3.0.0__tar.gz → 3.0.2__tar.gz

{devrev_python_sdk-3.0.0 → devrev_python_sdk-3.0.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: devrev-Python-SDK
-Version: 3.0.0
+Version: 3.0.2
 Summary: A modern, type-safe Python SDK for the DevRev API
 Project-URL: Homepage, https://github.com/mgmonteleone/py-dev-rev
 Project-URL: Documentation, https://github.com/mgmonteleone/py-dev-rev

{devrev_python_sdk-3.0.0 → devrev_python_sdk-3.0.2}/docs/changelog.md

@@ -13,6 +13,18 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+## [3.0.1] - 2026-04-22
+
+### Changed
+
+- **Shared pagination helper** (CUSS-457) — Extracted the per-request page-size
+  clamp (`_MAX_PAGE = 100`) and the `resolve_page_limit` helper from
+  `conversations.py` and `works.py` into a single shared module
+  `src/devrev/services/_pagination.py`. Non-breaking internal refactor; no
+  public SDK or MCP surface change. The shared helper adopts the tighter of
+  the two pre-existing semantics, eliminating a small over-fetch on the
+  final page of paginated `list_*_since` calls in `WorksService`.
+
 ## [3.0.0] - 2026-04-22
 
 ### Added

{devrev_python_sdk-3.0.0 → devrev_python_sdk-3.0.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "devrev-Python-SDK"
-version = "3.0.0"
+version = "3.0.2"
 description = "A modern, type-safe Python SDK for the DevRev API"
 readme = "README.md"
 requires-python = ">=3.11"

devrev_python_sdk-3.0.2/src/devrev/services/_pagination.py

@@ -0,0 +1,54 @@
+"""Shared pagination helpers for cursor-streaming list helpers.
+
+The DevRev ``*.list`` endpoints cap per-page responses at 100 items and the
+corresponding ``*ListRequest.limit`` fields are pydantic-constrained to
+``le=100``. Any service that drives cursor pagination from a user-supplied
+``overall_limit`` / ``page_size`` pair must therefore clamp the value it puts
+on the request body before construction, or the request will fail validation
+locally before it is ever sent.
+
+:func:`resolve_page_limit` centralises that clamping so every ``list_*_since``
+helper resolves the per-page ``limit`` identically.
+"""
+
+from __future__ import annotations
+
+from typing import Final
+
+_MAX_PAGE: Final[int] = 100
+
+
+def resolve_page_limit(
+    overall_limit: int | None,
+    collected: int,
+    page_size: int | None,
+) -> int | None:
+    """Compute the ``limit`` to send for the next page request.
+
+    The returned value is always clamped to :data:`_MAX_PAGE` so callers using
+    an ``overall_limit`` greater than the server maximum (e.g. ``limit=200,
+    page_size=None``) still paginate correctly. When both ``overall_limit`` and
+    ``page_size`` are supplied the tightest of ``{page_size, remaining,
+    _MAX_PAGE}`` wins.
+
+    Args:
+        overall_limit: Caller-supplied hard cap on total items to return across
+            all pages, or ``None`` to stream until the server runs out of
+            results.
+        collected: Number of items already accumulated across prior pages.
+        page_size: Caller-supplied per-page size, or ``None`` to defer to the
+            server default (still clamped to :data:`_MAX_PAGE`).
+
+    Returns:
+        The ``limit`` to put on the next ``*ListRequest``, or ``None`` to omit
+        it entirely (only when both ``overall_limit`` and ``page_size`` are
+        ``None``).
+    """
+    if overall_limit is None:
+        if page_size is None:
+            return None
+        return min(page_size, _MAX_PAGE)
+    remaining = overall_limit - collected
+    if page_size is None:
+        return min(remaining, _MAX_PAGE)
+    return min(page_size, remaining, _MAX_PAGE)

{devrev_python_sdk-3.0.0 → devrev_python_sdk-3.0.2}/src/devrev/services/conversations.py

@@ -23,6 +23,7 @@ from devrev.models.conversations import (
     ConversationsUpdateRequest,
     ConversationsUpdateResponse,
 )
+from devrev.services._pagination import resolve_page_limit
 from devrev.services.base import AsyncBaseService, BaseService
 
 
@@ -47,33 +48,6 @@ def _normalize_sort_by(sort_by: Sequence[str] | None) -> list[str] | None:
     return normalized
 
 
-# ``ConversationsListRequest.limit`` is pydantic-constrained to ``le=100``;
-# clamp any computed per-page limit so pagination loops do not construct a
-# request body that fails validation before it is ever sent.
-_CONVERSATIONS_MAX_PAGE = 100
-
-
-def _resolve_page_limit(
-    overall_limit: int | None,
-    collected: int,
-    page_size: int | None,
-) -> int | None:
-    """Compute the ``limit`` to send for the next page request.
-
-    The returned value is always clamped to ``_CONVERSATIONS_MAX_PAGE`` so
-    callers using an ``overall_limit`` greater than the server maximum (e.g.
-    ``limit=200, page_size=None``) still paginate correctly.
-    """
-    if overall_limit is None:
-        if page_size is None:
-            return None
-        return min(page_size, _CONVERSATIONS_MAX_PAGE)
-    remaining = overall_limit - collected
-    if page_size is None:
-        return min(remaining, _CONVERSATIONS_MAX_PAGE)
-    return min(page_size, remaining, _CONVERSATIONS_MAX_PAGE)
-
-
 def _is_before_cutoff(modified_date: datetime | None, cutoff: datetime) -> bool:
     """Return True if ``modified_date`` is strictly older than ``cutoff``.
 
@@ -135,7 +109,7 @@ class ConversationsService(BaseService):
         while True:
             if limit is not None and len(results) >= limit:
                 break
-            request_limit = _resolve_page_limit(limit, len(results), page_size)
+            request_limit = resolve_page_limit(limit, len(results), page_size)
             request = ConversationsListRequest(
                 cursor=cursor,
                 limit=request_limit,
@@ -267,7 +241,7 @@ class AsyncConversationsService(AsyncBaseService):
         while True:
             if limit is not None and len(results) >= limit:
                 break
-            request_limit = _resolve_page_limit(limit, len(results), page_size)
+            request_limit = resolve_page_limit(limit, len(results), page_size)
             request = ConversationsListRequest(
                 cursor=cursor,
                 limit=request_limit,

{devrev_python_sdk-3.0.0 → devrev_python_sdk-3.0.2}/src/devrev/services/works.py

@@ -31,6 +31,7 @@ from devrev.models.works import (
     WorksUpdateResponse,
     WorkType,
 )
+from devrev.services._pagination import resolve_page_limit
 from devrev.services.base import AsyncBaseService, BaseService
 
 if TYPE_CHECKING:
@@ -58,31 +59,6 @@ def _is_before_cutoff(timestamp: datetime | None, cutoff: datetime) -> bool:
         return False
 
 
-# ``WorksListRequest.limit`` is pydantic-constrained to ``le=100``; clamp any
-# computed per-page limit so pagination loops do not construct a request body
-# that fails validation before it is ever sent.
-_WORKS_MAX_PAGE = 100
-
-
-def _resolve_page_limit(
-    overall_limit: int | None,
-    collected: int,
-    page_size: int | None,
-) -> int | None:
-    """Compute the ``limit`` to send for the next page request.
-
-    The returned value is always clamped to ``_WORKS_MAX_PAGE`` so callers using
-    an ``overall_limit`` greater than the server maximum (e.g. ``limit=200,
-    page_size=None``) still paginate correctly.
-    """
-    if page_size is not None:
-        return min(page_size, _WORKS_MAX_PAGE)
-    if overall_limit is None:
-        return None
-    remaining = overall_limit - collected
-    return min(remaining, _WORKS_MAX_PAGE)
-
-
 def _normalize_sort_by(sort_by: Sequence[str] | None) -> _StrList | None:
     """Normalize sort_by entries to the server-expected ``field:direction`` form.
 
@@ -351,7 +327,7 @@ class WorksService(BaseService):
                 owned_by=owned_by,
                 applies_to_part=applies_to_part,
                 cursor=cursor,
-                limit=_resolve_page_limit(limit, len(collected), page_size),
+                limit=resolve_page_limit(limit, len(collected), page_size),
                 sort_by=sort_by,
             )
             stop = False
@@ -577,7 +553,7 @@ class AsyncWorksService(AsyncBaseService):
                 owned_by=owned_by,
                 applies_to_part=applies_to_part,
                 cursor=cursor,
-                limit=_resolve_page_limit(limit, len(collected), page_size),
+                limit=resolve_page_limit(limit, len(collected), page_size),
                 sort_by=sort_by,
             )
             stop = False

devrev_python_sdk-3.0.2/tests/unit/services/test_pagination.py

@@ -0,0 +1,48 @@
+"""Unit tests for the shared pagination helper."""
+
+from __future__ import annotations
+
+import pytest
+
+from devrev.services._pagination import _MAX_PAGE, resolve_page_limit
+
+
+class TestResolvePageLimit:
+    """Tight-clamping semantics for ``resolve_page_limit``."""
+
+    def test_max_page_constant_is_one_hundred(self) -> None:
+        """Server-side cap is exactly 100 items per page."""
+        assert _MAX_PAGE == 100
+
+    @pytest.mark.parametrize(
+        ("overall_limit", "collected", "page_size", "expected"),
+        [
+            # No overall cap, no page size → defer entirely to server.
+            (None, 0, None, None),
+            # No overall cap, small page size → pass through.
+            (None, 0, 50, 50),
+            # No overall cap, page_size above MAX → clamp to MAX.
+            (None, 0, 200, 100),
+            # Overall cap below MAX, no page size → send remaining.
+            (50, 0, None, 50),
+            # Overall cap with some collected, no page size → send remaining.
+            (50, 45, None, 5),
+            # Overall cap above MAX, no page size → clamp to MAX.
+            (500, 0, None, 100),
+            # Tightest of {page_size=10, remaining=5, MAX=100} wins.
+            (50, 45, 10, 5),
+            # Tightest of {page_size=50, remaining=500, MAX=100} wins.
+            (500, 0, 50, 50),
+            # page_size > MAX with large remaining → MAX wins.
+            (500, 0, 200, 100),
+        ],
+    )
+    def test_resolve_page_limit_cases(
+        self,
+        overall_limit: int | None,
+        collected: int,
+        page_size: int | None,
+        expected: int | None,
+    ) -> None:
+        """Exhaustive coverage of the clamping matrix from the ticket."""
+        assert resolve_page_limit(overall_limit, collected, page_size) == expected

{devrev_python_sdk-3.0.0 → devrev_python_sdk-3.0.2}/uv.lock

@@ -35,14 +35,15 @@ wheels = [
 
 [[package]]
 name = "authlib"
-version = "1.6.9"
+version = "1.7.0"
 source = { registry = "https://pypi.org/simple" }
 dependencies = [
     { name = "cryptography" },
+    { name = "joserfc" },
 ]
-sdist = { url = "https://files.pythonhosted.org/packages/af/98/00d3dd826d46959ad8e32af2dbb2398868fd9fd0683c26e56d0789bd0e68/authlib-1.6.9.tar.gz", hash = "sha256:d8f2421e7e5980cc1ddb4e32d3f5fa659cfaf60d8eaf3281ebed192e4ab74f04", size = 165134, upload-time = "2026-03-02T07:44:01.998Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/d9/82/4d0603f30c1b4629b1f091bb266b0d7986434891d6940a8c87f8098db24e/authlib-1.7.0.tar.gz", hash = "sha256:b3e326c9aa9cc3ea95fe7d89fd880722d3608da4d00e8a27e061e64b48d801d5", size = 175890, upload-time = "2026-04-18T11:00:28.559Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/53/23/b65f568ed0c22f1efacb744d2db1a33c8068f384b8c9b482b52ebdbc3ef6/authlib-1.6.9-py2.py3-none-any.whl", hash = "sha256:f08b4c14e08f0861dc18a32357b33fbcfd2ea86cfe3fe149484b4d764c4a0ac3", size = 244197, upload-time = "2026-03-02T07:44:00.307Z" },
+    { url = "https://files.pythonhosted.org/packages/ca/48/c954218b2a250e23f178f10167c4173fecb5a75d2c206f0a67ba58006c26/authlib-1.7.0-py2.py3-none-any.whl", hash = "sha256:e36817afb02f6f0b6bf55f150782499ddd6ddf44b402bb055d3263cc65ac9ae0", size = 258779, upload-time = "2026-04-18T11:00:26.64Z" },
 ]
 
 [[package]]
@@ -483,7 +484,7 @@ wheels = [
 
 [[package]]
 name = "devrev-python-sdk"
-version = "2.14.1"
+version = "3.0.2"
 source = { editable = "." }
 dependencies = [
     { name = "beautifulsoup4" },
@@ -729,6 +730,18 @@ wheels = [
     { url = "https://files.pythonhosted.org/packages/7b/91/984aca2ec129e2757d1e4e3c81c3fcda9d0f85b74670a094cc443d9ee949/joblib-1.5.3-py3-none-any.whl", hash = "sha256:5fc3c5039fc5ca8c0276333a188bbd59d6b7ab37fe6632daa76bc7f9ec18e713", size = 309071, upload-time = "2025-12-15T08:41:44.973Z" },
 ]
 
+[[package]]
+name = "joserfc"
+version = "1.6.4"
+source = { registry = "https://pypi.org/simple" }
+dependencies = [
+    { name = "cryptography" },
+]
+sdist = { url = "https://files.pythonhosted.org/packages/de/c6/de8fdbdfa75c8ca04fead38a82d573df8a82906e984c349d58665f459558/joserfc-1.6.4.tar.gz", hash = "sha256:34ce5f499bfcc5e9ad4cc75077f9278ab3227b71da9aaf28f9ab705f8a560d3c", size = 231866, upload-time = "2026-04-13T13:15:40.632Z" }
+wheels = [
+    { url = "https://files.pythonhosted.org/packages/b6/f7/210b27752e972edb36d239315b08d3eb6b14824cc4a590da2337d195260b/joserfc-1.6.4-py3-none-any.whl", hash = "sha256:3e4a22b509b41908989237a045e25c8308d5fd47ab96bdae2dd8057c6451003a", size = 70464, upload-time = "2026-04-13T13:15:39.259Z" },
+]
+
 [[package]]
 name = "jsonschema"
 version = "4.26.0"
@@ -1505,20 +1518,20 @@ wheels = [
 
 [[package]]
 name = "pygments"
-version = "2.19.2"
+version = "2.20.0"
 source = { registry = "https://pypi.org/simple" }
-sdist = { url = "https://files.pythonhosted.org/packages/b0/77/a5b8c569bf593b0140bde72ea885a803b82086995367bf2037de0159d924/pygments-2.19.2.tar.gz", hash = "sha256:636cb2477cec7f8952536970bc533bc43743542f70392ae026374600add5b887", size = 4968631, upload-time = "2025-06-21T13:39:12.283Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/c3/b2/bc9c9196916376152d655522fdcebac55e66de6603a76a02bca1b6414f6c/pygments-2.20.0.tar.gz", hash = "sha256:6757cd03768053ff99f3039c1a36d6c0aa0b263438fcab17520b30a303a82b5f", size = 4955991, upload-time = "2026-03-29T13:29:33.898Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/c7/21/705964c7812476f378728bdf590ca4b771ec72385c533964653c68e86bdc/pygments-2.19.2-py3-none-any.whl", hash = "sha256:86540386c03d588bb81d44bc3928634ff26449851e99741617ecb9037ee5ec0b", size = 1225217, upload-time = "2025-06-21T13:39:07.939Z" },
+    { url = "https://files.pythonhosted.org/packages/f4/7e/a72dd26f3b0f4f2bf1dd8923c85f7ceb43172af56d63c7383eb62b332364/pygments-2.20.0-py3-none-any.whl", hash = "sha256:81a9e26dd42fd28a23a2d169d86d7ac03b46e2f8b59ed4698fb4785f946d0176", size = 1231151, upload-time = "2026-03-29T13:29:30.038Z" },
 ]
 
 [[package]]
 name = "pyjwt"
-version = "2.11.0"
+version = "2.12.1"
 source = { registry = "https://pypi.org/simple" }
-sdist = { url = "https://files.pythonhosted.org/packages/5c/5a/b46fa56bf322901eee5b0454a34343cdbdae202cd421775a8ee4e42fd519/pyjwt-2.11.0.tar.gz", hash = "sha256:35f95c1f0fbe5d5ba6e43f00271c275f7a1a4db1dab27bf708073b75318ea623", size = 98019, upload-time = "2026-01-30T19:59:55.694Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/c2/27/a3b6e5bf6ff856d2509292e95c8f57f0df7017cf5394921fc4e4ef40308a/pyjwt-2.12.1.tar.gz", hash = "sha256:c74a7a2adf861c04d002db713dd85f84beb242228e671280bf709d765b03672b", size = 102564, upload-time = "2026-03-13T19:27:37.25Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/6f/01/c26ce75ba460d5cd503da9e13b21a33804d38c2165dec7b716d06b13010c/pyjwt-2.11.0-py3-none-any.whl", hash = "sha256:94a6bde30eb5c8e04fee991062b534071fd1439ef58d2adc9ccb823e7bcd0469", size = 28224, upload-time = "2026-01-30T19:59:54.539Z" },
+    { url = "https://files.pythonhosted.org/packages/e5/7a/8dd906bd22e79e47397a61742927f6747fe93242ef86645ee9092e610244/pyjwt-2.12.1-py3-none-any.whl", hash = "sha256:28ca37c070cad8ba8cd9790cd940535d40274d22f80ab87f3ac6a713e6e8454c", size = 29726, upload-time = "2026-03-13T19:27:35.677Z" },
 ]
 
 [package.optional-dependencies]
@@ -1528,15 +1541,15 @@ crypto = [
 
 [[package]]
 name = "pymdown-extensions"
-version = "10.20"
+version = "10.21.2"
 source = { registry = "https://pypi.org/simple" }
 dependencies = [
     { name = "markdown" },
     { name = "pyyaml" },
 ]
-sdist = { url = "https://files.pythonhosted.org/packages/3e/35/e3814a5b7df295df69d035cfb8aab78b2967cdf11fcfae7faed726b66664/pymdown_extensions-10.20.tar.gz", hash = "sha256:5c73566ab0cf38c6ba084cb7c5ea64a119ae0500cce754ccb682761dfea13a52", size = 852774, upload-time = "2025-12-31T19:59:42.211Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/df/08/f1c908c581fd11913da4711ea7ba32c0eee40b0190000996bb863b0c9349/pymdown_extensions-10.21.2.tar.gz", hash = "sha256:c3f55a5b8a1d0edf6699e35dcbea71d978d34ff3fa79f3d807b8a5b3fa90fbdc", size = 853922, upload-time = "2026-03-29T15:01:55.233Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/ea/10/47caf89cbb52e5bb764696fd52a8c591a2f0e851a93270c05a17f36000b5/pymdown_extensions-10.20-py3-none-any.whl", hash = "sha256:ea9e62add865da80a271d00bfa1c0fa085b20d133fb3fc97afdc88e682f60b2f", size = 268733, upload-time = "2025-12-31T19:59:40.652Z" },
+    { url = "https://files.pythonhosted.org/packages/f7/27/a2fc51a4a122dfd1015e921ae9d22fee3d20b0b8080d9a704578bf9deece/pymdown_extensions-10.21.2-py3-none-any.whl", hash = "sha256:5c0fd2a2bea14eb39af8ff284f1066d898ab2187d81b889b75d46d4348c01638", size = 268901, upload-time = "2026-03-29T15:01:53.244Z" },
 ]
 
 [[package]]
@@ -1629,20 +1642,20 @@ wheels = [
 
 [[package]]
 name = "python-dotenv"
-version = "1.2.1"
+version = "1.2.2"
 source = { registry = "https://pypi.org/simple" }
-sdist = { url = "https://files.pythonhosted.org/packages/f0/26/19cadc79a718c5edbec86fd4919a6b6d3f681039a2f6d66d14be94e75fb9/python_dotenv-1.2.1.tar.gz", hash = "sha256:42667e897e16ab0d66954af0e60a9caa94f0fd4ecf3aaf6d2d260eec1aa36ad6", size = 44221, upload-time = "2025-10-26T15:12:10.434Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/82/ed/0301aeeac3e5353ef3d94b6ec08bbcabd04a72018415dcb29e588514bba8/python_dotenv-1.2.2.tar.gz", hash = "sha256:2c371a91fbd7ba082c2c1dc1f8bf89ca22564a087c2c287cd9b662adde799cf3", size = 50135, upload-time = "2026-03-01T16:00:26.196Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/14/1b/a298b06749107c305e1fe0f814c6c74aea7b2f1e10989cb30f544a1b3253/python_dotenv-1.2.1-py3-none-any.whl", hash = "sha256:b81ee9561e9ca4004139c6cbba3a238c32b03e4894671e181b671e8cb8425d61", size = 21230, upload-time = "2025-10-26T15:12:09.109Z" },
+    { url = "https://files.pythonhosted.org/packages/0b/d7/1959b9648791274998a9c3526f6d0ec8fd2233e4d4acce81bbae76b44b2a/python_dotenv-1.2.2-py3-none-any.whl", hash = "sha256:1d8214789a24de455a8b8bd8ae6fe3c6b69a5e3d64aa8a8e5d68e694bbcb285a", size = 22101, upload-time = "2026-03-01T16:00:25.09Z" },
 ]
 
 [[package]]
 name = "python-multipart"
-version = "0.0.22"
+version = "0.0.26"
 source = { registry = "https://pypi.org/simple" }
-sdist = { url = "https://files.pythonhosted.org/packages/94/01/979e98d542a70714b0cb2b6728ed0b7c46792b695e3eaec3e20711271ca3/python_multipart-0.0.22.tar.gz", hash = "sha256:7340bef99a7e0032613f56dc36027b959fd3b30a787ed62d310e951f7c3a3a58", size = 37612, upload-time = "2026-01-25T10:15:56.219Z" }
+sdist = { url = "https://files.pythonhosted.org/packages/88/71/b145a380824a960ebd60e1014256dbb7d2253f2316ff2d73dfd8928ec2c3/python_multipart-0.0.26.tar.gz", hash = "sha256:08fadc45918cd615e26846437f50c5d6d23304da32c341f289a617127b081f17", size = 43501, upload-time = "2026-04-10T14:09:59.473Z" }
 wheels = [
-    { url = "https://files.pythonhosted.org/packages/1b/d0/397f9626e711ff749a95d96b7af99b9c566a9bb5129b8e4c10fc4d100304/python_multipart-0.0.22-py3-none-any.whl", hash = "sha256:2b2cd894c83d21bf49d702499531c7bafd057d730c201782048f7945d82de155", size = 24579, upload-time = "2026-01-25T10:15:54.811Z" },
+    { url = "https://files.pythonhosted.org/packages/9a/22/f1925cdda983ab66fc8ec6ec8014b959262747e58bdca26a4e3d1da29d56/python_multipart-0.0.26-py3-none-any.whl", hash = "sha256:c0b169f8c4484c13b0dcf2ef0ec3a4adb255c4b7d18d8e420477d2b1dd03f185", size = 28847, upload-time = "2026-04-10T14:09:58.131Z" },
 ]
 
 [[package]]