tango-python 1.18.1__tar.gz → 1.18.2__tar.gz

{tango_python-1.18.1 → tango_python-1.18.2}/CHANGELOG.md

@@ -7,6 +7,12 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+## [1.18.2] - 2026-10-19
+
+### Fixed
+
+- **`TangoRateLimitError.limit_type` no longer reports a daily lockout as `burst`.** It matched "minute" anywhere in the API's detail, and a daily 429 reads `Rate limit exceeded for daily. Please try again in 16 hours, 58 minutes.`, so callers were told to retry a burst limit they could not clear until midnight UTC. It now reads the rate's name from `exceeded for <name>` and checks for daily first.
+
 ## [1.18.1] - 2026-10-19
 
 ### Fixed

{tango_python-1.18.1 → tango_python-1.18.2}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.5
 Name: tango-python
-Version: 1.18.1
+Version: 1.18.2
 Summary: Python SDK for the Tango API
 Project-URL: Homepage, https://github.com/makegov/tango-python
 Project-URL: Documentation, https://docs.makegov.com/tango-python

{tango_python-1.18.1 → tango_python-1.18.2}/pyproject.toml

@@ -4,7 +4,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "tango-python"
-version = "1.18.1"
+version = "1.18.2"
 description = "Python SDK for the Tango API"
 readme = "README.md"
 requires-python = ">=3.12"

{tango_python-1.18.1 → tango_python-1.18.2}/tango/__init__.py

@@ -62,7 +62,7 @@ from .webhooks import (
 )
 from .webhooks.receiver import Delivery, WebhookReceiver
 
-__version__ = "1.18.1"
+__version__ = "1.18.2"
 __all__ = [
     "TangoClient",
     "TangoAPIError",

{tango_python-1.18.1 → tango_python-1.18.2}/tango/exceptions.py

@@ -1,7 +1,11 @@
 """Tango API exceptions"""
 
+import re
 from typing import Any
 
+# The API names the tripped rate in its 429 detail: "Rate limit exceeded for daily. Please try again in 16 hours, 58 minutes."
+_RATE_NAME = re.compile(r"exceeded for ([\w-]+)", re.IGNORECASE)
+
 
 class TangoAPIError(Exception):
     """Base exception for Tango API errors"""
@@ -88,11 +92,13 @@ class TangoRateLimitError(TangoAPIError):
         d = self.detail
         if not d:
             return None
-        lower = d.lower()
-        if "burst" in lower or "minute" in lower:
-            return "burst"
-        if "daily" in lower or "day" in lower:
+        # Read the rate's name, not the whole sentence: a daily lockout's wait reads "16 hours, 58 minutes", which a match on "minute" took for burst.
+        match = _RATE_NAME.search(d)
+        text = match.group(1).lower() if match else d.lower()
+        if "daily" in text or re.search(r"\bday\b", text):
             return "daily"
+        if "burst" in text or "minute" in text:
+            return "burst"
         return None
 
 

{tango_python-1.18.1 → tango_python-1.18.2}/tests/test_client.py

@@ -1502,6 +1502,22 @@ class TestErrorHandling:
         assert exc_info.value.limit_type == "daily"
         assert exc_info.value.wait_in_seconds == 3600
 
+    @pytest.mark.parametrize(
+        ("detail", "expected"),
+        [
+            # The API's own wording, with its waits as convert_seconds_to_human_readable_wait_time renders them.
+            ("Rate limit exceeded for daily. Please try again in 16 hours, 58 minutes.", "daily"),
+            ("Rate limit exceeded for daily. Please try again in 1 minute.", "daily"),
+            ("Rate limit exceeded for burst. Please try again in 45 seconds.", "burst"),
+            ("Rate limit exceeded for burst. Please try again in 1 minute.", "burst"),
+            ("Too many requests.", None),
+        ],
+    )
+    def test_limit_type_reads_the_rate_name_not_the_wait(self, detail, expected):
+        error = TangoRateLimitError("Rate limit exceeded", 429, {"detail": detail})
+
+        assert error.limit_type == expected
+
     @patch("tango.client.httpx.Client.request")
     def test_429_empty_body(self, mock_request):
         """Test 429 with no content body still works"""